Possible performance regression in v5

Hi, this is just a heads up. We tried upgrading Twitter Lite to use v5 but noticed that Tweet update timings increased significantly after we deployed to production. Reverting to v4 caused those timings to fall back to the baseline level. We haven't looked into the cause yet.
/cc @paularmstrong

Really.  That's... disturbing.
The benchmarks in #407 and #416 , and other testing, indicate that v5 should be faster for almost all use cases.  I think there were one or two exceptions where v4 was faster - think the discussion/examples were in https://www.reddit.com/r/reactjs/comments/5hf4d4/an_artificial_example_where_mobx_really_shines/ and https://github.com/dtinth/pixelpaint/pull/1 .
Any info you can provide would be greatly appreciated.

I'd like to help with this if you can provide more info.

I'm looking at narrowing down the cause, but it may take a long time and it's not really a priority at the moment. We only have some perf timings around Timelines and Tweets for "time from construct to mount" and "time from willUpdate to didUpdate". This is the graph of p95 and p50 for update timings:

(Ignore the noise toward the right on p95, that happens periodically) Please keep in mind that this could be something specific with our setup as well, though we're not doing anything special other than using connect with reselect selectors across many components.

Any further info you can offer on these performance regressions?

If @necolas or @paularmstrong could offer a flame graph of the app under 5.0, that would be helpful.

@necolas Have you had a chance to look at this again yet?

No, not yet

Okay, I've finally gotten some things together that are able to show this.
I've created a component that will run 50 updates to a tree and estimate the amount of time each update takes. I did this because I want to be able to do this in a production build of React and not need to rely on react-addons for performance measurements. The implementation of this shouldn't matter at the moment, but I hope to decouple it a bit and open source it in the next week or so.
Specs

System: Mac OS X
Chrome: 60.0.3112.113
CPU: 3.1GHz Intel Core i7 (simulated 10x slowdown in Chrome)
Packages:

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

Note: These timings are much more apparent on mobile browsers, but they still show up on desktop Chrome enough that these recordings should be good enough
React Redux v4.4.8
Chrome DevTools Timeline

React Redux v5.0.6
Chrome DevTools Timeline

Component Tree
The following is the connected component tree that I'm rendering in the above recordings.
import { connect } from 'react-redux';
import { createSelector } from 'reselect';
import React, { Component } from 'react';

const exampleMapStateToProps = createSelector(
  (state, props) => 'foobar',
  (foo) => ({ foo })
);

const foobar = () => {};
const exampleMapDispatchToProps = { foobar };

class Internal extends Component {
  render() {
    return
barfoo
;
  }
}

class InternalContainer extends Component {
  render() {
    return <Internal />;
  }
}

const InternalContainerConnected = connect(exampleMapStateToProps, exampleMapDispatchToProps)(
  InternalContainer
);

class Example extends Component {
  render() {
    return <InternalContainerConnected />;
  }
}

class ExampleContainer extends Component {
  render() {
    return <Example />;
  }
}

export default connect(exampleMapStateToProps, exampleMapDispatchToProps)(ExampleContainer);

Unfortunately haven't had time to swing back and look at this myself.  @jimbolla , you had a whole bunch of benchmarks when we did the v5 rewrite.  Any ideas on how this scenario compares so differently to those?

@cellog I used the source that's forked from: https://github.com/paularmstrong/react-component-benchmark

@paularmstrong I need more info. What is the createStore and so on look like? How are you setting up the default run of the benchmark? What props are passed to the sample component? What exactly are you trying to measure? I can't reproduce a benchmark that shows any difference in versions

@cellog I'm trying to create some reduced test cases in codesandbox, but suddenly not able to see the issue anymore. I'll need a few days to test this out at Twitter as well to see if perhaps the issue is mysteriously gone.

really appreciate the work, thank you

I know this issue is closed, but I see that Twitter is still using v4.
I'd love to know if v7 is an improvement!


Fix crash while applying bigmap diffs with complex keys

 Replace paths with ids

 Failed to apply big map diffs


Use a re-exported serde_json dependency in macros

Hello, here is a proposal to avoid shadow-depending on serde_json when using derives.
Fixes #1172

Just one thing, a CHANGELOG.md entry for the change would be good to add in utoipa/CHANGELOG.mdand utoipa-gen/CHANGELOG.md. Newest entry to the top and should be under the Unreleased and Changed title.


Feedback - Some suggestions

Hi @passivestar...
Thank you for this awesome theme, it's great, but I think can be improved a little bit because in some parts of the UI it's too flat, or in this case, too minimal, I don't known if it's intended 😂
Here are 2 screenshots with the default theme and other 2 with the minimal theme for comparison... Basically in the FileSystem tree, the arrow paths are missing, also some "containers" have no borders and can be a little confusing because everything looks like a label and not a clickable resource (Like the Sprite 2D in the create new node modal, below Recents label)
I hope you can consider this feedback 😄
BTW, the screenshots are for Godot 4.3 beta 2 and minimal theme high ppi v1.5

I don't known if it's intended 😂

It's not, it's just what we have to deal with considering godot's current theming limitations

Godot theming still lacks a lot, I'd very much like to have clear visible separation for those containers but since they affect all docks I had to make a choice. Fixing this will require a big amount of work on the engine side, probably something to explore in 4.4

the arrow paths are missing

some people mentioned that, I might bring those back but I'd like to see what it'll look like with just vertical lines, like it's done in vscode. might be a good compromise on minimalism/function
Thanks for the feedback! <3

:o... I understand... thank you for the quick response 👍🏻

Yeah I came here to say this also. The lack of line (arrow?) paths in the Scene and FileSystem windows makes it really hard to visualize what goes where.
I attached a screenshot for example (though not the best example since the tree is really small in this case so not that hard to follow but in a bigger project it gets unwieldy fast)
I understand it's currently a Godot limitation though as PassiveStar said.
Hope the org can make this change possible sometime soon.

There's now a relationship line for the currently selected list item since https://github.com/passivestar/godot-minimal-theme/commit/57a0d73988e8675e7a4421c4f62f00e9fbc42573 in the latest theme version. This is a half measure to make lists clean while providing some guidance. If I were to draw lines for all items some lists would become too noisy, that's especially noticeable on lists that don't need any guides to begin with, such as the sidebars of editor/project settings
Panels for sidebars were added in https://github.com/passivestar/godot-minimal-theme/commit/67ff2af7f03270543d7e78ec066b42ee5c24f545 and the PR to make them possible on the engine side was merged today. Wait for the next dev release of Godot for them to appear (4.4-dev6)


inputValidator attaches required validator even if there is no required attribute

There is a bug in the latest release (1.2.0).
Consider following scenario:
<textarea data-bind="value: $_message, validations: { someFn: function(){return false}}, inject: validation.inputValidator" name="test"></textarea>
With such element, the validation object should only attach someFn validator. The result though is that the kb.inputValidator attaches also required validator even if there is no required attribute on the element. From what I have investigated the bug is introduced in the commit b7db9d2c665b5dc6004547429bfe36238fe09133. In kb.inputValidator you replaced !el.getAttribute('required')) with _.isUndefined(el.getAttribute('required')) which is not the case.
The getAttribute function returns null if there is no such attribute and _.isUndefined(null) returns false. You should consider using element.hasAttribute function instead.

If one has this issue, workaround for now is to "override" required validator if it is not desired:
validations: { required: function(){return false}}

@wolasss thank you for tracing this down. I'll create a patch over the weekend.

I've released this as part of 1.2.1. Enjoy!

Thanks for such a quick fix @kmalakoff


Missing support of some java.time._ classes

I got linkage errors due missing of following classes when adding SN support for jsoniter-scala using this library (https://github.com/plokhotnyuk/jsoniter-scala/pull/640):
java.time.ZonedDateTime$
java.time.LocalDateTime$
java.time.OffsetTime$
java.time.ZoneOffset$
java.time.OffsetDateTime$
java.time.ZoneId$
java.time.format.DateTimeFormatter$
https://github.com/plokhotnyuk/jsoniter-scala/pull/640/checks?check_run_id=1801112956#step:7:524

@plokhotnyuk Thanks for the report.

For ZIO (namely zio-test), we would need

java.time.ZoneId.of(String zoneId)
java.time.OffsetDateTime.now()
java.time.OffsetDateTime.ofInstant(Instant instant, ZoneId zone)

https://github.com/zio/zio/pull/4671

@sideeffffect Thanks for the info.

@ekrich Small and efficient implementation of .parse and .toString implementation for java.time._ types can be borrowed from here.

Thank-you @plokhotnyuk. I will gladly take contributions if you or @sideeffffect have some cycles.


update description for Tensorflow Mechanics 101 Tutorial

I changed: "We use again MNIST as the example." to "We again use MNIST as the example." as this seemed to make more sense.

Can one of the admins verify this patch?

will submit again with signed git username


(WIP): Calculation for Boundary Layer Statistics

Originally implemented (incompletely) in master branch, but moved to separate branch until it's completed.

I will probably want to flip this into something more flexible. ie.

Have a series of functions that take in vpt.Profile and output boundary layer statistics. One function per statistic (unless it makes sense otherwise)
Have a function that takes in a list of BL thickness functions to evaluate at every point on the wall.

def BL99(profile):
    # Compute delta
return 'delta_99', delta

def BL_momentum(profile):
    # Compute Momentum delta
return 'delta_mom', delta

def computeBLStats(wall, functions):
    for pnt in wall:
        profile = getProfile(dataBlock, pnt)
        for deltafunc in functions:
              deltaName, delta = deltafunc(profile)

This would allow for easy extension in the future while also allowing "on-the-fly" BL stats to be computed easily. This should also generalize to 3D as well.... maybe.

Quick speed comparison between sampling all the points up front ("New BL heights") and sampling profiles one at a time ('Get Boundary layer heights'):
In[1]: %timeit runcell('Get Boundary layer heights', '/projects/tools/Models/BoeingBump/DNS/FlatPlate/Post/CRS_4d_HexWedgeCompare.py')
Out[1]: 23.3 s +/- 39.2 ms per loop (mean +/- std. dev. of 7 runs, 1 loop each)

In[2]: %timeit runcell('New BL heights', '/projects/tools/Models/BoeingBump/DNS/FlatPlate/Post/CRS_4d_HexWedgeCompare.py')
Out[2]: 2.91 s +/- 78 ms per loop (mean +/- std. dev. of 7 runs, 1 loop each)

Also confirmed that the results are identical (wall is from the original method, vortInt is from the new method):
In[79]: np.allclose(wall['delta_displace'], vortInt['delta_displace'])
Out[79]: True

In[80]: np.allclose(wall['delta_momentum'], vortInt['delta_momentum'])
Out[80]: True


Weird dashboard bug

What did you expect to happen?
The dashboard to look like the normal doom dashboard
What actually happened?
When I open emacs my dashboard looks like this

I doesnt matter if I am using emacsclient or not
Describe your attempts to resolve the issue
I tried reinstalling emacs, doom and rewritting my config, at first deleting everything and starting over works, but then randomly at after running doom sync or just restarting emacs, the dashboards looks like this
Steps to reproduce

Open emacs

System Information
https://pastebin.com/y0qYpuxE

I can't reproduce this on any of my vanilla Doom test environments (running 27.1, 27.2, 28.0.60, or 29.0.50; on Ubuntu, MacOS, NixOS, or Arch).

Check *Messages* for errors/warnings at startup. If you find an error produde a backtrace for it.
Comment out everything in ~/.doom.d/config.el then try to reproduce it.
Check the output of doom doctor.

The only thing that works out of the 3 you suggested is commenting / moving my config, that works but once I readd the config everything breaks again. if this only happened with my config I would have guessed there is something wrong with it but it seems that this happens with different configurations (distrotube, mine, mine with literate config etc..)

i just started removing my config bit by bit and it seems like this piece of code is causing the issue
(define-globalized-minor-mode global-rainbow-mode rainbow-mode
  (lambda () (rainbow-mode 1)))
(global-rainbow-mode 1 )

can you explain to me, why this causes an issue ? I stole it from his config when I saw that he added it and it seemed useful

Ah, it's because the dashboard (among other special buffers) do their own syntax highlighting, and rainbow-mode imposes its own on the buffers it's activated in, which interferes with Doom's dashboard in unpredictable ways. You're better off adding rainbow-mode to the hooks you want, rather than activating it everywhere. E.g.
;; add to ~/.doom.d/config.el
(add-hook! '(text-mode-hook prog-mode-hook conf-mode-hook) #'rainbow-mode)

Many major modes are derived from text-mode, prog-mode, or conf-mode, so this cases a pretty wide net.


How to write a custom service and call methods from it?

I'm writing a library in Ballerina and I'm looking for a way to emit events, just like the WebSocket's onError, onText etc'...
// library code
public type Client object {
    string token;
    http:WebSocketClient? wsConnection = ();
    service eventService;
    public function __init(string clientToken, service clientEventService) {
        self.token = clientToken;
        self.eventService = clientEventService;
        // I can't call any method which is inside the service because I'm not in a type descriptor (?)
    }
};

This is what I want to happen on the user's end:
Client myClient = new("TOKEN", myEventService);

service myEventService = service {
    resource function onTest(string myText) {
        io:println(myText);
    }
};

@TheOnlyArtz Can we have more context about your use case.

What type of events need to be emitted to the user?
How users use this object?
How this emit action is triggered. Is it from a client call?

@TheOnlyArtz Ballerina Client/Listener/Service/Resource concepts are introduced along with a standard protocol. Eg. HTTP, WS, gRPC. Hence the calling the remote functions of service is happening according to the protocol spec.
At the moment, Ballerina does not support service typing, hence the user cannot define custom services and write their own way of invoking its remote functions.
Also, I'm not clear of the emit events part of your question. Can you elaborate on it a little more?

@chamil321 Hey, and thanks for the response, what I mean by events is, in version 1.1.x of Ballerina there was this stream https://ballerina.io/v1-1/learn/by-example/streams.html which allowed well data streaming across the application, now it's impossible with the "new" stream? https://ballerina.io/v1-2/learn/by-example/streams.html

@TheOnlyArtz, sorry for the delayed response. BTW, I hope you are doing good.
Let me give a detailed response to this 😄
We are in the process of redesigning the streaming concepts of Ballerina. There will be two types of streams implementations in Ballerina, they are bounded streams (similar to java streams) and unbounded streams (which is used for event stream processing).
In this redesigning process, we have removed the existing streaming implementation (which is available in Ballerina 1.1.x) in the latest Ballerina release. As of now, we have completed the bounded stream implementation and you could find those functionalities in the latest preview releases.
AFAIU, you are looking for unbounded streaming capabilities and they will be available by mid next year (2021).
For now, if you are really looking for the event streaming capabilities then you could implement those using the existing Ballerina features such as lambda functions and etc but it involves some effort.

Answered, hence closing the issue.


feat/advice_slip_service

This PR includes:

The implementation of the remote data source of the application to fetch advice slips from the opensource backend API

LGTM!


Implement autocorrect to give suggestions for misspelled options

Implements issue #5 that will show possible options to use if a user misspells an option.

@a-morales Amazing work. I see there are some conflicts, can you resolve and I'll do a final pass?

There are a few conflicts with Options.scala, could you fix when you get a chance?

@jdegoes addressed comments and this is ready for a re-review.

Just one small comment on capitalization, then good to merge. This is going to be an amazing feature!

Addressed the comment around capitalization of CliConfig and CliApp.

@a-morales Awesome work! Congratulations on the merge...


nixos/tests: add a test for config.users.mutableUsers.

It's in preparation to add an assertion for #4990.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Hey @gleber you might check out my work here: https://github.com/NixOS/nixpkgs/pull/31034

As mentioned in #31034, I think this PR is useful on its own. Waiting for review

Added a comment on the code. Otherwise, LGTM. Given that this isn't part of release.nix, I don't think the hurdle to clear should be too high… I'm happy to merge once comment addressed.

LGTM. Deferring to @fpletz.

@GrahamcOfBorg @grahamc  I've ran nix-env --file ~/code/nixpkgs --query --available --json and it succeeds. I'm not sure how else to debug that failed check...

Thanks!

When seeing why the tests fail, make sure to be rebased on a recent master. That is how the tests are run, and sometimes this is caused by breakage already on master.


/etc/apk/repositories is missing trailing newline

I noticed this because certain docs (e.g. artifactory) have a command like:
echo https://package.mattmoor.io/reponame >> /etc/apk/repositories

Doing this with cgr.dev/chainguard/wolfi-base yields:
# cat etc/apk/repositories
https://packages.wolfi.dev/oshttps://package.mattmoor.io/reponame

cc @kaniini @deitch

See https://github.com/chainguard-dev/go-apk/pull/19


BI-13691 update auto offset reset to use latest

Updates the atuo offset reset config value to use 'latest' which starts reading from the latest message in the partition, ignoring all previous messages if there is no previously committed offset.

CI: No security warnings found


[AuthenticationMixin] Fix .authentication_status

Addresses https://github.com/ManageIQ/manageiq/issues/20115
Also, a HUGE thanks to @mzazrivec and @lpichler for the preliminary research in that issue that probably saved me a ton of time in diagnosing the issue (which I ultimately repeated because my brain is dumb)
Background:
As mentioned in the lengthy comment in AuthenticationMixin, this ensures that there is the following stanza in the WHERE clause:
'"authentications"."relationship_type" = ?'

Specifically, when .authentication_status is used in the .select of a ActiveRecord query.
Previously, it would generate something like this (using Host as the example class here):
SELECT "hosts".*,
       (SELECT  "authentications"."status"
        FROM "authentications"
        WHERE "authentications"."resource_id" = "hosts"."id"
        ORDER BY CASE
                   WHEN LOWER("authentications"."status") = ''            THEN -1
                   WHEN LOWER("authentications"."status") = 'valid'       THEN 0
                   WHEN LOWER("authentications"."status") = 'none'        THEN 1
                   WHEN LOWER("authentications"."status") = 'incomplete'  THEN 1
                   WHEN LOWER("authentications"."status") = 'error'       THEN 2
                   WHEN LOWER("authentications"."status") = 'unreachable' THEN 2
                   WHEN LOWER("authentications"."status") = 'invalid'     THEN 3
                   ELSE -1
                 END DESC
        LIMIT 1) AS "authentication_status"
FROM "hosts"
WHERE "hosts"."type" IN ('ManageIQ::Providers::Redhat::InfraManager::Host')
  AND (hosts.id IN (2))

Where you will see the nested SELECT matches on resource_id, but not resource_type.  In this case, if there was a EMS with the same id as the host in question, it's Authentication record could be returned instead of what is associated with the Host.  In particular, if the Host happened to have 0 Authentication records, it means that when doing this particular query, it could appear to be given a status of "None" in one case (the default when there are no records), and a say "Valid" state when the query is used via a nested SELECT and EMS's Authentication record is "Valid".

The only side effect of this approach (aside from the ugly code it produces) is that there are now cases where the resource_type check happens twice within the same query.  Since they results should be equivalent, this shouldn't matter besides slightly increasing the query string length length sent to Postgres (which shouldn't ultimately matter).
TODO

[ ] Add specs

(Note:  The QA steps below basically will outline the kind of test that will be written)
Links

BZ:  https://bugzilla.redhat.com/show_bug.cgi?id=1820012
Issue Addressed: https://github.com/ManageIQ/manageiq/issues/20115
PR adding this virtual_delegate:  https://github.com/ManageIQ/manageiq/pull/18196

Steps for Testing/QA
I have some comments starting here that can provide some background:
https://github.com/ManageIQ/manageiq/issues/20115#issuecomment-681096691
But effectively, you need any single provider/EMS with

least a single "Valid" Authentication record
couple of associated Host records with no Authentication records

When using the script in the comment above, the Host record with the ID that matches the EMS will show "None" when executing .authentication_status directly, but "Valid" when executing via a query for it that used a nested SELECT.
A reduced form of the script in a Rails console is just testing the following two queries:
irb(main):001:0> my_ems_id = ExtManagementSystem.first # change this as needed for your DB
irb(main):002:0> Host.find(my_ems_id).authentication_status
#=> "None"
irb(main):004:0> Host.where(:id => my_ems_id).select(:id, :authentication_status).first.authentication_status
#=> "Valid"

With the fix in, both results should return "None".

@miq-bot add_label bug
@miq-bot assign @kbrock
cc @lpichler @mzazrivec

Okay, tests added to this one that I tested failed on master first, so I am pretty content with this change.

@miq-bot ivanchuk/yes jansa/yes?
@Fryguy @dmetzger57 I have marked this for the above to versions since the associated BZ was unspecified in it's version and seems to have been tested on master by QE.  I will leave it to the two of you to determine if this is worthy of a backport or not.

Let me try that again...
@miq-bot add_labels ivanchuk/yes, jansa/yes?


Relying on const assets files instead of hard-coded xpaths

Thank you for your great work! I will be merging the PR in the weekend :)

Thank you for replaying , i will be updating the PR to exclude the extra changes keeping only the changes related to "Relying on const assets files instead of hard-coded xpaths"


OOM-kill in dockerd 20.10, tries to allocate 2³² IPAM subnets

Very briefly. With IPv6 enabled, and this entry in default-address-pools (daemon.json):
                , {"base":"fd0d:0ce5::/48", "size": 80}

Dockerd fails to start, crashes with fatal error: runtime: out of memory.
Crash log: ubQ795Mx.txt
As can be easily found in the crash log, OOM happens while in ipamutils.splitNetworks:
https://github.com/moby/libnetwork/blob/3797618f9a38372e8107d8c06f6ae199e1133ae8/ipamutils/utils.go#L92-L93
The config shown, {"base":"fd0d:0ce5::/48", "size": 80} — denotes 2³² /80 subnets. This is over 4 billion subnets.
Related issue: https://github.com/moby/moby/issues/40275
docker version
Client:
 Version:           20.10.21
 API version:       1.41
 Go version:        go1.18.1
 Git commit:        20.10.21-0ubuntu1~22.04.3
 Built:             Thu Apr 27 05:57:17 2023
 OS/Arch:           linux/amd64
 Context:           default
 Experimental:      true
Server:
Engine:
Version:          20.10.21
API version:      1.41 (minimum version 1.12)
Go version:       go1.18.1
Git commit:       20.10.21-0ubuntu1~22.04.3
Built:            Thu Apr 27 05:37:25 2023
OS/Arch:          linux/amd64
Experimental:     true
containerd:
Version:          1.7.12
GitCommit:
runc:
Version:          1.1.12-0ubuntu2~22.04.1
GitCommit:
docker-init:
Version:          0.19.0
GitCommit:

Thanks for reporting. This was also reported in moby/moby, under https://github.com/moby/moby/issues/40275.
The following PR introduced a new subnet allocator which doesn't suffer from this issue. It was released in v27.0, on June 25.

https://github.com/moby/moby/pull/47768

Note that:

Docker Engine v20.10 was EOL'd a year ago, and thus you might miss important security fixes.
The code here is now part of moby/moby. This repo is mostly defunct, and new bug reports should be opened there. This repo can't be archived yet as we need to migrate issues and PRs.

Thanks so much for accurate response @akerouanton


Development

This pull request used wrong branches for source and branch.
Refer to pull request #49.


Deprecated! Supabase Auth Helpers

This project is now out of date.

Supabase deprecated the Auth Helpers used in this project and now requires quite a bit of refactoring / restructuring.
Deprecated helpers currently used in this app

createBrowserSupabaseClient
createMiddlewareSupabaseClient
createServerSupabaseClient

New Helpers

Client Components — createClientComponentClient
Server Components — createServerComponentClient
Server Actions — createServerActionClient
Route Handlers — createRouteHandlerClient
Middleware — createMiddlewareClient

Secondly, since Supabase Auth UI is in beta and only offers nominal views, the auth functionality is not usable as of yet. Users cannot update passwords, email addresses, etc. Leaving the developer to build out Auth from scratch. Supabase is great but this project should build out Supabase Auth views on its own.

is there an example of how to do this?

There are links to the documentation on how to use the helpers and there is this very light weight run through here: https://supabase.com/docs/guides/auth/auth-helpers/nextjs

Hi @guylepage3
I was looking at this and trying to figure out what the update process might be like. However, I don't see the old deprecated function names like you are referring to in the code within the repo.
Unless I am missing something (I could be...I am relatively new to web development), it looks like all the new functions names are being used throughout the repo.
Could you point out some lines of code and file names that you are referring to that use the deprecated functions?

@guylepage3 isn't that just the custom name of the constant though?
That exports a constant that is a custom defined function with the name of createServerSupabaseClient for use throughout the application doesn't it?
The actual nextjs helper used is createServerComponentClient like you see here in that file:
import { createServerComponentClient } from '@supabase/auth-helpers-nextjs';
and here:
createServerComponentClient<Database>({ cookies })
I could be wrong as I am newer to web development.  Am I missing something?

Feel free to read up on Supabase in their docs as well. You'll quickly realize that the implementations vary considerably.

Thanks @guylepage3.  I guess I am confusing myself :).  Sorry I'm new trying to figure this out.
Isn't everything that has the words SupabaseClient in this repo actually just names of constants and not the actual imported helpers?

Nope. But SupabaseClient is a constant. Someone decided it was a good idea to name one of the constants part of the name of the helpers. Just to confuse everyone. Lol.
I thought I could save you some time but just do 3 searches instead of just one.
Search for;

createBrowserSupabaseClient
createMiddlewareSupabaseClient
createServerSupabaseClient

Thanks @guylepage3.  I am still confused lol.  I'll have to do more research to figure this out for myself

Lmao. Please read ALL the links I created in my initial comment.
I provided all the links to the deprecated posts from Supabase.
It's not confusing at all.

@micheltucker do what you will. But unless someone has completed the change, then it's incomplete. Zero confusion actually.

Would you mind sharing one diff of your proposed change?

It's provided by the supabase folks.. Please re-read. Thx


Naming of Breadcrumbs component seems wrong

The current storybook contains a component called Breadcrumbs which is not a breadcrumb but more a wizard step component.
Linkt to storybook

@Franklin89 I'm not sure if I agree. I get that HTML breadcrumb is more to indicate location in a hierarchy of pages and that in our case we do use it a bit differently but it still can be used as a breadcrumb. The idea for the components is to be deep to the most generic one we can relate to

As agreed, this will be a breadcrumb:


A CLI to compile an entire folder with StyleX

What changed / motivation ?
StyleX needs a bundler integration to work correctly. It also depends on Babel. However, this can be burden when trying to integrate into various projects built with project starters like Next.js, Solid Start and create-react-app. These projects have a complex bundler setup and it can be hard to integrate into them correctly. In some cases, like Next, the tooling may be written in Rust where an integration of the Babel plugin is impossible.
In order to provide a clean and reliable escape hatch for these scenarios, we should have a CLI that transforms an entire folder of JS files and outputs the resulting CSS file. (Tools like Tailwind and PandaCSS similarly side-step the built-in bundler because of how hard it is to integrate)
Expected Behaviour
For this example, consider a Next.js app configured to use the src/ folder with a src/app/ within that contains actual routes. When using the CLI, you'd be able to make another folder called, say, source/ and gitignore src/ folder. The source/ folder would contain all the same files that would normally be defined within the src/ folder, but you'd be able to use StyleX.
The CLI will then:

[ ] Transform every single JS file within the source/ folder with the StyleX Babel plugin and output it at the same relative path in the src/ folder.
[ ] While transforming it will collect the generated CSS for each file and output a CSS bundle at src/stylex-bundle.css.
[ ] The transformed JS files should be updated to include an import of the stylex-bundle.css file. (Configurable)

With this process, the StyleX CLI would run first and generate an src/ folder with your actual source code. After this, the usual next dev script will be able to take over and run as usual.
Additionally, the CLI should also:

[ ] Have a watch mode so that only the files that are edited are recompiled, and the generated CSS file is updated accordingly.
[ ] Can be configured to also pre-compile a given list of node_modules packages and write the compiled files in some locations.

[ ] Any processed JS files that import these packages, should be updated to rewrite their imports to point to the compiled location.

Finally, there are a few requirements for the configuration (This is probably a lot of the work):

[ ] The basic configuration options should be possible with both CLI args and with a file config.
[ ] It should be possible to configure the StyleX Babel plugin without a .babelrc file at the project root. (Next goes into Babel mode if it detect a .babelrc file)
[ ] It should be possible to run additional Babel transforms in addition to the StyleX Babel plugin

Stretch Goals

[ ] Avoid recompiling unchanged files between runs (even without watch mode).

Possible tools that can be used:

fb-watchman is a node client for watchman. A tool that tracks file changes within a folder
chokidar
node-watch

Can be configured to also pre-compile a given list of node_modules packages

Note that this is kind of how the RN ecosystem (Expo) handles compiling RN packages for web, and it's pretty fragile because someone has to know a list of all dependencies and sub-dependencies that might be need processing. Probably a bit of a challenge to reliably do that automatically. Do you know if the other CLI-based projects have any alternatives, or does everything expect the developer to list out the packages to be processes?

I know that Next has a transpilePackages option to compile packages manually. I'm following their lead on this since this.
I understand that the node_modules processing is going to be a challenge. I avoided doing this for a while, but there are simply too many esoteric build setups where such a system will always be needed. It unblocks StyleX usage without a huge ecosystem of bundler and framework plugins.

Converting to draft while it's progress. Mark it as ready for review, when it's ... ready for review.

This is ready for an initial review.
I have no idea how to make prettier and eslint ignore my test files that are transfomed by babel. I tried adding to the ignorePatterns in .eslintrc.js, but it doesn't seem to work. I'm open to ideas.

This is ready for an initial review.
I have no idea how to make eslint ignore my test files that are transfomed by babel. I tried adding to the ignorePatterns in .eslintrc.js, but it doesn't seem to work. I'm open to ideas.

I figured it out

@nmn any word on when this will land? If this is close I don't mind using a beta version and testing it out if you push a release!

Looks like a few improvements still need to be made. I'll try to do this on my end shortly.

@Jta26 I've fixed some of the issues, but it's not quite ready for primetime. Here are the blockers for the initial merge:

The CLI should have an option to take in a JSON configuration for StyleX.
The default configuration for StyleX should enable theming APIs
A working example app where the CLI is in use

I tried a standalone next app to test the CLI and the output was not compiled.

After the initial merge we can address the following issues before release:

Handling support for node_modules dependencies that use StyleX.

To simplify, this can be configured via an option in a config so it does not have to be detected manually.

Watch mode should only recompile files that have actually changed
Better logging - Log what files are being transformed.

Re the discussion about support for handling node_modules and how best list them, in large monorepos that have code in multiple packages (sometimes hundreds), shared by several apps, it could be very very useful to be able to define a namespace, e.g., @acme or a glob-like pattern, e.g., @acme/*. Otherwise the burden of listing all relevant packages will often likely be too high.

a glob-like pattern,

@TxHawks I don't think we will support that at first. It's important to note that you would only need to list dependencies that use StyleX within. As such, even in large mono-repos, there's usually only a handful of UI packages.
The challenge with supporting glob pattern is that we need something we can pre-compile without manually seeing what is being imported in the app.

Even in large mono-repos, there's usually only a handful of UI packages.
The challenge with supporting glob pattern is that we need something we can pre-compile without manually seeing what is being imported in the app.

With things like Turborepo and NX, UI packages are often broken into very small parts to allow for better caching. Manually handling this could be very difficult. Could possibly be solved with a pre-build script generating the list of namespace dependencies, but that too isn't really ideal.
Am I correct in assuming you'd also need a list of these dependencies on the bundler level to produce a single CSS file and remove duplicate declarations? E.g. in case two packages have fontWeight: 700 to not have two .x1xlr1w8 { font-weight: 700; } declarations in the final CSS file.

With things like Turborepo and NX, UI packages are often broken into very small parts to allow for better caching. Manually handling this could be very difficult.

Since we're pre-compiling we need to know all the code that could have StyleX usage. We could support globs but that would result in pre-compiling more code than necessary. The risk here is that this CLI would run before another build process, like Turbo+SWC. Compiling too much code may add to much runtime overhead.

Am I correct in assuming you'd also need a list of these dependencies on the bundler level...

No, a bundler already walks a file tree. It can detect files that use StyleX across package boundaries.

We could support globs but that would result in pre-compiling more code than necessary. The risk here is that this CLI would run before another build process, like Turbo+SWC. Compiling too much code may add to much runtime overhead.

I think I understand the footgun here, but since glob would be a non-default opt-in option that can be warned against in the docs, wouldn't it be a good escape hatch to provide?

wouldn't it be a good escape hatch to provide?

It wouldn't be that simple. We need a full list of packages that need to be transpiled and the names they're imported as. Globs don't generically have as much detail.
In particular codebases, you may be able to guarantee that all @acme/* packages are in a particular location. In those scenarios, it should be easy to resolve the glob in the config file itself.
We're starting with a JSON config file, but we should be able to support .js files later which would make this relatively simple for you to set up on your own end.

Closing to open a new one because something is messed up with github


bug/Vertical text in pdf inverted

Describe the bug
In a pdf, a line of vertical text is oriented text bottom facing page right. When doing "partition(filename)", the text is inverted and has random spaces added ("é fi i t r e c" instead of "certifié").
The rest of the text in the same page is properly treated.
This is not an OCR.
To Reproduce
Use a pdf with vertical text.
Expected behavior
Proper order conserved and no random spaces added.
Pdfium or pypdf do it properly.
Screenshots
non applicable
Environment Info
non applicable.
Additional context
non applicable

Hi @CorentinvdBdO - if this is still an issue, could you provide an example document we could use to reproduce?

Hello @MthwRobinson , sorry for bumping and old issue but I'm having the same problem, and I can provide an example document:
https://ddd.uab.cat/record/118259?ln=es
trecom_a2006m12n21p7.pdf


Hi! No module named 'mmcv._ext'

Hi i want to train a glean, but this error occurs.
I desperately want your help. thank you.

i solve it!
thank you!

Good to hear that!


Incorrect tuple union type deduction

TypeScript Version:  3.3.3 and 3.4.0-dev.20190220 with --strictNullChecks

Search Terms:
tuple, union, type deduction, destructuring
Code
function fn(): [null, number] | [string, null] {
  if (Math.random() > 0.5) {
    return [null, 2]
  }
  return ["Str", null]
}

// l: string | null (correct)
// r: number | null (correct)
const [l, r] = fn()
if (l === null) {
  // r: number | null (incorrect)
  console.log(r + 2)
} else {
  // l: string (correct)
  // r: number | null (incorrect)
  console.log(l.toLowerCase(), r === null)
}

Expected behavior:
No compilation errors. Correct type hints.
Actual behavior:
Compilation error:

TS2531: Object is possibly 'null'.

at console.log(r + 2). Incorrect type hints.
Playground Link:
Playground link. Make sure to enable "strictNullChecks".
Related Issues:
Did not find anything exactly like this issue.

This is tracked by this #12184. If you go to the linked issue #28981 the example is more-or-less the same as the one you provide.

Thanks.


Secret sharing function

User Story
Developers want to minimize the definition of cloud credentials. However, they do not always use the same credentials; they may be dynamically generated or use different credentials.
Acceptance Criteria
(Implemented also by #7 at the same time.)

Implemented also by #7 at the same time.
data kind allows passing authentication information, and references by expression allow dynamic authentication information to be passed.
The following is an example definition of data kind.
kind: data
name: azure_credential
provider: environment
parameters:
  - name: subscription_id
  - name: tenant_id
  - name: client_id
  - name: username
  - name: password
    secret: true

The following is an example of data kind reference.
parameters:
  credentials: ${{ data.azure_credential }}
  other_parameters: ...

The following is an example of a reference to a secret generated from a dependent resource.
parameters:
  app_settings:
    DOCKER_REGISTRY_SERVER_URL: ${{ resource.acr.url }}
    DOCKER_REGISTRY_SERVER_USERNAME: ${{ resource.acr.user }}
    DOCKER_REGISTRY_SERVER_PASSWORD: ${{ resource.acr.password }}
  other_parameters: ...


Bypass SIP on MacOS

Part of https://github.com/metalbear-co/mirrord/issues/649.
On MacOS, test if the file that the user specified to run is protected by SIP, or is a script with a shebang that leads to such a binary (also via multiple shebang scripts). If it is, create an unprotected version of the binary (and the shebang scripts that lead to it) and run that instead.
This does not yet bypass SIP on binaries that the executed application executes.

bors r+


Fix test json property name again

There was a missing related to #32.

Coverage remained the same at 76.421% when pulling 1c95b0742837c7651dee104af54f753f66ddd4d9 on y0za:fix-test-json-again into c417db51899517a62ac67ae16fae5bfc0a13efdc on mattn:master.

👍


RHBPMS-4576: A user can display tasks for which he is not Potential Owner or Bussines Admin

adding the fix to kie-server too!
This PR should me merged with:

https://github.com/droolsjbpm/jbpm/pull/756
https://github.com/droolsjbpm/jbpm-wb/pull/633

@mswiderski more reviews for you...


CLI Debian publishing

Here's a step-by-step breakdown of the provided Terraform code:
Step 1: S3 Bucket Configuration

Creates an S3 bucket named cli-prod-sunodo-origin with the specified name and tags.
Sets the bucket ACL to "private" using aws_s3_bucket_acl resource.
Configures bucket ownership controls to allow ObjectWriter access using aws_s3_bucket_ownership_controls resource.
Sets up public access block configuration to disallow public access to the bucket using aws_s3_bucket_public_access_block resource.

Step 2: IAM Configuration

Defines an IAM policy document using data.aws_iam_policy_document to specify the required actions and resources.
Creates an IAM policy named sunodo_cli with the defined policy document using aws_iam_policy resource.
Creates an IAM identity_providers for github OIDC

Step 3: CloudFront Configuration

Creates a CloudFront origin access identity named sunodo_cli_origin_access_identity using aws_cloudfront_origin_access_identity resource.
Defines an IAM role named cloudfront-iam-role with an assume role policy allowing CloudFront service to assume the role using aws_iam_role resource.
Defines an IAM policy named cloudfront-iam-policy with the required permissions using aws_iam_policy resource.
Attaches the IAM policy to the IAM role using aws_iam_role_policy_attachment resource.
Creates an ACM certificate for the domain cli.sunodo.io and *.sunodo.io using aws_acm_certificate resource. (wildcard)
Creates a CloudFront distribution named sunodo_cli_distribution with the specified settings, including the S3 bucket origin, viewer certificate, cache behavior, and restrictions using aws_cloudfront_distribution resource.
Associates the CloudFront distribution with the subdomain cli.sunodo.io using a CNAME record in Route 53 using aws_route53_record resource.

Looks good. I suggest we build on top of this PR before merging to main.
I still think tf code is so verbose, compared to cdk.

I still think terraform code is so verbose, compared to cdk.

Yes, it's more aligned with the AWS API, like CloudFormation, CDK has a lot of abstractions on top of the CloudFormation.

Yes, it's more aligned with the AWS API, like CloudFormation, CDK has a lot of abstractions on top of the CloudFormation.

But if terraform had good composability it was about time to have layers of abstraction as well to make a s3+cloudfront distribution with 10 lines of code

It's possible to create/reuse terraform modules and there are public registries like this https://registry.terraform.io/namespaces/terraform-aws-modules.

Yeah, We can even create our own terraform modules .

It's possible to create/reuse terraform modules and there are public registries like this https://registry.terraform.io/namespaces/terraform-aws-modules.

Yeah, We can even create our own terraform modules .

Ok, but the idea is to write as less code as possible, and reuse. Is it possible to write minimal cloudfront distribution + s3 code using cloudfront module?

I found this module https://registry.terraform.io/modules/cloudposse/cloudfront-s3-cdn/aws/latest
I don't know the reputation of the publishers but looks like it has what you want.

the idea is to write as less code as possible, and reuse. Is it possible to write minimal cloudfront distribution + s3 code using cloudfront module?

Yes, it is possible to reduce code duplication and reuse existing modules to create a minimal CloudFront distribution with an S3 bucket. By using a Terraform module specifically designed for CloudFront and S3 integration, we can abstract away the complexity and write less code.

I don't know the reputation of the publishers

That's why I suggested to create our own modules. we can even publish them as an opensource modules too.

It's possible to create/reuse terraform modules and there are public registries like this https://registry.terraform.io/namespaces/terraform-aws-modules.

Yeah, We can even create our own terraform modules .

Ok, but the idea is to write as less code as possible, and reuse. Is it possible to write minimal cloudfront distribution + s3 code using cloudfront module?

Fixed !

cloudposse/acm-request-certificate/aws module to create AWS ACM for cloudfront,

cloudposse/cloudfront-s3-cdn/aws module to create S3 bucket + cloudformation

cloudposse/iam-system-user/aws module to create oclif IAM user to upload packages to S3

LGTM
Add a changeset by running yarn changeset and selecting "minor" change to @sunodo/cli with change "add infrastructure code for CLI tarballs and debian release`

LGTM Add a changeset by running yarn changeset and selecting "minor" change to @sunodo/cli with change "add infrastructure code for CLI tarballs and debian release`

Okay
I think we don't need the oclif_user module anymore ! Am I right ?

I create a terraform cloud account and invited you both.
Still need to configure the trust between AWS and Terraform Cloud.
https://developer.hashicorp.com/terraform/cloud-docs/workspaces/dynamic-provider-credentials/aws-configuration

I create a terraform cloud account and invited you both. Still need to configure the trust between AWS and Terraform Cloud. https://developer.hashicorp.com/terraform/cloud-docs/workspaces/dynamic-provider-credentials/aws-configuration

One solution is to define variables and marks them as Sensitive in TF cloud

The apply role has write permissions to S3 and Cloudfront.

It also needs permission to create IAM policy and IAM role, Create Cname and A record in route53 and Create ACM certificate

Both amd64 and arm64 were built and uploaded as build artifacts, but for some reason only the amd64 package was uploaded by oclif upload deb.
Install of debian package in my machine failed, probably because there is no arm64 package.

I'd like to see the terraform content on a separate commit.

It was separated in the first place, then @tuler asked me to put them all in one commit.

I'd like to see the terraform content on a separate commit.

It was separated in the first place, then @tuler asked me to put them all in one commit.

Yeah, we don't agree all the time.
Do what you think is best.

I'd like to see the terraform content on a separate commit.
It was separated in the first place, then @tuler asked me to put them all in one commit.

Same PR, not necessarily same commit.

We'd have to measure to be sure about it.

Tarball takes 42 minutes and debian takes about 15 minutes. So, if we add them in a row, it will take about 60 minutes. but in parallel it will take 42 minutes.

One last thing, shouldn't the pack_* jobs depend on the build job?

The strategy is to run build on each PR but run the rest og the jobs whenever we have a new tag if: startsWith(github.ref, 'refs/tags/@sunodo/cli@')

whenever

yeah, but it doesn't make send to pack_* if the build job fails, right?
that's why I insist in making it a dependency

yeah, but it doesn't make send to pack_* if the build job fails, right?
that's why I insist in making it a dependency

Inside the pack_* jobs we should use composite action as like as the build job. so, if we make them depandent on the build job, we will increase our CI time. (the build should run then the pack_* jobs should run)
Your right If the build job runs steps that are totally unique to the pack_* jobs.

@oap75 this branch needs to be rebased

rebased

I think this is too complex. We will reconsider distribution strategies in the future.


Create bacvu.toml for tesntet-15

[validator.bacvu]
consensus_public_key = "00754b2c38f1f98e51b9592b03dd31a34baafc89d2c3c9157d5ff20e72f7f7f2f1"
eth_cold_key = "0103176180ad0ae1b522a0ae23a1956fe8f67b468396c2ca88ee3398047faed97594"
eth_hot_key = "0103a56b861adef4cc194bdca44bb741c42f38d7a2df5a94408783a32aece48f5772"
account_public_key = "00fd1f5d8953dbc5eb0c6b84b09c991910dfe8edc7a539774e8842ced3c9f71bf9"
protocol_public_key = "00aa83b8740012f2ab6f3609c397585f690eb9417ae959fcfdcd73c234bddd935a"
dkg_public_key = "60000000c0a52af24fd4bf1b169508a9f839936c04290c166ba9a41bb4d1ea72d8343189d7cdd753d6efaf0922104ddce1316317f888d215d989707f332ce1d755acba9ff0815bf7d5db89e34d94dcc30bc2db3562758c2e49eb65a44ebb2f828ab04485"
commission_rate = "0.05"
max_commission_rate_change = "0.01"
net_address = "<IP_ADDRESS>:26656"
tendermint_node_key = "00c9247e1bd89a60cc0c0e755a0f233c2c0506c0c48b4708ce1c3fa6c6aa06eaba"
email =<EMAIL_ADDRESS>discord = "@jessie479sjcb"
elements = ""
telegram = ""
twitter = "@CaitlynCoopg295"

Filename contains whitespace
This PR has been automatically closed because it does not follow the correct procedure for updating the genesis file.

Filename contains whitespace
This PR has been automatically closed because it does not follow the correct procedure for updating the genesis file.

Filename contains whitespace
This PR has been automatically closed because it does not follow the correct procedure for updating the genesis file.

Filename contains whitespace
This PR has been automatically closed because it does not follow the correct procedure for updating the genesis file.

Filename contains whitespace
This PR has been automatically closed because it does not follow the correct procedure for updating the genesis file.


Fix scoping of self in production

Fix scoping issue of self, which caused Sourcereel to crash in production.

Pull Request Test Coverage Report for Build 2425

0 of 4 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 35.027%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/components/sourcecast/util/recorder.js
0
4
0.0%

Totals

Change from base Build 2415:
0.0%

Covered Lines:
2174

Relevant Lines:
5509

💛  - Coveralls


[ADD] Comments on Experiments and examples JOSS paper

🚀 Feature Request
Suggestions regarding the Experiments and examples section of the paper submitted to JOSS :

[ ] The (Ben Hamner, Friedman, 2013) citation seems to be very Kaggle specific (for instance I didn't find a correspondance on Google Scholar), so could you add on the reference to add the link to the data. Kaggle website suggests:

Adam, Ben Hamner, Dan Friedman, SSA_Expedia. (2013). Personalize Expedia Hotel Searches - ICDM 2013. Kaggle. https://kaggle.com/competitions/expedia-personalized-sort

In general having links for each reference is a good practice in such journal.

[ ] Is there a notebook somewhere on this repo that shows the code used to generate this benchmark? If so, I would add the link to it in a footnote of the paper or at least a comment that it is in the repository. There is a lot of content here (congrat's!), but it would really help the reader to find its way.
[ ] The "customize choice models" is indeed a nice feature, however I'm not sure it is necessary to show code regarding this in the paper. It is probably sufficient to just mention it and link to the documentation of the package.

📎 Additional context
This comment are here to ensure data is accessible and reproducibility.
Connected to https://github.com/openjournals/joss-reviews/issues/6899

Hello @tmigot,
Thank you for your review !

Here are two notebooks for the different experiments (benchmark + custom model)
I will modify the paper following your recommendations.

Hello,
I fixed the the citation and removed the example, while adding a sentence earlier in the paper to link the documentation !

thanks @VincentAuriau


Replace support link for partners with a message to contact their bank

Resolves #4186
Description

As described in #4186, this PR makes a change to the error message displayed to partners. It removes the support link and replaces it with the name of the Partner's Bank and the email address of that bank.
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

The existing tests have been modified to reflect the changes in the error message. They now ensure that the name of the Bank and the email address exist in the error message
Screenshots

Before

After

/cc @sampart @kurgol @hharen

Looks good!


updated quickstart guide

Just updated quickstart to follow a logical sequence: first install all dependencies, then prepare environment, then start installation.
This was motivated after the resolution of issue #160.

Thanks!


Update _base.scss

trailing > removed

That's not an accident.
a > {
  b { color: red; }
}
a {
 b { color: white; }
}

Becomes:
a > b { color: red; }
a b { color: white; }


Some group_norm tests are failing in debug build

Repro steps:

build debug build
run test_group_norm tests. Some testing showed that some subsets of the below may fail (checked on GS and WH)
pytest tests/ttnn/unit_tests/operations/test_group_norm.py::test_group_norm_with_block_sharded_v2_8x8_grid
pytest tests/ttnn/unit_tests/operations/test_group_norm.py::test_group_norm_with_block_sharded_v2_8x8_grid_tile_layout
'pytest tests/ttnn/unit_tests/operations/test_group_norm.py::test_group_norm_with_height_sharded'

Expected Behaviour:

pass

Actual Behaviour:

some tests fail

See https://github.com/tenstorrent/pytorch2.0_ttnn/issues/538
Might be related to bad CB handling, e.g. see https://github.com/tenstorrent/tt-metal/issues/15458

On GS with Debug:
pytest "tests/ttnn/unit_tests/operations/test_group_norm.py::test_group_norm_with_block_sharded_v2_8x8_grid[N=2-C=320-H=64-W=64-num_groups=32-device_params={'l1_small_size': 0}]"

produces
E       RuntimeError: TT_ASSERT @ ../tt_metal/impl/buffers/circular_buffer_types.cpp:109: false
E       info:
E       Cannot set to globally allocated buffer. Circular buffer size 131072 B exceeds allocated L1 buffer bank size of 98304 B
...
E        --- tt::tt_metal::v0::CircularBufferConfig::set_globally_allocated_address(tt::tt_metal::v0::Buffer const&)
E        --- ttnn::operations::normalization::groupnorm_multi_core_sharded(tt::tt_metal::Tensor const&, std::__1::optional<tt::tt_metal::Tensor const> const&, std::__1::optional<tt::tt_metal::Tensor const> const&, std::__1::optional<tt::tt_metal::Tensor const> const&, tt::tt_metal::Tensor&, float, unsigned int, unsigned int, MathFidelity, tt::tt_metal::DataType, tt::umd::xy_pair, bool)


Splitting huge canvas into multiple pages

I was wondering if this can be done with jsPDF.
I have a canvas that is twice the size of the paper size and I would like the images drawn inside this canvas to be printed in multiple PDF pages. I am using the canvas.toDataURL method to output the content of the canvas.
Is there a way to do this with jsPDF?  When I add a page with addPage(), it just seems to render a blank page instead of the portions of the canvas content.  Perhaps I'm not understanding how to do this right.
Thanks for your help and advice.

+1 @palmoni @jtownsend99 if someone can write the solution it would be great :-) Thanks !

seems like multiple pages with jsPDF is a myth =/


Few Suggestions

Thanks for keeping this app for only one specific feature
its small and gets job well
feature suggestions
-stay on top feature will be very helpful, so dont have to click every time when i want to skipping tracks
-clicking on song info can reveal music playing tab for browsers or opens app
Also i guess this one is bug
-if no audio playing tray icon disappearing
have a nice day

Thanks for the suggestsions, I will consider implementing the first one.
It is already should open the application on click, but this unfortunately does not work with browsers, because it can't determine the handle of the browser window - it can detect only the process name, and modern browsers typically have a separate process for each tab, making it impossible to determine which process ID owns the window.
The icon in tray is used for two actions - double click or middle click on it to quickly play/pause media playback, or left click on it to open the flyout. If no media is playing, I don't know what it can show there.

also im developing some small apps for macos its possible on there with ui scripting, its weird not on windows . maybe they will add this to chromium in future
nice to know middle click stops audio, thanks
actually i thought app stopped or crashed until next play, maybe you can put an generic album art with no media playing text without buttons. also you can use app's icon (⏵I ) for tray if no audio playing
Also if you need any help about UI & icon designs i can help

hi again,
after 2 months of use i want to share my experiences
clicking 2 times for skipping next track gets me bored after a while
i tried audio flyout but it has too much bugs also i get an bsod after uninstall
so i switched back to your app, then i find an app named tray tools from ancient times, i added next tray icon with that and combination is perfect. tray tools can create a shortcut on tray i added a vbs script as shotcut
script i used > CreateObject("Wscript.Shell").Sendkeys(Chr(&hB0))
few more suggestions
1- if you add a delay/timer before switching play pause stays same while playback continues. play icon turning pause icon or disappearing for a short time when skipping / changing next track
2- here is a concept image what to show while playback stopped. Of course its not necessary , keeping icon on tray instead of disappear will be enough to show app is running

control buttons disabled on this
3- you can add menu with options, if you plan to add next previous icons hiding option would be cool

clicking about opens github page
4- as an alternative if you can map right scroll or ctrl click on tray icon for next track would work as well
thanks for taking your time on this, really appreciated

Would it be possible for you to add a feature such that we can control media of each browser tab?

Would it be possible for you to add a feature such that we can control media of each browser tab?

This is not possible


Update UserGuide.md for sample data and purge data.

Codecov Report

Merging #31 (2670244) into master (16f432c) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #31   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 16f432c...2670244. Read the comment docs.


Please publish a lock file of the nuget versions that are used in the function runtime

I was trying  to build a function that uses sendgrid output bindings, but things failed since I used incorrect nuget package version. Currently it's super hard for precompiled to know which version the runtime needs. A Lock File with a Versions listed would be very helpful

Here is how paket would do it: https://fsprojects.github.io/Paket/lock-file.html

+1000 to this
I've personally lost a long time fighting issues thinking it's configuration related only to find out I have a package that's slightly newer. Pinning the versions of everything is fine, assuming there's a way to easily figure out and track.
Scouring the web.config to figure out what versions are in play and then figuring out which packages those are for is not a great workflow.
I get that you probably don't want to tie down all the individual packages since folks with more control of the runtime can work, maybe a meta-package for users of the consumption runtime version with all packages locked is an easier solution?

@ahmelsayed I'll get the ball rolling on this one. I pointed @forki to a couple of spots to help in the meantime but the're similar to what @johlrich mentioned and quite painful.
We may simply start with a Wiki/Documentation page covering the dependencies.

Please no wiki. Please use something with fixed schema that is readable by
a machine.
Am 30.11.2017 23:20 schrieb "Fabio Cavalcante"<EMAIL_ADDRESS>
@ahmelsayed https://github.com/ahmelsayed I'll get the ball rolling on
this one. I pointed @forki https://github.com/forki to a couple of
spots to help in the meantime but the're similar to what @johlrich
https://github.com/johlrich mentioned and quite painful.
We may simply start with a Wiki/Documentation page covering the
dependencies.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Azure/azure-functions-cli/issues/312#issuecomment-348339348,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADgNNCpHCACVMn-zvH1X4pMOtIqkNHbks5s7yoJgaJpZM4QZ1tz
.

We'll likely need to have both. But yes, we'll make sure this is published in a way that is easy to consume programatically as well.

Here is my easy solution to get the list of the dependencies used by the runtime:
[FunctionName("DumpAssemblies"), NoAutomaticTrigger]
public static void DumpAssemblies(TraceWriter log, string input)
{
    log.Info(string.Join(Environment.NewLine, AppDomain.CurrentDomain.GetAssemblies().OrderBy(e => e.FullName).Select(e => e.FullName)));
}

Then I simply run this function manually in the Azure portal.

This is nice and gives assembly version. But we need nuget version which
can actually differ.
Am 06.03.2018 23:28 schrieb "Cédric Luthi"<EMAIL_ADDRESS>
Here is my easy solution to get the list of the dependencies used by the
runtime:
[FunctionName("DumpAssemblies"), NoAutomaticTrigger]public static void DumpAssemblies(TraceWriter log, string input)
{
log.Info(string.Join(Environment.NewLine, AppDomain.CurrentDomain.GetAssemblies().OrderBy(e => e.FullName).Select(e => e.FullName)));
}
Then I simply run this function manually in the Azure portal.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Azure/azure-functions-core-tools/issues/312#issuecomment-370952031,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADgNGD9LU8FM9vsi-Y4zG6_MCDWZVCeks5tbw2DgaJpZM4QZ1tz
.


Empty or null list of struct cannot be written to parquet

Describe the bug
When writing arrow batch with empty or null list struct, it fails with General("Inconsistent length of definition and repetition levels: 0 != 1")
To Reproduce
Write record batch which contain empty or null list of struct. Here's the PR with full repro behaviour https://github.com/apache/arrow-rs/pull/704
Expected behavior
The batch is successfully written
Additional context
WHat I could debug is that this line should fail https://github.com/apache/arrow-rs/blob/master/parquet/src/arrow/levels.rs#L646, because at some point the array_mask becomes empty slice which leads to the Inconsistent length of definition and repetition levels
I feel like this might be somehow connected https://github.com/apache/arrow-rs/issues/594.

I'm getting the same issue on darwin-aarch64 using the json2parquet CLI.

wget https://raw.githubusercontent.com/json-iterator/test-data/master/large-file.json
# Convert it into newline delimited JSON or else Arrow complains
cat large-file.json | jq -c '.[]' > lf.json
json2parquet  lf.json bork.pq

# Error: General("Inconsistent length of definition and repetition levels: 891 != 1388")

This line of JSON is barfing in json2parquet with:
thread 'main' panicked at 'Cannot filter indices on a non-primitive array, found List(true)'

https://github.com/apache/arrow-rs/blob/e0abda2c178be0c38d4257d22de2e4a3bfafde82/parquet/src/arrow/levels.rs#L757
{"ts":1331901001.88,"fuid":"Fd3cGk2agqUftBeFx4","tx_hosts":["<IP_ADDRESS>"],"rx_hosts":["<IP_ADDRESS>"],"conn_uids":["CaJMZy195M8cuXfxn4"],"source":"HTTP","depth":0,"analyzers":[],"mime_type":"text/html","duration":0.0,"is_orig":false,"seen_bytes":1433,"total_bytes":1433,"missing_bytes":0,"overflow_bytes":0,"timedout":false}

Looks like we might have to translate  https://github.com/apache/arrow/blob/master/python/pyarrow/tests/test_json.py
Better yet add a json directory for all the arrow clients:
https://github.com/apache/arrow-testing/tree/master/data


ng-if memory leak

I've noticed a possible memory leak with the ng-if directive. It appears to be failing to garbage collect HTML Elements.
I've set up two fiddles to compare ng-if with ng-show (which doesn't have the same issue).
ng-show - http://jsfiddle.net/neridum/8vo7ae8L/
ng-if - http://jsfiddle.net/neridum/hptt8wmu/
I simulated this by running a Heap Allocation profile and clicking between the views a few times (Incognito in Chrome Canary). With ng-if there is a left over HTMLDivElement that looks to be detached.
This doesn't appear to be for a specific version and is apparent with 1.2.1, 1.3.9 and 1.3.15.
I found a Stackoverflow post suggesting to run a cleanup method but I'm not sure that this is a suitable approach - http://stackoverflow.com/questions/26022481/ng-if-causing-memory-leak-in-angular-js
A related issue to this is https://github.com/angular/angular.js/issues/9748

@astalker:
Does this occur on 1.4.x?
Does it occur on all browsers?

I have de same problem on Angular 1.4.1 and Chrome 40.0.2214

Right! We ought to get to the bottom of this...

I've rerun the tests on Chrome 43 with Angular 1.4.1 and ng-if will still create detached HTMLDivElements versus ng-show. I'm not really sure how to run the same type of tests in any other browser I'm afraid.
ng-show shows 4 HTMLDivElements after 6 and after 9 view changes.
ng-if shows 4 HTMLDivElements after 9 view changes 2 of which are detached. It shows 6 elements 2 of which are detached after 6 view changes.
In my tests, regardless of how many times I change the view it only seems to create 2 detached DivElements.

Was not able to reproduce the leak using Chrome 44.0.2403.155 on OS X

I happen to stumble across this issue as well in v1.3.16.

Fixes issue: https://www.pivotaltracker.com/story/show/107443778

I don't know if it has anything to do with it, but i had a weirg leak regarding ng-if
I had a directive with an element that calls controller on ng-if, something like that:
<directive ng-repeat="vm.nodes">

</directive>
That one cuased vm.project to keep in memory even after scope is destroyed. the weird thing, is that the vm.isActiveNode and content inside the ng-if div never calls vm.project but it still leaked.
When I changed ng-if to ng-if="vm.activeNode === node.id" the leaking stopped..
I'm using Angular v1.4.7 and Chrome v46.0.2490.86 m

I also face this issue in a large scale application.Profiling with chrome developer tools produced a lot of DOM detached nodes.

I have struggled with this today. I can see that there are a limited number of detached nodes but they do not seem to grow during normal usage. I think that they are related to the initial compile step of directives that make use of transclusion - i.e. the transcluded DOM is detached but is referenced in a closure that is referenced by long lived functions such as the ngIf watcher function.

I can't find any leaks in the example.
The detached nodes are the templates cloned by ng-if when the statement becomes true.  This is how transclusion works for things like ng-if, ng-repeat, ng-view etc. where the transclusion may occur multiple times so the original element must be cloned and not used directly. If an ng-if wraps a large DOM this will result in a large detached DOM as the transclude template.
If you know your ng-if condition will never change you can use one-time binding which will allow the detached template nodes to be GCed.
652b83eb226131d131a44453520a569202aa4aac may reduce the detached nodes in cases where it prevents templates from being inserted into the detached DOM. I think.
I do wonder if there are cases where these template nodes could get jQuery data on them. Because they are never cleaned properly with jQuery.cleanData...

Also note that ng-show is completely different then ng-if. All ng-show it does is toggle a class, where ng-if creates/destroys the nodes + scope each time based on a template. You really can't compare the two...


Add support for globs in paths configuration

Is your feature request related to a problem? Please describe.
We are using mkdocs in a monorepo. In this monorepo, we have many packages and a top level mkdocs configuration. We currently need to list every one of our package paths individually in the paths configuration option. However, the paths to all the modules in our monorepo could be easily described with a couple globs (ex: lib/*/module).
Describe the solution you'd like
It would be great if the paths configuration option supported a glob syntax.
Describe alternatives you've considered
We've thought about making our mkdocs.yml a generated file, but this is a bit cumbersome.
Additional context
I could see this being done either in this package by resolving the globs before passing them to the griffe loader or as a native feature of griffe itself. Based on the code though, it seems like it would be simplest to add directly in this package.

Yup, good idea, valid use-case, easy addition.


Bracket Limitation Fix

XmlSerializer should remove the characters when needed when considering invalid characters like < and > when writing the name.

Taking a look into it, it seems to be about auto-implemented properties, probably will be a feature-bug kinda thing.

Fixed in latest commit


Add HiGHS solver and update documentation

Introduce the HiGHS solver to enhance optimization capabilities and update related documentation for clarity. Still have issues with HiGHS to work on. Additionally, remove unnecessary YAML files and include new static assets for improved user experience.

Closing to resolve conflicts and committing updated verison without issues from stashed version


Cut/Paste between cells does not work - Cut deletes the whole table

CUT works for me. Only the selected cell is deleted. Here is an example for a PASTE handler (set it on the parent collection of the table):


External Dialogs not visible with MainWindow set to IgnoreTaskbarOnMaximize

What steps will reproduce this issue?
This issue is happening in my application after I upgraded to 1.4.0. It is reproducible in the demo app by following these steps:

Run the MahApp.Metro Demo .Net45
Select the Window > Ignore Taskbar on Maximize option
Maximize the main window
Select Dialogs > Show ShowInputDialog Externally

The main window will dim and disable, the ShowInputDialog will appear briefly, then disappear behind the main window (which remains dim and disabled).
Expected outcome
The ShowInputDialog does not disappear behind the dim and disabled main window.
--
Environment

MahApps.Metro v 1.4.0
Windows 10
Visual Studio 2015
.NET Framework 4.6

I've found the cause and a fix for this issue...
The fix for #2694 set the Z-Order of a window to HWND_BOTTOM, HWND_TOP, then blindly set it to HWND_NOTOPMOST. If the window is an external dialog, this is no good. The Z-Order of an external dialog must be set to be after the Owner window.
I've fixed this by adding the following at line 377 in BorderlessWindowBehavior.cs:
// #2780 Don't blindly set the Z-Order to HWWND_NOTOPMOST. If this window has an owner, set
// the Z-Order to be after the owner. This keeps external dialogs appearing correctly above
// their owner window even when owner window is maximized and ignoring taskbar.
IntPtr hwndInsAfter = Constants.HWND_NOTOPMOST;
if (metroWindow.Owner != null)
    hwndInsAfter = new WindowInteropHelper(metroWindow.Owner).Handle;
UnsafeNativeMethods.SetWindowPos(this.handle, hwndInsAfter, left, top, width, height, Constants.TOPMOST_FLAGS);
I'm still trying to figure out how to make a pull request. It may be faster for someone else to make the change. Let me know if I should make a pull request, but I may need help with that.

@punker76  Thanks for applying the fix based on my comments...unfortunately, the fix was not quite right and there is some other issues now when external dialogs are shown. I should have been more clear in my message. Better yet, I should have just made a Pull Request.
I have cloned the repo, created a branch, committed my changes to fix this...but I can't seem to push my changes and create a pull request. I'm new to Git so I will need some help here. Can you point me to some wiki or something I can read to figure out how to create a pull request for my changes?

@fjlaga https://help.github.com/articles/about-pull-requests/

@fjlaga I don't see other issues, tested all external dialogs...

@punker76 There are no other issues in the demo app. I am seeing issues in my app where we use external dialogs much more. There are two issues with the fix you submitted:

You removed two calls to SetWindowPos which were for fix #2694. These calls set the z-order to HWND_BOTTOM, then to HWND_TOP, before setting it to HWND_NOTOPMOST. My fix was to replace only the last SetWindowPos call to use the owner window instead of HWND_NOTOPMOST. I'm not sure removing the two calls to SetWindowPos actually causes a problem, but the author of the fix for #2694 thought those two calls were important, so I didn't want to take them out.

My fix for #2780 changed the last call from using HWND_NOTOPMOST to the owner window, but it used the SWP flags Constants.TOPMOST_FLAGS. Your fix changes this to use 0x0040 which is the same as SWP_SHOWWINDOW. This is causing problems in my app...I have some custom dialogs that are used more or less like tooltip windows. With your fix, the main window flashes active/inactive every time one of these custom tooltip windows is shown.

I will work on a pull request.

@fjlaga It would be very helpful if you can create an app with such a 'tooltip' to reproduce this. I don't know if it's a good idea to use dialogs as tooltips, but a sample could be help to look what's going on...

to this point, I was the author of the fix...
create a sample to show your custom dialog tooltip, so we can look to this problems

thx

@punker76 I haven't had time to create a sample app that demonstrates the error. I have cloned the repo and made the changes that will fix it. The changes are minor and I think they can simply be reviewed without a sample app. My problem is when I try to push my changes I am denied access.
Error encountered while pushing to the remote repository: Response status code does not indicate success: 403 (Forbidden).

A little help would be appreciated.

I haven't had time to create a sample app that demonstrates the error. I have cloned the repo and made >the changes that will fix it. The changes are minor and I think they can simply be reviewed without a >sample app.
That's not the point. Yes I can review it, but you say that you get issues at your app and not at the main MahApps demo. So how can I find out what's going on? Maybe it's easy enough, maybe it has some other side effects...

My problem is when I try to push my changes I am denied access.
Error encountered while pushing to the remote repository: Response status code does not indicate >success: 403 (Forbidden).

It seems you didn't forked the repo.
To create a PR you must do the following steps

fork the repo
clone your fork to your local disk
create a branch Fix-for-XYZ
make your changes
commit your changes in your branch
push you branch on your repo
now you can make a PR from your branch to the dev branch of MahApps

hope this helps

Ah…fork me!
Ok, I think I got it right this time. Pull Request created.
-Frank
Frank Lagattuta
Phone<PHONE_NUMBER>
Mobile<PHONE_NUMBER>
Email     flagattuta<EMAIL_ADDRESS>Web       www.gsdx.ushttp://www.gsdx.us/
CONFIDENTIALITY NOTICE
This message and any files or text attached to it are intended only for the recipients named above and contain information that may be confidential or privileged. If you are not an intended recipient, you must not read, copy, use or disclose this communication. Please also notify the sender by replying to this message, and then delete all copies of it from your system. Thank you.
From: Jan Karger<EMAIL_ADDRESS>Sent: Wednesday, January 11, 2017 12:29 AM
To: MahApps/MahApps.Metro<EMAIL_ADDRESS>Cc: Frank Lagattuta<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MahApps/MahApps.Metro] External Dialogs not visible with MainWindow set to IgnoreTaskbarOnMaximize (#2780)
I haven't had time to create a sample app that demonstrates the error. I have cloned the repo and made >the changes that will fix it. The changes are minor and I think they can simply be reviewed without a >sample app.
That's not the point. Yes I can review it, but you say that you get issues at your app and not at the main MahApps demo. So how can I find out what's going on? Maybe it's easy enough, maybe it has some other side effects...
My problem is when I try to push my changes I am denied access.
Error encountered while pushing to the remote repository: Response status code does not indicate >success: 403 (Forbidden).
It seems you didn't forked the repo.
To create a PR you must do the following steps

fork the repo
clone your fork to your local disk
create a branch Fix-for-XYZ
make your changes
commit your changes in your branch
push you branch on your repo
now you can make a PR from your branch to the dev branch of MahApps

hope this helps
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MahApps/MahApps.Metro/issues/2780#issuecomment-271807746, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AHG0KNhBTmevAqQKjiAMviwhS2HBjn9oks5rRJLPgaJpZM4LVT5W.


Generation fails with multi-byte heading level-2 and up

Describe the bug
After having updates to<EMAIL_ADDRESS>generation of documents always fails.
As noted in the error message, the error occurs in headings that contain multi-byte characters in the Markdown.
This does not occur in level 1 heading, but does occur in level 2 and above.
Reproduction
schema.graphql
type Query {
  """
  ## ⌘
  """
  me: ID
}

magidoc.mjs
/**
 * @type {import("@magidoc/cli").MagidocConfiguration}
 */
const config = {
  introspection: {
    type: 'sdl',
    paths: [
      'schema.graphql'
    ]
  },

  website: {
    template: 'carbon-multi-page',
    output: '.output/public/schema-docs'
  }
}

export default config

Logs
$ npx magidoc generate
✔ Determine tmp directories [0.0s]
✔ Select Package Manager [0.3s]
  › Selected npm
    ⚠ This package manager is not well supported yet.
    ⚠ It is recommended to install pnpm instead.
↓ Template already unzipped. [SKIPPED]
↓ Template already unzipped [SKIPPED]
✔ Install dependencies [5s]
✔ Resolving template configuration [0.0s]
  › Found 8 supported keys
    Target schema location: ./src/_schema.json
✔ Load GraphQL Schema [0.0s]
✔ Write variables file [0.0s]
✖ Build template
◼ Move output
Command 'npm' failed with status 1 when executed in directory<EMAIL_ADDRESS>
---- Program Output----

><EMAIL_ADDRESS>build
> vite build

vite v3.2.4 building for production...
transforming...
✓ 443 modules transformed.
1:20:25 [vite-plugin-svelte] dom compile done.
package                                 files    time      avg
@magidoc/starter-carbon-multi-page        163   2.54s   15.6ms
rendering chunks...
vite v3.2.4 building SSR bundle for production...
transforming...
"GraphQLInterfaceType" and "GraphQLObjectType" are imported from external module "graphql" but never used in "src/lib/components/tags/TypeTag.svelte", "src/lib/components/common/text/TypeLink.svelte", "src/lib/model.ts", "src/lib/components/tags/NullableTag.svelte" and "src/routes/types/[type]/+page.svelte".
✓ 260 modules transformed.
1:20:28 [vite-plugin-svelte] ssr compile done.
package                                 files    time     avg
@magidoc/starter-carbon-multi-page        163   0.52s   3.2ms
Generated an empty chunk: "hooks"
rendering chunks...
.svelte-kit/output/server/vite-manifest.json                                           6.03 KiB
.svelte-kit/output/server/index.js                                                     73.29 KiB
.svelte-kit/output/server/entries/pages/_layout.svelte.js                              94.96 KiB
.svelte-kit/output/server/entries/pages/_error.svelte.js                               1.10 KiB
.svelte-kit/output/server/entries/pages/_page.ts.js                                    0.22 KiB
.svelte-kit/output/server/entries/pages/_...page_/_page.svelte.js                      0.88 KiB
.svelte-kit/output/server/entries/pages/_...page_/_page.ts.js                          0.38 KiB
.svelte-kit/output/server/entries/pages/mutations/_mutation_/_page.svelte.js           0.99 KiB
.svelte-kit/output/server/entries/pages/mutations/_mutation_/_page.ts.js               0.53 KiB
.svelte-kit/output/server/entries/pages/queries/_query_/_page.svelte.js                0.95 KiB
.svelte-kit/output/server/entries/pages/queries/_query_/_page.ts.js                    0.51 KiB
.svelte-kit/output/server/entries/pages/subscriptions/_subscription_/_page.svelte.js   1.00 KiB
.svelte-kit/output/server/entries/pages/subscriptions/_subscription_/_page.ts.js       0.56 KiB
.svelte-kit/output/server/entries/pages/types/_type_/_page.svelte.js                   23.27 KiB
.svelte-kit/output/server/entries/pages/types/_type_/_page.ts.js                       0.58 KiB
.svelte-kit/output/server/chunks/paths.js                                              0.17 KiB
.svelte-kit/output/server/chunks/index.js                                              9.93 KiB
.svelte-kit/output/server/chunks/ssr.js                                                0.12 KiB
.svelte-kit/output/server/chunks/index2.js                                             1.17 KiB
.svelte-kit/output/server/chunks/index3.js                                             2.27 KiB
.svelte-kit/output/server/chunks/ChevronDown.js                                        5.50 KiB
.svelte-kit/output/server/chunks/variables.js                                          0.26 KiB
.svelte-kit/output/server/chunks/stores.js                                             1.00 KiB
.svelte-kit/output/server/chunks/pages.js                                              3.59 KiB
.svelte-kit/output/server/chunks/index4.js                                             16.22 KiB
.svelte-kit/output/server/chunks/model.js                                              32.80 KiB
.svelte-kit/output/server/chunks/Button.js                                             6.82 KiB
.svelte-kit/output/server/chunks/PreviousNextPage.js                                   57.67 KiB
.svelte-kit/output/server/chunks/FieldDetails.js                                       18.49 KiB
.svelte-kit/output/server/chunks/ArgsList.js                                           7.18 KiB
.svelte-kit/output/server/chunks/hooks.js                                              0.00 KiB
  301 / -> /introduction/welcome
file:///private/var/folders/4d/5pyx_0k52vv0r0xlwd1vs66c0000gn/T/carbon-multi-page@2.14.4/node_modules/@sveltejs/kit/src/core/prerender/prerender.js:30
                                throw new Error(format(details));
                                      ^

Error: The following pages contain links to /queries/me#%E2%8C%98, but no element with id="%E2%8C%98" exists on /queries/me:
  - /queries/me
    at file:///private/var/folders/4d/5pyx_0k52vv0r0xlwd1vs66c0000gn/T/carbon-multi-page@2.14.4/node_modules/@sveltejs/kit/src/core/prerender/prerender.js:30:11
    at prerender (file:///private/var/folders/4d/5pyx_0k52vv0r0xlwd1vs66c0000gn/T/carbon-multi-page@2.14.4/node_modules/@sveltejs/kit/src/core/prerender/prerender.js:421:4)
[vite-plugin-svelte-kit] Prerendering failed with code 1
error during build:
Error: Prerendering failed with code 1
    at ChildProcess.<anonymous> (file:///private/var/folders/4d/5pyx_0k52vv0r0xlwd1vs66c0000gn/T/carbon-multi-page@2.14.4/node_modules/@sveltejs/kit/src/exports/vite/index.js:462:15)
    at ChildProcess.emit (node:events:527:28)
    at Process.ChildProcess._handle.onexit (node:internal/child_process:291:12)

For a more detailed output, run with --stacktrace

System Info
System:
    OS: macOS 12.5.1
    CPU: (8) x64 Intel(R) Core(TM) i5-8259U CPU @ 2.30GHz
    Memory: 784.23 MB / 16.00 GB
    Shell: 3.2.57 - /bin/bash
  Binaries:
    Node: 16.15.1 - ~/.nodebrew/current/bin/node
    npm: 8.11.0 - ~/.nodebrew/current/bin/npm
  Browsers:
    Chrome: 107.0.5304.121
    Edge: 107.0.1418.56
    Firefox: 107.0
    Safari: 16.1

### Severity

Blocking all usage of Magidoc

Thanks for raising the issue. Version 2.14.4 only bumped some dependencies. This must be a regression (or an improvement) in Svelte-Kit.
I'll have a look at it as soon as I get some time.

Thank you for taking the time.
For the time being, I will deal with this issue by setting the following configuration:
magidoc.mjs
/**
 * @type {import("@magidoc/cli").MagidocConfiguration}
 */
const config = {
  introspection: {
    // ...
  },
  website: {
    template: 'carbon-multi-page',
    templateVersion: '2.14.3'
  }
}

export default config

Hi @kulikala. I've just released a new version with bumped dependencies. It seems fine on my side now, so this must have been a bug in svelte-kit. Feel free to re-open if you still face the issue.

Hi @pelletier197 , thanks for your quick work in resolving this issue.
After upgraded to 2.14.5, it was generated without any problems!


The behavior of the menu item for hiding column becomes incorrect after the user reordered columns

Steps to reproduce the problem.
https://user-images.githubusercontent.com/4303130/149609204-82f15b6a-49e6-4480-a6d4-75ac3d8f28f5.mp4
It is caused by self.getSchema()[e.cell.columnIndex].hidden = true; in contextMenu.js.
And I will fix this bug along the PR for issue #420

https://github.com/hangxingliu/canvas-datagrid/commit/36d8714951c468992f3667d99136ece9f624463c


Licensing of trainings

Hey,
question @pbauer @polyester currently we do not have one.
Which one we should use ?

We do: https://training.plone.org/5/mastering_plone/about_mastering.html#license

Ak ok, sorry for the noise, I did not checked the sub dir :)
Closing now !
Thanks !


Dropdown in card-header

Hi, this is not a issue, is about a doubt.
By default can i put a dropdown menu in card-header? I was looking for in your docs but only found this:

The idea is a dropdown menu in that position like the uploaded image. Is it possible with your default CSS? Or i must to build a custom CSS?
Thanks in advance.

Works just fine - add the following markup to the card-header and edit as needed

It's also worth mentioning that app.js has markup specific to the behavior of anchors in that menu.
If your intention is to simply have them be anchors, then you can remove the markup in app.js - otherwise you'll need to alter it to suite your needs.

Thanks for ur answer @bellwood. This is my code put in card-header div and works fine!

            Link 1
            Link 1
            Link 1

Glad to help - feel free to close your issue if you are all set =)

Sure! Thank u!


Refactoring after a year

More hooks and more maintainable.

https://github.com/oxedom/parker/pull/29
Moving the whole project to Vue.js!


Unable to build: dependency error

When attempting to build using the rust version defined in docker, there is a dependency error. This can be worked around by using a more modern version of rust, such as 1.65. However, if using a newer version of rust, there are new compiler warnings and errors that prevent the program by building.
Can the codebase be updated to compile with the newer compiler and dependencies?

Dockerfile was updated, please try again

It builds in Docker now, but it won't start.
/usr/bgpexplorer/bgpexplorer: /lib/x86_64-linux-gnu/libm.so.6: version `GLIBC_2.29' not found (required by /usr/bgpexplorer/bgpexplorer)

This can be fixed by changing the runtime image to debian:bookworm-slim.

I would also suggest adding a .dockerignore file that is either a copy of the current .gitignore or a link to it so that docker builds don't include lots of extra unneeded build files.

Thank you. I have followed your recommendations.


Upgrade to elm-pages 2.0

Acceptance

Good docs on how to add content
better index & styling control

@Nikomus I'm part way through the port to elm-pages 2 and am removing the elm-ui stuff as part of it, so there may be some styling assist you can give soon. Maybe over the weekend?

Lol - one year on. I am actually doing this.

@katjam Will this help the site deploy on merge correctly? If not, we should probably wait to see what we need.

Yes, you are right. I am  just had a bit of time to lay a bit more groundwork - but I will get the deploy working today as a separate short task


Multiscale training

Added multi scale image loading to the oxparis dataset
Different scales can be given in a list of scales
scales can be chosen randomly or round robin

Added new scaling technique that preserves aspect ratio
Added square_pad option that must be activated in batched processing so that all images in same batch have same size
I use ImagePreprocessor for resizing and padding
I adapted joint_point_line_extractor to deal with padding in loss function
Multiscale learning is now implemented in oxparis and minidepth(partial)
Adapted merged_ds to allow multiscale learning

TODOs:
[] Add example configs for merged_ds training and multiscale learning
[] Test Merged DS with multiscale learning one minidepth gt is generated
[] If GT for minidepth is geenrated I need to finish minidepth and test it
[] If padding is used should I  add logic to remove keypoints that lie in padding area (altough this is unlikeli to happen)

FTR: @RaphaelKreft is currently generating MiniDepth ground truth and will merge this after he has also tested it with MiniDepth


Real devices are not detected by scan

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I have read the Contribution Guidelines

Issue Description
I need to run UI tests on multiple real devices. Even though they are visible through instruments scan is not able to detect them. Simulators are working fine.
Complete output when running fastlane, including the stack trace and command used
$ instruments -s devices
Known Devices:
Niklas iPhone (9.3) [AN-ID]
Test iPhone (9.2) [AN-ID]
iPhone 4s (9.3) [AN-UUID] (Simulator)
iPhone 5 (9.3) [AN-UUID] (Simulator)
iPhone 5s (9.3) [AN-UUID] (Simulator)
iPhone 6 (9.3) [AN-UUID] (Simulator)
iPhone 6 Plus (9.3) [AN-UUID] (Simulator)
iPhone 6s (9.3) [AN-UUID] (Simulator)
$ fastlane integration_test devices:"Niklas iPhone,Test iPhone"
16:54:08]: ------------------------------
[16:54:08]: --- Step: default_platform ---
[16:54:08]: ------------------------------
[16:54:08]: Driving the lane 'ios integration_test' 🚀
[16:54:08]: -----------------------
[16:54:08]: --- Step: skip_docs ---
[16:54:08]: -----------------------
[16:54:08]: ------------------
[16:54:08]: --- Step: scan ---
[16:54:08]: ------------------
[16:54:08]: xcrun xcodebuild -list -workspace ./Project.xcworkspace
[16:54:11]: Ignoring 'Niklas iPhone', couldn't find matching simulator
[16:54:11]: Ignoring 'Test iPhone', couldn't find matching simulator
[16:54:11]: Couldn't find any matching simulators for '["Niklas iPhone", "Test iPhone"]' - falling back to default simulator

+-----------------------+----------------------------------+
|                  Summary for scan 0.8.0                  |
+-----------------------+----------------------------------+
| scheme                | UITests                  |
| output_types          | html                             |
| devices               | ["Niklas iPhone", "Test iPhone"] |
| configuration         | Debug                            |
| workspace             | ./Project.xcworkspace           |
| clean                 | false                            |
| code_coverage         | false                            |
| address_sanitizer     | false                            |
| skip_build            | false                            |
| output_directory      | ./fastlane/test_output           |
| buildlog_path         | ~/Library/Logs/scan              |
| open_report           | false                            |
| skip_slack            | false                            |
| slack_only_on_failure | false                            |
| use_clang_report_name | false                            |
+-----------------------+----------------------------------+

Configuration Files
Please copy the complete content of your Fastfile and any other configuration files you use below:
Fastfile:
$:.unshift File.dirname(__FILE__)
require 'lib/utils.rb'
require 'shellwords'

default_platform :ios

platform :ios do
  lane :integration_test do |options|
    scan(
      scheme: 'UITests',
      output_types:'html',
      devices: options[:devices],
      configuration: 'Debug'
    )
  end
end

Environment
fastlane version (run fastlane -v): fastlane 1.95.0
Do you use bundler to execute fastlane (i.e. bundle exec fastlane)? No
Do you use a Ruby environment manager (e.g. chruby, rbenv, rvm)? No

I got it working using the destination parameter:
$:.unshift File.dirname(__FILE__)
require 'lib/utils.rb'
require 'shellwords'

default_platform :ios

platform :ios do
  lane :integration_test do |options|

    devices = options[:devices].split(',')
    destination = devices.map { | id | "platform=iOS,id=#{id}"}

    scan(
      scheme: 'UITests',
      output_types:'html',
      devices: options[:devices],
      configuration: 'Debug'
    )
  end
end