Convert mocha hooks

Forgot to do this in #3449... Mocha hooks need to be added via root hook plugins instead of somewhere in the test files, specifically because each file is treated as individual test and might run in a separate thread in parallel mode.
This didn't change the test results, but best to do the beforeEach cleanup anyway.

Good spot, thanks Ingvar, looks like there's a couple of linter failures to deal with then we can get this merged.
semistandard: Run `semistandard --fix` to automatically fix some problems.
  /__w/sharp/sharp/test/beforeEach.js:13:13: Missing space before function parentheses.
  /__w/sharp/sharp/test/beforeEach.js:19:12: Missing space before function parentheses.
  /__w/sharp/sharp/test/beforeEach.js:23:4: Unexpected trailing comma.

My bad, I was sleepy and didn't run lints. Will fix up once near the computer.

Fixed.
Btw, I wonder why Github says "First-time contributors need a maintainer to approve running workflows." even though I already sent #3449 so I'm not a first-time contributor 🤔

Also related: I didn't want to change it in case I don't understand it, but I find this line weird:
const concurrency = process.env.VIPS_CONCURRENCY ||
  (libcFamily === detectLibc.MUSL || process.arch === 'arm' ? 1 : undefined);

My understanding is that passing undefined to sharp.concurrency() is no-op, as it will only act as a getter instead of a setter in that case. Is that intentional? Or was this meant to be 0 (zero) instead, to reset concurrency to the number of CPU cores before each test?

First-time contributors...

This is a mis-labelling by GitHub, as the repo was originally configured to require approval for "outside collaborators". The macOS ARM64 CI environment runs on an M1 Mac mini kindly provided by MacStadium and there was a spate of cr*pto-mining spam PRs affecting GitHub when I was first setting this up that I didn't want running on hardware I am responsible for. It's probably time to relax this setting, which I've now done.

I find this line weird

Yeah, sorry about all the weird historic internals of sharp you're having to look at, I think this was to allow the tests to run on various hardware/conditions/CI providers. These can definitely be simplified e.g. look for a single environment variable.

Simplified via https://github.com/lovell/sharp/commit/4490a93430e5e85b03ed122bbde487c319e3acef - thank you for highlighting this.


Define page segmentation mode for the resource as a whole

pagesegmode values are:
  0 = Orientation and script detection (OSD) only.
  1 = Automatic page segmentation with OSD.
  2 = Automatic page segmentation, but no OSD, or OCR
  3 = Fully automatic page segmentation, but no OSD. (Default)
  4 = Assume a single column of text of variable sizes.
  5 = Assume a single uniform block of vertically aligned text.
  6 = Assume a single uniform block of text.
  7 = Treat the image as a single text line.
  8 = Treat the image as a single word.
  9 = Treat the image as a single word in a circle.
  10 = Treat the image as a single character.

Done in the file manager.


Update opencensus-exporter-trace-ocagent to 0.30.0

Updates io.opencensus:opencensus-exporter-trace-ocagent from 0.28.3 to 0.30.0.
GitHub Release Notes - Changelog - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.opencensus", artifactId = "opencensus-exporter-trace-ocagent" } ]

labels: library-update, early-semver-major, semver-spec-minor, commit-count:1

Superseded by #502.


YodaStyleFixer with function calls

The PHP version you are using:
$ php -v
PHP 7.1.8 (cli) (built: Aug 16 2017 17:19:54) ( NTS )
Copyright (c) 1997-2017 The PHP Group
Zend Engine v3.1.0, Copyright (c) 1998-2017 Zend Technologies
    with Xdebug v2.5.5, Copyright (c) 2002-2017, by Derick Rethans

PHP CS Fixer version you are using:
$ php-cs-fixer -v
PHP CS Fixer 2.6.0 Local Tavern by Fabien Potencier and Dariusz Ruminski

The command you use to run the fixer (and the configuration file you are using if any).
$ php-cs-fixer fix --verbose --rules='{"yoda_style":true}' [PATH_TO_FILE]

Before Fixer:
<?php
$foo = 123;
$bar = $foo === count($foo);

After Fixer (actually):
<?php
$foo = 123;
$bar = $foo === count($foo);

After Fixer (expected):
<?php
$foo = 123;
$bar = count($foo) === $foo;

Hi and thanks for the report.
The behavior of the YodaStyle fixer you observed is intended. It was discussed in various PR's and issue, both in this repo and over at Symfony.
There are requests for adding more options to this fixer to change its behavior, for example a similar discussion is going on here https://github.com/FriendsOfPHP/PHP-CS-Fixer/issues/3121
Please let us know if you've time to help out!

@SpacePossum Are you sure this is expected? I think the fixer is supposed to move the variable from the left side to the right side of the operator.
#3674 Fixes this issue.

Pretty sure, the fixer is the result of a very long discussion over multiple PR's and reverts, 3121 is a nice start/end on the process. I'm not in any way saying it is the only good way the fixer could work and I would welcome new options, however we should be careful not to change the default (and prevent doing the whole specification discussion again ;))
Part of the confusion and the long discussion also resulted in less good naming in the current implementation, such as the method name isVariable used to detect that, but ended up detected candidates (which are not always variables only).

(https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/3505 is however a proof of a real implementation and the related issue is a valid bug report)

If I understand correctly, you are referring to https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/693#discussion_r19977208, right?
I realize that the current behavior of the fixer is pretty opaque, I thought it was following these rules to apply Yoda style:

if left side is a variable and right side is not, move the variable to the right (initial purpose of Yoda style);
else if there are no variable but right side is a simple scalar constant, move it to the left (to ease readability, as most developers that apply Yoda style probably expect the scalar value to be on left side).

I submitted #3674 with those rules in mind. IMO the first rule is mandatory and should not have exception. I think this is the behavior most users expect and should be the default.
I'm a bit confused about what to do now: should I keep #3674 as is (bringing behavior of the fixer closer to the rules above)? Should the default remain unchanged and rules above applied strictly with a new option only? Something else?

It is hard to pin it down to one comment, as the discussion went over multiple PR's, both here and on the SF repo and in gitter as well IIRC.
To me it is not about what the current users would expect or not. Back in the day a compromise was reached between a lot of parties and I see no need to change the default. Now we have a nice configuration system that can bring opt-in/out functionality,  which was lacking back than.
Therefore IMHO I think the best way forward is fixing the clear bug and everything else would be nice to bring in with new config options.
Discussing one functional-optional-change at the time in one PR would be the best and the only way forward IMHO :)

Ok, WDYT about:

reducing #3674 scope by keeping the fix for #3335 only;
submitting a new PR to master introducing new options (e.g. always_move_variable and always_move_scalar_constant) which would make the fixer follow the rules I mentionned above and thus cover the case described here?

sounds like a solid plan to me :)


Error in Named Parameters in Equinox Bindings

At line 91 in equinox_bindings.py, you call vertex_eliminations_jaxpr with the following arguments,
out = vertex_elimination_jaxpr(closed_jaxpr.jaxpr,
                                        order,
                                        closed_jaxpr.literals,
                                        *_args,
                                        argnums=argnums,
                                        count_ops=count_ops,
                                        dense_representation=dense_representation)

however, the method at line 623 in core.py takes the argument called sparse_representation
def vertex_elimination_jaxpr(jaxpr: core.Jaxpr,
                            order: Union[Sequence[int], str],
                            consts: Sequence[core.Literal],
                            *args,
                            has_aux: bool = False,
                            argnums: Sequence[int] = (0,),
                            count_ops: bool = False,
                            sparse_representation: bool = False

yes, I renamed the kwarg. Have to fix this asap. Thank you for the request!


litecointoken

In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:

[ ] Filename in the lowercase format
[ ] Image format: png
[ ] Image size: 256 by 256 px

Added here https://github.com/TrustWallet/tokens/commit/ede0920ac0016af58820df8a655c31255810cda9
Also, you can use these graphics to say that your ERC20 token fully compatible with Trust Wallet (https://trustwalletapp.com/logo-assets). Tokens will show up automatically once the user receives them.
Feel free to share this on social media and tag Trust Wallet:

Twitter: @trustwalletapp

Facebook: @trustwalletapp


Endpoint fuzzing

Description of the Change
Fuzz tests for:

torii
status
find (shouldn't work well on the long run, since the storage isn't mocked)

Patch from google/libprotobuf-mutator#94
Benefits
Bugs now can be found in different way
Possible Drawbacks
None
Usage Example
This pr (as the previous one) depends on the clang (the bigger version the better), so you need to run cmake with -DFUZZING=ON flag, then build. After that in test_bin will appear related test that can be run (first argument maybe a folder with a corpus, stateless fuzzing will be considered otherwise):

torii_fuzzing
status_fuzzing
find_fuzzing

status_fuzzing test reaches out of memory state even if ~12G was allowed to use.
test_bin/status_fuzzing -rss_limit_mb=12000

...

==17953== ERROR: libFuzzer: out-of-memory (used: 12003Mb; limit: 12000Mb)
   To change the out-of-memory limit use -rss_limit_mb=<N>


Update samples readme for each package

Update samples for each package that we are releasing. Follow the guidelines about

[x]  Creating a /samples directory
[ ]  Adding a readme for the /samples directory
[ ]  Adding metadata to the samples for indexing
- [ ]  Including more advanced scenarios

@ramya-rao-a

Below are the applicable packages in the order of priority

storage-blob
storage-queue
keyvault-keys
keyvault-secrets
storage-file-share
keyvault-certificates
event hubs
app configuration

All packages have a samples directory with samples
All packages except Keyvault have a readme
@willmtemple Please sync up with @kaerm on what kind of advanced scenarios are we looking for and we can take a call on what needs to be done there

@kaerm The samples themselves are up to date. Please log new issue for any specific missing samples. Updating this issue title to reflect changes to only the readme files for samples.

Closing #4940 to prefer this issue.
I'll fix up the /samples/README.md files in addition to the /samples/typescript/README.md and /samples/javascript/README.md files for any errors as I'm going through.
@ramya-rao-a @kaerm FYI this is blocked on figuring out why the samples don't currently render in docs.microsoft.com. Since #6208 was merged I expected to see the TS/JS samples for file-share online but they're still not up.

This is not working because on our side we do not have the webhook configured. @scbedd - can you help enabling it for all the remaining repos?

@kaerm I'll sync with scotk today and get it done.

@kaerm @scbedd Any update on this? Not seeing the existing ones in docs.microsoft.com/samples if I browse to TypeScript/JavaScript and Azure Storage. Is it a matter of merging a change to those files and kicking off a web hook for the docs pipeline to start the ingestion process? #6496 should be merged soon, and I think it's all formatted and annotated correctly, but without being able to see the rendered pages it's impossible to know for sure.

@weshaggard was taking care of adding the webhook (I do not have admin perms on the repo). It was lower on his priority list, so it's a possibility it's not already done. Wes?

The webhooks are setup and enabled now.

Awesome, thanks! Do we have an idea of how long it takes to go from a commit being merged to master to seeing the changes live?

We're seeing the samples for storage live now. https://docs.microsoft.com/en-us/samples/browse/?products=azure-storage&languages=typescript
Thanks everyone!


Removed unused project.

Removed unused code from the core solution.

Why did we delete these files ?

These were not being used. No point keeping files in the repo when they are not being used.


_.deepFind won't work with _.matches shorthand

_.deepFind(objs, {'json.ref': objRef});
return the first item in obs even if json.ref dont match objRef.

This is by design. Currently all these functions take two parameters: collection and propertyPath.
I should investigate if it would make sense to create a _.deepMatches method, and apply it to the appropriate _.deep* methods we currently have.

Am I wrong in that I find that if you do specify the matches shorthand it still returns something? (possibly the first member of the collection)
var test = [{id: 1, ingredient: {id: 11, name: 'jake'}}, {id: 2, ingredient: {id: 22, name: 'phil'}}];
_.deepFind(test, {'ingredient.name': 'phil'}); <-- results in the first obj being returned.
I think people might have expected the match syntax to work because it's in find in Lodash - having it would be super handy!
One thing, I can't work out how you're supposed to use the propertypath to achieve a find. I specify the collection and the propertypath (in my case 'ingredient.name', but how do you specify the value it's trying to match?
Thanks, and great library btw... :)

Since 3.7 Lodash supports using property paths in all its (relevant) methods. So you can just use:
var test = [{id: 1, ingredient: {id: 11, name: 'jake'}}, {id: 2, ingredient: {id: 22, name: 'phil'}}];
_.find(test, 'ingredient.name', 'phil');

Ok, thanks. :)


.net Core 3.0.102 Kestrel Socket problem

Issue Title
.net Core 3.0.102 Kestrel Socket problem
General
Created a default Web Application with .Net Core 3.0, using Visual Studio 2019 v16.4.4
Just out of the box - Published the application, and run the executable.
When publishing this solution and try to run the executable, it raised a Kestrel error:
System.Net.Sockets.SocketException (10013): An attempt was made to access a socket in a way forbidden by its access permissions. at System.Net.Sockets.Socket.UpdateStatusAfterSocketErrorAndThrowException(SocketError error, String callerName) at System.Net.Sockets.Socket.DoBind(EndPoint endPointSnapShot, SocketAddress socketAddress) at System.Net.Sockets.Socket.Bind(EndPoint localEP) at Microsoft.AspnetCore ..... .....
This error screen came up really really fast, and then disappeared.
GitHub: https://github.com/yogigrantz/WebApplication1KestrelProblem
When this solution is published in a machine with VS2019 version 16.3.9 however, it works fine.
I am suspecting there is a faulty between 16.3.9 and 16.4.4 - Please see the screenshots in the GitHub

After further testing - uninstalled VS2019, uninstalled all .net Core versions. Rebooted. Reinstalled .net Core 3.0.102. Without VS 2019 I run dotnet publish from command line -> it produced the .exe.
Clicked the executable - > Same error perists:  System.Net.Sockets.SocketException (10013): An attempt was made to access a socket in a way forbidden by its access permissions.
Copied that executable to a colleague machine, ran it from there. It ran fine!
Something in my machine tripped this: System.Net.Sockets.Socket.UpdateStatusAfterSocketErrorAndThrowException

OK just found the cause of the problem. It was IIS. IIS was running in my machine and it was obviously causing a conflict for Kestrel. When I stopped IIS. Kestrel would work fine. Lesson learned!

Glad you were able to solve this. Closing.


Commands are not shown in help

The 'Commands' section in head comments block of the script should be shown when user types <botname> help, but it doesn't.
// Commands:
//   list birthdays - shows a list of users and their birthdays
//   birthday set <username> <date>/<month>/<year> - sets a birthday for the user
//   birthdays on <date>/<month>/<year> - shows a list of users with a set birthday date

This issue has been solved with https://github.com/tolstoyevsky/hubot-happy-birthder/pull/15 and should be successfully closed.


Create sentences on page open (for reading)

I'd like to include at least one sentence for Anki exports ( #3 ), but currently sentences aren't created until the page is marked read.
Instead, the sentences could be created when the page is rendered for reading.  The sentences still shouldn't be used until the page is marked "read" when showing the term references (using the Sentences link), but they should at least be available for export.  That way, if someone reads a page and creates a term, when they export to Anki at least one term would be available.

Done in develop.

Launched in 3.1.1


upgrade to .net 6

what it says on the label

Please upgrade to .Net 6. Your tool is amazing one.


vendor: Update govmm vendoring

Updating the govmm vendoring fixes the virtio-pci hotplug
when using NEMU in nested environments.
Shortlog:
f9b31c0 qemu: Allow disable-modern option from QMP
d617307 Run tests for the s390x build
b36b5a8 Contributors: Add Clare Chen to CONTRIBUTORS.md
b41939c Contributors: Add my name
dab4cf1 qmp: Add tests
5ea6da1 Verify govmm builds on s390x
ee75813 contributors: add my name
c80fc3b qemu: Add s390x support
ca477a1 Update source file headers
e68e005 Update the CONTRIBUTING.md
2b7db54 Add the CONTRIBUTORS.md file
b3b765c qemu: test Valid for Vsock for Context ID
3becff5 qemu: change of ContextID from uint32 to uint64
f30fd13 qmp: Output error detail when execute QMP command failed
7da6a4c qmp: fix mem-path properties for hotplug memory.
e4892e3 qemu/qmp: preparation for s390x support
110d2fa qemu/qmp: add new function ExecuteBlockdevAddWithCache
a0b0c86 qmp_test: Change QMP version from 2.6 to 2.9
10c36a1 qemu: add support for pidfile option
Fixes #983
Signed-off-by: Sebastien Boeuf<EMAIL_ADDRESS>

LGTM!

/test

@sboeuf does the runtime compiles? I changed the context ID froom uint32 to uint64 without my PR, you should have a conflict

@alicefr I simply did a cast of uint64() onto the cid value, but this is only to make this compile. I'm still expecting the proper fix for s390 arch to come from your PR #960 ;)

:) @sboeuf . I'll rebase mine if your gets merged before mine. I was planning to make another vendoring update with the s390x support. In this way, that PR gets simpler. I'll open the s390 support PR once I got the #960 merged :). So far I didn't find other endianness issues ;)

Nit-ette: Re-vendor commits not indented in commit so they don't render nicely in github :smile:
You can use the one-liner here to bling it:

https://github.com/kata-containers/community/blob/master/CONTRIBUTING.md#re-vendor-prs

lgtm

Ignoring metrics which is known to be broken atm and merging...
/cc @alicefr.


Live-reload not working when changing Component CSS

Is it normal that the live-reload functionality doesn't update the CSS if it is part of a Component.
Even on a cleanly installed seed and I change for example the background-color of the HomeComponent :host, the changes is only reflected when doing a 'hard' reload.
Might this have something to do with the fact that it is injected by Angular2 in the head of the index.html file?
Edit: this seems only to be the case when referencing an external CSS file, that is, using styleUrls inside the Component annotator. When you use the styles key and define the CSS directly in the Component it seems to work alright.

@Sjiep I just check that point and I don't see any ways to make this working. When a css file is modified, browsersync looks if it is in the document (which is not the case for components) so it doesn't inject anything, futhermore, component style sheets need to be compiled by Angular, those files would then need to trigger a full reload, but as there are css, browsersync doesn't trigger that one.
When you modify a component (styles or any else), you modify a js (compiled ts) file which triggers the full reload.

As a partial solution, injectChanges: false, in config will update window on every change.

I think disabling the CSS injection makes sense.

What are the implication of disabling CSS injection? thanks.

@sfabriece not much, when you will change a style sheet from css folder, it will trigger a full relaod instead of injecting that file only.

Ok, thanks @ludohenin .

Thanks, that indeed solved the issue!


fix(CommandPalette): activate first option after async search

🔗 Linked issue
Fixes https://github.com/nuxt/ui/issues/980
❓ Type of change

[ ] 📖 Documentation (updates to the documentation or readme)
[x] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
When using a group with async search in <CommandPalette> groups, the first item is not activated when the async search function resolves.
This PR activate the first option after async search in the <CommandPalette>, and reformat the code to use the same method activateFirstOption:

in the onMounted function (when using autoselect prop)
when the query changes (in the watch function)
in the debouncedSearch callback.

📝 Checklist

[x] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

Thanks! 😊


Single bracket issue

When parsing with non balanced brackets such as "};" the String.Format return an empty string.
Also when parsing a line ending by } such as "Struct}" The String.Format return a weird string

Smart.Default.Parser.ErrorAction = ErrorAction.ThrowError;
Smart.Format("Struct}")

results in a parsing exception "Format string has too many closing braces"
string.Format("Struct}")

throws a FormatException.
Both handle unbalanced curly braces in a similar way.


QObject::moveToThread:

Hello, Thanks for this great work
I have encountered some problems when I run this code on a remote server.
gs-process-data images --data Gaussian-Splatting-Toolkit/data --output-dir Gaussian-Splatting-Toolkit/output

So could you please give some recommends to solve this problem? Thank you very much

Hi! I've replaced the camera pose estimation with hloc and set the default descriptor extractor and matcher as superpoint and superglue. I should not depend on the colmap now. Could you have a try? Please let me know if there are any further issue.


Question about do_nms_obj and letterbox_image_into

Hi,
I have two questions about two methods used:

Does do_nms_obj do some kind of thresholding to filter the predicted bounding boxes before calculating and comparing IOUs?

void do_nms_obj(detection *dets, int total, int classes, float thresh)
{
    int i, j, k;
    k = total-1;
    for(i = 0; i <= k; ++i){
        if(dets[i].objectness == 0){
            detection swap = dets[i];
            dets[i] = dets[k];
            dets[k] = swap;
            --k;
            --i;
        }
    }
    total = k+1;

    for(i = 0; i < total; ++i){
        dets[i].sort_class = -1;
    }

    qsort(dets, total, sizeof(detection), nms_comparator);
    for(i = 0; i < total; ++i){
        if(dets[i].objectness == 0) continue;
        box a = dets[i].bbox;
        for(j = i+1; j < total; ++j){
            if(dets[j].objectness == 0) continue;
            box b = dets[j].bbox;
            if (box_iou(a, b) > thresh){
                dets[j].objectness = 0;
                for(k = 0; k < classes; ++k){
                    dets[j].prob[k] = 0;
                }
            }
        }
    }
}

The only threshold used is for the IOU value, so is nms performed on all bounding boxes produced by the network? Or is it performed only on filtered subset created? If so, where is this threshold set? I'm not sure where dets comes from exactly and what is done before it is passed to do_nms_obj.

Can someone explain exactly what letterbox_image_into does and what is the purpose?

Thanks in advance.

hopefully the source code can give you some insight. My understanding is this is an older, opencv 1.x standard ("IPLImage"). Newer versions of opencv use "Mat":
https://answers.opencv.org/question/10837/what-is-the-difference-of-mat-and-iplimage/
So, to some extent, this may be hard to come across good docs on just because it's fairly older ways of doing things. But here is the c code from darknet:
https://github.com/pjreddie/darknet/blob/61c9d02ec461e30d55762ec7669d6a1d3c356fb2/src/image.c#L796


Azure Pipelines

This brings builds and release on all platforms then packages it up into one npm installable package.
The next step is to actually pull the package down on each platform and test that it works but have been hitting some issues with it. It is basically the same setup as I did for bs-emotion-ppx https://github.com/alexfedoseev/bs-emotion/pull/15

Thanks!


Consider refactoring assemble_model() using model.frame()

Looking back at the assemble_model() function I added a while back, I'm wondering whether the same thing couldn't be accomplished in a simpler way.
The point of the thing is to force the lm() calls to keep all levels of all factors, rather than to drop unused levels as lm() does by default and possibly end up with a mismatching number of terms in groups A and B.  At the moment, assemble_model() does this by building the model frame of the DV and the model matrix of the IVs and then doing a clever lm() call to force the use of all the factor levels.  The clever call ruins the way the terms are stored in the fitted model, so assemble_model() saves the terms separately to be used elsewhere (in the coefficient extraction function, I think).
I just noticed that model.frame() seems to preserve all the factor levels by default, and passing the result to lm() seems to do what we want.  I haven't checked in detail, but it seems like that would preserve the terms better as well?  So I'm wondering whether we might be able to avoid the whole assemble_model() hack by simply passing the formula and data to model.frame() before passing the result to lm()...
Mind you, I'm not sure why I didn't do this before -- I could swear I considered model.frame back then, so it's possible that I rejected it for a good reason that I'm forgetting now.  But the assemble_model() solution is pretty ugly, so at some stage it might be worth trying to refactor it down to just the model.frame() call and seeing if it passes all the tests...

Reprex showing this.  Note that the naive lm() call ends up with differing numbers of terms, whereas fitting via model.frame() and assemble_model() both end up with the correct number for both groups.
data <- OaxacaBlinder::chicago_long |>
  dplyr::mutate(
    sex = ifelse(female == 1, "female", "male"),
    foreign_born = as.factor(foreign_born),
    tooyoung = age < 19
  )
fmla <- ln_real_wage ~ education + foreign_born
test <-
  data |>
  dplyr::mutate(education = as.factor(education)) |>
  tidyr::nest(.by = tooyoung) |>
  dplyr::mutate(
    lm = purrr::map(data, \(x) lm(fmla, x) |> coef()),
    mf = purrr::map(data, \(x) model.frame(fmla, x)),
    levels = purrr::map(data, \(x) levels(x$education)),
    mf_fit = purrr::map(mf, \(x) lm(x) |> coef()),
    am = purrr::map(data, \(x) OaxacaBlinder:::assemble_model(fmla, x)),
    am_fit = purrr::map(am, \(x) x$fit |> coef())
  ) |>
  identity()
test$lm
#> [[1]]
#>           (Intercept)      educationcollege  educationhigh.school
#>            3.35377962           -0.38800026           -0.77462231
#>         educationLTHS educationsome.college         foreign_born1
#>           -0.92343568           -0.66325923           -0.03572109
#>
#> [[2]]
#>   (Intercept) educationLTHS foreign_born1
#>     2.0154066     0.1672032    -0.2918628

test$mf_fit
#> [[1]]
#>           (Intercept)      educationcollege  educationhigh.school
#>            3.35377962           -0.38800026           -0.77462231
#>         educationLTHS educationsome.college         foreign_born1
#>           -0.92343568           -0.66325923           -0.03572109
#>
#> [[2]]
#>           (Intercept)      educationcollege  educationhigh.school
#>             2.1826097                    NA            -0.1672032
#>         educationLTHS educationsome.college         foreign_born1
#>                    NA                    NA            -0.2918628

test$am_fit
#> [[1]]
#>          X.Intercept.      educationcollege  educationhigh.school
#>            3.35377962           -0.38800026           -0.77462231
#>         educationLTHS educationsome.college         foreign_born1
#>           -0.92343568           -0.66325923           -0.03572109
#>
#> [[2]]
#>          X.Intercept.      educationcollege  educationhigh.school
#>             2.1826097                    NA            -0.1672032
#>         educationLTHS educationsome.college         foreign_born1
#>                    NA                    NA            -0.2918628

Created on 2024-08-01 with reprex v2.1.0

Ah, this is probably why I didn't do this before -- the problem happens when stats's functions convert characters to factors.  When I don't do that manually beforehand, they all fail, as in the amended reprex below that only changes that one mutate line.
I do wonder whether the better solution then is to convert the factors ourselves before splitting into groups?  Further research is required.
data <- OaxacaBlinder::chicago_long |>
  dplyr::mutate(
    sex = ifelse(female == 1, "female", "male"),
    foreign_born = as.factor(foreign_born),
    tooyoung = age < 19
  )
fmla <- ln_real_wage ~ education + foreign_born
test <-
  data |>
  # Doesn't work without this line
  # dplyr::mutate(education = as.factor(education)) |>
  tidyr::nest(.by = tooyoung) |>
  dplyr::mutate(
    lm = purrr::map(data, \(x) lm(fmla, x) |> coef()),
    mf = purrr::map(data, \(x) model.frame(fmla, x)),
    levels = purrr::map(data, \(x) levels(x$education)),
    mf_fit = purrr::map(mf, \(x) lm(x) |> coef()),
    am = purrr::map(data, \(x) OaxacaBlinder:::assemble_model(fmla, x)),
    am_fit = purrr::map(am, \(x) x$fit |> coef())
  ) |>
  identity()
test$lm
#> [[1]]
#>           (Intercept)      educationcollege  educationhigh.school
#>            3.35377962           -0.38800026           -0.77462231
#>         educationLTHS educationsome.college         foreign_born1
#>           -0.92343568           -0.66325923           -0.03572109
#>
#> [[2]]
#>   (Intercept) educationLTHS foreign_born1
#>     2.0154066     0.1672032    -0.2918628

test$mf_fit
#> [[1]]
#>           (Intercept)      educationcollege  educationhigh.school
#>            3.35377962           -0.38800026           -0.77462231
#>         educationLTHS educationsome.college         foreign_born1
#>           -0.92343568           -0.66325923           -0.03572109
#>
#> [[2]]
#>   (Intercept) educationLTHS foreign_born1
#>     2.0154066     0.1672032    -0.2918628

test$am_fit
#> [[1]]
#>          X.Intercept.      educationcollege  educationhigh.school
#>            3.35377962           -0.38800026           -0.77462231
#>         educationLTHS educationsome.college         foreign_born1
#>           -0.92343568           -0.66325923           -0.03572109
#>
#> [[2]]
#>  X.Intercept. educationLTHS foreign_born1
#>     2.0154066     0.1672032    -0.2918628

Created on 2024-08-02 with reprex v2.1.0

Hmph, so looking back at #27, which fixed this in the first place, it seems like using model.frame wouldn't actually add that much value, because model.matrix behaves the same way and assemble_model calls that anyway.  The hard part is getting the terms right, which is the real reason I hacked in assemble_model.
One hitch I noticed is that because model.matrix does drop unused levels in character variables that haven't been pre-converted to factors, it will, in fact, calculate models with mismatching numbers of factors.  Most of the current tests miss this because they pre-convert the character IVs to factors to ensure that the baselines match what comes from Stata.  Fortunately, the checksum test in #25 would catch this going wrong, and it passes.
So I think we're still getting correct results in spite of the mismatched terms from the models, and I think the reason for that is that they get matched up properly when the terms are put into a data frame for the final calculation (as in #20).  That means that #20 is not in fact redundant as I speculated in #27.  (And now I'm not sure whether #27 is redundant.)
So it looks like nothing is broken, but it does seem kinda rickety.  It'd be nice to have a better solution for this, and a test that directly tests the character-to-factor conversion...

Most of the current tests miss this

Ah, but "0-variance baseline-adjusted IV results match Stata" does not miss it -- it keeps education as a character.
So I'll rest a lot easier knowing that that test will break if the rickety setup in assemble_model breaks.  Still a good candidate for refactoring though.

And now I'm not sure whether #27 is redundant

Also pretty sure the reason it is not redundant is because naive lm calls will (by default, anyway -- can't remember if there's a way to change this) drop unused levels of a variable even if it is a factor.  So in theory a level could be dropped from both groups, especially in bootstraps, which could screw things up.

Sigh I'm realizing that the main function of this thread is to reveal how much I've forgotten what's in the codebase.
We already do convert all the characters to factors ourselves, at https://github.com/sinanpl/OaxacaBlinder/blob/ca3ced7994b5c6612fb946275418e8ee99de09b7/R/oaxaca.R#L219.
So, very possible that there's nothing to see here at all.  I'm going to close this thread and reopen it if I find a reason to.


Fix Optional[Annotated[...]] resolution.

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

Comparison is base (ef44925) 92.46% compared to head (f3d9947) 92.28%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main       #9      +/-   ##
==========================================
- Coverage   92.46%   92.28%   -0.19%
==========================================
  Files          10       10
  Lines        1062     1063       +1
  Branches      252      253       +1
==========================================
- Hits          982      981       -1
- Misses         45       46       +1
- Partials       35       36       +1

Flag
Coverage Δ

unittests
92.28% <100.00%> (-0.09%)
:arrow_down:

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.


More RESTful style endpoints

In #106 @Bluenix2 suggested the below. I really like this proposed change, so I think we should do it after 106 is merged.
I am not sure how thoroughly this has been discussed, and how much of a say I have in this. But I have some opinions on the new endpoints, you currently have:
GET /size: Get the size of the canvas
GET /pixels: Get the whole canvas
GET /pixel: Get a specific pixel
PUT /pixel: Set a specific pixel
One quick thing I want to mention about /pixels, it doesn't really make sense, we're already at pixels.pythondiscord.com. Making the URL pixels.pythondiscord.com/pixels..
In my opinion, the following seem more RESTful:
GET /canvas/size: Get the size of the canvas alternatively this info could be included in the headers and would be mainly requested using HEAD /canvas.
GET /canvas: Get the whole canvas
alternatively the following two endpoints could use /canvas/pixels
GET /canvas?x=x&y=y: Get a specific pixel
PUT /canvas: Set a specific pixel you could also use query parameters here and the body would only consist of the color

if we wanted to do multiple canvases at any point, might it make more sense to be /canvas/{canvas}/endpoint, similar to how Discord does it (i.e. /channels/1234/messages/5678)

We could do that fairly trivially with fastapi's path routing


add error handling in memorymapmanager

adds error handling to memory map manager so that problems like exhausted file handles get visible.
relates #60

Coverage decreased (-0.2%) to 93.593% when pulling 5421cdba29b8f867dcff146afba81f4d82a097f1 on hendrikmuhs:error-handling-memorymap into 3570797b2e629f4464295c7e30f9906e5d8b78b8 on KeyviDev:master.


Docs: how to develop a webhook for a job integration

What type of PR is this?
/kind documentation
What this PR does / why we need it:
Documents how to develop a webhook integration after all the caveats learned in #3132.
Which issue(s) this PR fixes:

Ref #2878
Special notes for your reviewer:
Does this PR introduce a user-facing change?

NONE

Leave a lgtm tag on @mimowo

/lgtm
Thanks! A cherry on top for the great work on fixing the long-standing issue.


Small scheduled task question - big thank you

Not an issue, just a small question and a big thank you for this script.
After updating my 1019+ to DSM 7.2-64570 Update 1 the storage pool on the m.2 disks was no longer available.
Ran this script and restarted after which the storage pool was available again.
Thanks!!
I've configured a scheduled task to run on shutdown.
My question is if it's needed to add the --noupdate after syno_hdd_db.sh or if can leave that out.
I did use --noupdate the first time running the script.
Also, would it be better to run these scripts at shutdown or at startup? I'd think on shutdown?
Current commands:

I would continue using the --noupdate option because without it DSM can update the drive databases whenever it wants.

Also, would it be better to run these scripts at shutdown or at startup? I'd think on shutdown?

I missed this question in my previous reply.
Originally I used to think running the scripts at shutdown was the better choice. But now I think at startup might be better.

Any arguments on why that would be better?
How about running the scripts on shutdown and startup?

There was one person who had the scripts scheduled to run at shutdown and they noticed that after a DSM update they had to run the run script again after it booted up.
It wouldn't hurt to run the scripts at shutdown and bootup.

Configured it to run at boot and shutdown. Better safe then sorry.
Thanks for the info!


[#42] [Backend] As a user, I can see the survey questions

Closed #42

What happened 👀
I created a backend for getting survey detail: including questions and answers.
Insight 📝

I updated the Survey model to have questions and answers.
I added tests for SurveyRepository and the new use case.

Proof Of Work 📹

As @hoangnguyen92dn approved this PR. I will merge this one :D


Add support for SN 0.4.1 and 0.4.2

Add support for 0.4.0 and 0.4.1 by values defined in build.sbt, the idea being that after a release, 0.4.0 specific code will be removed and shared and 0.4.1 will be merged.
CLI arguments are parsed via case-app dependency and an additional NativeConfig generation phase is introduced.
Includes initial project definition and testing infrastructure. Testing is comprised of scalatest unit tests and sbt-provided scripted tests acting as integration tests.

I think there are still some problems with WIndows CI (or I incorrectly rebased...). I'll fix those asap


About the building of vocabulary?

Hi, can you tell me why you use  the training data  to build vocabulary and not use the complete data?

because in general case training data is the biggest data among train,test and validate so it does not effect much. but you can use entire data for the same but it will not effect the result. as i tried with whole dataset for hindi on the same.


Error (maybe) in readme

Just reading through some of this README. I think there is an error in the example Slurm template in the README section.
From the section titled "Templates Files" (probably should be "template files" but thats a separate point?) the code should read "srun {CODE}" rather than just "{CODE}". You might also want to check the slurm_sig_template.sub which may need to be updated with "srun" as well?
Screenshot below.

Good call, thanks @rcorces . Fixed in https://github.com/pepkit/pepenv/commit/d5f86fbdbf39d1ffc8fd2496a784120d11e76f7a


Positional Audio

I was taking a look at the wiki about positional audio, but I didn't find anything referring to Mumble InputStream is it possible somehow to send an audio to the mumble server with vector3 ?

Not in the provided API. The input stream is also a bit tricky abstraction for it. Mumble doesn't handle positional audio as some hypothetical multichannel audio; instead the audio is still encoded as mono/stereo stream. The positional information is then added as extra metadata on the audio packets.
The input stream eats PCM audio, splits this in audio frames and encodes it. These encoded frames are then combined into a Mumble audio packet based on packet size settings. That audio packet has optional field for the positional data.
The issue here is that the input stream doesn't expose details/API on packet boundaries so there is no way to tell when the packet changes and positional data could be sent. Associating positional data with each sample would be quite expensive.
On the MumbleConnection-level the packets are formed in the sendEncodedFrame function:
https://github.com/Rantanen/node-mumble/blob/3387e1232dfec897c1d4b34015b1bf129625b08d/lib/MumbleConnection.js#L386-L396
The protocol docs indicate the positional data should follow the frames: https://mumble-protocol.readthedocs.io/en/latest/voice_data.html#encoded-audio-data-packet
Writing three float coordinates after the frames on line 396 would take care of that (and ensuring the length info on line 390 takes that added data into account).
A possible API for this would have current position as a state on the stream the user would keep up to date. Then when the stream queues the frames on the connection by invoking connection,sendVoiceFrame it would also pass the position data as a parameter. sendVoiceFrame would then pass the parameter to sendEncodedFrame. This would avoid the stream from having to expose the packet boundaries or from the user having to pass the location data with every sample.
Alternative to exposing the location as a field on the stream object would be to support some in-stream metadata that could be written to the stream through normal write APIs and the stream would update its internal state based on those - writing the location data would then proceed as above. I don't see much point with this though.
Now unfortunately this project is not only not maintained anymore, but I also lost the use of my other hand few months ago so I'm not going to be able to do any maintenance currently. The above instructions should enable you to make the necessary changes if you need them.


2016 date giving IncompleteRead error

Hi there,
I successfully installed and can run the demo and things for the 2015 season and previous.

However when I try to access the 2016 weeks I get this error:

Any help is appreciated. Much thanks and cool stuff!

Do you have write access to the \site-packages\nflgame folder?  All of the 2015 data in included with the initial install in gamecenter-json but anything after ~September gets built from nfl.com if/when it gets accessed.  In order for that data to be accessed by nflgame it needs to store a file in \gamecenter-json.
I thought there was actually a check built-in to make sure that write was possible, so maybe I'm sniffing around the wrong bush here...

I don't think permissions are a problem - I can write to a text file in those folders from the console no problem, but I will double check.,

It works on my laptop so it must be something specific to my desktop set up. Thanks again.

I'm having this same issue and I just made sure:
*\AppData\Local\Python Tools\CompletionDB\14.0\9a7a9026-48c1-4688-9d5d-e5699d47d074\3.5\nflgame
was not "ReadOnly".

Please see my comment on #279 if this is on a brand new install.


BOTH.SCREENS doesn't works properly

Works very well in download URL, LOCK and HOME , but, if you use BOTH.SCREENS, sometimes works at home or lock but never with two at the same time,
I use a elevate button on press this code;
void _setWallpaper(BuildContext context, String url) async {
String result;
// Platform messages may fail, so we use a try/catch PlatformException.
try {
result = await AsyncWallpaper.setWallpaper(
url: "$urlBase${url}_mobile.jpg",
wallpaperLocation: AsyncWallpaper.LOCK_SCREEN,
goToHome: true,
toastDetails: ToastDetails.success(),
errorToastDetails: ToastDetails.error(),
)
? 'Wallpaper set'
: 'Failed to get wallpaper.';
} catch (e) {
// No specified type, handles all
print('Something really unknown: $e');
}
}
flutter : Flutter 3.10.0 • channel stable
Dart: 3.0
Xiaomi Redmi note 11
Android : 12 SKW 1.21
MIUI 13

This is a problem with some device manufacturers such as Xiaomi, Realme, etc. For this you can use the external apply method setWallpaperFromFileNative or setWallpaperNative to use external app such as Gallery to apply both wallpapers.


Error make

Building C object CMakeFiles/LiveProxies.dir/Server.c.o
/LiveProxies/Server.c: In function ‘SendChunkPrintf’:
/LiveProxies/Server.c:56:3: warning: implicit declaration of function ‘vasprintf’ [-Wimplicit-function-declaration]
   vasprintf(&body, Format, args);
   ^
/LiveProxies/Server.c:61:2: warning: passing argumeQnt 1 of ‘sprintf’ from incompatible pointer type [enabled by default]
  sprintf(hex, "%x", bodyLen);
  ^
In file included from LiveProxies/GeoIP.h:42:0,
	         from /LiveProxies/Server.c:6:
/usr/include/stdio.h:364:12: note: expected ‘char * restrict’ but argument is of type ‘char **’
 extern int sprintf (char *__restrict __s,
	    ^
/LiveProxies/Server.c:61:2: warning: format ‘%x’ expects argument of type ‘unsigned int’, but argument 3 has type ‘size_t’ [-Wformat=]
  sprintf(hex, "%x", bodyLen);
  ^
/LiveProxies/Server.c:63:2: warning: passing argument 1 of ‘strlen’ from incompatible pointer type [enabled by default]
  all = malloc((strlen(hex) + 2 + bodyLen + 2) * sizeof(char) + 1); {
  ^
In file included from //LiveProxies/GeoIP.h:44:0,
	         from /LiveProxies/Server.c:6:
/usr/include/string.h:399:15: note: expected ‘const char *’ but argument is of type ‘char **’
 extern size_t strlen (const char *__s)
	       ^
/LiveProxies/Server.c:64:3: warning: format ‘%s’ expects argument of type ‘char *’, but argument 3 has type ‘char **’ [-Wformat=]
   sprintf(all, "%s\r\n%s\r\n", hex, body);
   ^
/LiveProxies/Server.c:65:3: warning: passing argument 1 of ‘strlen’ from incompatible pointer type [enabled by default]
   bufferevent_write(BuffEvent, all, (strlen(hex) + 2 + bodyLen + 2) * sizeof(char));
   ^
In file included from /LiveProxies/GeoIP.h:44:0,
	         from /LiveProxies/Server.c:6:
/usr/include/string.h:399:15: note: expected ‘const char *’ but argument is of type ‘char **’
 extern size_t strlen (const char *__s)
	       ^
LiveProxies/Server.c: In function ‘ServerFindHeader’:
LiveProxies/Server.c:81:16: warning: assignment makes integer from pointer without a cast [enabled by default]
    searchIndex = valIndex + inLen;
	        ^
/LiveProxies/Server.c: In function ‘ProxyCheckLanding’:
/LiveProxies/Server.c:125:3: warning: passing argument 2 of ‘Base64Decode’ from incompatible pointer type [enabled by default]
   if (!Base64Decode(keyRaw, &key, &len)) {
   ^
In file included from /LiveProxies/Server.c:1:0:
/LiveProxies/Base64.h:8:14: note: expected ‘unsigned char **’ but argument is of type ‘char **’
 bool MEM_OUT Base64Decode(char* b64message, unsigned char** buffer, size_t* length);
	      ^
/LiveProxies/Server.c:213:8: warning: passing argument 5 of ‘pcre_get_substring’ from incompatible pointer type [enabled by default]
	pcre_get_substring(Buff, subStrVec, regexRet, x, &(foundIpStr));
	^
In file included from /LiveProxies/Server.c:12:0:
/usr/include/pcre.h:465:20: note: expected ‘const char **’ but argument is of type ‘char **’
 PCRE_EXP_DECL int  pcre_get_substring(const char *, int *, int, int,
	            ^
/LiveProxies/Server.c: In function ‘ServerLanding’:
/LiveProxies/Server.c:313:3: warning: passing argument 3 of ‘bufferevent_setcb’ from incompatible pointer type [enabled by default]
   bufferevent_setcb(BuffEvent, HTTPRead, bufferevent_free, ServerEvent, NULL);
   ^
In file included from /LiveProxies/ProxyRequest.h:4:0,
	         from /LiveProxies/Server.c:7:
/usr/include/event2/bufferevent.h:299:6: note: expected ‘bufferevent_data_cb’ but argument is of type ‘void (*)(struct bufferevent *)’
 void bufferevent_setcb(struct bufferevent *bufev,
      ^
/LiveProxies/Server.c: In function ‘HTTPRead’:
/LiveProxies/Server.c:337:2: warning: implicit declaration of function ‘evbuffer_copyout_from’ [-Wimplicit-function-declaration]
  evbuffer_copyout_from(evBuff, &pos, buff, 4);
  ^
/LiveProxies/Server.c: In function ‘LevConnListenerBindCustom’:
/LiveProxies/Server.c:395:58: error: ‘LEV_OPT_REUSEABLE_PORT’ undeclared (first use in this function)
  int flags = LEV_OPT_CLOSE_ON_FREE | LEV_OPT_REUSEABLE | LEV_OPT_REUSEABLE_PORT | LEV_OPT_DEFERRED_ACCEPT;
	                                                  ^
/LiveProxies/Server.c:395:58: note: each undeclared identifier is reported only once for each function it appears in
/LiveProxies/Server.c:395:83: error: ‘LEV_OPT_DEFERRED_ACCEPT’ undeclared (first use in this function)
  int flags = LEV_OPT_CLOSE_ON_FREE | LEV_OPT_REUSEABLE | LEV_OPT_REUSEABLE_PORT | LEV_OPT_DEFERRED_ACCEPT;
	                                                                           ^
/LiveProxies/Server.c:407:2: warning: implicit declaration of function ‘evutil_make_listen_socket_reuseable_port’ [-Wimplicit-function-declaration]
  if (evutil_make_listen_socket_reuseable_port(fd) < 0)
  ^
/LiveProxies/Server.c:410:2: warning: implicit declaration of function ‘evutil_make_tcp_listen_socket_deferred’ [-Wimplicit-function-declaration]
  if (evutil_make_tcp_listen_socket_deferred(fd) < 0)
  ^
/LiveProxies/Server.c: In function ‘ServerBaseSSL’:
/LiveProxies/Server.c:445:3: warning: passing argument 3 of ‘LevConnListenerBindCustom’ makes pointer from integer without a cast [enabled by default]
   levServerListSSL4 = LevConnListenerBindCustom(levServerBaseSSL, ServerAccept, SSL4, &map, SSLServerPort);
   ^
/LiveProxies/Server.c:389:31: note: expected ‘void *’ but argument is of type ‘int’
 static struct evconnlistener *LevConnListenerBindCustom(struct event_base *Base, evconnlistener_cb Cb, void *Arg, IPv6Map *Ip, uint16_t Port)
	                       ^
/LiveProxies/Server.c:454:3: warning: passing argument 3 of ‘LevConnListenerBindCustom’ makes pointer from integer without a cast [enabled by default]
   levServerListSSL6 = LevConnListenerBindCustom(levServerBaseSSL, ServerAccept, SSL6, &map, SSLServerPort);
   ^
/LiveProxies/Server.c:389:31: note: expected ‘void *’ but argument is of type ‘int’
 static struct evconnlistener *LevConnListenerBindCustom(struct event_base *Base, evconnlistener_cb Cb, void *Arg, IPv6Map *Ip, uint16_t Port)
	                       ^
LiveProxies/Server.c: In function ‘ServerBase’:
/LiveProxies/Server.c:483:3: warning: passing argument 3 of ‘LevConnListenerBindCustom’ makes pointer from integer without a cast [enabled by default]
   levServerList6 = LevConnListenerBindCustom(levServerBase, ServerAccept, HTTP6, &map, ServerPort);
   ^
LiveProxies/Server.c:389:31: note: expected ‘void *’ but argument is of type ‘int’
 static struct evconnlistener *LevConnListenerBindCustom(struct event_base *Base, evconnlistener_cb Cb, void *Arg, IPv6Map *Ip, uint16_t Port)
	                       ^
/LiveProxies/Server.c: In function ‘ServerUDP’:
/LiveProxies/Server.c:512:3: warning: passing argument 1 of ‘RawToIPv6Map’ from incompatible pointer type [enabled by default]
   IPv6Map *ip = RawToIPv6Map(&remote); {
   ^
In file included from /LiveProxies/Global.h:3:0,
	         from /LiveProxies/Base64.h:3,
	         from /LiveProxies/Server.c:1:
/LiveProxies/IPv6Map.h:34:10: note: expected ‘struct sockaddr *’ but argument is of type ‘struct sockaddr_in *’
 IPv6Map *RawToIPv6Map(struct sockaddr *In);
	  ^
/LiveProxies/Server.c: In function ‘ServerUDP4’:
/LiveProxies/Server.c:579:2: warning: passing argument 4 of ‘pthread_create’ makes pointer from integer without a cast [enabled by default]
  int status = pthread_create(&wServerUDP, NULL, (void*)ServerUDP, hSock);
  ^
In file included from /LiveProxies/ProxyLists.h:6:0,
	         from //LiveProxies/Server.c:4:
/usr/include/pthread.h:244:12: note: expected ‘void * restrict’ but argument is of type ‘int’
 extern int pthread_create (pthread_t *__restrict __newthread,
	    ^
/LiveProxies/Server.c:582:3: warning: ‘return’ with a value, in function returning void [enabled by default]
   return status;
   ^
/LiveProxies/Server.c: In function ‘ServerUDP6’:
/LiveProxies/Server.c:604:2: warning: passing argument 4 of ‘pthread_create’ makes pointer from integer without a cast [enabled by default]
  int status = pthread_create(&wServerUDP, NULL, (void*)ServerUDP, hSock);
  ^
In file included from /h/LiveProxies/ProxyLists.h:6:0,
	         from /LiveProxies/Server.c:4:
/usr/include/pthread.h:244:12: note: expected ‘void * restrict’ but argument is of type ‘int’
 extern int pthread_create (pthread_t *__restrict __newthread,
	    ^
/LiveProxies/Server.c:607:3: warning: ‘return’ with a value, in function returning void [enabled by default]
   return status;
   ^
make[2]: *** [CMakeFiles/LiveProxies.dir/Server.c.o] Ошибка 1
make[1]: *** [CMakeFiles/LiveProxies.dir/all] Ошибка 2
make: *** [all] Ошибка 2

Your libevent lacks support for LEV_OPT_DEFERRED_ACCEPT, which was introduced in libevent 2.1 and LEV_OPT_REUSEABLE_PORT which was introduced in 2.1.5 (I think). Temporarily, you can remove those flags from int flags, but you will need to choose different ports accordingly for HTTP, HTTPS and UDP.
int flags = LEV_OPT_CLOSE_ON_FREE | LEV_OPT_REUSEABLE;

Another solution would be update libevent.
libevent version in dependencies README.me will be updated in next version.


Pin folders to top bar, like files

It would be nice to be able to pin a folder to the top bar and when you click it, it would show a dropdown of files in the folder. This would allow development without the sidebar open most of the time.

(Experimental duplicate detection)
Thanks for submitting this issue. Please also check if it is already covered by an existing one, like:

Pin tab (#93488)
Some filetypes do not appear on the taskbar jumplist and cannot be pinned (#48688)

It's a different suggestion.

This is a fair feature request, however we do not have it on our immediate plan. Thus assigning to backlog and we might add this in the future. Thanks

hi,this issue could be make better
please vote it
@bpasero @isidorn @TlStephen2011 @doudou @threeve
#99415


--primarygoogle

Add to primary calendar and delete 'My timetable', if present.

No worries! I'm about 10 years overdue a contribution to an open source project.. thanks for sharing your efforts in the first place.
Next step is learning to use Git properly. I think I've put your changes back in and merged your suggestions. Last remaining one is whether you want to revert the forward/back slash thing.

Implements #4


Bump jitsi-meet-sdk to 10.2.1

compileSdkVersion to 34
targetSdkVersion to 34
org.jitsi.react:jitsi-meet-sdk to 10.2.1

Fixes:
Starting FGS with type mediaProjection callerApp=ProcessRecord{8212e7e 22282:com.landa.app/u0a274}
targetSDK=34
requires permissions: all of the permissions allOf=true
[android.permission.FOREGROUND_SERVICE_MEDIA_PROJECTION] any of the permissions allOf=false
[android.permission.CAPTURE_VIDEO_OUTPUT, android:project_media]

Maybe #127
Test it with:
dependency_overrides:
  jitsi_meet_wrapper:
    git:
      url: https://github.com/efraespada/jitsi_meet_wrapper.git
      path: jitsi_meet_wrapper
      ref: main

Thanks for the PR. This package will very likely be discontinued, because there is now an official one (https://pub.dev/packages/jitsi_meet_flutter_sdk). Does the official one fulfill your needs?


seems not working on 1.20.2 paper fork

seems not working on 1.20.2 paper fork

Is it giving you an error you can share? I'm not having any issues with it on 1.20.2


Fix swagger KPI issue on additional property "tenantId" in deployment script with api-version 2020-10-01

Fix swagger KPI issue in s360: https://portal.azure-devex-tools.com/amekpis/correctness/detail?errorId=9ECBFB31-DDA6-4A34-B553-CE8D063BD3CB
This is not a breaking change, it simply fixes swagger to better reflect the behavior on service side.
Changelog
Please ensure to add changelog with this PR by answering the following questions.

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[ ] update existing version for new feature
[x] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When you are targeting to deploy new service/feature to public regions? Please provide date, or month to public if date is not available yet.
When you expect to publish swagger? Please provide date, or month to public if date is not available yet.
If it's an update to existing version,  please select SDKs of specific language and CLIs that require refresh after swagger is published.

[ ] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[x] No, no need to refresh for updates in this PR

Contribution checklist:

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[ ] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
Adding a new service

[ ] Please ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

@ArcturusZhang @msyyc , please check the potential SDK breaking changes, thanks.

@ArcturusZhang @msyyc , please check the potential SDK breaking changes, thanks.

Please note, this is NOT a breaking change. No behavioral change on service side, this PR simply fixes swagger correctness issue to better reflect service side.

@ArcturusZhang @msyyc , please check the potential SDK breaking changes, thanks.

The breaking changes in go SDK are expected


Use the "raw" not ("preprocessed") table name in MetastoreRelation

@markhamstra

LGTM; but I was just looking at the 1.6/master HiveMetastoreCatalog.scala.  It's a little bit different and will require some refactoring to re-apply the versioning-with-tableNamePreprocessor changes.


add skipUrlPattern option

I think this option is in the need of this kind of tool!
Useful for use cases where you need to skip some URLs

@davidtheclark I'll be happy if you could take a look and merge

Apologies if this is a stupid question, but can skipUrlPattern be used from the command line?
I'm getting a lot of broken links that are not really broken and would like to exclude them (I don't know why they're being listed as broken):

warning  Link to https://developer.android.com/training/location/permissions#request-location-access-runtime is dead  no-dead-urls  remark-lint
warning  Link to https://developer.android.com/jetpack/androidx/releases/biometric is dead                            no-dead-urls  remark-lint
warning  Link to http://developer.android.com/reference/android/app/Activity.html#ActivityLifecycle is dead           no-dead-urls  remark-lint
warning  Link to https://developer.android.com/training/location/request-updates is dead                              no-dead-urls  remark-lint

When I navigate to the URLs manually, they load fine. If it is supported, how do I modify my command to include it? I'm running:
remark -u remark-lint-no-dead-urls -q docs

It might be more manageable in a configuration file? https://github.com/unifiedjs/unified-engine/blob/main/doc/configure.md


Add Wintun support

Users required to install wireguard to get wintun driver.

Fix #116

This is awesome, thank you so much!
To merge this, I think we need to make sure this won't break any existing users of Nebula that are using the OpenVPN tun driver. Could you update your patch to retain support for the old driver, but select the wintun driver if it is installed? I'm thinking perhaps we have tun_water_windows.go and tun_wintun_windows.go and we have a small stub in tun_windows.go to select which one to use based on the installed driver. If you have a better idea for how to go about this, feel free to try another route too. I just want to make sure we don't leave any existing users scratching their heads when they upgrade (at least have a deprecation period for folks to switch over).
In the future after this PR has been merged I'd like to set up a build process to create a .msi that includes the wintun driver as well, which will also hopefully solve this for new users.
Thanks again, let us know if you aren't interested in making those changes as well and we can take the reigns.

@wadey Done, please review.

I don't know why, Microsoft Security will reported virus.
I tinkered a bit, it won't complain now.

Thanks for updating your PR, I just want to do a little more testing with this before we merge :+1:

I was working on finishing up some testing for this, but they just announced today the release of Wintun 0.9 which is supposed to make distribution of the driver much easier. So I guess we should look at what it will take to upgrade to 0.9
https://lists.zx2c4.com/pipermail/wireguard/2020-November/005921.html

The new version wintun is much easier to integrated.
But I found that wintun is licensed at GPL, which is incompatible with MIT.
Just send an email about license change to wireguard mail list.

I think https://github.com/WireGuard/wintun/commit/6af568d1e0edaa6809d99e7be5e13825c5e8bee6 solves license compatibility issues. wintun.h is now duel licensed under GPL and MIT.

I think WireGuard/wintun@6af568d solves license compatibility issues. wintun.h is now licensed under GPL and MIT.

Sorry, but it doesn't. The usage of wintun required linking to wintun library, which will make our code be GPL too.
In fact, I found that the TAP windows driver also be GPLv2 licensed.
BTW, I haven't received any response from the author of wintun/wireguard yet.
It seems that this pr can be closed now.

Did you send your email to Jason directly or to the mailing list? I was trying to find it on the mailing list and I didn't see it. Perhaps we are okay as long that file is MIT, but I need to brush up on GPL and DLLs.
Could you attach your previous email exchange about updating the header file license to this issue?

I send the email to mailing list. and received "Your message to WireGuard awaits moderator approval". that's all.

Hey, the signed binary wintun.dll from https://www.wintun.net is under a different license then gpl. It is not an open source license but it allows to redistribute and link to wintun.dll as long as it is used only with the api defined in wintun.h.
Here is a copy of the license.

Hey, the signed binary wintun.dll from https://www.wintun.net is under a different license then gpl. It is not an open source license but it allows to redistribute and link to wintun.dll as long as it is used only with the api defined in wintun.h.
Here is a copy of the license.

It looks like @ofer-dev is correct, and the .dll is distributed with a permissible license, one that we could bundle with an installer for Windows and not be concerned about licensing issues. The license is viewable in the wintun repo here: https://github.com/WireGuard/wintun/blob/master/prebuilt-binaries-license.txt
@CzBiX Would you like to take the effort of upgrading this PR for Wintun 0.10, or would you like to hand it over and we can work on it? Thanks again for your effort so far!

It looks like @ofer-dev is correct, and the .dll is distributed with a permissible license, one that we could bundle with an installer for Windows and not be concerned about licensing issues. The license is viewable in the wintun repo here: https://github.com/WireGuard/wintun/blob/master/prebuilt-binaries-license.txt
@CzBiX Would you like to take the effort of upgrading this PR for Wintun 0.10, or would you like to hand it over and we can work on it? Thanks again for your effort so far!

Also, if you decide to update this PR, don't worry about how to bundle the .dll for distribution. We will work on a different PR to build an installer with the DLL during the release process.

Also, if you decide to update this PR, don't worry about how to bundle the .dll for distribution. We will work on a different PR to build an installer with the DLL during the release process.

Since we will bundle the dll, I think we can remove water tun implementation now.

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

This seems to solve #9 on Windows 10. I had the same problem.
It is also a much better solution than installing the entire OpenVPN package.
Got it working by building @CzBiX fork and putting the wintun.dll into the same folder with nebula.exe
Hopefully this gets merged soon!
Awesome project, really appreciate your work!

any updates on this?

Just wanted to let you know that the CzBiX fork still appears to work well with the current Wintun version 0.13 (https://www.wintun.net/builds/wintun-0.13.zip)
Transfer speed on Windows is still markedly behind WireGuard in my situation but still ahead of the Nebula 1.4 Open-VPN-TAP-based version. But there's no point discussing that before there is a stable release.


Could not install typings files for JavaScript language features. Please ensure that NPM is installed or configure 'typescript.npm' in your user settings

From @sharno on October 19, 2017 1:55

VSCode Version: 1.17.2
OS Version: macOS 10.12.6

Steps to Reproduce:

Load a big JS project
open a file inside to edit

Checking the npm path, it's correct and even putting it manually gives the same error
checked the tsserver log, I found those errors:

Entry for package 'babel-loader' does not exist in local types registry - skipping...
Entry for package 'babel-polyfill' does not exist in local types registry - skipping...
Entry for package 'babel-preset-es2015' does not exist in local types registry - skipping...
Entry for package 'babel-preset-react' does not exist in local types registry - skipping...
Entry for package 'babel-preset-stage-0' does not exist in local types registry - skipping...
Entry for package 'babel-runtime' does not exist in local types registry - skipping...
Entry for package 'babel-eslint' does not exist in local types registry - skipping...

and the other file showed:
Info 38   [21:9:43.497] Received response: {"kind":"event::initializationFailed","message":"Command failed: \"/usr/local/bin/npm\" install --ignore-scripts types-registry"}
Info 39   [21:9:43.497] event: {"seq":0,"type":"event","event":"typesInstallerInitializationFailed","body":{"message":"Command failed: \"/usr/local/bin/npm\" install --ignore-scripts types-registry"}}

running these commands works from terminal but only after removing the quotations that are escaped
It's worth mentioning that I'm using Fish Shell but it's configured properly (I suppose)
Also this error appears only in this project
Reproduces without extensions: Yes
Copied from original issue: Microsoft/vscode#36520

From @sharno on October 19, 2017 3:5
Checked the path in VSCode's developer console and I found that /usr/local/bin is present there. Don't know why the command \"/usr/local/bin/npm\" install --ignore-scripts types-registry

@sharno Can you try upgrading to TypeScript 2.6 by following these instructions: https://code.visualstudio.com/Docs/languages/typescript#_using-newer-typescript-versions
I believe that @andy-ms added some additional logging for the typings installer that may help us track down what is going wrong

@mjbvz interestingly the issue totally disappears when I installed typescript@next (2.6) and configured VSCode to use it. VSCode stopped showing me the warning. It returns again though when I fall back to typescript@latest (2.5.3). Maybe it was a bug with (2.5.3)?
found in the logs this when using (2.6)
Exec: npm install --ignore-scripts types-registry
    Succeeded. stdout:
    +<EMAIL_ADDRESS>    updated 1 package in 0.817s

but still found those
Entry for package 'babel-loader' does not exist in local types registry - skipping...
Entry for package 'babel-polyfill' does not exist in local types registry - skipping...
Entry for package 'babel-preset-es2015' does not exist in local types registry - skipping...
Entry for package 'babel-preset-react' does not exist in local types registry - skipping...
Entry for package 'babel-preset-stage-0' does not exist in local types registry - skipping...
Entry for package 'babel-runtime' does not exist in local types registry - skipping...
...

and many other packages

closing for now. please reopen if you see the issue again on 2.6.


change github-server to git-server

Last pull request #28 is incomplete, try to fix it.

+1

Sorry, didn't see this. #47 switches it all to GIT_SERVER instead of GITHUB_SERVER.


Zone file export

Zone file export is listing DAB stations with the ISO code.
This is not necessary for DAB. ISO was necessary in the old spec for FM only. However to keep backward compatibility with older implementation, it is necessary to keep ISO for FM (but not DAB).
Example:
WRONG:
0.43ea.4081.4e1.dab                     	IN	CNAME	retedue.chrsi.radio.ebu.io.
0.43ea.4081.ch.dab                      	IN	CNAME	retedue.chrsi.radio.ebu.io.
0.43e2.4081.4e1.dab                     	IN	CNAME	retedue.chrsi.radio.ebu.io.
0.43e2.4081.ch.dab                      	IN	CNAME	retedue.chrsi.radio.ebu.io.
*.43e2.4e1.fm                           	IN	CNAME	retedue.chrsi.radio.ebu.io.
*.43e2.ch.fm                            	IN	CNAME	retedue.chrsi.radio.ebu.io.
CORRECT:
0.43ea.4081.4e1.dab                     	IN	CNAME	retedue.chrsi.radio.ebu.io.
0.43e2.4081.4e1.dab                     	IN	CNAME	retedue.chrsi.radio.ebu.io.
*.43e2.4e1.fm                           	IN	CNAME	retedue.chrsi.radio.ebu.io.
*.43e2.ch.fm                            	IN	CNAME	retedue.chrsi.radio.ebu.io.

For backward compatibility, it may also be better to keep ISO for DAB stations...


Data Importer to support relative property paths

It would be nice for Data Importer to support relative property paths in the first row.
For example:

path
./jcr:content/foo@int
./jcr:content/new/node/bar

/content/dam/test.png
123
hello world

Resulting in:

/content/dam/test.png/jcr:content@foo = 123  (as a Long)
/content/dam/test.png/jcr:content/new/node@bar = hello world (as a String)

There should be a way to specify the node-type for any. missing nodes that are part of the "relative property path".

Fixed in 5.3.0


Reduce the width of header and also responsive

Can I work on this?

Yes sure! Assigning to you

Hi @Any28Flo  any updates? have you started working on this issue?

Yes, today I work on it
El sáb., 2 de octubre de 2021 5:15 a. m., ruchika-01p <
@.***> escribió:

Hi @Any28Flo https://github.com/Any28Flo any updates? have you started
working on this issue?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ruchika-01p/Art_of_Living_YOGA/issues/1#issuecomment-932727479,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGFWEZNLGJLGFD5MWXLQKW3UE3LVTANCNFSM5E7WKI2Q
.

can i work on this?

@Shiyasmohd do you want to work on this ?

okay assigning to you

![Screenshot 2021-10-09 144209](https://user-images.githubusercontent.com/83513144/136652130-4821d470-b889-45f4-86dc-b67d1fdacb0c.png

)


Implement missing API endpoints for the train demo

Description
From https://github.com/raiden-network/light-client/issues/173#issuecomment-640456211

The CLI works perfectly fine, lightning-fast and provides almost every needed endpoint. It only misses the GET .../api/(version)/payments/:token/:partner_address.
This is currently a blocker for the demo train, as I need this endpoint to verify that the train sent a payment with a unique given identifier.

CC @eorituz
Acceptance criteria

endpoint PUT api/v1/channels is implemented
endpoint POST api/v1/payments/:token/:receiver_address is implemented
endpoint GET api/v1/payments/:token/:partner_address is implemented

Tasks

[ ]

For testing the integration image could get used

Finally the last PR was merged. Sorry for the delay.
Now, everything for the train demo should be there. Looking forward to get feedback. In this case would be no feedback the best, but... :wink:


Install failed: Connection timed out.

Help me sir, erorr after run script :
wget -O neko-rooms.sh https://raw.githubusercontent.com/m1k1o/neko-rooms/master/traefik/install
sudo bash neko-rooms.sh
Erorr :
--2024-03-12 10:44:10--  https://raw.githubusercontent.com/m1k1o/neko-rooms/master/traefik/install
Resolving raw.githubusercontent.com (raw.githubusercontent.com)... 2606:50c0:8002::154, 2606:50c0:8003::154, 2606:50c0:8001::154, ...
Connecting to raw.githubusercontent.com (raw.githubusercontent.com)|2606:50c0:8002::154|:443...
failed: Connection timed out.

Looks like your server does not have internet connection.

command appears "Enter your domain name: (e.g. example.com)" If I use IPv4, is it mandatory to fill it with water?

For https (lets encrypt) you need domain name. It does not work with only ips.

It looks like docker is not running.


forward and outputs of models

Hi, thanks for your great work!
I have two questions on the Bayesian CNN codes, which are as follows;
First, where is the forward method in the model class?
e.g., in class BBBAlexNet, there only is init, but isn't 'forward' prediction method.
Second, in main_bayesian.py, how does the network (i.e., model) output both so-called "outputs" and "_kl" ?

Hi. A little bit late, but still.
First, BBBAlexNet class is inherited from ModuleWrapper class that inherits from nn.Module and has a forward method.  Second, this forward method returns 2 values.


⚠ Bofrost.de has degraded performance

In 75e68ff, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1604 ms

Resolved: Bofrost.de performance has improved in e379abe after 7 minutes.


use wayback machine to fix offline link

that's a codeberg-pages problem and i dont whant to let our ci fail because of that
so until that's fixed I stick to the pined wayback machine ... no harm in having the wayback link


feature (#58): Enable spec running for Vue + TS files ; add base specs for each component

😀
Addresses #58
(re-opening #59 after that PR seemed to not update w/ all the commits on this branch)

hoping to see check fail for spec w/ intentional failure 🤞

Whooo! Fails intentionally & correctly w/ GitHub Action!

Now pushing spec fix...

Merging


removed direct dependency on /usr/bin/rake

Someone using rbenv could run into a situation where the ruby version
they are building the app on is a different ruby version that's
installed at /usr/bin/rake. In my specific case, I've never updated
the default system installed ruby.

motion-provisioning uses non ruby 2.0.0 features. Any help with the (CI  failed).


Pydantic2 adjustment and fixing dependencies

Remove extra arguments from pydantic Field that aren't supported in pydantic2 (warnings in v2, errors in v3)
Add missing decorator dependency (required in the storage.filesystem module)
Skip xarray_video test if library isn't installed (it is not a dependency in oceanum-python)

The following tests are failing (they are also failing in main):
=========================================================================== short test summary info ===========================================================================
FAILED tests/test_datamesh_query.py::test_query_dataset_lazy - AssertionError: assert (True and 0 == 3)
FAILED tests/test_datamesh_write.py::test_write_dataframe - pydantic_core._pydantic_core.ValidationError: 1 validation error for Datasource
FAILED tests/test_datamesh_write.py::test_write_metadata - pydantic_core._pydantic_core.ValidationError: 1 validation error for Datasource
FAILED tests/test_datasource.py::test_create_datasource - oceanum.datamesh.exceptions.DatameshConnectError: {"id":["data source with this id already exists."]}
FAILED tests/test_storage.py::test_file_not_found - aiohttp.client_exceptions.ClientConnectorError: Cannot connect to host localhost:8010 ssl:default [Connect call failed ('<IP_ADDRESS>', 8010)]
FAILED tests/test_storage.py::test_dir_not_found - aiohttp.client_exceptions.ClientConnectorError: Cannot connect to host localhost:8010 ssl:default [Connect call failed ('<IP_ADDRESS>', 8010)]
FAILED tests/test_storage.py::test_ls - aiohttp.client_exceptions.ClientConnectorError: Cannot connect to host localhost:8010 ssl:default [Connect call failed ('<IP_ADDRESS>', 8010)]
FAILED tests/test_storage.py::test_get - aiohttp.client_exceptions.ClientConnectorError: Cannot connect to host localhost:8010 ssl:default [Connect call failed ('<IP_ADDRESS>', 8010)]
FAILED tests/test_storage.py::test_open - aiohttp.client_exceptions.ClientConnectorError: Cannot connect to host localhost:8010 ssl:default [Connect call failed ('<IP_ADDRESS>', 8010)]
====================================================== 9 failed, 28 passed, 5 skipped, 52 warnings in 731.64s (0:12:11) =======================================================

Merged pyproject branch, this replaces setup.py, setup.cfg, MANIFEST.in, requirements*.txt all by just a pyproject.toml file.


[pan-cortex-xdr] Automate releaseDate and eol retrieval

Set auto config so that releaseDate and eol are retrieved from https://www.paloaltonetworks.com/services/support/end-of-life-announcements/end-of-life-summary.

Checked locally, merging.


notifications on active contact

when a message from an active contact is received, the status in notification.state isn't updated to "connected_notifications" but stays at "connected". this is somewhat impractical if trying to build some kind of taskbar/systray icon that reads notification.state.
so jackline obviously doesn't have any way to determine if I am looking. would it be possible (and useful) to update notification.state when an active contact sends a message and then let jackline wait for some kind of acknowledgement, like pressing 'return' (or teh anykey!) before changing notification.state again to just 'connected'?

yep... this is some regression, and connected with my attempts to fix #74

looks like there's some regression regarding this issue again... sorry

hmm, not really... there's a notification when talking to a@b -- focus on the window and unfocus again, next messages don't trigger an urgency thingy... likely related to xterm and i3.. (systray notification scripts shouldn't suffer from this)


update inverter and module API to be exact match to pvlib

Hi @chetan201 , @cedricleroy , and @BenBourne
I’ve been trying to make some changes to https://pvfree.alwaysdata.net to make it more useable to pvlib.

I’ve changed all of the fields to be a 1:1 match with pvlib so that you can use it directly

EG:
>>> from pvlib.pvsystem import snlinverter
>>> import requests
>>> r = requests.get('https://pvfree.alwaysdata.net/api/v1/pvinverter/?Name__iregex=^SMA')
>>> sb3000tl = r.json()['objects'][-1]
>>> snlinverter(v_dc=600, p_dc=2800, inverter=sb3000tl)
array(2750.2593123385373) AC power output

>>> sb3000tl
{'C0': -3.56e-06,
'C1': -5.4e-05,
'C2': -6.44e-05,
'C3': -0.00255,
'Idcmax': 18.0,
'Mppt_high': 480.0,
'Mppt_low': 175.0,
'Name': 'SMA America: SB3000TL-US-22 (240V) 240V [CEC 2014]',
'Paco': 3050.0,
'Pdco': 3139.323631,
'Pnt': 0.49,
'Pso': 18.62231941,
'Vac': 240.0,
'Vdcmax': 600.0,
'Vdco':<PHONE_NUMBER>,
'created_on': '2018-01-05',
'id': 2613,
'modified_on': '2018-01-05',
'resource_uri': '/api/v1/pvinverter/2613/'}

I’ve added all of the current SAM CEC inverters and Sandia modules from their https://github.com/nrel/sam repo

I changed all the links back to alwaysdata, because I can't edit this SunPower GitHub repo, and also, I'm not sure how you want to manage the Heroku app.

IMHO I really think that SunPower should let pvfree and this repo go, and let it move over to the @pvlib, assuming that @wholmgren is okay with it, so that it's easier to manage. There's not much advantage for @SunPower to hold on to this.

Thanks @cedricleroy  for merging this PR, someone needs to run the migrations manually on the dyno for the database. I can do it, but you should probably also have access to the pvfree heroku account; it's free. Do you have an Heroku account? Let me know and I'll give you access. Also, perhaps if sunpower is not willing to let pvfree go, maybe they want to pay $7/month for a dyno that never sleeps?


Revert Node to 22.11.0

Fix #11522
Related to https://github.com/nodejs/node/issues/56155

Thanks @qmonmert !

@murdos :)

So the problem was related to Node!
Thanks

@pascalgrimaud :)


[stable/prometheus] Fix strategy indentation

Move strategy under deployment instead of under container/pod
Fix comment wordings.

cc @viglesiasce

/ok-to-test


Problem matcher creates bogus annotations for failing tests

When running ExUnit tests with mix test after running this action as a step of an Actions job, the problem matchers defined create bogus annotations due to incorrectly parsed ExUnit results.
Minimal repository for reproduction: https://github.com/jswny/elixir-setup-actions-test.
The problem appears to be that it is picking up on stacktraces printed from failing tests, with each line being a single match from the problem matcher. In addition, it's pointing at the file .github, which isn't right either.
Bogus annotations created on the mix test step:

Thanks for reporting this! I'll try and fix it next week. I've noticed some odd annotations in the past but never had the chance to fix this. Good reminder that I should take a look.

@jswny I have a branch+PR that uses the VSCode plugin's problem matchers (with some minor adaptations). Can you test actions/setup-elixir@fix-37 and let me know if results seem better?

@jclem Thanks for this! It seems to have fixed the old ones (besides the process exited one, which I'm guessing is an Actions thing in general and not this specific action), and seems to nicely (as much as possible) match test failures!
I also tested with a more complicated repo with more types of test failures and it seems to work well! I agree that the multi-line matching for problem matchers is a big problem. You can't even match newlines, which would make it much easier to match ExUnit results, so you are forced to pick a single line and match that. I tried to write better matchers for ExUnit but ended up with basically the same thing you have, and deciding to try making a JS Action to be able to properly match those since they are multi-line. If I come up with anything better, I'll be sure to open up a PR and you can take a look.
Thanks for your effort!
Here's a screenshot of the new errors:

And output from a more complicated repo (too big to screenshot):
Compiling 1 file (.ex)
warning: this expression will fail with ArithmeticError
##[warning]  lib/test.ex:24

Generated test app

  1) test throws (TestTest)
##[error]     test/test_test.exs:9
     ** (throw) "Throw!"
     code: throw "Throw!"
     stacktrace:
       test/test_test.exs:10: (test)

  2) test internal function raises with stacktrace (TestTest)
##[error]     test/test_test.exs:25
     ** (ArithmeticError) bad argument in arithmetic expression
     code: assert Test.raise_stacktrace() == :ok
     stacktrace:
       (test 0.1.0) lib/test.ex:24: Test.raise_stacktrace/0
       test/test_test.exs:26: (test)

.

  3) test raises (TestTest)
##[error]     test/test_test.exs:17
     ** (RuntimeError) Raise!
     code: raise "Raise!"
     stacktrace:
       test/test_test.exs:18: (test)

  4) test multiple failing (TestTest)
##[error]     test/test_test.exs:29
     Assertion with == failed
     code:  assert 1 == 2
     left:  1
     right: 2
     stacktrace:
       test/test_test.exs:30: (test)

  5) test some passing some failing (TestTest)
##[error]     test/test_test.exs:35
     Assertion with == failed
     code:  assert 1 == 2
     left:  1
     right: 2
     stacktrace:
       test/test_test.exs:36: (test)

  6) test 1 == 2 (TestTest)
##[error]     test/test_test.exs:5
     Assertion with == failed
     code:  assert 1 == 2
     left:  1
     right: 2
     stacktrace:
       test/test_test.exs:6: (test)

.

  7) test internal function raises (TestTest)
##[error]     test/test_test.exs:21
     ** (RuntimeError) Application error
     code: assert Test.raise() == :okay
     stacktrace:
       (test 0.1.0) lib/test.ex:20: Test.raise/0
       test/test_test.exs:22: (test)

Finished in 0.1 seconds
1 doctest, 8 tests, 7 failures

Randomized with seed 986245
##[error]Process completed with exit code 1.
##[debug]Finishing: Run mix test

Great! Glad those other matchers are working better. I'm going to land that PR, then.
You are correct that the exit status matcher is a general Actions thing.

All set. Will cut a release later today.


Stored XSS #2

Steps:

Go to https://www.secretsantaorganizer.com/
Fill the fields but first, look at my photo (https://i.hizliresim.com/alRNWd.jpg)
Click "Create you event!" button.
Go to your mailbox and look at the mail from secretsanraorganizer
Click "Go to the manage page" button.
Click "Start your party" button.
Go to the mailbox, again and you have a new mail. Click it.
Click "Find out your person"

You will see xss alert.
PoC: https://www.secretsantaorganizer.com/participant/8fdfiqapde8so4gsc4k08sok8swk8cg
Regards.

Thanks a lot for your work! This kind of reviews help us stay sharp and improves the website for everyone that uses it. I just deployed some fixes.
I'll get back to you next week. This is a side-project and there are 7-800 support emails open atm. So forgive me for not spotting yours :)

Thx. I have a question. Will you reward me? :)

Yes we will figure something out. Obviously this is a valid issue and we want to know about these things. :+1: Did you get my email?

What about a mail?

I didn't see any mail.

I'll get back to you next week.

Ok. I am waiting

okay, ready
can you send me a mail at<EMAIL_ADDRESS>so I have your contact info? We'll handle it from there

I sent the mail


Follow through with PendingDeprecationWarnings

Summary
This commit promotes several pending deprecation warnings from earlier
releases to full deprecation warnings making them user facing.
Specifically, the qiskit.circuits.qpy_serialization module, the legacy
scheduling paths, and the qobj support in basic aer are now deprecated
as they have been pending deprecation for at least 1 release. This
starts the timer on pending removal per the qiskit deprecation
policy [1] so we can remove this functionality in >= 3 months and > 1
release. Meaning all of these are eligble for removal in qiskit-terra
0.23.0 (assuming that is at least 3 months after 0.21.0).
Details and comments
[1] https://qiskit.org/documentation/contributing_to_qiskit.html#deprecation-policy

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

6 of 8 (75.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.003%) to 84.361%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit/circuit/qpy_serialization.py
5
7
71.43%

Totals

Change from base Build<PHONE_NUMBER>:
0.003%

Covered Lines:
54494

Relevant Lines:
64596

💛  - Coveralls

@mtreinish The docstring of DynamicalDecoupling should indicate that it will be deprecated.

This is super stale at this point so I'm going to close this, we can revisit this in a new PR.


Update sbt-scalajs, scalajs-compiler to 1.1.1

Updates

org.scala-js:sbt-scalajs
org.scala-js:scalajs-compiler

from 1.1.0 to 1.1.1.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-js" } ]

labels: library-update, semver-patch

Codecov Report

Merging #1499 into master will decrease coverage by 0.18%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1499      +/-   ##
==========================================
- Coverage   86.02%   85.84%   -0.19%
==========================================
  Files          58       58
  Lines        2197     2197
  Branches      107      107
==========================================
- Hits         1890     1886       -4
- Misses        307      311       +4

Impacted Files
Coverage Δ

...d/src/main/scala/io/circe/NonEmptySeqDecoder.scala
91.66% <0.00%> (-8.34%)
:arrow_down:

...re/shared/src/main/scala/io/circe/JsonNumber.scala
92.77% <0.00%> (-3.62%)
:arrow_down:

.../core/shared/src/main/scala/io/circe/Decoder.scala
85.32% <0.00%> (-0.55%)
:arrow_down:

...rc/main/scala/io/circe/numbers/BiggerDecimal.scala
85.38% <0.00%> (ø)

...les/core/shared/src/main/scala/io/circe/Json.scala
80.44% <0.00%> (+0.55%)
:arrow_up:

...scala/io/circe/shapes/LabelledHListInstances.scala
96.29% <0.00%> (+3.70%)
:arrow_up:

Continue to review full report at Codecov.

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


🌱 Autorun make generate tasks on each dependabot PR and commit the changes to a branch

What this PR does / why we need it:
This improvement comes from cluster-api and allows to merge dependabot PRs more effectively, allowing them to pass CI jobs, but by sacrificing granularity of the bumps. If some dependency requires to be updated in the test folder as well, a dependabot PR will run a go mod tidy on test folder and commit the changes. This will allow for direct merge in many occasions.
Origin: https://github.com/kubernetes-sigs/cluster-api/pull/4845
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #

/lgtm