brew reinstall inconsistent with passed in formula options

Please note we will close your issue without comment if you delete, do not read or do not fill out the issue checklist below and provide ALL the requested information. If you repeatedly fail to use the issue template, we will block you from ever submitting issues to Homebrew again.

[x] are reporting a bug others will be able to reproduce and not asking a question. If you're not sure or want to ask a question do so on our Discourse: https://discourse.brew.sh
[x] ran a brew command and reproduced the problem with multiple formulae? If it's a problem with a single, official formula (not cask) please file this issue at Homebrew/homebrew-core: https://github.com/Homebrew/homebrew-core/issues/new/choose. If it's a brew cask problem please file this issue at https://github.com/Homebrew/homebrew-cask/issues/new/choose. If it's a tap (e.g. Homebrew/homebrew-php) problem please file this issue at the tap.
[x] ran brew update and can still reproduce the problem?
[x] ran brew doctor, fixed all issues and can still reproduce the problem?
[x] ran brew config and brew doctor and included their output with your issue?

I've found this while trying to debug https://github.com/Homebrew/homebrew-core/issues/30614.
Issue description
According to the manpage brew reinstall should

Uninstall and then install formula (with existing install options).

and should not honor passed in formula options (e.g. --without-dependency) unlike brew install.
But it looks like that this is only partly the case (they are ignored in some cases but taken into account in other cases) leading to a few inconsistencies. After a brew upgrade of that formula they are completely taken into account for said formula, which is the opposite of what is documented.
Reproduction:

brew uninstall yarn node if needed first
brew install yarn (installing node and yarn)
cat $(brew --prefix yarn)/INSTALL_RECEIPT.json will include (unimportant properties omitted): used_options":[],"unused_options":["--without-node"],"runtime_dependencies":[{"full_name":"node","version":"10.8.0"}] with --without-node correctly tagged as a unused_options and node correctly tagged as a runtime_dependencies
now run brew reinstall yarn --without-node

Actual behavior: cat $(brew --prefix yarn)/INSTALL_RECEIPT.json will include "used_options":["--without-node"],"unused_options":[],"runtime_dependencies":[{"full_name":"node","version":"10.8.0"}] On the one hand node is still a runtime_dependencies dependency and the passed in formula option (--without-node) is ignored and the existing install options are used instead as documented. But on the other hand --without-node is tagged as a used_options (it honors the passed in formula option here), while it should still be tagged as unused_options (the existing install options behavior as documented). This is quite inconsistent.
Expected behavior (as documented): Have node tagged as a runtime_dependencies and --without-node as a unused_options option = reinstall follows existing install options completely.

brew upgrade yarn to a new version (adding a fake revision to the formula is enough for the reproduction) and run cat $(brew --prefix yarn)/INSTALL_RECEIPT.json:

Actual behavior "used_options":["--without-node"],"unused_options":[],"runtime_dependencies":[] Now we completely respect the --without-node formula option passed to brew reinstall, which should have been ignored following the documentation.
Expected behavior (as documented): used_options":[],"unused_options":["--without-node"],"runtime_dependencies":[{"full_name":"node","version":"10.8.0"}] (the same as the expected behavior of 3.)

Possible fix
brew reinstall should set used_options and unused_options according to the existing install options and not the the one which are passed to brew reinstall. All passed in formula options should be ignored as documented.
Alternative fix
Reading the code at https://github.com/Homebrew/brew/blob/master/Library/Homebrew/cmd/reinstall.rb#L36-L39 I think the intended behavior might be to merge the newly passed in formula options with the existing ones (which is to be hones what I've expected before checking the man page). If that's the case it should be documented more clearly in the man page and we still have kind of the opposite bug: node shouldn't be a runtime_dependencies already after the brew reinstall yarn --without-node in step 3 anymore.
Note: The issue can also be reproduced by replacing brew reinstall yarn --without-node with brew upgrade yarn --without-node in step 3 too.
Can someone please confirm if it is intended that brew reinstall <formula> and brew upgrade <formula>  accept formula options and a supposed the merge them with the previously used ones (unlike documented in the manpage)?
brew doctor and config
$ brew doctor
Your system is ready to brew.
$ brew config
HOMEBREW_VERSION: 1.7.1-170-gf565646
ORIGIN: https://github.com/Homebrew/brew
HEAD: f5656463dd0e3ff1d4f40248e6833fe41ae7ca6d
Last commit: 2 hours ago
Core tap ORIGIN: https://github.com/Homebrew/homebrew-core
Core tap HEAD: 258820d458b0bc6dace635474a55b89cec25d282
Core tap last commit: 27 minutes ago
HOMEBREW_PREFIX: /usr/local
HOMEBREW_DEV_CMD_RUN: 1
CPU: quad-core 64-bit kabylake
Homebrew Ruby: 2.3.7 => /System/Library/Frameworks/Ruby.framework/Versions/2.3/usr/bin/ruby
Clang: 9.1 build 902
Git: 2.18.0 => /usr/local/bin/git
Curl: 7.54.0 => /usr/bin/curl
Java: N/A
macOS: 10.13.6-x86_64
CLT: <IP_ADDRESS>.1.1528165917
Xcode: 9.4.1
XQuartz: N/A

Can someone please confirm if it is intended that brew reinstall  and brew upgrade  accept formula options and a supposed the merge them with the previously used ones (unlike documented in the manpage)?

Yeh, I think this would be the ideal behaviour. At this point: who knows what's intended 😆

Yeh, I think this would be the ideal behaviour. At this point: who knows what's intended 😆

So than this bug report should be about having node still as a brew reinstall yarn --without-node  in step 3 after brew reinstall yarn --without-node, which is incorrect and about updating the man page to document the formula option merging for reinstall more clearly.


Topbar component as module(mixin)

It's a feature request rather than a bug. Now that Foundation 6 is on its way, I think it would be nice to have the Topbar (and any other components) as a module (mixins). They already have it for Grids, Blockgrids and such. Like for a 6 column grid you'd have something like this:
@include grid-column($columns: 6, $last-column: false, $center: false, $offset: 0, $push: 0, $pull: 0, $collapse: false, $float: left);

I have no clue if this feature has already been accounted for in Foundation 6. In Foundation 5.5.2 you have to set $include-html-top-bar-classes  to true to get the topbar html classes in your CSS. But think of it as a component which can be controlled with a mixin...

Top-bar is a very complex component that does so many things. For that reason it doesn't have a mixin. Everything else has mixins. In the new version we have componentized the menu pieces so that you can make a more custom nav easily. Each of these have their own mixin.


Can't successfully build on 3.3

During the build I am seeing
ERROR: _fs_changed: Cannot export project with preset 'Windows Desktop' due to configuration errors:
No export template found at the expected path:
/github/home/.local/share/godot/templates/3.3.stable.mono/windows_64_debug.exe
No export template found at the expected path:
/github/home/.local/share/godot/templates/3.3.stable.mono/windows_64_release.exe

   At: editor/editor_node.cpp:732.
ERROR: _fs_changed: Project export failed for preset 'Windows Desktop', the export template appears to be missing.
   At: editor/editor_node.cpp:755.

Which seems to indicate that there are export templates missing. Am I missing something?
name: "godot-ci export"
on: push

env:
GODOT_VERSION: 3.3
EXPORT_NAME: combat-engineers

jobs:
export-windows:
 name: Windows Export
 runs-on: ubuntu-latest
 container:
   image: barichello/godot-ci:mono-3.3
 steps:
   - name: Checkout
     uses: actions/checkout@v2
     with:
       lfs: true
   - name: Windows Build
     run: |
       mkdir -v -p build
       godot -v --export "Windows Desktop" build/$EXPORT_NAME.exe
   - name: Upload Artifact
     uses: actions/upload-artifact@v1
     with:
       name: windows
       path: build

export-linux:
 name: Linux Export
 runs-on: ubuntu-latest
 container:
   image: barichello/godot-ci:mono-3.3
 steps:
   - name: Checkout
     uses: actions/checkout@v2
     with:
       lfs: true
   - name: Linux Build
     run: |
       mkdir -v -p build
       godot -v --export "Linux/X11" build/$EXPORT_NAME.x86_64.bin
   - name: Upload Artifact
     uses: actions/upload-artifact@v1
     with:
       name: linux
       path: build

You need to download export templates before you can export.


Example using tfvars -> main -> modules

I'm sure I'm missing one singular piece of logic to connect these dots, but I'm trying to create the mechanism described in the section regardling "Larger Teams May Need to Use a Development Pipeline".  Specifically I've got tfvars in my specific environment dirs (aka:  Figure 6-4).  I have my source defined in that tfvars, but init, or get, or plan for that matter, are not downloading anything.  It's a constant "no configuration files present".
However, as I wrote this, I finally solved the problem.
The key issue deals with moving to the terraform init method.  In your book, although you stated that we would need to make a wrapper to perform terraform init, it was a small logic leap that I missed which is the terraform init /source-dir /target-dir mechanism.
I was assuming (incorrectly) that init should be fed the tfvars and realize/use the source.  this is not the case, and init needs to be done "manually" (seperately from tfvars) using the "SAME SOURCE" and choosing our tfvars dir as the destination (or not, but that would make the plan and apply much more difficult command lines).

Check out https://github.com/gruntwork-io/terragrunt. It does exactly what I describe in the book :)

So, did I understood correctly, example from the book not works without a terragrunt?

The book outlines the basic approach to using pipelines. That approach is not natively supported by Terraform itself yet, so you have to implement it yourself. There are many ways to do it. One way is to use Terragrunt. Another is to use S3 as described by Kief Morrison.

No - I don't believe there was specific code in the book to cover this - and honestly, I'm using all of the advanced topics covered - DRY, modular, remote state, sub-dirs, live layouts, larger teams, etc.  For this issue - i think Yevgeniy is spot on, and the way that I'm doing pipelines is using Terragrunt - which is an excellent wrapper for doing these advanced functions, but is slightly outside the scope of the book.
Besides, I am quite certain, if you run into issues in this area, and add to this thread, someone (maybe even the author himself) would jump in and help (he did several times for me!).

I believe the question here was answered. Closing this ~3 year old issue.


code modification & detailed documentation

I am working on building a code integrity checker for Electron apps but I am lost at sea trying to understand the code.
It would be really helpful if you could tell me how atom/browser/api/browser-window.js connects to the cpp code? and Which are the exact processes which start when the electron app is launched?
And
where can I find detailed documentation of the code in atom/ and chromium_src/ ?

There is no documentation on the code, the only things helpful are the code comments and code itself.

so if someone wants to contribute, then how to go about getting started and understanding where to begin?


Add demo with validation-engine error message style

See http://www.position-relative.net/creation/formValidator/demos/demoValidators.html

I'm sorry for the lack of activity on this issue. Instead of leaving it open any longer, I decided to close old issues without trying to address them, to longer give the false impression that it will get addressed eventually.
To the reporter (or anyone else interested in this issue): If you're affected by the same issue, consider opening a new issue, with a testpage that shows the issue. Even better, try to fix the issue yourself, and improve the project by sending a pull request. This may seem daunting at first, but you'll likely learn some useful skills that you can apply elsewhere as well. And you can help keep this project alive. We've documented how to do these things, too. A patch is worth a thousand issues!

We added some demos with newer frameworks lately. I guess we dont need this particular demos anymore.
Feel free to speak up in case you still think this is important Jörn.


Mislabeled PPC64LE Releases

I noticed on the releases the the PowerPC releases were named tini-ppc64el. This there any reason behind this?
When running on ppc64le, uname and other architecture identifiers return PowerPC as ppc64le not ppc64el. This naming confusion is causing problems with automation, since I usually just grab uname -m or System.getProperty("os.arch"), and form the URL for the download.
Like this:
ADD https://github.com/krallin/tini/releases/download/v0.18.0/tini-${tini_suffix} /tini

${tini_suffix} being the result I get back from  uname -m or System.getProperty("os.arch").
Is there anyway you would be willing to change this?

(Not a maintainer here, just offering info)
uname -m is really unreliable for userspace architecture detection, especially when using setarch/linux32 on a 64bit system, so I believe tini releases use dpkg architecture notation (dpkg --print-architecture), which should give you the architecture of your userspace environment instead of the architecture of your kernel.

@tianon I was mostly using uname -m as example. dpkg --print-architecture is good but not included by default in a lot of distros like CentOS or ClefOS. If I am not forgotten, I think you might need to install/enable epel which is not ideal.
I am not sure there is a good solution since in most languages like Java or Go, when you use GO ARCH or do os.arch they return ppc64le.
I want to avoid additional logic in build scripts or Dockerfiles with converting ppc64le to ppc64el.

Yeah, @tianon is basically spot on here with the reasoning behind why this is called the way it is. I wouldn't want to change this, but I sounds like it might be reasonable to add aliases to what uname would return. I'll take a look.


Custom model class number problem

Dear @jkjung-avt,
I replaced the yolov3-416.weights with my own model (after rename it). Then I tried to build the onnx from it and reach this issue.

I assume, after some research on a similar issue (#58) that is an class number problem.  I was wondering if the correction was still applying (change the filters lines in the cfg file) If so, every filter that contain 255 or only specific ones ?
thanks for your help !

Could you provide your custom yolov3-416.cfg and weights files for checking?  I will take a look.

It won't be necessary, I solved it by myself. It was, indeed, a number class pb, I changed the 3 good layers. Anyway,  after that I reached a tensort problem.

Sounded like the #51 but i solved it by commenting the line builder.fp16_mode = True
My current problem is this one : what does mean pos = np.where(box_class_scores >= conf_th)

I think this line of code, pos = np.where(box_class_scores >= conf_th), is trying to pick out inference results (corresponding to anchor boxes) with class_scores over the confidence threshold.
According to this SO post, the warning is likely due to "NaN" (Not a Number, or infinity) outputs by your model.  You could add code to filter those "NaN" values out, or try to train your model better.
By the way, the "weights are out of fp16 range" issue could usually be resolved if you re-train your model with proper "weight decay".

Thanks for your answer, yeah the model used on this image was not completely trained. I will try again with a more developped one in few hours.
Concerning the decay, unfortunaly, I do not have the time and experience to solve it by this way.
Thanks again, you, your repository and your answers

Is there a way to reduce the confidence threshold?


Please merge revision 289a22182e629e into dev channel

@whesse @mit-mit @kasperl
Make BASE-64 decoder accept URL-safe alphabet and percent-escaped padding.
Fixes issue #24603
Please merge revision 289a22182e629e
Merged cleanly on 1.13.0-dev.7.3

Merged to dev in version 1.13.0-dev.7.4


Fix testing DB creation

This PR fixes a bug introduced in #424 that prevents the MySQL server from starting.
The backticks to be escaped. Otherwise, the shell will try to execute the contents, resulting in the following error:
/docker-entrypoint-initdb.d/10-create-testing-database.sh: line 3: testing%: command not found
ERROR 1064 (42000) at line 2: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '.* TO 'sail'@'%'' at line 1

Thanks @jessarcher


error triggering graph validation

Reported by @ami-day

When i try to trigger graph validation I get a 401 error: https://contribute.data.humancellatlas.org/submissions/detail?uuid=fce1dc75-017f-40e7-bc17-b57af4957d46&project=ec6476ee-2949-41f3-947b-8eef41d6d3ac "an error occurred while triggering validation -  Http failure response for https://api.ingest.archive.data.humancellatlas.org/submissionEnvelopes/62b9d234b85bb84636c610da/graphValidationRequestedEvent: 401 OK "

There are no submissions for this dataset

@ami-day see comment above. Can we close this ticket? If not, provide more detail, please

no response - closing.


Allow to share an analysis with other users

It would be nice if it were possible to share the results of an analysis with other QIITA users.

Note for devs: the database and the analysis object already supports this. It should be just a matter of making it possible through the GUI.

This was solved in https://github.com/biocore/qiita/pull/1768


Remove say hi command

Description
Motivation
The say_hi command has nothing to do with privy. A better fit for it would be the dummy app in the discord engine
Solution
Removed the rake task that creates the command, the resolver, the serializer and called the discord API to remove the command from the Privy bot
Deploy TODOs
None
Checklist

[ ] I added/updated unit tests for these changes
[ ] I tested manually these changes
[ ] I updated the issue tracker and linked my PR to my ticket.

Steps to reproduce
say_hi now belongs to DummyBot, not privy

good call @ImMPrada! Done


Fix run's support of directories in the classpath

Fixes #3504

To validate this pull request against other sbt modules, please visit this link.

To validate this pull request against other sbt modules, please visit this link.


tests(balancer) increase timeout of #hostname does not perform health

checks when disabled test to avoid timeouts on CI environment

Superseded by https://github.com/Kong/kong/pull/5601/files as this PR did not address the root issue.


Fixed shapes' creation function declaration in TypeScript definition

Hello,
I recently updated the PlanckJS dependency of my project and it gave me problems with shapes.
Indeed the functions that are exposed to create shapes are:
exports.Circle = __webpack_require__(23);
exports.Edge = __webpack_require__(24);
exports.Polygon = __webpack_require__(21);
exports.Chain = __webpack_require__(28);
exports.Box = __webpack_require__(45);

source: dist/planck.js
Which doesn't correspond to TypeScript definitions:
export function CircleShape(position: Vec2, radius?: number): CircleShape;
export function CircleShape(radius?: number): CircleShape;
// [...]
export function EdgeShape(v1: Vec2, v2: Vec2): EdgeShape;
// [...]
export function PolygonShape(vertices: Vec2[]): PolygonShape;
// [...]
export function BoxShape(hx: number, hy: number, center?: Vec2, angle?: number): BoxShape;
// [...]
export function ChainShape(vertices: Vec2[], loop?: boolean): ChainShape;

source: lib/shape/index.d.ts
This PR solves the problem.

Oh actually, we still need to support circle = CircleShape() as well.


Prettier Standard formatting

This should be merged before #794 so I can rebase that branch with master. Thanks to @MSevey for the idea.
This PR is the simply the result of running prettier-standard 'js/**/*.js' 'plugins/**/*.js' in the repo.
Standard JS (https://github.com/standard/standard) is a popular style guide. I've used quite a few (Airbnb, Github, Google) and I always find myself coming back to Standard.
Prettier (https://github.com/prettier/prettier) is an auto-formatting tool that keeps code style consistent.

this mostly looks fine to me. It's a bit aggressive on splitting up long lines (which traditionally we don't care about so much), and I also dislike indenting case blocks in switch statements; is the formatter configurable? I imagine we'd want to add it to our package.json somehow for easy invocation?

The formatter is configurable - the scripts exist in package.json under npm run format as well as some git hooks on the larger PR, but for this one I just manually ran the script. I'm currently working on some failing tests that still exist on master. Once that's done, I'd like to rebase this PR so that Travis builds work properly again, so hold off on merging this for now.

@lukechampine I removed the unnecessary whitespacing. You are right. Lots of whitespace that was accidentally being left in the HTML. But that's also why I prefer using {' '} for whitespacing in JSX - it makes it really clear that you have spaces in text.
The default width for long lines in Standard JS is 80 chars. I'm inclined to stick with the defaults (look at how clean the eslintrc is compared to the old one) for now, just because it's a well-known standard that people are used to. It's super easy to change that in the future, we just add a custom rule and run npm run format.
I also updated the babelrc so that webpack builds properly, and so that tests passed based on my fix from yesterday's PR.
I've rebased from master and this PR should be good to go.

Is it able to distinguish between text lines and code lines?
Not a big deal / blocking factor for me either way. I would probably target 80 chars for text lines (since readability of a broken up line is not impacted) and 160 chars for code lines (since readability takes a huge hit if you split up a code line - unless you do it very elegantly, which I don't imagine a script is able to do.


Crosslink using publish link

This is merely a link-fixing and markdownlint fixing set of commits, there are no technical changes, so I'm merging this immediately.


Editing contract weirdness

I haven't been able to isolate the cause of the issue yet.

Haven't run into this one yet. Let us know if you figure out how to replicate it.

Seems to be fixed.


persisted.ts : Strengthen conditional checking for localStorage to prevent "null" from being used

Summary
DOM storage can be disabled for the Android WebView which results in "window.localStorage" being "null". Currently for technical reasons I believe Gopuff's Android app has DOM storage disabled. So because of this we are adding an extra condition to check for the existence of "window.localStorage". If not truthy, then ternary will inevitably return the default "loc" object as the stubbed implementation for "localStorage".
Testing
The discovery of the issue and what needed to be remediated involved the following in the context of me being a Gopuff developer:

Installed and launched debug APK build of Gopuff Android app
Opened chrome://inspect in Chrome browser tab
See the following error which originates from persisted.ts:

Unit Tests
I am not sure how we would want to go about nullifyinglocalStorage on the window for a unit test. But that's what we need to do in order to test this case where there's a window object(i.e. isBrowser) but the browser's local storage is disabled(i.e. localStorage === null)

Looks great! thanks for the PR 🙌


The request signature we calculated does not match the signature you provided.

SignatureDoesNotMatch: The request signature we calculated does not match the signature you provided. Check your key and signing method.
[tusd] event="ResponseOutgoing" status="500" method="POST" path="" error="s3store: unable to create multipart upload: SignatureDoesNotMatch: The request signature we calculated does not match the signature you provided. Check your key and signing method.
The error occurred whether Large files tried to upload

We can't help you if you don't fill out the questions for a bug report. Please use the template as you were prompted when creating a bug report (you can find the template here: https://github.com/tus/tusd/blob/master/.github/ISSUE_TEMPLATE/bug_report.md).

s3:AbortMultipartUpload
s3:DeleteObject
s3:GetObject
s3:ListMultipartUploadParts
s3:PutObject
These are the permissions allowed.

Describe the bug
There is no problem with Small Size Files It Will return 100%,(~50MB) , Whether Large files tried to upload Browser Shown The Error 500
POST https://server.com/files/ 500
tusd.php:27 Failed because: Error: tus: unexpected response while creating upload

In Server
[tusd] event="ResponseOutgoing" status="500" method="POST" path="" error="s3store: unable to create multipart upload: SignatureDoesNotMatch: The request signature we calculated does not match the signature you provided. Check your key and signing method.
To Reproduce
Steps to reproduce the behavior:

Execute command '...'
Start upload '....'
See error

Expected behavior
Same as Previous
Setup details
Please provide following details, if applicable to your situation:

Operating System: Linux
Used tusd version: n/a (not available)
Used tusd data storage: AWS S3 (Custom Server Wasabi)
Used tusd configuration:
-s3-bucket upload-store -s3-endpoint https://s3.eu-central-1.wasabisys.com
-max-size<PHONE_NUMBER>  -timeout 0
Used tus client library: tus-js-client

s3:AbortMultipartUpload
s3:DeleteObject
s3:GetObject
s3:ListMultipartUploadParts
s3:PutObject
These are the permissions allowed.

That looks very much like an incompatibility issue on Wasabi's side. tusd works fine with AWS S3 and Minio, and there are other problems reported in combination with Wasabi, e.g. https://github.com/shrinerb/shrine/issues/355
Are you sure you properly setup your credentials using the AWS_ACCESS_KEY_ID, AWS_SECRET_ACCESS_KEY and AWS_REGION environment variables? Are you able to get it working using AWS S3 itself?

metadata: {
	filename: {customname[a-zA-Z0-9]}.ext
	filetype: file.tye
}

Solved my Problem With wasabi

Hi
i use s3 as backend and got randomly error on SignatureDoesNotMatch but in the error do not mention s3store
these are the last 10 tusd's log:
[tusd] 2022/07/25 13:06:02 event="ChunkWriteComplete" id="e4cf9cf969f3d6563d0596747bb6f2a7+2~WSdh726Nr02zF1QhEgGpzWf6B9Lq1QR" bytesWritten="500000"  [tusd] 2022/07/25 13:06:02 event="ResponseOutgoing" status="204" method="PATCH" path="e4cf9cf969f3d6563d0596747bb6f2a7+2~WSdh726Nr02zF1QhEgGpzWf6B9Lq1QR" requestId=""  [tusd] 2022/07/25 13:06:02 event="RequestIncoming" method="PATCH" path="e4cf9cf969f3d6563d0596747bb6f2a7+2~WSdh726Nr02zF1QhEgGpzWf6B9Lq1QR" requestId=""  [tusd] 2022/07/25 13:06:02 event="ChunkWriteStart" id="e4cf9cf969f3d6563d0596747bb6f2a7+2~WSdh726Nr02zF1QhEgGpzWf6B9Lq1QR" maxSize="500000" offset="48500000"  [tusd] 2022/07/25 13:06:03.960933 event="HookInvocationStart" type="post-receive" id="e4cf9cf969f3d6563d0596747bb6f2a7+2~WSdh726Nr02zF1QhEgGpzWf6B9Lq1QR"  [tusd] 2022/07/25 13:06:03.969287 event="HookInvocationFinish" type="post-receive" id="e4cf9cf969f3d6563d0596747bb6f2a7+2~WSdh726Nr02zF1QhEgGpzWf6B9Lq1QR"  [tusd] 2022/07/25 13:06:04.961186 event="HookInvocationStart" type="post-receive" id="e4cf9cf969f3d6563d0596747bb6f2a7+2~WSdh726Nr02zF1QhEgGpzWf6B9Lq1QR"  [tusd] 2022/07/25 13:06:04.968753 event="HookInvocationFinish" type="post-receive" id="e4cf9cf969f3d6563d0596747bb6f2a7+2~WSdh726Nr02zF1QhEgGpzWf6B9Lq1QR"  [tusd] 2022/07/25 13:06:35 event="ChunkWriteComplete" id="e4cf9cf969f3d6563d0596747bb6f2a7+2~WSdh726Nr02zF1QhEgGpzWf6B9Lq1QR" bytesWritten="0"  [tusd] 2022/07/25 13:06:35 event="ResponseOutgoing" status="500" method="PATCH" path="e4cf9cf969f3d6563d0596747bb6f2a7+2~WSdh726Nr02zF1QhEgGpzWf6B9Lq1QR" error="SignatureDoesNotMatch:          status code: 403, request id: tx00000e18bb708b10ffb71-0062de5613-158baee2-default, host id: " requestId=""

@mosi-kha This issue has been closed for three years. Please open a new issue and include all relevant details as asked in the issue template.

#776


sign words images disapper

when not in debug mode the words images are not shown, need to be fixed!!!

So fix it


Some questions regarding the results

Thank you for the author's reply, but why is the value of video mAP that I received significantly different from the value given in your paper, about 3% or 5%? Should I make modifications to certain parameters to achieve the results shown in your paper？I hope the author can provide some suggestions. Thank you very much

From the configs, try replacing the structure from hitnet to serial. And please tell me the results you got

Do you mean to change the STRUCTURE: "hitnet" to STRUCTURE: "serial"? However, an error occurs when the training is performed after modification:
Traceback (most recent call last):
File "train_net.py", line 255, in
main()
File "train_net.py", line 245, in main
args.no_head)
File "train_net.py", line 100, in train
mem_active,
File "/17106/zwx/HIT-master/hit/engine/trainer.py", line 59, in do_train
loss_dict, weight_dict, metric_dict, pooled_feature = model(slow_video, fast_video, boxes, objects, keypoints, mem_extras)
File "/root/anaconda3/envs/hit/lib/python3.7/site-packages/torch/nn/modules/module.py", line 727, in _call_impl
result = self.forward(*input, **kwargs)
File "/17106/zwx/HIT-master/hit/modeling/detector/action_detector.py", line 20, in forward
result, detector_losses, loss_weight, detector_metrics = self.roi_heads(slow_features, fast_features, boxes, objects, keypoints, extras, part_forward)
File "/root/anaconda3/envs/hit/lib/python3.7/site-packages/torch/nn/modules/module.py", line 727, in _call_impl
result = self.forward(*input, **kwargs)
File "/17106/zwx/HIT-master/hit/modeling/roi_heads/roi_heads_3d.py", line 12, in forward
result, loss_action, loss_weight, accuracy_action = self.action(slow_features, fast_features, boxes, objects, keypoints, extras, part_forward)
File "/root/anaconda3/envs/hit/lib/python3.7/site-packages/torch/nn/modules/module.py", line 727, in _call_impl
result = self.forward(*input, **kwargs)
File "/17106/zwx/HIT-master/hit/modeling/roi_heads/action_head/action_head.py", line 44, in forward
x, x_pooled, x_objects, x_keypoints, x_pose = self.feature_extractor(slow_features, fast_features, proposals, objects, keypoints, extras, part_forward)
File "/root/anaconda3/envs/hit/lib/python3.7/site-packages/torch/nn/modules/module.py", line 727, in _call_impl
result = self.forward(*input, **kwargs)
File "/17106/zwx/HIT-master/hit/modeling/roi_heads/action_head/roi_action_feature_extractor.py", line 145, in forward
ia_feature, res_person, res_object, res_keypoint = self.hit_structure(person_pooled, proposals, object_pooled, objects, hands_pooled, keypoints, memory_person, None, None, phase="rgb")
ValueError: too many values to unpack (expected 4)
The result shows an incorrect number of parameters

The result before modification was:
VideoAP_ 0.5: 84.80
VideoAP_ 0.2: 86.40

Yes, hitnet to serial as you did.
Thanks for reporting the numbers. Others have pointed out disparities of results for different runs (though I used seed). I will fix the serial bug and get back to you.

By the way, what is the frame mAP result for that run?

frame mAP : 81.02

Hello author, I found that the weights of some parameters were not loaded during the training process, will this affect the final training results and mAP calculation? If so, what is the cause? Is the weight file I loaded incomplete? Could the author give me some suggestions, thank you very much.
Here are some log messages from the training process:
2023-11-12 10:39:16,803 hit.utils.model_serialization INFO: backbone.slow.res_nl4.res_2.btnk.conv3.bn.weight                                      loaded from backbone.slow.res_nl4.res_2.btnk.conv3.bn.weight                                      of shape (2048,)
2023-11-12 10:39:16,803 hit.utils.model_serialization INFO: backbone.slow.res_nl4.res_2.btnk.conv3.conv.weight                                    loaded from backbone.slow.res_nl4.res_2.btnk.conv3.conv.weight                                    of shape (2048, 512, 1, 1, 1)
2023-11-12 10:39:16,803 hit.utils.model_serialization INFO: roi_heads.action.feature_extractor.fc1.bias will not be loaded.
2023-11-12 10:39:16,803 hit.utils.model_serialization INFO: roi_heads.action.feature_extractor.fc1.weight will not be loaded.

In your case, weights not being loaded might be because you are training from an existing checkpoint.
I uploaded the pretrained model. Possibly due to the dataset being very small, different runs might give different results. I also tested different checkpoints (for JHMDB the model converges very fast then start overfitting).


Document named maps and static maps into the reference

We currently have documentation about the Maps API reference here: https://carto.com/developers/maps-api/reference/. Part of the documentation should be in there is being exposed as guides content:

Named Maps: https://carto.com/developers/maps-api/guides/named-maps/
Static Maps: https://carto.com/developers/maps-api/guides/static-maps-API/

In the reference, we don't provide any documentation

So, we should document those endpoints, so that the contents are displayed under the reference path.
cc/ @luisbosque @ramiroaznar

Is this issue in progress?

Is in TODO, do you need it ASAP?

Sorry, I assumed that it was assigned to be done in the RT.
I do not need ASAP. I just wanted to check its status. Thanks, man.

Let me know if you need this to be added again to the RT kanban.

Yes, we need that, @ramiroaznar.

Don't know why this was moved out of the RT flow, @jgoizueta will take care of this next turn.

Nice, @ramiroaznar.
@jgoizueta, here to help. If you need context, information, or any other thing that can help you to understand how we are currently documenting other similar APIs, just whisper. 😃

PR: https://github.com/CartoDB/Windshaft-cartodb/pull/1023
Ready for CR

You can check it here: ttp://test-mapsapi-975.developers.carto-staging.com/developers/maps-api/reference/#tag/Named-Maps

Done


Enable page-feedback form for doc pages

Contributes to #4769
Preview: e.g., https://deploy-preview-4772--opentelemetry.netlify.app/docs/getting-started/

Screenshots

I restarted the Netlify build, and the build runs fine now: https://app.netlify.com/sites/opentelemetry/deploys/66832c787ae60bb25b90968b. Merging.


404 on overview page

Thanx Pascal and Sebastian for mentioning this...

damned, it was yaml again.


Scene onPress doesn't work anymore?

I used to have an onPress on one of my Scenes, but is not working anymore.
<Scene key="add" component={Message}  title="Add" icon={TabIcon}  hideNavBar={true}
   onPress={()=> {
   alert('a');
   return false;
}}></Scene>

Tips?

Sorry, it was quite unclear syntax and it will not be supported by v4. v4 uses ReactNavigation TabNavigator as tabbar. You may pass own tabbar, please check TabNavigator docs.

Ok, thanks for the feedback!


fix: handle compressed timestamp

decoder: Compressed timestamp header message has local message type 0-3 positioned at bit 5-6 (note: normal message has local message type 0-15 positioned at bit 0-4). Bit 6 overlap with proto.MesgDefinitionMask, the only thing that differentiate them is the bit 7.  So we must take bit 7 into consideration when checking the header type, is it message definition or is it message data. Since only message has value at header bit 7, we we must include proto.MesgCompressedHeaderMask to do mask, only then we can get the correct header type.
encoder: Previous implementation of timestamp compression has a bug, where we always has timestampReference 1. It should  have only been updated when Rollover Event occurs.

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

Project coverage is 100.00%. Comparing base (1c69559) to head (90dc307).
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##            master      #438   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           42        42
  Lines         3659      3661    +2
=========================================
+ Hits          3659      3661    +2

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


iOS Text field textSelectionHandleColor is not correct

iOS Text field textSelectionHandleColor is not correct
My theme config contains this:
    cursorColor: ThemeColors.accent,
    textSelectionHandleColor: ThemeColors.accent,
    textSelectionColor: ThemeColors.accent.withOpacity(0.4),

But still my iOS text field textSelectionHandleColor is blue.

Latest Flutter verions: 1.17.5
No Warnings

Hi @vanlooverenkoen
I can reproduce the issue
I see there's an open issue addressing the case you described. https://github.com/flutter/flutter/issues/43887
Please follow up on that issue, I'm closing the current one as a duplicate.
If you disagree, please write in the comments and I will reopen it.
Thank you


Crash in Net::HTTP patch

In some capybara test, I get this:
NoMethodError: undefined method `finish' for nil:NilClass
from newrelic_rpm-<IP_ADDRESS>9/lib/new_relic/agent/instrumentation/net.rb:42:in `ensure in request_with_newrelic_trace'
Caused by NoMethodError: undefined method `add_request_headers' for nil:NilClass
from newrelic_rpm-<IP_ADDRESS>9/lib/new_relic/agent/instrumentation/net.rb:31:in `request_with_newrelic_trace'

Not sure why this happens, but it obfuscate a (normal) capybara failure on assert_current_path.

Interesting!
Can you give me some more details about the test that is failing?

Hey @cyrilchampier, if this is still an issue, please let me know here.  Otherwise, I'll close this issue tomorrow.

Hi, sorry was in holidays.
I do not remember how to reproduce this, but yes, I have this kind of logs:
[2019-10-31 14:06:48 +0100 dcyrils-macbook-pro.home (12006)] INFO : Installing notification based Action View instrumentation
[2019-10-31 14:06:48 +0100 dcyrils-macbook-pro.home (12006)] INFO : Installing notifications based Action Cable instrumentation
[2019-10-31 14:06:48 +0100 dcyrils-macbook-pro.home (12006)] INFO : Finished instrumentation
[2019-10-31 14:06:49 +0100 dcyrils-macbook-pro.home (12006)] INFO : Doing deferred dependency-detection before Rack startup
[2022-02-10 17:04:41 +0100 dcyrils-macbook-pro.home (12006)] WARN : Agent received a ForceDisconnectException from the server, disconnecting. (410: Gone)
[2022-02-10 17:05:20 +0100 dcyrils-macbook-pro.home (12006)] ERROR : Exception during Tracer.start_external_request_segment
[2022-02-10 17:05:20 +0100 dcyrils-macbook-pro.home (12006)] ERROR : RuntimeError: failed to get urandom
[2019-10-31 14:08:34 +0100 dcyrils-macbook-pro.home (12231)] INFO : newrelic_developer_mode does not have a corresponding configuration setting (developer_mode does not exist).
[2019-10-31 14:08:34 +0100 dcyrils-macbook-pro.home (12231)] INFO : Run `rake newrelic:config:docs` or visit https://newrelic.com/docs/ruby/ruby-agent-configuration to see a list of available configuration settings.
[2019-10-31 14:08:34 +0100 dcyrils-macbook-pro.home (12231)] INFO : Starting the New Relic agent version <IP_ADDRESS>9 in "test" environment.
[2019-10-31 14:08:34 +0100 dcyrils-macbook-pro.home (12231)] INFO : To prevent agent startup add a NEW_RELIC_AGENT_ENABLED=false environment variable or modify the "test" section of your newrelic.yml.
[2019-10-31 14:08:34 +0100 dcyrils-macbook-pro.home (12231)] INFO : Reading configuration from config/newrelic.yml (/Users/dcyril/src/doctolib)
[2019-10-31 14:08:34 +0100 dcyrils-macbook-pro.home (12231)] INFO : Environment: test

Please remark the dates:
during this Capybara test, I use Timecop to change current date.
It might have an impact?

This line makes me think that the problem is in the availability of /dev/urandom:
[2022-02-10 17:05:20 +0100 dcyrils-macbook-pro.home (12006)] ERROR : RuntimeError: failed to get urandom

Looks like there's a but in ruby >= 2.5.1 that causes this error when using SecureRandom with high concurrency: https://bugs.ruby-lang.org/issues/14716
I can see the code path that causes the agent issue you're seeing, and I can try to put together a fix for it.  I think the fix will simply expose the underlying error.

Maybe the fix would be to warn instead of throw?
From an user point of view, I would like newrelic to be completely invisible.
I would like it cannot modify my app (or test) behaviour in any case.

I think the fix is likely a little more complicated than that.  The problem occurs when we generate a random id for a segment.  And it only occurs very, very rarely.  (Which is why you've only seen it once, and we've never received a report of the problem before).
When this exception occurs, the segment initialize method returns nil.  We could check whether the segment is nil in this case, but that leaves many other places in our code that assume that a segment will never be nil immediately after initialization.
I've taken some notes of the problem, and I'm going to discuss with the other engineers on the team before we decide the appropriate way to handle it.  We'll update here when we have a plan.
Thanks again for bringing the problem to our attention!

thanks!

maybe this log can help you:
Errno::EMFILE: Failed to open TCP connection to <IP_ADDRESS>:9517 (Too many open files - socket(2) for "<IP_ADDRESS>" port 9517)
from /Users/dcyril/.rbenv/versions/2.6.5/lib/ruby/2.6.0/net/http.rb:949:in `rescue in block in connect'
Caused by Errno::EMFILE: Too many open files - socket(2) for "<IP_ADDRESS>" port 9517
from /Users/dcyril/.rbenv/versions/2.6.5/lib/ruby/gems/2.6.0/gems/socksify-1.7.1/lib/socksify.rb:178:in `initialize'
[2] pry(#<ZipperIntegration::WithZipperExtension::ZipperMessagesListViewRevampedTest::zipper agenda mapping>)> Unexpected error while processing request: failed to get urandom
	/Users/dcyril/.rbenv/versions/2.6.5/lib/ruby/2.6.0/securerandom.rb:106:in `urandom'
	/Users/dcyril/.rbenv/versions/2.6.5/lib/ruby/2.6.0/securerandom.rb:106:in `gen_random_urandom'
	/Users/dcyril/.rbenv/versions/2.6.5/lib/ruby/2.6.0/securerandom.rb:138:in `random_bytes'
	/Users/dcyril/.rbenv/versions/2.6.5/lib/ruby/2.6.0/securerandom.rb:236:in `uuid'
	/Users/dcyril/.rbenv/versions/2.6.5/lib/ruby/gems/2.6.0/gems/actionpack-5.2.3/lib/action_dispatch/middleware/request_id.rb:40:in `internal_request_id'
	/Users/dcyril/.rbenv/versions/2.6.5/lib/ruby/gems/2.6.0/gems/actionpack-5.2.3/lib/action_dispatch/middleware/request_id.rb:35:in `make_request_id'
	/Users/dcyril/.rbenv/versions/2.6.5/lib/ruby/gems/2.6.0/gems/actionpack-5.2.3/lib/action_dispatch/middleware/request_id.rb:26:in `call'
	/Users/dcyril/.rbenv/versions/2.6.5/lib/ruby/gems/2.6.0/gems/newrelic_rpm-<IP_ADDRESS>9/lib/new_relic/agent/instrumentation/middleware_tracing.rb:99:in `call'
	/Users/dcyril/.rbenv/versions/2.6.5/lib/ruby/gems/2.6.0/gems/rack-2.0.7/lib/rack/method_override.rb:22:in `call'
	/Users/dcyril/.rbenv/versions/2.6.5/lib/ruby/gems/2.6.0/gems/newrelic_rpm-<IP_ADDRESS>9/lib/new_relic/agent/instrumentation/middleware_tracing.rb:99:in `call'
	/Users/dcyril/.rbenv/versions/2.6.5/lib/ruby/gems/2.6.0/gems/rack-2.0.7/lib/rack/runtime.rb:22:in `call'
	/Users/dcyril/.rbenv/versions/2.6.5/lib/ruby/gems/2.6.0/gems/newrelic_rpm-<IP_ADDRESS>9/lib/new_relic/agent/instrumentation/middleware_tracing.rb:99:in `call'
	/Users/dcyril/.rbenv/versions/2.6.5/lib/ruby/gems/2.6.0/gems/activesupport-5.2.3/lib/active_support/cache/strategy/local_cache_middleware.rb:29:in `call'
	/Users/dcyril/.rbenv/versions/2.6.5/lib/ruby/gems/2.6.0/gems/newrelic_rpm-<IP_ADDRESS>9/lib/new_relic/agent/instrumentation/middleware_tracing.rb:99:in `call'
	/Users/dcyril/.rbenv/versions/2.6.5/lib/ruby/gems/2.6.0/gems/actionpack-5.2.3/lib/action_dispatch/middleware/executor.rb:14:in `call'
	/Users/dcyril/.rbenv/versions/2.6.5/lib/ruby/gems/2.6.0/gems/newrelic_rpm-<IP_ADDRESS>9/lib/new_relic/agent/instrumentation/middleware_tracing.rb:99:in `call'
	/Users/dcyril/.rbenv/versions/2.6.5/lib/ruby/gems/2.6.0/gems/actionpack-5.2.3/lib/action_dispatch/middleware/static.rb:127:in `call'
	/Users/dcyril/.rbenv/versions/2.6.5/lib/ruby/gems/2.6.0/gems/newrelic_rpm-<IP_ADDRESS>9/lib/new_relic/agent/instrumentation/middleware_tracing.rb:99:in `call'
	/Users/dcyril/.rbenv/versions/2.6.5/lib/ruby/gems/2.6.0/gems/rack-2.0.7/lib/rack/sendfile.rb:111:in `call'
	/Users/dcyril/.rbenv/versions/2.6.5/lib/ruby/gems/2.6.0/gems/newrelic_rpm-<IP_ADDRESS>9/lib/new_relic/agent/instrumentation/middleware_tracing.rb:99:in `call'
	/Users/dcyril/.rbenv/versions/2.6.5/lib/ruby/gems/2.6.0/gems/rack-cors-1.0.6/lib/rack/cors.rb:98:in `call'
	/Users/dcyril/.rbenv/versions/2.6.5/lib/ruby/gems/2.6.0/gems/newrelic_rpm-<IP_ADDRESS>9/lib/new_relic/agent/instrumentation/middleware_tracing.rb:99:in `call'

This is interesting.  Does this happen often for you?  Does it always follow an error like Too many open files?

I would say once a week for 2-3 months now. But one it's stuck, my only option is to comment theses 2 lines:
https://github.com/newrelic/rpm/blob/c9a467d22e6d4b893be33be330d258a125cf813a/lib/new_relic/agent/instrumentation/net.rb#L46-L47
And I recently discovered this "Too many open files", so I do not know.

OK, I have a repro.  The issue at play here is that your ruby interpreter is using all of its allotted file descriptors, which means it is unable to read from /dev/urandom.  This can be reproduced with the following script:
file_descriptors = []
begin
  while true do
    file_descriptors << IO.sysopen(__FILE__)
    SecureRandom.hex(8)
  end
ensure
  puts "failed at #{file_descriptors.length}"
end

This is because we're using SecureRandom, which passes through to Random.urandom, which looks like this (note the call to rb_raise):
random_raw_seed(VALUE self, VALUE size)
{
    long n = NUM2ULONG(size);
    VALUE buf = rb_str_new(0, n);
    if (n == 0) return buf;
    if (fill_random_bytes(RSTRING_PTR(buf), n, FALSE))
        rb_raise(rb_eRuntimeError, "failed to get urandom");
    return buf;
}

We don't actually need this random number to be secure, though.  In fact, we're not using SecureRandom to generate the guid of a Transaction.  See: https://github.com/newrelic/rpm/blob/master/lib/new_relic/agent/transaction.rb#L913
This method uses Random.rand, which does not rely on urandom.  I've used my repro to verify that Random.rand does not raise an exception when it runs out of file descriptors.
We'll refactor so that our Segment constructor can use the same generate_guid as the Transaction.
Thanks for bringing this to our attention!

thanks for finding the reproduction!

Hi @cyrilchampier, just wanted to let you know this issue has been fixed in the recently released version 6.8.0 of the agent. Thank you again!

Thanks a lot!


Add a feature to upload the videos automatically

Upload videoes automatically and generate title, description, tags,

Can you share where you mean uploading automatically


Does this work alone without NodeJS in Solaris

Hey,
I am new to this and wondered if this works in Solaris + Without NodeJS. I mean just like a C compiled binary.
We had problems in installing NodeJS itself so we look for something that can stand on its own.
Any help is appreciated.
Thanks in advance!
Suresh

@surighanta: Something like this?: https://www.alexkras.com/compile-node-js-to-native-binaries/


Add E2E tests for more ES scenarios

Which problem is this PR solving?

Add E2E tests for new scenarios when using Elasticsearch as storage in OpenShift:

Upstream Elasticsearch instance
Autoprovisioned ES instance using the Elasticsearch OpenShift Operator
Provision an ES instance using the Elasticsearch OpenShift Operator

All these scenarios are now supported for es-rollover and es-index-cleaner E2E tests.
Short description of the changes

Add templates for the new Jaeger and ES deployments
Add new production_managed_es value as deploy_mode parameter for render_install_jaeger macro
Add new parameter deploy_mode to render_install_elasticsearch
Remove wrong enabled parameter from esRollover

@frzifus if you check the logs, you'll see the test are not flaky. The template generation failing. It is failing because some macros changed the accepted parameters. I have to work on it.
Please, don't merge anything with failed tests.

Please, don't merge anything with failed tests.

I can not :)

@frzifus @rubenvp8510 this PR is ready for review :)

@iblancasa could you rebase?

This branch is out-of-date with the base branch


Kernel vulnerabilities in v20231230 build AMI

We are building the worker node AMI using this repo.
After running the scan we are getting vulnerabilities in kernel version - 5.10.201-191.748.amzn2.x86_64
Vulnerability Description:
The version of kernel installed on the remote host is prior to 5.10.205-195.804. It is, therefore, affected by multiple vulnerabilities as referenced in the ALAS2KERNEL-5.10-2024-045 advisory.

  - A race condition was found in the QXL driver in the Linux kernel. The qxl_mode_dumb_create() function dereferences the qobj returned by the qxl_gem_object_create_with_handle(), but the handle is the only one holding a reference to it. This flaw allows an attacker to guess the returned handle value and trigger a use-after-free issue, potentially leading to a denial of service or privilege escalation. (CVE-2023-39198)

  - An issue was discovered in the Linux kernel through 6.5.9. During a race with SQ thread exit, an io_uring/fdinfo.c io_uring_show_fdinfo NULL pointer dereference can occur. (CVE-2023-46862)

So could you please suggest the supported kernel version for EKS v1.26 worker nodes > 5.10.205-195.804

This is addressed in today’s release. Please open an AWS Support ticket for security issues in the future.


Duplicit physical drives panels in Home screen

Describe the bug
The Home screen's Drives panel list all physical drives twice. In the sidebar, the drives are listed correctly.
To Reproduce

Open Files.
See the double drives on Home screen.

Expected behavior
The physical drive entries should not be doubled.
Screenshots

Desktop (please complete the following information):

OS Version: Windows 10 Pro, version 20H2, build 19042.782
App version: v1.2.0

Additional context
Seems to be persistent. Refreshing the Home screen or restarting the app doesn't help.
Log file
debug.txt
debug_fulltrust.txt

After further investigation, it seems that this issue is occurring intermittently, as in some cases the duplicit physical drive entries weren't  present on the Home screen. However, the issue was still happening in most cases.

@dckface is this still an issue in Files v2?


ipaddress gem requirement missing from readme

As of this pull request, puppet-network uses ipaddress.
Not having it installed on the puppet agent will cause puppet agent -t to fail with an error like
 Could not autoload puppet/type/network_config: cannot load such file -- ipaddress
running sudo gem install ipaddress on the agent fixes this.
This requirement is not reflected in the Readme. I will work on a pull request that will add to the README an explanation of this along with an example on how to install this gem using puppet.

Also, maybe it's required on the master too. I'll look into this a bit more and make sure I know what is needed where, but this gem is definitely required somewhere at some point for puppet-network to work.

that should probably have an autorequire
…but then again, an autorequire isn't gonna really install things…
but another question: how, exactly, do you run a puppet infra without the ipaddress gem??

@igalic well, we use the ipaddress gem now.
still, I'd like to be able to able to install ipaddress on a node using puppet rather than some other way before puppet runs. Or this could a horrible hack that should be avoided and the gem should be installed using some other way after all.

It doesn't look like what I described in #129 will work so I might just close that issue. However I think that the docs should explicitly say that the ipaddress gem is required.

Re-opening this to comment that I thin you need to be clearer about where the ipaddress gem should be installed.
We're using PE 3.8.x with pe-puppetserver on the master, and I fixed the "could not autoload" issue by installing the ipaddress gem into the puppetserver environment, ie:
puppetserver gem install --no-ri --no-rdoc ipaddress

Is it actually required on the client as well? Or only the server?


feat: Add copy button to ssh connection

I have a lot of SSH connection profiles, but by now I need to add them one by one.
But in fact, all the profiles have some common fields like username, password, private key and even login scripts.
This PR adds a copy button to add a new connection based on the existing connection profile.

Amazing, thanks! @allcontributors please add @zend for code.


local file system support

Standalone NGL Viewer does not always work if used from local filesystem (file:///...). The error only shows up in Chromium, in Firefox everything works fine. The problem also disappears if security policies in Chromium are relaxed deliberately with "--allow-file-access-from-files".

As far is I understand access restrictions to the local file system are a security feature and can only be relaxed in Google Chrome/Chromium with the --allow-file-access-from-files flag, as you noted (for a discussion see 1, 2, 3 and 4).
As an alternative, depending what you want to achieve, serving the files you want to access through a simple web server could be the way to go.


Release Google.Cloud.Workflows.V1Beta version 2.0.0-beta05

Changes in this release:
New features

Add IServiceCollection extension methods for client registration where an IServiceProvider is required. (commit 022fab2)

Created release for Google.Cloud.Workflows.V1Beta-2.0.0-beta05


Visual Shader - Screen texture broken, crash on preview, white rectangle

Using the latest nightly build (https://hugo.pro/projects/godot-builds/ from 3rd october c3bb4ed), but this issue seems to be present even in the 3.1 alpha release.
Basically, when trying to use the "Texture" constant node and then selecting the "Screen" option, a white rectangle appears in the top right of the screen and nothing else happens, even if there are other nodes that should normally change the final picture. Note that this only happens when the texture is connected to the output somehow (whether there are other nodes between it and the output doesn't matter).
In order to reproduce this, add a TextureRect or a Sprite into a scene, create a new Visual Shader material for it and then add 3 nodes like this: Screen UV Input -> Texture constant (select Screen) -> Output (rgb and/or alpha).
https://1drv.ms/u/s!AvsEaC7oMEMagsQYfsbnSUG6tJ9mSQ
I'm using a GTX 1070 and 399.07 driver.
Edit:
Forgot to mention this, but when you try to open the screen texture node preview (rgb), the whole engine crashes. There are no error messages though.

The underlying issue is #12880
When the Visual Shader is translated to Godot Shading Language a temporary variable called _tex_read will be created. But because of #12880 Godot is can not handle variable names starting with _. The compilation will always fail.

#12880 just got resolved. Now you can use Screen Textures!


Add submenu support and ES2015ify the codebase

Fixes #5
Fixes #8

@sindresorhus if you would like to support node 0.12 and 0.10 let me know to rewrite the line where I'm using repeat() that is not supported by those versions.

Use const, let and fat-arrow functions to clean up the code.

@sindresorhus @SamVerschueren any more comments?

One small note. Otherwise look good to me :+1:

@sindresorhus @SamVerschueren comments have been addressed

Looks good when you've resolved Sam's feedback :)

Looks great! 🎉
Thanks for persevering through this @PedroFuentes. Really appreciate the contribution.

Added you to the repo since this is an open open source project :)

thank you @sindresorhus !

Nice work @PedroFuentes ! Also for not giving up after all those iterations :+1: !


#2815 Enhancedment StringUtils

Add isNotEmpty()
Move from bootstrap-core module to commons module

Deprecate StringUtils of bootstrap-core module

Change splitAndTrim() API
splitAndTrim() -> tokenizeToStringList()

Copy from spring framework

Codecov Report

Merging #2816 into master will decrease coverage by 0.01%.
The diff coverage is 74.39%.

@@            Coverage Diff             @@
##           master    #2816      +/-   ##
==========================================
- Coverage    40.8%   40.78%   -0.02%
==========================================
  Files        1775     1776       +1
  Lines       61272    61278       +6
  Branches     8608     8609       +1
==========================================
- Hits        24999    24992       -7
- Misses      36273    36286      +13

Impacted Files
Coverage Δ

...ceptor/HttpMethodBaseExecuteMethodInterceptor.java
0% <ø> (ø)
:arrow_up:

...ientExchangeHandlerImplStartMethodInterceptor.java
0% <ø> (ø)
:arrow_up:

.../pinpoint/profiler/util/AnnotationValueMapper.java
0% <ø> (ø)
:arrow_up:

...ootstrap/plugin/jdbc/bindvalue/BindValueUtils.java
90.9% <ø> (ø)
:arrow_up:

...r/HttpRequestExecutorExecuteMethodInterceptor.java
0% <ø> (ø)
:arrow_up:

...rap/plugin/jdbc/bindvalue/SimpleTypeConverter.java
50% <ø> (ø)
:arrow_up:

...pclient/interceptor/ExecuteRequestInterceptor.java
0% <ø> (ø)
:arrow_up:

...ptor/client/TServiceClientSendBaseInterceptor.java
0% <ø> (ø)
:arrow_up:

...nt/profiler/context/recorder/AbstractRecorder.java
35.71% <ø> (ø)
:arrow_up:

...ptor/ServerConnectionHandleRequestInterceptor.java
0% <ø> (ø)
:arrow_up:

... and 37 more

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 442f9d4...284ee9c. Read the comment docs.


KeyError: 'persistent-apps' error on 10.15.4

I am not able to add or remove anything with the latest release running on 10.15.4. I am getting the below error for each command.
dockutil --add /Applications/Microsoft\ Word.app
Traceback (most recent call last):
File "/usr/local/bin/dockutil", line 689, in
main()
File "/usr/local/bin/dockutil", line 388, in main
if addItem(pl, real_add_path, replace_label, position, before_item, after_item, section, displayas, showas, arrangement, tile_type, label_name):
File "/usr/local/bin/dockutil", line 565, in addItem
for existing_dock_item in (pl[section]):
KeyError: 'persistent-apps'

I am seeing the same behavior with 10.15.5.

Are you running this in a script via Jamf? Seems Catalina is not letting the root user modify the currently logged in user's Dock.
I've changed my script from;
/usr/local/bin/dockutil --remove 'App Store' --no-restart --allhomes
To;
user=ls -la /dev/console | cut -d " " -f 4
/usr/bin/sudo -u $user /usr/local/bin/dockutil --remove 'App Store' --no-restart "/Users/$user/Library/Preferences/com.apple.dock.plist"
Seems to work like that.

I'm also seeing this on Macos 12 developer beta 1.
dockutil worked for a while, but after a while of experimenting, I reset my dock (by deleting ~/Library/Preferences/com.apple.dock.plist) and I started to the get the same error.

It seems that if I run dockutil --list before dockutil --add I can work around this bug.

Not sure, but you might be experiencing the issue I filed as #109 , where the com.apple.dock.plist isn't setup yet. Dockutil expects the file to exist, and can't write elements to the persistent-apps, persistent-others, or recent-apps arrays if the arrays don't already exist.
And even if you make those manually, the Dock will completely reset itself back to a default state if you don't also have the version key set to the integer value 1
You can get around this by running this series of commands to initialize the version as 1, and fill in those expected keys with empty arrays.
defaults write <path/to/user>/Library/Preferences/com.apple.dock.plist version -int 1
defaults write <path/to/user>/Library/Preferences/com.apple.dock.plist persistent-apps -array
defaults write <path/to/user>/Library/Preferences/com.apple.dock.plist persistent-others -array
defaults write <path/to/user>/Library/Preferences/com.apple.dock.plist recent-apps -array

@aaronclarke I get a permission denied for this in root

It seems that if I run dockutil --list before dockutil --add I can work around this bug.

To anyone else who gets the "[Add|Remove] failed" error when running dockutil on a fresh user profile.... THIS IS THE SOLUTION. (Dockutil should really have some internal check to see if this is necessary and perform it automatically.) Oh well.
@aaronclarke your solution did not work for me, but I did try it.


Add candidate_contact_form_url to candidate model

In addition to the candidate field "candidate_url", we also need a "candidate_contact_form_url" field added to the Candidate model. Please search for all of the places we have "candidate_url" (edit form, lists, etc.) and add "candidate_contact_form_url" where appropriate.

Verified. Thank you @mattare2!


E164 method US only

TwilioVerifyService.e164_format is a one-liner that provides very little value, but makes this plugin unusable for anyone outside the US.
We have customers outside the US, so can't use this as-is.
How about just removing it and making a note in the README that the phone number provided needs to be validated to be in  E164 format?

Yeah i need to re-visit non US phone number support, I don't want to remove this method as this will introduce breaking changes for some apps.
I planned to swap out the hardcoded US format with a more versatile e164 formatting ruby gem like hexorx/e164


Maintenance: add biome to project (parser)

Summary
With #2791 being merged, we can now migrate the packages/parser workspace to use Biome.
As part of the migration I will fix any issue/inconsistency that arises. I'll do my best to minimize changes and leave the logic untouched.
Why is this needed?
So that we can gradually replace eslint, prettier, and all their dependencies with a single one, @biomejs/biome.
Which area does this relate to?
Other
Solution
No response
Acknowledgment

[X] This request meets Powertools for AWS Lambda (TypeScript) Tenets
[ ] Should this be considered in other Powertools for AWS Lambda languages? i.e. Python, Java, and .NET

Future readers
Please react with 👍 and your use case to help us understand customer demand.

I would like to take this issue :)

I think there will be some challenges with the conversion, especially with the classes within the packages/parser/src/envelopes, like this one.
All the classes are defined more or less like this:
export class ApiGatewayV2Envelope extends Envelope {
  public name = 'ApiGatewayV2Envelope';
  public static parse<T extends ZodSchema>(
    data: unknown,
    schema: T
  ): z.infer<T> {
    return Envelope.parse(
      APIGatewayProxyEventV2Schema.parse(data).body,
      schema
    );
  }

  public static safeParse<T extends ZodSchema>(
    data: unknown,
    schema: T
  ): ParsedResult {
    const parsedEnvelope = APIGatewayProxyEventV2Schema.safeParse(data);
    if (!parsedEnvelope.success) {
      return {
        success: false,
        error: new ParseError('Failed to parse API Gateway V2 envelope', {
          cause: parsedEnvelope.error,
        }),
        originalEvent: data,
      };
    }

    const parsedBody = Envelope.safeParse(parsedEnvelope.data.body, schema);

    if (!parsedBody.success) {
      return {
        success: false,
        error: new ParseError('Failed to parse API Gateway V2 envelope body', {
          cause: parsedBody.error,
        }),
        originalEvent: data,
      };
    }

    return parsedBody;
  }
}

which according to Biome has two issues:

The methods use the super. keyword within the static methods
The class itself only declares static methods

The first one is easily fixable thanks to the fact that the methods being called in the parent class are themselves static, so we can just replace all the occurrences of super.parse() with Envelope.parse() (and the same with safeParse).
The second one is a bit trickier. The way I see it, we have three options:

suppress the warnings
add some other property to the class (i.e. public name =  'ApiGatewayV2Envelope')
refactor the code and follow the suggestion from Biome

I'd like to avoid the former unless we have no other option, and the second seems useless. So let's explore the third.
These classes were created like that to allow customers to use them without having to instantiate them. The use case we want to enable is people doing ApiGatewayV2Envelope.parse(...) rather than new ApiGatewayV2Envelope().parse(...), which is sensible.
To maintain this functionality and also remove the warnings, we could refactor both classes to plain object, and set the two methods as object properties. Once refactored they would become this:
export const Envelope = {
  /**
   * Abstract function to parse the content of the envelope using provided schema.
   * Both inputs are provided as unknown by the user.
   * We expect the data to be either string that can be parsed to json or object.
   * @internal
   * @param data data to parse
   * @param schema schema
   */
  parse: <T extends ZodSchema>(
    data: unknown,
    schema: T
  ): z.infer<T> => {
    if (typeof data !== 'object' && typeof data !== 'string') {
      throw new ParseError(
        `Invalid data type for envelope. Expected string or object, got ${typeof data}`
      );
    }
    try {
      if (typeof data === 'string') {
        return schema.parse(JSON.parse(data));
      }
      if (typeof data === 'object') {
        return schema.parse(data);
      }
    } catch (error) {
      throw new ParseError('Failed to parse envelope', {
        cause: error as Error,
      });
    }
  },

  /**
   * Abstract function to safely parse the content of the envelope using provided schema.
   * safeParse is used to avoid throwing errors, thus we catuch all errors and wrap them in the result.
   * @param input
   * @param schema
   */
  safeParse: <T extends ZodSchema>(
    input: unknown,
    schema: T
  ): ParsedResult<unknown, z.infer<T>> => {
    try {
      if (typeof input !== 'object' && typeof input !== 'string') {
        return {
          success: false,
          error: new ParseError(
            `Invalid data type for envelope. Expected string or object, got ${typeof input}`
          ),
          originalEvent: input,
        };
      }

      const parsed = schema.safeParse(
        typeof input === 'string' ? JSON.parse(input) : input
      );

      return parsed.success
        ? {
            success: true,
            data: parsed.data,
          }
        : {
            success: false,
            error: new ParseError('Failed to parse envelope', {
              cause: parsed.error,
            }),
            originalEvent: input,
          };
    } catch (error) {
      return {
        success: false,
        error: new ParseError('Failed to parse envelope', {
          cause: error as Error,
        }),
        originalEvent: input,
      };
    }
  }
}

export const ApiGatewayEnvelope = {
  parse<T extends ZodSchema>(
    data: unknown,
    schema: T
  ): z.infer<T> {
    return Envelope.parse(APIGatewayProxyEventSchema.parse(data).body, schema);
  },

  safeParse<T extends ZodSchema>(
    data: unknown,
    schema: T
  ): ParsedResult<unknown, z.infer<T>> {
    const parsedEnvelope = APIGatewayProxyEventSchema.safeParse(data);
    if (!parsedEnvelope.success) {
      return {
        success: false,
        error: new ParseError('Failed to parse ApiGatewayEnvelope', {
          cause: parsedEnvelope.error,
        }),
        originalEvent: data,
      };
    }

    const parsedBody = Envelope.safeParse(parsedEnvelope.data.body, schema);

    if (!parsedBody.success) {
      return {
        success: false,
        error: new ParseError('Failed to parse ApiGatewayEnvelope body', {
          cause: parsedBody.error,
        }),
        originalEvent: data,
      };
    }

    return parsedBody;
  }
}

By doing this the logic would remain the same, but the resulting code would also be slightly simpler. Additionally, when bundling the current Parser code using esbuild, this is exactly the same transformation that is applied, so we'd get to this anyway at runtime.
What do you think?

Seeing this, I'm even thinking about whether there are reasons to keep the envelope classes as classes. I can't see any advantage why they should stay as a class, right?

Both are can be instantiated, but in my opinion neither of them has to be a class strictly speaking.
Regarding the breaking change, normally it would be one, but in this instance we have two points to consider:

the utility is still in beta and we don't guarantee stability until it's considered as generally available (next release)
all documentation shows how to use the object as static (aka without the new keyword)

I think overall if we want to make a change like this it's now or never for the next couple years

The refactoring to objects makes sense, there is nothing class specific in the implementation. I have checked and there is no impact on types or other parts of the parser, tests are green.


Feature/issue 848 map rect mpi, fixes #848

Submission Checklist

[ ] Run unit tests: ./runTests.py test/unit
[X] Run cpplint: make cpplint
[X] Declare copyright holder and open-source license: see below

Summary:
Add MPI backend to map_rect which executes tasks in parallel.
Intended Effect:
Use multiple cores to execute independent tasks passed to map_rect.
How to Verify:
Please first enable MPI, see the stan-math wiki for doing so. The tests to run are

test/unit/math/{prim,rev}/mat/functor/map_rect_mpi_test.cpp
test/unit/math/prim/mat/functor/mpi_parallel_call_test.cpp

Side Effects:
Introduces the new mpi_parallel_call facility which comes with its own doc and tests.
Documentation:
Lot's of doxygen documentation.
Copyright and Licensing
Please list the copyright holder for the work you are submitting (this will be you or your assignee, such as a university or company): Sebastian Weber
By submitting this pull request, the copyright holder is agreeing to license the submitted work under the following licenses:

Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

Cool. Now it's ready to go. I had to catch a few boundary cases for the code for which I added tests.
This is not the easiest thing to review, I am afraid. I tried to doc things quite a bit using doxygen and I hope this helps. Yet, keeping track of what happens where when and how caching and exceptions are handled is not trivial. So if we need a call over this, then we can arrange that.

https://github.com/stan-dev/math/blob/8c87ffc3fd84bb0e91cb2af43a4b45888be4183e/stan/math/prim/mat/functor/mpi_parallel_call.hpp#L295
Is it necessary for all processes to gather? Workers just need to be informed by the root to return or not.

All you need is all_ok. Let root do a reduce, and broadcast all_ok.
Regards,
Yi Zhang, Ph.D.
Metrum Research Group LLC
2 Tunxis Road, Tariffville, CT 06081
<EMAIL_ADDRESS>www.metrumrg.com
On Tue, May 8, 2018 at 2:23 AM, wds15<EMAIL_ADDRESS>wrote:

@yizhang-cae https://github.com/yizhang-cae I think this is needed,
yes. I know that all_gather is expensive as it blocks all asynchronous
operations, but note that I am requiring this only during the very first
invokation of the function. The first invokation is special since it is not
known yet what output sizes the specific task has (# of rows and # of cols
per job of local_output). This size information is cached upon the first
successful execution of the user-function on all nodes and for all
jobs. Thus I need to inform for the first execution all workers if all
workers did successfully execute their chunk. Only when all nodes were able
to execute things right then the information is cached. Whenever this is
done no further all_gather calls are performed. Makes sense?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/stan-dev/math/pull/854#issuecomment-387297598, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AfkhsEejjQXDRBXFy4LBHsbl5HwSNffiks5twTnfgaJpZM4TuC9a
.

Yeah, I can do that:

gather status at root
broadcast boolean all_ok flag

Fine with me.

Thanks much for going through that massive code block. I will address them probably over the weekend.

Yes, that's fine.  It'd also be OK to put a poitner back to the note
in the catch block.  It just looked odd to me that there wasn't any
overt action taking on exception other than to set a flag.

I think we want to allow that jobs can return no output. This is an odd function, but in Stan we allow for that. Thus we need num_outputs_per_job to be -1 to indicate that things are not yet initialized. Right?

OK.

Although I don't understand the reasoning, if you feel that strongly about it, I won't insist---I don't feel that strongly about a _t suffix convention.

I think it may be able to deal with that.  I think you can assign Eigen matrices of different shapes to each other.  But it's no big deal either way---just a simple loop.

Lot's of more doc here now. In going over it in this iteration I found in a few places simplifications. I hope all is fine now (including the way the concepts are described for ReduceF and CombineF).

Thanks.  I should have time to get to finish the final review tomorrow (Thursday) afternoon or Friday.

@wds15 --- assuming the build is all OK for this to not torch Jenkins, go ahead and merge.  Everything looks great.


Inappropriate way of assigning values to Client data in toy and vfl synthetic dataset

When applying feature transformation in ClientData, we must use deepcopy. Otherwise, the data in server and client will be inconsistent (in server, x = f(x), but in client, x=f(f(x))).
And I've fixed this in https://github.com/alibaba/FederatedScope/pull/486/commits/ae896cd5740dc0b34f03bc637a724edf3f3be77e, please help me to double-check if there are the same issues on other datasets, thanks! @xieyxclack

Thank you very much for pointing out and fixing this issue!


Add static analysis for the rotor

Add a method to calculate static displacement.

I just commited updates to the rotor_assembly.py. The Travis test has failed, but I believe it is because of the absence of the bokeh requirement in setup.py.

I changed the setup's code, now it's working again.
setup.py

This merged commit closes this issue


LCFS - User Activity table for BCeID users

Description:
Creating a User Activity react table on the view user page where certain user actions will be displayed
Screenshot of User Activity table in TFRS: Private Zenhub Image
Wireframe for LCFS:
Purpose and benefit to user:
Record for auditing purposes, transparency
Acceptance Criteria:

[ ] Given I am a , When I am , then
[ ] Given I am a , When I am , then

Development Checklist:

[ ] A
[ ] B
[ ] C

Notes:

Leaving this feature until we have the necessary features developed in LCFS (e.g., transfers, compliance reporting, initiative agreement issuance, etc.).


Support For Regex

I mean support for like first match, last match, group and groups methods
@ethanblake4 if you at least tell me how to add it (even a rough explanation would work)  i will try to follow through with the best of my abilities.
since i really love this project and would like to contribute.

For sure, thanks for your interest in helping! It's basically the same as making your own bridge wrappers, you'd just have to edit shared/stdlib/core/regexp.dart to add those methods. We don't currently have a wrapper for the RegExpMatch type, but you can just return a regular Match (using $Match) or you could add it if you want.

Thanks will do!

Hi, @ethanblake4 i was writing some tests for RegExpMatch and found some issues

with null assertion operator - if i try do this firstMatch(string)! it throws error

dart_eval runtime exception: Null check operator used on a null value
#0      $Object._equals (package:dart_eval/src/eval/shared/stdlib/core/object.dart:266:41)
#1      $Function.call (package:dart_eval/src/eval/runtime/function.dart:122:16)
#2      CheckEq.run (package:dart_eval/src/eval/runtime/ops/objects.dart:152:14)

but i use return match?[0] it returns chat
test('RegExp.firstMatch()', () {
      final runtime = compiler.compileWriteAndLoad({
        'example': {
          'main.dart': r'''
            String main() {
              final string = '[00:13.37] This is a chat message.';
              final regExp = RegExp(r'c\w*');
              final match = regExp.firstMatch(string)!;
              return match[0];
            }
          ''',
        }
      });
      expect(
          (runtime.executeLib('package:example/main.dart', 'main') as $String)
              .$value,
          'chat');
    });

when returning a specific type - when i try to return groupCount which is int it throws this error but if remove the type it compiles and return the int type and yeah the first issue applies to this too

CompileError: Cannot return RegExpMatch (expected: int) at unknown (file package:example/main.dart)
package:dart_eval/src/eval/compiler/compiler.dart 497:7   Compiler.compileSources
package:dart_eval/src/eval/compiler/compiler.dart 164:12  Compiler.compile
package:dart_eval/src/eval/compiler/compiler.dart 547:21  Compiler.compileWriteAndLoad

    test('RegExp.groupCount', () {
      final runtime = compiler.compileWriteAndLoad({
        'example': {
          'main.dart': '''
            int? main() {
              final string = '[00:13.37] This is a chat message.';
              final regExp = RegExp(r'c\\w*');
              final match = regExp.firstMatch(string);
              return match?.groupCount;
            }
          ''',
        }
      });
      expect(
          (runtime.executeLib('package:example/main.dart', 'main') as $int)
              .$value,
          0);
    });

if i don't use ! operator and don't specify return type on main all test that i have tried works fine

Hi, @ethanblake4 I did what you told but i am getting some issues with writing the tests

with null assertion operator - if i use ! ilke this firstMatch(string)! i get this error even tho it's not null because if i use ? it returns string

dart_eval runtime exception: Null check operator used on a null value
#0      $Object._equals (package:dart_eval/src/eval/shared/stdlib/core/object.dart:266:41)
#1      $Function.call (package:dart_eval/src/eval/runtime/function.dart:122:16)
#2      CheckEq.run (package:dart_eval/src/eval/runtime/ops/objects.dart:152:14)
at main()

2 with return type with if try to return int or string  i get an error but if i don't specify the return type it pass the test with no error
CompileError: Cannot return RegExpMatch (expected: int) at unknown (file package:example/main.dart)
package:dart_eval/src/eval/compiler/compiler.dart 497:7   Compiler.compileSources
package:dart_eval/src/eval/compiler/compiler.dart 164:12  Compiler.compile
package:dart_eval/src/eval/compiler/compiler.dart 547:21  Compiler.compileWriteAndLoad

here are the test cases
import 'package:test/test.dart';
import 'package:dart_eval/dart_eval.dart';
import 'package:dart_eval/stdlib/core.dart';

void main() {
  late Compiler compiler;

  setUp(() {
    compiler = Compiler();
  });
  group('Regex Tests', () {
    test('RegExp.firstMatch()', () {
      final runtime = compiler.compileWriteAndLoad({
        'example': {
          'main.dart': r'''
            String main() {
              final string = '[00:13.37] This is a chat message.';
              final regExp = RegExp(r'c\w*');
              final match = regExp.firstMatch(string)!;
              return match[0];
            }
          ''',
        }
      });
      expect(
          (runtime.executeLib('package:example/main.dart', 'main') as $String)
              .$value,
          'chat');
    });
    test('RegExp.groupCount', () {
      final runtime = compiler.compileWriteAndLoad({
        'example': {
          'main.dart': '''
            int? main() {
              final string = '[00:13.37] This is a chat message.';
              final regExp = RegExp(r'c\\w*');
              final match = regExp.firstMatch(string);
              return match?.groupCount;
            }
          ''',
        }
      });
      expect(
          (runtime.executeLib('package:example/main.dart', 'main') as $int)
              .$value,
          0);
    });
}

Just pushed (I think) a fix for the second issue to v0.7.0 branch. First one will have to wait until tomorrow, but I'm 99% sure it's not your fault, the ! and ?. operators were just added a few days ago and they definitely still have bugs.

Then I'll open a pr when all the tests pass

Released in v0.7.0


Add WindowShine 1.8.X (Unofficial) from SpaceDock

This pull request was automatically generated by SpaceDock on behalf of ChubbyHamster, to add WindowShine 1.8.X (Unofficial) to CKAN.
Please direct questions about this pull request to ChubbyHamster.
Mod details:
name = /mod/2335/WindowShine%201.8.X%20%28Unofficial%29
author = ChubbyHamster
abstract = This is a remade WindowShine 1.3 so that it can work with ksp 1.8.X (Unofficial)
license = MIT
Homepage =
description =
This mod needs TextureReplacer to work Dowload
This is a remade WindowShine 1.3 so that it can work with ksp 1.8.X (Unofficial)
This mod takes advantage of the TextureReplacer reflective module pared with re-texturing to make Solar PAnels reflective ; meaning they mirror any object, planet, kerbal, or thing that gets in front of it. Pure Eye candy.
Warning: I am not the developer of this mod! I just updated the config and decided to post this mod
Author: Avera9eJoe
Don't forget to ON reflections in the texture replacer
Examples:

ORIGINAL:WindowsShine 1.3

This author has been added as a co-owner of the original fork on SpaceDock and has uploaded a new release there. No need to add a new netkan.


Pre caching ad

This includes pre caching of ads. In Pubnative.init(<app_token>) caching starts and ad would be served from PubnativeLibraryCache.
Classes & Roles:
PubnativeLibraryNetworkCacheAdapter: Will hold different type of adapters those would be helping in caching the ads.
PubnativeLibraryNetworkCacheRequestAdapter: Will serve the ad by getting them from PubnativeLibraryCache.
PubnativeLibraryCache: Will be responsible to make network request, get ads from server & serve them when required. Initially it will get 5 ads from server and serve the ad if it's valid in terms of time (as of now it's 2 hrs) and when only 1 ad is left it again start getting 5 ads from server.

Coverage decreased (-1.005%) to 32.752% when pulling 226f71ccfa321900e655f3e8bd8c571865a93ad5 on pre-caching-ad into 07951b0f2441dd332ee2c85520f2d2fc78fdb1a6 on development.

@cerberillo and @anshumansworld do you think that realising this functionality like separated adapter is good? For me it looks not very bad, but I think, that better was realise caching process inside PubnativeNetworkRequest? For example, when we get a data for adapter, we put this one in cache and start new request if it needed.

@RagnarR Yeah it looks good, there is no point in adding this to the PubnativeNetworkRequest, since this is something totally independent from that.

For my thoughts we making our library very complicated with creating this new adapter. It will be harder to understand for developer how to use it and how is it works

@RagnarR It would be internal part of SDK as no one is going to access these adapters directly. Developers will only access classes those are under request.

Coverage decreased (-0.7%) to 33.079% when pulling d326d90c62cd6c89ec60d6f085d12e7b185b02a7 on pre-caching-ad into 07951b0f2441dd332ee2c85520f2d2fc78fdb1a6 on development.

Coverage decreased (-0.6%) to 33.139% when pulling e0ff7a3f3a9844ff34c3395a33435e04d7d47b97 on pre-caching-ad into 07951b0f2441dd332ee2c85520f2d2fc78fdb1a6 on development.

Coverage decreased (-0.7%) to 33.079% when pulling 5f006d4d35812d8174fbcd73701a67902e4a1fcd on pre-caching-ad into 07951b0f2441dd332ee2c85520f2d2fc78fdb1a6 on development.

Coverage decreased (-0.7%) to 33.016% when pulling 40bcceb9b7d57c9d8eb8d774ad3b5cd58687e2ca on pre-caching-ad into 07951b0f2441dd332ee2c85520f2d2fc78fdb1a6 on development.

Coverage decreased (-0.7%) to 33.028% when pulling 3b47b54fe0b256736db973d7c07574378e8606a7 on pre-caching-ad into 07951b0f2441dd332ee2c85520f2d2fc78fdb1a6 on development.

Coverage decreased (-0.7%) to 33.028% when pulling a3bd288685e46c0708d0d1c991dc209027754415 on pre-caching-ad into 07951b0f2441dd332ee2c85520f2d2fc78fdb1a6 on development.

All right, this is taking ages and it's not close to the solution, I'll be taking control of this task from now on

Coverage decreased (-0.7%) to 33.028% when pulling 1d95301e3446c8ffefb243512dfaf52241a894f1 on pre-caching-ad into 07951b0f2441dd332ee2c85520f2d2fc78fdb1a6 on development.

Ok, I've done the cache thing we need the following tests @anshumansworld please implement them as soon as you have time.
PubnativeLibraryCPICache

enqueue/dequeue behaviour with and without ads in the queue
getAd(context) and getAd(context, ad) tests checking the remaining number of ads and the refresh mechanism works
Check behaviour with outdated ads
PubnativeLibraryNetworkRequestAdapter behaviour on load and fail

when loaded and revenuemodel is CPA, we call getAds giving this one away.
when loaded and revenuemodel is not CPA, we need to serve the ad that is loaded
when failed we serve from the cache if there are ads from the cache
when failed we fail if there are no more ads in the cache

Coverage decreased (-0.7%) to 33.089% when pulling 6fa9083df1926192d027fed2d6c6b977a178be9c on pre-caching-ad into 07951b0f2441dd332ee2c85520f2d2fc78fdb1a6 on development.


POC for ROS2

Test out a setup including ROS2. Should be possible to replace RTSP and camera-input.
Motivation: having a foundation for autonomy going forward

https://docs.px4.io/main/en/ros/ros2_comm.html


Remove redundant logging of found config file

Why make this change?

With the overhaul of config system PR #1402 , we started seeing redundant logs about which config file is being used.

What is this change?

Store the environment based config file name found at construction time in the property ConfigFileName.
Remove Console writeline statements while checking for existence of file since this leads to multiple logs - one from CLI before starting the engine and second from within the engine itself when calling TryPargeConfig
Add a single log of the loaded file in CLI
Also catch exception for incorrect connection string parsing

How was this tested?

Manually, using the start command of CLI as well as triggering the engine directly with argument: --ConfigFileName

Starting with default config:
Before:

After:

Starting with user provided config:
Before:

After:

Trying to start with missing file name - same behvior as before

does this PR also address the issue here ?

does this PR also address the issue here ?

yes it does


Fixed #29534 -- Made dbshell use rlwrap on Oracle if available.

Design of the code is that another wrapper can easily be used and the way to wrap it can easily be modified.

@felixxm, any interest in giving this some feedback or an initial review?

@danizen Do you have time to finish this?

Updated in #10555.


Athena leaves temporary files in S3 directory

Dbeaever 6.2.1 on MacOS.
When I connect to Athena via dbeaver and run a query like
select * from table limit 100;
this causes dbeaver to leave leftovers .csv file in the S3 directory.  In our case this happens in Apache ORC files and that causes subsequent queries to break as these csv files are non ORC.
Here is a screenshot of the S3 bucket before the query was run.

Here after the query was run.  One can see the left over files.

Subsequent queries return this error:

I'm afraid we can't fix this from DBEaver side.
This is how Simba JDBC driver works (it is a driver recommended by Amazon).

OK fair enough.  Thanks.

There's a property in the driver to set the output location for query results. To change this setting in DBeaver go to the "Driver properties" tab you can set a value for S3OutputLocation.  To find what your default is go to the Athena page in your AWS console and look in "settings" for "Query result location".


General data functions

Add database functions for fetching feed content (i.e. from friends, populate the home feed, ability to search users, etc.)

From #26:

Still needed

checks for adding private friends (adding a requests field to the user document to store users who have requested to become friends)
aggregation functions, mainly for populating the feed (i.e. get posts from all of your friends, sort in reverse chronological order, etc...)
functions for liking and commenting

new branch for the functions to be added @rebeccaan8

from #23 (closing it soon, but still need to implement the following):

link to view friend requests

done now that friend requests are done
needs to be implemented on the front end, see #64