Send email and log communication on new submissions

Why
Alternative to #213, introducing some patterns around interactors, mailers, logging.
See notes here for details.
Pre-Merge Checklist

[x] All new features have been described in the pull request
[x] Security & accessibility have been considered
[ ] All outstanding questions and concerns have been resolved
[x] Any next steps that seem like good ideas have been created as issues for future discussion & implementation
[x] High quality tests have been added, or an explanation has been given why the features cannot be tested
[x] New features have been documented, and the code is understandable and well commented
[x] Entry added to CHANGELOG.md if appropriate

What

[x] Introduce EmailNewSubmission interactor`
[x] Extract Messenger module to handle error handling around email delivery. Also takes on delivery status logic.
[x] Extract thin Log utility for logging exceptions
[x] Wire interactor into SubmissionsController, Asks|OffersController.
[x] Add cc to admin

Testing

[ ] Test different delivery failure scenarios to make sure we're rescueing all the exception classes necessary.

Next Steps

The Messenger module sets up an interface that could be used when we add other delivery media, eg SMS.
Deferring some questions about mailer code patterns to #759

Outstanding Questions, Concerns and Other Notes

[ ] I've used a pattern here of refactoring away from constants towards symbols. In cases such as delivery statuses, which are essentially enumerations, i don't see a significant advantage to constants (and find them to be LOUD in the codebase :). Definitely open to discussion on this.
[x] The notes suggest this email should be CCed to someone from the org. @maebeale , i need some guidance figuring out who that should be.
[ ] CommunicationLog.log_email allows initiator (usually current_user) to be nil. Do we want to allow this?

Tested two failure scenarios in the test env:

set SMTP_HOST=localhost

Generates a 500 error all the way up to the user. Open to other thoughts but i think this is a good use of 'fail fast' -- the app  is unable to talk to the SMTP server, which IMO falls into the category of stop-the-press-and-fix it.

2020-10-02T20:12:37.259294+00:00 app[web.1]: I, [2020-10-02T20:12:37.259247 #13]  INFO -- : [d439358c-2e53-461e-aaf6-27fc58164a0d] Completed 500 Internal Server Error in 92ms (ActiveRecord: 35.5ms | Allocations: 19696)
2020-10-02T20:12:37.260032+00:00 app[web.1]: F, [2020-10-02T20:12:37.259978 #13] FATAL -- : [d439358c-2e53-461e-aaf6-27fc58164a0d]
2020-10-02T20:12:37.260033+00:00 app[web.1]: [d439358c-2e53-461e-aaf6-27fc58164a0d] Errno::ECONNREFUSED (Connection refused - connect(2) for "localhost" port 587):
2020-10-02T20:12:37.260033+00:00 app[web.1]: [d439358c-2e53-461e-aaf6-27fc58164a0d]
2020-10-02T20:12:37.260034+00:00 app[web.1]: [d439358c-2e53-461e-aaf6-27fc58164a0d] app/lib/messenger.rb:23:in `deliver_now'
2020-10-02T20:12:37.260034+00:00 app[web.1]: [d439358c-2e53-461e-aaf6-27fc58164a0d] app/interactions/email_new_submission.rb:8:in `execute'
2020-10-02T20:12:37.260034+00:00 app[web.1]: [d439358c-2e53-461e-aaf6-27fc58164a0d] app/controllers/submissions_controller.rb:24:in `create'
2020-10-02T20:12:37.260034+00:00 app[web.1]: [d439358c-2e53-461e-aaf6-27fc58164a0d] app/controllers/application_controller.rb:19:in `switch_locale'
2020-10-02T20:12:37.260320+00:00 app[web.1]: I, [2020-10-02T20:12:37.260252 #13]  INFO -- : source=rack-timeout id=d439358c-2e53-461e-aaf6-27fc58164a0d wait=3ms timeout=15000ms service=95ms state=completed

set<EMAIL_ADDRESS>
This also fails but is captured in the rails log and not propagated to the user. This specific example could arguably fall into the same category as above since w/o a good from address we can't send email, but i think is more representative of a class of errors that could be more ephemeral, like a request timeout (i'd like to test that directly, but not sure how?).

2020-10-02T20:18:50.417578+00:00 app[web.1]: E, [2020-10-02T20:18:50.417497 #13] ERROR -- : [dcfd2751-03fe-4fb4-a00e-e0cc7e17fc2e] Error encountered delivering new_submission_confirmation_email email
2020-10-02T20:18:50.417584+00:00 app[web.1]: 550 The from address does not match a verified Sender Identity. Mail cannot be sent until this error is resolved. Visit https://sendgrid.com/docs/for-developers/sending-email/sender-identity/ to see the Sender Identity requirements
2020-10-02T20:18:50.417585+00:00 app[web.1]: (Net::SMTPFatalError)
2020-10-02T20:18:50.417585+00:00 app[web.1]: app/lib/messenger.rb:23:in `deliver_now'
2020-10-02T20:18:50.417586+00:00 app[web.1]: app/interactions/email_new_submission.rb:8:in `execute'
2020-10-02T20:18:50.417586+00:00 app[web.1]: app/controllers/submissions_controller.rb:24:in `create'
2020-10-02T20:18:50.417587+00:00 app[web.1]: app/controllers/application_controller.rb:19:in `switch_locale'

Got verbal approval from @maebeale and created #764 to track outstanding issues.


CI

Hello, why you included bin folder to repository??
Do you need a help?

To publish binary that can be installed via curl. Gotta need to upload binary somewhere :)

You can use goreleaser for building and publish binary files to project artifacts.
For example:
https://github.com/juev/starred
All binary files you can view on Releases page.

Thanks, and yea I have heard about goreleaser and wanted to add that to the CI.
For following reasons I didn't do it

The Github Release, uploads a tar (as far as I know, I could be wrong I have only looked at it).
And I wanted to reduce the number of dependencies, such as tar might not be installed on a system for a user. Therefore, I simply provide direct binary file via curl.
In fact, insde the library gobrew it uses go to untar the source of go, instead of calling tar as a secondary os command.
Laziness :(

You can configure goreleaser or just your build system for storing binary files on release page without tar. 😆
If you want it, I can configured it.

Thanks for the hand.
There'd be one more issue then
https://github.com/juev/starred/releases/download/0.0.5/starred_0.0.5_Darwin_arm64.tar.gz
In order to download, how can I download always the latest version without providing the tag version info.
Currently I am doing this as you already saw:
curl -kLs https://raw.githubusercontent.com/kevincobain2000/gobrew/master/bin/$GOBREW_ARCH_BIN -o $GOBREW_BIN_DIR/gobrew

I believe after the release, it'd have to provide a version number on that URL

I create new issue for discuss this problem.


ng-knob directive for Angular2 and TypeScript

Hello, i adapted your work on ng-knob directive for Angular2 and TypeScript
Instructions:

Install d3 typings
import {Knob} from '../../knob/ng-knob2';
<ui-knob [(value)]="myKnobValue" [options]="myKnobOptions">

Value: 2 ways binding
Options: 1 way binding
Well tested in ionic2

Can you provide more details instructions for using this with ionic2?

Do you happen to have a example project with this.

No funcionó para mi: 'ui-knob' is not a known element

@DanCard0 same problem for me, did you resolve the issue?

I have this problem when I put this directive in a component:
`Can't bind to 'value' since it isn't a known property of 'ui-knob'.`
1. If 'ui-knob' is an Angular component and it has 'value' input, then verify that it is part of this module.
2. If 'ui-knob' is a Web Component then add 'CUSTOM_ELEMENTS_SCHEMA' to the<EMAIL_ADDRESS>of this component to .....

I have added the directive to the declarations section in my module but doesn't not work.
What's wrong?

Hi, does anyone have this working in angular2 and d3.js v4?

I had it working with d3 v3 and @types/d3 v3 ..

@prashantghimire yes, I has migrate all suggested changes from d3v3 to d3v4, I will fork the repo and provide you the link for it.

@prashantghimire here is the repo working with angular 4 and dv3v4 xZegga/angular2-knob

Hi, thanks @xzegga
When I tried it, I got a few typescript errors, ie:
Argument of type '{ r: (d: { r: any; }) => any; cx: (d: { cx: any; }) => any; cy: (d: { cy: any; }) => any; fill: a...' is not assignable to parameter of type 'string'.

Are you using this directive on angular for web or ionic framework? I'm don't have any experience with ionic, but this problem looks like a typescript version problem.

Hi, am using it on ionic 2 framework. Typescript is: "typescript": "2.3.4"

I've been able to remove the 'argument of type' errors by splitting out the functions given to attr(). However still getting the namespace issue

Check if you have a right version for d3.js and d3-selection
     "@types/d3-selection": "^1.1.0",
    "d3": "^4.10.0"

Hi, thanks. Have added an issue to the other repo. https://github.com/xzegga/angular2-knob/issues/1


[internal] upgrade to babel@7

This PR upgrades vx packages to use babel 7. babel-upgrade does most of the heavy lifting. A huge help for a monorepo.
Thanks to @skevy for pointing out how painless the migration is which motivated me to do it.
Runbook
For each package:

npx babel-upgrade --write info
remove external-helpers from rollup.config.js rollup-plugin-babel for babel 7 now dedups helpers
update build:babel cmd to babel 7 plugins
update rollup version to 0.67.4
rm -rf ./node_modules
npm install
npm run test

Testing

all tests pass
run vx-demo.now.sh/gallery locally with no errors

Changelog
:memo: Documentation

[gallery] add <BarStackHorizontal /> example code back

:house: Internal

[internal] upgrade to babel@7

woot! nice 👌 I did this for @data-ui recently and it was a bit painful ... mostly beemo learning curve. glad babel-upgrade made it easier!


Use 3000 as default rails server port

This rollbacks the random port for new suspenders up, while leaving the code configurable (as introduced in https://github.com/thoughtbot/suspenders/commit/4caa6123c83e726d6fcd23956ce9883ca36b1d42). That existing code is still buggy though, as it doesn't take sample.env file into account (https://github.com/thoughtbot/suspenders/issues/602#issuecomment-130671649). Should we rollback all the changes from that commit?

Ok I'll complete remove the changes from that commit!

ping @delphaber. Thank you!

@tute sorry for the lateness. I removed the random port logic completely. I hope it's fine!

Thank you! :clap:


up-zenoh-recorder & up-zenoh-utwin: How are they initialized and given the Zenoh Runtime they will use?

Hey folks :wave:
Tagging in @mallets, @evshary, and @stevenhartley in particular.
From what I recall @evshary had started an experiment to see how we can start Zenoh plugins from within C++ or something to that effect. So I'd be interested to hear the parameters and outcome.

Given the topology diagram:

I wanted to drill down a little deeper:

How will the up-zenoh-utwin and up-zenoh-recorder plugins be initialized?
a. From C++?
b. From Rust?
Is it even possible to pass the plugins the Zenoh Runtime they will require, from C++?
If the plugins are initialized from C++, would the plugins need to have a C shim wrapped around them?
I suppose we would add a new way to instantiate a UPClientZenoh based on a Zenoh Session?

A first attempt at laying out these components:
classDiagram

    %% Very important to understand the mechanism by which:
    %%  The plugins written in Rust (up-zenoh-recorder &
    %%  up-zenoh-utwin) will be started
    %%
    %% If they are started in a way that they are called into
    %% by Rust code, that is ideal.
    %%
    %% If they are started from C++ code, then it's less clear
    %% how this would work, as the plugins need to be given
    %% an instance of the Zenoh Runtime in order to create their
    %% own Zenoh Session to use

    class z["Zenoh Runtime"] {

    }

    class uzr["up-zenoh-recorder"] {
        +UPClientZenoh up_client
        +new(Zenoh Runtime) Result<>
    }

    class uzu["up-zenoh-utwin"] {
        +UPClientZenoh up_client
        +new(Zenoh Runtime) Result<>
    }

    class usc["up-streamer-cpp"] {

    }

    style usc fill:#647687,color:#fff
    style uzr fill:#A0522D,color:#fff
    style uzu fill:#A0522D,color:#fff
    style z fill:#A0522D,color:#fff

    usc --* z: Initializes
    usc --* uzr: How are these plugins initialized?
 From Rust or C++?
    usc --* uzu: How are these plugins initialized?
 From Rust or C++?
    uzr ..> z: Takes copy of Runtime,
 creates Zenoh Session
 and uses that to
 initialize held UPClientZenoh
    uzu ..> z: Takes copy of Runtime,
 creates Zenoh Session
 and uses that to
 initialize held UPClientZenoh

Hi @PLeVasseur

How will the up-zenoh-utwin and up-zenoh-recorder plugins be initialized?

The plugins are written in Rust and can be loaded based on our configuration. zenoh-c is just a wrapper of Zenoh and the plugins are loaded in Zenoh.

Is it even possible to pass the plugins the Zenoh Runtime they will require, from C++?

While opening a session in C++, it'll load the plugins with its runtime directly.

If the plugins are initialized from C++, would the plugins need to have a C shim wrapped around them?

No, plugins are written in Rust in loaded by Zenoh. No need to consider C++.

I suppose we would add a new way to instantiate a UPClientZenoh based on a Zenoh Session?

Yes. Since we only have runtime in plugins, we need to get Session based on that runtime and initialize UPClientZenoh with that Session. I can create a PR to update this.

Also, here are some references for you:

How to load plugins in C++

Using code: https://github.com/evshary/zenoh-cpp-example/blob/17060c33e2b8333014216fb65d96a38f512c1881/src/z_router.cxx#L83
Using config: https://github.com/eclipse-zenoh/zenoh/blob/7c64d99e9deb75070b8c3845e0a29b7027fb48e9/DEFAULT_CONFIG.json5#L377

The plugin example:

https://github.com/eclipse-zenoh/zenoh/blob/7c64d99e9deb75070b8c3845e0a29b7027fb48e9/plugins/zenoh-plugin-example/src/lib.rs#L53

Here are notes while building

https://github.com/evshary/zenoh-cpp-example/tree/17060c33e2b8333014216fb65d96a38f512c1881?tab=readme-ov-file#note

@Mallets Please correct me if I'm wrong.

Sounds to me based on @evshary's comment here and @Mallets's feedback on Slack that up-zenoh-utwin and up-zenoh-recorder can be standard Zenoh plugins written 100% in Rust.
Closing as resolved!


Generate report from selected rows/columns of the Resutlset

Hi,
this is only a new feature request.... Could be very useful generate a HTML table (in clipboard) of the selected rows/fields selected in the Result View.
Thanks, and good work!!!!

Now you can set the wizard to export the selected columns/rows only.


feat: supported findRecord (#304)

1. Description

1.1. Checklist

[ ] The title of my PR starts with a Conventional Commit prefix (fix:, feat:, docs: etc).
[ ] I have read the Contributor Guide and followed the process outlined for submitting PRs.
[ ] I have updated/added tests for ALL new/updated/fixed functionality.
[ ] I have updated/added relevant documentation in docs and added dartdoc comments with ///.
[ ] I have updated/added relevant examples in examples.

1.2. Breaking Change

[ ] Yes, this is a breaking change.
[ ] No, this is not a breaking change.

1.3. Related Issues

Codecov Report

Merging #332 (af2bc5a) into main (b89491d) will increase coverage by 0.18%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #332      +/-   ##
==========================================
+ Coverage   79.03%   79.22%   +0.18%
==========================================
  Files         123      124       +1
  Lines         892      900       +8
==========================================
+ Hits          705      713       +8
  Misses        187      187

Impacted Files
Coverage Δ

...ackages/atproto/lib/src/entities/record_value.dart
100.00% <100.00%> (ø)

...oto/lib/src/repositories/repositories_service.dart
90.90% <100.00%> (+2.02%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Not installing on Shopify store

There is a good chance this is a Shopify issue and not an issue with the code but I have been beating my head against a wall with Shopify support all day and getting nowhere so I figured I would throw something in here.
If I go to deploy this app without changing anything it works perfectly fine on a development store but I get the following on any production store using the merchant install link:

I currently have an app built on this exact template running on a production store but cannot get any new ones to install.  Here are the function logs:

This, like most things, was a Shopify problem. Thanks anyways bluebeel, this template is awesome and has save so many of us so much time!

This, like most things, was a Shopify problem. Thanks anyways bluebeel, this template is awesome and has save so many of us so much time!


Modules Are Empty Objects

The script tags:
<script src="~/Scripts/curl.js"></script>
<script src="~/Areas/Analytics/Scripts/Charting/ChartingMain.js"></script>

ChartingMain:
curl({ baseUrl: "/" }, ["Areas/Analytics/Scripts/Charting/ChartingApp"], function (ChartingApp) {
    var chartingApp = new ChartingApp();
});

ChartingApp:
define([
"require",
"exports",
"Areas/Analytics/Scripts/Charting/ChartViewModel",
"Areas/Analytics/Scripts/Routing/RouteManager",
"Areas/Analytics/Scripts/Charting/FusionCharts/DualYCombination2D",
"Areas/Analytics/Scripts/Charting/FusionCharts/Column2D",
"Areas/Analytics/Scripts/Charting/KendoDataViz/DataVizConfigGenerator",
"Areas/Analytics/Scripts/Charting/KendoDataViz/DataVizRenderer",
"Areas/Analytics/Scripts/Charting/FusionCharts/FusionChartRenderer"
], function(
require,
exports,
ChartViewModel,
RouteManagerImpl,
DualYCombination2D,
Column2D,
DataVizConfigGenerator,
DataVizRenderer,
FusionChartRenderer
) {
//some code
});

If I look in the network tab of the chrome debugger, I can see that the charting app modules are getting loaded. When I put break points in the modules, they get hit and return. The majority of modules return constructor functions, however when the ChartingApp define callback runs, all the modules in the callback are just base Objects with no parameters.
I was previously using requirejs to load these modules and everything worked. What might I be missing here?

I think that fix in the dev branch not fixes this issue, but I was able to use curl with Typescript.
Minimal fix is the next one:
// track exports
def.exportsReady = function executeFactory (deps) {
    when(isPreload || preload, function () {
        // only resolve early if we also use exports (to avoid
        // circular dependencies). def.exports will have already
        // been set by the getDeps loop before we get here.
        if (def.exports) {
            execute(deps);
            if (def.exports['__esModule']) {
                def.progress(msgFactoryExecuted);
            }
        }
    });
};

In my project it plays nice but I'm not sure that it is the best fix.
Warning: it breaks normal modules with exports unless you add magic __esModule: true property to them.


linux, checking THP current system settings.

Ah, this is starting to look rather complicated and it is unclear whether we actually need it. Currently, it is only called if we cannot do it directly and via madvise without failing on an error. That seems robust enough?

I see your point, that is fine if you prefer not merging it.


Allow translations in Keshif browser

Currently, the tooltips / information is shown in English.
Provide a means to change the language setting.

Keshif has been supporting some basic translation options for a while using kshf.lang object.


Changes in plugin settings

Do not store settings structure in config, always get it from plugin object. It also helps to hot-reload settings config during development

That's pretty good. The settings config is a good idea as well


Store started_at or duration info in cached steps too

This would help to better monitor how much time a step take, especially depending on dataset size, so that we can see how certain changes influence processing speed and make more informed decisions about these changes and our size limits (see https://github.com/huggingface/dataset-viewer/issues/2878). Especially useful for duckdb and statistics step.
I can work on this if you think it's useful (I think it is) and don't break any logic.

good idea

Note that it will help for https://github.com/huggingface/dataset-viewer/issues/2279, because we will be able to compute thresholds based on the compute spent for a specific dataset/user/org in the last... N hours/days.


liberation fonts updates

Motivation for this change
Fixes #51331
I pushed fixes to the prior but the histories were too divergent.
The feedback that needed to be addressed from the last pr has been resolved.
Where we're using fetchFromGitHub and we've dropped all binary fonts.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

:heart: thanks!

Great, merging in this case :fireworks:


Leaflet sub-directives should be prefixed

From @wayofspark on March 19, 2014 16:18
leaflet subdirective should be prefixed to avoid directive collision with other libs. Most dangerous directive is the 'legend' directive which require "^leaflet". Common directive names should be prefixed ( aka leafletLegend, lfLegend or any prefix ).
Copied from original issue: tombatossals/angular-leaflet-directive#332

From @tombatossals on March 20, 2014 15:35
You're totally right, I read an article about this after the directive was already released. :( Now it would break compatibility, but it would be not much problematic to solve.
Maybe we can think about prefixing the directives on a future release.
Thanks for sharing!


Orthographic camera projection

What happened?
Why orthographic projection so much differs from perspective on big distances from the globe (see images)?
Zooming in-out also feels weird. Also no moon or anything else is rendered in orthographic mode.
The orthographic view was previously the same as perspective. And after updating Cesium library (from one of 2020 versions to the latest) we've got this strange behavior.
What was the reason behind it and how can I replicate the same view as in perspective not by digging into the core of the library?

Reproduction steps

Go to sandcastle
Zoom in/out, rotate.
Zooming and rotating feel like you are looking through a pinhole at a flat map, but not 3d globe.

Sandcastle example
https://sandcastle.cesium.com/index.html?src=Projection.html
Environment
Everywhere

Thanks for the report @askmyselfwhy.

The orthographic view was previously the same as perspective. And after updating Cesium library (from one of 2020 versions to the latest) we've got this strange behavior.

Would you be able to identify the release where this has changed? My guess would be around version 1.97.

I found a release when it's happened
https://github.com/CesiumGS/cesium/releases/tag/1.93
Previous releases work fine, like this https://github.com/CesiumGS/cesium/releases/tag/1.92

Thanks for the report @askmyselfwhy.

The orthographic view was previously the same as perspective. And after updating Cesium library (from one of 2020 versions to the latest) we've got this strange behavior.

Would you be able to identify the release where this has changed? My guess would be around version 1.97.

I found a release when it's happened https://github.com/CesiumGS/cesium/releases/tag/1.93
Previous releases work fine, like this https://github.com/CesiumGS/cesium/releases/tag/1.92

Probably one of the changes here:
https://github.com/CesiumGS/cesium/pull/10063

Thanks so much @askmyselfwhy!

For anyone else running into this issue, setting scene.globe.showGroundAtmosphere = false seems to resolve the bulk of the problems. I'll continue playing around to see if I can get atmosphere shaders working with an orthographic frustum.


No such file or directory "temp" when making a screenshot using the webserver

Description:
When using the webserver to make a screenshot, folder "temp" isn't created by default, so it can't store the screenshot.
Fix:
Add an empty temp folder.
This issue was reported by Bun#1234 on Discord, thank you.

Resolved in https://github.com/scr-autopilot/scr-autopilot/commit/391e1f6167012e023afe1fe40c24be6d5779e746


Feature: Support for threshold error reporting to mail/slack

I'm looking to use this for reporting errors to slack/email using cronjobs as suggested.\
How I think about this is that I set up flags to check for:

if memory usage is more than 80% send out an alert
if disk usage is more than 80% send out an alert

How it would work is maybe by having an alert mode eg. python monitor.py --error-mode, and messaging to either slack or email can be set up by piping the output to other tools e.g. slacktee.
Would this be something you're willing to accept as a feature/PR?

I'm very busy at the moment so I won't have time to work on this but if you want to work on a reporting function to include I can see how to merge the PR

Cool. I'll work on this and submit a PR.
On 20/02/06 08:46PM, MF Softworks wrote:

I'm very busy at the moment so I won't have time to work on this but if you want to work on a reporting function to include I can see how to merge the PR
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/mfsoftworks/Server-Monitoring-Script/issues/3#issuecomment-583232163


payroll

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

dynamics-gp/TOC.md
:white_check_mark:Succeeded
View

dynamics-gp/payroll/media/2ba47052805e7f27d276a88b2252bc77.jpg
:white_check_mark:Succeeded
View

dynamics-gp/payroll/media/2c2296ebb02f986dbb208027f0a103de.jpg
:white_check_mark:Succeeded
View

dynamics-gp/payroll/media/30cd3027f837642f55e44f1bcf36cef7.jpg
:white_check_mark:Succeeded
View

dynamics-gp/payroll/media/5d89ab7faca588b16120e25e6609f746.jpg
:white_check_mark:Succeeded
View

dynamics-gp/payroll/media/5fe68484446f2385cc3c4fa0a4443791.png
:white_check_mark:Succeeded
View

dynamics-gp/payroll/media/6230e96ff4a8a0dfd645c8c83e5b3411.png
:white_check_mark:Succeeded
View

dynamics-gp/payroll/media/95196f36cdb4dbdcce5d37dbc9552912.jpg
:white_check_mark:Succeeded
View

dynamics-gp/payroll/media/9e6e4bda64b079f0be50f366b9f8c9d1.jpg
:white_check_mark:Succeeded
View

dynamics-gp/payroll/media/a46418ecb58f57c36692981e689d7631.jpg
:white_check_mark:Succeeded
View

dynamics-gp/payroll/media/a4d9e20a94bd7bdf275d5a701f20fbdb.jpg
:white_check_mark:Succeeded
View

dynamics-gp/payroll/media/abcbe843dec6b5e24b6f9d52be7f2576.jpg
:white_check_mark:Succeeded
View

dynamics-gp/payroll/media/cf99fb6ede6de86b98a9e4816be3b006.jpg
:white_check_mark:Succeeded
View

dynamics-gp/payroll/media/d3210406076487f1b0554484e04b57f0.jpg
:white_check_mark:Succeeded
View

dynamics-gp/payroll/media/d82a0116f4c6685328ad23e68fa36619.jpg
:white_check_mark:Succeeded
View

dynamics-gp/payroll/media/dd7cca7719191663d61705f74784fb7f.gif
:white_check_mark:Succeeded
View

dynamics-gp/payroll/media/e90a4bb19ddde66e966913bc656d2781.png
:white_check_mark:Succeeded
View

dynamics-gp/payroll/media/f27a989097195bbf9cd02d20a9737e57.jpg
:white_check_mark:Succeeded
View

dynamics-gp/payroll/media/image001.gif
:white_check_mark:Succeeded
View

dynamics-gp/payroll/media/image008.jpg
:white_check_mark:Succeeded
View

dynamics-gp/payroll/media/image009.jpg
:white_check_mark:Succeeded
View

dynamics-gp/payroll/media/image010.jpg
:white_check_mark:Succeeded
View

dynamics-gp/payroll/media/image011.jpg
:white_check_mark:Succeeded
View

dynamics-gp/payroll/media/image012.jpg
:white_check_mark:Succeeded
View

dynamics-gp/payroll/media/image013.jpg
:white_check_mark:Succeeded
View

dynamics-gp/payroll/media/image014.jpg
:white_check_mark:Succeeded
View

dynamics-gp/payroll/media/image015.jpg
:white_check_mark:Succeeded
View

dynamics-gp/payroll/media/image016.jpg
:white_check_mark:Succeeded
View

dynamics-gp/payroll/media/image017.jpg
:white_check_mark:Succeeded
View

dynamics-gp/payroll/media/image018.jpg
:white_check_mark:Succeeded
View

dynamics-gp/payroll/media/image019.jpg
:white_check_mark:Succeeded
View

dynamics-gp/payroll/media/image020.jpg
:white_check_mark:Succeeded
View

dynamics-gp/payroll/media/image021.gif
:white_check_mark:Succeeded
View

dynamics-gp/payroll/media/image022.jpg
:white_check_mark:Succeeded
View

dynamics-gp/payroll/media/image3.PNG
:white_check_mark:Succeeded
View

dynamics-gp/payroll/media/us-year-end-01.PNG
:white_check_mark:Succeeded
View

dynamics-gp/payroll/us-year-end.md
:white_check_mark:Succeeded
View

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


Other modules: tracking progress

Thanks to the awesome @roblav96 for all this beautiful work back over at https://github.com/pilwon/node-yahoo-finance/issues/43 (in 2017 :sweat_smile:)  Have added status [X]'s on each point to track progress, followed by any additional comments below.

Hey man, love what you're doing here. I've been reverse engineering Yahoo's API recently and thought I'd provide some of my findings.
[n/a] Crum Manager
sstrickx/yahoofinance-api
@sstrickx wrote a crum manager in Java.

As reported by @smeijer in https://github.com/pilwon/node-yahoo-finance/issues/73 (thanks again!) and confirmed by me (and @MichaelDeBoey?), this is no longer required.

[X] Recommended Symbols
GET https://query1.finance.yahoo.com/v6/finance/recommendationsbysymbol/PLUG

Thanks, @pudgereyem, who added on 2021-02-09 in #28.

[ ] Daily Gainers
GET https://query2.finance.yahoo.com/v1/finance/screener/predefined/saved?formatted=false&lang=en-US&region=US&scrIds=day_gainers&count=5&corsDomain=finance.yahoo.com
[X] Search Autocomplete
GET https://autoc.finance.yahoo.com/autoc?query=PLUG&region=1&lang=en
[ ] Symbol Conversations
GET https://finance.yahoo.com/_finance_doubledown/api/resource/canvass.getMessageList;count=20;oauthConsumerKey=finance.oauth.client.canvass.prod.consumerKey;oauthConsumerSecret=finance.oauth.client.canvass.prod.consumerSecret;query=namespace = "yahoo_finance" and (tag = "PLUG");region=US;sortBy=createdAt;userActivity=true
[ ] Symbol News
GET http://feeds.finance.yahoo.com/rss/2.0/headline?s=PLUG&region=US&lang=en-US
Returns in RSS format, couldn't figure out how to get JSON.
[X] Multiple Symbol Quote
GET https://query2.finance.yahoo.com/v7/finance/quote?symbols=NVDA,PLUG,AMD
[ ] Multiple Symbol Historical Charts
GET https://query1.finance.yahoo.com/v7/finance/spark?symbols=PLUG,AMD&range=1d&formated=true
Only returns close data though =/
[X] Trending Symbols
GET https://query1.finance.yahoo.com/v1/finance/trending/US?lang=en-US&region=US&count=5&corsDomain=finance.yahoo.com

Thanks, @PythonCreator27, who added this on 2021-02-18 in #66.

[ ] Symbol Options
GET https://query1.finance.yahoo.com/v7/finance/options/[symbol]

Found by @PythonCreator27, more details below in https://github.com/gadicc/node-yahoo-finance2/issues/8#issuecomment-802463467.

Live Streamer
import * as Promise from 'bluebird'
import * as Nstream from 'stream'
import * as Nhttp from 'http'
import * as Nnet from 'net'
import * as Axios from 'axios'

const YH_IDK_KEY = {
	a00: 'Ask',
	a50: 'AskSize',
	b00: 'Bid',
	b60: 'BidSize',
	c10: 'Change',
	c63: 'ChangeRealtime',
	c64: 'DisputedChangeRealtimeAfterHours',
	c85: 'ChangeRealtimeAfterHours',
	c86: 'PercentChangeRealtimeAfterHours',
	g53: 'DayLow',
	h53: 'DayHigh',
	j10: 'MarketCap',
	l84: 'PriceRealtime',
	l86: 'PriceRealtimeAfterHours',
	p20: 'PercentChange',
	p43: 'PercentChangeRealtime',
	p44: 'PercentChangeRealtimeAfterHours',
	t53: 'DisputedTimestampForCommodities',
	t54: 'DisputedTimestampForStocks',
	v53: 'Volume',
	v00: 'Volume2',
	l10: 'LastSalePrice',
	t10: 'LastSaleTime',
	l90: 'EcnQuoteLastValue',
}
const YH_KEY_IDK = {
	Ask: 'a00',
	AskSize: 'a50',
	Bid: 'b00',
	BidSize: 'b60',
	Change: 'c10',
	ChangeRealtime: 'c63',
	DisputedChangeRealtimeAfterHours: 'c64',
	ChangeRealtimeAfterHours: 'c85',
	PercentChangeRealtimeAfterHours: 'p44',
	DayLow: 'g53',
	DayHigh: 'h53',
	MarketCap: 'j10',
	PriceRealtime: 'l84',
	PriceRealtimeAfterHours: 'l86',
	PercentChange: 'p20',
	PercentChangeRealtime: 'p43',
	DisputedTimestampForCommodities: 't53',
	DisputedTimestampForStocks: 't54',
	Volume: 'v53',
	Volume2: 'v00',
	LastSalePrice: 'l10',
	LastSaleTime: 't10',
	EcnQuoteLastValue: 'l90',
}

let total = 0
let stream = null as Nhttp.IncomingMessage
function startYahooStream() {
	return Promise.resolve().then(function() {
		let keys = [
			YH_KEY_IDK.Ask,
			YH_KEY_IDK.AskSize,
			YH_KEY_IDK.Bid,
			YH_KEY_IDK.BidSize,
			YH_KEY_IDK.Volume,
			YH_KEY_IDK.Volume2,
			YH_KEY_IDK.PriceRealtime,
			YH_KEY_IDK.PriceRealtimeAfterHours,
			YH_KEY_IDK.LastSalePrice,
			YH_KEY_IDK.EcnQuoteLastValue,
			YH_KEY_IDK.LastSaleTime,
			YH_KEY_IDK.DisputedTimestampForStocks,
		]
		return Axios.default.get('https://streamerapi.finance.yahoo.com/streamer/1.0', {
			params: {
				s: 'NVDA,PLUG,AMD',
				k: keys.join(','),
				r: 0,
				callback: 'parent.yfs_u1f',
				mktmcb: 'parent.yfs_mktmcb',
				gencallback: 'parent.yfs_gencb',
				mu: 1,
				lang: 'en-US',
				region: 'US',
				localize: 0,
			},
			responseType: 'stream',
		}).then<any>(function({ data }) {
			return new Promise<string>(function(resolve, reject) {
				stream = data as Nhttp.IncomingMessage

				stream.on('data', function(chunk) {
					try {
						let input = chunk.toString()
						let jsons = [] as Array<{ [key: string]: any }>

						let split = input.split('yfs_u1f(')
						split.shift()
						total = total + split.length
						split.forEach(function(v) {
							let i = v.indexOf(');}')
							let parsed = v.substring(0, i).replace(/(['"])?([a-zA-Z0-9_]+)(['"])?:/g, '"$2": ')
							if (!parsed) return;
							jsons.push(JSON.parse(parsed))
						})

						let sendi = {}
						jsons.forEach(function(s) {
							let symbol = Object.keys(s)[0]
							if (!sendi[symbol]) sendi[symbol] = {};
							let v = s[symbol]
							Object.keys(v).forEach(function(k) {
								let n = parseFloat(v[k])
								if (n == 0) return;
								let key = YH_IDK_KEY[k] as string
								sendi[symbol][key] = n
							})
						})
						console.log('sendi >', JSON.stringify(sendi, null, 4))

					} catch (error) {
						console.error('quotes.startYahooStream > catch > error', error)
					}
				})

				stream.on('end', resolve)
				stream.on('error', reject)
			})
		})

	}).catch(function(error) {
		console.error('quotes.startYahooStream > error', error)
		return Promise.resolve()

	}).finally(function() {
		console.warn('quotes.stream > FINALLY')
		if (stream) {
			stream.removeAllListeners()
			stream.destroy()
			stream = null
		}
		return startYahooStream()
	})
}
startYahooStream()

console.log('sendi >', JSON.stringify(sendi, null, 4))
sendi >
{
    NVDA: {
        LastSalePrice: 149.6,
        Volume2: 92,
        Ask: 149.5,
        AskSize: 1800,
        Bid: 148.2,
        BidSize: 100,
        PriceRealtime: 149.6
    }
}

Hope you find this helpful!

Added quote module in https://github.com/gadicc/node-yahoo-finance2/compare/e0f8d35f857826d394c6e3e811fc054e53540533...1cbc17a1741392cd29e82f2a9763bbf28007840c,

Created a PR that adds a new module for recommendationsBySymbol here; https://github.com/gadicc/node-yahoo-finance2/pull/28

Added new module recommendationsBySymbol  in https://github.com/gadicc/node-yahoo-finance2/pull/28.
@gadicc, would you mind updating the Issue to reflect this since I can't do it myself? 🙏

Thanks, @pudgereyem!  Have updated checklist.  Thanks again for all your awesome work on this.

@gadicc Found another endpoint: https://query1.finance.yahoo.com/v7/finance/options/[symbol]. Not sure if this was found already.
Query Params

Param Name
Param Type
Required

formatted
Boolean
false

crumb
String
false

lang
String
false

region
String
false

corsDomain omitted as it does nothing.
Another finding: query1 and query2 are the same. We can fetch from query1.finance.yahoo.com OR query2.finance.yahoo.com for the same data. Probably want to unify what we use in our code, or allow users to choose from each domain.
Hope this helps!

Hey, that's awesome, thanks @PythonCreator27!  Have added it to the list in the first post.
And for noticing about the alternative host names, we should indeed accommodate that.

@gadicc Appreciate the creds friend =]
Take a peek at https://github.com/roblav96/robinhood.tools it has some additional sources of financial data n more.

More specifically:

https://github.com/roblav96/robinhood.tools/blob/master/src/common/webull.ts
https://github.com/roblav96/robinhood.tools/blob/master/src/server/adapters/webull.ts

For example:
curl 'https://quoteapi.webull.com/api/quote/tickerRealTimes/full?tickerIds=925334567,925353501,913255891'

Hey @roblav96, great to see you're still involved with all this stuff, and to see you on this new repo!
Thanks for sharing the above.  I recall on the original repo too people were looking for additional sources, and I guess we should add this to the README or an FAQ or something, so thank you!
What is your rough breakdown of consumption from Yahoo vs Webull now?  Just out of curiosity.

Another note: recommendationsBySymbol is using an older version of the endpoint. The newer version is: https://query1.finance.yahoo.com/v7/finance/recommendationsbysymbol/PLUG. Yahoo's website does seem to use the older version, though. Changing the endpoint, though, would be a definite breaking change. @gadicc Do you think that it is worth it, or should we just create a brand new module for the newer version of the endpoint? The newer endpoint seems to just return a Quote[] instead of its own response format.

Well found, @PythonCreator27.  I would just leave it for now, from what you've said (and thanks for checking this all out!), there doesn't seem to be a compelling reason to upgrade.  If something happens to the v6 one, maybe we could switch to v7 but return the results in the same format if that's the only difference (and our validation checks will let us know if we missed anything).
It is something to think about though for all modules.  Maybe in future we should name the modules with Yahoo's API version, e.g. recommendationsBySymbolV7().  That would make it easier to cover these kinds of cases without needing to publish a new major version of the library just for one end point.  Is a bit verbose though.
In short, I suggest to leave for now and see what other cases come up in the future and then make a call.  Thanks for bringing attention to this.

@gadicc Webull provides the best scrapable market data, but they make you work for it. lol Start by exploring the network traffic of https://app.webull.com/ using your web browser's network debug tools.

Thanks, @roblav96!  I've made a start of referencing this (a long with some of the older suggestsions from node-yahoo-finance (v1) issues) on the Wiki: https://github.com/gadicc/node-yahoo-finance2/wiki/Alternative-Sources
I invite anyone to help improve this page with anything useful.

Another finding (endpoint): https://query2.finance.yahoo.com/ws/insights/v2/finance/insights?lang=en-US&region=US&symbol=AMD&getAllResearchReports=true&reportsCount=2
Insights - seems interesting. I was looking for answers to #138, and found this along the way... Will send a PR soon to add this.


Add url auth support

Currently, the url is parsed which is great but it's missing the auth piece.
For example, the current code base strips the user/password from the url like so<EMAIL_ADDRESS>This pull request will parse the user/password properly when the request is made.

Was just about to make the same PR!


Preview Image Quality Very Poor

The preview image pane is very blurry when compared with viewing the image on the OS, or another image viewing application. As an example, I've attached an image showing the GeoTag preview vs. viewing the image on the OS preview.

You can see that the image on the left (from MacOS preview) is significantly clearer than the Image Preview Pane in GeoTag. The images are so blurry that I often have to open the image in another app to get a clear view of it.

Yes.   The preview is a thumbnail limited to fit in a box 512x512 pixels wide.  This is done to minimize memory use.  It also happens to be about the size of the preview window the way I use GeoTag.  It would be blurry or pixelated when the image pane is extended to larger sized, i.e. GeoTag is full screen.
The next version on GeoTag will raise the thumbnail size to 1024x1024. Perhaps that will address your complaint.

Yeah that would help for sure, I use it with the preview at at least 900x900


Add MemoizedInstanceVariableName cop

Check for memoized methods to match the memo instance variable name
   # Method foo is memoized using an instance variable that is
   # not `@foo`. This can cause confusion and bugs.
   def foo
     @something ||= calculate_expensive_thing
   end

   # good
   def foo
     @foo ||= calculate_expensive_thing
   end

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[x] Run rake default or rake parallel. It executes all tests and RuboCop for itself, and generates the documentation.

Rebased, resolved conflicts, needs re-approval.

Sorry for the delay. I've been extremely busy lately. You'll need to rebase one more time and then I'll merge this. Just ping me when ready.

@bbatsov - This looks ready to go :) !

This doesn't seem to take into account methods that end in a question mark.
def something?
  @has_something ||= figure_out_something
end

Rubocop wants me to change the instance variable to @something?, which is not valid.
Using v0.53.0.

Perhaps it's just suggesting it wrong?  This does have coverage to not register an offense for such an example - https://github.com/bbatsov/rubocop/pull/5445/files#diff-056322207e6bf75183c3ef1e2ddffbbdR108 .  Did changing the ivar to @something resolve the issue?

Thanks @garettarrowood, you are right.  It's just suggesting the wrong method name.

Honestly, I don't like this cop. I prefer _ prefix for memoization due to making distinguish between regular instance variables and memoized instance variables.
Example:
def my_method
  @_my_method ||= expensive_method_call
end

I'd love to have an option to turn this on.


Add .Template.BasePath to template system

Why is this useful?
This allows to reliably include template files from a charts templates directory for example as suggested here

checksum/config: {{ include (print $.Chart.Name "/templates/secret.yaml") . | sha256sum }}

The above example has a problem that it doesn't work when the chart containing this is included as a subchart because the template name changes from mychart/templates/secret.yaml to parentchart/charts/mychart/templates/secret.yaml.
We also include templates in other templates heavily to build ConfigMaps that contain templates files.
Right now we have to use an ugly hack to make this work.
This PR would allow to include a template regardless of the charts location by writing something like:
 {{ include (print $.Template.BasePath "/templates/secret.yaml") . }}

This is awesome! I've been wanting exactly this, but figured I could just work around it for now.

Perhaps this should also be documented in the doc you mentioned so others can learn from this. :)

This just needs to be briefly documented here:
https://github.com/kubernetes/helm/blob/master/docs/chart_template_guide/builtin_objects.md

added some docs and rebased on master


Fix Deprecated Code

Update app gradle to the latest version and refactor deprecated code.

Thanks for looking into this!


add more machine-readable data about the license

This small change is meant to make LibreJS happy.
Closes: #1273

Oops, yes, I should have said that...
It comes from https://www.gnu.org/software/librejs/free-your-javascript.html
.
On Nov 23, 2016 03:49, "Lon Ingram"<EMAIL_ADDRESS>wrote:

@lawnsea commented on this pull request.
In Gruntfile.js
https://github.com/wycats/handlebars.js/pull/1274#pullrequestreview-9800252
:

@@ -22,8 +22,9 @@ module.exports = function(grunt) {
dist: {
options: {
processContent: function(content) {

       return grunt.template.process('/**!\n\n @license\n <%= pkg.name %> v<%= pkg.version %>\n\n<%= grunt.file.read("LICENSE") %>\n*/\n')

           + content;

       return grunt.template.process('/**!\n\n @license magnet:?xt=urn:btih:d3d9a9a6595521f9666a5e94cc830dab83b65699&dn=expat.txt Expat\n <%= pkg.name %> v<%= pkg.version %>\n\n<%= grunt.file.read("LICENSE") %>\n*/\n')

Where does this magical value come from?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/wycats/handlebars.js/pull/1274#pullrequestreview-9800252,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AACBi7u1-sNqfn4LnQGjYOIfJBf0Uab-ks5rA7ezgaJpZM4Kxx5G
.

@marado Ok, thanks. Do we also need /* @license-end */?

That's how I read the documentation, yes.
On Nov 23, 2016 15:21, "Lon Ingram"<EMAIL_ADDRESS>wrote:

@marado https://github.com/marado Ok, thanks. Do we also need /*
@license-end */?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/wycats/handlebars.js/pull/1274#issuecomment-262543838,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AACBi7nPQhPRzsVETx888HDjBdE4zyotks5rBFnogaJpZM4Kxx5G
.

I just was a bit confused, because there is no explicit magnet-link for the MIT-license on the page on the libreJS-page. It just says:

Please note: When people speak of the "MIT license" they mean either the X11 license or the expat license. Please see which license the code uses, and label it accordingly.

But the expat-license is what's in Handlebars' LICENSE.md file, so everything is fine.
I would like to merge the PR. I would normally do 'squash and merge' or 'rebase and merge' but that would make me the author of the commit (I'm not sure with 'rebase and merge'). Could you rebase the branch onto of master so that I can merge it?

Since this is only one commit, there's nothing really to squash, and rebase and merge should not create a new commit (thus not crunching the authorship of the patch). So feel free to go ahead and 'rebase and merge'. If in any case github ends up munching the authorship, well, I won't loose any sleep over it either :-)

BTW, the magnet links are on that page, check section <IP_ADDRESS> (List of magnet links for valid licenses).

You're right. It's still yours.
Yes, the magnet links are on the page, yes, but there is no link for MIT, just for "expat" and "X11".
I just read "expat"-license for the first time, so I wanted to look.

Oh, I see.
"MIT License" doesn't really refer to one particular license: the term is used for several (slightly different) licenses, so the term 'expat' is used for the particular variant used in Handlebars.
More info, if you're curious: https://en.wikipedia.org/wiki/MIT_License#Variants


Set up typespec-conversion branches and ensure relevant checks block them

For context, see the email with subject Let's all agree on how to do API versions for TypeSpec conversion PRs: "a horizontal change that versions to preview"

Hi @kurtzeborn, who's picking up this task in Konrad's absence?

@mario-guerra, I've replied on the thread.  Short response: we need PM to drive this.

More context:
Description:
Create dedicated branches named "typespec-conversion" in both the public and private specifications repositories. These branches will be used exclusively for the conversion of existing OpenAPI specifications to TypeSpec. The purpose of these branches is to isolate the conversion work from the main branches, allowing for thorough validation and testing before any changes are merged.
Tasks:

Identify the public and private specifications repositories that require the "typespec-conversion" branches:

https://github.com/Azure/azure-rest-api-specs
https://github.com/Azure/azure-rest-api-specs-pr

Create the "typespec-conversion" branch in each identified repository.
Ensure that the new branches are properly configured and protected to prevent unauthorized changes.
Communicate the creation of these branches to the relevant service teams and provide guidance on their usage.

Latest plan is to perform conversions in regular PRs to main/RPSaaSMaster.  As we dug deeper, the benefits of using a dedicated branch went away.


Improved the styling of Curious Cases #211

I have made some changes in the styling of curious cases. I added javascript to it and now when the user clicks on the Question, the answer will be displayed.

Issue Assignment Guidelines and Collaboration Process 📌
📝 Issue Assignment: Before you start working on any issues, please make sure that the issue is assigned to you. If it's a new issue, create one if it doesn't already exist. 📌
👤 One person will be assigned one issue at a time. When you raise a pull request for that issue, only then can you create the next issue. Work on one issue at a time, and once the pull request for that issue gets merged, you can create another issue. This approach allows maximum contributors to have a chance to contribute, resulting in more work being done on the repository. 🏃♂
🚫 Avoid Direct PRs: Refrain from creating direct pull requests (PRs) for issues that are already assigned to someone else. This helps prevent duplicated efforts and keeps our work organized. Even if an issue is not assigned to anyone, you should create the issue first.


Do I need rooted device to use your program?

Hi.
Can't find the pass to a sys/  at all lol, I'm tne newbie Android user tho.
Any  #help will be appreciated.
Thank you.

Hi,
root not needed.
Screen your paths in /sys/class/power_supply/battery

I'm so sorry about delay.  And tyvm for your fasf reply.
However probably I wasn't enouh clear,   problem is I can't find sys directory at all, have no idea where it's

Hey!
I have found /sys/class/power_supply/battery
There's a screenshot you asked for


[4.x] Set up PHPStan on GitHub Actions

This PR sets up PHPStan to run on GitHub Actions, as discussed in discussions#469.
Overview

[x] Sets up PHPStan to run on GitHub Actions on PHP 8.1 only
[x] Configures PHPStan to run the analysis on the src and tests folders
[x] Generates the baseline so that static analysis passes immediately

Baseline
Because this PR aims to set up PHPStan and not address the errors it reports, I've generated a baseline to make the pipeline succeed. We'll then be able to incrementally fix the problems in future PRs.

Once this has been merged, I'll create PRs for the 3.x and 2.x branches.

@nhedger Was this PR closed intentionally (along with related ones as per https://github.com/orgs/reactphp/discussions/469)? I've also started looking into PHPStan on max level with no baseline a while ago, would you like to keep working on this PR or do you think it makes sense for me to file my PR in comparison? In either case, thank you for your patience and for looking into this!

Ah sorry, it got closed because I cleaned up my forks. Completely forgot to check for open PRs.

For the reference: We've recently merged PHPStan on max level into 4.x with #76 and backported to 3.x with #77. Unlike the changes in this PR, we've gone through the project to fix all PHPStan errors encountered so we don't have to rely on a baseline. We'll go ahead with this and apply PHPStan on max level to all our projects in the future.
Thank you very much for your contribution and sparking this discussion! :+1:


Enum Request - Warehouse Request (Table 5765) - Field 4 "Source Document"

Please implement the field 4 ("Source Document") in Table 5765 ("Warehouse Request") as Enum.

Enum "Warehouse Request Source Document" implemented


Debugging Not Working on .NET 4.8 Isolated Azure Functions Project both in VS 2022 and VS Code IDEs

Steps to Reproduce:

Created the .NET 4.8 V4 Isolated Azure Functions Project - basic Http Trigger

without debugging working fine from VS IDE but not from VS 2022

Running the function with debugging is not working - debugger points are not hitting the code.

Following this MS Doc,

Kept debugging points at the code FunctionsDebugger.Enable(); and also in HttpTrigger Function Code.

Not Sure which process to be attached for debugging the .NET 4.8 Functions Isolated one.

But tried this cmdlet func host start --dotnet-isolated-debug in the VS Code Terminal of the same project - without hitting the breakpoints, it directly showing the output
Please Assist!

@HariKrishna598 Can you confirm you have the latest version of Visual studio? 17.4.0 is the latest as of today.
Also what is the core-tools version you have in your machine?

@HariKrishna598 Can you confirm you have the latest version of Visual studio? 17.4.0 is the latest as of today.
Also what is the core-tools version you have in your machine?

VS 2022 Version 17.4.0, 64-bit
AF Core Tools Version 4

Just to confirm, do you have the latest worker packages? In this issue, that fixed a similar issue for you.

Just to confirm, do you have the latest worker packages? In this issue, that fixed a similar issue for you.

<Project Sdk="Microsoft.NET.Sdk">
  <PropertyGroup>
    <TargetFramework>net48</TargetFramework>
    <AzureFunctionsVersion>v4</AzureFunctionsVersion>
    <OutputType>Exe</OutputType>
  </PropertyGroup>
  <ItemGroup>
    <PackageReference Include="Microsoft.Azure.Functions.Worker" Version="1.8.0" />
    <PackageReference Include="Microsoft.Azure.Functions.Worker.Extensions.Http" Version="3.0.13" />
    <PackageReference Include="Microsoft.Azure.Functions.Worker.Sdk" Version="1.7.0" />
  </ItemGroup>
  <ItemGroup>
    <None Update="host.json">
      <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
    </None>
    <None Update="local.settings.json">
      <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
      <CopyToPublishDirectory>Never</CopyToPublishDirectory>
    </None>
  </ItemGroup>
  <ItemGroup>
    <Folder Include="Properties\" />
  </ItemGroup>
</Project>

This is my .csproj in .NET 48 v4 Azure Functions Project

I suspect your visual studio instance failed to refresh the azure functions tools automatically. You can manually do this refresh by following the instructions here. Let us know if that solves your problem.

I suspect your visual studio instance failed to refresh the azure functions tools automatically. You can manually do this refresh by following the instructions here. Let us know if that solves your problem.

After manually updating the Azure Functions by following the instructions in above link, it worked (both debugging and running).

Do we need to update it manually every time when it fails (What is the root cause of failing the automatic update in VS IDE)?
What about the debugging fix in VS Code for the same issue?

@HariKrishna598

Do we need to update it manually every time when it fails (What is the root cause of failing the automatic update in VS IDE)?

No, you do not need to do that. Typically Visual studio will automagically update it for you. Looks like there was an issue on the VS side which prevented it. We have notified the VS team about this and they are looking into it.

What about the debugging fix in VS Code for the same issue?

Can you confirm what version of core tools you have in your machine? VS code uses the core tools installed in your machine. You can get this info by running func command. The result will have both core tools version and functions runtime version. If you are not on the latest core tools, I suggest upgrading to the latest and try again. Also, ensure you are using the latest worker nuget packages(in your .csproj file).
Let us know if you are still running into issues with this.

func --version > 4.0.4785 is the current version installed in my pc.


Non admin user can't use interactor (HRNET) 403 permission denied error

My actions before raising this issue

[x] Read/searched the docs
[x] Searched past issues

Loging in as Admin user lets me use HRNET
Being logged in as a worker account throws an Error:
Interaction error occured
Error: Request failed with status code 403. {"detail":"You do not have permission to perform this action."}.

in this thread it seems to have been observed as well:
https://github.com/openvinotoolkit/cvat/issues/4548
Expected Behaviour
No error, same behaviour admin and worker
Current Behaviour
Error see above
Possible Solution
Steps to Reproduce (for bugs)

Add 2 users, one admin one worker
use HR net with both, admin works, worker throws permission error

Context
Installing it and using HRNET in an larger effort to annotate data for speeding up the annotation process.
Your Environment

Git hash commit (git log -1): 3a88677eb4b6d509c3640ca2ec4dab662f642a55

Issue by @nikste

hi, yasakova-anastasia
i have met that problem too, have u solve this problem already?

@yasakova-anastasia @zhangzhangzhiyuan here is the fix: https://github.com/opencv/cvat/issues/4996#issuecomment-1266185805


API refactor: Store optional

Overhaul of kleros-api. Separate out Store related code as much as possible and move most functionality to kleros.arbitrator and kleros.arbitrableContract

Use of the store provider is now optional besides in some high level functions
Tests no longer use store
Arbitrator and ArbitrableContract delegate calls to their contractWrapper. So if you want to pass period you can call kleros.arbitrator.passPeriod and it will be delegated to the contractWrapper to make the call

Pull Request Test Coverage Report for Build 562

204 of 288 (70.83%) changed or added relevant lines in 11 files are covered.
109 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-18.03%) to 54.232%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/contractWrappers/abstractWrappers/AbstractWrapper.js
5
6
83.33%

src/utils/delegateCalls.js
11
13
84.62%

src/resourceWrappers/EventListeners.js
4
6
66.67%

src/contractWrappers/abstractWrappers/ArbitrableContract.js
19
24
79.17%

src/kleros.js
6
14
42.86%

src/contractWrappers/KlerosWrapper.js
42
50
84.0%

src/resourceWrappers/Notifications.js
4
13
30.77%

src/resourceWrappers/ResourceWrapper.js
21
37
56.76%

src/resourceWrappers/Disputes.js
73
106
68.87%

Files with Coverage Reduction
New Missed Lines
%

src/utils/Web3Wrapper.js
1
27.03%

src/contractWrappers/KlerosWrapper.js
3
68.65%

src/utils/StoreProviderWrapper.js
105
0.0%

Totals

Change from base Build 547:
-18.03%

Covered Lines:
524

Relevant Lines:
886

💛  - Coveralls


is it possible to make template_paths accept wild card ?

similar to excludePaths so instead of absolute paths, we can use something like */views

Hi, not at the moment

But such feature would be great benefit and welcomed :+1:


To get rid off fake HelmSourceFailed alert, we need to silence it or fix it for KVM

More details here: https://gigantic.slack.com/archives/C01176DKNP4/p1680525966212559
To fix the false-positive alert, we need to either release prometheus-rules on KVM or silence that alert on KVM installations
There's also problem with China (akita).

Created the AlertManager silences until: 2023-08-31

Anubis: https://alertmanager.g8s.he.k8s.emea.adsint.biz/#/silences/1c1cdfee-7ecd-466d-b361-1d3624cb54ab
Buffalo: https://alertmanager.g8s.buffalo.cex.tribe.sh/#/silences/79166efc-b161-4dfa-8a26-862c2e8d0f60
Bumblebee: https://alertmanager.g8s.bumblebee.kvm.gigantic.io/#/silences/8cdbe742-1262-4620-b7f4-caae4fe3d7b4
Puma: https://alertmanager.g8s.p5x.telekom.de/#/silences/b3e11e59-79e7-4e27-8e76-af2646b19147

Akita silence: https://github.com/giantswarm/silences/pull/944


"pvsadm purge vms" is failing when GET of an instance errors out

pvsadm purge vms is failing when GET of one of the vms in the list throws an error.
+ pvsadm purge vms --instance-id 013b8948-6cf4-4fae-9b70-71ddadd4f91a --before 4h --ignore-errors --no-prompt
2020/12/11 00:41:43 the apiendpoint url for power is syd.power-iaas.cloud.ibm.com
2020/12/11 00:41:43 Calling the New Auth Method in the IBMPower Session Code
2020/12/11 00:41:43 Calling the crn constructor that is to be passed back to the caller  65b64c1f1c29460e8c2e4bbfbd893c2c
2020/12/11 00:41:43 the region is syd and the zone is  syd04
2020/12/11 00:41:43 the crndata is ... crn:v1:bluemix:public:power-iaas:syd04:a/65b64c1f1c29460e8c2e4bbfbd893c2c:013b8948-6cf4-4fae-9b70-71ddadd4f91a::
2020/12/11 00:41:46 Calling the New Auth Method in the IBMPower Session Code
2020/12/11 00:41:46 Calling the crn constructor that is to be passed back to the caller  65b64c1f1c29460e8c2e4bbfbd893c2c
2020/12/11 00:41:46 the region is syd and the zone is  syd04
2020/12/11 00:41:46 the crndata is ... crn:v1:bluemix:public:power-iaas:syd04:a/65b64c1f1c29460e8c2e4bbfbd893c2c:013b8948-6cf4-4fae-9b70-71ddadd4f91a::
2020/12/11 00:41:49 Calling the New Auth Method in the IBMPower Session Code
2020/12/11 00:41:49 Calling the crn constructor that is to be passed back to the caller  65b64c1f1c29460e8c2e4bbfbd893c2c
2020/12/11 00:41:49 the region is syd and the zone is  syd04
2020/12/11 00:41:49 the crndata is ... crn:v1:bluemix:public:power-iaas:syd04:a/65b64c1f1c29460e8c2e4bbfbd893c2c:013b8948-6cf4-4fae-9b70-71ddadd4f91a::
2020/12/11 00:41:52 Calling the New Auth Method in the IBMPower Session Code
2020/12/11 00:41:52 Calling the crn constructor that is to be passed back to the caller  65b64c1f1c29460e8c2e4bbfbd893c2c
2020/12/11 00:41:52 the region is syd and the zone is  syd04
2020/12/11 00:41:52 the crndata is ... crn:v1:bluemix:public:power-iaas:syd04:a/65b64c1f1c29460e8c2e4bbfbd893c2c:013b8948-6cf4-4fae-9b70-71ddadd4f91a::
2020/12/11 00:41:55 Calling the New Auth Method in the IBMPower Session Code
2020/12/11 00:41:55 Calling the crn constructor that is to be passed back to the caller  65b64c1f1c29460e8c2e4bbfbd893c2c
2020/12/11 00:41:55 the region is syd and the zone is  syd04
2020/12/11 00:41:55 the crndata is ... crn:v1:bluemix:public:power-iaas:syd04:a/65b64c1f1c29460e8c2e4bbfbd893c2c:013b8948-6cf4-4fae-9b70-71ddadd4f91a::
2020/12/11 00:41:58 Calling the New Auth Method in the IBMPower Session Code
2020/12/11 00:41:58 Calling the crn constructor that is to be passed back to the caller  65b64c1f1c29460e8c2e4bbfbd893c2c
2020/12/11 00:41:58 the region is syd and the zone is  syd04
2020/12/11 00:41:58 the crndata is ... crn:v1:bluemix:public:power-iaas:syd04:a/65b64c1f1c29460e8c2e4bbfbd893c2c:013b8948-6cf4-4fae-9b70-71ddadd4f91a::
2020/12/11 00:42:01 Calling the New Auth Method in the IBMPower Session Code
2020/12/11 00:42:01 Calling the crn constructor that is to be passed back to the caller  65b64c1f1c29460e8c2e4bbfbd893c2c
2020/12/11 00:42:01 the region is syd and the zone is  syd04
2020/12/11 00:42:01 the crndata is ... crn:v1:bluemix:public:power-iaas:syd04:a/65b64c1f1c29460e8c2e4bbfbd893c2c:013b8948-6cf4-4fae-9b70-71ddadd4f91a::
2020/12/11 00:42:01 Failed to perform the operation... [GET /pcloud/v1/cloud-instances/{cloud_instance_id}/pvm-instances/{pvm_instance_id}][404] pcloudPvminstancesGetNotFound  &{Code:0 Description:pvm-instance does not exist, id: f2fb0f34-6008-41a3-a6fb-2f4100cf9eb8 Error:pvm-instance not found Message:}
Error: failed to get the instance: {"description":"pvm-instance does not exist, id: f2fb0f34-6008-41a3-a6fb-2f4100cf9eb8","error":"pvm-instance not found"}
E1211 00:42:01.817516      19 root.go:38] failed to get the instance: {"description":"pvm-instance does not exist, id: f2fb0f34-6008-41a3-a6fb-2f4100cf9eb8","error":"pvm-instance not found"}

The purge vms should continue if failure to get one of the vm so that script will proceed and exit the overall script with that error.

/assign @Rajalakshmi-Girish

/kind bug


Remove Packages folder from .gitignore

It is very common to have Packages folder in domain business logic (it happened to me and to other colleagues), and because of this entry, it has been excluded from our repository. Therefore I would remove this folder from the ignore list.

But it is much more common to have a NuGet-Packages folder in your solution, so I would vote for keeping it, in the gitignore.
In many cases this Packages folder is probably in the root of the repository, so it might be an option to change the rule, so that it only matches the folder there.
But honestly I don't think a collision with a "Packages" namespace in the BL is very likely... If It actually happens in a particular project, you can always whitelist the specific Packages folder in the affected sub project.

Actually I removed it because I have never had in any project that nuget Package folder (can't it be something legacy?). However, if you mean it is really a common situation, then keeping it ignored does make sense. I was Just wondering how can I generate this folder (by nuget) in my project?

Well I think all .NET project, that I have worked on, or contributed to, did use Nuget for 3rd part dependencies (sometimes even for internal ones). But there certenly is no rule to use it :-)

+1 to changing Packages to Packages/ to only match the root, -1 to removing it entirely

@paulcbetts that's what we had before https://github.com/github/gitignore/pull/1131

@shiftkey Blargh, just leave it alone then I guess, Domain Business Logic'ers will just have to change their names, or customize their .gitignore

@mne1991 I'm 👎 on changing these defaults - lots of people don't want to store their package outputs in Git, which is what this change would introduce.
You can add a additional rule like this if you happen to have other folders which are caught by this convention which you want to track:
!src/some/path/to/[Pp]ackages/


Gradle Dependency Verification Will Always Fail with Dependabot

I'm using gradle to build my Java project https://github.com/marco-schmidt/am Dependabot is enabled for this project and regularly creates pull requests for new dependency versions.
I recently started using gradle's new dependency verification feature: https://docs.gradle.org/current/userguide/dependency_verification.html
In a nutshell, that feature will test all files belonging to dependencies (libraries, plug-ins) against a list of allowed hash values to make sure repositories cannot introduce malware with slightly modified versions of popular dependencies: https://github.com/marco-schmidt/am/blob/master/gradle/verification-metadata.xml
Naturally, whenever a new version of a dependency is being used, its hash value is not yet contained in that XML file. Therefore, dependabot updating a version number now always results in a failed attempt to build, making it less useful and interfering with the statistics for successful upgrades (the compatibility percentage value shown in the pull request).
In order to keep the verification feature happy, a parameter can be used to update the XML file:
./gradlew build --write-verification-metadata sha256
How can I make Dependabot use that additional argument? I don't want to use that argument all the time because I want the build to fail on developer machines (including mine) if unknown or modified file versions appear.
Or is there any other way to handle this situation?
Thanks for providing Dependabot,
Marco

I'd be happy if it simply bumped the version in gradle/verification-metadata.xml just like it does in build.gradle. That way as long as it's still PGP-signed by the same publisher, the publishing key very likely won't have changed, and the build would succeed.  If the publishing key does change along with the version bump, that's something I would want to see fail the build, so the maintainer knows to go hand-validate the new publishing key and version.

Today we do all our Gradle parsing in Ruby/Regex, which is brittle and doesn't easily allow supporting extra features like this.
Long term, we'd like for Dependabot to be primarily glue code that hands off to the native package managers for actual resolving. But doing that for Gradle is a larger lift, so until that happens, we won't be able to implement this.
That said, this is all open source, so if anyone is interested in submitting a PR, we'd definitely be interested.
Related:

https://github.com/dependabot/dependabot-core/issues/1164


SPL for F0 variant

I'm trying to use PlatformIO with custom STM32F0 boards (F051 and F030). Previously I've been using libopencm3 framework, all worked fine, but now I need SPL as a framework.
It turns out that SPL installed by PlatformIO comes with F4, F3 and L1 variants. Would it be possible add missing variants, too, so people could use all STM32 MCUs?

I tried to re-pack the framework-spl and replace it manually. It seems to do the trick, but I don't know how to automate it or offer a pull request. Attached it my hand-crafted package with SPL for F0 included.
framework-spl.tar.gz

Updated, thanks!

Sorry @ivankravets but I would like to re-open this.
The assembly file framework-spl/stm32/cmsis/variants/stm32f0/startup_stm32f0xx.s is named with a lower-case .s, causing build problems. Renaming to the upper-case .S helps, but it should better be renamed in the SPL F0 package.

@zserge Please open new issue here ( https://github.com/platformio/platform-ststm32/issues ) with further explanations what need to do. Thanks.


Flash is not working with left/right/up/down animation

Describe the bug
Flash is not working with left/right/up/down animation, currently, it is only working with fixed
Screenshots

Would you like to work on the issue?
Yes

Yes it has not been implemented.


Cannot use '!' character in Subnautica console

Could be just my Canadian Multilingual Keyboard layout. But if it's not, would be great to fix.
Currently commands are using forward slash and string replaced before sending, i.e:
say /hint

It cannot be fixed it seems. Even with US keyboard people were having the same issue. Cannot type ! in subnautica console.
Ultimately, the mod could override the console code and allow for it.


[common] Explain tag.num-retained-max

Purpose

About #3502

Tests

API and Format

Documentation

+1


Add translation support

Currently, the "Prev", "Next" and "End Tour" buttons can't be translated. Either pull them out of the template and make them configurable (preferably via $.fn), so that we can use our own translation mechanisms, or have a proprietary simple translation mechanism.
Please :)

Closing due to lack of response.

@edu2004eu @cleyan You can use:
https://github.com/airbnb/polyglot.js

@sorich87, could you please reopen this issue?

There is a template option that'll be backward compatible. I mean you could do:
Let me know if there's any issue with this approach.

Your approach is not very DRY. Also, when the default template will change in a future update, it will not show up because it is overwritten with the old custom template. So you have to check for changes with every update.
The solution @edu2004eu proposed simply is a cleaner approach.

Best proof why manual checking for changes is not a good idea: The global options' template string on your API documentation page is outdated and does not show the current version.

Hi all, today have you got the solution ?


Bug in FirstSuccessfulStrategy

The '!' lost in the process of migration from 1.3.x to 1.4.0.

https://github.com/apache/shiro/pull/89.diff


Ship Area of Interest - Questions

Symbol Set Code: 25
Code: 200400
Ship Area of Interest requires one anchor point, defining the center.
Is this an area or a point symbol?
Is there a set number of circles within the Ship Area of Interest? Both the template and the example is shown as 20 circles.
If area, is the shape always a rectangle and should there be additional attributes that determine the size of the symbol (i.e. length and width)?

Is this an area or a point symbol?
These are area symbols.
Is there a set number of circles within the Ship Area of Interest? Both the template and the example is shown as 20 circles.
No. The template should be empty, and the example is simply demonstrating how it could look.
If area, is the shape always a rectangle and should there be additional attributes that determine the size of the symbol (i.e. length and width)?
No, the shape is not always a rectangle. There probably should be additional attributes for size.

Basically, a ship will have an Area of Interest, shaped either as an ellipse or circle (...401), or a rectangle (...402). Inside that Area of Interest could be 1-n Maneuver Areas (...400), that are circles. All of these are dynamic polygons.
Active Maneuver Area (...500), is currently listed as a static symbol, to be drawn around the Own Ship Icon (30150000), but it's my opinion that this should be dynamic.
The USN and USMC will review this question and possibly submit change proposals.
FYI @wmcgrane


SPARKC-318: Fix mapping a column with null value

[SPARKC-318] - Failed to convert column X of Y.Z to java.lang.Long: null

This at least needs tests :) Especially the case that was broken before

test this please

Build against Scala 2.10 finished with success

Build against Scala 2.11 finished with success

@RussellSpitzer added tests in a new PR at #1015

@delyand we're rolling out a Contributor License Agreement for the Spark Cassandra Connector and need your signature to be able to use your contributions.
Please follow instructions to sign, the whole process is completely electronic and will only take minutes.
Don't hesitate to email cla at datastax.com if you have any additional questions.

Sorry for the long delay. I just now saw the CLA and signed it, if you're still planning on using this change. I didn't have the chance to support it further at the time being.

Hi, is there any plan to merge this PR?

Nope, please see the accompanying ticket, basically this is a huge code
change for little gain with an easy workaround. Also we fixed the issue in
Cassandra itself
On Mon, Jun 5, 2017, 3:27 AM Chen Bin<EMAIL_ADDRESS>wrote:

Hi, is there any plan to merge this PR?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/datastax/spark-cassandra-connector/pull/1008#issuecomment-306155627,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAZNYYt-TI1Xu5Oqk98oR8HqmhQEJ2tRks5sA9gAgaJpZM4JUAOs
.

Actually I am thinking of a different pr ...
On Mon, Jun 5, 2017, 7:03 AM Russell Spitzer<EMAIL_ADDRESS>wrote:

Nope, please see the accompanying ticket, basically this is a huge code
change for little gain with an easy workaround. Also we fixed the issue in
Cassandra itself
On Mon, Jun 5, 2017, 3:27 AM Chen Bin<EMAIL_ADDRESS>wrote:

Hi, is there any plan to merge this PR?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/datastax/spark-cassandra-connector/pull/1008#issuecomment-306155627,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAZNYYt-TI1Xu5Oqk98oR8HqmhQEJ2tRks5sA9gAgaJpZM4JUAOs
.

Ah this is the ticket where we still don't have a test case showing what fails currently and how it is fixed by this patch.


Add comments for modules so that the description for each module can be auto generated

@embbnux @u9520107   I think we need to choose a tool/standard for the code comment so that we can generate API documentation automatically.

I will start to work on this.


Allow :start_value in GPIO.start_link opts to explicitly set value on start

This helps in the cases that one might want to have the initial starting value of a pin be 1, especially when using this under supervision. Or just if one wants to ensure a specific value is set on port start.
A minor use-case, but definitely helpful.
I debated using something like start_link(16, {:output, 1}) vs putting this in the opts, but decided to go this route for more explicity options and see if there are opinions otherwise.
I think this could be accomplish in the gpio_port.c instead of making an extra call to write after starting the port. But, my C-fu is weak so opted for the easier route unless there is major objection.

This looks good to me. Thanks!

FYI - I'll post a new Elixir ALE release on Monday-ish.


Fix github-json output option

This PR fixes the following problem in Syft 0.44.1:
$ syft packages . -o github-json
1 error occurred:
	* bad output format: 'github-json'

Two things:

I'm testing Syft out and noticed this issue while trying out all the output options. I've not looked closely at the codebase at all. Feel free to tell me this is entirely the wrong approach :smiley:
I added a note to the docs, but couldn't find a suitable link/reference for the github format in #836 (nor from a quick search around). If there is a suitable link, I'm happy to add it to f82bcdedbd434116471a573c68d72f470ea057fe.

@StevenMaude this format isn't completely finalized. I'm not sure we should treat it as such until it is finalized, but at such time github-json sounds like an appropriate option 👍

@StevenMaude thanks for fixing this. Your PR has one last thing to pass our tests, there is a code formatting issue, to fix it just run the following command from the project directory in your local machine:
make lint-fix

Then commit the changes and it should be good. I will be here if you need help.

@jonasagx: thanks for your patience! Hopefully I've wrangled it, I think it was just a gofmt issue in the end.


replicate.exceptions.ReplicateError: HTTP error: (413, 'Request Entity Too Large')

So i'm using the replicate whisper API (openai/whisper:e39e354773466b955265e969568deb7da217804d8e771ea8c9cd0cef6591f8bc)
It's been wroking fantastic on smaller files but any file bigger than 80mb gives me the error
replicate.exceptions.ReplicateError: HTTP error: (413, 'Request Entity Too Large')
but when i'm using the website it works fine
Anyone know the problem?
This is my code
 output = replicate.Client().run("openai/whisper:e39e354773466b955265e969568deb7da217804d8e771ea8c9cd0cef6591f8bc",                                                     input={"audio": open(File_Path.get(), "rb")})         F.pack_forget()
(File_Path is getting obtained from a Tkinter input field)

Hey there,
This has been mentioned here and here. From my understanding, as I shoot to solve the same problem: the workaround is to provide it a URL since it is using HTTP, GET method has a specific size limit.
Hopefully these links can help you!

Thanks for responding, @natelalor.
Closing as a duplicate of https://github.com/replicate/replicate-python/issues/68


Repository dropdown doesn't refresh

In ReviewboardPollingBuilder.java, if 'repositories' has already been populated via loadRepositories(), it doesn't try again.
public ListBoxModel doFillReviewbotRepoIdItems() {
  // populate list of repositories with (name, id)
  ListBoxModel items = new ListBoxModel();
  // select option to allow polling requests for all repositories
  items.add("-- any --", "-1");
  //if no repositories - try to load them again
  if (repositories.isEmpty()) loadRepositories();
  // select options to filter by repository id.
  for (Map.Entry<String, Integer> e: repositories.entrySet()) {
    items.add(e.getKey(), e.getValue().toString());
  }
  return items;
}

It spent a couple hours this morning chasing down why my repository is not showing up in the dropdown and it seems to be because it was configured in ReviewBoard after my Jenkins job was configured for another repo..  Restarting Jenkins fixed the dropdown contents but it would be nice if it would fetch new content each time you go in to Configure in the job.

Originally repositories were fetched on the fly, then I added caching because we have insanely large list of Reviewboard repositories and a lousy connection between Jenkins and Reviewboard. I will add an option to disable it in plugin configuration.


Detachable webview

I've been looking into how to create detachable webviews, or better said something that can transfer the content into another webview instance. It might be a bit hacky currently, I'm pretty sure it's not prod quality yet, but I wanted something working to start the conversation.
I've seen #1682 which led me to the conclusion that I'm not alone with the desire to have something like this, hence this PR.

Nice! Just a few thoughts, could add WebViewManager::TransferGuest(sourceGuestInstanceId, targetElementInstanceId, targetGuestInstanceId) and let guest-view-manger use it to change mapping of guestinstanceid to embedder. Also ipc operations in web-view can be moved to guest-view-internaland ones' in web-view-ref can be handled by guest-view-manager. Not sure about the need for WebViewRef class.

@arsher This is awesome, good start!

Can you add some documents for it?

Sure thing, I'm sorry I had about zero amount of free time, I'll try to do some docs and responses for the comments during the weekend.

Is this the same thing as https://github.com/atom/atom/issues/9334 ?

I'm closing this since there were no updates and the PR is outdated, but I'll be glad to reopen once there are new updates.


Add  Reactive Streams Support

Type: Add Reactive Streams Support
Currently our project(Spring WebFlux+Kotlin/Kotlin Coroutines) uses AWS SDK for Kotlin which includes Kotlin Coroutines support, but it lacks Spring integration and the APIs are changed frequently.
This project(spring-cloud-aws) is great for Spring/AWS integration, but it does not support RectiveStreams  by default.
Please add ReactiveStreams/Reactor support(eg. ReactiveXXXXtemplate, ReactiveXXXListner) from scratch to align with the current Listner/AsyncListener , template/AsyncTemplate etc.

Hi @hantsy, the team decided not to add reactive components to this project to reduce maintenance scope and considering project Loom in JDK 21.
You may consider other libraries such as river from @gabfssilva which supports Kotlin Coroutines natively. If you're interested please try it out and provide feedback.
Thanks.


attic list bombs with "ValueError: year is out of range"

I'm trying to use attic to dump our design department NAS to our backup server. I have mounted the NAS over CIFS with the read-only option, and the dump ran fine (amazing compression/dedupe too; ~800gb resulted in a repo under 160gb!)
When I try to list the archive, I get this:
# attic list /bup/cobian/test-nas-dsn.attic::1416970025
drwxrwxrwx 171968813 171966977        0 Nov 25 10:32 mnt/nas-dsn1/WIP
-rw-rw-r-- 171968813 171966977    10244 Nov 14 09:06 mnt/nas-dsn1/WIP/.DS_Store
drwxrwxrwx 171968813 171966977        0 Jun 30 10:05 mnt/nas-dsn1/WIP/.TemporaryItems
<snipped about 30 files from here>
drwxrwxrwx 171968813 171966977        0 Mar 13  2012 mnt/nas-dsn1/WIP/Artwork Master Files/8. Reference
-rwxrwxrwx 171968813 171966977     6148 Mar 13  2012 mnt/nas-dsn1/WIP/Artwork Master Files/8. Reference/.DS_Store
Traceback (most recent call last):
  File "/root/build/prefix/lib/python3.4/site-packages/cx_Freeze/initscripts/Console.py", line 27, in <module>
  File "prefix/bin/attic", line 3, in <module>
  File "/root/build/prefix/lib/python3.4/site-packages/attic/archiver.py", line 715, in main
  File "/root/build/prefix/lib/python3.4/site-packages/attic/archiver.py", line 705, in run
  File "/root/build/prefix/lib/python3.4/site-packages/attic/archiver.py", line 275, in do_list
ValueError: year is out of range

It appears to be caused by this directory:
drwxrwxrwx   2 171968813 171966977    0 May 28  60056 BRIGHTON
I'm not sure exactly how the hell this folder has ended up with the year 60056 (goddam Apple!) and is something else I'll have to investigate, but not sure if there's a more graceful way attic could handle this? As it stands, I don't think I'll be able to restore from this archive.

Any update on this issue? I can't restore from this repo while attic dies when it comes across this file :(

why don't you set the file date to something reasonable (as a workaround)?

Thanks for the reply. I figured out I can extract the files I need, assuming I know the path (which I did in this case). It's only list that is affected by this.
I did fix the date (using touch) when I originally experienced this, but since then another file came up with this oddity and we didn't know until trying to list the repo for a file recovery.

Can you please add the attic version to your report above that gave this traceback?

I reproduced on current master branch. Working on it.

Awesome, that looks like it should resolve this issue :)


modify tests to be able to create containers

Optionally allowing to publish them on the test registry (implemented by issue #444 )

https://github.com/DBuildService/dockerfile-parse (by the Project Atomic folks) is a nice way to parse and update Dockerfiles


Json version

Updates Newtonsoft.Json version for Dev15 build and removes hard dependency on the older version.

I've manually tested this already and it works as expected.

I checked the latest Preview build's PrivateAssemblies folder and the one that's there is 8.0.3, so that must be right :)


Spyder 3.0.0: ImportError: No module named 'PyQt4' on fresh conda env Python 3.5/2.7

Upon importing Matplotlib.figure, get PyQt4 import error.
This happens even when not using Spyder, so now I can't use matplotlib at all even without Spyder.
What steps will reproduce the problem?

conda update conda; conda create -n spydtest; source activate spydtest
conda install spyder matplotlib
from matplotlib.pyplot import figure

What is the expected output? What do you see instead?

from matplotlib.pyplot import figure
Traceback (most recent call last):
File "", line 1, in
File "~/miniconda3/envs/spyd/lib/python3.5/site-packages/matplotlib/pyplot.py", line 114, in
_backend_mod, new_figure_manager, draw_if_interactive, _show = pylab_setup()
File "~/miniconda3/envs/spyd/lib/python3.5/site-packages/matplotlib/backends/init.py", line 32, in pylab_setup
globals(),locals(),[backend_name],0)
File "~/miniconda3/envs/spyd/lib/python3.5/site-packages/matplotlib/backends/backend_qt4agg.py", line 18, in
from .backend_qt5agg import FigureCanvasQTAggBase as _FigureCanvasQTAggBase
File "~/miniconda3/envs/spyd/lib/python3.5/site-packages/matplotlib/backends/backend_qt5agg.py", line 16, in
from .backend_qt5 import QtCore
File "~/miniconda3/envs/spyd/lib/python3.5/site-packages/matplotlib/backends/backend_qt5.py", line 31, in
from .qt_compat import QtCore, QtGui, QtWidgets, _getSaveFileName, version
File "~/miniconda3/envs/spyd/lib/python3.5/site-packages/matplotlib/backends/qt_compat.py", line 137, in
from PyQt4 import QtCore, QtGui
ImportError: No module named 'PyQt4'

Please provide any additional information below
Spyder 3.0 beta 1-6 worked fine. Spyder 3.0 release newly installed PyQT5 (i had pyqt4 before) then this error started
Versions and main components

Spyder Version: 3.0.0 release
Python Version: 3.5.2
Operating system: Ubuntu 16.04
Matplotlib: 1.5.3

Dependencies
jedi >=0.8.1     :  0.9.0 (OK)
matplotlib >=1.0 :  1.5.3 (OK)
nbconvert >=4.0  :  4.2.0 (OK)
numpy >=1.7      :  1.11.1 (OK)
pandas >=0.13.1  :  None (NOK)
pep8 >=0.6       :  1.7.0 (OK)
psutil >=0.3     :  4.3.1 (OK)
pyflakes >=0.6.0 :  1.3.0 (OK)
pygments >=2.0   :  2.1.3 (OK)
pylint >=0.25    :  1.5.4 (OK)
qtconsole >=4.2.0:  4.2.1 (OK)
rope >=0.9.4     :  0.9.4-1 (OK)
sphinx >=0.6.6   :  1.4.6 (OK)
sympy >=0.7.3    :  None (NOK)

This is resolved by editing ~/.config/matplotlib/matplotlibrc and setting
backend: qt5agg


int/ext + fragility handled incorrectly

This is a logical bug.

I think this is the correct procedure:

Combine exterior-interior function (exterior stage, interior stage) with interior stage-damage function (interior stage, damage) to get exterior stage-damage function (exterior stage, damage)
multiply exterior stage-damage function (exterior stage, damage) by the fragility function (exterior stage, prob of failure)
combine exterior stage-probability-weighted damage function (exterior stage, probability-weighted damage) with exterior stage-frequency (frequency, exterior stage) to get the damage frequency function (frequency, probability weighted damage)


Dont scroll to end when the diagnostics panel is open

Dont scroll to end when the diagnostics panel is open, but keep the clear selection, so that panel text is not selected when opened.
Closes  #446

Coverage increased (+0.009%) to 28.245% when pulling 6adc6c1091a90cd0d98b75ad61a3f3a722bf7131 on predragnikolic:improvement/diagnostics-dont-scroll-to-end into 2869978d8b46d717da27eb0ac7a7e234840b218d on tomv564:master.

We'll see how it goes!

Raoul, you got merge permission :confetti_ball: congratulation :)
What could possibly go wrong.

Well, for simple changes like these I think it's okay for me to merge ;) The serious pull requests I'll leave up to Tom.


Load source generators from framework references

Motivation
Source generators built into .NET 7 aren't being loaded when targeting .NET 7.
Modifications

Move some logic from NuGetReferenceGenerator into a shared extension
Split source generation loading into a separate class
Load source generators from framework references
Prevent duplicate loads from packages and framework references
When loading source generators also allow them to resolve additional assemblies from the same directory (required for loading Microsoft.Interop.SourceGeneration.dll)

/merge


libvirt_setup: fix cleanup_one_node function

cleanup function expects a dom object and not a string
this was changed (overlooked) as a last refactoring step in the PR #767
A manual test with this fix was already successful.

+1


Vue 3 Support

Are there any plans on updating this to support vue 3 now that quasar is using Vue 3?

https://github.com/stefanvanherwijnen/quasar-app-extension-http-authentication/issues/45


wrong link to next page

routes.rb
	['/test', '/test2'].each do |test|
		get test, :to => 'home#test'
	end

controller
class HomeController < ApplicationController
	def test
		@items = Item.all.page params[:page]
	end
end

view
=link_to_next_page @items, 'Next Page'
localhost:3000/test
next page href="test?page=2"
localhost:3000/test2
next page href="test?page=2", but need:
next page href="test2?page=2"
How describe route in paginate helper?

This could rather be a Rails issue than a kaminari issue, but could you explain why you need two routes pointing to the same action? In my opinion, you should not have to do that in the first place...

I'm closing this issue. In practice, there shouldn't be multiple routes to a single action and this is rather an edge case that shouldn't exist. Even if there was a case where you'd need two routes, Rails should be responsible for generating a link to the right URL since kaminari just uses Rails' #url_for.
Please submit a bug report with a test case that fails if you still think this is a kaminari issue.

example:
webmarket, categories: toys, food, wear, ... (more than 50 pieces).
mysite.com/toys
mysite.com/food
mysite.com/wear
...
routes.rb:
Category.routes.each do |category|
	get	"#{category}", :to=> 'search#items'
end

controller:
def items
cat = Categories.number current_route
items = Item.where category: cat
render something
end

I think in this case that feature is appropriate

It seems like a valid use case, there might be a better way that works well with kaminari, though. I'll re-open this one and see if I could reproduce the issue.

I solved this issue. I create many procedures dynamically, use meta programming method:

define_method

Cool, let us know if there's still an issue in kaminari!


feat(occupancy_grid_based_outlier_fillter): add config file to autoware_launch

Description
Related https://github.com/autowarefoundation/autoware.universe/pull/6745, add config param to autoware_launch.

Tests performed

Not applicable.
Effects on system behavior

Not applicable.
Interface changes

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[ ] I've confirmed the contribution guidelines.
[ ] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

It should be better to wait untill https://github.com/autowarefoundation/autoware.universe/pull/6745 is ready to merge?
I think we should merge this first to prevent autoware crush.


Package tendermint-rs

Closing for now, if there is an executable to be packaged then please re-open with more specific information.


How long time to training? pls answer!

With this training, it takes 1 second per session.
41/41 [==============================] - 1s 34ms/step - loss: 3.9857 - val_loss: 3.9261

This is repeated 600 times, so the time required will be at least 600 seconds.
And the time required for training depends on the environment and other factors, so we do not know exactly how many hours it will take.


extract_subset.py  not found

extract_subset.py not found, will you please take a look at this question ? thank you

Sorry for the confusion here. This part is actually moved to the run_glue_ima.py, where you can extract the subset with the argument "--train_data_percentage".

extract_feature.py: error: unrecognized arguments: -rp glue

extract_feature.py: error: unrecognized arguments: -rp glue

typo fix: "-rp" change to "-nlu_dataset"

extract_feature.py: error: unrecognized arguments: -rp glue

Hi, did you generate all images for glue dataset ?


Update Random Number Generators

I'm proposing to use Cryptographically Secure RNGs. I also changed the deprecated notation of try!() to ?, and put the tests in groups/mod.rs within a test module to only have rand and rand_chacha as dev-dependencies.

[clabot:check]


chat has ended shows up if you don't press next in the last step when you come back to the chatflow

chat has ended shows up if you don't press next in the last step when you come back to the chatflow, because the session still exists and the last step "chat has ended" is still to be show

@xmonader any suggestion how to solve this? Cause the chatflow engine does not know when the last step is the last step. unless we enforce the chatflows to indicate this somehow.

Hmm I think the chatflow_step decorator we can have last=True maybe and if we are that point we invalidate the session?

verified.


Route params can cause TransferState error

🐞 Bug report
Description
TransferStateService.useScullyTransferState() seems to cause fetchTransferState() to throw an error when navigating from a route without params to a route with params..
but only when refreshing the page on a route without params..
and not when refreshing the page on a route with params.
The result of the useScullyTransferState observable seems to behave differently depending on the route which Angular is freshly loaded from.
🔬 Minimal Reproduction
Step by step in the README. The error reveals itself when navigating in specific steps.
Repo: https://github.com/donmckenna/scully-transferState
Live Example: https://wizardly-johnson-148805.netlify.app/

New Angular app with Scully markdown blog.
Add 2 routes. I called them / and /search
Allow /search to take a param in search-routing.module.ts
Update routes in scully.--.config.ts to include this param and assign it a type of 'ignored'
Create a shared component which runs available$ Scully posts through TransferService.useScullyTransferState() named 'transferStateTest'
Include this component in each route component.
Refresh the page on / and, separately, on routes with params, and navigate around with routerLink to see their different behaviours.

💻Your Environment
Angular Version:
Angular CLI: 9.1.7
Node: 12.16.3
OS: win32 x64

Angular: 9.1.9
... animations, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... router
Ivy Workspace: Yes

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.901.7
@angular-devkit/build-angular     0.901.7
@angular-devkit/build-optimizer   0.901.7
@angular-devkit/build-webpack     0.901.7
@angular-devkit/core              9.1.7
@angular-devkit/schematics        9.1.7
@angular/cli                      9.1.7
@ngtools/webpack                  9.1.7
@schematics/angular               9.1.7
@schematics/update                0.901.7
rxjs                              6.5.5
typescript                        3.8.3
webpack                           4.42.0

Scully Version:
"@scullyio/init": "0.0.28",
"@scullyio/ng-lib": "latest",
"@scullyio/scully": "latest",

🔥 Exception or Error
ERROR TypeError: Cannot read property 'transferStateTest' of null
at MapSubscriber.mapper [as project] (pluck.js:13)
at MapSubscriber._next (map.js:29)
at MapSubscriber.next (Subscriber.js:49)
at FilterSubscriber._next (filter.js:33)
at FilterSubscriber.next (Subscriber.js:49)
at BehaviorSubject.next (Subject.js:39)
at BehaviorSubject.next (BehaviorSubject.js:30)
at TapSubscriber._tapNext (scullyio-ng-lib.js:398)
at TapSubscriber._next (tap.js:40)
at TapSubscriber.next (Subscriber.js:49)

fixed in #517


Can we add https to elm-lang.org and http://package.elm-lang.org

Can we please add https certificate added to the package site. Behind proxy, elm-repl cannot download the libraries.
Free certificate is provided by
https://letsencrypt.org/

We will be switching them to HTTPS relatively soon!