Retry logic on failed pruning

Currently on failed pruning,
web3signer waits until next interval,
but on wait amounts to prune,
if fails on first attempt,
it might fail on subsequents retries
2022-09-09 15:09:22.528+00:00 | main | INFO | Runner | Web3Signer has started with TLS disabled, and ready to handle signing requests on <IP_ADDRESS>:9000
2022-09-09 15:09:22.541+00:00 | pool-4-thread-1 | INFO | DbSlashingProtection | Pruning slashing protection database for 10000 validators
2022-09-09 15:25:17.219+00:00 | pool-4-thread-1 | INFO | DbSlashingProtection | Pruning slashing protection database complete
2022-09-10 15:09:22.536+00:00 | pool-4-thread-1 | INFO | DbSlashingProtection | Pruning slashing protection database for 10000 validators
2022-09-10 15:29:04.920+00:00 | pool-4-thread-1 | INFO | DbPrunerRunner | Pruning slashing protection database failed with error
2022-09-11 15:09:22.536+00:00 | pool-4-thread-1 | INFO | DbSlashingProtection | Pruning slashing protection database for 10000 validators
2022-09-11 15:16:01.933+00:00 | pool-4-thread-1 | INFO | DbPrunerRunner | Pruning slashing protection database failed with error
2022-09-12 02:30:04.399+00:00 | main | INFO | Runner | Web3Signer has started with TLS disabled, and ready to handle signing requests on <IP_ADDRESS>:9000
2022-09-12 02:30:04.407+00:00 | pool-4-thread-1 | INFO | DbSlashingProtection | Pruning slashing protection database for 10000 validators
2022-09-12 02:45:53.091+00:00 | pool-4-thread-1 | INFO | DbSlashingProtection | Pruning slashing protection database complete

Can you add back-off retry with failed pruning?
That would attempt retry pruning without waiting to next interval,
Thanks!

@jmcruz1983 Are you still observing this issue? Did lowering the pruning interval fixed it?

@usmansaleem all good from our side,
it has been a while since we experienced pruning errors,
Thanks


pyTensorArgs doesn't have to be owning

https://github.com/zou3519/functorch/blob/68df44f7cdb72864d5e43674357bca7a57b81220/functorch/csrc/PythonKey.cpp#L80
pyArgs is guaranteed to keep it live, so this can be a vector of handles rather than objects.

Fixed in core


Issue on migrating from 0.4.6 to 0.5.x

Issue Summary:
I am encountering an error after migrating from contentlayer2 version 0.4.6 to 0.5.3.
Error Details:
TypeError: Cannot read properties of undefined (reading 'ReactCurrentDispatcher')
    at eval (webpack-internal:///(rsc)/./node_modules/.pnpm/react@18.3.1/node_modules/react/cjs/react-jsx-runtime.development.js:328:51)
    at eval (webpack-internal:///(rsc)/./node_modules/.pnpm/react@18.3.1/node_modules/react/cjs/react-jsx-runtime.development.js:1332:5)
    at<EMAIL_ADDRESS>([...x]/.next/server/vendor-chunks/react@18.3.1.js:20:1)
    at __webpack_require__ ([...x]/.next/server/webpack-runtime.js:33:43)
    at eval (webpack-internal:///(rsc)/./node_modules/.pnpm/react@18.3.1/node_modules/react/jsx-runtime.js:4:20)
    at<EMAIL_ADDRESS>([...x]/.next/server/vendor-chunks/react@18.3.1.js:50:1)
    at __webpack_require__ ([...x]/.next/server/webpack-runtime.js:33:43)
    at eval (webpack-internal:///(rsc)/./node_modules/.pnpm/mdx-bundler@10.0.3_acorn@8.14.0_esbuild@0.24.0/node_modules/mdx-bundler/dist/client.js:9:44)
    at (rsc)/./node_modules/.pnpm/mdx-bundler@10.0.3_acorn<EMAIL_ADDRESS>([...x]/.next/server/vendor-chunks/mdx-bundler@10.0.3_acorn@8.14.0_esbuild@0.24.0.js:30:1)
    at __webpack_require__ ([...x]/.next/server/webpack-runtime.js:33:43) {
  type: 'TypeError',
  page: '/social/blog/how-to-book-train-tickets-for-your-pets'
}

Steps to Reproduce:

In Dev env

Migrate from contentlayer2 version 0.4.6 to 0.5.3.
Run the application.
Visit the page contentlayer implemented route.
Observe the error in the console/logs.

For Prod env

Build fails
Expected Behavior:
The application should run without any errors when visiting the specified page.
Actual Behavior:
Encountering a TypeError indicating an undefined property (ReactCurrentDispatcher).
Environment Details:
Node.js version: v20.18.0
Operating System: MacOS Sequoia 15.0.1
Package Manager: pnpm
Next.js version: 15.0.2
React version: 18.3.1
package.json:
{
  "name": "xxx",
  "version": "0.1.0",
  "private": true,
  "scripts": {
    "dev": "next dev -p 8000",
    "build": "next build",
    "start": "next start -p 8000",
    "lint": "next lint"
  },
  "dependencies": {
    "@react-google-maps/api": "^2.20.3",
    "@vercel/analytics": "^1.3.2",
    "agora-rtc-sdk-ng": "4.19.3",
    "contentlayer2": "^0.5.3",
    "cookies-next": "^4.3.0",
    "date-fns": "^4.1.0",
    "dayjs": "^1.11.13",
    "feed": "^4.2.2",
    "linkify-react": "^4.1.3",
    "linkifyjs": "^4.1.3",
    "mixpanel-browser": "2.55.1",
    "next": "15.0.2",
    "next-contentlayer2": "^0.5.3",
    "nprogress": "^0.2.0",
    "react": "^18.3.1",
    "react-dom": "^18.3.1",
    "react-hook-form": "^7.53.1",
    "react-icons": "^5.3.0",
    "react-lottie": "^1.2.4",
    "react-toastify": "^10.0.6",
    "reading-time": "^1.5.0",
    "rehype-slug": "^6.0.0",
    "remark-gfm": "^4.0.0",
    "sharp": "^0.33.5",
    "swiper": "^8.4.7"
  },
  "devDependencies": {
    "@tailwindcss/typography": "^0.5.15",
    "@types/cookie-parser": "^1.4.7",
    "@types/mixpanel-browser": "^2.50.2",
    "@types/node": "22.8.7",
    "@types/nprogress": "^0.2.3",
    "@types/react": "18.3.12",
    "@types/react-dom": "18.3.1",
    "@types/react-lottie": "^1.2.10",
    "@typescript-eslint/eslint-plugin": "8.12.2",
    "@typescript-eslint/parser": "8.12.2",
    "autoprefixer": "^10.4.20",
    "eslint": "9.14.0",
    "eslint-config-next": "15.0.2",
    "next-bundle-analyzer": "^0.6.8",
    "postcss": "^8.4.47",
    "readline": "^1.3.0",
    "tailwindcss": "^3.4.14",
    "typescript": "5.6.3"
  },
  "pnpm": {
    "overrides": {
      "@types/react": "18.3.12",
      "@types/react-dom": "18.3.1"
    }
  }
}

Additional Context:

I am not sure if this is a compatibility issue between the new contentlayer2 version and React 18.3.1 or Next.js 15.0.2.
The error does not appear with contentlayer2 version 0.4.6.

PS: Are there known compatibility issues between contentlayer2 version 0.5.3 and React 18?

If you are using Next 15, you should use React 19rc, I think it's required if you are using the app directory.
I tried using the latest contentlayer with React 18 and Next 14 - no issues with it.

Working once migrated to react-19


pull request for e2e test at 12/27/2016 06:26:07

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


Can't Read From appsettings.Development.json

[Enter feedback here]
Following these instructions doesn't seem to give you access to the configuration data from appsettings.Development.json. Do we need to set a particular environment variable first?

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

ID: b458a209-339d-e896-4e85-6d1c6a7aa1f9
Version Independent ID: 0d5de37c-8f89-b6a5-3132-2c5214196bfd
Content: Configuration - .NET
Content Source: docs/core/extensions/configuration.md
Product: dotnet-fundamentals
GitHub Login: @IEvangelist
Microsoft Alias: dapine

Just to check, I added this:
builder.UseEnvironment("Development");

And then the app read from appsettings.Development.json. But that's not a permanent solution.
I tried adding the environment variable ASPNETCORE_ENVIRONMENT set to "Development" in my project's profile, but that didn't seem help.

I figured it out from the XML comments of Host.CreateDefaultBuilder:

load host Microsoft.Extensions.Configuration.IConfiguration from "DOTNET_" prefixed environment variables

I changed my environment variable to DOTNET_ENVIRONMENT and now it works.
I think information should be included in the article.

Actually, the required information is on the webpage, but I didn't see it in the markdown file.
Sorry about that.


Fix for issues1 - added JUnit tests

Hi,
For the [Issues1 | https://github.com/SAP/cloud-jenkins/issues/1], added Junit Tests along with the fix.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Ignores --cacert curl option

Next option is ignored during code conversion:
--cacert ca-bundle.crt

You want output like this?
requests.get('https://github.com', verify='/path/to/ca-bundle.crt')

Closed by #341


Superficial spelling fix

I was hesitant to even mention it, but you have a minor spelling error in the smaller server status screen on the right side of the client:
Total service-wide bakups
An easy fix to check into git now so it gets included when you have other updates to make in the future.

Wow! I gotta say I am usually pretty good about fixing these before too long.... this one totally got past me lmao.... Thanks for that man!

Closing, pushed in latest build!


[$150] [Wave 8] [Ideal Nav] Change the global create and DM create tooltips to read Create

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: v1.4.36-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @muttmuure
Slack conversation: https://expensify.slack.com/archives/C036QM0SLJK/p1706889045249449
Action Performed:

Open NewDot on web/desktop
Hover over the + button in the bottom tab bar
Observe the tooltip reads  New
Click into a chat
Hover over the + button to the left of the composer
Observe the tooltip reads Actions

Expected Result:
Standardise on displaying the tooltip in both places as: Create
Actual Result:
As described above, we have an inconsistency where we use New and Actions.
Workaround:
N/A
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0121d4f02863128339
Upwork Job ID:<PHONE_NUMBER>597399040
Last Price Increase: 2024-02-02
Automatic offers:

aimane-chnaif | Reviewer | 28149278
esh-g | Contributor | 28149279

Is this open for anyone?

I'm available for this

Yeah, open to anyone. I dropped the price because it's so simple. :)

I've asked internally about the Spanish translation here.

Spanish translation is confirmed to be: Crear

@youssef-lr can you give this proposal a secondary look so we can move it on? Thanks!

Done @trjExpensify

Can't we close this if merged? @trjExpensify

Or payouts still needed?

PR is only on staging, so next steps::

PR needs to go to production
7 day regression period will kick in
All being well, we'll pay out on tat day and close the issue.

Right, we've had a deploy freeze because of the earnings call.

PR is now on production as of yesterday. Adjusting the title to reflect the 7 day hold.

Payment summary as follows:

$150 to @esh-g for the fix
$150 to @aimane-chnaif for the C+ review

Paid out both, closing!


Contribution

Is there space to free contribution to this project?
At first glance, it seems to be targeted to only Chinese people. Would you like to make it more universal so more people could appreciate it?
Maybe I could help you to translate documentation to English, and help to maintain it.
I look forward to hearing from you soon. 😀

Thank you. I'll get back to you later

Congratulations, a big step towards for the international open soure project.

This is a small step for @2881099 , but it is a big step for FreeSql. 😁

@ProximaB
Welcome to join the project, and we are looking forward to your translation.
(translation from fanyi.baidu.com 😀)

This is a small step for @2881099 , not only a big step for FreeSql but also a huge step for .Net Core! 😀😀

Hello , proximaB, soy chino en Madrid 😀

For a simple start we can make google translate a little bit smarter by not translating code via the html tag translate="no" or class="notranslate"
see https://cloud.google.com/translate/faq#technical_questions
and add a button

@mdissel Chinese Simple


Issue with Inline Image Upload in TinyMCE: Residual Data Retention

There seems to be an issue with how images are being handled within TinyMCE when using the Image/Inline feature. Upon uploading an image, it appears that the image file gets written somewhere within the data.db/data.db-wal file. However, after deleting the corresponding record, it's observed that the file size of data.db/data.db-wal remains unchanged. Even delete the entire collection didn't yield any results, indicating that the data isn’t being removed as expected.

The "Inline" option embeds the image as base64 text, aka. it will be part of your record db data.
When deleting entries from the database SQLite by default doesn't free the claimed disk space for optimization purposes to minimize allocations and frequent changes in the internal tree structure (at the end it is expected that this space will be filled again with future entries).
If you want to force releasing the disk space you can run VACUUM eg. with the sqlite3 cli tool:
sqlite3 ./pb_data/data.db 'VACUUM'

Thank you for your explanation.


Updates : New issue templates using .yml instead of .md 🛠

Is your feature request related to a problem? Please describe.
Currently we are using .md files instead of the brand new issue forms with .yml files which makes it more easier to fill in the data and raise an issue. Using the .md files can result in folks skipping out important details while making an issue.
Describe the solution you'd like
The solution is to build a new .yml file which can be customized. Adding an issue Template allows you to organize the issues made by folks in a much better way.

We can know if someone wants to write a test or maybe fix something in the website

The issue template forms makes writing details more efficient and quick

Also since there are required options, you can never miss on important values !

Here's a general idea :

Additional context
If the feature is approved, would you be willing to submit a PR?
YES ! ,  I'd love to contribute to the project !

Hey @IAmTamal, Thanks for opening the issue. Go ahead with this 🚀

Made PR for the issue : #89
Do let me know if changes are needed !


Problem with certificate's CA root cert (Comodo) is not in the approved list of root certs

Hi there, I want to register your Hardware Program for our drivers.
However, when submitting our certificate on your system, I received the error : "Your certificate's CA root cert is not in the approved list of root certs
Your code signing certificate's CA is not in the approved list of root certs. Please get a certificate from one of the certificate authorities listed on the code signing registration page.""
We use the Comodo certificate (Sectigo at the moment)
In this link,  https://docs.microsoft.com/vi-vn/windows-hardware/drivers/dashboard/get-a-code-signing-certificate#extended-validation-code-signing-certificates , I see that your list contains Sectigo.
Please answer why my submission is unsuccessful.

Chi tiết tài liệu
⚠ Không chỉnh sửa phần này. Điều này là bắt buộc cho docs.microsoft.com ➟ Vấn đề khi liên kết GitHub.

ID: 93d3f097-d81b-66c2-4835-d63758687d34
Version Independent ID: fe9d1cdd-966e-89b5-c66b-60531b0ec655
Content: Get a code signing certificate - Windows drivers
Content Source: windows-driver-docs-pr/dashboard/get-a-code-signing-certificate.md
Product: windows-hardware
Technology: windows-devices
GitHub Login: @EliotSeattle
Microsoft Alias: eliotgra

Hi @haicon232.
Sorry that you're having trouble with your certificate. The Hardware Dev Center Support folks can help you investigate the cause.

First, sign into your hardware dev center account, and then
Open a Hardware Dev Center support case for investigation.
Go to
https://docs.microsoft.com/windows-hardware/drivers/dashboard/hardware-dashboard-faq
and look for:

How do I contact Partner Center Support?
If you are having problems accessing the Dashboard or need Dashboard Support please open a support ticket here: https://developer.microsoft.com/windows/support. Select Contact us, Dashboard issue, and then Hardware submissions & signing (all OS version) from the dropdown menu.
Best of luck,
Eliot


[Release Radar Request] Keuss 2.0 deployed!

Open Source Project name
Keuss
What is your project?
A serverless, persistent and high-available queue middleware, build on Node.js, supporting delays/schedule, using a preexisting database to provide most of the backend functionality; we currently support mongodb, redis and postgresql
Version
2.0
Date
June 23, 2024
Description of breaking changes

Added a new major backend: now Keuss supports queues over PostgreSQL databases!
-The initialization for the queue system factory.queue(), is now asynchronous, and requires a callback.

GitHub Repo
https://github.com/pepmartinez/keuss
Website
https://pepmartinez.github.io/keuss/
Link to changelog
https://pepmartinez.github.io/keuss/docs/changelog/
Social media
none
Anything else to add?
You rock!

Congrats on being featured in the June 2024 Release Radar 🥳

Read the blog here
Feel free to retweet Mish's tweet featuring you
See and share LinkedIn post

Please share via your networks too, and congrats again ✨


Sequence support for triggerCharacters

It appears to me from the property name and the VS Code implementation that triggerCharacters only supports single characters in the array. Support for whole sequences would be very helpful for languages like PHP and C++, that have multi-character operators:
TestClass::staticMethod();
$obj->property;

I would like to trigger completion on -> and ::, but of course not on a single > (greater-than operator) or : (no meaning).
Currently, sending ["::", "->"] to VS Code seems to result in VS Code only triggering on the first character, : or -. I therefor send : and > instead.
Please document that the array can not only contain characters, but string sequences.

The current situation means that completions are triggered inside embedded HTML on every single closing tag because of the > trigger character (instead of ->), see https://github.com/felixfbecker/php-language-server/issues/372

@jrieken any comments from you?

   * @param triggerCharacters Trigger completion when the user types one of the characters, like `.` or `:`.

Well, the docs talk about characters only, but it does not say strings of length 1 (which is roughly the definition of a character). We might be able to implement multi-character triggers but it is not as trivial as it sound. Until then, an extension should use the last character in a potential sequence of trigger characters.

@jrieken That would be great. Then I can change the trigger characters to ["::", "->"] and  clients that already support sequences (e.g. https://github.com/eclipse/che/pull/3230/commits/61b2a93c32c7073f05b9f2a0c0a5a73642f356bd) can use that while VS Code would use the current behaviour until fully implemented.
Regarding the name, triggerCharacters is plural, so it is not defined whether that means an array where each element is a single character or where each element can also be multiple characters :)

@jrieken what's the status of the intermediate solution you proposed? I am planning to switch the trigger characters to string sequences soon as this issue is now also hitting more users through Atom https://github.com/atom/ide-php/issues/47 and it's hard to justify holding on to that behaviour just for VS Code. Of course I don't want to break VS Code users though. Let me know if there is a way I can help.

This reminds me of Microsoft/vscode#33697.

@rcjsuen thanks for linking that, I've not seen that issue. The proposed solution doesn't work over LSP though.

Actually the solution implemented by @jrieken in VS Code makes sense to me since it allows for the most flexibility. I will mirror that into the LSP in a backward compatible way. However it will make the node client implementation in compatible due to the middleware. But extension require this explicitly anyways.

OMG. LOVE YOU!

Implemented in https://github.com/felixfbecker/php-language-server/pull/527


[$500] [P2P Activation] [HIGH] Pay button in the workspace / DM view is different from the report view

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Reproducible in staging?: Y
Reproducible in production?: Y
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1697491822501769
Action Performed:

User A: select Request money
User A: enter User B as the person you're requesting from
User B: login
User B: Navigate to the IOU report, verify you see a split button with a dropdown
User B: navigate to the IOU report within the DM

Expected Result:
The pay button should be the “split pay” button in all cases, like we have in the report view, which allows the payment option a user chooses (first screenshot). And additionally, the split button should work as follows in all cases:

First time user with USD workspace/iouReport: Pay with Expensify (clicking it should go to wallet page to enable it)
First time user with non-USD workspace/iouReport: Pay elsewhere (clicking it should manually reimburse the report)
Returning user: display the lastPaymentMethod used i.e Pay with Expensify or Pay elsewhere

Actual Result:
The button within the DM/workspace chat view is not split, and therefore does not show a default payment method or the option to change it.
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01a1dc28bc812f8183
Upwork Job ID:<PHONE_NUMBER>518675456
Last Price Increase: 2023-10-17
Automatic offers:

s77rt | Reviewer | 27265389
akinwale | Contributor | 27265391

Proposal
Please re-state the problem that we are trying to solve in this issue.
The pay button in the report view for a DM does not show a dropdown with payment options.
What is the root cause of that problem?
The SettlementButton used in the ReportPreview component does not have the shouldShowPaymentOptions prop enabled.
What changes do you think we should make in order to solve the problem?
Add shouldShowPaymentOptions prop to the SettlementButton in ReportPreview.
What alternative solutions did you explore? (Optional)
None.

Confirmed. This need to be fixed.

@akinwale Thanks for the proposal. Your RCA is correct and the solution looks good to me 👍
🎀 👀 🎀 C+ reviewed
Link to proposal

@akinwale Thanks for the proposal. Your RCA is correct and the solution looks good to me 👍
🎀 👀 🎀 C+ reviewed Link to proposal

I agree, go for it!

@s77rt My PR is ready for review. Thanks.

@kevinksullivan this one is merged, but awaiting the next staging deploy today. It won't make it to prod ahead of Money2020 on Sunday, but likely Monday with a prod deploy if it passes on staging later.

The PR that introduced the bug has been identified: https://github.com/Expensify/App/pull/18692
The offending PR has been commented on: Not applicable since this was done on purpose and not really a bug
A discussion in #expensify-bugs has been started: Not needed (not a bug but a design change)
Determine if we should create a regression test for this bug: No

sorry was OOO yesterday, processing today

Paid


readme: update demo URL

Sorry for the embarrassing state of this project.  I really ought to make it into a proper node module and provide the core layout algorithm separate from all the poorly-done HTML goop.  My only excuse is it was a quick hack.

All good.
To be honest I think this project will probably attract a pull request or two to move it in the right direction.
I'm using it via https://github.com/danvk/source-map-explorer which is just fantastic.


[Scenario] La Mansión de Gaizka

What kind of mission do you want to add (mark with an X, e.g. [X] Descent)?

[x ] Mansions of Madness
[ ] Descent

What expansions does the scenario require?

[ ] Conversion Kit (CK)
[ ] First Edition (MoM1E) - Also Recurring Nightmares
[ ] Fatal Alchemy (FA) - Also Suppressed Memories
[ ] Call of the Wild (CotW) - Also Suppressed Memories
[x ] Beyond the Threshold (BtT)
[ ] Streets of Arkham (SoA)
[x ] Sunctum of Twilight (SoT)
[ ] Horrific Journeys (HJ)
[x ] Path of the Serpent (PotS)

What expansions can the scenario optionally use?

[ ] Conversion Kit (CK)
[ ] First Edition (MoM1E) - Also Recurring Nightmares
[ ] Fatal Alchemy (FA) - Also Suppressed Memories
[ ] Call of the Wild (CotW) - Also Suppressed Memories
[x ] Beyond the Threshold (BtT)
[ ] Streets of Arkham (SoA)
[x ] Sunctum of Twilight (SoT)
[ ] Horrific Journeys (HJ)
[x ] Path of the Serpent (PotS)

[Please enter the GitHub Url to your scenario here]
(https://github.com/JavierIceman12/LaMansionDeGaizka)

Added to manifest.
@antontimmermans , would you mind to add it to the statistics?

Added to the statistics sheets


Implement more integration tests

Takes a more-or-less standardized approach to integration testing. Only covering most of the dcim and ipam apps with this PR but looking to expand that in the future. Also implements a way to use an existing instance of NetBox by way of the --url-override arg to hopefully speed up dev iterations that may use the integration tests. Since taking this approach I don't think we need the Faker or Device Template dependencies so I pulled them.
cc: @raddessi

Right on! This looks good

I'll have to play around with it but I think this is very extendable. Do you feel comfortable with people adding tests with this structure then now or are you still feeling it out?

I think I'm comfortable extending this approach. One thing I do want to try tackling at some point is a few tests on .all() and .filter() with threading enabled. I think there's at least a couple ways to approach that though.


Updated xcode_summary to current version

xcode_summary version bump

Current coverage is 51.63% (diff: 100%)

Merging #28 into master will not change coverage

@@             master        #28   diff @@
==========================================
  Files            13         13
  Lines           459        459
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            237        237
  Misses          222        222
  Partials          0          0

Powered by Codecov. Last update 3bef147...33eee0b


Unable to specify a decimal number for sending

Hello,
In version 0.5.0 for Windows, impossible to specify a decimal number in the «amount» field. Keypress «.» doesn't work...
To take over this issue I had to check «Max available» checkbox to get the «.»
However, if I press «,» that works, but it's a separator for thousands, not for decimals...

Closing this since I can't repro anywhere, and it might be fixed because of the new Qt version. Please reopen if you're still seeing this with the latest zec-qt-wallet.

I have this bug on v0.5.9 on Ubuntu 18.04.1 LTS. Please reopen.

Thanks for the update. Can you confirm your country/locale please

Duplicate of #103


Wake and Raleigh Logos to Website

I could actually use some help with improving the style of the sponsors page

I worked on this some tonight, adding some header tags and NC open pass details. We'll continue to improve as we get more sponsors


loader refactor, type-safe __nextra_resolvePageMap, avoid code interpolation in loader.ts

I didn't think about this refactoring before, but now it looks much much better + everything is typesafe, also @shuding this should help you finish versioning
I tested remote and hot refresh - both works 🎉

@shuding I applied your suggestions, I will merge this one because I wanna push another pr with remote [...catchAll] support


New package: FlipGraphs v0.1.1

Registering package: FlipGraphs
Repository: https://github.com/schto223/FlipGraphs.jl
Created by: @schto223
Version: v0.1.1
Commit: c6fb69d8820c15db3a092ec41196d03a4189d523
Reviewed by: @schto223
Reference: https://github.com/schto223/FlipGraphs.jl/issues/5#issuecomment-2125333133
Description: My Master Thesis on Flip Graphs

Hi and congrats on the registration! Is "flip graph" a commonly accepted term in the literature? It's the first I'm hearing about it

Hi and congrats on the registration! Is "flip graph" a commonly accepted term in the literature? It's the first I'm hearing about it

Yes, as far as I know, flip graph is the common term. I have not yet found another term for this mathematical object.
Here is a short wikipedia article on this topic: https://en.wikipedia.org/wiki/Flip_graph

Okay, thanks for the pointer! I was checking because sometimes people register packages that are not useful outside of their own research work. But if you think your package can be used by other people, and if you add some documentation (or at least a temporary explanation in the readme), I see no reason to object here :)

https://en.wikipedia.org/wiki/Flip_graph

Okay, thank you. :) I have finally gotten the documentation up and running. And I've added a link to the former mentioned Wikipedia article into the README file.

[noblock]
Awesome, thanks! I added [noblock] to all my comments in order for the registration to proceed, you should to the same :)


Using security.txt to indicate TEA service availability

As an alternative, we could register a security.txt field.
The list of security.txt fields is also a registry maintained by IANA.

Originally posted by @ppkarwasz in #30

Security.txt is a good way to be able to find the API without having to have a product ID. It's different than the "ordinary" TEA discovery based on the TEI.

From the RFC: "Designated experts should determine whether a proposed registration or update provides value to organizations and researchers using this format and makes sense in the context of industry-accepted vulnerability disclosure processes such as [ISO.29147.2018] and [CERT.CVD]."


Crop Tokens, Not Text

Custom tokenizers might not skip special tokens during detokenization, which may result in the wrong prompt cropping.
ChatGLM3 similarity score jumps from 0.89 to 1 after the fix.

Thanks, @apaniukov, for the investigation and fix.

@apaniukov, I think we should consider adding something like crop_prompt to GenAI .generate() API? Or at least showcase to users how to use the generation API properly so that results will match what one can see in the framework.


Remove "test" from cache name

feel free to merge/close without comment :)

Do we want to add the -test- version to the restore keys for a while (~1m?) to not invalidate all existing caches?


Expose cycle counter and use it in /channelz.

We are using clock real time (which is very expensive) on every
RPC to update channelz. We should simply use cycle clock.
This patch exposes cycle clock, enables RDTSC on intel,
and use it for channelz.
This is part of a larger performance series.
Sorry I have to bother many of you:
@nicolasnoble  for portability
@AspirinSJL for time
@yashykt  for /channelz

This change is

The bazel failure are because of the test doesn't support precise clock (assumes precise clock is always realtime clock /facepalm).
Fixing it ...

Thank you for the review @AspirinSJL
I'll address them as soon as I figure out how I can fix the broken test. The test is still broken, so please don't review until I handle that. I think there is an unrelated bug that resurfaces here.

The bug at src/core/lib/gpr/time_precise.cc, line 71 seems real. Maybe try fixing that first?

The bug at src/core/lib/gpr/time_precise.cc, line 71 seems real. Maybe try fixing that first?

Thank you for catching that, but no that's definitely not the issue. :-)
Note that this is a new API so existing and unmodified tests don't use it. On top of that, the test breaks when RDTSC is enabled, and that code path is eliminated by the macro, and passes when the macro is undefined. It's something else, but I don't still know what it is.

Figured it out. The reason is that it fails is that gpr_precise_clock_init() loops for 10s to find the frequency, and that creates a 10s gap from the time origin of the cycle clock which is used for the deadlines in the iomgr.  The test uses 5s timeouts, and that doesn't seem to play well with that 10s of gap.
Lowering the spin time of  gpr_precise_clock_init() down to 1s, or in fixes the issue, but that's a hack I don't like, because I think something else is broken and this is just a sympom. I'll see which one is going to turn out working best tomorrow when I have more caffeine in my blood.

Once I figure why the deadline fires due to the 10s loop, and fix the root cause, I will change this patch to avoid looping for 10s. Abseil uses 64ms and that should be enough:
https://github.com/abseil/abseil-cpp/blob/master/absl/base/internal/sysinfo.cc#L180-L212
We can really afford delaying every grpc server start time by 10s on Intel. :-)

gpr_precise_clock_init() loops for 10s

This is actually the root cause. It results in the test server taking 10s to bootstrap causing all client-side deadlines to expire. Fix is what I said last night, to replicate what abseil does a shorter/smarter loop and potentially reading the tsc_freq file when it's exported.

The patch is working as intended. Essentially all end2end benchmarks show 19% to 45% now call reduction.  :-)

Failures are #18967 and #18511

@markdroth  I'm going to send the inlinedvector changes as a separate PR. Many tests started failing as a result of that change, so it might be worth having that separately.

I think I fixed the issue. One more try ...

yay! All tests pass ... :-)

I'm deferring to Mark's review on channelz :)

@nicolasnoble friendly ping. This is pending for ~2 months now.
I disabled it by default for open source, in case that helps expediting the review.

@hcaseyal cc'ing myself on this

Thank you @hcaseyal for the follow up and thank you @nicolasnoble for the approval!
Just squashed and rebased the patch series.  Will wait for the tests to pass.

Known issue #19819


Do not shadow variables

Preparing Bitcoin Core to be compiled with -Wshadow...
This will prevent compiler warnings about variable shadowing. I used the style already used (ie. append underscore).

utACK fceb4f8

utACK fceb4f8e842da86643b65f8afcd5f5e5c0777bbd

@jgarzik ping

@jgarzik ping

@jgarzik ping

Thank you Jeff!


Update kotlinx.serialization to 1.0.0

https://github.com/Kotlin/kotlinx.serialization/releases/tag/v1.0.0

Nice catch! Thanks a lot!


Callback for on attach (feature request)

Could you add a callback for on attach
i'd like to have custom mappings for LSP applied but only if the LSP is loaded

Its already there, simly pass {lspconfig = {on_attach = ...}}


Hyperion Buster releases

Summary
This PR will cause future releases to be made with Debian Buster (to avoid QT complications).
A few more tests will be necessary to identify and fix teething problems with the dependencies on different systems.
What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[ ] Feature
[ ] Code style update
[ ] Refactor
[ ] Docs
[x] Build-related changes
[ ] Other, please describe:

If changing the UI of web configuration, please provide the before/after screenshot:
Does this PR introduce a breaking change? (check one)

[x] Yes
[ ] No

If yes, please describe the impact and migration path for existing setups:
Upgrade the system if possible.
The PR fulfills these requirements:

[ ] When resolving a specific issue, it's referenced in the PR's body (e.g. Fixes: #xxx[,#xxx], where "xxx" is the issue number)

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature
[ ] Related documents have been updated (docs/docs/en)
[ ] Related tests have been updated

PLEASE DON'T FORGET TO ADD YOUR CHANGES TO CHANGELOG.MD

[x] No, CHANGELOG.md is not yet updated

To avoid wasting your time, it's best to open a feature request issue first and wait for approval before working on it.
Other information:

Got this artifact working with a debian:buster-slim install in a Docker environment on a Raspberry Pi 4. Anything to test specifically for you?

Got this artifact working with a debian:buster-slim install in a Docker environment on a Raspberry Pi 4

I guess so. Hyperion brings its own dependencies (with the exception of libcec). So if libcec is present it shouldn't be a problem (not tested)
Of course, hyperion only works to a limited extent in the Docker container.

Could not get release a7 to work, but this one did.
	libpython3.5 \
	libjpeg62-turbo \
	libavahi-compat-libdnssd1 \
	libdbus-1-3 \
	libqt5sql5 \
	libqt5serialport5 \
	libqt5network5 \
	libqt5widgets5 \
	libqt5gui5 \
	libqt5core5a \
	zlib1g-dev \
	curl \
	libusb-1.0-0 \
	libcec4

Has to get these installed on top of buster-slim variant. Or does this mean I only needed to get libcec installed with this artifact and the rest are included by default?

Or does this mean I only needed to get libcec installed with this artifact and the rest are included by default?

Yes


config turn-server with ALTERNATE-SERVER Mechanism failed

Hi there,
When construct a video conference project, I get a problem about TURNSERVER. details as following:

config one turnserver, everything is as well.
when use  ALTERNATE-SERVER Mechanism in turnserver with two turnserver, libnice.a print these log:
STUN demux: OK!
STUN auth error: No message integrity attribute!
STUN transaction retransmitted (timeout 399ms).
STUN transaction retransmitted (timeout 399ms).
STUN demux: OK!
STUN auth error: No message integrity attribute!
STUN demux error: no cookie!
STUN unknown: 0 mandatory attribute(s)!
Received 92-bytes STUN message
No XOR-MAPPED-ADDRESS: 1
Mapped address found!
STUN demux error: no cookie!
STUN demux error: no cookie!
STUN transaction retransmitted (timeout 796ms).
STUN demux: OK!
STUN auth error: No message integrity attribute!
STUN transaction retransmitted (timeout 1592ms).
STUN demux: OK!
STUN auth error: No message integrity attribute!

No generate correct SDP info, no relay address info.
Sorry, my English is not very good.

can you try using Nicer by setting this attribute in your licode_config.js please?
config.erizo.useNicer = true;  // default value: false
Also, please try always to use the template because it will help us trying to reproduce issues.

Hi, the issue  is caused by libnice. Details as follow:
https://gitlab.freedesktop.org/libnice/libnice/issues/66


EDA

Updated eda for numerical and categorical features.
Added pandas profile report.

Creating a new pull request with formatted files.


[WIP] feat: Relay compiler exercise

Adds a relay compiler exercise.
Progress:

[x] Tutorial written
[x] Code samples
[ ] Fill in line numbers & links within README.md

Included in this PR is a bit of reorganization: I moved all exercises into the src/ folder so that the READMEs can be directly next to the code they're referencing.
I also noticed that I was exporting things as default instead of exporting as named, and I started converting to named exports but didn't change it everywhere. I'll followup with the completion of that task in a separate PR.

Heads up friends that I'd love some review of the content of this one if you have some time!

Heads up friends that I'd love some review of the content of this one if you have some time!


On OpenBSD the filters keep running after shutting down smtpd

Hi ,
I use your opensmtpd-filters dkimsign and dkimverify on my OpenBSD server , currently version 7.5. and noticed that when I do
"rcctl stop smtpd" both filters don't stop and start using all of the CPU , they go wild.
I tried your code also on my Linux Arch servers and there it works fine.
So it's somekind of OpenBSD issue
I tried to attache signal handlers in your code but nothing worked.
I finally resorted to changing the rc file for smtpd.
and adding a pkill for the user _smtpd. This works
maybe you can find a more gracefull way
Thanks for the great work !
Here is my smtpd rc file:
#!/bin/ksh
#
# $OpenBSD: smtpd,v 1.8 2022/10/14 11:02:43 kn Exp $

daemon="/usr/sbin/smtpd"

. /etc/rc.d/rc.subr

rc_configtest() {
	# use rc_exec here since daemon_flags may contain arguments with spaces
	rc_exec "${daemon} -n ${daemon_flags}"
}
rc_stop() {
pkill -u _smtpd
}

rc_reload=NO

rc_cmd $1

Actually, OpenSMTPD is supposed to manage its filters. It starts them and should shut them down as well. So this sounds like an OpenSMTPD issue to me.
It’s still strange that the filters use up CPU however. Presumably, there is no OpenSMTPD process around to provide them with input any more, reading from stdin should result in a “Broken pipe” error. But it sounds like the self._stdin.readline() call returns an empty string instead.

According to documentation, readline() will in fact return an empty string on a stream past EOF. Could you try changing function serve_forever() in opensmtpd.py? After line = self.recv() insert the following:
            if line == '':
                break

According to documentation, readline() will in fact return an empty string on a stream past EOF. Could you try changing function recv() in opensmtpd.py? Replace return self._stdin.readline().rstrip('\r\n') by the following:
        line = self._stdin.readline()
        if line == '':
            raise Exception('No more input')
        else:
            return line.rstrip('\r\n')

Hi ,
Thx for your quick reply.
I tried your solution but no luck it still does the same.
I think it is indeed an OpenSMTPD problem in OpenBSD because if I just run the dkimsign in the shell. It does receive the SIGTERM signal and gracefully stops.
Kind regards,
Wim

I can reproduce a problem on Linux by running:
echo test | dkimverify

I fixed it: https://github.com/palant/opensmtpd-filters/commit/eedfd53c0302a6cb29c06dbdb8785bc9e006482c. This isn’t necessarily the problem you are seeing on OpenBSD, I don’t see any other way this code could get into an endless loop however.


Parallelized get target

6de5e3b2ab70317a7da2e638e2a5be87e89ba965 has shown great speed differences when graphing just one series.
When we deployed this to prod, where one typically renders 20ish series, things were still surprisingly slow.
I added some more metrics and verified that the parallel approach is much faster when you have several series.
approach taken:

use fake_metrics_to_nsq to backfill 1 month of dummy data for 1 org with 20 keys perorg:

/fake_metrics_to_nsq -keys-per-org 20 -orgs 1 -statsd-addr statsdaemon:8125 -nsqd-tcp-address nsqd:4150 -offset 31536000 -steps 1000 -interval 1

(i actually wanted to do a year but MT/nsqd started blocking and things failed, and i ended up with a bit less than a month but that's fine)
2) create a grafana dashboard that plots some.id.of.a.metric.* for this 1 month period:

restart MT from scratch so that it's not affected by previous ingest load
do a bunch of requests
visualize the duration metrics
rinse and repeat, switching to different commit hash, building and using that MT
metric is run with default args as in raintank-docker except no aggregation

./metric_tank \
	--chunkspan 600 \
	--numchunks 3 \
	--statsd-addr statsdaemon:8125 \
	--nsqd-tcp-address nsqd:4150 \
	--listen :6063 \
	--primary-node \
	--elastic-addr elasticsearch:9200 \
	--cassandra-addrs cassandra \
	&>> /var/log/raintank/nsq_metrics_tank.log &

(i did do a separate run with debug logging enabled to make sure it was doing runtime consolidation)
on the image you can see a bunch of requests with new code, then old, then new again.
note that the individual steps themselves are quite a bit slower then when executing serially, but all combined the duration of the total operation (red line on top) shows the picture as the user experiences it.  the parallel code takes us from 9s to 2.5s.

I should also note that the cpu usage is now more concentrated: with the parallelized code, cpu usage is higher for shorter periods of time.

i think this is fine and will be smoothed out since requests will rarely arrive at the same time anyway.

Looks great Dieter.


Clarification on multiple scope request when using token authentication registry

File: registry/spec/auth/token.md, CC @joaofnfernandes
Hello, I am not sure to understand completely the request to ask when you have multiple scopes.
Docs says:
"This query parameter should be specified multiple times if there is more than one scope"
Then "The above example would be specified as: scope=repository:samalba/my-app:push"
Nevertheless the "above example" is

Www-Authenticate:` Bearer realm="https://auth.docker.io/token",service="registry.docker.io",scope="repository:samalba/my-app:pull,push"

Where there are 2 scopes: pull and push
Nevertheless in the Example session, the request made is:

https://auth.docker.io/token?service=registry.docker.io&scope=repository:samalba/my-app:pull,push

My question is should we use the Example request or the follwing one:
https://auth.docker.io/token?service=registry.docker.io&scope=repository:samalba/my-app:pull&scope=repository:samalba/my-app:push
If the answer is the second request, then the doc should be updated and say:

"The above example would be specified as: scope=repository:samalba/my-app:push and scope=repository:samalba/my-app:pull"

Thank you for your time

@stevvooe PTAL

push and pull are not separate scopes. It is one scope with two actions. However, it can be represented as two scopes, but usually push alone is not useful, as one cannot figure out the information required to perform the push properly.


High CPU when using --watch on 2000+ files (roughly 40megs)

Code
tsc -w

TypeScript version:
2.6.1 (reproduces in previous versions as well)
Expected behavior:
To not hog CPU when watching (not while compiling).
Actual behavior:
tsc node process occupies 30-40% CPU while idle in watch.

Do you mean even while tsc is just waiting for changes?

Can you share the project?  we would be happy to sign any required NDAs.

Also, can you give us more information about your machines? e.g. OS, architecture, CPU count, etc.?

I'm also experiencing this on macOS 10.13.1, with Node v8.9.0.

@ip can you please share a repro project?

Sorry for the late response guys. Unfortunately,  I can't share the repo yet. Will get back to you about the NDA this week. It's happening on Win10 1511 to 1709 with node 6 and 8. Yes, it's hogging the CPU while simply watching for changes (before I save anything). The machine is i7 with 16gig ram and an SSD. The project is an angular 1.5 typescript project using the old school name spaces and references. I'll try and get more info about it and maybe somehow share the project this coming week.

I'm also experiencing this issue.
Machine i7 3632qm 12GB ram, SSD.
Project has almost 2000 ts files, and compiling in watch mode uses about 6-7% CPU constantly when waiting on changes:
18:59:26 - Compilation complete. Watching for file changes.
Unfortunately I cannot share project source, because it's closed software of my company.
I've seen similar issue when using grunt-watch task, which uses pooling to detect file changes. It's good for small projects but it's starting to stress cpu just watching hundreds of files. We switched to our custom implementation of watch using node fs.watch, which uses "native" host os watching for changes feature which is much more efficient with cpu than pooling (and much faster).
Isn't that the case? I couldn't find info about what typescipt compiler uses internally for watching changes.

@OlekNg, what OS are you using and what is the version of node?

I found that you can set TSC_NONPOLLING_WATCHER env var to 1, and then tsc -w uses non pooling watcher.

So you do not see the issue if you set TSC_NONPOLLING_WATCHER to 1?

@mhegazy
I'm on Win 10 Pro (1709) at home (1709) and Win 10 Premium at my company (also 1709 update).
Both running latest 6.x node version.
Setting TSC_NONPOLLING_WATCHER to 1 solved high CPU usage problem (it went down to 0%).
I'm not sure if this is recommended solution because I found it in tsc source - I couldn't find it in any official documentation.
From my previous post I also found that I can set env var for whole VSCode using setting "terminal.integrated.env.windows" which allowed me to run tsc compiler from VSCode with TSC_NONPOLLING_WATCHER flag set to 1.

How many files do you have in your project (tsc --listfiles --p tsconfig.josn). also what version of TS are you using?

tsc --listfiles --p tsconfig.json  reports 1854 files.

TSC_NONPOLLING_WATCHER=1 solves all my problems!
@OlekNg thank you so much for this tip! CPU went from 15-40% to 0.5%. I can definitely live with that.

Same problem, --list-files reports 596 files.
After some time the process takes roughly ~10% of one core all the time.


PR comments support

I like the idea of AI Agent supported code review! Giving a try to code rabbit ai now.
Generally want to reduce time when code review is triggered and focus it only on times, when humans also asked for code review.
One of the use cases highlighted is sometimes users create a PR and iteratively keep adding to it until it is fully ready for code review and even after code review changing code as well.
In that use case, the preference is to have AI code review just once. No need for summaries, release notes, nothing except code review and non trivial suggestions.
Right now using PR level comment trigger the action is skipped with the message:
Warning: Skipped: this action only works on push events or pull_request

Would it be possible add the support for PR level comments that would simply scan all changes in PR?
Here is the full trigger of the workflow we have:
on:
  issue_comment:
    types: [created]
  pull_request_review_comment:
    types: [created]

CodeRabbit Pro supports on-demand reviews but this doesn’t exist in OSS yet.
In Pro, automatic reviews can be disabled and reviews can be triggered manually using review commands.

Thank you for the explanation @guritfaq 🙏🏻
The review commands in code place do work as expected. However, when I run a review via PR level comment the Github Action application prevents this from being triggered. Even though the Pro version might indeed be more comprehensive, do you think it is still possible to let the open source variant to be triggered when it is best for community users?

@Tradunsky We are a small team and have to prioritize requests for our Pro version at this stage. However, we will add this to our backlog and address it as soon as possible.


Added Travis-CI and Cirrus-CI builds

Since examples are dependent on version of bash, I installed bash 4.4.
It's pretty easy to configure Travis to run examples against a matrix of bash versions if necessary in future.

BTW: You can see build on my fork here:
https://travis-ci.com/mmcc007/bash-oo-framework/builds/105608466

I added builds on Cirrus-CI which is running Bash 4.4.12 for much better results:
https://cirrus-ci.com/task/5121293144817664
Can also add a matrix of bash versions for testing on Cirrus-CI.

Added matrix and official docker builds of bash for both Cirrus and Travis. To seem them running:
https://cirrus-ci.com/github/mmcc007/bash-oo-framework
https://travis-ci.com/mmcc007/bash-oo-framework
Thanks to @fkorotkov for assistance on Cirrus-CI

Thank you so much @mmcc007 and @fkorotkov!

Sure, no problem. Might be using your logging module in another project.


fix-image-tag-behavior

The image tags when clicked render the expected results.
Fixes #203

@VatsalSin It would be better if you could squash your commits into one, since the changes are minor. :)

@VatsalSin if you merge this with the changes in #234 that are now in master, I think this should be good to go too


Add current status to button label

Is your feature request related to a problem? Please describe.
The set status dropdown is confusing for users becuase they cannot see what the current status is without opening the dropdown. As such, when they set the status, it is unclear what has happend.
Describe the solution you'd like
Option 1 (preferred)
A label should be added to the left of the button that says 'Status:' and the label on the button should become the current status e.g. Draft

Figure: Mockup - Option 1
The dropdown menu would still function the same.
Describe alternatives you've considered
Option 2
The 'Set status' dropdown should be labeled with the current status instead e.g.
'Status: Draft'

Figure: Mockup - Option 2
'Current status: Draft'  - Feel free to suggest different wording.
Additional context
See our YouTube video that goes through this as well as some related changes - https://youtu.be/8vhHKf0nCS4

See my comment here https://github.com/netlify/netlify-cms/issues/5555#issuecomment-871259430

Hey! I'm a new developer looking for issues and I would like to work on this issue

Hi, I made a PR and I want to mention that there is a text in my code ("Current Status:") that is not using the locales variables, so I think it will necessary to add another variable in en.editor.editorToolbar that is in the file /packages/netlify-cms-locales/en/index.js and the translate for the other languages

Closes via https://github.com/netlify/netlify-cms/pull/5574 (and released). Thanks everyone!


Object detection workflow

Planning to split this into small pull requests. This is the complete object detection work flow.
This includes frameworks for data and view extensions.
The object detection data extension creates datasets from object detection datasets. On the main page, select New dataset > Images > Object Detection

The dataset is configured with a form:

To create the model: on the main page, select New Model > Images > Object Detection.
After the model is trained, on the model page, select the bounding box view extension:

Then proceed to inference as usual. If you do Test Many, you may see something like:

This includes as a bonus:
- an image gradient work flow (the data extension creates datasets as in the image regression example) and the view extension displays arrows that point to the "light" in the images,
- an image processing work flow (the data extension creates datasets with images as inputs/outputs - useful for FCNs)
Example of gradient visualization:

  File "/home/lyeager/digits/digits/extensions/data/kittiObject/data.py", line 139, in encode_entry
    datum_bbox = caffe.io.bbox_to_datum(label,
AttributeError: 'module' object has no attribute 'bbox_to_datum'

I removed bbox_to_datum from caffe.io because I figured we could just include it in our code rather than checking it into Caffe.

Error handling for creating KITTI datasets is not good. Exceptions raised while the task is running should be caught and logged as errors, like so:
https://github.com/NVIDIA/DIGITS/blob/v3.3.0/tools/create_db.py#L779-L781
That will allow DIGITS to display the proper error message.
Also, I can't see any results on the dataset#show page, including the task log file. This is super helpful for debugging other failed tasks.

Exceptions raised while the task is running should be caught and logged as errors, like so:
https://github.com/NVIDIA/DIGITS/blob/v3.3.0/tools/create_db.py#L779-L781

I actually had that but the problem was I was missing something like this:
https://github.com/NVIDIA/DIGITS/blob/v3.3.0/digits/dataset/tasks/create_db.py#L218-L221
I have updated the "data extension" framework with better error handling (including those that occur in reader threads). So you would now get something like:

Updated to remove dependency on OpenCV bindings for Python.

Woohoo I just successfully completed the whole object detection workflow!

Woohoo I just successfully completed the whole object detection workflow!

Excellent! Which SHA1 did you use (there's been quite a few revisions so I'd like to make sure you've been using one of the last ones).

The latest for this branch.
I think we need to figure out how to add DetectNet as a "default/standard/zoo" network before this makes sense to merge, maybe? Also, maybe we should enable the gradient thing? It could be a cool walkthrough demo for DIGITS.


fix garbled characters in result of map_agg

Proposed changes

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall


When a link is using onmousedown event to somehow prevent click

Hi,
I came across with one case that the page is like this:
`>
`
When I capture the link the atrributes
            console.log('href: '+text);
            done();
}),```

I am getting this values:
onmousedown: b_Pag(this,2);
label: >
href: #

I am trying to access the link using
$('li.next a').go('onmousedown'),
$('li.next a').go('click'),

But It did not work. I imagine the page is using some technique to prevent click.. Anyone has any idea?

Sorry, Solved this with this code:
async.series([
        function( done ){
            $('li.next a').trigger('onmousedown',done);
        },
        function( done ){
            $('li.next a').attr('href',function(href){
                $.visit(href, done);
            })
        }
    ], function(){

        console.log('finished next page');
        done();
    });```


Scaled-down Bodies Not Colliding/Rotating Properly

I am working on this "speech to 2D word objects" project, which is available here:
https://www.ophirgal.com/Word Rain/Word Rain.html
And as you can see - when the words pile up / fall on each other they don't do so realistically, does anyone know what the problem is? I am guessing it has to do with how I am generating the word objects... I convert big SVG paths to vertices, then create bodies, then scale them down before adding them to the world. Below is my addBodyFromString() function:
var addBodyFromString = function(word='HELLO', indent=0){
// remove some older bodies if reached max
let all_bodies = Composite.allBodies(world);
if (all_bodies.length > MAX_BODIES){
    for (let i=0; i<=10; i++){
        let some_idx = 0;
        if (all_bodies[0].id === FLOOR_ID){
            some_idx += 1;
        }
        World.remove(world, all_bodies[some_idx]);
    }
}

var vertexSets = [];           // initialize empty list of vertex sets

// --------- alternative approach to generating the word ----------
// -------------- get vertices directly from strings --------------
/**
 * For each letter in word:
 *      - Get its points from pre-made dictionary.
 *      - For each point:
 *          Copy it and indent its x value according to the index
 *                          (x += index * some_magic_number).
 *      - Append the points to list of vertex sets.
 */

for (let i=0; i<word.length; i++){
    let letter = word[i];
    let points = [];

    if (letter < 'A' || letter > 'Z'){
        letter = '*';
    }

    for (let point of letterToVertixSet[letter]){
        let indented_point = {x: point.x + i*LETTER_INDENT,
                              y: point.y};
        points.push(indented_point);
    }
    vertexSets.push(points);
}

// ------------------------------------------------------------
// ------------------------------------------------------------

let word_color = Common.choose(colors); // choose random color

// create body
let word_body = Bodies.fromVertices(50 + indent, 80, vertexSets, {
    render: {
        fillStyle: word_color,
        strokeStyle: word_color,
        lineWidth: 1
    },
    restitution: 0.95
}, true);

Body.scale(word_body, .01, .01); // shrink its size

// add it to the world
World.add(world, word_body);

}

I got it to work, I just needed to scale the Vertices before creating the body, as well as use a high Sample length when creating the SVGs using the SVG module.


optimize: add column action_name to the tcc_fence_log

optimize: add column action_name to the tcc_fence_log
优化：tcc_fence_log表中，添加action_name字段。
主要用来查看空回滚的分支。

Codecov Report

Merging #3820 (0126c5e) into develop (0672229) will decrease coverage by 0.11%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             develop    #3820      +/-   ##
=============================================
- Coverage      40.84%   40.73%   -0.12%
  Complexity      2955     2955
=============================================
  Files            662      662
  Lines          22311    22315       +4
  Branches        2777     2746      -31
=============================================
- Hits            9113     9089      -24
- Misses         12349    12354       +5
- Partials         849      872      +23

Impacted Files
Coverage Δ

...src/main/java/io/seata/rm/tcc/TCCFenceHandler.java
0.00% <0.00%> (ø)

.../main/java/io/seata/rm/tcc/TCCResourceManager.java
0.00% <0.00%> (ø)

...a/rm/tcc/interceptor/ActionInterceptorHandler.java
13.88% <0.00%> (ø)

...rc/main/java/io/seata/rm/tcc/store/TCCFenceDO.java
0.00% <0.00%> (ø)

...eata/rm/tcc/store/db/TCCFenceStoreDataBaseDAO.java
0.00% <0.00%> (ø)

...o/seata/rm/tcc/store/db/sql/TCCFenceStoreSqls.java
0.00% <ø> (ø)

...obuf/convertor/BranchRegisterRequestConvertor.java
90.47% <0.00%> (-9.53%)
:arrow_down:

...otobuf/convertor/GlobalBeginResponseConvertor.java
92.59% <0.00%> (-7.41%)
:arrow_down:

...otobuf/convertor/GlobalCommitRequestConvertor.java
93.75% <0.00%> (-6.25%)
:arrow_down:

...otobuf/convertor/GlobalStatusRequestConvertor.java
93.75% <0.00%> (-6.25%)
:arrow_down:

... and 18 more


When sorting test result list, put vulnerable lists at the top

Added in b1126869ec9aae690ed481 // 44d95f63d7f397f4479a73 // a1ab9cf0760fbaee93d5bd


Regression: False positive in leak detection when an old pointer passed to realloc is used

Symbiotic now detects a non-existent leak in the following piece of code:
#include <stdlib.h>

int main(void)
{
    char *ptr1 = malloc(sizeof(char));
    if (ptr1 == NULL)
        return EXIT_SUCCESS;

    void *ptr2 = realloc(ptr1, 2 * sizeof(char));
    if (ptr2 == NULL) {
        free(ptr1);
        return EXIT_SUCCESS;
    }

    char ch = *ptr1; /* error */
    (void) ch;

    free(ptr2);
}

$ symbiotic --prp=memsafety 0059-test.c                                                                                [6/1696]
8.0.0-pre-llvm-11.0.0-symbiotic:1e1fb34a-dg:c04e6dd4-sbt-slicer:319209dd-sbt-instrumentation:ad1d134f-klee:aea4021c
INFO: Looking for invalid dereferences, invalid free, memory leaks, etc.
INFO: Starting instrumentation
wrapper: `which slllvm` failed with error code 1

INFO: Instrumentation time: 0.04898643493652344
INFO: Optimizations time: 0.01791858673095703
INFO: Starting slicing
INFO: Total slicing time: 0.017029762268066406
INFO: Optimizations time: 0.017657041549682617
INFO: After-slicing optimizations and transformations time: 2.0503997802734375e-05
INFO: Starting verification
KLEE: WARNING: undefined reference to function: __symbiotic_keep_ptr
KLEE: WARNING: undefined reference to function: klee_make_nondet
KLEE: WARNING ONCE: Alignment of memory from call "malloc" is not modelled. Using alignment of 8.
KLEE: WARNING ONCE: Alignment of memory from call "realloc" is not modelled. Using alignment of 8.
KLEE: ERROR: /home/lukas/aufover-benchmark/tests/single-c/mem-basic-realloc/0059-realloc-use-old-ptr-1/0059-test.c:15: memory error: memory not cleaned up
KLEE: ERROR: /home/lukas/aufover-benchmark/tests/single-c/mem-basic-realloc/0059-realloc-use-old-ptr-1/0059-test.c:15: memory error: out of bound pointer
KLEE: NOTE: now ignoring this error at this location

 --- Error trace ---

Error: memory error: out of bound pointer
File: /home/lukas/aufover-benchmark/tests/single-c/mem-basic-realloc/0059-realloc-use-old-ptr-1/0059-test.c
Line: 15
assembly.ll line: 70
Stack:
        #000000070 in main () at /home/lukas/aufover-benchmark/tests/single-c/mem-basic-realloc/0059-realloc-use-old-ptr-1/0059-test.c:15
Info:
        address: 30:0
        pointing to: none

 --- Sequence of non-deterministic values [function:file:line:col] ---

 --- ----------- ---

 --- Error trace ---

Error: memory error: memory not cleaned up
File: /home/lukas/aufover-benchmark/tests/single-c/mem-basic-realloc/0059-realloc-use-old-ptr-1/0059-test.c
Line: 15
assembly.ll line: 70
Stack:
        #000000070 in main () at /home/lukas/aufover-benchmark/tests/single-c/mem-basic-realloc/0059-realloc-use-old-ptr-1/0059-test.c:15
Info:
        address: 31:0
        pointing to: object at mpty() of size 2
                MO20[2] allocated at main():  %25 = call i8* @realloc(i8* %24, i64 2) #3, !dbg !24

 --- Sequence of non-deterministic values [function:file:line:col] ---

 --- ----------- ---
Error found.
NOTE: In the future, the result is going to be reported in SV-COMP format only with --report=sv-comp switch
RESULT: false(valid-memcleanup)
INFO: Total time elapsed: 0.5083410739898682

Yes, I can confirm. Look like a regression in KLEE.


Samples do not run correctly on Universal Windows (Windows 10)

Building and running Urho.Samples.UWP results in a blank app screen and no errors.  Closing the app doesn't trigger detach from the debugger.

@thargy are you trying to run it in Windows 10 Mobile or as a desktop app? And also do you use the latest revision of samples?

Can you get this simple test UWP UrhoSharp to work?
https://github.com/JimSEOW/UrhoSharpTest

@JimSEOW answered in https://github.com/xamarin/urho/issues/90

Ensuring UWP was run as x86 with a valid pak worked.


python310Packages.pyhocon: 0.3.58 -> 0.3.59

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.pyhocon is: "HOCON parser for Python"
meta.homepage for python310Packages.pyhocon is: "https://github.com/chimpler/pyhocon/"
meta.changelog for python310Packages.pyhocon is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
/nix/store/vd2npxlfw6d92n23mb56qrahg49h0jvy-python3.10-pyhocon-0.3.59/bin/.pyhocon-wrapped passed the binary check.
/nix/store/vd2npxlfw6d92n23mb56qrahg49h0jvy-python3.10-pyhocon-0.3.59/bin/pyhocon passed the binary check.
2 of 2 passed binary check by having a zero exit code.
0 of 2 passed binary check by having the new version present in output.
found 0.3.59 with grep in /nix/store/vd2npxlfw6d92n23mb56qrahg49h0jvy-python3.10-pyhocon-0.3.59
found 0.3.59 in filename of file in /nix/store/vd2npxlfw6d92n23mb56qrahg49h0jvy-python3.10-pyhocon-0.3.59

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
2 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
python310Packages.pyhocon
python39Packages.pyhocon

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/vd2npxlfw6d92n23mb56qrahg49h0jvy-python3.10-pyhocon-0.3.59 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.pyhocon https://github.com/r-ryantm/nixpkgs/archive/eeef4ca584cec1a2e59817e717a26ce288d9528e.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/vd2npxlfw6d92n23mb56qrahg49h0jvy-python3.10-pyhocon-0.3.59
ls -la /nix/store/vd2npxlfw6d92n23mb56qrahg49h0jvy-python3.10-pyhocon-0.3.59/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

python310Packages.pyhocon
python39Packages.pyhocon

Maintainer pings
cc @chreekat for testing.

@OfBorg eval


implement explicit MVN

Reference #229
Implemented large MVN and added test comparing to Kalman smoother.
Feedback welcome!

Thanks for contributing this! I'll do a code review asap, but at first glance this looks like exactly what we need!

Thanks for getting involved!
It might be useful to separate this test from the tfp-based tests (to give e.g. test_kalman_tfp() and test_kalman_vs_joint()) so that we get the results from the joint test even if the tfp test fails.
If you're up for it the ideal thing might be to refactor the tests to use the class-based approach as in e.g. info_infererence_test.py and parallel_inference_test.py (described here). No worries at all if that's not possible, I'm very happy to do it after this gets merged.

+1 to splitting the test into 2 - one for TFF, one for your code.

Cool! I'm on it!

I have done the refactor, following the pattern used in info_inference_tests.py. Let me know how it looks!

LGTM.

I will merge despite the failing test, which is unrelated to this PR (has something to do with tqdm).


⚠ ClamAV has degraded performance

In a1a56ab, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 555 ms

Resolved: ClamAV performance has improved in e5e8e8c after 10 minutes.


Support fish integration for fzf

Create the enableFishIntegration option to install the fzf key-bindings for
people who use fish shell.

Thanks! Rebased to master in 71c7aaee835e6fe2d67e53dd63d1cda52079e491.


No Ubuntu Terminal/ No Theme in Windows Terminal

I don't have an Ubuntu Terminal like I've seen with some users.

If I change the Settings.json file to customize my terminal, it doesn't work and there is no new theme applied.
i have add One Half Dark theme to "schemes": []
// This file was initially generated by Windows Terminal 1.4.3243.0
// It should still be usable in newer versions, but newer versions might have additional
// settings, help text, or changes that you will not see unless you clear this file
// and let us generate a new one for you.

// To view the default settings, hold "alt" while clicking on the "Settings" button.
// For documentation on these settings, see: https://aka.ms/terminal-documentation
{
    "$schema": "https://aka.ms/terminal-profiles-schema",

    "defaultProfile": "{61c54bbd-c2c6-5271-96e7-009a87ff44bf}",

    // You can add more global application settings here.
    // To learn more about global settings, visit https://aka.ms/terminal-global-settings

    // If enabled, selections are automatically copied to your clipboard.
    "copyOnSelect": false,

    // If enabled, formatted data is also copied to your clipboard
    "copyFormatting": false,

    // A profile specifies a command to execute paired with information about how it should look and feel.
    // Each one of them will appear in the 'New Tab' dropdown,
    //   and can be invoked from the commandline with `wt.exe -p xxx`
    // To learn more about profiles, visit https://aka.ms/terminal-profile-settings
    "profiles":
    {
        "defaults":
        {
            // Put settings here that you want to apply to all profiles.
        },
        "list":
        [
            {
                // Make changes here to the powershell.exe profile.
                "guid": "{61c54bbd-c2c6-5271-96e7-009a87ff44bf}",
                "name": "Windows PowerShell",
                "commandline": "powershell.exe",
                "hidden": false
            },
            {
                // Make changes here to the cmd.exe profile.
                "guid": "{0caa0dad-35be-5f56-a8ff-afceeeaa6101}",
                "name": "Command Prompt",
                "commandline": "cmd.exe",
                "hidden": false
            },
          {
            "guid": "{b453ae62-4e3d-5e58-b989-0a998ec441b8}",
            "hidden": false,
            "name": "Azure Cloud Shell",
            "source": "Windows.Terminal.Azure"
          }
        ]
    },

    // Add custom color schemes to this array.
    // To learn more about color schemes, visit https://aka.ms/terminal-color-schemes
  "schemes": [
    {
      "name": "One Half Dark",
      "foreground": "#DCDFE4",
      "background": "#282C34",
      "cursorColor": "#FFFFFF",
      "black": "#282C34",
      "red": "#E06C75",
      "green": "#98C379",
      "yellow": "#E5C07B",
      "blue": "#61AFEF",
      "purple": "#C678DD",
      "cyan": "#56B6C2",
      "white": "#DCDFE4",
      "brightBlack": "#5A6374",
      "brightRed": "#E06C75",
      "brightGreen": "#98C379",
      "brightYellow": "#E5C07B",
      "brightBlue": "#61AFEF",
      "brightPurple": "#C678DD",
      "brightCyan": "#56B6C2",
      "brightWhite": "#DCDFE4"
    }
  ],

    // Add custom actions and keybindings to this array.
    // To unbind a key combination from your defaults.json, set the command to "unbound".
    // To learn more about actions and keybindings, visit https://aka.ms/terminal-keybindings
    "actions":
    [
        // Copy and paste are bound to Ctrl+Shift+C and Ctrl+Shift+V in your defaults.json.
        // These two lines additionally bind them to Ctrl+C and Ctrl+V.
        // To learn more about selection, visit https://aka.ms/terminal-selection
        { "command": {"action": "copy", "singleLine": false }, "keys": "ctrl+c" },
        { "command": "paste", "keys": "ctrl+v" },

        // Press Ctrl+Shift+F to open the search box
        { "command": "find", "keys": "ctrl+shift+f" },

        // Press Alt+Shift+D to open a new pane.
        // - "split": "auto" makes this pane open in the direction that provides the most surface area.
        // - "splitMode": "duplicate" makes the new pane use the focused pane's profile.
        // To learn more about panes, visit https://aka.ms/terminal-panes
        { "command": { "action": "splitPane", "split": "auto", "splitMode": "duplicate" }, "keys": "alt+shift+d" }
    ]
}

I don't know about Ubuntu, but I think you need to use
colorScheme: "One Half Dark"
somewhere ... in a profile or in "defaults".

I don't know about Ubuntu, but I think you need to use
colorScheme: "One Half Dark"
somewhere ... in a profile or in "defaults".

The ubuntu option you see is more than likely the Windows Subsystem for Linux. If you install that you should have an Ubuntu shell on your machine.

The ubuntu option you see is more than likely the Windows Subsystem for Linux. If you install that you should have an Ubuntu shell on your machine.

@abdes-zakari The Terminal doesn't come bundled with Ubuntu, it only detects if you've already installed it. Can you confirm that you've installed an Ubuntu (or any other) distro from the store?

@abdes-zakari The Terminal doesn't come bundled with Ubuntu, it only detects if you've already installed it. Can you confirm that you've installed an Ubuntu (or any other) distro from the store?

No, I thought that the Ubuntu Terminal comes with Windows Terminal without prior installation of Ubuntu.
Thanks

No, I thought that the Ubuntu Terminal comes with Windows Terminal without prior installation of Ubuntu.
Thanks


Update even or odd.py

Use while loop and fast string
What type of PR is this? (check all applicable)

[ ] 🚀 Added Name
[ ] ✨ Feature
[ ] 🌟 ed the repository
[ ] 🐛 Grammatical Error
[ ] 📝 Documentation Update
[x] 🚩 Other

Description
Add Link of GitHub Profile

Add your name in CONTRIBUTERS.md file


feat:--20211112 add_images_datastructure

Description:
add images for some data structure, the image stores in images directory.
and these links are refer to the my temp branch, so I may change the links after the PR approved.

LGTM


[Issue-3579] [Hotfix] Handle balance for kusama

🚀 Deployed on https://pr-3580--sw-web-runner.netlify.app


GoLive Activities Checklist - MackTrucks Museum

Checklist of all activities needed for the GoLive of MackTrucks Museum:
Preparation tasks

[ ] #1

Pre, during, and post-GoLive:

[ ] #2

Pre-GoLive:

[ ] #3
[ ] #4
[ ] #5
[ ] #6
[x] #7
[ ] AEM GoLive steps:
Performance validation
Legacy redirects
Favicon
CDN configuration
Update production host

Post-GoLive:

[ ] AEM Post-GoLive Validation:
Performance validation
404 Report

@manuel-vara Can you make sure that Shirin, Vera, and Molly can access this ticket. I am unable to tag them. Thank you!


Maven: Failed to collect dependencies

Describe the bug
Failed to collect dependencies at com.github.appreciated:app-layout-addon:jar:2.0.1
Caused by: org.eclipse.aether.transfer.ArtifactNotFoundException: Could not find artifact com.github.appreciated:app-layout-root:pom:2.0.1-SNAPSHOT in vaadin-addons (http://maven.vaadin.com/vaadin-addons)
To Reproduce
Steps to reproduce the behavior:
Add dependency to pom.xml:

com.github.appreciated
app-layout-addon
2.0.1

Run maven:
mvn clean install -X -U
Expected behavior
app-layout-addon dependencies resolved
Screenshots
NA
** Additional Information (please complete the following information):**
mvn -version
Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-18T02:33:14+08:00)
Maven home: /opt/local/apache-maven
Java version: 1.8.0_191, vendor: Oracle Corporation, runtime: /Library/Java/JavaVirtualMachines/jdk1.8.0_191.jdk/Contents/Home/jre
Default locale: en_RU, platform encoding: UTF-8

Probalby a duplicate of #198

I stopped the Release and I'll see if I can fix it asap

2.0.2 is now in the shelves!

Thanks!

Thanks a lot!


Added Top Languages section

Description
I have added a top languages section using an npm package called GitHub Polyglot. This package can return the most used language of any user on GitHub. Usually, it returns the net usage of most used languages, but since we needed to show only the top 3 that's why I have only shown the top 3 languages. Also used shields.io to make badges for languages.
Fixes #61
Type of change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[x] Test A: tested on my local system. Screenshot of current look:-

Test Configuration:

Firmware version:
Hardware:
Toolchain:
SDK:

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

Well done, @ritvij14! Looks fine to me, approved. @garimasingh128 please have a look and merge.


Improve efficiency of stats

Back down to ~2 sec

Version should be bumped to 1.10.1 before being merged


add vccs

@jeck5001 I have to close this request without merging, because you requested to add a non-existent domain to this repository: Host vccs.edu.vccs not found: 3(NXDOMAIN) Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for understanding!


calculate canvas size dynamically

... but always overridable via the command line parameters

fontsize and repeat factor should be taken into consideration

What metrics are you using for this calculation Jorg?

Haven't started this, I am open to suggestions

Want to look at the data that the fontbox.ttf library is able to access?  Happy to provide some guidance if you have documentation of what is available.


Fix channel loss on reconnect

The scenario requires two or more close-by disconnections

Connect to Redis
Subscribe successfully to a channel
Forcibly kill the connection (by taking down Redis for example)
Let lettuce reconnect successfully
Forcibly kill the connection again right before activate() successfully subscribes
Let lettuce reconnect successfully
RedisPubSubConnectionImpl.channels is empty and the channel will stay unsubscribed

There doesn't seem to be a reason to clear out channels and patterns.
It is only used for persistence over different connections.
There is no downside to having a channel in that set even if the subscription was not successful.

Coverage increased (+0.08%) to 96.29% when pulling f6b869aede4b6c6438de970e376384c6d800f35e on kichik:master into cc638d450f2f74fe95233e9721022c15555a73fb on mp911de:master.

Looks good to me. Both collections patterns and channels are sets. A redis reply of subscribed to the pattern/channel won't cause duplicates. Thank you for the PR.

Thanks for the super fast response (again)!
Any chance you can release 3.0.3 with this?

Will do. Could you check the memory leak fix I pushed earlier this day?
Von meinem iPhone gesendet

Am 31.01.2015 um 20:02 schrieb kichik<EMAIL_ADDRESS>Thanks for the super fast response (again)!
Any chance you can release 3.0.3 with this?
—
Reply to this email directly or view it on GitHub.


Fix PendingDeprecationWarning in xonsh 0.5.1

FORMATTER_DICT is deprecated since xonsh 0.5.1, so there's an annoying warning on every shell update:
PendingDeprecationWarning: FORMATTER_DICT is an alias of PROMPT_FIELDS and will be removed in the next release

This small change switches to the new PROMPT_FIELDS to resolve this issue.

I gave this fix a try and it's working well for me

Sorry for taking so long to respond, I was on vacation. I will look at the fixes and release a new version of powerline that I have been developing on my machine unfortunately (it all started as test for the new toolbar prompt changes).

And just to say, thanks both for using the xontrib (and xonsh) and for xontributing ;).


SwiftUI Todo List App

SwiftUI Todo List App
https://www.youtube.com/watch?v=wEf1YS4vyW8&list=PLwvDm4VfkdpheGqemblOIA7v3oq0MS30i

https://github.com/SwiftfulThinking/SwiftUI-Todo-List-MVVM-UserDefaults

https://github.com/SwiftfulThinking/SwiftUI-Advanced-Learning


AD&D 1e Legacy

trying to fix the layout issues that happening on the live server vs sandbox and custom game.  Same code renders a different look on the live server for some reason...

Submission Checklist
Required

[x] The pull request title clearly contains the name of the sheet I am editing.
[x] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[x] The pull request makes changes to files in only one sub-folder.
[x] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

New Sheet Details

The name of this game is: <   >  (i.e. Dungeons & Dragons 5th Edition, The Dresden Files RPG)

The publisher of this game is: <   > (i.e. Wizards of the Coast, Evil Hat)

The name of this game system/family is: <   > (i.e. Dungeons & Dragons, FATE)

[x] I have followed the Character Sheets Standards when building this sheet.

[ ] I have authorization from the game's publisher to make this an official sheet on Roll20 with their name attached.
[ ] This game is not a traditionally published game, but a copy of the game rules can be purchased/downloaded/found at: <   >
[ ] This sheet is for an unofficial fan game, modification to an existing game, or a homebrew system.

Changes / Description

Character Sheet Info Roll20 Internal Use only.


Introduce support for node v6.3.x

Node v6.3.1 release notes

Thanks @cilindrox ! Merged and released.


Welcome page Easter egg...

I found the landing page...but it was buried under scRNA seq... should this be shown upon loading?

correct - this is WIP (work in progress)


Can we enable more flexibility with NODE_ENV?

What problem does this feature solve?
https://github.com/Financial-Times/dotcom-reliability-kit/blob/f4fb427b50302ae6d25b800d9688a4c6013f5752/packages/app-info/lib/index.js#L68
Only three values are handled by the logger:

'production'
null
'development'
The aim being to only show pino pretty logs whilst working in development

Ideal solution
We make extensive use of test as our pre-production value for NODE_ENV. It would be great if that was treated the same as development 😄 so we could see the pretty logs
Alternatives
We could change all our references from test to development but our environments and aws phases are named in the same way using test and production 🤔

Hey Geoff 👋 I'm happy for logs in the test environment to also get prettified. It'd be a relatively simple change, this line (and relevant tests) needs updating to also return true if the value is test.
I might get to this at some point but also happy for you to open a pull request.

https://github.com/Financial-Times/dotcom-reliability-kit/pull/481


Add a simple Dockerfile to rmbt-server

This PR adds a simple Dockerfile to rmbt-server, to be able to quickly and easily run it in Docker.
For ease of use, the token check on the server has been disabled.

Dear Leo,
thank you for the PR, this is great functionality!
If that's ok with you, I'll merge the PR and then re-reoute the links to the rtr-nettest-user as to not have instructions in the README that reference an external repository.

Yes, sure, no problem.


add more data-only hotspot data to api

add count of data-only Hotspots

right now, we only show a count of all hotspots on chain, it'd be nice to show the subset of those that are data-only or light/full.
food for thought: is it worth adding a hotspots/stats, like there is for validator/stats?

generate paginated list of data-only hotspots for explorer map filter

the explorer team would like to plot data-only hotspots on a map, and so we'll need a paginated list of these hotspots

note to marc: lower priority, should be looked at after finishing "inverse of witness list" request.

Fixed by #295

@cokes518 Likely yes on stats, but less put a different issue in for it when we actually need it?