Add Personal Bank Account via Plaid flow

Details
This issue is sort of holding on https://github.com/Expensify/Expensify/issues/163233 where we'll add the Android app package name to the Plaid dashboard. I don't have the access level to do this so until that is done Android cannot be tested on production - but should be testable on dev and staging which are both using the sandbox. This is done now so we can test on release builds.
Fixed Issues
Fixes https://github.com/Expensify/Expensify/issues/161736
Fixes https://github.com/Expensify/Expensify/issues/149395
Notes on testing this on native platforms
This flow is only accessible via deep link at the moment so we'll need to follow instructions here
If you are using ngrok on iOS simulator make sure you also have certs on your simulator device. I wasted some time trying to figure out why iOS couldn't call secure.
Notes on testing on Android
In order to access local web-secure on Android it's necessary to create a separate endpoint for web-secure. I created a whole new one by following instructions here but a temporary one should work too.
After setting up a new "secure" ngrok url to use I ran this and everything worked fine.
ngrok http -hostname=secure-expensify-marc.ngrok.io --host-header rewrite secure.expensify.com.dev:443
Tests

Navigate to /add-bank-account
Enter "Chase" for the bank
Enter the test credentials user_good/pass_good
Select "Plaid Checking"
Enter Password
Tap Continue
Verify that the alert pops up and says the account has been added
Verify in expensify.com.dev that the account has been added

QA Steps (Internal)
QA is largely the same for staging. Making it internal because it may be tricky to get the deep links to work.
I can also do the QA for this (or help out) if whoever is assigned asks.
Tested On

[x] Web
[x] Mobile Web
Desktop / Not Accessible via Desktop
[x] iOS
[x] Android

Screenshots

Web

Mobile Web

Desktop
N/A
iOS

Android

Updated and fixed the conflicts.

I was unable to get the deeplink command to work on my physical iOS device or a simulator, so I'll need to finish testing that tomorrow.

Thanks @Jag96 let me know if you need any help with that - IIRC the expensify-cash:// style link is the one we need to use.

It looks like the font family/color isn't being set correctly for things like Select an account or this account has already been added

@shawnborton I was thinking we could do styling stuff in a follow up with a contributor if that works for ya. Mostly just trying to scaffold some stuff out here.

Cool, that works for me. Though it is a bit strange that our default  component wouldn't have the default font/color baked into it? But yeah, we can investigate that elsewhere.

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

@Jag96 Heyo! Will this be internally tested? Let me know so we can check it of the list. 🙇

This should be QA testable. With the exception of desktop since there are no deep links.

I'm happy to takeover the testing for this. I think I intended it to be Internal just because of how weird it is.

Web is working fine 🎉
Android doesn't work 🤔
Will test iOS next...
I think no need to block on this as long as the rest of the app works. Nothing is using the flow yet.

Oh hmm I think judging from logs the native Android app is not using the staging server...
I assumed that it would, but that seems wrong since there is no badge in the sidebar header.
cc @roryabraham to confirm.

Probably we will need to test this on production anyway to be sure it works - I think it will, but I don't have an account to test with as my bank is not an option in Plaid.

@marcaaron Are you asking if the E.cash Android beta (staging) app communicates with the staging API? If so, the answer is no – E.cash staging only communicates with the production API, by design. There was a fair amount of discussion around this already, but basically the reason is that if E.cash staging can't talk to our staging API, then there's no chance we deploy code that breaks E.cash production because it is dependent upon a staging API which has not yet been deployed.

Let me ask another way... 😅
https://staging.expensify.cash appears to use the .env.staging variables but native apps do not?

Ah, yes that's true. The reason for that is because when we run an iOS/Android deploy we just take an existing beta build and release it for production.


Refactor/call logger

Coverage decreased (-0.01%) to 1.209% when pulling 33bbefa4f88cc9e1f45eedbf4e99541186b2021b on u9520107:refactor/call-logger into 9911993a3bb1d8c8234eb68897df66100e7e0383 on ringcentral:master.


Keyboard navigation to scroll the response

It is not impossible to scroll up/down a long response in Github Copilot Chat window. I know that shift+tab moves the cursor to previous code blocks, but this is not enough. Furthermore, when a code block outside of the current view the cursor moves to a code block which is invisible.
Ideally, I would love to have something like caret mode from Firefox/Chrome, so I can freely navigate the response (even outside of code blocks). Having vscode commands to just scroll up/down by page is something minimal, but functional.
Screencast from 2024-08-18 13-52-48.webm

There are commands for this, list.scrollUp/list.scrollDown

I don't think list.scrollUp / list.scrollDown is a real solution. The scrolling speed is to slow. Check the demo:
Screencast from 2024-08-20 10-20-23.webm
Also, I couldn't figure out a when context I can use to limit the key bindings to the chat window. I have tried listFocus, but it doesn't work. Any suggestions? In my case Copilot Chat window is on the panel, not sidebar, so I assume it should be something like panelFocus && activePanel == 'workbench.view.???'?
  {
    "key": "pageup",
    "command": "list.scrollUp",
    // "when": "listFocus"
  },
  {
    "key": "pagedown",
    "command": "list.scrollDown",
    // "when": "listFocus"
  }

It seems reasonable to me, do you want something that would scroll by page, or the same as here but faster?
You could use the inChat context key

Thank you for inChat hint, it works!

It seems reasonable to me, do you want something that would scroll by page, or the same as here but faster?

If you ask me I believe there several improvements need to be done:

Scrolling by page is must have
Commands to scroll to the previous next answers, similar to workbench.action.terminal.scrollToPreviousCommand / workbench.action.terminal.scrollToNextCommand
The chat window should scroll automatically when the cursor goes out of the view via Tab/Shift+Tab shortcuts (in chat window these keys move the cursor to next/previous code blocks)

For 1, I opened https://github.com/microsoft/vscode/issues/226194
For 2, you can just use the arrow keys, or do I misunderstand?
Also agree for 3, there are some oddities with focus handling here, there is an open issue for this

For 2, you can just use the arrow keys, or do I misunderstand?

I don't think arrow keys are related. Let's imagine you talk to Copilot and it generated several responses, so the chat window looks like this:
Response 1
...long body of Response 1...

Response 2
...long body of Response 2...

Response 3
...long body of Response 3...

So, it would be great to have commands to scroll the chat window to previous/next Response.

Yeah, so you can use cmd+up to move focus from the input to the chat list, then arrow keys to navigate by response


questions on parameter x_j of message funcation in source code

❓ Questions & Help

I still don't understand what the message function parameter x_j means in the source code. In GCNConv, I only see x_j, while in GATConv there are x_i and x_j. I studied it for hours without understanding it

x_j corresponds to the features which gets aggregated, and x_i denotes the features which aggregate. The reason GATConv uses both x_i and x_j is because the attention scores gets computed based on both source and target node features.

x_j corresponds to the features of node j which gets aggregated, and x_i denotes the features of node i which aggregate. The reason GATConv uses both x_i and x_j is because the attention scores gets computed based on both nodes i and j.

Thank you very much for your timely reply!
When I studied the source code of GCNConv, I found that message function also needs x_i, because it is a sum of neighbor nodes and itself. But the reason, I think, why there's not x_i in the argument to the message function is because you've added  self-loops, which makes each node's neighbor contain itself, in other words, x_j contain x_i. Is my understanding correct?
Also, I saw in the source code for GTNConv that the self-loops were first removed and then added, I don't quite understand why to do so.

Yes and no. x_i and x_j are in edge parallel space, denoting the source and target node features for each edge. For aggregating the root node itself, we therefore have to add an additional self-loop. In message, you simply denote which message vector gets send to node i for each neighboring node j. This can be just based on the information given by j itself (GCNConv) or also based on node i (GATConv).
We remove self-loops first in case the user already provides self-loops in edge_index. Normally, the removal is a no-op.

Yes and no. x_i and x_j are in edge parallel space, denoting the source and target node features for each edge. For aggregating the root node itself, we therefore have to add an additional self-loop. In message, you simply denote which message vector gets send to node i for each neighboring node j. This can be just based on the information given by j itself (GCNConv) or also based on node i (GATConv).
We remove self-loops first in case the user already provides self-loops in edge_index. Normally, the removal is a no-op.

Thank you very much. Now I realize that the reason I cannot understand it is because I cannot understand how the propagate function is implemented. I am not a very professional programmer. This leads me to be unclear about what I need to write myself when I try to define my own layer. I tried DGL and found that I could easily implement GCN and GAT. Maybe this library will be more suitable for me

The MessagePassing interface is really similar in both libraries. In PyG, propagate first computes messages for each edge (j,i) in the message function and aggregates those afterwards using on the of the aggregation operators add, mean or max for each neighborhood. In the message function, you have access to source and target node features and edge features to define messages, .e.g.,
def message(self, x_j, x_i, edge_weight):
    return self.mlp(torch.cat([x_j, x_i], dim=1) * edge_weight.view(-1, 1)

which concatenates source and target node features, transforms them using a MLP and weights them based on edge weights.

Thank you for your patient answer! You must be an angel! I think I'm close to mastering the message function. However I still don't know the argument 'edge_index', 'edge_index_i', 'edge_index_j' and how do I know the arguments of self.propagate()?

edge_index describes the connectivity of your graph in a sparse COO format. This is needed as an argument to propagate in order to derive which messages should get passed to which nodes. This is the only required argument to propagate, all other ones are optional. You simply pass in all arguments there which are needed to create messages. edge_index_i is just a way to say derive all the target node indices from all the edges, and can be used to, e.g., normalize features around each neighborhood.

I think I will master it after studying more source code, thank you for your help!
BTW, I am currently dealing with a problem that requires updating the node features and updating the edge features. I noticed that the MessagePasing framework is based on updating node features. Is there any way I can add the update edge feature process to the model?
I think this process will become more and more important in the future for some tasks such as link prediction and link classification, etc.

Well, updating edge features is relatively easy using plain PyTorch methods, so an own interface is not really necessary. Simply do something like:
row, col = edge_index
new_edge_attr = self.mlp(torch.cat([x[row], x[col], edge_attr], dim=1))

You are right, it is pretty easy, I just need to add this code to the forward function. I should have thought of it...

In pytorch_geometric lib you always treat the col as target and row as source right? By default the message flows from row to col which the edge_index list can be denoted as (j,i)?


HIVE-27698 : Backport of HIVE-22398: Remove legacy code that can cause issue with new Yarn releases

JIRA link - HIVE-27698

@vihangk1 @sankarh Can you please approve this. A clean cherry pick


Estimate difficulty of mapping XBRL tables to historical tables (or vice versa)

Since we're moving away from trying to fully transform XBRL data back to the structure of the historical data, we should provide an estimate of the difficulty to perform that transformation for a subset of tables that are in use with RMI, and in PUDL.

Created spreadsheet with difficulty estimates.


Utilising AWS Pseudo-parameter types for optional parameters

PR Opened to resolve the issue discussed here
I have just gone through the .yaml files and identified where we can use AWS specific parameter types and updated them from String to the specified parameter type. The supported AWS specific parameter types are outlined in the following AWS Docs
For example: String -> AWS::EC2::SecurityGroup::Id

Have you tested these yet? e.g. deployed the templates with values supplied for those changed parameters as well as without an explicit value given for those that are optional ("(optional)" in description and Default: '' property supplied).
if tested: return "approved"
else: return "Request changes"

Ah no haven't tested this yet!

Maybe try AWS::NoValue instead of '' for default values

Currently this does not work --
Parameter validation failed: parameter value AWS::NoValue for parameter name HostedZoneID does not exist. Rollback requested by user.


Invalid opcode (node binary fails in qemux86)

Hello!
I'm having issues getting node to work on a core-image-minimal build. In conf/local.conf, I've set RDEPENDS_${PN} = " nodejs" but when the image builds and I emulated it with QEMU, I get this error:
$ node
[142.374544] traps: node[361] trap invalid opcode ip:85f9238 sp:bfc1f030 error:0
[142.374706] in node[8048000+ee8000]
Illegal instruction

What am I doing wrong?

I'm having similar problems. Only qemux86 generate ´trap invalid opcode´, while qemux86-64, qemuarm64 work with no problems.
I'm just not sure whether this should be reported here or on openembedded-core.
I'm not using meta-nodejs, just arrived here after googling for the problem.


Move ios 'Building your app for production' to 'Publishing to Apple App Store'

Description
Currently, there is a section in the docs on Publishing to Google Play Store.
However, there is no section in the docs on Publishing to Apple App Store.
The problem is that vital information about building for the Apple App store is buried here: https://reactnative.dev/docs/running-on-device, in the section 'Building your app for production'. For example, this section mentions that you need to enable app transport security before submitting to the App Store.
I think this information should be moved to a new section called 'Publishing to Apple App Store', which would make it much easier to find.
Happy to PR this, if people agree.
Documentation version
0.63

@sradc Thank you for bringing this up, I think that extracting Building your app for production section to the separate file under Guides (iOS) is a good idea! Feel free to work on this PR!
Ideally iOS Running on Device should also contain the small paragraph with the link to the new page like the Android guide does:


question about loss function

Thanks for your great work!
My question is about the loss function of 'chordal_distance' which measure the eliment-wise error of the transformation matrix. Intuitively, it seems that this approach would perform worse than using geodesic distance. Could you please share the reasons for using 'chordal_distance' as the loss function?

Hello @Ray0089,
Both chordal and geodesic distance are valid distance metrics on SO(3) and SE(3).
Please refer to the following resources:

Rotation Averaging, sec 4
Partial Bi-Invariance of SE(3) Metrics, A Survey of Metrics
On the Continuity of Rotation Representations in Neural Networks

We found that using chordal distance during training is more stable and less sensitive to numerical errors, while geodesic distance is more sensitive and the gradient is unstable in the edge cases (e.g. values near pi).
Note that chordal distance shares similarities with the 6D rotation representation, as both uses matrix representationsto calculate the distances in the embedding space but with a different number of columns: the latter uses the first two columns of the rotation matrices, while the former uses all three columns.


Global Content actions

Now that the Indexing module is a multi-usage way to track all content items, we should have a module that would accept a Job/Task/Activity (find correct name) and track it has been applied on all existing content items.
Scenarios:

When adding a part to a content type, we also want to add the data to all existing content items
During a migration, we need to change the data of all existing content items, or a set of them
Detecting data corruption

This module would provide a UI and some storage to track all the registered activities and their status (like for the search module that tracks all lucene-indexed items)
A background task would be responsible to applying the activities to all items (like in the search module).
Because these activities have to run only once if multiple nodes, the storage should be in the db (as opposed to what the Lucene module does). Or should we support both local and global tasks explicitly and make the lucene module reuse this logic?
@Piedone

With global/local tasks I understand you mean at tenant level vs at node level. However maybe is interesting to also think in at app level.
In Orchard 1 when you have several tenants with  number of background task per tenant you can find cpu can have a level of consumption higher than the desired because it is consumed by background task even your sites doesn't have traffic. The reason is each tenant is competing for the cpu with other tenants for executing its background tasks when in fact, those task are not so priotitary for you to want it happen.
As a solution, what do you think of having a general scheduler of background tasks that will execute background tasks for all the tenants in sequential order avoiding tenants compete for the cpu? First background tasks of tenant 1, second background tasks of tenant 2, and so on.
This scheduler should be configurable to decide how many threads do we want at the same time. So, setting a number equal to the current number of tenants will be equivalent to how orchard works now.
If we think it is interesting we also could have the possibility of having more than one scheduler working in parallel. We could assign to each scheduler the background tasks we are interested it executes through Admin UI. We would assign to each of them different numbers of threads depending on the priority we want to give to background tasks assigned to that scheduler.
What do you think?

I like the idea of implementing explicitly local task logic by Orchard a reuse that logic in Lucene module.  Maybe Orchard 2 should have an app scheduler, a global scheduler(per tenant) and a local scheduler(per node). For app scheduler user can decide how many threads will be executing it. but for the other two not.
Then user also would be able to decide in which scheduler will be executed each background tasks depending on its priority (app vs global) or the location where it needs to be executed (local vs the other two).
Problem I see for that is it could be confusing for people using only one tenant. Maybe app scheduler should be a feature disabled by default

Well, if app scheduler can behave similar to global scheduler when it is configured to use same number of threads as tenants running then I would have only app scheduler and local scheduler. The option of having an extra app scheduler with different configurations of number of threads for giving more or less priority to its tasks is sth I'm not sure it is needed.

I haven't been following the meeting the last two-three weeks or so I'm not familiar with what Indexing does now. However if the question is what the ideal task scheduler/background task/job manager in Orchard 2 should do, I have this to add (sorry if I'm reiterating something already discussed or implemented):

It should have some mechanism to ensure that a task is only executed on one node of a multi-node setup (like IDistributedLock in O1).
They should handle app shutdown other then simply being halted (https://github.com/OrchardCMS/Orchard/issues/4417 and https://github.com/OrchardCMS/Orchard/issues/4620).
Good point about priority from @jersiovic .
Ideally the tasks executing should be visible to the user (at least on a level that "I have this and that bg tasks/jobs executing on this tenant"). Also see the comments under https://github.com/OrchardCMS/Orchard/issues/4842.

@jersiovic I don't want to digress and talk about the design of bg tasks in this issue, and in O2 you can already group bg tasks by groups, each group having it's own thread loop and timer, which you can use for priority management. Also an app scheduler and a node scheduler are the same thing. But I understand and would agree with your idea of trying to have a common scheduler for all tenants on a node. Switching service providers is already in place, so having a common scheduler would not be hard at all. I will think about it
@Piedone @jersiovic Because I don't see any remark about the original subject, please read it again and give any comment without mentioning bg tasks but just content management.
@Piedone the indexing module in O2 is just the part from O1 that is creating an indexing task record for each content item (Delete/Update) and when a content item is changed. It doesn't do anything else, it more a Tracking module than indexing, but the primary goal is to index content, either from a search module, or the projection module, or anything that needs to apply some logic to all content items, and store the cursor location to do the processing in order, and in multiple times if necessary (batches).

Got it. So what I would do (again, I may have outdated knowledge) is the following:

Have a generic service for executing tasks for content items.
Make Indexing use that.

I would imagine something like scheduled tasks (sorry to mentioning it again), e.g.:
public interface IContentItemTaskHandler : IDependency
{
    void Process(IContent contentItem, string taskType);
}

So have the infrastructure, then let the consumers decide how they want to utilize it. In case of Indexing there would be nothing too special, but otherwise you could e.g. wrap the body of your task's Process into a distributed lock.

Make sure it's async :)
Sent from my iPhone

On 19 Dec 2016, at 20:26, Zoltán Lehóczky<EMAIL_ADDRESS>wrote:
Got it. So what I would do (again, I may have outdated knowledge) is the following:
Have a generic service for executing tasks for content items.
Make Indexing use that.
I would imagine something like scheduled tasks (sorry to mentioning it again), e.g.:
public interface IContentItemTaskHandler : IDependency
{
void Process(IContent contentItem, string taskType);
}
So have the infrastructure, then let the consumers decide how they want to utilize it. In case of Indexing there would be nothing too special, but otherwise you could e.g. wrap the body of your task's Process into a distributed lock.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


[Bug]: could not read Username for 'https://github.com': No such device or address

Describe the bug
The error just occurs when obsidian tries to read from github, the commits are fine from what i know
Relevant errors (if available)
plugin:obsidian-git:10760 Uncaught (in promise) Error: fatal: could not read Username for 'https://github.com': No such device or address

    at Object.action (plugin:obsidian-git:10760:25)
    at PluginStore.exec (plugin:obsidian-git:10785:25)
    at eval (plugin:obsidian-git:8505:43)
    at new Promise (<anonymous>)
    at GitExecutorChain.handleTaskData (plugin:obsidian-git:8503:16)
    at GitExecutorChain.eval (plugin:obsidian-git:8487:44)
    at Generator.next (<anonymous>)
    at fulfilled (plugin:obsidian-git:7339:24)
action @ plugin:obsidian-git:10760
exec @ plugin:obsidian-git:10785
eval @ plugin:obsidian-git:8505
handleTaskData @ plugin:obsidian-git:8503
eval @ plugin:obsidian-git:8487
fulfilled @ plugin:obsidian-git:7339
plugin:obsidian-git:15062 git obsidian message: Committed 0 file

Steps to reproduce
install obsidian via .deb
clone the repository
open the folder on obsidian
enable the plugin and set up active sync
1 and 2 minutes respectively
watch the error as you cry of frustation
Expected Behavior
An error on the top left corner saying "could not read Username for 'https://github.com': No such device or address"
Addition context
I have seen that the deb inst commonly used, but its an official version as shown in this screenshot

So that could be strangely it
Operating system
Linux
Plugin version
1.28.0

Also, can this extension work with shared repositories like with other accounts instead of only 1? or it could broke that way

Authentication is kind of a problem, especially on Linux, because git doesn't have credential caching by default.
When pushing via terminal you have to enter your credentials, which is not possible in Obsidian, so it fails.
An alternative is to use ssh. You either use an ssh key with passphrase and add it to the ssh-agent or use a ssh key without passphrase.
What exactly do you mean with multiple accounts? Using multiple accounts on the same machine or your friend who is using the same repo? Both cases shouldn't be a problem.

Alright, i will change to ssh, yeah, i was using a Personal access tokens because on top of linux dont having credential caching, for some reason i couldnt use the password, it tells me that for security reasons now i cant use passwords.
brb

It worked! thank you!
I have seen that there are many common/easily fixable problems that aren't on the wiki and i would like to help adding them to it so there's less "filling" on the issues section and everything is streamlined


Profane inside word

Hi, I'd like to use this to check if usernames include profane words, however, usernames are all one word and is_profane isn't able to pick up profane words when they are connected to other words. Can you add support for this? For example, if an inputted username is f***youusername it should mark as profane.

Noted. Thanks.

Thanks!

This doesn't seem to apply to is_clean

Could you provide the sample code? Because is_clean is passing in https://github.com/areebbeigh/profanityfilter/blob/master/tests/test_profanity_noboundries.py

Sorry, my mistake, I was testing without boundries turned off.


Warning when using HMR and

<StylesProvider injectFirst={true}> produces the following warning:
Warning: Material-UI: you cannot use a custom insertionPoint and <StylesContext injectFirst> at the same time.
The warning only comes when the code is reloaded or rerendered (I'm not quite sure). It may be related to hot-module reloading. For me it occurs with storybook 5 after reloading via HMR. However, it can also be seen in the codesandbox (after the sandbox is loaded change something and wait a short time, the warning appears then):

[x] This is not a v0.x issue.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior 🤔
no warning
Current Behavior 😯
warning
Steps to Reproduce 🕹
https://codesandbox.io/s/2pzvn6l49j
Context 🔦
I use <StylesProvider injectFirst={true}>  together with emotion 10 (css-prop).
Your Environment 🌎

Tech
Version

Material-UI
v4.0.0-beta.0

I see this was merged 2 years ago into next does that mean it's in version 5? I'm in 4.12.3 and still have this same error occurring.

Same issue occuring here. I am on v4.12.3,
Is there no more support for v4? Are we SOL until we upgrade to v5.....?

Is there no more support for v4?

Correct, I'm locking


feat: improve rendering performance on windows

It works fine on my windows. I felt a little rendering improvements.


Ic-Step no cursor pointer when on click provided

Summary of the bug
When creating a stepper with steps when you provide an onclick to the step and hover the mouse over the step the cursor does not change to the pointer, it stays as the default. Also the clickable region for the Ic-Step includes the line to the left/right of the circle which feels incorrect, you should only be able to click the step itself.
🪜 How to reproduce
Tell us the steps to reproduce the problem:

Create a stepper with three steps
Provide an onclick function to all steps
Hover over the step and see that the cursor does not change
If you provide manual style to Ic Step to change the cursor style you'll see you can click also click the line which feels wrong

🧐 Expected behaviour
When an onclick is provided to Ic Step and the mouse hovers over the circle of the step the mouse changes to a pointer.
You can only click the circle and not the line to call the onclick
Additional info
On ICDS version 2.1.0-beta.8

Thanks very much for raising this issue, hopefully the following information will help.
The current version of the stepper does not support clickable steps; it has been designed as a progress indicator and not a navigational component. The component guidance page describes the stepper's role and it is suggested to use a separate navigation control to move between step content. As the addition of the onClick event is a customisation to the component, the cursor management would need to also be provided as a customisation here.
In the future, we would look at including other patterns for allowing navigation between different steps in a process. One of these will be a clickable stepper, whilst another will be a summary page component with edit links.
We will close this ticket as other tickets will be raised to for the other components mentioned above.
Thanks very much


Add inspireHEP to sources

Inspire HEP (  https://inspirehep.net/  ) is one of the most important platforms to search for papers in High Energy Physics.
Every paper has a DOI link or an e-print link that is simply a link to arxiv.

Ok! Will do in the next release (probably September)


Automation block has a scroll that only leads to white space

Automation blocks with for example just 1 button have a scroll that can only be used to scroll down to white space.

Link:
https://demo.crust.tech/crm/pages/69199831280648209/record/69766934266118161

Solved along with https://github.com/crusttech/webapp-crm/issues/37


Tile Inspector can't be opened when Mine Beacon is on SE or SW edge

Most likely the same off-by-one issue as #788, could perhaps be centralized (DRY).

TileMap.cpp, line 292:
bool TileMap::tileHighlightVisible() const
{
	return NAS2D::Rectangle<int>::Create(mMapViewLocation, NAS2D::Vector{mEdgeLength - 1, mEdgeLength - 1}).contains(mMapHighlight);
}

Quite right, both about the bug, and about the proposal to clean it up.


investigate acceleration structures

Spatial partitioning over BVh probably, due to near uniformity of primitive distribution. Does traversing a connectivity list beat out a spatial partitioning structure?
Models conform to rectilinear grid, but additional partitioning below that may be necessary. Should a MPT or full mesh be traced? Better parallelism but harder partitioning if MPT?

Single level or multilevel structured grid is best


require() of ES Module federation.config.js  from load-federation-config.js not supported.

error:
require() of ES Module app/federation.config.js from<EMAIL_ADDRESS>not supported.

federation.config.js
import {
  shareAll,
  withNativeFederation,
} from "@angular-architects/native-federation/config";

export default withNativeFederation({
  shared: {
    ...shareAll({
      singleton: true,
      strictVersion: true,
      requiredVersion: "auto",
    }),
  },

  skip: [
    "rxjs/ajax",
    "rxjs/fetch",
    "rxjs/testing",
    "rxjs/webSocket",
    // Add further packages you don't need at runtime
  ],
});

env
node v20
npm v

 dependencies"
 "dependencies": {
    "@angular/animations": "^17.0.0",
    "@angular/cdk": "^17.0.0",
    "@angular/common": "^17.0.0",
    "@angular/compiler": "^17.0.0",
    "@angular/core": "^17.0.0",
    "@angular/forms": "^17.0.0",
    "@angular/material": "^17.0.0",
    "@angular/platform-browser": "^17.0.0",
    "@angular/platform-browser-dynamic": "^17.0.0",
    "@angular/platform-server": "^17.0.0",
    "@angular/router": "^17.0.0",
    "@angular/service-worker": "^17.0.0",
    "@angular/ssr": "^17.0.0",
    "@angular-architects/module-federation": "^17.0.0",
    "@ngx-formly/core": "^6.2.2",
    "@ngx-formly/material": "^6.2.2",
    "express": "^4.18.2",
    "http-proxy": "^1.18.1",
    "ngx-quill": "^24.0.4",
    "rxjs": "~7.8.1",
    "tslib": "^2.6.2",
    "zone.js": "~0.14.2",
    "es-module-shims": "^1.5.12"
  },
  "devDependencies": {
    "@angular-devkit/build-angular": "^17.0.0",
    "@angular-devkit/core": "^17.0.0",
    "@angular-devkit/schematics": "^17.0.0",
    "@angular/cli": "^17.0.0",
    "@angular/compiler-cli": "^17.0.0",
    "@nx/angular": "18.2.4",
    "@nx/workspace": "18.2.4",
    "@schematics/angular": "^17.0.0",
    "@types/express": "^4.17.21",
    "@types/jasmine": "~5.1.4",
    "@types/node": "^20.10.4",
    "jasmine-core": "~5.1.1",
    "karma": "~6.4.2",
    "karma-chrome-launcher": "~3.2.0",
    "karma-coverage": "~2.2.1",
    "karma-jasmine": "~5.1.0",
    "karma-jasmine-html-reporter": "~2.1.0",
    "nx": "18.2.4",
    "typescript": "~5.2.2"
  },

`

Any news on this?


Consider using Microsoft.Bcl.AsyncInterfaces for IAsyncEnumerable

Now that Microsoft.Bcl.AsyncInterfaces (https://github.com/dotnet/corefx/blob/bfcbf22e72bd5f814af92c0980993a8b4e9c7232/src/Microsoft.Bcl.AsyncInterfaces/ref/Microsoft.Bcl.AsyncInterfaces.cs) is being produced, contains relevant IAsyncEnumerable interfaces and targets netstandard2.0 consider using it for netstandard2.0/2.1 builds of System.Linq.Async
cc @onovotny

Will do once it's on NuGet.org.


Leveraging PhantomJS settings, viewportSize, and others

Hi,
Great work on an awesome plugin! I used to use Chrome DevTools for this task but it's not very friendly when you need to output a clean CSS file. This works nicely. Kudos!
I was wondering if there was a way to leverage PhantomJS settings (e.g. userAgent) and viewportSize?
Thanks.

Not as far as I'm aware. All options are passed through to UnCSS, it's probably worth opening an issue there for tracking this. If/when this lands it'll be automatically available in this module.


Favorite subreddits

I have completed the "Favorites" feature for subreddits.

I added one page in left menu (called Favorites) where you can see the list of your favorite subreddits.
In subreddit page, i have added a heart button next to title where you can add or remove the subreddit from favorite list.
I used the ionic storage  for saving user's preferences

Thanks @meletakis
I will review changes closer to the weekend
have a good day :)

I think that we have to restructure the left menu options cause i saw that @crock started a settings page.
I suggest to develop one page called subreddit on the left menu, where you should see the textfield "Go to subreddit" AND the list of your favorite subreddits.

@meletakis @crock you can chat about menu here https://gitter.im/ionic2-reddit-reader/Lobby?utm_source=share-link&utm_medium=link&utm_campaign=share-link


Docker Local Node stops working with PLATFORM_NOT_ACTIVE error after a while

Description
I am running the local node on my Mac Studio. Set up with
npm install @hashgraph/hedera-local -g
then
hedera start
Docker specs are set as requested.
Localnode works fine for a while, then it seems to time out. Last log in network-node container is:
Hedera - HederaNode#0 is CHECKING
As you can see, early it goes CHECKING -> ACTIVE

The error I now get run trying to create tokens is:
Error: max attempts of 10 was reached for request with last error being: PLATFORM_NOT_ACTIVE
Steps to reproduce
Run local node for a while, then stop, then try again (sorry... not very specific)
Additional context
No response
Hedera network
other
Version
2.31.0
Operating system
macOS

Something else (minor I think)
I just got this error:
Error on xterm-256color.Setulc:
"\u001b[58::2::%p1%{65536}%/%d::%p1%{256}%/%{255}%&%d::%p1%{255}%&%d%;m"

var v,
 stack = [],
 out = ["\x1b[58::2::"];
(stack.push(v = params[0]),
 v),
(stack.push(v = 65536),
 v),
(v = stack.pop(),
 stack.push(v = (stack.pop() / v) || 0),
 v),
out.push(stack.pop()),
out.push("::"),
(stack.push(v = params[0]),
 v),
(stack.push(v = 256),
 v),
(v = stack.pop(),
 stack.push(v = (stack.pop() / v) || 0),
 v),
(stack.push(v = 255),
 v),
(v = stack.pop(),
 stack.push(v = (stack.pop() & v) || 0),
 v),
out.push(stack.pop()),
out.push("::"),
(stack.push(v = params[0]),
 v),
(stack.push(v = 255),
 v),
(v = stack.pop(),
 stack.push(v = (stack.pop() & v) || 0),
 v),
out.push(stack.pop())}out.push("m");
return out.join("");

when running hedera restart

Make sure that Docker's resource configuration (such as CPU and memory) is sufficient to prevent the node from stopping responding due to insufficient resources.
If it still doesn't work, you can try using Servbay instead of docker, which is better than docker on mac and has fewer bugs

Okay, CPU and memory is set up to the levels suggested in the docs. CPU is actually way higher than whats recommended. What would you recommend increasing memory to? +20%?
hedera restart command always seems to fix it at least

@natanasow  seems better now


Full java 8 migration and Spring upgrade

Hi folks I've started an upgrade of the esup-filemanager portlet.
I've just pushed a new branch here  (not all work finished), so please provide a feedback on use (I'm on hollydays now and won't have really more time to work on).
For testing you can:

switch to WIP_java8 branch
make a package locally with:
export JAVA_HOME with java 8 link
run mvn clean package install

and deploy it into a local uPortal install by upgrading to 4.0.1-SNAPSHOT version on gadle.properties

go on My Drive portlet
navigate on directories and try to upload/download

In b6794e0, the esup and esup-third repositories were removed, but it appears they are needed for the com.googlecode.sardine:sardine dependency as it does not exist in Maven Central and is pulled from esup/esup-third.
Once that dependency is resolved, I was able to build the portlet and deploy it to uPortal-start. From there I was able to navigate, upload, and download.

@jonathanmtran thanks a lot ! For the esup an esup-third repositories it remains to update dependencies version, I will try to have a look on so...
After, I don't know if I try to update the jquery version, or I let all UI part for a bigger improvisent like à WC UI.


Daisy chaining of MCPWM timers (IDFGH-13953)

Is your feature request related to a problem?
Yeah, so I need PWM (for lighting leds, not just some signalling) that is capable of 16 bits resolution or more. The 14 bits of the LED-PWM on the ESP32-S3 simply doesn't cut it, shame on you for leaving out the "high frequency" LED PWM module ;-)
So I am now turning to the MCPWM module (2x) which will probably do better (it looks like timer width is 16 bits).
Of course I am looking for more. The frequency is not the problem, I am aiming at 100-200 Hz and both types of modules can achieve that easily at the highest resolution.
Now my question is, according to the documentation, MCPWM timers within a group can be linked/synced. Would it be possible to daisychain two of them? Like this scenario: timer 0 counts up to 0xffff, when it overflows it kicks timer 1 and timer 1 progresses than exactly one value (0x0000 -> 0x0001 etc). Like timer 0 is the clock for timer 1. The generators could be linked to timer 1 and up to 32 bits of resolution could be achieved. I don't think it can be done, timer 1 will start running at the rate of the connected clock, not? Maybe there is a workaround? Maybe something likewise can be done with the LED-PWM module, that would be fine too. I really don't need all of the 8 channels ;-)
Thanks for your input!
Describe the solution you'd like.
No response
Describe alternatives you've considered.
No response
Additional context.
No response

Yeah that would be great and probably not too complex to implement in silicon I assume? Something like timer 0 overflow is optional clock level toggle for timer 1.
However a wider timer would be even better of course. Something like 20 bits? Even better: a wider timer on the led pwm controller, I am using the mcpwm purely for the timer width now, which is quite a bit complex for just driving leds.
Thanks!


WebGLPointsLayer

Hello. I am using OL to display 250k+ points in different layers. Every layer create how WebGLPointsLayer, but it loads the graphics card a lot. If I create one WebGLPointsLayer with 250k+ points it works great. How do I need to load points correctly in different layers?

You could probably put all the points in the same layer with different styling. Still, that is a lot of data, how much FPS do you end up with typically?


rustup

@bors r+

:pushpin: Commit 5f825ae895adbd2cfeff2fb0fc1a2e411bcac91a has been approved by RalfJung

:hourglass: Testing commit 5f825ae895adbd2cfeff2fb0fc1a2e411bcac91a with merge 7decf8c7202663e58ccc9ad84f01abe1ac77e128...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing 7decf8c7202663e58ccc9ad84f01abe1ac77e128 to master...


test: Tests for connection limit, compaction stats

- What I did
Added functional tests for compaction and connection limit
- How I did it
Used the config:set verb to set the limit to the required value
- How to verify it
tests should pass
- Description for the changelog

Closing this PR as this is very old and 'll be raising a new PR with the new changes for the same.


Add Checkbox and CheckboxGroup components

This PR adds two new components, and their supporting documentation:

Checkbox
CheckboxGroup

They are based on RAC Checkbox and CheckboxGroup.
Checkbox renders a focusable/selectable <div> and a hidden <input>. The text label is set via the children slot:

Checkbox supports the following states:

required
disabled
read-only
indeterminate

Note: Checkbox can be used independently of a CheckboxGroup, but using it as a standalone component requires additional work to add validation and error handling that CheckboxGroup has built-in.
CheckboxGroup provides a semantic wrapper for one or more checkboxes:

It includes these slots:

children (expects an array of Checkbox components)
label
description
errorMessage (via FieldError)

Additionally, the orientation prop can be used to toggle the layout of a group between vertical (default) and horizontal.

I've made changes to the Checkbox and CheckboxGroup to bring in our latest changes, update the styles to match our most up to date design, and add tests. With these changes, I think we're ready to merge. Will mark this as approved and leave it up to you to confirm, @mkernohanbc.

Functional updates

flexWrap prop added to CheckboxGroup to allow wrapping horizontal options rather than overflowing caf74a5, eb24ea6
Custom TypeScript interface removed from Checkbox in favor of just using the RAC defaults (no difference in how the component is used, just cleaner this way) 9cb5205
children removed from interface in CheckboxGroup in favor of the RAC default (no difference in use) 4976771

Style updates

Cursor styles are applicable for the entire checkbox + label, not just the box itself a9bfa01
Disabled checkbox label gets disabled font color 9a036ab
Add styles for isIndeterminate + isDisabled dd65760
Add styles for isIndeterminate + isInvalid 01f9f22
Checkbox flex gap changed to 10px to match current design 6206192
Checkbox border styles updated to match current design (border color matches background color in most filled states) 8073da6

Tests

Checkbox d103d15, 9d26371
CheckboxGroup bd6f5ce, 400cd6c

Storybook updates

Add isInvalid and isSelected props to Checkbox template b4d9797

Merge latest from main afce1ce

isIndeterminate + isDisabled:

isIndeterminate + isInvalid:

flexWrap defaulted to nowrap:

flexWrap set to wrap:

Thanks for all your work to clean this up @ty2k — looks good to me! Will squash and merge now.


Executable 'rviz_targetpose_marker.py' not found error

Hi.
The following error was found during program execution.
Were is 'rviz_targetpose_marker.py'?
$ ros2 launch px4_manipulation run.launch.py
[INFO] [launch]: All log files can be found below /home/quad/.ros/log/2024-02-25-00-46-06-522995-Thinkpad-684214
[INFO] [launch]: Default logging verbosity is set to INFO
[ERROR] [launch]: Caught exception in launch (see debug for traceback): executable 'rviz_targetpose_marker.py' not found on the libexec directory '/home/quad/ros2_ws/install/px4_manipulation/lib/px4_manipulation'

I found!!!
The file was in the /tools directory. :)


bug: another divide by zero error in design_optimisation.py

File "../../bad/optimisation.py", line 89, in design_optimisation
    U = (1/nD)*np.ones(nD)
ZeroDivisionError: division by zero

Maybe zero designs are being passed in? Overaggressive culling of designs in the heuristic code?
This definitely happens when we forget to provide inputs to one of the design variables. So:

[x] add validation checker of provided design variables before we build all the designs. Do this in DARC_Designs.__init__()
[x] refactor DARC_Designs.__init__()

There may or may not be other things which cause this error.
Add some asserts:

[x] design_optimisation() should error if there are no (or very few) in designs
[x] add a logging.warning in DARC_Designs.refine_design_space() if there are very few designs being generated

No designs being available was the primary (only?) cause of this error, and it is now being caught and warned about in multiple places. The problem was primarily happening when we had a mis-structured design space spec that we were asking for in the magnitude effect style design spaces. This is a separate issue, see #28.


Support for Extended GTFS Route Types

NOTE: This issue system is intended for reporting bugs and tracking progress in software development. Although this software is licensed with an open-source license, any issue opened here may not be responded to in a timely manner. Conveyal is unable to provide technical support for custom deployments of this software unless your company has a support contract with us. Please remove this note when creating the issue.
Observed behavior (please include a screenshot if possible)
No support for creating and editing routes with Extended GTFS Route Types
Expected behavior
Support for editing Extended GTFS Route Types
Steps to reproduce the problem
I would like to create 1500 Water Taxi Service and 1000 Water Transport Service route types.
Any special notes on configuration used
none
Version of datatools-ui and datatools-server if applicable (exact commit hash or branch name)
UI Version: | 872d61
Server version: | 11e9a8
Is the support for Extended GTFS Route Types on the roadmap? Or is as simple as editing the .yml config files and creating a pull request for this?

Hi @mvanlaar, there is support for this in datatools-server:
https://github.com/conveyal/gtfs-lib/blob/048778369dd9d62180fd01cd403b30f9d0ce6966/src/main/java/com/conveyal/gtfs/loader/Table.java#L178-L180
But you're correct in that the UI does not support these values in the Editor. I hesitate to simply update the gtfs.yml file with all of the extended route types (or even just cherry-picking a few), just because they are less commonly used and would complicate that form element for most users. Another approach (though this complicates the implementation) would be to show the primary options and allow custom input for values between, say, 13 and 1800. This issue also highlights that the routes#route_typefield in gtfs.yml is missing values 8 through 12, which have already been incorporated into the GTFS spec.
I'll take a look at the code and will update this ticket with more info.

I've added in my local copy the google supported values. Thats working.

We could try to make it a optgroup for the primary values and a optgroup for the secondary options.


Remove and disallow termination checks for exec functions

I do not believe this opens to any potential unsoundness, especially because it's already possible to write a non-terminating recursive exec function with a loop.
Reported by @matthias-brun.

@Chris-Hawblitzel I'm going to land this but please have a look when you get the chance.

I am not confident this is okay.

Can you say what the motivation for this change is?  I could imagine having an option that either disables termination checks for both exec recursive functions and exec loops, or enables termination checks for both exec recursive functions and exec loops.

I was already under the impression we didn't support termination checking for exec functions, seeing as, exec functions support while loops, which don't support termination checking. As far as I can tell, Verus never complains about the existence of loops and there isn't support for opting in to exec-mode termination checking (at the moment) so permitting decreases on an exec function would be misleading.

well, i don't object to supporting it in the future. I'm just saying Andrea's change makes sense now and until such support does exist.

I think in the design I'm proposing, though, we wouldn't be disallowing and removing the decreases clauses that already exist.

We would, however, remove by default the requirement that exec functions have decreases clauses.  I support that part of the change.

The initial impetus was due to the fact that the support was incomplete: decreases did not work for functions with mutable references as arguments. That turned out to be a simple fix, so those are now supported.
The need to check termination and the guarantees provided by the check were unclear to me. From the discussion:

termination checks for exec functions aren't necessary for soundness,
the current check does not take into account loops.

So, this PR, now:

makes the termination check for exec functions optional (only enabled when there's a decreases clause),
warns the user that a termination check on an exec passing doesn't guarantee that the function terminates if there are loops.

can you add a warning or error if the user tries to use decreases on a loop? It looks like that's still missing, I think.


Fix sorting, it should use column instead of member

Fixes out of bound problem for most views.
Test: build, run and see if sorting works.

Hi,
just a meta-comment since there are probably more changes coming regarding the datamodels.
There are Qt classes which basically implement sorting and filtering of table models. They work
out of the box if lexicographical sorting is all what is needed and can be customized for more
specific sorting and filtering behaviour. Ref: https://doc.qt.io/qt-5/qsortfilterproxymodel.html
This can probably replace a lot of code in the data views. It's probably too much of a refactoring
right now. Just thought I mention that as a future consideration.
More sources:

https://doc.qt.io/qt-5/qtwidgets-itemviews-customsortfiltermodel-example.html
https://doc.qt.io/qt-5/qtwidgets-itemviews-basicsortfiltermodel-example.html


Consequencer updates dec 22

Added a hard upper limit to the number of CV Patterns (35) to avoid a memory allocation error
Updated generateNewRandomCVPattern() to capture a memory allocation error in case it ever happens again
Added code to pull the button pins up to (hopefully) avoid phantom long button presses
Updated the behaviour of B2 in Mode 3. B2 no longer changes the selected CV pattern in mode 3 - this is now only done using CV, but only in mode 3

I have no idea why git is showing that there is an additional comment line in cvecorder. I have removed this line from my main branch and the branch this PR was created from, but github still thinks the change is there for some reason.


Add a PR comment that tells readers how to install the current dev version of EpiAware

As the title. I think this could help streamline review and make sure we more often test out changes locally. It might also help those new to Julia begin reviewing.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 93.33%. Comparing base (b91d467) to head (efadb11).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #399   +/-   ##
=======================================
  Coverage   93.33%   93.33%
=======================================
  Files          53       53
  Lines         540      540
=======================================
  Hits          504      504
  Misses         36       36

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


iputils: 20151218 -> 20161105

Motivation for this change
after the upgrade of iputils from 20151218 to 20161105
functionality of ping6 and tracepath6 was merged into ping and tracepath.
Ping is now mostly a drop-in replacment for ping6, except that selecting a
specific interface is done by encoding it into the address (ex.: fe80::1%eth0)
rather then specifing it with the -I flag.
This is still marked as work in progress, because I want to use travis to test for package breakage due my slow internet connection at the moment. Also I did not run any test I have modified yet.
Things done

[x] 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
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

We should mention in the release notes that ping6 and tracepath6 are not available anymore. These are common tools in the Linux userland.
Also note that there are multiple references to ping6 in pkgs and nixos.

We might want to link {ping,tracepath}6 to {ping,tracepath} for the time being?

@globin I thought about that, but this might be a bad idea as the flags are different. Commands might check for the existence of ping6/tracepath6 but break because of incompatible syntax. I would rather want to see command not found  in which case the programs can be patched rather then subtle bugs.

Yes, right.

Is there a convienient way to run all tests in nixos/tests/networking.nix @wkennington?
Currently I do something like:
nix-build nixos/release.nix -A tests.networking.networkd.bridge


Add EsStore plugin

The approach to using the buffer in this plugin is slightly different
when compared to other plugins. The documentation on Elasticsearch's
bulk API mentions the following (Ref:
https://www.elastic.co/guide/en/elasticsearch/reference/current/docs-bulk.html).
There is no "correct" number of actions to perform in a single bulk
call. You should experiment with different settings to find the
optimum size for your particular workload.

Hence, the buffer size is the maximum number of bytes to be sent in the
HTTP request body of the bulk API.

One thing that stands out in this pull request is the decision to define buffer_size as the number of bytes to keep in buffer rather the number of records to keep in buffer. This is a deviation from the definition of buffer_size in other store plugins.
This deviation seems okay because we normally do not expect users to modify the buffer_size parameter of the plugin in configuration. In most cases, the defaults should be good. This deviation appears to be necessary because the Elasticsearch performance appears to suffer based on the number of bytes, not the number of documents. Quoting another reference from Elasticsearch documentation at https://www.elastic.co/guide/en/elasticsearch/guide/current/bulk.html:

How Big Is Too Big?
The entire bulk request needs to be loaded into memory by the node that receives our request, so the bigger the request, the less memory available for other requests. There is an optimal size of bulk request. Above that size, performance no longer improves and may even drop off. The optimal size, however, is not a fixed number. It depends entirely on your hardware, your document size and complexity, and your indexing and search load.
Fortunately, it is easy to find this sweet spot: Try indexing typical documents in batches of increasing size. When performance starts to drop off, your batch size is too big. A good place to start is with batches of 1,000 to 5,000 documents or, if your documents are very large, with even smaller batches.
It is often useful to keep an eye on the physical size of your bulk requests. One thousand 1KB documents is very different from one thousand 1MB documents. A good bulk size to start playing with is around 5-15MB in size.

@susam , It's a very interesting observation, in your previous comment quoting the "How Big Is Too Big?" passage from the Elasticsearch documentation. I somehow did not stumble upon it while I was looking for optimal sizes because I was only referring to the most recent documentation of ES. Thank you for pointing it out.


hipLaunchCooperativeKernel slowdown

Problem Description
I wanted to try using Cooperative groups as appears as if it would be helpful in some future work. I began by trying to launch a kernel using hipLaunchCooperativeKernel. Note that this kernel did not use any cooperative features I was just trying to get the framework in place to start implementing them. The kernel ran correctly but was an order of magnitude slower than when launched in the normal way. I tried running it with only 1 thread block and observed the same behavior. My question is if this is the expected behavior for the current implementation. If so I'll just shelve the idea for now.
Operating System
22.04.4 LTS (Jammy Jellyfish)
CPU
AMD EPYC 7443P 24-Core Processor
GPU
AMD Radeon VII
ROCm Version
ROCm 6.0.0
ROCm Component
HIP
Steps to Reproduce
No response
(Optional for Linux users) Output of /opt/rocm/bin/rocminfo --support
No response
Additional Information
No response

Hi @elbriggs, I will confirm the expected behaviour with our internal team and let you know.


Add Parse Dashboard to the example

It would be nice to add the dashboard to this example

I think we should link to it, but right now it's best to run the dashboard locally.

+1

The dashboard is now available on npm so this would be much easier to add.

Hello @drew-gross
Can you help me please to configure reset user password with parse Version 2.1.6 ?
Regards


Dynamic library prevents code signing from working in Xcode Cloud

In Package.swift this line forces you to choose either "Embed and Sign" or not to code sign when including ConfettiSwiftUI into an app (in Xcode):
type: .dynamic

This line isn't really needed.
I discovered this when attempting to build one of my apps with Xcode Cloud. Here's the error you get during the archive process:
ITMS-90334: Invalid Code Signature Identifier. The identifier "ConfettiSwiftUI-55554944d4fbc3d9c42f38d3b8bfbbc8cd81fa38" in your code signature for "ConfettiSwiftUI" must match its Bundle Identifier "ConfettiSwiftUI"

Thanks for the PR -> new release 0.3.0. Please close if it is resolved.

Awesome. Thanks for getting this in. My app now builds perfectly with Xcode Cloud.


Add some acceptance/feature tests with Hound/Wallaby

The UI part of the backend is not that important, but we should still ensure it works.

24d79a4a9cdfbb3a2d9c9327c761bf8146515fd1


Missing header defines for ATxmega128A1U in avr libc

A couple of defines are missing in the avr libc header files for the ATxmega128A1U. Need to add them as a work around.
Upstream bug for this issue, seems to have gone stale..

Latest header definitions can be found here: http://distribute.atmel.no/tools/opensource/Atmel-AVR-GNU-Toolchain/
Specifically, this issue is fixed by using the these header files: http://distribute.atmel.no/tools/opensource/Atmel-AVR-GNU-Toolchain/3.6.1/avr8-headers.zip . Copying them to /usr/avr/include/avr or equivalent will fix this issue.
Added them to the travis build script here: 00d9cc0f0b0f97c4b32ba94a522d521041ea08cf


fix: hostname needs to be computed

What does this fix or implement?

Checklist

[ ] PR name added as appropriate (e.g. feat:/fix:/doc:/test:/refactor:)
[ ] Tests added or updated
[ ] Documentation updated
[ ] Changelog updated and version incremented (label: upcoming release)
[ ] Github Issue linked if any
[ ] Jira task updated

I also suggest, for testAccCheckServerConfigEmptyNicIps test configuration, in resource_server_test.go, (the configuration for which we don't specify the hostname) add a resource.TestCheckResourceAttrSet check only to make sure that the hostname is set in the state even though it isn't provided in the configuration.


Update documentation for feathers-config

We need to update the docs for feathers-configuration to account for the changes in feathersjs/feathers-configuration#27

:shipit:


Add textentry hotkeys

This stacks onto #930.

Sorry for leaving this hanging for so long. To be short, I cherry picked some of the changes from this PR, and committed to master branch. They will be in the next release.
On careful consideration, the on_resize methods were not included. This is a bit out of scope for the default classes, and not easy for us to support in a uniform way across widgets. It's easy enough to add this behavior in a base class. With that in mind, it would make more sense for us to add something like that as an example, instead of supporting it in the main code.
I'll close this for now, but please feel free to follow up if you want to discuss further.


Patch 1

I submitted this pull request to the main branch instead of the intended 20230903 branch. Consequently, Hence, Ig I should close this pull request.


Intercepting and processing formik state when submitting the form

Find a way to (maybe externally outside of react) intercept and process (adapt) formik state before it gets shipped to BE.

@Ducica I think this has been solved by #46 ?

Definitely solved in FE-31


EDA ON SALES DATA

Description
Here in data analysis section i am contribution "Exploratory Data Analysis on sales data"
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
#156

@theyashwanthsai
Can you please review it and merge it?


Improve CI time

This issue captured ideas brought up in offline discussions with @rchiodo for improving the testing experience in the repo, including faster CI time and tighter feedback loop for changes (locally)

Shared document/kernel for test suite when appropriate. We currently open and close document for every individual test, which will launch kernel for every single. The kernel startup contributes to main portion of our CI time. When document open/close and kernels state don't matter in a test suite, we could see how we can reuse the same document and kernel for faster testing.
A launch.json option (with properly tuned envs) to test non-conda tests locally. Currently the tests on a reasonably powerful machine takes too long and some of us rely on the CI for the test results.

We could also work on improving

Compile time
Conda install time
Not install .NET/Julia for python tests

Partial improvements made here https://github.com/microsoft/vscode-jupyter/pull/11867

Closing this as majority of the work has been done (faster compilation, faser tests, etc)
If we can improve more, I think it can be done as a regular debt task.
This used to take 45 minutes or longer back a year ago, now its around 15 minutes


Allow to change order of widgets dragging

Suggested new feature or improvement
📝 Description
Will be good if in the panel the widgets could be rearranged by yourself, by drag and drop, and not just for addition order.
Scenario
🖼 Scenario
Rearranging widgets to reorder them as is needed for the project or wants the user.
Additional details
No response

Duplicate of #673


Reduce the overhead of deallocating read-write set

The overhead of deallocating a read-write set at a commit time is high, if we have many tuples in a read-write set.  It is the case with analytics queries.  There are many ways to handle this, such as having more light-weight read-write set, using a Hyper-way of snapshotting the memory space so that we don't need these to be kept, or moving this out of its execution path, etc. There may be pros and cons, so we may need a decision before changing it.
We have not concluded how to handle this, so it would be nice if someone looks more deeply at it.

@sivaprasadsudhir is fixing it so that we deallocate the TransactionContext object in the GarbageCollector. This removes deallocation from the txn's critical path.
More optimized ReadWrite set implementations can be a future project.


Is rebar3_hex required in modern/recent versions of rebar3?

While reading the docs on pkg management at rebar3.org, the subcmd hex is presented as a built-in thing instead of a separate plugin that needs to be installed before usage. I'm not very familiar with rebar3, so I'm not sure if I need to "install" rebar3_hex in my rebar3.config or if it's something that comes bundled in modern rebar3.
I'm using Erlang v<IP_ADDRESS> and rebar3 v3.22.1, both installed through asdf.
Whichever the answer, I believe it would be great to have it stated in README.md, perhaps also describing which versions of rebar3 would require installing it manually, and which ones would bring rebar3_hex bundled in itself.

The rebar3_hex plugin is required for publishing packages only. Installing packages works fine without any plugin.
Agreed this should be stated clearly in the docs, it has been a source of confusion.

Yes, as stated by @tsloughter, rebar3_hex is not bundled (or installed) with rebar3, even in the latest version. Maybe a pull request to https://github.com/tsloughter/rebar3.org, to update the documentation, is the best option, at the moment.
In case it wasn't made clear, or you still have doubts, it suffices to do {deps, [rebar3_hex]} in your rebar.config to have the plugin and its commands (under rebar3 hex) available for use.

Not in deps. You want {project_plugins, [rebar3_hex]}. for doing stuff like publishing a package.

Yeah, sorry, project_plugins 👍 I'll also fix it above.

Thanks for clarifying, @tsloughter @paulo-ferraz-oliveira!

Feel free to close the issue if you feel there's nothing else we can do from this repo. 👍

Thanks for clarifying, @tsloughter @paulo-ferraz-oliveira!
I avoided closing this issue to remember coming back here to submit a PR about it – and I got the time to do so today. :)
https://github.com/erlef/rebar3_hex/pull/334


fix-3264-change-download-urls

#3264

There seems to be problems with datasets metadata, of which I dont have access to. I think one of the datasets is from reddit. Can anyone help?

Hello @LashaO , I think the errors were caused by _DATA_FILES in definite_pronoun_resolution.py. Here are details of the test error.
self = BuilderConfig(name='plain_text', version=1.0.0, data_dir=None, data_files={'train': 'train.c.txt', 'test': 'test.c.txt'}, description='Plain text import of the Definite Pronoun Resolution Dataset.')

    def __post_init__(self):
        # The config name is used to name the cache directory.
        invalid_windows_characters = r"<>:/\|?*"
        for invalid_char in invalid_windows_characters:
            if invalid_char in self.name:
                raise InvalidConfigName(
                    f"Bad characters from black list '{invalid_windows_characters}' found in '{self.name}'. "
                    f"They could create issues when creating a directory for this config on Windows filesystem."
                )
        if self.data_files is not None and not isinstance(self.data_files, DataFilesDict):
>           raise ValueError(f"Expected a DataFilesDict in data_files but got {self.data_files}")
E           ValueError: Expected a DataFilesDict in data_files but got {'train': 'train.c.txt', 'test': 'test.c.txt'}

Hi ! Thanks for the fixes :)
Instead of uploading the definite_pronoun_resolution data files in this PR, maybe we can just update the URL ?
The old url was http://www.hlt.utdallas.edu/~vince/data/emnlp12/train.c.txt, but now it's https://www.hlt.utdallas.edu/~vince/data/emnlp12/train.c.txt (https instead of http)

Actually the bad certificate creates an issue with the download
import datasets
datasets.DownloadManager().download("https://www.hlt.utdallas.edu/~vince/data/emnlp12/train.c.txt")
# raises: ConnectionError: Couldn't reach https://www.hlt.utdallas.edu/~vince/data/emnlp12/train.c.txt

Let me see if I can fix that

I uploaded them to these URLs, feel free to use them instead of having the text files here in the PR :)
https://s3.amazonaws.com/datasets.huggingface.co/definite_pronoun_resolution/train.c.txt
https://s3.amazonaws.com/datasets.huggingface.co/definite_pronoun_resolution/test.c.txt

Thank you for the tips! Having a busy week so anyone willing to commit the suggestions is welcome. Else, I will try to get back to this in a while.

@LashaO Thanks for working on this. Yes, I'll take over as we already have a request to fix the URL of the Jeopardy! dataset in a separate issue.

Still have to fix the error in the dummy data test of the WikiAuto dataset.

Thank you, Mario!

The CI failure is only related to missing tags in the dataset cards, merging :)


srain: 1.3.0 -> 1.3.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/srain/versions.
meta.description for srain is: "Modern IRC client written in GTK"
meta.homepage for srain is: "https://srain.im"
meta.changelog for srain is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
/nix/store/9m4vd6h3kcivm2a4mvpngw4jc1qlskch-srain-1.3.1/bin/.srain-wrapped passed the binary check.
/nix/store/9m4vd6h3kcivm2a4mvpngw4jc1qlskch-srain-1.3.1/bin/srain passed the binary check.
2 of 2 passed binary check by having a zero exit code.
2 of 2 passed binary check by having the new version present in output.
found 1.3.1 with grep in /nix/store/9m4vd6h3kcivm2a4mvpngw4jc1qlskch-srain-1.3.1
found 1.3.1 in filename of file in /nix/store/9m4vd6h3kcivm2a4mvpngw4jc1qlskch-srain-1.3.1

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
1 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
docbookrx
srain

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/9m4vd6h3kcivm2a4mvpngw4jc1qlskch-srain-1.3.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A srain https://github.com/r-ryantm/nixpkgs/archive/3ca0d9b0a562c6169528b44c4354d3a945559701.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/9m4vd6h3kcivm2a4mvpngw4jc1qlskch-srain-1.3.1
ls -la /nix/store/9m4vd6h3kcivm2a4mvpngw4jc1qlskch-srain-1.3.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

srain

Maintainer pings
cc @wineee for testing.

Seems to be working fine


1382987 update content

Updated document with a new section. Please review

Docs Build status updates of commit 5745c52:
:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/concepts/deploy-and-publish/appsource/prepare/frequently-failed-cases.md
:warning:Warning
View
Details

msteams-platform/concepts/deploy-and-publish/appsource/prepare/frequently-failed-cases.md

Line 209, Column 1: [Warning-hard-coded-locale] Link 'https://docs.microsoft.com/en-us/microsoft-365-app-certification/docs/attestation' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 210, Column 1: [Warning-hard-coded-locale] Link 'https://docs.microsoft.com/en-us/microsoft-365-app-certification/docs/userguide#3publisher-attestation-workflow' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 211, Column 1: [Warning-hard-coded-locale] Link 'https://docs.microsoft.com/en-us/azure/active-directory/develop/troubleshoot-publisher-verification' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 9c7e271:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/concepts/deploy-and-publish/appsource/prepare/frequently-failed-cases.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit af36a81:
:clock10: Pending: waiting for processors (9 builds ahead of you)
:warning: Docs Build is busy, currently there are 9 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit af36a81:
:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/concepts/deploy-and-publish/appsource/prepare/frequently-failed-cases.md
:warning:Warning
View
Details

msteams-platform/concepts/deploy-and-publish/appsource/prepare/frequently-failed-cases.md

Line 209, Column 16: [Warning-file-not-found] Invalid file link: '../../microsoft-365-app-certification/docs/attestation'.
Line 210, Column 14: [Warning-file-not-found] Invalid file link: '../../microsoft-365-app-certification/docs/userguide'.
Line 211, Column 16: [Warning-file-not-found] Invalid file link: '../../azure/active-directory/develop/troubleshoot-publisher-verification'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit f84b546:
:clock10: Pending: waiting for processors (9 builds ahead of you)
:warning: Docs Build is busy, currently there are 9 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit f84b546:
:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/concepts/deploy-and-publish/appsource/prepare/frequently-failed-cases.md
:warning:Warning
View
Details

msteams-platform/concepts/deploy-and-publish/appsource/prepare/frequently-failed-cases.md

Line 209, Column 16: [Warning-file-not-found] Invalid file link: '../../microsoft-365-app-certification/docs/attestation'.
Line 210, Column 27: [Warning-file-not-found] Invalid file link: '../../microsoft-365-app-certification/docs/userguide'.
Line 211, Column 16: [Warning-file-not-found] Invalid file link: '../../azure/active-directory/develop/troubleshoot-publisher-verification'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 4c33351:
:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/concepts/deploy-and-publish/appsource/prepare/frequently-failed-cases.md
:warning:Warning
View
Details

msteams-platform/concepts/deploy-and-publish/appsource/prepare/frequently-failed-cases.md

Line 209, Column 16: [Warning-file-not-found] Invalid file link: '../../microsoft-365-app-certification/docs/attestation'.
Line 210, Column 27: [Warning-file-not-found] Invalid file link: '../../microsoft-365-app-certification/docs/userguide'.
Line 210, Column 233: [Warning-file-not-found] Invalid file link<EMAIL_ADDRESS>Line 211, Column 16: [Warning-file-not-found] Invalid file link: '../../azure/active-directory/develop/troubleshoot-publisher-verification'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 6948ac9:
:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/concepts/deploy-and-publish/appsource/prepare/frequently-failed-cases.md
:warning:Warning
View
Details

msteams-platform/concepts/deploy-and-publish/appsource/prepare/frequently-failed-cases.md

Line 210, Column 228: [Warning-file-not-found] Invalid file link<EMAIL_ADDRESS>
For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit a1c9bf7:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/concepts/deploy-and-publish/appsource/prepare/frequently-failed-cases.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 886123f:
:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/concepts/deploy-and-publish/appsource/prepare/frequently-failed-cases.md
:warning:Warning
View
Details

msteams-platform/concepts/deploy-and-publish/appsource/prepare/frequently-failed-cases.md

Line 212, Column 95: [Warning-file-not-found] Invalid file link: '../../microsoft-365-app-certification/media/picture5.png'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 1083031:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/concepts/deploy-and-publish/appsource/prepare/frequently-failed-cases.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit fd51921:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/concepts/deploy-and-publish/appsource/prepare/frequently-failed-cases.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit f432265:
:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/concepts/deploy-and-publish/appsource/prepare/frequently-failed-cases.md
:warning:Warning
View
Details

msteams-platform/concepts/deploy-and-publish/appsource/prepare/frequently-failed-cases.md

Line 209, Column 16: [Warning-file-not-found] Invalid file link: '../microsoft-365-app-certification/docs/attestation.md'.
Line 210, Column 27: [Warning-file-not-found] Invalid file link: '../microsoft-365-app-certification/docs/userguide'.
Line 211, Column 16: [Warning-file-not-found] Invalid file link: '../azure/active-directory/develop/troubleshoot-publisher-verification.md'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit f5b4715:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/concepts/deploy-and-publish/appsource/prepare/frequently-failed-cases.md
:white_check_mark:Succeeded
View

wiki
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

All updates have been done and it is ready-to-merge.

Docs Build status updates of commit 03885a3:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/concepts/deploy-and-publish/appsource/prepare/frequently-failed-cases.md
:white_check_mark:Succeeded
View

wiki
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 1278f49:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/concepts/deploy-and-publish/appsource/prepare/frequently-failed-cases.md
:white_check_mark:Succeeded
View

wiki
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Read host and token from config.json

Description
created feature to be able to read host and token from config.json
Works with both
absolute paths
C:\\users\username\onedrive\desktop\config.json

Path object (recommended)
(Path(__file__) / Path("../../config.json)).resolve()

Changes

added a few lines of code to constructor of api.py
added config.json to .gitignore

Tests

wrote a simple test for it

Known Issues
Screenshot
Updated documentation

test that is failing is because it expects the db schema to have 70 sections but it has 74 currently

I think I can put the host and token initialization from the constructor into a utility function and clean up our constructor a bit

I can approve when tests are passing.

@InnocentBug I think I'm going to make a new PR because I was indecisive if I was putting this PR into validate branch or develop branch, so this PR history became a mess, but luckily it is a small change, so I can add the changes to a new PR and put it into develop branch


Button style not showing as expect

When i add OutlinedButton to page,its not showing right

import flet as ft

def main(page):
    page.title = "Example"
    for i in range(1, 10):
        page.add(
            ft.OutlinedButton(
                str(i),
                style=ft.ButtonStyle(shape=ft.CountinuosRectangleBorder()),
                width=40,
                height=40,
                on_click=print,
            )
        )

ft.app(target=main)

I changed style to style=ft.ButtonStyle(shape={"": ft.CountinuosRectangleBorder()}),,it looks right(althought content is not in center)

Then i add padding=1 to ButtonStyle, if looks good

final code
import flet as ft

def main(page):
    page.title = "Example"
    for i in range(1, 10):
        page.add(
            ft.OutlinedButton(
                str(i),
                style=ft.ButtonStyle(shape={"": ft.CountinuosRectangleBorder()}, padding=1),
                width=40,
                height=40,
                on_click=print,
            )
        )

ft.app(target=main)

I have tried OutlinedButton ElevatedButton FilledButton FilledTonalButton TextButton, and WEB_BROWSER view, same issue.
Flet version (pip show flet):
Name: flet
Version: 0.7.0.dev1399
Summary: Flet for Python - easily build interactive multi-platform apps in Python
Home-page:
Author: Appveyor Systems Inc.
Author-email<EMAIL_ADDRESS>License: Apache-2.0
Location: c:\users\14438\documents\github\private-taichi\venv\lib\site-packages
Requires: flet-core, httpx, oauthlib, packaging, watchdog, websocket-client, websockets
Required-by:

Operating system:
Windows  11 22H2 22621.1702
Additional environment details:

Update to the latest Flet 0.7.1 - this issue has been fixed there.

Yeah, 0.7.1 works great, thanks!


Parking AI seems unreliable

My city is filled with parking spaces, many of which are empty, yet I see vehicles travelling to many destinations roaming around looking for parking.
Context:

Left side driving
Lots of cul-de-sacs around areas where cims are trying to get to

If you need a save or any other infos, let me know.
https://github.com/VictorPhilipp/Cities-Skylines-Traffic-Manager-President-Edition/issues/204

Since the recent update I've noticed a big improvement, in that there is far less 'stray traffic' (eg. my industrial areas are no longer filled with 'looking for parking' traffic from distant leisure and tourist districts).
However, cims still aren't finding parking spaces near their destinations. #184, #150 and #209 would greatly alleviate this issue.

Closing as obsolete.
Updated links to issues mentioned above: #189, #176, #196


Commands not showing in help

I have recently noticed that some of my commands were not showing up in the help command list, I recently found out what was causing it. If you use a permissions number instead of its name, for example for a number "permLevel: 1" having it as a number makes the command not show up in help except the command still works fine... I fixed it by using the roles name, "permLevel: "User". Sorry if this is terribly explained...
P.S. I haven't tried much to fix this myself as it hasn't effected me to much.

permLevel is by level name and not number

Yes, this was a breaking change when we went from permission levels (what some of your commands were using) to a more use friendly approach with the permission level names, such as User, Moderator, Administrator etc

Oh, alright Thanks guys.


Do check as same as calculateVertices

I already ask a question about this  https://github.com/pixijs/pixi.js/issues/3408  , but no one reply .
I think opening PR is better than asking question.

@staff0rd  ， I can't understand what's your mean.
What should I do to this PR ?

Instead of this;
if (this._transformID === this.transform._worldID && this._textureID === this._texture._updateID) { return; }
...
if (this._transformID === this.transform._worldID && this._textureID === this._texture._updateID) { return; }

Self-documenting and reduced;
isAlreadyUpdated() {
    return this._transformID === this.transform._worldID && this._textureID === this._texture._updateID;
}

if (isAlreadyUpdated()) { return; }
...
if (isAlreadyUpdated()) { return; }

@staff0rd but it will create a new function call. In this place I think performance is very important.

Ah I see - I wasn't aware javascript had a cost associated with function declaration.  TIL.

If you're lucky it gets inlined, but not guaranteed. That is one of the shitty parts of working with such a dynamic lang.

I want @ivanpopelyshev to see this before we merge, since he did a lot of trim work.

I don't think we can use same temporary variables in two different functions. If you update one of them, second wont be updated.

@ivanpopelyshev  ， you're right . I changed them.

good catch @ivanpopelyshev !


AJ-1011: openapi cleanup

Prefactoring for AJ-1011. In service/src/main/resources/static/swagger/openapi-docs.yaml:

clean up some formatting
add some code comments
sort paths first by their tag (Records, Schema, Instances, etc) and second alphabetically
sort all components alphabetically

A lot of lines of code have changed, but there should be zero functional changes; it's all reordering and formatting. You can view commit-by-commit to see it in smaller chunks.

While you're at it, can you put the API name over on the right side, like here?

@calypsomatic I did that back in #331! Note that you'll only see it when hitting the UI from a running WDS; you won't see it when viewing the spec via https://petstore.swagger.io/ - because the change is specific to the HTML/JS that interprets the spec.

@calypsomatic I did that back in #331! Note that you'll only see it when hitting the UI from a running WDS; you won't see it when viewing the spec via https://petstore.swagger.io/ - because the change is specific to the HTML/JS that interprets the spec.
Huh!


Add method to load a conversation

It is useful to have a method that allows to load a previous conversation:
Task<Guid> LoadConversationAsync(IEnumerable<ChatGptMessage> messages);

Task<Guid> LoadConversationAsync(Guid conversationId, IEnumerable<ChatGptMessage> messages, bool replaceHistory = true);

Consider the following guidelines:

The first method must call the second one with Guid.NewGuid() as conversationId
If replaceHistory = false, messages must be added to existing ones in the given conversation
The total number of messages must not exceed the message limit defined in ChatGptOptions

I think in the method
Task<Guid> LoadConversationAsync(IEnumerable<ChatGptMessage> messages, bool replaceHistory = true);
the optional parameter **replaceHistory ** will not be necessary, since the messages will be bound to a new conversation with a new conversationId, or possibly I am missing the point.

You're right, it is a typo. I have updated the description.

Yes, I have assigned the task to you

Thanks a lot

One more thing: should I ensure that messages are not empty in
Task<Guid> LoadConversationAsync(Guid conversationId, IEnumerable<ChatGptMessage> messages, bool replaceHistory = true);
or should it be a feature like clearing the history of the conversation?
My point is messages should not be empty since there already is a method to clear the history.

I think it is not necessary to check if messages are empty, because the user could have created its own conversationId,  so it is not already assigned to a real conversation.


Notification for overdue projects is not working properly.

Describe the bug
Setting of Notification for overdue project is on still not getting notification for overdue projects.

Highlighted Notifications are not working.

More than 3 days is only for owners

More than 3 days is only for owners

it's also not working


Add SysVInit artifact, and aggregate LinuxServices artifact.

Review status: 0 of 1 files reviewed, 1 unresolved discussion, all commit checks successful.

definitions/linux.yaml, line 250 [r1] (raw file):
Do you need the first one?  Is this the stat'ing parent directory GRR problem?

Comments from the review on Reviewable.io

Review status: 0 of 1 files reviewed, 1 unresolved discussion, all commit checks successful.

definitions/linux.yaml, line 250 [r1] (raw file):
Yep, I want to collect both the dirs and the files.

Comments from the review on Reviewable.io