Submit expense by email

Related to #1860
As the MVP, we want to focus on someone that takes a picture of a receipt and wants to submit it as an expense.
Today, they have to go to opencollective.com/:collectiveSlug/expenses/new, upload the photo and enter the details.
Members of the Open Collective Pro Tier should be able to send an email to expenses@:collectiveSlug.opencollective.com. This would create a PENDING expense with the photo in the email as the attachment.
We shouldn't try to do any OCR but the user should be able to specify the total amount in the subject or body.
It's beyond the scope of this issue, but if this can help designing the architecture, in a future version we would love to be able to forward a receipt received by email (without attachment).
That means potentially developing a template architecture so that people can improve the code by adding support for different service providers. E.g. here is a way to detect that this is an email receipt from Lyft and this is how you can extract the amount.
People could forward or directly add the expenses@:collectiveSlug.opencollective.com email address as the billing email address so that they don't even have to forward anything.

These specs are good, having adapters based on the sender (with a smart default) is a very good idea to make this feature extensible.
Having the PENDING status and requiring a manual validation is very important too because the FROM address in emails can be faked.
Things that are not covered here:

Do we prefill the payout method? How?

There's a possible answer in https://github.com/opencollective/opencollective/issues/1830#issuecomment-474579881. In short, we add the ability to add a set of rules in /collective/expenses/automations like if sender is @.com, use this payout method and set the description to ______.
If no automation configured, set the payout field to null or manual.

How to deal with emails with multiple attached documents

We can have a smart default here that loops through the attached files and takes the first one with invoice in it, falling back on the first pdf.
If it doesn't work for a given provider then we add a custom adapter.

A first version could automatically create a DRAFT expense and send back a link that people can use to update it manually and confirm. Ideally, there could be a list somewhere of all the DRAFT expenses so that you can easily go through them all.
Just that would already be a great feature. I could forward by email or configure the billing email to be expenses@:collectiveSlug.opencollective.com and once a week/month I just go through all the DRAFT expenses and confirm them. Already a big time saver on its own.
From there we could indeed think about improving the experience by defining default values based on recipes (or automation rules).
E.g.

if sender is<EMAIL_ADDRESS>set tags to web service, payout method to paypal, etc.
if subject contains xxxxxx

If there are more than one attachment, I'd just pick the PDF if any, or the first image (with a threshold of minimum 100KB to avoid picking up inline images from signatures for example).
If there is no attachment, ideally we would want to take a screenshot of the email and use that as image. But that could be for later.

@xdamman What's the next step? Should we create a separate opencollective-email-automations service?

Requested by @ninavizz in https://github.com/opencollective/opencollective/issues/4609#issue-984256814:
User story
As a paid contributor to an OpenCollective project, I would like a workflow for invoicing that aligns with the rest of my customers, so that invoicing my OC customers can take less time and be less annoying.
Currently, most of my customers have a specific email address for their billing system's intake, that they send to their vendors plug-in to their invoicing software. Tagged fields in the email body can also map to filled-out form fields on today's form.
TL;DR, it's a pain for me as a contributor to have to effectively do two separate workflows, to file an invoice for my one client who's an OpenCollective project. First step, on my invoicing software; and second, uploading it to my customer's OpenCollective profile on the website.
Nope, do not need any fancy callback features to tell my invoicing software (Harvest) you've paid me or anything.
Best solution for this problem
A simple email address that projects can send to contributors, for us to plug-into our invoicing software as a "Send to:" customer email address. Bill.com does this, and its terrific.
MVP
What will be built as an MVP? (Please refer to our wiki for what's acceptable as an MVP.)
I am a freelancer.
My customer sends me an email address.
I plug that email address into my invoicing software so that when I "send" my invoice, it automagically uploads everything to the Open Collective form and I no longer have to endure that step on my own.
Metrics
?! I'm a UX contributor. I'm offering my feedback that this would be a beneficial feature that will save me time-on-task in my invoicing. Time on task for other people who submit invoices to OC, I guess.
Documentation
I'm not a developer. Look at how bill.com does it?

If we go back to this issue, we may consider using some third-party APIs out there (https://bill.com ?) that can provide us with structured info from PDF files as a service. If not too expensive, such a service would drastically reduce the complexity required to implement this feature. Parsing PDFs is a painful job.

I the spirit of "MVP" it could also be a simpler option to just create that email address that automagically posts a PDF to a project's "invoices;" and then the humans with the project that have to approve the expense, plug-in information themselves, manually. After the vendor has setup their "profile" in OC w/ payout options, etc.
Literally the only "new" information I ever post into the webform, is the total $$ and a subject for the expense.  I don't need to verify my address and payout info with each expense.
Having project leads either cut-and-paste or compose "Expense" titles would also provide some uniformity for community members, when they review named expenses; eg, different contributors will interpret the ask for that information differently, and put different information in there... which can confuse donors (which I am guilty of!).

This looks like a very good AI model to automatically parse receipts: https://github.com/clovaai/donut
It could be trained on the database of open collective for even better accuracy.

@xdamman Extracted this in https://github.com/opencollective/opencollective/issues/6793 as having some kind of automatic expense generation from images could be helpful for many different features (including this one).


[core] Fix docs:dev not working after upgrading next to 12.1.0

[x] I have followed (at least) the PR section of the contributing guide.

The problem
Hey 👋
In MUI X we have updated next package recently (https://github.com/mui/mui-x/pull/4625) and docs:dev script started to fail with the following error:
error -<EMAIL_ADDRESS>Module parse failed: Cannot use 'import.meta' outside a module (87:16)
File was processed with these loaders:
 *<EMAIL_ADDRESS> * ../node_modules/next/dist/build/babel/loader/index.js
You may need an additional loader to handle the result of these loaders.
|                 // still a Refresh Boundary later.
|                 // @ts-ignore importMeta is replaced in the loader
>                 import.meta.webpackHot.accept();
|                 // This field is set when the previous version of this module was a
|                 // Refresh Boundary, letting us know we need to check for invalidation or

The fix
I've searched a bit and found this comment with a proposed solution https://github.com/vercel/next.js/issues/35412#issuecomment-1079482890, then I discovered "type": "commonjs" here:
https://github.com/mui/material-ui/blob/c919863947b23689b7a3958a1c8602e1d6f9baba/docs/packages/markdown/package.json#L5
Testing
To confirm that removing this line fixes the issue, I have:

updated next locally to the latest version
removed nextjs cache (.next)
ran yarn docs:dev to reproduce the error above
removed "type": "commonjs" from markdown package
removed nextjs cache (.next) again
ran yarn docs:dev to make sure the error is gone

I'm reopening https://github.com/mui/material-ui/pull/32430 then

@oliviertassinari did you close it because of docs:dev issue?

I closed it because there were no security issues for us.

@oliviertassinari does it make sense to merge next upgrade into this branch, so that it's easier to check that it works?

Are we progressing with this one? It also failed for me and I suspect it will be a problem for the release.


Add dependency org.xmlresolver

Description
Replace dependency Apache Resolver with org.xmlresolver.
Motivation and Context
The org.xmlresolver library has features the old Apache Resolver lacks and is under more active development.
How Has This Been Tested?
Old tests pass.
Type of Changes

Breaking change because dependencies change from CatalogResolver to Resolver at Java level. This only affects plug-ins that use custom Java code that use the catalog resolver.

Documentation and Compatibility
Plug-in developer migration documentation to use Resolver class instead of CatalogResolver.

This is proving to be somewhat difficult to implement. Ant relies on the old catalog resolver and and we may have to keep on shipping the old catalog resolver for plug-in backwards compatibility.

Some questions, what are those extra features the org.xmlresolver would have? As it may not be used in lots of implementations, it may have various issues that the stable Apache resolver does not.
About the ANT resolver which builds on top of the Apache resolver, it is indeed buggy, at some point I suggested removing the need for the ANT resolver: https://github.com/dita-ot/dita-ot/issues/3376

Removing from 3.6 project board based on discussion at contributor call: https://github.com/dita-ot/dita-ot/wiki/Meeting-minutes-2020-10-01

It looks like too large and out of scope for DITA-OT to write a wrapper that would reimplement the old resolver's URI to be backed by the new resolver. So it's easier to just have both the new and old resolver as dependency.

Old resolver is used by Ant when you explicitly use <xmlcatalog>
New resolver is used in all other cases.

This will be included in #4130

Implemented when updating to Saxon 12.3

Implemented when updating to Saxon 12.3

@jelovirt Does this mean that we can remove “Apache XML Commons Resolver” from the list of Third-party software?


Add cucumber checker for Cucumber files

This adds cucumber as a checker for Cucumber files (https://cucumber.io/).
For speed purposes, it is only feasible to use the --dry-run option, which means the only meaningful check that can be done is for undefined steps. Luckily, that is exactly the thing a user would most want to check.
Tested only with Cucumber for Ruby, but it should work for any flavor of Cucumber that follows a substantially similar directory structure, with tests in a directory named features/ and step definitions somewhere below that. Each flavor of Cucumber has a slightly different interface.
Closes #1374.

The command callback test is failing on Windows due to the path syntax. I'm not sure how to handle that.

The test is probably just failing due to different slashes, and it can probably be fixed by using the ale#path#Simplify function.

Thanks; changes made.


Fix initialization of HookCaller in requirementslib.

Call Pep517HookCaller.__init__() (see pep517/wrappers.py) from
HookCaller.__init__() to ensure Pep517HookCaller.python_executable
and other required attributes are initialized.
Fixes #4772
Thank you for contributing to Pipenv!
The issue
Fixes #4772 and https://github.com/sarugaku/requirementslib/issues/297
The fix
Correctly constructs the base class of HookCaller in requirementslib so that it doesn't throw exceptions due to uninitialized attributes when calling methods on the base class Pep517HookCaller.
The checklist

[x] Associated issue
[ ] A news fragment in the news/ directory to describe this fix with the extension .bugfix, .feature, .behavior, .doc. .vendor. or .trivial (this will appear in the release changelog). Use semantic line breaks and name the file after the issue number or the PR #.

Important I haven't forked requirementslib to pipenv/patched yet as I first wanted to get some visibility from pipenv team on this issue to poke the maintainer of requirementslib to merge the patch and potentially fix the use of catch all Exception handling that hid the issue in the first place. If maintainers of requirementslib are still unresponsive I'm happy to run through the process to fork this vendored package for pipenv.

The library is in the same organisation, but I'm not an active maintainers to it, unfortuantely, and not really familir with the code base to review changes.


vkdt-cli: add i-format option

to let export non raw files through vkdt-cli

wait what are you trying to do there? and why are you not passing a cfg file to the cli? this looks wrong to me but also i don't understand what you're trying to do here.

I just want to create thumbnails with vkdt-cli. That works implicitly for raw (in graph_export I-raw is set by default) but jpg are ignored. Then vkdt-cli needs a way to know that the image is a jpg.

i'm guessing that you call vkdt-cli -g something.jpg.cfg where the cfg
does not in fact exist

Exactly, I've missed to mention this.

could you try whether that works

Currently a bit far of the computer, I'll try this asap.
Thanks!


[Automated workflow] upgrade-unity from 2021.3.16f1 to 2021.3.1f1 - URP: true

Automated changes by create-pull-request GitHub action

This was a test run to see if everything works as expected with the new naming 👍


ISPN-6491 Remove total order thread pool

https://issues.jboss.org/browse/ISPN-6491

Shouldn't we mention in the docs that we are deprecating this? That's the only bit I'm missing in this PR, otherwise looks ok to me.

@slaskawi where should I mention it?
the thread pool was deprecated (and no longer used) since 7.1. I just left the configuration and parser, that is removed in this PR.

@pruivo I normally mention changes like this at https://github.com/infinispan/infinispan/blob/master/documentation/src/main/asciidoc/upgrading/upgrading.adoc

@wburns Yes, that's exactly what I mean

done!

Integrated, thanks Pedro!


Web worker must use import instead of importScripts to work properly

Describe the bug
Web workers must use import instead of importScripts to work properly
In the document

The worker script can also use import statements instead of importScripts()

Reproduction
✅  Correct

❌  No execution

System Info

vite version: 2.0.5
chrome： 88.0
Operating System: mac OS
Node version: v12.17.0

When importing a worker like this
import Worker from './myWorker.js?worker';

Vite is generating this output-
function e() {
  return new Worker('/assets/myWorker.81e2507d.js', { type: 'module' });
}

the type indicates that its a module worker Module workers are a new feature which supports imports instead of importScripts hence the error. If you use ES module imports then Vite bundles in the worker file also.
Vite is being future proof here. Hope it helps.


optimize: Change Settings to allow InitialLocation

This modifies Settings to allow specifying an initial location and properties of the function (value, gradient, etc.). This allows to work with local optimizers that are seeded with initial settings. This has two fields that must be specified (both Location and Operation). The difficulty is that the default value of the function is 0, so we either must require the user to specify it is set (somehow), or require the user to change the default value away if it is not set. The former seems much safer.
Please take a look.

It's unfortunate that this version requires users to deal with Operations, which otherwise are only needed by the methods. I think the other viable alternative is to have
type Settings {
    InitX []float64
    InitValues *Location
}

where InitValues ignores the X location (using InitX) and must have the initial function value set if it is non-nil. It's not clear which is less confusing. Maybe this current way is worse because setting the initial X will be the most common thing to do (far more common than setting any of the values).
Thoughts @vladimir-ch @kortschak ?

As an addendum, we only really need to know a boolean for FunctionValue, so that's an additional argument that this other approach is better, since this current approach makes setting everything a two-line process and introduces additional complexity to the user, while the alternate approach only has one odd pathology.
I'll switch and resubmit a PR


[Windows] build progress output broken on Windows since version 6.0

Bug Report or Feature Request

[x] bug report
[ ] feature request

Area

[x] devkit
[ ] schematics

Versions

Windows 10 1803 Build 17134.137
node 10.5.0
npm 6.1.0
ng version

Angular CLI: 6.0.8
Node: 10.5.0
OS: win32 x64
Angular: 6.0.7
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.6.8
@angular-devkit/build-angular     0.6.8
@angular-devkit/build-optimizer   0.6.8
@angular-devkit/core              0.6.8
@angular-devkit/schematics        0.6.8
@angular/cli                      6.0.8
@ngtools/webpack                  6.0.8
@schematics/angular               0.6.8
@schematics/update                0.6.8
rxjs                              6.2.1
typescript                        2.7.2
webpack                           4.8.3

Repro steps

create a new project with ng new
try to build/serve the project on Windows
see the build progress output

I tried both powershell and cmd on Windows
I'm not sure if this is a Windows issue and I also tried on the Windows Insiders as well
I think the issue Microsoft/vscode#19665 may helps probably

The log given by the failure

Desired functionality
the build progress output should not be broken on Windows
BTW: the progress output works nicely as always in Bash on Ubuntu 16.04
SORRY for my poor English 😆

it's<EMAIL_ADDRESS>with<EMAIL_ADDRESS>and the issue is still there 😢

I'm not sure what's happening, but I am on Windows, using Git Bash, and don't see this behaviour.

hi @filipesilva
After doing some experiments, I think this issue is related to the font setting of the terminal (powershell, git bash , etc)
when I change the font setting of Git Bash like this the build output goes into a mess

but when I change the font setting of Git Bash like this the output goes well

just changed the font size from 8x16 to 8x12

SORRY for my poor English 😆

I redo the experiment above on the old angular project with<EMAIL_ADDRESS>and<EMAIL_ADDRESS>the output goes very well no matter what the font setting of Git Bash is

Thanks for investigation 👍
It might be related with changes on Webpack's side as well, we'll have to investigate. Best I can say for now is to turn off progress reporting via --progress=false or in the angular.json file.

@filipesilva
this issue has been fixed with<EMAIL_ADDRESS>and<EMAIL_ADDRESS>in my scenario 🎉


Training getting slower with more  more Spark executors?

Training with cifar10 datasets following the steps in GetStarted_yarn:
with 1 executor, it take 7 minites,
with 2, it takes 12 minites
with 4 it takes 23 minites

First, distributed training does not help in all cases. As you add more and more nodes to the cluster, communication cost increases. This is especially true if your model is large.
Second, you did not mention the batch size. Maybe you were comparing apples and oranges. Let's say you set batch size = 32. With 4 executors (and 1 gpu per executor), you are getting effectively 4*32=128 batch size. So the 4-node cluster has as 4X work-load as 1-node cluster. If you set batch size = 32 for single-node, and batch_size = 8 for 4-node cluster, then it is a fair comparison. But In the latter case, communication becomes bottleneck since the GPUs are likely idle most of the time, waiting to be fed.

Thanks,@junshi15
I think I get what you mean.
Anyway,with more executors,I could set bigger batch size(the actual batch size = batch size * num of executors) to make full use of GPUS comparing to single-node.
Is that correct?

Another question,
If I have two executers(1 gpu per executor), the gpu of one is idle and another is busy.Does the time cost of training depends mainly on  the training time of the busy executer without consideration of  communication?

This  is synchronous training. The speed is limited by the slowest executor.

What's the main factor that affect the communication, the bandwith?

bandwith, latency, etc. depending on your network.

Just to clarify: Does the accuracy improves, when I don't decrease the batchsize but increase the number of executors? When I understood it correctly, more batches are processed then.
Or is there any other measurable "benefit", when I don't deacrease the batchsize?

If you fixed the batch size in the prototxt file, but increase number of executors, you process more images per batch. It is not clear you will get better accuracy. There are many things you want to tune.
Folks at Facebook manage to do just this.


Fix a false positive for RSpec/ContextWording when context is interpolated string literal or execute string

This PR is fix a false positive for RSpec/ContextWording when context is interpolated string literal or execute string.

Before submitting the PR make sure the following are checked:

[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[-] Updated documentation.
[x] Added an entry to the CHANGELOG.md if the new code introduces user-observable changes.
[x] The build (bundle exec rake) passes (be sure to run this locally, since it may produce updated documentation that you will need to commit).

I rebased and resolved the conflict.

FYI: This change helped to catch context "has #{} something", but makes unable to write something like context "#{variable ? 'with' : 'without'} something".

context "#{variable ? 'with' : 'without'} something"

I think a rubocop:disable is appropriate for this specific case.

context "#{variable ? 'with' : 'without'} something"

I think a rubocop:disable is appropriate for this specific case.

Yeah, I thought about it. Thanks. I've changed the string completely. So, it was as "FYI", not an issue. You can support such things or you can leave it.

I think we could only support it by evaling the interpolated string, or by matching it against some regex. Either way, it would be a mess, and would probably still not work properly 😅

I think we could only support it by evaling the interpolated string, or by matching it against some regex. Either way, it would be a mess, and would probably still not work properly sweat_smile

I believe so, so it's not even a purpose, just a note, maybe for others who will find this, or for contributors who like challenges or have great ideas. :sweat_smile:


mgmt, update test on ARM SDKs (Missing parts)

Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

/azp run java - loadtesting - mgmt - tests

Why we have no LIVE test on iothub?
It was checked in the task

If we mistakenly deleted the test, could you add it back?

/azp run java - iothub - mgmt - tests

The live test of iothub has been re-added, and the package definition is com.azure.resourcemanager.iothub.


Defaults Github url parameters to an empty string

The application code currently has defaults built in for these URLs. For example:
const getApiEndpoints = (apiBaseUrl = GITHUB_API_URL || 'https://api.github.com',
                         loginBaseUrl = GITHUB_LOGIN_URL || 'https://github.com') => ({

This PR adds defaults for these urls as the empty string so that the previous defaults can still work.
Another approach would be to move these defaults into template.yml instead of using an empty string.
Cheers!

Thanks for this!
Your suggestion of moving the defaults to the yaml seems to leave the fewest surprises for future developers- would you be able to update the PR?

Sure thing! I prefer that as well. Btw is your library working for you currently? I tried deploying and I get an error from Cognito about processing the authorization code. Should have that change today!


web-dropin 5.24.0 error reporting issue

Hello, when using the Test 3D Secure 2 authentication card (amex 3714 4963 5398 431), sometimes the component's onError is called, but instead of an error object (with name/message props) I get a string (see the console.log('error =' , error) screenshot)
This happened in a couple of tests (out of 50~60) just after sending the card information and waiting for the security code form to be displayed.
kind regards

Hey @pine3ree
If I understand correctly the error that you got was seems to be a network error cause by your own server correct?
So the issue is regarding why this doesn't go through the library onError instead of throwing an exception?

Hello @m1aw ,
in this case the issue is not the network error per se, but the fact that i do not get an error object. I get a string instead. It does go through the onError(error, component) handler, but error is a string (the first line in the screenshot is my call console.log('error =', error) inside onError()). I expect and error object with a name property and a message property, also according to adyen-web js api.

@m1aw I would expect:
error.name = 'NETWORK_ERROR'
error.message === 'Call to https://.,,ect'

Inside my handler:
    function handleError(error, dropin) {
        IS_DEBUG && console.log('error =', error);
        setTimeout(function() {
            window.location.href = HREF_ADYEN_OUTCOME + '&rc=' + encodeURIComponent(error.name) + '&mref=' + MREF;
        }, REDIRECT_TIMEOUT);
    }

since error.name is undefined 'rc=' + encodeURIComponent(error.name) resolves to 'rc=undefined', while it should resolve to 'rc=NETWORK_ERROR'.
As a temporary patch I added a type check inside my handler (just before the setTimeout call), hoping that if error is a string there is a first part before a colon char that represents one of adyen error/result codes
        if (typeof error === 'string') {
            var regexp = /^[A-Z]+(?:\_[A-Z]+)*\:/i;
            if (regexp.test(error)) {
                var parts = error.split(':', 2);
                error = { name: parts[0].trim() };
            }
        }

Hello @pine3ree
I was taking another look at this issue, could you let us know which request is generating this error?
Additionally could you let us know which version of browser are you using and which version of @adyen/adyen-web are you using?
Thank you,

Hello @m1aw ,
I was testing the native 3DS2 Drop-in integration. The (uncaught?) network error occurred just after sending the credit card info in order to receive the additional 3ds2 secure code form. I test mostly in FF (linux, latest version at the time of this issue so probably 103~105). But i remember I had the same issue even with chromium-linux (v.90). I updated my customer frontend libs up to version 5.29.x (and the php server library too), but since with my workaround the issue was circumvented I cannot tell if the issue itself is still there. Another strange fact that i remember is that the issue would not happen in a week during tests and the maybe twice within a few minutes. Kind regards.


Logging interval setting issue

Hello,
Using iOS MyAmbience application (v1.4.0), going through the SmartGadget settings' menu to set the logging interval does not properly work.
For example, if we set it to 1 minute, let the device run 2 or 3 minutes, export the data, then the measurements are on a 1 minute basis, perfect.
But if you export the data again, the new measurements are back on a 1 second basis.
For example, below is the exported data when I set the interval to 1 minute.
First, by default, it is set to 1 second.
Then we clearly see the 1 minute basis.
And then it's back to 1 second basis, as per the second export.
1597499034.566	28.990
1597499035.562	29.020
1597499036.565	29.020
1597499037.572	28.990
1597499038.556	29.010
1597499100.000	28.980
1597499160.000	29.020
1597499220.000	29.020
1597499220.028	29.040
1597499221.025	29.040
1597499222.022	29.020
1597499223.021	29.040
1597499224.018	29.040

It's as if reaching the SmartGadget settings' menu performed a reset of the logging interval...
Could it be possible to have a look at this please ?
Thank you very much 👍

Hi
Thank you for your comments.
Long story short: Indeed, this is a confusing, it's subject to change but originally intended.
I hope I followed your description correctly, let's look at your log and see how this happened.
The app, when running and connected to the Gadget, is receiving live data from the Smart Gadget in a 1 second interval. This is fixed and can not be changed, but only happens, while being connected to the Gadget.
The logging interval feature sets the interval in which the sensor measurement is cached on the Smart Gadget for later download.
Both of these are combined in the App's storage, e.g. if you connect to your smart gadget, then you receive live values in 1 second interval. If you download the data from the gadget, the data of that time range will have the logging interval set (in your example: 1min). If you stay connected with the gadget, the live values with 1 second interval will keep coming in for all the time after the "sync"/download.
Connected with Gadget -> interval is 1 second:
1597499034.566	28.990
1597499035.562	29.020
1597499036.565	29.020
1597499037.572	28.990
1597499038.556	29.010
Downloaded from gadget -> interval is 1 minute, as adjustable in the gadget settings
1597499100.000	28.980
1597499160.000	29.020
1597499220.000	29.020
Connected with Gadget -> interval is 1 second:
1597499220.028	29.040
1597499221.025	29.040
1597499222.022	29.020
1597499223.021	29.040
1597499224.018	29.040
What you export, is what you have in your app and see on the screen, a mix of live values and values downloaded from the gadget.
Does this explain the issue described?
As I've mentioned, this feature is subject to change. We have some ideas how to improve this in a future version of MyAmbience, as it is indeed quite confusing. If you have any suggestions, don't hesitate to share.

Hi @MBjoern, and thank you very much for your detailed explanation.
I'm able to reproduce the behaviour you just explained ; the internal device logging interval is and remains the user-defined one.
But while connected, MyAmbience receives data on a 1 second basis, adding it to the data exported from the device.
That's clear :)
Perhaps you should use "live data" for real-time counters display only, and not add it to the exported data.

Hi @MBjoern, and thank you very much for your detailed explanation.
I'm able to reproduce the behaviour you just explained ; the internal device logging interval is and remains the user-defined one.
But while connected, MyAmbience receives data on a 1 second basis, adding it to the data exported from the device.
That's clear :)
As an enhancement, perhaps you should use "live data" (1 second basis) for real-time counters display only, and not add it to the exported data (as it then "breaks" the user-defined interval).
Anyway, thank you very much for your great support 👍

Hi @sensusr , I'm glad that it got clear now.
Agreed, that would be a great enhancement.
Stay tuned for future updates of the app. We'll be glad to hear your comments and feedback on the adaption of the feature.
Best regards


Extend the handler for other events: VoiceStateUpdate

The current event handler system is only implemented for Message and Reaction(Add/Remove) events and needs to be extended to the other events used by the bot.
The steps required for this should be common for all events:

[ ] Create a new file for the event in src/event-distribution/events
[ ] Implement 4 details for the event (have a look at the other events for inspiration):

[ ] An interface for the options
[ ] A type defining the function parameters for the handler function
[ ] A function that adds a new handler in the preset structure so it can be retrieved later
[ ] A function that extracts information the distributor uses to determine which handlers should be run

[ ] Update src/event-distribution/events/index.ts to include the above things
[ ] Update src/index.ts to call the distribution with the event information

This issue tracks progress of the process of adding the VoiceStateUpdate event.

This was not closed in #346


Running Error [Bug]

after dnu restore
System.TypeLoadException: Could not load type 'Microsoft.Dnx.Runtime.PlatformServices' from assembly 'Microsoft.Dnx.Runtime.Abstractions, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'.
   at Microsoft.AspNet.Hosting.Program.Main(String[] args)
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at Microsoft.Dnx.Runtime.Common.EntryPointExecutor.Execute(Assembly assembly, String[] args, IServiceProvider serviceProvider)
   at Microsoft.Dnx.ApplicationHost.Program.ExecuteMain(DefaultHost host, String applicationName, String[] args)
   at Microsoft.Dnx.ApplicationHost.Program.Main(String[] args)
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at Microsoft.Dnx.Runtime.Common.EntryPointExecutor.Execute(Assembly assembly, String[] args, IServiceProvider serviceProvider)
   at Microsoft.Dnx.Host.Bootstrapper.RunAsync(List`1 args, IRuntimeEnvironment env, String appBase, FrameworkName targetFramework)
   at Microsoft.Dnx.Host.RuntimeBootstrapper.ExecuteAsync(String[] args, BootstrapperContext bootstrapperContext)
   at Microsoft.Dnx.Host.RuntimeBootstrapper.Execute(String[] args, BootstrapperContext bootstrapperContext)
dnx 1.0.0-rc1-15838
Please help someone I can't run application :|

What's in your project.json?


DEMON GAZE 2 [PCSB01150]

App summary

App name: DEMON GAZE 2
App serial: PCSB01150
App version: 01.00

Vita3K summary

Version: v0.1.5
Build number: 2614
Commit hash: https://github.com/vita3k/vita3k/commit/0f386175
CPU backend: Dynamic

Test environment summary

Tested by: othmanead
OS: Windows 11
CPU: Intel i5-10300H
GPU: RTX 3060 mobile
RAM: 16 GB

Issues
Small crackling sound in bgm, low performance and minor graphical issues
Screenshots

Log
Vita3k log.txt
Recommended labels

Ingame +
Audio issues
Slow

When attempting to install either this version or the USA variant, Vita3k installs all DLC, but crashes when attempting to install the game proper, listing only the line
[21:44:31.807] |C| [logging::init::<lambda_2>::operator ()]: Unhandled C++ exception. bad allocation
This has been observed for probably half a year now, but is present in v0.1.6 3091-e3956248 as well.

sadly the install crash is still happening in v0.1.9 3540-34565e3e

Additionally, if the installation crash is worked around using a vpk file to install the base game, the game itself does display graphics when it runs.

App summary

App name: DEMON GAZE II
App serial: PCSB01150
App version: 1.01

Vita3K summary

Version: v0.2.0
Build number: 3691
Commit hash: https://github.com/vita3k/vita3k/commit/31826697
CPU backend: Dynarmic
GPU backend: Vulkan

Test environment summary

Tested by: Slade
OS: Microsoft Windows 10 Professional (build 19045)
CPU: Intel(R) Core(TM) i7-3770K CPU @ 3.50GHz
GPU: NVIDIA GeForce GTX 1060 6GB
RAM: 16 GB

Issues

The game will boot to a black screen with audio playing in the back ground.
You can fumble through the menus and start a game, but no images will appear on screen.
Using the OpenGL renderer has the same black screen issue.

Screenshots

Log
vita3k.log
Recommended labels

Bootable
black screen
regression

Also unable to play now.


Create port_moody.json

Created source file for Port Moody, BC.

@KHobbs3 It looks like the data link here is invalid:
https://opendata.arcgis.com/datasets/7bc8ca82182141d2b3f0452fe593d528_30.zip

@KHobbs3  @ingalls https://services9.arcgis.com/XMHGoEc3tdJ8TphG/arcgis/rest/services/Addresses/FeatureServer/37
source:
https://data.portmoody.ca/datasets/d485aaa4f27641bf88ccb5fe79180bed_37

@KHobbs3 @ingalls https://services9.arcgis.com/XMHGoEc3tdJ8TphG/arcgis/rest/services/Addresses/FeatureServer/37
source:
https://data.portmoody.ca/datasets/d485aaa4f27641bf88ccb5fe79180bed_37

Thanks, took note of that and will look into this

@KHobbs3 @ingalls https://services9.arcgis.com/XMHGoEc3tdJ8TphG/arcgis/rest/services/Addresses/FeatureServer/37
source:
https://data.portmoody.ca/datasets/d485aaa4f27641bf88ccb5fe79180bed_37

@justinelliotmeyers What would the "number" and "street" fields be for this dataset?

@KHobbs3 yikes - that actually looks like junk! Sorry about that - I usually always check to make sure the data looks good, but that is missing everything!
try this little guy:
https://services9.arcgis.com/XMHGoEc3tdJ8TphG/ArcGIS/rest/services/Parcel_Information/FeatureServer/0

@KHobbs3 yikes - that actually looks like junk! Sorry about that - I usually always check to make sure the data looks good, but that is missing everything!
try this little guy:
https://services9.arcgis.com/XMHGoEc3tdJ8TphG/ArcGIS/rest/services/Parcel_Information/FeatureServer/0
use address field

Thank you! I'll just verify the license information before making another PR...


Update SQL spec descriptions

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[x] The title of the PR is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[x] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[x] If applicable, the PR references the bug/issue that it fixes.
[x] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[x] I have read the contribution guidelines.
[x] My spec meets the review criteria:

[x] The spec conforms to the Swagger 2.0 specification.
[x] Validation errors from the Linter extension for VS Code have all been fixed for this spec. (Note: for large, previously checked in specs, there will likely be many errors shown. Please contact our team so we can set a timeframe for fixing these errors if your PR is not going to address them).
[x] The spec follows the patterns described in the Swagger good patterns document unless the service API makes this impossible.

Hi @jaredmoo, 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 (jaredmoo). 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;

Fixed, thanks. Node build seems to be hitting a transient error.

@jaredmoo thanks, restarted job and now it is passing.

Can I get a review and merge here?

@jaredmoo thanks, merging.

 https://github.com/Azure/azure-sdk-for-ruby/pull/700

No modification for NodeJS

 https://github.com/Azure/azure-sdk-for-python/pull/1010

Thanks!


[Workshop] There is no an error message when the application limit is exceeded when the Parent applies to the workshop for his children.

Environment: Windows 10, 64-bit, Google Chrome 107.0.5304.107.
Reproducible: always.
Preconditions
1.Go to link https://oos.dmytrominochkin.cloud/#/ as registered user, Parent role (Login<EMAIL_ADDRESS>Password: 'Qazwsx789!')
2.At least one child is created in the Parent`s profile.
3.The location is selected (e.g. Kyiv)
4.There is at least one workshop registered by Service provider in selected location (Login<EMAIL_ADDRESS>Password: 'Qazwsx789!').
5.A parent hasn't applied to chosen workshops within week before test execution.
6.The interested workshop is found and the 'Workshop' page is opened (by clicking the "Детальніше" button of the workshop card).
7.The 'Application' page is opened (by clicking the 'Подати заяву' button).
8.The option "Подати дані про дитину" is chosen.
9.The child is chosen from "Виберіть дитину" dropdown list AND all checkboxes are selected.
Steps to reproduce

Submit an application to the workshop for the first time by clicking 'Відправити' button on the 'Application' page and then the 'Підтвердити' button in the confirmation popup.
Log in as SP and change status "Очікує підтвердження" on the "Відмовлено" status.
Log in as Parent.
Submit an application to the same workshop for the same child for the second time.
Log in as SP and change status "Очікує підтвердження" on the "Відмовлено" status.
Log in as Parent.
Submit an application to the same workshop for the same child for the third time.
Pay attention to the absence of a notification that the limit has been exceeded.

Actual result
There is no notification when the application limit is exceeded when the Parent applies to the workshop for his children.
https://drive.google.com/file/d/1Yr-51FZd_YV1VIsk9DCrhAv38KmqZy8_/view?usp=share_link
Expected result
An error message appears on the screen: "Перевищено ліміт заявок. Спробуйте ще раз через тиждень". The application is not submitted.
User story and test case links
https://jira.softserve.academy/browse/OS-431
Labels to be added
"Bug", Priority ("medium"), Severity ("medium"), Type ("UI, "Functional").

Fixed


UserAvatar: Fail gently on bug where auth info is missing.

This fixes #4388 with a quick workaround.  The underlying issue is
still present -- we shouldn't have this component mounted at all in
this situation.  But fixing that may involve some tricky changes to
our navigation, and this is a high-severity issue -- a crash in an
essential flow for using the app for many people -- so we want to fix
it promptly even if the underlying fix takes some time.
Fixes: #4388

Thanks, merged! I've just gone through your repro at #4388 (the first one) and the app didn't crash.

Thanks, merged! I've just gone through your repro at #4388 (the first one) and the app didn't crash.

Thanks for the fast review!

Thanks for the fast review!


umd bundles are not minified since 12.0.0 release

Which @angular/* package(s) are the source of the bug?
animations, common, compiler, core, forms, platform-browser-dynamic, platform-browser, router, upgrade
Is this a regression?
Yes
Description
Since 12.0.0 release there are no more bundle minified

https://unpkg.com/browse/<EMAIL_ADDRESS>https://unpkg.com/browse/<EMAIL_ADDRESS>https://unpkg.com/browse/<EMAIL_ADDRESS>

which are available in 11.x

https://unpkg.com/browse/<EMAIL_ADDRESS>https://unpkg.com/browse/<EMAIL_ADDRESS>https://unpkg.com/browse/<EMAIL_ADDRESS>
Please provide a link to a minimal reproduction of the bug
https://unpkg.com/browse/<EMAIL_ADDRESS>Please provide the exception or error you saw
When I upgrade my scripts to use umds I have 404

Please provide the environment you discovered this bug in
No response
Anything else?
No response

This is indeed by design https://github.com/angular/angular/pull/41425.


Add PrettyTables@v2 to compat bounds

I found that this was needed to allow usage of ReadStatTables with<EMAIL_ADDRESS>From what I can tell the PrettyTables usage in this package isn't using anything that broke between version 1 and 2 so this is probably safe. Locally one of the tests (readstat sas7bdat) failed (we'll see what happens in CI) both with this update and without, so that one appears unrelated.

Thanks for raising this issue! There has been a plan to make further changes to adopt new features from DataFrames.jl and DataAPI.jl. The compatibility issue will be fully resolved by that time, with some other changes that will result in a version v0.2 of this package.

Great, thanks for the feedback @junyuan-chen. I assume that those changes are ongoing in #6, or somewhere else I can keep track of?

Ok, great, I'll close this PR then since it'll not be needed, thanks.


Code Snippets are not showing the construction of request bodies

e.g.

I'm pretty sure we used to construct an object for request bodies.  Can someone investigate why we are not creating an example object based on the request body schema?
@andrueastman Can you confirm my recollection that we used to do this?

We have tested this. When you edit the request body, the query isn't updated. This is why the changes you make to the request body are not reflected on the snippet response.
A workaround for now is to run the query with the new request body. This will update the query and consequently return the correct snippet response with your new request body.
We are fixing it :)

Hey @darrelmiller...
While testing this further... @Onokaev noticed there was an issue on the GE side with the snippets not updating when the request body is updated.
Perhaps that's what is causing the issue above.
This shall be fixed.

Closing this as this is currently not reproducible


Update packages troubleshooting doc to re-enable cronjob

Updating doc to add command for un-suspending the cronjob in cases of forgotten credentials.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm

/cherry-pick release-0.12


Use biomejs for VSCode extension code formatting and linting

We've migrated the console to use biomejs, we should do the same for the extension code.
Perhaps we can share the biome.json file at the root of the repo (or ui folder if we go that route)

Fixed with #2475


updated the formatting of release date

Closes #120 - This should fix #120 when the full database repopulation is run


Expand README

This PR Expands the README by documenting the general idea of how the custom copilot integrations work. More detailed information is found under each of the custom copilots.

Should we get this merged, to get something started? And improve over time? 🤔
i want to incentivize folks to add different copilots here
@IgorWounds @mnicstruwig

Should we get this merged, to get something started? And improve over time? 🤔
i want to incentivize folks to add different copilots here
@IgorWounds @mnicstruwig

We still need to edit this further, and parts related to user files will also need to change now that we are treating everything as a widget. I'll try to wrap this up soon.

This is ready to merge, but we'll have to wait until we deploy to PROD.


⚠ Fortnite has degraded performance

In b7893e5, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1596 ms

Resolved: Fortnite performance has improved in b8f18c5.


Fail Breaking Changes CI on exception

Automation for azure-sdk-for-python

Automation for azure-sdk-for-go

Looks reasonable :D


Maybe split in jaxrs-client / jaxrs-server modules?

Curious how you feel about this @pavolloffay.
Reason: as a JAX-RS client I have no reason to include the opentracing-web-servlet-filter dependency.
Currently I actively exclude the transient dependency for a JAX-RS client-only jar.
Maybe this is related to #13 ?

There is also an option to create a common artifact in servlet filter because at the moment it uses only some utility class from there. I'm working on adding @Priority, then we should know if we need to implement  #13.


[enhancement] Adding Parameter in config for better Username Detection.

Add a new parameters in debug configuration which taking as input a string : idusername
This string should be the value of the id on the MainPage, to detect the username.
for example, a main login page with the html code :
<input class="mdc-text-field__input pwd" type="password" id="pwd_pass" size="25"  name="password" value="">
The Parameter isUsername should be pwd_pass, then the extension can automatically take the username when logging and store it for better logging/information.
New code should have for example in content.js
function runUsernameScraper() {   const username = <HTMLInputElement>document.getElementById(IdUsername)   if (username != null) {      return username.value   }

👍
Great feedback. Will add this.


Unable to create delta log metadata files for Iceberg table

We have created iceberg data in a GCS bucket, which includes a 'data' folder and a 'metadata' folder. I am attempting to generate delta metadata after cloning the 'onetable' repository. Although the process creates the '_delta_log' folder under base path of table, it does not generate any delta log files, and I am encountering the following error:
error: -

my_config.yaml -
sourceFormat: ICEBERG
targetFormats:

DELTA
datasets:
tableBasePath: gs://iceberg_data/test.db/people
tableDataPath: gs://iceberg_data/test.db/people/data
tableName: people
partitionSpec: date:VALUE

@Manoranjan93 are you using a catalog for your Iceberg table? Also can you confirm whether that version hint file exists in GCS?

@the-other-tim-brown - Yes, we are using a catalog for the Iceberg table. In GCS, under the table directory, there are two main folders: 'data' and 'metadata'. In the 'data' folder, there are partition folders, each containing Parquet files or other data files. In the 'metadata' folder, you can find the 'metadata.json' file along with two additional '.avro' files. Please find the sample folder structure in gcs

ls iceberg/
data/
metadata/

ls iceberg/data
creation_date=2015–01–10/
creation_date=2015–01–11/

ls iceberg/metadata

00000–55gcadf-xxxx-xxx.metadata.json
0swscs-kcdedc-xxxx-xxxx-m0.avro
snap-xxx-x-xxx.avro

@Manoranjan93 did you pass in the iceberg catalog configuration options for your source? If you are using the RunSync tool you can pass in -i catalog_options.yaml where that file is structured like:
catalogImpl: com.my.catalog.Impl
catalogName: name
catalogOptions:
  option1: value1
  option2: value2

@the-other-tim-brown - I have already attempted to create a catalog.yaml file for source, as highlighted below, for the Iceberg table, and I am able to generate a delta log file. Could you please review the following scenarios and confirm if this is the expected behavior? Also, please verify if I am performing the steps correctly.
catalog.yaml file -
catalogImpl: org.apache.iceberg.gcp.biglake.BigLakeCatalog
catalogName: iceberg
catalogOptions:
gcp_project:
gcp_location: us-west1
warehouse: gs://path/to/warehouse
my_config.yaml
sourceFormat: ICEBERG
targetFormats:

DELTA
datasets:
tableBasePath: gs://bucket_name/db/table_name
tableDataPath: gs://bucket_name/db/table_name/data
tableName: table_name
namespace: biglake_dataset_id
partitionSpec: date:VALUE

I am using google cloud shell editor to run all the commands.
java -cp utilities/target/utilities-0.1.0-SNAPSHOT-bundled.jar:/home/fexxx/onetable/biglake-catalog-iceberg1.2.0-0.1.0-with-dependencies.jar io.onetable.utilities.RunSync  --datasetConfig my_config.yaml --icebergCatalogConfig catalog.yaml

@Manoranjan93 is there content in the _delta_log/00...000.json file? The steps you are following are what I would expect but I have not tested this path with BLMS. If there are entries in the json, then you can try reading the table as delta from dataproc to validate it is readable.

@the-other-tim-brown -- I have content in the _delta_log/0000..00.json

I will try to read the table as delta.

@Manoranjan93 were you able to get this working?

@the-other-tim-brown Yes, I was able to read as a delta table using spark

@Manoranjan93 close issue?


Explain "Don't use Kotlin lambdas in your public API"

I've been reading your guide, pretty cool summary. I got hung up on this suggestion, could you please expand on it a bit more or link to some resources?

Don't use Kotlin lambdas in your public API

Why is that? What will break?

Gradle enhances the bytecode at runtime to provide a nicer DSL experience for users of your plugin.

That's great, but what if my users are using includeBuild(gradle/plugins), at that point my plugin will be a normal dependency and Gradle doesn't provide a nice DSL magic transformation. Any suggestion to support both?

Don't use Kotlin lambdas in your public API

I think this should go with

Don't use Groovy closures in your public API

The rationale is that one is pretty much unusable from the other language - also both from Java.
When you use Action<T>, it stays idiomatic in plugins implemented in plain Java, Groovy or Kotlin.
In the Groovy DSL (build scripts and precompiled scripts), there's some bytecode magic to also add Closure based overloads mostly for backwards compatibility.
In the Kotlin DSL (build scripts, precompiled scripts and .kt files in source sets with the kotlin-dsl applied), Action<T> is declared as a "lambda with receiver" equivalent to T.() -> Unit in order to remove the need to use it..

@TWiStErRob, if your users are using includeBuild(gradle/plugins) they should get what Gradle provides according to the following, otherwise it's a bug

@eskatos Ah, thanks, that's some proper magic there 🙇🏻♂!!!
Sorry, I confused two of my projects. I double-checked now and I see it actually working in practice:

In one project I have includeBuild(gradle/plugins) and in gradle/plugins/build.gradle.kts have implementation(...) dependencies on Detekt Gradle Plugin, and then it defines plugins that configure Detekt reports. This fun reports only defines Action<T> overload, yet the usage compiles as if it was T.() overload. This works fine as you described.
In another project (also defining plugins) I have a standard java-gradle-plugin + kotlin(jvm) subproject that pulls in the same dependency and if I put the same code in there, I have to use it:project.tasks.withType<Detekt>().configureEach {
	it.reports {
		it.html.required.set(true) // human
		it.txt.required.set(true) // console
	}
}

otherwise it doesn't compile. This is what I was referring to. So the answer here is that I need to apply kotlin-dsl to these modules to get the magic of @HasImplicitReceiver interface Action<T>.

It was always confusing not being able to use them the same way, thanks for clarifying!
(Now I understand some IntellIJ errors I saw related to this when sync was broken! TIL)


Visual Enhancement: Filter columns need more space in between each other

Current:

Columns have 7 px space between each other horizontally
Rows have 7px space between each other vertically
"X" is 7 pixels away from the row

Expected:

Columns have 10px space between them
Rows have 10px space between each other vertically
"X" is 20 pixels away from the row
(px-data-grid_filtering_v6-1_ap.pdf - Page 10)

Nice to have


Fix trailspace issues

Fixes two issues:

Highlight group can't be changed pending load order as it stands right now, use default when calling hi to handle these cases
Wrap the call to vim.fn.matchdelete with a pcall to avoid breaking things like Telescope if the match doesn't exist

Also changes some calls that use vim.fn to use vim.api and check valid state, etc.

Curious, is there a reason to defer_fn on the highlight :h autocmd call? If we do that, the call to get the "current window" might not be accurate. Wanted to check before trying to change it here.

Nope :( Having vim.fn.expand("<abuf>") in here (and removing defer_fn) doesn't seem to work.
What I believe might be happening is that modified is still false at the moment of autocommand execution. It will be changed later by plugins themselves.

Gotcha. So it seems then that one other solution would be to save off the winid while the autocmd is executing, but then perform the rest of the functionality we want later. Something like:
  vim.api.nvim_exec(
    [[augroup MiniTrailspace
        au!
        au WinEnter,BufWinEnter,InsertLeave * lua MiniTrailspace.defer_highlight()
        au WinLeave,BufWinLeave,InsertEnter * lua MiniTrailspace.unhighlight()

        au FileType TelescopePrompt let b:minitrailspace_disable=v:true
      augroup END]],
    false
  )

  -- Create highlighting
  vim.api.nvim_exec([[hi link MiniTrailspace Error]], false)
end

-- Module config
MiniTrailspace.config = {}

function MiniTrailspace.defer_highlight()
  local win_id = vim.api.nvim_get_current_win()

  vim.schedule(function()
    MiniTrailspace.highlight(nil, { win_id = win_id })
  end)
end

-- Functions to perform actions
--- Highlight trailing whitespace
---
---@param check_modifiable boolean: Whether to check |modifiable| (if it is off, don't highlight). Default: `true`.
---@param opts table: Additional options/state for highlighting, currently:
---                     - win_id: The window when the autocommand was invoked
function MiniTrailspace.highlight(check_modifiable, opts)
  check_modifiable = check_modifiable or true

  -- Highlight only in normal mode
  if H.is_disabled() or vim.fn.mode() ~= 'n' then
    MiniTrailspace.unhighlight()
    return
  end

  if check_modifiable and not vim.bo.modifiable then
    return
  end

  local win_id = opts.win_id or vim.api.nvim_get_current_win()
  if not vim.api.nvim_win_is_valid(win_id) then
    return
  end

...

would work then?

Just let you know that I removed defer_fn() by slightly reframing the problem it used to solve. Now instead of modifiable, tracking of buftype option is done via if-else and OptionSet autocommand. Activated with config.only_in_normal_buffers.


ci: update error reference docs

This PR is auto-generated by a nightly GitHub action to update the error reference docs from source code in withastro/astro.

Fixing at source, to correct URLs changed by #6620 !


Expose Sync at the top level

External consumers need to be able to trigger syncs!

Closed by  #557


Use specific page url for view live link

Use specific url for  'View live' PageListingButtons - subclassed Pages weren't using the correct live url

Hi @lambertkuang - thanks for the PR!
Can you clarify exactly where the incorrect URL is appearing, please? As far as I'm aware, the index view already handles fetching the subclassed pages (with the exception of the 'reorder' view, where it's disabled for performance reasons):
https://github.com/wagtail/wagtail/blob/master/wagtail/admin/views/pages.py#L81-L85

Ah right, I'm on an older version of wagtail that wasn't yet using the most specific pages on search - closing this PR 👍


Info: pvekclean works on PBS

Hi Jordan,
just to let you know that the script also works on Proxmox Backup Server (PBS).
First time it found 9 old kernels, I deleted them and rebooted.
Ran the script again and found another one that wasn't in the previous list, and deleted it. That was the only strange thing.
Rebooted again, ran it again, and no old kernels. :)

Hey @alexdelprete,
Thanks for testing on PBS! PVE Kernel Cleaner has been significantly updated lately with the release of v2.0.
If possible, please give this a try and let me know how it goes.


Docker support

Hello,
I added an easy to use docker configuration to the project. More on how to use it is in Readme.md.
Since I have very low experience with Python and the repo itself had little documentation on how to deploy it, I tried to put together parts from dokku repo and this repo to make this as good as possible. I would recommend to inspect and apply fixes.
Along with docker I fixed a few bugs that were causing exceptions.
I hope you find this as a valuable addition to the repo, and I hope this is something that can bring it back to life.
Best regards,
Milan

Coverage decreased (-0.02%) to 28.039% when pulling cdcb34d972a5879830cd62256c3f97ae452ecda4 on viewpnt1:docker-support into ab1ae7f1280273d42fdaf9300db5b48148c10b08 on ActiDoo:develop.

@viewpnt1
Hello Milan,
thank you for your pull request. Most of the changes look fine, I have just a few points to discuss:

If we are sticking with alpine I'd like to pin the version to 3.10. I had a bad experience with alpine making incompatible changes during updates. Otherwise we could also use buster.
How are you using the custom_tasks folder? If you want to include custom code in your application, you can use the pyramid.includes setting in the ini file.

pyramid.includes =
    pyramid_debugtoolbar
    pyramid_tm
    mygengine_ext

With mygengine_ext as
def includeme(config):
    init_extension()

def init_extension():
    from gengine.app.registries import get_admin_extension_registry
    get_admin_extension_registry().register("mygengine_ext.admin_extension", admin_extension)

    from gengine.app.registries import get_task_registration_points_registry
    get_task_registration_points_registry().register("mygengine_ext.task_registration_point", task_registration_point)

def admin_extension(adminapp, admin):
   pass

def task_registration_point():
   pass

In the production nginx config, you have

location = / {
    rewrite ^/(.*)$ /admin/$1 last;
}

I think this is redirecting all requests to /admin/, which will probably break the /api calls, right?

If you are going to contribute more, I'd love to talk to you about your use-case and the project roadmap (we are also planning to integrate some new features next year)

Regards
Marcel

Hello Marcel,
Thanks for the response. To answer your questions:

Alpine version pinning - I totally agree. Pining versions is always a safer approach.

Custom tasks might not be necessary. I only added it so I can add my own @EngineTask definitions somewhere, and keep demo task and its respective files unchanged. By doing so I believed the concept will be least intrusive to the existing codebase. Although, I would prefer your solution if it doesn't require config.include('gengine.custom_tasks') in gengine/init.py.

This route will redirect only exact location "/" to admin, although now that you mention it, it is kinda weird. I got the whole config made out of your dokku config, but haven't given it much thought.

My project roadmap:
I work on a web app that along core features offers gamification concept to the users. So I found your repo and decided to give it a try. Along the way I adjusted/tweaked/fixed not-so-small number of features I thought weren't working as intended (but I'm actually not sure for all of them) - for example route add_or_update_subject didn't really offer possibility to add subject, only update - stuff like that. Also, there might be some opinionated fixes I applied that were necessary for my project's use cases, so that would need to be discussed and reviewed or discarded. I would be happy to contribute parts that are suitable.
The project is going live by the end of year so, if anything, I will be applying bug fixes regularly in the near future.

Best regards,
Milan


Deprecate scope for discovery list for self help 2024 03 01 preview

This checklist is used to make sure that common guidelines for a pull request are followed.
Related command

General Guidelines

[x] Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)

[x] Have you run python scripts/ci/test_index.py -q locally? (pip install wheel==0.30.0 required)

[ ] My extension version conforms to the Extension version schema

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

Thank you for your contribution! We will review the pull request and get back to you soon.

Utilizing breaking change window to remove scope parameter from discovery

AAZ Repo PR:
https://github.com/Azure/aaz/pull/483

We are deprecating the "scope" field for "az self-help discovery-solution list" command. We were previously advised to add a deprecation message for that parameter and remove the parameter altogether during the breaking change window. The PRs involved here take care of removing the parameters, while the previous PRs that are already merged have included the warning message regarding upcoming deprecation in version 0.4.0.

AAZ Repo:
https://github.com/Azure/aaz/pull/484


Mic feature not working

Describe the bug
Mic still not working in rooms, moderators receive my rais hand and Iam not receive the request to accept it and join to communicate with them, kindly advice thank you.

Device: [e.g. oppo reno 2f]
OS: [e.g. Android 10]
Version [e.g. 1.0.8]

Why can you still use, most people's have an error ！


Bumping golang.org/x/net to v0.8.0

What this PR does / why we need it:

Bumping golang.org/x/net to v0.8.0

Which issue(s) this PR fixes:

Fixes #
Does this PR introduce a user-facing change?

Additional Notes for your reviewer:
Review Checklist:

[ ] Follows the developer guidelines
[ ] Relevant tests are added or updated
[ ] Relevant docs in this repo added or updated
[ ] Relevant carvel.dev docs added or updated in a separate PR and there's
a link to that PR
[ ] Code is at least as readable and maintainable as it was before this
change

Additional documentation e.g., Proposal, usage docs, etc.:

LGTM


Top app bar position relative?

What vue-mdc-adapter Version are you using?

0.18.2

What OS are you using?

Windows 10 home

What is the expected behavior?

Position should be relative instead of fixed with fixed being set as false.

What is the actual behavior?

The component appears to be fixed no matter what.

This might be because the wrong underlying foundation is being used due to a bug in the vue-mdc-adapter top-app-bar implementation.
It is supposed to select between three foundations like this
    this.foundation = this.short
      ? new MDCShortTopAppBarFoundation(adapter)
      : this.fixed
        ? new MDCFixedTopAppBarFoundation(adapter)
        : new MDCTopAppBarFoundation(adapter)

but currently doesn't.


AttributeError: module diffusers has no attribute LCMScheduler

In a v1.6.0 Automatic1111 installation, I followed the installation steps for this extension and an attempt to generate an image in the LCM extension yields the following:
The config attributes {'original_inference_steps': 50} were passed to LCMScheduler, but are not expected and will be ignored. Please verify your scheduler_config.json configuration file.
The config attributes {'requires_safety_checker': True} were passed to LatentConsistencyModelPipeline, but are not expected and will be ignored. Please verify your model_index.json configuration file.
Keyword arguments {'requires_safety_checker': True} are not expected by LatentConsistencyModelPipeline and will be ignored.
Traceback (most recent call last):
  File "/home/m/stable-diffusion-webui/venv/lib/python3.10/site-packages/gradio/routes.py", line 488, in run_predict
    output = await app.get_blocks().process_api(
  File "/home/m/stable-diffusion-webui/venv/lib/python3.10/site-packages/gradio/blocks.py", line 1431, in process_api
    result = await self.call_function(
  File "/home/m/stable-diffusion-webui/venv/lib/python3.10/site-packages/gradio/blocks.py", line 1103, in call_function
    prediction = await anyio.to_thread.run_sync(
  File "/home/m/stable-diffusion-webui/venv/lib/python3.10/site-packages/anyio/to_thread.py", line 33, in run_sync
    return await get_asynclib().run_sync_in_worker_thread(
  File "/home/m/stable-diffusion-webui/venv/lib/python3.10/site-packages/anyio/_backends/_asyncio.py", line 877, in run_sync_in_worker_thread
    return await future
  File "/home/m/stable-diffusion-webui/venv/lib/python3.10/site-packages/anyio/_backends/_asyncio.py", line 807, in run
    result = context.run(func, *args)
  File "/home/m/stable-diffusion-webui/venv/lib/python3.10/site-packages/gradio/utils.py", line 707, in wrapper
    response = f(*args, **kwargs)
  File "/home/m/stable-diffusion-webui/venv/lib/python3.10/site-packages/gradio/utils.py", line 707, in wrapper
    response = f(*args, **kwargs)
  File "/home/m/stable-diffusion-webui/extensions/sd-webui-lcm/scripts/main.py", line 99, in generate
    pipe = LatentConsistencyModelPipeline.from_pretrained(
  File "/home/m/stable-diffusion-webui/venv/lib/python3.10/site-packages/diffusers/pipelines/pipeline_utils.py", line 1098, in from_pretrained
    maybe_raise_or_warn(
  File "/home/m/stable-diffusion-webui/venv/lib/python3.10/site-packages/diffusers/pipelines/pipeline_utils.py", line 282, in maybe_raise_or_warn
    class_obj = getattr(library, class_name)
  File "/home/m/stable-diffusion-webui/venv/lib/python3.10/site-packages/diffusers/utils/import_utils.py", line 677, in __getattr__
    raise AttributeError(f"module {self.__name__} has no attribute {name}")
AttributeError: module diffusers has no attribute LCMScheduler

The following resolved the problem for me.  Perhaps you want to update your README.md?
$ source bin/activate
$ pip install --upgrade diffusers transformers accelerate peft

Yes. Update the diffursers to the latest version, that will fix the problem!


Filter Deployment logs

Is your feature request related to a problem? Please describe.
Cyclops shows logs for running pods here. Scope of the task is to add a search for logs that you can use to filter out the log you are looking for.
Describe the solution you'd like
Add a search bar to the top of the logs modal and show all logs that contain the search input.

Hi @petar-cvit would the search box that comes with React Ace would fit your need?

@doncicuto Actually yeah, I actually searched for something out of the box like this, but didn't find it. Exactly what we need. Could you open a PR for it?

Perfect @petar-cvit you can assign the issue to me then and do you want the Search button to show for every React Ace editor or just for the one for Deployment logs?

Just the one with the logs will do

Hey @doncicuto, do you have any updates on the issue?

Next week I'll prepare a PR, sorry for the delay, busy summer!

@quest-bot embark

@petar-cvit I'm taking this one!

Hey @petar-cvit a lot has changed since you opened this issue, could you please let me know where this deployment code is located at, the reference you mentioned in the description aren't there any more in the main.

Would something like this do the job @petar-cvit ? I just tested it locally.
https://github.com/user-attachments/assets/eb3de279-c195-482c-af0b-8efd85c1e1fd

@Sheikh-Abubaker thanks! Can you check if we can implement something similar to https://github.com/cyclops-ui/cyclops/issues/341#issuecomment-2205549708 since it looks more clean in the UI and seems like it supports regex and casing in the search?

@quest-bot loot #341


Added bme688 sensor to Thingy:53 i2c1 instance

Updated the thingy:53 device tree to include the bme688 sensor on the i2c1 instance.
Signed-off-by: Ylvisaker<EMAIL_ADDRESS>

Incorrect zephyr version used as base, will open a new PR.

Incorrect zephyr version used as base, will open a new PR.

Please don't do this. Instead you should update PRs by rebasing and force pushing your branch


feat!: shades, fallback previews, drop tailwind < 3.1

@nekowinston what was the reason for closing this?

I retired from Catppuccin and stopped contributing to the project.


🧙 [Shard Request] -  #3

Shard Name
Eternium
Shard admins
SohoRoom#4121
Shard ip/port
<IP_ADDRESS>,2593
Shard description
More than 800+ quests, a dungeon raid, a rich history of the world, more than 2k artifacts, Dota style PVP, 8 races, 6 classes (WoW Style), a corrected Sosaria map with completely transparent water in the world. 50 levels, free PVP zone

Shard type
PvM, PvP
Shard era
ML
What customisations does your shard currently have?
Assets (modified UOP/MUL etc)
Link to shard files
https://uoem.ru/
Version used for the basis of you shard files
<IP_ADDRESS>
List of modified UO assets
Cliloc.rus, Cliolic.enu, anim,anim1,anim2,anim3,anim4,anim5 (all Anim idx)
map0.mul map1.mul map2.mul map3.mul map4.mul
tiledata.mul
radarcol.mul
statics0.mul (all 1 2 3 4)
staidx0.mul (all 1 2 3 4)
stadif* (all 0 1 2 3 4)
Gumpart.mul
art.mul
lightidx.mul
multi.mul
speech.mul
skills.mul
Gumpidx.mul
Hues.mul
Prof.txt

ClassicUO Customisations
no
Comments/Requests
DISCORD LINK SHARD:      https://discord.gg/WVnyPBbSeF
http://<IP_ADDRESS>:8080/
Code of Conduct

[X] I agree to be patient and acknowledge that the CUO devs are working on their own free time to help me.

LINK FILES ---- RAR
https://uoem.ru/download/Ultima%20Online%20Eternium.rar

Update: Live now in the testing section as we work on fixing a packet bug causing crash on login.

Fixed

Good afternoon, I went in and checked that the game stopped hanging almost everywhere, but not everywhere, when you approach the market it freezes 100%, I tried to delete scripts associated with the tooltip update and message over the seller's head did not help * (I also identified a number of problems with animation and cliolic prepared files for replacement so that it correctly displayed animals. Thanks for the feedback.! slight_smile https://disk.yandex.ru/d/TjbVphecuUvKXQ

Upd up:
I think these files should close the errors
List:
Sound.def
soundidx.mul
Body.def
Bodyconv.def
Cliloc.enu
Cliloc.rus
https://disk.yandex.ru/d/kFKpKZ9c6e7Oog
https://disk.yandex.ru/d/TjbVphecuUvKXQ


[WIP][Post] AR VIR in Eigen

WIP, I have some sketch docs that show the workflows that are current in <li>s. Think this won't work without diagrams.

Got the ability to do a mouse over process thingy to show visually how all this works

OK, this is good for review


Handful of issues on Remaining Games list

Noticed a few games that could be removed from Remaining Games

"Toki" is already ranked
"Indiana Jones and the Temple of Doom" was already played.  The licensed and unlicensed versions of that game are identical.
"Donkey Kong Classics" is a combination cart
"Sesame Street: A-B-C/1-2-3" is also a combo cart

Fixed. Thanks @ianschmidt83 !


Week1 assignment2 instruction  not clearly on coursera website

on coursera
week1 assignment2 instruction doc recommend you set up ssh connection and generate key pair first.
then it instruct you
git remote add origin < url >

But! once you configure the ssh, the < url > were not just you repository https address anymore.
instead you need change from
origin	https://github.com/cu-ecen-aeld/assignment-2-123zmz123 (fetch)
origin	https://github.com/cu-ecen-aeld/assignment-2-123zmz123 (push)

to
origin	git@github.com:cu-ecen-aeld/assignment-2-123zmz123.git (fetch)
origin	git@github.com:cu-ecen-aeld/assignment-2-123zmz123.git (push)

at my pc I can not push success  when I use https address. (it report (fatal: Authentication failed for 'https://github.com/cu-ecen-aeld/assignment-2-123zmz123/'))
seems like instruction doc on coursera were not very clear.

Hi @123zmz123 thanks for the feedback.  I've updated the assignment 2 instructions to reference https://github.com/cu-ecen-aeld/aesd-assignments/wiki/Setting-up-SSH-Access-To-your-Repo


core/iwasm: Support mapped file system access on non-libuv WASI

Similar to the for-loop above it, this patch just adds another one iterating through map_dir with the ability to set the guest path to a different target. It follows the format "guest-path::host-path".

@wenyongh thanks for looking into it, I intend to add the necessary changes soon. I'm currently in the process of another WAMR-based IDE release (this time on macOS) hence it might take a few days until I get to it.
I'm embedding WAMR inside the IDE using a plugin mechanism, and it loads specific implementations of the same interface wherever it sees fit (debugger-enabled interpreter build during debugging, "fast interp" or maybe even JIT/AOT otherwise).
I have only tested the current set of changes inside this confined environment, but I don't have a libuv-enabled build planned atm (no idea of differences and benefits). What would you like to see happen with libuv, doesn't it already pass down the map_dir list to libuv for it to do the right thing?

@wenyongh thanks for looking into it, I intend to add the necessary changes soon. I'm currently in the process of another WAMR-based IDE release (this time on macOS) hence it might take a few days until I get to it.
I'm embedding WAMR inside the IDE using a plugin mechanism, and it loads specific implementations of the same interface wherever it sees fit (debugger-enabled interpreter build during debugging, "fast interp" or maybe even JIT/AOT otherwise). I have only tested the current set of changes inside this confined environment, but I don't have a libuv-enabled build planned atm (no idea of differences and benefits). What would you like to see happen with libuv, doesn't it already pass down the map_dir list to libuv for it to do the right thing?

Thanks, I checked the code of init uvwasi, it seems that map_dir_list[i] is used as a mapped dir of dir_list[i], the format of map_dir_list[i] is not <guest_dir::host_dir>:
https://github.com/bytecodealliance/wasm-micro-runtime/blob/main/core/iwasm/common/wasm_runtime_common.c#L3182-L3183
If to apply similar changes like this PR, seems the init_options.preopenc should be dir_count + map_dir_count, and for init_options.preopens[0 to dir_count-1], its real_path and mapped_path are the same (=dir_list[i]); for init_options.preopens[dir_count to dir_count+map_dir_count-1], its real_path is host_dir in map_dir_list[i], and itsmapped_path is guest_dir in map_dir_list[i].
I think you can just leave it unchanged, we can open another PR to apply similar changes for libc-uvwasi after this PR is merged.

In case this doesn't cause any test build failures I consider this completed.

Fixed up minor issues and tested the functionality with this Python 3 build: https://github.com/vmware-labs/webassembly-language-runtimes/releases/tag/python%2F3.11.4%2B20230714-11be424
Using: ./iwasm --map-dir=/::<path-to-python-root>


add Multirate Infinitisemal Step (MIS) methods

I was able to reuse the existing multirate infrastructure, which is really nice, so this lets you add the various MIS methods as the outer (slower) solvers.
Fixes #10.

Also added Wicker Skamarock methods.


PullRefreshIndicator in LazyColumn on Desktop

Already checked:

https://stackoverflow.com/questions/70774033/compose-for-desktop-lazyrow-lazycolumn-not-scrolling-with-mouse-click
https://github.com/JetBrains/compose-multiplatform/issues/1555

Just want to use androidx.compose.material.pullrefresh on Desktop and Browser.
But I cannot pull down the LazyColumn on Desktop.
This Composable works on Android & iOS.

Duplicate of #4436

it will not trigger this logic because it explicitly waits dragging, but scrollable modifier disables dragging for mouse.

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


ruff 0.0.237

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Closes https://github.com/conda-forge/ruff-feedstock/pull/84
Closes https://github.com/conda-forge/ruff-feedstock/pull/87

@conda-forge-admin, please rerender

+ /Users/runner/miniforge3/conda-bld/ruff_1674973256938/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_plac/bin/python -m pip install ruff-0.0.237-py3-none-macosx_11_0_arm64.whl
ERROR: ruff-0.0.237-py3-none-macosx_11_0_arm64.whl is not a supported wheel on this platform.

It's a bit odd that this works fine with Python 3.9 & 3.10 but fails on 3.8 & 3.11.


Added /auth/user/email to get user info via email

Ref #1

@WhatTheFar Please check if it's okay please deploy new version 😁 🙏

This is not gonna work, there is a GET /user/:username already and GET /user/:email will not be called.
I will figure it out by tomorrow how to dynamically query the user with a custom model.

🎉 🎉


Spike: can the Public VM native/circuit use Brillig VM/interpreter

Hopefully yes eventually, but it will require changes to the core Brillig instruction set. And we'll need to figure out how gas will be tracked per-instruction :thinking:


Allow alphanumeric PIN for Portal Twenty-Two

Portal 22 device supports an alphanumeric PIN code (Pair Code) for better security.
Please allow that within Bitcoin Keeper. Currently, it's only numeric.
Optionally we can store this PIN encrypted in the app and allow biometric unlock for Twenty-Two as well.

Verified this issue on keeper app v 1.3.2(436)


issues with chromedriver.exe

I cannot for the love of the lord get chromedriver.exe to load anywhere.
Tried putting in path, probably copied it into every folder i know, yet, i can't get the script to work because it says chromedriver.exe has to be on PATH.
Any ideas?

Hi @emilianosalatino,
As long as the 'main.py' and 'chromedriver.exe' exist on the same path, there shouldn't be any problems. If the problem still exists, provide more information like where is 'main.py' and 'chromedriver.exe' are present and the error message


Strip out HTML comments

Hey @tompazourek, thank you for this thing, absolutely love the idea of compile-time transformation.
Though, do you think it's possible to add an ability to remove <!-- HTML comments --> from the outcome as well? I can work on it if you don't want or don't have time - would appreciate if you then accept my pull request and redeploy the lib in Nuget.
Also it would be great if this thing was somehow configurable so that a user can choose whether they want to strip out comments or not, and maybe they don't want newlines at all so they can be completely removed (or replaced with a single space so that nothing breaks). Do you know if any configuration may be read from web.config at the compile time?
Thanks!
Andrew K.

Actually just read your comment in #4 and it actually makes sense that no one should use HTML comments in CSHTML files so I may just replace all existing comments with Razor ones and then force all the developers to only use those. Also there's a place where we use <!--[if mso]> which shouldn't be stripped which makes the logic even more complicated, so probably can close the ticket. Stripping newlines can still be useful though if we have a way to configure the minification.

Hey @andkorsh, the newlines might actually be significant, so we cannot just easily strip them out.
Consider the following HTML:
Hello
world

It gets rendered as Hello world in the browser. If we'd strip newlines, it'd get rendered as Helloworld.
You might argue that a newline is two bytes (\r\n) where it can be shortened just to a space   (one byte). The library actually shortens the Windows-like \r\n endings to Unix-like \n endings, so the result is still one byte. That means that the minified result is as short as it would be if we replaced newlines with spaces, but it's closer to the original.
What the library also tries to do is to minify consecutive sequence of whitespace or newlines to a single byte (either a \n or  ). It's still not working 100% perfectly (I am not completely sure why), but a lot of the whitespace gets stripped out.


1.0.6 not available through composer

It seems that the newest version (1.0.6) is not available through composer.
Is it possible to make the newest release available there too?

@Adaphath just releases 1.0.7.
This version can now be downloaded again without problems via composer.


Graph

Ooo add a graph in the terminal right next to or below the info when monitoring

Yess!!

Good idea, that would be awesome


ci/test: e2e issue

add this for e2e testing for ci

/close
will re open for next feat/enhancement


Revert "Merge pull request #2956 from muraee/fix-nodepool-rolling-upgrade

DNM: debugging HO read budget overrun

/test e2e-aws

/test e2e-aws-metrics

/test e2e-aws-metrics

closing in favor of https://github.com/openshift/hypershift/pull/2988
/close


Add GitHub status formatter

Solves #121.
The formatter creates one commit status per runner with information about issue counts.
Issues in status description are ordered by severity.
By default all Pronto Messages with level :warning or higher are considered a failed state for GitHub status (red status).
GithubStatusFormatter allows to override this behaviour by accepting :level_mapping option in the initializer (which is not supported when running the formatter directly with command line, e.g. pronto run -f github_pr_status).
There is no README info yet but I'll provide it once we agree on the interface and the way this formatter should work.
The result of running Pronto with the new formatter:

@mmozuras Please let me know whether the PR adheres to the project conventions.
I got some RuboCop line to long warnings in specs, should I fix those?
Also I'm going to squash refactoring commits before merging.

@mknapik amazing, thank you, honestly 👏 😄
Just address a couple of comments, add info to README and CHANGELOG and it's gonna get merged! 😄

@mmozuras Apart from adhering to your remarks I've also did some refactoring.
Additionally I renamed formatter cli name from github_pr_status to github_status.
Let me know what do you think.
I haven't squashed everything yet so you can review the changes easily.

@mknapik looks good, just change that one last thing 😄

@mknapik @mmozuras would it be too much to ask to have the status contexts formatted more like how other services format them? i.e. instead of "Pronto::Rubocop", something like "pronto/rubocop" or "lint/pronto/rubocop" to be more similar to how Travis CI uses "continuous-integration/travis-ci/pr"

@aergonaut Actually I had no idea what was the convention.
The docs use continuous-integration/jenkins and security/brakeman as examples.
I guess I can change it to pronto/$runner.

@mmozuras @aergonaut Regarding context names:
Do you think a simple function like runner.class.downcase.gsub('::', '/') is enough or should I implement something like ActiveSupport::Inflector#underscore?
The difference:

Pronto Runner
downcase
underscore

Pronto::Brakeman
pronto/brakeman
pronto/brakeman

Pronto::RailsBestPractices
pronto/railsbestpractices
pronto/rails_best_practices

@mknapik looks good to me 😄

@mknapik looks good, amazing contribution 👍 🙇

@mmozuras Thanks!
The gem is well written and contributing was a pleasure!

This is awesome. Was trying to make it work using version 0.6.0, then realized that this isn't released yet 😉 .
I look forward to using it in the next version bump.


test node 18/20, minimum version 18.19.0

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 55.85%. Comparing base (31b978f) to head (b7e1e27).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1517   +/-   ##
=======================================
  Coverage   55.85%   55.85%
=======================================
  Files         105      105
  Lines        4510     4510
  Branches      689      689
=======================================
  Hits         2519     2519
  Misses       1437     1437
  Partials      554      554

Flag
Coverage Δ

caliper-core
44.94% <ø> (ø)

caliper-fabric
80.46% <ø> (ø)

generator-caliper
83.62% <ø> (ø)

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


other sources

May have to use cve.org, or other as Ubuntu doesn't have all cve records
ex: https://ubuntu.com/security/CVE-2024-10824

And done! Much better.


Remove docker installation steps from Administer Cluster docs

Part of #30970

/assign @celestehorgan

@sftim
i did not check the diff here properly.
this PR is not needed after my change in https://github.com/kubernetes/website/pull/31309

also looks like @marosset from SIG Windows just LGTM-ed my PR.
follow ups on formatting can be done separately.

/hold

Since

this PR is not needed after @neolit123's change in #31309, and
follow ups on formatting can be done separately,

and also Netlify build fails,
and author is not responding to comments or feedback in 2 or more weeks,
it might be better to close this PR. 😊
/close


GH-1079 INLINE_BLANK_NODES-lose-bnode

Signed-off-by: Mirr99<EMAIL_ADDRESS>GitHub issue resolved: #1079
Briefly describe the changes proposed in this PR:

Added a function call to queueBlankStatements(SubjectInContext key) to ensure that the statements related to a repeated BNode are also written.

Please let me know if a test case is required.

Thanks for looking into this! A regression test would be great, yes. The most logical place, I think, is to add a test case to the ArrangedWriterTest.

I have added a test case testWriteRepeatedInlineBlankNode(). Please let me know if this test case will be suitable.


jupyterhub ssh have support for ssh keys instead of just password based

This issue need discussion and potentially waiting on jupyterhub or jupyterlab-ssh to gain features:

jupyterhub needs some way to store public metadata about users in api
jupyterlab-ssh needs to have more flexible authentication


[EMB-347] Add ember_auth_register waffle flag

Purpose
Add waffle flag for toggling sign up page at /register between legacy page and Ember app.
Changes

Add migration to add ember_auth_register waffle flag
Check ember_auth_register waffle flag for auth_register in framework/auth/views

QA Notes
Make sure Ember redesigned sign up page is served when the ember_auth_register waffle flag is enabled (and legacy sign up page when it's not).
Documentation
Are we documenting these anywhere?
Side Effects
Shouldn't be.
Ticket
https://openscience.atlassian.net/browse/EMB-347

@sloria, look good?


Allow repositioning while scrolling

It would be nice if we repositioned the content on scroll so it follows the toggle.

Looks like we already do this with absolute positioning!

👉 #31


Error deploying to Azure Function App service

Hi,
I am using the functions-action v1 GitHub action to deploy to the Azure Function App service, and am getting the following error -
Step Details –

name: 'Run Azure Functions Action'
uses: Azure/functions-action@v1
id: fa
with:
app-name: ${{ env.AZURE_FUNCTIONAPP_NAME }}
package: ${{ env.AZURE_FUNCTIONAPP_PACKAGE_PATH }}
publish-profile: ${{ env.AZURE_FUNCTIONAPP_PUBLISH_PROFILE }}
scm-do-build-during-deployment: true
enable-oryx-build: true

Error Message -
Generating summary of Oryx build
Deployment Log file does not exist in /tmp/oryx-build.log
The logfile at /tmp/oryx-build.log is empty. Unable to fetch the summary of build
Triggering recycle (preview mode disabled).
Linux Consumption plan has a 1.5 GB memory limit on a remote build container.
To check our service limit, please visit https://docs.microsoft.com/en-us/azure/azure-functions/functions-scale#service-limits
Writing the artifacts to a squashfs file
Parallel mksquashfs: Using 1 processor
Creating 4.0 filesystem on /home/site/artifacts/functionappartifact.squashfs, block size 131072.
[==========-                                                   ] 1200/6677  17%
[================|                                             ] 1800/6677  26%
[=============================================================|] 6677/6677 100%
Exportable Squashfs 4.0 filesystem, gzip compressed, data block size 131072
compressed data, compressed metadata, compressed fragments, compressed xattrs
duplicates are removed
Filesystem size 42239.93 Kbytes (41.25 Mbytes)
30.36% of uncompressed filesystem size (139119.65 Kbytes)
Inode table size 65659 bytes (64.12 Kbytes)
28.87% of uncompressed inode table size (227400 bytes)
Directory table size 66076 bytes (64.53 Kbytes)
35.44% of uncompressed directory table size (186420 bytes)
Number of duplicate files found 456
Number of inodes 7018
Number of files 6214
Number of fragments 507
Number of symbolic links  0
Number of device nodes 0
Number of fifo nodes 0
Number of socket nodes 0
Number of directories 804
Number of ids (unique uids + gids) 1
Number of uids 1
root (0)
Number of gids 1
root (0)
Creating placeholder blob for linux consumption function app...
Malformed SCM_RUN_FROM_PACKAGE when uploading built content.
Generating summary of Oryx build
Deployment Log file does not exist in /tmp/oryx-build.log
The logfile at /tmp/oryx-build.log is empty. Unable to fetch the summary of build
Deployment Failed. deployer = GITHUB_ZIP_DEPLOY deploymentPath = Functions App ZipDeploy. Extract zip. Remote build.
Error: Failed to deploy web package to App Service.
Error: Execution Exception (state: PublishContent) (step: Invocation)
Error:   When request Azure resource at PublishContent, zipDeploy : Failed to use /home/runner/work/_temp/temp_web_package_031966780744437484.zip as ZipDeploy content
Error:     Package deployment using ZIP Deploy failed. Refer logs for more details.
Error: Deployment Failed!

The app is active and running on Azure. Some details are as follows:
Azure Configuration
Location: West Europe
Operating System: Linux
Runtime Version: <IP_ADDRESS>
Status: Ready
Zone Redundant: Disabled
ENABLE_ORYX_BUILD: 1
FUNCTIONS_EXTENSION_VERSION: ~4
FUNCTIONS_WORKER_RUNTIME: python
SCM_DO_BUILD_DURING_DEPLOYMENT: 1
App Service authentication: Enabled
Restrict access: Require authentication
Unauthenticated requests: Return HTTP 401 Unauthorized
Token store: Enabled

And the repo I'm trying to deploy has roughly the following file structure:
GitHub Repo
├── .github
│ ├── workflows
│ │ └── config.yml
├── bin
│ ├── dir1
│ │ ├── file1.py
│ │ ├── file2.py
│ │ └── init.py
│ ├── dir2
│ │ ├── file1.py
│ │ ├── file2.py
│ │ └── init.py
│ └── init.py
├── host.json
├── local.settings.json
├── requirements.txt
├── otherfile1
└── otherfile2

@will300 Is app setting AzureWebJobsStorage set with correct connection string for your function app?

@patelchandni same issue

after adding the AzureWebJobsStorage  config with the blob conn string are running ok but with nothing to see on the azure portal:

@dvegaeygds Do you see AzureWebJobsStorage app setting in the portal? After adding it, please restart your app and then redeploy.

@patelchandni I did, but still doesn't work I re created the function and now is ok. weird?

I got the same error, in my case the app wasn't running. After starting the App again the deployment was successful.


Fix flaky getdate tests (#2985)

Description
This commit fixes flaky getdate tests by validating for datetime difference spillover.
Cherry-picked: https://github.com/babelfish-for-postgresql/babelfish_extensions/pull/2985
Authored-by: Anikait Agrawal<EMAIL_ADDRESS>Signed-off-by: Anikait Agrawal<EMAIL_ADDRESS>Test Scenarios Covered

Use case based -   NA

Boundary conditions - NA

Arbitrary inputs - NA

Negative test cases - NA

Minor version upgrade tests - NA

Major version upgrade tests - NA

Performance tests - NA

Tooling impact - NA

Memory tests - NA

Client tests - NA

Check List

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

By submitting this pull request, I confirm that my contribution is under the terms of the Apache 2.0 and PostgreSQL licenses, and grant any person obtaining a copy of the contribution permission to relicense all or a portion of my contribution to the PostgreSQL License solely to contribute all or a portion of my contribution to the PostgreSQL open source project.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>4
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.003%) to 74.451%

Totals

Change from base Build<PHONE_NUMBER>1:
0.003%

Covered Lines:
44935

Relevant Lines:
60355

💛  - Coveralls


Create Dashboard

https://aws.amazon.com/blogs/security/deploy-dashboard-for-aws-waf-minimal-effort/

Done in release https://github.com/globaldatanet/aws-firewall-factory/releases/tag/3.0.1


there is no such menu fixed assets>Adjustment

[Enter feedback here]
Though I can see following part in this page, I cannot even find Adjustment menu under Fixed assets menu. please show us correct menu.
Impairment recognition test
The primary page is Fixed assets > Adjustment. The sub-menus different, depending on whether Method I or Method II is applied. The first step of impairment is the recognition test. Future cash flow that isn't discounted is used as a standard that the carrying amount (or net book value) is compared against. If this future cash flow isn't enough to cover the net book value of the asset, the impairment measurement must recognize the journal amount.

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

ID: 3e96bc99-19d3-5510-c13e-085778239910
Version Independent ID: 7eceeba8-fa13-5b66-5a1e-4340f4e8496c
Content: Fixed asset impairment accounting on cash-generating units for Japan - Finance | Dynamics 365
Content Source: articles/finance/localizations/apac-jpn-impairment-accounting-cash-generating-unit.md
Service: dynamics-ax-applications
Product: ****
Technology: ****
GitHub Login: @yijialuan
Microsoft Alias: riluan

@Izmiyash, thanks for the feedback.
@yijialuan, please review.

Bug 527680 logged to track this issue.

Just assign the bug to @EricWangChen who is the owner now

Content deliverable 529674 created