How to include a message on why the action is not authorized?

I would like to add additional information to the exception thrown to the user on why she/he can not access the resource and I can not find how to do it? Is there a way?
Example:
I have a resource, a user and a subscription. The Subscription has a name
I allow access to a resource only if the user.subscription has a specific name.
can :access, Resource do |resource|
    user.subscription.name == "Subscription 1" || user.subscription.name == "Subscription 3"
end

I would like to have a message that says "You must have subscription 1 or 3". How to add this info to the exception?

@thebravoman The message is set when the authorization takes place, normally in the Controller. You can customize the message in a number of ways. See https://github.com/ryanb/cancan/wiki/exception-handling

Yes, the message can be set in the controller.
But I would like to "access" a resource only of the user has a subscription
X or Y and not subscription Z. This check is made in the Ability class
which understands the logic. But then I should implement a very similar
logic in the controller to find out what are the user subscriptions and if
they need the rules. And this would make the logic redundant and called
twice.
I have the knowledge of what is the reason for access denied directly in
the Ability class the same way I now what is the error on saving an active
record directly in the model.
How would one approach to add a message in the Ability?
On 20 Aug 2016 7:45 p.m., "Alex Sanford"<EMAIL_ADDRESS>wrote:

@thebravoman https://github.com/thebravoman The message is set when the
authorization takes place, normally in the Controller. You can customize
the message in a number of ways. See https://github.com/ryanb/
cancan/wiki/exception-handling
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/CanCanCommunity/cancancan/issues/347#issuecomment-241210387,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABQDBYVtpxxy6QWK4Iq4UGc8hF5Z8ONHks5qhy9EgaJpZM4JoY63
.

This is something we've been trying to figure out as well. At the moment we have to check the combination of what role a user has, the subject and the action to set the exception message. It'd be better if we could set the message in the ability.rb where these rules are set.

Unfortunately is not possible to define messages on ability.rb because rules are composed together when checked and not executed one by one. You have to define the messages on your own somewhere else. Cancancan may not be able to realize which authorization rule is not satisfied. Example:
can :read, Book, published: true
can :read, Book, user: user.id

is converted in something like:
select * from books where published == true and user_id = 1

and
@book.published? || @book.user_id == 1


Laravel 7 support

Any plans to support laravel 7. I am migratign my app from laravel 5.5. Thanks.

i would like an answer to this as well

Hey ✌!
https://github.com/demency/laravel-friendships
some good contributor help me to enable L5.8 to L7 versions.

@demency OK, let me check it out. Thanks.

@demency I have been a repo link and it is not there. I saw it previously  but have till am ready to use it but now it is not there.

Yes @pkitatta
But for some reason i had to delete it. Some weird things happens. Probably in future i'll release a compatible version but with different license and code. Not sure when, because im too busy.

I have forked it and done some small changes and it works on L8, maybe i will maintain it

@DevKhris Thank you sir. In the mean time I was using multicaret/laravel-acquaintances which seems to be identical in everything except the provider class name.


use field.value_to_string(model) for any non protected type

Fix issue #39

Thanks! Could you add a unit test for this, please?

@gasman: Yes.  I made a comment about tests in the issue.  It would have been more appropriate here

@gasman I am having trouble running tests against wagtail to make sure I have a failing test before and successful test after changeset.  I posted on wagtail-dev at https://groups.google.com/forum/#!topic/wagtail-developers/Q-JeY81IOLU

So I've added a test.  I've pushed the test to a separate branch as well without the fix so you can easily compare pass/fail.
The issue I am observing appears to be split between wagtail and modelcluster.  The modelcluster case occurs with a related field pointing to a field that returns a custom object.

So tests are passing on dj18 py3 now.  Having a little bit of an issue with dj17 not liking Subfieldbase on the test models.  Not sure if it has something to do with the way the test project is laid out? I've not had this issue with simple models like this before: "metaclass conflict: the metaclass of a derived class must be a (non-strict) subclass of the metaclasses of all its bases"

Is it acceptable to skip the test on django 1.7?

If my previous comments emailed to you, ignore them.  I attached SubfieldBase to the model instead of the ModelField in a copy/paste error from having them all in the same file.
But the models for the issue test case really needs the from_db_value() methods introduced in Django 1.8 without some ugly hacks.  I think it would be better to keep the test model/field simple and skip the test on Django 1.7.

So I've hit this in several places under wagtail too while trying to use a custom field that returns an object that cannot be natively serialized to json.  Due to the nature of this particular field it is going to be faster for me to change the value's base classes to be compatible with json.dumps.  It would probably be a good idea to add a model like this to your serialization tests since Django added the from_db_value method in 1.8.
These are related:
https://github.com/torchbox/wagtail/pull/1752
https://github.com/torchbox/wagtail/pull/1753

Hi @thenewguy - afraid I'm not really keen on all the Foos and Bars in the test case, as they obscure the details of what's being tested. Would it be possible to update these to a more "real-world" example? It looks like the CashField from the Django tests https://github.com/django/django/blob/master/tests/from_db_value/models.py would be a good candidate. Happy to have a go at this myself, but it might be a week or two before I get round to it...
Having skimmed over the Django docs to refamiliarise myself with from_db_value and the pre-1.8 equivalent SubfieldBase, I'm happy to keep this as a 1.8-specific test. It may be that old-style SubfieldBase custom fields are buggy too (and may or may not be fixed by the fix here), but lack of a fix/test for that shouldn't block us from fixing the behaviour for new-style from_db_value fields.


Changed overflow: visible to overflow: hidden for hr element

Applied the expected overflow value for hr element, accordingly to HTML5 spec

Do you have a link to the spec?
I can't find anything regarding css overflow.

Do you have a link to the spec?
I can't find anything regarding css overflow.

This is the link to the spec: https://html.spec.whatwg.org/multipage/rendering.html#the-hr-element-2

fixes #860


gpsrec plot map does not work with light theme, colors are bangle 1 values.

See attached screenshot.  In the Light Theme the background is white but the color scheme does not adjust to the theme.  The route line disappears, the word 'Back' disappears and long with 68m and 'N'.  All you see is the Green and Red dots and the Red Arrow.


use security builders?

via @wsargent
https://github.com/tersesystems/securitybuilder#passwordbuilder might simplify the code introduced in #257

No longer needed after #260.


data.py doesn't work on MacOS Python 3.6

I got a lot of errors where Python doesn't like doing things interchangably with strings and bytearrays. If fixed them and got running. If you want the fixed file its here: https://drive.google.com/open?id=0B3d1md_6mdBRcVlraVRsWHUyR0U

your modified code worked for me, thanks!

Thanks! Worked for me as well MacOS High Sierra 10.13.3 Beta. Python 3.6.3.


Performance degrade on Windows 10 (app or MS Edge)

Ionic version:  (check one with "x")
[ ] 1.x
[x] 2.x
I'm submitting a ...  (check one with "x")
[x] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:

When building an app for Windows 10, noticeable performance degrade was observed. When running the app built for Windows 10, or using MS Edge browser, the performance when clicking on items, radiobutton much slower than using Chrome or Opera (in my example, I was using Opera). It takes more than one second delay for the click event to be registered to the component
Expected behavior:

There is no delay between click, the performance should be comparable to iOS / Android build or on Chrome browser
Steps to reproduce:

Related code:
the code is cloned from
https://github.com/driftyco/ionic-conference-app
Other information:

Here the link to the similar thread on ionic forum:
https://forum.ionicframework.com/t/ionic2-very-slow-on-windows-phone-10/76051/22
Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
Cordova CLI: 6.3.1
Ionic Framework Version: 2.3.0
Ionic CLI Version: 2.2.2
Ionic App Lib Version: 2.2.1
Ionic App Scripts Version: 1.1.4
ios-deploy version: Not installed
ios-sim version: Not installed
OS: macOS Sierra
Node Version: v6.5.0
Xcode version: Xcode 7.3.1 Build version 7D1014

Hello, thanks for opening an issue with us! I am going to close this issue as a duplicate of https://github.com/driftyco/ionic/issues/10009.

Thought this was different with #10009 because it was reported on Windows 10 Phone. Anyway I have posted a comment there to make sure it covers Windows 10 desktop as well.
Kind regards,


Problems with properties

In Context/Folder view:
-Yes/No properties doesn't render Yes/No button in Table view if is marked in the properties of the actual note.
-Option property renders properties, but when a filter for said property is applied with "is", selector doesn't show the options, so it can't filter anything.

Please confirm that this is still happening after the properties update (0.9.26)

Now it work. Thanks :-)


Two TLDs added via zlint-gtld-update.

Adds "ss" and "xn--mgbah1a3hjkrd".

@cpu any thoughts here? Why it's weird to not have a delegation date, I'm not sure if that we means we should block. You've been acting as the code owner around this recently.

@zakird I'm out of the office until Feb 11th but will give this a :mag: when I return.

While it's weird to not have a delegation date, I'm not sure if that we means we should block

All new country code TLDs  added to the map by zlint-gtld-update will have a delegation date of 1985-01-01 (the .com delegation date) for the reason @ezekiel pointed out that was outlined in the original PR that added the tool: I couldn't find an IANA or ICANN data source that included this information for ccTLDs.
@zakird If you or @cardonator have a suggested data source for this information that can be processed automatically by zlint-gtld-update I'd be happy to write a separate PR to use it. Without another data source I can't think of a better way to resolve this short of removing zlint-gtld-update and maintaining the map file manually. Either way I think its best to handle this discussion separate from this PR - I'm +1 to seeing it merged as-is.


Add Linux Admin Username rules to naming restrictions section

When deploying from AKS quickstart ARM template, parameter validation for Linux Admin Username (as well as the other parameters) redirects to here via https://aka.ms/aks-naming-rules, but there is no information to help fix the problem.  Can we add the following line to this doc:
"Linux Admin Username must only contain letters, numbers, hyphens, and underscores and may not start with a hyphen or number."
(Or a link relevant Azure/Ubuntu documentation?)
Thanks!

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

ID: 55fe52d9-5526-de32-7e16-3e3207fc8aef
Version Independent ID: 28d7cd54-bc74-5a08-684d-5d64ea589750
Content: Troubleshoot common Azure Kubernetes Service problems
Content Source: articles/aks/troubleshooting.md
Service: container-service
GitHub Login: @sauryadas
Microsoft Alias: saudas

@montebhoover Thank you for your feedback! We will review and provide an update as appropriate.

@montebhoover  Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.
@sauryadas Please validate this request and alter the doc as needed.

nobody will give us an hint, isnt it?

@jakaruna-MSFT can you take a look at this?

Thank you for the feedback @montebhoover
I have incorporated your feedback and the changes should be available soon.


If filename in path has spaces - signature becomes invalid

Consider the following example:
var ImgixClient = require('imgix-core-js');

var client = new ImgixClient({
  host: "example.com",
  secureURLToken: "12345abcde",
  secure: true
});

var url = client.buildURL("/a/b/c/image with spaces.jpg", {
  fm: 'jpg',
  dpr: 2,
  w: 1500,
  sharp: 15
});

The above outputs: https://example.com/a/b/c/image with spaces.jpg?fm=jpg&dpr=2&w=1500&sharp=15&ixlib=js-1.0.7&s=5d03453c163fb3c22722a1b7d3fc874a when tool from web app outputs spaces URL encoded.
Request to the resulting URL returns 403 Invalid Signature. I needed to pin imgix-core-js version to 1.0.5 to avoid this.

This looks like regression of https://github.com/imgix/imgix-core-js/issues/7

Augh, this method will be the death of me. I'll take a look at this sometime in the next few days and put together a fix. Thanks for the report, Oleg!

Sorry for the delay here, @oaleynik. There's a PR open for this fix now: https://github.com/imgix/imgix-core-js/pull/29

:tada: This issue has been resolved in version 0.3.0-beta.1 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Make final image smaller

 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.

@dvf how does this make the final image smaller? I doubt it would make any realistic difference

Final images are much smaller with the merging of #597. Additional optimizations are always welcome!


revising license classes

I'm not sure if I've captured everything correctly, but we can discuss any failures of the current models in the morning. Please take a look at LicenseDataServiceTest so we can fill in any blanks to best serve the end integration.

Coverage decreased (-0.8%) to 13.544% when pulling e4786db57410855e3b0154c94b20f1282307d7cf on revise_license_data_service into 3b9eb4e657fefd4e9021569e39ac0b471c1aa594 on component-license-dataservice.


creating a page with the same name as an existing page crashes the application

At edit_functions.create_new_page(). Should probably take you to the existing page instead.

Tested this and found that it’s been fixed with an information flash saying “Article already exists”.


Update site URL does not work

When using https://fabioz.github.com/startexplorer/update/ I get a HTTP 404 error. If this is the current update site URL, please make it work again. If not, please update all references, especially those in the Eclipse market place.

You get 404 when opening with browser, as browser is looking for index.html
Open with Eclipse Help -> Update side

You get a 404 because Github apparently changed some hosting rules :

Subdomains of github.com are deprecated for GitHub Pages. They will not redirect to github.io after April 15, 2021.

There is already an issue (since 2016) for that : #71

https://fabioz.github.com/startexplorer/update/
=> KO in browser and in eclipse Update Sites AND in market place :

http://fabioz.github.io/startexplorer/update/
=> OK in browser and in Eclipse Update Sites :

It's broken from Eclipse Market place too

I created some pull requests to fix this issue in readme and on the web page: #79 #80


Added heartpad

Ready to make a PR? Great! Make sure your directory structure looks like this:
├── CAD
│   ├── case.stl
│   └── plate.dxf
├── firmware
│   └── rp2040.bin
└── PCB
├── macropad.pro
└── macropad.sch
(The file names do not matter, just the folders)

Okay, done!

Looks good to me! Get your hackpad here: https://forms.hackclub.com/hackpad


Can the proof name be changed?

From James Haines email on 7/17/15, issue 9.
Is there a way to make a proof start with something other than “Proof.”? If we do:
\documentclass[final,leqno]{siamart}
\begin{document}
\begin{proof}[of main theorem]
A proof with a new name.
\end{proof}
\end{document}
The proof comes out as “Proof (of main theorem). A proof with a new name. [space plus end-of-proof box]”. We can’t see anything in the instructions about how to get “Proof of main theorem. A proof with a new name. [space plus end-of-proof box]” instead.

From John at Techsetters: Would require setting up a new proof, showing the desired text, something like \newtheorem{proofofmaintheorem}{Proof of main theorem}

Added appendix to describe this.


CI for Node

Contains the compiled JavaScript

Initializes with two players

Starts the game with a random player

Sets the current player to be whoever it is not

Starts the game with a random player


User should get a clear error message when using a non-existent profile

Now ETMoses just tells you that:

Which should read "LES or Local Energy Solution" BTW

@antw Is this the only thing that can go wrong or are there any other situations which might need some custom error message?


1.26.0 release?

The last NPM release was September and there are some new JavaScript improvements (like property handling) that would be great to get without hand-crafting the dependency.
Please could you release v1.26.0 whenever it's convenient? 🙏 @RunDevelopment
Thanks for such an amazing package!

I just finished the changelog for v1.26.0. I'll make a new release after #3292 gets merged.
/cc @mAAdhaTTah

Just published 1.26.0.


sub-label clipped in firefox

it's ok on Chrome:

@jpellizzari what's up with this? Are you working on it?

This got pushed to the back-burner in favor of other things. I still plan on getting to it on one of our "fix niggling UI issues" days.

Hmm. I'd rather not leave issues assigned for long w/o any work taking place, unless the assignee really, really doesn't want anybody else to tackle the issue.

I took a stab at this and could not observe this issue for the label in the screenshot. What I did see was very long labels that broke out of the label container (due to missing word-separators for line breaking or too many lines) and were clipped by Firefox.
A reaffirming overflow seems to stop FF from clipping these
.node-labels-container {
    overflow: visible;
}

but only for the x direction, there is still clipping at the bottom (if it turns into 4 lines).
Whenever text overflows and we manage to make it visible, these things happen intermittently:

The text part that is outside of the container gets stuck (in relation to the window viewport) once we start moving the graph view. There is no way to get it to invalidate it just stays there even when moving other SVG items over it.
it is rendered with a delay; while the in-container label text is rendered instantly, the overflown part shows up half a second later. Specifically visible when a node is selected.

These things might only happen if other nodes/labels are close-by to interfere with rendering or on the right-hand side of the zoom-content SVG group.
In Chrome, this all works as intended.

I cannot see the clipping anymore either. And i don't care about the label container break-out. -> closing.


Addition of CONTRIBUTING.md

I would like to request to be assigned the issue for adding a Contributing.md file to this Project Under Hacktoberfest. This file will provide clear guidelines for potential contributors, enhancing our community's onboarding experience and encouraging more participation.

@AnuranjanJain Great !! make sure to star the repo>create fork


Chrome vs Emails with subject lines of more than 168 Characters

Drag&Dropping emails with a subject line of more than 168 Characters will not work on Chrome browsers.
Tested with:
Chrome version 68.0.3440.106 (Official Build) (64-bit)
Outlook Pro 2010
At two totally different machines/networks, Windows 10 Pro (1803) (64bit, x64) and Windows 7 Home Premium
PS. Not an issue with IE
PS. Fixing those who actually write emails with that long subject lines is outside the scope of this report... 😋

I just fixed an issue where a PathTooLong exception was generated when the temporary filename was longer than MAX_PATH (260 characters).  The filename will now be truncated if it is too long, which is typically caused by e-mails with very long subject lines.
This fix has been released in v1.0.2.

So much appreciating you taking care of this :). It work perfectly. It only truncates the filename, so it can be accepted by html5, but the actual original subject lines remains as is. Thank you 💯


Enable mcad_ray_test

Closes #143
Description

This test was being blocked by this issue https://github.com/project-codeflare/codeflare-sdk/issues/190 which is now closed and merged.

/retest

/retest

This one is hard to catch from logs:
File /opt/app-root/lib64/python3.9/site-packages/ray/_private/runtime_env/validation.py:131, in parse_and_validate_pip.<locals>._handle_local_pip_requirement_file(pip_file)
    129 pip_path = Path(pip_file)
    130 if not pip_path.is_file():
--> 131     raise ValueError(f"{pip_path} is not a valid file")
    132 return pip_path.read_text().strip().split("\n")

ValueError: /opt/app-root/notebooks-ray/requirements.txt is not a valid file

Can you try to revert the notebooks-ray change? That should help.

Can you try to revert the notebooks-ray change? That should help.

Good catch, I reverted now.
I noticed that all tests still make use of this import: "github.com/project-codeflare/codeflare-operator/test/support"
Should I update them to use "github.com/project-codeflare/codeflare-common/support" instead? Based on the recent Support Package Refactor

@ChristianZaccaria well, you can do it as part of this PR or as a separate PR.
There are more changes to be done with that migration - leveraging implemented helper methods instead of current implementation in tests (i.e. creation of ConfigMaps, exposing service using Route/Ingress and such).

/lgtm

@ChristianZaccaria @sutaakar I'd vote to get this in and work on the refactoring in a separate PR.

/approve

Thanks a lot @ChristianZaccaria!


docs: pfe-number update for storybook

Fixes #633
Testing Instructions

Run storybook locally: npm run storybook
Validate that the Storybook demo looks like this

There seem to be some issues with the README along this same vein.  Would this be a good time to update that as well?

@castastrophe Let's just push this out and we can have a follow-up PR for the README issues.


Management Console won't open

Expected Behavior 🌈
The Management Console should open up, when I click the "UHM Management Console.exe.
Current Behavior 💥
I've tried installing on three different machines, but the program won't open. When I press the symbol nothing happens.
Relevant log output 📝
No response
Steps to Reproduce 🔍
Step 1: Install "UMH Management Console_latest_x64_en-US.exe"
Step 2: Click the shortcut "UMH Management Console"
Context (Environment) 🌎
Windows 10 machine.
Possible Solution 🛠
No response
Version 📦
v0.9.x
Anything else? 📌
No response

Thanks for the report.
Can you send us the logs of the management console (%localappdata%\management-console) ?
Either here, or per e-mail at<EMAIL_ADDRESS>

Thanks for the report. Can you send us the logs of the management console (%localappdata%\management-console) ?
Either here, or per e-mail at<EMAIL_ADDRESS>
[00:00:00.061] (1674) ERROR  thread 'main' panicked at 'Failed to spawn sidecar: Io(Os { code: 2, kind: NotFound, message: "The system cannot find the file specified." })': src-tauri\src\main.rs:199
0:
1:
2:
3:
4:
5:
6:
7:
8:
9:
10:
11:
12:
13:
14:
15:
16: BaseThreadInitThunk
17: RtlUserThreadStart

I have found & fixed the bug.
Here is a new version, which includes the bug fix:
https://download.mgmt.docs.umh.app/UMH Management Console_0.3.102_x64_en-US.exe
Please note that this version, contains some features (The Datasources tab), which are not yet released and might contain bugs.

This fix is now also released offically


无法在 Python 3.10.6 中运行

请将下面信息填写完整，便于我们快速解决问题，谢谢！
问题描述
Paddle2ONNX 依赖 ONNX 1.9.0，不支持在 Python 3.10.6 中运行 issue#3723
手动升级 ONNX 到 1.12.0 之后无报错
更多信息 :

用于部署的推理引擎:
为什么需要转换为ONNX格式：
Paddle2ONNX版本:

Ubuntu 22.04 <IP_ADDRESS>-microsoft-standard-WSL2
Python 3.10.6
Paddle（nightly）Version: 0.0.0
Paddle2onnx Version: 0.9.2
PaddleOCR release/2.6

你的联系方式(Email/Wechat/Phone):

报错截图

运行指令/Command Code：

paddle2onnx --model_dir ./inference/ch_PP-OCRv3_det_infer \
--model_filename inference.pdmodel \
--params_filename inference.pdiparams \
--save_file ./inference/det_onnx/model.onnx \
--opset_version 10 \
--input_shape_dict="{'x':[-1,3,-1,-1]}" \
--enable_onnx_checker True

完整报错/Complete Error Message：

/home/username/.local/lib/python3.10/site-packages/onnx/mapping.py:27: DeprecationWarning: `np.object` is a deprecated alias for the builtin `object`. To silence this warning, use `object` by itself. Doing this will not modify any behavior and is safe.
Deprecated in NumPy 1.20; for more details and guidance: https://numpy.org/devdocs/release/1.20.0-notes.html#deprecations
  int(TensorProto.STRING): np.dtype(np.object)
/home/username/.local/lib/python3.10/site-packages/paddle2onnx/constant/dtypes.py:47: DeprecationWarning: `np.bool` is a deprecated alias for the builtin `bool`. To silence this warning, use `bool` by itself. Doing this will not modify any behavior and is safe. If you specifically wanted the numpy scalar type, use `np.bool_` here.
Deprecated in NumPy 1.20; for more details and guidance: https://numpy.org/devdocs/release/1.20.0-notes.html#deprecations
  np.bool: core.VarDesc.VarType.BOOL,
/home/username/.local/lib/python3.10/site-packages/paddle2onnx/constant/dtypes.py:48: DeprecationWarning: `np.float` is a deprecated alias for the builtin `float`. To silence this warning, use `float` by itself. Doing this will not modify any behavior and is safe. If you specifically wanted the numpy scalar type, use `np.float64` here.
Deprecated in NumPy 1.20; for more details and guidance: https://numpy.org/devdocs/release/1.20.0-notes.html#deprecations
  core.VarDesc.VarType.FP32: np.float,
/home/username/.local/lib/python3.10/site-packages/paddle2onnx/constant/dtypes.py:53: DeprecationWarning: `np.bool` is a deprecated alias for the builtin `bool`. To silence this warning, use `bool` by itself. Doing this will not modify any behavior and is safe. If you specifically wanted the numpy scalar type, use `np.bool_` here.
Deprecated in NumPy 1.20; for more details and guidance: https://numpy.org/devdocs/release/1.20.0-notes.html#deprecations
  core.VarDesc.VarType.BOOL: np.bool
2022-10-27 13:46:43 [WARNING]   You are use develop version of paddlepaddle
I1027 13:46:43.140570  5001 interpretercore.cc:235] New Executor is Running.
2022-10-27 13:46:43 [WARNING]   The model is saved by paddlepaddle v2.2.2, but now your paddlepaddle is version of 0.0.0, this difference may cause error, it is recommend you reinstall a same version of paddlepaddle for this model
/usr/lib/python3/dist-packages/apport/report.py:13: DeprecationWarning: the imp module is deprecated in favour of importlib and slated for removal in Python 3.12; see the module's documentation for alternative uses
  import fnmatch, glob, traceback, errno, sys, atexit, locale, imp, stat
Traceback (most recent call last):
  File "/home/username/.local/bin/paddle2onnx", line 8, in <module>
    sys.exit(main())
  File "/home/username/.local/lib/python3.10/site-packages/paddle2onnx/command.py", line 218, in main
    program2onnx(
  File "/home/username/.local/lib/python3.10/site-packages/paddle2onnx/command.py", line 173, in program2onnx
    p2o.program2onnx(
  File "/home/username/.local/lib/python3.10/site-packages/paddle2onnx/convert.py", line 94, in program2onnx
    return export_onnx(
  File "/home/username/.local/lib/python3.10/site-packages/paddle2onnx/convert.py", line 35, in export_onnx
    onnx_graph = ONNXGraph.build(paddle_graph, opset_version,
  File "/home/username/.local/lib/python3.10/site-packages/paddle2onnx/graph/onnx_graph.py", line 328, in build    onnx_graph.build_parameters(paddle_graph.parameters)
  File "/home/username/.local/lib/python3.10/site-packages/paddle2onnx/graph/onnx_graph.py", line 184, in build_parameters
    node = helper.make_node(
  File "/home/username/.local/lib/python3.10/site-packages/onnx/helper.py", line 112, in make_node
    node.attribute.extend(
  File "/home/username/.local/lib/python3.10/site-packages/onnx/helper.py", line 113, in <genexpr>
    make_attribute(key, value)
  File "/home/username/.local/lib/python3.10/site-packages/onnx/helper.py", line 343, in make_attribute
    is_iterable = isinstance(value, collections.Iterable)
AttributeError: module 'collections' has no attribute 'Iterable'

其他信息
出现报错时 onnx 的版本为 Version: 1.9.0
Name: onnx
Version: 1.9.0
Summary: Open Neural Network Exchange
Home-page: https://github.com/onnx/onnx
Author: ONNX
Author-email<EMAIL_ADDRESS>License: Apache License v2.0
Location: /home/username/.local/lib/python3.10/site-packages
Requires: numpy, protobuf, six, typing-extensions
Required-by: paddle2onnx

使用 python -m pip install onnx --upgrade 更新 onnx 版本
Defaulting to user installation because normal site-packages is not writeable
Looking in indexes: https://pypi.tuna.tsinghua.edu.cn/simple
Requirement already satisfied: onnx in ./.local/lib/python3.10/site-packages (1.9.0)
Collecting onnx
  Downloading https://pypi.tuna.tsinghua.edu.cn/packages/53/b7/0a595a49bd5bc9af85498cd336f98cd1eaf4783f6eeed03908b12c5d11a4/onnx-1.12.0-cp310-cp310-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (13.1 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 13.1/13.1 MB 1.3 MB/s eta 0:00:00
Requirement already satisfied: typing-extensions>=<IP_ADDRESS> in ./.local/lib/python3.10/site-packages (from onnx) (4.4.0)
Requirement already satisfied: numpy>=1.16.6 in ./.local/lib/python3.10/site-packages (from onnx) (1.23.4)
Requirement already satisfied: protobuf<=3.20.1,>=3.12.2 in ./.local/lib/python3.10/site-packages (from onnx) (3.20.0)
Installing collected packages: onnx
  Attempting uninstall: onnx
    Found existing installation: onnx 1.9.0
    Uninstalling onnx-1.9.0:
      Successfully uninstalled onnx-1.9.0
ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
paddle2onnx 0.9.2 requires onnx<=1.9.0, but you have onnx 1.12.0 which is incompatible.
Successfully installed onnx-1.12.0

之后再运行相同的命令
/home/username/.local/lib/python3.10/site-packages/paddle2onnx/constant/dtypes.py:47: DeprecationWarning: `np.bool` is a deprecated alias for the builtin `bool`. To silence this warning, use `bool` by itself. Doing this will not modify any behavior and is safe. If you specifically wanted the numpy scalar type, use `np.bool_` here.
Deprecated in NumPy 1.20; for more details and guidance: https://numpy.org/devdocs/release/1.20.0-notes.html#deprecations
  np.bool: core.VarDesc.VarType.BOOL,
/home/username/.local/lib/python3.10/site-packages/paddle2onnx/constant/dtypes.py:48: DeprecationWarning: `np.float` is a deprecated alias for the builtin `float`. To silence this warning, use `float` by itself. Doing this will not modify any behavior and is safe. If you specifically wanted the numpy scalar type, use `np.float64` here.
Deprecated in NumPy 1.20; for more details and guidance: https://numpy.org/devdocs/release/1.20.0-notes.html#deprecations
  core.VarDesc.VarType.FP32: np.float,
/home/username/.local/lib/python3.10/site-packages/paddle2onnx/constant/dtypes.py:53: DeprecationWarning: `np.bool` is a deprecated alias for the builtin `bool`. To silence this warning, use `bool` by itself. Doing this will not modify any behavior and is safe. If you specifically wanted the numpy scalar type, use `np.bool_` here.
Deprecated in NumPy 1.20; for more details and guidance: https://numpy.org/devdocs/release/1.20.0-notes.html#deprecations
  core.VarDesc.VarType.BOOL: np.bool
2022-10-27 13:48:37 [WARNING]   You are use develop version of paddlepaddle
I1027 13:48:37.315676  5086 interpretercore.cc:235] New Executor is Running.
2022-10-27 13:48:37 [WARNING]   The model is saved by paddlepaddle v2.2.2, but now your paddlepaddle is version of 0.0.0, this difference may cause error, it is recommend you reinstall a same version of paddlepaddle for this model
2022-10-27 13:48:37 [INFO]      ONNX model generated is valid.
2022-10-27 13:48:37 [INFO]      ONNX model saved in ./inference/det_onnx/model.onnx

因为 PaddleOCR 本身不支持 Ubuntu 22.04 的 GCC 11.2.0 ，所以 PaddleOCR 使用的是 nightly （看 issue#44571 pr#45351已经在 rc 中修复了。
但是 Paddle2ONNX 依赖的 ONNX 1.9.0 不支持 Python 3.10.6 issue#3723 ，所以在更新 ONNX 版本到 1.12.0 之后可以正常导出。

后来排查了一下，问题确实出现在 Python 3.10.6 上，Python 3.10.6 从 pypi 上下载的包版本为 0.9.6，实际上最新版本应该是 1.0.1
重装 Python 3.9.15 和 Paddle2ONNX 1.0.1 之后问题同样可以解决。

如果使用 Python 3.10.6 进行转换，命令如下
paddle2onnx --model_dir ./inference/ch_PP-OCRv3_rec_infer \
--model_filename inference.pdmodel \
--params_filename inference.pdiparams \
--save_file ./inference/rec_onnx/model.onnx \
--enable_onnx_checker True

报错信息如下
/home/username/.local/lib/python3.10/site-packages/paddle2onnx/constant/dtypes.py:47: DeprecationWarning: `np.bool` is a deprecated alias for the builtin `bool`. To silence this warning, use `bool` by itself. Doing this will not modify any behavior and is safe. If you specifically wanted the numpy scalar type, use `np.bool_` here.
Deprecated in NumPy 1.20; for more details and guidance: https://numpy.org/devdocs/release/1.20.0-notes.html#deprecations
  np.bool: core.VarDesc.VarType.BOOL,
/home/username/.local/lib/python3.10/site-packages/paddle2onnx/constant/dtypes.py:48: DeprecationWarning: `np.float` is a deprecated alias for the builtin `float`. To silence this warning, use `float` by itself. Doing this will not modify any behavior and is safe. If you specifically wanted the numpy scalar type, use `np.float64` here.
Deprecated in NumPy 1.20; for more details and guidance: https://numpy.org/devdocs/release/1.20.0-notes.html#deprecations
  core.VarDesc.VarType.FP32: np.float,
/home/username/.local/lib/python3.10/site-packages/paddle2onnx/constant/dtypes.py:53: DeprecationWarning: `np.bool` is a deprecated alias for the builtin `bool`. To silence this warning, use `bool` by itself. Doing this will not modify any behavior and is safe. If you specifically wanted the numpy scalar type, use `np.bool_` here.
Deprecated in NumPy 1.20; for more details and guidance: https://numpy.org/devdocs/release/1.20.0-notes.html#deprecations
  core.VarDesc.VarType.BOOL: np.bool
2022-10-27 17:31:32 [WARNING]   You are use develop version of paddlepaddle
I1027 17:31:33.009392 14354 interpretercore.cc:235] New Executor is Running.
/usr/lib/python3/dist-packages/apport/report.py:13: DeprecationWarning: the imp module is deprecated in favour of importlib and slated for removal in Python 3.12; see the module's documentation for alternative uses
  import fnmatch, glob, traceback, errno, sys, atexit, locale, imp, stat
Traceback (most recent call last):
  File "/home/username/.local/lib/python3.10/site-packages/paddle2onnx/op_mapper/op_mapper.py", line 119, in mapping
    mapper_func(graph, node, **kw)
  File "/home/username/.local/lib/python3.10/site-packages/paddle2onnx/op_mapper/nn.py", line 548, in opset_7
    if isinstance(normalized_shape, collections.Iterable):
AttributeError: module 'collections' has no attribute 'Iterable'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/username/.local/bin/paddle2onnx", line 8, in <module>
    sys.exit(main())
  File "/home/username/.local/lib/python3.10/site-packages/paddle2onnx/command.py", line 218, in main
    program2onnx(
  File "/home/username/.local/lib/python3.10/site-packages/paddle2onnx/command.py", line 173, in program2onnx
    p2o.program2onnx(
  File "/home/username/.local/lib/python3.10/site-packages/paddle2onnx/convert.py", line 94, in program2onnx
    return export_onnx(
  File "/home/username/.local/lib/python3.10/site-packages/paddle2onnx/convert.py", line 35, in export_onnx
    onnx_graph = ONNXGraph.build(paddle_graph, opset_version,
  File "/home/username/.local/lib/python3.10/site-packages/paddle2onnx/graph/onnx_graph.py", line 331, in build
    onnx_graph.build_op_nodes(paddle_graph.node_map)
  File "/home/username/.local/lib/python3.10/site-packages/paddle2onnx/graph/onnx_graph.py", line 209, in build_op_nodes
    OpMapper.mapping(self, node, self.operator_export_type)
  File "/home/username/.local/lib/python3.10/site-packages/paddle2onnx/op_mapper/op_mapper.py", line 121, in mapping
    raise Exception(
Exception: Error happened when mapping node ['layer_norm_0'] to onnx, which op_type is 'layer_norm' with inputs: {'Bias': ['layer_norm_5.b_0'], 'Scale': ['layer_norm_5.w_0'], 'X': ['transpose_9.tmp_0']} and outputs: {'Mean': ['layer_norm_15.tmp_0'], 'Variance': ['layer_norm_15.tmp_1'], 'Y': ['layer_norm_15.tmp_2']}, specific error: module 'collections' has no attribute 'Iterable'

最新版Paddle2ONNX，暂时还没有支持3.10（由于编译环境还没升级，所以只编译到了3.9）
https://github.com/PaddlePaddle/Paddle2ONNX/blob/develop/docs/zh/compile.md 当前你可以参照这个文档，自行编译一个3.10下的paddle2onnx


Make umap.__version__ return current version

For replication purposes it is useful to print/log the current version of UMAP.
Is there are variable that holds this information?

It should now ... Hopefully I can keep this all synced.

Thank you very much!

Sorry, I think:
__version__ = 0.2.1

should be:
__version__ = "0.2.1"

Good catch. Thanks.


Modify font-family for prism-material-light.css

Modify font-family to make the monospace font take effect.

@gyj1109 Please explain what problem you are trying to solve. The author of the theme intentionally chose these fonts. Why did you change them?

@RunDevelopment Because those monospace font won't take effect on my computer. "Roboto Mono" isn't available on every computer.

Well, Material Design is for Android, and Roboto Mono is the default monospace font on Android. It is reasonable to set it as the monospace font of the material theme, but almost no one has this font on the computer. So it is unreasonable to use it as a web font.

Well, Material Design is for Android, and Roboto Mono is the default monospace font on Android. It is reasonable to set it as the monospace font of the material theme, but almost no one has this font on the computer. So it is unreasonable to use it as a web font.

While this is true, it's also not an issue because it will default to monospace if the font isn't defined on the client.

Because those monospace font won't take effect on my computer.

What do you mean by that? Can you share a screenshot?

because it will default to monospace if the font isn't defined on the client.

After trying, I found that it did fallback to the default monospace font, because it not looks like with any general monospace font, which made my mistake. Sorry about that and I will close this PR.

@gyj1109 You can also use Google Fonts for this if you want to ensure your users have it: https://fonts.google.com/specimen/Roboto+Mono

@gyj1109 You can also use Google Fonts for this if you want to ensure your users have it: https://fonts.google.com/specimen/Roboto+Mono

That's a great idea! I'll try it and thanks a lot.


Remove dependency on lastVsyncInfo(), except for PreemptStrategyNormal internal impl, to improve encapsulation and possibly remove the whole lastVsyncInfo in the future

p.s.

#29


  This must be called on the main thread, but kicks off background initialization of service syncing.  Docs say:

This is an opaque object. If your application often creates and destroys view or service classes provided by the Google Maps SDK for iOS, it may be useful to hold onto this object directly, as otherwise your connection to Google may be restarted on a regular basis. It also may be useful to take this object in advance of the first map creation, to reduce initial map creation performance cost.

Just retaining it is enough to cache for subsequent views, without needing to pass it into the view itself.
See also GM source (internal links)
cs/piper///depot/google3/googlemac/iPhone/Maps/SDK/Maps/GMSServices+Google.h;l=15-20
cs/piper///depot/google3/googlemac/iPhone/Maps/SDK/Maps/GMSServices.mm;l=259-269;bpv=1;bpt=1
Fixes https://github.com/flutter/flutter/issues/109057
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the relevant style guides and ran the auto-formatter. (Unlike the flutter/flutter repo, the flutter/plugins repo does use dart format.)
[x] I signed the CLA.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy, or this PR is exempt from version changes.
[x] I updated CHANGELOG.md to add a description of the change, following repository CHANGELOG style.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

If this is expensive enough to warrant precaching, it's probably expensive enough that we don't want to kick it off before main. Even though it's a background thread, adding resource contention that early will likely cause a startup time hit for any application using the plugin (and I would expect that most clients of the plugin aren't showing a map immediately on launch, making it an entirely avoidable hit).
I would vote for only automatically caching when the plugin gets its first message, and then adding explicit API to allow precaching, so that app developers can decide when to take that cost.

If this is expensive enough to warrant precaching, it's probably expensive enough that we don't want to kick it off before main. Even though it's a background thread, adding resource contention that early will likely cause a startup time hit for any application using the plugin (and I would expect that most clients of the plugin aren't showing a map immediately on launch, making it an entirely avoidable hit).

This is a good point.  I looked at more traces, I was hoping the maps SDK would essentially be doing what you describe and lazily initialize more when the map is actually created, but it does indeed do a bit of work just setting up, which would be a startup cost.

I would vote for only automatically caching when the plugin gets its first message, and then adding explicit API to allow precaching, so that app developers can decide when to take that cost.

Will move to the first time a map is created.  Re: adding an API that's more work than I'm willing to spend on @jonahwilliams's nerd snipe, I'll file a new issue for that. 🙂

Sounds good!


Error in the generated file

Describe the bug
Error in the generated file
To Reproduce
Steps to reproduce the behaviour:
retrofit_generator: '>=3.0.0 <4.0.0'
retrofit: ^3.0.1
Expected behaviour
Should generate .g file normally
Screenshots

Smartphone (please complete the following information):

Device: [e.g. iPhone11]
OS: [e.g. iOS12.1]

Additional context
Add any other context about the problem here.

Did you try generator 4.0.1?

That error is because you are using @required annotation and Retrofit generates required keyword.
@required annotations were used before null safety, just remove the @ and it should work.

Yes. you are right.   Please use required instead of @required


Test failure: ReadAndWriteEchoTwice

This issue was made automatically. If there is a problem contact ryanbrandenburg.
This test failed with the following error:
Assert.Equal() Failure
Expected: 20000
Actual:   2549
   at Microsoft.AspNetCore.Server.IISIntegration.FunctionalTests.SynchronousReadAndWriteTests.ReadAndWriteEchoTwice() in /_/test/Common.FunctionalTests/Inprocess/SynchronousReadAndWriteTests.cs:line 67
--- End of stack trace from previous location where exception was thrown ---
------- Stdout: -------
| [0.001s] TestLifetime Information: Starting test ReadAndWriteEchoTwice at 2018-11-06T10:13:20
| [0.020s] FIXTURE:HttpTestSite Debug: Method: POST, RequestUri: 'http://localhost:49907/ReadAndWriteEchoTwice', Version: 2.0, Content: System.Net.Http.StringContent, Headers:
|                                      {
|                                        Content-Type: text/plain; charset=utf-8
|                                      }
| [0.020s] FIXTURE:HttpTestSite Warning: StatusCode: 500, ReasonPhrase: 'Internal Server Error', Version: 1.1, Content: System.Net.Http.HttpConnection+HttpConnectionResponseContent, Headers:
|                                        {
|                                          Server: Microsoft-IIS/7.5
|                                          Date: Tue, 06 Nov 2018 10:13:20 GMT
|                                          Content-Type: text/html
|                                          Content-Length: 2549
|                                        }
| [0.020s] FIXTURE:HttpTestSite Warning: <!DOCTYPE html>
|
|
|                                                <meta charset="utf-8" />
|                                                <title> HTTP Error 500.0 - ANCM In-Process Handler Load Failure </title>
|
|
|
|
 HTTP Error 500.0 - ANCM In-Process Handler Load Failure

|
 Common causes of this issue:

|

|
 The specified version of Microsoft.NetCore.App or Microsoft.AspNetCore.App was not found.

|
 The in process request handler, Microsoft.AspNetCore.Server.IIS, was not referenced in the application.

|
 ANCM could not find dotnet.

|

|
 Troubleshooting steps:

|

|
 Check the system event log for error messages

|
 Enable
...

Other tests within that build may have failed with a similar message, but they are not listed here. Check the link above for more info.
This test failed on master.
CC Eilon

This comment was made automatically. If there is a problem contact ryanbrandenburg.
There were 4 failures with about the same error on master at 1:31:38 AM:

ReadAndWriteEcho
ReadAndWriteSynchronously
ReadAndWriteEchoTwice
ReadAndWriteEcho


Refactoring ready check state handling and integeration test package

This PR address several issues:

refactered ready check state handling, it creates a transfer data object StateInfo which carries non error state with additional info, so that make manifest error state only based on returned error, which distingushed with other state handling.

it refactered manifest cr test which use remote host that brings unnessary dependencies, replaced it with mocked oci host.

renamed integration test package name by it's controller name to make it more obverse which controller is involved in the tests

/test pull-lifecycle-mgr-tests

/test pull-lifecycle-mgr-tests

/test pull-lifecycle-mgr-tests

/test pull-lifecycle-mgr-tests

/test pull-lifecycle-mgr-lint

/test pull-lifecycle-mgr-tests

/test pull-lifecycle-mgr-lint

/test pull-lifecycle-mgr-lint

/test pull-lifecycle-mgr-lint


Adding user with password attribute throws error

Add a user to AD:
const entry = {
cn: accountID,
sAMAccountName: accountID,
userPrincipalName<EMAIL_ADDRESS>description: description,
objectClass: ['top', 'person', 'organizationalPerson', 'user'],
};
This works, but the user does not have a password. Additionally, the user account is disabled and the User must change password at next logon is set.
I want to enable the user upon creation and set the password and also set the Password never expires in account options.
To achieve this, I added the password attribute and the userAccountControl attribute:
const passwordBuffer = Buffer.from("${password}", 'utf16le');
const encodedPassword = passwordBuffer.toString('base64');
const entry = {
cn: accountID,
sAMAccountName: accountID,
userPrincipalName<EMAIL_ADDRESS>unicodePwd: encodedPassword,
description: description,
objectClass: ['top', 'person', 'organizationalPerson', 'user'],
userAccountControl: 512, // Normal user
};
This throws an error: Unwilling to perform.
There are no details of the error and I tried debugging this but couldnt find many resources either. Is there a way to achieve this?

Please use formatting to make your question legible -- https://docs.github.com/en/get-started/writing-on-github/working-with-advanced-formatting/creating-and-highlighting-code-blocks
Also, please explain how this question related to the ldapjs library.

Please use formatting to make your question legible -- https://docs.github.com/en/get-started/writing-on-github/working-with-advanced-formatting/creating-and-highlighting-code-blocks
Also, please explain how this question related to the ldapjs library.

Updated the issue description

That error maps to the standard LDAP error code 53. See https://github.com/ldapjs/protocol/blob/58ae926cbe694c83b3d000674245b3c0115cfc77/index.js#L75
It is the server that is returning the error code. The library is translating it to a human readable representation. I imagine that if you posted the full stack trace of the error, we would see that being shown.

That error maps to the standard LDAP error code 53. See https://github.com/ldapjs/protocol/blob/58ae926cbe694c83b3d000674245b3c0115cfc77/index.js#L75
It is the server that is returning the error code. The library is translating it to a human readable representation. I imagine that if you posted the full stack trace of the error, we would see that being shown.

Failed to add user:  LDAPError [UnwillingToPerformError]: Unwilling To Perform
at messageCallback (C:\Users\M_Bhansali\Desktop\prep\node_modules\ldapjs\lib\client\client.js:1261:45)
at Parser.onMessage (C:\Users\M_Bhansali\Desktop\prep\node_modules\ldapjs\lib\client\client.js:919:14)
at Parser.emit (node:events:512:28)
at Parser.write (C:\Users\M_Bhansali\Desktop\prep\node_modules\ldapjs\lib\messages\parser.js:135:8)
at Socket.onData (C:\Users\M_Bhansali\Desktop\prep\node_modules\ldapjs\lib\client\client.js:875:22)
at Socket.emit (node:events:512:28)
at addChunk (node:internal/streams/readable:324:12)
at readableAddChunk (node:internal/streams/readable:297:9)
at Readable.push (node:internal/streams/readable:234:10)
at TCP.onStreamRead (node:internal/stream_base_commons:190:23)
at TCP.callbackTrampoline (node:internal/async_hooks:130:17) {
lde_message: 'Unwilling To Perform',
lde_dn: null
}

{"name":"ldap","component":"client","hostname":"Z2-IZZI","pid":14744,"clazz":"Client","ldap_id":"3__ldap://patton:389","level":10,"msg":"sending request {\"messageId\":3,\"protocprotocolOp\":104,\"type\":\"AddRequest\",\"entry\":\"CN=9999-awsd,OU=XYZ,DC=Xyz,DC=abc\",\"attributes\":[{\"type\":\"cn\",\"values\":[\"9999-awsd\"]},{\"type\\"ccou":\"sAMAccountName\",\"values\":[\"9999-awsd\"]},{\"type\":\"userPrincipalName\",\"values\":[\"9999-awsd@xyz.abc\"]},{\"type\":\"unicodePwd\",\"values\":[\"IgBAAHkAACYAIgQABSACUAOQAzAHYAIgA=\"]},{\"type\":\"description\",\"values\":[\"Checking the account options\"]},{\"type\":\"objectClass\",\"values\":[\"top\",\"person\",\"organizationalP",\"coerson\",\"user\"]}],\"controls\":[]}","time":"2023-07-31T15:08:54.122Z","v":0}

You can see that the error originates in the onMessage handler. That shows the error came from the server. You will need to send an LDAP message the server is willing to handle.


Warn when scopes in TOML do not match scopes on Shopify

WHY are these changes introduced?
Fixes #706 https://github.com/Shopify/internal-cli-foundations/issues/706
If the scopes in the TOML do not match the scopes on the platform, the user should be notified when running dev.
WHAT is this pull request doing?
Check for scopes mismatch in dev and warn the user.
Screenshot

Measuring impact
How do we know this change was effective? Please choose one:

[ ] n/a - this doesn't need measurement, e.g. a linting rule or a bug-fix
[ ] Existing analytics will cater for this addition
[ ] PR includes analytics changes to measure impact

Checklist

[ ] I've considered possible cross-platform impacts (Mac, Linux, Windows)
[ ] I've considered possible documentation changes
[ ] I've made sure that any changes to dev or deploy have been reflected in the internal flowchart.

@RyanDJLee how does this mismatch happen? And what happens when these are mismatched, like will the toml overwrite what's in the Dashboard? Or does that only happen when you shopify app config push?

@RyanDJLee how does this mismatch happen?
This mismatch will happen when a user modifies the scopes in the TOML file and decides not to push before running dev.

And what happens when these are mismatched, like will the toml overwrite what's in the Dashboard? Or does that only happen when you shopify app config push?
We simply raise a warning as we do not know whether this was intentional, however it seems more likely that the user forgot to push. Once opted into scopes on the platform, the user should always push prior to running dev IMO. The user will not get the expected behaviour when there is a mismatch and they are opted into scopes on the platform, so they should at least be aware of this. However, there is no guarantee that the scopes in the TOML are 'correct' / the ones they actually want to push (i.e. accidentally modified or experimenting) and I think we mention in the tech design that we do not want the dev command to be dangerous, so we decided to go with a warning for now.
@rezaansyed  @rachel-carvalho @ryancbahan curious to hear thoughts on this.

that makes sense, I think we just need to make the what push clearer does in that case, and we should use Partner Dashboard instead of Shopify, so:
The scopes in toml file name don’t match the scopes in your Partner Dashboard

Scopes in toml file name:
- read orders
- write products

Scopes in Partner Dashboard:
- write products

Next steps:
- Run ‘shopify app config push’ to push your shopify.app.toml scopes to the Partner Dashboard


The ProgLoad command blows away the TOML options file.

Summary
I suspect that the ProgLoad command is clearing the existing contents of the program directory, including the options TOML file. Need to test this and determine the best work-around for the issue. I was previously unaware of this as I never use the ProgLoad command for loading programs and had forgotten that the command clears the program directory.
Requirements

[x] Change the location that the configuration file is saved by default.
[x] Add the ability to specify where the file will be saved and what it will be named.

This will be included in the v1.1.0 release.


Add a wrapper for accessing the URL launcher service.

cc @HansMuller
https://gist.github.com/chinmaygarde/99b3c6bc700a973a51639526d835319f if anyone needs the snippet.

I see, I didn't pull in the changes made recently and was basing this off of an older pattern https://github.com/flutter/flutter/commit/a8f6f44a5539ba4c11a1e76bc932a5031f60a260#diff-7d620ba5e4aa11648c48a973a928da2fR33

Android will parse the string as conforming to RFC 2396 while iOS expects the string to conform to RFC 1738. I believe 2396 updates 1738 and RFC 3986 updates or obsoletes both. So depending on the platform, there will be subtle changes in the conformance requirements of the string. That is why the mojom specifies that the platform will parse this instead of us. I associate URL with 1738 and URI with the others.

The URL standard obsoletes RFC 3986, RFC 2396, and RFC 1738.

Done. Another look?

Done. Rebased.


Question: Release file decompression error with PyLZMA in Python script

I was trying to make my automation script decompressing the waifu2x-DeadSix27-vc15-cv401-win64_v523.7z file with pylzma and py7zlib for automatic setup, but it throws me a ValueError when I was trying to decompress conv.exe. It looks something like this:
>>> z.getmember('conv.exe').read()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "C:\Users\K4YT3X\AppData\Local\Programs\Python\Python37\lib\site-packages\py7zlib.py", line 632, in read
    data = getattr(self, decoder)(coder, data, level, num_coders)
  File "C:\Users\K4YT3X\AppData\Local\Programs\Python\Python37\lib\site-packages\py7zlib.py", line 702, in _read_lzma
    return self._read_from_decompressor(coder, dec, input, level, num_coders, with_cache=True)
  File "C:\Users\K4YT3X\AppData\Local\Programs\Python\Python37\lib\site-packages\py7zlib.py", line 686, in _read_from_decompressor
    data = decompressor.decompress(input, self._start+size)
ValueError: data error during decompression
>>>

I know this is not waifu2x-converter-cpp's problem, but packaging the released files in a 7z seems to be really difficult for scripts like python to decompress. In comparison, the released zip files from waifu2x-caffe works pretty well with Python's zipfile library.
Is there a particular reason why 7z is used? Is it possible to also release in zip format in the future?
Thank you.

Can't you just run p7zip from a subprocess?
7z compresses like 30% more than gzip (a not insignificant amount), hence me using it.

My script is designed for Windwos platform only at the moment, and in any case, that will require the client machine to have 7zip installed, which is yet an extra step.
I'll try libarchive and see if that solves the problem. That library looks nice so far.

zip @ max: ~37mb
7u @ max: ~27mb
I'd also argue that most users have that installed anyway, especially the target audience of this program.

EDIT: Sorry, didn't mean to close

No worries, and in any other ways, I understand this is not really a bug or defect on waifu2x-converter-cpp's side. It's more of a problem that I'm facing by automating the deployment of this repository.

I'd also argue that most users have that installed anyway, especially the target audience of this program.

For those that use this program directly, I agree. However, for secondary developments like my automated script, I see a lot of people not knowing the basics of command line. Also, the sole reason of an automated script is to ease the hassle of the users figuring everything out themselves.
I just looked into libarchive, and it does indeed decompress 7z conveniently enough. However, according to this following issue, it requires manual steps to compile and setup on Windows, so it's therefore even more of a problem than pylzma.
At this point, I would say that 7z really doesn't do well with Python on Windows. Indeed, 7zip provides better compression ratio, which is easier for higher-level users to download due to its smaller size, but as for the high-level applications that want to use this program as a component, 7z might cause integration problems.
As I mentioned before, it's purely up to you on how and in what format to publish the files, but I just want to at least make you aware that some might have problems with 7z. I will suggest zip format to be included in future releases, so people with different needs take what they need.
Also, thank you for writing this program. It has been really useful for both me and downstream users of mine.

I use zip now.

I use zip now.

Much appreciated. I was able to continue with my development, and now this program can be deployed automatically.


Refactoring union find algorithm

Making the code nice to understand.

[ ] Adhere all comments to convention of lowercase.
[ ] Remove all occurrences of comments on the same line as a piece of code.
[ ] Add JSDocs to all newly created functions.


Scrolling instead of Paging

Describe the problem/challenge you have
[A description of the current limitation/problem/challenge that you are experiencing.]
The most frustrating thing about Octant - other than it crashes and uses gobs of CPU, is that I can only see 10 resources at a time. I have to explicitly look at the bottom right of any list to see if there are more pages available.  This user interface idiom is old and pedantic.
If my resources are not in the first 10, I must find the Pagination Dialog at the BOTTOM of the list and either expand the list to have more in it or use the little paging icons. - Which when looking at screen real estate represent just a few pixels in a vast sea.
At the end of the day, this is a bug in the user experience.  I have to do a lot of navigation and clicking to get to subsequent resource page listings.
Describe the solution you'd like
[A clear and concise description of what you want to happen.]

Let me set the page size for the entire application
Let the page size setting I(and other UI choices) persist between application restarts as well as between pages.

OR
Detect a scroll at bottom event for a particular list and retrieve and display the additional requested resource rows.  This eliminates pagination completely - 5 STARS for this!   If you're worried about memory, use the "page size" as the local resource list buffer size, dropping entries off of either end as scroll requests demand either previous or next rows of data.
Anything else you would like to add:
[Miscellaneous information that will assist in solving the issue.]
Environment:

Octant version (use octant version): Version:  0.16.3
Git commit:  656c7404e529262861eacb13e88d33dccd6035bf
Built:  2020-12-10T01:32:53Z

Kubernetes version (use kubectl version): Client Version: version.Info{Major:"1", Minor:"20", GitVersion:"v1.20.3", GitCommit:"01849e73f3c86211f05533c2e807736e776fcf29", GitTreeState:"clean", BuildDate:"2021-02-18T12:10:55Z", GoVersion:"go1.15.8", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"18", GitVersion:"v1.18.9", GitCommit:"94f372e501c973a7fa9eb40ec9ebd2fe7ca69848", GitTreeState:"clean", BuildDate:"2020-09-16T13:47:43Z", GoVersion:"go1.13.15", Compiler:"gc", Platform:"linux/amd64"}

OS (macOS 10.15, Windows 10, Ubuntu 19.10 etc):
macOS Big Sur v11.2.1

@ReggieCarey Thanks for reporting this. We agree. It is a common complaint. We already have the first option merged in and it will be part of the next release (0.18) of Octant. See: #2047. I'm not going to close this issue as a duplicate though because you mention an alternative solution that I think is worth exploring.
I like the idea of automatically scrolling, but I have no idea how that would work in practice with our existing datagrid tables, that's something we'd need investigate, but I'm also wondering is a big least even the best thing here? These are just open questions I have in general:

Is scrolling through an endless list really the desired experience?
Do we know what we are looking for?
Sorting Warning/Error status resources first?
Can we find it easier with label selectors?

I'm also sorry to hear that Octant is crashing all of the time for you, would love to get the logging output captured in an issue the next time that happens.
Finally, I wanted to circle back to the comment about CPU, this has been reported before as well, see #2013 . This seems to be a macOS specific issue and we currently only have one member of the development team using macOS so getting a pprof profile when it happens has been difficult, but we are aware that folks are experiencing this and are actively working to get it solved.

"Is scrolling through and endless list really the desired experience"
The list is not endless - although I'm not sure what a 1000 node cluster looks like viewed through octant.
"Do we know what we are looking for?"
Often the answer is "yes" I'm trying to view a particular resource.  I suppose filters might help but they're limited to labels.
"Sorting Warning/Error status resources first?"
Depends on the use case.  For me Octant is useful in two roles - Kubernetes Admin, and Data Scientist.  I think it's incredibly useful to provide this but its not the best sort for all use cases.
Note : The filtering capability is valuable - it's not at all obvious that it's a tool or a clickable icon on screen. Can I do wildcarding? How about open and closed ranges (for # restarts for example).
"Can we find it easier with label selectors?"
Sure, one can find a label and down select on that.  It's an idiom of K8s resource selection.
Why is filter by label so distinctly different in the UI from filtering on name? How does one type a filter into the pulldown?
"I'm also sorry to hear that Octant is crashing all of the time for you, would love to get the logging output captured in an issue the next time that happens."
The failures seem to be centered around resources that go away (namespaces, etc.) that apparently Octant can't do without having discovered them once. Also excessive CPU is definitely seen if networking goes away temporarily but this is not the only excessive CPU utilization - as others have reported, it just sucks compute resources just sitting idle.
Not sure on getting a pprof report to you.
The latest release (0.17.0) is a huge improvement b.t.w.

Thank you for feedback, @ReggieCarey - lot's of valuable info there!
I feel the #2047 is a good first step to help with issues you experienced.  Improving the filtering would hopefully help as well, I opened #2074 to track that.
As @wwitzel3 noted, experimenting with automatic scroll is certainly worth exploring, especially if we can find a way to improve usability without affecting the performance.

Regarding infinite scroll, we on the vRA team have spent a significant amount of time looking at this issue. The TLDR is that it is very difficult to make infinite scroll accessible. If a user has low or no vision, they depend on a screen reader to tell them where they are in a data grid, how to locate content, and when they are at the end. https://www.levelaccess.com/infinite-scrolling-impact-on-assistive-technologies-series-1/
The Clarity team took a long look at it as well and if you want to read through why the path you've taken for page size and filtering / searching is preferable, read this issue. (https://github.com/vmware/clarity/issues/41#issuecomment-294220411) Red Dolan's comments are particularly insightful. This was back in 2016. Today they fully support paginated data grids and we have switched over from our old infinite scroll in order to have a more fully accessible product.

I'm not wedded to the idea of an infinite scroll. I recognize 508 compliance implications mentioned. One thing that could be done is to reduce the number of utterances required to access page N of a multi page list of resources and to maintain the page placeholders when using the back button or after looking at resource details. It's extremely frustrating to look at a resource on page 7 of a list then have to click through from the beginning of the list just to look at the next resource in a list. There appears to be no local caching of state or data making the tool frustrating possibly even more so for those using screen reader technology. Imaging audio paging to find a resource and one open you realize it's the wrong resource and now you're forced to audio scroll through the list from the beginning.
If you were serious about 508 compliance and supporting vision impaired users, you would design an alternate user experience optimized for voice interaction, not the current visual layout - a very challenging thing to do and possibly beyond the scope of the octant project.
As it stands now, you're serving neither the sighted or sight impaired community when you limit the user experience based on what screen readers can provide. Screen reader tech is intended as a band aide for web apps not designed for the vision impaired. Screen readers are not a solution.
I assume you've got UX testers validating the utility of the app given a screen reader interface.


Incorrect derivation path

Here https://docs.api3.org/airnode/v0.3/concepts/sponsor.html#derivation-path the derivation path is
m / 0 / sponsor && 0x7FFFFFFF / (sponsor >> 31) && 0x7FFFFFFF / (sponsor >> 62) && 0x7FFFFFFF / (sponsor >> 93) && 0x7FFFFFFF / (sponsor >> 124) && 0x7FFFFFFF / (sponsor >> 155) && 0x7FFFFFFF

This belongs to pre-alpha and changed in v0.2+. It should be
m/44'/60'/0'/0/...
  /1st least significant 31-bits of the sponsor address (sponsor && 0x7FFFFFFF)…
  /2nd least significant 31-bits of the sponsor address (sponsor >> 31 && 0x7FFFFFFF)…
  /3rd least significant 31-bits of the sponsor address (sponsor >> 62 && 0x7FFFFFFF)…
  /4th least significant 31-bits of the sponsor address (sponsor >> 93 && 0x7FFFFFFF)…
  /5th least significant 31-bits of the sponsor address (sponsor >> 124 && 0x7FFFFFFF)…
  /6th least significant 31-bits of the sponsor address (sponsor >> 155 && 0x7FFFFFFF)

This needs to fixed for all v0.2+ versions

Thanks I will update all versions requeired.

As a note, this will be changed again in v0.4 according to https://api3dao.atlassian.net/browse/AN-453 (so the base path will be m/44'/60'/0'/1/) but you can ignore that for now and we can do the docs update while handling that issue.

@bbenligiray v0.2+ has been updated. A TODO comment has been placed into the docs for v0.4 to make an update for AN-453 when it hits the Documentation column on the JIRA board. See PR 429:


Fix simultaneous runs with the libvirt hypervisor

Previously, if any two test runs occurred where the systems had identical
hostnames, then one of the runs would fail indicating that there was a
domain conflict.
This patch ensures that a unique name is created for each run when using
the libvirt provider.
Additionally, the string beaker_ is prepended to the path name so that
it is easier to find your systems when large numbers of VMs are running
at the same time.
It also chooses a different base network when using libvirt vs
anything else so that users can use both VirtualBox and libvirt
simultaneously.

This broke BEAKER_provision=no:
$ BEAKER_provision=no BEAKER_destroy=no be rspec spec/acceptance/bootstrap_rpm_spec.rb
/home/ekohl/.gem/ruby/gems/beaker-rspec-6.3.0/lib/beaker-rspec/helpers/serverspec.rb:43: warning: already initialized constant Module::VALID_OPTIONS_KEYS
/home/ekohl/.gem/ruby/gems/specinfra-2.82.23/lib/specinfra/configuration.rb:4: warning: previous definition of VALID_OPTIONS_KEYS was here

Hosts file 'centos7-64{hostname=centos7-64.example.com}' does not exist.
Trying as beaker-hostgenerator input.

/home/ekohl/.gem/ruby/gems/beaker-hostgenerator-1.2.10/lib/beaker-hostgenerator/parser.rb:78: warning: URI.unescape is obsolete
Hypervisor for centos7-64.example.com is vagrant_libvirt
Beaker::Hypervisor, found some vagrant_libvirt boxes to create

An error occurred while loading ./spec/acceptance/bootstrap_rpm_spec.rb.
Failure/Error:
  configure_beaker(modules: :fixtures) do |host|
    if fact_on(host, 'os.family') == 'RedHat'
      unless fact_on(host, 'os.name') == 'Fedora'
        # don't delete downloaded rpm for use with BEAKER_provision=no +
        # BEAKER_destroy=no
        on host, 'sed -i "s/keepcache=.*/keepcache=1/" /etc/yum.conf'
      end
      # refresh check if cache needs refresh on next yum command
      on host, 'yum clean expire-cache'
    end
RuntimeError:
  Beaker is configured with provision = false but no vagrant file was found at /home/ekohl/dev/puppet-foreman_proxy_content/.vagrant/beaker_vagrant_files/beaker_centos7-64{hostname=centos7-64.example.com}/Vagrantfile. You need to enable provision

# /home/ekohl/.gem/ruby/gems/beaker-vagrant-0.6.7/lib/beaker/hypervisor/vagrant.rb:251:in `configure'
# /home/ekohl/.gem/ruby/gems/beaker-4.28.0/lib/beaker/network_manager.rb:98:in `block in configure'
# /home/ekohl/.gem/ruby/gems/beaker-4.28.0/lib/beaker/network_manager.rb:97:in `each_key'
# /home/ekohl/.gem/ruby/gems/beaker-4.28.0/lib/beaker/network_manager.rb:97:in `configure'
# /home/ekohl/.gem/ruby/gems/beaker-rspec-6.3.0/lib/beaker-rspec/beaker_shim.rb:50:in `configure'
# /home/ekohl/.gem/ruby/gems/beaker-rspec-6.3.0/lib/beaker-rspec/spec_helper.rb:66:in `block in <top (required)>'
# /home/ekohl/.gem/ruby/gems/beaker-rspec-6.3.0/lib/beaker-rspec/spec_helper.rb:5:in `<top (required)>'
# /home/ekohl/.gem/ruby/gems/beaker-rspec-6.3.0/lib/beaker-rspec.rb:5:in `require'
# /home/ekohl/.gem/ruby/gems/beaker-rspec-6.3.0/lib/beaker-rspec.rb:5:in `<module:BeakerRSpec>'
# /home/ekohl/.gem/ruby/gems/beaker-rspec-6.3.0/lib/beaker-rspec.rb:1:in `<top (required)>'
# /home/ekohl/.gem/ruby/gems/voxpupuli-acceptance-0.3.0/lib/voxpupuli/acceptance/spec_helper_acceptance.rb:34:in `require'
# /home/ekohl/.gem/ruby/gems/voxpupuli-acceptance-0.3.0/lib/voxpupuli/acceptance/spec_helper_acceptance.rb:34:in `configure_beaker'
# ./spec/spec_helper_acceptance.rb:5:in `<top (required)>'
# ./spec/acceptance/bootstrap_rpm_spec.rb:1:in `require'
# ./spec/acceptance/bootstrap_rpm_spec.rb:1:in `<top (required)>'
# /usr/share/gems/gems/bundler-2.1.4/libexec/bundle:46:in `block in <top (required)>'
# /usr/share/gems/gems/bundler-2.1.4/libexec/bundle:34:in `<top (required)>'
No examples found.

Finished in 0.00006 seconds (files took 0.96779 seconds to load)
0 examples, 0 failures, 1 error occurred outside of examples
$ ls .vagrant/beaker_vagrant_files/
beaker_centos7-64{hostname=centos7-64.example.com}_1c88cd/ centos7-64{hostname=centos7-64.example.com}/
$ ls .vagrant/beaker_vagrant_files/beaker_centos7-64\{hostname\=centos7-64.example.com\}_1c88cd/
Vagrantfile

You can argue changing the path on disk was a backwards incompatible change as well. Certainly not a patch.

I've submitted https://github.com/voxpupuli/beaker-vagrant/pull/46 which I think is the proper way to do it.


use an ofSoundBuffer internally

also:

Added a couple of text overlays on example
Moved method definitions from .h to .cpp
Added public static bool ofxAudioDecoder::load(ofSoundBuffer & buffer, string filePath, size_t framesToRead = 0);
Renamed getChannels() to getNumChannels()

looks good. out of curiosity, why do you write this-> inside the getter methods?

for some methods the reason is obvious, e.g.:
void MyClass::setChannels(int channels) {
    this->channels = channels;
}

So to keep consistency and reduce surprises, I just keep the consistent pattern of using strict scopoing for large scope catchments (e.g. outside the current function) and lazy scoping for small catchments

and thanks for accepting!


ignore area if the page has no text. closes #130

Basically ignore the page with area restrictions creation when there's no text elements on the page
https://github.com/jjelosua/tabula-java/blob/master/src/main/java/technology/tabula/Page.java#L56

Thanks Juan! I know I'm being a PITA, but could re-submit your PR without so many whitespace changes? I'd love to keep the change history as clean as possible so it's easy to review.
Thanks!

this will close #105, #107, #77 as well. Thanks @jjelosua! Also probably https://github.com/tabulapdf/tabula/issues/573

@jazzido Done

You rock, @jjelosua. Gracias!

@jjelosua @jazzido: Please see the error reported by @blomdahldaniel here: https://github.com/tabulapdf/tabula-java/issues/107
I think this PR introduces a new bug. Perhaps the early return (if there are no texts) should be a new Page object with no texts and with the rulings uncropped (it's lines 68-78 that cause the original problem).


update configuration for etcd; leverage variables in other roles to k…

…eep things DRY; clustering maintained

LGTM


Only execute TimeUnitConversion as needed in preset passmanagers

Summary
This commit makes a change to the preset pass managers to only run the
TimeUnitConversion pass in the preset passmanagers in situations where
it is really needed. For circuits with a high number of gates this pass
can become a bottleneck. Instead of always running the pass this commit
changes the logic to only run the time unit conversion pass when a
scheduling_method is set or if there are instructions durations set and
the circuit contains delays. To facilitate the latter case a new
analysis pass, ContainsInstruction, is added which will simply set a
boolean in the property set if the dag contains a specified instruction
name.
Details and comments
Fixes #6726

@taalexander: We could write a test that replaces TimeUnitConversion.run with a mock, and assert that it was called/not called.  Probably the test should first test that the mock's method does get called when it should, to avoid cases where the pass never gets called.  This might be a larger change, though - that's probably something we'd want to do for most of the logic in the preset managers, if we were going to do it anywhere.

Sure, that's a good call. I'll add some tests to this asserting whether the pass should is called (or not). They would be good example tests on how to do this too, especially as the control flow logic in the preset PassManager gets more complex (see #6830).

Is there an update on this?

Sorry, this fell through the cracks I've rebased it and am working on adding tests right now. That being said looking at the code I had forgotten about the implementation here. There is a bunch of overlap with #6832 for the new transpiler pass but #6832 will do the instruction detection much more efficiently.

https://github.com/Qiskit/qiskit-terra/pull/6803/commits/d53f565d0605b40dae7d9cdd21deb1e5df64c9b4 adds the tests


Reopen on desktop wasn't clear to me

Testing #167433
It was probably wishful thinking, but I was misled by the last entry in this list

I thought I would get to open my existing local clone on desktop

Sadly we have no way to acquire a full filepath on disk via the current state of the web filesystem api (that only gives you the name of the folder), so we can't support that transition from web to desktop (unless Remote Tunnels are enabled 🤔). I can rename this entry for now to make it clear that it's not about an existing local repo clone--maybe Reopen Virtual Workspace on Desktop?

marking as verified as the new wording is there and a bit more clear, though it does introduce some terminology I'm not sure we have been using a lot elsewhere in this experience. A possible alternative: Reopen as Remote Repository on/via Desktop


Revert "Explicit pod security"

This reverts commit a76ba2ff6cc663eb1672c7f3203439a8a81e2048.
Pods won't run with these changes on clusters with version 4.10 and below. There is no restricted SCC that allows setting seccompProfile RuntimeDefault. Reverting and investigating options for either conditionally deploying with these settings or creating an SCC that allows pods to run with this securityContext.

/assign @2uasimojo


System.StringException: Starting position out of bounds: 201590400

During testing through the developer console. I got below error.
please help
System.StringException: Starting position out of bounds: 201590400
--

Hello @Akashkumar12 did you resolve it. I am also getting the same issue. Let me know if there's any solution for this. Thank you.

Hi
I am getting a similar error:
Line: 402, Column: 1
System.StringException: Starting position out of bounds:<PHONE_NUMBER>

The Exception is thrown from the file Zippex.cls at line 402.
Based on the values of the variables it looks like the theStart and theEnd is way to high as the ZipFileString is only 1822 characters.
400: Integer theStart = offset+46;
401: Integer theEnd = theStart + HexUtil.hexToIntLE(fileNameLength);
402: fileName = zipFileString.substring(theStart*2, theEnd*2);// (f bytes)       Offset 46

The file I am trying to read is an excel file.
Here is values of the variables used:
theStart = 909596500
theEnd = 909596500
fileNameLength =
zipFileString.length() = 1822

Here is the code I am running using Anonymous Apex in Developer Console.
List<Attachment> attList = [SELECT Id, Name, Body FROM Attachment WHERE ParentId='<Case ID>'];
if(!attList.isEmpty()){
	Zippex sampleZip = new Zippex(attList[0].Body);
}

while reading Pdf file am getting System.StringException: Starting position out of bounds: 646326178.Can you please help me on this.
Thank you.

I am getting the same error:
System.StringException: Starting position out of bounds:<PHONE_NUMBER>

I am getting the same error:
System.StringException: Starting position out of bounds:<PHONE_NUMBER>

I am facing the same issue.. Please let me know if anyone has resolved this issue.
Thank you in advance

I got this error everytime I tried to extract an corrupted ZIP File. Pay Attention which is the body of the BinaryAttachment. It should be a blob. During Testing convert your ZIP File into an HEX Value oder Base64String!
HEX
binaryAttachment.body = EncodingUtil.convertFromHex('YOUR HEX VALUE GOES IN HERE');
BASE64
binaryAttachment.body = EncodingUtil.base64DECODE('YOUR BASE64 ENCODED VALUE GOES IN HERE');


Bug: Automation triggered by bot showing as door doesn’t work

Describe The Bug
A bot showing as door doesn’t trigger an automation whereby the stairwell lights of my building (controlled by a second bot) should turn on. I think if the bot shows as anything but a switch, the automation fails to happen. Can this be fixed?
To Reproduce
N/A
Expected behavior
When the bot (showing as door) that opens the building’s front door turns on, the bot that turns on stairwell lights should turn them on.
Relevant log output
N/A

Config for homebridge-switchbot
N/A

Screenshots
No response
Device & Model
N/A
Node.js Version
16.13.1
NPM Version
8.1.2
Homebridge Version
1.3.9
Homebridge Switchbot Plugin Version
1.7.0
Homebridge Config UI X Plugin Version
No response
Operating System
Raspbian

@FranciRoosters I believe this an iOS issue. Can you try iOS 15.2.1 or 15.3 beta 2?

iOS 15.2.1 didn’t fix it. Like I said, automations triggered by bots showing as switches work fine, but when they show as anything else, they automations fail. Having said this, I’m not sure it’s an iOS bug. 🤷♂

On 12 Jan 2022, at 20:16, Donavan Becker @.***> wrote:

@FranciRoosters I believe this an iOS issue. Can you try iOS 15.2.1 or 15.3 beta 2?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you were mentioned.

iOS 15.3 was released today. Please check to see if this is fixed for you now.

Seems to be working now. Will test more tomorrow and report back. Thanks!

On 26 Jan 2022, at 19:48, Donavan Becker @.***> wrote:

iOS 15.3 was released today. Please check to see if this is fixed for you now.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you were mentioned.

Please close once you you have verified.

I’ve verified it and it doesn’t work. Before I tested it with the wrong automation. If my automation says “when the front door opens, turn on the stairwell lights”, the bot commanding the stairwell lights doesn’t activate. Again, could it be because one bot is set as “door”?

On 26 Jan 2022, at 21:10, Francesco Galli @.***> wrote:

Seems to be working now. Will test more tomorrow and report back. Thanks!

On 26 Jan 2022, at 19:48, Donavan Becker @.***> wrote:

iOS 15.3 was released today. Please check to see if this is fixed for you now.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you were mentioned.

I would recommend install HomeLog and watching the the logs in HomeLogs while activating the Automation and see if it is triggering. I have no control over an automation. There is no characteristic on a door service that would cause an automation to work.

HomeLogs doesn’t see the automation working.

On 26 Jan 2022, at 22:37, Donavan Becker @.***> wrote:

I would recommend install HomeLog and watching the the logs in HomeLogs while activating the Automation and see if it is triggering. I have no control over an automation. There is no characteristic on a door service that would cause an automation to work.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you were mentioned.

Then I would report this to Apple. Like I said, I have no control over automation.


providers/multipart: RESTEASY-754 Support apache-mime4j 0.7.2

Apache mime4j 0.7.2 has been out for a while now. Update the
multipart provider to support it.
The multipart provider had extended some classes from 0.6 that are
nolonger available in 0.7.2. Therefore a new approach was taken of
copying a few bits of code from mime4j and rewrite small parts.
See Mime4JWorkaround.java, and Mime4jWorkaroundBinaryEntityBuilder
for all such code.

Superseded by #1890


upgrade @oasisprotocol/ledger and related

so we can support version 2.x of the Oasis Ledger app

cc #148

note that this puts us over the edge where common.js exists, and the extension won't work in production build

with the upgrade to client 0.1.0-alpha7+, I'm adding a thing to remove the eval ): ): ):

Nice! Still has an issue with building, right?

yup, should still have that issue!

Let's revisit this after #156

hi , does this version support paratime ?

I'm mostly aware that it adds support for on-chain governance voting. I haven't heard about whether it adds paratime support or not.

great, we've merged #156, and I've freshened this up.

rebased again. now builds


testground/app: app consistently produces blocks every minute

Description to be added @Bidon15

Closing as not relevant anymore


ChattoAdditions causes resource name collisions in Xcode 11

Because the ChattoAdditions Podspec file is using the resources attribute instead of the resource_bundles attribute, an "Assets.car" file is generated that conflicts with the one generated by the host target when using static Pod libraries. You need to change over to resource_bundles, which is recommended anyway. From the docs:

For building the Pod as a static library, we strongly recommend library developers to adopt resource bundles as there can be name collisions using the resources attribute.

Is fixed by PR #615


Location aging seems wrong

Some location backends, like the IchnaeaNlpBackend, report a position only on change in detected Cells/WiFis. Last position reported is still good until the next report. At present that means that UnifiedNLP will start aging the report even when the position is still current and accurate.
The aging should only start when a null (unknown position) report is given by the backend at which point the last non-null report returned by the backend should be aged.

Cancel this. . . Looks like it is possible for the backend helper to no scan. So just because the backend hasn't seen a change in cells or wifi APs, it does not mean they are unchanged.


error when I run preprocess_raw_data.py. How can I fix it?

hello,
when I run the command: python preprocess_raw_data.py -n_jobs 20 -data db5 -graph_nodes residues -graph_cutoff 30 -graph_max_neighbor 10 -graph_residue_loc_is_alphaC -pocket_cutoff 8

I got the following error:

Processing split  1
Processing  ./cache/db5_residues_maxneighbor_10_cutoff_30.0_pocketCut_8.0/cv_1\label_val.pkl
Traceback (most recent call last):
File "C:\Users\equidock_public-main\src\preprocess_raw_data.py", line 37, in
Unbound_Bound_Data(args, reload_mode='val', load_from_cache=False, raw_data_path=raw_data_path,
File "C:\Users\equidock_public-main\src\utils\db5_data.py", line 78, in init
with open(os.path.join(split_files_path, reload_mode + '.txt'), 'r') as f:
FileNotFoundError: [Errno 2] No such file or directory: './data/benchmark5.5/cv/cv_0\cv_1\val.txt'
Any suggestions?
Thanks

Ups, thanks for this. The current code only works for a single split. I just pushed this modification. If you want to do cross validation on multiple splits, you would have to modify the code. Also, I notice that you are using a Windows machine, but the current code is using the Linux/Mac file paths with forward slashes. You would need to adjust the code for Windows.

Thanks. That works.
I have another question regarding how to run it:

I have run through your instruction, it seems all about processing data, training etc.
Can I just use the pre-trained model to do docking?
What is the actual command do I need to run with my own pdb?
I may miss something here.
Appreciate your help.
Thanks

You can use our pretrained models. Just check inference_rigid.py and our instructions in the README.

got it. thanks


requirement management unclear and prone to misuse

Package management is unclear for new users, they may accidentally add requirements to .txt instead of .in files.
Suggested solution:

make requirements.txt a 'hidden' file > '.requirements.txt'
and / or improve documentation, for example with pip-tools

fixed. now only one .txt is created from multiple .in files that are separated for logic


Applied fixes from StyleCI

This pull request applies code style fixes from an analysis carried out by StyleCI.

For more information, click here.

Coverage remained the same at 52.67% when pulling 1cea5db89da7cd77fac5460af62b252b2b59dae6 on analysis-ze6RW7 into 49a709cc3365451b6dedc7eaadd770813e0e3e83 on master.


completionAPI only changes for ++

when updating completionApi it does not seem to change when you use the / but works when pressing ++

Facing the same issue.


Validity of JSON files should be tested before meta-data availability

At present, meta-data checks (e.g. RepetitionTime specification) run before JSON validity checks. In case of a typo in the JSON (it happended to me once!) this leads to countless "false-positives". If nothing speaks against it, switching the order of these checks could help people retain more hair ;-)
See #94 for an example of me being misguided.

This does indeed create confusing warnings.
I believe the checks do run in the appropriate order, but an invalid JSON file does not prevent the meta-data presence checks from running.
I can change this so if a JSON file is present and invalid, meta-data checks for info that could be in that file will not be tested.
There are however, dataset configurations that could complicate this strategy. If for example you have a scan that has more than one JSON files that refer to it. If one of those JSON files is invalid it's impossible to know how metadata is split between the JSONs and what checks we should ignore. The simplest solution would be to still ignore the metadata checks in this scenario and they'll pop back up next time the user tests the dataset with the fixed JSON file.
@chrisfilo What do you think about ignoring meta-data presence checks when any possible location for them is an invalid file? Also if multiple JSONs can't refer to one scan then this is moot, but I believe that is allowed.

@constellates this sounds like a great solution. Sorry for a terribly slow responce.

@constellates is this resolved? can we close it?

I add a PR just now.


WIP: Removes routes and redirects everything to fns

This change leaves the 'call' command as a directive to users to
migrate their routes over to functions.

I'm not a fan of deleting tests we can't make pass... seems like the classic haste vs speed thing.
What do we think is required to write a test for migrate that doesn't depend on the old-style init?

@riconnon We just need to rewrite it the way that the init command itself does but pointing to a V1FuncFile. I'm happy to do it before the end of the day, I was just under the impression this shouldn't be blocked any further.

@JadeRedworth I guess we can keep this as an option if someone else is desperate to open an MR for something that depends on this, but I don't think the work to remove routes from the server has begun yet, so probably you have time to get this done.

Can we also get https://github.com/fnproject/cli/issues/411 fixed as part of this PR?

@denismakogon I don't think this PR should be blocked by this issue #411. I will look into this issue now!

@JadeRedworth routes from V1 is the only one way to run async functions at this moment. We have lots of demos that require async functions. But at this moment, fns and triggers are not capable to provide the same features. So, we need to have async calls in the first place before completely switching to V2.

@denismakogon async calls are not going to be supported for the time being on fns and triggers.


NO-JIRA: csv: update spec.replaces to a released version

SSIA

/override ci/prow/e2e-aws-operator


"@azure/msal-angular" has missing dependencies msal

Please follow the issue template below. Failure to do so will result in a delay in answering your question.

Library

[ ]<EMAIL_ADDRESS>or<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[X ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>
Important: Please fill in your exact version number above, e.g<EMAIL_ADDRESS>@azure/msal: "^1.3.1"
@azure/msal-angular: ^1.0.0
Framework
Angular CLI: 9.1.11
Node: 12.16.2
OS: win32 x64
Angular: 9.1.12
Package                           Version
@angular-devkit/architect         0.901.11
@angular-devkit/build-angular     0.901.11
@angular-devkit/build-optimizer   0.901.11
@angular-devkit/build-webpack     0.901.11
@angular-devkit/core              9.1.11
@angular-devkit/schematics        9.1.11
@angular/cdk                      9.2.4
@angular/cli                      9.1.11
@angular/material                 9.2.4
@ngtools/webpack                  9.1.11
@schematics/angular               9.1.11
@schematics/update                0.901.11
rxjs                              6.5.5
typescript                        3.8.3
webpack                           4.42.0
Description
After importing MsalModule and set the default configurations. try to build the app using ng serve or ng build --prod it will fail
Error Message
ERROR in The target entry-point "@azure/msal-angular" has missing dependencies:

msal

Security

[ ] Is this issue security related?

Regression

[X ] Did this behavior work before?
Version:

MSAL Configuration
MsalModule.forRoot({
auth: {
clientId: 'clientId',
authority: 'tenant',
redirectUri: 'redirectUri',
},
cache: {
cacheLocation: 'localStorage',
storeAuthStateInCookie: isIE, // set to true for IE 11
},
},
{
popUp: !isIE,
consentScopes: [
'user.read',
'openid',
'profile'
],
unprotectedResources: [],
protectedResourceMap: [
['https://graph.microsoft.com/v1.0/me', ['user.read']]
],
extraQueryParameters: {}
})

Reproduction steps
After importing MsalModule and set the default configurations. try to build the app using ng serve or ng build --prod it will fail

Expected behavior
Browsers/Environment

[ ] Chrome
[ ] Firefox
[ ] Edge
[ ] Safari
[ ] IE
[ ] Other (Please add browser name here)

@HadyAllam This is expected behavior as MSAL.js is a peer dependency of @azure/msal-angular, which means your application needs to install msal directly. When you install @azure/msal-angular, you should install msal at the same time. https://github.com/AzureAD/microsoft-authentication-library-for-js/tree/dev/lib/msal-angular#installation


[SPARK-5074][Core] Fix a non-deterministic test failure

Add assert(sc.listenerBus.waitUntilEmpty(WAIT_TIMEOUT_MILLIS)) to make sure sparkListener receive the message.

Test build #25032 has started for   PR 3889 at commit e61c198.

This patch merges cleanly.

Test build #25032 has finished for   PR 3889 at commit e61c198.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/25032/
Test PASSed.

Thanks for looking at this, I'll pull it in.

Thanks - merged this in master.

(Oops Patrick I didn't see your comment earlier)


Sync won't work

None of your recent builds have sync working!
I just says that I have to login to my google account, but nothing happens when I login...
It still says that I have to login to my google account...

@ecsjjgg  Add yourself to this google group. > https://groups.google.com/u/1/a/chromium.org/g/google-browser-signin-testaccounts
This is per the official documentation at https://www.chromium.org/developers/how-tos/api-keys/#signing-in-to-chromium-is-restricted


feat: make metrics scraping optional

What type of PR is this?
/kind enhancement
What does this PR do / why we need it:
To make metrics scraping optional in GitOps Operator
Have you updated the necessary documentation?

[x] Documentation update is required by this PR.
[x] Documentation has been updated.

Which issue(s) this PR fixes:
Fixes #?
Test acceptance criteria:

[x] Unit Test
[x] E2E Test

How to test changes / Special notes to the reviewer:
Edit the spec.monitoring field to false to disable scraping of metrics, like so
apiVersion: argoproj.io/v1beta1
kind: ArgoCD
metadata:
name: argocd-sample
namespace: default
spec:
monitoring:
disableMetrics: true
OR patch the resource using cli, like so
oc patch argocd argocd-sample -n default --type='json' -p='[{"op": "replace", "path": "/spec/monitoring/disableMetrics", "value": true}]'
To enable metrics for your instance, just change the spec.monitoring field to false, this will create the required role, rolebindings and service monitors and add the cluster-monitoring label to the namespace resource.
Disabling it will delete these resources and will also remove the cluster-monitoring label from the namespace resource.

/test v4.14-kuttl-parallel

/test v4.13-kuttl-parallel

/test v4.14-kuttl-parallel

Thanks for reviewing @svghadi

/test v4.14-kuttl-parallel

/test v4.13-kuttl-parallel

/test v4.14-kuttl-parallel

/test v4.12-kuttl-parallel

/test v4.12-images
/test v4.14-e2e

/test v4.12-images

/test v4.14-e2e
/test v4.13-kuttl-parallel

/retest

/lgtm

/approve

/lgtm
/approve