[STORM-2998]Wrong className in LoggerFactory.getLogger method

https://issues.apache.org/jira/browse/STORM-2998
If we use LoggerFactory.getLogger method in a class,the most accurate approach is use this class as a parameter.

+1

Thanks @liu-zhaokun, merged to master


Windows 10 Problem to start insomnia

Is there an existing issue for this?

[X] I have searched the issue tracker for a bug that matches the one I want to file, without success.

Insomnia Version
2021.5.3
What operating system are you using?
Windows
Operating System Version
Windows 10 1709
Installation method
download from Insomnia
Last Known Working Insomnia version
No response
Expected Behavior
I should see something if I start the App. I never used Insomnia before therefore an empty black panel is not I expected.
Actual Behavior
After Starting there is only black panel nothing more.
Reproduction Steps

Start the app
Black empty Panel
I can navigate to View-DevTools
There is an Error in the console: Not allowed to load local resource: file:///C:/Users/OW#1114\AppData\Local\insomnia\app-2021.5.3\resources\app.asar/renderer.html
I can navigate to the Directory with Filemanager and copy, read, and delete app.asar.

Additional Information

Hi @trireme, thank you for reporting this.
Are you by any chance still able to reproduce this using the latest Insomnia release (2022.3.0)?

I am not able to reproduce it as before, the latest version of insomnia worked for me, I assume the main problem is the path "file:///C:/Users/OW#1114" I used before, the # is a common problem for most apps. As I don't use it now, it could have been the cause for the error. For me the issue is closed.


Not working on server with Windows7

Stopped working 3 or 4 days ago.
Dowloaded the latest verstion - error "cannot load user and info"
What it can be?
Server is in Moscow

Maybe some one have ideas why does it happen?


Standardize linters for all repos

Is your feature request related to a problem? Please describe.
Note from community member: Maybe it would also be interesting to define which linters do we want to use for the different file-types; for example:
Hadolint -> Docker
ESLint -> JavaScript
YAMLlint -> YAML
Describe the solution you'd like
Finalize standard set of linters for Pyrrha repos.
Describe alternatives you've considered
N/A
Additional context
N/A

Hello. I am interested to Contribute to the Project can you describe more about this issue?


docs: Write a blog post about why we need a new plugin, and why you do not need a seperate vector DB

/cc @VoVAllen

https://modelz.ai/blog/pgvector


FEAT: WikiDP File Format Information PAR API endpoint

A read-only PAR endpoint for format information held in Wikidata:

GET methods for FileFormat and FormatFamily.

I've now filtered out this to a repository all of it's own: https://github.com/openpreserve/par-wikidp so closing this PR.


Is it possible to update/refresh the CodeLens?

My plan is to display the results (as done here in VS) of unittests against each method. Clicking the code lens would allow one to run (or re-run) that test making it necessary to update the code lens with the latest result.
As far as I can tell this isn't possible.

We should add an optional event to the provider, ala onDidChange similar to have document providers allow that.

@DonJayamanne @eamodio @pflannery The first cut for this just landed and I'd like to collect feedback. Have a look at the changes or to code against the latest API do the following:

open your extensions package.json file
change the engine.vscode property to *
run npm run postinstall (assuming you have the vscode module as dev-dependency)
undo the engine-version change from step 2

start verifying...

@Jakeii , @aeschli , thanks for the work guys. Missed this notification. Will look into this this weekend. 👍

verified with a code sample. New API works.

@jrieken sorry I haven't gotten a chance to look at this yet (been swamped) -- I will hopefully have some time this weekend.

@jrieken looks great -- its now in the latest GitLens -- Thanks!


[Feature]: Give tap/click/hover state styles to actionable cards #412

Is your feature request related to a problem?
Currently it's not very obvious that a button or card has been interacted with until HA changes the state of the entity and the icon lights up.
With slow or non-instant entities like Hue light groups, it could be upto 1 second later that all HA recognises a change in state and tells the card about it. This is fairly bad from an accessibility and UX point of view as there is no feedback to the user that their action has been recognised.
https://user-images.githubusercontent.com/44971861/165781756-e8347086-d4a1-4445-8aa2-823a5708d19b.mov
Without instant feedback of interaction, the user could very easily tap the entity again the entity and end up turning it off.
Describe the solution you'd like
Interaction states, e.g. :hover and :active states on interactive cards
For example:
https://user-images.githubusercontent.com/44971861/165783770-1708fbfd-0bb8-400f-be3a-cccf60c94361.mov
Describe alternatives you've considered
No response
Additional context
No response

Would you be up to make a PR for this yourself?

@sc8696 How are you doing it in the demo? I want this functionality too and tried:
style: |
  ha-card :active {
    background-color: rgba(147, 227, 215, 0.5) !Important;
  }

In the card's YAML but for some weird reason I get this:

Where the card highlights as well as the 'active' internal section, which looks like mushroom-card
Would be really good to get the confirmation 'clicks' like this, on my phone I get haptic feedback but sometimes it's hard to be sure I clicked the right button, this solves that.

Your CSS highlight every active element. You want ha-card:active.

Your CSS highlights every active element. You want ha-card:active.

You're totally right, I missed that, thanks mate!
Don't suppose you'd have an answer for this loosely related issue would you?


How can i configure elastic search appender?

Hi, help me with this, please
To work with ES i configure a connection:
  require 'faraday_middleware/aws_signers_v4'

  transport_configuration = lambda do |f|
    f.request :aws_signers_v4,
              credentials: Aws::Credentials.new(ENV['AWS_ACCESS_KEY_ID'], ENV['AWS_SECRET_KEY']),
              service_name: 'es',
              region: 'eu-west-1'

    f.response :logger
    f.adapter Faraday.default_adapter
  end

  transport = Elasticsearch::Transport::Transport::HTTP::Faraday.new \
  hosts: [{scheme: 'https', host: ENV['ELASTICSEARCH_URL'], port: '443'}],
  &transport_configuration

Elasticsearch::Model.client = Elasticsearch::Client.new transport: transport, trace: true

And it works, but how can i do the same for semantic logger ES appender?
        SemanticLogger.add_appender(
            appender: :elasticsearch
            url: 'http://some-search-server-here.eu-west-1.es.amazonaws.com'
        )

Semantic Logger only supports the url based option for configuring elastic search. If you need it to support some other configuration mechanism, feel free to submit a PR.

I found it possible by creating a custom appender using this AWS ES Documentation


Not an issue, but tips for printing

Certain models will print nicer and will get a smoother surface, when printed "standing and upside down". Especially P**-Models.
Not a huge difference on an Prusa, but older printers will might get way better prints.
*Standing means: Printed while standing on the upper or lower surface (website view).

Fixed.


Why can't edit WPF forms with vscode?

I've found old yt movie, where POSH had wpf editor for xaml wpf gui files. But now, there's only windows forms editor in vscode? Why?
How can I integrate PSSCriptPad with vscode for xamls?

You'd have to go back to an extension version prior to 2024.7.0, as looking at the changelog, that's when PSScriptPad was replaced with the Forms Editor, which doesn't support WPF. I'm not sure why this was done, but there you have it. Unfortunately, those versions are also licensed (and I can't imagine them playing nice with the current version of VS Code), so unless you have a license, you'll have to make do by using VS Code and PSScriptPad separately.
However, you could create the initial XAML in PSScriptPad, then edit it and the script further in VS Code. I think those are the only two answers at the moment that I can think of.


Add queue in enc/dec group message

Reference from libsignal-protocol-ee5b8ba.min.js
I have a question, why don't we maintain libsignal-node instead of just Baileys?

@BochilGaming , I have the ideia of create a new repository here with the libsignal. I want to rewrite it to typoscript

@BochilGaming , I have the ideia of create a new repository here with the libsignal. I want to rewrite it to typoscript

Thank you for your response. It's a good idea to also maintain libsignal and rewrite it to TypeScript. Maybe in the future, I can also help with the new libsignal codebase.

@BochilGaming , repository created: https://github.com/WhiskeySockets/libsignal-node


Add support for Zapier CLI

Overview

Add support for Zapier CLI. The shell plugin provisions a configuration file at ~/.zapierrc. The shell plugin also imports the deploy key from the same file at the same path, if it exists.
Type of change

[x] Created a new plugin
[ ] Improved an existing plugin
[ ] Fixed a bug in an existing plugin
[ ] Improved contributor utilities or experience

Related Issue(s)

Resolves: #
Relates: #

How To Test

Install Zapier CLI and set up the shell plugin.
Create a Zapier project with zapier init and enter that Zapier folder.
Test with zapier register. You should get a response asking for ? What is the title of your integration? Any other response is an indication of a failure of the provisioner.

Changelog

Additional information

[x] Check this box if this is a Hashnode Hackathon submission

Hello! Don't forget to also write an accompanying blog post on Hashnode with the tags 1Password and BuildWith1Password (not just putting #1Password in the text - but use the Hashnode tags in the CMS. :)
The full instructions are here.
The blog post is due by midnight Pacific (San Francisco) time - so you have about 8 hours from now.

Thank you, @techcraver. I will be submitting a few more shell plugins that I had been working on. Can I submit one blog post for all shell plugins?

Thank you, @techcraver. I will be submitting a few more shell plugins that I had been working on. Can I submit one blog post for all shell plugins?

That's awesome!
It's best to do separate blog posts for each shell plugin - it maximizes visibility and chances at winning.

Corresponding blog post: https://rajapri28613.hashnode.dev/1password-shell-plugin-for-zapier

Great!
On Fri, Jun 30, 2023 at 7:38 PM Maniraja @.***> wrote:

Corresponding blog post:
https://rajapri28613.hashnode.dev/1password-shell-plugin-for-zapier
—
Reply to this email directly, view it on GitHub
https://github.com/1Password/shell-plugins/pull/337#issuecomment-1615381883,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AF2UZ3GVRDNW3S2JXGL7N73XN6ETTANCNFSM6AAAAAAZ2MZ6YA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hey @rajapri28613 , I see there hasn't been any activity on this PR in a bit, is there anything that we can help you with?
We'd love to get this to the finish line together! 🚀


Inputs Excel file

For the input file, all the sheets will have the multi year option. However the user will have to fill out the data for all the files. Although I might do a macro for them to just repeat the data. The copying shouldn't be too hard tho.
So now I have to fix all the inputs template macro. I also want to separate the sales volume and outbound cost sheet.
So now I would have the customer list, then a sales volume list from template an outbound cost list from template

[ ] 1. Do the template for customer list

[ ] 2. Do a macros that ask for the years, and generate all the sheets with the columns that changes with years as Year - Name

[ ] 3. Do I or do I not need to create a macro?

I think they can just do Ctrl+A to select the one year and Ctrl+Shift+Right to select all the years and paste


Add diag aug as default for ASE, update diag aug behavior for directed

Reference Issues/PRs
Fixes #287
Fixes #59
What does this implement/fix? Explain your changes.

Updates diag aug to use in and out degree
Adds diagonal augmentation as a default for ASE with the option to turn it off

Any other comments?

Seems like we should definitely do augment diagonal for ASE, but what about omni?
Should I add a short tutorial about diag-aug (I already wrote the simulation, would just have to write up some text to go with it)
Is it worth saving the modified matrix that is actually embedded? Sometimes I like to make screeplots and the like, or just heatmap the matrix itself.

Codecov Report

Merging #288 into master will increase coverage by 0.62%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #288      +/-   ##
==========================================
+ Coverage   87.51%   88.13%   +0.62%
==========================================
  Files          33       33
  Lines        1954     2090     +136
==========================================
+ Hits         1710     1842     +132
- Misses        244      248       +4

Impacted Files
Coverage Δ

graspy/utils/utils.py
91.26% <100%> (+0.08%)
:arrow_up:

graspy/embed/ase.py
100% <100%> (ø)
:arrow_up:

graspy/models/rdpg.py
100% <100%> (ø)
:arrow_up:

graspy/simulations/simulations.py
96.55% <0%> (+0.18%)
:arrow_up:

Continue to review full report at Codecov.

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


Added ablility to set headless mode for webdriver with firefox too

Is this repository still maintained?

Yes, it is :rofl:
Thank you!


Update Wasmtime's policy on cargo vet

This was discussed at today's Wasmtime meeting out of some concerns around our current policies. Namely I felt the current state of affairs is not striking the right balance between cost and benefit with our usage of cargo vet. After discussion we've reached consensus around two changes to our cargo vet policy documented here in this PR:

An exemption can be added for "popular crates" at any time with no review required. This should handle most big crates that are needed for various dependencies. The thinking behind this is that a supply-chain attack against these crates is highly likely to be detected in a short time due to their popularity. Coupled with the fact that changes to Wasmtime take a minimum of two weeks to get released means that it's an unlikely exploitation vector.

Maintainers are recommended to push directly to contributor's PRs for cargo vet entries instead of making separate PRs. This avoids the need for contributor rebasing and additionally solves the problem where the vet entries land in a separate PR but then the contributor's PR takes much longer to land. In the interim some vet entries have been cleaned up by accident which requires re-landing the PR to add the entries.

Some points of note:

I kept the 10,000 crate download metric since spot checking a few dependencies seemed to show that as a resonable threshold. For example from the ONNX PR the compact_str dependency clears this threshold but the capstone dependency does not. That makes sense to me since I don't expect disassembling to be all that popular on crates.io
I briefly looked into what it might take to "comment on a PR if a github actions fails". I'd love to drop a comment on PRs from contributors pointing to this documentation when the cargo vet entry fails in CI. Turns out GitHub Actions has an example of doing exactly that (well, mostly), but it's pretty nasty so I didn't pursue it yet.


double vision in the chart

Why does the chart have double vision
1
2

maybe the reason is in xib files  ，the linechartview  backgrou color is default （no color）

The screenshots are 403.

http://f.picphotos.baidu.com/album/s%3D900%3Bq%3D90/sign=b2080a14be99a9013f3557362dae7b46/8d5494eef01f3a2980cd03ce9f25bc315c607cad.jpg
http://c.picphotos.baidu.com/album/s%3D900%3Bq%3D90/sign=7b7d4928912bd40746c7dffd4bb2ef6c/c8ea15ce36d3d539c459d8c33c87e950352ab04e.jpg

403 again

http://i.imgur.com/A0WIVHC.png

http://i.imgur.com/x6HIJSS.png

Man you messed something up real good! Maybe your
opaque/backgroundColor properties are wrong, or you initialized too
many instances if the chart in one view, or many other things.
Take a step back, create a new view and start from there.
The other option is to post a "working" project so that the good people of
"Charts" can help you investigate.

it s the  backgroundColor, i set it as default transparent,.
why the transparent backgroud lead to this?

Try to set opaque=NO. And read a little bit about it in Apple's docs :)

thanks  ,I understand


Regular expressions with /g are stateful and Node 0.10 has a bug in string replace with /g RegExp.

Node 0.12 onwards string replace works fine.
https://groups.google.com/forum/#!topic/v8-dev/VVuLI2VwHGU
http://stackoverflow.com/questions/1520800/why-regexp-with-global-flag-in-javascript-give-wrong-results

var postcss = require('postcss'),
    plugin = require('./dist/index.js');

console.log('\n\nFollowing output will be invalid every third time on Node v0.10.\n\n');

for (var i = 0; i < 10; i++) {
    var result = postcss([ plugin({}) ]).process(".cls{background-image:url('http://example.com/img1.jpg?etag=%22foo%22')}");
    console.log(result.css);
}

Output:
Following output will be invalid every third time on Node v0.10.

.cls{background-image:url('http://example.com/img1.jpg?etag="foo"')}
.cls{background-image:url('http://example.com/img1.jpg?etag="foo"')}
.cls{background-image:url(http://example.com/img1.jpg?etag="foo")}
.cls{background-image:url('http://example.com/img1.jpg?etag="foo"')}
.cls{background-image:url('http://example.com/img1.jpg?etag="foo"')}
.cls{background-image:url(http://example.com/img1.jpg?etag="foo")}
.cls{background-image:url('http://example.com/img1.jpg?etag="foo"')}
.cls{background-image:url('http://example.com/img1.jpg?etag="foo"')}
.cls{background-image:url(http://example.com/img1.jpg?etag="foo")}
.cls{background-image:url('http://example.com/img1.jpg?etag="foo"')}

Hi @ben-eb ... can you consider this pull request?

Hi @paras20xx,
Node 0.10 has been on the way out for some time, and in 5 months time their long term support will come to an end. Therefore I don't think it is worth patching in support for it now, especially as there are much better releases of Node! And, I have not "officially" supported Node 0.10 since cssnano v2; the reason being is that there is a maintenance burden for that platform, & I use Node 5 right now.
However, if you need the 0.10 support then you can link your fork as a top level dependency of your application, using npm 3;
{
  "devDependencies": {
    "cssnano": "^3.6.0",
    "postcss-normalize-url": "paras20xx/postcss-normalize-url#patch-1"
  }
}

I appreciate that this patch is "only" a two line change but I don't want to commit to supporting 0.10 for the next 5 months.

@ben-eb ... The "prepublish" script is generating the dist/index.js file. So, I'll probably need to publish a new package.
Yeah, we are also planning to do update node version, but that will take sometime, till then we are stuck.


Review package.json

optimize name and description
add repository and keywords

No functional changes.
Main reason of the PR are the missing keywords which are missing in this new module list :slightly_smiling_face:

Thanks! Now it looks nice in the list :slightly_smiling_face:


[oauth2-proxy] deprecate chart

Description of the change

oauth2-proxy has an official helm chart. see linked issue below.
Benefits

Possible drawbacks

Applicable issues

fixes #1131

Additional information

Checklist

[x] Chart version bumped in Chart.yaml according to semver.
[x] Title of the PR starts with chart name (e.g. [home-assistant])
[x] Variables are documented in the README.md (this can be done with using our helm-docs wrapper ./hack/gen-helm-docs.sh stable <chart>)

@onedr0p no warning before removing a deprecated chart?

@EternalDeiwos Old versions should still work but you should migrate to the official chart. https://github.com/oauth2-proxy/manifests


Nested attributes support

Hey @janko-m,
I've just found another bug I believe, because I was trying to do exact same thing as you described here:
https://github.com/shrinerb/shrine/blob/master/doc/multiple_files.md
But with Mongoid (and currently supported embedded models on my branch), and well, it just skips all the Shrine uploaders stuff (kind of). It just saves file_data (if file attribute is just called file), but there's no uploading or validations. (I must use Mongoid 6.4 though).

I've added tests for it, confirming that bug exists, but only for embedded models:
https://github.com/LinkUpStudioUA/shrine-mongoid/tree/nested-attributes

OK, this is actually bug in #swap method for embedded records, once it will be fixed,
cascade_callbacks association option will help.


fix(module:affix): position offset when has deferred render componets.

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/NG-ZORRO/ng-zorro-antd/blob/master/CONTRIBUTING.md#commit
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Application (the showcase website) / infrastructure changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: #671
What is the new behavior?
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information

Coverage decreased (-0.003%) to 41.712% when pulling 0b7512dfb720804d67a91cd731c94667d306035c on cipchk:issue-671 into 9073bc8f0d05eaac5be8474ad319a9d33346f2c9 on NG-ZORRO:master.


ocrmypdf: 9.8.0 -> 9.8.2

version bump also fixes build after pikepdf update:
https://github.com/NixOS/nixpkgs/pull/88680

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
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 nixpkgs-review --run "nixpkgs-review wip"
[x] 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)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

builds and works for me
nixpkgs-review
1 package built:
ocrmypdf

though many tests fail in not a lot of time
= 105 failed, 134 passed, 29 skipped, 8 deselected, 1 xfailed in 240.64s (0:04:00) =
builder for '/nix/store/a2nd36bfdnfhx5glk8hymrrv2pyvmbmw-ocrmypdf-9.8.2.drv' failed with exit code 1
error: build of '/nix/store/a2nd36bfdnfhx5glk8hymrrv2pyvmbmw-ocrmypdf-9.8.2.drv' failed
nix-build -A ocrmypdf  0.43s user 0.11s system 0% cpu 4:04.21 total

short test summary
----------------------------- Captured stderr call -----------------------------
Scan: 100%|██████████| 1/1 [00:00<00:00, 360.89page/s]
OCR:   0%|          | 0.0/1.0 [00:00<?, ?page/s]
------------------------------ Captured log call -------------------------------
INFO     ocrmypdf:_sync.py:253 Using Tesseract OpenMP thread limit 3
=========================== short test summary info ============================
FAILED tests/test_acroform.py::test_acroform_message - FileNotFoundError: [Er...
FAILED tests/test_ghostscript.py::test_ghostscript_feature_elision - FileNotF...
FAILED tests/test_image_input.py::test_image_to_pdf - FileNotFoundError: [Err...
FAILED tests/test_image_input.py::test_jpeg_in_jpeg_out - FileNotFoundError: ...
FAILED tests/test_main.py::test_quick - FileNotFoundError: [Errno 2] No such ...
FAILED tests/test_main.py::test_oversample[hocr] - FileNotFoundError: [Errno ...
FAILED tests/test_main.py::test_oversample[sandwich] - FileNotFoundError: [Er...
FAILED tests/test_main.py::test_force_ocr - FileNotFoundError: [Errno 2] No s...
FAILED tests/test_main.py::test_argsfile - FileNotFoundError: [Errno 2] No su...
FAILED tests/test_main.py::test_maximum_options[pdf-hocr] - FileNotFoundError...
FAILED tests/test_main.py::test_maximum_options[pdf-sandwich] - FileNotFoundE...
FAILED tests/test_main.py::test_maximum_options[pdfa-hocr] - FileNotFoundErro...
FAILED tests/test_main.py::test_maximum_options[pdfa-sandwich] - FileNotFound...
FAILED tests/test_main.py::test_german - FileNotFoundError: [Errno 2] No such...
FAILED tests/test_main.py::test_missing_docinfo - assert <ExitCode.other_erro...
FAILED tests/test_main.py::test_uppercase_extension - FileNotFoundError: [Err...
FAILED tests/test_main.py::test_pagesegmode[hocr] - FileNotFoundError: [Errno...
FAILED tests/test_main.py::test_pagesegmode[sandwich] - FileNotFoundError: [E...
FAILED tests/test_main.py::test_algo4 - AssertionError: assert 1 == <ExitCode...
FAILED tests/test_main.py::test_jbig2_passthrough - FileNotFoundError: [Errno...
FAILED tests/test_main.py::test_masks - FileNotFoundError: [Errno 2] No such ...
FAILED tests/test_main.py::test_linearized_pdf_and_indirect_object - FileNotF...
FAILED tests/test_main.py::test_very_high_dpi - FileNotFoundError: [Errno 2] ...
FAILED tests/test_main.py::test_destination_not_writable - AssertionError: Ex...
FAILED tests/test_main.py::test_form_xobject - FileNotFoundError: [Errno 2] N...
FAILED tests/test_main.py::test_skip_big_with_no_images - FileNotFoundError: ...
FAILED tests/test_main.py::test_no_contents - FileNotFoundError: [Errno 2] No...
FAILED tests/test_main.py::test_compression_preserved[baiona.png] - Assertion...
FAILED tests/test_main.py::test_compression_preserved[baiona_gray.png] - Asse...
FAILED tests/test_main.py::test_compression_preserved[baiona_alpha.png] - ass...
FAILED tests/test_main.py::test_compression_preserved[congress.jpg] - Asserti...
FAILED tests/test_main.py::test_compression_changed[baiona.png-jpeg] - Assert...
FAILED tests/test_main.py::test_compression_changed[baiona_gray.png-lossless]
FAILED tests/test_main.py::test_compression_changed[congress.jpg-lossless] - ...
FAILED tests/test_main.py::test_sidecar_pagecount - FileNotFoundError: [Errno...
FAILED tests/test_main.py::test_sidecar_nonempty - FileNotFoundError: [Errno ...
FAILED tests/test_main.py::test_pdfa_n[1] - FileNotFoundError: [Errno 2] No s...
FAILED tests/test_main.py::test_pdfa_n[2] - FileNotFoundError: [Errno 2] No s...
FAILED tests/test_main.py::test_pdfa_n[3] - FileNotFoundError: [Errno 2] No s...
FAILED tests/test_main.py::test_text_curves - FileNotFoundError: [Errno 2] No...
FAILED tests/test_main.py::test_output_is_dir - AssertionError: assert 1 == <...
FAILED tests/test_main.py::test_output_is_symlink - AssertionError: Traceback...
FAILED tests/test_main.py::test_fast_web_view[1.0-0-pdfa-False] - FileNotFoun...
FAILED tests/test_main.py::test_fast_web_view[1.0-0-pdf-False] - FileNotFound...
FAILED tests/test_main.py::test_fast_web_view[0.0-0-pdfa-True] - FileNotFound...
FAILED tests/test_main.py::test_fast_web_view[0.0-0-pdf-True] - FileNotFoundE...
FAILED tests/test_main.py::test_fast_web_view[1.0-1-pdfa-False] - FileNotFoun...
FAILED tests/test_main.py::test_fast_web_view[1.0-1-pdf-False] - FileNotFound...
FAILED tests/test_main.py::test_fast_web_view[0.0-1-pdfa-True] - FileNotFound...
FAILED tests/test_main.py::test_fast_web_view[0.0-1-pdf-True] - FileNotFoundE...
FAILED tests/test_main.py::test_image_dpi_not_image - FileNotFoundError: [Err...
FAILED tests/test_metadata.py::test_preserve_metadata[pdfa] - FileNotFoundErr...
FAILED tests/test_metadata.py::test_preserve_metadata[pdf] - FileNotFoundErro...
FAILED tests/test_metadata.py::test_override_metadata[pdfa] - AssertionError:...
FAILED tests/test_metadata.py::test_override_metadata[pdf] - AssertionError: ...
FAILED tests/test_metadata.py::test_creation_date_preserved[pdf-trivial.pdf]
FAILED tests/test_metadata.py::test_creation_date_preserved[pdf-jbig2.pdf] - ...
FAILED tests/test_metadata.py::test_creation_date_preserved[pdfa-trivial.pdf]
FAILED tests/test_metadata.py::test_creation_date_preserved[pdfa-jbig2.pdf]
FAILED tests/test_metadata.py::test_xml_metadata_preserved[pdf] - FileNotFoun...
FAILED tests/test_metadata.py::test_xml_metadata_preserved[pdfa] - FileNotFou...
FAILED tests/test_metadata.py::test_kodak_toc - FileNotFoundError: [Errno 2] ...
FAILED tests/test_optimize.py::test_jpg_png_params - FileNotFoundError: [Errn...
FAILED tests/test_optimize.py::test_jbig2_lossy[False] - FileNotFoundError: [...
FAILED tests/test_optimize.py::test_jbig2_lossy[True] - FileNotFoundError: [E...
FAILED tests/test_optimize.py::test_flate_to_jbig2 - FileNotFoundError: [Errn...
FAILED tests/test_optimize.py::test_multiple_pngs - FileNotFoundError: [Errno...
FAILED tests/test_page_numbers.py::test_limited_pages - FileNotFoundError: [E...
FAILED tests/test_preprocessing.py::test_deskew - FileNotFoundError: [Errno 2...
FAILED tests/test_preprocessing.py::test_remove_background - FileNotFoundErro...
FAILED tests/test_preprocessing.py::test_exotic_image[pdf-sandwich-palette.pdf]
FAILED tests/test_preprocessing.py::test_exotic_image[pdf-sandwich-cmyk.pdf]
FAILED tests/test_preprocessing.py::test_exotic_image[pdf-sandwich-ccitt.pdf]
FAILED tests/test_preprocessing.py::test_exotic_image[pdf-sandwich-jbig2.pdf]
FAILED tests/test_preprocessing.py::test_exotic_image[pdf-sandwich-lichtenstein.pdf]
FAILED tests/test_preprocessing.py::test_exotic_image[pdf-hocr-palette.pdf]
FAILED tests/test_preprocessing.py::test_exotic_image[pdf-hocr-cmyk.pdf] - Fi...
FAILED tests/test_preprocessing.py::test_exotic_image[pdf-hocr-ccitt.pdf] - F...
FAILED tests/test_preprocessing.py::test_exotic_image[pdf-hocr-jbig2.pdf] - F...
FAILED tests/test_preprocessing.py::test_exotic_image[pdf-hocr-lichtenstein.pdf]
FAILED tests/test_preprocessing.py::test_exotic_image[pdfa-sandwich-palette.pdf]
FAILED tests/test_preprocessing.py::test_exotic_image[pdfa-sandwich-cmyk.pdf]
FAILED tests/test_preprocessing.py::test_exotic_image[pdfa-sandwich-ccitt.pdf]
FAILED tests/test_preprocessing.py::test_exotic_image[pdfa-sandwich-jbig2.pdf]
FAILED tests/test_preprocessing.py::test_exotic_image[pdfa-sandwich-lichtenstein.pdf]
FAILED tests/test_preprocessing.py::test_exotic_image[pdfa-hocr-palette.pdf]
FAILED tests/test_preprocessing.py::test_exotic_image[pdfa-hocr-cmyk.pdf] - F...
FAILED tests/test_preprocessing.py::test_exotic_image[pdfa-hocr-ccitt.pdf] - ...
FAILED tests/test_preprocessing.py::test_exotic_image[pdfa-hocr-jbig2.pdf] - ...
FAILED tests/test_preprocessing.py::test_exotic_image[pdfa-hocr-lichtenstein.pdf]
FAILED tests/test_preprocessing.py::test_non_square_resolution[hocr] - FileNo...
FAILED tests/test_preprocessing.py::test_non_square_resolution[sandwich] - Fi...
FAILED tests/test_preprocessing.py::test_convert_to_square_resolution[hocr]
FAILED tests/test_preprocessing.py::test_convert_to_square_resolution[sandwich]
FAILED tests/test_rotation.py::test_autorotate_threshold[1-correlation > 0.80]
FAILED tests/test_rotation.py::test_autorotate_threshold[99-correlation < 0.10]
FAILED tests/test_stdio.py::test_stdin - AssertionError: assert 1 == <ExitCod...
FAILED tests/test_stdio.py::test_stdout - AssertionError: assert 1 == <ExitCo...
FAILED tests/test_stdio.py::test_dev_null - AssertionError: could not send ou...
FAILED tests/test_unpaper.py::test_clean - FileNotFoundError: [Errno 2] No su...
FAILED tests/test_unpaper.py::test_unpaper_args_valid - FileNotFoundError: [E...
FAILED tests/test_unpaper.py::test_unpaper_args_invalid_filename - assert 'No...
FAILED tests/test_unpaper.py::test_unpaper_args_invalid - AssertionError: ass...
FAILED tests/test_userunit.py::test_userunit_qpdf_passes - FileNotFoundError:...
FAILED tests/test_userunit.py::test_rotate_interaction - FileNotFoundError: [...

Humm… I'm a bit worried about the tests being disabled. We won't spot breakages (in fact, this already regressed in 20.03), and should better have tests enabled by default.
Chromium takes hours to build, so enabling tests here (once we get them to work) seems to be a no-brainer. Also note it only took my machine ~2mins to fail :-)
The failures all seem to be due to the check_ocrmypdf pytest helper not finding tesseract. Error output on 20.03 looks similar.
As it just invokes api.run_pipeline(options, api=True), this might be a real breakage - maybe we also just need to provide tesserract in $PATH before executing the tests, not sure.

Result of nixpkgs-review pr 89710 1

1 package built:
- ocrmypdf

The tests are already disabled and the update is working for me.
Leave enabling the tests to a future pr.


Accuracy mismatch between model.evaluate() and TensorFlow

I am a bit confused regarding the use of model.evaluate() which produces a different accuracy compared to what I get with TensorFlow :
#data_test is (?,28,28,1) and labels_test are categorical
scores = model.evaluate(data_test, labels_test, verbose=0)
print("Accuracy1 :  " + str(scores[1]))
#########################
from keras import backend as K
from sklearn.metrics import *
ground_truth = np.argmax(labels_test,axis=1)
K.set_learning_phase(0)
sess = K.get_session()
softmax_tensor = model.output
prediction = np.zeros(data_test.shape[0])
for index in range(data_test.shape[0]):
	image_data = data_test[index]
	image_data = image_data[None, ...]
	pred = sess.run(softmax_tensor, feed_dict={'data:0': image_data, 'keras_learning_phase:0':0})[0]
	prediction[index] = np.argmax(pred)
print("Accuracy2 : "+ str(accuracy_score(ground_truth, prediction)))

Results :
Accuracy1 : 0.9964
Accuracy2 : 0.9825

Update:
mode.evaluate() calculates the accuracy on categorical labels.


prep for mvp

Added raw server files and manual import files

I have comments...However, we can address them at a later time, post-MVP.
Can we renamed serverts to server?
Rememer to delete server.tsx.


Voice OTP Authenticator implementation

Purpose
This PR includes Voice OTP Authenticator implementation.
Discussion
The pre discussion that we had about this authenticator can be found from [1]
Design document
Design document can be found here[2]
[1] Mail - "[Architecture] Voice OTP Connector"
[2] https://docs.google.com/document/d/1H_WrcFetIbULV2bZ5RDxR6jrJV0Um-e2MXGxbjYTGsw/edit?usp=sharing

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

Shall we run the check style plugin against the files ?
It would be much easier to correct the formatting.
[1] https://medium.com/@wijithbandara/how-to-configure-wso2-code-style-template-5d4e9f50e3a4


Wavedrom in mediawiki

Does somebody know how to integrate wavedrom in mediawiki ?

@Martoni It would be cool :+1:
It should not be too difficult to do. Do you have good mediawiki plugin template? example?
Does anybody have access to mediawiki installation to try?

@drom Yes I have an access to a mediawiki installation. I've got a private mediawiki. But I don't know how to make a plugin template.
I'll keep trying to do it, if I find how to I will explain it here.

@drom I wrote a quick&dirty plugin. It works for me, but not perfect.

Great job. That was quick!

and dirty ;)

It is fine. Keep working and you will make it clean :smirk:


Use correct stdin utils

Use the correct utils for reading from stdin. This resolves #14.

Coverage increased (+1.4%) to 84.783% when pulling 77c6079e4df0385454d65205e289a4f9e57cebae on raugturi:master into 3af44adf226e52677f72466e92e5490c36e1243b on tk0miya:master.

Thanks!


This Is Football/V-Rally: Championship Edition 2 - Libcrypt Protection Effects

Hello there
I am not sure where to post this as this is my first time contributing to adding in information to a project here, but I would like to let you know that This Is Football's LibCrypt protection kicks in whenever you try to load in a game with two teams selected, whereby the loading bar gets stuck at approximately 80-85%.
Also, V-Rally: Championship Edition 2 has the same protection with similar effects where if you try to choose your car and game mode, the game will try to load, and then permanently gets stuck on the loading screen with the ECG patterns and heartbeats played consistently.
To bypass this under normal circumstances, either create a CCD/IMG/SUB dump with the subchannel data intact and untampered so that the game can load correctly, or use a PPF patch to patch out the LibCrypt protection, but as you already have created such tools, this has only been posted to inform you as to how the protection gets triggered and how it exhibits itself to the end-user.

Hello there
I am not sure where to post this as this is my first time contributing to adding in information to a project here, but I would like to let you know that This Is Football's LibCrypt protection kicks in whenever you try to load in a game with two teams selected, whereby the loading bar gets stuck at approximately 80-85%.
Also, V-Rally: Championship Edition 2 has the same protection with similar effects where if you try to choose your car and game mode, the game will try to load, and then permanently gets stuck on the loading screen with the ECG patterns and heartbeats played consistently.
To bypass this under normal circumstances, either create a CCD/IMG/SUB dump with the subchannel data intact and untampered so that the game can load correctly, or use a PPF patch to patch out the LibCrypt protection, but as you already have created such tools, this has only been posted to inform you as to how the protection gets triggered and how it exhibits itself to the end-user.

As specified in https://github.com/alex-free/tonyhax/blob/master/anti-piracy-bypass.md both these games suggest using the LibCrypt Partcher I have developed. Both games are explicitly supported by the patcher :) let me know what you think...
I'd like to get tonyhax international bypassing libcrypt, but I am unsure how and when I will implement that functionallity.

thankyou, added this info to https://github.com/alex-free/tonyhax/blob/master/anti-piracy-bypass.md . If you know of more libcrypt effects I can add to further complete the document, open a new issue.


Incorrect Item and Sequence Number Display in Count Details After Switching Tabs

Current behavior
An incorrect item is displayed in the Count Details panel.
The sequence number does not match the item shown in the Pending tab.
Expected behavior
When switching to the Pending tab, the first item in the tab should be displayed in the Count Details panel.
The sequence number should correctly match the first item of the Pending tab.
Steps to reproduce the issue
Navigate to the Assigned section and select the Counted tab.
Select an item from the Counted tab and note its sequence number in the Count Details panel.
Switch to the All tab.
Navigate to the Pending tab.
Observe the item and sequence number displayed in the Count Details panel.
Environment details
App version : v2.3.1
App environment : UAT
Additional information
https://jam.dev/c/a9b02b99-87f3-4c9a-bbcb-9e6413acc209

@Adameddy26 This is also fixed in the release v2.3.4 please verify.


Define release policy

Explain the current release policy for Android and iOS in writing.

Semantic versioning.
Pre-releases
Releases are made when the community asks for it
No LTS releases are made but link to a discussion about this topic

The release policy can be found here: https://github.com/maplibre/maplibre-native/wiki/Release-Policy-Android-and-iOS


Prod - Create dump process to save database.

Context
As application owner we want to regulary save the database in case of any potential problem to not lose data.
Expected behavior
a database dump must be created and compressed in a .gz files. This file will be saved on the localhost

Tickets done, dump process and restore prepared, aviable on yaki drive until used for the production server


Add shiny deploy to CLI

I made a fun little app that I want to share. I want shiny deploy my-app to just work. Something along the lines of quarto publish. I'd like to have the complexity of configuring rsconnect-python, connecting to shinyapps.io or Posit Connect, or even bundling and pushing to shinylive to be handled for me as much as possible.

I liked that idea, I'm a bit worried about recommending multiple deployment workflows. We do need to keep the rsconnect one because that's how people deploy other python applications to Posit Connect and Cloud. Can you say more about the complexity you found with rsconnect, maybe we can improve that package and have shiny deploy just be a pass through.

Right, I think shiny deploy would wrap rsconnect-python to deploy to the targets that use it.

Can you say more about the complexity you found with rsconnect

I started reading the docs and gave up without trying. Not to say that I won't give it a try later--I'm just wearing my impatient new user hat on, and when I run into friction, I take it as a sign that we need to reduce friction.

@wch that's nice, but that only works if I'm already at shinylive.io developing. Suppose shiny deploy shinylive would generate that link? (is it just b64encoding the app file(s)?)

@nealrichardson I can imagine a workflow like this:

Edit app in VS Code
Click on a Play button to try to run the app on shinylive.io. It could also support autoreload on save.
Click on a Share button on the shinylive.io page to get the sharing URL.

Or, without VS Code:

Edit app in some editor
Run shinylive watch . to open the app on shinylive.io and autoreload on save.
Click on a Share button on the shinylive.io page to get the sharing URL.

Other commands for shinylive could be:

shinylive url myapp: Get share URL for app
shinylive open myapp:  Open app in shinylive.io editor page. (Maybe have an option for opening without the editor)

I'm writing this as though these would use the shinylive command from the Python shinylive package. I think it conceptually makes sense to have the command be there, but on the other hand, that's one more thing the user would have to install, and if it were instead a shiny command, that would remove one step that the user would need to do.

I think the CLI command is a good idea, but one more thing that might be useful for you now: the shinylive.io editor page has a button to load all files from a directory on disk (works in Chrome only, I think).

I'm writing this as though these would use the shinylive command from the Python shinylive package. I think it conceptually makes sense to have the command be there, but on the other hand, that's one more thing the user would have to install

We could put it in shinylive, have shiny depend on shinylive (or does that create a circular dependency problem?), and have shiny deploy shinylive call out to shinylive to render?

Two recent updates:

shinylive now supports "deploying" an app to shinylive.io by creating a shinylive.io URL from local files. The command is shinylive url encode.
The Shiny VS Code extension also supports deploying to shinylive with two commands

For single-file apps, e.g. app.py or app.R, run the Create ShinyLive Link from Active File from the command palette with the app file open and active.
For multi-file apps, select all of the files or directories in the Explorer pane that you want to include in your Shinylive app. Then right click on the selection and choose Create ShinyLive Link from Selected Files.


Refactor thread

Add configmap and clusterpolicy parser and encoder
Inject http client
Controller should create clients in factory

Config interface is excessive, make interface over authentication schemes (keyless,default)

Alot of duplicate code can be made common using functions

remove update code


Leave app/js as a potential source for bower

@LFDM can you comment on the implications of this (if any) for the main arethusa build?

No implications, it just makes the bower package larger, which is not really a big deal. Initially the idea was to ship only the necessary parts through bower - the missing piece is was @PonteIneptique uncovered in #628 (who would have though that someone already wants the bower package ;))
The main Arethusa deployment builds don't run through bower, so it's an unrelated change in this regard.

Ah, sorry, started to merge other things - can you rebase this tomorrow morning @PonteIneptique? Will merge then.

Coverage remained the same at 42.04% when pulling 83d03399cced0db3ca1e0d3ccd3932e6f590bdac on PonteIneptique:patch-1 into 83505855aa5a5115f9f50bdcc235d3cdd2c82034 on latin-language-toolkit:master.


Tag a release?

It would be useful for people depending on test_frameworks to have a release tagged.  This makes dependency management a bit easier for consumers, and works around some issues with exact SHA dependencies in dependency management tools (e.g. dep tends to be more picky then usual with exact SHAs).  Even just a v0.1.0 release would be useful

Hey @DirectXMan12.
We can see to tag some releases if that helps with dep or such. Is it enough when we tag the current HEAD and then continue tagging according to semver or would you also need tags for some earlier versions?

Tagging the current HEAD and then continuing with semver should work for us.  Thanks!

@DirectXMan12 Sorry for the long delay. I just tagged the current state with v0.1.0. I will try to make sure to follow semver for upcoming tags.


[1.36][studio] Copy button not considering special space character NBSP

Describe the bug
I'm having an issue with maestro studio's copy command feature.
We use some language currency formatters to format a double value, for instance, into a currency string like R$ 10.000,00. The thing with those formatters is that they don't use a normal space character between the R$ and the 10.000,00. They use this following character:   which is usually rendered as a space, but when maestro compares it with a normal space it doesn't match the string.
In my IDE this characters renders as follows:

Also, if I just tap the  icon and CMD+A and CMD+C, I can copy the NBSP character as expected, so it's probably a parsing issue or something on the "copy to clipboard" feature .
Environment information (please complete the following information):

Maestro version [1.36]
Platform: [Both]
Framework: [Flutter]
Device model and OS version: [iPhone 14 Pro - iOS 17.2]
Simulator
Host [Mac M1]

Hey, thanks for creating this issue! Unfortunately, it's not reproducible to us.
Please provide a minimal, reproducible example that clearly explains how to reproduce the problem with the detailed, step-by-step instructions (that can be easily run by me or other maintainers).
This issue does not contain such an example, therefore I'm going to reluctantly close it.
If you don't have time right now, that's totally OK. Feel free to open a new issue in the future with all the required information provided.


Blazor - make parameters forced public

I would like to suggest component parameters with getter and setter qualifications, as per https://docs.microsoft.com/en-us/aspnet/core/blazor/components?view=aspnetcore-3.0#lifecycle-methods  should be public, as opposed to:
@code {
    [Parameter]
    private bool IsCompleted { get; set; }
}

Or ditch the getter and setter altogether. I fully realize it's a matter of my personal preference and this likely won't go further, yet would like to state a case for the 'public' encapsulation referring to a front-facing element, such as these kind of Blazor parameters. The above feels and looks odd, IMPO.

This is going to be covered by #8825

Thank you @stsrki , I did look for a similar topic but not good enough.


Persistent storage and manifest.json

A couple issues..
i) when I close down firefox (v76) the stored blacklist gets wiped.  My settings allow cookies and do not wipe them upon closing.
ii) the manifest.json in your addon won't work with any of the recent versions.  It's missing:
    "applications": {
        "gecko": {
            "id"<EMAIL_ADDRESS>        }
    },

iii) lastly, the toolbar icon doesn't appear to do anything.  Should it open the blacklist panel?
Nice little tool for filtering out results.  Thanks!

On point iii)- No, currently the blacklist panel list is only accessible via View>Sidebar>Wiper
I'll try to reproduce (i),
and thanks for the tip in (ii)- I'll fix it and update here.

Oops I think you have opened the issue in the wrong repo!
Here's the correct repo: https://github.com/davidahmed/wiper/issues

@canDry Please do open the issue in the correct repo. It's really helpful I think :) If not, then I'll recreate it myself at a later point in time.


Null pointer exception in Input Ticket Audit

Hi and thanks for writing this plugin.
I'm having this error:
2017/12/06 14:16:08 - Zendesk Input Ticket Audits.0 - ERROR (version <IP_ADDRESS>-73, build 1 from 2017-11-10 10.59.25 by marc) : Unexpected error
2017/12/06 14:16:08 - Zendesk Input Ticket Audits.0 - ERROR (version <IP_ADDRESS>-73, build 1 from 2017-11-10 10.59.25 by marc) : java.lang.NullPointerException
2017/12/06 14:16:08 - Zendesk Input Ticket Audits.0 - 	at org.pentaho.di.trans.step.BaseStep.handlePutError(BaseStep.java:1669)
2017/12/06 14:16:08 - Zendesk Input Ticket Audits.0 - 	at org.pentaho.di.trans.step.BaseStep.access$300(BaseStep.java:147)
2017/12/06 14:16:08 - Zendesk Input Ticket Audits.0 - 	at org.pentaho.di.trans.step.BaseStep$DefaultRowHandler.putError(BaseStep.java:4189)
2017/12/06 14:16:08 - Zendesk Input Ticket Audits.0 - 	at org.pentaho.di.trans.step.BaseStep.putError(BaseStep.java:1650)
2017/12/06 14:16:08 - Zendesk Input Ticket Audits.0 - 	at org.pentaho.di.trans.steps.zendesk.ZendeskInputTicketAudit.processRow(ZendeskInputTicketAudit.java:121)
2017/12/06 14:16:08 - Zendesk Input Ticket Audits.0 - 	at org.pentaho.di.trans.step.RunThread.run(RunThread.java:62)
2017/12/06 14:16:08 - Zendesk Input Ticket Audits.0 - 	at java.lang.Thread.run(Thread.java:748)

Looking at the code, if I understand correctly,
     if ( 404 == zre.getStatusCode() ) {
        putError( getInputRowMeta(), row, 1L, zre.toString(),
          getInputRowMeta().getValueMeta( ticketIdFieldIndex ).getName(), zre.getStatusText() );

I get a 404 from the server. But I get a null pointer exception instead of the ticketID in the error message. No idea why... so I'm stuck.
Please tell me what I can provide…
Regards.

Thanks for the stack trace.  It sounds like you may not have an "Error handling" hop in your transformation, which may be causing the NullPointerException when it tries to find that hop to write the error row.
If that doesn't resolve the issue, can you send me an email<EMAIL_ADDRESS>with the attachment, so I review it? Be sure to strip any username/password settings from the file before sending.
Let me know if that helps.

It's exactly the problem… I had no error step. Thanks a lot.

Sorry I'm a beginner, but I have almost 10% of these records in error. I see in the example in the wiki  a «remove invalid ticket ids» step, is this linked ?

The example screenshot is from my own PDI transformation that I run daily.  In my case, there are some tickets that for whatever reason exist only in the Incremental Tickets API, but not in any other API endpoint.  I've never gotten a good answer as to why this happens, unfortunately.
When I come across those specific Ticket IDs that I want to avoid, I just add those Ticket IDs into a Data Grid step, and have a filter to prevent me from asking for Ticket Audit records for those Ticket IDs.
If you're getting 10% of your tickets responding with a 404 error, do you see them in the Zendesk UI when you're logged in as an admin? If you do see them, you might need to raise a ticket with Zendesk Support asking why the API does not return the ticket information.  If you do not see them in the Zendesk UI, you may want to ignore, or actively filter them like in my example.

Ok, that's much clearer, thanks.
Another question (I hope it's the last one), I see in the example that you are using the table output. I don't understand how this doesn't produce duplicates, if a ticket has been updated ? I would, instinctively, have used an insert/update step...

I send everything into a truncated staging table, and then have another SQL query that handles the insert/update into the final table after that transformation.
This helps to avoid any PDI<->Database latency as Insert/Update typically has to do a round-trip for each record.

Ok, we dug a bit into the zendesk UI with a colleague which is more proficient than me. It seems that these are delete tickets. As we only have the latest deleted tickets in the GUI, we can only check for those, but there is a 100% match with recent deleted tickets. So the API returns even ids of deleted tickets, which cannot be retrieved. And they confirm me that they did a lot of tests a while a go, and the big amount of deleted tickets and their range is consistent with that.


Cleric Turn Unholy missing the Luck modifier

DCC Core Rulebook page 31 states, "Luck: A cleric’s Luck modifier applies to all spell checks to turn unholy creatures."

Thanks for the report! I’ll have this fixed in the next release!


[Flink 31170] [docs]The spelling error of the document word causes sql to fail to execute

What is the purpose of the change

Fix spelling errors in the create table statement

Brief change log

The create table statement provided by the document is incorrect, and an error is reported during execution

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (no )
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? ( no)
If yes, how is the feature documented? (not documented)

What's wrong with this PR?
@reswqa  @healchow

Thanks @BoYiZhang, would you mind referring Apache Flink development guidelines to modify the commit message and squash all to only one commit?


remove unneded deps to pub_semver

Last usage was apparently removed by https://github.com/flutter/flutter/commit/24eeddc05359458203de7457718c092be4af7b91.

LGTM


code optimizations

Codecov Report
Attention: Patch coverage is 68.42105% with 24 lines in your changes missing coverage. Please review.

Project coverage is 59.52%. Comparing base (636f0c7) to head (79a774d).

Files with missing lines
Patch %
Lines

...eb/src/components/App/CaseFiles/CaseFileDrawer.tsx
61.90%
8 Missing :warning:

...eb/src/components/Shared/Drawer/DrawerTitleBar.tsx
53.33%
7 Missing :warning:

...ance-web/src/routes/_authenticated/admin/staff.tsx
28.57%
4 Missing and 1 partial :warning:

...e-web/src/routes/_authenticated/admin/agencies.tsx
40.00%
3 Missing :warning:

...c/routes/_authenticated/ce-database/case-files.tsx
50.00%
1 Missing :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##           develop      #41      +/-   ##
===========================================
+ Coverage    58.30%   59.52%   +1.21%
===========================================
  Files           61       63       +2
  Lines          861      877      +16
  Branches       223      226       +3
===========================================
+ Hits           502      522      +20
+ Misses         335      331       -4
  Partials        24       24

Flag
Coverage Δ

compliance-web
59.52% <68.42%> (+1.21%)
:arrow_up:

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

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


UTF-8 error

this corrects the error :
UnicodeDecodeError: 'utf8' codec can't decode byte 0xff in position 0: invalid start byte

Thanks for the contribution.


Convention based route builder

I have another suggestion - which I can help you implement.
The idea would be to have an option to activate a convention based route builder.
This means that you don't have to specify your route if you want it to follow the namespace in which you are working.
From your same Web application:
namespace Customers.List.Recent;
public class Endpoint : EndpointWithoutRequest
{
    public override void Configure()
    {
        Verbs(Http.GET);
        Routes("/customer/list/recent");
    }
}

So basically the Routes can be build automatically using the namespace.
You would still be able to set (additional) routes manually if you choose for this convention based option.
In this option you can add "ignore" words (for example, if your namespace would contain MyCompany.Features.Customers.List.Recent you could add "MyCompany" and "Features" as ignore words).
Furthermore you could decide on formatting: lower-case, pascal-wording, etc.
Inventory.GetProduct -> inventory/get-product for example.
Again, this would be totally optional as all the rest and I don't think it would affect performance when building the routes.
What do you think? Would it be an idea to add (I can make a PR if you want).
If you don't like it, or think the lib we be "too much" just say :-)

i personally do not like convetion based routing. because i like to see the exact full route/url when i look at the endpoint. so i don't have to spend even a minute amount of brain capacity to figure out what the route is for an endpoint.
also KISS is the only design philosophy i try to follow religiously. and i believe this feature would be poking a stick at KISS ;-)
let's see... if there's more interest in this from the community and enough ppl ask for it, then we will reopen this ticket and work on it in the future.
but hey... keep the ideas coming... you have been bloody awesome so far mate!

Okay understood! No problem. I can easily add it for my own project in base classes. So I am good.

Hi @dj-nitehawk - sleeping over it :-) You are right, it's not a good idea to have.
I was experimenting with it yesterday, and found out that a small refactoring in your code could break your api endpoints (because they would change automatically). I will implement for myself a SegmentBuilder utility; just want to standardise the routes (so to easily use the feature name, the action, etc) - but that's something that belongs in the projects itself, not in this library. Thanks for your insight.

yeah i frequently move around and re-organize my vertical slice/features as the app evolves. having to manage/update the JS clients everytime that happens is a pain in the you know what. route urls should be considered a static contract with the outside world.
quick question... how/where did you hear about fastendpoints?

quick question... how/where did you hear about fastendpoints?

I was googling about architectures for .net. Been a long time user of servicestack in the past, current environment is clean architecture/onion and just peeking around for alternatives. Then I found the vertical slice architecture, found https://mehdihadeli.github.io/awesome-software-architecture/architectural-style/vertical-slice-architecture/ this site. One of the articles lead to here 👍

good stuff :-)


Add design for workload cluster upgrade isolation

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve


KNL-1449 new authz groups should be refreshed immediately

Comment added in jira: https://jira.sakaiproject.org/browse/KNL-1449?focusedCommentId=210439&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-210439


addMavenResolverPlugin breaks sbt 0.13.8 on Windows

For test I created a project with only "hello world" hw.scala (from scala-sbt.org website) and added a build.sbt with a single line of
addMavenResolverPlugin

I then ran sbt run and it failed with:
...\proj\scala\test\build.sbt:1: error: not found: value addMavenResolverPlugin
addMavenResolverPlugin
^
[error] Type error in expression
Project loading failed: (r)etry, (q)uit, (l)ast, or (i)gnore?

$ sbt --version
sbt launcher version 0.13.8
$ cat project/build.properties
#Project properties
sbt.version=0.13.8

:+1: this is impacting all ENSIME users on Windows (it's worth pointing out that @fspeech is using Windows here, if it is not obvious from the context of the path errors)

BTW, why on earth are sbt still distributing their own sbt script when everybody ends up using https://github.com/paulp/sbt-extras/blob/master/sbt anyway to work around the plethora of issues that it addresses?

You need to add it to project/maven.sbt. See http://www.scala-sbt.org/0.13/docs/sbt-0.13-Tech-Previews.html#Maven+resolver+plugin

errr... ok. That's a bit magical.

To enable this plugin, add the following to project/maven.sbt (or project/plugin.sbt the file name doesn’t matter):

recommend re-open.

@fspeech the ENSIME project puts the maven plugin in the project/plugins.sbt. I think the effect is the same, but I don't think it's ever going to work in build.sbt

https://github.com/sbt/sbt/issues/1963


add process immediately feature

When this checkbox is enabled on the channel, any messages that come in for that channel will be processed in the enqueue script. This essentially saves all queue ids to a list as they are created and then runs a new script that searches by id and clicks the "retry" button for that queue record.

It's immediately marked as delivered but the subscribed channel script never executes.


Public Hostnames "Path" is appended to Service URL?

Which Cloudflare product does this pertain to?
Zero Trust
Existing documentation URL(s)

https://developers.cloudflare.com/cloudflare-one/connections/connect-apps/routing-to-tunnel/

Section that requires update
N/A
What needs to change?
The Public Hostname Page for a Cloudflare Tunnel has three fields:

(optional) Subdomain
(required) Domain
(optional) Path

As the documentation clearly states, this can be used to expose http resources like so:
subdomain.domain.com >> http://localhost:1234

However, it seems that the beheviour is currently such that instead of:
subdomain.domain.com/path1 >> http://localhost:1234
subdomain.domain.com/path2 >> http://localhost:5678

we get:
subdomain.domain.com/path1 >> http://localhost:1234/path1
subdomain.domain.com/path1 >> http://localhost:1234/path2

How should it change?
If this is expected behaviour, it should be added to the documentation. Otherwise, the setup of the Public Hostname Page in the tunnel dashboard might lead users to believe a routing like this is possible:
subdomain.domain.com/path1 >> http://localhost:1234

Additional information
No response

@obezuk, is this an issue that is relevant to the documentation repo, or should I file this over at the cloudflared repo?

@ranbel / @obezuk, any update on this?

Hi @michaelweinold, apologies for the delay. I have talked to the product team and confirmed that this is not a documentation issue. Please open an issue in the cloudflared repo so that engineering can investigate this behavior.


CFE-1168: Add CMPSMachineNamePrefix feature-gate

Add new CMPSMachineNamePrefix feature-gate to guard the implementation changes done to support customized control plane machine names.

Part of CFE-1168
Implements https://github.com/openshift/enhancements/pull/1714

/assign @JoelSpeed

/test e2e-aws-serial-techpreview

@JoelSpeed this can also be merged, I believe.

/lgtm

/retest-required


documentation, SLIM-534

 You should sign our Contributor License Agreement in order to get your pull request merged.

All is well.


After evaulation, Authentik was chosen for the PoC


Create Cardano.CLI.Helpers.hs

Removing some of the clutter from Cardano.CLI.Run.

Just a few concerns that I wouldn't mind a response to
Unrelated to my request for changes:
I tend to be wary of "util" or "helper" modules since they can end up becoming a dumping ground for things when it becomes a bit more difficult to decide where they belong in the project. However, I do find this to be nicer than it was before so it's fine.
In the future (assuming we'll be expanding upon the work done in this project) we should give some thought to where these things belong long-term.

Agreed but this is temporary. There was too much clutter in Cardano.CLI.Run and as I continue cleaning up cardano-node I'll find a better home for these guys.

bors r+

@Jimbo4350, @intricate, https://github.com/input-output-hk/cardano-node/pull/162#discussion_r320041073 makes me wonder -- what other pieces of https://github.com/input-output-hk/cardano-node/pull/152 were undone.

I bet Tx submission no longer works.

In the future (assuming we'll be expanding upon the work done in this project) we should give some thought to where these things belong long-term.

@intricate, I also had slightly different plans about restructuring this code.
I also find it somewhat sad that Jordan never talked to me about this module reorganisation.
Never even mentioned it once.

@intricate, I also had slightly different plans about restructuring this code.
I also find it somewhat sad that Jordan never talked to me about this module reorganisation.
Never even mentioned it once.

@deepfire I requested your review on this PR 4 days ago.

I requested your review on this PR 4 days ago. Fixes pushed to jordan/rebase-fix.

@Jimbo4350, are you serious?
The four days include today (which I don't even know how you would theorize it qualifies), sunday and saturday. and the PR was open on Friday's evening.
Given how much change it introduces, how about at least making sure I'm aware of it, like you did with many other PRs?


Google map looks grey

My template:
{{ form.posicion_pedido.errors }}
{{ form.posicion_pedido }}
{{ form.media }} is in the form.
Using jQuery v2.1.1 in the base template.

the position field seems to look gray initially but if i move to another position it looks better

When I render this in the admin site works perfectly. I've heard that sometimes this is a google maps issue that could be resolved by resizing the map via JQuery but I can't instance the map with this plugin.
Any idea of what can be wrong in here?

Seems that was a bootstrap issue. sory!


iOS can't operate the app with Voice Over

Running the demo that comes with the module in iOS 7 with Voice Over enabled, elements and controls won't receive focus. Voice Over will only focus to the iOS 7 Status bar and read out the current time, battery level, device name and network state.
However, it works perfectly fine in Android with the TalkBack. Everything focus and read out fine.
Would you have some hint on why it wouldn't work on iOS?

I'm facing the same issue, Is there any fix or work around?

I got a solution,

setting the accessibilityHidden property of left and center window to false (to be set on the open event of drawer, setting it before open event didn't work)

$.drawer.centerWindow.accessibilityHidden = false;
$.drawer.leftWindow.accessibilityHidden = false;
Also setting accessibilityLabel property of drawer instance to some string makes voice over works.
$.drawer.accessibilityLabel = Ti.App.name;
@viezel - from this, do you get any idea to fix this within the module itself. You may also have a look at  this issue https://github.com/mutualmobile/MMDrawerController/pull/276 which may related to this I guess.

I have one more problem now, when I open the left window the voice over doesn't focus the first item there automatically. Any tips?

I have updated the iOS code to add support for tapping on elements in VoiceOver. It doesn't support swiping left and right, but it is a start.


AppleAccessToken can not be serialized

I'm not sure how common it is, but at least the integration we are using expects the AcessToken object to be serializeable. The parent implements \JsonSerializable, but it is serialized with serialize(), not JSON.
FYI, our integration is Drupal: https://www.drupal.org/project/social_auth / https://www.drupal.org/project/social_auth_apple. Combined with storing the access token in the session and custom Guzzle handler stacks that have their own dependencies injected, we end up with a Serialization of 'Closure' is not allowed exception/error
I believe I can work around it in the social_auth_apple module by serializing the AccessToken class and then again back to a plain AcessResult object, but maybe the logic here could be refactored to do that API request already in \League\OAuth2\Client\Provider\Apple::createAccessToken() and make the access result a simple  value object?

Hello,
Isn't this a BC ?
I used it as is and it broke until i downgraded to 0.2.5.


Don't use JBang runner for examples

I don't like that the examples are running through another runner using JBang. java -jar qDup.jar  should be just enough - if you need it e.g. to obtain username, rather than reading it from system properties in the runner you should support ${{SYS.user.name}} as the default.
IIUC JBang does not even run qDup in the current dir but instead fetches qDup 0.6.2 - a version one and half year old!

You do not need to use jbang to run the examples, the first section in examples docs shows exactly that:
java -jar ../../target/qDup-*-uber.jar -b wildfly.yaml
But, Java is an implementation detail of qDup.  We can not assume that users of qDup are java users, or maven users, or have java toolset installed in their machines.
using jbang, users do not need to learn how to download java, install java, build a java proejct etc, they just have a one-line command that installs a JVM for them if it is missing, downloads the qDup maven artefacts and executes the qDup script. This happens without them having to know anything about Java
jbang fetching an older version of qDup is an issue and it should be using the latest release

Anywhere I've seen qDup used it's used directly; we should not recommend a tool (the runner) that we don't use ourselves. That approach leads to different UX and problems as the old version being used.
Having java pre-installed in the system (and available on the classpath) is the same level of requirement as with jbang, and Java is more 'standard' one. I agree that Maven might be too much.

Having java pre-installed in the system (and available on the classpath) is the same level of requirement as with jbang, and Java is more 'standard' one. I agree that Maven might be too much.

I don't agree with this statement. in order to set up an environment a user needs to

download and install a Java runtime and ensure the classpath is correctly setup
download the qDup binary from (know where to look for that binary)
know how to start a Java program with that binary file. there is risk that they have put the jar in the wrong directory

There is nothing stopping you, as a Java developer, invoking the jar directly.  But for users we shouldn't expect them to have any knowledge of the Java runtime, it is a distraction for their learning.
We need to put ourselves in the position of someone new to qDup, who does not have any Java experience, their aim is to run a qDup script.  Just because we always written documentation assuming that we are writing for Java devs, we shouldn't continue to make that assumption, esp if we want non-java users to use qDup (and there are a lot of non-Java users)

I suppose my point is, leaving it as is does not impact your use of the tool.
But taking it out makes life more difficult for a new user

@rvansa In #130 you mention;

And the current jBang command does not work with 0.6.12...

I checked the jbang script with the hello-world example. There is an error in the hello-world.yaml, but once that is fixed, i don't see an error;
$ ./runQDup.java
[jbang] Building jar...
16:37:23.201 [main] INFO  io.hyperfoil.tools.qdup.QDup - Starting with output path = /tmp/20220914_163723
16:37:23.320 [main] INFO  io.hyperfoil.tools.qdup.JsonServer - listening at localhost.localdomain:31337
16:37:23.365 Running qDup version 0.6.12 @ unknown
16:37:23.367 run-1663169843146 starting state:
 GREETING = Hello qDup!
 USER = johara
 HOST = localhost.localdomain

16:37:23.369 [main] INFO  i.h.tools.qdup.cmd.Dispatcher - starting 0 scripts
16:37:23.439 [qdup-command-1] INFO  o.a.s.c.u.s.e.EdDSASecurityProviderRegistrar - getOrCreateProvider(EdDSA) created instance of net.i2p.crypto.eddsa.EdDSASecurityProvider
16:37:23.472 [qdup-command-1] INFO  o.a.s.c.i.DefaultIoServiceFactoryFactory - No detected/configured IoServiceFactoryFactory using Nio2ServiceFactoryFactory
16:37:23.476 [qdup-command-1] INFO  o.a.s.c.c.h.DefaultConfigFileHostEntryResolver - resolveEffectiveResolver(johara@localhost.localdomain:22) loaded 1 entries from /home/johara/.ssh/config
16:37:24.181 [qdup-command-0] INFO  i.h.tools.qdup.cmd.Dispatcher - starting 1 scripts
16:37:24.181 [qdup-command-0] INFO  i.h.tools.qdup.cmd.Dispatcher - starting nanny
16:37:24.183 hello-qdup:10@localhost:script-cmd: hello-qdup
16:37:24.183 hello-qdup:10@localhost:hello-qdup
16:37:24.493 hello-qdup:10@localhost:echo Hello qDup!
Hello qDup!
16:37:24.499 [qdup-command-6] INFO  io.hyperfoil.tools.qdup.Run - run-1663169843146 runPendingDownloads
16:37:24.500 [qdup-command-6] INFO  io.hyperfoil.tools.qdup.Run - run-1663169843146 -> /tmp/20220914_163723.queueCleanupScripts
16:37:24.500 [qdup-command-6] INFO  i.h.tools.qdup.cmd.Dispatcher - starting 0 scripts
16:37:24.501 [qdup-command-7] INFO  io.hyperfoil.tools.qdup.Run - run-1663169843146 runPendingDownloads
16:37:24.501 run-1663169843146 closing state:
 GREETING = Hello qDup!
 USER = johara
 HOST = localhost.localdomain
 localhost.localdomain :
   hello-qdup :
     id=5 :

Finished in 01.141 at /tmp/20220914_163723
16:37:25.509 [shutdown-abort] WARN  io.hyperfoil.tools.qdup.Run - Skipping cleanup - Abort has been defined to not run any cleanup scripts
16:37:25.509 [shutdown-abort] WARN  i.h.tools.qdup.cmd.Dispatcher - ignoring stop call when already stopped

Do you see an error message?

Ouch, mea culpa - I had an older version of qDup checked out. It really works, even with 0.6.15 as the last version.

I was also thinking about the possibility to expose the script as jbang alias, so a user could run
$ curl -Ls https://sh.jbang.dev | bash -s - app setup
$ jbang qDup@run https://raw.githubusercontent.com/Hyperfoil/qDup/master/docs/examples/helloWorld.yaml

That way they would not need anything pre-installed (including java), they would not even have to clone the repo


Bug: Routing failure with accents / non-ascii characters

Routes with accents or arabic characters seems to throw a 404.
If you encode the accent in the filename (ie, routée.tsx => rout%C3%A9e.tsx) the page loads, but the client-side is broken.
Bug reported on discord

I think this is a problem with react router too https://codesandbox.io/s/react-router-forked-pvdsdf?file=/index.js

Has this been filed over in React Router land? This should be fixed over there.

Okay issue is literally this line: https://github.com/pillarjs/send/blob/master/index.js#L514
Dependency chain Remix => express => serve-static => send
Some explanation here and workarounds: https://github.com/pillarjs/send/issues/30#issuecomment-51772908
Does this seem sensible? Happy to rework some of this to mitigate this issue

@jacob-ebey any thoughts on how you want to proceed with this? The issue is two fold I think, we can solve it by enforcing that routes are encoded and the express server would need to be fixed. If accepting decoded routes is desirable then react router needs to be fixed (and possibly other areas).

I think the correct fix definitely lies in fixing the remix-serve package. It didn't occur to me that cloudflare (for example) uses a slightly different setup with assets served by wrangler. This does work when the routes are encoded.
I think I'm going to put a PR together that replaces the usage of express.static as I've outlined comments that indiciate it's not usable for filesystems where encoding is used.
The wrangler pages does not provide any cache headers (which is negligable when running locally), but is this still needed for the remix-serve package? Does that actually get used "in production"?

Fixed


Segment percentage split evaluation is not consistent between Core API and local evaluation

Describe the bug
Identities are evaluated differently for segments containing a percentage split rule when evaluated against the Core API versus local evaluation mode in SDKs.
To Reproduce
Steps to reproduce the behavior:

Run the Core API (e.g. self hosted)
Create a segment with a percentage split rule of 50% and add a segment override to a basic feature in a given environment
Instantiate two instances of a client (e.g. Node.js), one running in local evaluation mode and one in remote
Evaluate the flag for multiple identities in both instances of the client and compare the results
Notice that the state of the flag can differ for certain identities between local and remote evaluation

Expected behavior
The result is the same for both local and remote evaluation for all identities
Screenshots
N/a
How are you running Flagsmith?

[x] Self Hosted with Docker
[x] Self Hosted with Kubernetes
[x] SaaS at flagsmith.com (only for Core projects)
[ ] Some other way

Additional context
This will affect the following clients when running in local evaluation mode:

[ ] PHP (Matt)
[ ] Node.js (Matt)
[ ] Java (Kim)
[ ] Elixir (Matt)
[ ] .NET (Kim)
[ ] Go (Kim)

Note that python, ruby, rust are not affected as they will use the django id if it exists.

To fix this, we have a couple of options:

Change the SDKs listed above to use the django id if present
Change the API to use the composite key and change those that are 'not affected' to always use the composite key as well

Option (1) is by far the simplest option but is slightly less clean than option 2. It also somewhat diverges from the logic that was implemented for MV V2 evaluations (details of which can be found here)

I'm in favour of 2. FWIW it's an API change + 3 SDKs vs 6 SDKs so simplicity if Option (1) isn't that apparent for me?

The ability to major version the SDKs is much easier than the API, however and I would suggest that a change in behaviour like this would require a major version - or an approach similar to the one done for #1882 where users need to opt in to the consistent evaluation on the API.

If the some of the other languages are already doing option 1 I think lets stick with 1? This avoids API versioning and makes the SDKs consistent with each other?

PHP PR: https://github.com/Flagsmith/flagsmith-php-client/pull/48

Node.js PR: https://github.com/Flagsmith/flagsmith-nodejs-client/pull/119

Golang PR: https://github.com/Flagsmith/flagsmith-go-client/pull/88

.NET PR: https://github.com/Flagsmith/flagsmith-dotnet-client/pull/69

Java PR: https://github.com/Flagsmith/flagsmith-java-client/pull/122

I've just realised that using the DjangoId attribute is not a valid solution to this issue since the identity information from the Core API is, in fact, never passed to the SDKs, particularly in local evaluation mode.
More research needed into the scope of the issue and a new solution.

To confirm, we only need to make this change in the Core API. All the clients are now consistent that they use the Django ID if it exists, otherwise, they use the composite key.

Latest update: percentage split is still prioritising the django_id in our Core API, even if using consistent hashing is enabled. Since v2.85.0 we are using the flagsmith engine for evaluating segments and hence will need to make the change in the engine to verify if the environment is configured to use consistent hashing.


Shared Pull To Refresh

Feature request
You may want to implement a pull to refresh that is shared across all tabs. This might just be good to be included in the examples.
Current behavior
Currently, the examples load a pullToRefresh from each tab's scene independently. I.e. if you do not implement a pull to refresh on each tab, then depending on what tab you are on you you'll get different behaviour.
TAB

Scene1: FlatList with images
Scene2: Static user information
Scene3: ScrollView with other information

Proposed solution
A shared pullToRefresh that implements a "refreshing" value that can be used trigger a useEffect in each scene.

@alexpchin we definitely can benefit from a shared pull to refresh example. Would you like to implement it in a new example? Or do you think we should completely replace the current behavior?
I'm also want to add an example for #19

@PedroBern I'll start exploring some options today.

@alexpchin I think the easiest way is to create a context with value something like this:
type RefreshingContext = {
    isRefreshing: boolean
    setRefreshing: (value: boolean) => void
}

Then, consume the context inside each tab. All tabs can start refreshing, triggering the useEffect on the others.
To create the context, there is this helper createCtx from ./src/CollapsibleTabViewContext, it's not exported, so we need to either copy or export:
const [useRefreshingContext, RefreshingContextProvider] = createCtx<
  RefreshingContext
>();

What do you think?


Parallelize Actions where possible

The production build takes 25 minutes.

Adding to 2022-10-15, because I'm considering it part of gh-445.

De-prioritizing to focus on the all-important plan-ahead flow. Developer productivity stories like this can be worked on in support of such user-facing features. That is preferable to dev productivity stories blocking new features.

I have some pretty good parallelization in place, closing.


Introducing ROTP::OTP::URI

Resolves https://github.com/mdp/rotp/issues/98
Working with the v5.1.0 release we had a workaround for spaces encoded as + in issuer values. We couldn't upgrade to the v6.0.0 release because of the double encoding bug. Looking over https://github.com/google/google-authenticator/wiki/Key-Uri-Format I felt it was sufficiently awkward enough to warrant it's own home to contain all the non-standard munging.
I added docker-compose.yml to assist with testing across all versions.
$ docker-compose up

This looks really good, and I agree that URI deserves it's on home at this point. Let me take a look at it tonight when I have a bit more time to go over it. Thanks Alex.

Thanks @atcruice. Sorry it took so long to get this out. It's in 6.1.0 on Rubygems.


Cannot accept new pages.


Color picker fails to update RGB values when selecting different color properties in the Inspector

Operating system or device - Godot version: Windows 10 - Godot Git https://github.com/godotengine/godot/commit/bd33fd0435b2d0dc97a7abb3e68b594185f4e6c3
Issue description:
The color picker will fail to update the displayed RGB values when selecting different color properties in the Inspector. It always happens when switching colors, and can thus be reproduced in a consistent manner. Clicking the color property again after switching will display the color's RGB values correctly.
Steps to reproduce:

Select a node or a resource with at least two color properties, then click on one of the color properties in the inspector. This works fine.

Deselect it, and select another color property in the same node or resource. The correct color is displayed, but notice how the RGB values below it are not the correct ones - they are the values of the previously selected color.

Click on the second color property again, and the RGB values will display correctly.

I have a fix for this coming.

Closing as fixed by #9249.


RPI4 support for i2c3, i2c4, i2c5, i2c6

Raspberry PI 4 supports up to 6 I2C busses. It looks like in both 2.51.1 and 2.53.9 these DTO aren't exposed. Could this be added in near future release so I don't have to resort to i2c-gpio?

Hi. This should be fixed by https://github.com/balena-os/balena-raspberrypi/pull/521

Thanks I see them now in 2.53.12

You are welcome.


Update links

Hello,
There's on
https://github.com/jonblack/arduino-menusystem#documentation
currently 4 links.
They should be updated.
Especially http://www.humblecoder.com/animated-led-matrix-menu-transitions-on-the-arduino/
Kind regards

Fixed in 3.0.0


[QUESTION] How to get req object in the service

I'm trying to unit testing. But I can't pass req object in the test.
// src/services/APIGatewayService.ts
@Service()
export class APIGatewayService implements OnInit, AfterRoutesInit
    @Constant('APIGW.baseUrl')
    private baseUrl: string;

    async getPackageList(req: Req, data: any = {}): Promise<Array<Package>> {
        req.logger.debug({ name: `APIGatewayService getPackageList start`, data });
        const key = 'apigw:packagelist';
...

// test/services/APIGatewayService.spec.ts
import 'mocha';
import { expect } from 'chai';
import { APIGatewayService } from '../../src/services/APIGatewayService';
import TestServer from '../../src/TestServer';

describe('ParseService', () => {
    beforeEach(TestServer.create);
    afterEach(TestServer.reset);
    describe('eval()', () => {
        it('should evaluate expression with a scope and return value', () => {
            const service = TestServer.get<APIGatewayService>(TestServer);
            service.getPackageList(req, {});  //  <---- req is undefined

        });
    });
});

Hello @tsetsee
you have to mock the request object. Create simply a req variable and implement methods/props used in your tested method.
See you

Hello @tsetsee
you have to mock the request object. Create simply a req variable and implement methods/props used in your tested method.
See you


Update Bing Search's Node JS quickstart

Context
This was originally posted by @searls at https://gist.github.com/searls/d65259f79b3f8f48032f3d54c492845b after he explained his frustrations with the existing docs here https://twitter.com/searls/status/1040965099958280192 cc @noopkat, as well :)
I've modified his original snippet for compatibility with the stated Node versions, to add helpful errors, and to try and use Bing's terminology.
Contributing
I know there is a contributor guide, but it's behind an auth wall. I did glance through the last historical versions in the git history, and I'm still not sure I'm doing this right 😝 It seems there's a private copy of this repo that I'm supposed to submit through, presumably at https://github.com/MicrosoftDocs/azure-docs-pr, but it's private, so I can't access it. Apologies to the reviewer if I'm doing this all wrong.
Decisions and their rationale

I split explanations into multiple lines because git's diff algorithm is line based by default, so the default diff output will be better when someone  changes something within the description. Additionally, it works better for  editors like vim, which have poor support for long lines.
I mention that https comes from the stdlib, if someone is new to the  language, they might not know where to get it, this lets them know they already have it and don't need to deal with a package manager.
@searls gets the key from the environment variables. This is slightly more difficult, but too many new people accidentally publish their keys, this  protects them from that security issue. There are other ways to do this, too,  but an environment variable is probably the easiest / most reliable. It references a Twilio page about how to set env vars. I glanced at the page and it looked really good, I did search for a similar article by Microsoft, but didn't find anything equivalent.
I added the link to find the subscription key. I assume this link works for everyone, at the very least, it works for me, using a trial account. Someone should probably verify that: https://azure.microsoft.com/en-us/try/cognitive-services/my-apis/?apiSlug=search-api-v7
If the key is not set, immediately error with a helpful message. This is similar to the original code's check for the key length, and is, IMO, a best practice.
I struggled a bit with how to describe the bingWebSearch function. The existing explanations are pretty detailed about the code, but there's a lot going on in this function. You could reasonably argue this implies we should break it out into helper functions. I started to do this, but found it was a slippery slope, quickly leading to a lot of abstraction that I think hindered the goal of showing how to query with as little barriers as possible.
I used console.dir(...) instead of pretty printing JSON. Initially the purpose was to remove the confusing looking console.log(JSON.stringify(JSON.parse(body), null, ' ')), but it had the side benefits of allowing color and depth cutoffs. I left the color off b/c I don't know that it works on all consoles, I set depth to null because it seemed reasonable to print the full body by default.
In the Put it all together section, since I've changed the code sample, it no longer matches the example in the other repository. Looking through the other quickstarts, the one for php dealt with this by providing the entire file inline, so I used that approach.
I did not update the sample response because my responses seemed to have different keys depending on what I queried for. I assume the sample response is still valid since the versions in the URL still match.
Looking at the diff on Github, I accidentally removed trailing whitespace, which adds some clutter to the diff. My editor does that automatically and my git diff is configured to ignore whitespace changes since they get spammy, so I only just realized it. It wasn't intentional 🤷♂

Based on the CLA, I need to provide evidence that this does not violate @searls IP:

Alright, I've refactored the example to be half as long, be relatively idiomatic, and continue printing the same results (difference being it also works if you copy-paste it) https://gist.github.com/searls/d65259f79b3f8f48032f3d54c492845b#file-2-search-js
I gotta run, but someone please open an issue or PR!
-- @searls

@JoshCheek thanks for reaching out. Your PR is currently under review.

Hey @JoshCheek, just wanted to let you know that we've merged in your changes. They will be live by tomorrow. Thanks again for all your help!
https://github.com/MicrosoftDocs/azure-docs-pr/pull/53602

#please-close


The lines in the .coverage file do not match the executed line numbers in the report.

Describe the bug
The Python environment is 2.7.13. When using coverage 4.4.1, I encountered a discrepancy between the lines data in the generated .coverage file and the executed line numbers shown in the HTML report. For example:
"/usr/zk/sync.py": [     297,     298,     57,     22,     687 ]
However, the file /usr/zk/sync.py has fewer than 100 lines, and line 22 was not executed.

Coverage 4.4.1 is quite old.  Coverage 5.5 still supported Python 2.  Try upgrading to 5.5.  I can't support Python 2 anymore, but that version might work for you.

Coverage 4.4.1 is quite old. Coverage 5.5 still supported Python 2. Try upgrading to 5.5. I can't support Python 2 anymore, but that version might work for you.

I remember that the .coverage file in coverage version 5.5 is a binary file, which means it cannot be directly accessed to extract data. In version 4.4.1, I can directly read the content and convert it to JSON. I need to access the lines data in the .coverage file for further processing.

You should take a look at the coverage json command.  It writes a supported JSON data file.


Remove IBM_Validated_OSS_List_Ubuntu_1804.json

As seen recently, in the live list https://www.ibm.com/community/z/open-source-software/ for IBM validated open source software Ubuntu 18.04 is removed as it is no longer supported.
Therefore, we might need to remove it from our data files too. As of now creating a file for it through ./bin/package_build.py would result in an empty file. We also have to comment it out from our supported distros file in our core repo. @pleia2, @arshPratap Please take a look and let me know if there's anything else that I could add here.

Agreed, we should only be shipping 20.04 and 22.04 sources for this now that the source has dropped 18.04.

Yup. I've made the required changes in my PR too. As of now, only IBM_Validated_OSS_List_Ubuntu_2004.json and IBM_Validated_OSS_List_Ubuntu_2204.json files could be created through it.


Can widechar be supported? Such as chinese and japanese characters.

Please refer to #48
https://www.github.com/kspook/Attention-OCR-1/


ImportError : torch.utils.ffi requires the cffi pacakge

Hi
when trying to run the training script experiments/scripts/train_ycb.sh on ubuntu with python2.7.12,
it fails with the stack trace:
set -e
export PYTHONUNBUFFERED=True
EXPORT CUDA_VISIBLE_DEVICES=0
python2 ./tools/train.py --dataset ycb --dataset_root ./datasets/ycb/YCB_VIdeoDataset
/root/catkin_ws/densefusion/DenseFusion/lib/transformations.py:1912: UserWarning: failed to import module _transformations
warnings.warn('failed to import module %s' % name)
Traceback (most recent call last):
File "./tools/train.py", line 26, in
from lib.loss import Loss
File "/root/catkin_ws/densefusion/DenseFusion/lib/loss.py", line 9, in
from lib.knn.init import KNearestNeighbor
File "/root/catkin_ws/densefusion/DenseFusion/lib/knn/init.py", line 7, in
from lib.knn import knn_pytorch as knn_pytorch
File "/root/catkin_ws/densefusion/DenseFusion/lib/knn/knn_pytorch/init.py", line 2, in
from torch.utils.ffi import _wrap_function
File "/usr/local/lib/python2.7/dist-packages/torch/utils/ffi/init.py", line 14, in
raise ImportError("torch.utils.ffi requires the cffi package")
ImportError: torch.utils.ffi requires the cffi package
I access DenseFusion/lib/knn folder and revised Makefile PYTHON:= python2
I run $make
python2 build_ffi.py
Traceback (most recent call last):
File "build_ffi.py", line 5, in
from torch.utils.ffi import create_extension
File "/usr/local/lib/python2.7/dist-packages/torch/utils/ffi/init.py", line 14, in
raise ImportError("torch.utils.ffi requires the cffi package")
ImportError: torch.utils.ffi requires the cffi package
Makefile:31: recipe for target 'build/knn_pytorch/_knn_pytorch.so' failed
make: *** [build/knn_pytorch/_knn_pytorch.so] Error 1

Please refer to Issue#2.