EnrichmentContainer interface

This PR introduces EnrichmentContainer interface so that generated EnrichmentMessages can obtain instances from the container.
This interface is implemented by message contexts (which are defined in core-java sub-project, see this PR).

@armiol, PTAL.


Fixed Expception being thrown for non-Android and non-iOS platform

This fixes issues #404

https://github.com/hurbes/stacked/blob/d2311ba317842e1bd3bace396b98599cb5919f0c/packages/stacked_themes/lib/src/theme_manager.dart#L160
I think it should be || not &&.

I apologize you're correct, it should be || not &&. Though it has been already updated, it has been a bad miss on my part.

@hurbes A hotfix is needed since last change broke everyone’s status bar.


Add settings for controlling pan and zoom

Adds a couple of extra settings for pan and zoom:

Disable pan/zoom with mouse wheel/touchpad
Invert pan direction on touchpad (this one is inverted for me on my MacBook)
Adjust pan sensitivity on touchpad
Allow panning even when cursor is over empty space

Do you think it's possible to detect pan sensitivity and direction? It would be best to automatically determine correct settings for a particular device. I can't really check myself as I don't have any Mac devices at hand.
Maybe there is a better alternative to normalize-wheel package I'm using that translates Mac touchpad events better so they don't require any special handling.
It would be really cool if things can just work everywhere =).

I'm by no means a web developer (so you're probably in a better position to investigate this), but I'm more than happy to test things for you on macOS.


Not Working anymore?

Python version:
2.7.12
Bandcamp-dl version:
0.0.6-01
Bancamp-dl options:
????
url:
options:
Describe the issue:
C:\Users\admin\Music\bandcamp>bandcamp_dl.py https://virginbabylonrecords.bandca
mp.com/album/laideronnette
Traceback (most recent call last):
File "C:\Users\admin\Music\bandcamp\bandcamp_dl.py", line 75, in
main()
File "C:\Users\admin\Music\bandcamp\bandcamp_dl.py", line 63, in main
album = bandcamp.parse(url)
File "C:\Users\admin\Music\bandcamp\bandcamp.py", line 9, in parse
r = requests.get(url)
File "C:\Python27\lib\site-packages\requests\api.py", line 70, in get
return request('get', url, params=params, **kwargs)
File "C:\Python27\lib\site-packages\requests\api.py", line 56, in request
return session.request(method=method, url=url, **kwargs)
File "C:\Python27\lib\site-packages\requests\sessions.py", line 488, in reques
t
resp = self.send(prep, **send_kwargs)
File "C:\Python27\lib\site-packages\requests\sessions.py", line 609, in send
r = adapter.send(request, **kwargs)
File "C:\Python27\lib\site-packages\requests\adapters.py", line 485, in send
raise ProxyError(e, request=request)
requests.exceptions.ProxyError: HTTPSConnectionPool(host='virginbabylonrecords.b
andcamp.com', port=443): Max retries exceeded with url: /album/laideronnette (Ca
used by ProxyError('Cannot connect to proxy.', NewConnectionError('<requests.pac
kages.urllib3.connection.VerifiedHTTPSConnection object at 0x027D5510>: Failed t
o establish a new connection: [Errno 11004] getaddrinfo failed',)))
I've turned off my proxy and i get the same error. Any other ideas?

test

Unsupported version of Python.


NPM support

Hi,
Awesome directive.
I need to start using this via NPM rather than bower. The package name in package.json is a bit unclear as just "mask" and it does not seem to have been registered in the NPM registry.
Would you please rename it to "ngMask" and register on the registry...
Cheers,
Andy

I have sent a pull request: https://github.com/candreoliveira/ngMask/pull/36

I have found this one (https://www.npmjs.com/package/ng-mask) but it is not an official

+1 This should really be added to npm registry.

I've reached out to @enriquecastl for permissions to publish the official ngMask to the npm registry under ng-mask.


proxy is failing for newly-created modules

What version of Go are you using (go version)?

$ go version
go version go1.13beta1 darwin/amd64

Does this issue reproduce with the latest release?
I assume so.
What did you do?
deklerk at deklerk-macbookpro2 in ~/workspace/google-cloud-go/firestore on firestore_carveout*
$ go mod tidy
go: finding cloud.google.com/go v0.45.1
everifying<EMAIL_ADDRESS><EMAIL_ADDRESS>reading<EMAIL_ADDRESS>410 Gone
deklerk at deklerk-macbookpro2 in ~/workspace/google-cloud-go/firestore on firestore_carveout*
$ echo $?
1

What did you expect to see?
Proxy fails to fetch a recently-released version of our module.
What did you see instead?
Proxy should go fetch it if it doesn't know about it. It certainly does exist: https://github.com/googleapis/google-cloud-go/releases/tag/v0.45.1

I am not sure if that's the upstream issue (Go module mirror shells out 'go mod download' and it doesn't provide any more details). The module mirror indicated it tried to fetch the version multiple times but couldn't for a while. Finally it managed to access the origin at 14:25:59 and it's available.
<EMAIL_ADDRESS><EMAIL_ADDRESS>


Fix typos

Fix some typos in comments

Hello @flaub, could you take a moment to help review? Thank you very much.


Compile project with Gemfile and/or gemspec

Hi.
First of all, I would like to thank you for the great project.
I use latest ruby-compiler from the GitHub.
When I try to compile a project with Gemfile and gemspec I get an error:
Cannot find entrance /Users/user/ruby/crowdin-cli/bin/crowdin-cli, available entrances are bundle, bundler, crowdin-cli.

I look through the code. And it seems like it tries to get the absolute path to the binary.

Have you tried use the entrance string crowdin-cli instead of the whole path?

Yup
crowdin-cli(master ✗) ruby ../ruby-compiler/bin/rubyc crowdin-cli

[...]

Cannot find entrance /Users/user/ruby/crowdin-cli/crowdin-cli, available entrances are bundle, bundler, crowdin-cli.

Thanks for reporting. Indeed a bug. Let me fix it ASAP.

@mamantoha Fixed in c9cb1a0, works on my local machine:
,

Thank! It's almost working.
https://github.com/pmq20/ruby-compiler/commit/c9cb1a0df7bd18302c44e1eca2d3ded4998517d3#commitcomment-23200482

@pmq20 I have a question that does not relate to this issue.
Why do you use utf-8 with dos line endings and not unix for files?

@mamantoha Sorry I did not pay attention. Does this matter to any issues?


Fix build error caused by mkdocs (v1.5.0)

Fix https://github.com/timvink/mkdocs-charts-plugin/issues/16#issue-1823961327 as suggested by mkdocs' release note:

Breaking change: config.extra_javascript is no longer a plain list of strings, but instead a list of ExtraScriptValue items. So you can no longer treat the list values as strings. If you want to keep compatibility with old versions, just always reference the items as str(item) instead. And you can still append plain strings to the list if you wish.

I would rather write list(map(str, config["extra_javascript"])) instead of [str(item) for item in config["extra_javascript"]] as the same fn. is applied to all elements.
This is a breaking change, would it be possible to review this M.R. A.S.A.P. ?
cc @timvink @mgao6767 @squidfunk

I'm not sure why I was CC'd.

Thanks for the efforts @mgao6767 and @oprypin !


[ko] replace all references to storage.googleapis.com/kubernetes-release with dl.k8s.io

fixes: https://github.com/kubernetes/website/issues/41069

/assign @gochist

Hello @9bany :wave:! It was assigned to @jongwooo in issue #38458.
As you can see in issue #38458, Korean localization uses a working branch to keep up with upstream changes from the previous working branch. Currently it is dev-1.26-ko.1 .
In order to prevent conflicts when merging the working branch into the main branch in the future, it seems necessary to coordinate with @9bany and @jongwooo.
I suggest to @9bany to change the base branch of this PR from main to dev-1.26-ko.1.

Hello @gochist, i don't have any idea how can i create a new branch base from dev-1.26-ko.1
as you can see, i was change base from main to dev-1.26-ko.1, but it too big PR

Hello @gochist, i don't have any idea how can i create a new branch base from dev-1.26-ko.1 as you can see, i was change base from main to dev-1.26-ko.1, but it too big PR

Hey @9bany, I think what you'll need to do is:
Go to the top of this PR and click Edit:

Select the base:main dropdown:

Search for dev-1.26-ko.1 then select it:

Confirm that you want to change the base:

Verify that the new target is `dev-1.26-ko.1:

Note: This was done in my own fork so it shows rjsadow:dev-1.26-ko.1. When you're finished it should say "merge 1 commit into kubernetes:dev-1.26-ko.1 from 9bany/...."

Hey @9bany I think what you're going to need to do is:

Ensure you have all the latest branches available in your repo. git pull upstream
Checkout dev-1.26-ko.1
Cherry pick your commit from your main to dev-1.26-ko.1. git cherry-pick 36cf98b08515f204875338a4d5ff534bc17cb153
Re push.  git push .... --force
Update the target branch to also be dev-1.26-ko.1


LSMI / LMI in Peer overview should be CMI / LMI

As shown in the hornet dashboard:

https://github.com/iotaledger/node-dashboard/blob/b0524e88df5b893c15c4e1fbfac89b18e6845fda/src/app/routes/Peer.tsx#L248
Small issue, easy to fix ;)

For peers that is correct, we only know LSMI from them


Added RN60 values for accessibilityRole and accessibilityStates

Please select one of the following

[ ] I am removing an existing difference between facebook/react-native and microsoft/react-native :thumbsup:
[ ] I am cherry-picking a change from Facebook's react-native into microsoft/react-native :thumbsup:
[ ] I am making a fix / change for the macOS implementation of react-native
[x] I am making a change required for Microsoft usage of react-native

Description of changes
Added the remaining RN60 accessibilityRole and accessibilityStates property values.
RNW uses these values here: https://github.com/microsoft/react-native-windows/commit/74cff3e1dc239ed02d2f3b2bf116f61993c7980e
This change just adds the values so that TS doesn't complain as we start using these values.
Focus areas to test
(optional)

to clarify, TS will still complain until someone updates the @types (I'll probably try to do that just to see what the maintainers say, there are lots of props we'd like to add to the definitive types).
this does fix yellowboxes that show even if you bypass TS checking.

to clarify, TS will still complain until someone updates the @types (I'll probably try to do that just to see what the maintainers say, there are lots of props we'd like to add to the definitive types).
this does fix yellowboxes that show even if you bypass TS checking.

What else needs to be updated? My changes were a little 'shot in the dark', just searching for the property names, and comparing with FB 60-stable and master. Note, they've done a lot more refactoring in this area so I took the 'easy' route and just added the new values, without taking all of their changes.

This will unblock using the properties to not generate runtime warnings.  To get typescript fully happy these typing files -- https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/types/react-native/index.d.ts -- which are written by hand and maintained by the community will also need updating.  Those are not microsoft forked, but given these new props are in RN60 they should be fine to contribute.
we have some overrides in our project that also enable to extend @types/react-native, I haven't successfully extended these ones yet - still playing with it.

/azp run


Merge the Spring Cloud Azure 4.0 back to main branch

We have GAed the Spring Cloud Azure 4.0 from the 4.0 branch, this PR is used to merge it back to the main branch.

@hallipr, please take a look at the changes made in eng/scripts and eng/pipelines. If no concerns, we're planning to merge the pr back to main by the end of this week.

API change check for com.azure.spring:spring-messaging-azure-servicebus
API changes are not detected in this pull request for com.azure.spring:spring-messaging-azure-servicebus

API change check for com.azure.spring:spring-messaging-azure-storage-queue
API changes are not detected in this pull request for com.azure.spring:spring-messaging-azure-storage-queue

API change check for com.azure.spring:spring-integration-azure-core
API changes are not detected in this pull request for com.azure.spring:spring-integration-azure-core

API change check for com.azure.spring:spring-integration-azure-eventhubs
API changes are not detected in this pull request for com.azure.spring:spring-integration-azure-eventhubs

API change check for com.azure.spring:spring-integration-azure-servicebus
API changes are not detected in this pull request for com.azure.spring:spring-integration-azure-servicebus

API change check for com.azure.spring:spring-integration-azure-storage-queue
API changes are not detected in this pull request for com.azure.spring:spring-integration-azure-storage-queue

API change check for com.azure.spring:spring-cloud-azure-autoconfigure
API changes are not detected in this pull request for com.azure.spring:spring-cloud-azure-autoconfigure

API change check for com.azure.spring:spring-cloud-azure-actuator
API changes are not detected in this pull request for com.azure.spring:spring-cloud-azure-actuator

API change check for com.azure.spring:spring-cloud-azure-actuator-autoconfigure
API changes are not detected in this pull request for com.azure.spring:spring-cloud-azure-actuator-autoconfigure

API change check for com.azure.spring:spring-cloud-azure-stream-binder-servicebus-core
API changes are not detected in this pull request for com.azure.spring:spring-cloud-azure-stream-binder-servicebus-core

API change check for com.azure.spring:spring-cloud-azure-stream-binder-servicebus
API changes are not detected in this pull request for com.azure.spring:spring-cloud-azure-stream-binder-servicebus

API change check for com.azure.spring:spring-cloud-azure-stream-binder-eventhubs
API changes are not detected in this pull request for com.azure.spring:spring-cloud-azure-stream-binder-eventhubs

API change check for com.azure.spring:spring-cloud-azure-stream-binder-eventhubs-core
API changes are not detected in this pull request for com.azure.spring:spring-cloud-azure-stream-binder-eventhubs-core

API change check for com.azure:azure-security-keyvault-jca
API changes are not detected in this pull request for com.azure:azure-security-keyvault-jca

API change check for com.azure:azure-data-appconfiguration
API changes are not detected in this pull request for com.azure:azure-data-appconfiguration

API change check for com.azure.spring:spring-cloud-azure-core
API changes are not detected in this pull request for com.azure.spring:spring-cloud-azure-core

API change check for com.azure.spring:spring-cloud-azure-service
API changes are not detected in this pull request for com.azure.spring:spring-cloud-azure-service

API change check for com.azure.spring:spring-cloud-azure-resourcemanager
API changes are not detected in this pull request for com.azure.spring:spring-cloud-azure-resourcemanager

API change check for com.azure.spring:spring-cloud-azure-trace-sleuth
API changes are not detected in this pull request for com.azure.spring:spring-cloud-azure-trace-sleuth

API change check for com.azure.spring:spring-messaging-azure
API changes are not detected in this pull request for com.azure.spring:spring-messaging-azure

API change check for com.azure.spring:spring-messaging-azure-eventhubs
API changes are not detected in this pull request for com.azure.spring:spring-messaging-azure-eventhubs

/azp run azure - spring - tests

API change check for com.azure.spring:azure-spring-cloud-appconfiguration-config
API changes are not detected in this pull request for com.azure.spring:azure-spring-cloud-appconfiguration-config

@hallipr, please take a look at the changes made in eng/scripts and eng/pipelines. If no concerns, we're planning to merge the pr back to main by the end of this week.

Hi @hallipr, since no comments were left on this PR, I will go ahead to merge it to main now. If you have concerns we can change them in the main.

The linting failed because the graalvm sample, so I will override the check enforcer. https://dev.azure.com/azure-sdk/public/_build/results?buildId=1478454&view=results

/check-enforcer override


Remove Editor component

Whether the Editor is displayed or not shouldn't have to be micro-managed by the consumer of the library. The editor is a development time tool. It should automatically be displayed at development time, and excluded otherwise.

Closed by e631fc4eadaa37e91deb98e033c59afbfe72b133


Build Canceled: Broccoli Builder ran into an error with Concat plugin

For of all thank you for the plugin. I very much appreciate your work.
Using version 3.4.0 everything was fine. Upgraded to 3.11.0 and I am getting:
Build Canceled: Broccoli Builder ran into an error with `Concat` plugin. 💥
ENOENT: no such file or directory, open '/var/www/tmp/source_map_concat-input_base_path-bAyyG1nq.tmp/node_modules/blueimp-md5/js/md5.js'
Error: ENOENT: no such file or directory, open '/var/www/tmp/source_map_concat-input_base_path-bAyyG1nq.tmp/node_modules/blueimp-md5/js/md5.js'
    at Error (native)
    at Object.fs.openSync (fs.js:641:18)
    at Object.fs.readFileSync (fs.js:509:33)
    at SourceMap.addFile (/var/www/node_modules/fast-sourcemap-concat/lib/source-map.js:74:33)
    at /var/www/node_modules/broccoli-concat/concat.js:200:16
    at Array.forEach (native)
    at Concat.<anonymous> (/var/www/node_modules/broccoli-concat/concat.js:198:24)
    at /var/www/node_modules/fast-sourcemap-concat/lib/source-map.js:416:14
    at SourceMap.end (/var/www/node_modules/fast-sourcemap-concat/lib/source-map.js:403:12)
    at Concat._doLegacyBuild (/var/www/node_modules/broccoli-concat/concat.js:183:22)

Thank you for reporting! I will look at it. In the meanwhile did you tried already to remove everything?
I mean: rm -rf tmp dist node_modules package-lock.json && npm i

And one more Q; Could you try with v3.10.0?

Thank you for your quick response. Yes, I tried rm -rf tmp dist node_modules and install, but didn't help. I have also tried v3.10.0 now, the result is the same.

npm -v
3.10.10

ember -v
ember-cli: 2.13.2
node: 6.11.0
os: linux x64

package.json
{
  "name": "XXXX",
  "version": "0.0.0",
  "description": "XXXX",
  "license": "MIT",
  "author": "",
  "directories": {
    "doc": "doc",
    "test": "tests"
  },
  "repository": "",
  "engines": {
    "node": ">= 4"
  },
  "private": true,
  "scripts": {
    "build": "ember build",
    "start": "ember server",
    "test": "ember test"
  },
  "devDependencies": {
    "@html-next/vertical-collection": "^1.0.0-beta.8",
    "broccoli-asset-rev": "^2.4.5",
    "ember-ajax": "^3.0.0",
    "ember-chrome-devtools": "^0.2.0",
    "ember-cli": "2.13.2",
    "ember-cli-app-version": "^3.0.0",
    "ember-cli-babel": "^6.0.0",
    "ember-cli-dependency-checker": "^1.3.0",
    "ember-cli-eslint": "^3.0.0",
    "ember-cli-htmlbars": "^1.1.1",
    "ember-cli-htmlbars-inline-precompile": "^0.4.0",
    "ember-cli-inject-live-reload": "^1.4.1",
    "ember-cli-mirage": "^0.4.0",
    "ember-cli-moment-shim": "^3.5.0",
    "ember-cli-notifications": "3.2.0",
    "ember-cli-qunit": "^4.0.0",
    "ember-cli-shims": "^1.1.0",
    "ember-cli-sri": "^2.1.0",
    "ember-cli-tutorial-style": "^1.0.1",
    "ember-cli-toggle": "3.0.0",
    "ember-cli-uglify": "^1.2.0",
    "ember-composable-helpers": "^2.0.3",
    "ember-concurrency": "0.7.15",
    "ember-data": "^2.13.0",
    "ember-export-application-global": "^2.0.0",
    "ember-i18n-iso-countries": "^1.4.0",
    "ember-intl": "2.31.1",
    "ember-intl-cp-validations": "2.3.01",
    "ember-cp-validations": "3.5.1",
    "ember-initials": "3.10.0",
    "ember-load-initializers": "^1.0.0",
    "ember-moment": "^7.5.0",
    "ember-pikaday": "2.2.2",
    "ember-power-select": "^1.10.4",
    "ember-resolver": "^4.0.0",
    "ember-route-action-helper": "^2.0.6",
    "ember-sortable": "^1.10.0",
    "ember-source": "~2.13.0",
    "ember-truth-helpers": "^2.0.0",
    "ember-uploader": "1.2.2",
    "ember-world-flags": "^1.0.0",
    "ember-google-charts": "1.5.3",
    "loader.js": "^4.2.3",
    "semantic-ui-ember": "^2.0.1"
  },
}

It should be fixed in v3.11.1 :) Please, feel free to reopen


Weapons not registering ticks

Keris and the upgraded keris, ornament kit weapons, plugin needs updating

everything added https://github.com/winterdaze/tick-counter/pull/18 thank you guys


Geocoder returning undefined on name lookup

This was working fine for me the other day, but now I can't seem to run even the example using a "name" location.
I'm getting the following error:
/home/kyle/web/Pokemon-GO-node-api/poke.io.js:569
                var _data$results$0$geome = data.results[0].geometry.location;
                                                           ^

TypeError: Cannot read property 'geometry' of undefined
    at /home/kyle/web/Pokemon-GO-node-api/poke.io.js:569:60
    at Request._callback (/home/kyle/web/Pokemon-GO-node-api/node_modules/geocoder/providers/google.js:20:5)
    at Request.self.callback (/home/kyle/web/Pokemon-GO-node-api/node_modules/geocoder/node_modules/request/main.js:122:22)
    at emitTwo (events.js:106:13)
    at Request.emit (events.js:191:7)
    at Request.<anonymous> (/home/kyle/web/Pokemon-GO-node-api/node_modules/geocoder/node_modules/request/main.js:654:16)
    at emitOne (events.js:101:20)
    at Request.emit (events.js:188:7)
    at IncomingMessage.<anonymous> (/home/kyle/web/Pokemon-GO-node-api/node_modules/geocoder/node_modules/request/main.js:616:14)
    at emitNone (events.js:91:20)

This just fixed itself...so I have no idea what happened...


Docs: Native middleware 'SchemaValidationMiddleware' has invalid link

What were you searching in the docs?
Missing documentation for native middleware 'SchemaValidationMiddleware'. The link given in Doc-EventHandler-ApiGatreway is invalid. Therefor the use of 'SchemaValidationMiddleware' isnt't clear.
Is this related to an existing documentation section?
https://github.com/aws-powertools/powertools-lambda-python/blob/b901f231bd99f6f6cd2763470cfbb0ef2b208288/docs/core/event_handler/api_gateway.md?plain=1#L578
How can we improve?
Add valid documentation for 'SchemaValidationMiddleware'.
Got a suggestion in mind?
No response
Acknowledgment

[X] I understand the final update might be different from my proposed suggestion, or refused.

Hey @jvnsg! Thanks for opening this issue and in fact the link is wrong. This should be https://docs.powertools.aws.dev/lambda/python/latest/api/event_handler/middlewares/schema_validation.html.
What do you think about sending a Pull Request to fix this? We'd love to have your first contribution here.


sciond: Add server support for local app requests

SCIOND should be able to respond to the following local app requests:

Path requests
Revocation notifications
AS Info requests
IF Info requests
SVC Info requests

Fixed in https://github.com/scionproto/scion/pull/1632.


Include response output: teams message until teams user commands

Related to #3754

Add the response output for a few of the commands under the service:

teams documentation location

This in place for the following command groups:

teams message
teams messagesettings
teams report
teams tab
teams team
teams user

You can reference the commands planner plan and planner bucket. They are already contain the Response heading in their documentation.
For more information what guidelines to follow, be sure to check out the epic issue #3754

can i work on this? :wave:

You sure can @nicodecleyre 🚀


install for RHEL/CentOS

Coverage remained the same at 89.305% when pulling 1c12338369386b8a7a35db57838bb0342a36814e on remicollet:issue-rpm2 into 06701bce2857d9c18d980dae11ef1361f7b5a04a on jbboehr:master.


setup mongodb app

we need to setup mongodb app with clustering support

@rasheedamir is this issues still valid?


GetRows() is non-deterministic

Description

ChaincodeStub.GetRows() is a function that enable us table-like access to KVS.
Since the order of items from GetRows() is random, the non-deterministic function will damage the blocks when used in invoke.
(The current implementation of PBFT will break the blocks if inconsistent data is written in KVS.)
My suggestion is to rewrite the function to read the all items first, and automatically sort the items in prior to return.
If the developers are OK, I will send the pull request.
Describe How to Reproduce

Call GetRows() twice.

@Ohsaworks  - can you explain what exactly you were doing here?  I definitely understand the fact that almost all iterators in Go are non-deterministic, but what exactly was the chaincode doing that exposed this?

Reading all items is not safe as we don't know how many items there are. This could potentially run the system out of memory. Agree with @mastersingh24, a use case would be helpful.
A way to pass a "sort" to the database may be something we want to consider in the next ledger architecture, but this is often a source of performance issues with large datasets.


Is this project abandoned?

While I don't want to tell anybody in an open source project how to spend their time, I've posted an issue (#14) a month ago and received no responses.
I also see that the last merge (#9) is from late 2018 and the last reply from the repo owner is early 2019 (#10) with a fix for an issue from another user that was never merged (#13).
I'm guessing if I don't get a reply that this project is abandoned, I hope this issue will help other (potential new) users understand the status!

It was, I'm working on it now. Sorry, the pandemic got to me rough.

Just an FYI: I'm going to extensive dive into debugging issues in this issue. If anyone would like to try to help identify issues (no significant coding has to be done, I'll show an example below), it would be more than appreciated since I'm aiming to debug all corner cases in this stylesheet.
The single-widget tests are being done in single.py, which has tests of the following format:
def main(argv=None):
    'Application entry point'

    app = QtWidgets.QApplication(argv)

    # setup stylesheet
    if args.stylesheet != 'native':
        file = QtCore.QFile(f':/{args.stylesheet}.qss')
        file.open(QtCore.QFile.ReadOnly | QtCore.QFile.Text)
        stream = QtCore.QTextStream(file)
        app.setStyleSheet(stream.readAll())

    if args.widget == 'progress_bar_horizontal':
        child = QtWidgets.QProgressBar(widget)
        child.setProperty('value', 24)

    ...
    # Add your tests cases here

# run
    window.show()
    app.exec_()

if __name__ == '__main__':
    main(sys.argv)

Then, you can call the script via:
 python3.9 single.py --stylesheet=windows_dark --widget=progress_bar_horizontal

It also supports a lot of other cases, and allows you to detect differences between windows_dark and native as a great source of initial bugs. Other options include --style (Fusion, Windows, Native), --font-size (change the size of the application font, useful for testing the app at different system font sizes), --scale (multiplier for the application scale, useful for testing bugs with HiDPI screens, etc.), --font-family (useful for detecting bugs for monospace, serif, and sans-serif fonts).
Any tests you wish to add are more than appreciated, and will be immediately merged if you submit a PR.

A sample test and a sample outcome (look at how nice this is now) is:
    elif args.widget == 'scrollarea':
        # For us to have both scrollbars visible.
        child = QtWidgets.QTableWidget(widget)
        child.setColumnCount(100)
        child.setRowCount(100)
        for index in range(100):
            row = QtWidgets.QTableWidgetItem(f'Row {index + 1}')
            child.setVerticalHeaderItem(0, row)
            column = QtWidgets.QTableWidgetItem(f'Column {index + 1}')
            child.setHorizontalHeaderItem(index, column)

Which was then run with the command:
python single.py --stylesheet=windows_dark --widget=scrollarea --font-size=12 --scale=1

Pictures of actual behavior along with expectations of the desired behavior are strongly recommended, as are additional test cases.

I'm happy to say the updates have been completed, and the UI looks better than ever. If you have any specific requests for improvements, @marc-vdm or @RPGHacker, let me know and I'm more than happy to make them.


Error using package manager

Hello.
I got the following error when I run :PackerSync:
packer.nvim: Error running config for statuscol.nvim: .../site/pack/packer/start
/statuscol.nvim/lua/statuscol.lua:134: Vim:E355: Unknown option: statuscolumn

I'm using:
NVIM v0.9.0-dev
Build type: RelWithDebInfo
LuaJIT 2.1.0-beta3
[...]
Features: +acl +iconv +tui
I took a look at that file (to me located at $HOME/.local/share/nvim/site/pack/packer/start/statuscol.nvim/lua/statuscol.lua),
noticed the code local o = vim.o, then the assignment o.statuscolumn = stc (at line 134).
I don't know Lua language to propose a solution.

The error indicates that your neovim version does not support 'statuscolumn'. Please check that you are on the latest dev version, or build from source.


turn into proper project

You rescued me. I tried to install this package and was not able to. Then I cloned the package added your files and it worked. Thank you very much.


Nightly rustfmt

I noticed in 84d7bfc2275e69e5faf497a731e2379e28a3ee15 that you changed the cargo fmt job to the stable toolchain. I think this is fine but I guess I should explain why I would using +nightly to begin with. In lspower I had a custom rustfmt.toml which uses several nightly features (which for some reason have existed for a long time but not been moved to stable).
Anyway, I enabled those settings here and created a PR to compare. I don't think it's really important to merge unless you prefer this formatting as well, but at least I think the merged imports are quite nice. (I'd be happy to apply the formatting to those manually as well).

Thanks so much for the pull request, @silvanshade! :heart: I can definitely see now why +nightly was used originally.
While I'm not entirely against the formatting changes, and I do really like the look of the increased max line width with some of the longer function and method signatures, I do intentionally strive to stick to standard rustfmt settings as closely as possible and whenever possible. I presume the majority of downstream users will likely be working with some version of the standard configuration, and I would like to do the same to motivate and inform the design of the public tower-lsp API, i.e. to avoid excessive line breaks in most common usage.
Still, I may consider adding this again down the road as the standard rustfmt configuration changes. For this reason, I think this pull request is still quite welcome and valuable. Thank you again for bringing this to my attention!


feat: expose checkConsistency with public function

Description
Added a public function for testhelper/tests/checkConsistency. This is needed to reduce code duplication in another project.
Release required?

[ ] No release
[ ] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

/run pipeline

:tada: This PR is included in version 1.26.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Overflow in Recipes catalog

Bug description
The version needs an ellipsis to avoid overflowing:

Operating system
macOS 15.0
Installation Method
None
Version
next (development version)
Steps to reproduce
No response
Relevant log output
No response
Additional context
No response

Duplicates #1833


feature/databricks-delta-incremental-support

PR Overview
This PR will address the following Issue/Feature: Internal tickets and Issue #128
This PR will result in the following new package version: v1.8.0

When I tested this locally for Databricks there was actually no error when running without a full refresh. However, the table format did not change. Therefore, a breaking change should be leveraged to ensure a full refresh is ran and the delta table format is applied.
Please provide the finalized CHANGELOG entry which details the relevant changes included in this PR:

🚨 Breaking Changes 🚨

⚠ Since the following changes result in the table format changing, we recommend running a --full-refresh after upgrading to this version to avoid possible incremental failures.

For Databricks All Purpose clusters the fivetran_platform__audit_table model will now be materialized using the delta table format (previously parquet).

Delta tables are generally more performant than parquet and are also more widely available for Databricks users. Previously, the parquet file format was causing compilation issues on customers managed tables.

Documentation Updates

Updated the sync_start and sync_end field descriptions for the fivetran_platform__audit_table to explicitly define that these fields only represent the sync start/end times for when the connector wrote new or modified existing records to the specified table.

Under the Hood

The is_databricks_sql_warehouse macro has been renamed to is_databricks_all_purpose and has been modified to return true if the Databricks runtime being used is an all purpose cluster (previously this macro checked if a sql warehouse runtime was used).

This update was applied as there have been other Databricks runtimes discovered (ie. an endpoint and external runtime) which do not support the insert-overwrite incremental strategy used in the fivetran_platform__audit_table model.

In addition to the above, for Databricks users the fivetran_platform__audit_table model will now leverage the incremental strategy only if the Databricks runtime is all purpose. Otherwise, all other Databricks runtimes will not leverage an incremental strategy.

PR Checklist
Basic Validation
Please acknowledge that you have successfully performed the following commands locally:

[X] dbt run –full-refresh && dbt test
[X] dbt run (if incremental models are present) && dbt test

Before marking this PR as "ready for review" the following have been applied:

[X] The appropriate issue has been linked, tagged, and properly assigned
[X] All necessary documentation and version upgrades have been applied
[x] docs were regenerated (unless this PR does not include any code or yml updates)
[ ] BuildKite integration tests are passing
[x] Detailed validation steps have been provided below

Detailed Validation
Please share any and all of your validation steps:

To validate these changes the validation tests were included and you can see they were successful for the following destinations:

BigQuery

Databricks All Purpose Cluster

Databricks SQL Warehouse

Additionally, I validated that the All Purpose Cluster appropriately runs an incremental strategy and the non All Purpose (SQL Warehouse in this case) does not run an incremental strategy.

Databricks All Purpose Cluster

Databricks SQL Warehouse

Finally, I confirmed that the Delta format runs as expected and without issue on the Databricks All Purpose cluster on incremental runs.

If you had to summarize this PR in an emoji, which would it be?

🌳

The SQL Server buildkite test is currently failing, but that is due to a permission issue which should hopefully be resolved soon. I will re-kick off the integration tests once that is resolved. I don't imagine SQL Server would be failing for any of these changes; therefore, this should be good to review even with the failing buildkite test.

@fivetran-avinash really really good catch. I ran this only in Databricks so definitely didn't catch that then!
@fivetran-joemarkiewicz Tagging on to Avinash's comments, a more future-proof way to handle the logic update might be:

Put the macro back to just check if it's runtime.
Set the config like:

...
materialized='table' if target.type == 'databricks'
  and not is_databricks_runtime(or whatever the old version name was)() else 'incremental'
...

That way we don't have to list out the other warehouses. What do you think?

@fivetran-catfritz I like that idea, but the benefit of listing out each of the warehouses is we are explicitly only using the incremental strategy if we know the destination is supported. If it is not in our supported list, then we use the table materialization. This likely will provide the greatest opportunity for success if for some reason a destination not supported is used to run this model.

@fivetran-joemarkiewicz Makes sense--in that case approved on my end!


REST API To Import Project to castlemock

We want to create a script to install the docker version of castlemock, and then import a pre-defined project to the container instance. More broadly, are there REST endpoints to perform some of the functions that are done via the provided UI?
Thank you

Hi @oscarAtNava.
Ironically, there isn’t a REST interface for Castle Mock. However, the whole infrastructure in Castle Mock is built in such a way that it would be easy to add a REST interface in the future. Up until this moment, it has been down prioritised. I was waiting for someone to request it. With that being said, it should be very easy to add these functionalities as a REST interface. If you want, I can add this to Castle Mock in the beginning of next week and then create a new release. How does that sound for you?
Would it be possible for you to provide a list of suggestions on which functionalities you would like the RESR interface to provide?
Thank you.
Best regards,
Karl Dahlgren

Hi Karl,
Thank you for the quick response. The most urgent need for us would be the
Import Project function in the UI. We plan to set up a SOAP project and
export it using the UI export function. Then, we are writing a rake task to
programmatically set up the docker container, start it, and import the
pre-defined project.
In the future I can see developers wanting to add to the imported project,
and I can see where they fill spreadsheets of test data, and I
programmatically create a response for a particular end point. Finally the
Update Operation function to be able to enable or disable an operation, or
change it from Mocked to Xpath, etc.
Let me know if you have any questions.
Thanks,
Oscar
On Fri, Mar 16, 2018 at 4:13 PM Karl Dahlgren<EMAIL_ADDRESS>wrote:

Hi @oscarAtNava https://github.com/oscaratnava.
Ironically, there isn’t a REST interface for Castle Mock. However, the
whole infrastructure in Castle Mock is built in such a way that it would be
easy to add a REST interface in the future. Up until this moment, it has
been down prioritised. I was waiting for someone to request it. With that
being said, it should be very easy to add these functionalities as a REST
interface. If you want, I can add this to Castle Mock in the beginning of
next week and then create a new release. How does that sound for you?
Would it be possible for you to provide a list of suggestions on which
functionalities you would like the RESR interface to provide?
Thank you.
Best regards,
Karl Dahlgren
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/castlemock/castlemock/issues/49#issuecomment-373831792,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AiTzFl4b-kJX7W-lVeUpq1PtKl26sxNwks5tfBzRgaJpZM4SuSDN
.

--
Oscar A. Ramirez
Software Engineer
(646) 504-6536
Nava http://navapbc.com/
http://navapbc.com/

Hi @oscarAtNava.
Sorry for the late response. I just finished releasing the latest version of Castle Mock (1.19). The release contain a new REST interface for importing projects and SOAP resources. It currently only has those two services, but I will add more in the upcoming releases. I just wanted to get this version released as soon as possible so that you weren't blocked.
The REST API can be viewed with Swagger UI. There is a link to the API in the Castle Mock footer, or you can go to the following page: {CastleMock}/doc/api/rest.
The services requires either modifier or admin authorization. Either a token or basic authentication can be used to authenticate.
For more information regarding basic authentication, please see the following page: https://en.wikipedia.org/wiki/Basic_access_authentication
In your scenario, you need to use both services to import everything. The exported project file does not contain the WSDL file. That one has to be added through the resource service.
Here are two examples on how to use the services. I hope you find them useful.
Import project example:
curl -X POST "http://localhost:8080/castlemock/api/rest/core/project/import?type=SOAP" -H "accept: */*" -H "Authorization: Basic YWRtaW46YWRtaW4=" -H "Content-Type: multipart/form-data" -F "file=@project-soap-QrTaXl.xml;type=text/xml"
Import WSDL example:
curl -X POST "http://localhost:8080/castlemock/api/rest/soap/resource?projectId=QrTaXl&resourceType=WSDL" -H "accept: */*" -H "Authorization: Basic YWRtaW46YWRtaW4=" -H "Content-Type: multipart/form-data" -F "file=@wsdl.wsdl;type="
I hope that this will resolve your problem. I wish you the best of luck with your automated Castle Mock setup. Please feel free to contact me if you have any further questions, suggestions on improvements or if you simply want to tell me how the process is going. All feedback is appreciated. :)
Best regards,
Karl Dahlgren

Thank you Karl, I will take the new version for a spin and let you know.

Hi @oscarAtNava.
I just wanted to inform you that the upcoming release 1.20 will contain some minor modifications to the REST url. Instead of using request parameters for the identifiers, we will instead be using path parameters.
You will find all the available REST services in the Swagger page in Castle Mock. I hope that this doesn't cause too much problems for you. The functionality is still the same, but the URLs has changed a little bit.
Please contact me if you face any difficulties.
Best regards,
Karl Dahlgren

Thank you for the heads up Karl. We will take a look.
On Sat, May 19, 2018 at 6:31 AM Karl Dahlgren<EMAIL_ADDRESS>wrote:

Hi @oscarAtNava https://github.com/oscarAtNava.
I just wanted to inform you that the upcoming release 1.20 will contain
some minor modifications to the REST url. Instead of using request
parameters for the identifiers, we will instead be using path parameters.
You will find all the available REST services in the Swagger page in
Castle Mock. I hope that this doesn't cause too much problems for you. The
functionality is still the same, but the URLs has changed a little bit.
Please contact me if you face any difficulties.
Best regards,
Karl Dahlgren
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/castlemock/castlemock/issues/49#issuecomment-390395837,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AiTzFt6vhuwCJ5ELe9nPPZHfiX5DLpiJks5tz_RrgaJpZM4SuSDN
.

--
Oscar A. Ramirez
Software Engineer
(646) 504-6536
Nava http://navapbc.com/
http://navapbc.com/

Could you please give me the rest api for import rest xml into the latest version of castlemock


How can I pass a className to individual points?

Hi, I know I can pass a custom className for the whole canvas. But I was wondering if there is any way to pass a custom className to the individual rendered points right now?
To put this in context, I am basically trying to hide individual points on click trigger. If passing the custom className to individual points after the render is not supported atm, do you have any thoughts/suggestions on how I can do that with the functionality that is already there?
Thank you guys, you are doing a great job ❤

I'm pretty much sure you can not at all, since points rendered on canvas are not HTML elements.
If you want to dynamically hide/show certain points, you should change the data source during the runtime and trigger re-render (there may be better ways to do so, though that's the only one I know, and most certainly it should work).

@neilm-code Doable by storing the index of the point in the properties of its GeoJSON. Then write your click handler to get the index from the feature and pass it to .remove(). Here's an example of how I've used this method to fetch data about the point on click: see renderGeography()


Update System.IO.Abstractions to latest

Oh, I just realized there are conflicts that need to be resolved. The code is good otherwise.


resource already exists

Hello.  We have been using control tower and this customization tool since they both came out.    We have had issues with the templates failing on the 2nd run or when a new account is created.   It appears that the resource is not registering.  This may also be a template issue or completely a USER ERROR and for that I apologize wasting your time :)  .   Error below:
The error message is the resource already exists
C Wong was on our account for a long time and introduced us to the tool when it was called Addon Framework.
Extra question do you have AWS addons that can be shared?

Thanks for reaching our to us.
We recommend you to open a AWS support case to help you troubleshoot the error in the CloudFormation template.
In regards to Add-On, it is on our roadmap for the next release.

@fivehorizons Has this been resolved? The 2nd run would likely fail as the template does not delete certain key resources which get created. Think of this is a precaution for some users making changes to the template (without appropriate guardrails) which may result in the resource, such as the Transit Gateway, getting replaced which would result in operational disruption. The new account creation is interesting, this this the spoke or the hub account?

@fivehorizons Did this get resolved for you? The template used by solution does make non-unique names so it would have to be that those resources already exist with the name. Perhaps you can share the full row from the CFN output which would show the resource which already exists?


No ad displayed if CustomEvent adapter has no fill

Issue Description
Banner ads stop being served if the highest-priority network implemented via the Custom Event framework comes back with an error, including No Fill.
Here is an example log file. In it, you can see that a banner is requested every 30 seconds, but when the Amazon request comes back with error no fill, no further attempt at serving a banner ad is made. This is not the expected behaviour.
11-20 12:45:48.211 27809-27809/com.snowleopardgames.lifetotal D/MobFox: dbg: ### MobFoxPlugin >> showBanner
11-20 12:45:48.211 27809-27809/com.snowleopardgames.lifetotal D/MobFoxBanner: request params: {"s":"a03392acae5a7979ecd31244db5cc7b0","o_andadvid":"ee1f7301-f1ba-4625-af74-873692ae88eb","type":"waterfall","adFormat":"banner","autoplay":"true","skip":"","debug":"","rt":"android_app","adspace_width":320,"adspace_height":50,"adspace_strict":0,"sub_bundle_id":"com.snowleopardgames.lifetotal","auto_pilot":true,"v":"Core_3.2.8e","secure":false,"start_muted":false,"dev_js":1,"banner_pos":1,"dev_dnt":1}
11-20 12:45:48.931 27809-27809/com.snowleopardgames.lifetotal D/MobFoxWebView: data >>> :{"customEvents":[{"class":"Amazon","parameter":"002da47d74654bb8982653583e0a947f","pixel":"http:\/\/nvirginia-callbacks.mobfox.com\/customevent.pixel.php?h=a5ac609cd5ec2fd78619200afc9e6462&custom_event_id=stack393247","id":"stack393247"},{"class":"AdColony","parameter":"","pixel":"http:\/\/nvirginia-callbacks.mobfox.com\/customevent.pixel.php?h=a5ac609cd5ec2fd78619200afc9e6462&custom_event_id=stack393209","id":"stack393209"}]}
11-20 12:45:48.931 27809-27809/com.snowleopardgames.lifetotal D/MobFoxBanner: ad response
11-20 12:45:48.931 27809-27809/com.snowleopardgames.lifetotal D/MobFoxTime: ad response time: 717
11-20 12:45:48.931 27809-27809/com.snowleopardgames.lifetotal D/MobFoxBanner: iterator class not found exception
11-20 12:45:48.931 27809-27809/com.snowleopardgames.lifetotal D/MobFox: MobFox Amazon Custom >> constructor
11-20 12:45:48.931 27809-27809/com.snowleopardgames.lifetotal D/MobFox: MobFox Amazon Custom >> loadAd
11-20 12:45:49.481 27809-27809/com.snowleopardgames.lifetotal E/MobFox: MobFox Amazon Custom >> error
11-20 12:45:49.481 27809-27809/com.snowleopardgames.lifetotal D/MobFoxTime: MobFoxBanner >> banner load start time: 1266
11-20 12:45:49.481 27809-27809/com.snowleopardgames.lifetotal E/MobFox: dbg: ### MobFoxPlugin >> onBannerError: errorCode: NO_FILL errorMsg: Server Message: no results. Try again in 10 seconds
11-20 12:46:18.211 27809-27809/com.snowleopardgames.lifetotal D/MobFox: dbg: ### MobFoxPlugin >> showBanner
11-20 12:46:18.211 27809-27809/com.snowleopardgames.lifetotal D/MobFoxBanner: request params: {"s":"a03392acae5a7979ecd31244db5cc7b0","o_andadvid":"ee1f7301-f1ba-4625-af74-873692ae88eb","type":"waterfall","adFormat":"banner","autoplay":"true","skip":"","debug":"","rt":"android_app","adspace_width":320,"adspace_height":50,"adspace_strict":0,"sub_bundle_id":"com.snowleopardgames.lifetotal","auto_pilot":true,"v":"Core_3.2.8e","secure":false,"start_muted":false,"dev_js":1,"banner_pos":1,"dev_dnt":1}
11-20 12:46:18.921 27809-27809/com.snowleopardgames.lifetotal D/MobFoxWebView: data >>> :{"customEvents":[{"class":"Amazon","parameter":"002da47d74654bb8982653583e0a947f","pixel":"http:\/\/nvirginia-callbacks.mobfox.com\/customevent.pixel.php?h=c92afd6bbd31ac741423dedc50a4e6e4&custom_event_id=stack393247","id":"stack393247"},{"class":"AdColony","parameter":"","pixel":"http:\/\/nvirginia-callbacks.mobfox.com\/customevent.pixel.php?h=c92afd6bbd31ac741423dedc50a4e6e4&custom_event_id=stack393209","id":"stack393209"}]}
11-20 12:46:18.921 27809-27809/com.snowleopardgames.lifetotal D/MobFoxBanner: ad response
11-20 12:46:18.921 27809-27809/com.snowleopardgames.lifetotal D/MobFoxTime: ad response time: 714
11-20 12:46:18.921 27809-27809/com.snowleopardgames.lifetotal D/MobFoxBanner: iterator class not found exception
11-20 12:46:18.921 27809-27809/com.snowleopardgames.lifetotal D/MobFox: MobFox Amazon Custom >> constructor
11-20 12:46:18.921 27809-27809/com.snowleopardgames.lifetotal D/MobFox: MobFox Amazon Custom >> loadAd
11-20 12:46:19.441 27809-27809/com.snowleopardgames.lifetotal E/MobFox: MobFox Amazon Custom >> error
11-20 12:46:19.451 27809-27809/com.snowleopardgames.lifetotal D/MobFoxTime: MobFoxBanner >> banner load start time: 1240
11-20 12:46:19.451 27809-27809/com.snowleopardgames.lifetotal E/MobFox: dbg: ### MobFoxPlugin >> onBannerError: errorCode: NO_FILL errorMsg: Server Message: no results. Try again in 10 seconds
11-20 12:46:48.211 27809-27809/com.snowleopardgames.lifetotal D/MobFox: dbg: ### MobFoxPlugin >> showBanner
11-20 12:46:48.211 27809-27809/com.snowleopardgames.lifetotal D/MobFoxBanner: request params: {"s":"a03392acae5a7979ecd31244db5cc7b0","o_andadvid":"ee1f7301-f1ba-4625-af74-873692ae88eb","type":"waterfall","adFormat":"banner","autoplay":"true","skip":"","debug":"","rt":"android_app","adspace_width":320,"adspace_height":50,"adspace_strict":0,"sub_bundle_id":"com.snowleopardgames.lifetotal","auto_pilot":true,"v":"Core_3.2.8e","secure":false,"start_muted":false,"dev_js":1,"banner_pos":1,"dev_dnt":1}
11-20 12:46:48.751 27809-27809/com.snowleopardgames.lifetotal D/MobFoxWebView: data >>> :{"customEvents":[{"class":"Amazon","parameter":"002da47d74654bb8982653583e0a947f","pixel":"http:\/\/nvirginia-callbacks.mobfox.com\/customevent.pixel.php?h=321cefb0944ba8e9aae1f097f4cc8d52&custom_event_id=stack393247","id":"stack393247"},{"class":"AdColony","parameter":"","pixel":"http:\/\/nvirginia-callbacks.mobfox.com\/customevent.pixel.php?h=321cefb0944ba8e9aae1f097f4cc8d52&custom_event_id=stack393209","id":"stack393209"}]}
11-20 12:46:48.751 27809-27809/com.snowleopardgames.lifetotal D/MobFoxBanner: ad response
11-20 12:46:48.751 27809-27809/com.snowleopardgames.lifetotal D/MobFoxTime: ad response time: 549
11-20 12:46:48.751 27809-27809/com.snowleopardgames.lifetotal D/MobFoxBanner: iterator class not found exception
11-20 12:46:48.751 27809-27809/com.snowleopardgames.lifetotal D/MobFox: MobFox Amazon Custom >> constructor
11-20 12:46:48.761 27809-27809/com.snowleopardgames.lifetotal D/MobFox: MobFox Amazon Custom >> loadAd
11-20 12:46:49.441 27809-27809/com.snowleopardgames.lifetotal E/MobFox: MobFox Amazon Custom >> error
11-20 12:46:49.451 27809-27809/com.snowleopardgames.lifetotal D/MobFoxTime: MobFoxBanner >> banner load start time: 1242
11-20 12:46:49.451 27809-27809/com.snowleopardgames.lifetotal E/MobFox: dbg: ### MobFoxPlugin >> onBannerError: errorCode: NO_FILL errorMsg: Server Message: no results. Try again in 10 seconds

Expected Behaviour
When a custom event calls the CustomEventBannerListener's onBannerError() method, the SDK should continue down the waterfall and request an ad from the next network on the list.
Environment
I am working within a Cordova app, so the calling code is based on your example Cordova plugin. However I have put a lot of effort into improving it, so it may not be the same as your test environment. You can see my version here.

Any movement or advice on this issue? It's hurting our revenue significantly.

Hi Andre,
Unfortunately we didn't get a chance to address these issues on latest release as we were focused on MOAT development for the past month.
We will include a fix for that on our next release which should occur during next week.
Sorry for the inconvenience.
Best,
-Shahaf

Fixed in 3.6.0


Remove Java dependency for HTML linting

Increasing Access
Removing the Java dependency for HTML linting would make it easier for people to build the website on their own machines—it would be one less possible point of failure. This would also make it easier to automate building and deploying the website to Google Cloud, where we hope to migrate the site in the future.
Most appropriate sub-area of p5.js?
Other (specify if possible)
Feature request details
Remove or reconfigure the htmllint build step so that Java is not a dependency. npm install should install all needed dependencies for building the site.

Removing the need for Java would be great! @kjhollen Would you like to explore this and make a PR when you are done? htmlhint seems like a possible replacement.


RegisterDevice.py Fehler

Hello while running RegisterDevice.py i get this error:
pi@raspberrypi:~ $ sudo python3 /home/pi/RPi-Jukebox-RFID/scripts/RegisterDevice.py
Choose the reader from list
0 C-Media Electronics Inc.       USB PnP Sound Device
1 HXGCoLtd Keyboard
Device Number: 1
Exception ignored in: <function BaseEventLoop.__del__ at 0x76158618>
Traceback (most recent call last):
  File "/usr/lib/python3.7/asyncio/base_events.py", line 622, in __del__
    warnings.warn(f"unclosed event loop {self!r}", ResourceWarning,
  File "/usr/lib/python3.7/asyncio/base_events.py", line 390, in __repr__
    f'<{self.__class__.__name__} running={self.is_running()} '
  File "/usr/lib/python3.7/asyncio/base_events.py", line 1794, in get_debug
    return self._debug
AttributeError: '_UnixSelectorEventLoop' object has no attribute '_debug'
Exception ignored in: <function BaseEventLoop.__del__ at 0x76158618>
Traceback (most recent call last):
  File "/usr/lib/python3.7/asyncio/base_events.py", line 622, in __del__
    warnings.warn(f"unclosed event loop {self!r}", ResourceWarning,
  File "/usr/lib/python3.7/asyncio/base_events.py", line 390, in __repr__
    f'<{self.__class__.__name__} running={self.is_running()} '
  File "/usr/lib/python3.7/asyncio/base_events.py", line 1794, in get_debug
    return self._debug
AttributeError: '_UnixSelectorEventLoop' object has no attribute '_debug'

What could be the reason? Thanks!

Was fixed in https://github.com/MiczFlor/RPi-Jukebox-RFID/pull/1791, but this is not part of master yet.

How was that fixed? Just changing the protocol in requirements.txt? I've the same issue and would like to solve it manually for my Neuftech Reader EM4100.
Do I have to reinstall the whole applilcation? Just running
cd /home/pi/RPi-Jukebox-RFID/scripts/ python3 RegisterDevice.py
does not solve the issue.
Thanks.

Even forking the develop branch and install again it is not fixed.

Please help I'm having exactly the same problem. I was trying to do a fresh install to get rid of the Spotify version (since it doesn't work). The RFID and everything else was working before, but now I'm getting the same error as becksen when I try to install the RFID, in my case its the RC522.

same problem here, but with usb-Sycreader_RFID_Technology_Co.__Ltd_SYC_ID_IC_USB_Reader_08FF20140315-event-kbd

Same problem here.

V2.4.0 was released today - fixed.


Event bubbling

Test the convenience of a consumed parameter to input events which will set the event as consumed or not.
If the event is not consumed, the input subsystem will continue traversing the scene graph finding suitable actors to consume the event.
Implications:

create a method findAllActorsAtPos which will traverse the whole scene graph and return an Array of Actor elements which lie at any given on-screen coordinate.

Does this have to do with the following too?
I want to create an ActorContainer that can be dragged and that contains Actor children. The Actor children should respond to clicks. At the moment this is not possible. If the Actor has enabled events, all events stop at the actor and the Actor Container's mouseClick and mouseDrag callbacks are never called. Another example is when a Actor Container contains Actors that have events disabled. The events again stop at the actor because the Actor has mouseEnabled set to false.

Yes. With the possibility of programmatically let the input traversal function to find other suitable actors to receive the input event.
Will make some tests to realize whether it is suitable to be added to caat or not.

What is also possible is a manual bubble to parents. At the moment Actors contain all mouse handler stubs. That stub could also do a call on the object's parent (mouseDown: function(eventObject) {if (this.parent.mouseEnabled) this.parent.mouseDown.call(eventObject);}
That way you would not have to change anything in the findActor part but just let actors that are found pass the event along
(don't know exactly if this works with the mouse coordinates though)

This would be great, I'm having an issue with this right now (I want to be able to drag the mouse with the button held down over multiple Actors and have them all notice).

CAAT 0.3 build 208.

Added to CAAT.Scene the following methods:

findActorAtPosition(CAAT.Point) : custom actor find procedure. This is overriden to attend at the priority input lists.
enableInputList(number) :  number of priority lists to set for this scene. A priority list is a list of actors to
which are suitable to receive input first. If no actor on the priority lists are under
the cursor, the whole scene graph is traversed instead.
addActorToInputList( actor, index, position ) : add an actor to a given priority list at certain position.
emptyInputList( index ) : remove all elements from a list.
removeActorFromInputList( actor, index ) : remove an actor from a list, or if no index list is supplied remove
the actor from every list in which appears.


Fix phpdocs

the current comments are inline /* but the suggested would be docblock `/**``  comments. so the current comment should be fine. no?

PHPStorm's intention adds /**, but /* is sufficient for inline comments, I believe.

Note that /** is a docblock comment, not a normal comment. Don't use it except for docblocks

The comments are correct as they are /* is right.


11 nofile

Refactor large parts of Def
Now creates modules for nested schemas in the right context, which makes coverage tools behave nicely. Also had some trouble in Elixir 1.3.1 without this where the compiled code would "disappear" somehow.
Reimplemented defparams in terms of defschema.

I have also tested this branch in my own project and can verify that coverage tools again are happy and that it doesn't bork in Elixir 1.3.1.
Closes #11

Awesome, thank you so much !

Thanks for the review ;)


Update app.ts

fixed some grammar

Thanks for the request. However, the sync folder is for downloading videos, not downloaded videos. It's where all the files are downloaded to, prior to being merged and moved to the library folder. The description is accurate as is.


Add FP16/BF16 fused_attention support with max_seqlen=512

This PR extends the support matrices of the following APIs: nvte_fused_attn_fwd_qkvpacked, nvte_fused_attn_bwd_qkvpacked, nvte_fused_attn_fwd_kvpacked, nvte_fused_attn_bwd_kvpacked to have the 16 bits fused attention with max sequence length = 512. Besides, this PR also adds the JAX sides custom_calls (used internally in TE) for self/cross fused attention. The TransformerLayer and MultiHeadAttention module now will automatically enable the fused attention if available.
Tasks:

[x] Rebase #105 onto the main
[x] Setup the JAX unittests for the fused_attention
[x] Add fused attention 16 bits cuDNN frontend implementations under nvte_fused_attn_{fwd,bwd}_{qkv,kv}packed. (Limitation: padded shape and max sequence length = 512). cuDNN 8.9.1 is required.
[x] Merge https://github.com/NVIDIA/TransformerEngine/pull/185 to update the latest dBias I/O
[x] Update the documents
[x] Fix a bug of generateMatrixStrides and move *_op_create to the fused_attn_utils.cu
[x] Add JAX customcalls SelfFusedAttnMax512FwdPrimitive, SelfFusedAttnMax512BwdPrimitive, CrossFusedAttnMax512FwdPrimitive, CrossFusedAttnMax512BwdPrimitive and the related custom vjp
[x] Automatically enable fused attention if available

Future works:

Support dropout for 16 bits fused attention
Add JAX DotProduct module

/te-ci

/te-ci

/te-ci

/te-ci

/te-ci

/te-ci

/te-ci

/te-ci


iPadOS and iOS ‘move cursor with pressed space bar’ does not work

Hi,
on iPadOS and iOS there is a feature which allows you to move the cursor in text fields while press and hold the space bar. It makes it very easy to place the cursor exactly where you want.
Unfortunately, it does not work on the Better Go Playground. Would it be possible to implement that feature?
Thanks!
Remy

@remyzandwijk Better Go playground uses Monaco editor which also powers VSCode.
The problem is that Monaco doesn't play nice with touch devices and was never designed to support them.
I'm working atm to replace Monaco with CodeMirror for touch devices to mitigate the issue but it will take some time (contributions are appreciated).

Thanks for the explanation @x1unix!

@remyzandwijk can you try this please and tell me if it works there: https://codemirror.net/try/

@x1unix this works, although a bit different than I am used to.
In this editor you need to put your finger ‘on’ the text you want to place the cursor. The magnifier appears above that text. In other editors I know you need to put your finger a bit below the target text and the magnifier appears on top of the line, which makes moving the cursor a bit more natural. I guess it is a matter of getting used to. Thanks!


[Bug]: Building from 0.14.2 tag results in CLI which returns 0.14.1 version when using "-v"

Is there an existing issue for the same bug?

[X] I have checked the existing issues.

Describe the bug and reproduction steps
% git status
HEAD detached at 0.14.2
nothing to commit, working tree clean

% make build
...

% poetry run python -m openhands.core.cli -v
DEBUG:openhands:DEBUG mode enabled.
08:52:41 - openhands:DEBUG: logger.py:265 - Logging initialized
08:52:41 - openhands:DEBUG: logger.py:277 - Logging to file in: /Users/steve_cosenza/workspace/OpenHands/logs
2024-11-25 08:52 - openhands_aci:DEBUG - Logger initialized
0.14.1

### OpenHands Installation

Development workflow

### OpenHands Version

0.14.2

### Operating System

MacOS

### Logs, Errors, Screenshots, and Additional Context

_No response_

Anyone know where this version is coming from? I'll circle back to this later.

Oh I think I understand now. It's due to the way we release.
We actually tag the commit of the release first and then update the versions. So when you checkout the specific tag, it will have the previous version but in reality it's actually using the correct version.


Live word counter giving wrong no. of alphabet

Describe the bug
On giving integer as input alphabet count is increased
To Reproduce
Steps to reproduce the behaviour:
Make some in changes in count_alphabet function in word-count-calcultor
Expected behavior
Alphabet return wrong input and reload the page
Screenshots

**Smartphone (please complete the following information):

Device: realme narzo
Browser chrome

Steps Followed✅
Check the box with [x] If you followed all the steps.

[ ] Describe the bug
[ ] To Reproduce
[ ] Expected behavior
[ ] Screenshots
[ ] Additional context

Code of Conduct✅

[ ] I agree to follow this project's Code of Conduct.
[ ] I checked the current issues for duplicate problems.
[ ] I have gone through project README.

can I work on this issue under GSSoC'22?

@Ashish4321-hub Will you work on it? or I can assign someone else?

@Astrodevil if you are reassigning, please consider me.

@Astrodevil if you are reassigning, please consider me.

I have assigned this issue to the issue creator. Please feel free to raise a new one

hey @Astrodevil , I am a GSSOC'22 contributor . Can you please assign this issue to me . I have send the PR solving the above issue . Can you please review and merge it .

hey @Astrodevil , I have solved this issue and PR is also merged . but it has not been counted in my GSSOC PRs .

@Rishita489 I will check the labels, don't worry. Will let you know in few hours.


Bump Dependencies #1230141978

Bumps dependencies for product-is. Link : https://github.com/wso2/product-is/actions/runs/1230141978

 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.jenkins-is-staging seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


PostgreSQL table and schema asset link is missing from the announcement header

Affected module
UI
Describe the bug
In the announcement header the asset link is missing. The asset is a PostgreSQL table or schema.
To Reproduce

Select an asset with the menitioned type
create an announcement for that asset, save
check the appearing announcement header ("USER posted on table "

Screenshots or steps to reproduce

Expected behavior
A link should be in the header, with the asset name or description
Version:

OS: docker
Python version: docker
OpenMetadata version: 1.2.0
OpenMetadata Ingestion package version: ?

Additional context

@hejatibor I am not able to reproduce the issue on Postgres or any other service. On Sandbox it's working fine. can you help me with more details and api response for the same so we can track the exact issue here.

I found other strange behavior which lead me to that idea, maybe the non-ascii characters in the database service name causes this problem.

At the task view:

Now I can't delete and recreate with ascii-name the db service, on next Monday I can do that.

@hejatibor This was the issue previously and we have fixed it in the 1.2.1 release with this PR. Please update your OM to the latest version.
Closing this issue for now. Let us know if this issue still persists by re-opening the same issue with more details.


"Failed to compile" "PostCSS Loader Invalid Options" useConfigFile is an invalid additional property

npm run dev fails to compile fresh new nuxt.js project.
Nuxt.js v1.4.0
How to reproduce:
vue init vuetifyjs/nuxt my-project
cd my-project
npm i
npm run dev

You will see this error in the console:
⋊> ~/c/my-project npm run dev

><EMAIL_ADDRESS>dev /Users/vasyl/code/my-project
> nuxt

  nuxt:build App root: /Users/vasyl/code/my-project +0ms
  nuxt:build Generating /Users/vasyl/code/my-project/.nuxt files... +1ms
  nuxt:build Generating files... +48ms
  nuxt:build Generating routes... +12ms
  nuxt:build Building files... +28ms
  nuxt:build Adding webpack middleware... +818ms
  ████████████████████ 100%

Build completed in 7.46s

 ERROR  Failed to compile with 1 errors                                                                           16:53:08

 error  in ./assets/style/app.styl

Module build failed: ValidationError: PostCSS Loader Invalid Options

options['useConfigFile'] is an invalid additional property

 @ ./assets/style/app.styl 4:14-195 13:3-17:5 14:22-203
 @ ./.nuxt/App.js
 @ ./.nuxt/index.js
 @ ./.nuxt/client.js
 @ multi webpack-hot-middleware/client?name=client&reload=true&timeout=30000&path=/__webpack_hmr ./.nuxt/client.js

Temporary not really stable workaround.

Open the file ./node_modules/nuxt/lib/builder/webpack/postcss.js
Comment out the line with useConfigFile: false
Run npm run dev - all OK this time.

The same

The same

Thanks!
I made a script to do this through npm run (for Ubuntu).

make a file called replace.sh in project's root folder containing:

#!/usr/bin/env bash
echo "sed -i -e 's/useConfigFile: false,/ /g' ./node_modules/nuxt/lib/builder/webpack/postcss.js

add this in scripts' package.json

    "replace": "./replace.sh"

it will look like this:
  "scripts": {
    "dev": "nuxt",
    "build": "nuxt build",
    "start": "nuxt start",
    "generate": "nuxt generate",
    "lint": "eslint --ext .js,.vue --ignore-path .gitignore .",
    "precommit": "npm run lint",
    "replace": "./replace.sh"
  },

wait for update ;)

It seems to be a problem after postcss-loader updated to 2.1.2.
Setting the version of postcss-loader to  2.1.1 solved the problem form me.

Look at postcss-loader github:
https://github.com/postcss/postcss-loader/issues/347

Locking the version of postcss-loader at 2.1.1 for nuxt dependencies in package-lock.json resolves the issue.

for now you can solve this by installing in dev dependancies manually<EMAIL_ADDRESS>

same issue. use npm install --save-dev<EMAIL_ADDRESS>fixed.

The same problem for me! Downgrade package doesn't works for me, when I push on production, I get back this error.

hmm very odd, what do you mean with pushing on production @pirmax

@pirmax do u try npm install --save-dev<EMAIL_ADDRESS>

@derevandal Yes, I tried, with Yarn :+1:
@bovas85 I mean, I pushing on production with Dokku, while push, the build is OK, but in console the build got me an error: https://pastebin.com/05vunrzx

Did you try removing the node_modules folder or doing npm rebuild ?

same here! i solved with<EMAIL_ADDRESS>in devDependencies.
@pirmax check the yarn.lock maybe the version is locked here.

How I can update only yarn.lock instead of package.json? I don't know packages very well...

<EMAIL_ADDRESS>is released

The owner of the postcss-loader is a very kind person. He solved the issue for all of us.
Thank you @ai !


singleSelect() error

When multiselect field created as singleSelect(), one option selected, and when I click on same option again I get error in console.
Picture attached.

Thank you for reporting, this issue is fixed now in [1.0.4]. Sorry for the inconvenience.


Cmake version 3.5.1does not work (git ff26b35 2018-11-06)

Behavior
cmake failed
Steps to Reproduce the Problem
cmake -DCMAKE_BUILD_TYPE={Release,Debug}     -DHPX_WITH_MALLOC=system  ..
-- Performing Test HPX_WITH_RDTSCP - pre-set to TRUE
-- Boost version: 1.58.0
-- Found the following Boost libraries:
--   atomic
--   filesystem
--   program_options
--   regex
--   system
-- WARNING: HPX will perform poorly without tcmalloc or jemalloc. See docs for more info.
-- Using system allocator.

CMake Error at examples/resource_partitioner/CMakeLists.txt:14 (if):
if given arguments:
"CMAKE_CXX_COMPILER_VERSION" "VERSION_GREATER_EQUAL" "5.0"
Unknown arguments specified
-- Configuring incomplete, errors occurred!
See also "/home/aaa/hpx/hpx_git_x/bld/CMakeFiles/CMakeOutput.log".
Specifications
... Please describe your environment

HPX Version:  git ff26b35 2018-11-06
Platform (compiler, OS):
gcc (Ubuntu 5.4.0-6ubuntu1~16.04.10) 5.4.0 20160609
Ubuntu 16.04 x86-64
cmake version 3.5.1

VERSION_GREATER_EQUAL
I investigated and this command is only available at CMake 3.7+ and we should bump our minimal cmake version to 3.7. I will do a PR soon.

Couldn't we fix this without raising the cmake requirements?

Yes, we would need to add to all related files a check for the cmake version and depending on the version we would need the use the correct syntax.
I checked and all major linux derivations have at least CMake 3.7, even debian stable has it. So I would recommend to upgrade the cmake version instead to add the check to all related files.
@Yeahhhh Is using Ubuntu 16.04, which is around 2.5 years old and a little bit outdated.

I am not interested in supporting very old compilers, or old cmake installations (it takes 5 minutes to build your own cmake on almost any machine). I suggest we bump the cmake requirement to 3.7 as suggested by @diehlpk

I suggest that we should bump the CMake requirement to 3.8 instead. CMake 3.8 introduces the ability to require a specific language standard on a per target basis without manually specifying the (compiler-dependent) flag.
In my experience, HPX's handcrafted solution for selecting the requested standard and propagating this decision to downstream users is error-prone and hard to maintain. I would like to get rid of this particular part of the build system sooner rather than later.

@chinz07 Do you intend to submit these changes to the release 1.2? If yes, I would upgrade to 3.8, if not to 3.7 so people can build it on debian stable. Once you integrate this, we could upgrade to 3.8.

I am not interested in supporting very old compilers, or old cmake installations (it takes 5 minutes to build your own cmake on almost any machine)

@biddisco you're the one advocating to reduce dependencies (in particular Boost linked libraries). Now you're suggesting to add another burden onto the user?

Do you intend to submit these changes to the release 1.2?

@diehlpk, @chinz07 I don't think that adding such changes to the release is advisable.
In general, I'd be fine with raising the cmake requirements to a reasonable version, I'd strongly suggest to do that after the release, though - possibly in relation with a redesign of our build system. For the release I'd suggest to find a different solution for the discussed problem.

@hkaiser The fix for the release will be a little bit messy because we would have to add to all CMake files an additional if case where we check for the cmake version. I could do this, but I do not think it is worth since all major Linux systems provide at least 3.7.

Why, just use LESS and EQUAL everywhere. No harm done.

@diehlpk @hkaiser Just to clarify, I wanted to suggest to bump the requirement after the 1.2 release. I wrongly assumed that the proposed change would already target HPX 1.3.
VERSION_GREATER_EQUAL seems to be used only once. Am I missing something?

@chinz07 Ok, I will provide a PR soon and just update this one occurrence.

Just to clarify, CMake 3.5 support VERSION_LESS and VERSION_GREATER. Those should be used at the place in question. LESS_EQUAL has slightly different semantics and should be avoided.
This should do the trick:
NOT CMAKE_CXX_COMPILER_VERSION VERSION_LESS 5.0

@Yeahhhh please try #3534 and see if it fixes your problem.

I switched to Ubuntu1804+cmake 3.10.2.

@biddisco you're the one advocating to reduce dependencies (in particular Boost linked libraries). Now you're suggesting to add another burden onto the user?

Any serious developer who wants to use HPX should be using a more recent compiler and toolchain. It's just not our business to support older technologies. The principle of open source is not that every user in the world is allowed to complain that X doesn't work on their system, it's there so that if they care enough to complain about it, they can take the 5 minutes to try to fix it. As it turns out, @sithhell cares enough, and he has supplied a patch. There are lots of places in HPX where I do care, and I do spend my time trying to help: I've spent a great deal of time trying to provide build system tools to help our coverage of problems on all PRs, but at some point, we must draw a line. Mine is just a tadge higher than yours in this case - and I do not feel bad about it (so don't try to make me feel guilty).

On reflection. I do feel guilty. I should have taken the 5 minutes to change greater_equal to not less since it is so simple. And I checked and cmake 3.5 only dates back to 2016, which is not such a long time ago. So please accept my apologies for being grumpy about this.


Installation is insecure by default

Exposing services unauthenticated is insecure, and could lead to the system being compromised unless the deployer takes additional actions to secure their system. The deployment defaults should be as secure as they can be.
I'd suggest disabling ingress for the agent, grafana and prometheus, and letting users enable those ingresses with the security configuration that is relevant to their setup.

Thanks for the fast comment @chriswessels  !!
That can be taken care of. So your suggestion is to only leave enabled the service endpoint and flag the other disabled ingresses with comments for the user ?

Yes. Then I'd expect the easiest secure way access those services is with kubectl port-forward instead of with an ingress.

Thanks for the feedback.
We adapted the configuration considering  your inputs.


[WIP] Indent functionality

Heya! Really grateful to have found your RichTextKit swift package, Daniel! I thought it would be great to add indentation, and eventually bullet points, so I've started implementing this in this fork!
This is very much a rough draft but I tried to follow the standards and practices you employed in the original package. I'm also still finding my way through xcode, github, and swiftui so please let me know if there are any questions/recommendations/comments from what you see! In it's current state there are some hiccups - namely an issue where when you return to a new line and hit either indent option and then begin to type, that indentation will apply to the previous line. Planning to smooth out that issue over the next couple days but I'm sure it's an easy enough bug to squash!
Thanks again!

Hi @jamesbradleym
Thank you so much for looking into this. I will take a look at it when I have time 👍

@danielsaidi Still working on trying to bring this up to fully implemented but I'm hitting a roadblock with indentation after a newline.
If you provide a new character/string after a newline then changing the paragraph style attributes with an updated firstline head indent and head indent then it works, but if there is no new character or string to apply this styling to then it styles the previous line which the newline is technically part of in the NSAttributedString. Still investigating how to apply a style to characters following the newline where the cursor is sitting..

Additional testing shows this problem also persists with text alignment. When 1 newline is given at the end of a NSAttributedString alignment does not apply, after typing into the richtexteditor the alignment is applied to the previous line/paragraph. When 2 newlines are given following a visual paragraph these features work as expected.

Hey @jamesbradleym - great finds! There are some platform-specifics that I'd love to iron out. Unfortunately, some of the changes have side-effects on other handling, so it's quite tricky. I need to dedicate a few full days on this soon.

Hi again @jamesbradleym - I've been busy with other projects, but I'm very curious to hear about your progress with this.

Hey @danielsaidi , sorry for the delay here - I haven't had a chance to track down the unexpected indent behavior with this added functionality so I wouldn't say it's ready to be merged necessarily but I'd be grateful for any review or comments you may have!

Perfect, I hope that I'll have time later in May 👍

I've had a chance to look at this more and I think I may be on to a solution though it feels a bit hacky.. will commit sometime this week when it's a bit more fleshed out hopefully

Sounds great @jamesbradleym - just let me know when you want me to take a look at it. 👍

@danielsaidi still hunting down a proper method to work this in. This implementation has a hang-up when attempting to indent at a location without characters, such as after a new line or at the very beginning of a document without text. If I'm being honest the work around (at least for the 'after newline' condition, need to hone in on the beginning of document issue) it feels a bit hacky, unfortunately I wasn't able to get indentation to work without giving NSAttributedString a character to work with. So I've given it "\u{00A0}", a non-breaking space, the cursor position is kept right before this character so the UI updates as expected. However my first attempts at having this space removed resulted in the paragraph styling being removed as well because it was using that character to style..
Other indent situations work as expected, indenting within a paragraph, at the beginning of a paragraph, end of a paragraph, etc have not had issues.
It would be great to get your feedback on this, even in its current state, if you have an opportunity. It would be very helpful to get your insight on both solving this issue and also getting a better understanding of what level of resolution this would require before you'd be comfortable accepting the feature.
Thanks!

Hi @jamesbradleym
Wow, sounds like you've put quite a lot of time into it - that's amazing!
I am working with a client that uses RTK now, so I will have more time to put on this project. I'll have a look in a day or two 👍

Hey @danielsaidi ,
So scratch everything I just said, after taking a look at why the first indent of a blank document wasn't working I stumbled across the bug, I was creating a fresh style that had no reference to our current typingAttributes paragraph style! I fixed this and it led me to the solution for newline as well, the exact same implementation! So I moved up the newline at index check and have it run the same function which updates our typingAttributes with an updated indent styling. Looks like it's working from my end! Should probably test it a bit longer but I need to put this down and come back with fresh eyes.
Should be good to review when you're ready though!

Wow, that's great! I'll take a look tomorrow or the day after that 🎉

I've removed quite a bit of the tags code, there are still some remnants along with some additional code that I had been toying with as a workaround to the indent walls I was hitting before the last update. Will continue cleaning up and let you know when its pure indent updates!

@danielsaidi Ready for review! 👍🏻

@jamesbradleym
This PR looks absolutely amazing - such impressive work! I manage to make some changes to the toolbar, which cause some merge conflicts. I will fix the conflicts then merge the PR.
I however don't think the indent actions should be added to the toolbar by default, since the space is so limited on iPhone. I will instead add the actions to the the rich text format sheet, and figure out how to best add them to macOS.
The merge conflict was caused by some changes I made to the toolbar, where I have moved some init arguments to the style and also added a way to make it possible to modify the format sheet from the initializer.
I used this new modifier to apply undimmed presentation detents to the sheet, so that you can edit the underlying text editor while the sheet is opened:

With all these additions, I think the 0.6 will be a quite nice update! :)

Fantastic, thanks for all your work! Definitely agree on conserving space with iOS, bit tricky!

I redesigned the iOS/iPadOS format sheet, so that we have the indentation buttons there as well:


Roll Flutter Engine from f20657354d8b to 1a4db4a76966 (10 revisions)

https://github.com/flutter/engine/compare/f20657354d8b...1a4db4a76966
2024-01-16<EMAIL_ADDRESS>[Flutter GPU] Run unittests on CI and fix HostBuffer. (flutter/engine#49789)
2024-01-16<EMAIL_ADDRESS>[Flutter GPU] Fix playground shader paths. (flutter/engine#49790)
2024-01-16<EMAIL_ADDRESS>[Windows] Add README (flutter/engine#49779)
2024-01-16<EMAIL_ADDRESS>Fix typo (flutter/engine#49725)
2024-01-16<EMAIL_ADDRESS>Truncate thread names on Linux to the maximum allowed length (flutter/engine#49781)
2024-01-16<EMAIL_ADDRESS>[Impeller] Fix hot reload for shaders on Impeller (flutter/engine#49739)
2024-01-16<EMAIL_ADDRESS>Add flutter prefix to import (flutter/engine#49793)
2024-01-16<EMAIL_ADDRESS>Support running sound null safe kernels from dart_runner (flutter/engine#49598)
2024-01-16<EMAIL_ADDRESS>Manually bump Dart to 3.4.0-34.0.dev (flutter/engine#49792)
2024-01-16<EMAIL_ADDRESS>Bump actions/upload-artifact from 3.1.3 to 4.1.0 (flutter/engine#49788)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jonahwilliams@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Added groundwork for storing additional feature image data

requires https://github.com/TryGhost/SDK/pull/387 (checkout and yarn link)
We want to allow setting captions for feature images but there are technical hurdles that prevent a straightforward implementation:

we can't push more columns into the posts table (we're hitting max row length)
we can't push more columns into the posts_meta table (this will become a generic meta data table in future)
we don't want to store feature images inside mobiledoc (feature images are used outside of post content)

To work around these the best solution will be to store images as their own model and link them to parent models. This change takes the approach of using Bookshelf's polymorphic associations to link an image to a parent via an imageable association.

adds images table
migrates existing posts.feature_image values to image records

posts.feature_image needs to be kept around until a major release where we are able to drop it

adds Image model
updates Post model

adds feature_image() association
updates default fetch behaviour to include feature_image association

updates v2 and v3 API serializers to convert the embedded feature_image record back into a URL string to maintain backwards compatibility
updates {{img_url}} helper to handle a plain URL string or an embedded image resource to maintain compatibility across APIs

Closing in favour of adding fields directly to posts_meta (https://github.com/TryGhost/Ghost/pull/13030).


GAM only checks current directory for client_secrets.json

Platform: Windows 64-bit
Version: 3.51
I have GAM installed to C:\users\user\Applications\gam and have that included in my PATH variable so I can use GAM anywhere. After updating to 3.51 I'm being prompted to create my client_secrets.json file whenever the gam directory is not my current path. Here's a sample CMD session which recreates the issue:
C:\Users\user> gam info user test

<Blurb about creating client_secrets.json>

C:\Users\user> cd Applications\gam

C:\Users\user\Applications\gam> gam info user test

<Displays information about username test>

I don't remember what version it was I upgraded from (it was quite old) but using GAM from arbitrary paths worked fine back then. It seems like GAM expects client_secrets.json to exist in your current directory and fails if it does not--even if it exists in the path GAM is installed in.

Getting this too with a recent install after upgrade to Windows 10. Use to work fine.

I think the problem is due to the GC_Defaults array setting GC_CONFIG_DIR to null. I'm going to test with using os.path.dirname(os.path.realpath(file)) as the default value
183: GC_CONFIG_DIR: u''


Remove zombie branches created by greenkeeper.io

Hmmm... looks like using greenkeeper will require some occasional lawn mowing...


Not able to navigate using navigator service

Unhandled Exception: You are trying to use contextless navigation without
a GetMaterialApp or Get.key.
If you are testing your app, you can use:
[Get.testMode = true], or if you are running your app on
a physical device or emulator, you must exchange your [MaterialApp]
for a [GetMaterialApp].
import 'package:get_it/get_it.dart';
import 'package:injectable/injectable.dart';
import 'locator.config.dart';

final locator = GetIt.instance;

@InjectableInit()
void setupLocator() => $initGetIt(locator);

@AdaptiveAutoRouter(
    routes: <AutoRoute>[
      AutoRoute(page: SplashView, initial: true),
      AutoRoute(page: LanguageView),
    ]
  )
  class $Router{}

void main() {
  setupLocator();
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return GetMaterialApp(
      navigatorKey: locator<NavigationService>().navigatorKey,
      home: SizedBox(),
      builder: ExtendedNavigator.builder<route.Router>(
        initialRoute: route.Routes.splashView,
        router: route.Router(),
        builder: (context, extendedNav) => Theme(
            data: appTheme(),
            child: extendedNav),
      ),
    );

  }
}

class SplashViewModel extends BaseViewModel{
  final _navigationService = locator<NavigationService>();

  Future initialise() async {
    new Future.delayed(const Duration(seconds: 3), () async {
      await _navigationService.pushNamedAndRemoveUntil(Routes.languageView);
    });
  }
}

import 'package:injectable/injectable.dart';
import 'package:stacked_services/stacked_services.dart';
@module
abstract class ThirdPartyServiceModule{
  @lazySingleton
  NavigationService get navService;
}

you need to call onGenerateRoute
onGenerateRoute: auto_router.Router().onGenerateRoute,
for example :
`void main() async {
setupLocator();
// setupDialogUi();
runApp(MyApp());
}
class MyApp extends StatelessWidget {
@override
Widget build(BuildContext context) {
return MaterialApp(
debugShowCheckedModeBanner: false,
title: 'Git Comment',
initialRoute: auto_router.Routes.homeView,
onGenerateRoute: auto_router.Router().onGenerateRoute, // this line is needed
  navigatorKey: locator<NavigationService>().navigatorKey,
);

}
}`

you need to call onGenerateRoute
onGenerateRoute: auto_router.Router().onGenerateRoute,
for example :
`void main() async {
setupLocator();
// setupDialogUi();
runApp(MyApp());
}
class MyApp extends StatelessWidget {
@override
Widget build(BuildContext context) {
return MaterialApp(
debugShowCheckedModeBanner: false,
title: 'Git Comment',
initialRoute: auto_router.Routes.homeView,
onGenerateRoute: auto_router.Router().onGenerateRoute, // this line is needed
  navigatorKey: locator<NavigationService>().navigatorKey,
);

}
}`

You need to call onGenerateRoute first. in your MaterialApp
onGenerateRoute: auto_router.Router().onGenerateRoute
My Code:
void main() async {   setupLocator();   runApp(MyApp()); }
class MyApp extends StatelessWidget { @override   Widget build(BuildContext context) { return MaterialApp(       debugShowCheckedModeBanner: false,       title: 'Dj Jag',       home: LoginView(), initialRoute: auto_router.Routes.homeView, onGenerateRoute: auto_router.Router().onGenerateRoute, navigatorKey: locator<NavigationService>().navigatorKey, ); } }

You need to call onGenerateRoute first. in your MaterialApp
onGenerateRoute: auto_router.Router().onGenerateRoute
My Code:
void main() async {   setupLocator();   runApp(MyApp()); }
class MyApp extends StatelessWidget { @override   Widget build(BuildContext context) { return MaterialApp(       debugShowCheckedModeBanner: false,       title: 'Dj Jag',       home: LoginView(), initialRoute: auto_router.Routes.homeView, onGenerateRoute: auto_router.Router().onGenerateRoute, navigatorKey: locator<NavigationService>().navigatorKey, ); } }

@SrikanthKoti
Added onGenerateRoute to the main file, but there's no use

@SrikanthKoti
Added onGenerateRoute to the main file, but there's no use

@SrikanthKoti
Added onGenerateRoute to the main file, but there's no use

I had the same error, for me it solved the issue....
Is It showing the same error now ?

@SrikanthKoti
Added onGenerateRoute to the main file, but there's no use

I had the same error, for me it solved the issue....
Is It showing the same error now ?

Is this still a problem?

Is this still a problem?

@FilledStacks Yes, still getting the issue

@FilledStacks Yes, still getting the issue

@charantft are there any awaits in your setupLocator function?

@charantft are there any awaits in your setupLocator function?

@FilledStacks I have the same problem and no, there are not any awaits in the setupLocator function.

@sebastianbuechler this usually happens if you try to navigate before the navigation service is setup or before the locator registrations has completed. Can you paste your locator code below please

I debug the project:  https://github.com/murilosandiego/delivery-app-getx.git
same problem.

flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 1.22.5
[√] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
[√] Android Studio (version 3.5)

I/flutter (17710): ══╡ EXCEPTION CAUGHT BY WIDGETS LIBRARY ╞═══════════════════════════════════════════════════════════
I/flutter (17710): The following message was thrown building GetMaterialApp:
I/flutter (17710): You are trying to use contextless navigation without
I/flutter (17710):       a GetMaterialApp or Get.key.
I/flutter (17710):       If you are testing your app, you can use:
I/flutter (17710):       [Get.testMode = true], or if you are running your app on
I/flutter (17710):       a physical device or emulator, you must exchange your [MaterialApp]
I/flutter (17710):       for a [GetMaterialApp].
I/flutter (17710): The relevant error-causing widget was:
I/flutter (17710):   GetMaterialApp file:///D:/src/delivery-app-getx/app/lib/main.dart:26:12
I/flutter (17710): When the exception was thrown, this was the stack:
I/flutter (17710): #0      GetNavigation.global (package:get/get_navigation/src/extension_navigation.dart:1046:7)
I/flutter (17710): #1      GetNavigation.toNamed (package:get/get_navigation/src/extension_navigation.dart:571:12)
I/flutter (17710): #2      AuthStore.fireRoute (package:delivery_app_getx/core/stores/auth_store.dart:37:11)
I/flutter (17710): #3      ever. (package:get/get_rx/src/rx_workers/rx_workers.dart:69:44)
I/flutter (17710): #4      GetStream._notifyData (package:get/get_rx/src/rx_stream/get_stream.dart:47:21)
I/flutter (17710): #5      GetStream.add (package:get/get_rx/src/rx_stream/get_stream.dart:97:5)
I/flutter (17710): #6      RxObjectMixin.value= (package:get/get_rx/src/rx_types/rx_core/rx_impl.dart:101:13)
I/flutter (17710): #7      AuthStore.onInit (package:delivery_app_getx/core/stores/auth_store.dart:31:16)
I/flutter (17710): #8      GetLifeCycleBase._onStart (package:get/get_instance/src/lifecycle.dart:66:5)
I/flutter (17710): #9      _InternalFinalCallback.call (package:get/get_instance/src/lifecycle.dart:12:25)
I/flutter (17710): #10     GetInstance._startController (package:get/get_instance/src/get_instance.dart:290:18)
I/flutter (17710): #11     GetInstance._initDependencies (package:get/get_instance/src/get_instance.dart:244:11)
I/flutter (17710): #12     GetInstance.find (package:get/get_instance/src/get_instance.dart:334:17)
I/flutter (17710): #13     GetInstance.put (package:get/get_instance/src/get_instance.dart:103:12)
I/flutter (17710): #14     Inst.put (package:get/get_instance/src/extension_instance.dart:90:21)
I/flutter (17710): #15     InitialBinding.dependencies (package:delivery_app_getx/core/routes/bindings/initial_binding.dart:14:9)
I/flutter (17710): #16     GetMaterialApp.build. (package:get/get_navigation/src/root/get_material_app.dart:229:25)
I/flutter (17710): #17     _GetBuilderState.initState (package:get/get_state_manager/src/simple/get_state.dart:124:23)
I/flutter (17710): #18     StatefulElement._firstBuild (package:flutter/src/widgets/framework.dart:4765:58)
I/flutter (17710): #19     ComponentElement.mount (package:flutter/src/widgets/framework.dart:4601:5)
I/flutter (17710): ...     Normal element mounting (13 frames)
I/flutter (17710): #32     Element.inflateWidget (package:flutter/src/widgets/framework.dart:3569:14)
I/flutter (17710): #33     Element.updateChild (package:flutter/src/widgets/framework.dart:3327:18)
I/flutter (17710): #34     RenderObjectToWidgetElement._rebuild (package:flutter/src/widgets/binding.dart:1252:16)
I/flutter (17710): #35     RenderObjectToWidgetElement.mount (package:flutter/src/widgets/binding.dart:1223:5)
I/flutter (17710): #36     RenderObjectToWidgetAdapter.attachToRenderTree. (package:flutter/src/widgets/binding.dart:1165:17)
I/flutter (17710): #37     BuildOwner.buildScope (package:flutter/src/widgets/framework.dart:2683:19)
I/flutter (17710): #38     RenderObjectToWidgetAdapter.attachToRenderTree (package:flutter/src/widgets/binding.dart:1164:13)
I/flutter (17710): #39     WidgetsBinding.attachRootWidget (package:flutter/src/widgets/binding.dart:974:7)
I/flutter (17710): #40     WidgetsBinding.scheduleAttachRootWidget. (package:flutter/src/widgets/binding.dart:955:7)
I/flutter (17710): (elided 11 frames from class _RawReceivePortImpl, class _Timer, dart:async, and dart:async-patch)

@dashan11 I have no idea what's happening. It's working when setting up with a brand new project. If you can provide me a sample where it's happening using the navigation service then please upload that to git and provide me with a link here.

I had started a new project and I had the same problem.
I added the navigatorKey: StackedService.navigatorKey to my ExtendedNavigator and navigation started to work.
I hope this new line will not bring me future problems.
class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      builder: ExtendedNavigator(
        router: router.Router(),
        initialRoute: router.Routes.startupView,
        **navigatorKey: StackedService.navigatorKey,**
     ),
    );
  }
}

@charantft yes, if you're using the extended navigator you can pass the key into that instead. That will work just fine