text:
Prevent datagram fragmentation during first registration

In case the data size of a registration exceeds the 1510 bytes, the datagram in the IPv6 stack will be fragmented. If the client device has not communicated with its destination device before (in this case the server), fragments will be dropped until the address resolution is finished. See also [https://github.com/ARMmbed/sal-stack-nanostack/issues/11#issuecomment-215640678].
The CoAP retry by re-sending the complete registration datagram will recover from the fragment drop. I would propose to avoid fragmentation and split up datagram in the multiple CoAP messages.

Hi, This needs to be checked on end-to-end level because mbed-client supports Blockwise CoAP message transfer but this is a unique case where the initial registration message is fragmented, so it can be issue from server side as well if it is not handling this use case.
We will check this and get back with fix proposal.

Off-topic from mbed-client, but I'll respond to the initial report here.
To pin down the issue a little tighter - it's not a client->destination association issue. It's more precisely:
"If any of the nodes on the path from the client to the destination have not communicated with their next-hop node before, fragments may be dropped until their address resolution for the next hop is finished."
So you shouldn't see this with every node start up on a 6LoWPAN mesh. All nodes after the first should find the path to the common server clear, as all nodes across the mesh to the border router have definitely got their links up, and the border router will have a path to the server.
Multiple nodes on Ethernet would show the issue, as each of them would need to do their own resolution.
In that case, if the server's not local, then if you could ensure the Ethernet segment's default router includes the Source Link-Layer Address option in its Router Advertisements, you could avoid the need for resolution.
DTLS would also side-step the issue, as the DTLS handshake would be small packets.

ARM Internal Ref: IOTCLT-727

Hi Ed,
We have published fixes for blockwise operation. Could you try is your use case working any better now? Fixes are in yotta so yotta up should fetch the changes.
Thanks,
Antti

Hello,
These are the versions I have used to test:
|_ mbed-client 1.12.2
| |_ mbed-client-c 2.7.1 yotta_modules\mbed-client-c
| | _ nanostack-libservice 3.5.1 yotta_modules\nanostack-libservice
| |_ mbed-trace 1.1.3 yotta_modules\mbed-trace
| |_ mbed-client-mbed-os 3.0.0 yotta_modules\mbed-client-mbed-os
But unfortunately I still see fragmentation on the Ethernet layer and not in the CoAP layer. Do I need to enable the blockwise implementation in a config file?

Hi, blockwise feature is disabled by default so you need to add following flag in config.json.
"coap_max_blockwise_payload_size": 1024
Acceptable values for the coap_max_blockwise_payload_size flag are:
0, 16, 32, 64, 128, 256, 512 and 1024. Value 0 means that the feature is not used.
https://docs.mbed.com/docs/mbed-client-guide/en/latest/Introduction/

Hi Ed,
Is this duplicate to one you created in mbed-client-c repository?
https://github.com/ARMmbed/mbed-client-c/issues/52
If so maybe we can close this one.

Indirectly yes. The mbed-client code of LWM2M is not responsible for the "invalid payload bytes #52".
I assume that if a fix is required in CoAP, this is will be done in mbed-client-c and not in the mbed-client one. That's why I created a new issue in this repository. Also to keep issue reports clean and not stacking of issues in one report. Thanks.

source: gharchive/issue

authors: EduardPon anttiylitokola ciarmcom kjbracey-arm yogpan01

repo: ARMmbed/mbed-client

license_type: permissive

license_source: github-api


text:
Fix memory leak in deserialize_object_instances

til might be left unreclaimed if (TYPE_OBJECT_INSTANCE != til->_type)

Hi,
Thanks for your finding. Our CI does not handle building if change comes through the fork so I made a new PR for this https://github.com/ARMmbed/mbed-client/pull/325.

source: gharchive/pull-request


text:
unknown type name 'mbedtls_ecp_restart_ctx'

Description

Type: Bug
Priority: Minor

mbed TLS build:
Version: 2.14.0
Configuration: MBEDTLS_ECP_ALT defined and MBEDTLS_ECP_RESTARTABLE not defined
Expected behavior
Compiles without error.
Actual behavior
error: unknown type name 'mbedtls_ecp_restart_ctx'
Steps to reproduce

Define MBEDTLS_ECP_ALT and do not define MBEDTLS_ECP_RESTARTABLE in config.
Compile library/ecdh.c.

I could simply add typedef void mbedtls_ecp_restart_ctx; to my config to fix the above issue, but is that expected?

As you can see from my previous comment, mbedtls_ecp_restart_ctx  should be defined even if MBEDTLS_ECP_RESTARTABLE is not defined, as it is used in other API, such as ecdh and ecdsa. ecp_alt.h is used to define your alternative implementation structures. Since it is not predictable how every platform will need to define their contexts.
Having said that, I agree that in matters of mbedtls_ecp_restart_ctx, when MBEDTLS_ECP_RESTARTABLE is not defined, it is perhaps not the best design, and we may consider moving the definition referenced earlier, to a location independent on MBEDTLS_ECP_ALT.

Hi @jwhui and thanks for your report! Sorry for replying so late. After analyzing the issue, I fully agree with my colleague @RonEld:

currently you have to add something like typedef void mbedtls_ecp_restart_ctx; somewhere and the ideal place is your ecp_alt.h
however, we acknowledge that's a bit of an annoyance, and I can confirm that's one we didn't fully anticipate when designing restartable ECC
so I think we'll consider removing the requirement for you to define that type in your ecp_alt.h, probably by moving its definition out of the the MBEDTLS_ECP_ALT guards as @RonEld suggests, as it looks like the simplest way to solve the issue.

I'm relabeling as a bug, as I think that's a regression: existing ecp_alt code that was working and correct with Mbed TLS 2.13.0 will no longer build with Mbed TLS 2.14.0 (until it's modified to add the type definition). This is a break of our compatibility promises, it should not have happened, and should be fixed.

@mpg I am not entirely convinced this is a bug.
One can have an alternative implementation for the restartable feature as well, in which case, mbedtls_ecp_restart_ctx will be defined differently.

@RonEld I think we ruled out alternative implementations of the restartable feature: https://github.com/ARMmbed/mbedtls/blob/development/include/mbedtls/check_config.h#L111 - when we did that, we should have moved the type out of the ALT guard, as it's not ALTable.

@mpg thank for the reminder! ( I believe I even made that change:) )

@mpg @RonEld, thanks for looking into this!

authors: RonEld jwhui mpg

repo: ARMmbed/mbedtls

license_source: bigquery


text:
Fix uninitialized variable in x509_crt (#2392 resubmit)

This patch fixes an issue we encountered with more stringent compiler
warnings.  The signature_is_good variable has a possibility of being
used uninitialized.  This patch moves the use of the variable to a
place where it cannot be used while uninitialized.
Signed-off-by: Andy Gross andy.gross@linaro.org
Notes:

Pull requests cannot be accepted until:

The submitter has accepted the online agreement here with a click through
or for companies or those that do not wish to create an mbed account, a slightly different agreement can be found here
The PR follows the mbed TLS coding standards

This is just a template, so feel free to use/remove the unnecessary things

Description
A few sentences describing the overall goals of the pull request's commits.
Status
READY/IN DEVELOPMENT/HOLD
Requires Backporting
When there is a bug fix, it should be backported to all maintained and supported branches.
Changes do not have to be backported if:

This PR is a new feature\enhancement
This PR contains changes in the API. If this is true, and there is a need for the fix to be backported, the fix should be handled differently in the legacy branch

Yes | NO
Which branch?
Migrations
If there is any API change, what's the incentive and logic for it.
YES | NO
Additional comments
Any additional information that could be of interest
Todos

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

Steps to test or reproduce
Outline the steps to test or reproduce the PR here.

This is resubmit of https://github.com/ARMmbed/mbedtls/pull/2392, implementing following change request there (https://github.com/ARMmbed/mbedtls/pull/2392#pullrequestreview-202535560):

Could you remove the double space, and perhaps say something like "Improve code clarity in X.509 module, removing false-positive uninitialized variable warnings on some toolchains."? "Fix compiler warning" is a bit too generic, and it doesn't entail whether the warning was justified and potentially hinting at a serious bug.

Comparing to the original patch, only changes to the ChangeLog file content was made. Everything else: the actual code patch, commit message, sign-off, author - stays the same.
@hanno-arm, @mpg, @RonEld (reviewers of #2392), could you please have a look?

@pfalcon Thanks for opening this new PR. Could you check if the warning also happens in our LTS branches (currently mbedtls-2.16 and mbedtls-2.7)? If it does, we'll need PRs backporting the fixes to those branches as well. Thanks!

@mpg, @k-stachowiak: Thanks for the review!

Could you check if the warning also happens in our LTS branches (currently mbedtls-2.16 and mbedtls-2.7)?

This does happen in 2.16, that's actually which version it was originally prepared for. I submitted https://github.com/ARMmbed/mbedtls/pull/2813 for that, the only change is moving ChangeLog entry to the appropriate section.
While doing so, it occurred to me that this PR contained ChangeLog entry in an old section (circa 2.16.0), so I moved to the next-version section here (while rebasing on the latest development). That dismissed your reviews, sorry about that, hopefully we can recover them soon ;-).
This patch doesn't apply to 2.7 - the code considerably different there.

This has two approving review, passes CI, and so does the only relevant backport, so labeling "ready for merge".

authors: mpg pfalcon


text:
Which flight software is used to flight test?

@rkintada I'm curious which flight controller you use to test this repo in flight with the Snapdragon Flight? Do you use PX4 or some other? Do you use additional sensors? Some prescribed usage in the README might benefit user.

@Seanmatthews for the basic testing we use the bench-testing with hand-motions.  This is also tested with Qualcomm's Snapdragon Navigator Flight Controller( the flight controller is in the process of being released on Qualcomm Developer Network soon ) with a recommendation of the Flight Platform.   The sensors are pretty standard ones we use( IMU(MPU9x50), Pressure Sensor(BMP280), Optic Flow- OV7251( not when using VISLAM), external GPS).

authors: Seanmatthews rkintada

repo: ATLFlight/ros-examples


text:
帮忙看着这个报错，这个路径下没有node.js文件

failed to load config from /Users/liuguisheng/code/github/sugar-blog/packages/blogpress/.vitepress/config.ts
failed to start server. error:
Error: Cannot find module '/Users/liuguisheng/code/github/sugar-blog/packages/blogpress/node_modules/@sugarat/theme/node.js'
at createEsmNotFoundErr (node:internal/modules/cjs/loader:960:15)
at finalizeEsmResolution (node:internal/modules/cjs/loader:953:15)
at resolveExports (node:internal/modules/cjs/loader:482:14)
at Function.Module._findPath (node:internal/modules/cjs/loader:522:31)
at Function.Module._resolveFilename (node:internal/modules/cjs/loader:919:27)
at Function.Module._load (node:internal/modules/cjs/loader:778:27)
at Module.require (node:internal/modules/cjs/loader:1005:19)
at require (node:internal/modules/cjs/helpers:102:18)
at Object. (/Users/liuguisheng/code/github/sugar-blog/packages/blogpress/.vitepress/config.ts:35:19)
at Module._compile (node:internal/modules/cjs/loader:1103:14)

辛苦看一下readme先构建主题包

issue 我先关了喔，如果还有问题可以再打开

repo: ATQQ/sugar-blog

license: MIT


text:
Update README

https://nus-cs2103-ay1920s1.github.io/website/schedule/week7/project.html#1-update-project-website-aboutus-contactus-readme

Closed by #10


text:
Implement parser and model for done command

When a (person's) visit is marked as done, it will remove the existing visit for the person, and update the last visited date for the person.
Implementation

[X] Implement parser and model for done command
[ ] Perform unit testing
[ ] Replace invocation of edit with delete command
[ ] Update last visited value using relevant methods

Codecov Report

Merging #50 (714c6f9) into master (09b2b0a) will decrease coverage by 1.53%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master      #50      +/-   ##
============================================
- Coverage     72.41%   70.87%   -1.54%
  Complexity      421      421
============================================
  Files            73       75       +2
  Lines          1294     1322      +28
  Branches        131      134       +3
============================================
  Hits            937      937
- Misses          324      352      +28
  Partials         33       33

Impacted Files
Coverage Δ

...main/java/seedu/address/commons/core/Messages.java
0.00% <ø> (ø)

...java/seedu/address/logic/commands/DoneCommand.java
0.00% <0.00%> (ø)

.../seedu/address/logic/parser/AddressBookParser.java
84.21% <0.00%> (-4.68%)
:arrow_down:

.../seedu/address/logic/parser/DoneCommandParser.java
0.00% <0.00%> (ø)

...c/main/java/seedu/address/model/person/Person.java
95.74% <0.00%> (-2.09%)
:arrow_down:

...rc/main/java/seedu/address/model/person/Visit.java
77.77% <0.00%> (-9.73%)
:arrow_down:

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 09b2b0a...714c6f9. Read the comment docs.

Addresses #43 but still need to incorporate delete command and last visited field.

authors: Chen-Yiyang codecov-commenter


text:
Dates: fix bugs and add tests

Strings which match the date format but are not actual dates (eg.2020-02-32) should not be recognized as valid dates.
Closes #88, #97, #105.

A bug in the logic caused the next occurence of recurring dates to always be the date in the future when the reference date was the same date as the recurring date.
Closes #92.

Codecov Report

Merging #150 (b46fbd8) into master (2c92ae9) will increase coverage by 0.18%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master     #150      +/-   ##
============================================
+ Coverage     65.52%   65.71%   +0.18%
- Complexity      614      615       +1
============================================
  Files            95       95
  Lines          2080     2091      +11
  Branches        238      239       +1
============================================
+ Hits           1363     1374      +11
  Misses          602      602
  Partials        115      115

Impacted Files
Coverage Δ

src/main/java/socialite/model/person/Date.java
77.08% <100.00%> (+6.81%)
: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 2c92ae9...b46fbd8. Read the comment docs.

I think it would be better to merge this in first, and I'll work on the UI fix in another PR. Thanks for the reviews!


text:
Debug Sample data

Previously all classes share the same StudentList object and several
Person share the same Class object, which resulting in several classes
or students being updated together.

Codecov Report

Merging #57 (14e499b) into master (6df0039) will decrease coverage by 0.04%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master      #57      +/-   ##
============================================
- Coverage     47.68%   47.64%   -0.05%
  Complexity      432      432
============================================
  Files           102      102
  Lines          2099     2101       +2
  Branches        233      233
============================================
  Hits           1001     1001
- Misses         1056     1058       +2
  Partials         42       42

Impacted Files
Coverage Δ

.../java/seedu/address/model/util/SampleDataUtil.java
31.25% <0.00%> (-2.09%)
:arrow_down:

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 6df0039...14e499b. Read the comment docs.

authors: NoraYUuu codecov-commenter


text:
[PE-D][Tester A] No way to view marked tasks.

As soon as I mark a task as completed, it disappears from the task list. There's no way to view marked tasks (in case I accidentally marked it as completed and I want to unmark now).
Also, I think if there's no way to view marked tasks, I'm not sure if there's a way to unmark a task (since I can't view them). This means that potentially, the command unmark is not very useful to this application.
Perhaps you can consider allowing users to view marked tasks as well.
Example of Bug
Before marking task as done

After marking task as done

Labels: severity.High type.FunctionalityBug
original: Devanshshah1309/ped#1

The task list shows only INCOMPLETE tasks. Once a task has been marked (either by the command or the by ticking the white box), it will be completed and disappears from the task panel as it is NOT incomplete.
Proposed Changes

Add message to indicate that task has been marked as completed and needs to be viewed via the task list -a command
Update UG

authors: KIANSEONG nus-pe-script


text:
Improve code coverage and update PPP

Added tests for RecordList and AddressBookParser which were previously lacking, in order to improve code coverage.
Updated PPP.

Codecov Report

Merging #153 (2054d62) into master (44d7d4c) will increase coverage by 1.01%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master     #153      +/-   ##
============================================
+ Coverage     74.20%   75.22%   +1.01%
- Complexity      645      661      +16
============================================
  Files           100      100
  Lines          1989     1994       +5
  Branches        236      238       +2
============================================
+ Hits           1476     1500      +24
+ Misses          424      406      -18
+ Partials         89       88       -1

Impacted Files
Coverage Δ

...in/java/seedu/address/model/record/RecordList.java
88.57% <100.00%> (+25.23%)
:arrow_up:

.../seedu/address/logic/parser/AddressBookParser.java
100.00% <0.00%> (+34.61%)
:arrow_up:

...del/person/exceptions/RecordNotFoundException.java
100.00% <0.00%> (+100.00%)
:arrow_up:

...el/person/exceptions/DuplicateRecordException.java
100.00% <0.00%> (+100.00%)
:arrow_up:

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

authors: LJXSean codecov-commenter


text:
[UI] Design and Update UI for Classify application

[x] Design and share new UI
[x] Implement new UI

Resolved in #165 and #170


text:
Modify delete command

Modify delete command so that user can remove multiple internships

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@77a32bf). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master      #25   +/-   ##
=========================================
  Coverage          ?   72.92%
  Complexity        ?      412
=========================================
  Files             ?       72
  Lines             ?     1267
  Branches          ?      130
=========================================
  Hits              ?      924
  Misses            ?      310
  Partials          ?       33

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 sugiyem


text:
Migrate doc from Google Docs to Markdown

As stated in the prompt.
I also updated the design of the documentation so that we will be using a modal design.

@tangyuantyy do you mind having a look at this and merge this PR?

I think we may need to fix the space issue in Markdown styling.


text:
Command parser accepts suffix to command words causing potential confusion

For example, "/notes" are accepted as execute note command and add note with the text "s". If user does a typo to the command word suffix, the application would not be able to flag and the application would not work too unless user realizes the typo.
@AY2324S2-CS2103-F15-3/developers

Is this considered a bug under v1.4?

[ ] Verify

Out of scope?


text:
misc: Tutorial add command

DO NOT MERGE

Closing this PR as it doesn't need to be merged.

authors: ForAeons tituschewxj


text:
Add Joshua's photo

Fixes #16

LGTM!


text:
Deprecated selector in scroll-searcher/styles/scroll-searcher.less

In scroll-searcher/styles/scroll-searcher.less:
Starting from Atom v1.13.0, the contents of atom-text-editor elements are no longer encapsulated within a shadow DOM boundary. This means you should stop using :host and ::shadow pseudo-selectors, and prepend all your syntax selectors with syntax--. To prevent breakage with existing style sheets, Atom will automatically upgrade the following selectors:

atom-text-editor .scroll-searcher, atom-text-editor::shadow .scroll-searcher => atom-text-editor .scroll-searcher, atom-text-editor.editor .scroll-searcher

atom-text-editor .scroll-marker, atom-text-editor::shadow .scroll-marker => atom-text-editor .scroll-marker, atom-text-editor.editor .scroll-marker

Automatic translation of selectors will be removed in a few release cycles to minimize startup time. Please, make sure to upgrade the above selectors as soon as possible.

Scroll searcher has been updated. Due to recent shadow dom update, this new version will only work with Atom >= v1.13.0.
https://atom.io/packages/scroll-searcher

license: mit


text:
Can't get it to run

Thanks for creating this.
I'm new to python. What is that I have to do to run it? I've tried the following:
python app.py
Traceback (most recent call last):
  File "crypto-signal-master/app.py", line 1, in <module>
    from bittrex import Bittrex
  File "/Users/bernat/Desktop/crypto-signal-master/bittrex.py", line 18, in <module>
    import requests
ImportError: No module named requests

python crypto-signal-master
/usr/bin/python: can't find '__main__' module in 'crypto-signal-master'

Create a virtual env by doing "virtualenv env" and then do "source env/bin/activate". Then do "pip install requests". That should take care of it. It's also added to the requirements for the project

Would this work on Windows?

"pip install -r requirements.txt" also works :)

Yes, it should work on Mac, Windows, or Linux. Anywhere that you can run Python you should be able to run this

Yes
On Thu, Sep 21, 2017 at 10:16 AM NerosBerg notifications@github.com wrote:

Would this work on Windows?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/AbenezerMamo/crypto-signal/issues/5#issuecomment-331222963,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABwVIaeZgH_kABfS5Qgsfn-mmISVp9UVks5skpn9gaJpZM4Pfm1o
.

--
Sent from my iPhone 7

I’m new to this too. When you say creat a virtual environment what do mean exactly? I have VMware installed on my windows computer. Are you saying we should create a VE on there. I honestly have no clue to know where to start but I’m very much interested in knowing where to start and get this running. I’d appreciate your help, thanks in advance.

@adbitcoin http://docs.python-guide.org/en/latest/dev/virtualenvs/

authors: AbenezerMamo NerosBerg adbitcoin bernatfortet rmarchandTen24

repo: AbenezerMamo/crypto-signal


text:
Barchart with LineData incorrect.

The lineData is incorrect in the Barchart. It doesn't confirm my data set that I"m using.
As you can see in the screenshot the same data set is used but the line data is above the bars.
To Reproduce
Here the example code that I used:
`  const barData = [
{ value: 250, label: 'M' },
{ value: 500, label: 'T' },
{ value: 745, label: 'W' },
{ value: 320, label: 'T' },
{ value: 600, label: 'F', frontColor: '#177AD5' },
{ value: 256, label: 'S' },
{ value: 0, label: 'S' }
]
<BarChart
barWidth={14}
barBorderRadius={4}
noOfSections={3}
maxValue={750}
stepValue={750}
height={238}
yAxisLabelTexts={['0', '5km', '10km', '15km']}
frontColor={themeColors['tint']}
data={barData}
yAxisThickness={0}
xAxisThickness={1}
isAnimated={true}
hideRules={true}
hideYAxisText={false}
lineData={barData}
roundedTop={true}
showLine={true}
spacing={28}
lineConfig={{
color: 'black',
thickness: 1,
curved: true,
isAnimated: true,
delay: 10,
hideDataPoints: true,
}}
xAxisLabelTextStyle={{
color: themeColors['text-primary'],
fontSize: graphTextX,
textAlign: 'center'
}} // text X style
/>
`
Just to clarify. The themeColors['text-primary'] is just a color
The result should be that the lines are at the same point as the Bar. And not above it.

Hi @DannyBloky
This has been fixed in the latest version of the library. Please update to the latest version.

authors: Abhinandan-Kushwaha DannyBloky


text:
Fix typo

implementatios -> implementations

Thankios!

authors: jbo-ableton nakajima

repo: AbletonAG/atria


text:
#19 ColumnImplicits and StructFieldImplicits

Closes #19
Closes #20

Do we have tests for these?

There aren't in Enceladus, but I can add some of course

There aren't in Enceladus, but I can add some of course

Please

Could someone check that he checked behavior in the tests is correct?

authors: AdrianOlosutean Zejnilovic

repo: AbsaOSS/spark-commons


text:
Adsk Contrib - RB-2.0 release

Cherry-pick all commits from master to update branch to V2.0 release.
Add UNIFORM_FLOAT3, setStrictParsingEnabled to PyOpenColorIO (#1280) Michael Dolan michdolan@gmail.com 73be29a4bacfd560193514af496c0cab6673f77e
Restructure PyOpenColorIO, complete PyGpuShaderDesc (#1275) Michael Dolan michdolan@gmail.com 081b8de4d2f005b12ef7183d21744c2259bcb0c4
Adsk Contrib - More python tests and bindings fixes (#1274) 574ccd856eba78d493b56b4e63d8beb24ef8fb9b
Adsk Contrib - Avoid NaN values in certain Lut1D arrays (#1272) 50e5d8b37db24c1979c9406476479fa21864e261
Transform Python Binding Fix (#1277) Frankie Eder frankaeder@gmail.com c324ba5438cabf0de777404c1709bd87a6d47491
Use OIIO-provided CMake find module (#1266) Michael Dolan michdolan@gmail.com 8774ebf739b0303850cd85b09eff754653d40154
Fix issue #1262 (#1271)  d51962acf067287ddbafce98028f0a71843989b3
Adsk Contrib - Python repr and LogCameraTransform fixes (#1270)  c033d11ee321c6c9492163868d353ed3db1d066d
Add roles to API and fixes (#1267) 8cf0211c3fe367326e54fdf373d3a6dd0d7d8c42
Adsk Contrib - Shader textures now available via Python (#1268) 711042573275c4080b3445b276e9d64d1de28461
Do not squash to keep list of changes.
Signed-off-by: Bernard Lefebvre bernard.lefebvre@autodesk.com

We need to setup new branch protection for RB-2.x. It's currently following the RV-1.x rules, which is why the required checks are off. I'll update it.

We need to setup new branch protection for RB-2.x. It's currently following the RV-1.x rules, which is why the required checks are off. I'll update it.

Branch protection updated so all checks are now passing

Branch protection updated so all checks are now passing

Thank you @michdolan

Thank you @michdolan

This is missing 2 commits

This is missing 2 commits

authors: BernardLefebvre michdolan

repo: AcademySoftwareFoundation/OpenColorIO


text:
Click by element fix

Fixes #211

Coverage decreased (-0.2%) to 99.756% when pulling 73d299ab076e11257f02531473f8c601e3efa06e on clikc_by_element_fix into b473e6dc358d1fda333cc00ab5a465d9b7e22f2a on master.

repo: Accruent/robotframework-zoomba


text:
MadMax create invalid achi plots

Hi Sten,
in FAQ is given info, that achi plots can be created with MadMax. I installed Achi on Ubuntu 20.04, also installed MadMax and created first plot with poolkey & farmerkey from achi keys show.
Used MadMax command: ./chia_plot --count 10 --threads 14 --tmpdir /mnt/nvme/madmax/ --tmpdir2 /mnt/ramdisk/ --finaldir /mnt/data-96/achi/.
After plots check by achi plots check -n 100, achi told that generated plot is not valid (see log below). Why this happens? Is there any special custom MadMax needed, or something like this?
Thank you for your check and response. Also BIG Thank you for Achi ;)
2021-08-16T22:14:23.008  achi.plotting.plot_tools         : INFO     Found plot /mnt/data-96/achi/plot-k32-2021-08-16-21-39-82bc6667e7f932543ad7e555b4cd452f44597148b6b77d02f43cab0d3eb42b10.plot of size 32
2021-08-16T22:14:23.008  achi.plotting.plot_tools         : INFO     Loaded a total of 1 plots of size 0.0989849086608956 TiB, in 0.03962993621826172 seconds
2021-08-16T22:14:23.008  achi.plotting.check_plots        : INFO
2021-08-16T22:14:23.008  achi.plotting.check_plots        : INFO     Starting to test each plot with 1000 challenges each
2021-08-16T22:14:23.009  achi.plotting.check_plots        : INFO     Testing plot /mnt/data-96/achi/plot-k32-2021-08-16-21-39-82bc6667e7f932543ad7e555b4cd452f44597148b6b77d02f43cab0d3eb42b10.plot k=32
2021-08-16T22:14:23.009  achi.plotting.check_plots        : INFO        Pool public key: REMOVED
2021-08-16T22:14:23.016  achi.plotting.check_plots        : INFO        Farmer public key: REMOVED
2021-08-16T22:14:23.016  achi.plotting.check_plots        : INFO        Local sk: <PrivateKey REMOVED>
2021-08-16T22:14:23.071  achi.plotting.check_plots        : ERROR    <class 'AssertionError'>:  error in proving/verifying for plot /mnt/data-96/achi/plot-k32-2021-08-16-21-39-82bc6667e7f932543ad7e555b4cd452f44597148b6b77d02f43cab0d3eb42b10.plot
2021-08-16T22:14:23.071  achi.plotting.check_plots        : ERROR       Proofs 1 / 100, 0.001
2021-08-16T22:14:23.072  achi.plotting.check_plots        : INFO
2021-08-16T22:14:23.072  achi.plotting.check_plots        : INFO
2021-08-16T22:14:23.072  achi.plotting.check_plots        : INFO     Summary
2021-08-16T22:14:23.072  achi.plotting.check_plots        : INFO     Found 0 valid plots, total size 0.00000 TiB
2021-08-16T22:14:23.072  achi.plotting.check_plots        : WARNING  1 invalid plots found:
2021-08-16T22:14:23.072  achi.plotting.check_plots        : WARNING  /mnt/data-96/achi/plot-k32-2021-08-16-21-39-82bc6667e7f932543ad7e555b4cd452f44597148b6b77d02f43cab0d3eb42b10.plot

Hi ag3ntt,
I have special MadMax plotter,
use this one https://github.com/Achi-Coin/achi-plotter.
Please let me know if it works.
Cheers,
Sten

Hi @Achi-Coin,
thank you - with your custom MadMax plotter it works great.

repo: Achi-Coin/achi-blockchain


text:
Strong name assemblies

Inspired by Microsoft Open-source library guidance we should strong name the assemblies to support beeing referenced from other strong named assemblies.

CONSIDER strong naming your library's assemblies.

I'd like to open up the discussion whether or not we would strong name our assemblies, because if so, we should do it before 1.0.0 as it is a Binary breaking change.
Microsoft does recommend it and it would open up fore more consumers, but it might have some downsides.
Input from @viktorvan and @fredrik-lundin maybe?
Making this the descussion for both ActiveLogin.Identity and ActiveLogin.Authentication (https://github.com/ActiveLogin/ActiveLogin.Authentication/issues/25).

I can't see any issues with this. I don't think we depend on anything not strongly named?

Another thought that occured, do we know any strongly named assemblies that would reference us?

authors: PeterOrneholm Zonnex


text:
update org.activiti.cloud.api:activiti-cloud-api-dependencies to 7.0.26

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

UpdateBot commands:
updatebot push-version --kind maven org.activiti.cloud.api:activiti-cloud-api-dependencies 7.0.23

UpdateBot commands:
updatebot push-version --kind maven org.activiti.cloud.api:activiti-cloud-api-dependencies 7.0.24

UpdateBot commands:
updatebot push-version --kind maven org.activiti.cloud.api:activiti-cloud-api-dependencies 7.0.25

UpdateBot commands:
updatebot push-version --kind maven org.activiti.cloud.api:activiti-cloud-api-dependencies 7.0.26

UpdateBot commands:
updatebot push-version --kind maven org.activiti.cloud.api:activiti-cloud-api-dependencies 7.0.27

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.2 out of 3 committers have signed the CLA.:white_check_mark: salaboy:white_check_mark: jx-activiti-cloud:x: mergify-botmergify-bot seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

authors: CLAassistant jx-activiti-cloud

repo: Activiti/activiti-cloud-dependencies


text:
fix(versions): update Activiti/activiti-cloud-query-service versions

UpdateBot pushed maven dependency: org.activiti.cloud.common:activiti-cloud-service-common-dependencies to: 7.1.11

UpdateBot commands:
updatebot push-version --kind maven org.activiti.cloud.common:activiti-cloud-service-common-dependencies 7.1.11

repo: Activiti/activiti-cloud-query-service


text:
fix(versions): update Activiti/activiti-dependencies versions into develop

UpdateBot pushed maven dependency: org.activiti:activiti-core-dependencies to: 7.1.62

UpdateBot commands:
updatebot push-version --kind maven org.activiti:activiti-core-dependencies 7.1.62 --merge false

repo: Activiti/activiti-dependencies


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

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

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

repo: Activiti/ttc-query-campaign


text:
final pr request

only styleLint error remains

Not merging this PR.

authors: manojown shoaibmerchant

repo: Actonate/angular-starter


text:
Legg til funksjon for å sjekke om objekt_type objekt er i objektlista.

if objekt_type(98).objekt_liste():
do_code

Er ikke dokumentert.

repo: Acurus/pnvdb


text:
dyld[10175]: Symbol not found: OBJC_CLASS$_GULNetworkInfo

dyld[10175]: Symbol not found: _OBJC_CLASS_$_GULNetworkInfo
Referenced from: <7EFCE371-0EAC-3F87-8683-C6B15DDA9669> /Users/immons/Library/Developer/CoreSimulator/Devices/9948F228-BE7E-4732-A5F7-91302C50CE3E/data/Containers/Bundle/Application/1F806E73-1923-4820-BDA8-7029ED7ECAC6/Mobile.Core.app/Frameworks/FirebaseSessions.framework/FirebaseSessions
  Expected in:     <34E0B44E-4E10-3028-9875-DC18A41938E9> /Users/immons/Library/Developer/CoreSimulator/Devices/9948F228-BE7E-4732-A5F7-91302C50CE3E/data/Containers/Bundle/Application/1F806E73-1923-4820-BDA8-7029ED7ECAC6/Mobile.Core.app/Frameworks/GoogleUtilities.framework/GoogleUtilities

AdamE.Firebase.iOS.Analytics version: 10.24.0
this one shows in Console output, application crashes immediately on simulator when starting.

I had a similar issue with just getting the other Firebase components working, I had to do this to get it working. dotnet add package AdamE.Firebase.iOS.Installations --version 10.24.0

@Immons does the above help?

@AdamEssenmacher no, still crashing immediately

Do you have a repro project?

Any solution to this?

I tried to create a repro only to find out that in a clean repro project the Frameworks/GoogleUtilities.framework/GoogleUtilities file had the missing symbol, so it seems to be some binary file mix-up. Clean build resolved the issue.

authors: AdamEssenmacher Immons eostarEmployee80 filipnavara rezamohamed


text:
Added Chevron and Icon

Related Issue
Please use one of the well-known github fixes keywords to reference
the issues fixed with this PR (eg Fixes #). If an issue doesn't yet exist please create one if reasonable to aid
in issue tracking.
NOTE: For multiple issues resolved by this PR use the corresponding keywords every time in a comma-delimited list per the reference
page above.
Description
Create a descriptive title and provide all of the relevant details in the description.
This information lives with the code and will help future readers (including yourself) and will serve as documentation.
At the very least please describe the issue you are addressing, and what your fix entails.
Sample Card
If appropriate, please include a link to a card in one of the samples directories that can be used to validate this change. This can be an existing card or a card added with this PR.
How Verified
How you verified the fix, including one or all of the following:

New unit tests that were added if any. If none were added please add a quick line explaining why not.
Existing relevant unit/regression tests that you ran
Manual scenario verification if any; Do include .gif's or screenshots of the testing you performed here if you think that it
will aid in code reviews or corresponding fixes on other platforms for eg.

Most of the unused Assets can be removed and I guess we can avoid the changes in RootViewController.xib.

repo: AdaptiveCards-Desktop/AdaptiveCards


text:
Class adldap does not exist

Class adldap does not exist

That has to be one of the vaguest issues ever.
Have you tried turning it off an on again? :D
That said the class is case sensitive as is Adldap depending how you use it you may have to refer to it as \Adldap

Hi,
I have one error
BindException in Guard.php line 86:
Can't contact LDAP server
Can you help me?
thank you

Have you properly completed the details in config\adldap.php for your domain controller etc?
'domain_controllers' => ['myserver'],
'port' => 389,

You may want to try with just an IP address. Also is the port you are using correct?

I use localhost then what myserver ??

localhost or virtualhost

You need to specify your Active Directory Domain Controller

Closing due to no information about the issue as well as no LDAP server knowledge.

authors: stevebauman thuyhyvg warlord0


text:
Username Update Broken when using GUID FK

Laravel Version: 5.5.45
Adldap2-Laravel Version: v3.0.9
PHP Version: 7.2
LDAP Type: ActiveDirectory

Description:
We are using the user AD GUID as the primary key. However, when a user changes their username, a SQL error is thrown about trying to create a new account instead of updating the existing one.
I need a way to tell ADLDAP2-Laravel what I want they key attribute to be, so it can do a lookup even with a changed username.

Steps To Reproduce:

Import user
Change AD username
sql error thrown in logs. Redacted data in [brackets]

[2019-07-16 10:17:38] production.ERROR: Unable to import user [name] SQLSTATE[23000]: Integrity constraint violation: 1062 Duplicate entry '[guid]' for key 'PRIMARY' (SQL: insert into `users` (`email`, `name`, `username`, `phone`, `department`, `distinguishedname`, `guid`, `[private attribute name]`, `[private attribute name]`, `[private attribute name]`, `nav_username`, `emailtopia_address`, `full_ad_username`, `password`, `updated_at`, `created_at`) values ([user]@[domain]com, [name], [username], [phone], [dept], [dn], [hash], , , , , , , [hash], 2019-07-16 10:17:38, 2019-07-16 10:17:38))

Hi @benyanke,
This is actually a feature that was added in v6.0.
However, to do this in v3.0, you would have to extend the DatabaseUserProvider as well as the Importer.
For the DatabaseUserProvider you will need to override the retrieveByCredentials() method and for the Importer, you will need to override the findByCredentials() method.
I can't give you example code since this is implementation dependent and [v6.0] supports this feature.
Hope this helps!

Thank you! I'll give this a try.

No problem! Let me know if you run into any issues and I'll do my best to help you out.

authors: benyanke stevebauman


text:
Feature/http cache

Minor adjustments to get it working on my local. It's awesome :)
One change I made was removing the customAttributes OSGi Property from HttpCacheConfigImpl. The reasoning is this list of strings may not be sufficient for all customizers. Instead I collapsed the GroupCacheKeyFactory into GroupCacheConfigExtension, and created a impl specific userGroups OSGi Property that both the Extension and CCFactory can leverage.
Technically these 2 classes could be decoupled since they are registered to their respective Service interfaces, and HttpCacheConfig's configs still let you specific the service filter for each individually, but i expect these 2 things (Extension and CCFactory) will be very tightly coupled in most impls.
Other minor changes included handling CacheKey exceptions on key-based invalidation (i have it invalidating the entire cache if that happens)
If you can take a look at the the MemPersistenceObject and make sure the Header logic looks right, would be handy. I was getting an exception when it tried to unwrap the Header values from a wrappedSet to a List.
Made some minor tweaks to the MBean to get it to come up; but think there may be a few issues w it still.
25.11.2015 00:07:40.629 *TRACE* [0:0:0:0:0:0:0:1 [1448428060356] GET /content/geometrixx/en.html HTTP/1.1] com.adobe.acs.commons.httpcache.filter.impl.HttpCacheFilter Delivered un-cached request [ /content/geometrixx/en.html ] in 265 ms
25.11.2015 00:07:54.323 *DEBUG* [0:0:0:0:0:0:0:1 [1448428074309] GET /content/geometrixx/en.html HTTP/1.1] com.adobe.acs.commons.httpcache.filter.impl.HttpCacheFilter Delivered cached request [ /content/geometrixx/en.html ] in 13 ms

25.11.2015 00:25:02.799 *TRACE* [0:0:0:0:0:0:0:1 [1448429099625] GET /content/geometrixx/en/events/techsummit.html HTTP/1.1] com.adobe.acs.commons.httpcache.filter.impl.HttpCacheFilter Delivered un-cached request [ /content/geometrixx/en/events/techsummit.html ] in 3171 ms
25.11.2015 00:27:24.362 *DEBUG* [0:0:0:0:0:0:0:1 [1448429244347] GET /content/geometrixx/en/events/techsummit.html HTTP/1.1] com.adobe.acs.commons.httpcache.filter.impl.HttpCacheFilter Delivered cached request [ /content/geometrixx/en/events/techsummit.html ] in 15 ms

How is this different than the in memory HTTP response cache provided since AEM 6.0?

Far_ more flexible. That only works w anon requests and has a 60 ls ttl

I accidentally PR'd here, and shouldve done got @Sivaramvt's - thus the closing.
@justinedelson btw - there is more/better info on this feature here: https://github.com/Sivaramvt/acs-aem-commons/pull/1
OOTB Cache

Listens on a series of path prefixes
Cache Invalidation is TTL-based on 60s timer
Cache is only for anonymous requests

Our Cache

Defines cache configs for each cache scheme, which includes

RequestURI patterns whitelist
RequestURI patterns blacklist
Authentication support (anon-only, auth-only, both)
Pluggable accept rules (roll your own)
Pluggable cache key creation (want to cache results by group's X and Y + profile attribute x? and do NOT cache requests by Group z ever?  no problem, just write it.)
Cache invalidation rules (TTL or CUD event on resource by path pattern)

Pluggable persistence (mem right now; disk and jcr possible)
MBean to inspect cache
Other stuff im forgetting

authors: davidjgonzalez justinedelson


text:
PWA-2447: quick port of missing Commerce features/coverage table

Description
This PR provides a quick-and-dirty port of the HTML output from the old Jekyll-based PWA Studio docs site. The HTML for this table will soon be replaced with an interactive, sortable React table component that we can use for all tables in the documentation.
Related Issue
This topic cannot currently be accessed in the old PWA docs site because of the redirections applied.
Motivation and Context
The quick-and-dirty approach gets the missing table out there quickly, and gives me some more time to create a React table component that can be reused throughout the documentation.
How Has This Been Tested?
Generated a staging build here:
Screenshots (if appropriate):
Works fine on mobile devices and desktops.

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Definitely interested in what you come up with here @bdenham and how we might be able to leverage it for the public roadmap: https://devdocs.magento.com/release/roadmap.html.

@jcalcaben Yep! :) I had to make a few css tweaks. But yeah, this gets it out there so I can continue to create a solid React table with all the features interactive sorting features and data feeds from json and/or yml files.

Definitely interested in what you come up with here @bdenham and how we might be able to leverage it for the public roadmap: https://devdocs.magento.com/release/roadmap.html.

@jeff-matthews Yeah, for sure. That's why I put this out there for the short term — to buy me time to do a thorough table component that caters to our conversion needs for porting over our jekyll-based tables.

authors: bdenham jeff-matthews

repo: AdobeDocs/commerce-pwa-studio


text:
Count tests skipped at runtime as passing

TestNG returns a specific exit code if all tests are skipped at
runtime. For such cases we should count the test as passing
because (presumably) the test knows what it's doing.
Signed-off-by: Younes Manton ymanton@ca.ibm.com

I believe test count is for counting test targets, NOT specific testcases within the target.  As such, I believe it is inappropriate to include TestNG count in this total, though I will ask @renfeiw to weigh in on this also.

Sorry, by count I meant classify/mark. Currently test targets whose test cases are skipped are marked as as failed. With this change they'll be marked as passing.

Just wondering why do we want to keep a test target whose all test cases are skipped? Should we just remove this test or exclude this test on that platform?

I want to be able to decide to skip tests at runtime (based on whether or not the JDK being tested supports certain features, the environment, etc), and in a more granular way, rather than at test build time.

Could you please give me an example that you need to skip the test cases in the runtime while it cannot be done in the playlist.xml? TKG is a general framework not only for testng, your change may have issue when non-testng tests fail with return code 2.

In my opinion it is important not to hide 'SKIPPED' status into 'PASSED' status, because PASSED implies the tests were run.  SKIPPED clearly shows the tests were not run.  The 2 states are distinct and should not be muddied together.

Could you please give me an example that you need to skip the test cases in the runtime while it cannot be done in the playlist.xml? TKG is a general framework not only for testng, your change may have issue when non-testng tests fail with return code 2.

For my immediate concerns I can do what's needed in the playlist, but that seems less desirable because it will have to be repeated per playlist test, and the commands in most of the tests in the playlists are already hard to read. This is a much tidier solution that doesn't encourage duplication.
However if we want TKG to not be aware of TestNG's exit codes then I guess it will have to be.

In my opinion it is important not to hide 'SKIPPED' status into 'PASSED' status, because PASSED implies the tests were run. SKIPPED clearly shows the tests were not run. The 2 states are distinct and should not be muddied together.

Agreed. I'm suggesting turning FAILED with exit code 2 to PASSED (or some other non-failure status) because TestNG returns exit code 2 when all test cases are skipped and we consider all non-zero exit codes as failing.

As mentioned in the previous comment, this change may cause issue for non-testng tests.

For my immediate concerns I can do what's needed in the playlist, but that seems less desirable because it will have to be repeated per playlist test

Could you give us a specific example of the need for repeating the change for all playlist tests? It is unclear to me why your change will cause all playlist tests to skip.

Could you give us a specific example of the need for repeating the change for all playlist tests? It is unclear to me why your change will cause all playlist tests to skip.

Sorry, I think you've misunderstood me. I wasn't suggesting that our existing playlists had to change. I'm suggesting that we can either make this kind of change in one place (like settings.mk) or in potentially many places (like playlist.xml) and that making it in one place is far better than making it in many places.
However, if we have to constrain ourselves to only allowing exit code 0 to signal success then I guess that's a limitation we'll have to accept. I can change the playlist to handle the exit code before invoking TEST_STATUS instead, or do something else to work around it.

I've worked around it in the playlist and no longer require this change.

authors: renfeiw smlambert ymanton

repo: AdoptOpenJDK/TKG


text:
Set timeout of sub-test to a reasonable value

This PR reduces the 'math' variant of math load test's timeout value from 20h to 2h- which is a more practical timeout value. This prevents cases like the one reported here where the test took more than 5 hours to complete.
Related to https://github.com/AdoptOpenJDK/openjdk-systemtest/issues/389
Signed-off-by: Mesbah_Alam@ca.ibm.com Mesbah_Alam@ca.ibm.com

Test: https://ci.adoptopenjdk.net/view/Test_grinder/job/Grinder/5946/console

Test: https://ci.adoptopenjdk.net/view/Test_grinder/job/Grinder/5946/console

Note: We will be adding a time-based equivalent  of these load tests which should cover the testing that was potentially being covered by the amount of workload which is taken off by this PR.

Note: We will be adding a time-based equivalent  of these load tests which should cover the testing that was potentially being covered by the amount of workload which is taken off by this PR.

repo: AdoptOpenJDK/openjdk-systemtest


text:
Repository incompatible with Travis CI

Due to unsatisfied requirements all pushes cannot compile. pytest should be configured. Maybe Travis config file should be changed.

Now we will always pass Travis tests, cause tests file is not connected with our source files. We have to think about that. Create link between them.

I guess it works just fine now. At least it looks like that.

repo: AdvenamTacet/EMP


text:
Blik svg not available after upgrade to 5.50.0. (404)

Describe the bug
When paying with blik, an svg fetch error is returned and a broken image is displayed.
GET https://cdf6519016.cdn.adyen.com/checkoutshopper/images/logos/await.svg 404 (Not Found).
on the previous version everything was fine(5.29.0).
To Reproduce
Pay by blik
Expected behavior
The application is able to fetch the image or dorpin should not display the broken image(as in the previous version).
Screenshots

Desktop (please complete the following information):

OS: MacOs
Browser: Chrome
Version: 120.0.6099.62
dropin 5.55.0

@Oskii2311 Could you confirm the version you are using? Is it v5.55.0 or 5.50.0?

@longyulongyu
Sorry, my bad.
it is:
5.55.0

@Oskii2311 could you please try the newest version and let us know if the problem is solved? Thanks

I updated dropin to version 5.55.1 and everything looks good.
Thank you!

authors: Oskii2311 longyulongyu ribeiroguilherme

repo: Adyen/adyen-web


text:
Storing the checkoutAttemptId on sessionStorage

Summary
This PR handles the storage and usage of the checkoutAttemptId on the sessionStorage. This change aims to improve the calculation of the conversion rates, in scenarios like the shopper reloads the checkout page or comes back from a redirect payment method.
Highlights on how it works:

If checkoutAttemptId is passed during the initialization of AdyenCheckout, then we don't handle the storage of the ID - the PBL app or the merchant itself is handling it in this scenario
In case that there is no checkoutAttemptId passed, the SDK will request it and save on the sessionStorage along with a timestamp of 15 minutes - so if a redirect happens, or if the shopper decides to move out from the checkout and add another extra item to the cart, we would still use the same checkoutAttemptId if the timespan is within the 15 minutes.

Tested scenarios

Passing checkoutAttemptId to AdyenCheckout doesn't trigger a request to get new ID, neither use the one on the session storage
After saving the value in the session storage and refreshing the page, the UI will sill use the same ID
After 15 min passed, the UI requests another ID

Fixed issue:  COWEB-1099

As discussed - is it possible to write any tests for this?

authors: ribeiroguilherme sponglord


text:
SDK v2

Is your feature request related to a problem? Please describe.
There are many limitations and issues with the current implementation of the SDK:

The parameters for the initial served variant are not saved in the backend. They are nowhere but in the code.
The parameters for variant (after being modified in the playground) need to be given in the api call. As a result, after each change to the variant parameter, the code in the user's software need to be updated with the new parameters.
Creating playgrounds for code-written applications in agenta is quite difficult.

Describe the solution you'd like
A new version of the SDK, where the simplest single prompt application would look like this:
ag.init() # no need to give the infos here since they are loaded from the environment variables
ag.config.default(prompt1=ag.ChatPrompt(), model_config=ag.LLMConfig())

@post
def something(myinp:str):
	output = ag.ChatCompletion.create(**ag.config.model_config, messages=ag.config.prompt1.format(my_inp))
  return output

A more complicated application with two prompts would look like this
ag.init()

p1 = ag.ChatPrompt([
    {"role": "human", "content": "What is the capital of {text}."},
    {"role": "system", "content": "You are an expert in geography."}])

c1 = ag.LLMConfig(model="gpt-3.5-turbo-16k-0613")

p2 = ag.LLMPrompt([
    {"role": "human", "content": "What is the capital of {text}."},
    {"role": "system", "content": "You are an expert in geography."}])

c2 = ag.LLMParams(model="gpt-3.5-turbo-16k-0613")

@ag.post
def generate(
    inputs: ag.DictInput = ag.DictInput(default_keys=["text"]),
) -> str:
    output = ag.ChatCompletion.create(**ag.config.c1 messages=ag.config.p1.format(**inputs))
    output = ag.ChatCompletion.create(**ag.config.c2 messages=ag.config.p2.format(**inputs))
    return output

The prompts can also be used if the code is not deployed with agenta
ag.init(app_name="app_name", environment="deployment")
ag.config.pull()
output = openai.ChatCompletion.create(**ag.config.c1, messages=ag.config.p1.format(user_message="msg"))

Both LLMPrompt and LLMConfig are parameters that are shown in the playgound. LLMConfig encapsulates all the parameters for configuring an LLM, while LLMPrompt is the same for prompts.
**Tasks **

[x] Implement the new SDK so that it works exactly like the previous one:

Deploying the app injects the name of the app and the variant as environment variable
ag.init() which uses per default the values in the environment variables and saves them
ag.init() which creates ag.config
ag.config.default which sets the parameters (from the cmd) for the object
@ag.post which creates the same openapi.json but now based on the parameters saved in ag.config.default

[x] Saves and pulls the configuration

Modify ag.config.default to

Save the configuration to the variant name (if no configuration exists)
Pull the latest version of the configuration from the backend

Modify the @post endpoint

to pull the last version of the configuration from the backend before running (with caching)
Add a new parameter to the endpoint to get the variant name

[x] Use the environment

Modify @post endpoint to add a new parameter environment to get the variant name from the environment
Modify the endpoint view to show the variant name with input being environment

[ ] Create LLMConfig, LLMPrompt which work with litellm.ChatCompletion
[ ] Modify the frontend to hardcode parameters for the llms
[ ] Modify traefik

Use traefik to route to a new endpoint with the environment name

[x] Edit documentation

@devgenix

Rewrite the code for the examples and templates to use SDK v2
Test all capabilities for the platform with these examples (overwriting, serving, deleting, publishing, testing the published endpoint, evaluation...)
Reading and following the documentation for SDK v2 and finding issues

repo: Agenta-AI/agenta


text:
Fix - removed warnings in playground page

Hi @mmabrouk,
Let me know if there's something I'm not doing right! Merging this PR closes #153.

@CodiumAI-Agent /review

Preparing review...

PR Analysis

🎯 Main theme: Removal of warnings in the playground page
📌 Type of PR: Bug fix
🧪 Relevant tests added: No
✨ Focused PR: Yes, the PR is focused as it addresses a specific issue (#153) which is about removing warnings in the playground page. The title, description and the code changes are all aligned with this objective.
🔒 Security concerns: No, the changes made in this PR do not introduce any apparent security concerns. The changes are primarily related to the user interface and type definitions, which do not typically pose security risks.

PR Feedback

💡 General PR suggestions: The PR seems to be well-structured and the changes are clear. However, it would be beneficial to include tests that verify the removal of the warnings. This would ensure that the changes are effective and do not introduce new issues.

How to use

Tag me in a comment '@CodiumAI-Agent' and add one of the following commands:
/review [-i]: Request a review of your Pull Request. For an incremental review, which only considers changes since the last review, include the '-i' option.
/describe [-c]: Modify the PR title and description based on the contents of the PR. To get the description as comment instead of modifying the PR description, include the '-c' option.
/improve: Suggest improvements to the code in the PR. These will be provided as pull request comments, ready to commit.
/ask <QUESTION>: Pose a question about the PR.

@CodiumAI-Agent /improve

Preparing review...

authors: CodiumAI-Agent aybruhm mmabrouk


text:
Updated Cake build and SonarQube runner files to v4.1.1.1164

https://github.com/SonarSource/sonar-scanner-msbuild/releases/tag/4.1.1.1164

https://github.com/SonarSource/sonar-scanner-msbuild/releases/tag/4.2.0.1214

This PR related to issue #11


text:
Issue while running agora flutter sdk in web browser ('createIrisApiEngine')

Describe the bug
I am trying to build a video call feature for a web app using the Agora Flutter SDK. When I try to run it in a web browser, it shows 'TypeError: Cannot read properties of undefined (reading 'createIrisApiEngine').' All the UI elements are loaded properly, but the Agora engine does not work as I expect.
To Reproduce
Steps to reproduce the behavior:

create a flutter project
add the agora_rtc_engine: ^6.3.0
permission_handler: ^11.0.1 dependency in the project
create a code for run a simple video call using agora and add app Id and channel name to the code .
try to run it in web browser

Expected behavior
Expected that agora engine will work
Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

OS: windows
Browser : chrome
Version : 122.0.6261.95 (Official Build) (64-bit)

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
video call is an important feature of my web app. so please reply as soon as possible

Have you set up the project
https://github.com/AgoraIO-Extensions/Agora-Flutter-SDK?tab=readme-ov-file#web-alpha

repo: AgoraIO-Extensions/Agora-Flutter-SDK


text:
希望提供对MPU-6050加速度计/陀螺仪的支持

1. SDK version（SDK 版本）
{
所有版本
}

2. In what kind of operation problems appear, and how to reproduce the problem ?（什么样的操作步骤问题会出现，是否是稳定复现，如何复现问题？）
{
新特性需求
}

如题，目前Pudding开发板默认无加速度计，MPU-6050是目前较为通用和经济的一类加速度计传感器，希望增加相应的代码和样例，以弥补Pudding开发板的缺陷。

6050通过iic连接，github上就有很多例程，移植一下就可以用的

authors: Neutree mayonglei


text:
Access blocked: rclone’s request is invalid

how to fix?

whoever watching gdrive limit expire everyday at 2 pm try at the time

repo: Aicirou/goindex-theme-acrou


text:
Amend git commit when fixing linting issues / test failures

Issue
I am using prettier in my project and it is a bit opinionated, which is often ignored by aider. So I almost always end up with 2 commits for one feature, where the most recent one is just a 'fix linting issues'. The same goes with when tests fail, it often ends up 2-4 commits for one feature.
Instead, it would be great if those post-fixes would be amended to the previous commit.
Version and model info
openai default model. aider @ 0.60.1

Thanks for trying aider and filing this issue.
Best thing would just be to squash commits from time to time?

Yeah that would do the trick I think, at least for one feature/workflow until all tests are passing.

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.

not stale

Sorry, my suggestion was that you should just squash the commits whenever you want?

Ah. I guess yeah. But you could also code yourself, yet we let AI do it :D

Aider relies on the git history to allow users to safely undo changes. Automatically squashing commits would defeat that.

Alright, fair.

Apologies for reopening this thread, but I propose we introduce a flag (similar to —lint-before-commit) that allows users to create a single commit containing the code changes and already linted. What do you think?
In most cases, users undo changes because they want to revert only the last “thing I asked aider,” not just the listing of that request.

authors: lucacri paul-gauthier steebchen

repo: Aider-AI/aider


text:
Uncaught ImportError in init.py line 9

Aider version: 0.63.2
Python version: 3.11.5
Platform: Windows-10-10.0.22621-SP0
Python implementation: CPython
Virtual environment: Yes
OS: Windows 10 (64bit)
Git version: git version 2.41.0.windows.1
An uncaught exception occurred:
Traceback (most recent call last):
  File "<frozen runpy>", line 198, in _run_module_as_main
  File "<frozen runpy>", line 88, in _run_code
  File "__main__.py", line 7, in <module>
    sys.exit(main())
             ^^^^^^
  File "main.py", line 639, in main
    main_model = models.Model(
                 ^^^^^^^^^^^^^
  File "models.py", line 815, in __init__
    res = self.validate_environment()
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "models.py", line 1030, in validate_environment
    res = litellm.validate_environment(model)
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "llm.py", line 23, in __getattr__
    self._load_litellm()
  File "llm.py", line 30, in _load_litellm
    self._lazy_module = importlib.import_module("litellm")
                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "<frozen importlib._bootstrap>", line 1204, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1176, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1147, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 690, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 940, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "__init__.py", line 9, in <module>
    from litellm.llms.custom_httpx.http_handler import AsyncHTTPHandler, HTTPHandler
  File "http_handler.py", line 6, in <module>
    import httpx
  File "__init__.py", line 2, in <module>
    from ._api import *
  File "_api.py", line 6, in <module>
    from ._client import Client
  File "_client.py", line 32, in <module>
    from ._transports.default import AsyncHTTPTransport, HTTPTransport
  File "__init__.py", line 3, in <module>
    from .default import *
  File "default.py", line 33, in <module>
    import httpcore
  File "__init__.py", line 1, in <module>
    from ._api import request, stream
  File "_api.py", line 5, in <module>
    from ._sync.connection_pool import ConnectionPool
  File "__init__.py", line 1, in <module>
    from .connection import HTTPConnection
  File "connection.py", line 14, in <module>
    from .http11 import HTTP11Connection
  File "http11.py", line 17, in <module>
    import h11
  File "__init__.py", line 9, in <module>
    from h11._connection import Connection, NEED_DATA, PAUSED
ImportError: cannot import name 'Connection' from 'h11._connection' (C:\Users\Fayez\aider-project\venv\Lib\site-packages\h11\_connection.py)

Thanks for trying aider and filing this issue.
How did you install aider? What environment are you using? It looks like you don't have the correct set of dependencies installed. This doc may be helpful:
https://aider.chat/docs/troubleshooting/imports.html
I'm going to close this issue for now, but feel free to add a comment here and I will re-open. Or feel free to file a new issue any time.


text:
Future: 油管订阅插件

包含 youtube 用户、频道、直播订阅功能

考虑各种现实因素，该计划功能废弃

repo: Ailitonia/omega-miya


text:
Cannot run docker image

I try running the docker image as said in the installation guide and get this message

and then in just hangs like that. I can't keyboard interrupt, all I can do is just quit the terminal (PowerShell on Windows 11)
I also tried running the docker image on docker desktop in which I get this error

The test command works locally, but I can't attempt to run it inside the docker container at all, since it immediately crashes.
Only changes I've made was to install gpu version of torch via their guide and to rename the docker-compose-nvidia.yaml file to docker-compose.yaml. I also skipped the step where it says to install the NVIDIA Container Toolkit because link gives guide on how to do it on Linux and not on windows (or wsl 2).

EDIT: I have (or think) that I have installed the Container Toolkit, but it still gives the same error

Hi,
Regarding the second error. You need to first implement an agent. For testing purposes, you can put the dummy agent example:
https://air-hockey-challenges-docs.readthedocs.io/en/latest/framework.html#dummy-agent
in the air_hockey_agent/agent_builder.py
Unfortunately, we can not support the docker installation on windows. You can try to develop the agent locally and evaluate in the docker without visualization.
In addtiona, here is some instructions on the docker usage with windows:
Instructions for Docker on Windows:

- Install vcxsrv (to let Docker use the display of Windows system for the render)
- Install docker-desktop
- Change docker-compose.yaml like this:

--->  replace     DISPLAY=${DISPLAY}
with          DISPLAY=host.docker.internal:0.0

--->  replace     "${PWD}/:/src/2023-challenge"
with          "./:/src/2023-challenge"
where ./ can be filled with any path inside the docker-container.

- Build the image with command docker compose build from Powershell
- Run the container with docker compose run challenge from Powershell

So, I've been following the steps again, and I installed the docker container on wsl 2 but when I try to run the test command in wsl 2 but not on docker I get this error.

When I try checking my glx version with glxinfo I get this error

I have looked far and wide for an answer for this but have found nothing useful.

Thats strange. What is you Windows Version? According to Microsoft Windows 10 Build 19044+ and Windows 11 should support running Linux GUI application (X11 and Wayland) out of the box with wsl2 (source). You should not install Xming on your own.
One issue could be that your GPU driver is not up to date.  If xeyes or glxgears still fail with the latest GPU driver you can create a wsl issue for your hardware.
Another participant shared this setup to get the GUI output of docker on windows.  Thanks to @amarildolikmeta.
Instructions for Docker on Windows:

- Install vcxsrv (to let Docker use the display of Windows system for the render)
- Install docker-desktop
- Change docker-compose.yaml like this:

--->  replace     DISPLAY=${DISPLAY}
with          DISPLAY=host.docker.internal:0.0

--->  replace     "${PWD}/:/src/2023-challenge"
with          "./:/src/2023-challenge"
where ./ can be filled with any path inside the docker-container.

- Build the image with command docker compose build from Powershell
- Run the container with docker compose run challenge from Powershell

I have not tested this but it might be an option for you.

So xeyes works completely fine its just glxgears and glxinfo that gives me weird errors. They were working at some point, but I think I may have upgraded some kind of package (attempting to get them to upgrade to 1.3) and they just broke after that. I'm not running any weird hardware (3060, i9-11900k) and my drivers are up to date. I'll definetly check out how to get GUI output of docker onto windows, but is there any point  if the test command doesn't.

So I don't know if you saw  but xeyes does work (sorry for not highlighting earlier) and I am on my gpus latest driver (3060). I''m also on the latest windows 11 update.

One of the main problems I'm having is running `glxinfo`

I also did run the commands you suggested but they take no effect. I do have mesa-utils installed and everything and like I said I'm not running weird hardware or anything.

If glxinfo does not work you i would suggest to reset wsl. Im sorry but we do not have the resources to help you troubleshoot your wsl setup.
To participate in the challenge i suggested a workarround above, or just use linux.

Ok, should I delete Ubuntu as well or just WSL?

It should be enough to just reset Ubuntu. This article outlines how to reset a wsl distro without deleting and reinstalling it.

Completely unrelated question (bored while everything is installing again) what do you think is a good book or course for reinforcement learning such as this. I don't plan on winning any award or anything at this competition but I thought it would be a good way to learn a bit more.

It should be enough to just reset Ubuntu. This article outlines how to reset a wsl distro without deleting and reinstalling it.

Ok I just reinstalled and it still gives me the same exact error

I think I will try just running local, thank you for trying to help.

authors: ChaoticChaosTheGreat PuzeLiu cube1324


text:
Feature: Add front end for topic events view based on offset range

Linked issue
Resolves: #2583
What kind of change does this PR introduce?

[ ] Bug fix
[x] New feature
[ ] Refactor
[ ] Docs update
[ ] CI update

What is the current behavior?
Currently it is possible to view only last few selected offsets of partitions, the backend was added in #1987 .
What is the new behavior?
Now topic events can be views based on offset range and partition.
Requirements (all must be checked before review)

[x] The pull request title follows our guidelines
[x] Tests for the changes have been added (if relevant)
[x] The latest changes from the main branch have been pulled
[x] pnpm lint has been run successfully

Before

After

Before

After

New Feature

Hi @khatibtamal - thanks for your PR 🎉
Would you feel comfortable adding this change also to the new UI - the React app in /coral?

Hi @khatibtamal - thanks for your PR 🎉
Would you feel comfortable adding this change also to the new UI - the React app in /coral?

Hi @programmiri I will push the changes in a day or two, just need some time getting used to the coral code thanks.

Hi @programmiri I will push the changes in a day or two, just need some time getting used to the coral code thanks.

Thanks for the update - and take your time!
We've tried to document the processes and approaches in coral a lot, I hope the docs can support you 🤞 Please feel free to ping me if you have any question! And just to be clear - it's also ok if you don't want to do that, we can add a follow up PR later. We appreciate the work you've done already.

Hi, the documentation was excellent,  I only had a couple of minor issues in running the new UI. They are as follows.

Step 9 as listed here did not create the zookeeper and kafka containers for me. I manually ran docker-compose-testEnv.yaml for those containers.
Step 12 first bullet here states kafka bootstrap server is  http://klaw-kafka:9092 but while adding the cluster in klaw, just need to ensure the protocol http is skipped.

I have coral working locally now and I am able to make changes. I dont mind attempting this, I feel within a day or two I should be able to put up a PR. I will certainly ping you if I feel the issue gets too hot to handle.
thanks.

@programmiri just and update. I have made decent progress in Coral and now I am working on fixing the tests, will commit soon. thanks.

Hi @kathia-barahona thanks so much for your feedback, I'll use that to update our documentation, that was super valuable! ❤  Will take a look at your PR later (probably tomorrow)

Hi @kathia-barahona thanks so much for your feedback, I'll use that to update our documentation, that was super valuable! ❤ Will take a look at your PR later (probably tomorrow)

Cool thanks! I will push my changes with all tests in a day or two.

@programmiri now this is ready for review.
Topic messages can now be fetched using 3 options, they are as follows

Default: Behaviour exactly same as before

Custom: Behaviour exactly same as before

Range: this is the new behaviour, where we can now give an offsetStart and offsetEnd for a partitionId

@programmiri I have had some issues

When I first login for some reason I am directed to a localhost:9097/coral path, and my changes do not give an intended look as follows

But when I change the port to 1337 then it looks fine. Not sure why this is happening, is there anything else I need to do in the code to avoid this?

But when I change the port to 1337 then it looks fine. Not sure why this is happening, is there anything else I need to do in the code to avoid this?

This is the expected behaviour at the moment, the proxy setup is not perfect yet :D It's described here:
https://github.com/Aiven-Open/klaw/tree/main/coral/proxy#installation
At the moment, if you're not logged in as a user (or the authentication expired), the redirect will go to port 9097 and not port 1337, so in this moment you're looking at Klaw is it runs in the docker container and not at the dev server. So after login, we've to manually change to the correct port (1337) again.

When I press the "Fetch messages" button, I never get any data back, even though I ensured there is data in the kafka instance running in the docker container. This also happened to me using legacy code, with no changes, maybe I missed something during setup?
When I run the complete application without docker containers I have no problem in fetching data using the legacy UI, but I cannot run coral that way. So I could not test end to end using Coral.

That's a great input! I didn't notice that, but I also don't have any topics with messages on my local env at the moment. I will tag @aindriu-aiven  and @muralibasani  here to see who can take a look so we can make sure it's only an issue with the proxy (and maybe see how we can fix it).

Also, I just checked the frontend changes in coral and it looks good - great job! Thank you also for adding all needed tests :)

👍 for changes in /coral, looks great!

@programmiri Thanks a lot for you review!

@khatibtamal thanks again. I will take a look at it today too. If it looks good, we can merge.

@khatibtamal UI looks good.
Can we add a small validation on the partition id field. Based on the number of partitions, we should not allow the user to enter any other number.
If topic has 2 partitions for ex: then partition ids are 0 and 1.
If topic has 10 partitions for ex: then partition ids are 0 , 1, 2....9
If user enters number above or below this range, then warn with a error "Invalid partition id"

@khatibtamal UI looks good. Can we add a small validation on the partition id field. Based on the number of partitions, we should not allow the user to enter any other number. If topic has 2 partitions for ex: then partition ids are 0 and 1. If topic has 10 partitions for ex: then partition ids are 0 , 1, 2....9
If user enters number above or below this range, then warn with a error "Invalid partition id"

sure will add them and make commit soon. thanks!

@khatibtamal UI looks good. Can we add a small validation on the partition id field. Based on the number of partitions, we should not allow the user to enter any other number. If topic has 2 partitions for ex: then partition ids are 0 and 1. If topic has 10 partitions for ex: then partition ids are 0 , 1, 2....9
If user enters number above or below this range, then warn with a error "Invalid partition id"

@muralibasani @programmiri pushed changes, my last commit

@khatibtamal UI looks good. Can we add a small validation on the partition id field. Based on the number of partitions, we should not allow the user to enter any other number. If topic has 2 partitions for ex: then partition ids are 0 and 1. If topic has 10 partitions for ex: then partition ids are 0 , 1, 2....9
If user enters number above or below this range, then warn with a error "Invalid partition id"

@muralibasani @programmiri pushed changes, my last commit.
@muralibasani in this comment I have described how the back end when using coral does not return any data, even though I ensured there is data, did you have the same issue?

Thank you for the changes @khatibtamal Will take a look very soon.

@khatibtamal changes look good. Big thanks for this submission.

@muralibasani thanks for your review. Also do you have similar issues on your local when using Coral proxy, as mentioned in the second point Valid urls give no messages in this comment?

@khatibtamal changes look good. Big thanks for this submission.

@muralibasani thanks for your review. Also do you have similar issues on your local when using Coral proxy, as mentioned in the second point Valid urls give no messages in this comment?

@khatibtamal  will take a look at the issue soon.

authors: khatibtamal muralibasani programmiri

repo: Aiven-Open/klaw


text:
added image of app mockup

added image of mockup

Added image to assets folder


text:
Implement real agent using chains and remove python code

Real Agent can now be updated using UniChat's chains based on LangChain.

See https://github.com/AkiKurisu/UniChat/releases/tag/v0.0.6-alpha

Next Gen Dialogue dependency should be removed after using UniChat.

See https://github.com/AkiKurisu/Next-Gen-Dialogue/releases/tag/v1.0.1

Python code should be removed for cross platform since simple RAG can now be done in Unity directly.

Update in 48e0d87

repo: AkiKurisu/Real-Agents


text:
Having "Sugar crystal cookies" upgrade reduces CookiesPerSecond

Hello again!
With the new update having the "Sugar crystal cookies" prestige upgrade reduces the CookiesPerSecond when Cookie Monster is loaded, as displayed by the game itself below the cookies count.
Reloading Cookie Clicker reverts the CookiesPerSecond back to the original, correct number.
This affects the Game.cookiesPs variable as well, so it doesn't seem like it is only visual but a temporary change to the game calculations.
The upgrade gives a +5%, but gets counted only somewhat half with Cookie Monster on.
The upgrade also gives +1% for every lvl 10+ building, which doesn't seem to be counted at all with Cookie Monster on. Which leads to a much bigger percentage drop the more lvl 10+ buildings you have.
Reproduction:

Have "Sugar crystal cookies" upgrade
Load Cookie Monster
CookiesPerSecond drop

In the test save file below CookiesPerSecond are 100 nonillion per second, loading Cookie Monster drops it to 98.1 nonillion.
Removing the "Sugar crystal cookies" upgrade from the save gives a CookiesPerSecond of 95.242 nonillion, loading Cookie Monster it stays the exact same at 95.242 nonillion.
Test save files:

Test save file, all achievements and upgrades, with "Sugar crystal cookies" upgrade
[encoded data, 7452 characters]%21END%21

Test save file, all achievements and upgrades, - without - "Sugar crystal cookies" upgrade
[encoded data, 7463 characters]%3D%21END%21

Test save file, all achievements and upgrades, with "Sugar crystal cookies" upgrade and all buildings lvl 10
[encoded data, 8320 characters]%21END%21

Test save file, all achievements and upgrades, - without - "Sugar crystal cookies" upgrade and all buildings lvl 10
[encoded data, 8315 characters]%3D%21END%21

Screenshots:
With upgrade - Cookie Monster not loaded

With upgrade - Cookie Monster loaded

Without upgrade - Cookie Monster not loaded

Without upgrade - Cookie Monster loaded

Thanks! :-)

repo: Aktanusa/CookieMonster


text:
Show rankings on choices based on Payback Period

Describe the function you'd like to see added to CookieMonster
I could be wrong but it feels like in order to be most efficient with your cookies (money), you want to buy the item/building/addon/powerup/whatever with the lowest payback period. I love the color coded scale, but even when there are 4-5 actions stuck to choose from in the "yellow" or "orange" category, there isn't a clear way to easily see "ok, what is my number 1 smartest, most efficient move based on PBP"

maybe if you added 1, 2, 3, 4, 5, ... rankings next to payback period here to suggest next move
and/or also on hover

Doesn't the green building already show what you need to buy to be most efficient? If you buy that building it will automatically code the previous second best building as green.

https://user-images.githubusercontent.com/8949910/111695644-b48aa380-8809-11eb-9a8e-7f2f45c5f095.png
Take a look at this. How do you know which "yellow" is the best option to buy? You have to do mental less than math and observe the list over and over as it changes with every action. It is easier to sort through a list and find 1 than to try to compare every item to each other.
Look at this one:

best ranked option would be Alchemy here since it has the lowest PBP but there are actually 9 items in the "yellow" group

Could you share your save-file? I think I know what is going on but I would like to check!

[encoded data, 6978 characters]%3D%3D%21END%21

If you change your buy-mode to buy 10 you will see that in this save there is actually a "buy 10 option" that is better than any of the buy 1 options (specifically, Farm in this case). The yellow options you see when you are in buy 1 mode are indeed the top 2-10 and the buy 10 farm is the best option.
I don't think we need to show ranking together with the colors as any user that is looking for the most optimal strategy can always the green building. These will sometimes not be a "buy 1" option but rather a "buy 10" instead.

authors: DanielNoord brandonros


text:
Add widget(s)

This is arguably the most important feature in a weather app, atleast for me.
The Rain app just has a standard one in the oreo style, which I really like:

@Stradvr I ll think about it but dont know when yet. Will close this issue to prevent too many duplicate issues
duplicate of https://github.com/Akylas/oss-weather/issues/37

authors: Stradvr farfromrefug

repo: Akylas/oss-weather


text:
Interview issues

So pleased to have found this library after struggling with openzwave-shared. Thank you @AlCalzone for all the hard work you've put in on it.  However I have a question regarding node interviewing.
I have various types of device.  The controller and wall sockets report interview completed after a short while.  However I have door sensors an movement sensors that never report completed.  The cache file shows the interviewStage as ProtocolInfo.
I have tried setting node.keepAwake = true on each node and then triggering the sensors but I still don't see interview completed.
Am I doing something wrong?

The log output might show what's happening. Can you redirect that to a file and send it to me?

Hmm... Winston seems to be suppressing output if it's not directed to a terminal.

That behavior is in this library. I haven't found a good solution yet. Basically I want to suppress the output when it won't be seen anywhere.
You can trick it by setting the NODE_ENV environment variable to "test".

Setting NODE_ENV to "test" suppresses all logging.  See shared.ts line 239.
However, I did managed to get it to log with:
export function createConsoleTransport(): Transport {
	return new winston.transports.File({
		level: transportLoglevel,
                filename: 'winston.log'
          });
}

Please see attached.  For completeness I've attached the output, the cache file and my test code.
For reference, the types are:

Controller
Movement sensor
Door sensor

Wall socket

Not in use (needs deleting)
Not in use (needs deleting)

Wall switch
Wall socket

I can give you more precise details if it's useful.
Only nodes 1, 5, 11 and 12 have been interviewed correctly, despite triggering the three sensors.  11 interviewed after a button was pressed.
winston.log
f943537f.zip
zw.zip
Thanks again for your help.

Ok, so what I can see at a first glance:

Nodes 2, 3 are asleep, but never send a wake up notification, despite reporting changes. Can you try waking them up manually a couple of times?
Nodes 4 and 6 seem to be missing completely (unknown to the controller), but I guess you meant that by that symbol.
Node 7 never got the ping because it was dropped by the controller --> I should add some fallback for this case to https://github.com/AlCalzone/node-zwave-js/issues/403

Nodes 2, 3 and 7 never respond as interviewed, even when triggered multiple times during the start-up sequence.  Nodes 4, 6 and 10 were removed from the controller which does not appear to reuse these slots.
Nodes 8 and 9 also need deleting.  But I have found no way to delete them without the original devices.
Incidentally, rerunning this script multiple times, Node 11 still only interviews when I press it, even though it is shown as "interviewStage": "Complete" in the cache.
Any ideas how to move forward?

Let me explain the interview procedure and the concept of sleeping nodes first, maybe my answers become clearer then.
Battery powered nodes have a small window where they respond to controller messages (status awake). This window is announced by the node using a WakeUp message. After 10s without communication or after actively sending the node to sleep, the node won't respond (status asleep).
Even if a node sends us messages (sensor readings), that does not mean that it will respond to requests.
Mains powered nodes always respond, except if they are not reachable (status dead).

The controller "pings" every node to test if it is reachable/alive. Several things can now happen:

The node is not battery powered and responds.
The node is battery powered and responds. The messages to this node are prioritized so the node can go back to sleep sooner.
The node is battery powered and does not respond. The node gets marked as asleep and the controller waits until the node wakes up.
The node is not battery powered and does not respond. The controller marks it as dead.

The controller immediately interviews all nodes that are awake or not battery powered.

If a node was previously interviewed, some parts of the interview process are repeated (i.e. requesting current values).
Once the complete or partial interview has finished, the interview completed event is emitted.

As soon as a battery-powered node wakes up, step 2 is performed for that node.

Nodes 2, 3 and 7 never respond as interviewed, even when triggered multiple times during the start-up sequence

I'm not sure what the issue with nodes 2 and 3 is. As I wrote, they do send some messages, but they don't send the wake up notification. This means they are never marked as awake and the interview is never performed. Can you tell me which devices they are? Maybe a startup log from OpenZWave would also help.
The problem with node 7 (at least in the log you sent me) is that the controller timed while sending the ping, so the node never got a chance to respond. This is something I need to fix in this library.

This is the ozwcache file from the same controller which I think will give you all the information you need (and a lot that you don't).
When openzwave-shared starts, it takes some time (around 50 seconds) but eventually everything is recognised.  FYI, I am using a version from Github:  github:OpenZWave/node-openzwave-shared#7558a309879169ea8787405c82028ccd6e1aba26.
ozwcache_0xf943537f.zip
Thanks again for all your work on this (and coap/tradfri).

Thanks for the cache file, I'll have a look later. I'm also interested in the OZW_log.txt to see which messages are being exchanged by the nodes in question. Could you send me those too please?

Oops.  I had turned logging off.  But here is a short log.
OZW_Log.zip

From reading the log, OZW behaves exactly like this library. Both Node 2 and 3 fail to respond to the ping, so they are marked asleep and are not interviewed (at least not during the logged period).
Looking at the manual, node 2 has a default wake up interval of 2 hours and must be triggered using the B key to send a node information frame (which the driver interprets as "Hi, I'm awake!"). I guess you did that when you "triggered" it. If that doesn't work, simply waiting 2 hours should do it.

I had not triggered node 2 with the B key, simply ensured it had seen movement.  As you predicted, running the test code for 2 hours, node 2 was successfully interviewed.
I still haven't seen the door sensors: nodes 3 and 7.  In the cache file, their interviewStage is still ProtocolInfo.
Maybe I'm worrying too much about the interview completed?  I was working on the basis that I needed to wait for interview completed on the driver before continuing.  Should I just carry on regardless?  On openzwave-shared, I wait for scan complete which takes about a minute.

Maybe I'm worrying too much about the interview completed?

Yes and no. If the driver has interviewed the node before, it knows enough so you can work with it right away. However some values in the value DB might not be up to date yet.
If the interview stage is still ProtocolInfo, the node hasn't been interviewed yet.
Maybe I should add an event node ready similar to driver ready to let library users know its safe to perform operations on the node? That event would be triggered either immediately if the node was already interviewed or after the first interview is completed. However in the case of sleeping nodes, all commands will stay in queue until the node wakes up - so I'm not sure its very useful.
Your nodes 3 and 7 have a wakeup interval of 86400s:
<Value type="int" index="0" value="86400" max="2147483647" min="-2147483648" poll_intensity="0" verify_changes="false" write_only="false" read_only="false" units="Seconds" label="Wake-up Interval" instance="1" genre="system"/>

so I guess you need a lot of patience for the interview to complete if you don't want to wake it up manually.

I think a node ready would be a great idea.  Also an all nodes ready on the driver would be useful.  Then the interview becomes far less important.  I really don't want to have to wait 24 hours after restarting the server. :)
Is there any way I can remove nodes 8 & 9 from the controller without the actual devices?

@CliffS I created issues for the events (will be in the next release) and removing the failed nodes. Is the interview working for you now?

@AlCalzone Yes, if I leave it long enough, it interviews.  So I think the node ready will solve my problems and allow me to dump OpenZwave.
Actually, I have one more problem.  I'll open a separate issue shortly.

Alright, closing this issue then :)

authors: AlCalzone CliffS

repo: AlCalzone/node-zwave-js


text:
Download feature for excel: "We found a problem with some content..."

When I run the download excel example, and then I open the excel file, excel (2013) is always giving me a warning that,

We found a problem with some content in '....xlsx'. Do you want us to try to recover as much as we can? If you trust the source of this workbook, click Yes.

http://jsfiddle.net/agershun/00nfeq12/

Thanks for a well documented error.
@Agershun have you seen this before?

I see the same warning as @mg1075 .
Also, after I click Yes, and click 'Enable editing' there is a popup with a message:

Excel completed file level validation and repair. Some parts of this workbook may have been repaired or discarded.
Repaired Records: Worksheet properties from /xl/workbook.xml part (Workbook)

Thanks for letting us know. Looks like we have a general bug not related to the specific example.
We are looking into it.
Any help on what's wrong with the generates file is much appreciated...

I found what is wrong in generated Excel file.
Its worksheet is named Sheet[object Object], which contains prohibited characters [ and ].
I am not sure if it is supposed to be named like that (maybe a bug somewhere).
The file will open ok, when the worksheet has a proper name.

You are awesome !!!

You are welcome :)
I have no time to prepare a proper pull request for this issue right now, I leave it to you guys.
Before it is implemented, a workaround is to just provide name (sheetid option) when calling export.
Like this:

alasql('SELECT * INTO XLSX("filename.xlsx",{sheetid:"Sheet name",headers:true}) FROM ?',[data]);

Found the source to the bug: https://github.com/agershun/alasql/commit/20ac08ca84cb0993d3060f0a13fdf5f8b8a1a780
Please use
https://rawgit.com/agershun/alasql/11eb8f8a7424a5eb90c4c3a5cc41b1f55f239ef0/dist/alasql.min.js
while we prepare a new release

Hmmm... this will merge all the results into one.... Its not solved yet. Ill dig deeper...

@mathiasrw, Thank you for providing a link to a fixed version of the code.  Do you have any idea when a release including the fix will be pushed?

Hmmm. Right now should be a good time. Ill have a chat with @agershun about pushing the updates.

@mathiasrw, Thanks for pushing that release.

👍🏼

hhhhhhhh kkkkkkkkk

omg!

repo: AlaSQL/alasql


text:
Json isn't deserialized properly

Any playlist that has uppercase as the first letter will not be read, won't even show up in the playlists.
For example, playlistTitle works, but PlaylistTitle doesn't. Same goes for image, description, song hash etc.

Not affiliated with this project, but  tried to reproduce this, but it seems uppercase letters work fine. Do you literally mean that you changed the "playlistTitle" part? That is the key, not the value and that would indeed not work.

Yeah I mean the case sensitivity. It seems like BMBF processes things like this correctly though.

I finally understood what you meant by playlistTitle. As @SpellcheckerExtraordinaire said, the json key of that property should be case sensitive under normal condition. That's how it should be. Now it is possible to make it case insensitive but it's extra work to add inconsistency and I judge it to be unnecessary.
I'm more interested to know from where you got this PlaylistTitle with a uppercase P. Because if it's just one tool that generate it like this, I'm tempted to instead tell that author to change (or PR if open source). If it's multiple tools that use that uppercase P, I'd better make it more flexible here.
There's no proper definition for this playlist format, I had to look at how others were generating it.

I'm more interested to know from where you got this PlaylistTitle with a uppercase P. Because if it's just one tool that generate it like this, I'm tempted to instead tell that author to change (or PR if open source). If it's multiple tools that use that uppercase P, I'd better make it more flexible here.
There was this guy on bsaber that wrote a crawler for ranked maps and that's how I found it.

I checked how this plugin was loading the key. Judging quickly from the code, it doesn't seems it would load with the right title if beatlist was less restrictive.
https://github.com/rithik-b/PlaylistLoaderPlugin/blob/cf90997a158e2160134e5a4a2170ac37ffeaff98/PlaylistLoaderPlugin/LoadPlaylistScript.cs#L52
For now I will close the issue as wont-fix. I prefer keeping beatlist strict so once it load on beatlist, you can be confident it would load in the game as well.
As reminder there's this panel available in beatlist, a bit hidden but it is here.

repo: Alaanor/beatlist


text:
Multiple Requests in a Row

I'm making a request using Alamofire using the following code.
let headers: HTTPHeaders = [
	"myheader": "test",
]

Alamofire.request("http://localhost:3000/user", headers: headers).authenticate(user: "test@test.com", password: "testtedfnajksdst").validate().responseJSON { response in

On the server end I'm using Express.js and Node.js to handle the requests. For some reason tho my /user route gets called 3 times even tho I'm only calling Alamofire once. It seems to happen three times when I provide incorrect login information. The server responds with a 401 status code all 3 times. Only one of those requests actually includes the auth information tho.
When I provide the correct login information the route only gets called twice looks like. First time gives a 401 status code, second time responds with a 200 status code. After that if I make another request on the client side I get one 304 status code.
My question is why is the server showing multiple requests even tho on the client it looks like Alamofire is only making one request. Why also would only one request include the auth information? I'm using Xcode 8 on the iPhone Simulator 10.0

@fishcharlie This is due to URLSession providing authentication handling by automatically retrying failed requests. #1425 is another example of this issue.

authors: fishcharlie jshier

repo: Alamofire/Alamofire