Clean up case study images

Closes #22086
☔ Umbrella issue: #22024
What is changed

Rename the pre-existing case_studies directory to case-studies
Move /images/CaseStudy_<company-name>_<banner-name> to /images/case-studies/<company-name>/<banner-image-file>
Move /images/<compnay-name><banner-image-file> to /images/case-studies/<company-name>/<banner-image-file>
Rewrite reference to the images on each index.html an stylesheets

I also rename some banner image files to be consistent.
How I tested

Open http://localhost:1313/case-studies/
Click each company logo to visit each case study page
Check if I can see the banner images
Check if the number of banner images equals to the number of image files in /images/case-studies/<company-name>/

/assign celestehorgan

/area web-development

Thank you for reviewing.

/approve
I looked through every case study to check that images still worked correctly.
Thank you for taking this on!!


Add messages delete handler

All Submissions:

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?

Description
Adds the messages.delete endpoint handler.

Thanks, @mochetts!


Add support for jupyter-server

Proposed change
@fcollonval mentioned in https://github.com/jupyterlab/extension-examples/issues/151#issuecomment-787419416 that Binder doesn't support jupyter-server. Is this something we can add in a backwards compatible manner, so that old versions of Notebook and JupyterLab still launch?
Alternative options
Do nothing and document a workaround.
Who would use this feature?
Anyone using applications based on jupyter-server.
(Optional): Suggest a solution
I don't know what's required to add support (or if it should already be supported but isn't working due to a bug)

This will be handled in repo2docker:

https://github.com/jupyterhub/repo2docker/issues/956
https://github.com/jupyterhub/repo2docker/issues/1195


Add Server, Client, and Models Package

Motivation
To implement a Rosetta server, it was necessary to setup your own codegen toolchain and keep it working across interface updates. This was a drain on engineering resources for anyone working on a server.
Secondly, there were no examples demonstrating how to use the different packages.
Solution
Create a server package that makes it easy to write a Rosetta server, break apart the Rosetta models into a separate package for anyone looking to build their own server, and add a ton of examples.

Pull Request Test Coverage Report for Build 384

113 of 122 (92.62%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+3.9%) to 82.889%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

asserter/request.go
71
73
97.26%

asserter/block.go
25
28
89.29%

asserter/network.go
12
16
75.0%

Totals

Change from base Build 158:
3.9%

Covered Lines:
373

Relevant Lines:
450

💛  - Coveralls


Make Milestone-Jefferson work with M44 again.

This patch set reverts a few changes that went into the branch and broke the build, as the respective upstream changes are not part of M44.
Fixes #374.

sigh, wrong target branch


installation issues

I installed requirements and torch as per the document. but when i run
Install MultiScaleDeformableAttention package: python src/trackformer/models/ops/setup.py build --build-base=src/trackformer/models/ops/ install
i ran into below error after adding arch 8.6 in supported_arches list in cpp_extension.py
File "/AI_Data/Vamsi_Data/anaconda3/envs/trackformer/lib/python3.7/site-packages/torch/utils/cpp_extension.py", line 437, in unix_wrap_ninja_compile
with_cuda=with_cuda)
File "/AI_Data/Vamsi_Data/anaconda3/envs/trackformer/lib/python3.7/site-packages/torch/utils/cpp_extension.py", line 1163, in _write_ninja_file_and_compile_objects
error_prefix='Error compiling objects for extension')
File "/AI_Data/Vamsi_Data/anaconda3/envs/trackformer/lib/python3.7/site-packages/torch/utils/cpp_extension.py", line 1441, in _run_ninja_build
raise RuntimeError(message)
RuntimeError: Error compiling objects for extension

I've encountered the same problem, I've searched for many tips on the internet but to no avail, has the blogger solved it yet?


Extra code pictures causes error

Only when first configuring Android app:
Even when the + sign goes away on the preview/picture list screen, you can still return to the bio/info setup screen and then click save there, and you will be brought to the code creation screen again. After uploading the code image you can return to the bio screen as many times as you want to add an arbitrary amount of code images. However, when you go to upload a number of snippets from the preview screen that is too large, you get an error.
Expected behavior:
Returning to the bio screen when the max number of snippets have been added and then clicking save should bring you to the preview screen where all of your code snippets are displayed (not the code snippet creation page).
Maybe this has already been fixed, but I scanned the open and closed issues and couldn't find anything. Also, it's worth mentioning the only reason I even found this error was that it was not clear that I could hold down on the snippet previews to delete or move them. A quick message about this would be much appreciated by all even if the main error above is fixed.

Thanks for the quick response.
I should have said this in my original message, but thanks for creating the service. It's a great idea and really well executed.


Problem upgrading to Gemini 2.0 with Helm

What happened?
I upgraded from a working 1.x installation to 2.0 by running
helm repo update
helm upgrade -n gemini gemini fairwinds-stable/gemini

Then when trying to apply a CR:
kubectl apply -f snapshotgroups.yaml
error: resource mapping not found for name: "navidrome-config" namespace: "owncloud" from "snapshotgroups.yaml": no matches for kind "SnapshotGroup" in version "gemini.fairwinds.com/v1"
ensure CRDs are installed first

So I removed & reinstalled:
helm delete gemini
helm install -n gemini gemini fairwinds-stable/gemini

No change.
What did you expect to happen?
I expected CRDs to be upgraded with the Helm upgrade
How can we reproduce this?
Install 1.x, then upgrade to 2.x
Version
2.0.0
Search

[X] I did search for other open and closed issues before opening this.

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Additional context
kubectl version
WARNING: This version information is deprecated and will be replaced with the output from kubectl version --short.  Use --output=yaml|json to get the full version.
Client Version: version.Info{Major:"1", Minor:"26", GitVersion:"v1.26.3", GitCommit:"9e644106593f3f4aa98f8a84b23db5fa378900bd", GitTreeState:"clean", BuildDate:"2023-03-15T13:33:11Z", GoVersion:"go1.19.7", Compiler:"gc", Platform:"linux/amd64"}
Kustomize Version: v4.5.7
Server Version: version.Info{Major:"1", Minor:"26", GitVersion:"v1.26.3", GitCommit:"9e644106593f3f4aa98f8a84b23db5fa378900bd", GitTreeState:"clean", BuildDate:"2023-03-17T19:59:21Z", GoVersion:"go1.19.7", Compiler:"gc", Platform:"linux/amd64"}

Inconsistent state of CRD/api-resource:
kubectl api-resources | grep snapshotgroups
snapshotgroups                                        gemini.fairwinds.com/v1beta1           true         SnapshotGroup

kubectl get crd snapshotgroups.gemini.fairwinds.com
NAME                                  CREATED AT
snapshotgroups.gemini.fairwinds.com   2023-03-06T15:46:48Z

I have just updated to Gemini 2.0.1. No errors reported during the upgrade, but the Gemini controller is repeatedly logging:
I0515 14:22:14.143125       1 reflector.go:257] Listing and watching *v1.SnapshotGroup from pkg/mod/k8s.io/client-go@v0.26.1/tools/cache/reflector.go:169
W0515 14:22:14.146389       1 reflector.go:424] pkg/mod/k8s.io/client-go@v0.26.1/tools/cache/reflector.go:169: failed to list *v1.SnapshotGroup: the server could not find the requested resource (get snapshotgroups.gemini.fairwinds.com)
E0515 14:22:14.146448       1 reflector.go:140] pkg/mod/k8s.io/client-go@v0.26.1/tools/cache/reflector.go:169: Failed to watch *v1.SnapshotGroup: failed to list *v1.SnapshotGroup: the server could not find the requested resource (get snapshotgroups.gemini.fairwinds.com)

This was resolved by doing:
helm delete gemini
kubectl delete crd snapshotgroups.gemini.fairwinds.com
helm install ...


Warnings vs Errors

Some rules are marked as warnings and some as errors.
Is this because some rules are more important then others?
If it is not breaking code, shouldn't it always be marked as a warning?

Can you paste an example of an error here? There should be only warnings, are you sure it's not some other plugin marking them as errors?

I see, so maybe the "too many blank lines" comes from pep8?
I saw D102 thought it was it, but it refers to other warning in the same line right?

Too many blank lines is pep8 not pep257, pep257 is only about docstrings.


⚠ Bofrost.lu has degraded performance

In 441fac4, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1744 ms

Resolved: Bofrost.lu performance has improved in 5ec3876 after 1 hour, 5 minutes.


param slider outside of visual

I am having issues with params in Deneb.
The following example will generate a chart with a slider.
{
  "data": {"values": [{"x": 50, "c": 1}, {"x": 250, "c": 2}]},
  "params": [
    { "name": "max_x_axis", "value": 500,
      "bind": {"input": "range", "min": 1, "max": 500, "step": 10}}
  ],
  "mark": "bar",
  "encoding": {
    "x": {
      "field": "x",
      "type": "quantitative",
      "scale": {
        "domain": [0,{"expr": "max_x_axis"}]
      }
    },
    "color": {"field": "c", "type": "ordinal"}
  }
}

In the deneb preview, you can already see, that this slider is outside of the visuals bounds:

And indeed, when you look at the visual, you will need to use the scrollbar to get to the slider:

Example in Vega-Lite Editor

Hi, and thanks for your question. Vega views are typically a known height and/or width. Due to the fact that Power BI visuals can be flexible height and width, the Vega view is synced to these dimensions if you're not manually specifying width and/or height.
If you want a parameter to appear within the bound of the visual, I'd recommend that you manually set the height in the top-level of the specification as necessary, e.g.:
{
  "height": 300,
  "data": {"values": [{"x": 50, "c": 1}, {"x": 250, "c": 2}]},
  "params": [
    { "name": "max_x_axis", "value": 500,
      "bind": {"input": "range", "min": 1, "max": 500, "step": 10}}
  ],
  "mark": "bar",
  "encoding": {
    "x": {
      "field": "x",
      "type": "quantitative",
      "scale": {
        "domain": [0,{"expr": "max_x_axis"}]
      }
    },
    "color": {"field": "c", "type": "ordinal"}
  }
}

Omitting the width will ensure that this is still automatically assigned but you can then tailor the height as necessary if you need to add more parameters,

As there's been no further feedback on the provided response for a couple of weeks I'm assuming that this resolves the issue, so am closing to ensure that the backlog is up to date. If there are further questions, then this can be re-opened or a new issue raised as applicable.


Question in 02_classification.ipynb

Information
The question or comment is about chapter:

[ ] Text Classification

Question or comment
def label_int2str(row):
return emotions["train"].features["label"].int2str(row)
but I received error report as: 'Value' object has no attribute 'int2str'.
I printed the types of emotions[label] with emotions["train"].features["label"], I received Value(dtype='int64', id=None)

Hey @pangniuniu95,
You might have done this, but can you cross-check again on emotions.set_format(type='pandas')? I think your emotions dataset is not set to pandas format, leading to the above error.
If you're still getting the error, can you provide the part of the script - function calling and other relevant code?

I'm getting the same error when I run 01_introduction.ipynb on Colab. I've double checked that I have run emotions.set_format(type='pandas').
AttributeError: 'Value' object has no attribute 'int2str'

It seems I can get around this by manually defining the value labels:
`# Manually define the mapping from int to str
label_mapping = {
0: "sadness",
1: "joy",
2: "love",
3: "anger",
4: "fear",
5: "surprise"
}
Function to convert label integer to string
def label_int2str(label):
return label_mapping[label]
Apply the function to the label column
df["label_name"] = df["label"].apply(label_int2str)
print(df.head())`

It seems that I can get around this by manually defining the labels

# Manually define the mapping from int to str`
label_mapping = {
    0: "sadness",
    1: "joy",
    2: "love",
    3: "anger",
    4: "fear",
    5: "surprise"
}

# Function to convert label integer to string
def label_int2str(label):
    return label_mapping[label]

# Apply the function to the label column
df["label_name"] = df["label"].apply(label_int2str)

print(df.head())

There is no issue if I run a seperate notebook with
!pip install datasets==2.8.0
import pandas as pd
from datasets import load_dataset

# Load the emotions dataset
emotions = load_dataset("emotion")

emotions.set_format(type="pandas")
df = emotions["train"][:]

def label_int2str(row):
    return emotions["train"].features["label"].int2str(row)

df["label_name"] = df["label"].apply(label_int2str)
df.head()

So it could be that it's the version of the datasets library that is used in the notebook? When running setup_chapter() it reports Using datasets v1.16.1.


Cannot use Find to access temporary table in in transaction operation

type MyObject struct {
MyParentObject    xorm:"extends"
}
func (m MyObject) TableName() string {
return "my_object"
}
session := db.NewSession()
if err := session.Begin(); err != nil {
return
}
q="CREATE TEMP TABLE temp1 AS SELECT * FROM A"
if _, err := session.Exec(q); err != nil {
return
}
var array []MyObject
if err := session.Find(&array); err != nil {
return
}
I got an error pq: relation "public.temp1" does not exist occur during executing Find function in above code.

I think maybe you could use session.Table("temp1").Find(&array).

I will close this. Please feel free to reopen it.


Provdided a way to retrieve response from context

This is usefull for custom checking.
For example, if you want to do more advanced checking in your project context, you could gather the ApiContext and grab the response from here like so (shamelessly copied from https://gist.github.com/stof/930e968829cd66751a3a):
<?php

namespace Context;

use Behat\Behat\Tester\Exception\PendingException;
use Behat\Behat\Context\SnippetAcceptingContext;
use Behat\Behat\Hook\Scope\BeforeScenarioScope;

class FeatureContext implements SnippetAcceptingContext
{
    protected $apiContext;

    /**
     * @BeforeScenario
     */
    public function gatherApiContext(BeforeScenarioScope $scope)
    {
        $environment = $scope->getEnvironment();

        $this->apiContext = $environment->getContext('Knp\FriendlyContexts\Context\ApiContext');
    }

    /**
     * @Then the response should contains :count elements in the :key key
     */
    public function theResponseShouldContainsElementsInTheKey($count, $key)
    {
        $response = $this->apiContext->getResponse();
        // Check whatever you want
    }
}

Pretty useful don't you think?

:+1:

@gquemener like from me :+1: .

Sure, I've just did


[FEATURE] Use AnyDataFrame and AnyExecutionEngine annotations

We created these two TypeVars for readability. We need to refactor the entire code space to use them.

https://github.com/fugue-project/fugue/blob/49d37249fd74dddddc94795a5474a31f3fbfdd45/fugue/dataframe/dataframe.py#L25
https://github.com/fugue-project/fugue/blob/49d37249fd74dddddc94795a5474a31f3fbfdd45/fugue/execution/execution_engine.py#L40


Can you make this compatible with tamed respawn

What i expected to happen when i had both mods on is this
i give my horse extra lives and if it runs out i can use the pet bandage, but instead saveyourpets just overules
can you make it wait till its depleted the extra lives from that mod before triggering? <3
https://www.curseforge.com/minecraft/mc-mods/tamedrespawn

That mod isn't open source, so I can't legally look at the code to see if I can make them compatible.


VTK and other files to native space

Apply combined transform to cropped .vtk files using greedy.
Could also propagate a few other useful files, like coords-AP.nii.gz, coords-PS.nii.gz, and coords-IO.nii.gz

Can definitely propagate the other images too.
For the vtk let me let me know when you have the physical xfm issue sorted out within the corobl space.

Just pushed the change that should have the xfm working correctly for generating .vtk files now. Still only about 80% sure that it'll work in all cases, but the nifti toolbox i'm using for matlab doesn't have a straightforward way to extract the correct xfm.
https://www.mathworks.com/matlabcentral/fileexchange/8797-tools-for-nifti-and-analyze-image (see xform_nii())

pushed to Hippocampal_AutoTop that is. will still need to recompile, etc.

Looks like it fixed the xfm issue..
However, something (likely unrelated) is that the midsurface doesn't actually appear in the mid-IO layer -- e.g.:

might be worthwhile generating both inner and outer surfaces as well, useful anyways, but also could be useful for debugging..

I can confirm that this fixed the issue but see the same thing as @akhanf . @jordandekraker I remember we had a similar issue couple months ago and that there was an offset of 1 voxel (0.3 mm) in each direction. Screenshot below shows when applying these offsets in the LR (+1 voxel), AP (-1 voxel) and IS (-1 voxel) directions:

Seems to align better?

Good points, thanks. I'll add the inner and outer surface, and for the time being I can add +/-1 to the needed dimensions but I still haven't been able to identify where this off-by-1 issue comes from. I'll leave a TODO note in the code to this effect, and maybe we can leave this thread open until we can definitely nail down why this is occurring.

Hi @jordandekraker, I had a look at your updated code and tested it but I think line 143 in coords_SurfMap.m should be like:
v = reshape(v,[APres*PDres,3]) + [-1 -1 -1];
It looks fine after that change.

Thank Roy! Just made the update

closing this since all the gifti files are now provided, with folder structure finalized in #24

closing this since all the gifti files are now provided, with folder structure finalized in #24


Project Requirements: have a polished UI

AS A developer
I WANT to have a polished UI
SO THAT our application looks nice and we follow the project guidelines

Project UI structure MVP complete.


let users customize aspect ratio of charts

again, something to scratch my own itch. Maybe it can help with #41 .
Thank you!!

Thanks for the PR @erdostom
I tried to change the heightRatio to 1 and the graph seems not right. (Haven't get the chance to debug it)

Besides, do you think aspectRatio might be a better option name?


out_forward: Support mutual TLS. fix #1879

Signed-off-by: Masahiro Nakagawa<EMAIL_ADDRESS>

@okkez Yes but we need feedback from other users.

No feedback but okkez confirmed the behaviour and this change doesn't break existing behavior.
So I will merge this patch and release v1.3.2.


How Do I Customize Prompt with Additional Custom Prompts?

I have a folder of prompt ps1 files that allow for loading customized prompts, such as AWS Profile and Region name, posh-git status, and others. I'd like to see in the readme an example of how do I load these by customizing the prompt profile

Hey @sheldonhull - I will likely roll this into the main module, but right now the Prompts functionality is in one of the included plugins. Here's how you currently add the plugin to your profile config:
Import-Module PSProfile
$PSProfile.Plugins += @{
    Name = 'PSProfile.Prompt'
}
Save-PSProfile
$PSProfile.Load()

That should load the updated profile once you've saved it. This will be a bit more streamline in the near future (likely something like Add-PSProfilePlugin -Name PSProfile.Prompt)
Once you have it added, you should have some new functions available to enable prompt interaction:
Save-Prompt: This will do what you're looking to do and save your current prompt as a named prompt to call back later. For your use case, you'll want to...

Set your current prompt like you normally would
Run Save-Prompt -Name $handyShortNameToRememberItBy to save the current prompt in your PSProfile config.
Repeat for any additional prompts you have, giving each a unique name.

Set-Prompt: This is what you use to switch between prompts on your PSProfile config. If you only want to temporarily switch (i.e. not set the selected prompt as your default for persistence), include the -Temporary switch.
Get-Prompt: This will read back your current prompt's definition. Use the -Raw switch if you only want the prompt body, otherwise it will return a string containing the function prompt { ... } enclosure around it
Edit-Prompt: This will open your prompt for editing in Visual Studio Code using a temporary file. Once you make your edits, it will re-save your prompt on your PSProfile config and update it live so you can see your changes. VS Code is the only editor currently supported for this action.

Update to this process using the latest version of the repo:
Add-PSProfilePlugin -Name PSProfile.Prompt -Save

Sweet. Gonna try this out for sure

Updates on the released version, for reference:
PSProfile.Prompt has been rolled into the main code. All of the existing commands are still there, just as aliases instead pointing at the actual *-PSProfile* version:
Edit-Prompt

Also, if you want to try out my prompt, I include it with the default configuration (have a few people that have asked for it). You can check it out temporarily by running:
Switch-Prompt SCRTHQ -Temporary

Keeps getting better and better

Hey @sheldonhull - going to close this out! Please let me know if you have any other questions regarding this 😄

So I have a function that is a prompt for aws metadata. I want to leverage your prompt with all the existing functionality and add aws metadata, or lets say remove the [#1] session name you provided and instead plugin my prompt section.
How do I build a prompt with "components" like you did?
I'm struggling to find the configuration file that shows this so I can customize.
Another good example. I use nameit to generate random names for sessions instead of numbered sessions. There's something great about [Obtuse Penguin] [PS 5.1.1] [0:00:00.0910] 😄 . Do you have a help doc I can refer to that shows how to construct the sections of a prompt so I can customize?

@scrthq is it possible to use this approach like the Add-PowerlinePrompt or whatever does, where you can add several prompts together as "building blocks" like I mentioned? This would make it super easy to add to this or to an extensions repo a set of things like prebuilt aws prompt or other console features.
Having to put it all in a single prompt block is pretty brittle for edits. Just wanted to follow-up and see if I'm missing something here or if it's an enhancement follow-up request


Passing vmArgs (JavaFX11+ support)

Unless I am missing something which could very much be the case, It seems impossible to specify some vm arguments for the JVM.
This is mandatory with JavaFX11+ where one has to pass something like that:
--module-path /usr/local/javafx/javafx-sdk-13.0.1/lib/ --add-modules javafx.controls,javafx.fxml
The java launcher supports passing a vmArgs configuration for example to address this.

Looks like this was resolved by https://github.com/fwcd/vscode-kotlin/commit/44092af74bf597c65a56427ccbc5d0fa428c466b


Get purchaseToken by productID

Is it possible to receive a purchaseToken for a single productID without making a InAppBilling.purchase() or InAppBilling.subscribe()?

Yes, I would think so.
Not with the current API of this module, but it should be possible by wrapping getPurchaseTransactionDetails(productId) and getSubscriptionTransactionDetails(productId) from the anjlab library.
I might look into it sometime this week. If it is wanted sooner a PR would be welcome! I do not think it is that much work.

Okay cool, i just digged into anjlabs library and i will check it this week and then make a PR.

Cool!
Looks like the purchase history is grabbed synchronous, so you don't have to fool around with caching promises.
Just resolve it normally, with the same WritableMap which is created in the onProductPurchased-callback.
Wouldn't be a bad idea to refactor the TransactionDetails to WritableMap mapping out into its own method so you can re-use the same logic in "all" the methods (onProductPurchased, getPurchaseTransactionDetails, getSubscriptionTransactionDetails).
Something like this in the InAppBillingBridge;
@ReactMethod
public void getPurchaseTransactionDetails(final String productId, final Promise promise){
    if (bp != null) {
        TransactionDetails details = bp.getPurchaseTransactionDetails(productId);
        if (details != null && details.productId == productId)
        {
              WritableMap map = mapTransactionDetails(details)
              promise.resolve(map)
        } else {
            promise.reject("Could not find transaction details for productId.");
        }
    } else {
        promise.reject("Channel is not opened. Call open() on InAppBilling.");
    }
}

private WritableMap mapTransactionDetails(TransactionDetails details) {
    WritableMap map = Arguments.createMap();

    map.putString("receiptData", details.purchaseInfo.responseData.toString());

    if (details.purchaseInfo.signature != null)
        map.putString("receiptSignature", details.purchaseInfo.signature.toString());

    map.putString("productId", details.productId);
    map.putString("orderId", details.orderId);
    map.putString("purchaseToken", details.purchaseToken);
    map.putString("purchaseTime", details.purchaseTime.toString());

    ResponseData responseData = details.purchaseInfo.parseResponseData();
    map.putString("purchaseState", responseData.purchaseState.toString());

    return map;
}

The above is completely untested though. Good luck :+1:

Thanks @cbrevik it works !

My implementation: https://github.com/thstarshine/react-native-billing/commit/8f0dd3e1ddbe1d93aa83cb7156d7e7934006c40a for reference.
Just modify string comparison to use equals instead of ==.

Great @thstarshine! Could you possibly create a PR from this?

Sure! Thanks for this great addon.
https://github.com/idehub/react-native-billing/pull/10

This has now been resolved, should be available in v1.2.0. Thanks!


Debug Port Range

Debug port is now a random port in specified range, also features are documented in readme. #16

This should have a test.

ok, but how sensible is jsunit about changing process.execArgv in the setup method? It can't be tested otherwise

true, this would be pretty hard to get a good test going in travis-ci. if there's any problems you still own it ;)


Upload built spec as artifact to ease debugging

After building each document, it should be uploaded (along with all the relevant assets) as an action artifact. This could help in diagnosing validation/publish failures. Retaining the artifacts for like 3 weeks should be enough.

Helpful: https://github.blog/changelog/2021-08-25-github-actions-reduce-duplication-with-action-composition/


How to generate the labels for a new design

Hello.
I have generated some new data with verilog files by OpenROAD, and exstracted the features needed by the TimingPredict's gcn model.
For the labels, such as net delays, I generated them by OpenSTA, which embedded in the OpenROAD and read from the sdf files.
But it turns out that the model's results(R2_score) are not so satisfactory.

Is my appoach to genrate labels wrong? And how do you genrate the labels?
Have you preprocessed the features saved as bin files(eg. data/8_rat/aes128.graph.bin)?
Thank you!

Feature preprocessing such as distribution transformation is implemented during reading from the bin files but not inside the bin files themselves.
Can you provide more details on your setup? Following are are few notable questions.

Are you using the same PDK (sky130) as our setup?
Have you completed the whole implementation process including parasitics extraction and computed the delays and arrival times in OpenSTA?
How do you evaluate the R2 score (e.g., with net delay or arrival time? if with arrival time, are you using our modified OpenSTA repo to output the arrival times for comparison)?

Thanks for your interest in our project.

Hello! Thanks for your quick response.
Here are the details on my setup and results.

I have used the nangate45 PDK to genrate the train and test datasets. I will try the sky130 PDK for the later experiments.
I have used the OpenROAD-flow-scripts to completed the whole implementation process. And read the net delays from the given final sdf files's INTERCONNECT
(e.g., (INTERCONNECT split850/Z 26514/B3 (0.001::0.001) (0.001::0.001))).
I have only evaluted the R2 score with the net delay.

Thanks you!

Hello! Thanks for your quick response.
Here are the details on my setup and results.

I have used the nangate45 PDK to genrate the train and test datasets. I will try the sky130 PDK for the later experiments.
I have used the OpenROAD-flow-scripts to completed the whole implementation process. And read the net delays from the given final sdf files's INTERCONNECT
(e.g., (INTERCONNECT split850/Z 26514/B3 (0.001::0.001) (0.001::0.001))).
I have only evaluted the R2 score with the net delay.

Thanks you!

The difference in PDK might be the reason, because our current model cannot generalize to different PDKs yet. You can try training a new model or with some fine tuning and see if it works.

Thank you so much!
I will try sky130 PDK later.


Initial CPU credits?

Quickly creating a test B-series VM, I can see (from the "CPU Credits Remaining" in the metrics pane in the Azure Portal) what looks like 500 credits.  This isn't mentioned in this article.  Is this just an initial allocation so that the instance has reasonable performance during setup/build?  I assume these expire after a certain amount of time?

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

ID: f435b3d1-2e0b-378c-ce89-97defa822eff
Version Independent ID: 32c3ea31-ae89-5a61-f99e-368bd7d72abc
Content: B-series Azure Windows VM sizes
Content Source: articles/virtual-machines/windows/b-series-burstable.md
Service: virtual-machines-windows
GitHub Login: @jonbeck7
Microsoft Alias: jonbeck

This is potentially a case of unclear numbers in the metrics view in the Portal.
The above was using the "Sum" aggregation in the chart.
Having had the machine running for a couple of hours now and choosing the "Avg" aggregation in the chart the numbers align better with that given in the documentation above.  So, I think "Sum" is misleading - not sure what it is summing over, but I assume there are multiple values per minute.

@cbailiss Thanks for the feedback! We are currently investigating and will update you shortly.

@cbailiss All Burstable VM's get a initial credit and that will not expire automatically. Like you guessed, That's like earned credit and  decreases when usage is above the base performance.
I will check about the "sum" in the chart and post a comment.

I am assigning to the content author to review and provide additional information.
@jonbeck7 can you take a look?

@MicahMcKittrick-MSFT  can you reassign to @joelpelley?
@joelpelley it sounds like there is some confusion about how the B-series is represented in the Azure portal.

B-Series is now owned by @Aysha<EMAIL_ADDRESS>Please assign to her.
Best regards,
Joel Pelley PMP
Principal Program Manager - Azure Compute VM Sizes and Hardware
Email  joelpell<EMAIL_ADDRESS>|  office<PHONE_NUMBER>
[MSFT_logo_Gray DE sized SIG1.png]
“You have power over your mind – not outside events.  Realize this, and you will find strength.”

Marcus Aurelius

This email message may contain confidential and proprietary information.  Any unauthorized use is prohibited.  If you are not the intended recipient, please contact the sender by reply email and destroy all copies of the original message.
From: Cynthia Nottingham<EMAIL_ADDRESS>Sent: Monday, March 4, 2019 12:29 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Joel Pelley<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Initial CPU credits? (#25126)
@MicahMcKittrick-MSFThttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicahMcKittrick-MSFT&data=02|01|Joel.Pelley%40microsoft.com|6954127dee88459decb508d6a0e0174a|72f988bf86f141af91ab2d7cd011db47|1|0|636873281658801020&sdata=2DVd2uHI7LBsbFz%2FZJ802a0%2FmCskqtdhFaCwlNSS7AU%3D&reserved=0 can you reassign to @joelpelleyhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fjoelpelley&data=02|01|Joel.Pelley%40microsoft.com|6954127dee88459decb508d6a0e0174a|72f988bf86f141af91ab2d7cd011db47|1|0|636873281658811016&sdata=N8KoLTQ8bcrtHqigQ%2B2B115i%2FblL9ngUXDcDr2y7IYI%3D&reserved=0?
@joelpelleyhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fjoelpelley&data=02|01|Joel.Pelley%40microsoft.com|6954127dee88459decb508d6a0e0174a|72f988bf86f141af91ab2d7cd011db47|1|0|636873281658811016&sdata=N8KoLTQ8bcrtHqigQ%2B2B115i%2FblL9ngUXDcDr2y7IYI%3D&reserved=0 it sounds like there is some confusion about how the B-series is represented in the Azure portal.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F25126%23issuecomment-469409554&data=02|01|Joel.Pelley%40microsoft.com|6954127dee88459decb508d6a0e0174a|72f988bf86f141af91ab2d7cd011db47|1|0|636873281658821016&sdata=KknZFI0Jt6aL5Rc6SqtmK6CFTlrFS2yelYBlBtYFlFM%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAjgTWwlQS6NQpcXh8Y8AqTNVy1wOl4kSks5vTYIkgaJpZM4a_d3F&data=02|01|Joel.Pelley%40microsoft.com|6954127dee88459decb508d6a0e0174a|72f988bf86f141af91ab2d7cd011db47|1|0|636873281658821016&sdata=6VQ5aLzrKOfOVX6T18lzDdSmkN8g2nCXqhytVHCOKp8%3D&reserved=0.

@joelpelley looks like @aysha is not available to be assigned issues in this repo

Please assign to @Jon Beck (jonbeck<EMAIL_ADDRESS>Best regards,
Joel Pelley PMP
Principal Program Manager - Azure Compute VM Sizes and Hardware
Email  joelpell<EMAIL_ADDRESS>|  office<PHONE_NUMBER>
[MSFT_logo_Gray DE sized SIG1.png]
“You have power over your mind – not outside events.  Realize this, and you will find strength.”

Marcus Aurelius

This email message may contain confidential and proprietary information.  Any unauthorized use is prohibited.  If you are not the intended recipient, please contact the sender by reply email and destroy all copies of the original message.
From: Micah<EMAIL_ADDRESS>Sent: Monday, March 4, 2019 12:53 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Joel Pelley<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Initial CPU credits? (#25126)
@joelpelleyhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fjoelpelley&data=02|01|Joel.Pelley%40microsoft.com|741cec4f7c3441c6ee2e08d6a0e37592|72f988bf86f141af91ab2d7cd011db47|1|0|636873296121882336&sdata=iJkO%2FA9dlg8tt%2BcVu2SWLCpz0H1BFnw%2B46TJ3OPXJ7Q%3D&reserved=0 looks like @ayshahttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Faysha&data=02|01|Joel.Pelley%40microsoft.com|741cec4f7c3441c6ee2e08d6a0e37592|72f988bf86f141af91ab2d7cd011db47|1|0|636873296121882336&sdata=1zVmEL6R%2FmxnP%2BbXKeaq1Sq1OCj1OfzteFgV8xkr%2Bgo%3D&reserved=0 is not available to be assigned issues in this repo
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F25126%23issuecomment-469417338&data=02|01|Joel.Pelley%40microsoft.com|741cec4f7c3441c6ee2e08d6a0e37592|72f988bf86f141af91ab2d7cd011db47|1|0|636873296121892332&sdata=ddtXZE8jKFSPHHcqZCQQDMzdN9NTOooA95UfLIszwUI%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAjgTW9BHYRYIXe4uqmdjhvTi46tcrWauks5vTYfJgaJpZM4a_d3F&data=02|01|Joel.Pelley%40microsoft.com|741cec4f7c3441c6ee2e08d6a0e37592|72f988bf86f141af91ab2d7cd011db47|1|0|636873296121892332&sdata=EkDxnHgRDT3twPkfanB0qbcVTECKap9KboPewFSCG%2F0%3D&reserved=0.

Please assign to @Jon Beck (jonbeck<EMAIL_ADDRESS>Best regards,
Joel Pelley PMP
Principal Program Manager - Azure Compute VM Sizes and Hardware
Email  joelpell<EMAIL_ADDRESS>|  office<PHONE_NUMBER>
[MSFT_logo_Gray DE sized SIG1.png]
“You have power over your mind – not outside events.  Realize this, and you will find strength.”

Marcus Aurelius

This email message may contain confidential and proprietary information.  Any unauthorized use is prohibited.  If you are not the intended recipient, please contact the sender by reply email and destroy all copies of the original message.
From: Micah<EMAIL_ADDRESS>Sent: Monday, March 4, 2019 12:51 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Joel Pelley<EMAIL_ADDRESS>Assign<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Initial CPU credits? (#25126)
Assigned #25126https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F25126&data=02|01|Joel.Pelley%40microsoft.com|52da2765fca94436ff9908d6a0e31921|72f988bf86f141af91ab2d7cd011db47|1|0|636873294620370350&sdata=uE8f%2FTBwEs9DlSjnWez9lQYk8JOHZQi86le4sbR94nc%3D&reserved=0 to @joelpelleyhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fjoelpelley&data=02|01|Joel.Pelley%40microsoft.com|52da2765fca94436ff9908d6a0e31921|72f988bf86f141af91ab2d7cd011db47|1|0|636873294620370350&sdata=Br5qm4%2FQh9LDPPfqBS4FisNPGTVndGtmiVFFEBCX0s0%3D&reserved=0.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F25126%23event-2179079668&data=02|01|Joel.Pelley%40microsoft.com|52da2765fca94436ff9908d6a0e31921|72f988bf86f141af91ab2d7cd011db47|1|0|636873294620380354&sdata=38JC5bFEypCWBUyHfghiHYhvMCjv2fIJRPm%2BP%2BE%2Bap8%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAjgTWzSbsEuFYRLdMqO6Da4CVDcLGttXks5vTYcugaJpZM4a_d3F&data=02|01|Joel.Pelley%40microsoft.com|52da2765fca94436ff9908d6a0e31921|72f988bf86f141af91ab2d7cd011db47|1|0|636873294620380354&sdata=2yfbYzQxix2dBvXC1hCo%2BSpG4lSpqP4F7LiILK3W6%2FQ%3D&reserved=0.

#reassign: @MicahMcKittrick-MSFT

We call out the initial credits you get after deploying a new B series VM based on the size. This information is called out in the table. I will close this but let me know if there is further confusion I can help with.


Auto-deployments are unreliable - periodic tests are flaky

Filing this as an uber issue for why the auto-deployments are unreliable leading to flaky periodic tests.

https://kf-v1-1-0714-295.endpoints.kubeflow-ci-deployment.cloud.goog/
The deploy step failed.
Looks like it timed out waiting for the cluster to be ready.
The clusters are ready now; so maybe we just need to bump the timeouts?


[AF-323] Update iframe only template

🍞
/cc @zendesk/vegemite
Description
Update iframe only template to include new APIs
References

JIRA: https://zendesk.atlassian.net/browse/AF-323

Risks

None (template changes only)

wait for resize to hit prod?

:+1: for when API is working in production

👍


Add: Rotting Oranges

Codecov Report

Merging #380 into master will decrease coverage by 0.29%.
The diff coverage is 85.18%.

@@           Coverage Diff            @@
##           master    #380     +/-   ##
========================================
- Coverage     100%   99.7%   -0.3%
========================================
  Files         131     132      +1
  Lines        1341    1368     +27
========================================
+ Hits         1341    1364     +23
- Misses          0       2      +2
- Partials        0       2      +2

Impacted Files
Coverage Δ

problems/rotting-oranges/rotting_oranges.go
85.18% <85.18%> (ø)

Continue to review full report at Codecov.

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


Add 1-click-bom file

This can be used with the 1-click BOM browser extension to quickly put all the parts into the Digikey cart.
Also, if someone turns it into a PCB it would be really nice to add it as a project on kitnic.it.

Thank you! I changed the line but also made it link to the tsv file.

Thank you!


adding navbar in DevProfiles main page

@oyepriyansh, I'm interested in taking on the task of adding a navbar to the main page.
Could you please assign it to me?

@oyepriyansh, assign me this i think i can make this work

assigned to you

thankyou @oyepriyansh ..


Support non english locales for fixie's tests

Fixes the problems with some of Fixie's own tests on non-english localized machines according to issue #90.
I am not sure about the similar cases that are not failing on my machine:

NUnitXmlReportTests
XUnitXmlReportTests

This has been fixed in master.


Add prefer-button rule

This is a proposal to add a rule to check for onClick handlers on anchors and hint to rather use a button element.
Still outstanding:

Documentation: Will document this fully if this proposal gets accepted in principle.
Complete href checks: Companion to this rule there should be a check that extends the current check for "#' in the href to also warn when no href is present or when something like javascript:void(0) is used. Will also make these changes if this proposal gets accepted in principle.

I have also added the check to the strict rules section only, pending discussion.
Looking forward to hearing your ideas and opinions around this proposal.
P.S. Following issue has been solved below:
On a clean clone the following rule in the eslint-config-airbnb-base file causes an error on just about every single line of the repo on Windows:
    // disallow mixed 'LF' and 'CRLF' as linebreaks
    // http://eslint.org/docs/rules/linebreak-style
    'linebreak-style': ['error', 'unix'],

Also the flow script in package.json fails with an ! was unexpected at this time. error.
I could work around it but should I raise an issue? Or do would you rather stay with the current config and propose a workaround for Windows devs in the docs?

Coverage decreased (-0.1%) to 99.413% when pulling 3e15f97aa48c8a4d6b56e878bc8ba49d696a3a86 on AlmeroSteyn:master into 6be5eab70af08c3aaa3bcb05042bf7d59bf42d62 on evcohen:master.

@AlmeroSteyn you can fix your windows issues by setting up git to automatically convert between LF and CRLFs, and then delete and re-clone all your repos. I'm not aware of a way around doing that for Windows users for general dev across platforms.

@ljharb Aaaah! That'll sort it, thank you.

 is a pretty solid indication that this element is
being used a button. That's with the href set to '#'. The href content
could be made configurable.
On Mon, Apr 24, 2017 at 11:40 PM Almero Steyn<EMAIL_ADDRESS>wrote:

@AlmeroSteyn commented on this pull request.
In src/rules/prefer-button.js
https://github.com/evcohen/eslint-plugin-jsx-a11y/pull/219#discussion_r113116977
:

meta: {

docs: {},
schema: [schema],
},

create: (context: ESLintContext) => ({
JSXOpeningElement: (node: JSXOpeningElement) => {

 const type = elementType(node);

 if (type && type !== 'a') {

   return;

 }

 const anchorClick = getProp(node.attributes, 'onClick');

 if (!anchorClick) {

That's the hook then. If you're putting role="button" on elements, well,
it should just be a button then.
react-router uses an anchor with a click handler fwiw:
https://github.com/ReactTraining/react-router/blob/master/packages/react-router-dom/modules/Link.js#L65
@evcohen https://github.com/evcohen Ah yeah there's a use case. Dealing
with context things on navigation.
The check for just onClick is therefore not always valid.
The role="button" trigger could be more sane but thats kinda like Tortuga
in the Pirates movies, can only be found by those who already know where it
is. A dev will generally need to know about ARIA already to have added this
to be warned about ARIA.
Would a combination of role="button" and / or a missing or invalid href
value perhaps do the job? Or sticking to onClick in conjunction with a
missing or invalid href while allowing onClick if the href is properly
filled?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/evcohen/eslint-plugin-jsx-a11y/pull/219#discussion_r113116977,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAVHOdReVpywjqagMnrZ4HAStGGiYyTGks5rzZVbgaJpZM4NFXOR
.

An href of "#" or an href starting with "javascript:" should both be flagged as "should be a button", imo. Anything else could be a link with progressively-enhanded onClick behavior, which could be fine.

An href of "#" or an href starting with "javascript:" should both be flagged as "should be a button", imo. Anything else could be a link with progressively-enhanded onClick behavior, which could be fine.

If we are going in the direction with a more semantic rule like this, should we deprecate href-no-hash and move that logic into here? This can report on the following:
 -> use an href that is a valid reference to something a user can navigate to
 -> use an href that is a valid reference to something a user can navigate to
 -> use a button
 -> use a button
 -> PASSES
is this correct?

I'd say  is "use a button", not "use a better link" - otherwise that sounds good.

Likewise, sounds good.

If we are combining the two validations, maybe the name should change as it then covers more than hinting at buttons.
How about anchor-is-valid with the rules mentioned above?
And what about anchor-has-content? Should that be pulled in as well?
Will make these changes in the coming day or three in the PR.
So as I see it (for now without anchor-has-content):
Should be button

Add a valid href

PASSES

Thank you all for considering this, by the way!

button-is-button?

Coverage increased (+0.02%) to 99.544% when pulling c51abc4bc18f4fc98f7c97b368117de7fcb4887d on AlmeroSteyn:master into 5be6cd62b8c7cbeaa4192dc6869a210a56561f01 on evcohen:master.

Coverage increased (+0.02%) to 99.544% when pulling 6079474a2fab0c8dc3c4ea8a16a39920163e2710 on AlmeroSteyn:master into 5be6cd62b8c7cbeaa4192dc6869a210a56561f01 on evcohen:master.

Coverage increased (+0.02%) to 99.544% when pulling e996b226c4bf5865c7ef837aef9ec0fde709ac6f on AlmeroSteyn:master into 5be6cd62b8c7cbeaa4192dc6869a210a56561f01 on evcohen:master.

Coverage increased (+0.02%) to 99.544% when pulling e996b226c4bf5865c7ef837aef9ec0fde709ac6f on AlmeroSteyn:master into 5be6cd62b8c7cbeaa4192dc6869a210a56561f01 on evcohen:master.

I have added the logic discussed.
Unfortunately, I seemed to have accidentally pulled some old commits into the history here. My latest commit shows only the files I have changed.
If required I can open up a new pull request with a clean history and use the new rule name.
Does this cover everything? And do you think we should add the empty anchor check here?

@AlmeroSteyn can you rebase? I think a bunch of @jessebeach commits are in here not sure what happened

It looks like she's just the pusher; i.e., she pushed their commits (two avatars show up)

@evcohen It happened when i squashed my commits. Something went wrong so it pulled these old commits in. Will see what I can do in the morrow. My very last commit contains all the changes related to this PR.

Hmm sorry everyone, in trying to fix it I properly broke it. Reset back to upstream as the commits were already registered in the rebase and rebasing did not seem to work and that autoclosed this PR. So I had to create a new one in #224. Sorry for the inconvenience.
¯(°_°)/¯

I find git rebase -i master to be very helpful when history gets wrinkled. It let's you choose the commits to include in a rebase, so you can ignore the extra commits that are already in the base.

I use git rebase -i origin/master (and usually just always delete my local master) to ensure that i'm always using the latest commits.

@jessebeach @ljharb Thanks! Oh the dark art of Git history (which loosely translates to I flipped every switch Google could find till I broke it and it did not include this switch so I have made an immediate note of it for future).


Resource Manager Acceptance Tests

We would like to have acceptance tests, but service accounts don't have permissions to access most of the Resource Manager API. This issue is a placeholder for these tests.
See https://github.com/GoogleCloudPlatform/gcloud-python/issues/1105#issuecomment-135467296 and https://github.com/GoogleCloudPlatform/gcloud-node/issues/851#issuecomment-140620629

Closing this issue as the behavior of Resource Manager that blocks the use of service accounts is by design and not something that is likely to change.
If the situation changes and these acceptance tests are wanted in the future, please open an new issue and reference this issue.


Improve Bower project setup documentation

As per issue #131, we should mention how to make sure external Sass dependencies should be linked using native Sass includePaths option.
Grunt
sass: {
  options: {
    outputStyle: 'expanded',
    sourceMap: true,
    indentedSyntax: true,
    sassDir: 'scss',
    cssDir: 'dist/css',
    includePaths: [
      'bower_components/mathsass/dist'
    ]
  }
}

Gulp
const SASS_INCLUDE_PATHS = [path.join(__dirname, 'bower_components/mathsass/dist')];
gulp.task('sass', function() {`
    gulp.src([path.join(DIRS.css, 'main.scss')])
        .pipe(echo())
        .pipe(sass({
            includePaths: SASS_INCLUDE_PATHS
        }).on('error', sass.logError))
        .pipe(sourcemaps.init())
        .pipe(minifyCSS())
        .pipe(rename({extname: '.min.css'}))
        .pipe(sourcemaps.write())
        .pipe(gulp.dest(DIRS.css));
});

@jrah We should also add the Grunt example above, for reference.


Deploy in Container apps job?

Maybe I'm missing where the evaluator gets deployed but would it make sense as a container app job?
@anthonychu

I just run it locally, but it could be a good fit for ACA jobs. Not sure if we'd want to parallelize it (chunk up the data) or just run it all. Runs take about an hour on 200-question ground truth data sets.


AttributeError: module 'mlx.core' has no attribute 'simplify'

File "./gpt2-mlx/transformer.py", line 168, in loss
mx.simplify(loss)
AttributeError: module 'mlx.core' has no attribute 'simplify'
simplify was removed.
#1 https://github.com/ml-explore/mlx-examples/pull/379

mlx should match version 0.0.10
0.0.10

hey @sweetcard thanks for pointing out the issue. Please let me get back to you on this 🙂. I'll upgrade to the latest version of MLX, which will require a few other changes as well, particularly using their new learning rate schedulers. I implemented a custom learning rate scheduler in this project before they supported it, but my custom implementation is now broken with newer versions of MLX

@dx-dtran Really sad. Wondering what the solution is for this situation?

same issue here. would this be fixed? thanks

just go to terminal and use the 0.0.10 version it worked for me
pip3 install mlx==0.0.10


Special characters not displayed in embedded terminal

VSCode Version: 1.25.1
OS Version: xubuntu 16.04.4 LTS

Steps to Reproduce:

run tig in embedded terminal
special chars used to display branching are not shown correctly unless you highlight line by moving selection up and down

Not selected:

Selected:

Does this issue occur when all extensions are disabled?: Yes

Experimental duplicate detection - is awesome!


Synthetic Data Component Added (Backend Commit) / Classification and Regression

Hi Guys,
This pull request introduces new scripts for generating synthetic data for classification and regression tasks.
Key changes mentioned below:

Added synthetic_data.py

Implements the CreateSyntheticData class to handle synthetic data generation

synthetic_data_opts.py

Defines the SyntheticDataOptions dataclass to manage options for synthetic data creation.

assertion.py

data_loader_asserts.py:
Added assertions to validate synthetic data options.

data.py

Modified data.py for integrating the synthetic data component.
Kindly review the code and do the needful.
Thanks and regards,
Karthik

Just those last two unnecessary problem type checks

Ok Dan.

Ok Dan. I'll merge it on the main.
May I know what would be the next step? Is it working on the front-end part of the code for Synthetic data and Bayesian Regularised Neural Network yeah or have you got any other tasks?


Unable to override button higlight group

(https://github.com/goolord/alpha-nvim/blob/d54ce84559ee50500a1133d1c58daf9cd715525b/lua/alpha/themes/dashboard.lua#L44)
I am unable to change this highlight from "Keyword"

local function button(sc, txt, keybind, keybind_opts)
  local b = dashboard.button(sc,txt,keybind,keybind_opts)
  b.hl_shortcut = "foo"
  return b
end

should be sufficient


assets-for-api-docs repo has expired cirrus token

The cirrus/GCP token for the assets-for-api-docs repo is https://github.com/flutter/assets-for-api-docs/blame/master/.cirrus.yml#L1 is 4 months old and (I think) expired. For example, this PR's pre-submit tests are failing: https://github.com/flutter/assets-for-api-docs/pull/174

Fix is in.


ARROW-719: [GLib] Release source archive

I don't know about his approach is good but it will help you to consider what approach is better.

It seems Apache Thrift (which also has a GLib-based C implementation) makes source releases requiring autotools. Could you tell us more about the use case? One issue with creating a signed release archive for arrow-glib is that it's an extra step to validate the additional artifact when making releases.
The make dist source release could be thought of as a sort of "binary release" because it is not strictly a source release as it includes files generated by autotools, so it's possible we could publish the arrow-glib tarball as a separate artifact after the main source release happens.
@ryanblue @tomwhite sorry to pull you into this thread, but could you advise on what other ASF projects have done in these kinds of situations? Thanks

It seems Apache Thrift (which also has a GLib-based C implementation) makes source releases requiring autotools.

Really? https://www.apache.org/dist/thrift/0.10.0/thrift-0.10.0.tar.gz includes configure (= GNU Autotools isn't required for users). configure isn't included in https://github.com/apache/thrift/ .

Could you tell us more about the use case?

Sure. I should add a link to JIRA: https://issues.apache.org/jira/browse/ARROW-719
If we bundle generated configure and related files, users don't need autoconf-archive. The autoconf-archive dependency may confuse users like you did at https://github.com/apache/arrow/pull/418#issuecomment-288415708
If user's platform is old and packaged GNU Autotools is old, the user may fail to generate configure. If we bundle generated configure and related files, we can avoid the case.
I hope that this description helps you.

Really? https://www.apache.org/dist/thrift/0.10.0/thrift-0.10.0.tar.gz includes configure

oops, sorry about that! I was looking in lib/c_glib. Is there a way we can generate only a single tarball in the source release, then, like Thrift?

Is there a way we can generate only a single tarball in the source release, then, like Thrift?

How about the following?
git archive ... | tar xf -
...
cd apache-arrow-XXX/c_glib
make dist
mv apache-arrow-glib-XXX.tar.gz ../
cd -
rm -rf apache-arrow-XXX/
git archive ... | tar xf -
tar xf apache-arrow-glib-XXX.tar.gz
rm -rf apache-arrow-XXX/c_glib
mv apache-arrow-glib-XXX apache-arrow-XXX/c_glib
tar czf apache-arrow-XXX.tar.gz apache-arrow-XXX

That seems fine to me -- do you want to put this in the 02-source.sh? @xhochy?
If cross-platform release becomes an issue we can always add a Dockerfile, I don't think it's an issue for the moment since my understanding is that autotools does not generate a platform specific configure file

That's fine, put in in 02 and if it's really a problem, I'll spent the 10min to add a Docker option if really necessary.

I've force pushed commit that uses 02 instead of creating 03.


Finalise changes for Authenticator Module / Connect to this Repo

Finalise changes for Authenticator Module / Connect to this Repo
This task is to finalise the changes required within the authenticator module flow and connect the authenticator to this repository.

@oleggrib - I've assigned you to this task as you have been working on most of the parts. But I can help connect this if needed etc to this repo.

Amazing work on this task Oleh. Closing this with an updated diagram of the final flow now implemented.


#974 remove one or two escapes for supporting JSON

Hi.  This is an attempt to fix #974 (cc @ryanjbaxter )
As mentioned previously in the ticket, the original code tries to mask out escaped variables (i.e. \${hello}) before performing replacement, and then finally removing the escapes before returning.
This works fine for .yaml or .properties, but not for JSON.
The jackson serialization that happens before the property resolution ends up adding an extra \ (so that the JSON is well-formed) EnvironmentController.labelledJsonPropertes.
I've updated the resolverPlaceholders to accept one or two escapes (\ or \\).
I realize this will now treat \\ as an escape in yml or properties, which may be unintended, but seems unlikely.
An alternative might be to add a flag to this method to indicate whether or not to allow double escape, and updating the labelledJson method to pass the flag?
I'm open to any feedback on this approach or another.
Thanks!

@ogrodnek Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@ogrodnek Thank you for signing the Contributor License Agreement!

Codecov Report

Merging #982 into master will decrease coverage by 0.12%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master     #982      +/-   ##
============================================
- Coverage     75.88%   75.76%   -0.13%
- Complexity      816      817       +1
============================================
  Files           100      100
  Lines          3119     3120       +1
  Branches        419      419
============================================
- Hits           2367     2364       -3
- Misses          584      587       +3
- Partials        168      169       +1

Impacted Files
Coverage Δ
Complexity Δ

...nfig/server/support/EnvironmentPropertySource.java
100% <100%> (ø)
7 <2> (+1)
:arrow_up:

...environment/MultipleJGitEnvironmentRepository.java
75.79% <0%> (-3.19%)
32% <0%> (-1%)

...erver/environment/NativeEnvironmentRepository.java
84.78% <0%> (-1.45%)
43% <0%> (-1%)

.../server/environment/JGitEnvironmentRepository.java
81.81% <0%> (+0.36%)
89% <0%> (+1%)
:arrow_up:

...erver/environment/SvnKitEnvironmentRepository.java
75.3% <0%> (+2.46%)
19% <0%> (+1%)
:arrow_up:

Continue to review full report at Codecov.

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


Installation instructions not working on Win7

I have GitHub for windows installed but the pip install instructions in the demo notebook didn't work for me (cannot find command 'git'). Could you help? Looks like a very cool project.

We'll likely register the qgrid package on pypi at some point, so pip install can be used without needing git.

It looks like there is still no qgrid package on pypi and the only way to install qgrid on windows is after git installation... Maybe it's worth to mention it in readme.md.

@wikiped qgrid is on PyPI: https://pypi.python.org/pypi/qgrid.  pip install qgrid should Just Work :tm: if you have numpy installed.

Thanks, it did work indeed.
For some reason when I was doing pip search qgrid before installing it the only result I was getting is js.jqgrid.


Bug: v4 Children of implied parent are no longer siblings

Hi ScepticMystic!
I built my breadcrumbs structure in the old version using just d:: [[Child Note Name]] in all my MOCs. It worked great!
In v4, Matrix view in the right-hand side pane no longer shows a note's siblings.
In the hierarchy settings I have selected 'opposite direction', which means that the parent of my note is shown in the Matrix view.
And I have selected 'Same Parent -> Siblings' too, but the notes which have the same implied parent are not showing.
It is as though it doesn't recognize the children of an implied parent as siblings, if that makes sense.

Hey Brett, thank you for the feedback!
Indeed, I hadn't considered that, but you're right. Currently, implied edges don't build off each other. Only real edges are considered.
I hear you though. Perhaps this can be added as an option. I want to think about how best to do it. Maybe you can choose a number of rounds? If you only go one round, then only real edges are considered. But if you go two or more, then real and implied and checked together.
I don't know if 3 or more rounds would have any difference tho... so maybe just a toggle between Real Only and Real And Implied.
What do you think?

Thanks for looking at my feedback!
A toggle between Real Only and Real And Implied sounds like it would work perfect for my system, so long as it's not too much trouble.

Making a note for reference.
I think the solution here is to introduce a "rounds" option, where you can decide how many rounds of implied edges to add. The narrower version of this is a toggle between 1 and 2 rounds.
On the first round, only real edges are considered. Instead of the builders adding edges directly, they should return a list of new edges, which all get merged together, deduped, then added in one go. This solves the duplicate issue, and ensures that the order the edge builders are called doesn't matter. They all start with the same info.
On rounds 2+, only implied edges are considered (maybe? I think that's right). Again, return a list of edges, merge, dedupe, add.

Hey @Brett-ai
Happy to announce that I'm implemented a feature to be able to control this in a fine-grained way :)
If you update to 4.0.19-beta, then go to Breadcrumbs Settings > Hierarchies > Hierarchy Settings, you can read a little about the new system. I've introudced "rounds" to implied relations, where you can decide how many times each implied relation kind gets added. Each time a kind starts a new round, it will consider all real relations, as well as the implied relations added in previous rounds.
You can read a little more in the beta README

Woo! Nice!
It works great now. Thanks so much for your work on this addon.


Route doesn't render when redirecting from nested layout route with autoCodeSplitting enabled

Which project does this relate to?
Router
Describe the bug
When redirecting from a nested layout route (routes/nested/_layout.foo.tsx), the route that is redirected to does not render. The issue does not occur when redirecting from an un-nested layout route (routes/_layout.foo.tsx). It also appears to have something to do with the code splitting as the problem does not occur when none of the routes are lazy loaded. In the reproduction example, autoCodeSplitting is enabled. When autoCodeSplitting is disabled, I'm not sure which routes need to be lazy loaded for the problem to occur.
Your Example Website or App
https://stackblitz.com/edit/tanstack-router-hdebcw?file=src%2Froutes%2Fnested%2F_layout.foo.tsx
Steps to Reproduce the Bug or Issue

Go to the reproduction link
Navigate to /nested/foo which redirects to /nested/bar in beforeLoad
See that 'Hello /nested/bar!' does not render
Navigate to /foo which redirects to /bar in beforeLoad
See that 'Hello /bar!' does render

Expected behavior
I expected that redirects will always render the resulting route no matter what level of nesting or layouts
Screenshots or Videos
No response
Platform

OS: macOS
Browser: Chrome
Version: 129.0.6668.71

Additional context
No response

#2820 has not fixed this issue.


No output

I get no output, I;m currently thinking that my connection to Postgres is somehow not working as it should.
Is there a way of testing that?

Not working and no error shown? That is a bit strange.
You can test the connection to postgress in many ways, best would be to test using pg gem and maybe pry interactive shell. Please se pg readme on how to connect to the database and perform some queries:
https://github.com/ged/ruby-pg
In case you need further assistance, please upload anonymized config file and the run output. You can just paste the output, or record whole terminal session e.g. by using https://asciinema.org/


not  enough values to. unpack

hi,guys.I pulled the code again from GitHub and reinstalled it, but why am I still getting the following error?

Can you share the code you tried?


Don't run tests for LGTM build - fixes #3652

Fixes #3652

CI build failure was on the master leg of the LGTM build and the merge leg completed without error.


cronjob: batch/v1 stable since k8s 1.21

The api is stable since k8s 1.21 and batch/v1beta1 is no longer served: https://kubernetes.io/docs/reference/using-api/deprecation-guide/#cronjob-v125

Signed-off-by: Lionel Hubaut<EMAIL_ADDRESS>

Fixed in the following PR #76


There is no formatter for 'json' files installed. when a big json file opened 53.8 MB

VSCode Version:
Version: 1.50.1
Commit: d2e414d9e4239a252d1ab117bd7067f125afd80a
Date: 2020-10-13T15:06:15.712Z
Electron: 9.2.1
Chrome: 83.0.4103.122
Node.js: 12.14.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.18362
Just open a big json,use Alt Shift F to format.

@jrieken I think most language services are disabled for large files, correct?

Yes, files larger than a certain limit (I recall 50MB but might be wrong) never make it to extension host

Closing. Behavior is as expected and necessary for performance reasons.


needs lock protection

https://github.com/prometheus/prometheus/blob/302148fd69a852aa8a25c37032349852038beeb1/discovery/manager.go#L168-L176
Function cancelDiscoverers  itself has no lock.
https://github.com/prometheus/prometheus/blob/302148fd69a852aa8a25c37032349852038beeb1/discovery/manager.go#L283-L285

The analysis is correct. I would probably modify  cancelDiscoverers to only call the cancel() functions and move the rest to ApplyConfig as when the manager's context is cancelled, we don't need to reset the fields anyway.

@simonpasquier I can work on this if no else is currently working on it


Recipes App - Version 1.0.0

Pull Request: Feature/RecipesAppEnhancement
Hello, dear reviewer. We are a team of two @Nessrine88  and me @Alejandroq12
General Requirements

Ensured no linter errors are present.
Followed correct Gitflow practices.
Documented work professionally.

Ruby Requirements

Adhered to the provided best practices for Ruby.

Project Requirements
Login and Registration

Implemented with Devise for authentication.

Recipes List

Displays a list of recipes created by the logged-in user, following the wireframe layout.
Clicking a recipe leads to its details.

Public Recipe List

Shows all public recipes ordered by newest, according to the wireframe.
Each recipe detail is accessible.
Owners can delete their recipes.

Recipe Details

Features a toggle for making a recipe public or private.
Displays details for public recipes or if the user owns the recipe, in line with the wireframe.
Owners can add new food to the recipe.
Ensured no N+1 query issues.

Navigation Menu

Created to allow users to access all pages.

Testing

Wrote necessary unit and integration tests using RSpec and Capybara.

Additional Requirements for Teams with Less Than 3 Members
Food List

Lists food added by the user, showing quantities.
Provides access to a form for adding new food.

General Shopping List

Displays food needed for the user's recipes not in their food list.
Calculates total items and price of missing food.

Technical Setup

Repository set up on GitHub with Gitflow.
Devise set up for authentication.
RSpec and Capybara configured for testing.

Workload Distribution

Established a Kanban board with GitHub projects to organize tasks.
Assigned time estimates to all tasks for fair distribution among team members.
Decided on common tasks to be divided or completed together (pair programming).
Ensured all tasks are based on our Kanban board cards.

Thank you for your valuable time. Have very good day.

@coderabbitai review it

@Alejandroq12, @Nessrine88, I've observed that a new commit was pushed after the review request. This practice can hinder the effectiveness of the review process. I recommend making all necessary changes before submitting the project for review. Doing so will facilitate a more comprehensive and constructive review. Best of luck, and if you require any assistance, feel free to reach out. Thank you!


[$1250] [HOLD for payment 2024-04-03] CRITICAL: [Track Expense] - FE - Add Track Expense Flow

https://docs.google.com/document/d/1gACVXuJS2ojNV_hwQRBLYFK_dR7vXSNlXfTnMOsdd_w/edit#bookmark=id.jw3ehgyt25yl
First, add a beta for 'trackExpense' so this only shows up for those in the beta
Add the Track Expense Button to the FAB
This will open up the standard money request selector
Instead of choosing a participant, it will automatically select the self-DM
This will optimistically create the reportAction to the self-DM

Held on 36361
Held on #368199
Note: We will have to hide the Distance option on the selector - this is held on P2P distance requests which is being worked on currently (cc @neil-marcellini )
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0115a976392ebf591e
Upwork Job ID:<PHONE_NUMBER>251933696
Last Price Increase: 2024-03-27

cc @shubham1206agra While Ishpaul is working on the self-DM support, could you get started on this issue?
This is holding on a few things, but there's a few things that can be done in advance like

Adding the beta and Track to the global create
Updating the IOU flow to skip the participant selector

The API call is almost done, and it would be nice to test with a draft PR to make sure we're not missing anything from the BE

@shubham1206agra Has a draft PR up, we're waiting on the self-DM to be merged first https://github.com/Expensify/App/issues/36361
Hopefully we can get something merged sometime late next week / early week after - does that sound like an accurate timeline once we have the self-DM for you to test with @shubham1206agra?

Still on track for 3/8?

3/8 is here!  Is it done?  If no, what is the next step, who is doing it, and when?

We've found some issues that need to be handled in the BE - I'm testing and coordinating changes with @shubham1206agra. We'll have a better sense of where things stand on Wednesday

Ok, so on Wed you'll have an ETA?

All accounts should have a selfDM now - we added everyone to the beta. I'll add you to the Track expense beta though

@shubham1206agra @ishpaul777 please apply in Upwork thanks -
upwork.com/jobs/~017fda999369b97b34

@shubham1206agra bump to apply in Upwork please
@ishpaul777 offer sent 👍

Payment Summary:
$1250 - @shubham1206agra bump to apply in Upwork please
$1250 - @ishpaul777 paid in Upwork

@ishpaul777 can you confirm if there are any regression test steps required here?

@ishpaul777 can you confirm if there are any regression test steps required here?

@thienlnam ^^


Added multiple k8s version for adapter tests

Signed-off-by: Tathagata Paul<EMAIL_ADDRESS>Description
Closes #5547
Notes for Reviewers
This change will essentially mean that we can remove the workflow input parameter: k8s_version from every adapter's e2e test file which references the test_adapters file
Signed commits

[x] Yes, I signed my commits.

cc @Revolyssup

hey @4molybdenum2, could you please share this topic on the call.

@4molybdenum2 If you want to test out the github workflow, you can just fork the meshery-istio repository and change the uses section of the workflow to 4molybdenum2/meshery/.github/workflows/test_adapters.yaml@allow-multiple-k8s-version

@Revolyssup @gr455 tests seems to be running fine for the k8s versions https://github.com/4molybdenum2/meshery-istio/runs/6703379423?check_suite_focus=true. I believe the update docs failure is due to missing credentials?

@Revolyssup @gr455 tests seems to be running fine for the k8s versions https://github.com/4molybdenum2/meshery-istio/runs/6703379423?check_suite_focus=true. I believe the update docs failure is due to missing credentials?

Yep. Looks good to me! @Revolyssup


Broken images while scaling with Firefox 58

Type of issue

[x] Bug report
[ ] Feature request

Uploader type

[x] Traditional
[ ] S3
[ ] Azure

Note: Support requests cannot be accepted due to lack of time.

Bug Report
Fine Uploader version
5.15.17
Browsers where the bug is reproducible
Firefox 58
Operating systems where the bug is reproducible
Mac OS X 10.13.3
Exact steps required to reproduce the issue

Select Many images
Wait for the gallery template to show all thumbnails

All relevant Fine Uploader-related code that you have written
https://jsfiddle.net/judm9dhh/11/
Your Fine Uploader template markup (if using Fine Uploader UI and the issue is UI-related)
https://jsfiddle.net/judm9dhh/11/
Detailed explanation of the problem
I experienced a bug which is not reproducible all the time, because i haven't found the actual reason why this happens. When a lot of images are added, the chance is bigger, that the problem is visible.
The scaled image is broken sometimes, where as the start of the thumbnail is shown normally but then turns white till the end.
Example:
https://imgur.com/a/tfFG1
We turned scaling off and this doesn't happen anymore. Also it does only happen with firefox, we also tested with chrome and safari but these don't show these errors. The error first came up in January, but we weren't able to track it down until recently.
Also we tried to replace the scaling methods using pica, but this also had broken images. So if pica and fine-uploader aren't using the same methods, it has to do with recent changes on firefox.
Maybe there is something like a memory limit per tab or so, which leads to this behavior. Also when uploading the files to a server with concurrent connections the error happens more often (2 is enough to test).
We can't turn off scaling, because our customers are usually uploading 200+ images and therefore it would take hours to upload. Hope you guys have an idea what i can do now.
Will also try with firefox on windows right now and report back if it happens there also.
Thanks in advance!
Edit: tried with 2600 images from windows 10 with firefox 58.0.2 and had 12 broken pictures.

Hi, it seems they had the same issue on https://github.com/nodeca/pica/issues/130. There is now a reported bug on firefox: https://bugzilla.mozilla.org/show_bug.cgi?id=1450302.

Thanks for looking into this @nicompte. I was unable to find the time to do that myself. I’ll close this as a 3rd party bug.


Question,How to use  search with languages Polish and Ukrainian

Have a. question, in support languages, cant found Polish and Ukrainian - how I can add it -  or how to search using full-text search for these languages.
Thanks

Hi,
Please refer to this ticket about adding stemmers etc.

closing as solved duplicate


Update design guidelines on tertiary auro-button and hyperlink

General Support Request

Support request
Designers do not have any guidelines on how to choose between a tertiary button and a hyperlink.
Possible Solution
According to user research, the current solution of buttons is accessible. Next steps are to write better guidelines on how to choose between a tertiary button and hyperlink. This can be done in "Do's and Don'ts" or general usage guidelines in the current doc site.
Context
Issue #141
Your Environment

Version used:
Browser Name and version:
Operating System and version (desktop or mobile):
Link to your project:

In addition to this, I'd like to expand on this content update to include a11y specific pages to the Auro doc site button and hyperlink sections.
The current demo pages speak specifically to a dev's ability to use the elements. I'd like to have content that speaks to the WHY things are done in order to address UX and a11y needs.

This is contradicting? Specifically speaking to the use of the underlined links in each example.

@blackfalcon The left example is an example of a piece of content. To my understanding, the "no underline" hyperlink should only be used as a navigational element. The right example is a "header menu" so it makes sense to use a "no underline" hyperlink... if that is unclear I will edit the examples.

Completed the decision tree after receiving feedback from the UI team.

I'd like to have content that speaks to the WHY things are done in order to address UX and a11y needs.

Can you elaborate on this more @blackfalcon? What are some examples of "things" that are done? Keyboard interactions? Visual elements?

Accessibility guidelines related to design will be added to a README file.


fix await error

Task exception was never retrieved
future: <Task finished name='Task-19' coro=<Client.on_message() done, defined at /home/Caeden/Github/revolt.py/__main__.py:6> exception=TypeError("object str can't be used in 'await' expression")>
Traceback (most recent call last):
  File "/home/Caeden/Github/revolt.py/__main__.py", line 8, in on_message
    await message.channel.send("hi how are you")
  File "/home/Caeden/Github/revolt.py/revolt/messageable.py", line 62, in send
    message = await self.state.http.send_message(await self._get_channel_id(), content, embed_payload, attachments, reply_payload, masquerade_payload)
TypeError: object str can't be used in 'await' expression

_get_channel_id was not an asynchronous function

make api_url and max_messages a kwarg

Thanks. Please undo the changes on examples/commands.py.

Wouldnt Bot be a more suitable name considering its a Bot with commands?

Thanks. Please undo the changes on examples/commands.py.

Wouldnt Bot be a more suitable name considering its a Bot with commands?

No, its a Client with commands.


Book Editing

This should make book editing work on Spigot+ servers. It sends the Java book update packet when it receives movement or inventory interactions from the player.

I just updated master on my bookediting branch (the idea was that I could look at your changes better and not get caught up in the rest of the changes made to Geyser) - do you think you could rebase your changes?

I'll do it - don't want to bother you with the hassle!


How to compile

Instructions because I have never compiled go before.

After installing some dependencies, just do go build or go install


[staging] gfortran: default to same gcc version as stdenv

This is a more expected default behavior, so that the unversioned
gcc/gfortran toplevel attrs have the same version. It's also more
robust in cases where C and Fortran objects are being linked together.
With this commit, gcc.version == gfortran.version == "10.3.0"
(on x86_64-linux), bumping the default gfortran version from 9 to 10.

Motivation for this change
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

The only change noted on the 9 -> 10 migration guide for fortran is that mismatched arguments cause errors now: https://gcc.gnu.org/gcc-10/porting_to.html
This usage was already invalid fortran, and I haven't encountered any affected packages, but I'll keep testing to see if it's something we need to worry about.

The elmerfem changes are fine. Definitely need to pull in someone more knowledgeable about the gfortran change than myself though.

Thanks, feel free to cherry-pick to master if you'd like to update separately. Grepping for gfortran only nets about 100 matches so I can probably make sure all the direct dependers are still working. Gonna mark this as draft until then.

Rebased to include https://github.com/NixOS/nixpkgs/commit/cc690c83f43c155e4046c5bc3b4b8cd6eb7d08b4

Note that the gcc patch for riscv 0d1750e9c041fa24752adb128625fee0952a6086 is only included by gcc 10 and 11. Thus on riscv platforms, current gfortran using gcc9 will suffer from sysdir issues.
I'd also think we should bump the default one.

This is a pretty big deal, this should be merged immediately. Example of an issue that can arise from this:
python
>>> import scipy.spatial
>>> import torch

15935 /nix/store/afi0ysqw20yiiw2gr2d28dx40bc4ddf8-python3-3.9.10/bin/python
python  15935 micahc  mem    REG  254,0   1952240 35015494 /nix/store/v4i14fk4ddy86jcw0k4s1n2d91p2m3dk-gfortran-9.3.0-lib/lib/libstdc++.so.6.0.28
$ lsof -p 15935 | grep libstd

python
>>> import torch

$ pgrep -f -a python
16636 /nix/store/afi0ysqw20yiiw2gr2d28dx40bc4ddf8-python3-3.9.10/bin/python
python  16636 micahc  mem    REG  254,0   1903088 34770303 /nix/store/ndnqiz3nnifj1blhg9q626xlmkqq1nmh-gcc-10.3.0-lib/lib/libstdc++.so.6.0.28

This means that torch when imported after scipy.spatial will get a difference libstdc++ than it was compiled with. I haven't tested everything in torch, but I can guarantee you there are bugs when this occurs. I suspect this probably happens in many places with code that is compiled with both stdenv and gfortran.

This is a pretty big deal, this should be merged immediately. Example of an issue that can arise from this:
python
>>> import scipy.spatial
>>> import torch

15935 /nix/store/afi0ysqw20yiiw2gr2d28dx40bc4ddf8-python3-3.9.10/bin/python
python  15935 micahc  mem    REG  254,0   1952240 35015494 /nix/store/v4i14fk4ddy86jcw0k4s1n2d91p2m3dk-gfortran-9.3.0-lib/lib/libstdc++.so.6.0.28
$ lsof -p 15935 | grep libstd

python
>>> import torch

$ pgrep -f -a python
16636 /nix/store/afi0ysqw20yiiw2gr2d28dx40bc4ddf8-python3-3.9.10/bin/python
python  16636 micahc  mem    REG  254,0   1903088 34770303 /nix/store/ndnqiz3nnifj1blhg9q626xlmkqq1nmh-gcc-10.3.0-lib/lib/libstdc++.so.6.0.28

This means that torch when imported after scipy.spatial will get a difference libstdc++ than it was compiled with. I haven't tested everything in torch, but I can guarantee you there are bugs when this occurs. I suspect this probably happens in many places with code that is compiled with both stdenv and gfortran.

Can this be re-evaluated and un-drafted? It seems like a smart choice.

Ah yes, forgot about this for a minute. As I recall there are actually quite a few packages broken by this, but fixing them is usually as simple as adding -std=legacy or -fallow-argument-mismatch. (I'm not sure which one is more correct to use, the first seems more self-documenting and may be more correct going forward, but is potentially broader in scope.) Let me rebase what I have locally and see where we're at.

Some good news, some of the packages I previously needed to work around such as scalapack and elmerfem have been fixed upstream in the meantime.

Successfully ran:
nix build -j8 -f . arpack python3.pkgs.{numpy,scipy} openblas suitesparse sundials liblapack fftw{,Float} mpich cholmod-extra cernlib magma

Not an exhaustive list, but a good sample of direct dependers.

@GrahamcOfBorg build mpich mvapich

@tpwrules thanks for reporting, I've made mpich only build with -fallow-argument-mismatch for gfortran 10+. Latest mpich does seem to solve the issue but I'd rather keep this PR narrow in focus.

See https://github.com/NixOS/nixpkgs/pull/168772 for mpich update.

For some reason this broke cross-compilation with gfortran.
I'm building gfortran with this:
nix build .#pkgsCross.aarch64-multiplatform.buildPackages.gfortran
When I try to execute it, it says this:
./result/bin/aarch64-unknown-linux-gnu-gfortran: line 210: /nix/store/y8yb50h4vba4bngybkd64sgm5kab8sn3-gfortran-11.2.0/bin/aarch64-unknown-linux-gnu-gfortran: No such file or directory

When I look into the  /nix/store/y8yb50h4vba4bngybkd64sgm5kab8sn3-gfortran-11.2.0/bin/ directory, I see this:
total 36852
lrwxrwxrwx 1 root root        3 Jan  1  1970 c++ -> g++
-r-xr-xr-x 1 root root  1226088 Jan  1  1970 cpp
-r-xr-xr-x 1 root root  1230184 Jan  1  1970 g++
-r-xr-xr-x 1 root root  1226088 Jan  1  1970 gcc
-r-xr-xr-x 1 root root    34792 Jan  1  1970 gcc-ar
-r-xr-xr-x 1 root root    34792 Jan  1  1970 gcc-nm
-r-xr-xr-x 1 root root    34792 Jan  1  1970 gcc-ranlib
-r-xr-xr-x 1 root root   715472 Jan  1  1970 gcov
-r-xr-xr-x 1 root root   563912 Jan  1  1970 gcov-dump
-r-xr-xr-x 1 root root   584392 Jan  1  1970 gcov-tool
-r-xr-xr-x 1 root root  1230184 Jan  1  1970 gfortran
-r-xr-xr-x 1 root root 29489536 Jan  1  1970 lto-dump
lrwxrwxrwx 1 root root        3 Jan  1  1970 x86_64-unknown-linux-gnu-c++ -> g++
lrwxrwxrwx 1 root root        3 Jan  1  1970 x86_64-unknown-linux-gnu-g++ -> g++
lrwxrwxrwx 1 root root        3 Jan  1  1970 x86_64-unknown-linux-gnu-gcc -> gcc
lrwxrwxrwx 1 root root        3 Jan  1  1970 x86_64-unknown-linux-gnu-gcc-11.2.0 -> gcc
-r-xr-xr-x 1 root root    34792 Jan  1  1970 x86_64-unknown-linux-gnu-gcc-ar
-r-xr-xr-x 1 root root    34792 Jan  1  1970 x86_64-unknown-linux-gnu-gcc-nm
-r-xr-xr-x 1 root root    34792 Jan  1  1970 x86_64-unknown-linux-gnu-gcc-ranlib
-r-xr-xr-x 1 root root  1230184 Jan  1  1970 x86_64-unknown-linux-gnu-gfortran

I didn't expect the cross-compiler for gfortran to refer to the native/x86_64 compiler.
Maybe related to https://github.com/NixOS/nixpkgs/issues/132651 as well?

Ah thank you for pointing that out, I see what you mean. I think it may be a simple host/target offset issue. Instead of wrapCC (gccStdenv, I think I should have used wrapCC (targetPackages.gccStdenv. Trying that out to see if it fixes it.

Will check that, let me rebuild gfortran and check if fftw now cross-compiles again.

Oh I see. How about instead of wrapCC (gccStdenv.cc.cc, simply wrapCC (gcc.cc? That seems like the most straightforward way to do it.

Sounds fine to me. Haven't tested but assuming that'll work. The splicing stuff is kind of leaky at times.


[WIP] Fixes related to types-publisher now supporting multiple minor versions in parallel

This is barely a work-in-progress, tbh, but mostly wanted to make you aware of the types of issues that exist right now in the failing CI builds (e.g. this one) due to these changes.

I agree with @sandersn to take a more thorough look after the weekend. Based on some quick research it seems like it may be a slightly larger change than I anticipated, but that’s mainly due to lacking context on this codebase.

Huh, my intention was to lock the version of types-publisher I use to prevent something like this from happening. I’ll help take a look next week. In the meantime, I’ll turn the PR benchmarking off. Thanks!

@alloy, assuming you’re not about to wrap this up, I’m planning on taking a look today (and however many more days it takes 😬)

Ohh 😮 My bad, this fell off my radar. Let me know if you get anywhere; if it turns out tough I can take a look tomorrow too.

I think I’m going to use this as an opportunity to clean things up a bit and stop depending on types-publisher internals. I’m currently working on extracting the repo-reading-and-metadata-gathering bits from types-publisher into its own package. As such, my progress will be kind of slow, but may have some questions for you when I get into the substance of your changes.
By the way, I noticed that react-native on DT doesn’t seem to be using the functionality you added yet (unless I’m totally missing something). Do you know of any packages that are?

Alas not. This was preparation work for when RN 0.62 gets released at which time we’ll want to be able to further improve 0.61’s typings separately.

👋 FYI, I fixed this as I was moving it into https://github.com/microsoft/DefinitelyTyped-tools. Preparing to archive this repo now. Thanks @alloy!


Add hardware info about BADRING water leakage sensor (E2202)

Technically not a teardown but this folder seems to be the most appropriate place for it. The pinout was identified with a multimeter, some of them were verified with some custom code I was testing.

Thank you.


[Proposal] @fastlane-bot V2

Introduction
A bot on fastlane repository allows maintainers to automate and improve their workflow. It helps to manage the project effectively and focus on what is really important by getting rid of tedious tasks.
Recently a bunch of improvements have been requested:

@fastlane-bot could also update linked issues when a PR is released
@fastlane-bot should link to "contribution guides" in stale issues comment
@fastlane-bot should not handle issues that have status: has-pr label
Fastlane Bot: Do not ask for "fastlane env" in "feature request" issues

The recent activity on the issue-bot project (the project has been started 3 years ago) made us thinking that it may be a good time to improve the bot.
This is a proposal about a bot revamp aka @fastlane-bot V2.
What does the bot do?
The @fastlane-bot does the following tasks:

Asking about fastlane env information
~Adding code signing issue suggestion~ (removed in https://github.com/fastlane/issue-bot/pull/68)
Marking issues as stale
Closing inactive issues
Locking old issues / PRs
Reminding about PRs that waiting for a review by adding a needs-attention label
Saying thank you for a contribution
Informing about including the changes from a PR in a release
Reminding about unpinning closed issues

Reasons for switching to GitHub Apps
According to the documentation:

Apps on GitHub allow you to automate and improve your workflow. You can build apps to improve your workflow.
...
GitHub Apps are the officially recommended way to integrate with GitHub...

Below there are the benefits of a migration to GitHub Application:
Security improvements

Fine-grained permissions
Short-lived tokens

Code base improvements

Probot

A dedicated framework for building GitHub Apps in Node.js

GitHub GraphQL API v4

It offers the ability to define precisely the data we want and replacing multiple REST requests wit a single call.

Testing

issue-bot project has 0 tests. It relays on manual testing. A @fastlane-bot V2 should be covered by tests which is very simple to do with Probot framework.

Pagination

According to the issue-bot codebase:

Doing pagination ourself is a pain, but it's important for keeping a reasonable memory footprint

Probot framework handles a pagination for us.

Typescript

Safer and predictable code by providing a strict syntactical superset of JavaScript.

Clear and centralised configuration file

Reaction improvements

Built-in, centralised web hooks

By using web hooks, reactiveness of a bot would improve significantly.
Others

Bot accounts
API rate limits for bot accounts
Documentation
Easier contribution
Bot tasks clean-up (re-thinking behaviour of the bot and get rid of the tasks we do not need anymore)

Reasons for NOT switching to GitHub Apps

Regression risk
Maintaining two bot accounts for time of a migration
If we do not expect working intensively on a bot project in the future, the migration may not be worth an effort and time
Some improvements like an icon and messages redesign can be done without a codebase migration

GitHub Actions
GitHub introduced GitHub Actions lately and even though it is still in public beta, it looks very promising. This proposal will not describe how GitHub Actions work.
Switching to GitHub Actions seems to be easier than building GitHub Application due to well separated and easy to test action's code. Here you can find a sample first-interaction action from GitHub.
Basically GitHub actions bring all the benefits that GitHub Applications do plus a few more:

Regression risk is minimal due to well tested code and small, single job focused code (action)
No need for running a dedicated server for a bot application
There are plenty of actions ready to be used in production, i.e. stale
Switching the mindset from fastlane bot code to reusable and useful for others (thanks @janpio for bringing this idea 🙇 )
Potential future migration from CircleCI to GitHub workflows (GitHub provides runners for free for open source projects!). Although this point is out of scope for this proposal.

Summary
Even though @fastlane-bot V2 looks like quite a big project to work on, it could provide a bunch of improvements over the existing solution and make the whole effort worth doing.
Feel free to share your thoughts, thank you! 🤖

This proposal should be updated included a recent announcement from GitHub:
GitHub Actions, now with CI/CD.

The proposal has been updated:

added GitHub Actions section

We decided to migrate a bot to GitHub Actions. We created a repository, where we will be keeping all the actions: https://github.com/fastlane/github-actions.
I am closing the proposal 🙇


window size issue

the window is small asf and there is no options to make the window bigger
i can't even see text :(

Would require extensive rewrite, and I am unsure as to when it will be done.

I found a cheaper solution, it has been implemented in an inelegant way.
See Commit #159.


[docs] Demo DraggableDialog crashes

[ ] I have searched for similar issues in this repository and believe that this is not a duplicate.

Steps to reproduce

Visit https://mui.com/material-ui/react-dialog/
??
demo DraggableDialog crashes

Your environment

Tech
Version

Version
v6.2.0

Netlify deploy
https://6758f30d6b946400086963e3--material-ui-docs.netlify.app

Browser
Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36

Search keywords:

Duplicate of https://github.com/mui/material-ui/issues/44732


[webview_flutter_android] Adds support for use of old hybrid composition

Adds support for Hybrid Composition to deal with: https://github.com/flutter/flutter/issues/104686
Also fixes: https://github.com/flutter/flutter/issues/102284
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the relevant style guides and ran the auto-formatter. (Unlike the flutter/flutter repo, the flutter/plugins repo does use dart format.)
[x] I signed the CLA.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy, or this PR is exempt from version changes.
[x] I updated CHANGELOG.md to add a description of the change, following repository CHANGELOG style.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@stuartmorgan This is ready for another review.
I switched the implementation to use the expensive android view when the background color is set as transparent. Since background color is a creation param, changes to it are ignored after the webview is already built.

@stuartmorgan I reran the test and it passed. I believe it was just a timeout. If you believe this is good, could you approve the PR?


tslint is deprecated

Greetings, and thanks for the excellent library.
I'm using @protobuf-ts/plugin for the first time and I'm noticing that it creates the following comment at the top of the file:

Some things:
1) Comment Stacking
Is it a bug that there are commas instead of newlines between the comments? Naively, I would expect it to look like:
// comment 1
// comment 2
// comment 3

Instead of:
// comment 1,// comment 2,// comment 3

2) TSLint is Deprecated
TSLint has been deprecated for over 3 years, so the // tslint:disable should be replaced with /* eslint-disable */.

Oh, I'm noticing now that the "comment stacking" problem seems to be fixed by #289.

Oh yes, that was a regression, sorry about that.
Agree on TSLint - this is going to be replaced by eslint in the next major release. In the meantime, you can set the plugin option.


fixing syntax error

When running the async clients I am getting this error:
gridbot_celery  |     from binance import Client
gridbot_celery  |   File "/usr/local/lib/python3.10/site-packages/binance/init.py", line 9, in
gridbot_celery  |     from binance.client import Client, AsyncClient  # noqa
gridbot_celery  |   File "/usr/local/lib/python3.10/site-packages/binance/client.py", line 2699
gridbot_celery  |     if self.tld == 'us'
gridbot_celery  |                        ^
gridbot_celery  | SyntaxError: expected ':'
I've added the colon in the relevant line

@gabriele16 It was fixed in the meantime, thanks for reporting it!


📝 Update includes in docs/en/docs/tutorial/static-files.md

#12554

Thank you! @lucaromagnoli :nerd_face: