text:
Mobile app for teleoperation

I would like to collect more data. I know Robosuite has a script to collect human demos using Keyboard or SpaceMouse. I was wondering if the data collection app and the web interface as mentioned here https://roboturk.stanford.edu/ available?

The app and web interface are not publicly available at this time. I'd recommend using the SpaceMouse interface for now.

authors: amandlek nsojib


text:
[docs/setup] Fix order of commands when installing LISA from github

The current order runs a git fetch prior to moving into the repository directory. Fix this by changing the order of commands.

Indeed, thanks

source: gharchive/pull-request

authors: douglas-raillard-arm luis-machado-arm

repo: ARM-software/lisa

license_type: permissive

license_source: github-api


text:
Parameter validation tests for Blowfish

Description
This PR introduces unit test code the validation of the public API function parameters against NULL pointers.
Status
READY
Requires Backporting
NO
Migrations
NO
Additional comments
None
Todos

[ ] Tests
[ ] Documentation
[ ] Changelog updated
[ ] Backported

This PR has been superseded by PR #2282, so can now be closed.

authors: k-stachowiak sbutcher-arm

repo: ARMmbed/mbedtls

license_source: bigquery


text:
Automatic component API documentation

This PR adds API documentation for the four components that are currently documented. The API documentation is entirely programatically generated.
In the end, this was fairly straightforward. Instead of passing the docstring, we can just parse metadata.json with methods in dash/development/base_component.
Because this requires a tiny change to the Python package (exposing dbc.METADATA_PATH), I can't deploy this as an app. To view the app:

checkout this branch
npm i
pip install -e .
cd docs && python run.py

I have included a screenshot of the Buttons page with this change.

This looks great! It will also be good motivation to go through and improve the annotations. Thanks for setting it up!

Released in 0.0.11 and deployed at https://dash-bootstrap-components.opensource.asidatascience.com/l/components.

authors: pbugnion tcbegley


text:
Document Tabs

This PR adds documentation for tabs.
@pbugnion I would be particularly interested if you could take a look at the modifications in helpers.py, I had to pass some module imports into exec in order to get the callbacks to work, previously it was complaining about dbc not being recognised etc. What I've got works, but there may be better ways to do it.

I think this looks great! I've made a couple of comments inline, specially around helpers.

Thanks for the review @pbugnion! I agree that we should be careful to keep snippets relatively self-contained. I'll take another look at the load_source_with_app function and implement your suggestions.

@pbugnion this is now ready for another review
The main outstanding question I have is on naming. The function name load_source_with_locals doesn't quite sit right with its argument globals_dict which takes its name from exec's globals parameter. Any preference on how best to resolve? Or whether you think there is anything to resolve?

I was originally thinking of passing the variables as locals to exec, not globals, hence the name. According to the docstring on exec, though:

The globals must be a dictionary and locals can be any mapping,
defaulting to the current globals and locals.
If only globals is given, locals defaults to it.

Thus, we can't pass just locals. I think we should just rename the function to load_source_with_globals, or load_source_with_environment?

I actually tried something very similar to this originally, and checked the above snippet just to be sure, but if you do that the documentation page for tabs throws a name error saying tab1_content is not defined. For a reason I haven't been able to figure out yet that needs to be passed to globals in exec to work. In which case would the best thing be to pass everything in to globals, leave the current implementation more or less as is but rename to load_source_with_environment as suggested in a previous comment?

Ah that makes sense. Not sure why it's not working, but my understanding of exec is fairly limited.

In which case would the best thing be to pass everything in to globals, leave the current implementation more or less as is but rename to load_source_with_environment as suggested in a previous comment?

I think that make sense, yes.
Thanks for tackling this!

This looks perfect, thank you!


text:
Write lens report format version to report and validate it on load

This introduces a _version field in the report that indicates the version of the report format.
Eventually, the report will have a stable version and not introduce breaking changes, but for now the format version is set to be the same as the lens version, and validated to be equal on loading the report.
Fixes #9.

Could this be _schema_version or something equivalent rather than _version? _version suggests a version of a report rather than the report format to me.

I've changed the key to be _schema_report as @acroz suggested, and moved it into the Summary class as an integer as opposed to using lens' version. Even if the format is not considered stable, I think it will be useful to have that as an indicator of changes to the format, as requiring the same lens version on generation and loading might be too restrictive.

@zblz Is this ready to merge?

I changed the PR significantly since you approved it by moving the key into the summary (see message above). I forgot to ping you to re-review. Can you take a look at current changes and confirm you are OK with it?

@zblz LGTM!

authors: acroz srstevenson zblz

repo: ASIDataScience/lens


text:
Added udev rules and .sh file to copy them and also changed the name of the device node of the camera

@Mayankm96 i have added udev rules file and .sh file to copy them to the required folder of the system .... also i have changed the name of the device node of the camera as per the symlink added in the rules ......

Also @Mayankm96  please remove the cam_logitech.launch file from this directory since it implements libuvc_cam for 2 cameras which is not possible at all and this has been tried by me and @naishadhparmar already ..... for that purpose we are using the usb_cam package

@Suryansh470 Remove the file you are saying as part of this PR.

Also add other udev rules too.

okay i am committing after removing that file and also other udev rules will be added by tonight...

Added udev rules for arduino as discussed...

repo: AUV-IITK/fourtran


text:
Statistics Tab

[ ] #177 #173 Tag Usage UI (173 says it's after disliking a meme)
[x] #172 Dislikes affecting likes
[ ]  #171 Most Used Templates
[x]  #163 Dislike implementation

Most used templates will be pushed to v2.0

source: gharchive/issue

license: MIT


text:
Update glossary in developer guide

Removed unnecessary terms in Developer Guide

Codecov Report

Merging #66 (8a637a7) into master (86416d6) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #66   +/-   ##
=========================================
  Coverage     71.48%   71.48%
  Complexity      407      407
=========================================
  Files            73       73
  Lines          1287     1287
  Branches        130      130
=========================================
  Hits            920      920
  Misses          332      332
  Partials         35       35

Continue to review full report at Codecov.

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


text:
Update view command in UG/DG

Updates to be made to (but not limited to) user stories, use cases, implementation, features sections.
Tag teaming here, let's update each other on progress and see what we can get done! :)

tagging myself just for fyi that am working on the view command related sections atm! :)


text:
Update version number of Trackermon

Version 1.4.1

Codecov Report

Merging #327 (84cdd04) into master (368f4a1) will increase coverage by 0.05%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master     #327      +/-   ##
============================================
+ Coverage     62.93%   62.98%   +0.05%
- Complexity      500      501       +1
============================================
  Files            88       88
  Lines          1713     1713
  Branches        195      195
============================================
+ Hits           1078     1079       +1
+ Misses          553      552       -1
  Partials         82       82

Impacted Files
Coverage Δ

src/main/java/seedu/trackermon/MainApp.java
0.00% <0.00%> (ø)

...edu/trackermon/logic/parser/SortCommandParser.java
78.94% <0.00%> (+1.31%)
:arrow_up:

Continue to review full report at Codecov.

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

authors: JonathanHoshi codecov-commenter


text:
[PE-D] deleteTask command results not showing helpful message for index 0

When inputting index 0, command returns invalid command format, instead of flagging it as an invalid index

Labels: severity.Low type.FeatureFlaw
original: onefangg/ped#8

Update UG

authors: Korochii nus-pe-script


text:
Exit command

Note:
Closed previous pr that requests to merge from branch-exit-command as it does not branch off from master, thanks @zupey for pointing it out :")
Current features:
Exit command that exits the app.
Command:
exit
Todo:

test

What is ExitCommandParser for?  Currently, its not being called.
Maybe we use it to check if args is empty 🤔

Codecov Report
Base: 30.50% // Head: 30.40% // Decreases project coverage by -0.09% :warning:

Coverage data is based on head (bad9207) compared to base (7ef16cd).
Patch coverage: 0.00% of modified lines in pull request are covered.

:exclamation: Current head bad9207 differs from pull request most recent head 922a7ab. Consider uploading reports for the commit 922a7ab to get more accurate results

Additional details and impacted files
@@             Coverage Diff              @@
##             master      #65      +/-   ##
============================================
- Coverage     30.50%   30.40%   -0.10%
  Complexity      292      292
============================================
  Files           103      105       +2
  Lines          2816     2825       +9
  Branches        309      309
============================================
  Hits            859      859
- Misses         1873     1882       +9
  Partials         84       84

Impacted Files
Coverage Δ

...n/java/nus/climods/logic/commands/ExitCommand.java
0.00% <0.00%> (ø)

...n/java/nus/climods/logic/parser/CliModsParser.java
0.00% <0.00%> (ø)

...va/nus/climods/logic/parser/ExitCommandParser.java
0.00% <0.00%> (ø)

src/main/java/nus/climods/ui/MainWindow.java
0.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

authors: codecov-commenter sarrrdin zupey


text:
[PE-D][Tester E] Undocumented Edit behaviour when calling edit with only name as parameter

Calling edit with only the name parameters or incorrectly formatted command led to calling Find with the inputted name instead. The user guide only documents this behaviour when calling edit with a name that is common between multiple people
Edit: Also occur with editLoan

Labels: type.FunctionalityBug severity.Medium
original: zicotjia/ped#7

Intended feature, documentation updated

authors: nus-se-script rui-han-crh


text:
Add attribute for major programme information to Person class

As a student looking to pick up a second major, I want to be able to see students from certain majors, so that I can find out more about said major.

Closed with the new PR.

authors: hansstanley igezt


text:
[PE-D][Tester D] Incorrect message output for clear

Description
When i run the clear command, it gives me the following output message. This can be confusing as the name of the product is Strack.io.

Steps to reproduce

Run command clear

Suggestion
I believe the line of code that handles it is here.
https://github.com/AY2324S2-CS2103T-T08-2/tp/blob/08977d33f7eb0adf95d066ed3eda295f09000591/src/main/java/seedu/address/logic/commands/ClearCommand.java#L14
This can be changed to Strack.io has been cleared instead

Labels: severity.VeryLow type.FeatureFlaw
original: g-tejas/ped#15

Won't fix as this works as intended, with the exception of the message, would be a good to fix bug


text:
Update test cases to accommodate job field

Fixes #59

Great work siriousguy! Just a few nits. I have attached a screenshot of the remaining 9 test cases that have yet to pass. Perhaps you could resolve them before merging this PR. Keep up the effort!

LGTM!

authors: siriousguy ywllowsensor


text:
Missing '.' in link to blog post

You are missing a dot ('.') in front of html at the end of the link to the blog post.

Sorry, I missed you posting this issue! I have fixed it, thank you!

authors: AZHenley lukamac

repo: AZHenley/riscv-parser


text:
Can Metajo 2 -Zai  (hdi) Successfully start?  / Viper CTR

I have not been able to start this game nor the the Viper CTR game hdi.
Nothing at all happens. Just those 2 hdi's so far.
Nothing else since 2022.

creator of the repo hasn't been active since April...

creator of the repo hasn't been active since April...

Well...can you have a go at these 2 games and just tell me if it just me or can you get them to work? lol. Thanks!

authors: Duglor MesaBlack

license: mit


text:
针对 hexo new 与 hexo publish 分别建立 after hook 脚本

预期交付：

[ ] scripts/posts.afternew.js
[ ] scripts/drafts.afternew.js
[ ] scripts/drafts.afterpublish.js

熟悉 Title 中提到的 hexo 命令，利用 commander、gray-matter 库以及 scripts 中已封装的工具函数等解析当前脚本的调用归属，并规范处理对应 hexo 命令的执行结果（处理规范请见负责人 @GarlicGo 下发的子 issues）。

这三个脚本的初步功能预期可以简要介绍一下吗？

这个预期规范就是让你制定来着，可以参考：（以下均为假想实现：具体还得按照各命令的实际效果与可能会出现的分支情况来定）：
posts.afternew.js：1. 在 hexo new 后转义其所创建的文件&文件夹到相应的年/月/日文件夹下；2. 通用化：整理所有不符规范的文档及其资源文件夹
scripts/drafts.afternew.js：drafts 组织形式与 posts 保持一致
scripts/drafts.afterpublish.js：将发布到 _posts 中的文章亦按照时间分类目录整理

authors: GarlicGo Morilence

repo: Abyssers/blog


text:
Fix python module install destination

We were installing python modules in the wrong locations (and double installing pyimath).

Get rid of the double install.
Make the one install go to the right destination: <install>/lib/pythonX.Y/site-packages
With that in place, we can restore the EXPORT that makes the python module show up as a target in the exported cmake config.

@lamiller0 Can you please try this on your end and test if it installs the libraries in the right locations AND provides you with the expected target in the exported cmake?

The CI has a test that is supposed to validate this location, looks like that needs to be updated along with the change:
          # Confirm the python module loads. Query the site-packages directory and substitute ../_install
          export PYTHONPATH=`python -c "import site; print('../_install%s' % site.USER_SITE[len(site.USER_BASE):])"`
          python -c "import imath;print(imath.__version__)"

This is attempting to set the path to the site-packages directory underneath the _install directory. I'm not sure the right way to get that location.

I think I don't have it quite right, will revise

Question for @lamiller0 : Is the only reason you needed the target in the first place that the python module was given a nonstandard name and location? If it followed the usual conventions, would you no longer need the target in the cmake exports?

Does anybody know if there is a reason that Imath's CI only builds (let alone tests) python on Linux, so we aren't currently even trying on Windows or Mac in our CI? @xlietz do you happen to remember how this came about?

Question for @lamiller0 : Is the only reason you needed the target in the first place that the python module was given a nonstandard name and location? If it followed the usual conventions, would you no longer need the target in the cmake exports?

Is it considered non-standard to embed the python major and minor version as part of the package?  I suspect this was done as a way to accomodate multiple python versions installed on the same machine.

@lgritz sorry I missed the notification on this - yes the CI is intentionally NOT building the python bindings on Windows and Mac because of the frustrating Boost dependency. On Linux, Boost is installed in the docker image. But on Windows and Mac we need to get it from somewhere else. Originally I was downloading it from the boost.org in the CI but because it's so large, it seemed to be contributing to an overflow of the Boost host's download caps. We had some discussion about it at the time (I will try to find it) but ultimately there was never a resolution about how to create an image of boost for the Windows and Mac builds.

For historical context:
6/8/20 - Chris Horvath reported the boost downloads issue to the TAC on June 8, 2020 which he had seen on Boost's message boards and github repo.
6/9/20 - I disabled the Windows builds on June 9 and had planned to get boost from sourceforge instead getting the source from boost.org. But I never succeeded in getting that to work.  https://github.com/AcademySoftwareFoundation/openexr/pull/751
11/9/20 - Kimball reenabled Windows builds and removed all python references. https://github.com/AcademySoftwareFoundation/openexr/commit/cc7b31aecd5889ffbf58f75802cdb33aabc677ed (part of 3.0 release prep PR)

For OIIO, I use boost on all 3 platforms, so I know it can be done and is not a big deal. (With the caveat that I just use some regular boost components, not boost python! OIIO has been pybind11 for many years.) I get Boos via Homebrew on Mac and VcPkg on Windows. The VcPkg build does add a lot of time to the CI, but even if we need to disable it later, I think it's vital that we get it working to test these changes (which I'm still failing at, ugh).

That's great to know it's working on the OIIO CI! I can take a look at it this weekend too if you think it would help.

That's great to know it's working on the OIIO CI! I can take a look at it this weekend too if you think it would help.

Wow, yeah, I would love any help you could give. I feel like I'm in above my head on this.
Feel free, also, to start with my patch (I think that having commit privileges, you are able to push onto my branch from this PR), maybe we can collaboratively get there in one joint PR.
I'm going to continue to tinker if I have free time, so there's some chance I will have it mostly done before the weekend, but even if that's the case, I would sure like you to look it over and make sure you think it's ok.

I’d like to resolve this issue and get another release out, but it’s not clear where we’ve left this.
The issue started with #360, describing a problem where we want to build Imath with all features enabled, including the python bindings, but make the bindings optional when building a downstream package that uses Imath via CMake. #361 removed the EXPORT from the python install line to resolve that issue, which subsequently led to the build failure in #395, Imath::PyImath_Python3_9 missing from ImathTargets-release.cmake.
The current theory appears to be that the original problem sprung from installing the module in the wrong location, but have we determined exactly what the correct location is?
Fixing the location will like resolve the build failure in #395, but it’s not clear it solves the original issue described in #360.
@josch, @lamiller0, @OlivierArgentieri, @lgritz, @kmilos, any advice about how to proceed?

I dont have a full solution, but would like to reiterate that I suspect defining Imath::PyImath_Python3_9  will likely still be necessary going forward for systems in which more than 1 python version is available.  (instead of just making Imath::PyImath available)

authors: cary-ilm lamiller0 lgritz xlietz

repo: AcademySoftwareFoundation/Imath


text:
Performance degrades while loading large number of documents

We notice scalability issues with scenes containing 500+ MaterialX documents. MaterialX load, validation and code generation takes longer than expected.
Current bottleneck seems to be in Document::importLibary which is required for every mtlx file load.

I've added a catch benchmark to repro here
MaterialXTest.exe "GenShader: Scalabilty Check" --benchmark-samples 10
Filters: GenShader: Scalabilty Check
MaterialXTest.exe is a Catch v2.13.10 host application.
Run with -? for options

-------------------------------------------------------------------------------
GenShader: Scalabilty Check
-------------------------------------------------------------------------------
F:\source\MaterialX-adsk-fork-current\source\MaterialXTest\MaterialXGenGlsl\GenGlsl.cpp(100)
...............................................................................

benchmark name                       samples       iterations    estimated
                                     mean          low mean      high mean
                                     std dev       low std dev   high std dev
-------------------------------------------------------------------------------
Load documents and ShaderGen                    10             1      1.8764 s
                                        196.507 ms     191.65 ms    205.391 ms
                                        10.2555 ms    4.27542 ms    15.2796 ms

4.284 s: GenShader: Scalabilty Check
===============================================================================
All tests passed (88 assertions in 1 test case)

Similar test with 100 samples (2000 documents)
GenShader: Scalabilty Check
-------------------------------------------------------------------------------
benchmark name                       samples       iterations    estimated
                                     mean          low mean      high mean
                                     std dev       low std dev   high std dev
-------------------------------------------------------------------------------
Load documents, validate and
generate shader                                100             1     4.52616 m
                                         2.50697 s     2.49358 s     2.52304 s
                                         74.617 ms     63.377 ms    90.9251 ms

256.122 s: GenShader: Scalabilty Check
===============================================================================
All tests passed (8888 assertions in 1 test case)

Clarifying questions:

Is there a fundamental reason to import the library into each document?
Can the library be data that be referred to?
Does each document need to self contained?

These are great notes and observations, thanks @ashwinbhat!
As one initial proposal, perhaps we could add support for passing library documents directly to the shader generator, rather than requiring content and library documents to be merged in advance?

Yes, I believe we could improve many workflows by defining the notion of a library document that is referred to for validation, shader generation, baking and other workflows. I think this might also help with USD integrations where the library document will be registered and referred to as needed. Separating the library document and the material document will be a rather significant API change.

@jstone-lucasfilm I did a quick prototype by introducing an API to allow registering node library. This seems promising i.e. validation and shader generation seem be to ok. I have not run full suite of tests yet, but wanted to ask if there will be more interest in this approach.
see https://github.com/autodesk-forks/MaterialX/commit/88cb02fa18ecfc0c638c576a52a8ff81ad453085#diff-446cebc815ec59703ca40bb07af4fe92e68aa357d5e4c3f3e53e3d90f1dc8f54
Running the benchmark test with 1000 samples i.e. 20 x 1000 documents.

benchmark name                       samples       iterations    estimated
                                     mean          low mean      high mean
                                     std dev       low std dev   high std dev
-------------------------------------------------------------------------------
Load documents, validate and
generate shader                               1000             1     5.19299 m
                                        288.903 ms    286.796 ms    291.134 ms
                                        34.9009 ms    33.2086 ms    36.8215 ms

297.767 s: GenShader: Scalabilty Check

Profile report shows the hotspot are no longer the issue.

Wow, that looks very promising, @ashwinbhat!  I would probably recommend the name "dataLibrary" instead of "nodeLibrary", but otherwise this definitely looks like a worthwhile proposal for the main repository.

Hey @ashwinbhat , Very interesting what you've found here. Seems the big gain is to reuse the same in memory document for reference definitions.
Note that you will probably want to watch out that definitions which reside in other documents must also be found, and that it may be useful to cache a set / list of libraries instead of just one. As such the search logic could be made more robust by searching in these reference libraries first and if not found then try the local document. Does this sound reasonable ?
Some other thoughts pop to mind: who owns "the" reference librarie(s) ? They are referenced by 1 or more documents but they don't own it.  Also if it can be dynamically settable this can cause problems if cleared or set to another library file. I can see it's clean to have a Node API hide the logic but wondering if it should be something more explicitly passable instead?
Also, I can see the workflow where a library can be embedded into a saved document would need to be considered as it's no longer intrinsically part of the document -- which is actually probably a good thing.
My final 2.5 cents is that for previous work, Niklas and I added a "scene" concept which encaspulated 1 or more documents + reference libraries but this of concept of course does not even exist here...

Hey @bernardkwok
Modifying the search logic to search Data library first and fallback to local sound reasonable. It will probably help with the embedded workflow, though I think we should try to define more clearer workflows and understand why embedding a library in document would be necessary.
Ownership of the data library is undefined. In the current workflow an application can load a set of libraries, import it into a document. It can then load another set of libraries for another document. Though this is flexible I'm not sure how practical this workflow is.
By requiring an API that ensures the datalibrary is set once, will help setting a clear state. Is there a clear use-case where applications might want to dynamically set datalibrary between document loads? I thought about the option to pass around the datalibrary but then it would be up to the application to keep track of them. But I guess my key question here is do we foresee a use-case where there are multiple instances dataLibraries?

authors: ashwinbhat jstone-lucasfilm kwokcb

repo: AcademySoftwareFoundation/MaterialX


text:
Add enabled Flag to Transitions

New Feature
Description
In #1175 the enabled flag was added to the Item schema. This would also be useful to add to transitions as well.
#1187 addresses adding this flag for Effects.

Would this be a good first issue for Dev Days?

Since this landed for Effects we should prioritize this for implementation, otherwise we've got a straggling outlier.

authors: jspada200 meshula reinecke


text:
Sonar scans require newer version of builder wrapper

Sonar scans are expecting a newer version of the build wrapper. Here is warning output from the most recent Sonar scan of the Imath repository:
" File  /__w/Imath/Imath/_build/bw_output/build-wrapper-dump.json
was generated using 6.16 build-wrapper version,
which does not match analyzer 6.21.0.32709 version.
Please download matching version from the server
https://sonarcloud.io/static/cpp/build-wrapper-linux-x86.zip "

Thanks for the report @xlietz ! I'll upgrade to the sonar scanner v4.6.2.2472 and the latest build wrapper and re-release new CI images.

authors: aloysbaillet xlietz

repo: AcademySoftwareFoundation/aswf-docker


text:
Removed boost/mpl methods in favour of new methods on openvdb::TypeList

Signed-off-by: Nick Avramoussis 4256455+Idclip@users.noreply.github.com
One important consideration in this PR is compile times - my implementation probably differs from boost so this may have been impacted (could have been improved as well as impaired). Need to test.

Very cool stuff, nice one! I like the readability of the new methods much better than the old ones.
Would be good to get the TestTypes::testTypeList() unit test updated to test this new functionality, otherwise I'm happy with this.

I've added a set of static asserts to TestTypes::testTypeList() to test these new methods

This is great. In case you weren't aware, the Houdini plugin is failing at the moment and will likely need updating to use this (see the Azure job).

In case you weren't aware, the Houdini plugin is failing at the moment and will likely need updating to use this (see the Azure job).

Oops thanks, will fix this

I'd quite like @kmuseth to look over this change before we merge it in.

I won't merge any of these boost PRs until we've got consensus in the tsc

I'll review it today!

@kmuseth I've added documentation to all the type internal meta-programming structures - I think I went a bit overboard, I added doxygen documentation but didn't realise that there exists a @cond OPENVDB_TYPES_INTERNAL doxygen rule which disabled this documentation generation by default. Oh well, it's there should we ever want it in the future!
Regarding the comment on compile times, this was something I wanted to test and hadn't yet. I didn't anticipate any adverse effects but I wanted to make sure that this solution performed at least as fast as the boost solution. I've tested with gcc and am getting similar results (across 32 threads, building the .so, the binaries and the tests, obviously no ccache):
before: real 4m52.941s
after: real 4m53.310s

great, thanks Nick

authors: Idclip danrbailey kmuseth

repo: AcademySoftwareFoundation/openvdb


text:
Rez-Bind: Override Built-in Bind Modules

Hi Rez folks!
I'm interested in overriding the built-in behavior of the os.py and arch.py bind modules, so I added custom versions to bind_module_path in rezconfig.
Unfortunately, search path ordering in package_bind.py means that the builtin bind modules end up taking priority over user-defined modules.
I would expect the user-defined modules to override the builtins (this is how plugin_path works, for example -- a custom env.py overrides the builtin env.py command).
https://github.com/AcademySoftwareFoundation/rez/blob/73e7493fba2275115463d7716ccf08e46c52854f/src/rez/package_bind.py#L25
Environment

macOS 13
Rez 2.111.3
Python 3.9.10

To Reproduce

Create a custom bind_modules directory
Create a file named "bind_modules/os.py"
Add a custom bind module creation script to os.py
Add bind_modules directory to REZ_BIND_MODULE_PATH
Delete current os package, and call rez-bind os

Expected behavior
My expectation is that my custom os.py module will be called.
Actual behavior
The builtin os.py is called.
Related Issues/PRs
Happy to file a PR for this one - I think it's a matter of changing:
searchpaths = config.bind_module_path + [builtin_path]

to:
searchpaths =  [builtin_path] + config.bind_module_path

Hi @Pantsworth! Indeed that seems weird and is not what I would expect. Feel free to open a PR for this. It would be very appreciated!

authors: JeanChristopheMorinPerso Pantsworth

repo: AcademySoftwareFoundation/rez


text:
Added the missing paths=self.package_paths

Fixes: https://github.com/nerdvegas/rez/issues/1274
Added missing self.package_paths so get_resolve_diff produces correct results. This is safe to do because of the existing self.package_paths != other.package_paths check, above.

This looks alright to me. Can you rebase your branch please? Rebasing should fix the Windows CI failures.

@ColinKennedy See https://github.com/AcademySoftwareFoundation/rez/runs/7476834008?check_suite_focus=true.
I think the copyright text is missing in the test file you added.

@JeanChristopheMorinPerso I added the copyright header manually but probably should just run the autofix script. If I run that and it still fails, I will let you know

authors: ColinKennedy JeanChristopheMorinPerso


text:
update org.activiti.cloud.dependencies:activiti-cloud-modeling-dependencies to 7.0.65

UpdateBot pushed maven dependency: org.activiti.cloud.dependencies:activiti-cloud-modeling-dependencies to: 7.0.65

UpdateBot commands:
updatebot push-version --kind maven org.activiti.cloud.dependencies:activiti-cloud-modeling-dependencies 7.0.65

repo: Activiti/activiti-cloud-modeling


text:
Pulling new versions

UpdateBot pushed version changes from the source code in repository: . ref: develop

UpdateBot commands:
updatebot push --ref develop --merge=false

UpdateBot commands:
updatebot push --ref develop --merge=false

UpdateBot commands:
updatebot push --ref develop --merge=false

UpdateBot commands:
updatebot push --ref develop --merge=false

UpdateBot rebasing due to merge conflicts

repo: Activiti/activiti-cloud-query


text:
update org.activiti.cloud.dependencies:activiti-cloud-dependencies to 7.1.87

UpdateBot pushed maven dependency: org.activiti.cloud.dependencies:activiti-cloud-dependencies to: 7.1.87

UpdateBot commands:
updatebot push-version --kind maven org.activiti.cloud.dependencies:activiti-cloud-dependencies 7.1.87 --merge false

repo: Activiti/ttc-campaigns-service


text:
update org.activiti.cloud.dependencies:activiti-cloud-dependencies to 7.1.0

UpdateBot pushed maven dependency: org.activiti.cloud.dependencies:activiti-cloud-dependencies to: 7.1.0

UpdateBot commands:
updatebot push-version --kind maven org.activiti.cloud.dependencies:activiti-cloud-dependencies 7.1.0 --merge false

repo: Activiti/ttc-query-campaign


text:
Update webView protocol to open external links in web browser

Problem
Links inside the webview that opened into new window (with target="_blank") were not working. You can reproduce the problem by typing "link" in the ExampleApp.
Solution
Open links into a new browser window. (Unless there is a better way?)
https://stackoverflow.com/questions/30603671/open-a-wkwebview-target-blank-link-in-safari/50924004#50924004

Great!!

repo: AdaSupport/EmbedFramework


text:
added options

Create options

List all music albums
List all genres (e.g 'Comedy', 'Thriller')
Add a music album

@rugwizangoga we appreciate your work bro,

authors: Lul-Abdifan rugwizangoga

repo: Adams-Ghub/catalog-of-my-things


text:
Possible to use with institutional log-ins?

This looks like a great tool. Is there any way to make it work for books which require institutional log-ins?

Hmm 🤔 , probably it requires some changes to the code.
I don't have an institutional account which integrates with HathiTrust, so I'm not completely sure what it would take to get that to work.
My guess is that they use cookies, so then it could probably be done by:

Using a browser to login as usual
Copy the cookie from the developers tools
Add an additional flag to this command line tool where you can paste the content of this cookie

It won't be the most user-friendly approach (opening a browsers dev-tools can sound daunting), but it will probably be the simplest way to implement it.
Shouldn't be too hard to implement, but I don't have a need for it nor an account to give it a shot, so sadly won't be able to spend time on it.
Feel free to give it a shot though. The entire code-base of this project is literally only 40 lines, where https://github.com/Addono/HathiTrust-downloader/blob/09e140e8bd61459604fcf1d9615e95f34eea10de/hathitrustdownloader/cli.py#L25 is the crucial part where you would need to add a parameter cookies= with the value of the cookie(s) needed for authorization. Finding out which cookie/cookies are used by HathiTrust to handle auth would need some investigating.

Thanks for taking the time to respond! I'll take a look at this when I get the chance.

Sure. Feel free to send a PR if you get it to work, or ask something if you're stuck.
No need to feel pressured to give it a shot though 😄

Any chance anyone has gotten this to work?

I have not. Here's the process I'm currently using. I'm not likely to spend the time to figure out how to automate it.

Wow thank you. Been looking for something like this for a while now.

authors: Addono mdelhey mendelkranz

repo: Addono/HathiTrust-downloader


text:
Compact/compresesed hosts File compatibility?

N.B.: I'm not sure if this is the right place to ask this. Please move as you see fit.
Do the various AG products handle hosts file compacting (usually called "compression" which seems like wrong terminology), such as that performed by https://github.com/ScriptTiger/Hosts-Conversions?  (For that matter, can they handle actual hosts/filterlist compression, like gzip, zip, 7z, &c?) There's some discussion that this will need to be done on some very large adblocking lists due to various problems elseplace (OS issues, browser storage restrictions, &c), so I thought to ask.

AdGuard supports hosts-file syntax only in DNS blocklists, and any valid hosts file should work there. If that is not the actually the case, please file an issue here.
Regarding filter lists compression, there's little point in supporting it for AdGuard, since our apps are bound only by the amount of storage and RAM.

@ngorskikh The major reason I asked re: support for such is for a subscription for such from a 3rd party, so, if the (hosts|filterlist) file is served in 1 or more of the above ways (e.g., 9-hosts-per-line compacted & gzipped), would that be directly supported?

"Compacted" hosts files are valid hosts files, so they are supported.

We don't support loading of gzipped files.

DNS filter lists are not loaded into memory. We rely on reads from disk cache. If decompress them, they will take place in memory, which is not acceptable, e.g. on iOS with hard memory limit for network extensions.

Thanks very much for the clarification! 🙇🏾♂

Another question related to this:  Would there be any advantage/disadvantage to AG apps in using these compacted forms?  Or do they resolve to the same representation internally as the equivalent uncompacted file?

Pretty much the same for AdGuard

Again, thanks very much.

authors: TPS ameshkov ngorskikh sfionov

repo: AdguardTeam/CoreLibs


text:
Clean .bazelrc file

Signed-off-by: jinshiying jin.shiying@zte.com.cn

bors r+

authors: amadeus-zte shiyingjin

repo: Adlik/Adlik


text:
Changelog - Feature -TextButton

В компонент Text Button, в сценарии Multi Button, добавлена опциональная возможность включения иконки слева
https://www.figma.com/file/EGEGZsx8WhdxpmFKu8J41G/Admiral-2.1-UI-Kit?type=design&node-id=146368-112749&t=tcGEsxLVu3YwG8Ru-4

Дубликат
https://github.com/AdmiralDS/react-ui/issues/901

authors: Parasjona dariawish

repo: AdmiralDS/react-ui


text:
Allow uniform download links in JCR Compare

Fix for 1836.

Pull Request Test Coverage Report for Build 4404

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 52.774%

Totals

Change from base Build 4400:
0.0%

Covered Lines:
13423

Relevant Lines:
25435

💛  - Coveralls


text:
Adding letterboxing & pillarboxing when aspect ratios of original and final image are different

Adding letterboxing and pillarboxing accordingly to the image when the expected final image aspect ratio is not same as the original image aspect ratio

@ramyareddyg @bstopp did we decide this should be a different Transformer from the original Resize? I think that makes sense opposed to 1) changing behavior of the original  2) packing too much logic into the Resize transformer.

Getting back into this now: It looks like there's no way to turn off the letter/pill boxing. I agree @davidjgonzalez, a subclass of the resize makes more sense; ensuring backwards compatibility with in-the-wild use cases.

Closed via #563

authors: bstopp davidjgonzalez ramyareddyg


text:
Feature / HTTP cache (not ready for merge)

HttpCache caches the full output of a http request per the configurations and then bypass the request processing entirely on each subsequent requests achieving high performance.
Features

Caches anonymous and authenticated requests.
Supports caching personalized requests. Group based caching provided OOTB. Mechanism exposed to plugin custom logic for handling personalized requests.
Flexible cache configs tied to URIs supported. Allows extending cache configs. Allows multiple cache configs.
Developer hook exposed to plugin custom rules on key cache handling events such as request receive, response cache, response deliver and cache invalidate.
Provides mechanism to plugin custom cache keys.
Pluggable cache persistence model. In-Memory implementation provided OOTB. Allows multiple cache stores to co-exist.
Invalidation mechanism based on JCR paths and Sling jobs. Sample JCR change event handler based invalidation supplied. In-Memory cache store supports TTL.
Provides instrumentation based on JMX mbeans.

Credits to @davidjgonzalez for this wonderful idea, contribution and guidance.
Please review.

FYI - going to try to merge the servlet changes into your branch now

Looks like there is a conflict in bundle/pom.xml

@Sivaramvt added some Felix namehints for config factories so life in 6.2 is nice. Moved off the default diamond so we can keep 1.6 for now (probably doesnt matter, but was a benign change and were changing a bit already this release).. lets see what Travis says....

Thanks @davidjgonzalez . Super excited that this work would be used by bigger community.

authors: Sivaramvt davidjgonzalez


text:
Remove duplicated downstream NODE_LABEL

Fixes: #54
Signed-off-by: Morgan Davies morgandavies2020@gmail.com

run tests

What about this line?
https://github.com/AdoptOpenJDK/ci-jenkins-pipelines/blob/e84ab0f5463acbfe8401c2b441351f1047c09808/pipelines/build/common/openjdk_build_pipeline.groovy#L358

PR Tester failures look unrelated

What about this line?

I don't think that line should matter as I think it's setting up the sign job NODE_LABEL rather than interacting with the existing parameter. The PR doesn't allow for testing of the sign job so I'll confirm this on merge with a test pipeline

[2021-03-17T22:07:00.219Z] ===RELEASE FILE GENERATED===
[2021-03-17T22:07:00.219Z] From /cygdrive/c/Jenkins/temp/workspace/./build//src
[2021-03-17T22:07:00.219Z]  * branch                  HEAD       -> FETCH_HEAD
[2021-03-17T22:07:00.219Z] awk: warning: escape sequence `\-' treated as plain `-'
[2021-03-17T22:07:00.219Z] From /cygdrive/c/Jenkins/temp/workspace/./build//src
[2021-03-17T22:07:00.219Z]  * branch                  HEAD       -> FETCH_HEAD
[2021-03-17T22:07:00.219Z] awk: warning: escape sequence `\-' treated as plain `-'
[2021-03-17T22:07:00.920Z] From /cygdrive/c/Jenkins/temp/workspace/./build//src
[2021-03-17T22:07:00.920Z]  * branch                  HEAD       -> FETCH_HEAD
[2021-03-17T22:07:01.623Z] awk: warning: escape sequence `\-' treated as plain `-'
[2021-03-17T22:07:02.328Z] From /cygdrive/c/Jenkins/temp/workspace/./build//src
[2021-03-17T22:07:02.328Z]  * branch                  HEAD       -> FETCH_HEAD
[2021-03-17T22:07:02.665Z] awk: warning: escape sequence `\-' treated as plain `-'
[2021-03-17T22:07:02.665Z] Removing unnecessary files now...
[2021-03-17T22:07:02.665Z] Should have the source, I'm at /cygdrive/c/Jenkins/temp/workspace/build/src
[2021-03-17T22:07:02.665Z] Currently at '/cygdrive/c/Jenkins/temp/workspace/build/src/build/windows-x86_64-normal-server-release/images'
[2021-03-17T22:07:02.665Z] moving j2sdk-image to jdk8u292-b06
[2021-03-17T22:07:02.665Z] mv: cannot move 'j2sdk-image' to 'jdk8u292-b06': Permission denied
script returned exit code 1

I couldn't see the obvious place to check what host this ran on though.

Running https://ci.adoptopenjdk.net/job/build-scripts/job/openjdk11-pipeline/1451/console with signing and installers enabled to test https://github.com/AdoptOpenJDK/ci-jenkins-pipelines/pull/101#issuecomment-801533126

authors: AdamBrousseau M-Davies karianna

repo: AdoptOpenJDK/ci-jenkins-pipelines


text:
Retrieve RISC-V Version String job

Instead of retrieving the faked version string, we run a downstream job on a riscv machine after the build that runs java version and passes it back to the pipeline in time for parseVersionOutput()
Also clears up white spaces in openjdk_build_pipeline.groovy

Fixes: #1773
Signed-off-by: Morgan Davies morgandavies2020@gmail.com

Just because I'm picky ... Can you change the title of the PR and the commit message to write the architecture as RISC-V please?

Sure but it'll still appear as riscv in the code as that's what the architecture is listed as in the config file

Can this come out of draft now?

Hitting a node issue at the moment. Java version will not run in the azure docker container, for ref see https://ci.adoptopenjdk.net/job/build-scripts/job/utils/job/riscv-version-out (builds 12-14):
13:45:16  ./java: 1: ./java: ELF��: not found
13:45:16  ./java: 2: ./java: @h@8: not found
13:45:16  ./java: 3: ./java: : not found
13:45:16  ./java: 3: ./java: 0: not found
13:45:16  ./java: 20: ./java: Syntax error: Bad function name

I'm pretty sure it's running on the binary due to all the debug I'm spitting out to verify it but I get this error no matter where I execute the binary from. It's likely the environment (since it's actually running on the azure build agent, not the riscv machine itself) which begs the question, which riscv machine am I supposed to use from this list?

I would literally go with "anything that's online" which at the moment is just one from that list - so just have it select the riscv label and it'll pick whatever we happen to have online at the time. We don't have enough RISC-V systems to get picky about it ;-)

I would literally go with "anything that's online" which at the moment is just one from that list - so just have it select the riscv label and it'll pick whatever we happen to have online at the time. We don't have enough RISC-V systems to get picky about it ;-)

I am already picking the riscv label and it picks up the docker container every time which is why it's failing. I'll try and pick a node from that list and go with that then

I would literally go with "anything that's online" which at the moment is just one from that list - so just have it select the riscv label and it'll pick whatever we happen to have online at the time. We don't have enough RISC-V systems to get picky about it ;-)

I am already picking the riscv label and it picks up the docker container every time which is why it's failing. I'll try and pick a node from that list and go with that then

Hmmm OK we need to fix that - the x64 machines used to build should NOT have riscv on them - I set up a tag of riscvcross when we were buiding natively.
But if that's been broken use riscv&&ci.role.test

Note when using the jenkins sh tag and java -version, make sure you specify 2>&1 to redirect to stdout, even if returnStdout is set 🤦

run tests

🟠 PR TESTER RESULT 🟠
❎ Some pipelines failed or the job was aborted! ❎
See the pipeline-build-check below for more information...

Failed due to existing OpenJ9 problems https://github.com/eclipse/openj9/issues/11236

JSON files aren't being produced (due to early archive)

So we try archiving the binaries first, and then the metadata

Fixed by archiving the binaries, and then the metadata

JSON data shows it's using the faked version.txt file and not new one

Change name of faked version to ensure it's not picked up
Add additional check in formMetadata() to look for riscv info

Fixed by description above
See https://ci.adoptopenjdk.net/view/work-in-progress/job/build-scripts-testing/job/jobs/job/jdk11u/job/jdk11u-linux-riscv64-openj9/26/ for evidence this works. I'll run tests one more time to ensure that I haven't broken anything new and open this up for review

🟠 PR TESTER RESULT 🟠
❎ Some pipelines failed or the job was aborted! ❎
See the pipeline-build-check below for more information...

Failed due to issues with connecting to the docker agent. Existing issue unrelated to this PR https://github.com/AdoptOpenJDK/openjdk-build/issues/2244
17:35:46  Compiling /home/adoptopenjdk/workspace/build-scripts-pr-tester/build-test/jobs/jdk8u/jdk8u-linux-x64-hotspot/workspace/build/src/hotspot/src/share/vm/code/dependencies.cpp
17:35:47  Cannot contact azurebuildagentd92090: hudson.remoting.RequestAbortedException: java.io.IOException: Unexpected termination of the channel
01:28:39  Cancelling nested steps due to timeout
01:28:39  Could not connect to azurebuildagentd92090 to send interrupt signal to process
[Pipeline] }
[Pipeline] // timeout
[Pipeline] echo
01:28:39  [ERROR] Build JDK timeout (8 HOURS) has been reached. Exiting...
[Pipeline] }
[Pipeline] // withEnv
[Pipeline] }
[Pipeline] // stage
[Pipeline] }
01:28:39  ERROR: Issue with creating launcher for agent azurebuildagentd92090. The agent has not been fully initialized yet

run tests

Running one final test at https://ci.adoptopenjdk.net/view/work-in-progress/job/build-scripts-testing/job/jobs/job/jdk11u/job/jdk11u-linux-riscv64-openj9/27/console but assuming that passes and no more changes are requested, this should be good to merge at an admin discretions

One observation (and it doesn't need to be done in this PR). Are you able to generalise this job to be for any cross platform builds. I'm thinking of windows arm64 and macOS arm64 which will likely need the same logic

run tests

authors: M-Davies gdams karianna sxa

repo: AdoptOpenJDK/openjdk-build


text:
inventory: add m1 build macOS machines + playbook support for m1

Checklist

[x] commit message has one of the standard prefixes
[ ] FAQ.md updated if appropriate
[ ] other documentation is changed or added (if applicable)
[ ] playbook changes run through VPC or QPC (if you have access)
[x] for inventory.yml changes, bastillion/nagios/jenkins updated accordingly

Running VPC to make sure the changes to adoptopenjdk_install work: https://ci.adoptopenjdk.net/job/VagrantPlaybookCheck/1132/

Could you possibly remove the brew cask --uninstall adoptopenjdk8 line from the macos (or whatever else may need to be done to fix the GitHub check). That can then test that the macOS specific stuff works, as I have no means of testing it otherwise.
The Unix side of things seem okay: https://ci.adoptopenjdk.net/job/VagrantPlaybookCheck/1132/OS=Debian8,label=vagrant/console

Could you possibly remove the brew cask --uninstall adoptopenjdk8 line from the macos (or whatever else may need to be done to fix the GitHub check).

I double checked and the github env is meant to have adoptopenjdk8 so this is an upstream issue on their end. I'll raise a bug at GitHub

authors: Willsparker gdams

repo: AdoptOpenJDK/openjdk-infrastructure


text:
Upgrade to ASm 7.3.1

Defines a property to hold the ASM version to be used and then uses that property in the rest of top.xml -- this will make it easier for future upgrade of ASM library
Resolves
Signed-off-by: Mesbah_Alam@ca.ibm.com Mesbah_Alam@ca.ibm.com

Test with JDK 15 : Test_grinder/job/Grinder/6433/ (note- it fixes the failure reported in : https://github.com/eclipse/openj9/issues/8178)
Test with JDK 14: Test_grinder/job/Grinder/6437/console
Test with JDK 8 : Test_grinder/job/Grinder/6438/console

repo: AdoptOpenJDK/stf


text:
DJI *.rst new feature

Hi, will gyroflow work with DJI *.rst files? If this data is useful, I will be happy to write the code for this part.

What's in these files? Do you have a sample?

well it is telemetry so it's in scope of telemetry-parser. If you want to write a parser for it, then feel free, I'll accept a PR.
But in terms of Gyroflow, it's not useful for stabilization, because it's too low frequency, and it doesn't have camera angles

authors: AdrianEddy Rufics

repo: AdrianEddy/telemetry-parser


text:
[ECP-9155] Apple Pay on product page creates a new quote every time the page is loaded

Describe the bug
When enabling the Apple Pay express checkout on the product page, a new quote is created in Magento's quote table and its associated tables such as quote_item every time a product page is visited, including when it's refreshed. This happens for devices/browsers that both do and don't support Apple Pay.
To Reproduce
Steps to reproduce the behavior:

In the Magento Admin, configure and enable Apple Pay express checkout on the product page
Visit a product page, and note a new record inserted into Magento's quote table
Visit a different product page (or refresh the current product page!)
View Magento's quote table

Expected behavior
No new database records for customers who did not even initiate an Apple Pay session, such as:

only creating a new quote when a customer initiates an Apple Pay session (e.g. by clicking the button)
deleting the quote if a customer abandons an Apple Pay session (e.g. closing the Apple Pay form)
never creating a quote for non-supported devices. The current behavior is reproducible on browsers such as Firefox, which doesn't even support Apple Pay.

Screenshots
Here's an example of records being created in the quote table when refreshing a product page quickly a few times.

Desktop (please complete the following information):

OS: reproduced on MacOS
Browser: reproduced on Firefox, Safari

Smartphone (please complete the following information):

Device: iPohne
OS: iOS
Browser: reproduced on Safari

Additional context
This is what looks like the sequence of operations that leads to the creation of quotes each time a product is viewed

https://github.com/Adyen/adyen-magento2-express-checkout/blob/develop/view/frontend/web/js/applepay/button.js#L83
https://github.com/Adyen/adyen-magento2-express-checkout/blob/develop/view/frontend/web/js/actions/getExpressMethods.js#L57
https://github.com/Adyen/adyen-magento2-express-checkout/blob/develop/etc/webapi.xml#L16
https://github.com/Adyen/adyen-magento2-express-checkout/blob/develop/Api/ExpressInitInterface.php
https://github.com/Adyen/adyen-magento2-express-checkout/blob/develop/Model/ExpressInit.php#L176

Hello @pignion,
Thank you for raising this issue and providing all the stack which will be very helpful during our investigation. That was a design decision back in the days of the initial kickstart of the project. However, it looks like the time has come to revisit the approach of initiating express payments on product detail pages since it creates an overhead.
I've already created an investigation ticket for this issue. You will be informed when it's done.
Best regards,
Can

Hello,
We raised this issue to Adyen support a few months ago as we had the same problem for one of our clients. At the time this was not in Adyen's roadmap we therefore had to come up with our solution. Sadly we aren't allowed to share the code we made, which is anyway is not compatible as we are not using a Luma-based theme.
It's quite simple, we created a new API /V1/adyen/express/methods, we use the same API for guest users & logged users
This service will call the adyen methods API endpoint with the product SKU & the current store code.
$product  = $this->productRepository->get($sku);
$store = $this->storeManager->getStore($storeCode);

$currencyCode = $store->getCurrentCurrencyCode();

$requestParams= [
    "channel"         => "Web",
    "merchantAccount" => $merchantAccount,
    "countryCode"     => $this->countryHelper->getDefaultCountry($store->getId()),
    "shopperLocale"   => $this->adyenHelper->getCurrentLocaleCode($store->getId()),
    "amount"          => [
        "currency" => $currencyCode
    ]
];

$paymentMethodRequest["amount"]["value"] = (int) $product->getFinalPrice() * 100;

$client = $this->adyenHelper->initializeAdyenClient($store->getId());
$service = $this->adyenHelper->createAdyenCheckoutService($client);
$responseData = $service->paymentMethods($requestParams);

On the frontend side we are not using the expressInit API but our own to display the button.
After that when the Apple pay button is being created, we modified the on-click event here. We validate the form and call the getExpressMethods which will create the cart and apply any cart price rule that is needed.
This way unless the user clicks on the applepay button he is not creating a cart/quote in Magento.
Finally we also added a
if (!window.ApplePaySession) {
      console.log("Not compatible with ApplePay. Missing ApplePaySession session")
  }

To try and display the apple button with a call to the new /V1/adyen/express/methods API only when needed.
Hopefully, this helps someone to make it happen.

Hello,
The fix would help with this case but I do not believe this has been fixed by storing the quote id. Some website have thousands of PDP views per minute of different customers. We would still overwhelm the database.
The fix would be to initialize the only when the quote is clicked.

I agree with @oliverde8. PR #93 does not solve the issue.

authors: aedmonds candemiralp oliverde8 pignion


text:
Cannot test 3dsecure 2 credit card without cvc, cannot make CVC not mandatory

Describe the bug
We're trying to test different credit card numbers. In the page below, there are some cards listen. We tried to use the one marked in the screenshot:
https://docs.adyen.com/development-resources/test-cards/test-card-numbers#test-3d-secure-2-authentication

But there's one issue: CVC is always mandatory in the dropin, and I could not find any option to make the dropin recognise when the CVC is mandatory or not, or at least make it not mandatory at all. The only option I found is hideCVC, but that's not what we intend to do.

To Reproduce
Steps to reproduce the behavior:

Set up the dropin as you would normally do.
Fill the fields with the previously mentioned test credit card.
Try to submit the dropin.
Verify that the CVC field is marked as incorrect

Expected behavior
The dropin should recognise when the CVC is required or not, or at least allow to instantiate the dropin with the CVC not mandatory at front-end level.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

iOS
chrome
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Hi lluishi93,
Can you tell me what version of adyen-web you are using?
Because we cannot recreate this issue. This is what we see when we try to pay with this maestro number:

This was tested with v3.6.1, v3.13.1 & v3.14.1

@sponglord we're using the npm package with the following version:

How did you set the CVC to optional?

Could it be related with the supported card types? ATM, we do not provide any array with supported cards, and according to documentation:

Indeed - you are not saying that you support maestro. Therefore we cannot recognise the brand and make decisions about whether the cvc is optional or not.
What happens in this case is we say: "we don't recognise the card brand from this number, but if it validates correctly we'll allow the shopper to try and use it". But then of course we treat it as a "regular" credit card (needs date & cvc)

Ok, so we can close this for now then! TY

repo: Adyen/adyen-web


text:
Fixing tests and merging to default branch

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #

🚢

authors: Rostifar franciscojavierarceo

repo: Affirm/feast


text:
Suggestion : can you increase the size of the board?

The size of the board is small, can you increase the size? So we can work with larger PCB.

Hi, I plan to in the future.
I'm also planning to use grub screws instead of motherboard standoffs as this should allow boards to overhang without being obstructed.
Stay tuned to see what I get up to

authors: AfterEarthLTD Habibjackson

repo: AfterEarthLTD/Solder-Reflow-Plate


text:
Fix : fixed tiny mistake

Core => Chore

gitmessage.txt is English and the other is Korean now, which is opposite.
Reviewer should solve it and merge it.

repo: AgainIoT/Open-Set-Go


text:
Upgrade SDK version to the latest release

It seems the python SDK is currently stuck on SDK version 3.1.2 while the latest release of the c++ SDK is 4.1.1.
That is quite a gap and i would appreciate it if the SDK could be updated to include all or at least some of the more recently added functionalities.
I would try my hand at porting the SDK myself but unfortunately i have no experience with c++ or the porting method used.
Thanks for the hard work.

Could someone provide some info on this?

repo: AgoraIO-Community/Agora-Python-SDK


text:
Callback for when audio recording starts

Currently, when we start recording audio there is no callback function which allows us to retrieve the time, or do some other operations.
Having this callback is an essential component to the audio recording operation.
Thanks, Elijah

not sure i understand. the recording should start right away once you call the api

If a user joins muted it doesn't start recording till the first audio frame. There's an event for that, you can start recording on the first occurrence

Thanks guys :)

authors: elijahleinkram fareesh plutoless

repo: AgoraIO/Agora-Flutter-SDK


text:
Window build - agora-iris.log error with permission denied

Describe the bug
Permission denied of logging when build the installer of window
To Reproduce
Steps to reproduce the behavior:

Create window build with: flutter pub run msix:create
Installing the build on window
The error of permission denied as image attached

Thanks for reporting, may I ask is it causing crashes?

Yes, it it crashing with the logs above.

@littleGnAl Could you please help to look into this issue soon, i am still waiting for releasing the window app. Thank you so much.

I am so sorry that I have no experience publishing the Windows app, but I just wonder why the AgoraRtcScreenSharing.exe has been executed directly.

@littleGnAl The error occurs because we do NOT have permission for iris-logger on the WorkSpace folder when installing. Could we change this logs to others path? Thank you.

@littleGnAl Could you pls help on this one?

Sorry for the late response, we are working on this fix.

After a little dig into the msix package, the AppxManifest.xml file will be created by default, and the Executable value will be set the first .exe file name https://github.com/YehudaKremer/msix/blob/f003af4ead7f536259a44985232412d45ce37714/lib/src/configuration.dart#L250, it will be set to AgoraRtcScreenSharing.exe in this case.
So if you are not using the screen sharing feature, you can chagne the Executable value to the actual .exe file name should be worked for this case.

and maybe this link is helpful
https://github.com/YehudaKremer/msix/issues/120

@littleGnAl Thank so much for your information, the app is using the Share Screen feature.

This should be fixed on agora_rtc_engine: ^6.0.0.

authors: ThangD littleGnAl


text:
Facebook Provider problem

Local logins work without a problem. I tried to add facebook provider.
Pressing Facebook button at /account/login page redirects to facebook and after successful authentication, it returns to my authority server with an url like this:
https://myserver/?code=somecode&state=somestate
and this page gives a 404 error without returning to the calling client page.
I tried almost any option in the provider setup page. My last configuration is like this:

The callback url configured on facebook should end with /signin-{providerId} for exemple https://myserver/signin-facebook

I tried /signin-facebook also. When it is like that, after successful login within facebook, it returns to my server like this:
https://myserver/signin-facebook?code=somecode&state=somestate and that page shows a 404 error without redirecting to calling client page.

And your provider id is facebook right ? Put the log in debug mode and share it please.

Yes:

How to put the log in debug mode?

in the appsettings.json :
"Serilog": {
    "LevelSwitches": {
      "$controlSwitch": "Debug"
    },

Do you see the 404 in logs ?

No log containing 404 text.
The 302 responded page gives the 404 error:
https://auth.bebyaz.com/External/Challenge?provider=facebook&returnUrl=%2Fconnect%2Fauthorize%2Fcallback%3Fclient_id%3Dtheidserveradmin%26redirect_uri%3Dhttps%253A%252F%252Fauth.bebyaz.com%252Fauthentication%252Flogin-callback%26response_type%3Dcode%26scope%3Dopenid%2520profile%2520openid%2520profile%2520theidserveradminapi%26state%3D59363698e69c4a5ca1bcc50d474eabfd%26code_challenge%3Dau1VNsUJ4unfM0SS77szPXyHUFC9lPt25uQJ7d9qClY%26code_challenge_method%3DS256%26response_mode%3Dquery
you can also try it with this url:
https://auth.bebyaz.com/account/login

The problem was fixed. I added these to the appsettings.json:
"CustomEntriesOfStringArray": {
"token_endpoint_auth_signing_alg_values_supported": [
"RS256",
"ES256",
"ES384",
"ES512",
"PS256",
"PS384",
"PS512",
"RS384",
"RS512"
]
"AdditionalSigningKeyType": {
"RS384": {
"SigningAlgorithm": "RS384"
},
"RS512": {
"SigningAlgorithm": "RS512"
},
"PS256": {
"SigningAlgorithm": "PS256"
},
"PS384": {
"SigningAlgorithm": "PS384"
},
"PS512": {
"SigningAlgorithm": "PS512"
},
"ES256": {
"SigningAlgorithm": "ES256"
},
"ES384": {
"SigningAlgorithm": "ES384"
},
"ES512": {
"SigningAlgorithm": "ES512"
}
}

No 404 in logs. Now when I pressed the Facebook button within the Admin UI application, it works. But if the login page is a result of an OWIN client request. It doesn't returns to that client and shows 404 page. Local Login works for that Owin client.

I guess the host is wrong then, double check it, it's looks like it do not return to your TheIdServer instance. The callback url should callback your identity server. Then the identity server callback to the app.

404 error is showing in TheIdServer URL page. Which callback setting do you mean? I'm try to implement this logic to an old ASP.NET MVC 4 application and here is my owin startup code:
JwtSecurityTokenHandler.DefaultInboundClaimTypeMap = new Dictionary<string, string>();
app.UseCookieAuthentication(new CookieAuthenticationOptions
{
AuthenticationType = "Cookies"
});
app.UseOpenIdConnectAuthentication(new OpenIdConnectAuthenticationOptions
{
AuthenticationType = "oidc",
UsePkce = true,
ClientId = myclientid,
ClientSecret = myclientsecret,
Authority = mytheidserver,
#if DEBUG
RedirectUri = "https://localhost:44326",
#else
RedirectUri = my web client,
#endif
ResponseType = "id_token",
//ResponseType = "code",
//ResponseMode = "query",
Scope = "openid email",
UseTokenLifetime = false,
SignInAsAuthenticationType = "Cookies",
SaveTokens = true,
});

On Facebook Developers app you need to provide the redirect URI for your client, this is the callback url. In this case the client is your TheIdServer, facebook should redirect to your TheIdServer instance not to your MVC client directly. As you don't see a 404 entry in your TheIdServer log, it means the callback do not return to your TheIdServer instance, do you agree ? If it was redirected to an unknown page of TheIdServer, the admin app should display:

I don't know why the 404 error isn't showing in the logs. I took a snapshot of the flow with burp proxy. domain starting with auth is my idserver and domain staring with batu is my mvc client:

Here is my Provider latest setup:

And my client setup:

Are you behind a proxy ? If we can not found the 404 request in the log, it's because the request is not processed by TheIdServer, do you agree ?

Only one entry containing "signin-facebook" :

https://myidserver/signin-facebook?code=somecode&state=somestate
URL gives 404 error. That means the server code can't find a matching route for this URL, isn't it?  When this happens, is there a global error handling routine in the code and does it writes to log?

And what if you try to login with facebook using TheIdServer admin ?
What's the TheIdServer version you use ?
if you use SEQ, try to filter logs with : RequestPath = '/signin-facebook'

And your MVC application should use the Hybrid grant type, not Token exchange, nor Implicit, nor Client credentials

I'm using the latest commits from this repo now. Yesterday I was using the old commits that contain a Duende version of 6.1.0-preview.4"
When I filter the log with requestpath, I'm only seeing the successful logins from TheIdServer admin two days ago:

Now I even can't login from TheIdServer admin.  My current settings is like this:
Facebook developer page contains only one redirect url: https://auth.bebyaz.com/signin-facebook
Admin->Provider:  id is "facebook", sign-in path is "/signin-facebook".
Admin->Client:  id is theidserveradmin, grant type is code. "https://auth.bebyaz.com/signin-facebook" is in the redirect uri list. Scopes contain theidserveradmin, profile, openid and email.
If only I take a snapshot of the settings when I successfully login :( But now I learn a lot in 2 days about oauth, openid, sso..etc and this is the main reason I try to install the system. Thanks for your time and quick responses.

The theidserveradmin redirect uri should be : "https://auth.bebyaz.com/authentication/login-callback" ,"/signin-facebook" should not be used as redirect path for clients.
Drop the database. it'll recreate at startup. You probably have issue with stored security keys.

I found that the IIS was filtering the request because of long query string. Adding this to web.config solved the problem:

  ...

    ...

Thank you for your support. I learned OAuth, OpenId flows and learning new things is always a joy :)

authors: aguacongas bulente

repo: Aguafrommars/TheIdServer


text:
Claude Haiku 3.5 not working with AWS Bedrock

Output is blank for Haiku 3.5 using Bedrock. Sonnet V2 had this same issue (#2120), but that one was fixed.
Issue is present in 0.62 and main branch.
aider
Aider v0.62.1.dev5+ge699968b
Model: bedrock/anthropic.claude-3-5-haiku-20241022-v1:0 with whole edit format
Git repo: none
Repo-map: disabled
Use /help <question> for help, run "aider --help" to see cmd line args
─────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
> test

Tokens: 593 sent, 0 received. Cost: $0.00059 message, $0.00059 session.
─────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
>

litellm.NotFoundError: BedrockException - Bedrock HTTPX: Unknown provider=us, model=us.anthropic.claude-3-5-haiku-20241022-v1:0

Thanks for trying aider and filing this issue.
Sorry, it would be difficult for me to help debug your bedrock configuration.

I'm labeling this issue as stale because it has been open for 2 weeks with no activity. If there are no additional comments, I will close it in 7 days.Note: A bot script made these updates to the issue.

Hi, I have the same error.
I am using the models enabled in aider:
aider --list-models bedrock/anthropic.claude-3-5

aider --model "anthropic.claude-3-5-sonnet-20241022-v2:0" --message "hi"

aider --model "bedrock/anthropic.claude-3-5-sonnet-20241022-v2:0" --message "hi"

aider --model "bedrock/anthropic.claude-3-5-sonnet-20240620-v1:0" --message "hi"

All the Claude 3.5 models are enable on Amazon Bedrock:

Maybe in LiteLLM those models are not supported yet:
https://docs.litellm.ai/docs/providers/bedrock#supported-aws-bedrock-models

Error: litellm.BadRequestError: BedrockException - b'{"message":"Invocation of model ID anthropic.claude-3-5-sonnet-20241022-v2:0 with on-demand throughput isn’t supported. Retry your request with the ID or ARN of an inference profile that contains this model."}'
Diagnosis: indicates that the specific model you are trying to invoke does not support on-demand throughput without being associated with an inference profile
Solution: Model Invocation: The model ID anthropic.claude-3-5-sonnet-20241022-v2:0 requires invocation through an inference profile. This means that you cannot directly call this model unless you are using an appropriate inference profile that includes it.
https://docs.aws.amazon.com/bedrock/latest/userguide/inference-profiles-support.html#inference-profiles-support-system
us.anthropic.claude-3-5-haiku-20241022-v1:0

Test:
aider --model "bedrock/us.anthropic.claude-3-5-sonnet-20241022-v2:0" --message "hi"

Test with haiku:
aider --model "bedrock/us.anthropic.claude-3-5-haiku-20241022-v1:0" --message "hi"

@danielorozco06 that worked for me. Thanks

authors: RenFraser danielorozco06 paul-gauthier scottyeung xerebz

repo: Aider-AI/aider


text:
Disable options

Hello!
I've been playing around with certain ideas with the options and i was wondering if there is a way to disable certain options?
Thanks,
Karan Kumar

Hi @karankumar89
I would use the render prop to render custom list items then disable them as needed. On your array of items you can add a disabled property and reference that:
 <Picky
  {...restOfYourProps}
  render={({
    style,
    isSelected,
    item,
    selectValue,
    labelKey,
    valueKey,
    multiple
  }) => {
    const cssClass = classNames({ disabled: item.disabled, selected: isSelected })
    return (

 selectValue(item)}
      >
        {/* required to select item */}
        <input type="checkbox" checked={isSelected} readOnly disabled={item.disabled} />
        {item[labelKey]}

    );
  }}
/>

The example uses: https://www.npmjs.com/package/classnames
I've not tested the above code but it should put you on the right track. I hope this helps 👍

repo: Aidurber/react-picky


text:
dev版本无限报错

requirements.txt安装了,旧版本全部删除,重新从github下的就是这样

两种报错

删掉之前pip安装的包,重新安装,也是不行

我是ubuntu20,python3.9.9, x86_64架构

这个我也配置了,就是还是403

dev 需要 python 3.10 第一个是 3.10 类型提示的新特性 第二个是 3.10 新加入的结构化模式匹配

改了3.10之后能启动了

改用http模式也不行

你反向 ws 地址没对啊，应该是
ws://127.0.0.1:port/onebot/v11/ws/
新版 adapter 不是早就改了吗，看文档

你反向 ws 地址没对啊，应该是 ws://127.0.0.1:port/onebot/v11/ws/ 新版 adapter 不是早就改了吗，看文档

终于可以了,刚才还自作聪明用v12....

authors: Ailitonia lancer-wang

repo: Ailitonia/omega-miya


text:
[Feature Request] Firework: Add the ability to specify HEX

Hello. I would like to specify not only the values from spigot but also my own HEX colors.
Because specifying like this colors:WHITE,FUCHSIA,AQUA, is not very convenient and the variability is not very presentable.
Thank you!

I'm currently working on New Year's fireworks and I noticed that some components are missing.
has_trail:
has_twinkle:
ShotAtAngle: & Motion:
It would be great to have some more parameters.

Thanks!


text:
banAdd event

Featuring Type
Handler
So, what is it about?
I want to get the ID of the banning user in the banAdd event but it is not possible now. If you add me, I want to ban protection system.
Code of Conduct

[X] I agree to follow this project's Code of Conduct

i have no idea if you mean the user who banned or the user who got banned but u can use the following for the result on banAdd event code.
To get id of user who banned someone
$getAuditLogs[$guildID;;1;22;{executor.id}]
To get id of user who got banned
$getAuditLogs[$guildID;;1;22;{target.id}]

note: $authorId in banAdd gives id of user who was banned

authors: GBlade0607 USERSATOSHI aho-emi


text:
Running on Android

I see in the documentation that it can run on Android, but the demo does not support it. I want to know more about compiling on Android, but my attempt to compile the demo here has failed.I hope to receive your help.Thank you.

UATHelper: 打包 (Android (ASTC)): Building NNEngineDemo...
UATHelper: 打包 (Android (ASTC)): [Adaptive Build] Excluded from CustomizedOpenCV unity file: CustomizedOpenCV.cpp
UATHelper: 打包 (Android (ASTC)): [Adaptive Build] Excluded from NNEngineDemo unity file: NNEngineDemo.cpp, NNEngineDemoGameModeBase.cpp
UATHelper: 打包 (Android (ASTC)): [Adaptive Build] Excluded from WebcamFacialCapture unity file: FaceDetector.cpp, FaceEstimatorAttention.cpp, WebcamFacialCapture.cpp
UATHelper: 打包 (Android (ASTC)): [Adaptive Build] Excluded from WebcamMotionCapture unity file: PoseDetector.cpp, PoseEstimator.cpp, WebcamMotionCapture.cpp
UATHelper: 打包 (Android (ASTC)): Determining max actions to execute in parallel (24 physical cores, 32 logical cores)
UATHelper: 打包 (Android (ASTC)):   Executing up to 24 processes, one per physical core
UATHelper: 打包 (Android (ASTC)): Building 2 actions with 2 processes...
UATHelper: 打包 (Android (ASTC)): [1/2] clang++ NNEngineDemo-arm64.so
UATHelper: 打包 (Android (ASTC)): ld.lld: error: undefined symbol: OrtSessionOptionsAppendExecutionProvider_Nnapi
UATHelper: 打包 (Android (ASTC)): >>> referenced by OnnxModel.cpp:49 (C:/Users\DELL\Desktop\NNEngine-Demo-ue5.0\NNEngine-Demo-ue5.0\Plugins\NNEngine\Source\OnnxRuntimeNNEngine\Private\OnnxModel.cpp:49)
PackagingResults: Error: undefined symbol: OrtSessionOptionsAppendExecutionProvider_Nnapi
UATHelper: 打包 (Android (ASTC)): >>>               C:/Users/DELL/Desktop/NNEngine-Demo-ue5.0/NNEngine-Demo-ue5.0/Plugins/NNEngine/Intermediate/Build/Android/UnrealGame/Development/OnnxRuntimeNNEngine/Module.OnnxRuntimeNNEngine.cppa8.o:(OnnxModel::OnnxModel(FString, EOnnxProvider, int, EOnnxGraphOptimizationLevel))
UATHelper: 打包 (Android (ASTC)): clang++: error: linker command failed with exit code 1 (use -v to see invocation)
PackagingResults: Error: linker command failed with exit code 1 (use -v to see invocation)
UATHelper: 打包 (Android (ASTC)): Took 11.9995843s to run UnrealBuildTool.exe, ExitCode=6
UATHelper: 打包 (Android (ASTC)): UnrealBuildTool failed. See log for more details. (C:\Users\DELL\AppData\Roaming\Unreal Engine\AutomationTool\Logs\D+Unreal+Engine+UE_5.0\UBT-NNEngineDemo-Android-Development.txt)
UATHelper: 打包 (Android (ASTC)): AutomationTool executed for 0h 0m 13s
UATHelper: 打包 (Android (ASTC)): AutomationTool exiting with ExitCode=6 (6)
UATHelper: 打包 (Android (ASTC)): Updating environment variables set by a Turnkey sub-process
UATHelper: 打包 (Android (ASTC)): The system cannot find the path specified.
UATHelper: 打包 (Android (ASTC)): The system cannot find the path specified.

repo: Akiya-Research-Institute/NNEngine-Demo


text:
💡[FEATURE]: Web application For a Project

Is your feature request related to a problem? Please describe.
I would love to add a web application for the House Price Prediction Project that was built earlier .
Describe the solution you'd like
I would be using flask as backend and manage different data pipelines that were used for data preprocessing
I would also provide a good front end which will be user-friendly
Additional context
@Akshat111111 Hey bro 👋 , Can you please assign this issue to me with level tag under GSSoC'24 so that i can work on it.

Good idea, You can proceed further.

@Akshat111111 , Thank you I'll be responding with a PR soon


text:
Lucky warning shouldn't show if you are already under the limit

Does it make sense to complain about going under max Lucky reward threshold when you're already under it?

Well there is still info on how far you are from it...  It also adds the mental annoyance that you are still under and don't buy anything =D

Also, if you're like me, and you click a bunch of cheap buildings, and suddenly you are below the threshold, I'd like to see that :). Otherwise you would only see the warning precisely on the building putting you under the threshold. Plus what @Aktanusa said.
But I see what you're getting at. In general my flow is like this: just after a reset it's better to just buy every next best upgrade as soon as you can afford them. But after a while, things slow down, and it's better to wait for the lucky threshold. If you're really annoyed by the warning you could just disable it in the first phase, and when you hit the second phase enable it again.

Closing due to inactivity

authors: Aktanusa CatsAreFluffy svschouw

repo: Aktanusa/CookieMonster


text:
[BUG] cannot continue after taking photo

Version
F-Droid IzzyOnDroid 1.4.15.56
What platform are you using?
Android
OS Version
Pixel 8 Pro, android 14, march 5 2024 update
What happened?
After a photo is taken (in manual or automatic mode), there is a popup that says:
Cannot read properties of null (reading 'unshift')
The only option to close the popup is OK. When I tap the checkmark to continue, the popup happens again. I cannot proceed.
Relevant log output
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

@xxyxxyxyx1 sorry for the latest answer. Can you tell me if this is still happening with latest version? Any chance to get a logcat once the error happens?
Thanks

@xxyxxyxyx1 ok so just run the app, reproduce the issue. Then use adb logcat to get the current logcat. content.
You can call adb logcat -c to clear the logcat before reproducing the issue.
Then share the content returned by adb logcat
Thanks

It is attached
logcat.txt

@xxyxxyxyx1 sadly i dont see anything in the log. Could you try this build https://github.com/Akylas/OSS-DocumentScanner/releases/tag/test_android? I should even receive a bug report directly

done

@xxyxxyxyx1 did you restart the app after the crash? (to ensure the log is sent)

I have caused the bug more than once.
The app does not actually crash when the bug happens. After I press "OK" on the error popup, I can continue using the app. I just cannot continue in the scanning process.

@xxyxxyxyx1 ok got the error!  But to me here the erorr is Cannot read properties of null (reading 'length') is that what you see?

@xxyxxyxyx1 ok so i made a fix where it should not give an error for you anymore. Though there is still something wrong.
Could you test that build again that i updated https://github.com/Akylas/OSS-DocumentScanner/releases/tag/test_android ? You should not see an error but i should still get a report with more details allowing to understand why it fails for you

It seems to work now with the new version. I am able to take a photo and export a PDF.
I have not observed an error including the text "(reading 'length')"
In brief testing, I found two other problems, but they seem unrelated:

in 2 out of 3 papers (A4 two-column text-only documents), the app chose only 1 paragraph on the page, instead of the whole page. The blue preview rectangle looked approximately correct when I took the pictures.
after exporting a PDF, a filename is displayed with an "open" popup. The displayed filename is wrong, starting with "primary%3A", and clicking "open" results in a file-not-found in the pdf viewer. However, the PDF is saved correctly: I can find the PDF in a file browser with the correct filename in the correct location.

@xxyxxyxyx1 it does not fail for you anymore but the issue is still there. But i got the correct bug report from you now and i can investigate it ! thanks a lot.
Could you open new issues for the other 2 with more details? I dont understand what you mean by he app chose only 1 paragraph on the page . are you talking about OCR? Please explain in details with screenshots if relevant.
And the second one maybe a screenshot of the dialog with the wrong text could help.
Thanks

Just a +1, as the same is happening for me; test_android version works fine.

@catfluoride are you seeing an error message? if so what is it? Or does it simply crash?

@catfluoride are you seeing an error message? if so what is it? Or does it simply crash?

Hi! I got an error message similar to that of OP: Cannot read properties of null ("reading 'image')

@catfluoride ok this one is a regression in latest release. I am on it. Sorry!

@catfluoride ok this one is a regression in latest release. I am on it. Sorry!

No need to apologize, man. You created a wonderful app. Thank you for your hard work!

@catfluoride should be fixed in new release (only on github for now)

@catfluoride should be fixed in new release (only on github for now)

Indeed. It works perfectly now. Thanks a lot!

authors: catfluoride farfromrefug xxyxxyxyx1

repo: Akylas/OSS-DocumentScanner


text:
Ikea Rollos:  found unknown property with key 8

see since 3.0.x upgrade the following message in log:
tradfri.0 | 2022-01-03 10:49:21.538 | warn | (13574) object was: {"0":"IKEA of Sweden","1":"FYRTUR block-out roller blind","2":"","3":"2.3.073","6":3,"7":4487,"8":0,"9":94}
-- | -- | -- | --
tradfri.0 | 2022-01-03 10:49:21.537 | warn | (13574) found unknown property with key 8

Duplicate of #704

Please update to 3.1.0

authors: AlCalzone haarchri


text:
Stop forcing response headers

Can you please provide a config so that we can turn off the manipulation of cache HTTP headers? It makes ResponseCacheAttribute useless.
https://github.com/Alachisoft/NCache/blob/b8bb6f06372c1f79397e95b3ca8612cff24eb911/SessionState/ASP.NET Core/NCacheSessionServices/NCacheSessionServices/Utilities/SessionKeyManager.cs#L56

The best workaround I could come up with is something like this in Startup:
app.Use((context, next) => {
    context.Response.OnStarting(() => {
        if (context.Items.Keys.Contains("_ncache-header-reset")) {
            var (cacheControl, pragma, expires) = ((StringValues, StringValues, StringValues))context.Items["_ncache-header-reset"];
            context.Response.Headers["cache-control"] = cacheControl;
            context.Response.Headers["pragma"] = pragma;
            context.Response.Headers["expires"] = expires;
        }
        return System.Threading.Tasks.Task.CompletedTask;
    });
    return next.Invoke();
});
app.UseNCacheSession();
app.Use((context, next) => {
    context.Response.OnStarting(() => {
        context.Items["_ncache-header-reset"] = (context.Response.Headers["cache-control"], context.Response.Headers["pragma"], context.Response.Headers["expires"]);
        return System.Threading.Tasks.Task.CompletedTask;
    });
    return next.Invoke();
});

@bradlis7, I have forwarded this request to our Engineering team. They will review this on their end. In the meantime, please try the workaround that you have come up with and see if this works.

repo: Alachisoft/NCache


text:
Document more workflows for vcs packages

Initially this PR removed aur-vercmp-devel and introduced documentation changes that were biased towards a repo-based workflow for vcs packages.
Since these are actually two actions, the removal was split into PR #622.

See #961 for a new idea on this topic. If we store the "seen" upstream git commits somewhere, they can be queried them via APIs (github, gitlab) instead of running aur-srcver. This require some effort though and is specific for a small subset of VCS packages...
The "seen" commit would be more informative if it is linked to the built package. Apart from storing pkgname-pkgver <-> upstream commit manually, this could be part of the package itself: see https://lists.archlinux.org/pipermail/pacman-dev/2022-February/025511.html for a proposal on this regard.

authors: AladW rafasc

repo: AladW/aurutils

license: isc


text:
Signup issue

Page Id and Page name is not being save upon signup with Facebook

Sorry posted on the wrong repo

repo: Alamofire/Alamofire


text:
Add AlamofireObjectMapper to README

Add info about AlamofireObjectMapper to new Third-Party Libraries section

Hi @phatmann...thanks for putting this together. Unfortunately, that section of the README is only for promoting libraries officially supported by the AlamofireSF. We want to avoid promoting libraries in which we don't have any control over.
Thanks anyways! 🍻

authors: cnoon phatmann


text:
[5.10] Initial Requirement 11 Lofi-Prototyping

Description: Prototyping based on Initial Requirement 11.
Goals: Create a Prototype based on the Requirement.
Success or Completion Criteria: Prototype Figma.
Allocated Time: 2 days
Start Date: 31/08/23
End Date: 01/09/23
Main Task :  Create lo-fi prototypes/wireframes based for the initial requirements

Description: Update prototype to allow employees to only indicate preference and inavailability. Any other time slots that are not indicated will automatically be implied as "available"
Changes to success or completion criteria?: n
Start Date: 07/09/23
End Date: dd/mm/yy, hhmm

Issue Comments
Description
Close issue for first draft of the prototyping
Changes to success or completion criteria? y/n (if y, describe)
n
Start Date: 31/08/23, 1000
End Date: 01/09/23, 2100


text:
Whats the issue?

https://github.com/AleoHQ/snarkOS/blob/1d942fdc3924b401f1381f73624301b139df3067/node/rest/src/lib.rs#L115

duplicate of https://github.com/AleoHQ/snarkOS/issues/3053

me too

Linux iZrj94x9iunw7yki5u7tioZ 5.15.0-91-generic #101-Ubuntu SMP Tue Nov 14 13:30:08 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Closing as duplicate.

authors: BeckerFelix demoso idontusehax1 ljedrz

repo: AleoHQ/snarkOS


text:
[Fix] Add limits on program struct definitions

Motivation
Implements a limit on the number of struct definitions in a given program per SnarkOS issue 3141. This limit of 31 in this PR is up for debate. We also may want implement limits on number of closures number of characters.
(cc @vicsn)
Test Plan

(Write your test plan here)
Related PRs

(Link your related PRs here)

Superseded by https://github.com/AleoHQ/snarkVM/pull/2384 - thank you for the start

authors: christianwwwwwwww vicsn

repo: AleoHQ/snarkVM


text:
fix the updated hook with several v-draggables: use a WeakMap

Hi again,
#4 has some issues when using multiple v-draggables
binding.dir.updated is global for all v-draggables, so this wasnt the right thing ;)
i changed it, so it is using a WeakMap on the element

Sorry for the delay in response. Thanks again for your contribution, good idea with the WeakMap. Released this as 3.0.1.

Sorry for the delay in response. Thanks again for your contribution, good idea with the WeakMap. Released this as 3.0.1.

authors: Aletheios schelmo

repo: Aletheios/v-drag-drop


text:
When pasting images, editing of text becomes very slow

Description
I am mostly using Trumbowyg on macOS Safari an have enabled the respective plugins to allow pasting images into the editor which are stored as base64 HTMLs.
However, when pasting an image, the text editing becomes very laggy and slow.
How to reproduce?
You can try it yourself on the trumbowyg demos webpage where after an image was pasted, text editing becomes very slow.
Any chance to speed this up, so it is still realistically usable for HTMLs with pasted images?

Commenting to note that I have experienced this too, and limiting the size of the images down to a quite small 500px maximum on either side has helped a lot, though it will still be an issue if your users fill the text box with a bunch of images. This can be done in the trumbowyg.base64.js file down around line 143 within the if (isValidImage(e.target.result)) { function, something like:
var image = new Image();
image.src = e.target.result;
image.onload = function() {   if(this.width > 500) {   alert('Image is too large. Sorry! Files pasted in the editor have a limit of 500px on either side to avoid high load times.');   return false; } else if(this.height > 500) {   alert('Image is too large. Sorry! Files pasted in the editor have a limit of 500px on either side to avoid high load times.');   return false; } else {   trumbowyg.execCmd('insertImage', fReader.result, false, true);   $(['img[src="', fReader.result, '"]:not([alt])'].join(''), trumbowyg.$box).attr('alt', values.alt);   trumbowyg.closeModal();   } };
That being said, the issue was not apparent until we launched with the editor and it's definitely a pain point!

Is it only on Safari?

From my perception this is not only a Safari issue but also seen in Chrome: it seems solely be an issue with slightly larger pictures. Question is why it slows it so much down, as while typing the image is simply "moved" and should ideally not consume so much CPU power.

Hi.
Faced with the same problem. The problem appears when pasting an image from the clipboard. An example image is attached. The problem also manifests itself in the editor on the site: https://alex-d.github.io/Trumbowyg/
Adding the "max-height: 300px;" style helps. The brakes continue, but the editor is working.

authors: lithxe mkrecek234 shme-adm

repo: Alex-D/Trumbowyg