text:
check_keystone_admin gets 404 from /healthcheck on xenial:mitaka

Keystone on xenial:mitaka is missing the endpoint that check_keystone_admin checks for. This endpoint is example.com/healthcheck. This endpoint is enabled through /etc/keystone-paste.ini option by adding:
[filter:healthcheck]
use = egg:oslo.middleware#healthcheck
This is missing from xenial:mitaka, thus this check is invalid and returns a 404.
One possible solution is adding a check to see if keystone version is > mitaka. If so, one possible endpoint to check is just / which gives some info (including a very brief status field).

Imported from Launchpad using lp2gh.

date created: 2019-08-08T19:58:49Z

owner: zzehring

assignee: None

the launchpad url

(by eric-chen)
Close this very old issue. Welcome to reopen it if it is still valid.


text:
requirer/provider pattern

It seems to me that this collection of interfaces is biased towards the provider/requirer pattern.
Which is perfectly fine, but we could explain/state this more clearly in the toplevel README.
Are we only going to spec here provider/requirer interfaces? Or also other types of interfaces? (e.g. rolling-ops or other 'relation models')?
If so: we can make this clear in the toplevel readme and each interface's own readme doesn't have to repeat it or rephrase it.
If not: we have to be more clear in how we state it on a per-interface basis, and realize that that will have an effect on how the testers are setup.

You're not talking nonsense, but I don't think the patterns you describe all fit within what we already have. For instance, a pub-sub model would be a provider/requirer where one of them exposes a list of unacknowledged messages and the other end is expected to ack them as they have been processed.

source: gharchive/issue

authors: PietroPasotti simskij

repo: canonical/charm-relation-interfaces

license_type: permissive

license_source: github-api


text:
Sidecar rewrite: envoy

Context
We rewrite all of our charms using the sidecar with base charm pattern instead of the old podspec.
What needs to get done
Rewrite the charm using sidecar with base charm pattern.
Definition of Done
Charm is rewritten with sidecar with base charm pattern.
All of the tests are rewritten and passing.

When we work on this task, we have to have in mind that we will need to migrate the k8s-service relation interface from its SDI implementation to the newer one handled by a library. See https://github.com/canonical/mlops-libs/issues/2 for more information about the library and what it entails.

This'll take a bit more effort than usual because of https://github.com/canonical/charmed-kubeflow-chisme/issues/89.  We need to solve https://github.com/canonical/charmed-kubeflow-chisme/issues/89 before we can write Envoy using the CharmReconciler because we need to push a config file to the container that requires inputs from other components.
I have a working solution to this that wont be a huge effort, but it still adds maybe an extra half a day or so to the rewrite

authors: DnPlas ca-scribner misohu

repo: canonical/envoy-operator


text:
Create a spec for NGC container Notebook integration

What needs to get done
A spec that outlines the context, implementation details and the user experience of how we will enable running a Notebook in Charmed Kubeflow using and NGC contaienr.
The implementation section must include:

The definition of the PodDefault that configures JupyterLab
Explain how the PodDefault gets applied (via config? via relation to resource dispatcher? charm action?)
what namespace the PodDefault is applied in?

Why it needs to get done
To define and discuss the epic of NGC container Notebook integration.

This task has been already completed. The spec has been reviewed and approved by @ca-scribner, @kimwnasptd and @DnPlas.

authors: DnPlas NohaIhab

repo: canonical/notebook-operators


text:
JavaFX basic view: some refactoring

This changes are based on some use cases.
Added some JavaDoc
Classes are more easy to use.

Reviewed 3 of 3 files at r1.
Review status: all files reviewed at latest revision, 2 unresolved discussions, some commit checks failed.

java-client-javafx/src/main/java/com/canoo/dolphin/client/javafx/AbstractFXMLViewBinder.java, line 70 [r1] (raw file):
typo s/Thsi/This/

java-client-javafx/src/main/java/com/canoo/dolphin/client/javafx/FxmlLoadException.java, line 1 [r1] (raw file):
missing headers, serialVersionUID

Comments from the review on Reviewable.io

Review status: 1 of 3 files reviewed at latest revision, 2 unresolved discussions.

java-client-javafx/src/main/java/com/canoo/dolphin/client/javafx/AbstractFXMLViewBinder.java, line 70 [r1] (raw file):
Done.

java-client-javafx/src/main/java/com/canoo/dolphin/client/javafx/FxmlLoadException.java, line 1 [r1] (raw file):
Done.

Comments from the review on Reviewable.io

source: gharchive/pull-request

authors: aalmiray hendrikebbers

repo: canoo/dolphin-platform

license_source: bigquery


text:
High CPU usage, seemingly related to watcher notifications about folders

For various reasons, I want to synchronize a Windows user home folder from a laptop to another computer.  (I'm perfectly aware that sync is not backup, and I have some specific reasons for wanting to do this anyway.)
Unfortunately this has led to comparatively high CPU usage for SyncTrayzor / Syncthing.  Even when the computer is left idle, and most files that are regularly modified are ignored, the sync process uses an average of about 15%-20% of one core (i7-4600u at 2.70 GHz).  By "average" I mean that there are bursts where it uses a full core and idle intervals inbetween.
Turning on some of the Syncthing debug log options shows that changes to individual files are handled quickly: Going from "Walk..." to "Walk progress done..." typically takes 10 to 20 milliseconds.  But then the file watcher also quite often reports changes to folders such as the home folder itself:

[YXMEM] 2017/08/26 20:15:15.006664 walk.go:102: DEBUG: Walk \?\C:\Users\jonkv82 [] 131072 &{...}

I don't know what causes this (what actually happened to the jonkv82 folder to cause a notification).  But after this message, it usually takes up to 18-20 seconds of high CPU usage until we get the corresponding "Walk progress done".  This is consistent with walking the entire subtree of about 140,000 files due to what I assume is some kind of change in the folder itself.  This seems to happen rather regularly once every 15 minutes, plus some "random" times in between.
Similarly, for AppData:

[YXMEM] 2017/08/26 20:19:09.323512 walk.go:102: DEBUG: Walk \?\C:\Users\jonkv82 [AppData] 131072 &{...}

This typically takes about 11-12 seconds for 90,000 files, and can happen more often.
I took a brief look at the Syncthing source in walk.go where these messages are generated.  I've never used Go before so I could be mistaken, but it seems that Filesystem.Walk does walk the entire directory tree beneath my home folder or beneath AppData.  I assume this is in fact not necessary because if there were other modified files beneath these folders, the file system watcher would specifically notify SyncTrayzor about each file without the need for a full directory traversal?
I'd be happy to provide any additional information that might be useful.

If we see several changes to files in a folder, then we tell Syncthing to rescan the folder. This has the upside that we don't constantly bombard it with requests if there are lots of changes, but the downside that if you have a folder with many many files, Syncthing will end up doing unnecessary scanning. There are other cases which will cause a scan of the folder, such as a file being deleted (or moved).
Syncing your whole home directory is normally a bad idea. Things like ntuser.dat, and all of your temporary files, live in your home directory. This includes application-specific data, and applications normally don't like that being changed under them. It also includes things like the SyncTrayzor log file, which will of course log every time SyncTrayzor records a change to the log file....
I'd instead recommend syncing specific directories (you can use ignores to ignore everything except some specific folders). A less preferable option is to ignore files which shouldn't be synced (ntuser.dat, etc), and all temporary frequently-changing files.

I see.
To clarify my use case, I currently use Crashplan to back up my laptop to several other computers with a 5-minute interval, but they are cancelling their service for home users and I need an alternative.  I'm using BorgBackup for many other purposes, but its Windows version is not well tested and it needs to rescan the entire folder hierarchy every time it runs, which isn't well suited for a more "continuous" form of backup on a laptop where you might save a file and then put the laptop to sleep a minute later.  Many other backup programs seem to have the same weakness relative to Crashplan.
Therefore I thought it would be an interesting experiment to (1) run SyncTrayzor to immediately mirror everything of importance to a stationary computer that's permanently on, and then (2) run BorgBackup to back up the mirrored folder on the stationary computer at regular intervals, where I don't have to worry about power usage or about the computer shutting down before the entire folder is scanned and all files are backed up.  Kind of a weird setup but the experiment actually seems to be working pretty well so far, apart from the CPU usage.  An additional advantage over many backup programs is that Syncthing takes care of finding my computer despite its dynamic IP address, opening ports in the firewall, and so on.
I have set up various exclusions so that temporary files and other "non-restoreable" files are not mirrored.  I have also ensured the folder is "send only" on the laptop because the intention is to never synchronize anything back to the home folder.  In case of a disaster I'll selectively restore files from the BorgBackup archives.
Setting up individual folders would perhaps be a workaround but as far as I can tell this has some drawbacks.  First, to actually decrease the number of files scanned, I'd have to set up a rather large number of individually synchronized folders -- at least one for each folder under AppData\Roaming, and then there are plenty of other folders of interest as well.  Second, I'd have to add new folders every time something new appears.  For example, I use IntelliJ IDEA which sets up a new config folder for each new major version, three times per year.  I'd probably forget to do this at some point.  Having one main folder with explicit exclusions would work better for me in this specific use case, and if I happen to include something that shouldn't be restored, it won't really hurt that it's part of the backup archives.
Thanks for the information that SyncTrayzor consolidates multiple calls into a single folder update!  I have excluded a number of frequently changing files and thought I had basically caught them all, but I found them by analyzing the Syncthing log entries which won't work if SyncTrayzor has already consolidated entries before Syncthing can log them.  I will look for other ways of finding that information and maybe this will reduce the scan frequency.  I'll comment here as soon as I've tested (probably tomorrow).

I'm trying to understand exactly how the notification logic works relative to .stignore.  The main github page for SyncTrayzor says that "Folder watching respects the ignores configured in Syncthing".  Could you point me to the place in the code where these ignores are applied?
As far as I can tell almost all files that have been modified on my laptop during the last hour are actually ignored by .stignore, and this is confirmed by the "global changes" log that shows very few changes are actually propagated. Despite this the folders are scanned very often, far more often than any files are transferred.  If the .stignore patterns are applied after the notification logic in DirectoryWatcher.OnPathChanged finds a common prefix for all changed files, then it would explain what I'm seeing here:  A few ignored files are modified, the file watcher tells SyncTrayzor, we don't know yet that those files are ignored, their only common prefix turns out to be [AppData], and [AppData] isn't ignored so the entire folder is scanned.  But that is just speculation so far.

Looks like that bit of the README is out of date: Syncthing stopped telling me its ignores, so I stopped being able to apple them.

That explains a lot!  Thanks for the info about adding notification support directly to Syncthing.  I'll wait and see what happens there.
Meanwhile I've moved a couple of larger folders (in terms of the number of files they contain) to a different location.  Since they are no longer nested within the main home folder I can sync them separately, which seems to save a lot of time. Then I created a link in the home folder and ignored the link in .stignore.
I'll have to test this experiment for a while longer to see how it works, but given the circumstances this issue should probably be closed.
Thanks again for the very quick answers!

license: mit


text:
Improve vim comipler and add instruction of machineout reporter

Improve sample vim compiler script and add some instruction of machineout reporter to end of the Readme.md

I decide to release nodeunit vim compiler as nodeunit.vim on my github so fix README and remove share/nodeunit.vim

repo: caolan/nodeunit

license: MIT


text:
Do not fail if symlink exists

Stack traces from customers indicate that symLink() fails with EEXIST, which would mean it wasn't deleted by the previous unlinkSync.
npm ERR! Error: EEXIST: file already exists, symlink '../@cap-js/cds-types' -> '/tmp/app/node_modules/@types/sap__cds' (STDOUT, STG)#
npm ERR!     at Object.symlinkSync (node:fs:1810:11) (STDOUT, STG)#
npm ERR!     at Object.<anonymous> (/tmp/app/node_modules/@cap-js/cds-types/scripts/postinstall.js:28:4) (STDOUT, STG)#

Fix is to ignore EEXIST errors.

Need to merge this to unblock a customer

repo: cap-js/cds-types


text:
Compatibility with Capacitor 5

Describe the bug
The plugin has configured source & target compatibility with JDK 11 (https://github.com/capacitor-community/photoviewer/blob/main/android/build.gradle#L45) as well as kotlinOptions with jvmTarget 17 (https://github.com/capacitor-community/photoviewer/blob/main/android/build.gradle#L49). Capacitor 5 requires AGP 8, which has JDK 17 as a prerequisite.
To Reproduce
Install the plugin in an app with capacitor 5 (or upgrade an existing project with the plugin to capacitor 5).
Expected behavior
Installation should work as expected. Either by no longer specifying the source & target compatibility within the plugin or by changing to JDK 17.
Additional context
Additionally, the package name should be moved from the AndroidManifest.xml file to build.gradle in the android DSL with namespace (though some capacitor 5 auto-magically does this with a migration script for plugins which have not already done it themselves). This is a breaking change with AGP 8. And the peer dependencies should be updated to signal compatibility with capacitor 5.

@adnbrownie it has not been converted yet to Capacitor5

When is the conversion to Capacitor 5 scheduled to happen?

@anthonyplews @adnbrownie You have a beta version published under the tag next can you please test it and revert to me in case of issues or success.

@jepiqueau I can confirm that it works on both Android and iOS, thanks for the quick response.

@anthonyplews thanks for your feedback i will publish it soon as the latest release

closing since version 3.0.0 of the plugin was released with Capacitor 5 support

authors: adnbrownie anthonyplews jcesarmobile jepiqueau

repo: capacitor-community/photoviewer


text:
fix: Fix Turing replicaCount value

Motivation
Replica count for the Turing app was wrongly read from the values file, in the deployment Helm template.
Modification

charts/turing/templates/turing-deployment.yaml - Fix the replicaCount spec.
charts/turing/templates/turing-ingress.yaml - Fix the ingress annotations parsing and indentation
charts/turing/tests/turing_ingress_test.yaml - Add unit test for ingress

Other Modifications

.github/workflows/test.yaml - Putting back (changed in a recent commit) and pinning the quintush/helm-unittest plugin version to v0.2.11 as the latest commit on the main branch is broken and has some regressions in the matchRegex assertion (paths like stringData.[config.yaml] where there is a dot in the path, are not processed properly anymore). Since the latest commit in the plugin repo need not always be stable (as with this case), perhaps we can keep the plugin version pinned and bump it from time to time.

Logged the issue here: https://github.com/quintush/helm-unittest/issues/207
Checklist

[x] Chart version bumped
[x] README.md updated

Hey @caraml-dev/platform, this is a small bugfix PR for the Turing chart that will unblock the release process. Could you give it a pass? Thanks!

repo: caraml-dev/helm-charts


text:
Feat: add useRouter property to sidenav

Closes carbon-design-system/carbon-components-angular#
{{short description}}
Changelog
New

{{new thing}}

Changed

{{change thing}}

Removed

{{removed thing}}

Apologies, this MR was not ready yet to be pointed at the main repo.


text:
chore(devenv): fix dev env for older browsers

Overview
Fixes #640. Sorry @alisonjoseph for this!
Removed
Inadvertent addition of object spread operator, etc. in <script> tag in .dust
Testing / Reviewing
Testing should make sure the dev env is not broken.

Thanks @alisonjoseph for reviewing! Merging. @alisonjoseph and everybody feel free to let me know if any of you find other issues here!

repo: carbon-design-system/carbon-components


text:
Change the default setting for modal videos to be unmuted

The problem
Currently, the modal video player is muted by default. See videos on https://www.ibm.com/testing/services/ for reference. From usability perspective, this requires users to perform an extra click of unmuting to hear the videos. This request is to change the default setting to unmuted.
The solution
Change default setting of video player to unmuted and playing at volume around 20-40%.
Default:  change to unmuted, with option for user to unmute
Additional information
For more info, please reach out to Jina Yoon.

Kenny to reach out to Jina for more info.

Hi @lily-kenney @jyoon25, inline embedded videos currently aren't muted by default, whereas videos embedded in a modal are. The difference is because videos in a modal autoplay and best practices are that videos which autoplay should be muted. But looking at it now, I'm not sure videos in a modal need to be muted even if they autoplay, since the modal is explicitly triggered by the user.
@wonilsuhibm @jeffchew thoughts?

@kennylam  Thanks for confirming!
I think autoplay with muted is right action because it can prevent several video audio crushes.

@kennylam @jyoon25
I wonder if the video should NOT auto-play in a modal:

Muted autoplay is a clear cut case for embedded autoplaying videos,
But in this case, the user did click a button with the intent to watch the video. So muting feels wrong.
However, the opening of the modal feels like a big change and could be a bit jarring.

With these things considered, I am wondering if the video should not autoplay when opening in the modal.
@photodow @oliviaflory @JennySanchez @jacobottesen-dgc @jeffchew any thoughts?

@wonilsuhibm Agreed with the video should NOT auto-play or muted in a modal.

Closing.  We are going to leave the design in place.

authors: JennySanchez RobertaJHahn jyoon25 kennylam lily-kenney wonilsuhibm


text:
feat(tutorial): complete step 5

Closes #
https://carbon-tutorial-codeline128.mybluemix.net
Changelog
New

{{new thing}}

Changed

{{change thing}}

Removed

{{removed thing}}

Per instructions in the tutorial step 5, please include your mybluemix.net URL for your deployed app in your pull request description.

Hello @kellychurchill, I have updated the pull request comment including my Bluemix URL

👍  Looks good!
Feel free to stop this running application on IBM Cloud to avoid the possibility of incurring any charges 💵
Once all your steps are approved, please see the wrapping up step to apply for your badge!
https://www.carbondesignsystem.com/tutorial/wrapping-up

repo: carbon-design-system/carbon-tutorial


text:
Complete Step 4

Closes #
{{short description}}
Changelog
New

{{new thing}}

Changed

{{change thing}}

Removed

{{removed thing}}

cancel


text:
feat(boe): complete step 2

Closes 2. Building pages
Complete second step of the tutorial
Changelog
New

Build a few static pages

Changed

Extend app.scss by import font-family (required after upgrade of carbon-components to v10.3.0)

@sergey-siuchanka looks spectacular 🔝! I've recorded your completion of step 2.

authors: ZrianinaMariia sergey-siuchanka


text:
[Button] Disabled buttons emit click event

Description
Click event is triggered on disabled buttons when clicking the button text.
Steps to reproduce the issue

Visit https://web-components.carbondesignsystem.com/?path=/story/components-button--default
Check "disabled" in Knobs
Open Actions tab
Click button background (no click event is triggered)
Click button foreground text (click event is triggered)

These are Storybook actions only. onClick needs to be set when authoring a page, whether the component is disabled or not.

No this is a problem in actual production applications. the onClick event is fired when clicking a disabled button, but only when clicking the foreground text. It should not be fired at all when disabled.

Weighing in; It is just my hunch but I have an impression that the disabled state of button in shadow DOM is not applied to the slotted light DOM child. Need PoC test case to see it's correct or not, though.

@asudoh @RobertaJHahn @kennylam Can you please re-open this issue since it is indeed valid? Thanks!

authors: asudoh frodesigns kennylam


text:
fix(spacing): fix spacing issues around the site

Closes https://github.com/carbon-design-system/carbon-website/issues/1258
Closes https://github.com/carbon-design-system/carbon-website/issues/1259
Closes https://github.com/carbon-design-system/carbon-website/issues/1067

Fixed spacing on homepage tiles

Fixed spacing on top anchor links

Fixed spacing images, aligned the rows to the top

Ensured all h2-h4 are display: inline-block; so they properly respect margins placed on them

Much better, real quick:

the distance between the intro paragraph and the anchor links needs to be 32px (not 24px) sorry, my bad, looks like it's still 24px

Also, distance between anchor links and the first H2 (Carbon and accessibility) still looks too large by about 8px... can we just double check that it's 80px?

We miss you! But we know that you're following your heart by moving to Humbolt county to trim weed.

Hey guys, sorry I left this comment in #1259
This margin bottom distance is 24px now and i just need to change it to 32px! Thanks!

merging this in. will add spacing fix in another PR

@alisonjoseph Sorry I'm just looking after this for Jeanie so hopefully I'm not misunderstanding the goal here. Jeanie madea  comment that this spacing below should be 32px, or 2rem. It's currently 1rem it seems.

@tw15egan looks like one little update then we can merge this in

authors: alisonjoseph jeanservaas shixiedesign tw15egan

repo: carbon-design-system/carbon-website


text:
Revert spacing changes in content block segmented

The problem
After talking to @ltitle, we realized that with updates from #2620 that we might not need the variant spacing options in the content block segmented that we originally identified, and instead Drupal can use content block mixed or content block media instead now that it has the optional copy.
The solution
We just want to revert the spacing that was hardcoded in https://github.com/carbon-design-system/ibm-dotcom-library/commit/0246fb66e6236ed8dc0003d9552b9633cde1c76e#diff-288b4cfefcc3ec25bbec0d037c68e5d1R17-R24
It was supposed to be a variant originally, so there was going to be an updated needed to fix that anyway, but according to @ltitle we don't need it anymore.
Additional information

#2620
#2396

There might be a small change to functional specs here that need to be made.
https://ibm.box.com/s/k1s20tmml53qfcie7bavgb0h3pap8211

repo: carbon-design-system/ibm-dotcom-library


text:
Fix/masthead search

Related Ticket(s)
#117, #156
Description
Addresses various focus/active styles for search buttons and inputs. Also fixes search input and suggestion interactions.
Changelog
Changed

add search icon focus and active styles
improves active/inactive states and transitions for search input and suggestions
various css fixes
adds missing component props

@oliviaflory Since Lara has her hands full, could you help give a quick review of the related tickets? Thanks!


text:
feat: side panel for create and edit + type check

Closes #3752
Updates to the side panel AI expanding it's usage to the patterns.
What did you change?

SidePanel to add a type check to the slug and to mark the prop as Experimental as per Carbon/React.
CreateSidePanel added feature
EditSidePanel added feature
Added design updates which included moving the

How did you test and verify your work?
Storybook.

See #3752 in which @aagonzales and @Laura-Marshall comments regarding close button position.

repo: carbon-design-system/ibm-products


text:
CIP-0068 | question on reference code

https://github.com/cardano-foundation/CIPs/blob/5bcdb9b679f35a28801adedfd7114090c6bc5472/CIP-0068/ref_impl/offchain.ts#L66
Looking at the reference Code provided for CIP-68, Im not sure i understand this line and what is used for, it could be clarified.

@alessandrokonrad would you care to field this one? 🙏

@jerrybassat7 this line has nothing to do directly with CIP-68. The example minting policy requires to spend a certain UTxO to create a unique NFT.

authors: alessandrokonrad jerrybassat7 rphair

repo: cardano-foundation/CIPs


text:
CEXIO getTrades() returned duplicate results

Hello,
I'm trying to use CEX.IO but am met with an error. I've looked around and noticed another exchange was fixed by changing it from 2 hours to 12. Any idea why this error would happen? It seems even after wiping the collection and trying again it still fails at the same spot.
fetching pre-roll data:

skipping 10457 hrs of previously collected data

cexio.BTC-USD saved 1000 trades 750 days left
.
error: getTrades() returned duplicate results
{ product_id: 'BTC-USD', from: 376024 }
{ product_id: 'BTC-USD', from: 376024 }
               DATE            PRICE     DIFF            VOL         RSI                         ACTIONS                      BAL                PROFIT
2017-10-17 21:00:00  5621.85 BTC-USD                3
2017-10-17 21:30:00  5587.00 BTC-USD   -0.62%       48
2017-10-17 22:00:00  5580.00 BTC-USD   -0.13%       16
2017-10-17 22:30:00  5589.96 BTC-USD   +0.18%       44
---------------------------- STARTING LIVE TRADING ----------------------------
Press  l  to list available commands.
/home/hype/zenbot/extensions/exchanges/cexio/exchange.js:77
        balance.currency = n(body[opts.currency].available).format('0.00000000')
                                                 ^

TypeError: Cannot read property 'available' of undefined
    at /home/xxx/zenbot/extensions/exchanges/cexio/exchange.js:77:43
    at Request.request [as _callback] (/home/xxx/zenbot/node_modules/cexio-api-node/rest.js:67:16)
    at Request.self.callback (/home/xxx/zenbot/node_modules/cexio-api-node/node_modules/request/request.js:186:22)
    at emitTwo (events.js:125:13)
    at Request.emit (events.js:213:7)
    at Request.<anonymous> (/home/xxx/zenbot/node_modules/cexio-api-node/node_modules/request/request.js:1163:10)
    at emitOne (events.js:115:13)
    at Request.emit (events.js:210:7)
    at IncomingMessage.<anonymous> (/home/xxx/zenbot/node_modules/cexio-api-node/node_modules/request/request.js:1085:12)
    at Object.onceWrapper (events.js:314:30)
    at emitNone (events.js:110:20)
    at IncomingMessage.emit (events.js:207:7)
    at endReadableNT (_stream_readable.js:1057:12)
    at _combinedTickCallback (internal/process/next_tick.js:138:11)
    at process._tickCallback (internal/process/next_tick.js:180:9)

I am having this same issue. Any help would be greatly appreciated.

Hello, I'm aware of this issue and have fix for it. Will prepare PR, but DeviaVir is on vacation, so I'm testing for more bugs.
Quick fix at #75 change to:
if (err || typeof body === 'undefined' || body === 'error: Nonce must be incremented') return retry('getBalance', func_args, err)

@nedievas Didn't seem to fix it.
skipping 10457 hrs of previously collected data

cexio.BTC-USD saved 1000 trades 750 days left
.
error: getTrades() returned duplicate results
{ product_id: 'BTC-USD', from: 376024 }
{ product_id: 'BTC-USD', from: 376024 }
               DATE            PRICE     DIFF            VOL         RSI                         ACTIONS                      BAL                PROFIT
2017-10-17 21:00:00  5621.85 BTC-USD                3
2017-10-17 21:30:00  5587.00 BTC-USD   -0.62%       48
2017-10-17 22:00:00  5580.00 BTC-USD   -0.13%       16
2017-10-17 22:30:00  5589.96 BTC-USD   +0.18%       44
---------------------------- STARTING LIVE TRADING ----------------------------
Press  l  to list available commands.
/home/hype/zenbot/extensions/exchanges/cexio/exchange.js:78
        balance.currency = n(body[opts.currency].available).format('0.00000000')

Note The line is moved down one due to a comment I added.

Check if cexio-api-node module is 1.0.4 version

avery@vps130953:/home/hype/zenbot$ npm view cexio-api-node version
1.0.4

skipping 10457 hrs of previously collected data

cexio.BTC-USD saved 1000 trades 751 days left
.
error: getTrades() returned duplicate results
{ product_id: 'BTC-USD', from: 376024 }
{ product_id: 'BTC-USD', from: 376024 }
               DATE            PRICE     DIFF            VOL         RSI                         ACTIONS                      BAL                PROFIT
2017-10-17 21:00:00  5621.85 BTC-USD                3
2017-10-17 21:30:00  5587.00 BTC-USD   -0.62%       48
2017-10-17 22:00:00  5580.00 BTC-USD   -0.13%       16
2017-10-17 22:30:00  5589.96 BTC-USD   +0.18%       44
---------------------------- STARTING LIVE TRADING ----------------------------
Press  l  to list available commands.
/home/hype/zenbot/extensions/exchanges/cexio/exchange.js:79
        balance.currency = n(body[opts.currency].available).add(body[opts.currency].orders).format('0.00000000')
                                                 ^

TypeError: Cannot read property 'available' of undefined
    at /home/hype/zenbot/extensions/exchanges/cexio/exchange.js:79:50
    at Request.request [as _callback] (/home/hype/zenbot/node_modules/cexio-api-node/rest.js:67:16)
    at Request.self.callback (/home/hype/zenbot/node_modules/cexio-api-node/node_modules/request/request.js:186:22)
    at emitTwo (events.js:125:13)
    at Request.emit (events.js:213:7)
    at Request.<anonymous> (/home/hype/zenbot/node_modules/cexio-api-node/node_modules/request/request.js:1163:10)
    at emitOne (events.js:115:13)
    at Request.emit (events.js:210:7)
    at IncomingMessage.<anonymous> (/home/hype/zenbot/node_modules/cexio-api-node/node_modules/request/request.js:1085:12)
    at Object.onceWrapper (events.js:314:30)
    at emitNone (events.js:110:20)
    at IncomingMessage.emit (events.js:207:7)
    at endReadableNT (_stream_readable.js:1057:12)
    at _combinedTickCallback (internal/process/next_tick.js:138:11)
    at process._tickCallback (internal/process/next_tick.js:180:9)
``` Updated the file. This is what I'm getting.

Updated but still getting this
`fetching pre-roll data:
skipping 10457 hrs of previously collected data
cexio.BTC-USD saved 1000 trades 751 days left
.
error: getTrades() returned duplicate results
{ product_id: 'BTC-USD', from: 376024 }
{ product_id: 'BTC-USD', from: 376024 }
DATE            PRICE     DIFF            VOL         RSI                         ACTIONS                      BAL                PROFIT
2017-10-17 21:00:00  5621.85 BTC-USD                3
2017-10-17 21:30:00  5587.00 BTC-USD   -0.62%       48
2017-10-17 22:00:00  5580.00 BTC-USD   -0.13%       16
2017-10-17 22:30:00  5589.96 BTC-USD   +0.18%       44
---------------------------- STARTING LIVE TRADING ----------------------------
Press  l  to list available commands.
/home/hype/zenbot/extensions/exchanges/cexio/exchange.js:79
balance.currency = n(body[opts.currency].available).add(body[opts.currency].orders).format('0.00000000')
^
TypeError: Cannot read property 'available' of undefined
at /home/hype/zenbot/extensions/exchanges/cexio/exchange.js:79:50
at Request.request [as _callback] (/home/hype/zenbot/node_modules/cexio-api-node/rest.js:67:16)
at Request.self.callback (/home/hype/zenbot/node_modules/cexio-api-node/node_modules/request/request.js:186:22)
at emitTwo (events.js:125:13)
at Request.emit (events.js:213:7)
at Request. (/home/hype/zenbot/node_modules/cexio-api-node/node_modules/request/request.js:1163:10)
at emitOne (events.js:115:13)
at Request.emit (events.js:210:7)
at IncomingMessage. (/home/hype/zenbot/node_modules/cexio-api-node/node_modules/request/request.js:1085:12)
at Object.onceWrapper (events.js:314:30)
at emitNone (events.js:110:20)
at IncomingMessage.emit (events.js:207:7)
at endReadableNT (_stream_readable.js:1057:12)
at _combinedTickCallback (internal/process/next_tick.js:138:11)
at process._tickCallback (internal/process/next_tick.js:180:9)
`

Looking into it. It backfills to much data. I get error: Rate limit exceeded on BTC-EUR.

Should be OK now. #644

The problem is not linked with error: Rate limit exceeded message.
I got the same error on the same trade 376024.
root@fa4488802bab:/app# node zenbot.js backfill cexio.BTC-USD

skipping 10457 hrs of previously collected data

cexio.BTC-USD saved 1000 trades 757 days left
.
error: getTrades() returned duplicate results
{ product_id: 'BTC-USD', from: 376024 }
{ product_id: 'BTC-USD', from: 376024 }


text:
Support forcing basic authentication for git

Describe the problem/challenge you have
Using older versions of Team Foundation Server (now renamed to Azure DevOps Server, no relation to Azure) require git basic authentication, which requires a form of tricking the git cli that vendir uses in order to properly authenticate.  Specifically, Use a PAT is asking for -c http.extraHeader="Authorization: Basic ${B64_PAT}" to be passed to the git clone step for cloning a repository.
Describe the solution you'd like
If vendir detects the presence of an environment variable, like VENDIR_GIT_ARGS or something similar then those are added to the git commands that are executed.
Alternatively, and probably more robust, would be adding this directly into the vendir sync, although I'm unsure of how to pass credentials through that process then.  Also, since my ultimate solution needs to work in kapp-controller, that is going to require updates to the App CR spec to support as well.
Anything else you would like to add:
I am unsure of how else to trick git into properly using basic authentication, it looks like Microsoft doesn't know either as that link is their official documentation.

Vote on this request
This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.
👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"
We are also happy to receive and review Pull Requests if you want to help working on this issue.

In argocd they faced the same issue. Their solution was to add the auth in an environment variable.
For vendir I think we could expand the spec with a field in the git section called
forceHTTPBasicAuth: false # by default it will be false
And the behavior could be if this variable is set to true and you provide HTTP Basic auth we would set the extra header via -c flag in git.
What do you think? Also if you need this on kapp-controller and we decide to follow the above approach we might need an issue there to bubble up this feature.

This should now allow https://github.com/carvel-dev/kapp-controller/issues/1485 to proceed.  :champagne: :tada:

authors: joaopapereira voor

repo: carvel-dev/vendir


text:
fix: Add netlify deploy and pr ci

Signed-off-by: ErikQQY 2283984853@qq.com
Fix: #5

@hsluoyz Plz review

repo: casdoor/casdoor-website


text:
Update user address failed

Whether I'm in a local docker or demo environment, when I update user address, it alert

json: cannot unmarshal string into Go struct field User.address of type []string

maybe the data type of Go is not match

User.address is a string array. The web UI should be modified to support "Address line 1" and "Address line 2" like below. So the update will succeed.

repo: casdoor/casdoor


text:
When call Duktape.close() , native crash occured.

**I'm using duktape-android in Thread and multi.
when I run duktape-android about over 300 times, out of memory occured.
So I called Duktape at end of script.
When I called Duktape.close(), native crash occured.
Below is the log.**
12-18 10:00:12.897 29666-714/com.kwic.smartaibmonitormaster E/libc++abi: Pure virtual function called!
12-18 10:00:12.898 29666-714/com.kwic.smartaibmonitormaster A/libc: Fatal signal 6 (SIGABRT), code -6 in tid 714 (Thread-4)
                                                                [ 12-18 10:00:12.899   476:  476 W/         ]
                                                                debuggerd: handling request: pid=29666 uid=10207 gid=10207 tid=714

12-18 10:00:12.972 759-759/? A/DEBUG: *** *** *** *** *** *** *** *** *** *** *** *** *** *** *** ***
12-18 10:00:12.973 759-759/? A/DEBUG: Build fingerprint: 'lge/lucye_lgu_kr/lucye:7.0/NRD90U/172411421f9f3:user/release-keys'
12-18 10:00:12.973 759-759/? A/DEBUG: Revision: '12'
12-18 10:00:12.973 759-759/? A/DEBUG: ABI: 'arm64'
12-18 10:00:12.973 759-759/? A/DEBUG: pid: 29666, tid: 714, name: Thread-4  >>> com.kwic.smartaibmonitormaster <<<
12-18 10:00:12.973 759-759/? A/DEBUG: signal 6 (SIGABRT), code -6 (SI_TKILL), fault addr --------
12-18 10:00:12.973 759-759/? A/DEBUG:     x0   0000000000000000  x1   00000000000002ca  x2   0000000000000006  x3   0000000000000008
12-18 10:00:12.973 759-759/? A/DEBUG:     x4   ffffffffffffffff  x5   8080800000000000  x6   0000000000808080  x7   fefeff2063646b6b
12-18 10:00:12.973 759-759/? A/DEBUG:     x8   0000000000000083  x9   ffffffffffffffdf  x10  0000000000000000  x11  0000000000000001
12-18 10:00:12.973 759-759/? A/DEBUG:     x12  ffffffffffffffff  x13  0000000000000000  x14  0000000000000000  x15  00357c6e098c12a3
12-18 10:00:12.973 759-759/? A/DEBUG:     x16  00000075b8551ed0  x17  00000075b84fb3f4  x18  00000000ffffffff  x19  00000075ac8c14f8
12-18 10:00:12.973 759-759/? A/DEBUG:     x20  0000000000000006  x21  00000075ac8c1450  x22  0000000000000000  x23  00000000ffffffc8
12-18 10:00:12.973 759-759/? A/DEBUG:     x24  00000075ac8bdd60  x25  00000075ac8bdc30  x26  00000075ac8bdc70  x27  0000000000000001
12-18 10:00:12.973 759-759/? A/DEBUG:     x28  00000075ad48f728  x29  00000075ac8bdb40  x30  00000075b84f8884
12-18 10:00:12.973 759-759/? A/DEBUG:     sp   00000075ac8bdb20  pc   00000075b84fb3fc  pstate 0000000060000000
12-18 10:00:12.983 759-759/? A/DEBUG: backtrace:
12-18 10:00:12.983 759-759/? A/DEBUG:     #00 pc 000000000006c3fc  /system/lib64/libc.so (tgkill+8)
12-18 10:00:12.983 759-759/? A/DEBUG:     #01 pc 0000000000069880  /system/lib64/libc.so (pthread_kill+64)
12-18 10:00:12.983 759-759/? A/DEBUG:     #02 pc 0000000000024658  /system/lib64/libc.so (raise+24)
12-18 10:00:12.983 759-759/? A/DEBUG:     #03 pc 000000000001d074  /system/lib64/libc.so (abort+52)
12-18 10:00:12.983 759-759/? A/DEBUG:     #04 pc 000000000008c72c  /data/app/com.kwic.smartaibmonitormaster-2/lib/arm64/libduktape.so
12-18 10:00:12.983 759-759/? A/DEBUG:     #05 pc 000000000008a5b0  /data/app/com.kwic.smartaibmonitormaster-2/lib/arm64/libduktape.so (__cxa_pure_virtual+16)
12-18 10:00:12.983 759-759/? A/DEBUG:     #06 pc 0000000000017b64  /data/app/com.kwic.smartaibmonitormaster-2/lib/arm64/libduktape.so
12-18 10:00:12.983 759-759/? A/DEBUG:     #07 pc 0000000000017504  /data/app/com.kwic.smartaibmonitormaster-2/lib/arm64/libduktape.so
12-18 10:00:12.983 759-759/? A/DEBUG:     #08 pc 0000000000078d5c  /data/app/com.kwic.smartaibmonitormaster-2/lib/arm64/libduktape.so
12-18 10:00:12.983 759-759/? A/DEBUG:     #09 pc 000000000007f564  /data/app/com.kwic.smartaibmonitormaster-2/lib/arm64/libduktape.so
12-18 10:00:12.983 759-759/? A/DEBUG:     #10 pc 000000000007a1b0  /data/app/com.kwic.smartaibmonitormaster-2/lib/arm64/libduktape.so
12-18 10:00:12.983 759-759/? A/DEBUG:     #11 pc 0000000000078e24  /data/app/com.kwic.smartaibmonitormaster-2/lib/arm64/libduktape.so
12-18 10:00:12.983 759-759/? A/DEBUG:     #12 pc 0000000000021b98  /data/app/com.kwic.smartaibmonitormaster-2/lib/arm64/libduktape.so
12-18 10:00:12.984 759-759/? A/DEBUG:     #13 pc 0000000000024fd4  /data/app/com.kwic.smartaibmonitormaster-2/lib/arm64/libduktape.so (duk_eval_raw+188)
12-18 10:00:12.984 759-759/? A/DEBUG:     #14 pc 0000000000014500  /data/app/com.kwic.smartaibmonitormaster-2/lib/arm64/libduktape.so
12-18 10:00:12.984 759-759/? A/DEBUG:     #15 pc 0000000000015bf8  /data/app/com.kwic.smartaibmonitormaster-2/lib/arm64/libduktape.so (Java_com_squareup_duktape_Duktape_evaluate__JLjava_lang_String_2Ljava_lang_String_2+52)
12-18 10:00:12.984 759-759/? A/DEBUG:     #16 pc 00000000001f324c  /data/app/com.kwic.smartaibmonitormaster-2/oat/arm64/base.odex (offset 0x1e9000)

I have published Duktape-Android 1.4.0 which bundles Duktape 2.6.0. This will be the final release of a Duktape repackaging as we are switching to QuickJS.

authors: JakeWharton jeonhoeun

repo: cashapp/quickjs-java


text:
Mint working with Postgres

This is the branch I am using to run the mint with postgres. I don't expect you to merge, but you can inspect the changes I have been making.

Views were broken. I fixed by moving the 'AS' sub-clause to where postgres likes it. Also, postgress doesn't like the select clauses so I hacked to get past the errors.
Timestamps - postgres is looking for a different format, so I complied with using the datetime module.

This is all working in my deployment with postgress, but I haven't yet tested back onto SQLite. Also, an unexpected benefit to this is that now I can use postgres database cluster to backend user wallets instead of sqlite files. sqlite is only designed to work with local filestores, so moving to postgres is a big scalability bonus.
Also, I still have the description_hash parameters - I need this for lightning wallet integration.

Codecov Report
Patch coverage: 33.33% and no project coverage change.

Comparison is base (c31f008) 54.75% compared to head (a13c19a) 54.75%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #194   +/-   ##
=======================================
  Coverage   54.75%   54.75%
=======================================
  Files          36       36
  Lines        2933     2933
=======================================
  Hits         1606     1606
  Misses       1327     1327

Impacted Files
Coverage Δ

cashu/lightning/lnbits.py
14.28% <0.00%> (ø)

cashu/mint/migrations.py
100.00% <ø> (ø)

cashu/mint/router.py
0.00% <0.00%> (ø)

cashu/wallet/migrations.py
100.00% <ø> (ø)

cashu/wallet/wallet.py
80.44% <33.33%> (-0.37%)
:arrow_down:

cashu/mint/ledger.py
32.06% <66.66%> (ø)

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

I opened another with just the key changes

authors: codecov-commenter trbouma

repo: cashubtc/cashu


text:
feat：Add Qwen Model Provider to Casibase

Description: [Add Qwen-Chat Model Provider to Casibase]
Related Issues: [https://github.com/casibase/casibase/issues/813]
TODO

[✓ ] add qwen-chat model; support qwen-turbo;qwen-max;qwen-plut;qwen-max-longcontext

@zjuldt why close?

authors: hsluoyz zjuldt

repo: casibase/casibase


text:
Upgrade R to v3.2.0 for Mac OS X 10.9+

Update R Cask to install R v3.2.0 on Mac OS X 10.9+

Thanks.

authors: avsmith ndr-qef

repo: caskroom/homebrew-cask


text:
PhpStorm.app delete WebIde90

This should be update depending on the version number. E.g. the version number is 9 then it should be WebIde90.

Thank you for the contribution. It was merged directly as e1ffd71c3512e65e2eab90d00fffd07f67dbba4d to keep commit history cleaner. Your contribution is still credited to you.

:+1:

authors: vinkla vitorgalvao


text:
Audacity update url

Changes to a cask
Editing an existing cask

[x] Commit message includes cask’s name (and new version, if applicable).
[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.

Changed Audacity download URL.
Currently, the official website points to download URL leading to an HTML page, although seemingly pointing to a file. This HTML page is now parsed using open-uri to extract the temporary (protected) download URL that is embedded within an iframe.
The previous download URL that was hardcoded (http://app.oldfoss.com:81/download/Audacity/audacity_macosx_ub_2_1_2-1453294898.dmg) seems not to be working anymore, so I'm proposing this fix.

This fix can’t be accepted, because this will run every time the cask is loaded, which is not only when trying to install it specifically. This means that URL will try to be opened many times, and can hang and make every operation lower.
Members of the Audacity team know fosshub cannot be used with curl and that as such we cannot use their download URLs, and must resort to oldfoss. They don’t like it, but also have no solution. We either get a working URL, or we remove the cask.

Visited the download page, and the URL is still accurate, it just appears to be down at the moment.

Thanks for the feedback. Indeed it is still accurate, but has been down for at least 36 hours now.
But you're right, hacking through their redirections in embedded iframes is not a clean way to get a valid URL. That was just an attempted contribution but I understand the downside of this.

Believe me, I do like your solution, it just introduces bad things.

Just ran into this issue, went back a version and the download started instantly (http://www.oldfoss.com/Audacity/download/audacity-macosx-ub-2.1.1.dmg).
Can we downgrade the version used by cask to 2.1.1?

@shakeelmohamed Not until someone at least contacts the developers upstream to let them know, to see if they know about the issue.

@vitorgalvao @shakeelmohamed @tobadia I’m just about to finish an enhancement to the core which allows the URL stanza to support lazy evaluation:
url do
  base_url = 'http://www.fosshub.com/Audacity.html'
  open("#{base_url}/audacity-macosx-ub-#{version}.dmg") do |io|
    content = io.read
    %r{^\<iframe.*src=\"(http.*\.dmg)\".*>}.match(content)[1].to_s
  end
end

As @vitorgalvao pointed out earlier, we have had this issue with other casks in the past. I figure this justifies a DSL enhancement to make us less dependent to upstream doohickeys
As the PR is imminent, I would suggest that this issue be closed. Also, no need to contact upstream.

@shakeelmohamed probably not a good idea since this would not keep up with the current release.
@claui glad to have contributed to other enhancements, just wondering : how will be handled the fact that open() will be called at each load, as @vitorgalvao mentioned earlier? (naive question, really)

how will be handled the fact that open() will be called at each load

From @claui’s post:

an enhancement to the core which allows the URL stanza to support lazy evaluation

“Lazy evaluation” basically means “don’t evaluate this until it is needed”. That should mean that with @claui’s improvement, url will no longer be called at each load, just when there are operations that require it.

url will no longer be called at each load, just when there are operations that require it.

Correct. Writing url with the new block syntax means: it will only be called when some code wants the URL, and even then only once.
Anything outside that url block is still bound to run at each load.
The fix requires the open block to be inside the url block, as the example shows.
Writing url <url_string> will remain the recommended way to specify an URL for almost every Cask.

Got it, thank you both for detailing this.

@tobadia FYI – the fix has just been merged, including your contribution.

I was following the threads on the other edits you were both documenting. Thanks for the beads up!

Heads* damn autocorrect

authors: claui shakeelmohamed tobadia vitorgalvao


text:
Update ilya-birman-typography-layout to 3.5.

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} reports no offenses.
[x] The commit message includes the cask’s name and version.

After installing by brew cask install ilya-birman-typography-layout this layout appeared twice.

@a1ip try
brew cask uninstall ilya-birman-typography-layout
brew cask install ilya-birman-typography-layout

This helped me.

@sashkab doesn't work for me. After uninstall and install - nothing changed. Then I tried to remove "Russian - Ilya Birman Typography" layout between uninstall and install, - the same result, nothing changed, still two identical items in Russian layouts named "Russian - Ilya Birman Typography" as it was in screenshot. 😞


text:
Fix sourcetree appname on case-sensitive filesystem

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

The Appname is Sourcetree.app not SourceTree.app

Thanks for the PR!

authors: commitay nexeck


text:
Fix: Move previous created casks from root to /Cask folder

This commit fixes issue #909

@reitermarkus Shouldn’t Travis have caught this? It used to (if I recall correctly).

authors: guerrero vitorgalvao

repo: caskroom/homebrew-fonts


text:
fix: Fix mobile view error

Signed-off-by: ErikQQY 2283984853@qq.com

@ErikQQY it's still not fully right:

@ErikQQY any update?

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

authors: CLAassistant ErikQQY hsluoyz

repo: casnode/casnode-website


text:
AttributeError in retrieving passages for MS MARCO passage ranking

In the replication of the MS MARCO passage ranking experiment, we retrieve passages by running
python ./src/main/python/msmarco/retrieve.py --hits 1000 --threads 1 \
 --index indexes/msmarco-passage/lucene-index-msmarco --qid_queries collections/msmarco-passage/queries.dev.small.tsv \
 --output runs/run.msmarco-passage.dev.small.tsv

This leads to the following error:
Traceback (most recent call last):
  File "./src/main/python/msmarco/retrieve.py", line 58, in <module>
    hits = searcher.search(query.encode('utf8'), int(args.hits))
  File "/home/kelvin/.local/lib/python3.6/site-packages/pyserini/search/pysearch.py", line 99, in search
    return self.object.search(JString(q.encode('utf8')), k)
AttributeError: 'bytes' object has no attribute 'encode'

This error is also encountered in https://github.com/castorini/anserini/pull/1220.

@kelvin-jiang thanks for reporting. yup, we're aware of this issue... the repo is in rapid flux right now due to EMNLP deadlines and TREC COVID runs. We'll return to this once things quiet down.

I believe this has been closed by #1238
@kelvin-jiang can you please confirm and then close issue?

Confirmed!

authors: kelvin-jiang lintool

repo: castorini/anserini


text:
Fix directory directives

Load configuration from .reek.yml
Update version.rb
Update solargraph-reek.rb
Fix directory directives

Thanks!

authors: castwide edward-mb

repo: castwide/solargraph-reek


text:
Argument '2' passed to select() is expected to be of type array, string given

Information

Version of Medoo: 2.1.12
Version of PHP: 8.2.22
Type of Database: MySQL
System: Linux

Describe the Problem
Argument '2' passed to select() is expected to be of type array, string given

You are passing a string to the second select() parameter.
Please read the documentation. https://medoo.in/api/select.

Hi, according the docs that i"ve read, to select an entire column :
$data = $database->select("account", "user_name");
Second argument is a string.

@stefankummer Sorry, my mistake. However, it should work. It's on the test case.
https://github.com/catfan/Medoo/blob/d1ff4e0e873a3e032e3d8804031dd6be4e013b22/tests/SelectTest.php#L60-L73
Please provide the full code you called.

authors: catfan stefankummer

repo: catfan/Medoo


text:
Various areas of chess.com unthemed

Is there an existing issue outlining your problem?

[X] I am using the latest version of the provided userstyle.
[X] I have searched the existing issues and they do not solve my problem.

What userstyles are you seeing the problem on?
lbl:chess.com
Describe your problem.
I installed Stylus extension for Firefox and then clicked Stylus Install button on chess.com catppuccin page. Button redirected me to some css file and i enabled it, but it doesn't work.
Attach screenshots.
No response
What browser(s) are you seeing the problem on?
Firefox 127.0.2
Any additional comments?
No response

Are you sure you clicked install when it opened the CSS file?

It only changes the board color but background etc. is same

Quite a lot of the userstyle is unthemed, but you are able to install it? @coldenate do you mind taking a look at the userstyle and seeing what can be fixed?

Changed all themes to default settings, problem solved.

authors: kocosmanberke uncenter

repo: catppuccin/userstyles


text:
Fix: DO-1452, DO-1713 fix table sort does not return the right index row

Motivation and Context

A bug was found where if table was sorted then the wrong chosen role was passed to the backend.
Implementation Description

The issue turned out to be an issue with getting the value within the getItem which created a discrepancy when data was sorted from initial index to the one chosen by the user. This was also an issue for if the table selected value was changed from outside the component. The easier fix was to simply remove that part of the logic. We do need to fetch the data more often, but in this way we guarantee that regardless if selection is done in a sorted table or changed from outside that the correct index is still passed to both onclick_row action context and selected_indices Variables.
Any new dependencies Introduced

None
How Has This Been Tested?

Locally by single and multi selecting table rows, sorted and unsorted and checking corresponding Variables were updated as expected.
PR Checklist:

[x] I have implemented all requirements? (see JIRA, project documentation).
[x] I am not affecting someone else's work, If I am, they are included as a reviewer.
[ ] I have added relevant tests (unit, integration or regression).
[x] I have added comments to all the bits that are hard to follow.
[x] I have added/updated Documentation.
[x] I have updated the appropriate changelog with a line for my changes.

Screenshots (if appropriate):

@krzysztof-causalens looked at the error in the CI and their docs recommend to sanitize the data beforehand, is there a reason why we aren't?

@krzysztof-causalens looked at the error in the CI and their docs recommend to sanitize the data beforehand, is there a reason why we aren't?

Sanitizing is mostly important where the HTML comes from the user. Yes ideally we'd sanitize it but "fine" since it's provided by the app developer (to the best of my knowledge, not a security expert), hence the error was ignored. Not sure why suddenly the bearer-ignore stopped working, perhaps they changed how their ignore system works and the comment needs updating?

@krzysztof-causalens looked at the error in the CI and their docs recommend to sanitize the data beforehand, is there a reason why we aren't?

Sanitizing is mostly important where the HTML comes from the user. Yes ideally we'd sanitize it but "fine" since it's provided by the app developer (to the best of my knowledge, not a security expert), hence the error was ignored. Not sure why suddenly the bearer-ignore stopped working, perhaps they changed how their ignore system works and the comment needs updating?

Note for future: The message was relating to a Bearer issue that is currently up, meantime workaround was found in a separate PR and added here.

Checked and with persist_value flag the table selection persists, tested for an initial value selected and for a combination of filter + sort

authors: krzysztof-causalens patricia-causalens

repo: causalens/dara


text:
🛑 Janax Soft is down

In 3239b55, Janax Soft (https://www.janaxsoftware.com.ar/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Janax Soft is back up in 60dc7ce.

repo: cavalicenti/upptime


text:
Related Services

Eg, topic service, recon service -- whether in Cayley itself or as part of the greater project.

Moved to forum: https://discourse.cayley.io/t/related-services/216

repo: cayleygraph/cayley


text:
Test the /assign command

This tests assigning a user using a ChatOps command

/assign cjxd-bot-test


text:
chore: bdd-gh-1581613658 to 0.0.1

chore: Promote bdd-gh-1581613658 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


text:
chore: bdd-spring-1585319214 to 0.0.1

chore: Promote bdd-spring-1585319214 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


text:
How can i edit the template?

Hi, how can i add greatsword and blunderbuss? i need to edit every singular .js file? or is any ez solution? I can implement, but then to edit the template, i need to edit like 7-8 files.. Can you add it or if you check: https://github.com/DrugAddict420/quellopulito
I edit the files and add mutated + blunderbluss and greatsword.. could you add it to your repo? OR what else can i do?

if you have added all the code changes, simply run the following command in the directory:
npm run build
this should produce all needed files in the ./dist folder

you can also create a pull request in this repo, I can trigger a release for the changes

oh thats sad, you've created a fork, but didnt use it for the changes
https://github.com/DrugAddict420/nw-guild-cp
if you commit your changes over there, I can pull them into this repo and create a release out of it

@cbartel done i change the files inside: https://github.com/DrugAddict420/nw-guild-cp What now?

Check if everyhing is allright! i did both, edit the forked, and even ask for a pull request! Let me know if is everyhing ok!
@cbartel-ci

#50 was merged. Thanks for the update. It should now upload the release.. I will check if everything went right. You should be able to use the updater in the admin panel afterwards

as I've feared, I have some authorization issues in the action that uploads the release. Let me get to that quickly, should be done in a few minutes ;)

There was an error building the server
`ERROR in src/expedition/expedition.service.ts:15:10
TS2305: Module '"@prisma/client"' has no exported member 'ExpeditionParticipant'.
13 |   User,
14 | } from '@nw-company-tool/model';

15 | import { ExpeditionParticipant } from '@prisma/client';
|          ^^^^^^^^^^^^^^^^^^^^^
16 | import { DatabaseClient } from '../database/database.client';
17 | import { EventService } from '../event/event.service';
18 |

webpack 5.58.2 compiled with 1 error in 4298 ms`
Need to see, what went wrong there
As I said, this hasn't been touched for a while. These problems occur after not updating for a while :/

dont use the update yet, cannot guarantee what happens to your server if there is no build for the server included :) will try to solve this issue on my own in the meantime

@cbartel-ci @cbartel waiting for updates! I got another error, i post in the issue category!

I got it working locally ;) was a bit dusty.. but its doing pretty well
was able to put get the expeditions running aswell

will need to see how to get the build running such that we can create a proper release

You are the best! Hope you can release asap! If you release tonight i buy you 2 more coffe cuz u deserve it! @cbartel @cbartel-ci

no need ;) it just caught my curiosity .. don't like if things don't work! :)

@cbartel doesnt matter you are helping a lot! You deserve it! Hope you can fix all asap <3

please check if you can get v2.2.3 running without, you might have to manually download the node distribution and extract it. Do not delete your data directory :) it should work as before.
If not, you can at least try to rename the database.db file to for example database.db.tmp
In this case, the server will create a fresh database (and your user data will unfortunately be lost)

prisma:info Starting a sqlite pool with 3 connections.
[Nest] 2819  - 02/04/2023, 10:40:09 PM   ERROR [ExceptionsHandler]
Invalid this.client.user.findUnique() invocation in
/opt/nwct/nwct-server/dist/server.js:899:33
896     };
897 }
898 async findByDiscordId(discordId) {
→  899     return this.client.user.findUnique(
The table main.User does not exist in the current database.
Error:
Invalid this.client.user.findUnique() invocation in
/opt/nwct/nwct-server/dist/server.js:899:33
896     };
897 }
898 async findByDiscordId(discordId) {
→  899     return this.client.user.findUnique(
The table main.User does not exist in the current database.
at RequestHandler.request (/opt/nwct/nwct-server/node_modules/@prisma/client/runtime/index.js:49022:15)
at async DatabaseClient._request (/opt/nwct/nwct-server/node_modules/@prisma/client/runtime/index.js:49919:18)
at async LoginController.callback (/opt/nwct/nwct-server/dist/server.js:1358:26)
at async /opt/nwct/nwct-server/node_modules/@nestjs/core/router/router-execution-context.js:46:28
at async /opt/nwct/nwct-server/node_modules/@nestjs/core/router/router-proxy.js:9:17
same error.. im doing something wrong?

whenever im trying to create the first user with discord i get this error.. is some table missing in the db? or is another the problem?

How can we use https on port 8080? Im reading that we should use 8443.. is that true? Or is another solution to the ssl problem?

hey, i added you at discord. I don't think we should communicate through github issues, as the problems are mainly configuration issues, not code  :)
Closing this issue

authors: DrugAddict420 cbartel cbartel-ci

repo: cbartel/nw-company-tool


text:
[MacOS] USB controller failed to open

I'm trying to connect some controllers via USB and I'm unable to open them. I was able to use the psmoveapi psmovepair utility without difficulty. PSMoveService error logs below.
The bluetooth radio is currently off, so it's not surprising that the bluetooth serial came up empty.
Chadwicks-MacBook-Pro:Debug chadwickboulay$ ./PSMoveService
[2016-11-24 20:23:05.897]: main - Starting PSMoveService
Config file name: "/Users/chadwickboulay/PSMoveService/ControllerManagerConfig.json"
Config file name: "/Users/chadwickboulay/PSMoveService/ControllerManagerConfig.json"
Config file name: "/Users/chadwickboulay/PSMoveService/TrackerManagerConfig.json"
Config file name: "/Users/chadwickboulay/PSMoveService/TrackerManagerConfig.json"
[2016-11-24 20:23:05.980]: PSMoveController::open - Opening PSMoveController(IOService:/AppleACPIPlatformExpert/PCI0@0/AppleACPIPCI/XHC1@14/XHC1@14000000/HS01@14100000/Motion Controller@14100000/IOUSBHostInterface@0/IOUSBHostHIDDevice@14100000,0)
[2016-11-24 20:23:05.980]: PSMoveController::open -   with serial_number:
[2016-11-24 20:23:05.983]: PSMoveController::open - Failed to get bluetooth address of PSMoveController(IOService:/AppleACPIPlatformExpert/PCI0@0/AppleACPIPCI/XHC1@14/XHC1@14000000/HS01@14100000/Motion Controller@14100000/IOUSBHostInterface@0/IOUSBHostHIDDevice@14100000,0)
Config file name: "/Users/chadwickboulay/PSMoveService/PSMoveControllerConfig.json"
[2016-11-24 20:23:05.984]: DeviceTypeManager::update_connected_devices - Device device_id 0 (IOService:/AppleACPIPlatformExpert/PCI0@0/AppleACPIPCI/XHC1@14/XHC1@14000000/HS01@14100000/Motion Controller@14100000/IOUSBHostInterface@0/IOUSBHostHIDDevice@14100000,0) failed to open!
[2016-11-24 20:23:05.989]: DeviceTypeManager::update_connected_devices - Closing device 0 (PSMove) since it's no longer in the device list.
[2016-11-24 20:23:05.989]: PSMoveController::close - Closing PSMoveController(IOService:/AppleACPIPlatformExpert/PCI0@0/AppleACPIPCI/XHC1@14/XHC1@14000000/HS01@14100000/Motion Controller@14100000/IOUSBHostInterface@0/IOUSBHostHIDDevice@14100000,0)
[2016-11-24 20:23:06.980]: PSMoveController::open - Opening PSMoveController(IOService:/AppleACPIPlatformExpert/PCI0@0/AppleACPIPCI/XHC1@14/XHC1@14000000/HS01@14100000/Motion Controller@14100000/IOUSBHostInterface@0/IOUSBHostHIDDevice@14100000,0)
[2016-11-24 20:23:06.981]: PSMoveController::open -   with serial_number:
[2016-11-24 20:23:06.984]: PSMoveController::open - Failed to get bluetooth address of PSMoveController(IOService:/AppleACPIPlatformExpert/PCI0@0/AppleACPIPCI/XHC1@14/XHC1@14000000/HS01@14100000/Motion Controller@14100000/IOUSBHostInterface@0/IOUSBHostHIDDevice@14100000,0)
Config file name: "/Users/chadwickboulay/PSMoveService/PSMoveControllerConfig.json"
[2016-11-24 20:23:06.985]: DeviceTypeManager::update_connected_devices - Device device_id 0 (IOService:/AppleACPIPlatformExpert/PCI0@0/AppleACPIPCI/XHC1@14/XHC1@14000000/HS01@14100000/Motion Controller@14100000/IOUSBHostInterface@0/IOUSBHostHIDDevice@14100000,0) failed to open!
[2016-11-24 20:23:06.991]: DeviceTypeManager::update_connected_devices - Closing device 0 (PSMove) since it's no longer in the device list.
[2016-11-24 20:23:06.991]: PSMoveController::close - Closing PSMoveController(IOService:/AppleACPIPlatformExpert/PCI0@0/AppleACPIPCI/XHC1@14/XHC1@14000000/HS01@14100000/Motion Controller@14100000/IOUSBHostInterface@0/IOUSBHostHIDDevice@14100000,0)

Also failing when connected via Bluetooth.
Chadwicks-MacBook-Pro:Debug chadwickboulay$ ./PSMoveService
[2016-11-24 20:33:42.733]: main - Starting PSMoveService
Config file name: "/Users/chadwickboulay/PSMoveService/ControllerManagerConfig.json"
Config file name: "/Users/chadwickboulay/PSMoveService/ControllerManagerConfig.json"
Config file name: "/Users/chadwickboulay/PSMoveService/TrackerManagerConfig.json"
Config file name: "/Users/chadwickboulay/PSMoveService/TrackerManagerConfig.json"
[2016-11-24 20:33:42.751]: PSMoveController::open - Opening PSMoveController(IOService:/IOResources/IOBluetoothHCIController/AppleBroadcomBluetoothHostController/IOBluetoothDevice/IOBluetoothL2CAPChannel/IOBluetoothHIDDriver)
[2016-11-24 20:33:42.751]: PSMoveController::open -   with serial_number: 00-06-f7-18-cd-f3
Config file name: "/Users/chadwickboulay/PSMoveService/00_06_f7_18_cd_f3.json"
[2016-11-24 20:33:47.755]: PSMoveController::open - PSMoveController(IOService:/IOResources/IOBluetoothHCIController/AppleBroadcomBluetoothHostController/IOBluetoothDevice/IOBluetoothL2CAPChannel/IOBluetoothHIDDriver) has invalid calibration. Reloading.
[2016-11-24 20:33:52.818]: PSMoveController::open - Failed to open read initial controller state after 10 attempts.
Config file name: "/Users/chadwickboulay/PSMoveService/00_06_f7_18_cd_f3.json"
[2016-11-24 20:33:52.822]: DeviceTypeManager::update_connected_devices - Device device_id 0 (PSMove) opened
[2016-11-24 20:33:53.121]: ServerDeviceView::poll - Device id 0 closing due to no data (100 failed poll attempts)
[2016-11-24 20:33:53.121]: PSMoveController::close - Closing PSMoveController(IOService:/IOResources/IOBluetoothHCIController/AppleBroadcomBluetoothHostController/IOBluetoothDevice/IOBluetoothL2CAPChannel/IOBluetoothHIDDriver)

I solved the USB problem. It's an old problem. The solution is a bit of a hack. hid_get_feature_report returns one less byte on Mac than on Windows. (I'm pretty sure this is true for Linux too). I'm not sure why this problem cropped up again.
Still working on the bluetooth problem. So far it seems that hid_get_feature_report is failing when the first byte is PSMove_Req_GetFirmwareInfo = 0xF9 I've tried a bunch of different buffer lengths but that's not working. Maybe the firmware on my controller is borked again.

I couldn't solve the bluetooth problem. I was able to discern that in PSMoveController::poll(), hid_read was always returning a result of 0. As it is set to non-blocking, this happens whenever dev->input_reports is empty, which in my case seems to be always. I guess I need to explicitly tell the controller to fetch a packet. Or is it more complicated than that?
I wonder if this might be related to problems you solved in the CAPI branch with https://github.com/cboulay/PSMoveService/commit/a147707b36957dc4696d33ead8b33ee032a5bb14 ?

This is a really tricky problem. I just added a commit that includes a test utility to help me narrow this down: ac71279
My controller is frequently ending up in a state where I can't read anything using hid_read. If I disconnect then reconnect the controller then its state is good again and I can use hid_read. Using my utility, I discovered that this breaks as soon as I attempt a hid_get_feature_report that fails.
Still more work to do. Next up:

Verify that good feature report requests don't break my ability to hid_read.
Find all the bad feature report requests and fix them.

I solved the USB problem. It's an old problem. The solution is a bit of a hack. hid_get_feature_report returns one less byte on Mac than on Windows. (I'm pretty sure this is true for Linux too). I'm not sure why this problem cropped up again.

Ah I think we switched from our custom fork of hidapi back to the most recent main line version a few weeks ago. In the windows implementation I think they include the report id prefix byte in the resulting report size. I think we originally had a fork of hidapi that removed this +1 change.

Still working on the bluetooth problem. So far it seems that hid_get_feature_report is failing when the first byte is PSMove_Req_GetFirmwareInfo = 0xF9 I've tried a bunch of different buffer lengths but that's not working. Maybe the firmware on my controller is borked again.

No that's actually a problem on windows too. The most recent PSMove controllers  have updated firmware that zeros out the magnetometer readings. I had hoped to detect these newer controllers with the firmware report. I got the report ID from Alex's HID report page. Unfortunately I could never get that HID report to work from me on any of the controllers I tried. I left the code in requesting the bad report because I had wanted a few other people to try it out (and it didn't seem to effect the rest of the controller startup when it failed). However if that is causing problems in OS X we should just comment that out.
Sorry for the trouble there!

I finally narrowed down my problem. The attempt to get the firmware here was failing when connected via only bluetooth. This failure would stop all subsequent hid_reads to fail, making the controller useless until I disconnected/connected it.
@HipsterSloth , can you please verify that you can get ANY feature report over bluetooth, especially the firmware in the line above? For now I've wrapped most of that function in a if (!getIsBluetooth()). I'm willing to make this platform-specific if necessary, but I'm hoping it's something simpler.
Another feature report that we might want over bluetooth is PSMove_Req_GetExtDeviceInfo.
I can finally get back to debugging undistort_points.

Crossed wires...
OK you can't get the firmware when connected via bluetooth. Then I don't need a platform-specific solution.

authors: HipsterSloth cboulay

repo: cboulay/PSMoveService


text:
Holy Week information

Merge the holy week data into master

Lets do it.

repo: cbounds/nativitygit


text:
Refactor fileLoop() helper function?

Common pattern I am using is to loop through lines of a file, abstracting this may or may not be a good idea.

That pattern is used twice, whereas a readAll() was used five or six times and has now been factored into a function.

repo: cc-paw/cc-paw


text:
Remove whitelisting of functions

This decreases build time and prevents functions that should be
whitelisted to not be.
For example, when working on my fork of the leptess API, I didn't have access to the file format enums.
And anyway, <leptonica/allheaders.h> is the api. There is no reason some whitelisting should be done on top of it.
Also increase bindgen version.

Thank you @NilsIrl.
I'm getting a warning when running the tests:
warning: `extern` block uses type `u128`, which is not FFI-safe
    --> /home/chris/Documents/git/github.com/ccouzens/leptonica-sys/target/debug/build/leptonica-sys-3634b718d278f931/out/bindings.rs:1448:10
     |
1448 |     ) -> u128;
     |          ^^^^ not FFI-safe
     |
     = note: `#[warn(improper_ctypes)]` on by default
     = note: 128-bit integers don't currently have a known stable ABI

Is it possible to prevent this without hiding it? I'll investigate too.
I think the reason I didn't do it this way initially is because I was worried about it including functions from the c-standard library like printf. But this doesn't seem to be a problem.

The functions generating these warnings are:
// part of <stdlib.h>
extern "C" {
    pub fn strtold(
        __nptr: *const ::std::os::raw::c_char,
        __endptr: *mut *mut ::std::os::raw::c_char,
    ) -> u128;
}

// part of <stdlib.h>
extern "C" {
    pub fn qecvt(
        __value: u128,
        __ndigit: ::std::os::raw::c_int,
        __decpt: *mut ::std::os::raw::c_int,
        __sign: *mut ::std::os::raw::c_int,
    ) -> *mut ::std::os::raw::c_char;
}

// part of <stdlib.h>
extern "C" {
    pub fn qfcvt(
        __value: u128,
        __ndigit: ::std::os::raw::c_int,
        __decpt: *mut ::std::os::raw::c_int,
        __sign: *mut ::std::os::raw::c_int,
    ) -> *mut ::std::os::raw::c_char;
}

// part of <stdlib.h>
extern "C" {
    pub fn qgcvt(
        __value: u128,
        __ndigit: ::std::os::raw::c_int,
        __buf: *mut ::std::os::raw::c_char,
    ) -> *mut ::std::os::raw::c_char;
}

// part of <stdlib.h>
extern "C" {
    pub fn qecvt_r(
        __value: u128,
        __ndigit: ::std::os::raw::c_int,
        __decpt: *mut ::std::os::raw::c_int,
        __sign: *mut ::std::os::raw::c_int,
        __buf: *mut ::std::os::raw::c_char,
        __len: usize,
    ) -> ::std::os::raw::c_int;
}

// part of <stdlib.h>
extern "C" {
    pub fn qfcvt_r(
        __value: u128,
        __ndigit: ::std::os::raw::c_int,
        __decpt: *mut ::std::os::raw::c_int,
        __sign: *mut ::std::os::raw::c_int,
        __buf: *mut ::std::os::raw::c_char,
        __len: usize,
    ) -> ::std::os::raw::c_int;
}

I'll see if I can exclude stdlib and any other libraries.

https://github.com/rust-lang/rust-bindgen/issues/1549

I think the reason I didn't do it this way initially is because I was worried about it including functions from the c-standard library like printf.

Theses functions are indeed included

It that's really a problem. These could be removed

Changing wrapper.h to be
#define _STDLIB_H       1

#include <leptonica/allheaders.h>

Seems to exclude stdlib.h. The warning goes away, and the tests still pass

Actually, I'll merge it as is. The warnings don't show for dependent crates.

I've pushed this as version 0.3.0.
I'm currently updating tesseract-sys and tesseract-rs to use it.

I didn't have access to the file format enums.

@NilsIrl, can you let me know one of the enums you need? I'll add a test so to make sure it doesn't get removed in future

https://github.com/DanBloomberg/leptonica/blob/50c3c353f75b0a5c97b645b3c050dab66243ee20/src/imageio.h#L91-L112

authors: NilsIrl ccouzens

repo: ccouzens/leptonica-sys


text:
Text misaligned/not wrapping

iPhone SE 10.2.1, Slide 1.0.0 (13)
Username:

Wrapping:

Slide 1.0.0 (14)

I believe I have this all fixed now!

@ccrama sorry, could you be a little more clear? I'm kinda confused what you want me to do. Like, switch accounts then close and reopen the app? Or something else?

Yes, still cut off if I open the app, switch accounts, close the app, and then reopen the app and then the toolbar.

This is in build 20 not 19 (same with your other replies, sorry should have been more clear!)

Ah ok. I'll wait for the new build to hit TF then 👌

Unfortunately, still happens on 20.

authors: ccrama incIined

repo: ccrama/Slide-iOS


text:
accordian Widget configuration

SUMMARY:
This feature will add an accordion widget to the widget field. This will allow for the addition of accordion elements to be added to a page. Each accordion can have as many expanding points as needed.
BACKEND (Drupal): https://accordion-chug.pantheonsite.io/

Paragraph Accordion Section

which only has one field that is a reference to the Accordion Block paragraph

Paragraph Accordion Block

Two fields, both of which are formatted so that basic html can be used

Title
Text

FRONTEND (Gatsby): gus/accordion branch

gatsby-node.js - add paragraph__accordion_section to widgetParagraphUnion
widget.js - add code to display paragraph
basic-page.js - added to query to include the accordion paragraphs from drupal

TESTING (Gatsby Cloud):
https://www.gatsbyjs.com/dashboard/2e86c058-a370-4898-ae51-9698cd178e8d/sites/4d03c6f6-9271-4faf-8419-e36460e489d7/deploys
https://gusaccordion.gatsbyjs.io/
Existing page

https://gusaccordion.gatsbyjs.io/masters-of-biomedical-sciences-mbs the accordions on that page are done by the widget.
New page
Add new basic page and add an accordion widget with at least one accordion block.
trigger build on gatsby cloud and examine page to ensure the accordion widget is working.
edit widget and add   around the title field and re-trigger build to ensure the formatting works.
Add a second accordion to the page to ensure that multiple accordion widgets can work on one page.
-re-trigger build

Not sure if this is exactly related to the changes you made, but halfway down the page, starting with:
Applied Toxicology Contact
Gordon M. Kirby, DVM, MSc, PhD
the page content layout looks off.

https://build-e7a1ba8f-ca59-474b-989a-389ee322cda9.gtsb.io/test-for-accordions-pr
I added two separate acc blocks, the second has only one item (Another Block). If I click
to open that one, it also open the first item from the first block. Not sure it that's because
there's no other text/widget between the acc blocks?

This is due to a cut and paste - just it is not related to the accordion.

https://build-e7a1ba8f-ca59-474b-989a-389ee322cda9.gtsb.io/test-for-accordions-pr
I added two separate acc sections, the second has only one item (Another Block). If I click to open that one, it also open the first item from the first block. Not sure it that's because there's no other text/widget between the acc blocks?
UPDATE: I added a text widget between the two acc sections, and the same issue is happening.

I have fixed this

ok, seems to be working now after the fix

@snschau Build is still failing. See https://www.gatsbyjs.com/dashboard/2e86c058-a370-4898-ae51-9698cd178e8d/sites/fc8e90a4-54ce-4012-a5fc-3302f05a3d6d/builds/b7a6225d-2999-4794-9b20-9f6ab8710642/details#rawLogs - looks like further additions are needed for gatsby-node.js

Tanya and Joy - i have been trying to figure out what I have done wrong in the Schema - I can't seem to figure it out - it in gatsby-node.js currently the items are from line 1132 to 164 - it builds - but get an error when the page displays.  If you have time to look at it in early January - that would be great. I will be around on the 10th maybe we can have a look at it then.  Thanks. (Hope you had a good break).
Scott.

authors: abanuog snschau tqureshi-uog

repo: ccswbs/gus


text:
Regression due to ruamel.yaml

Potentially dangerous regression in config box, due to the new ruamel.yaml dependency.
>>> from box import ConfigBox
>>> b = ConfigBox.from_yaml("kill_all_humans_mode: no")
>>> if b.kill_all_humans_mode:
...     print("ok, killing all humans...")
...
ok, killing all humans...

Same problem with box.Box. Our test suite caught this, but others might not be so lucky.  ruamel isn't exactly a drop-in replacement for yaml, despite claims to the contrary (ruamel defaults to spec 1.2 and yaml is spec 1.1).
I would advise to make the toml and ruamel.yaml dependencies optional (or "soft" dependencies, like it was in 3.x) since users might prefer other providers such as pytoml and PyYAML and you don't want to tie their hands here.
Personally, I can not use box>=4 at all because the ruamel.yaml has an insane installer which, for various reasons, does not work in the prod environment at $EMPLOYER.

For ruamel.yaml it is possible to use version 1.1 by passing that argument to the loader which should remove any differences that I am aware of.
Box.from_yaml("kill_all_humans_mode: no", version='1.1')
<Box: {'kill_all_humans_mode': False}>

I am against the soft dependencies as it makes it more cumbersome for the 95% percent case, and also makes it so dependencies like that can exist due to external libraries (aka with Box 3.x if someone has PyYAML on one machine and ruamel.yaml on another can produce different results.)
However, as there is no way to monkey patch PyYAML in without having ruamel.yaml imported successfully that does cause a conundrum in the case where one cannot install ruamel.yaml at all. I will need to think of a way to allow for fallbacks if those other dependencies are already on the system, or somehow manually flagged to use instead of the defaults.

Testing this out https://github.com/cdgriffith/Box/commit/92439a7889ac493e3aa9b3d80565d0efac53ed9a
I can't think of an easy way other than checking on install if PyYAML is alraedy installed and ruamel.yaml is not. I am going to sleep on this, but let me know if you have any other ideas for it!
Also not planning to bother to do that to toml as pytoml is depreciated.
To be honest when working on 4.x initially, before the PyYAML 5.1 release, I was blindly assuming that we wasn't going to see another major release, so it's nice to see that both projects are still alive and well!

https://github.com/cdgriffith/Box/commit/92439a7889ac493e3aa9b3d80565d0efac53ed9a is really not the right way to go about it. Since you are shipping .whl distributions, the setup.py is typically not executed at all for the end user. The distribution dependencies are determined at build time, not at install time (unless you're shipping source distributions only, forcing the end user to execute a setup.py at install time).
Maybe it's best to just make the toml/yaml supported as optional "extras". Perhaps not every user even wants these features, and just wants to use Box in pure Python without loading from markup language.
Then the syntax for end user will look like this:
pip install python-box  # pure python version
pip install python-box[yaml]   # with yaml support
pip install python-box[toml]  # with toml support
pip install python-box[toml,yaml]  # with both yaml and toml support

This allows you to choose your preferred yaml/toml providers in the metadata, but also allows users who have some other preference to use some other one without getting a dependency conflict.

I actually have not seen that syntax / usage before. How do you access or use that metadata info? (Have a handy link with examples perhaps?)
Thanks!

See https://setuptools.readthedocs.io/en/latest/setuptools.html#declaring-extras-optional-features-with-their-own-dependencies

documentation updates slated for 4.1 https://github.com/cdgriffith/Box/pull/132

Addressed in 4.1.0

As an update to this, Box 5.0 will go back to being able to be pure python, and installing additional extras as requested.  https://github.com/cdgriffith/Box/wiki/Installation

authors: cdgriffith wimglenn

repo: cdgriffith/Box


text:
Better handling of species annotations...

Could be nicer. An import option would be good.

Multiple IDs are now handled better in 1516ecfbc1f3779d227b0a882d08f039a36aea26 opening the door for a species data frame

repo: cdiener/dycone


text:
feat(deps): upgrade runtime dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-runtime-dependencies-2.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 1131

Questions ?
Please refer to the Backport tool documentation


text:
chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-2.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 2188

Questions ?
Please refer to the Backport tool documentation


text:
chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-1.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 540

Questions ?
Please refer to the Backport tool documentation


text:
chore(deps): upgrade configuration

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-configuration-2.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 2601

Questions ?
Please refer to the Backport tool documentation


text:
CronJob L2

https://kubernetes.io/docs/concepts/workloads/controllers/cron-jobs/

+1


text:
chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-k8s-25-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 2127

Questions ?
Please refer to the Backport tool documentation


text:
chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-k8s-27-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 2696

Questions ?
Please refer to the Backport tool documentation


text:
Helm support to take account of Capabilities.APIVersions

Description of the feature or enhancement:
As I am sure you must be aware by now, Kubernetes has many, many, many versions. Each version supports different APIs and each new version deprecates APIs that were considered "beta" in the past.
For instance extensions/v1beta1 for Ingress was deprecated in  v1.14+ and unavailable since v1.22+.
When you deploy a helm chart, helm can take into account the specific clusters API capabilities and adapts the manifests accordingly (if implemented in code).
However, helm support in CDK8s is currently a "one size fits all" with no option of selecting what APIs to use meaning that the cdk8s output might not work on the cluster you are deploying to.
Use Case:
I am trying to render a helm chart for kong and the manifest cannot be deployed to a version 1.22 Kubernetes cluster
Proposed Solution:
I can see 3 possible ways of addressing this

Adding a parameter to the Helm method allowing for selection of major Kubernetes version
Adding helm support to the version-specific implementations such as cdk8s-plus-22
Adding a parameter to manually override API versions such as --api-versions=networking.k8s.io/v1

[ ] :wave: I may be able to implement this feature request
[ ] :warning: This feature might incur a breaking change

This is a :rocket: Feature Request

Been doing some more research on this and it appears to be a major limitation of the helm template command.
Helm template will only honour --api-versions=networking.k8s.io/v1 if --validate is also passed. However, this needs a connection to a cluster and will fail if there are conflicting resources on the cluster breaking your build process.
There really needs to be a way to tell Helm template some information about your cluster's capabilities without having to resort to connecting and validating using a real cluster as build servers typically don't have access to Kubernetes clusters.
As cdk8s relies on the helm template command, there doesn't appear to be a workaround unless I am missing something.

Can we reopen this issue? This is a priority/p1 issue and I think it's still relevant.

@dcharbonnier It works for me with the flag. Does it not work for you?
helmFlags: ["--api-versions", "monitoring.coreos.com/v1"]

For me, ArgoCD helm chart requires it for generating ServiceMonitors.
{{- if and (.Capabilities.APIVersions.Has "monitoring.coreos.com/v1") .Values.repoServer.metrics.enabled .Values.repoServer.metrics.serviceMonitor.enabled }}

authors: dcharbonnier dniasoff erikschul


text:
Compatibility to aws-cdk v2.56+

I really liked the CodeFirst approach which allows us to do certain things which is not easily possible with a schema file.
As this library was separated from the appsync package per this PR I find myself in a situation where the library is not in sync anymore with the CDK constructs.
The package does not seem to be released with the rest of the CDK packages anymore. The package.json shows a dependency for aws cdk v 2.55.1 here without a semver range.
How is the future of this package planned? Can it be re-integrated with the aws-cdk releases?
Best
Friedrich

So since you can't express semver ranges across multiple pre-release versions of a package as far as I can tell, we will have to release as new versions of the appsync alpha package are released. I can figure out how to include cdk dependencies in our automated package upgrades as part of the projen workflows and then it should all happen automatically.

Since the appsync module is now stable, taking a peerDependency on aws-cdk-lib ^2.60.0 allows users to depend on newer versions as needed.

Thanks for that, do you have a timeline for the merge of the open pull request? Is there any blocker I can help with?

authors: MrArnoldPalmer pfried

repo: cdklabs/awscdk-appsync-utils


text:
How to configure ALB health check

Hello.
We are trying to replace a deployment of an application that uses ApplicationLoadBalancedFargateService with one made using @aws-cdk-containers/ecs-service-extensions. We have been able to configure almost everything but are a little stuck when configuring the health check for the application. Specifically, we are not able to find how to set up the health check path and port.
With our previous deployment we used :
application.targetGroup.configureHealthCheck({
      path: '/ping',
      port: '3000',
      healthyHttpCodes: '200-299',
      interval: Duration.seconds(30),
      timeout: Duration.seconds(5),
      healthyThresholdCount: 5,
      unhealthyThresholdCount: 5,
    });

But using service extensions, specifically the HttpLoadBalancerExtension there is no straightforward way to access the load balancer instance nor the target group.
We tried using a ContainerMutatingHook but werw only able to access the docker native health check, which didn't work for our use case.
We have a couple more ideas of how to do it but were wondering if there is a recommended way since we understand that ServiceExtensions should allow us to do the same as ApplicationLoadBalancedFargateService but having more freedom to configure the infrastructure.
Thanks!

Thanks so much for bringing this to our attention. There are a number of parameter gaps in the default constructs, and I think we should be exposing the target group on the HttpLoadBalancerExtension. I'll discuss getting this prioritized.

authors: bvtujo ztrange

repo: cdklabs/cdk-ecs-service-extensions


text:
Dynamic Dashboard Default Range Incorrect

Version
5.0.0
Steps and/or minimal code example to reproduce
Create any MonitoringFacade with dashboardConfigs and DynamicSegments:
new MonitoringFacade(this, `MonitoringFacade`, {
            dashboardNamePrefix: dashboardName,
            dashboardConfigs: [
                { name: DashboardTypes.Type1, range: Duration.days(7). },
            ],
            ...

Expected behavior
Duration should be converted into a properly formatted string, and the default range of the dashboard should be set correctly according to CloudWatch api spec in the CloudFormation template:
start

    The start of the time range to use for each widget on the dashboard.

    You can specify start without specifying end to specify a relative time range that ends with the current time. In this case, the value of start must begin with -PT if you specify a time range in minutes or hours, and must begin with -P if you specify a time range in days, weeks, or months. You can then use M, H, D, W and M as abbreviations for minutes, hours, days, weeks and months. For example, -PT5M shows the last 5 minutes, -PT8H shows the last 8 hours, and -P3M shows the last three months.

    You can also use start along with an end field, to specify an absolute time range. When specifying an absolute time range, use the ISO 8601 format. For example, 2018-12-17T06:00:00.000Z.

    If you omit start, the dashboard shows the default time range when it loads.

    Type: String

    Required: No

Example:
   "Type": "AWS::CloudWatch::Dashboard",
   "Properties": {
    "DashboardBody": {
     "Fn::Join": [
      "",
      [
       "{\"start\":\"-P7D\",\"widgets\":[{\"type\":\"text\",\"width\":......,
       {
        "Ref": "AWS::Region"
       },

Actual behavior
In the constructor of DynamicDashboardFactory, this is used in the following way:
      const start: string =
        "-" + (dashboardConfig.range ?? Duration.hours(8).toIsoString());

      const dashboard = this.createDashboard(
        renderingPreference,
        dashboardConfig.name,
        {
          dashboardName: `${props.dashboardNamePrefix}-${dashboardConfig.name}`,
          start,
          periodOverride:
            dashboardConfig.periodOverride ?? PeriodOverride.INHERIT,
        }
      );

dashboardConfig.range is restricted to type Duration, and this translates into the cloudformation template, resulting in no default start and the behavior is that it shows 3 hours.
Example:
{
   "Type": "AWS::CloudWatch::Dashboard",
   "Properties": {
    "DashboardBody": {
     "Fn::Join": [
      "",
      [
       "{\"start\":\"-Duration.days(7)\",\"periodOverride\":\"inherit\",\"widgets\"......,
       {
        "Ref": "AWS::Region"
       },

Other details
Maybe let the interface DynamicDashboardConfiguration range property be able to be string?
Or fix from
      const start: string =
        "-" + (dashboardConfig.range ?? Duration.hours(8).toIsoString());

to
      const start: string =
        "-" + (dashboardConfig.range.toIsoString() ?? Duration.hours(8).toIsoString());

#436 has the basic fix, although it could potentially accept an absolute date as a separate feature request.

authors: echeung-amzn ryxli

repo: cdklabs/cdk-monitoring-constructs