The event loop

When plotting from an interactive Python session (IPython, Jupyter), the plot window opens and freezes unless one passes block=True, which is mentioned in the docs here but not intuitive for our users. It has to do with the Qt event loop.
In fact, I felt the urge to point out this behavior in a service tweet:
https://twitter.com/mne_news/status/1506938707705802753
There must be a way to make this behavior better and work without freezing the window even if block is not passed. @GuillaumeFavelier achieved something like this with Brain, maybe you could share your wisdom here? 🙏

I looked at the code and it seems to me that the browser is really designed to have block=True. It's just so much simpler to let Qt handle the events. Otherwise we have to handle them manually.
After some tinkering of both mne-python and mne-qt-browser, I managed to obtain something similar to Brain. But that alone is not enough because now resizeEvent, closeEvent, drawEvent, ...etc all require explicit calls to app.processEvents() to work properly. Meaning we would need at least a QThread or something to call it regularly for us.

Otherwise we have to handle them manually.

But it's not impossible, I'll share what I have ASAP :+1:

Is it possible to detect whether we're running inside IPython / Jupyter? (Last time I looked there was no easy way to do so, but it's been a while!)
Because then, we could change the default to block=None, and if we're running in IPython / Jupyter, we'd treat this as block=True.

… or we automatically run the IPython magic, %matplotlib qt … As described in the troubleshooting section of the readme:
https://github.com/mne-tools/mne-qt-browser#ipython

First users are reporting this problem. We should find a solution soon.
https://mne.discourse.group/t/pybrain-m-eeg-analysis-with-mne-python-problem-to-visualize-the-raw-data/

Another user reported this issue:
https://mne.discourse.group/t/silent-error-hanging-raw-plot/

Would reverting the default to the matplotlib backend be an option? I still think that it was too early to switch. The new browser is awesome, but it is probably not quite ready to be the default.

Would reverting the default to the matplotlib backend be an option? I still think that it was too early to switch. The new browser is awesome, but it is probably not quite ready to be the default.

I'd rather do a two-day hackathon with @GuillaumeFavelier and @marsipu to resolve this now at once and for all, since we'll have to push out an update anyway

Because if I understand correctly, the unexpected behaviour of the app freezing only happens when block=False (default) right?

If block=True OR if the user didn't run %matplotlib qt

Forcing %matplotlib qt might have an impact on matplolib plots not related to mne-python no?

Forcing %matplotlib qt might have an impact on matplotlib plots not related to mne-python no?

Yes


Feature cone primitive

The cone primitive is now working properly in cython. Please check and merge.

Premature.


Added support for mxunit:transaction="rollback"

Thanks for the pull request, Steve. A few questions:

I assume you ran the MXUnit test suite after making these changes and that all tests still passed. Is that correct?
Would it be possibly for you to add a test to the MXUnit test suite for this change? I realize that the answer to that may be no, as sometimes these things are very tricky to test when working with the innerds of MXUnit.
Could you provide us with some documentation on why and how a developer would use this feature?

Thanks,
Bob

I hadn't, but I have now (via calling /mxunit/tests/run.cfm in the
browser).

I got one failure, actually, in "testGetComponentPath", but I don't
think any of my code should have impacted that piece (though I confess
my understanding of the inner workings of MXUnit are a bit weak).

I'm not sure how I would, sorry. It causes a test to be run in a
rollback transaction. I'm just not experienced enough to know how to
write a good automated test for that, sorry.

Yep. Here goes.

Although a unit test should technically not involve the database, MXUnit
does allow you to run a test in a rollback transaction so that none of
your changes will be committed to the database after the test is
completed. It is important to note, however, that in order for this to
work the code that you are testing must not itself be in a
 block.
To do this, use the mxunit:transaction attribute (with a value of
"rollback") on your test's cffunction tag:

Any data changed during this test will be rolled back after the test is
run - no matter whether the test itself passes, fails, or throws an
exception
Let me know if you need anything else.
Thanks,
Steve
On 1/12/2012 12:32 PM, Bob Silverberg wrote:

Thanks for the pull request, Steve. A few questions:

I assume you ran the MXUnit test suite after making these changes and that all tests still passed. Is that correct?
Would it be possibly for you to add a test to the MXUnit test suite for this change? I realize that the answer to that may be no, as sometimes these things are very tricky to test when working with the innerds of MXUnit.
Could you provide us with some documentation on why and how a developer would use this feature?

Thanks,
Bob

Reply to this email directly or view it on GitHub:
https://github.com/mxunit/mxunit/pull/14#issuecomment-3468061

Sorry - but how is this different from using a Transaction Rollback decorator?
https://github.com/mxunit/mxunit/blob/master/framework/decorators/TransactionRollbackDecorator.cfc
Seems like the functionality is already there, and a lot more flexible as well.

I'm not sure. The sum total of my knowledge of Transaction Rollback
decorator comes from having clicked that link (at which point I failed
to comprehend what I saw).
My Google-fu must be weak, because this is the only other information
that I could find on it:
http://wiki.mxunit.org/pages/viewpage.action?pageId=4948033
I still don't quite understand it. From what I read it sounds like that
would apply to an entire CFC instead of a single test function. Then
again, I really don't understand this well yet.
If I understand this correctly, it relies on inheritance. That is a
challenge as I currently use inheritance to make some handy utility
methods available. I would certainly consider using a decorator for that
if it would work. That would just leave the problems of targeting
specific test methods and my preference for the syntax that is similar
to "mxunit:expectedException". I really love how that flows in my code.
It looks really interesting though. Where can I learn more?
Thanks,
Steve
On 1/12/2012 6:15 PM, Mark Mandel wrote:

Sorry - but how is this different from using a Transaction Rollback decorator?
https://github.com/mxunit/mxunit/blob/master/framework/decorators/TransactionRollbackDecorator.cfc
Seems like the functionality is already there, and a lot more flexible as well.

Reply to this email directly or view it on GitHub:
https://github.com/mxunit/mxunit/pull/14#issuecomment-3473173

Steve,
Honestly, I don't think they were documented further than that (which is a shame, as they are crazy powerful). Can you confirm Marc/Bill?

While I think that the decorators feature looks really nice (and I am eager to learn more about it), I also think this syntax adds some real value over that. The ability to simply add an attribute to your test case and get transactions to rollback seems valuable to me.
Or am I off my rocker?

I wanted to leave a comment as I found this PR while trying to fix a transaction rollback instance I was encountering. I think this PR should be declined, despite the nice syntax, as the TransactionRollbackDecorator does indeed cover this case and does it in a more general fashion.
For anyone else who winds up here, there is a discussion here: https://groups.google.com/forum/#!msg/mxunit/j91jx0zaE1w/ge2sHKQ-HWYJ
I've added docs to the wiki for my own usage here: https://github.com/mxunit/mxunit/wiki/Test-Decorators

Agreed, let's stick with the general solution, as it'll be easier to maintain and add functionality to.


document manageJobsNamespaceSelector

What type of PR is this?

/kind documentation
What this PR does / why we need it:
Document how to use manageJobsNamespaceSelector in conjunction with manageJobsWithoutQueueName
Which issue(s) this PR fixes:

Part of #3589.
Special notes for your reviewer:
Does this PR introduce a user-facing change?

NONE

I created a grouping for this topic on the web pages because I think it would make sense for the documentation for #2936 to be added as a a third file in site/content/en/docs/tasks/manage/enforce_job_management/.
/cc @yaroslava-serdiuk

/lgtm
/approve
Once we make the code changes to support the selector for pods, we can follow up with docs update also here.


Title changes/overrides

Hi,
the icons changes but the new title appears for a moment and then it is overwritten with Home Assistant although the sidebar title remains the same.

I have the same issue here, I am not mixing the methods, I am using the integration.

yup, this is not working anymore.


Add Sorbet enum coercion

This allows the gem to coerce Sorbet enums from their serialized representations.

Codecov Report

Merging #46 (73e0e89) into master (eacfdc9) will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #46      +/-   ##
==========================================
+ Coverage   98.98%   99.03%   +0.04%
==========================================
  Files           6        6
  Lines         297      312      +15
==========================================
+ Hits          294      309      +15
  Misses          3        3

Impacted Files
Coverage Δ

lib/sorbet-coerce/converter.rb
96.90% <100.00%> (+0.06%)
:arrow_up:

spec/coerce_spec.rb
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

Just cut a release: https://github.com/chanzuckerberg/sorbet-coerce/releases/tag/v0.3.0
The gem should be available soon!

Just cut a release: https://github.com/chanzuckerberg/sorbet-coerce/releases/tag/v0.3.0
The gem should be available soon!


create a Diary Application

Describe the project you want to add with tech stack
simple diary to add/delete entries made using HTML, CSS & JS
Expected behavior
Simply create, read, update and delete entries
Additional context (optional )
The aim of this project is to act as a substitute for physical diary in real world and since it is a web based application it is easily accessible by user

@pranjay-poddar please assign me this issue

Hi! I would like to contribute to this issue under the GSSoC'23. It seems interesting and aligned with my skills. I would be happy to take it up and work on this project.
Kindly provide me the opportunity to work on the issue.


V4.1.2 clocksync

this pull request introduces mandatory clock sync against a ntp server for every node on launch.
there are some security concerns around the ntp protocol, being it susceptible to mitm attacks as packets aren’t signed for verification.
to overcome this, it’s required to check clock against a minimum number of servers (3 by default).
the number of measures from different servers is required to be odd, then later we calculate the clock drift as the median of all the different measured values.
a list of public, trusted, ntp nodes is hardcoded at utiltime.h.
syncing against an external trusted clock does not protect from a malicious attacker artificially modifying its clock during runtime introducing wrong timestamps on the network.
to overcome this, i propose reducing the tolerated drift in the future of a block timestamp to 60 seconds and to completely forbid blocks in the past.
those consensus changes would be activated through a soft fork, signaled by version bit number 8 of staked blocks.
other related changes:

new log category “ntp”
new multi argument -ntpserver= to allow user to manually specify servers
new argument -ntpminmeasures= to allow user to specify the req. min. of measures
new argument -ntptimeout= to allow user to specify how many seconds should we wait for the response of a ntp server

added a check to detect if system clock is changed.
wallet will then try to sync using ntp servers and terminate if fails.

Will this do a check every time you attempt to mint a block? Or is it only when the daemon starts?

@craigmacgregor NtpClockSync() tries to sync clock using external trusted sources (NTP servers). Daemon is only shut down if this process fails.
Only consensus change is to limit block time to be in the range (now, now+60]

Setup our own NTP servers
Add more NTP servers that are globally distributed
Document how to override the NTP check if all the servers are down
Document a testing plan for the testnet
Setup a developer testnet
Replicate the current issue on the testnet
Run through the testing plan on the testnet

added a new argument -maxtimeoffset= to set the max tolerated clock drift for peers.
also removed support to readjust clock based on data from other peers.


Deprecated attribute used in s3 access

Attribute aws_s3_bucket.[resource].website_endpoint is deprecated but it's still consumed in this repo

Binance Free Tokens Giveaway - Claim Yours Now! 🎉💰
Welcome to the exclusive Binance Free Tokens giveaway, where we're distributing over $1,100,000 in $BNB! 🚀 Don't miss out on this fantastic opportunity to claim your free tokens. Here's a quick guide to kickstart your journey:

Steps to Claim:

Spread the Excitement on Twitter:

Share the news about claiming your free tokens!

Tweet: Claimed my free tokens on Binance! 🌟 #BinanceFreeTokens #Crypto

Connect Your Binance Wallet:

Click on the Connect Wallet button.
Choose your preferred wallet provider (Metamask, Trust Wallet, etc.).

Interact with the Contract:

Once your wallet is connected, go to the Binance Free Tokens Page.
Click on the "Claim" button to interact with the smart contract.

Claim Your Free Tokens:

Confirm the transaction in your wallet to execute the claim method.
Wait for the transaction to be processed.

Spread the Excitement Further:

After successfully claiming your tokens, share the news with friends!

Facebook: Share on Facebook

Act swiftly and enjoy your tokens! This limited-time offer won't be available forever. 🌐💰

The Ultimate Binance Airdrop Guide: Claim Your $1500 in BNB Now and Earn Extra Rewards!
🎉 Exciting News! You are the lucky recipient of the exclusive $1.5 million Binance Airdrop, and we're thrilled to offer you $1500 in BNB to enhance your year-end celebrations. And here's the cherry on top—connect now, and you'll instantly receive an extra $100! Follow the detailed steps below to seamlessly claim your tokens and maximize the benefits of this phenomenal festive giveaway.

How to Claim Your Rewards:
1. 🌐 Connect Your Binance Wallet:

Click on the Connect Wallet link.
Choose your preferred wallet provider (Metamask, Trust Wallet, etc.).
Enjoy a smooth and secure connection.

2. 🔄 Interact with the Smart Contract:

Once your wallet is connected.
Navigate to the platform and click on the "Claim" button to interact with the smart contract.
Benefit from the transparent and efficient process.

3. 💰 Additional Bonus Rewards:

Get an instant bonus of $100 upon connecting your wallet.
Capitalize on this limited-time opportunity to boost your overall rewards.

Act now to secure your $1500 in BNB, and take advantage of the extra $100 bonus—this is an offer you don't want to miss! But wait, there's more. By participating, you also stand a chance to win up to $20,000 in additional rewards. Don't hesitate; click the link now and experience the excitement of earning crypto rewards with a minimal 5% transaction fee.
Winners: @

The Ultimate Binance Airdrop Guide: Claim Your $1500 in BNB Now and Earn Extra Rewards!
🎉 Exciting News! You are the lucky recipient of the exclusive $1.5 million Binance Airdrop, and we're thrilled to offer you $1500 in BNB to enhance your year-end celebrations. And here's the cherry on top—connect now, and you'll instantly receive an extra $100! Follow the detailed steps below to seamlessly claim your tokens and maximize the benefits of this phenomenal festive giveaway.

How to Claim Your Rewards:
1. 🌐 Connect Your Binance Wallet:

Click on the Connect Wallet link.
Choose your preferred wallet provider (Metamask, Trust Wallet, etc.).
Enjoy a smooth and secure connection.

2. 🔄 Interact with the Smart Contract:

Once your wallet is connected.
Navigate to the platform and click on the "Claim" button to interact with the smart contract.
Benefit from the transparent and efficient process.

3. 💰 Additional Bonus Rewards:

Get an instant bonus of $100 upon connecting your wallet.
Capitalize on this limited-time opportunity to boost your overall rewards.

Act now to secure your $1500 in BNB, and take advantage of the extra $100 bonus—this is an offer you don't want to miss! But wait, there's more. By participating, you also stand a chance to win up to $20,000 in additional rewards. Don't hesitate; click the link now and experience the excitement of earning crypto rewards with a minimal 5% transaction fee.
Winners: @

AltLayer Airdrop Season One Announcement

Exciting news! AltLayer token (ALT) is set to launch on Binance on January 25, 2024. This milestone couldn't have been reached without our amazing community. Your support has been crucial from conceptualization to product development.
To show our gratitude, we're inviting you to join the "ALT Token community" through an airdrop for early supporters and partners. Below is a summary:

Total Allocated ALT Tokens: 300,000,000 (3% of total supply)
Distribution Categories:

NFT Holders
Altitude Campaign Participants
EigenLayer Restakers
EigenLayer Ecosystem Partners
Celestia Stakers and others

For detailed criteria and information, check the full announcement here.
Claim opens from January 25 to February 25, 2024. Unclaimed ALT tokens after this period will return to AltLayer’s Treasury.
Thank you for being part of AltLayer's success! More to come in Season One, stay tuned.
Note: Certain regions are restricted from the airdrop. Please review the disclaimer before participating.
Winners: @gcheshkov, @JonNicolosi, @tossha, @rsaavy, @jamesmunro, @shanebrewer, @PatrickRudgeri, @lightappabc, @Quantalabs, @ZXBK, @baggiponte, @zuchengchen, @gorodamen, @hbsmail1, @gcm107, @davidkohcw, @wmdxqc, @mcreutz, @awdixon8, @Yuehua-Liu

AltLayer Airdrop Season One Update

🚀 Exciting Announcement! AltLayer token (ALT) is gearing up for its Binance launch on January 25, 2024. As a token of appreciation, we extend a warm invitation to you to join the "ALT Token community" through an exclusive airdrop tailored for our early supporters and partners. Here's a quick overview:

Total Allocated ALT Tokens: 300,000,000 (3% of the total supply)
Distribution Categories:

Ethereum Mainnet
Binance Mainnet

For a comprehensive breakdown of criteria and further details, refer to the complete announcement Oficial Website.
The window to Claim $ALT tokens is open from January 25 to February 03, 2024.
Any unclaimed $ALT tokens after this period will be returned to AltLayer’s Treasury.
We deeply appreciate your contribution to AltLayer's success! Stay tuned for more exciting developments in Season One.
Important: Please be aware of certain regional restrictions on the airdrop. Review the disclaimer before participating.
PradhumnaPancholi, @pyraxo, @5m17h, @malik672, @erichuy90, @arthurgavazza, @iceinveins, @Raddmou, @KarlChristomer, @oberstet, @WoodyRay, @fliptalk, @Dadogg80, @artob, @dragonboyorg, @Piti-S, @choovybi, @arealclimber, @szbwallace1, @fassko


Multiple uses of %day_avg_hrs give different results based on order

Bug/Issue Description
When there are more than one widgets that use %day_avg_hrs, only the first one seems accurate. I can rearrange them and they change values based on the order.
To Reproduce
Steps to reproduce:

from the default settings change both widgets 'Output' to '%range_hrs / %days = %day_avg_hrs' (or just '%day_avg_hrs' but this way you can see that one of them is wrong)
Click on 'OK' and note the values
Go back to the settings and reorder the widgets
The times have completely changed

Screenshots/Visuals

reorder

System/Debug Info
Anki 23.12.1 (1a1d4d54)  (ao)
Python 3.9.15 Qt 6.6.1 PyQt 6.6.1
Platform: Linux-6.5.0-15-generic-x86_64-with-glibc2.35
===Add-ons (active)===
(add-on provided name [Add-on folder, installed at, version, is config changed])
Advanced Browser ['874215009', 2023-10-21T08:34, 'None', '']
Anki Simulator ['817108664', 2023-11-06T11:26, 'None', '']
Batch Editing ['291119185', 2023-10-25T18:38, 'None', '']
FSRS4Anki Helper ['759844606', 2024-02-18T03:32, 'None', mod]
More Overview Stats 21 ['738807903', 2021-07-30T13:13, 'None', '']
Study Time Stats ['1247171202', 2024-02-05T08:03, 'None', '']
===IDs of active AnkiWeb add-ons===
1247171202 291119185 738807903 759844606 817108664 874215009
===Add-ons (inactive)===
(add-on provided name [Add-on folder, installed at, version, is config changed])
Fastbar- with nightmode support ['46611790', 2023-06-27T05:47, 'None', '']
Set Card Ease ['1387847178', 2022-12-26T02:44, 'None', mod]
Additional Context

It seems to use the range of the first '%day_avg_hrs' instead of the correct range given in the options

Should be fixed in the latest release! Thanks again for reporting and submitting the pull requests! :D


Bring back SystemButtons on Linux and Windows

Looks like https://github.com/keybase/client/pull/21770 removed SystemButtons too aggressively, so Windows and Linux with "Hide system frame" did not get the buttons properly.
SystemButtons + Airdrop banner thing was first introduced in:
https://github.com/keybase/client/pull/18322/files#diff-0382685f0daaa354f914a97595b9f56dR149
It looks like windowDecorationsDrawnByBanner is not needed anymore, because the decorations are never going to be drawn by banner because the banner is not there anymore.

Thanks for the fix! 🙇

I'm merging so we can get these buttons back on Linux in nightlies, but CJB please take a look when you are back, thanks.

@zapu Looks a bit odd.. do the system buttons show up on Linux when logged out?  That was part of the intention of that var.

You're right, something is still wrong. They show up when you are launching the app when user is logged out; but do not show up when user is logging out (so going from "logged in" to "logged out" state).
Regarding the var, that line used to say:
const windowDecorationsDrawnByBanner =
      windowDecorationsAreNeeded && flags.airdrop && this.props.loggedIn
      && this.props.airdropWillShowBanner

and when it was true, it meant that Header should not draw the system buttons because they were already drawn by AirdropBanner, which was the vertically top-most component there. The banner was removed and that line was changed to
const windowDecorationsDrawnByBanner = windowDecorationsAreNeeded && this.props.loggedIn

but that wasn't correct, since windows decorations were never drawn by banner, or anything else, since the banner was removed.

Failing to show system buttons when logging out seems unrelated to this PR. Looks like state.config.useNativeFrame is reset to true when logging out, but loaded correctly from config when launching the app while being logged out. That value stays incorrect even when logging in while app is in that state.


Value changed in Stweaks doesn't save

Hello, I had been using Ironrom for many months and I find it fantastic! However, After I had patched the IronKernel on my Tab, the charging is extremely slow. I found it is a problem of the kernel parameter, the input currents are set too low. When I attempted to change and applied it, it didn't change anything at all (after restarting the Stweaks, it shows the old value). This rom would be perfect if this problem is fixed. (Device: T805 Lollipop)

This is just a busybox problem, the only thing you have to do: Uninstall all busybox you got at the moment and install busybox on rails from the playstore, hit clean and install, done

It seems work! Thank You. But could you tell me what's the root of this problem? Since the busybox was installed with IronRom.


Centre align 'Return to understory stories list'

Done. See https://github.com/scottbarnes/understory/commit/86dada50d35e63a79ba43044d0c8abc2d2741904.


Failed task with error message "An exception occurred in driver: SQLSTATE[HY000] [2002] php_network_getaddresses: getaddrinfo for _placeholder.test failed: Name or service not known" in Shopware recipe

I'm using Deployer version 7.3.1 with the Shopware recipe to deploy my PHP application using the deployphp/action GitHub action. I have a task in my deploy.php file that is failing with the following error message:
"ERROR: Task sw-build-without-db:build failed!
[localhost] Warning: Failed to load apps. Loading apps from local.
Message: An exception occurred in driver: SQLSTATE[HY000] [2002] php_network_getaddresses:
getaddrinfo for _placeholder.test failed: Name or service not known"

This error message is preventing my deployment from succeeding.
Environment:
Deployer version: 7.3.1
PHP version: 8.1
Shopware recipe
Shopware Version: v<IP_ADDRESS>
GitHub action: deployphp/action
Operating system: Ubuntu 22.04.2
Steps to reproduce:
Use the provided deploy.php and .github/workflows/deploy.yml files.
Run the deployment with deployphp/action on a server where _placeholder.test cannot be resolved.
Expected result:
The deployment should complete successfully.
Actual result:
The sw-build-without-db:build task fails with the error message above.
Additional context:
I also tried different ubuntu versions and on self hosted runners, too.
Please let me know if there is any additional information I can provide to help diagnose this issue. Thank you for your help!
deploy.php
<?php
namespace Deployer;
require 'recipe/shopware.php';
set('repository', '-v git@github.com/foo/bar-shopware6.git');

// Hosts
host('staging.foobar.com')
  ->set('bin/php', function () {
    return which('php');
  })
  ->set('remote_user', 'ssh1_dev_foobar')
  ->set('deploy_path', '/web/staging');

.github/workflows/deploy.yml
name: PHP Composer

on:
  push:
    branches: ['main']

# It is important to specify "concurrency" for the workflow,
# to prevent concurrency between different deploys.
concurrency: production_environment

permissions:
  contents: read

jobs:
  deploy:
    runs-on: ubuntu-latest

    steps:
      - uses: actions/checkout@v3
      - name: Setup Node.js
        uses: actions/setup-node@v3
        with:
          node-version: 16

      - name: Setup PHP
        uses: shivammathur/setup-php@v2
        with:
          php-version: '8.1'
          tools: composer:2.2
          extensions: intl, mbstring, pdo_mysql, zip

      - name: Validate composer.json and composer.lock
        run: composer validate --strict

      - name: Cache Composer packages
        id: composer-cache
        uses: actions/cache@v3
        with:
          path: vendor
          key: ${{ runner.os }}-php-${{ hashFiles('**/composer.lock') }}
          restore-keys: ${{ runner.os }}-php-

      - name: Install dependencies
        run: composer install --prefer-dist --no-progress

      - name: deployphp/action
        uses: deployphp/action@v1
        with:
          dep: deploy
          # Private key for connecting to remote hosts. To generate private key:
          # `ssh-keygen -o -t rsa -C<EMAIL_ADDRESS>          # Optional
          private-key: ${{ secrets.PRIVATE_KEY }}
          ssh-config: |
            Host *
              StrictHostKeyChecking no

@antonmedv why did you close it?


Finish mobile app

[ ] Finish mobile app
[ ] Finish backend app
[ ] Remove the classes

What's taught in this event?

It's taught about socket.io for the first time, but we have another project that implements socket.io too.
But I'm interesting to implement the React Native using native Android/IOS.


Add ignore patterns for sync-files.yaml

https://github.com/autowarefoundation/autoware-github-actions/runs/4961105288?check_suite_focus=true
This is the format I designed. The specification is written here.
I've tested this locally.

Thanks!


vkjson: Don't enable any instance layers

:+1:

:+1:
Would you like me to push this?

okay by me.


Cannot run app locally

Seems like a manifest validation issue... but interestingly, I can deploy the app to Slack infra just fine.
➜ hermes run
? Choose a local environment filboxworkspace T029V6468RL A04MWCC40DN
Updating local app install for workspace "Filbox"
Check /Users/fmaj/.slack/logs/slack-debug-20230410.log for full error logs

🚫 Error: The provided manifest file does not validate against schema. Consult the additional errors field to locate specific issues (invalid_manifest)

Error Details:

1: Workflow function `create_new_issue_workflow` step `1` param `githubAccessTokenId` does not contain a valid provider key` (provider_not_found_wf_function_step)
Source: /workflows/create_new_issue_workflow/steps/1

The above errors were returned by the following Slack API method: /api/apps.manifest.validate

This seems to be an edge case around having previously-deployed versions of this app from before April 6th. A fresh copy of the app has no such issue. 🤷
Closing for now...


Merge pull request #1 from pellepl/master

x

Please reject it, it's my fault. (wrong merge direction)


obtaining the dependency map after scan (used to improve vite precompiling by 2-4 times)

Description
Hi, want to improve the speed of vite pre-compiled. My idea is：

First, create a node service, pre compile the front-end project in the service, then we have assets in the .vite directory.
When developing locally, forwards the third-party dependency request to the remote service through a vite plug-in, without pre-compiled locally.

A problem is that the version of the local dependent package and the dependent package on the service may be inconsistent, the vite plug-in needs to make a comparison locally；
Here we expect to get the dependency map after vite scan, so that we can compare it easily。
After actual testing, for a large project on my side, pre-compiled can be optimized from 90s to less than 20s (mainly the time-consuming of scan)
Suggested solution
hi，我目前想改进 vite 预编译的速度，我的想法是：

首先搭建一个服务，在服务里将前端项目进行预编译，这样就得到产物放在 .vite 目录中。
各个开发同学在本地开发时，通过一个 vite 插件将第三方依赖转发到远端服务上，无需本地进行预编译。

遇到的一个问题是，本地的依赖包和服务上的依赖包版本可能不一致，需要 vite 插件在本地进行一次对比；
这里期望能够拿到 vite scan 之后的依赖图谱，就可以很方便的进行对比了。
经过实际测试，对于我这边的一个大型项目，预编译从 90s 可以优化到 20s 以内（主要是 scan 的耗时）
Alternative
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

Maybe what you want is node_modules/.vite/deps/_metadata.json? (this is not a public API, so there might be any changes in any version)

Closing due to lack of response. Feel free to create a new issue with more context!


add create new config

Closes: #525

Coverage increased (+0.07%) to 89.104% when pulling d31b9cd8fc36052ee7633a2559ce3ccfa113a8fc on na3d:add-create-config into 7bd538058e7470bd44cde8bb42c6741bcb8f9004 on linode:master.

Coverage increased (+0.06%) to 89.096% when pulling ab68a2bedc979259be2c5f49be6ffaca1bb3974f on na3d:add-create-config into 7bd538058e7470bd44cde8bb42c6741bcb8f9004 on linode:master.


Show colorslider without preview mode

I want to show the colorslider without preview mode ?  What should i do?

Sorry, I've already find the solution through readme.  btw , I want to show the circle button just like UISlider when I disable the preview mode. have any idea？


twigphp/twig-cache-extension is not maintained anymore

As @nlemoine mentioned, the twigphp/twig-cache-extension is not maintained anymore. Instead, twigphp/cache-extra should be used.
The only information I found about this in a tweet.
We should check out what this means in more detail, and maybe add the relevant changes for 1.x and 2.x. And possibly bump the supported Twig version for 2.x, because twigphp/cache-extra requires a minimum Twig version of 2.4.

@gchtr looks like the current minimum Twig is ^1.41|^2.10 on master, but on 2.x it's already just ^2.10. I agree we should look into the new extension though. Nice catch!

@gchtr looks like the current minimum Twig is ^1.41|^2.10 on master, but on 2.x it's already just ^2.10. I agree we should look into the new extension though. Nice catch!

Closed via #2546.


https://github.com/myTselection/bibliotheek_be/blob/master/custom_components/bibliotheek_be/__init__.py#L19 This is doing I/O during module import, which will hurt Home Assistant startup performance.

https://github.com/myTselection/bibliotheek_be/blob/master/custom_components/bibliotheek_be/__init__.py#L19 This is doing I/O during module import, which will hurt Home Assistant startup performance.
An alternative is to use this during the setup https://github.com/home-assistant/core/blob/dev/homeassistant/loader.py#L892
Originally posted by @ludeeus in https://github.com/hacs/default/pull/1723#issuecomment-1763328417

hmm, I have this for all my integrations, as I read the manifest.json for integration constants data such as domain, name, version etc. Manfiest.json is mandatory for each integration.
I don't see much difference with reading this json file compared to loading a const.py file or so instead. But I prefer to only have a single source instead of having duplicated data to maintain. Else I'll forget to increment the version number in manifest or const.py over time.
Loading/importing python files would result in IO too I suppose...the code needs to be loaded with IO as some time anyway...
So I guess I'd prefer to keep as is.

There is a big difference between reading a file and a static const.
The fact that you do it for all does not make the situation any better.
The suggested alternative provides the same data in a safe manner that does not hold up the startup, while still providing a single source to update (manifest.json), I use this method myself for HACS.

Ok thx for the info! I'll further look into it and update all.


win

Codecov Report

Merging #26 (089b46b) into main (8b14356) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #26   +/-   ##
=======================================
  Coverage   73.53%   73.53%
=======================================
  Files           1        1
  Lines         873      873
  Branches      239      239
=======================================
  Hits          642      642
  Misses         17       17
  Partials      214      214

Continue to review full report at Codecov.

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


Since version 0.9.0 it is not possible to reinstall a worker node:

Is your feature request related to a problem? Please describe.
Not sure, if this is a bug or a feature.
With version 0.8.0 it was possible to simply wipe the harddisk and restart the installation process. The worker did automatically reregister and would continue as before.
With 0.9.0-rc5 this is not possible anymore. k3s-service stops with the error message:
"Node password rejected, duplicate hostname or contents of '/etc/rancher/node/password' may not match server node-passwd entry, try enabling a unique node name with the --with-node-id flag"
Also when deleting the node does not help.
kubectl delete node worker03
I could use the --with-node-id flag, but then the nodes get a random suffix.
Describe the solution you'd like
Some mechanism to recreate a node with the same name and IP, for example in case of disaster recovery.
Describe alternatives you've considered
Alternative solutions could be some command to delete the node fully from the "memory of k3s".
Additional context
n/a

@amiga23 I believe that this is expected behavior with the newer version of k3s in scope. Does a wipe and re-install (retaining node name/id) work if you retain /etc/rancher and /var/lib/rancher and overlay those prior to starting k3s?

The node can be deleted from /var/lib/rancher/k3s/server/cred/node-passwd and then it is possible to register a fresh node with the same name.
This is sufficient for me.
Backing up /etc/rancher and/or /var/lib/docker does not make sense to me, as the advantage of k3os would be lost.


Fix typo, consistency

Correct 'Vet' to 'VNet'
Standardise casing of 'VNet'
Expand ER to Express Route as it hadn't been referenced previously

Hi @stuartleeks, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (stuartle). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

@stuartleeks : Thanks for your contribution to the Azure documentation! The author, @narayanannamalai, has been notified to review your proposed change.

#Hold-off , lot of typos are being corrected in a current PR that is under review

Hi @staurtleeks thanks for your contribution to improving the documentation, the original content has been modified and hence this change is no longer required.
Please close this PR.

#please-close

Closing as per request. New article is much improved. Have submitted a minor change from this PR that wasn't carried forwards: #7560


txt2video generation failure

when i try to do txt2video it only generates 1 frame then stops

same here. Nothing happening in console :(

There is some error happening but a part of the code that handles the exception does not print it. My bad. This will be fixed in the next update. Regarding the problem itself I'm pretty sure that it is the same as described here: https://github.com/volotat/SD-CN-Animation/issues/33#issuecomment-1537991750

Just to be sure. I checked default resolution of 768 x 512 and still nothing. Btw vid2vid works ok. Is it for sure the same error? Thx for reply.

UPDATE from me: checked out on clean auto1111 install and text2video works. Probably some crash with other 283424 extensions i have on old auto1111 instance.


Setup pypi publishing action test

This PR setup pypi publishing action.

I've requested pypi token from @John-Ragland. This action will need that token before it works.

I am going to merge this for now to test things out.


Consolidate changing variables to one config file

Should help future release diffs be much nicer!

r? @nikomatsakis
(rust_highfive has picked a reviewer for you, use r? to override)

r? @brson


[Linux] Couldn't setcap vrcompositor, steamvr will ask for permissions every single launch.

Hi, I have an issue with setcap (and probably with getcap). I already have get- and set-cap installed. (actually no, apt said that these two packages already been installed, but I had to manually move these to /bin folder from /sbin:
ln -s /usr/sbin/getcap /usr/bin/getcap
ln -s /usr/sbin/setcap /usr/bin/setcap

). Okay, on my main system (Debian 12 also) I have these two packages. But the Distrobox seems to not have it, and I have no clue how to install these packages.
distrobox last logs:
2 : Installed base packages and Steam. Opening steam. Please install SteamVR from it.
2 : Installation will continue when steamvr will be installed
2 : Next prompt for superuser access prevents annoying popup from steamvr that prevents steamvr from launching automatically.
2 : Couldn't setcap vrcompositor, steamvr will ask for permissions every single launch.
2 : Running steamvr once to generate startup files.

(NOTE: steamvr don't launching by this script, idk why)
full installer logs:
setup.log
when launching SteamVR from distrobox manually, I'll get this:

(clicked yes)

SteamVR's closing.

I couldn't get working ALVR on my main Debian system, so I referred to this "100% working method", but it kinda doesn't work..

NOTE: after clicking "yes" for "SteamVR requires superuser access to finish setup..." no promt for superuser password is made. It's because of missing *cap libraries, I'm already tested this on my main Debian system


⚒ Saved Structures

Save all neglected structures.

[ ] Crafting Benches
[ ] Camptents
[ ] Fireplaces

Structures table should have a type column that distinguishes structures and tells them apart.


Earphones volume

To revive the issue #5. If the volume of the audio in the speaker is set lower than default then the earphone volume may be a bit low to be heard.
There are projects like https://github.com/jpeddicord/speedofsound and https://github.com/brightify/autoamplifier that do clever things with adjusting sound to ambient levels.
I wonder would a solution involve a setting that sets the volume to a percentage of the media volume. This is the way https://github.com/yuttadhammo/BodhiTimer does it.
BTW, there are surprisingly few FOSS meditation timers out there and the former served me well for ages but it's not getting maintained. If you are thinking of customising the meditation timer then I will be glad to test them out .

Regarding testing: Please register as beta user to test new versions and leave non-public feedbacks. Thanks.

Choosing an audio stream and using volume control of this audio stream - introduced with issue #19 - should be enough handling for earphone IMHO.


Consider replacing uSpec with Rspec

When I started this project, Rspec was a lumbering monolith, and many times larger than config_module's codebase, so I used my testing micro-framework uSpec instead.
Rspec has seen some serious improvements over the intervening time and some of its features might come in handy.
Basically the question that needs to be answered is if there's a clear win in using Rspec over uSpec.

RSpec is nice because rails developers know it really well. minitest is nice because rubyists know it even if they've never used the library before.
RSpec seems a better direction if you want to make it easy for people to contribute.

@Spaceghost true, but uSpec is "just Ruby". The syntax isn't weird - because there's almost none.
Here's some example code (from config_module's specs):
spec 'modules extended with ConfigModule will load configurations' do
  ExampleConfig.foo == 'bar'
end

spec 'modules extended with ConfigModule have "namespace" methods' do
  ExampleConfig.methods.include? :namespace
end

Here's the output up on Tavis-CI:

I'm not against it. It looks pretty simple and clean and I get how I'd use it just from seeing a single test.

uSpec seems to be doing just fine right now.


warning: multiple different client offset_encodings detected for buffer, this is note supported yet

Neovim version
NVIM v0.9.0-dev-329+ged27f0e93
Operating system/version
wsl Ubuntu20.04
Terminal name/version
WezTerm
$TERM environment variable
xterm-256color
Branch info
main (Default/Latest)
Affected language servers
clangd
How to reproduce the issue
打开任意一个c文件
Actual behavior
一直弹出警告框，如下：

Expected behavior
能够正确修复，不再弹出类似警告消息。
Support info

Logs
没有在提示路径中找到log文件

Additional information
这是user/config.lua中clangd部分配置：
function config.lang_c_cpp()
	vim.api.nvim_command([[packadd nvim-lspconfig]])
	vim.api.nvim_command([[packadd lsp_signature.nvim]])
	return require("clangd_extensions").setup({
		server = {
			-- options to pass to nvim-lspconfig
			-- i.e. the arguments to require("lspconfig").clangd.setup({})
			capabilities = { offsetEncoding = "utf-8" },
			cmd = { "clangd" },
			root_pattern = {
				".clangd",
				".clang-tidy",
				".clang-format",
				"compile_commands.json",
				"compile_flags.txt",
				"configure.ac",
				".git",
			},
			single_file_support = true,
		},
		extensions = {
			-- defaults:
			-- Automatically set inlay hints (type hints)
			autoSetHints = true,
			-- These apply to the default ClangdSetInlayHints command
			inlay_hints = {
				-- Only show inlay hints for the current line
				only_current_line = false,
				-- Event which triggers a refersh of the inlay hints.
				-- You can make this "CursorMoved" or "CursorMoved,CursorMovedI" but
				-- not that this may cause  higher CPU usage.
				-- This option is only respected when only_current_line and
				-- autoSetHints both are true.
				only_current_line_autocmd = "CursorHold",
				-- whether to show parameter hints with the inlay hints or not
				show_parameter_hints = true,
				-- prefix for parameter hints
				parameter_hints_prefix = "<- ",
				-- prefix for all the other hints (type, chaining)
				other_hints_prefix = "=> ",
				-- whether to align to the length of the longest line in the file
				max_len_align = false,
				-- padding from the left if max_len_align is true
				max_len_align_padding = 1,
				-- whether to align to the extreme right or not
				right_align = false,
				-- padding from the right if right_align is true
				right_align_padding = 7,
				-- The color of the hints
				highlight = "Comment",
				-- The highlight group priority for extmark
				priority = 100,
			},
			ast = {
				--[[ These are unicode, should be available in any font
                role_icons = {
                    type = "🄣",
                    declaration = "🄓",
                    expression = "🄔",
                    statement = ";",
                    specifier = "🄢",
                    ["template argument"] = "🆃",
                },
                kind_icons = {
                    Compound = "🄲",
                    Recovery = "🅁",
                    TranslationUnit = "🅄",
                    PackExpansion = "🄿",
                    TemplateTypeParm = "🅃",
                    TemplateTemplateParm = "🅃",
                    TemplateParamObject = "🅃",
                },]]
				-- These require codicons (https://github.com/microsoft/vscode-codicons)
				role_icons = {
					type = "",
					declaration = "",
					expression = "",
					specifier = "",
					statement = "",
					["template argument"] = "",
				},

				kind_icons = {
					Compound = "",
					Recovery = "",
					TranslationUnit = "",
					PackExpansion = "",
					TemplateTypeParm = "",
					TemplateTemplateParm = "",
					TemplateParamObject = "",
				},

				highlights = {
					detail = "Comment",
				},
			},
			memory_usage = {
				border = "none",
			},
			symbol_info = {
				border = "none",
			},
		},
	})
end

这是user/plugins.lua
lang["p00f/clangd_extensions.nvim"] = {
	opt = true,
	ft = {
		"c",
		"c++",
		"cpp",
	},
	capabilities = { offsetEncoding = "utf-8" },
	cmd = { "clangd" },
	root_pattern = {
		".clangd",
		".clang-tidy",
		".clang-format",
		"compile_commands.json",
		"compile_flags.txt",
		"configure.ac",
		".git",
	},
	single_file_support = true,
	config = conf.lang_c_cpp,
}

There's already a clangd setting, did u disable it?

set capabilities = { offsetEncoding = "utf-16" } will be fine.
This have been set at here: https://github.com/ayamir/nvimdots/blob/main/lua/modules/completion/lsp.lua#L112

We have (and you probably should have) configured all language servers here. If you really intended to use this plugin, please delete these lines and move the options you need to:
require("clangd_extensions").setup({
    server = {
        -- options to pass to nvim-lspconfig
    },
    -- ... --
})

(otherwise all the options will be set again - this is currently not allowed. See clangd_extensions.nvim#configuration):
However, it is not recommended to use separate plugins to configure any native LSP. The built-in settings can already satisfy most of the features provided by this plugin, installing another version in parallel can cause all kinds of trouble. You might want to see the wiki.

We have (and you should have) configured all language servers here. If you really intended to use this plugin, please delete these lines and move the options you need to

所以只有在这个文件里没有相关的配置时我才需要新建一个user文件来配置对吗？

mostly ys.

那我是应该添加在completion文件夹中，还是lang文件夹，也许是我没理解透，这两个文件夹在wiki中的描述让我有些困惑

那我是应该添加在completion文件夹中，还是lang文件夹，也许是我没理解透，这两个文件夹在wiki中的描述让我有些困惑

It's fine no matter where u put it, both will work. Follow your heart.🙂

好的，感谢解答

那我是应该添加在completion文件夹中，还是lang文件夹，也许是我没理解透，这两个文件夹在wiki中的描述让我有些困惑

This is an enhancement only for clangd, so as described in wiki, it is recommended to put the settings for specific languages in the lang folder. But plugins can work normally wherever they are placed 👍 This is just a convention.

This is an enhancement only for clangd, so as described in wiki, it is recommended to put the settings for specific languages in the lang folder. But plugins can work normally wherever they are placed 👍 Just a convention.

Thank you for the detailed answer.


Display Cargo config in UI

Display the needed config piece for Cargo to use the current Kellnr instance prominently in the UI.

[ ] Copy with one click on a button
[ ] Depending on the enabled features (proxy, required auth) show different config

For example:
Copy this code to your .cargo/config.toml to use this Kellnr instance
[registries]
kellnr = {index = "sparse+https://kellnr.com/api/v1/crates", token = ""}

# IF PROXY IS ENABLED
[source.crates-io]
replace-with = "kellnr-cratesio-sparse"

[source.kellnr-cratesio-sparse]
registry = "sparse+https://kellnr.com/api/v1/cratesio/"

# IF REQUIRED AUTH IS ENABLED
# https://doc.rust-lang.org/nightly/cargo/reference/registry-authentication.html
[registry]
global-credential-providers = ["cargo:token"]

Decided not to implement this feature.


Updating Dockerfile to read go files from cmd/module

Updating Dockerfile to read go files from cmd/module

I'll take a look at it later! :)

I'll take a look at it later! :)


Cleanup on headers handling

While cleaning up a bit the code (I wanted to remove the delete as it may be not great for v8's optimisation, I had the chance to look again at all the tests and I found a couple of things that I'd like to change.

When doing the GET /component I think headers should be set as headers and not included in the response
When doing the POST / for the batch endpoint I think headers should not be set but included in the response. Still, I moved them from being in the response.headers to just headers (one level down). I think this allows us to have "response" to be exactly identical as a GET /component response.

I would be tempted to publish this as a bugfix instead of breaking change if you agree @ivan-dimitrov-skyscanner
Also, we may need to revisit your PR #347 as I think if we have all the headers in the POST perhaps we don't need that?
More details inline.

@matteofigus,
Regarding PR #347 - I think we'll still need it because even though we'll have all the headers in the POST, the majority of the HTTP proxies (CDN, etc.) which stay on the request-response path don't cache responses of POST requests. For this reason I think we still need a GET request when dealing with just single component.

Thanks, so let me merge this and then we'll review that one ;)


Add: Contributors to website index.md

-Added contributor section to website, based on https://github.com/vicheanath and contrib.rocks
Closes #70

@parkerstafford Looks great! Can you add to README.md as well?

Just added!


Allow user to export audio files to video

Like --export_as_audio converts video files to audio wav files,
--export_as_video should convert audio files to video ones.
Of course, the obvious question is what should the actual video be. The default may be simply a black screen with the audio, perhaps at a 1920x1080 resolution with 1 fps, but with the ability to change these somehow.

Closing since it's hart to figure what size the new generated video will be. May be a good idea for a subcommand though.


ESS Investigation functionality

Description

What does this PR introduce?

New functionality for ESS Investigations. It allows to create ESS Investigations via the Parts as Planned List and via the new ESS Investigations page. Furthermore it displays all existing ESS Investigations in a new page.

@bajaioana please do not push the code to the main branch. Get in contact with @ds-mwesener to define a branch on which to push the code


Replace KeyValuePair with Tuples

An example replacement of KVPs with Tuples as discussed in #340. It enables code that looks like this: -
let lines = sparkContext.TextFile filePath
let counts =
   lines.FlatMap(fun x -> x.Split ' ' :> _)
        .Map(fun w -> KeyValuePair(w, 1))
        .ReduceByKey(fun x y -> x + y)
        .Collect()
for count in counts do
   printfn "%s: %d" count.Key count.Value

to be replaced with this
let lines = sparkContext.TextFile filePath
let counts =
   lines.FlatMap(fun x -> x.Split ' ' :> _)
        .Map(fun w -> w, 1)
        .ReduceByKey(fun x y -> x + y)
        .Collect()
for (key, value) in counts do
   printfn "%s: %d" key value

as well as all the other usual things like fst and snd etc. etc.

Hi @isaacabraham, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.
TTYL, MSBOT;

Any thoughts on this PR?

[Current coverage][cc-pull] is 82.02%

Merging [#407][cc-pull] into [master][cc-base-branch] will decrease coverage by -<.01%

3 files in ...ark.CSharp/Streaming were modified. more

Misses -1
Partials -14
Hits -13

5 files in ...ft.Spark.CSharp/Core were modified. more

Misses +17
Partials -5
Hits -77

2 files (not in diff) in ...crosoft.Spark.CSharp were modified. more

Misses +1
Partials -1
Hits -7

9 files (not in diff) in ...oft.Spark.CSharp/Sql were modified. more

Misses -3
Partials -6
Hits -42

4 files (not in diff) in ...k.CSharp/Interop/Ipc were modified. more

Misses -3
Hits -33

1 files (not in diff) in ...Spark.CSharp/Interop were modified. more
9 files (not in diff) in ...ft.Spark.CSharp/Core were modified. more

Misses -3
Partials -4
Hits -30

1 files (not in diff) in ...crosoft.Spark.CSharp were modified. more

@@             master    #407   diff @@
=======================================
  Files            45      45
  Lines          4907    4646   -261
  Methods           0       0
  Branches        516     418    -98
=======================================
- Hits           4029    3811   -218
- Misses          637     643     +6
+ Partials        241     192    -49

Powered by Codecov. Last updated by [ada2e3e...a73b03f][cc-compare]
[cc-base-branch]: https://codecov.io/gh/Microsoft/Mobius/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/Microsoft/Mobius/compare/ada2e3ea7c29856fd3600799b28909cf7cdcdaf2...a73b03f94d0da665901a9889185ea9b25036405e
[cc-pull]: https://codecov.io/gh/Microsoft/Mobius/pull/407?src=pr

@isaacabraham - following samples (also used as functional tests) are failing in this PR (output of RunSamples.cmd copied below)
* SparkContextAccumulatorSample (category: default), duration=00:00:04.2144522
* RDDTreeReduceSample (category: default), duration=00:00:00.9291795
* RDDCountByValueSample (category: default), duration=00:00:02.5838769
* PairRDDCountByKeySample (category: default), duration=00:00:02.1073810
* DFForeachSample (category: default), duration=00:00:00.9013544
* DFForeachPartitionSample (category: default), duration=00:00:02.2719630

I see the following test output printed to console for PairRDDCountByKeySample. Looks like wrong type is used in the assert statement.
---- Error running sample PairRDDCountByKeySample (duration=00:00:02.1073810) -----
System.Reflection.TargetInvocationException: Exception has been thrown by the target of an invocation. ---> NUnit.Framework.AssertionException:   Expected: 2
  But was:  < <(a, 2)> >

   at NUnit.Framework.Assert.That[TActual](TActual actual, IResolveConstraint expression, String message, Object[] args)
   at Microsoft.Spark.CSharp.Samples.PairRDDSamples.PairRDDCountByKeySample() in c:\projects\sparkclr\csharp\Samples\Microsoft.Spark.CSharp\PairRDDSamples.cs:line 122
   --- End of inner exception stack trace ---
   at System.RuntimeMethodHandle.InvokeMethod(Object target, Object[] arguments, Signature sig, Boolean constructor)
   at System.Reflection.RuntimeMethodInfo.UnsafeInvokeInternal(Object obj, Object[] parameters, Object[] arguments)
   at System.Reflection.RuntimeMethodInfo.Invoke(Object obj, BindingFlags invokeAttr, Binder binder, Object[] parameters, CultureInfo culture)
   at Microsoft.Spark.CSharp.Samples.SamplesRunner.RunSample(MethodInfo sample) in c:\projects\sparkclr\csharp\Samples\Microsoft.Spark.CSharp\SamplesRunner.cs:line 79

Try running these examples that are failing using the instructions at https://github.com/Microsoft/Mobius/blob/master/notes/windows-instructions.md#running-in-local-mode-dev-environment and fix the failures.
Once all the tests are passing, I will run perf tests on this PR as discussed in https://github.com/Microsoft/Mobius/issues/340. Once that passes, this PR will be ready for merge as a part of upgrade to Spark 2.0 (since it changes public API).

@skaarthik Thanks for looking at this. I'm not in a position to fix this at the moment (the machine with Spark etc. installed isn't here) and won't be for at least a week or so - would you mind looking at this as it seems that you've already identified the issue? Thanks!

@isaacabraham - sure. I only looked at the failure in one test. Will take a look at other test failures & possibly push a fix when I get a chance.

I looked at other test failures - some are due to wrong types (need to be updated to reflect the changes in this PR) and some are due to mismatch in expected and actual values that need deeper investigation. I will do the investigation later.

Anyone interested in picking this up, resolve conflicts and fix failing tests?

I will try to - can't commit to any time at the moment - but please leave this PR open :-)

@isaacabraham - this PR will remain open. I will also try to find time to get this PR updated.


Create follow-up issues for 3.0 tasks that were split

A number of tasks for 3.0 were split into "the minimum that need to be done in 3.0" and "things that could be done in 3.x". Sometimes when closing the task after the 3.0 part had been done, we didn't check that a ticket had been created for the remaining work.
This task is to go back and check that for all 3.0 tasks were work has been left for 3.x, there's a github issues to track than work (generally in the 3.x EPIC), and that it correctly describes what's left to do (usually the original task will have a description that can be copy-pasted).

Done. I went over all 3.0 and 3.0 tasks to check if there was any remaining work to be tracked. Turned out most follow-up issue had already been created and I only have to created 2 of them (related to the CHECK_PARAMS option).


What are the bounds of the threshold parameter?

The docs have a default threshold of 15. Is there a minimum / maximum number, eg: between 0 to 100?
Also, what are some mappings to values? Eg: 30 requires a "vigorous shake", and 10 is "any slow rotation"

Whatever works for you - the default of 15 works for most mobile device sizes. You may need to adjust it for larger / heavier devices to be more vigorous.

I think the range of threshold should be written in the documentation @alexgibson


FIX [mqtt-tcp test] fix memleak of connmsg.

Codecov Report
Patch coverage has no change and project coverage change: +0.16 :tada:

Comparison is base (706d922) 51.13% compared to head (9554145) 51.29%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #491      +/-   ##
==========================================
+ Coverage   51.13%   51.29%   +0.16%
==========================================
  Files         120      120
  Lines       36258    36258
==========================================
+ Hits        18539    18599      +60
+ Misses      17719    17659      -60

see 19 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


HBASE-24334 Removed duplicated dependency declaration of metrics-api

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 19s
Docker mode activated.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 33s
the patch passed

+1 :green_heart:
compile
0m 19s
the patch passed

+1 :green_heart:
javac
0m 19s
the patch passed

+1 :green_heart:
shadedjars
5m 54s
patch has no errors when building our shaded downstream artifacts.

-0 :warning:
javadoc
0m 18s
hbase-hadoop-compat in the patch failed.

_ Other Tests _

+1 :green_heart:
unit
0m 41s
hbase-hadoop-compat in the patch passed.

25m  3s

Subsystem
Report/Notes

Docker
Client=19.03.8 Server=19.03.8 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1665/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1665

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 3f6f55cce1cc 4.15.0-74-generic #84-Ubuntu SMP Thu Dec 19 08:06:28 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / c1cb22f0b3

Default Java
2020-01-14

javadoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1665/1/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-hadoop-compat.txt

javadoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1665/1/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-hadoop-compat.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1665/1/testReport/

Max. process+thread count
264 (vs. ulimit of 12500)

modules
C: hbase-hadoop-compat U: hbase-hadoop-compat

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1665/1/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.


Add support for analyzers to JSON API

CQL analyzers are now available in datastax/cassandra fork and Astra Vector DB (docs).
Let's define syntax for adding analyzers to find and similar operations supporting filtering on JSON API

Analyzers are what is called "text search"?
In addition to syntax, need to figure out how they'd work for data model: since we have no schema to work on ("JSON" model) for anything but couple of special fields (_id, $vector), I think we'd require another special field.
So maybe another issue for figuring out how this would work from data model perspective?


transitionTo encoding the url

Hello ,
When using react router's transitionTo with query params, I send the params as {runs:"1|2|3"}
"|" being my delimiter. The transitionTo method encodes the "|" character and the resultant url looks like
subruns=1%7C%7C2%7C3
While functionally I have no issues , Is there any way to let the router not to encode the params so that the url is readable?
Thanks,

Supply your own query string parsing: https://github.com/ReactTraining/react-router/blob/master/upgrade-guides/v2.0.0.md#custom-query-string-parsing


Python 3.8 really required?

Hi, just a question: Is python 3.8 really required? Unfortunately the 40.4 version did nothing any more since the API change two days ago, not even spitting out debug messages with -v.
But upgrading on an old buster Raspbian just installs 3.7 - will do the bullseye upgrade, just wanted to ask if the 42.x path could still be compatible with python 3.7 meanwhile, as splitting up the PV controlled charging between 2 EVs only works with correct SoC. (Just buying time here, if there is a reason, just close this)

Thanks, you are completely right, it is not urgent and python upgrade would be possible, without dist upgrade. Thanks for the explanation!

Hi @foscoj ,
I just updated to python 3.8 on Raspbian Buster like this today:
sudo apt-get install -y build-essential tk-dev libncurses5-dev libncursesw5-dev libreadline6-dev libdb5.3-dev libgdbm-dev libsqlite3-dev libssl-dev libbz2-dev libexpat1-dev liblzma-dev zlib1g-dev libffi-dev tar wget vim
wget https://www.python.org/ftp/python/3.8.16/Python-3.8.16.tgz
sudo tar xzvf Python-3.8.16.tgz
cd Python-3.8.16
sudo ./configure --enable-optimizations
sudo make -j 4
sudo make altinstall
echo "alias python3=/usr/local/bin/python3.8" >> ~/.bashrc
echo "alias pip3=/usr/local/bin/pip3.8" >> ~/.bashrc

After this you can use python 3.8 with your normal user. If you want to use python 3.8 systemwide you have to replace the python3 and pip3 files with symlinks to the fresh installed 3.8 files. After a pip3 install weconnect-mqtt I was back online with my EV.
Yes I know, you won't get security updates if you leave your distro's package manager. But this solution will give you some time to upgrade to bullseye ;-)

one more comment: When you have to update you can check if you cannot directly go to e.g. python 3.11 this makes you more future proof and you also profit from performance improvements.

Thanks again, it was time for bullseye anyway, backup is running ;)


Add license

Adds an MIT license, as per Tarides's policy.
If any contributors feel that they should be included in the copy right notice explicitly,
please add yourself, in this PR or in a followup!

Hello, @hannesm! Would you consent to to us adding an MIT license to this repository, to which you have made contributed?

sure, please go for it.


Pull upstream master

Pull upstream master, to use token auth instead of username+password.
(Pairing with @lafave)

+1

:shaved_ice:

Let's make sure when we upgrade to this we keep the old username/password env vars out there so we can easily rollback if things break.


PS-3705 CI challenge

Basic summary of what is changed here.

extracted all the redshift tests to it's own folder
pulled out all workspace tests to their own folder
all classes should be using their class name for buckets/file tags

Tady by to Odin snesl, protože to celkem dává smysl


How to build on linux without having protobuf installed

How can I build this from source without having to install protobuf first?
In my current project, I build grpc from source, including also its third party projects including protobuf.
And I do not want to install protobuf on my build system for various reasons.
So, how may I override the existing build rules in the qmake project file for linux, so that it will use the protobuf build from grpc?

I temporarily fixed this by applying a patch to protoc-gen-doc.pro
diff --git a/protoc-gen-doc.pro b/protoc-gen-doc.pro
index 38a35ce..ab40d30 100644
--- a/protoc-gen-doc.pro
+++ b/protoc-gen-doc.pro
@@ -18,9 +18,10 @@ lessThan(QT_MAJOR_VERSION, 5):error(This program requires Qt 5.x.)
 linux {
     # Use pkg-config to find libprotobuf.
     CONFIG += link_pkgconfig
-    PKGCONFIG = protobuf
+    #PKGCONFIG = protobuf

-    LIBS += -lprotoc # Has no .pc, so add manually.
+    INCLUDEPATH += "${ROOT_DIR}/externals/grpc/third_party/protobuf/src/"
+    LIBS += -lprotoc -lprotobuf -L"${ROOT_DIR}/externals/grpc/third_party/protobuf/src/.libs/"
 }

 msvc|mac {

Closing as it works for me.


Update based on latest llama.cpp sources - better llama 2 support

update README for cuda, metal and opencl support

It looks like the main branch of the repository you cloned was a little old. These llama.cpp changes have already been reflected and released. The development and release speed of llama.cpp is very fast. So, to save my daily life, I have decided to update the codes about llama.cpp every Friday. In addition, regarding the optimization options, I do not actively disclose them because I do not have a sufficient verification environment.
Thank you for your understand.


What about DS214play x86 ?

it's not realy an issues, but DS214play is x86 with the Intel Atom CE5335

also docker is not available on this NAS

It seems to use the Evansport package architecture, which is untested but is likely working. You could try building it yourself:
git clone https://github.com/runfalk/synology-wireguard.git
cd synology-wireguard/
sudo docker build -t synobuild .
sudo docker run --rm --privileged --env PACKAGE_ARCH=evansport --env DSM_VER=6.2 -v $(pwd)/artifacts:/result_spk synobuild

You don't need docker on the NAS, just on the computer you use to build the package.

then I created artifacts directory myself and

The minimum required kernel version for WireGuard is 3.10. The kernel version of your NAS is 3.2.40, so unfortunately, WireGuard will not work.

Yeah, the kernel version is too old. The only way for it to be supported would be to use a userspace implementation (maybe https://github.com/cloudflare/boringtun or https://github.com/WireGuard/wireguard-rs could work?) or backport WireGuard to this version.


Added TAG_SEPARATOR_PATTERN option for splitting tags

Summary
This PR adds the TAG_SEPARATOR_PATTERN option.  This option is a regex character class that will be used when splitting the tag string.
It defaults to [,] which will split tags on commas.  Splitting tags is currently done on commas so this will be backwards compatible and should not introduce breaking changes.
Some Examples:

Splitting on spaces: TAG_SEPARATOR_PATTERN="[ ]"
Splitting on spaces and commas: TAG_SEPARATOR_PATTERN="[ ,]"
Splitting on spaces, commas, and semicolons: TAG_SEPARATOR_PATTERN="[ ,;]"

All previously passing tests still pass.
I can give the Wiki text so that the Wiki page for the configuration can be updated with this new option.

Related issues
#725

Changes these areas

[ ] Bugfixes
[ ] Feature behavior
[ ] Command line interface
[x] Configuration options
[ ] Internal architecture
[ ] Snapshot data layout on disk

This looks awesome, thanks for working on it!
One small note: If it's a REGEX we should probably change the name from TAG_SEPARATORS which implies list (usually a CSV/JSON string in configs), to something like TAG_SEPARATOR_PATTERN which implies a REGEX.

Understood.  I have updated the code to rename TAG_SEPARATORS to TAG_SEPARATOR_PATTERN.

Wiki does not appear to have been updated with instructions on how to use this. @hannah98 or @pirate ?

I'm also having a bit of trouble with implementing this in a docker-compose use case. Should I be able to just add TAG_SEPARATOR_PATTERN=r'[ ,;]' to the environment variables in the docker-compose.yml?
I'm having no luck if that's the case.

Oops You're right I haven't updated it yet, the configuration page is a bit behind. It used to be publicly editable but some people abused it by adding virus links all over, so I had to lock it down. Unfortunately I don't know if there's a way to accept PRs for the wiki?
If anyone wants to contribute to wiki, I can just copy paste suggestions in from comments here and commit them myself.


Fix test adapter pipeline workflow

Description
This PR fixes manual test adapter workflow
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

@Revolyssup feedback? @asubedy @hershd23

@AugustasV Merge conflilcts...

@AugustasV Let's discuss this on the Meshery Dev call. Please add this as an agenda item in the meeting minutes if you would. :)

@AugustasV to sign-off on that/those commits, there are a couple git commands to run --> https://github.com/meshery/meshery/pull/6365/checks?check_run_id=9314658641

#6498 for PR merge


Add SimBrief Rate Limiting

This code limits the number of requests to the route to 10 per minute. Once this limit is exceeded, requests are blocked, but the block only lasts for one or two minutes, so users are not locked out for several hours.
Additionally, the SimBrief code (frontend) has been updated to avoid sending more than 5 requests, with a delay of 1500ms between each request. This allows us to smooth out the requests we send afterward.
Example below:
You can see that I triggered the rate limiter, received 429 errors, then got the popup notifying me of a problem. If I regenerate about a minute later, I get a 200 status, and the generation works.

Thanks!!


README

#README
##Story
As a boot camp student,
I want the prework notes to be structured on a webpage,
So that I can easily find and read the information.
#Acceptance
GIVEN the prework project repository,
WHEN I view the README for the repository,
THEN I can see a professional README that provides information on the project and how to use it

README file structure
Overview
Motivation
Features
Technologies
Future Improvements


Add Chainkorea remote node

Node used and maintained by Chainkorea (https://github.com/chainkorea)

HTTPS & Load Balancing through AWS (https://github.com/chainkorea/docker-etc-lb)
Blockchain up to date.
Response time ~150-350ms
KR servers
Live stats page on https://classicstats.net
Already used by ClassicEtherWallet

 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.

Next version of wallet will have ability to add custom remote endpoint - https://github.com/ETCDEVTeam/emerald-wallet/issues/314


Hydrodynamic added mass

Desired behavior
Gazebo's Hydrodynamics plugin can be used to apply forces due to added mass (i.e. the forces required to dislocate the fluid around a body moving underwater). However, using a plugin to apply such forces causes the simulation to be less stable, because it's reacting to the acceleration results at the end of a time step. Ideally, the added mass effect would be accounted for as part of the body's 6x6 inertia matrix.
Alternatives considered

Keep using the approach on the Hydrodynamics plugin.
Implementation suggestion

DART's Inertia class can be initialized with any arbitrary 6x6 spatial inertia tensor. So we could expose that to Gazebo users by:

[x] Extending the math::Inertial class to support all the terms, or creating a new class that wraps the current Inertial. It would be interesting to hold the mass matrix separately from the current mass matrix so they can be inspected and manipulated separately. https://github.com/gazebosim/gz-math/pull/459
[ ] Extending the SDF spec to expose the 21 unique coefficients in the 6x6 added mass matrix. https://github.com/gazebosim/sdformat/pull/1077
[ ] Making sure all the inertial data is contained in a Sim component like Inertial https://github.com/gazebosim/gz-sim/pull/1592
[ ] Extending the Gazebo Physics API to load these values and populate DART's spatial tensor and populating that through the Sim Physics plugin. https://github.com/gazebosim/gz-physics/pull/384

Other nice-to-haves:

[ ] Displaying the inertial component's data in the GUI's component inspector so it's easier for users to debug (part of https://github.com/ignitionrobotics/ign-gazebo/issues/158) https://github.com/gazebosim/gz-sim/pull/1592

Additional context

This would be immediately useful to maritime projects, such as buoy_sim and LRAUV.
This feature has been long-wanted by users, see:

this question from 2014: SDF: Specifying 6x6 inertia (mass) matrix
this question from 2016: Hydrodynamics effect - added mass

TODO in uuv_simulator

SDF proposal: https://github.com/gazebosim/sdf_tutorials/pull/76

Very interested in making Gazebo simulate the added mass better. Any extra links I can read up that is not mention here?


Cookies approval won't go away

What is the problem?
Trying to follow the CDK workshop here https://cdkworkshop.com/15-prerequisites.html and no matter what preferences I set for the cookies the approval banner wont go away and comes back every time I try and turn the page. It covers half the screen and makes the workshop unusable.
Reproduction Steps
Visit the website. Select cookie preferences so banner goes away. Turn the page in the workshop. See the cookie banner come back.
Verbose Log
No response
CDK CLI Version
No response
Section
No response
Browser
Chrome
Language
No response
Other information
No response

This same bug is hitting me as well.  Every page I go to in the workshop requires clicking 'Accept'  It's annoying.

What internet browsers are you using? I'm not able to reproduce this on a couple different browsers.

Chrome
Version 102.0.5005.115 (Official Build) (64-bit)

Do you have any sort of settings on regarding cookies? I can't even get the banner to appear on multiple devices and fresh browser installs. Mostly have been trying on Chrome around the same version as you

occurs for me on:
Chrome Version 104.0.5112.81 (Official Build) (64-bit) (and another version on work laptop)
Edge Version 104.0.1293.54 (Official build) (64-bit)
doesn't occur for me on:
Internet Explorer Version 21H2 (OS Build 19044.1889) (the cookie notice doesn't show up at all)
then it gets interesting, on Opera 89.0.4447.83 it occurs if VPN is in asia or europe, but in the americas I don't see cookie notice at all

I wasn't able to reproduce this originally, but we've just made a change to our website and have specified that we don't have cookies. Hopefully this will fix things
I was never able to reproduce so I can't confirm if this is going to remove the approvals you guys are seeing, so I'm going to close this assuming the changes we've made have worked. Changes should be up now


T284598 gutenberg search format

Phabricator Ticket: https://phabricator.wikimedia.org/T284598
The prefix search feature is implemented, the previous temp button of Add Remove are removed.

[x] Mobile
[x] prefix search
[x] fulltext search
[x] RTL
[x] Keyboard
[x] UI final tune

LTR
RTL
Fulltext Search

When you clear the search box with backspace it keeps the previous search results. It should probably go back to the initial (blank) state.

Thanks for the feedback, all should be updated now.

When you clear the search box with backspace it keeps the previous search results. It should probably go back to the initial (blank) state.

Thanks for the feedback, all should be updated now.

Doesn't work for me. The only way I can clear the result is by typing a space character.

We should be able to do the following:

type in "Paris"
type ENTER without selecting any entry from the search results
since "paris" is a valid search result, it should just work, now it errors out

This is available now through the ENTER key,
the only thing missing clarification is the behaviour of how the typing/keyboard/mouse trigger the hover style/selection, should it behave the same as wikipedia search feature?

the only thing missing clarification is the behaviour of how the typing/keyboard/mouse trigger the hover style/selection, should it behave the same as wikipedia search feature?

I think it works well as it is. I'll merge it so it can move forward to QA and design review.


Question: How to determine encoding of the file

Hi there, again thanks for making this since it saves tons of time.
Could you point me in the code or explain how does the importer determine what type of encoding the file is in when importing. I need to somehow extract this information and not sure how to do that. Maybe you can give me a hint where to look. not a bug more like request for information. And is there actually an automatic encoding determination or am i misinterpreting things?
   ```

guard let csv = CSVImporter<[String: String]>(url: fileURL) else {
return
}
    csv.startImportingRecords(structure: { (headerValues) -> Void in
        print(headerValues)

    }) {$0}.onFinish {importedRecord in
        print(importedRecord)

    }

I think what you're looking for is this line. This means, when creating a CSVImporter object you can pass a parameter names encoding with your files encoding. By default it's set to .utf8.

I see, so no way to determine encoding automatically then. Tough :( there is a method on NSString that computes that from an NSData object. Will try to look into that then on my own. Thanks for having a look.

As you can see here we already have logic in place which will automatically determine the type of line ending of the file when .unknown is specified by the user. I'm not against using the exact same strategy for encoding as well. This could be done e.g. by making encoding an optional parameter on the init method, and if it is nil we could use the FileSource to determine the encoding.
Would you be up adding this feature youself and sending a PR with tests and docs updated? That would be really awesome. I'm reopening this issue and renaming it to describe this feature.

I can make a PR, probably in next few days, I have already found a solution to this by the way and made a simple String extension that returns the encoding to me in String.Encoding format.
The only other issue and a bit off topic here is the delimeters (can be ; as well sometimes) and if one can process a string from memory as a CSV file. Because the NSString method i am referring to not only guesses the encoding but also returns the string to you which would potentially need to be handled by the importer on the fly rather than from a file.

Sounds good. Note that one of the advanatages of CSVImporter is that it's able to read big files faster and more safely since it doesn't read the entire file at once, which your solution probably does. So that's another plus on implementing this in CSVImporter.
I don't really understand your other problems though. You probably would need to post some code so I can understand. Note though, that if it's a different problem than this one, it's probably better you open another issue for each problem.

how does csvimporter handle garbage?  i have a specific data structure but it can be corrupted or fields missiing or added so i need to add some regex.

CSVImporter generally expects a valid CSV file according to RFC 4180 which specifies:

Within the header and each record, there may be one or more
fields, separated by commas.  Each line should contain the same
number of fields throughout the file.  Spaces are considered part
of a field and should not be ignored.  The last field in the
record must not be followed by a comma.

When a line for example doesn't have the same number of fields, then – at the moment – the entire line is simply ignored. That's not required by the RFC though (that's why it's a "should" not a "must"), so we could implement multiple different fallback strategies and let the user choose between them.
Can you give examples of lines and how they are "corrupted"? Depending on the case, I'm perfectly okay with a little more accommodating behavior, so long as it doesn't conflict with the RFC.


Fixed NameError coming from a typo.

I tried running the current main, however there was a NameError when saving the checkpoint. No checkpoint could be saved.
I renamed the variable from the non-existent hpt to hps name and everything works again.

Thank you very much, solve my problem, I couldn't train


[EXID] Added custom metadata to leave-the-organization (ADO-2105783)

Added "ms.custom: engagement-fy23” metadata to the article.

Learn Build status updates of commit ea49a7d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/active-directory/external-identities/leave-the-organization.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

#sign-off


S3 support

Hello again! You've mentioned in a readme, that were planning at some point to add other disk drivers support, would +1 an S3 please. We have common "file uploads" going to/serving from s3, but images that require Glide cropping and so on are still on local disk, ideally would like to move all the uploads to Amazon. Not sure though about the best practices regarding cache for example. Have seen a few stackoverflow questions/discussions on having originals on s3, cache on local or disabling cache completely and just serving from s3...

All take a look at it soon. If you need S3 uploads now, you could take a look at our medialibrary-package

Thanks heaps, looking at medialibrary!

+1 for S3. Would be a fantastic addition to this package.

Due to time constraints, S3 support isn't coming any time soon. If you really need this, feel free to submit a PR.

+1 for S3 read/write support.

@tillkruss Take a look at our medialibrary package which can copy Glide manipulated images to S3.


chore: fix function names

PR checklist

[ ] Tests written/updated
[ ] Changelog entry added in .changelog (we use unclog to manage our changelog)
[ ] Updated relevant documentation (docs/ or spec/) and code comments
[ ] Title follows the Conventional Commits spec

Thanks @MarkDaveny ❤


RuntimeError: abort(SyntaxError: JSON5: invalid character 'w' at 1:1)

I ran $ npx @squoosh/cli --webp wordpress-lighthouse.png wordpress-sources.png lighthouse-score-improve.png
I got:
RuntimeError: abort(SyntaxError: JSON5: invalid character 'w' at 1:1). Build with -s ASSERTIONS=1 for more info.
    at process.w (/home/codespace/.npm/_npx/1912/lib/node_modules/@squoosh/cli/build/index.js:7:376640)
    at process.emit (events.js:315:20)
    at processEmit [as emit] (/home/codespace/.npm/_npx/1912/lib/node_modules/@squoosh/cli/build/index.js:7:79620)
    at processPromiseRejections (internal/process/promises.js:245:33)
    at processTicksAndRejections (internal/process/task_queues.js:94:32)

--webp takes a JSON parameter to configure the encoder, that’s why it’s trying to decode the first file name using JSON5. If you use --webp '{}', it should work.
For now I’d recommend using the web app to generate the CLI invocation, until we get around to extend the documentation (as outlined in #900). Closing this in favor of #900


The wallet doesn't open if it was connected to  not working node

If you specify a remote node in the settings of the wallet, a not working node and close the wallet. Then when you re-open, the wallet will not open after entering the password
Step

go to settings screen
open remote node tab
enter us-nodes.testnet.beam.mw:8100 in address field
click on apply changes button
close wallet
open wallet > enter password > click open your wallet button

Expected: wallet is open

checked

checked


debug_toolbar: sqlparse 0.2.0 api changed

Reproduce
http://<IP_ADDRESS>:8000/en-us/accounts/login/?next=/en-us/dashboard/
Enter any username and password -> Login
Error message
builtins.TypeError
TypeError: process() missing 1 required positional argument: 'stream'
Solution
https://github.com/jazzband/django-debug-toolbar/pull/854/files
Regards

So is the real solution upgrading Django-Debug-Toolbar? What version would it work?

Upgrading to 1.5 solves the problem. Upgrading to 1.6, however, needs some tweaking.


Update standard IDistributedCache implementation.

I was unable to find anything about IDistributedInMemoryCache anywhere, but MS provides MemoryDistributedCache as part of the Microsoft.Extensions.Caching.Memory package.

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

Yep, thanks.


Add and test methods to get current position of an element on the diagram

DiagramElement has x and y members but I'm not sure these are actually correctly updated, certainly not if the diagram is interactive.
Also how is the thickness of the lines/borders affecting this?

It seems I already thought of these. x, y are properties and move(x, y) is defined on DiagramElement. There are also tests but they can be improved.


focusedBeforeDialog does not work in Chrome

Focus is not properly returning to the element that captured focus before the dialog was created.

I can reproduce on Chrome, indeed.

Fixed in 5.0.2, thank you for reporting!
https://github.com/edenspiekermann/a11y-dialog/releases/tag/5.0.2


Fix switching to the target branch

This is to fix a git failure in dhat test

@adonagy merging it without your approval.


Please add a mouseover background color, a mouseover foreground color, and a mouseover border color for the icon

Please add a mouseover background color, a mouseover foreground color, and a mouseover border color for the icon

"workbench.colorCustomizations": {
      "activityBarTop.iconHoverForeground": "#0b1691",
      "activityBarTop.iconHoverBorder": "#ff0000",
      "activityBarTop.iconHoverBackground": "#ff0000",
  },

vscode 1.86
Windown 10

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


"[Fixes #4991] Misleading MissingMethodException message for incorrec…

…t routes"
Related: https://github.com/aspnet/Routing/pull/345
@rynowak Let me know if this is ok and I can add tests

:watch:

Needs tests

:up: :date:

:shipit:

a4ec3bb24a7af9ad30e8ef526b2528248e1ed840


Local Development

Hey long time lurker here, I'm just now setting things up for Lamby and getting familiar. However, I don't see any clear instructions on how I should be using Lamby in a local environment. It's a bit unclear to me if the app container service has any entrypoint that I can call via HTTP?
I can just run the rails server and go about my business but I'd be curious if there is any proposed approach here. Is there a proxy server or something that we're responsible for setting up? (eg. like localstack with a gateway lambda)
I'm a bit unfamiliar with Codespaces. Not really interested in inheriting any costs related for that service either.
I couldn't quite pinpoint where the config/environment.Lamby.cmd script is and didn't see any port expose method calls in the Dockerfile.

I guess its really up to you. It seems @ metaskills has a fondness of codespaces and using vscode .devcontainer .
For other languages I use .devcontainer but for rails I normally just locally install things.
to answer your other question
Lamby.cmd method is designed to handle AWS Lambda events by invoking the handler method .
https://github.com/rails-lambda/lamby/blob/master/lib/lamby.rb#L24

Answer might be more along the lines of this is how the SAM (Serverless Application Model) works. As it builds the final docker image, the Entrypoint & Command are done for you as part of the sam build process. The devcontainers are a CI/CD convention I explored at one time. It is neat... and benefits for development using the same image as the build system. But at the end of the day you could take any of the CI/CD conventions in the cookiecutter and replace them with your own linux env that has SAM installed, etc. Hope that helps?

Hey, thanks guys. I ended up using dev containers locally, however I did run into some issues with query params support (array of objects does not work the same locally via standard Rails server as in the AWS environment). I never got the chance to investigate if that was a limitation of AWS Lambda or just something that wasn't handed in the Lambda event handler mapping code of Lamby.
I thought maybe I would mess with serverless to see if I could get a closer 1:1 of live environments to local someday when I get the chance.


create kapitan binary (using pyinstaller or similar)

This seems interesting to me. I would like to work on this project as part of gsoc. It would be helpful if you throw some more insight into this project.

Ramaro has a test branch with some initial implementation of this https://github.com/ramaro/kapitan/tree/cython_build
The goal is to get automated binaries of kapitan that can be easily deployed anywhere.
Minimal maintenance is desired. All build artifacts should be auto-tested and released on github as well.

@uberspot Thanks , I will look into it and report my progress.

Just curious because I'm also interested in this project: do we intend to Cythonize the code before converting it into binary/one-folder using pyinstaller or something similar?

That's the idea.


Duplicate background entry

Duplicate background color entry in .loader-ui-grabber

UPDATE 21 June 2023:
Probably a mistake

@steve9164 checking if the developer has signed the CLA

CLA has been signed.


Update assets for ACS

ACS currently does not use the islet's assets. Thus, it would be common for users to face with build errors with the following command: ./scripts/fvp-cca --normal-world=acs --rmm=islet. For example, cmake should be updated to a version over 3.19 and srec_cat should be installed.

I actually tried it before.. it was hard to use cmake in assets. :(

I actually tried it before.. it was hard to use cmake in assets. :(

Thank you for the information! Maybe we can leave it as it is until more urgent tasks are done.


How to set/get null values?

How to clear date picker and get null or empty value from it?

You can use removeDate property of DatePicker component.
I added a sample of removeDate usage here.


chore: imporve adjustments of ancestor

ancestorProps is an array there, so it's always not equal undefiend. There should validate its length

Please see https://github.com/lit/lit/blob/main/CONTRIBUTING.md#pull-requests
We really need more of a description of what this change is for. What package? Is there an associated issue? We also need a test.

Please see https://github.com/lit/lit/blob/main/CONTRIBUTING.md#pull-requests
We really need more of a description of what this change is for. What package? Is there an associated issue? We also need a test.

This is just a code optimization, it has not caused any actual issues, and there is no need to add additional test cases. You only need to look at the code changes in these few lines to understand the meaning of this modification: diff


Preloading / optimistic loading

@aleffert  This is not an active pull request as of now...
I am thinking of using this approach to preload the uncached ViewControllers optimistically. Could you skim by it and give me your thoughts on this?
The basic idea is to have a previousPreloadedViewController and a nextPreloadedViewController in case it is not present in the cache. Thereafter, replace the siblingAtOffset with the relevant preloadedViewController

Instead of explicitly saving the next and previous (which is more instance variable state to maintain), a better way might be to wrap up controllerForBlock and instead of having it load one view controller, have it load three (current/before/after), but just return current. This is a much simpler piece of state than the explicit next/prev since it doesn't have anything to do with the current location beyond knowing what to load in the first place.
Relatedly, I'm making a major change to how we keep track of where we are in the list, which should land today that will make it easier to get the next and previous if they exist.

Closing this, using a different approach


Fix stream wrapper (tweaked #193)

Context
Tweaked version of #193 to pass all tests and Travis builds
What has been done

Just syntax tweaks on top of #193

Notes
See #193 for other details

What should we do to get this code in master and a new tag ?

Hi there - is there any plans if this fix would go into master?


Fix P4Dummy Backend Failure with External Input Program

Fixes #1
Let me know if this is ok.

How are you invoking the program? This should not be necessary.

Also what is the failure you are seeing?

How are you invoking the program? This should not be necessary.

I'm invoking it as ./build/p4dummy example.p4, relative to the p4c dir. This doesn't generate any error, but exits by printing the usage.
Apologies if i've missed something obvious here(?)

Thanks for finding this issue:
This patch should fix things, please test it and see whether it fixes your issue:
diff --git a/dummy.cpp b/dummy.cpp
index 722875a..4507902 100644
--- a/dummy.cpp
+++ b/dummy.cpp
@@ -132,18 +133,16 @@ V1Switch(p(), vrfy(), ingress(), egress(), update(), deparser()) main;)";
 int main(int argc, char *const argv[]) {
     AutoCompileContext autoP4DummyContext(new P4Dummy::P4DummyContext);
     auto &options = P4Dummy::P4DummyContext::get().options();
-    options.langVersion = CompilerOptions::FrontendVersion::P4_16;

     if (options.process(argc, argv) == nullptr) {
         return EXIT_FAILURE;
     }

-    const IR::P4Program *program = nullptr;
-    if (options.file == nullptr && !options.useFixed) {
-        options.usage();
-        return EXIT_FAILURE;
+    if (!options.useFixed) {
+        options.setInputFile();
     }

+    const IR::P4Program *program = nullptr;
     if (options.useFixed) {
         program = P4Dummy::parseDummyP4(options);
     } else {

see whether it fixes your issue

Thanks, this works.
I've updated the PR with the new patch.


Create intracitygeeks.txt

@ribex Could you please provide us with an URL of a page on the official website where an IT-related long-term (>1 year) course is offered by the school. Thank you.

@ribex You might miss my previous comment. Please reply.