Randomly pick a set of colors from the color-swatches.php setup

I would like to randomly get a set of color from my color-swatches.php setup (I have currently only one set).
This can be useful in my case when the field is not filled, so one is randomly picked.
From a native Dropdown field, I could probably pick them using craft.fields.getFieldbyHandle('handle').settings.options.
How can I achieve this using the colour-swatches custom plugin?

Hey @romainpoirier
Currently we don't have any possibilities to pick a random colour on save when nothing has been selected, but I do like the idea.
We'll tinker about this on how to best implement in the plugin. Thank you for the idea!

@romainpoirier we had some tinkering about this.
Would it make sense to introduce an extra field setting ( lightswitch maybe ) when a field gets created, and when no value is selected it picks a random colour. Or would it make more sense to add an option to the actual colour palette where "random" can be selected?

Hello @romainpoirier,
We've implemented the feature, there's now a settings switch on the field where you can select if a random color needs to be picked if the user doesn't.


Offset contextmenu by 2px

Fixes https://github.com/Microsoft/vscode/issues/57399

@sbatten I have to check, but this is literally the same code we use for native context menus too.

@bpasero I might be wrong, but I think it isn't an issue in native because the menu can be drawn to the right and down in almost all cases because the menu can be drawn outside the window. You might be able to test on Linux if the issue arises in fullscreen clicking in the bottom right?

This is not a good fix for the reasons mentioned.


How about a join command?

Has any thought been given to a command/shortcut that joins/merges the present line with the next? That'd be handy.

@reagle can you give an example?

The question is if this should be in the main codebase or rather a plugin like the  comment plugin.
I never wrote a Lua script but it doesnt look tooo crazy.

Don't know about that; I've always taken it for granted.

I'm working on a little plugin. I hope that I'm able to push the first version today.

I finished the plugin. At the moment it is just available from my Repo. But I will make a pull request so that is installable via the plugin channel.

Thanks, look forward to installing it once it is!


NCM Stress Test Report

This PR includes the report of NCM Stress Test.

3rd file: where is the gray area? Is it 100% so covered by blue or red ?
4th file: in excel, there's a way to show numbers of the height (Y axis), it will add some level of clarity to the slope chart

The gray area is so small that it is not noticable in either of the graphs.
I added the values for the Y axis, and changed the color of the 1 VPC testcases to green.


1.0.2 showing unstable behavior

Hi!
Cool idea, definitely more fun than smartctl.
I'm running version 1.0.2. When I start crazy, one of three things happens:

Segmentation fault
Starts up showing /dev/sda (as good) and /dev/sr0 (as bad), latter of which is my DVD drive
Starts up showing /dev/sda (good) only (i.e. no /dev/sr0)

When I run echo -n q | sudo crazy | fgrep /dev in a loop, I can see all of these without waiting for long.
Besides the segfault, it would be cool if display was stable and the DVD drive would not be shown at all, at least not by default.
Thanks!

On a side not, libatasmart seems to change its mind about sr0 from time to time:
# sudo /usr/sbin/skdump /dev/sr0  # first take
Device: sat16:/dev/sr0
Type: 16 Byte SCSI ATA SAT Passthru
Size: 1023 MiB
Model: []
Serial: []
Firmware: [y !]
SMART Available: yes  <----
Quirks:
Awake: no
SMART Disk Health Good: Input/output error
Failed to dump disk data: Operation not supported

# sudo /usr/sbin/skdump /dev/sr0  # second take
Device: sat16:/dev/sr0
Type: 16 Byte SCSI ATA SAT Passthru
Size: 1023 MiB
Model: []
Serial: []
Firmware: [y !]
SMART Available: no  <----
Quirks:
Awake: no
ATA SMART not supported.

Hi @otakuto
Gentoo users reported segmentation faults when compiling crazydiskinfo with GCC. However, for the record, I compiled it with clang and I cannot reproduce the segfault. The program starts and runs just fine. See https://bugs.gentoo.org/show_bug.cgi?id=629520 for more details.

Here's a backtrace:
(gdb) bt
#0  0x00000000004099ae in attributeToHealth(Attribute const&) ()
#1  0x0000000000409abb in smartToHealth(SMART const&) ()
#2  0x0000000000409da7 in drawDeviceBar(_win_st*, std::vector<SMART, std::allocator<SMART> > const&, int) ()
#3  0x000000000040b611 in main::{lambda()#2}::operator()() const ()
#4  0x000000000040cbed in std::_Function_handler<void (), main::{lambda()#2}>::_M_invoke(std::_Any_data const&) ()
#5  0x000000000040ed36 in std::function<void ()>::operator()() const ()
#6  0x000000000040c171 in main ()

@otakuto ping?

pong!
I'm sorry to have kept you waiting.
smartmontools support json output recently!
https://www.smartmontools.org/changeset/4640
Also, I will use smartctl as backend on 2.0.0.
So I think that this problem will be solved.
Thank you.

Thanks for the response. Looking forward to 2.0.0 then.

Closing to keep https://github.com/issues clean for myself…


FEATURE REQUEST: Change Hostname

Not quite sure how to set this up, but is there any way to either re-run the installer without having to sit for 1/2 hour waiting for the Diffie-Hellman (?) primes to be produced?
Specifically, I've twice fudged up my Dynamic DNS settings and it'd be really helpful if we could do this bit without having to dive back in and start over.
For example:
If I enter, mydomain.ddns.com (when it should be ".net") I have to go through the whole process again. It's particularly bad for beginners and sleepy admins who aren't paying sufficient attention.
If this is connected to the DH keys, then obviously that's an issue, but if that's the case, a final "ARE YOU SURE" check might be an idea.
I seem to have similarly goofed my outbound DNS forwarder - I've pointed it to the PiHole but that's not working either.
These are minor issues of course, but if if were possible to fix them post install that would be terrific.
Thanks

https://github.com/pivpn/pivpn/wiki/FAQ#can-i-change-the-hostname-the-ip-address-protocol-or-port-after-the-install

Updating the wiki link for others:
If using WireGuard: https://github.com/pivpn/pivpn/wiki/WireGuard#how-do-i-change-the-public-ipdns-name-of-the-pivpn-after-the-install
If using OpenVPN: https://github.com/pivpn/pivpn/wiki/OpenVPN#how-do-i-change-the-public-ipdns-name-of-the-pivpn-after-the-install


issue when adding custom headers for  External Authorization

I am trying to send a custom header from each route to External Authorization server , this header will be used to identify the service and do Authorization .When I try adding the header at route level , getting unable to unpack error.
Can you please help in figuring out what is missing ?
Below is the config in Yaml
virtual_hosts:               - name: local_service                 domains: ["*"]                 routes:                 - match: { prefix: "test/v1/service" }                   route:                      cluster: backend                                       typed_per_filter_config:                     envoy.filters.http.ext_authz:                       "@type": type.googleapis.com/envoy.extensions.filters.http.ext_authz.v3.AuthorizationRequest                                             headers_to_add:                          -  {"key": "x-prefix","value": "servicev1"}
Error:
 Unable to unpack as envoy.extensions.filters.http.ext_authz.v3.ExtAuthzPerRoute: [type.googleapis.com/envoy.extensions.filters.http.ext_authz.v3.AuthorizationRequest] { envoy_1      |   headers_to_add { envoy_1      |     key: "x-prefix" envoy_1      |     value: "servicev1" envoy_1      |   } envoy_1      | }

The yaml you provided is using @type: "type.googleapis.com/envoy.extensions.filters.http.ext_authz.v3.AuthorizationRequest" for the per filter config, but I think you actually want @type: "envoy.extensions.filters.http.ext_authz.v3.ExtAuthzPerRoute"
Instead of sending a custom header to the external authorization server to identify the source route, I believe you need to instead use the context_extensions field on the gRPC check request message https://github.com/envoyproxy/envoy/blob/12537506f8bb42797c0bfe6fa9418e02cb34ae63/api/envoy/extensions/filters/http/ext_authz/v3/ext_authz.proto#L295 - this requires using a gRPC auth service instead of an HTTP one, though.


Add Github App authentication

A small proposal how to add Github Apps authentication #212
Definately should get some e2e tests later.
Documentation is missing.
Please let me know what you think!

I added documentation and also build the docker image to try it in a live example.
It worked perfectly fine, except installing git-crypt. But I just disabled that for my test.
Let me know if there is anything else I should do!

For everyone trying it out:
https://hub.docker.com/r/tenjaa/github-pr-resource

This is a feature that I'd like!   What's the story here?   Need a hand rebasing?  Who should we ping to review it?

@kevincantu Have a look at https://github.com/telia-oss/github-pr-resource/issues/212 for the current status

Accdentially closed this  :D
My goal was to create a branch with the update and then write some automation to keep the master branch of my fork up to date. Something went wrong..
Have a look at https://github.com/tenjaa/github-pr-resource/tree/github-app for commits from this PR.
And a watch the master branch of that repo for up-to-date builds.
I am not going to reopen this as it wouldnt get merged anyway. But I will probably keep my fork up to date (from time to time at least) until var sources are fully available.


Constants might not work in external styles - investigate

Hi I'm trying to add constants in an external css file like so
/* constants.js */
export default { green: 'green' }

/* styles.js */
import color from './constants'
export default css`{ div { color: ${color.green} } }`

/* comp.js */
import styles from './styles'
export default () =>
blah

I see the following message:
      Found an `undefined` or invalid value in your styles: `color.green`.

      If you are trying to use dynamic styles in external files this is unfortunately not possible yet.
      Please put the dynamic parts alongside the component.

uhm odd, those are not dynamic styles. You are using a constant and it should work. However dynamic styles like this.props.something shouldn't work. I'll investigate

@giuseppeg sorry let me clarify on the example, this only comes up when I try to use constants imported from another file, I updated the example to reflect that

hmm interesting it works if I do the following
/* constants.js */
const green = 'green';
export { green }

/* styles.js */
import { green } from './constants'
export default css`{ div { color: ${green} } }`

@TonyHYK yep there is a bug indeed when using color.green it throws. green should work so as a workaround for now you could use:
import color from './constants'
const { green } = color
export default css`{ div { color: ${color.green} } }`

or
import { green } from './constants'
export default css`{ div { color: ${green} } }`

@giuseppeg thanks for the fix. Will this be available in NextJS anytime soon? It looks like NextJS is using style-jsx before this fix.


Can the C language format be set to the C99 version?

I've tried migrating code from MRS, but it runs on C99, causing some code to not compile. correctly

Now I've tested three IDEs to find the one that's easiest to use.

MRS
Clion
VSCode

But there's a problem with C99.

I thought in C you always had to use
#include <stdbool.h>

to get the bool type. Per https://en.cppreference.com/w/c/types/boolean.

this not support print %f ?


Support for dotnet 7 and boostrap 5.0

Please how can we adapt this plugin to fit dotnet 7 and bootstrap 5.0

System.TypeLoadException: 'Method 'Handle' in type 'ChangeRouteHandler' from assembly 'Blazor-State, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null' does not have an implementation.'
I am using Blazor with dotnet 7 and when I use services.AddAdminLte(); that is the error I get each time.
Is support for .NET 7 not there yet?

.NET 7 should not be a problem but the base AdminLTE still uses Bootstrap v4.


Remove code-duplication from NativeLibraryLoader

Motivation:
NativeLibraryLoader has some code-duplication that can be removed.
Modifications:
Remove duplicated code and just use provided methods of PlatformDependent.
Result:
Less code duplication, fixes [#3756].

Cherry-picked into 4.1 (bdb0a39c8a) and 4.0 (1a0ef664c5d5371654c5a1680fb11df0ddd5efeb)


Error creating UUID,

Hi there, when running the default docker-compose I get the following error inside the umami app:

umami_1  | Prisma schema loaded from prisma/schema.prisma
umami_1  | Datasource "db": PostgreSQL database "umami", schema "public" at "db:5432"
umami_1  |
umami_1  | 1 migration found in prisma/migrations
umami_1  |
umami_1  | Applying migration 01_init
umami_1  |
umami_1  | The following migration have been applied:
umami_1  |
umami_1  | migrations/
umami_1  |   └─ 01_init/
umami_1  |     └─ migration.sql
umami_1  |
umami_1  | All migrations have been successfully applied.
umami_1  |
umami_1  | ✓ Database is up to date.
umami_1  | $ node scripts/update-tracker.js
umami_1  | $ node server.js
umami_1  | Listening on port 3000 url: http://dddf4ef6cb75:3000
umami_1  | PrismaClientKnownRequestError:
umami_1  | Invalid prisma.website.findUnique() invocation:
umami_1  |
umami_1  |
umami_1  | Inconsistent column data: Error creating UUID, invalid character: expected an optional prefix of urn:uuid: followed by [0-9a-zA-Z], found u at 1
umami_1  |     at pn.handleRequestError (/app/node_modules/@prisma/client/runtime/library.js:176:6477)
umami_1  |     at pn.handleAndLogRequestError (/app/node_modules/@prisma/client/runtime/library.js:176:5907)
umami_1  |     at pn.request (/app/node_modules/@prisma/client/runtime/library.js:176:5786)
umami_1  |     at async t._request (/app/node_modules/@prisma/client/runtime/library.js:179:10484)
umami_1  |     at async WEBPACK_DEFAULT_EXPORT (/app/.next/server/pages/api/websites/[id].js:154:25)
umami_1  |     at async Object.apiResolver (/app/node_modules/next/dist/server/api-utils/node.js:372:9)
umami_1  |     at async NextNodeServer.runApi (/app/node_modules/next/dist/server/next-server.js:514:9)
umami_1  |     at async Object.fn (/app/node_modules/next/dist/server/next-server.js:828:35)
umami_1  |     at async Router.execute (/app/node_modules/next/dist/server/router.js:243:32)
umami_1  |     at async NextNodeServer.runImpl (/app/node_modules/next/dist/server/base-server.js:432:29) {
umami_1  |   code: 'P2023',
umami_1  |   clientVersion: '4.13.0',
umami_1  |   meta: {
umami_1  |     message: 'Error creating UUID, invalid character: expected an optional prefix of urn:uuid: followed by [0-9a-zA-Z], found u at 1'
umami_1  |   }
umami_1  | }

This is after setting up the project, and adding a site. After I start doing some things on the site that are trackable, it just fails?
I am using Remix.js, and it is loading the script.js file fine.

 Can you check your session / website_event table and seeing if there's data? I'm wondering where its failing to create the UUID.


update libxml2

这里下载失败了，需要重跑下

note: the following packages are unsupported on macosx/x86_64:
  -> libiconv 1.17

Is it caused by this commit?
https://github.com/XmacsLabs/mogan/actions/runs/4096346496/jobs/7063919459


Laravel 5.5 Support ???

I am getting error when using it in Laravel 5.5 same code worked perfectly on 5.2
Target [Softon\Sms\Gateways\SmsGatewayInterface] is not instantiable while building [\Softon\Sms\Sms].

@softon I am getting the same error in L5.4 also.

@softon @salahy2  I tried to edit the SmsServiceProvider.php file as given below -
`
    $this->app->bind('sms', 'Softon\Sms\Sms');

    $this->app->bind('Softon\Sms\Gateways\SmsGatewayInterface','Softon\Sms\Gateways\\'.$gateway.'Gateway');
    $this->app->bind('Softon\Sms\SmsViewInterface','Softon\Sms\Sms'.$view.'View');

`
here I have removed the initial backslash \from the namespace and it started working.

Give this a try: https://packagist.org/packages/gr8shivam/laravel-sms-api

Hello,
I am getting same message with L5.5. Is there any official solution for it?

Hi @musamamasood , would suggest you to try (https://github.com/gr8shivam/laravel-sms-api). If the issue arises with that too, just raise an issue here (https://github.com/gr8shivam/laravel-sms-api) and I would love to resolve that.

Is it possible that I can use multiple custom gateway? Also is it possible to setup blade template for multiple sms message?
Thanks.

Multiple Custom Gateways: Yes, you can configure them easily.
Blade Template: Well not exactly in a blade template as it ain't required for a small message. Though you can use pre-defined notification templates to achieve the same use case.

Thanks.


Version footer user display

It should be the user's name, rather than User + user.id

(This is an issue for all classes of versioned objects.)

Joe - this was intentional, i have been going through trying to squash the proliferation of expensive lookups for trivial info on these views, that is ballooning our load times (everywhere, on every view — i don't think anyone has ever gone through the code to audit these).
However, maybe double-check the version helpers,  and trace the args back to the callers, because at this point, we may be passing the user in. There was a lot of refactoring. If the helper is calling version.user, and the user hasn't been eager loaded, that's initiating a lookup in the helper, and as always you can audit the lookup in the console - these are surprisingly (and frustratingly) expensive. If on the other hand the user is passed in or it's eager loaded, feel free to change it back.

Thanks for the response and pointers.
I'll try to look into it in a few days or so.
On Mon, Feb 5, 2024 at 1:33 PM andrew nimmo @.***>
wrote:

Joe - this was intentional, i have been going through trying to squash the
proliferation of expensive lookups for trivial info on these views, that is
ballooning our load times (everywhere, on every view — i don't think anyone
has ever gone through the code to audit these).
However, maybe double-check the version helpers, and trace the args back
to the callers, because at this point, we may be passing the user in. There
was a lot of refactoring. If the helper is calling version.user, and the
user hasn't been eager loaded, that's initiating a lookup in the helper,
and as always you can audit the lookup in the console - these are
surprisingly (and frustratingly) expensive. If on the other hand the user
is passed in or it's eager loaded, feel free to change it back.
—
Reply to this email directly, view it on GitHub
https://github.com/MushroomObserver/mushroom-observer/issues/1877#issuecomment-1928132929,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAALDFAUWHT7VZLOCQD6TCDYSFFZTAVCNFSM6AAAAABC2DZZUWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRYGEZTEOJSHE
.
You are receiving this because you authored the thread.Message ID:
@.***>


Using --target in new arg forwarding

What happened?
Thanks a lot for the changes from https://github.com/mitsuhiko/insta/issues/301.
Almost everything works, but I'm not sure how to pass --target. Specifically:

It seems that -p and --lib need to go before the --, and then other items go after; e.g. from 0.21.0 to 0.21.2:-cargo insta test --accept -- -p mdbook-prql run_examples
+cargo insta test --accept -p mdbook-prql -- run_examples

But I can't get cargo insta test --accept -- --target=wasm32-unknown-unknown to work, with --target either before the -- or after. (after two -- doesn't raise an error but also doesn't seem to do anything)  Finished test [unoptimized + debuginfo] target(s) in 0.36s
   Running unittests src/main.rs (target/debug/deps/mdbook_prql-c1048722bc7cd3a6)
error: Unrecognized option: 'target'

(ref https://github.com/prql/prql/pull/1172)
Sorry if this is a pain...
Reproduction steps
No response
Insta Version
0.21.2
rustc Version
rustc 1.64.0 (a55dd71d5 2022-09-19)
What did you expect?
No response

Thank you @mitsuhiko !


How do I access the inky parser

I making a app where user can edit mail, and i want to let them use in inky for convince.
In your doc you say "The Inky parser can be accessed directly for programmatic use." But how do i do that exactly???

You'll have to look at the source... I think you can use: Inky::Core.new.release_the_kraken(input) (yeah, I know, it's a bit of a crazy method name...)


Move CodeBuild Sources & Artifacts to a separate package

According to what we agreed to in #1743, we need to move CodeBuild's Sources and Artifacts to a separate package.
We should also make a decision whether we should have 2 new packages: one for Sources, one for Artifacts, or should we put both Sources and Artifacts in one new package.

Yes, I have some questions about this: I try to model a stack with a pipeline inside made of 4 step (GitHubSourceAction, CodeBuildBuildAction, S3DeployAction and a LambdaInvokeAction): the use case is a Github repo and a github webhook (also managed with CDK) that contains a static website hosted on s3. The pipeline run, build (jekyll build and put artifact in a transition bucket), deploy (unpacking the artifact to an origin bucket) and run a lambda (cloudfront cache invalidation) as a final step. Now I'm encountering some issue in defining the actions: the GitHubSourceAction should be fine. For the CodeBuildBuildAction I'm already encounering some issue: looking at the CodeBuildBuildActionProps interface it seems that only outputArtifactName is required other then CommonCodeBuildActionProps properties. But I cannot init an Artifact object because I don't have visibility of it. Do you have any suggestions about how to work with CodeBuildBuildAction? I'm working with codepipeline 0.28.0 and using already the aws-codepipeline-actions package

Hey @made2591 ,
you need to have the GitHub source Action as a local variable, and use its outputArtifact property. Like this:
const sourceAction = new cpactions.GitHubSourceAction({
    actionName: 'Source',
    owner: 'skinny85',
    repo: 'test-code-build-deploy-spring-boot-project',
    oauthToken: cdk.SecretValue.secretsManager('my-github-token'),
    branch: 'webhook-test',
    outputArtifactName: 'SourceOutput',
});

And then in your CodeBuild Action:
new cpactions.CodeBuildBuildAction({
    actionName: 'CodeBuild',
    project,
    inputArtifact: sourceAction.outputArtifact,
}),

Hope this helps!
Thanks,
Adam

Thank you @skinny85 I will try it right now and let you now!

We decided not to pursue this direction, instead opting to leave the sources and artifacts in the same package.


broken link to webpack info

Description
https://lit-html.polymer-project.org/guide/tools#build-your-project-with-webpack links to https://open-wc.org/building/building-webpack.html#default-configuration but that page doesn't exist (anymore).
Would be great if you could fix that.

We recently changed the docs and decided we want to focus support and our building recommendation on Rollup, and less on webpack, so we removed it from the main docs site. The webpack building docs are still available on github: https://github.com/open-wc/open-wc/tree/master/packages/building-webpack


Duplicate layer when moving or zooming map

Describe the bug
I've created a RasterSource which has as source a WebGLTileLayer containing a GeoTIFF. When I display it on the map using an ImageLayer it seems all ok, but when I move or zoom in/out the map the layer starts to duplicate.
As you can see in the image the correct one is on the right point (bottom right), but all the others spawn and move when I do something on the map.

To Reproduce
Use the following code:
import { GeoTIFF } from 'ol/source';
import RasterSource from 'ol/source/Raster';
import Map from 'ol/Map';
import { WebGLTile as TileLayer } from 'ol/layer';
import ImageLayer from 'ol/layer/Image';
import View from 'ol/View';
import proj4 from 'proj4';
import { register } from 'ol/proj/proj4';
import OSM from 'ol/source/OSM';
import filetif from './COPERNICUS_EUDEM_165252.tif'

// Register projection definition
proj4.defs(
	'EPSG:32633', '+proj=utm +zone=33 +ellps=WGS84 +datum=WGS84 +units=m +no_defs'
);
register(proj4);

const elevation = new GeoTIFF({
	sources: [
		{
			url: filetif,
			projection: 'EPSG:32633',
		},
	],
	normalize: false,
	interpolate: false,
})

const elevationTile = new TileLayer({
	source: elevation,
})

const raster = new RasterSource({
	sources: [elevationTile],
	operationType: "image",
	operation: (inputs, data) => {
		const image = inputs[0];
		return { data: image.data, width: image.width, height: image.height }
	},
});

const dem = new ImageLayer({
	source: raster,
	opacity: 0.6,
})

const osm = new TileLayer({
	visible: true,
	preload: Infinity,
	source: new OSM({ wrapX: true })
})

const map = new Map({
	target: 'map',
	layers: [
		osm,
		dem
	],
	view: new View({
		projection: 'EPSG:32633',
		center: [306821.2, 4879578.4],
		zoom: 13,
	}),
})

Then move or zoom in/out the map
COPERNICUS_EUDEM_165252.zip

Restricting the extent of the Image layer, for example
const dem = new ImageLayer({
  source: raster,
  opacity: 0.6,
});

elevation.getView().then((options) => {
  dem.setExtent(options.extent);
});

seems to fix it as a workaround, but the underlying problem needs investigating.

It works, thanks a lot


Incorporate feedback from alpha test

I included most of the feedback that we got from our alpha test.
Still open

[ ] Write a "Quickstart" (or Getting Started/Introduction) page to the doc. It should explain what cockpit is and crucially what the quantities are and possibly how the whole thing interacts with BackPACK. I only created a placeholder.
[x] I noticed that the UpdateSize quantity is broken. E.g. running example 02 produces update sizes of zero. @f-dangel could you look into this?

What I fixed (or at least partially fixed. @f-dangel could you check them?)

More prominent pip install blocks on the main pages and the examples.
Fix the bug with the utils files of the examples. They are now expected to be in the same directory and are ignored when testing.
Added install instructions for both DeepOBS and BackOBS.
Removed sphinx copybutton (bug with highlighted lines).
Ignored irrelevant torch warning (now all examples (almost) run without warnings from some external package).
Added print message when plotting (thus providing more feedback on what the cockpit is currently doing. For the future, we could wrap all these print statements with a togglable option similar to verbose.
First example now shows a blocking plot at the end (is ignored in testing due to proper switching the MPLBACKEND).
Improved the tick labels for most plots. It is still far from perfect, but it is better.
Made examples 2-3 more interesting (mostly by letting them run longer).

Fixed the bug in UpdateSize with #3

Status (ping @fsschneider):

Double-checked installation and examples from a fresh environment

Had to manually install requests, it's required in torchvision to download data sets, but was somehow not automatically installed in my setting. Nothing we have act on, I think.
Basic and advanced examples work. I also understood how to grab the utility files.

Semi-crucial for the merge/release:

DeepOBS example

The backobs installation does not work with the latest version of pip (21.1). It works with older versions though, but we should update it to work with the latest pip. The following works:
pip install 'git+https://github.com/f-dangel/backobs.git@master#egg=backobs'

I get an ImportError when running the script:
File ".../python3.8/site-packages/backobs/utils.py", line 3, in <module>
from deepobs.pytorch.testproblems import (
ImportError: cannot import name 'cifar10_vgg16' from 'deepobs.pytorch.testproblems' (.../python3.8/site-packages/deepobs/pytorch/testproblems/__init__.py)

I guess this is because the installed DeepOBS version does not (yet) include the cifar10_vgg16 problem. We could fix this by installing DeepOBS from a later commit.

Further suggested improvements (not crucial for merging/releasing):

Basic/Advanced/DeepOBS example: Put reminder to grab the utility files inside a .. note: block. In a rush, this may easily be forgotten, so this should be visually more emphasized.
Advanced example: Add download link of 02_advanced_fmnist.py script like in the basic example.
Advanced example: The utilities save everything to ../logfiles. I dropped the files inside my test folder, but the results were saved outside, i.e. in test/../logfiles, which was a bit annoying, but at least the files are easy to locate because of the messages printed to the terminal. The optimal solution would be to create logfiles relative to 02_advanced_fmnist.py, but I'm not sure how much effort this will take given that the functionality to do so comes from another utility file.
DeepOBS example: Add download link of 03_deepobs.py script like in the basic example.
The pip install ... statement for Cockpit shows up multiple times. Maybe we can reduce duplication, while keeping the statement easy to spot at a first glance.
I like the idea of giving feedback to the user what is happening inside cockpit (especially where plots are saved or files are written). Keep in mind thought that enabling verbose will provide very verbose feedback. For instance, the 2d histogram will be printed to stdout. Maybe we can have multiple verbosity levels, but we only need to think about this if users explicitly request this silencing option because they are disturbed/confused/annoyed by print messages that show where plots and logs are saved.


"First App Open" event not being tacked on iOS

"mixpanel-react-native": "1.3.5",
Only ios has the issue.

hi @billkim84 , the issue should have been fixed in v1.3.7. I'm closing this one now, but feel free to reopen if the problem still exits.


Stream Enrich: enforce a time-to-live on the enrichments

related to #3407 and #3703

Nice idea

Migrated to https://github.com/snowplow/enrich/issues/83


Documenting gulp.task arguments

Could someone please explain the callback fed to gulp.task? I'm not sure if it's supposed to take arguments, what it's supposed to do, or if gulp depends on my plugin/function listening for this callback function to invoke a/sync responses. I was looking at stackoverflow as well as https://github.com/gulpjs/gulp/blob/master/docs/API.md#gulptaskname-deps-fn and couldn't find a sufficient explanation of this information.

The callback can be whatever you want it to be, whether you want to accept arguments or not. Usually the callback consists of what you want that gulp task to do, after it's done with the array of dependency tasks (if any).
I noticed that the documentation for gulp.task() is a little off from the rest of the API documentation. I'll fix that.

@kevdez thanks! Undocumented function arguments can be really confusing.

Closed by #1429


Remove unused imports

I think https://github.com/getodk/central-backend/issues/544 will expose a wider set of dead code that could be removed in the test/ directory.


Two Deposit Block can have same block hash

Issue Type

[x] bug report
[ ] feature request

Current Behavior
Given two identical deposit transaction and deposit twice, they would end up putting in the same block root hash. This make it possible to trick the deposit position for IFE (?)
Expected Behavior
Each block hash should be unique?
Suggested Fix
I don't have one yet.....
Motivation for Change
Avoid bug

@boolafish thinking about this, do you agree with my conclusion.

Just looking at this again. @boolafish do you still have concerns that this is possible?


feat: add use-all-catalogers flag

Description
Add a new switch --use-all-catalogers that allows for the use of both image and file catalogers to run on the subject. This is useful when scanning an image built to use as a build environment, where artifacts may be installed using the files present soon.
Closes https://github.com/anchore/syft/issues/1049

Unnecessary if https://github.com/anchore/syft/pull/1038 gets merged

Added small comment about use of new flag vs extending a newly merged flag

I think extending the existing flag makes a lot of sense. I'll switch over to using that flag and part of the config :+1:


docs: How to use Ionicons in Ionic 2 application?

From @rhuldip on August 10, 2016 15:5
Type: docs
Ionic Version: 2.x
Platform: all
Please provide a documentation to use Ionicons in Ionic 2 applications.

Copied from original issue: driftyco/ionic#7640

This is already covered in the component docs and the API docs
http://ionicframework.com/docs/v2/components/#icons
http://ionicframework.com/docs/v2/api/components/icon/Icon/


Distributed logger

Might end up being its own package, but this would allow logging across instances.

On second thought, this is unnecessary to an extent.

Users could just add a hook to send errors to sentry.io, etc.


Enhancements menu disappeared in MacOS?

What is happening?
Notion Repackaged: No way to access Enhancements Menu and apply mods / enhancements after updating to 2.0.18-1 by replacing folder in Applications.
What should be happening?
I should be able to access such menu with Cmd+E or Option+E or some similar hotkey, and there should be a Notion icon on my taskbar.
Screenshots (if applicable):
N/A
Operating system/browser name & version:
MacOS Monterrey 12.0.1, M1Max Macbook Pro
notion-enhancer and/or notion-repackaged version:
2.0.18-1
notion-enhancer configuration (enabled/disabled mods and their settings):
N/A? Can't see.

please read the warning at the very top of the installation instructions. this is expected if you have not followed the instructions fully.

also, the default hotkey has changed to alt+ctrl+e (even if you're on mac). you can change it in the notion-enhancer settings (core -> menu -> hotkey) after you perform the above fix.


feat(commands): slash command migration

Resolves #1  by implementing slash commands.

Perfect, I'll look into it in couple of hours


watchman send dsk for desktop

Does Andrey's backend account for this? It may throw an error if he hasn't added it to the enum values


Why is it super slow ? the rendering time is too much,

Hi
SLAHMR give excellent result! Probably one of the best if not the best mocap on github, but the rendering time is extremely slow even with an RTX 4090
there is  WHAM that is much faster,
https://github.com/yohanshin/WHAM
but SLAHMR give better mocap results.
I just hope if you can improve the rendering speed because it's truly excessive, a 20s video rendered with WHAM take 1minute  but with SLAHMR at least 10 hours.
Thank you!

It's probably the problem of Pyrender. You should make sure that your Pyrender supports GPU rendering. While the pyrender library hasn't been updated for several years, so you could try to render them utilizing PyTorch3D.


Craft 5 support

Hey there! We've added this to our Craft 4-5 migration project, as people are keen to see this plugin on Craft 5.
If you're no longer developing this plugin, we'd be happy to continue development at Verbb (including updating to Craft 5) - but just let us know if you've got plans to continue things and update to Craft 5!

Thanks for the 2.0.0 release for Craft 5 support!


Sagemaker endpoint doesn't use GPU (instance ml.g4dn.xlarge)

I have spent the whole day trying to deploy a custom HF model to Sagemaker endpoint and making sure it uses the GPU, and I had no luck, hoping to get some insight here.
Here's my script for the model deployment
img_url_old_lib='763104351884.dkr.ecr.us-west-2.amazonaws.com/huggingface-pytorch-inference:1.13.1-transformers4.26.0-gpu-py39-cu117-ubuntu20.04'

huggingface_model = HuggingFaceModel(
    model_data=model_uri,
    role=role,
    source_dir='code',
    entry_point='inference.py',
    name='hf-inference-1-13-gpu',
    image_uri=img_url_old_lib
)

predictor = huggingface_model.deploy(
    initial_instance_count=1, # number of instances
    instance_type='ml.g4dn.xlarge', #has 1 GPU
    endpoint_name='hf-inference-1-13-gpu',
)

And here's my code/inference.py script
device = torch.device("cuda" if torch.cuda.is_available() else "cpu")

def model_fn(model_dir, context=None):
    """
    Load the model for inference
    """

    model_path = os.path.join(model_dir, 'model/')

    processor = TrOCRProcessor.from_pretrained("microsoft/trocr-base-handwritten")
    print("Loaded Processor")

    model = VisionEncoderDecoderModel.from_pretrained(model_path)
    print("Loaded Model")

    model_dict = {'model': model.to(device), 'processor': processor}

    return model_dict

def predict_fn(images, model, context=None):
    """
    Apply model to the incoming request
    """

    images = [Image.open(io.BytesIO(content)) for content in images]
    print("Opened Image")

    processor = model['processor']
    model = model['model']

    pixel_values = processor(images, return_tensors="pt").pixel_values
    pixel_values = pixel_values.to(device)
    generated_ids = model.generate(pixel_values)
    generated_text = processor.batch_decode(generated_ids, skip_special_tokens=True)
    print("Generated Text: " + str(generated_text))

    return generated_text

I've read the threads here and here, and followed the suggestions made by @philschmid , I tried changing the version of the transformers_version arg variable but it still doesn't use the GPU (see pic below). I tested the model in the SM notebook using the same GPU instance(ml.g4dn.xlarge) and I can confirm the inference code does use the GPU as expected. So I'm not sure why when it's deployed to the endpoint with the docker image it doesn't use the GPU.
I'd appreciate any help on this, thanks!

I figured it out, I need the env parameter.
So something like this, and now it works on the GPU!
env = {'HF_TASK': 'image-to-text'}

huggingface_model = HuggingFaceModel(
    model_data=model_uri,
    env=env,
    source_dir='./code',
    entry_point='inference.py',
    role=role,
    name='hf-inference-2-1-gpu-v4',
    image_uri=img_url
)


Text in messages doesn't seem to be limited to page width

Attached image shows text leaving the box that it was created in

Thanks, ill up the aggressiveness of the word-wrap


Command Framework : Hooks Edition

Overview
This is an an attempt to refactor #1701 using a pattern adding logic to a command rather than using a single, replaceable command class.
Patterns:

There is no longer a 1:1 correspondence between the commands to run (liquibase update) and a LiquibaseCommand (UpdateCommand). Instead, LiquibaseCommand register commands they are a part of and give an order to run in, and Liquibase will execute the logic in all LiquibaseCommand instances that have registered for the command string asked.
LiquibaseCommand objects no longer have getter/setter fields on them for properties, but instead define CommandArgumentDefinitions which can be used to type-safely get and set the stored string/object argument values
There is a new CommandScope class (no subclasses) which is what the code trying to call a command uses. They construct that object with the name of the command, add argument values to it, pass it to CommandFactory.execute, and get the results off it.

Questions (see below):

Thoughts so far?
Do we like the "crosscut" pattern for making database connections?

┆Issue is synchronized with this Jira Story by Unito

Thinking out loud: Do we manage database connections from "cross cutting" command logic?
With this "hook" pattern, one thing I've been considering is being able to have a LiquibaseCommand instance not be tied to a particular command name, but handle a variety of them. For example, someone could make an AuditCommand that adds itself to the command chain for every command and sends a record of what was executed off somewhere. That allows cross-cutting logic to be defined in one place and add itself to all the commands, without the other commands needing to know or care about that particular logic.
That cross-cutting support is one of the bigger advantages I see to this "hook" style. It's not in the ticket's "patterns" section yet, since it's not yet implement.
So where I'm at now is I have the HistoryCommand, which takes a "database" argument. Partly that is because that is how the existing code works, but I think that is also the pattern we'd want to follow: command like HistoryService shouldn't be in charge of making the database connections, they should just focus on their logic. Keeps away the duplication, and also because commands will be composed together, like updateCheckingRollback may call the update and rollback commands and we don't want to be re-opening connections.
NOTE: I have some ideas for a new ConnectionManager class vs. the Database we have now, but am keeping the discussion with what we have for code now vs. adding unrelated changes...
Option 1: Cross-Cut
I could leverage the cross-cutting support to make some sort of a OpenConnectionCrosscutCommand which takes arguments of "url, username, password" and creates a database parameter and adds it to the commandScope for the later commands to use. And if the database parameters is already on the commandScope, it would be a no-op. I have some ideas on framework sugar to make this sort of cross-cut easy to write.
The advantage to this is that it would make the command metadata correct. Something like "liquibase update" would show arguments of url, username, and password with datatabase marked as "taken care of". For integrations like the CLI which dynamically expose those arguments and need to have something make the database connections it works well. For integrations like Spring which already has the database connection and doesn't need the url, username, password then the metadata will look a bit off.
Option 2: Integration managed
Alternately, we could have the connection management hard-coded into the integration. Spring would get its connection object passed to it, CLI and maven would add url/username/password arguments. But, the fact that the CLI needs url/username/password would be in CLI-specific code. Same with maven. The Commands themselves would just say they need a database parameter.
The issue will be that for dyanamic integrations like the CLI, it needs to know what commands need a url/username/password argument and which do not and build up the configuration accordingly. Looking for a database argument on the command could be one way to do that, but is that the best approach? We'd also have to figure out how to handle commands that take multiple connections like diff which take a database and a referenceDatabase. Is it just "find all Database arguments, and make url/username/password arguments prefixed with the same prefix on 'database' " ?
Initial Thoughts
While I like the cross-cutting option's ability to leverage the command framework and a more common path, I'm thinking that the fact that the connection management is very much an integration-specific thing makes that code really belong in the integration layer.
For security reasons, it's also probably best to continue to not save the passwords on the commandScope where other parts of the code can grab it.
Thoughts?

I like the idea of being able to create a database connection and make it available to the commandScope. In the titan works, databases are running in containers and the titan context knows how to connect to them. For example, some pseudo workflow like:
liquibase update --sourceDatabase=example

beforeUpdate:
Get connection information for `example` database and construct a connection object that satisfies the requirements for update.
Prepare connection with any additional logic as required by the extension.

runUpdate:
runs update as expected based on connection

afterUpdate:
Inspect or Clean up as needed by the exension.

The above before/after hook example is not titan specific, but a generic approach to satisfy the requirements of whichever command is being extended. As long as the commands exposed their requirements and the extension satisfies them the before and after logic can be very flexible.

Thinking out loud: How do we handle output?
In the previous version, we had a CommandResponse which contained a message field which was supposed to be what gets sent back to the user. It also could contain other data that could be used programmatically when the command is being used in embedded or composed ways. For example, the SnapshotCommand would include the snapshot object on the CommandResponse.
What I was finding with that old style was that:

The commandResult.successful was too easy to forget to check (so moved failures to an exception)
The commandResult.message was too easy to forget to print out
Not all commands have a string-based output
Printing the consolidated message at the end of the command makes the interleaving with log messages wrong.
Some commands have multiple output streams, not a single "message" output
Commands with large output would take up a lot of memory storing it all in the string

I completely removed the CommandResult object in favor of storing programmatic data on the commandScope object.
I've also switched the commands that output data to use the UIService directly rather than the message variable.
The trouble is that it's not as simple as "just use UIService":

updateSQL sometimes needs to send SQL to stdout and messages to stderr, sometimes send both to stdout
When a command is called in an embedded scenario, the command output may not want to be shown at all

We could add a getOutputStream() and getDataStream() type methods to commandScope which the integration can set up as needed. Often times, the output is generated further down in the code where there isn't the commandScope but that could be refactored. Or, maybe a more complex CommandOutput object that goes on commandScope that does more than just those two separate functions?
We could enhance the UIService to handle those cases better, but I'm not sure if that is too generic to have the command-level logic we sometimes need?
I'm also not sure if we should just drop the UIService.sendMessage() type functions anyway in favor of just using the LoggingSystem as the way everything outputs messages. If we did that, maybe it's less that we disable output and more change the log level? Does that have the same "too global" problems that UIService has?

I agree that Option 2 seems like the best choice.  The integrations are just too different in their connection needs.
As for the output, I would vote for keeping the UIService.  I prefer to manage the console output separately from the logging.

I like the idea of an output stream where updates can be sent. I agree that logging should be separate. It makes sense to me that there would be 3 potential outputs: an output stream, a logging stream, or an exception. It's probably that each extension method can throw an exception that can be caught and handled upstream.

I added pseudo-support into the NewMain class to see how a "find Database arguments and make connections for them in the integration" would work.
It seems to be just fine, so going to stick with that approach for now and see how it goes.

Where I struggle with the separated UIService vs. LoggingService is that for most types of ways Liquibase is ran, there is no UI besides the logs. Spring, Maven, Ant, Gradle, and even "cli with --logLevel set" all use the logs as their primary output. It's really only one mode of the CLI that has any sort of distinction.
Because we are managing the UIService and the LogService separately, we are dealing with duplicated messages in the output and/or duplicate lines of code that send something to both the UI and the logs.
Part of our log level definition of "INFO" is "messages that would normally be shown to the user in a console application" which makes me ask "Instead of UIService, what if we just log at INFO level, and configure the CLI by default to show an unformatted INFO level log?"

Trying to add the outputStream, it ended up making the most sense to add it to the CommandScope itself. I tried adding it to the regular Scope and the the UIService but both ended up feeling harder to use and too "global" for something that really is command-specific. Having it on the CommandScope makes it feel like the special-case CommandScope.result that it is.
Having it there will mean that any lower-level functions called by the command will need to take the stream as a parameter, but that is manageable.


Add the Ability to Send Anonymous Events

Is your feature request related to a problem? Please describe.

please add the ability to send anonymous events, which is available on the API anonymous-events

Well, our apps don't require a person to log in to use the app, so a person can use the app anonymously. if the user finds a house/roommate (our app is about finding roommates) suitable for him/her, they are most likely going to signup. we want to use the opportunity to for example send a coupon to the user, based on his anonymous activity.

Thanks for the quick response!  One more detail, for the coupon, how would it be delivered to the anonymous person?

Of course we are going to use the anonymous_id to like that data to person, when he/she sightedUp
Otherwise we could have just used a static identifier like<EMAIL_ADDRESS>for all the anon users

This feature request has been logged internally. Because of that, I will close this issue.
Anyone can feel free to comment on the issue if they have questions or comments.

Hey folks, checking back in on this. This seems like a huge omission for customers who are using the Data Pipelines product. We use the Customer.io CDP to send data to downstream analytics sources (Mixpanel, etc.).
Without the ability to log anonymous profiles and then merge them with identified profiles once a user authenticates, we're unable to measure perhaps the most important metric in our app: App Launch -> Sign In conversion.
Tracking anonymous events and then associating them with a new account once authenticated seems like a table stakes feature of most other analytics platforms.
Did we migrate from Segment to Customer.io Data Pipelines too soon? Is this product not mature enough to support a production use-case?

Any update?

Hey there, @devontivona. Thanks for asking.
Thanks for letting us know the importance of this feature in your project. This request is on our roadmap for early this year, but at this time we can't share more than that in terms of timing. We appreciate the feedback as it helps us to prioritize this work.


Idva5 851 delete old application

Short description outlining key changes/additions
Added endpoint for deleting application
JIRA Ticket Number
IDVA5-851
Type of change

[ ] Bug fix
[ ] New feature
[ ] Breaking change
[ ] Infrastructure change

Pull request checklist

[ ] I have added unit tests for new code that I have added
[ ] I have added/updated functional tests where appropriate
[ ] The code follows our coding standards

An exhaustive list of peer review checks can be read here

CI: No security warnings found


Refresh checkpoints for 1.14.5

Full reindex not yet done, will note here when completed, but opening for visibility.

CI has an issue with the p2p-policy test (has that about 1 in 8 runs so I'm looking into it) - have hit the re-run button.

Verified this syncs past the checkpoints on both test & mainnet

Hashes, timestamps and tx counts match. Waiting for another 1200 blocks on mainnet to be sure it won't get reversed. So far I have no chain tip forks around this across all nodes on mainnet, and only a single orphan on testnet.  Looking good so far.


Reduce scope of CSS transition:

I found that when I dynamically set a property of either .front or .back (in my case the background image and position), these would be animated on page load which I did not want. I traced the cause to this line in the source. The CSS transition is set for all properties on both divs.
This pull request reduces the scope of that transition to only the properties the library is concerned with animating. The rest is left for the user to style and animate as they wish.

Could you show the issue in a jsfiddle and also one for your solution?

I have been wrestling with the animation running on page load as well when I implemented the dynamic size...
@jeshuamaxey Could you please try the latest version? I think it might be fixed... If not, could you have a look and see if your fix still applies to the new version? I will be happy to merge it in.


Write role descriptions for volunteers on FreePROBATE

For developers, this project is mainly a data engineering/machine learning task as we process the printed images to create OCR, parse the OCR to create structured data, then load the results into our multi-modal MyopicVicar code.  As a result, this might be an ideal project for students in data science or data engineering programs.


Correction of the comparator in util.sort()

Function passed to Array.prototype.sort should return number, not boolean, i.e., it should return a negative value if the first argument is smaller than the second, zero if they are equal and a positive value otherwise, see the specification.

Thanks @iamstolis !


I will successfully merge your PR for 50$(Open to see Proof of Work)

I will automerge your PR succesfully for a mere 50$ fee payable in crypto only after i merge the PR!( Since  i am not accepting payment before work consider this as genuine!)
Merging your PR to token-list is a daunting task many. For those who are tired by rigorously creating a PR without any success for the nth time.. I am willing to automerge your PR and get rid of this headache for you all so that you can focus on distributing and providing liquidity to your tokens and other meaningful tasks. Below is the list of tokens that i have succesfully merged in token-list repo:
MetaVersana :https://github.com/solana-labs/token-list/pull/12715
Fone token: https://github.com/solana-labs/token-list/pull/12697
Zen token: https://github.com/solana-labs/token-list/pull/6005
SolSquid: https://github.com/solana-labs/token-list/pull/5798
Gravity: https://github.com/solana-labs/token-list/pull/5794
So this is how its gonna work: You will provide me all the necessary details and assets for your token.  Later when its passed all the checks you will need to make the payment in the next 10 minutes. If you fail to do so, i will close the PR and you wont benefit in any way!
Contact:
Telegram(preferred): @zakimzf
Email<EMAIL_ADDRESS>with the subject "New PR needed for token-list"

that would be great. how do we get started


Rework repository extraction

Need to be tested.
Tested on this repo and on a large repo "microsoft/vscode"

@SebastianZug If there are some problems you can make an Issue. I will merge this now in v2.0.0


image component

possible component we want to copy the functionality of or just straight up use: https://github.com/stereobooster/react-ideal-image
Features

src should extrapolate to srcset (or straight up take in a srcset string[])
aspect ratio
width/height
loading state

 Image Component


env: node\r: No such file or directory

executing:
basel-init myApp result on this message error:
env: node\r: No such file or directory
It seems there is a none unix return somewhere x(
I fix it using:
sudo find /usr/local/lib/node_modules/basel -type f -exec perl -pi -e 's/\r\n|\n|\r/\n/g' {} \;
Using a yosimite in my mac.

Great!
Can You help to fix it in installation?

@acidfilez thanks for your fix!

Getting the same message "env: node\r: No such file or directory" tried @acidfilez fix to no avail. Returns all "Can't open : No such file or directory".
Thanks.

@acidfilez work for me on mac 👍

issue confirmed on Ubuntu

So, i looked in to this...
Seems that in the distributed version on both MacOS and Linux there are non-unix line-endings being used in at least:

bin/basel-crud.js
bin/basel-database.js
bin/basel-init.js

I forked the repo to fix it, but it turned out that the windows line endings where not present here.
This leads me to conclude that it must be an issue with the build tools that placed the package on npm.
Can you rebuild this package and push it to npm again?
This version has been online on npm for a year now and it affects 2 of the 3 available platforms which is quite a huge part of your audience.

Hi @woutervddn
Did you submit a Pull Request?

Try to read my post again, the code in this repo is fine.
During the checkout to your build system the windows line endings where appended. This is due to your personal git configuration on Windows. The npm package that got build and published from there contained the these CR + LF endings where Linux and MacOS only work using LF.
DOS uses carriage return and line feed ("\r\n") as a line ending, which Unix uses just line feed ("\n").
In order to make this work the project own on npm must change his personal git configuration, rebuild and republish to npm
More information about this can be found here:
https://en.wikipedia.org/wiki/Newline

Just had the same problem on Ubuntu?
Have you guys found a way to fix it?


Update list of known breaking changes compared to 4.0-stable

#76647 was just merged and the list is already outdated.
Not adding compat binds here as #76577 has not been merged yet.

Thanks!


[MWPW-156809] add alt text for placeholder image

add alt text for placeholder image

Resolves: MWPW-156809
Test URLs:

Before: https://stage--cc--adobecom.hlx.live/drafts/ruchika/premiere-pro?martech=off
After: https://alt--cc--adobecom.hlx.live/drafts/ruchika/premiere-pro?martech=off

Validation done after the fix :
https://alt--cc--adobecom.hlx.page/drafts/ruchika/premiere-pro

when verified on product page url
https://alt--cc--adobecom.hlx.page/products/premiere the block is not seen loaded ,can you please take a look and advise

Thank you @Ruchika4  , As advised validation is done with mep=off and see the page loaded fine without any errors
https://alt--cc--adobecom.hlx.page/products/premiere?mep=off

Thank you @Ruchika4 , As advised validation is done with mep=off and see the page loaded fine without any errors https://alt--cc--adobecom.hlx.page/products/premiere?mep=off

Thanks @spadmasa . Promo issue is because authors are replacing only marquee using promo manifest hence breaking the interactive experience. I have already shared this with authors and they are aware of selector to be used when using interactive marquee.


BlockingIO Error

Describe the bug
I apologize in advance that this might be a bit annoying to reproduce.
When loading an app like the example given below, some fraction of the time it will fail to run and yield the following type of error (after which point the user has to refresh the app to get things going again):
Exception in callback <bound method Distributor._on_change of <marimo._utils.distributor.Distributor object at 0x7fbf4d755c60>>
handle: <Handle Distributor._on_change>
Traceback (most recent call last):
  File "uvloop/cbhandles.pyx", line 61, in uvloop.loop.Handle._run
  File "/ceph/home/andrew/temp/absel/env/lib/python3.10/site-packages/marimo/_utils/distributor.py", line 54, in _on_change
    response = self.input_connection.recv()
  File "/ceph/home/andrew/miniconda3/envs/test_env/lib/python3.10/multiprocessing/connection.py", line 255, in recv
    buf = self._recv_bytes()
  File "/ceph/home/andrew/miniconda3/envs/test_env/lib/python3.10/multiprocessing/connection.py", line 426, in _recv_bytes
    return self._recv(size)
  File "/ceph/home/andrew/miniconda3/envs/test_env/lib/python3.10/multiprocessing/connection.py", line 384, in _recv
    chunk = read(handle, remaining)
BlockingIOError: [Errno 11] Resource temporarily unavailable
Exception in callback <bound method Distributor._on_change of <marimo._utils.distributor.Distributor object at 0x7fbf4d755c60>>

handle: <Handle Distributor._on_change>
Traceback (most recent call last):
  File "uvloop/cbhandles.pyx", line 61, in uvloop.loop.Handle._run
  File "/ceph/home/andrew/temp/absel/env/lib/python3.10/site-packages/marimo/_utils/distributor.py", line 54, in _on_change
    response = self.input_connection.recv()
  File "/ceph/home/andrew/miniconda3/envs/test_env/lib/python3.10/multiprocessing/connection.py", line 256, in recv
    return _ForkingPickler.loads(buf.getbuffer())
_pickle.UnpicklingError: invalid load key, '\x07'.
Exception in callback <bound method Distributor._on_change of <marimo._utils.distributor.Distributor object at 0x7fbf4d755c60>>
handle: <Handle Distributor._on_change>
Traceback (most recent call last):
  File "uvloop/cbhandles.pyx", line 61, in uvloop.loop.Handle._run
  File "/ceph/home/andrew/temp/absel/env/lib/python3.10/site-packages/marimo/_utils/distributor.py", line 54, in _on_change
    response = self.input_connection.recv()
  File "/ceph/home/andrew/miniconda3/envs/test_env/lib/python3.10/multiprocessing/connection.py", line 255, in recv
    buf = self._recv_bytes()
  File "/ceph/home/andrew/miniconda3/envs/test_env/lib/python3.10/multiprocessing/connection.py", line 426, in _recv_bytes
    return self._recv(size)
  File "/ceph/home/andrew/miniconda3/envs/test_env/lib/python3.10/multiprocessing/connection.py", line 384, in _recv
    chunk = read(handle, remaining)
BlockingIOError: [Errno 11] Resource temporarily unavailable

but other times it loads totally fine.
I initially noticed the error above on a linux server I'm using to deploy the app, but I've also observed something similar locally on my mac but seemingly much less frequently (I had to sit there refreshing for a good bit, whereas is much easier to catch on the server):
Exception in callback <bound method Distributor._on_change of <marimo._utils.distributor.Distributor object at 0x15d9f67a0>>
handle: <Handle Distributor._on_change>
Traceback (most recent call last):
  File "uvloop/cbhandles.pyx", line 61, in uvloop.loop.Handle._run
  File "/Users/andrewhill/Desktop/absel/env/lib/python3.10/site-packages/marimo/_utils/distributor.py", line 54, in _on_change
    response = self.input_connection.recv()
  File "/Users/andrewhill/.pyenv/versions/3.10.4/lib/python3.10/multiprocessing/connection.py", line 255, in recv
    buf = self._recv_bytes()
  File "/Users/andrewhill/.pyenv/versions/3.10.4/lib/python3.10/multiprocessing/connection.py", line 426, in _recv_bytes
    return self._recv(size)
  File "/Users/andrewhill/.pyenv/versions/3.10.4/lib/python3.10/multiprocessing/connection.py", line 384, in _recv
    chunk = read(handle, remaining)
BlockingIOError: [Errno 35] Resource temporarily unavailable
Exception in callback <bound method Distributor._on_change of <marimo._utils.distributor.Distributor object at 0x15d9f67a0>>
handle: <Handle Distributor._on_change>
Traceback (most recent call last):
  File "uvloop/cbhandles.pyx", line 61, in uvloop.loop.Handle._run
  File "/Users/andrewhill/Desktop/absel/env/lib/python3.10/site-packages/marimo/_utils/distributor.py", line 54, in _on_change
    response = self.input_connection.recv()
  File "/Users/andrewhill/.pyenv/versions/3.10.4/lib/python3.10/multiprocessing/connection.py", line 255, in recv
    buf = self._recv_bytes()
  File "/Users/andrewhill/.pyenv/versions/3.10.4/lib/python3.10/multiprocessing/connection.py", line 426, in _recv_bytes
    return self._recv(size)
  File "/Users/andrewhill/.pyenv/versions/3.10.4/lib/python3.10/multiprocessing/connection.py", line 384, in _recv
    chunk = read(handle, remaining)
BlockingIOError: [Errno 35] Resource temporarily unavailable

Environment
Linux
{
  "marimo": "0.6.11",
  "OS": "Linux",
  "OS Version": "6.1.80-1.el9.elrepo.x86_64",
  "Processor": "x86_64",
  "Python Version": "3.10.4",
  "Binaries": {
    "Browser": "--",
    "Node": "--"
  },
  "Requirements": {
    "click": "8.1.7",
    "importlib-resources": "missing",
    "jedi": "0.19.1",
    "markdown": "3.6",
    "pymdown-extensions": "10.8.1",
    "pygments": "2.18.0",
    "tomlkit": "0.12.5",
    "uvicorn": "0.30.1",
    "starlette": "0.37.2",
    "websocket": "missing",
    "typing-extensions": "4.12.1",
    "black": "24.4.2"
  }
}

Mac
{
  "marimo": "0.6.11",
  "OS": "Darwin",
  "OS Version": "23.3.0",
  "Processor": "arm",
  "Python Version": "3.10.4",
  "Binaries": {
    "Browser": "--",
    "Node": "v21.7.1"
  },
  "Requirements": {
    "click": "8.1.3",
    "importlib-resources": "missing",
    "jedi": "0.19.1",
    "markdown": "3.6",
    "pymdown-extensions": "10.8.1",
    "pygments": "2.17.2",
    "tomlkit": "0.12.4",
    "uvicorn": "0.29.0",
    "starlette": "0.37.2",
    "websocket": "missing",
    "typing-extensions": "4.11.0",
    "black": "24.4.2"
  }
}

Code to reproduce
import marimo

__generated_with = "0.6.11"
app = marimo.App(width="full")

@app.cell
def __():
    import pandas as pd
    import numpy as np
    import marimo as mo
    import polars as pl
    import altair as alt
    alt.data_transformers.enable("marimo_csv")

    def create_random_dataset(N):
        x = np.arange(1, N+1)
        y = np.random.rand(N)
        labels = map(str, y)

        return pl.DataFrame({'x': x, 'y': y, 'label': labels})

    # Define the number of points
    N = 200000  # You can change this value to any desired number of points

    def make_plot(n):
        diagonal = (
            alt.Chart(pd.DataFrame({"x": [1, 10000], "y": [1, 10000]}))
            .mark_line(color="red", strokeDash=[10, 10])
            .encode(x="x", y="y")
        )

        dataset = create_random_dataset(n)
        return diagonal + (
            alt.Chart(dataset.to_pandas())
            .mark_point()
            .encode(
                x=alt.X(
                    f"x:Q",
                    title=f"log10(UMI + 1) for",
                    scale=alt.Scale(type="log", base=10),
                ),
                y=alt.Y(
                    f"y:Q",
                    title=f"log10(UMI + 1) for",
                    scale=alt.Scale(type="log", base=10),
                ),
                tooltip=["x", "y", "label"],
            )
        )

    # Display the plots
    interface_elements = [mo.md("#My App"), mo.accordion({"Help": "Help text"})]
    interface_elements.append(mo.hstack([make_plot(N) for _ in range(0, 3)]))
    interface_elements.append(mo.hstack([make_plot(N) for _ in range(0, 3)]))
    mo.vstack(interface_elements)

    return (
        N,
        alt,
        create_random_dataset,
        interface_elements,
        make_plot,
        mo,
        np,
        pd,
        pl,
    )

if __name__ == "__main__":
    app.run()

Note that increasing the number of points also seems to increase the chances that this will happen, which is why I've set it to a high value here.

Also the times I've caught this have all been when running the app with marimo run and refreshing, etc.

Thanks everyone for documenting this issue.
PR #1822 is an attempt to mitigate this issue by just retrying the socket recv() after a short wait.
Re @ross-at-finix's hypothesis: we do use a TCP socket to facilitate communication between the kernel process and the server. Right now that socket is created and managed by a multiprocessing.connection.Listener object, which as far as I can tell doesn't allow for increasing the socket buffer size. Perhaps we should just with a socket directly.
More context: In marimo run mode, the kernel(s) and server are actually in the same process, so in theory we could just use a simpler communication method in run mode to get around this problem. The socket would still be needed for edit mode, during which the kernel is run in a separate process (so that its execution can be easily interrupted).

Thanks @akshayka ; I've done a very surface level check with our app by building that PR and using it in our QA deployment and the standard table elements are working great.

That's great, thank you for checking! Version 0.7.8 includes the fix, and should be available on PyPI soon.

after upgrading to  Version 0.7.8, I am facing same issue but with different error.
Exception in callback <bound method Distributor._on_change of <marimo._utils.distributor.Distributor object at 0x7f7728327090>>
handle:
Traceback (most recent call last):
File "uvloop/cbhandles.pyx", line 61, in uvloop.loop.Handle._run
File "/root/.cache/pypoetry/virtualenvs/justask-4h5SbXZT-py3.11/lib/python3.11/site-packages/marimo/_utils/distributor.py", line 56, in _on_change
response = self.input_connection.recv()
^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/multiprocessing/connection.py", line 250, in recv
return _ForkingPickler.loads(buf.getbuffer())
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
_pickle.UnpicklingError: invalid load key, '\x07'.
[W 240719 11:19:59 distributor:59] BlockingIOError in distributor receive: [Errno 11] Resource temporarily unavailable

I have the same error as @Sonali-bapte with version 0.8.12, uvicorn version 0.30.6

just to update, I also pretty regularly see the @Sonali-bapte mentioned

@andrewhill157 @TedSinger @Sonali-bapte can you share more context of when you see this error?
@andrewhill157 does your original reproduction still surface this issue?

The example I provided above doesn't seem to reproduce the issue reliably for me at this point (current marimo release) in either my linux or mac environments. I have different app that is more intensive but tricky to share that regularly produces the same error message as @Sonali-bapte mentioned above (this was the app that led me to try and make the simpler example in the first place). It looks like you have a PR in progress, but let me know if trying to boil it down to something simpler and usable on your end would still be helpful

Thanks for writing back @andrewhill157.
I've merged a fix, and can release it later today. It solves the issue by using an in-memory queue instead of a socket, which is not needed for run mode. So there's no pickling or TCP connection involved.

Version 0.8.19 is available on PyPI and contains the fix.

Thank you, much appreciated! Seems much more reliable for the app I mentioned so far


DOM Blocking/Locking

I was trying to put in place a status bar during a long operation that inserts records to the DB. The idea is that prior to each add I would update the x of y value which could easily be a progress bar then too. However I'm noticing that the HTML wont update until after all of the transactions are done. I'm not clear on exactly what's  blocking the DOM from updating. I note that I can log the value to the console and watch it tick up as it goes.
Any ideas are very helpful, I notice that this does prevent things from occurring on the page as well other than the loading that I was trying to accomplish. It's minor, but a load of 10 seconds can be noticeable as the recordset grows >10,000 rows.
Here's some code as an example of how that would work via an ajax call where dbW is my dexie instantiated database:
function getInventory(){
    if(gettingInventory == 1) return;
gettingInventory=1;
dbW.inventory.clear().then( function(){
    return new Dexie.Promise(function (resolve, reject) {
		$.ajax('/Transactions/sync/getInventory.cfm', {
			type: 'get',
			dataType: "json",
			error: function (xhr, textStatus) {
				reject(textStatus);
				gettingInventory=0;
			},
			success: function (data) {
				// Resolving Promise will launch then() below.
				//alert(data);
				resolve(data);
			}
		});
	}).then(function (data) {
		return dbW.transaction('rw', dbW.inventory, function () {
			data=queryToObject(data);
			var invLoadCount=0
			data.forEach(function (item) {
				invLoadCount++;
				$('#invLoadStatus').text(invLoadCount + ' of ' + data.length);
				console.log(invLoadCount + ' of ' + data.length)
				dbW.inventory.add(item);
			});
		});
	}).then(function () {
		console.log ("GetInventory Sync committed");
		dbW.inventory.count(function(val){$('#inventoryCount').html(val)});
		gettingInventory=0;
		$('#inventoryTime').html(curTime())
	});

});

}

As blogged about by @nolanlawson, this is an issue in some browsers but not others (http://nolanlawson.com/2015/09/29/indexeddb-websql-localstorage-what-blocks-the-dom/). Can be worked around by using indexedDB from a WebWorker.

I'm unsure about how to integrate the syncs with a web worker exactly but will look into it more. Generally I have the sync functions updating a few dom elements and my understanding is that the web worker based functions then cant access the dom, just the indexeddb...

You will need to communicate from the WebWorker to the Dom through postMessage ()

@tolian49 These days I recommend using IndexedDB inside of a WebWorker 100% of the time because of this issue. Dexie, in my tests, works perfectly inside of a WW. :)
Web Workers are dead easy to work with; here's an example: http://bl.ocks.org/nolanlawson/28eebac3b91feeeabc6e

Thanks for the info! I've had success with setting it up as a worker
process and youre right it doesnt lock now. Many thanks.
On Thu, Feb 18, 2016 at 9:56 AM, Nolan Lawson<EMAIL_ADDRESS>wrote:

@tolian49 https://github.com/tolian49 These days I recommend using
IndexedDB inside of a WebWorker 100% of the time because of this issue.
Dexie, in my tests, works perfectly inside of a WW. :)
Web Workers are dead easy to work with; here's an example:
http://bl.ocks.org/nolanlawson/28eebac3b91feeeabc6e
—
Reply to this email directly or view it on GitHub
https://github.com/dfahlander/Dexie.js/issues/180#issuecomment-185788642
.


Stop making .menu files

The fontbakery-build.py process is still creating fontFileNameStem + ".menu" files.
These are no longer needed.

Looks good


Refactor: Single-Module Wrappers By Default

closes: https://github.com/polywrap/monorepo/issues/886 (and all sub-tasks)
Checklist
Schema

[x] Parse
[x] Bind
[x] Compose

Client

[x] Manifest
[x] Core
[x] Client
[ ] Plugins

Integration

[x] Test APIs
[ ] CLI
[ ] Templates

lmao @ the lines of code being removed here :fire:


my pull request for hacktoberfest

here i have submitted the fizzbuzz code in c

Thank you for your PR!
Unfortunately a solution similar to yours has already been submitted :/
The goal is to find creative/abstract new solutions to FizzBuzz.
Please consult the readme for more information and see The top 5 samples here.
Thanks anyway! :)


environment variables are not available within my react app

I'm trying to use environment variables as documented in the user guide, but for some reason none of my variables are available within my app.
Adding Environment Variables
When I console out process.env, the only thing that shows up is NODE_ENV and PUBLIC_URL. My variables.env looks like the following:
REACT_APP_SECRET=mySecreT
REACT_APP_DOES_IT_WORK=hello
REACT_APP_DOESNT_LOOK_LIKE=blah
REACT_APP_IT=fail
REACT_APP_TIME_TO_SUBMIT_ISSUE=okay

I'm using the version of create-my-app that's using TypeScript. Another thing worth mentioning, is that this is not an isomorphic project. I'm using the frontend by itself.
System Specs:
OSX Sierra
Node: v7.10.0
NPM: 4.6.1

@MaxOrelus it might have to do with the fact that you named your .env file variables.env.

.env: Default.
.env.local: Local overrides. This file is loaded for all environments except test.
.env.development, .env.test, .env.production: Environment-specific settings.
.env.development.local, .env.test.local, .env.production.local: Local overrides of environment-specific settings.

Ahhh, I see. Thanks that did the trick. The documentation might need to be updated, because just from reading it I did not infer you that the file name had to start with .env. I was under the impression that the filename had to end in any of those prefixes.
variables.env
variables.env.local
variables.env.production

Regardless, thank you for helping me solve this 👍

Want to send a PR to clarify the docs? Thanks.


workspaces are not showing in Hyprland

Hi,
Since the last update to Waybar today. Workspaces seemed to be broken as they wont display.

my configuration
config
"modules-left": [
"custom/launcher",
"wlr/workspaces",
"cpu",
"memory"
],
Module:
"wlr/workspaces": {
"all-outputs": false,
"sort-by-number": true,
"on-click": "activate",
"on-scroll-up": "hyprctl dispatch workspace e+1",
"on-scroll-down": "hyprctl dispatch workspace e-1",
"format": "{icon}",
"format-icons": {
"urgent": "",
"active": "",
"default": ""
},
"persistent_workspaces": {
"1": "",
"2": "",
"3": "",
"4": "",
"5": ""
}
},

wlr/workspaces isn't supported in latest Hyprland. Need to use hyprland/workspaces.


[Next.js] [EE] Finalize editing secret following JSS.Server implementation

This is a follow-up to #529, which finalizes the editing secret now that JSS.Server implementation is complete.
Updated names (SITECORE_SECURITY_TOKEN > JSS_EDITING_SECRET and SecurityToken > JssEditingSecret) and added doc comments.

@nickwesselman Could you please take a look at the .env and JssNextWeb.config changes and confirm these make sense (from product standpoint / container setup / secret complexity)? You can ignore the rest of the updates as these are just renames.

@nickwesselman Could you please take a look at the .env and JssNextWeb.config changes and confirm these make sense (from product standpoint / container setup / secret complexity)? You can ignore the rest of the updates as these are just renames.

@ambrauer I'm a little confused about the difference in names between the two env vars, I guess there are different conventions in platform vs JSS though? Suggested the SITECORE_ prefix for the server since all the other env vars there use it. Better to be consistent between the two environments, or with other env vars used in JSS?

Yeah, I was struggling with that. I'm not sure there are any (as of yet) shared between server and client. Personally I think it'd be just as confusing if they did match (since they really are independent, I could see cases of "I SET THIS ALREADY!"). I lean toward consistency with other env vars in Node / JSS (we've got things like JSS_APP_NAME so far). And I thought this lined up nicely for non-container environments who are just setting the <setting "JavaScriptServices.ViewEngine.Http.JssEditingSecret"> directly.

@ambrauer I'm a little confused about the difference in names between the two env vars, I guess there are different conventions in platform vs JSS though? Suggested the SITECORE_ prefix for the server since all the other env vars there use it. Better to be consistent between the two environments, or with other env vars used in JSS?

Yeah, I was struggling with that. I'm not sure there are any (as of yet) shared between server and client. Personally I think it'd be just as confusing if they did match (since they really are independent, I could see cases of "I SET THIS ALREADY!"). I lean toward consistency with other env vars in Node / JSS (we've got things like JSS_APP_NAME so far). And I thought this lined up nicely for non-container environments who are just setting the <setting "JavaScriptServices.ViewEngine.Http.JssEditingSecret"> directly.


coveralls code coverage

Would it be useful to have a code coverage badge for the project?
There is a tutorial here to enable it in CMake:
http://gronlier.fr/blog/2015/01/adding-code-coverage-to-your-c-project/
and another
https://github.com/JoakimSoderberg/coveralls-cmake
The only downside would be if we had really low coverage...

Would it be useful to have a code coverage badge for the project?

Code coverage is always useful, badge or no, but would take some time to accurately generate. Each platform & build config's tests would need to have coverage results merged in the CI pipeline. We don't even have all of the configs building in CI yet, so that would have to come first before we could generate an accurate number.
Also macro expansion is a challenge for some coverage tools and might make this more difficult.

The only downside would be if we had really low coverage...

While I doubt this is an issue for UnitTest++, I don't see a downside to publishing a badge with low coverage as long as the number is accurate. Proactively communicating information about a project's health is part of what makes it a good community project.


Cannot get php-worker container to work

I get an error when i put this code in my docker-compose.yml file:
php-worker:
      build:
        context: ./php-fpm
        dockerfile: Dockerfile-70 # or Dockerfile-56, choose your PHP-FPM container setting
      volumes_from:
        - applications
      command: php artisan queue:work

And the error i get:
PHP Warning:  PHP Startup: Unable to load dynamic library '/usr/local/lib/php/extensions/no-debug-non-zts-20160303/sqlsrv.so' - /usr/local/lib/php/extensions/no-debug-non-zts-20160303/sqlsrv.so: cannot open shared object file: No such file or directory in Unknown on line 0

PHP Warning:  PHP Startup: Unable to load dynamic library '/usr/local/lib/php/extensions/no-debug-non-zts-20160303/pdo_sqlsrv.so' - /usr/local/lib/php/extensions/no-debug-non-zts-20160303/pdo_sqlsrv.so: cannot open shared object file: No such file or directory in Unknown on line 0

It seems it cannot load the extensions sqlsrv and pdo_sqlsrv. Anyone know how to fix this issue?

I know, I find that annoying too,... but, that's just a warning. It shouldn't be causing the container not to work. Also, have you created this Dockerfile-70 in the php-worker directory?
After you docker-compose up -d php-worker ...
can you do a docker-compose ps and see if the php-worker container is up?

The container is now running, but the command: php artisan queue:work doesn't work. It just returns the warnings that I mentioned before.


feat(router): update app with router using react-router-dom

Description
Please describe the reason related to the changes:
Type
Please select the type related to the changes:

[ ] bugfix
[ ] build
[ ] continuous delivery (CD)
[ ] continuous integration (CI)
[ ] documentation
[ ] feature
[ ] formatting
[ ] refactoring
[ ] styling
[ ] other (please describe below):

Breaking Change
Does your changes introduce a breaking change?

[ ] no
[ ] yes

Behavior
Please describe the current behavior:
Changes
Please describe the new behavior:
Information (issue, links, logs, etc.)
Please list any other useful information:
Checklist
Please make sure this pull request follow the requirements

[ ] commit guidelines @commitlint/config-conventional
[ ] 100% code coverage
[ ] tests passing

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

33 of 35 (94.29%)  changed or added relevant lines in 10 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-4.7%) to 95.349%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/app/src/pages/about.tsx
2
3
66.67%

packages/app/src/pages/lost.tsx
2
3
66.67%

Totals

Change from base Build<PHONE_NUMBER>:
-4.7%

Covered Lines:
41

Relevant Lines:
43

💛  - Coveralls


refactor: use block.timestamp instead of block.number

🤖 Linear
We probably need to update events as well.
Closes OPO-647

Tried bumping husky to fix a weird error I'm getting in prophet-modules when I use the dependency@defi-wonderland/prophet-core#de18a05 but husky v9 dropped support Node 14 and 16.


PR new setup

An attempt at a more streamlined setup.
The new setup should work by running pip install -r requirements.txt -e .[develop], where all needed packages should be installed.
The setup has only been tested locally as hep is very slow today (atleast for me).
Currently, the new setup needs to be tested on hep:

[ ] installation from scratch, new python (not sure of version, but higher than 3.8), no packages
[ ] run examples/train_model.py

Thanks for the PR, @MortenHolmRep! I am trying to run the two checks above. Will drop a few comments on the PR in the meantime. :)

Thank you, my python 3.10.4 download failed on hep twice, at ~90% done.


[WIP] Enhance cover section by enabling user to change bg color

やりたいこと
ユーザーが自在にバックグラウンドの背景の色を変更できる様にする。
#3 で問い合わせた内容の対応
PR #3 で問い合わせた内容の返事

Hey YassLab,
I think I described it a bit short. Lets assume you have in your model the following attribute:
attribute: background_color_select, :enum, values: ["red","green","blue"], default: "red"
Then in your details view you would offer this attribute for editing as follows:
scrivito_tag(:div, widget, :background_color_select)
Finally in your widgets view you would have something along the following lines:
 ...
This way the background colour of your widget is changeable by your used.
Bartek from Scrivito
PS: Did you know that there's a one-hour Google Hangout with our developers called "Ask Scrivito" every Monday, starting at 4:00 pm CET (Berlin Time) / 10:00 am EST?
https://plus.google.com/hangouts/_/infopark.de/ask-scrivito

やること

[ ] Codeが実際に動くか確認する。
[ ] ユーザー側の入力の誤認を対応できる様にする。

こっちも解決しているので閉じちゃいます! ✅


i18n(de): add solid integration guide

Description (required)
supersedes #8148
This PR adds the missing german translation for the SolidJS integration guide
I checked the original docs again and added what was missing

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

de
guides/integrations-guide/solid-js.mdx
Localization added, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

oh, did not see that this is still open...
@Maxframe I don't have any more questions here, so this should be good to go


Upgrade and type

This pull request supersedes #121 .
Resolves #119
Closes #121

I may have bitten off more than I can chew with this PR. I don't know enough TypeScript to get this over the finish line.
Unless there are any serious objections I'd like to take a crack at this without TypeScript. I believe this effort ought to be done in a separate PR, perhaps by someone who understands it better.

I may have bitten off more than I can chew with this PR. I don't know enough TypeScript to get this over the finish line.
Unless there are any serious objections I'd like to take a crack at this without TypeScript. I believe this effort ought to be done in a separate PR, perhaps by someone who understands it better.
cc @igorT

What's the current blocker regarding typescript? I haven't glanced at the whole PR, but looks like tests are passing.

@Samsinite full disclaimer: I'm fairly certain the problems I'm facing are due to my inexperience with TS.
Let's try to resolve this:
https://github.com/ember-data/active-model-adapter/pull/121#discussion_r436348998
/cc @igorT
This is exported by @ember-data/store as a private method (maybe this should be public @igorT ????). However...

I hit a wall at this point. Resolving a few other Ember Data related imports results in similar challenges.

@igorT did you ever resolve https://github.com/ember-data/active-model-adapter/pull/121#discussion_r436348978 ?

Outside of the above I think this is good to go.

@jherdman Thanks for pushing work forward on this project! FWIW I just tested the branch associated with this PR in my company's app and it has worked thus far in my testing. The current version of the add-on stopped working in our app in ember-data 3.24 (app worked fine with ember-data 3.23). I went through the diffs between 3.23 and 3.24 and tried reverting some of the changes without any luck. Swapping your PR branch in brought everything back to life!

Looking at the PR. We discussed moving this addon into adopted addons in the data meeting today, and have the 👍 . @runspired is going to coordinate the move.

@jherdman you might have to update your remote's url once this moves, move is occurring :)

I think any additional fixes should be covered after this lands as it will be easier to review them. I've gone over the structural changes and ensured the key pieces remain for each thing and any deletions are sensible deletions.

Given CI is failing I propose:

I remove the Travis CI config completely in this PR, and update badges accordingly
Ticket remaining changes
Merge as is
Open PRs to resolve any remaining issues

Thoughts?

It looks like the tests pass and then the job exits with 1 for some reason...
🤔
Maybe try triggering it again?
# tests 33
# pass  33
# skip  0
# todo  0
# fail  0
# ok
The command "yarn test:ember" exited with 0.
cache.2
store build cache
Done. Your build exited with 1.

If anyone wants to help with the type errors that'd be super. I'm completely lost in this mess at this time.
Personally I think this PR has grown out of control and typing should be done at another time </rant>

Seems we've gotten all the type errors resolved. @MelSumner what are the next steps for getting this accepted and released as a new major?

I'll look into these fails ASAP.

And we're green! @MelSumner — I'd love to merge this then do a quick documentation pass (#146) before a major.


Revert "[Hub Generated] Publish private branch 'dev-maintenance-Microsoft.Maintenance-2023-04-01'"

Reverts Azure/azure-rest-api-specs#23832

ignore the breaking changes since we are reverting  a PR


Extract html into separate module

This should satisfy splitting out for yo-yoify support.
Another 3.0 roadmap fix (see #42)
Fixes #3
Fixes #60

This is the best :tada::tada::tada::tada:


feat: add multiple additive scenes feature

This will internally calculate if the server should load another additive scene, or only load once.

merged here https://github.com/MirageNet/Mirage/commit/f39414be4f072981dd75612ef526cd0eb8aa3f91


Usage together with json_serializable

I am currently in an updating session for one of our projects which also included updating grapqhl_codegen from json_serializable times (0.10.x) to the current 0.13.4. Unfortunately, this broke two instances where we are using a generated type in a new class that's tagged with @JsonSerializable(). Using in this case means either of:

Extending a generated type (this is also not possible anymore because the generated constructor is a factory constructor which may not be extended; I can open a new issue for this if needed).

@JsonSerializable()
class Example extends Query$something {
  Example({required super.id});

  factory Example.fromJson(Map<String, dynamic> json) =>_$ExampleFromJson(json);

  @override
  Map<String, dynamic> toJson() => _$ExampleToJson(this);
}

This fails with:
Cannot populate the required constructor argument: id.
package:example/example.dart:9:3
  ╷
9 │   Example({required super.id});
  │   ^^^^^^^^^^^^

Using a generated type in a field

@JsonSerializable()
class Example {
  final Query$something? value;

  Example({required this.value});

  factory Example.fromJson(Map<String, dynamic> json) =>_$ExampleFromJson(json);

  @override
  Map<String, dynamic> toJson() => _$ExampleToJson(this);
}

This fails with:
Could not generate `fromJson` code for `value`.
To support the type `InvalidType` you can:
* Use `JsonConverter`
  https://pub.dev/documentation/json_annotation/latest/json_annotation/JsonConverter-class.html
* Use `JsonKey` fields `fromJson` and `toJson`
  https://pub.dev/documentation/json_annotation/latest/json_annotation/JsonKey/fromJson.html
  https://pub.dev/documentation/json_annotation/latest/json_annotation/JsonKey/toJson.html
package:nebolus/common/entities/fuckery.dart:10:46
   ╷
10 │   final Query$something? value;
   │                                              ^^^^^
   ╵

The error persists, even when adding
valueToJson(Query$something value) => value.toJson();
// snip

  @JsonKey(
    fromJson: Query$something.fromJson,
    toJson: valueToJson,
  )

Now I am curious: Is there any way to avoid these problems? I don't know if this is connected to the order of the runners perhaps but I also tried adding runs_before: ["json_serializable", "json_serializable:json_serializable"] to the build options for graphql_codegen in my build.yaml but to no avaail.
One option would obviously be to keep the older version around but there are so many things attached to it that would be held back as well that I don't really consider this a good option.

Thanks for creating this issue @mvarendorff. Honestly, I have no idea why the second example doesn't work or if this is caused by the order of the builders.
I am curious, what kind of benefits are you getting from extending the query class?

That is a very valid question! Turns out... None, really. The snippet that did it was written many years ago without much experience and there wasn't anything there that couldn't be handled by an extension instead. I believe the original idea was to enrich the object from the server with additional data from the app and then write that to disk to be retrieved later. In our case that data was also included in the server object already, so there wasn't really a point in doing what we were doing but if that wasn't the case, it would probably make more sense to store them separately and join them in code rather than in storage.

As for the other, I did some digging and testing and trying. It turns out, the other was indeed the culprit as json_serializable by default appears to run before graphql_codegen.
Adding
    runs_before:
      - json_serializable:json_serializable

to graphql_codegen's build.yaml and removing the custom @JsonKey made things work. I opened a PR at #299 which in my tests solves both portions of the issue described originally (eventhough the former has been worked around otherwise).

Thanks for submitting a fix!


Intuit python version for compatible Ray image

Issue link

What changes have been made

Verification steps

Checks

[ ] I've made sure the tests are passing.
Testing Strategy

[ ] Unit tests
[ ] Manual tests
[ ] Testing is not required for this change

/hold require the SHA for ray py3.11


Add automatic deploy from Github

Flynn is awesome.
It will great for adding automatic deploy from a chosen branch of GitHub. Every push to the chosen branch will trigger a new deployment. For a reference, Heroku already has this feature Automatic deploys.

@jxltom perhaps you want this?


Add parameter for increasing max characters when using bulk mode

When using bulk mode it leverages com.univocity.parsers.csv.CsvParser class however it has a default limit of 4096 for each column, it can be changed by using the method setMaxCharsPerColumn on the class CsvParserSettings. Exposing the setting as a parameter would alleviate this issue.

Link to PR -> https://github.com/springml/spark-salesforce/pull/46


DROOLS-1272 document Rule Execution Modes - no doc exists for fireUnt…

…ilHalt
My early draft : @mariofusco @etirelli @mdproctor what do you think, please?

Build successful! See generated HTML docs (may take a while till they are available):
http://ci.kiegroup.org/job/kie-docs-pullrequests/172/artifact/drools-docs/target/generated-docs/html_single/index.html
http://ci.kiegroup.org/job/kie-docs-pullrequests/172/artifact/jbpm-docs/target/generated-docs/html_single/index.html
http://ci.kiegroup.org/job/kie-docs-pullrequests/172/artifact/optaplanner-wb-es-docs//target/generated-docs/html_single/index.html

Implemented Mark's feedback except this only comment still missing to be actioned just yet.

Build successful! See generated HTML docs (may take a while till they are available):
http://ci.kiegroup.org/job/kie-docs-pullrequests/174/artifact/drools-docs/target/generated-docs/html_single/index.html
http://ci.kiegroup.org/job/kie-docs-pullrequests/174/artifact/jbpm-docs/target/generated-docs/html_single/index.html
http://ci.kiegroup.org/job/kie-docs-pullrequests/174/artifact/optaplanner-wb-es-docs//target/generated-docs/html_single/index.html

+1

Build successful! See generated HTML docs (may take a while till they are available):
http://ci.kiegroup.org/job/kie-docs-pullrequests/176/artifact/drools-docs/target/generated-docs/html_single/index.html
http://ci.kiegroup.org/job/kie-docs-pullrequests/176/artifact/jbpm-docs/target/generated-docs/html_single/index.html
http://ci.kiegroup.org/job/kie-docs-pullrequests/176/artifact/optaplanner-wb-es-docs//target/generated-docs/html_single/index.html


DropwizardAppRule use of ConfigOverride changes default config

I have an integration test that needs to emulate 2 pods in a cluster. Since I can't change the IP, I'm changing the port using ConfigOverride passed to the DropwizardAppRule like this:
      private fun createPodRule(vararg overrides: ConfigOverride): DropwizardAppRule<ProcessorAPIConfiguration> {
            return DropwizardAppRule<ProcessorAPIConfiguration>(
                    IntegrationTestApplication::class.java,
                    "etc/processor.yaml", *(configOverrides + overrides)
            )
        }

For my first pod, I just use the defaults and don't pass in any overrides, but for my second pod, I override the application and admin ports.
What I'm expecting to happen is that the first pod uses the ports from the yaml file, but what ends up happening is that both rules use the override ports
If I explicitly override the ports for both rules, than everything works as expected.

Can't repro now.