[iOS] TextInput placeholder position changes after a long drag of cursor and text clear

Description
Following the steps in 'Steps To Reproduce', the placeholder text has moved forward a little bit, and the cursor is blinking in between the placeholder, like so:

React Native version:
System:
OS: macOS 10.15.3
CPU: (4) x64 Intel(R) Core(TM) i5-7287U CPU @ 3.30GHz
Memory: 53.38 MB / 16.00 GB
Shell: 5.7.1 - /bin/zsh
Binaries:
Node: 10.16.3 - /usr/local/bin/node
Yarn: 1.15.2 - ~/.yarn/bin/yarn
npm: 6.9.0 - /usr/local/bin/npm
Watchman: 4.9.0 - /usr/local/bin/watchman
Managers:
CocoaPods: 1.8.4 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: iOS 13.4, DriverKit 19.0, macOS 10.15, tvOS 13.4, watchOS 6.2
Android SDK:
API Levels: 21, 23, 24, 25, 26, 27, 28
Build Tools: 23.0.1, 26.0.2, 27.0.3, 28.0.0, 28.0.2, 28.0.3
System Images: android-23 | Intel x86 Atom_64, android-23 | Google APIs Intel x86 Atom_64
Android NDK: 13.1.3345770
IDEs:
Android Studio: 3.4 AI-183.6<IP_ADDRESS>22156
Xcode: 11.4.1/11E503a - /usr/bin/xcodebuild
Languages:
Java: 1.8.0_172 - /usr/bin/javac
Python: 2.7.15 - /usr/local/bin/python
npmPackages:
@react-native-community/cli: Not Found
react: 16.11.0 => 16.11.0
react-native: 0.62.2 => 0.62.2
npmGlobalPackages:
react-native: Not Found
Steps To Reproduce

Setup a single line TextInput with some placeholder text in it
Set the clearButtonMode to 'while-editing'
Type in some long content, they should be long enough to cause TextInput to scroll horizontally  a long way
Drag the cursor without stopping to the begin of the content
Clear the content with the clear button

Expected Results
Placeholder should be in the right place
Snack, code example, screenshot, or link to a repository:
Like the screenshot above, I am using the project created by npx react-native init MyTestApp.
After that, just put a TextInput like this below the Header：
<TextInput
     style={{
         height: 30,
         backgroundColor: 'aqua',
         flex: 1,
     }}
     clearButtonMode={'while-editing'}
     placeholder={'This is some placeholder'}
/>

It should be reproducible with any long content, but I am not sure how long it should be. Here is the long content I used to take the screen shot：
UITextPosition UITextPosition UITextPosition UITextPosition UITextPosition UITextPosition UITextPosition UITextPosition UITextPosition UITextPosition UITextPosition UITextPosition UITextPosition UITextPosition UITextPosition UITextPosition UITextPosition UITextPosition UITextPosition UITextPosition UITextPosition

Debugging view hierarchy in Xcode I find that the UIFieldEditor is having a negative x value but I don't know why.

I just found that setting the value property can fix this issue, will dig in if I get more time.


Meteor.users.helpers not defined

Hi,
I'm trying to use this package to link to another document on the users collection.
Its throwing an error at the line 'Meteor.users.helpers({....});'
It does this on a freshly created meteor project version <IP_ADDRESS>
Any ideas what it could be?
Thanks

Hey @JimHough do you have more info on the exact error that you saw?

Same problem here. I installed latest windows preview release from https://github.com/meteor/meteor/wiki/Preview-of-Meteor-on-Windows
When I start the app - I get this error:
W20150306-15:24:50.920(5.5)? (STDERR)
W20150306-15:24:50.922(5.5)? (STDERR) C:\Users\username\AppData\Local\.meteor\packages\meteor-tool\1.1.0-winr.12\mt-os.windows.x86_32\dev_bundle\server-lib\node_modules\fibers\future.js:245
W20150306-15:24:50.922(5.5)? (STDERR)                                           throw(ex);
W20150306-15:24:50.922(5.5)? (STDERR)                                                 ^
W20150306-15:24:50.922(5.5)? (STDERR) TypeError: Object [object Object] has no method 'helpers'
W20150306-15:24:50.922(5.5)? (STDERR)     at app\collections\users.js:3:14
W20150306-15:24:50.922(5.5)? (STDERR)     at app\collections\users.js:10:3
W20150306-15:24:50.923(5.5)? (STDERR)     at D:\code\apps\dummyapp\dummyapp\.meteor\local\build\programs\server\boot.js:222:10
W20150306-15:24:50.923(5.5)? (STDERR)     at Array.forEach (native)
W20150306-15:24:50.923(5.5)? (STDERR)     at Function._.each._.forEach (C:\Users\username\AppData\Local\.meteor\packages\meteor-tool\1.1.0-winr.12\mt-os.windows.x86_32\dev_bundle\server-lib\node_modules\underscore\underscore.js:79:11)
W20150306-15:24:50.923(5.5)? (STDERR)     at D:\code\apps\dummyapp\dummyapp\.meteor\local\build\programs\server\boot.js:117:5
W20150306-15:24:56.408(5.5)? (STDERR)
W20150306-15:24:56.409(5.5)? (STDERR) C:\Users\username\AppData\Local\.meteor\packages\meteor-tool\1.1.0-winr.12\mt-os.windows.x86_32\dev_bundle\server-lib\node_modules\fibers\future.js:245
W20150306-15:24:56.409(5.5)? (STDERR)                                           throw(ex);
W20150306-15:24:56.409(5.5)? (STDERR)                                                 ^
W20150306-15:24:56.409(5.5)? (STDERR) TypeError: Object [object Object] has no method 'helpers'
W20150306-15:24:56.409(5.5)? (STDERR)     at app\collections\users.js:3:14
W20150306-15:24:56.409(5.5)? (STDERR)     at app\collections\users.js:10:3
W20150306-15:24:56.409(5.5)? (STDERR)     at D:\code\apps\dummyapp\dummyapp\.meteor\local\build\programs\server\boot.js:222:10
W20150306-15:24:56.410(5.5)? (STDERR)     at Array.forEach (native)
W20150306-15:24:56.410(5.5)? (STDERR)     at Function._.each._.forEach (C:\Users\username\AppData\Local\.meteor\packages\meteor-tool\1.1.0-winr.12\mt-os.windows.x86_32\dev_bundle\server-lib\node_modules\underscore\underscore.js:79:11)
W20150306-15:24:56.410(5.5)? (STDERR)     at D:\code\apps\dummyapp\dummyapp\.meteor\local\build\programs\server\boot.js:117:5
=> Exited with code: 8
W20150306-15:25:01.627(5.5)? (STDERR)
W20150306-15:25:01.627(5.5)? (STDERR) C:\Users\username\AppData\Local\.meteor\packages\meteor-tool\1.1.0-winr.12\mt-os.windows.x86_32\dev_bundle\server-lib\node_modules\fibers\future.js:245
W20150306-15:25:01.628(5.5)? (STDERR)                                                 ^
W20150306-15:25:01.628(5.5)? (STDERR) TypeError: Object [object Object] has no method 'helpers'
W20150306-15:25:01.628(5.5)? (STDERR)     at app\collections\users.js:3:14
W20150306-15:25:01.629(5.5)? (STDERR)     at app\collections\users.js:10:3
W20150306-15:25:01.629(5.5)? (STDERR)     at D:\code\apps\dummyapp\dummyapp\.meteor\local\build\programs\server\boot.js:222:10
W20150306-15:25:01.629(5.5)? (STDERR)     at Array.forEach (native)
W20150306-15:25:01.629(5.5)? (STDERR)     at Function._.each._.forEach (C:\Users\username\AppData\Local\.meteor\packages\meteor-tool\1.1.0-winr.12\mt-os.windows.x86_32\dev_bundle\server-lib\node_modules\underscore\underscore.js:79:11)
W20150306-15:25:01.628(5.5)? (STDERR)                                           throw(ex);
W20150306-15:25:01.630(5.5)? (STDERR)     at D:\code\apps\dummyapp\dummyapp\.meteor\local\build\programs\server\boot.js:117:5
=> Exited with code: 8
=> Your application is crashing. Waiting for file change.

I wrote the following code in users.js
Meteor.users.helpers({ some code here });

Does it work okay on other collections?

Actually, I am using the package on users collections only. Let me check.

Yes. the package is working on other collections. I think the problem is with users at the moment.

I have the same when updating meteor. Actually had the issue for quite a time. It could have to do with the need to go from Meteor.collection to Mongo.collection but this is just an educated guess.

I wonder if this current bug is windows specific, @JimHough are you running Meteor on windows by any chance?

@dburles using Ubuntu 14.04. Not working on any of my three systems (all Ubuntu 14.04)

I am also running on Ubuntu 14.04. It seems as though the helpers prototype is not applied or is deleted from the users collection.

@dburles Will you be able to work on this in the foreseeable future?

I made the warning go away by doing:
var Users = Meteor.users;

Users.helpers({});

This was quite helpful: https://github.com/ongoworks/meteor-security/issues/5
Can the others confirm that they are also using collection-behaviours?

@JimHough tried this project today on Ubuntu 14.04 and couldn't reproduce your error, are you sure that the issue isn't caused by a package conflict like @Levino ?

Having problems meself with this

@AlexFrazer any further info you may have, such as system etc would be super useful thanks!

@dburles, I am sorry, it seemed this was a result of pub/sub. My solution was to put it in a null publication.
Meteor.publish(null, function () {
  return Meteor.users.find({
    _id: this.userId
  }, {
    fields: {
      services: false
    }
  });
}, { is_auto: true });

I am getting another error related to Meteor.users.helpers.
While the exception is generated by your code, the error is not caused by your code.
I inherited this project and am getting the error
Error: [Can't apply helpers to 'users' a transform function already exists!]
when I try to implement Meteor.users.helpers for
kaoskeya:admin
kaoskeya:accounts-autoform-admin
Have searched my app and cannot locate a duplicate of Meteor.users.helpers.
Any suggestions as to how to fix this problem?
Thanks.

meteor list
255kb:meteor-status               1.2.0* Meteor Status automatically alerts u...
3stack:presence                   1.0.3  Track who' online on your app, and w...
accounts-base                     1.2.0  A user account system
accounts-facebook                 1.0.4  Login service for Facebook accounts
accounts-google                   1.0.4  Login service for Google accounts
accounts-password                 1.1.1  Password support for accounts
alanning:roles                    1.2.13  Role-based authorization
aldeed:autoform                   4.2.2* Easily create forms with automatic i...
aldeed:collection2                2.3.3  Automatic validation of insert and u...
aldeed:template-extension         3.4.3  Adds template features currently mis...
avatar                            0.7.6+ Consolidated user avatar template (t...
base-model                        0.1.3+ A model for all other models to inhe...
bootstrap-contextmenu             0.0.1+ A meter package wrapper to https://g...
canvas-roles                      0.0.0+
cfs:graphicsmagick                0.0.18  Adds gm to scope with the ability...
circular-menu                     0.0.0+
dburles:collection-helpers        1.0.3  Transform your collections with help...
dburles:jquery-tinysort           0.5.5  TinySort, a small and simple jQuery ...
dsyko:meteor-node-csv             0.3.7  node-csv npm packaged for meteor.
email                             1.0.6  Send email messages
fds:keycodes                      1.0.2  Names for event key codes
force-ssl                         1.0.4  Require this application to use HTTPS
fortawesome:fontawesome           4.3.0  Font Awesome (official): 500+ scalab...
hammer:hammer                     2.0.4_2  Hammer.js (official) - multi-touch...
html-parser                       0.0.0+
html2canvas                       0.0.0+
http                              1.1.0  Make HTTP calls to remote servers
intercom-local                    1.2.4+ Intercom.io basic integration (uses ...
iron:router                       1.0.7* Routing specifically designed for Me...
jag:pince                         0.0.6* A logger for Meteor inspired by log4...
jquery                            1.11.3_2  Manipulate the DOM using CSS sele...
jstree                            3.1.0+ Package wrapper for jstree plugin.
kaoskeya:accounts-autoform-admin  0.0.1  Manage users module for kaoskeya:adm...
kaoskeya:admin                    0.2.5  Simple autoform based admin that int...
matb33:collection-hooks           0.7.13  Extends Mongo.Collection with befor...
matdutour:outdated-browser        0.1.1  Detects outdated browsers and advise...
meteor-platform                   1.2.2  Include a standard set of Meteor pac...
meteorhacks:cluster               1.6.4* Clustering solution for Meteor with ...
meteorhacks:kadira                2.20.3* Performance Monitoring for Meteor
meteorhacks:kadira-profiler       1.1.0* CPU Profiler for Kadira
meteorhacks:npm                   1.3.0  Use npm modules with your Meteor App
meteorhacks:zones                 1.4.0* Zone.Js integration for meteor
mirageglobe:html5shivcdn          1.2.3  Latest version html5shiv loaded via ...
mizzao:jquery-ui                  1.11.2  Simple lightweight pull-in for jQue...
momentjs:moment                   2.10.3  Moment.js (official): parse, valida...
mrt:cron                          0.0.1  Simple meteor.cron
npm-container                     1.0.0+ Contains all your npm dependencies
ongoworks:security                1.1.0* Logical security for client-originat...
pascoual:pdfkit                   1.0.5  PDFKit, the PDF generation library
pauli:accounts-linkedin           1.1.2  Accounts service for LinkedIn accounts
pfafman:multer                    0.1.0  Add Multer to meteor
plotter                           0.0.0+
redactor                          10.0.5+ Redactor WYSIWYG html editor.
reywood:iron-router-ga            0.6.0  Google analytics (universal edition)...
s3-server                         0.0.0+ Upload files to S3. Allows use of Kn...
sacha:spin                        2.0.4* Simple spinner package for Meteor
selection-box                     0.0.0+
selectionchange-polyfill          0.0.0+
sergeyt:livequery                 1.3.7  Live Query utilizes the power of jQu...
server-time                       0.1.0+ Package to compensate for client/ser...
service-configuration             1.0.4  Manage the configuration for third-p...
socialize:messaging               0.2.1+ A social messaging package LOCAL
tmeasday:html5-history-api        4.1.2  HTML5 History API expansion for brow...
twbs:bootstrap                    3.3.4* The most popular front-end framework...
underscore                        1.0.3  Collection of small helpers: _.map, ...
user-model                        0.1.2+ A social user package
useraccounts:bootstrap            1.9.1* Accounts Templates styled for Twitte...
uzumaxy:prevent-backspace         0.0.1  Prevents the backspace from navigati...
voidale:better-emails             0.0.1+ Send Better Emails
voidale:bootstrap-alerts          0.3.5  Bootstrap styled alerts with halflings
voidale:bootstrap-dynamic-modal   0.0.1+ Simple shortcut of bootstrap 3 modals.
voidale:meteor-notification       0.0.2+ Adds notification functionality
voidale:minimap                   0.0.1+ Wrapper for mgMiniMap Jquery minimap...
voidale:nano-scroller             0.8.6+ A wrapper for nano Custom Scrollbar.
voidale:offcanvas-chat            0.0.1+ Simple off canvas chat
voidale:stripe-subscriptions      0.0.1+ Complete package for Stripe Subscipt...
webshot                           0.0.0+ Webshot - NPM Screenshot Library usi...
zimme:iron-router-active          1.0.4* Active route/path template helpers f...

@dougserfass it appears as though another package has added a transform function to the Meteor.users collection.

as you can see from meteor list, i inherited a lot of added packages in this project
i am working with kaoskeya to try to identify the package with the transform function
do you know of any easy way to determine which package contains the transform function?

I am getting the same error
W20150729-18:44:26.862(7)? (STDERR) /Users/username/.meteor/packages/meteor-tool/.1.1.3.hn2ytd++os.osx.x86_64+web.browser+web.cordova/mt-os.osx.x86_64/dev_bundle/server-lib/node_modules/fibers/future.js:245
W20150729-18:44:26.862(7)? (STDERR) 						throw(ex);
W20150729-18:44:26.862(7)? (STDERR) 						      ^
W20150729-18:44:26.867(7)? (STDERR) TypeError: Object [object Object] has no method 'helpers'
W20150729-18:44:26.867(7)? (STDERR)     at RouteController.extend.layoutTemplate (packages/me:post/collections/news.js:2:1)
W20150729-18:44:26.867(7)? (STDERR)     at /Users/username/Documents/lab/project1/myapp/meteor/.meteor/local/build/programs/server/packages/me_post.js:161:4
W20150729-18:44:26.867(7)? (STDERR)     at /Users/username/Documents/lab/project1/myapp/meteor/.meteor/local/build/programs/server/packages/me_post.js:222:3
W20150729-18:44:26.867(7)? (STDERR)     at /Users/username/Documents/lab/project1/myapp/meteor/.meteor/local/build/programs/server/boot.js:222:10
W20150729-18:44:26.867(7)? (STDERR)     at Array.forEach (native)
W20150729-18:44:26.867(7)? (STDERR)     at Function._.each._.forEach (/Users/username/.meteor/packages/meteor-tool/.1.1.3.hn2ytd++os.osx.x86_64+web.browser+web.cordova/mt-os.osx.x86_64/dev_bundle/server-lib/node_modules/underscore/underscore.js:79:11)
W20150729-18:44:26.867(7)? (STDERR)     at /Users/username/Documents/lab/project1/myapp/meteor/.meteor/local/build/programs/server/boot.js:117:5

Here is my usage, even if I comment out my implementation and leave only the News.helper I still get an error.
News = new Mongo.Collection('news');
News.helpers({
	imageUrl: function() {
		var image = Images.find(this.image_id);
		return image.url({store:'images'});
	},
	thumbnailUrl: function() {
		var image = Images.find(this.image_id);
		return image.url({store:'thumbs'});
	}
});

Here is my package list, I saw in another issue that collectionFS might not work with this package?
accounts-password           1.1.1  Password support for accounts
accounts-ui                 1.1.5  Simple templates to add login widgets to...
aldeed:autoform             5.3.2  Easily create forms with automatic inser...
aldeed:collection2          2.3.3  Automatic validation of insert and updat...
cfs:ejson-file              0.1.4  CollectionFS, FS.File as EJSON type
cfs:filesystem              0.1.2  Filesystem storage adapter for CollectionFS
cfs:graphicsmagick          0.0.18  Adds `gm` to scope with the ability to ...
cfs:standard-packages       0.5.9  Filesystem for Meteor, collectionFS
dburles:collection-helpers  0.1.6* Collection helpers
fortawesome:fontawesome     4.3.0  Font Awesome (official): 500+ scalable v...
iron:router                 1.0.9  Routing specifically designed for Meteor
jquery                      1.11.3_2  Manipulate the DOM using CSS selectors
less                        1.0.14  The dynamic stylesheet language
meteor-platform             1.2.2  Include a standard set of Meteor package...
sacha:spin                  2.3.1  Simple spinner package for Meteor
underscore                  1.0.3  Collection of small helpers: _.map, _.ea...
zcurtis:uikit               2.21.2  UIKit repackaged for Meteor (from fork)

Hey @harinsa somehow your project is using a very old version of collection-helpers.

@dburles whoops, sorry about that, did not specify the version and somehow meteor gave me that version. Change to @1.0.3 and it works now.

Hi, i am having the same problem :(

we can't attach helpers to Meteor.users

@crapthings Please elaborate, give some references. Thanks.

Meteor.users.helpers
organization: ->
	Organizations.findOne @organizationId

ypeError: Object [object Object] has no method 'helpers'
W20151005-23:47:17.861(8)? (STDERR)     at collections/users.coffee:1:14
W20151005-23:47:17.861(8)? (STDERR)     at collections/users.coffee:1:1
W20151005-23:47:17.862(8)? (STDERR)     at /Volumes/Data/dev/nexushubs/lawflow/mobile1/.meteor/local/build/programs/server/boot.js:242:10
W20151005-23:47:17.862(8)? (STDERR)     at Array.forEach (native)
W20151005-23:47:17.862(8)? (STDERR)     at Function..each..forEach (/Users/monsterstep/.meteor/packages/meteor-tool/.<IP_ADDRESS>mnd3gj++os.osx.x86_64+web.browser+web.cordova/mt-os.osx.x86_64/dev_bundle/server-lib/node_modules/underscore/underscore.js:79:11)
W20151005-23:47:17.862(8)? (STDERR)     at /Volumes/Data/dev/nexushubs/lawflow/mobile1/.meteor/local/build/programs/server/boot.js:137:5

@crapthings This is not how this works...

FYI - this happens when you add any package which applies a _transform on the Meteor.users collection... for example it can be replicated very easily by adding both this package and https://github.com/copleykj/socialize-user-model/
$ grep -r '_transform' .meteor/local/build/programs/server/packages/

Hey @zeroasterisk are you applying helpers to Meteor.users or Users?

Though from glancing at the source code, it's unlikely that it wouldn't conflict with collection-helpers

Well. I'm having the same kind of error, but not exactly with the same setup.
I can't use collection-helpers inside any packages. I have some local packages, one of them (app-core) declares a bunch of collections and helpers and export them for use by other local packages and the main app.
I'm getting `Object [object Object] has no method 'helpers'"
I tried api.use('dburles:collection-helpers') on the same package as the collections, on another package required by the app-core package and on the main app, but nothing works.
The only way to make it work is putting my collections.js in the main app and requiring dburles:collection-helpers from the main app too.

@VitorHP it should work alright, I'm pretty sure telescope does exactly that, maybe take a look at the source?

@dburles Alright! For some reason I was getting a very old version of the lib. When I specified the right version, it started to work again.
Probably my fault, since I started learning Meteor just these days.
Thank you very much for the help!

I am having a similar issue using the TAPi18n-db package. Is there any workaround?
Error thrown is from here:
https://github.com/dburles/meteor-collection-helpers/blob/master/collection-helpers.js#L5

@in0ni Unfortunately that package isn't compatible with collection-helpers, I see you have opened an issue over there, hopefully it can be resolved

if u have package that doesn't compatible with collection-helpers
try change the order of package

Meteor.users.helpers undefined

mikowals:batch-insert
dburles:collection-helpers

work

dburles:collection-helpers
mikowals:batch-insert

Hello! I'm having the same problem as this error
server/collections/sources.ts (21, 20): Property 'helpers' does not exist on type 'Collection<Source>'.
server/collections/users.ts (6, 14): Property 'helpers' does not exist on type 'Collection<User>'.

I have version 1.1.0 of collection-helpers
Meteor runs after that alerts, and in meteor shell I can access the helpers, but it is annoying to see that warnings/errors
Any way to add a property called helpers to that type?
Thanks!

Ok, I made it work.
I installed the meteor typings from this url https://github.com/meteor-typings/meteor
Then changed the meteor typing in typings/globals/meteor/index.d.ts
search for interface Collection<T> { and at the end of that interface add something like this
helpers(helpers: any): any;
Then everything worked without the warning.


Webhooks should have a type field to differentiate event types

As per discussion.
It would be convenient for webhook consumers to have a type field in the webhook payload, so that they can determine what type of webhook is being sent, e.g. whether it's "deployment created" or "promotion completed" or whatever.

@tommysullivan makes a good point in Gitter about using specific media types on the POST to represent the different kinds of data. I'm not opposed to that

And I'm not opposed to a type field but media types are a nice solution thanks for reposting @rtyler

U can always add annotations for media type too, let jaxrs route for u (if that's what u r using as a hook recipient)

I'm assigning this to myself to further define what we should actually do here

@rtyler I think we need this now, unless there's an alternative way just now for consumers (e.g. Jenkins plugins) to determine what type of event is being sent by inspecting the webhook payload?

@GiriDandu @maheshkelkar I propose the following for "typing" webhook payloads:
A Contentn-Type header on the payload request that would be: application/vnd.deploydb.<type>.v1+json, e.g.:

application/vnd.deploydb.deploymentcreated.v1+json
application/vnd.deploydb.deploymentcompleted.v1+json

Provided that is well documented, I do not think we need a type field in the JSON

+1!
Thank You,
Tom Sullivan

On Mar 24, 2015, at 8:51 AM, R. Tyler Croy<EMAIL_ADDRESS>wrote:
@GiriDandu @maheshkelkar I propose the following for "typing" webhook payloads:
A Contentn-Type header on the payload request that would be: application/vnd.deploydb..v1+json, e.g.:
application/vnd.deploydb.deploymentcreated.v1+json
application/vnd.deploydb.deploymentcompleted.v1+json
Provided that is well documented, I do not think we need a type field in the JSON
—
Reply to this email directly or view it on GitHub.


Rescan button over the wordlists manager

#291

The name is misleading, one might think it works the same way as the Configs if you call it Rescan. Please call it something like Delete Not Found (both the text on the button and the variable names).

@openbullet
Now it's fine i guess, Would you mind adding it? ^_^

Yes of course, I wasn't home

Next time please don't add those merges to the PR since it's not editing the same piece of code you don't need to do that ok?


Runtime: Using lamba version of AddService when adding MemoryStorage.

Fixes #5325

Closing this PR.  The issue will be resolved when we switch to AdaptiveDialogBot in another PR.


Clean up test fixtures

Current dependencies on/for this PR:

main

PR #361

PR #362

PR #363   👈

This comment was auto-generated by Graphite.


Uniform indentation with other functions.

Uniform indentation with other functions.

Since the line fits within 80 columns this formatting is not needed. Hence the formatter did not change it. I believe we only split lines for arguments if all the arguments can't fit within 80 columns.


ffmpeg: compile with harfbuzz

Description of changes
Enables building ffmpeg with harfbuzz, which is needed for the drawtext filter.
It already contains fontconfig, freetype, and fribidi. Harfbuzz is the last missing piece.
I've tested this with ffmpeg-headless, and it works fine.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

What's the closure size increase like?

How can we test this?

I want to test that the filter works.


regenerated test harness with updated xml e2e test result file name

Regenerated harness scaffolding to fix report file name

/lgtm
/approve

Codecov Report

Merging #199 (c1a82fb) into main (455b3c1) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #199   +/-   ##
=======================================
  Coverage   38.91%   38.91%
=======================================
  Files          31       31
  Lines        1971     1971
=======================================
  Hits          767      767
  Misses       1113     1113
  Partials       91       91


Delete vconcat/vsplit instructions

Feature
The vconcat and vsplit instructions are not currently implemented in any Cranelift backend. At a quick glance through the history, I'm not sure they've ever been implemented, except as a legalization pass that's since been deleted. Let's remove them.
Benefit
We won't have to maintain the instruction definitions or add new rules in the backends to implement these instructions.
Implementation
Remove the instruction definitions from cranelift/codegen/meta/src/shared/instructions.rs and the references to these opcodes from cranelift/interpreter/src/step.rs. I just merged #5462 which implemented vsplit in the interpreter and added a test for it, cranelift/filetests/filetests/runtests/simd-vsplit.clif, which would also need to be deleted. I think that's it!
This would supersede #5457.
Alternatives
We could clarify the semantics of these instructions and implement them in the backends and interpreter, but without a compelling use case it's not clear why we would.

Sounds good to me,  I did also have a quick ctrl+f for the instruction and saw it wasn't being used — but didn't know the reason. I'd be happy to pick this up!

The original purpose for the iconcat/isplit and vconcat/vsplit instructions was that it allows legalization to split full-width operations into two half-width operations without having to implement the full-width operations in every backend. So for example a 256bit vector add could be emulated using two 128bit vector adds. After legalization all concat and split instructions would be gone.

Ah, so would they still be needed then for that use case? Also - what exactly is a "legalization" pass? Is that just a pass that lowers the IR to backend instructions?  For example until recently we had a heap_addr instruction which lowered to a series of clif ir instructions that does bound checking. Most of the work that legalization did has been subsumed by the move to the new backend framework 2 years ago. I believe the plan is to eventually remove it entirely at some point.

Thanks for the context, that is pretty fascinating. Sorry for more questions, what was different about the new backend framework that made a separate legalisation pass unnecessary?

The old backend framework used clif ir all the way to the end. So high level instructions that are performed using multiple machine instructions needed to be lowered to simpler clif ir instructions that could then be encoded using a single machine instruction each. The new backend framework however lowers from clif ir to a backend specific ir. This lowering can split high level instructions as necessary for the specific architecture, avoiding the need to modify the clif ir during compilation (except for optional optimization passes and the last remaining bits of the legalizer). For example https://github.com/bytecodealliance/wasmtime/blob/93ae9078c5a2588b5241bd7221ace459d2b04d54/cranelift/codegen/src/isa/x64/lower.isle#L92-L103 lowers an iadd.i128 as add + adc.


All images are not displayed in a message tile.

For messages with multiple images, all image thumbnails are not shown in the inline preview. Only the first image is shown in the preview.
Current behaviour

Expected behaviour

This might be a backend thing -- I think we may only render previews for the first image if there are more than N images linked to in a message...

Just read the code; I guess the backend logic limits us to showing <5 images; so it's probably not that?  Maybe worth testing on web though just to be sure.

Just tested on the web, <5 images are being shown. But in the app, only the first img tag src is being extracted.

OK, makes sense, yeah we should fix that.


pull request for e2e test at 12/27/2016 03:26:10

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


fix: change model names to include Huggingface organization

Fixes #362
What does this PR do?
Change model names
Screenshot / video of UI

What issues does this PR fix or reference?
#362
How to test this PR?
Go to the models page

It's happening something weird to me. I went to the models page of a recipe and it's empty and also the right sidebar is different like this is based on an old version but it seems to correctly rebased.


Logo Proposal for Nebulator

Hi, I'm a graphic designer and I like to collaborate with open source projects. Do you know that the graphic image of a project is very important? thinking about it I would like to design a logo for your Project Nebulator.
I will be pleased to collaborate with you.

Interesting. I'll think about it.

I can present you a proposal, do you have something in mind that you would like to use in the design of the logo?

Well if you want to have fun with it, go ahead.
Nebulator is named for a similar application a friend made called Nebula.
It refers to the space object. But nebulator would be a thing that makes
nebulas maybe?
Try google "nebula icon".
Chris
On Wed, Jul 11, 2018 at 6:43 AM, area55<EMAIL_ADDRESS>wrote:

I can present you a proposal, do you have something in mind that you would
like to use in the design of the logo?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/cepthomas/Nebulator/issues/1#issuecomment-404139914,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIiE2Atc93g3vb_cfLpwBzlYEUob-QWOks5uFeTIgaJpZM4VIIN7
.


Fix #17766: Avoid caching on response.

fix  #17766
This PR is a part of the audit proposed in #17766
The PR adds add_never_cache_headers on @require_server_admin decorator to avoid caching on response.
we assign the add_logging_data to the response and further pass it as a paramater to the add_never_cache_headers and then we finally return the response.

@timabbott @alexmv  need your review

Closing as this does not explain why it's correct, which is the main work of this audit.


pyopae: add pycontext to setup.py

Installation from package will fail without compiling this as this file
implements functions needed for context management.

Is compilation needed for installation of the package?

Short answer, yes.
The long anser:
Our cmake build system builds the extension module using cmake only. This is done to allow building in different build types (Debug, Coverage, Release, ...).
The cmake build system includes another step that processes the setup.py file to create both:

A source distribution (as a tar.gz file)
A universal wheel package which includes the compiled extension module.

When I was testing the binary distribution I noticed it was missing some symbols (the ones provided by that pycontext file).


Backport "HBASE-28605 Add ErrorProne ban on Hadoop shaded thirdparty jars (#5918)" to branch-3

This change results in this error on master at 3a3dd66e21.
[WARNING] Rule 2: de.skuzzle.enforcer.restrictimports.rule.RestrictImports failed with message:

Banned imports detected:
Reason: Use shaded version in hbase-thirdparty
        in file: org/apache/hadoop/hbase/backup/TestBackupDelete.java
                org.apache.hadoop.thirdparty.com.google.common.collect.Sets (Line: 34, Matched by: org.apache.hadoop.thirdparty.**)

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 38s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ branch-3 Compile Tests _

+1 :green_heart:
mvninstall
4m 33s
branch-3 passed

+1 :green_heart:
compile
8m 44s
branch-3 passed

+1 :green_heart:
spotless
1m 13s
branch has no errors when running spotless:check.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 38s
the patch passed

+1 :green_heart:
compile
7m 11s
the patch passed

+1 :green_heart:
javac
7m 11s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
xml
0m  2s
The patch has no ill-formed XML file.

+1 :green_heart:
hadoopcheck
8m  2s
Patch does not cause any errors with Hadoop 3.3.6.

+1 :green_heart:
spotless
1m  9s
patch has no errors when running spotless:check.

_ Other Tests _

+1 :green_heart:
asflicense
0m 15s
The patch does not generate ASF License warnings.

45m 51s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5932/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5932

Optional Tests
dupname asflicense javac hadoopcheck spotless xml compile

uname
Linux d712a73af15c 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-3 / cdb79ea5d8

Default Java
Eclipse Adoptium-11.0.17+8

Max. process+thread count
81 (vs. ulimit of 30000)

modules
C: . U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5932/1/console

versions
git=2.34.1 maven=3.8.6

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 27s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-3 Compile Tests _

+1 :green_heart:
mvninstall
3m 15s
branch-3 passed

+1 :green_heart:
compile
2m  1s
branch-3 passed

+1 :green_heart:
shadedjars
5m 59s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m 51s
branch-3 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 10s
the patch passed

+1 :green_heart:
compile
2m 24s
the patch passed

+1 :green_heart:
javac
2m 24s
the patch passed

+1 :green_heart:
shadedjars
6m 53s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
2m  5s
the patch passed

_ Other Tests _

-1 :x:
unit
286m 51s
root in the patch failed.

320m 11s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5932/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5932

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux c40995ef66af 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-3 / cdb79ea5d8

Default Java
Eclipse Adoptium-11.0.17+8

unit
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5932/1/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-root.txt

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5932/1/testReport/

Max. process+thread count
5901 (vs. ulimit of 30000)

modules
C: . U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5932/1/console

versions
git=2.34.1 maven=3.8.6

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 35s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-3 Compile Tests _

+1 :green_heart:
mvninstall
3m 20s
branch-3 passed

+1 :green_heart:
compile
2m 28s
branch-3 passed

+1 :green_heart:
shadedjars
7m 59s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
2m 27s
branch-3 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 40s
the patch passed

+1 :green_heart:
compile
2m 12s
the patch passed

+1 :green_heart:
javac
2m 12s
the patch passed

+1 :green_heart:
shadedjars
7m  3s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
2m  4s
the patch passed

_ Other Tests _

-1 :x:
unit
434m 34s
root in the patch failed.

472m  9s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5932/1/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5932

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 54096a2515e8 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-3 / cdb79ea5d8

Default Java
Temurin-1.8.0_352-b08

unit
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5932/1/artifact/yetus-jdk8-hadoop3-check/output/patch-unit-root.txt

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5932/1/testReport/

Max. process+thread count
5479 (vs. ulimit of 30000)

modules
C: . U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5932/1/console

versions
git=2.34.1 maven=3.8.6

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


Fix wrong step on backspace for ambiguous content

Closes ProseMirror/prosemirror#726

Looks great! Can you add a test? (I think what would work well is to add it to test/test-domchange.js and have a test that does two deletions like this, one with lastKeyCode set to 8, and one with it set to something else.)
Also, it may be worth resetting lastKeyCode after each change, or keeping a time counter with it and ignoring it if it's too old, to avoid spurious effects when a non-key-related change happens after a backspace press.

Yes, I can. I actually tried to add a test to test/test-domchange.js but didn't because the output is same. I didn't think about setting lastKeyCode manually though. Thanks for the suggestion.
I was also worried about unwanted states. So, I manually tried Shit + Backspace, Ctrl + Backspace, etc. Keeping a time counter is a great idea, it didn't occur to me.

I tried to incorporate your suggested changes. Let me know if you have any opinion about them 😃

Thanks for following up! I've merged your work as eb194674f5095f and followed up with 0398b2b4076330de to make it work for ambiguous ctrl-backspace. The changes have been released as prosemirror-view 1.0.3

Thanks for the release. Don't know why didn't I think about dispatchTransaction 🤦♂


Incorrect description for startTime and endTime

The documentation states that startTime and endTime:

are optional and if not provided the last 24 hours time range will be used
both must be specified or both omitted

Currently what I observe is:

startTime or endTime must be provided otherwise the response is a 400 HTTP error with following body:

{
    "error": {
        "code": "String reference not set to an instance of a String.\r\nParameter name",
        "message": "s"
    }
}

The API can be called with only startTime or endTime specified until they respect the 7 days constraint

@vergnes If still applicable, please create a PR with your proposed changes. Or open a new issue with the link to the topic with the incorrect description. Thanks, linda


aluminium wire

when i attach an aluminium wire  to the big reactor it crashes and when i go back on the world it crashes immediately

Mind posting a crash report?

---- Minecraft Crash Report ----
// Don't be sad, have a hug! <3
Time: 3/11/15 5:24 PM
Description: Exception in server tick loop
java.lang.NoClassDefFoundError: buildcraft/api/power/IPowerReceptor
at micdoodle8.mods.galacticraft.core.energy.tile.TileBaseUniversalConductor.initBC(TileBaseUniversalConductor.java:45)
at micdoodle8.mods.galacticraft.core.energy.tile.TileBaseUniversalConductor.(TileBaseUniversalConductor.java:33)
at micdoodle8.mods.galacticraft.core.tile.TileEntityAluminumWire.(TileEntityAluminumWire.java:16)
at micdoodle8.mods.galacticraft.core.blocks.BlockAluminumWire.func_149915_a(BlockAluminumWire.java:104)
at net.minecraft.block.Block.createTileEntity(Block.java:1446)
at net.minecraft.world.chunk.Chunk.func_150806_e(Chunk.java:850)
at net.minecraft.world.World.func_147438_o(World.java:2540)
at erogenousbeef.core.multiblock.MultiblockTileEntityBase.getNeighboringParts(MultiblockTileEntityBase.java:317)
at erogenousbeef.core.multiblock.MultiblockTileEntityBase.attachToNeighbors(MultiblockTileEntityBase.java:45)
at erogenousbeef.core.multiblock.MultiblockWorldRegistry.tickEnd(MultiblockWorldRegistry.java:130)
at erogenousbeef.core.multiblock.MultiblockRegistry.tickEnd(MultiblockRegistry.java:36)
at erogenousbeef.core.multiblock.MultiblockServerTickHandler.onWorldTick(MultiblockServerTickHandler.java:22)
at cpw.mods.fml.common.eventhandler.ASMEventHandler_134_MultiblockServerTickHandler_onWorldTick_WorldTickEvent.invoke(.dynamic)
at cpw.mods.fml.common.eventhandler.ASMEventHandler.invoke(ASMEventHandler.java:54)
at cpw.mods.fml.common.eventhandler.EventBus.post(EventBus.java:138)
at cpw.mods.fml.common.FMLCommonHandler.onPostWorldTick(FMLCommonHandler.java:255)
at net.minecraft.server.MinecraftServer.func_71190_q(MinecraftServer.java:645)
at net.minecraft.server.MinecraftServer.func_71217_p(MinecraftServer.java:547)
at net.minecraft.server.integrated.IntegratedServer.func_71217_p(IntegratedServer.java:111)
at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:427)
at net.minecraft.server.MinecraftServer$2.run(MinecraftServer.java:685)
Caused by: java.lang.ClassNotFoundException: buildcraft.api.power.IPowerReceptor
at net.minecraft.launchwrapper.LaunchClassLoader.findClass(LaunchClassLoader.java:99)
at java.lang.ClassLoader.loadClass(ClassLoader.java:306)
at java.lang.ClassLoader.loadClass(ClassLoader.java:247)
... 21 more
A detailed walkthrough of the error, its code path and all known details is as follows:
-- System Details --
Details:
Minecraft Version: 1.7.10
Operating System: Mac OS X (x86_64) version 10.10.1
Java Version: 1.6.0_65, Apple Inc.
Java VM Version: Java HotSpot(TM) 64-Bit Server VM (mixed mode), Apple Inc.
Memory: 114372008 bytes (109 MB) / 406102016 bytes (387 MB) up to<PHONE_NUMBER> bytes (1011 MB)
JVM Flags: 5 total; -Xmx1G -XX:+UseConcMarkSweepGC -XX:+CMSIncrementalMode -XX:-UseAdaptiveSizePolicy -Xmn128M
AABB Pool Size: 0 (0 bytes; 0 MB) allocated, 0 (0 bytes; 0 MB) used
IntCache: cache: 0, tcache: 0, allocated: 0, tallocated: 0
FML: MCP v9.05 FML v<IP_ADDRESS>1 Minecraft Forge <IP_ADDRESS>1 28 mods loaded, 28 mods active
mcp{9.05} [Minecraft Coder Pack] (minecraft.jar) Unloaded->Constructed->Pre-initialized->Initialized->Post-initialized->Available->Available->Available->Available
FML{<IP_ADDRESS>1} [Forge Mod Loader] (forge-1.7.10-<IP_ADDRESS>1.jar) Unloaded->Constructed->Pre-initialized->Initialized->Post-initialized->Available->Available->Available->Available
Forge{<IP_ADDRESS>1} [Minecraft Forge] (forge-1.7.10-<IP_ADDRESS>1.jar) Unloaded->Constructed->Pre-initialized->Initialized->Post-initialized->Available->Available->Available->Available
appliedenergistics2-core{rv1-stable-1} [AppliedEnergistics2 Core] (minecraft.jar) Unloaded->Constructed->Pre-initialized->Initialized->Post-initialized->Available->Available->Available->Available
CodeChickenCore{<IP_ADDRESS>} [CodeChicken Core] (minecraft.jar) Unloaded->Constructed->Pre-initialized->Initialized->Post-initialized->Available->Available->Available->Available
{000} [CoFH ASM Data Initialization] (minecraft.jar) Unloaded->Constructed->Pre-initialized->Initialized->Post-initialized->Available->Available->Available->Available
Micdoodlecore{} [Micdoodle8 Core] (minecraft.jar) Unloaded->Constructed->Pre-initialized->Initialized->Post-initialized->Available->Available->Available->Available
NotEnoughItems{<IP_ADDRESS>} [Not Enough Items] (NotEnoughItems-1.7.10-<IP_ADDRESS>-universal.jar) Unloaded->Constructed->Pre-initialized->Initialized->Post-initialized->Available->Available->Available->Available
VeinMiner_coremod{0.21.0_build-35} [Core mod] (minecraft.jar) Unloaded->Constructed->Pre-initialized->Initialized->Post-initialized->Available->Available->Available->Available
appliedenergistics2{rv1-stable-1} [Applied Energistics 2] (appliedenergistics2-rv1-stable-1.jar) Unloaded->Constructed->Pre-initialized->Initialized->Post-initialized->Available->Available->Available->Available
CoFHCore{1.7.10R3.0.0RC5} [CoFH Core] (CoFHCore-[1.7.10]3.0.0RC5-207.jar) Unloaded->Constructed->Pre-initialized->Initialized->Post-initialized->Available->Available->Available->Available
BigReactors{0.4.2A2} [Big Reactors] (BigReactors-0.4.2A2.jar) Unloaded->Constructed->Pre-initialized->Initialized->Post-initialized->Available->Available->Available->Available
BuildCraft|Core{6.4.2} [BuildCraft] (buildcraft-6.4.2.jar) Unloaded->Constructed->Pre-initialized->Initialized->Post-initialized->Available->Available->Available->Available
BuildCraft|Transport{6.4.2} [BC Transport] (buildcraft-6.4.2.jar) Unloaded->Constructed->Pre-initialized->Initialized->Post-initialized->Available->Available->Available->Available
BuildCraft|Silicon{6.4.2} [BC Silicon] (buildcraft-6.4.2.jar) Unloaded->Constructed->Pre-initialized->Initialized->Post-initialized->Available->Available->Available->Available
BuildCraft|Builders{6.4.2} [BC Builders] (buildcraft-6.4.2.jar) Unloaded->Constructed->Pre-initialized->Initialized->Post-initialized->Available->Available->Available->Available
BuildCraft|Energy{6.4.2} [BC Energy] (buildcraft-6.4.2.jar) Unloaded->Constructed->Pre-initialized->Initialized->Post-initialized->Available->Available->Available->Available
BuildCraft|Factory{6.4.2} [BC Factory] (buildcraft-6.4.2.jar) Unloaded->Constructed->Pre-initialized->Initialized->Post-initialized->Available->Available->Available->Available
DamageIndicatorsMod{3.2.0} [Damage Indicators] (Damage-Indicators-Mod-1.7.10.jar) Unloaded->Constructed->Pre-initialized->Initialized->Post-initialized->Available->Available->Available->Available
exnihilo{1.37} [Ex Nihilo] (Ex-Nihilo-1.37.jar) Unloaded->Constructed->Pre-initialized->Initialized->Post-initialized->Available->Available->Available->Available
ExtraUtilities{1.2.1} [Extra Utilities] (extrautilities-1.2.1-2.jar) Unloaded->Constructed->Pre-initialized->Initialized->Post-initialized->Available->Available->Available->Available
GalacticraftCore{3.0.6} [Galacticraft Core] (GalacticraftCore-1.7-<IP_ADDRESS>.jar) Unloaded->Constructed->Pre-initialized->Initialized->Post-initialized->Available->Available->Available->Available
GalacticraftMars{3.0.6} [Galacticraft Planets] (Galacticraft-Planets-1.7-<IP_ADDRESS>.jar) Unloaded->Constructed->Pre-initialized->Initialized->Post-initialized->Available->Available->Available->Available
Mantle{1.7.10-0.3.1.jenkins180} [Mantle] (Mantle-mc1.7.10-0.3.1.jar) Unloaded->Constructed->Pre-initialized->Initialized->Post-initialized->Available->Available->Available->Available
TConstruct{1.7.10-1.6.0.build611} [Tinkers' Construct] (TConstruct_mc1.7.10_1.6.0d40.1.jar) Unloaded->Constructed->Pre-initialized->Initialized->Post-initialized->Available->Available->Available->Available
VeinMiner{0.21.0} [Vein Miner] (VeinMiner-1.7.10_0.21.0.jar) Unloaded->Constructed->Pre-initialized->Initialized->Post-initialized->Available->Available->Available->Available
VeinMinerModSupport{0.21.0} [Mod Support] (VeinMiner-1.7.10_0.21.0.jar) Unloaded->Constructed->Pre-initialized->Initialized->Post-initialized->Available->Available->Available->Available
wearmc{1.7.10g} [WearMC] (WearMC-Forge-1.7.10g.jar) Unloaded->Constructed->Pre-initialized->Initialized->Post-initialized->Available->Available->Available->Available
AE2 Version: stable rv1-stable-1 for Forge <IP_ADDRESS>7
CoFHCore: -[1.7.10]3.0.0RC5-207
Mantle Environment: Environment healthy.
TConstruct Environment: Environment healthy.
AE2 Integration: IC2:OFF, RotaryCraft:OFF, RC:OFF, BC:ON, MJ6:OFF, MJ5:OFF, RF:ON, RFItem:ON, MFR:OFF, DSU:OFF, FZ:OFF, FMP:OFF, RB:OFF, CLApi:OFF, Waila:OFF, InvTweaks:OFF, NEI:ON, CraftGuide:OFF, Mekanism:OFF, ImmibisMicroblocks:OFF, BetterStorage:OFF
Profiler Position: N/A (disabled)
Vec3 Pool Size: 0 (0 bytes; 0 MB) allocated, 0 (0 bytes; 0 MB) used
Player Count: 0 / 8; []
Type: Integrated Server (map_client.txt)
Is Modded: Definitely; Client brand changed to 'fml,forge'

Use pastebin for crash reports.

Someone is using an old Buildcraft API! naughty!

This is a Galacticraft bug. BR is just causing the Galacticraft TE to be loaded.

so do i need to download a different version of gc?

go report it to them...
http://forum.micdoodle8.com/index.php?forums/bug-reports.15/


Auto Tag / Push DockerHub & GHCR

For some reason I wasn't able to directly push to your fork before merging.

I have no idea why you weren't able to push to my fork... Might be because I did not base my PR on a branch but on master instead ?
Anyways, thanks a bunch !


[5.6] Add callback support to "optional"

Lets you pipe the value into an outside function, with the guarantee that you're not passing it null:
return optional($modelOrNull, function ($definitelyTheModel) {
    return doSomethingWithTheModel($definitelyTheModel);
});

how is this better than a ternary or an if statement?
I love the optional helper, but I think it's getting stretched a bit far lately.

A ternary is insufficient when the value comes from a function/expression.

Ah ok, so this would be for cases where you might have a result of one function that you want to pass as an argument to another function. I don't think that's a common pattern for me, but that may be because I haven't been looking for it. It would probably be helpful to update the example since the current one would be more succinct as a ternary.
There was another PR recently where a contributor wanted to add similar functionality to tap and I thought creating a when helper might be a better idea. Maybe that would be the same case here?
Personally, I like keeping optional as nothing more than a PHP version of Ruby's & operator, so I have a bit of a bias.

transform() used internally by API resources can do this: https://laravel.com/docs/5.6/helpers#method-transform.
It will call the Closure when the value passes filled().
https://github.com/laravel/framework/blob/5.6/src/Illuminate/Support/helpers.php#L1121-L1142

I find it odd that optional() may now not return Illuminate\Support\Optional.
optional($user, function ($user) {
    return $user->sendResetPassword();
})->getInvalidProperty->dump;

It now would throws exception.

I love the optional helper, but I think it's getting stretched a bit far lately.

Do a PR to remove the Optional helper. I'd +1 it. tap tap tap tap dat all day 😄

@derekmd the transform helper checks for filled, as you've noted. Not null.
@crynobone I don't see that confusing at all. These are two different uses of the optional helper. Think of it as an overloaded method with a different return type.


Error: The following properties were null: name

Description
Yesterday i have installed Rocket.Chat on my Ubuntu Server.
Everything works fine but today the Android App got this Error: The following properties were null: name
and does not show anymore Rooms or Chats.
Devices and Versions

Your Rocket.Chat.Android version: (2.4.0)
Your Rocket.Chat Server version: (0.66.2)

Huawei P9 Lite with Android 7.0
Samsung Galaxy S5 with Android 6.0.1
Steps to reproduce
When is start the Android App i got this Error: The following properties were null: name
Only during search button i found the members of the chat but no channel anymore
Logs
I20180707-02:48:15.316(2) API ➔ debug GET: /api/v1/settings.public?count=0&fields=%7B%22type%22%3A%201%7D&query=%7B%22_id%22%3A%20%7B%22%24in%22%3A%20%5B%22LDAP_Enable%22%2C%22CAS_enabled%22%2C%22CAS_login_url%22%2C%22Accounts_RegistrationForm%22%2C%22Accounts_ShowFormLogin%22%2C%22Accounts_PasswordReset%22%2C%22Accounts_CustomFields%22%2C%22Accounts_OAuth_Google%22%2C%22Accounts_OAuth_Facebook%22%2C%22Accounts_OAuth_Github%22%2C%22Accounts_OAuth_Linkedin%22%2C%22Accounts_OAuth_Meteor%22%2C%22Accounts_OAuth_Twitter%22%2C%22Accounts_OAuth_Wordpress%22%2C%22Accounts_OAuth_Gitlab%22%2C%22API_Gitlab_URL%22%2C%22Site_Url%22%2C%22Site_Name%22%2C%22Assets_favicon_512%22%2C%22Assets_favicon_192%22%2C%22UI_Use_Real_Name%22%2C%22UI_Allow_room_names_with_special_chars%22%2C%22Favorite_Rooms%22%2C%22FileUpload_Storage_Type%22%2C%22FileUpload_MaxFileSize%22%2C%22FileUpload_MediaTypeWhiteList%22%2C%22Message_HideType_uj%22%2C%22Message_HideType_ul%22%2C%22Message_HideType_au%22%2C%22Message_HideType_mute_unmute%22%2C%22Message_HideType_ru%22%2C%22Message_AllowDeleting%22%2C%22Message_AllowEditing%22%2C%22Message_AllowPinning%22%2C%22Message_AllowStarring%22%2C%22Message_ShowDeletedStatus%22%2C%22Message_ShowEditedStatus%22%2C%22Assets_tile_310_wide%22%2C%22Store_Last_Message%22%2C%22Message_Read_Receipt_Enabled%22%2C%22Message_Read_Receipt_Store_Users%22%5D%7D%7D
I20180707-02:48:15.319(2) API ➔ debug Success { statusCode: 200,   body:     { settings:        [ [Object],         [Object],         [Object],         [Object],         [Object],         [Object],         [Object],         [Object],         [Object],         [Object],         [Object],         [Object],         [Object],         [Object],         [Object],         [Object],         [Object],         [Object],         [Object],         [Object],         [Object],         [Object],         [Object],         [Object],         [Object],         [Object],         [Object],         [Object],         [Object],         [Object],         [Object],         [Object],         [Object],         [Object],         [Object],         [Object],         [Object],         [Object],         [Object],         [Object],         [Object] ],      count: 41,      offset: 0,      total: 41,      success: true } }
I20180707-02:48:15.329(2) API ➔ debug GET: /api/v1/rooms.get?updatedAt=1970-01-01T00%3A00%3A00.000Z
I20180707-02:48:15.330(2) Meteor ➔ method rooms/get -> userId: 5bn5RLtWRzd2H6i5H , arguments:  { '0': undefined }
I20180707-02:48:15.334(2) API ➔ debug Success { statusCode: 200,   body:     { update: [ [Object], [Object], [Object], [Object], [Object], [Object] ],      remove: [],      success: true } }
I20180707-02:48:15.339(2) API ➔ debug GET: /api/v1/subscriptions.get?updatedAt=1970-01-01T00%3A00%3A00.000Z
I20180707-02:48:15.340(2) Meteor ➔ method subscriptions/get -> userId: 5bn5RLtWRzd2H6i5H , arguments:  { '0': undefined }
I20180707-02:48:15.341(2) API ➔ debug Success { statusCode: 200,   body:     { update: [ [Object], [Object], [Object], [Object], [Object], [Object] ],      remove: [],      success: true } }

Same configuration and same problem.

Help dame thing

@Michelle011997 @exceed73 Can you please paste the JSON response from the rooms.get and subscriptions.get API? There are probably some Subscription or Room object without a property name and that should never happen.

Fixed since 2.5.0 Build 2034 (update this morning).
Thanks @rafaelks for your interest.

That's right @rafaelks, and some of them have no name since... a version I don't remember.
Yesterday, Rocket.chat 0.66.3 arrived via snap package on my server. + Client 2.5.0 : Everything is fine now.

Yes the problem is solved with the Update v2.5.0

Great! Closing the issue in this case. Thank you guys!


JSON format MQTT messages

I'm using Home Assistant, and it would be nice to view the status of my tesla on my home assistant.
I can use MQTT to share the data with Home Assistant, however the data need to be in a JSON format. Would it be possible to also have a topic with all info in a JSON format?

You don't need JSON for that :) Take a look at this part of my Home Assistant configuration:
https://github.com/adriankumpf/HomeAssistant-Config/blob/master/config/includes/sensors/teslamate.yaml

Oh Wow, thanks for that! Will do a straight copy paste!
I wasn’t aware that you can just take the value, the home assistant docs clearly say it must be in JSON format. Learned something today!


Obsoletion request: GO:0044845    chain elongation of O-linked mannose residue

Please provide as much information as you can:

GO term ID and Label

GO:0044845    chain elongation of O-linked mannose residue

Reason for deprecation Put an x in the appropriate box:

[x] The reason for obsoletion is that this term represents a molecular function.

"Replace by" term (ID and label)
If all annotations can safely be moved to that term

"Consider" term(s) (ID and label)
Suggestions for reannotation

Are there annotations to this term?

How many EXP:

2 PomBase EXP, removing today

Are there mappings and cross references to this term? (InterPro, Keywords; check QuickGO cross-references section)

Is this term in a subset? (check the AmiGO page for that term)

Any other information

Checklist for ontology editor
Check term usage and metadata in Protégé

[ ] check term usage in the ontology
[ ] check internal mappings: RHEA, EC, MetaCyc
[ ] check subset usage
[ ] check taxon constraints

Check annotations

[ ] create Annotation Review spreadsheet

Annotation Review template: https://docs.google.com/spreadsheets/d/1izV3q1jnq_Ybg5MJ53O_b5BCTkGtjl_Fi-cOg3-7eBE/edit#gid=0

Instructions: https://wiki.geneontology.org/Annotation_Review_Request

[ ] create annotation review ticket: https://github.com/geneontology/go-annotation/issues/new?assignees=&labels=annotation+review&template=annotation-review-for-obsoletion.md&title=Review+annotations+to+[INSERT+GO+TERM]
[ ] find external mappings (via IEAS), include in obsoletion notice
[ ] tag curators in Annotation Review ticket. Responsible annotator for each group: https://github.com/geneontology/go-site/blob/master/metadata/group-contacts.csv

Notification

[ ] create obsoletion announcement
[ ] announce to GO friends<EMAIL_ADDRESS>[ ] paste the text in the ontology ticket

There are other siblings:
GO:0035269    protein O-linked mannosylation | is_a
GO:0018406    protein C-linked glycosylation via 2'-alpha-mannosyl-L-tryptophan
I think these should also be obsoleted?
Also, can you suggest which MF the terms correspond to?
Thanks!
Pascale

I   don't think there is a direct mapping  for these, they are only describing the type of linakge.
For GO:0035269    protein O-linked mannosylation
they can be GPI anchor biosynthetic process (pombe dpm1,2,3,4 )
or  ogm2,4 omh3
call wall mannoprotein biosynthetic process
and I think some could be
GO:0006488 | dolichol-linked oligosaccharide biosynthetic process
although I only have these with MF mannosyltransferases

I realise you asked about MF. These are multiple.
dolichyl-phosphate-mannose-protein mannosyltransferase activity
dolichyl-phosphate beta-D-mannosyltransferase activity
alpha-1,2-mannosyltransferase activity
enzyme regulator activity

@pgaudet  should I still go ahead with these obsoletions. I can't suggest precise  replacements, but I can suggest that
MF & BP are reviewed.

@pgaudet  let me know if I can go ahead with this one.

Hi Val,
How do you plan for the glycosylation branch to be re-organized? I have a hard time making comments if we obsolete some terms, in isolation from the rest of the branch.
Do you want to merge all into GO:0006486 protein glycosylation ?
We could discuss this on Monday's ontology call.
Thanks, Pascale

Yes I now think that most of the branch can go. I have been holding off because I am not sure about the
N-linked, O-linked etc.
So far I have only done the  N-linked GO:0006488   dolichol-linked oligosaccharide biosynthetic process
so I noted any terms that were clearly MF terms during that work
Lets discuss this one on Monday

It would be clearer and possibly more efficient if you did a large tickets with all the terms that should be obsoleted, so we have a better idea of the work involved.
Thanks, Pascale

Nice review: https://academic.oup.com/glycob/article/19/8/816/1988198

See also https://pubmed.ncbi.nlm.nih.gov/37565810/

For GO:0035269 protein O-linked mannosylation, MetaCyc has these multi-step pathways, each involving several different gene products:
protein O-mannosylation I (yeast) PWY-7921
protein O-mannosylation II (mammals, core M1 and core M2) (PWY-7922)
protein O-mannosylation III (mammals, core M3) (PWY-7979)
So we may not want to obsolete GO:0035269, unless that GO term is defined/used in a much more restricted way than the MetaCyc pathways.

note that I'm not proposing obsoletion of "GO:0035269
protein O-linked mannosylation"
but I was unsure about the reason for the "O:0044845 chain elongation of O-linked mannose residue" term (or why they aren't related),
probably one term would cover this pathway?

I don't know enough yet to be sure, so any input is appreciated.

@pgaudet was asking above about obsoleting  "GO:0035269 protein O-linked mannosylation", so I wanted to point out those MetaCyc pathways.
But looking more closely at those 3 pathways:
protein O-mannosylation I (yeast) PWY-7921:
This includes:

Man-GDP -> Man-dolichyl-P (single-step = EC:<IP_ADDRESS>/GO:0004582 MF)
transfer of Man from Man-dolichyl-P to O-linked Ser/Thr on protein (single step = EC:<IP_ADDRESS>/GO:0004169 MF)
transfer of additional Man (elongation) from Man-GDP to initial Man on the protein (multi-step = GO:0044845 BP)

protein O-mannosylation II (mammals, core M1 and core M2) (PWY-7922):
This includes:

Man-GDP -> Man-dolichyl-P (single-step = EC:<IP_ADDRESS>/GO:0004582 MF)
transfer of Man from Man-dolichyl-P to O-linked Ser/Thr on protein (single step = EC:<IP_ADDRESS>/GO:0004169 MF)
transfer of 2x GlcNAc (ie. not Man) by two different enzymes

protein O-mannosylation III (mammals, core M3) (PWY-7979):

Man-GDP -> Man-dolichyl-P (single-step = EC:<IP_ADDRESS>/GO:0004582 MF)
transfer of Man from Man-dolichyl-P to O-linked Ser/Thr on protein (single step = EC:<IP_ADDRESS>/GO:0004169 MF)
transfer of GlcNAc and then GalNAc (ie. not Man) by two different enzymes

From this, it seems "protein O-linked mannosylation" could be considered a process involving the first two steps shared between the three MetaCyc pathways above.
However, the current def of "GO:0035269 protein O-linked mannosylation" is just:
def: "The transfer of mannose from dolichyl activated mannose to the hydroxyl group of a seryl or threonyl residue of a protein acceptor molecule, to form an O-linked protein-sugar linkage." [GOC:bf, PMID:9878797]
which is just the second step, which is covered by the MF term GO:0004169:
id: GO:0004169
name: dolichyl-phosphate-mannose-protein mannosyltransferase activity
def: "Catalysis of the reaction: dolichyl phosphate D-mannose + protein = dolichyl phosphate + O-D-mannosylprotein." [EC:<IP_ADDRESS>, GOC:pr]
So, I'd say we either:

Change the def of "GO:0035269 protein O-linked mannosylation" so that it covers both these steps:

Man-GDP -> Man-dolichyl-P (single-step = EC:<IP_ADDRESS>/GO:0004582 MF)
transfer of Man from Man-dolichyl-P to O-linked Ser/Thr on protein (single step = EC:<IP_ADDRESS>/GO:0004169 MF)
(and request an annotation review)
or

Obsolete "GO:0035269 protein O-linked mannosylation" because its a single step process whose current def is the same as "GO:0004169 dolichyl-phosphate-mannose-protein mannosyltransferase activity" (in which case we wouldn't have a BP term for those 2 steps, but that's probably OK)

"GO:0044845 chain elongation of O-linked mannose residue" appears to be specific to yeast/fungi, corresponding to the second half of MetaCyc protein O-mannosylation I (yeast) PWY-7921.  All 8 annotations are to yeast species.

It seems reasonable to extend the definition of GO:0035269 protein O-linked mannosylation" so that it covers both these steps.
As for "GO:0044845 chain elongation of O-linked mannose residue". Maybe the term should be kept but the name improved?  @ValWood what do you think?

We can keep if the name is improved , but I still think we don't need it. I see only 2 EXP annotations and they are both pombe?
https://www.ebi.ac.uk/QuickGO/annotations?goUsage=descendants&goUsageRelationships=is_a,part_of,occurs_in&goId=GO:0044845&evidenceCode=ECO:0000269&evidenceCodeUsage=descendants
One of these is an ER protein.....
Is all elongation in the Golgi? I'm not sure that it is?
How does "elongation" differ from  GO:0035269 protein O-linked mannosylation (I don't  know because I didn't look in detail)
Could we not use GO:0035269 protein O-linked mannosylation occures_in blah?


Add metadata for image

What does this PR do?

Adds image metadata in public html

@Developerayo  I added the metadata for the image in the public HTML already but from the document I used, it says the project has to be built with npm run build for it to reflect
Here's the document I used: https://medium.com/@weberzt/how-to-add-a-display-image-to-your-react-app-using-meta-tags-1b84fd50f57


fix(cdk): reduce ephemeral storage in Eliza task definition from 50GB to 20GB

TL;DR
Added Redis caching support to the ECS infrastructure and reduced memory allocation for task definitions.
What changed?

Added Redis ElastiCache instance with security group configuration
Configured Redis connection details in character environment variables
Reduced ECS task definition memory from 16GB to 4GB
Changed caching strategy from database to Redis for Trump character
Disabled automatic deployment on main branch pushes

How to test?

Deploy the infrastructure changes using CDK
Verify Redis instance is created and accessible
Confirm ECS tasks can connect to Redis using the provided endpoint
Monitor memory usage of running tasks to ensure 4GB is sufficient

Why make this change?
Redis provides better performance for caching compared to using the database directly. The memory reduction helps optimize costs while maintaining adequate resources for the application. The deployment trigger change allows for more controlled releases.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#33  👈 (View in Graphite)
#32
#31
#29 : 1 other dependent PR (#30 )
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Merge activity

Dec 28, 10:51 PM EST: A user started a stack merge that includes this pull request via Graphite.


Revert caching on ModelAdmin permissions

Fixes #9207. Revert cb4017c649ee5e7be373d7a35e700d1c322d5b63, and extend tests to verify that changes to permissions immediately take effect on subsequent page requests, rather than being improperly cached.

@gasman I agree the lru_cache is harmful,  but I'm less convinced the cached_property is - this value should only change when permissions themselves are changed, rather than who has them, which should usually only be on migration. I feel like a value that generates extra queries to find, but changes extremely rarely (pretty much only on actual code changes) is an ideal candidate for caching...

@jacobtoppm Good point - will try reinstating the cached_property, and hopefully we should find that the new tests still pass.

@jacobtoppm Have reinstated the cached_property, and confirmed that we successfully get the reduction in queries without the sticky permissions problem - nice!
Have also taken the opportunity to fix the assertNumQueries tests so that they give the same results when run individually (which is kind of important when preparing a patch like this...)
No idea why CircleCI and the other third-party CI tools are choking on this - presumably they don't like the force-push.

Merged in 77ed99b16e2fa67a982d106e025d23e9b88cbe77 (main) and 6b795c8c16108d14f93ceef0fda39c1a3cb31184 (stable/4.0.x)


Fabricization

PR for #5

@kvverti remember to move the entrypoints to the impl package as well

If you use idea, I suggest you perform a reformat of code over the whole main source set java directory


Converting snapshot creation timestamp into UTC

Converting the Snapshot creation timestamp returned by REST API from local timezone to UTC.

Will open a new PR for this


createPresignedHttpRequestFromCtx() should escape both path and query args

Describe the bug
createPresignedHttpRequestFromCtx() in s3api/utils/utils.go does not generate a canonical URL,
and presigned url check will  fail for  any url if  its query args contains ';' or  its path contains  UTF-8 code.
This problem is caused by ctx.Request().URI() returning decoded  path and query args. Thus the newly
assembled url  by createPresignedHttpRequestFromCtx  is not canonical and may contain unescaped semicolon ';'.
This will make signer.PresignHTTP() in CheckPresignedSignature fails to generate correct signature because
query args containing semicolon is omitted by net/url.ParseQuery, as the stack below shows
net/url.parseQuery() /snap/go/10553/src/net/url/url.go:944 (PC: 0x79ef33)
   939:        func parseQuery(m Values, query string) (err error) {
   940:                for query != "" {
   941:                        var key string
   942:                        key, query, _ = strings.Cut(query, "&")
   943:                        if strings.Contains(key, ";") {
=> 944:                                err = fmt.Errorf("invalid semicolon separator in query")
   945:                                continue
   946:                        }
   947:                        if key == "" {
   948:                                continue
   949:                        }
(dlv) bt
bt
 0  0x000000000079ef33 in net/url.parseQuery
    at /snap/go/10553/src/net/url/url.go:944
 1  0x000000000079ed85 in net/url.ParseQuery
    at /snap/go/10553/src/net/url/url.go:935
 2  0x00000000007a050d in net/url.(*URL).Query
    at /snap/go/10553/src/net/url/url.go:1122
 3  0x00000000009b80f7 in github.com/aws/aws-sdk-go-v2/aws/signer/v4.(*httpSigner).Build
    at ./go/pkg/mod/github.com/aws/aws-sdk-go-v2@v1.24.1/aws/signer/v4/v4.go:153
 4  0x00000000009b9c25 in github.com/aws/aws-sdk-go-v2/aws/signer/v4.(*Signer).PresignHTTP
    at ./go/pkg/mod/github.com/aws/aws-sdk-go-v2@v1.24.1/aws/signer/v4/v4.go:377
 5  0x00000000012c97ba in github.com/versity/versitygw/s3api/utils.CheckPresignedSignature
    at ./go/src/github.com/versity/versitygw/s3api/utils/presign-auth-reader.go:98
 6  0x0000000001493ec5 in github.com/versity/versitygw/s3api/middlewares.VerifyPresignedV4Signature.func1
    at ./go/src/github.com/versity/versitygw/s3api/middlewares/presign-auth.go:62
 7  0x0000000001297b29 in github.com/gofiber/fiber/v2.(*Ctx).Next
    at ./go/pkg/mod/github.com/gofiber/fiber/v2@v2.52.0/ctx.go:1027

The same analysis applies to ute-8 code.
According to https://docs.aws.amazon.com/IAM/latest/UserGuide/create-signed-request.html,  url should
be canonical when checking presigned url. A proposed fix is listed below, subject to consideration by the VGW author.
diff --git a/s3api/utils/utils.go b/s3api/utils/utils.go
index aa4346b..7567e28 100644
--- a/s3api/utils/utils.go
+++ b/s3api/utils/utils.go
@@ -20,11 +20,13 @@ import (
        "fmt"
        "io"
        "net/http"
+       "net/url"
        "regexp"
        "strconv"
        "strings"
        "time"

+       "github.com/aws/smithy-go/encoding/httpbinding"
        "github.com/gofiber/fiber/v2"
        "github.com/valyala/fasthttp"
        "github.com/versity/versitygw/s3err"
@@ -107,16 +109,18 @@ func createPresignedHttpRequestFromCtx(ctx *fiber.Ctx, signedHdrs []string, cont
        }

        uri := string(ctx.Request().URI().Path())
+       uri = httpbinding.EscapePath(uri, false)
        isFirst := true

        ctx.Request().URI().QueryArgs().VisitAll(func(key, value []byte) {
                _, ok := signedQueryArgs[string(key)]
                if !ok {
+                       escapeValue := url.QueryEscape(string(value))
                        if isFirst {
-                               uri += fmt.Sprintf("?%s=%s", key, value)
+                               uri += fmt.Sprintf("?%s=%s", key, escapeValue)
                                isFirst = false
                        } else {
-                               uri += fmt.Sprintf("&%s=%s", key, value)
+                               uri += fmt.Sprintf("&%s=%s", key, escapeValue)
                        }
                }
        })

To Reproduce
Use AWS SDK to presign  a URL whose query argument contains a semicolon, like below
2024/03/15 21:55:24 Request query arguments:
2024/03/15 21:55:24 response-content-disposition: inline; filename="The_Rust_Programming_Language.pdf"; filename*=UTF-8''The_Rust_Programming_Language.pdf
2024/03/15 21:55:24 response-content-type: application/pdf

Server Version
output of
./versitygw --version
Version  : v0.14
Build    : 905b283
BuildTime: 2024-03-16_04:27:15AM

Additional context
Describe s3 client and version if applicable.

@zhucaifeng81 Thanks for the bug report and a fix! Feel free to open a PR with this patch if you want to want to get the github contribution credit for it.  If not, no problem and I'll open a PR for this.

No credit is needed.  It is already a pleasure to study  the clean VGW code. Thanks for your  excellent work.


(SP: 1) [TinyMCE] Generate token

Description

Register TinyMCE and generate token
Update the environment configuration (.env) with the token.

⚠ Use project email for TinyMCE sign up

register account on TinyMSE using project email (add reg info to data)
generated api_key token
add token to environment configuration (.env) in google doc

Let's try adding this editor for question creation.


Can't edit a previously saved service fee in  Settings > Moderator

Brief Description: Can't edit a previously saved service fee in  Settings > Moderator
Operating System (OS and version): OSX Yosemite v 10.10.4
OpenBazaar version (shown on About OpenBazaar page in menu): 1.0.7
Hardware: MacBook Pro (Retina, 13-inch, Late 2013)
Reproducible (Always / Almost Always / Sometimes / Rarely / Couldn't Reproduce): Always
Steps to reproduce:

Go to Settings > Moderator and set a service fee and save
Go back to Settings > Moderator and try to revert the service fee to zero - after saving it reverts to the previous setting. N.B. this only happens if reverting to zero and not any other value

Observed Behavior: Get confirmation of saved edit but still not updating
Expected Behavior: For the new fee I've entered to be correct
Additional info (links, images, etc go here):

There is something wrong with setting the value to 0%. This might be a server side thing @cpacia .

Other values seem to work properly.

Actually @bangers89 you can set it properly from your User Page Moderator Settinsg to get around this for now.

This appears to be a server issue. It's being sent to the API, but the original value is returned regardless of where you safe it.
I'll file a server issue.


Repeated download of labels

Repeated download of labels:

I get an error massage while downloading labels repeatedly: "Parcel label is already generated".
I get no parcel Id in this error message. That can be problem when downloading bigger amount of labels

It can happen that user makes mistake or there is a downloading error etc. Repeated download of labels is sometimes necessary.
How can I download labels repeatedly if needs to download it again? Thanks
`
private function printPackages(array $packages, ?Shipment $shipment): ?string
{
$client = new Client(new HttpClient());
$request = new GetPrintedLabels();
    foreach ($packages as $package) {
        $request->addParcelId($package);
    }

    /** @var GetPrintedLabelsResponse $response */
    $response = $client->on($this->account)
        ->request($request);

    if ($response->successfull()) {
        return $response->getPdf();
    }

    if ($shipment) {
        $shipment->update(
            [
                'error_message' => $this->glsErrorsToText($response->errors()),
            ]
        );
    }
    Log::error('printLabel: ' . $this->glsErrorsToText($response->errors()));
    return null;
}

`

Hello,
You can download/generate the labels only once, unfortunately MyGLS doesn't support calling GetPrintedLabels multiple times for the same parcel.

Thanks for fast reaction. Jan


Clicking on pipeline name redirects the user to the Pipeline list page

Steps to reproduce

Go to Pipeline list section
Click on the name of any pipeline to go to the Pipeline Details page
Click on the pipeline name

Current result: Clicking on pipeline name redirects the user to the Pipeline list page
Expected result: Clicking on pipeline name should not do any action at all. In order to allow the user to get back to Pipeline list section, let's add an arrow before the Pipeline name, as it happens in Target inner pages. Clicking on the return arrow will redirect the user to Pipeline list section

Working as expected


how to use lightgbm of hdfs version?

Build from source with HDFS enabled, detailed could be found at :
https://github.com/Microsoft/LightGBM/pull/1243

@chivee  well I know how to build it but wonder how to use it from python api

@chivee  need I train it like https://github.com/Microsoft/LightGBM/issues/396?

@janelu9 it supports the hdfs file. I don't think it can support spark rdd.
For the spark version, please refer to https://github.com/Azure/mmlspark, which support LightGBM


azure cli - download certificate as pfx format

az keyvault certificate download --file
[--encoding {DER, PEM}]
On azure web-portal (also in PS) there is an option to download as pfx. Why it is missing in az cli then?

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

ID: a9c661d7-bf43-5354-015e-3859f30a42a2
Version Independent ID: fa69e552-5904-ce97-d02c-915c819bdde1
Content: az keyvault certificate
Content Source: src/command_modules/azure-cli-keyvault/azure/cli/command_modules/keyvault/_help.py
Service: key-vault
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

You can do that today in two steps.
First, download the certificate with az keyvault secret download, which will download the raw PFX as Base64-encoded text. Then undo the Base64 encoding, and you have the binary PFX file.
I don't know why, but the certificate APIs only want to return the public part of the certificate, which is why az keyvault certificate download doesn't offer PFX as an option. If you download the certificate as a secret (even though it's not a secret), it downloads the raw PFX (with no password).
Go figure. 🤷♂

@bradwilson :+1: thx, but still there in inconsistent API between CLI and Powershell (with Azure Portal).
IMHO at least it should be mentioned in the docs how to download pfx through CLI with your workaround.

My use case was getting the pfx from keyvault and upload that in Azure Apim instance, we tried what @bradwilson has suggested and then uploaded the pfx via Rest API.
Not sure if that is the best option but that has worked.

My use case is:
Generate pem with cert and key, after this imported in azure:
az keyvault certificate import --file certandkey.pem --name test --vault-name kevaulttest
When download in Azure:
az keyvault certificate download --vault-name kevaulttest -n test -f certificate.pem
Only certificate with:
-----BEGIN CERTIFICATE----- -----END CERTIFICATE-----
Private key not show in certificate.pem
But when I download the secret, comes the private key + certificate

This is just how these APIs work in Azure Key Vault. The "get certificate" API is only intended to permit access to the public part of the certificate; if you need access to the private key, then you request the certificate through the "get secret" API.

I had to use the cert in a tomcat server. The problem was that the downloaded cert didnt contain the password, hence I had to import and then export the cert again with the password.
Here is the code :
az keyvault secret download
--file inputCert.pfx
--vault-name MyKeyvault
--encoding base64
--name inputCert
2>&1
| tee -a $SETUP_LOG
echo "Converting downloaded cert to pem format"
openssl pkcs12 -in inputCert.pfx -out temp.pem -nodes -password pass:""
echo "Converting the cert with the password"
openssl pkcs12 -export -out outputCert.pfx -in temp.pem -password pass:"yourpassword"

I import a pkcs12 certificate via the cli but cannot get it back?
why use key vault for this at all?
I'm going to store my certs as blobs.
This is not a vault, if I put something into a vault I expect to get it back exactly as it was when I put it in.

You can do that today in two steps.
First, download the certificate with az keyvault secret download, which will download the raw PFX as Base64-encoded text. Then undo the Base64 encoding, and you have the binary PFX file.
I don't know why, but the certificate APIs only want to return the public part of the certificate, which is why az keyvault certificate download doesn't offer PFX as an option. If you download the certificate as a secret (even though it's not a secret), it downloads the raw PFX (with no password).
Go figure. 🤷♂
Edit: You can specify --encoding base64 and it will un-encode it for you automatically.

its almost 3 years later and this saved me.  Powershell az.keyvault does not grab the chain when you get the secret and turn it into a pfx.  az keyvault secret download does.  Its a work around and not ideal, but it unblocks me.  Thanks.

Thank you @bradwilson, saved me too!
While the documentation at https://docs.microsoft.com/en-us/azure/key-vault/certificates/how-to-export-certificate?tabs=azure-cli goes over this, what it doesn't mention is that you need to use the same ID for the certificate but amend it such that it is in the format http://${VAULTNAME}.vault.azure.net/secrets/${CERTNAME}. If you use the ID provided by Azure directly, i.e. replacing secrets with certificates, you receive an error:
ValidationError: invalid id: The collection "certificates" does not match the expected "secrets"

It seems to me that Azure should either allow those to redirect in some way if they are certificates or add something into the az cli to allow the PFX to be downloaded. I suspect adding a cli command for it might also open up opportunities to create a password on the PFX too.

I created new version of the certificate through az keyvault certificate create
Then I downloaded the cert using az keyvault secret download  with --encoding base64
Thentried to apply openssl pkcs12 -in to get .pem file but I got the following error
140633834518336:error:0D0680A8:asn1 encoding routines:asn1_check_tlen:wrong tag:crypto/asn1/tasn_dec.c:1149:
140633834518336:error:0D07803A:asn1 encoding routines:asn1_item_embed_d2i:nested asn1 error:crypto/asn1/tasn_dec.c:309:Type=PKCS12
What am I doing wrong?

The problem seams to be related to the command "az keyvault secret download with --encoding base64" which get the private key unprotected with no password >"-----BEGIN PRIVATE KEY-----".
Azure cli library expects the PEM file to have an encrypted key > "-----BEGIN ENCRYPTED PRIVATE KEY-----"
As mentioned by @akshita31 you need to download and export the PEM with a password
# Login with credentials having access to the Key Vault certificate. Role > "Key Vault Certificate User"
# az login -t $tenantId

# Private key pfk is exported from KeyVault with an empty password.
passphrase=""

# Store the certificate into variable
certificate_pfx_private=$(az keyvault secret show --name ${certificate_name} --vault-name ${KeyVaultName} --query value --output tsv)

# Close the current Azure session to clean the local cli cache
az account clear

# Create temp file
temp_file="$(mktemp)"

# Set the output PEM to empty password is working but not recommended
openssl pkcs12 -in <(echo -n "$certificate_pfx_private" | base64 --decode) -passin "pass:$passphrase"  -out $temp_file -passout pass:$passphrase

# check the PEM file is in the correct format
check_private_key=$(cat $temp_file | grep -e "-----BEGIN ENCRYPTED PRIVATE KEY-----")
if [ $? -ne 0 ]; then
  echo "Error: Private key not in the correct format."
  #exit 1
fi

az login --service-principal -u "${clientId}" -p $temp_file --tenant "${tenantId}"

# Do you work on Azure. Note the file is required to allow the az cli to request new access token as needed.
# delete file to logout
# rm $temp_file

You can do that today in two steps.
First, download the certificate with az keyvault secret download, which will download the raw PFX as Base64-encoded text. Then undo the Base64 encoding, and you have the binary PFX file.
I don't know why, but the certificate APIs only want to return the public part of the certificate, which is why az keyvault certificate download doesn't offer PFX as an option. If you download the certificate as a secret (even though it's not a secret), it downloads the raw PFX (with no password).
Go figure. 🤷♂
Edit: You can specify --encoding base64 and it will un-encode it for you automatically.

az keyvault secret download --encoding base42 ... doesn't work, the downloaded PFX seems corrupted. It has 1 extra byte.

You can do that today in two steps.
First, download the certificate with az keyvault secret download, which will download the raw PFX as Base64-encoded text. Then undo the Base64 encoding, and you have the binary PFX file.
I don't know why, but the certificate APIs only want to return the public part of the certificate, which is why az keyvault certificate download doesn't offer PFX as an option. If you download the certificate as a secret (even though it's not a secret), it downloads the raw PFX (with no password).
Go figure. 🤷♂
Edit: You can specify --encoding base64 and it will un-encode it for you automatically.

There's no need to decode the downloaded file. But the downloaded PFX is not valid.

openssl pkcs12 -info -noout -in downloaded.pfx
F8420000:error:068000A8:asn1 encoding routines:asn1_check_tlen:wrong tag:../openssl-3.2.1/crypto/asn1/tasn_dec.c:1186:
F8420000:error:0688010A:asn1 encoding routines:asn1_item_embed_d2i:nested asn1 error:../openssl-3.2.1/crypto/asn1/tasn_dec.c:349:Type=PKCS12


Zone Keyboards Show "Unknown" Status Instead of Real Status

Issue: Zone Keyboards Show "Unknown" Status Instead of Real Status
Description:
The integration was functioning well for a few days, accurately displaying status updates (armed, disarmed, pending, etc.) for my Zone Keyboards (Strefy). However, after a couple of days, the Zone Keyboards in HASS began displaying an "Unknown" status instead of the correct real status.
I've been using original HASS satel integration for around a year before and I've not encountered any issues with zone status updates during that time.
Possible Cause:
I made some adjustments in the DloadX program to add additional outputs, which should not have affected the zones directly. However, this could still be related to the issue.
Additionally, I encountered a strange behavior with the new outputs:

New outputs configured on output numbers 11-15 did not appear in the Home Assistant GUI, even though they were properly set up in the YAML file.
Same new outputs configured on another output numbers (121-125) appeared in the HA GUI immediately.
This inconsistency might be relevant to the unknown status issue.

Troubleshooting Steps Taken:
Integration Reconfiguration - I deleted the entire satel integration configuration from the YAML file, removed manually all entities from the Home Assistant GUI, and re-added the satel configuration in YAML again. Unfortunately, this did not resolve the issue.
Request for Assistance:
Any insights or suggestions for resolving this would be greatly appreciated. If there are specific diagnostics or logs I should provide, please let me know.
Firmware:
I have not made any changes to the firmware for around 1 year. Currently using version:
Communication module: ETHM-1 V2.10
Alarm system: Integra 256 Plus 1.22 PL

Please enable the debug log in configuration.yaml:
logger:
  default: info
  logs:
    satel_integra_ext.satel_integra: debug

and check the HA log when the issue happens.

Thank you! Here are the logs:
PARTITION (STREFA) ISSUE
Logger: homeassistant.components.alarm_control_panel
Source: components/alarm_control_panel/init.py:210
integration: Alarm control panel (documentation, issues)
First occurred: 22:59:53 (3 occurrences)
Last logged: 22:59:53
Entity alarm_control_panel.zone_house (<class 'custom_components.satel_integra_ext.alarm_control_panel.SatelIntegraAlarmPanel'>) is setting state directly which will stop working in HA Core 2025.11. Entities should implement the 'alarm_state' property and return its state using the AlarmControlPanelState enum, please report it to the custom integration author.
Entity alarm_control_panel.zone_garage (<class 'custom_components.satel_integra_ext.alarm_control_panel.SatelIntegraAlarmPanel'>) is setting state directly which will stop working in HA Core 2025.11. Entities should implement the 'alarm_state' property and return its state using the AlarmControlPanelState enum, please report it to the custom integration author.
Entity alarm_control_panel.zone_outside (<class 'custom_components.satel_integra_ext.alarm_control_panel.SatelIntegraAlarmPanel'>) is setting state directly which will stop working in HA Core 2025.11. Entities should implement the 'alarm_state' property and return its state using the AlarmControlPanelState enum, please report it to the custom integration author.
This was the only error concerning the alarm_control_panel. Seems like there are some issues but theoretically it should be fine till HA Core 2025.11
MISSING OUTPUT ISSUE
Logger: homeassistant.components.binary_sensor
Source: helpers/entity_platform.py:764
integration: Binary sensor (documentation, issues)
First occurred: 22:59:53 (1 occurrences)
Last logged: 22:59:53
Platform satel_integra_ext does not generate unique IDs. ID $satel_integra_ext.input$15 already exists - ignoring binary_sensor.input_15
It seems like these two parts of configuration (zone and output) collide with each other while creating unique_id
This is my config:
zones:
15:
name: "Input 15"
type: "opening"
outputs:
15:
name: "integra_zalanie_detect"
type: "safety"
I changed the entity.id of my input 15 (binary_sensor.input_15 to binary_sensor.input_15x), but it didn't solve the issue.
Logger: homeassistant.components.binary_sensor
Source: helpers/entity_platform.py:764
integration: Binary sensor (documentation, issues)
First occurred: 23:37:36 (1 occurrences)
Last logged: 23:37:36
Platform satel_integra_ext does not generate unique IDs. ID $satel_integra_ext.input$15 already exists - ignoring binary_sensor.input_15x

Are you sure you're using the latest version 1.0.1 of Satel Integra Extended?

Yes, I'm sure, please take a look a the screenshot:

this one is my "dev" fork which I always sync to lashqo before a release.
I think I see the bug. Non switchable outputs use "input" in the unique_id while it should use "output" (or yet something else more unique).
I'll see if I can make a fix in the evening.

Great! Thank you! Do you have any ideas about the other issue? - Partition (Strefa) showing "Unknown" status

That one I don't know yet.
Which version of HA do you have?

Core 2024.11.0
Supervisor 2024.11.2
Operating System 13.2
Frontend 20241106.0

I see they changed something in 2024.11.
I will have to test it with this release. Before that you can try to downgrade.

Can you please test v1.0.2-beta1 release?

Just tested v1.0.2-beta1 release. Both issues are resolved. The Partitions (Strefy) showed up, I see the status of each Partition (Strefa) and I'm able to arm/disarm. Also the new outputs showed up after restarting.
Thanks you for quickly fixing these issues. Much appreciated!

Great to hear that, thanks for quick confirmation!


Add tomopy plugin

A Tomopy plugin could be handy for:

centre of rotation, https://tomopy.readthedocs.io/en/latest/api/tomopy.recon.rotation.html#module-tomopy.recon.rotation
alignement

We will need some user input/dataset

May be algotom instead or in addition to tomopy.
wrapped functions:

ring remover artifacts
paganin


scat2gridgauss functions should check that output grid is REGULAR

Reported by Ansley B Manke on 14 Jan 2004 19:17 UTC
Algorithm assumes regular output axes, but will
compute a (wrong) result on irregular axes!
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/761

Comment by<EMAIL_ADDRESS>on 19 May 2005 17:55 UTC
done a while back...


⚠ Science Web Services - QState has degraded performance

In 1379bd8, Science Web Services - QState (https://sws.labbricks.com/qstate/) experienced degraded performance:

HTTP code: 523
Response time: 3258 ms

Resolved: Science Web Services - QState performance has improved in 5270bc8 after 19 minutes.


Enhancement: Run php-cs-fixer on GitHub Actions

This PR

[x] runs php-cs-fixer on GitHub Actions

Follows #27.
💁♂ As you can see in this pull request, using GitHub Actions can be quite verbose.

Thank you, @nikolaposa!