[index.html] Adding section animations using aos.js library

Overview
This pull request introduces AOS (Animate On Scroll) animations to the About page, utilizing the AOS library.
Implementation

Library Usage: AOS library has been integrated to achieve the animations.
Future Considerations: If the implementation meets your expectations, I can extend this functionality to other pages as well.

Alternatives
If there are concerns about using external libraries, I am happy to implement similar animations using pure CSS and JavaScript. Just let me know your preference!
Demo
https://github.com/user-attachments/assets/44997213-9220-44f3-9684-deea8cd25629

Hi @raysan5, thank you for updating the branch and improving the pull request title! The AOS animations are working fine on my end. Is there anything else I need to do, or should I wait for the pull request to be merged? Let me know. Thanks!

@amine-a11 thanks, I'm just considering the usefulness of this addition compared for the additional code complexity, maintenance cost and external dependency cost. I need to think about it.


Adding property selectedTextStyle

Adding a property to define the text style to use when the button is in its selected state.

Hello,
Thanks for continuing the older repo.
With the many configuration options this component is great to customise!
While using this library, I was missing an option that I need: defining the text style for the selected button. So added this option to the library.

This Pull Request provides the option selectedTextStyle to the SegmentedButton:

The available typeface options are the same as the ones for the app:textStyle
By default, the selected text style is the text style as defined by the app:textStyle attribute.
Also added a getter and setter for this, since all properties provide it like this
Added the property to the README

Hello, thanks for the contribution! It all looks good to me.
Do you mind adding an example demonstrating this capability? If you're comfortable with it, you can even include the exact setup you're using in code just to see some real-world usages.
If not, I can merge without an example.

Thanks! As for the example, I've added it to the showcase of the Draggable Segmented Group in the sample app. The selected button of that row state gets the bold text on that set of buttons.

Okay, works for me. I missed that the first time reading through the code.
Thanks for the contribution and sorry for the delay in merging it.

No problem, thanks for merging and releasing it! 👍


Add and Update abs() function.

Added and Updated definition of abs() function for complex number with geometrical explanation.Provided sample code for the implementation of abs function in python.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

@saintpopo Thanks for this pull request (PR).
Unfortunately, we are marking this PR invalid for hacktoberfest. We believe it does not follow the recommended guidelines:

Quality Standards
In line with Hacktoberfest value #2 (Quantity is fun, Quality is key), we have provided examples of the quality >standards we encourage. This applies mainly to beginners.

PRs that are automated e.g. scripted opening PRs to remove whitespace / optimize images.
PRs that are disruptive e.g. taking someone else's branch/commits and making a PR.
PRs that are regarded by a project maintainer as a hindrance vs. helping.
Something that's clearly an attempt to simply +1 your PR count for October.

Learn more at https://hacktoberfest.digitalocean.com/details


start.js & app.js = Internal Server Error in production b/c Response header content-type is "text/html; charset=UTF-8"

Describe the bug

Locally for app.js & start.js the response header Content-Type is application/javascript
In production the content type for these 2 files are text/html; charset=UTF-8
No error in cloudflare wokers terminal but browser errors say

Loading module from “https://sensethenlove.com/_app/immutable/entry/start.59d6b0b2.js” was blocked because of a disallowed MIME type (“text/html”).
Loading module from “https://sensethenlove.com/_app/immutable/entry/app.646327ce.js” was blocked because of a disallowed MIME type (“text/html”).
The browser console errors also include a link to https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/X-Content-Type-Options
Example: https://sensethenlove.com/ + https://github.com/sensethenlove/sensethenlove
Reproduction
Deploy to cloudflare workers w/ most recent svelte packages
Logs
Loading module from “https://sensethenlove.com/_app/immutable/entry/start.59d6b0b2.js” was blocked because of a disallowed MIME type (“text/html”).
Loading module from “https://sensethenlove.com/_app/immutable/entry/app.646327ce.js” was blocked because of a disallowed MIME type (“text/html”).

System Info
System:
    OS: macOS 12.6
    CPU: (12) x64 Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
    Memory: 95.89 MB / 16.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 18.14.1 - ~/.nvm/versions/node/v18.14.1/bin/node
    npm: 9.3.1 - ~/.nvm/versions/node/v18.14.1/bin/npm
  Browsers:
    Chrome: 110.0.5481.177
    Firefox: 110.0
    Safari: 15.6.1
  npmPackages:
    @sveltejs/adapter-cloudflare-workers: ^1.0.6 => 1.0.6
    @sveltejs/kit: ^1.8.8 => 1.8.8
    svelte: ^3.55.1 => 3.55.1
    vite: ^4.1.4 => 4.1.4

Severity
blocking all usage of SvelteKit
Additional Information
No response

Looking at that demo site, the real issue seems to be that these requests are returning 500 errors (which is rendered as an HTML page), not that the correct file is being returned with the wrong MIME type.
The error page instructs the owner of the site to log into the Cloudflare console and check its logs. Do you see anything helpful in there?

@Conduitry Now the requests are returning as 200's! Thank you for helping me comprehend the content-type, idk what changed, glitch in the matrix, thank you to all @ Svelte!


Unicode Keyboard not working on TextFormField Windows, but i can copy same value and paste them into TextFormField

Platform: flutter windows
Target:
Unicode Keyboard not working, but i can copy same value and paste them . See video...
Another platform like: web or android, it work without error
AlertDialog alert = AlertDialog(
        title: Text(S.current.change_defaut_open_bot),
        content: SingleChildScrollView(
          child: ListBody(children: <Widget>[
            TextFormField( // <-- TextBox error
                onChanged: (value) => {newName = value},
                obscureText: false,
                initialValue: app.auto_redirect_hash,
                decoration: InputDecoration(
                  labelText: S.current.change_defaut_open_bot,
                )),
          ]),
        ),
        actions: [
          cancelButton,
          continueButton,
        ]);

Doctor:
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 2.8.1, on Microsoft Windows [Version 10.0.22000.376], locale en-US)
[!] Android toolchain - develop for Android devices (Android SDK version 32.0.0)
    X cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    X Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/windows#android-setup for more details.
[√] Chrome - develop for the web
[√] Visual Studio - develop for Windows (Visual Studio Professional 2019 16.11.8)
[√] Android Studio (version 2020.3)
[√] IntelliJ IDEA Ultimate Edition (version 2021.1)
[√] VS Code (version 1.63.2)
[√] Connected device (4 available)

https://user-images.githubusercontent.com/1293057/149307819-aabb4db5-9940-4105-964d-7c5c670aad7b.mp4

HI @nghuyy, Thanks for filing the issue. Could you please share, How can I enter Unicode characters on windows 10?

I'm using windows 11 with Vietnamese touch keyboard... You can type some special character like "đ" or "ê" ...

HI @nghuyy, Thanks for filing the issue. Could you please share, How can I enter Unicode characters on windows 10?

Another platform like: web or android, it work without error

Also can you please share what error are you seeing when running on windows Desktop?

I'm using 3rd keyboard for Windows  like:
https://www.unikey.org/
or
https://www.trankynam.com/gotv/
With Vietnamese language typing type Telex
But build in Windows Keyboard work.

@nghuyy This looks like a duplicate of #82673, Can you please confirm?


Improve the site UI Using color theory

The page colors are not visually appealing and don't meet the accessibility goals for colourblind individuals.
I would use color theory using palettes from a site which shows how the site looks like for people with different vision. There should be better contrast between the font and background colors.
Kindly assign this issue to me so that I can start working on it.

hey @Thi-MindSpace
i would like to contribute on this issue 'Improving color theory of the web page'
the reason i choose this issue, recently i have did some good looking web pages with attractive color and ui
I'll do my best improving the color theory of the page
this is a web app that aims to improve access to mental health, and to achieve that we need a pleasant look to the site
I hope you will assign thsi job to me

hey @The-MindSpace
i would like to contribute on this issue 'Improving color theory of the web page'
the reason i choose this issue, recently i have did some good looking web pages with attractive color and ui
I'll do my best improving the color theory of the page
this is a web app that aims to improve access to mental health, and to achieve that we need a pleasant look to the site
I hope you will assign thsi job to me

Hi, I would like to work on this issue, please can you assign this to me. I am a UI UX designer as well as a frontend developer


Cherry pick onError:fail rename and release signing metadata in to 0.27.x branch

Changes
This PR brings in two commits for the release of 0.27.1:

Change "fail" to "stopAndFail"
Add metadata to our publish task for Tekton Chains to observe and sign

/kind misc
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Release notes block below has been filled in or deleted (only if no user facing changes)

Release Notes
NONE

/kind misc

@sbwsg I am wondering if introducing signed releases in a patch would cause any conflict with a .0 (point) release? 🤔  As in, when the release is applies using:
kubectl apply -f https://storage.googleapis.com/tekton-releases/pipeline/previous/v0.27.1/release.yaml

vs
kubectl apply -f https://storage.googleapis.com/tekton-releases/pipeline/previous/v0.27.0/release.yaml

I understand we are just singing the images but still wanted to bring it up ...

@sbwsg I am wondering if introducing signed releases in a patch would cause any conflict with a .0 (point) release? 🤔

I don't think the signing process modifies the deployed resources in any way.  It allows users to verify the images that they're installing with the Pipelines YAML but doesn't require that YAML to be any different to that of previous releases.

I can test this, though, once I've made the release. If something is terribly wrong then I can immediately roll back the publish yaml in the cluster and perform another release. wdyt?

I was more concerned about the image these files are pointing to:
 gcr.io/tekton-releases/github.com/tektoncd/pipeline/cmd/controller:v0.27.0@sha256:015e8f9cf38416652545ce5e16776379a979af64373250ec489861886a468fcf

Not signed vs Signed?  0.27.1 will generate a new image which will be signed where 0.27.0 is pointing to an image which is not. How do the users consume the new releases? There is a very small window between a point release and a patch in this case so should be fine.
/approve

Not signed vs Signed? 0.27.1 will generate a new image which will be signed where 0.27.0 is pointing to an image which is not. How do the users consume the new releases?

my understanding is that users consuming this signed release won't actually notice any difference, unless they want to start verifying the signature (which is something i think in the long run we'll want to start recommending and/or requiring as part of installation - maybe the operator can enforce it?)
but maybe you're imagining a different scenario @pritidesai ? in the meantime it looks like you're okay with going ahead with this so adding my sign off as well:
/lgtm


It seems DECIMAL_PLACES is no effect

I find Its description in settings.py only.
And one question: how I can display unit_price without zero after comma in template. For example:
{{ product.unit_price }}

I wan't see $ 12 (not $ 12.00).
Now I use own custom templatetags for fix it.

This is hard coded in shop.money.iso4217.py for each currency according to it's decimal places.
Hmm, didn't think about the use-case for selling real estate :/

In our country it seldom used dime (or cents for USA) in web-shops. Also I'd like to separate big values by space. For example, $ 25050 -> $ 25 000. And I have my work around for it too. I'd like use shop's functions for it.

Ok, then that mapping should be configurable

It would be good! Say, SHOP_DECIMAL_SEPARATED_PLACES = True by default.

Isn't it better to just write a template filter?

Yes, now I use my custom filter:
@register.filter
def price_format(value, use_currency_title=True):
    if not value:
        return value

    if type(value) == unicode:
        value = Decimal(value.replace(',', '.'))

    if settings.SHOP_DECIMAL_PLACES == 0: precision = '1'
    else: precision = '0.{}1'.format('0' * (settings.SHOP_DECIMAL_PLACES - 1))

    price = separate_by_space(value.quantize(Decimal(precision)))

    if not use_currency_title:
        return price

    return settings.SHOP_MONEY_FORMAT.format(currency=settings.SHOP_DEFAULT_CURRENCY_TITLE, amount=price)

@register.filter
def separate_by_space(value):
    result = ''
    parts = unicode(value).split()
    for part in parts:
        try:
            n = decimal.Decimal(part)
            result += '{0:,} '.format(n).replace(',', ' ')
        except decimal.InvalidOperation:
            result += '{} '.format(part)
    return result.replace('  ', ' ').strip()

Maybe Django Shop is needed like such price_format filter?

@vivazzi
I think the best place to look how money is rendered is here. shop.money.money_maker.AbstractMoney.__format__.
USE_THOUSAND_SEPARATOR = True makes things even worse, because it alway adds a dot, so that an amount of 987 would be rendered as € .987,00.
So this issue describes two separate problems. On one side you want to configure the decimal places yourself. Would it make sense for you, if I add a hook where you can use your own version of money.currencies.CURRENCIES?
The second is what to use as the thousands separator. Could you please describe your use cases.

I need think about it..
Where I can see USE_THOUSAND_SEPARATOR implementation? In Django Shop I don't found.

On one side you want to configure the decimal places yourself. Would it make sense for you, if I add a hook where you can use your own version of money.currencies.CURRENCIES?

Can we bind SHOP_DECIMAL_PLACES with money.currencies.CURRENCIES? Other words, can we add SHOP_DECIMAL_PLACES to money.currencies.CURRENCIES?
For example:
# in settings.py:
DECIMAL_PLACES = getattr(settings, 'SHOP_DECIMAL_PLACES', None)

# in money.currencies.CURRENCIES
'RUB': ('643', DECIMAL_PLACES or 2, '₽', _("Russian Ruble")),

Decimal places are a per currency configuration. There are currencies with 0 DP: Japanese Yen, Hungarian Forint; some with 2 DP: Pound, Euro, Dollar and even some with 3 DP: Omani rial, Kuwait dinar. Therefore I don't like the idea of a global decimal places setting.
Therefore I will add a configuration directive which can be used to replace the whole iso4217.CURRENCIES table.

Maybe your variant is better because our products can be in different currencies. You are right

Where I can see USE_THOUSAND_SEPARATOR implementation?

This is part of Django itself, but currently it is not supported.


Fix length of Product Group Code in Shopify Connector

The length of the field Product Group Code in the table  "Shpfy Shop Collection Map" was 10. Which gave the error with longer VAT Product Posting Groups.
It was possible to add the VAT Product Posting Group more than 10 chars but there was an issue when renaming the value.

I checked where the field is used and looks like there are no references in Shopify Connector that would break anything else.

If it would be an public table I'd say this was a breaking change, but since the table is set to internal this should be possible to change.

I'll try to process this internally and see what our build system thinks about this 😉


2024-2.0: move some dependencies to the pip section

This should resolve the building problems.

❯ diff envs/env-py310.yml envs/env-py311.yml
1c1
< name: 2024-2.0-py310-tiled
---
> name: 2024-2.0-py311-tiled
10c10
<   - python >=3.10,<3.11
---
>   - python >=3.11,<3.12


Adds footer pages, top level sections and site settings logo

Added footer pages in base
Added top level sections in base
Used site settings logo in header

@dannyparsons kindly check and let me know


craco changes the output content of the package file to package the original configuration file and own custom folder

What's happening
Hello, When I use craco to change the package file name and static resource directory,The packaging generates not only the original default build file, but also my own custom folder，The default build file is a file directory after deployment, and dist is a normal file
What should happen
I want to package to generate only custom folders and directories
To reproduce
npm i @craco/craco -D
npm run build
CRACO version
(ex. 7.1.0)
CRACO config
const path = require('path');
const addPath = dir => path.join(__dirname, dir);
const { whenProd } = require('@craco/craco')

const webpackConfig = {
    webpack: {
        alias: {
            "@": addPath("src"),
        },
        configure: (webpackConfig, { env, paths }) => {
            path.appBuild = 'dist';
            webpackConfig.output = {
                ...webpackConfig.output,
                path: path.resolve(__dirname, 'dist'),
                publicPath: '/',
            }
            webpackConfig.output.path = path.resolve(__dirname, 'dist');
            return webpackConfig;
        }
    },
    babel: {
        plugins: [
            // ['import', { libraryName: 'antd', style: true}],
        ]
    }
}

module.exports = webpackConfig;

package.json
"dependencies": {
    "@testing-library/jest-dom": "^5.16.5",
    "@testing-library/react": "^13.4.0",
    "@testing-library/user-event": "^13.5.0",
    "@types/jest": "^27.5.2",
    "@types/node": "^16.18.14",
    "@types/nprogress": "^0.2.0",
    "@types/react": "^18.0.28",
    "@types/react-dom": "^18.0.11",
    "@types/react-router-dom": "^5.3.3",
    "antd": "^5.3.1",
    "axios": "^1.3.4",
    "mobx": "^6.8.0",
    "mobx-react": "^7.6.0",
    "nprogress": "^0.2.0",
    "react": "^18.2.0",
    "react-dom": "^18.2.0",
    "react-router-dom": "^6.8.2",
    "react-scripts": "5.0.1",
    "typescript": "^4.9.5",
 },
"scripts": {
    "start": "craco start",
    "build": "craco build",
    "test": "craco test",
    "eject": "react-scripts eject"
 },
"devDependencies": {
    "@babel/plugin-proposal-decorators": "^7.21.0",
    "@craco/craco": "^7.1.0",
    "babel-plugin-import": "^1.13.6"
 }

Additional information
I think it may be a problem between react- Script 5.x and Craco 7.x

If you want to only generate only your own custom folder without generating /build folder as default, then you only need to adjust this line of code : paths.appBuild = webpackConfig.output.path;
Details as below:
const webpackConfig = {
    webpack: {
        alias: {
            "@": addPath("src"),
        },
        configure: (webpackConfig, { env, paths }) => {
            webpackConfig.output = {
                ...webpackConfig.output,
                path: path.resolve(__dirname, './dist/'),
                publicPath: '/',
            }
            paths.appBuild = webpackConfig.output.path;
            return webpackConfig;
        }
    },
    babel: {
        plugins: [
            // ['import', { libraryName: 'antd', style: true}],
        ]
    }
}

module.exports = webpackConfig;


Create a pattern for having a function be aware of extents

example:
grid needs to know what the ticks on axis are so it can render pretty.
adding text relative to the extents, which are changing
And we want this to fit w/ the builder pattern, so when the axis limits change, this changes

might use: grconvertX, grconvertY


MsTest DataRow support

(This is a redo of PR #52 by @livioc, to be able to fix the generation issues.)
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue).
[x] New feature (non-breaking change which adds functionality).
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected).
[ ] Performance improvement
[ ] Refactoring (so no functional change)
[ ] Other (docs, build config, etc)

Checklist:

[x] I've added tests for my code. (most of the time mandatory)
[x] I have added an entry to the changelog. (mandatory)
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

@livioc Reqnroll adds an additional string[] parameter to the method to know about the tags (regardless of whether the test framework supports it) and that had to be provided.
The rest of the issues seem to be related to how we do the testing: we let the tests run and then parse the TRX file to get the results. But MsTest generates the TRX file in a slightly different format in case of DataRow, so that causes the problems. I have a working env now, I can try to debug the rest.

Oh God. This was a tricky issue. Somehow the code we generated was not working in case there was only one column in the examples table with MsTest v2.*...
Locally it is passing now. Let's hope it works.
But I had to learn that the local testability is really a pain, especially for generator changes. Need to improve this (https://github.com/orgs/reqnroll/discussions/38).

Hi @gasparnagy , today I have done some tests on a new .net48 test project but I still have problem.
I created the nuget packages with .\build.ps1 -Configuration Release   on https://github.com/reqnroll/Reqnroll/commit/e999ecfd87517a4d91c841ade7571fb612f4fb8e.
Then I generated the test code using <PackageReference Include="Reqnroll.MsTest" Version="1.0.2-local" />  with this gherkin
 Scenario Outline: Simple scenario
		Given there is <count> <color> items

		Examples:
			| count | color |
			|     0 | blue  |
			|     0 | black |

When I run the generated test code
        [Microsoft.VisualStudio.TestTools.UnitTesting.TestMethodAttribute()]
        [Microsoft.VisualStudio.TestTools.UnitTesting.DescriptionAttribute("Simple scenario")]
        [Microsoft.VisualStudio.TestTools.UnitTesting.TestPropertyAttribute("FeatureTitle", "Calculator")]
        [Microsoft.VisualStudio.TestTools.UnitTesting.DataRowAttribute("0", "blue")]
        [Microsoft.VisualStudio.TestTools.UnitTesting.DataRowAttribute("0", "black")]
        public async System.Threading.Tasks.Task SimpleScenario(string count, string color, string[] exampleTags)
        {
            string[] tagsOfScenario = exampleTags;
            System.Collections.Specialized.OrderedDictionary argumentsOfScenario = new System.Collections.Specialized.OrderedDictionary();
            argumentsOfScenario.Add("count", count);
            argumentsOfScenario.Add("color", color);
            Reqnroll.ScenarioInfo scenarioInfo = new Reqnroll.ScenarioInfo("Simple scenario", null, tagsOfScenario, argumentsOfScenario, featureTags);
#line 15
this.ScenarioInitialize(scenarioInfo);
#line hidden
            if ((TagHelper.ContainsIgnoreTag(tagsOfScenario) || TagHelper.ContainsIgnoreTag(featureTags)))
            {
                testRunner.SkipScenario();
            }
            else
            {
                await this.ScenarioStartAsync();
#line 16
  await testRunner.GivenAsync(string.Format("there is {0} {1} items", count, color), ((string)(null)), ((Reqnroll.Table)(null)), "Given ");
#line hidden
            }
            await this.ScenarioCleanupAsync();
        }

I got the exception
 SimpleScenario (0,black)
   Source: Calculator.feature line 15
   Duration: 71 ms

  Message:
Test method ReqnrollProject1.Features.CalculatorFeature.SimpleScenario threw exception:
System.Reflection.TargetParameterCountException: Parameter count mismatch.
TestCleanup method ReqnrollProject1.Features.CalculatorFeature.TestTearDownAsync threw exception. System.NullReferenceException: Object reference not set to an instance of an object..

  Stack Trace:
RuntimeMethodInfo.InvokeArgumentsCheck(Object obj, BindingFlags invokeAttr, Binder binder, Object[] parameters, CultureInfo culture)
RuntimeMethodInfo.Invoke(Object obj, BindingFlags invokeAttr, Binder binder, Object[] parameters, CultureInfo culture)
TestCleanup Stack Trace
<OnScenarioEndAsync>d__34.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
ExceptionDispatchInfo.Throw()
TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
TaskAwaiter.GetResult()
<OnScenarioEndAsync>d__17.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
ExceptionDispatchInfo.Throw()
TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
TaskAwaiter.GetResult()
<TestTearDownAsync>d__9.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
ExceptionDispatchInfo.Throw()
TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)

  Standard Output:
-> Loading plugin C:\dev\ExtGitHub\Reqnroll-PersonalTest\ReqnrollProject1\bin\Debug\net48\Reqnroll.MSTest.ReqnrollPlugin.dll
-> Loading plugin C:\dev\ExtGitHub\Reqnroll-PersonalTest\ReqnrollProject1\bin\Debug\net48\ReqnrollProject1.dll
-> Using default config
-> warning: The previous ScenarioContext was already disposed.

TestContext Messages:
-> warning: The previous ScenarioContext was already disposed.

Are you sure that we must pass the addition tags array parameter? Can't we just ignore them?

@livioc The current test system is not working properly. Actually you are not testing with the version you have built (you can see it because the [Microsoft.VisualStudio.TestTools.UnitTesting.DataRowAttribute("0", "black")] has only two parameters, but it should have 3.
The problem is that the package has been cached to your C:\Users\<your user>\.nuget\packages folder and therefore it does not matter if you compile it locally, you will still test with the previous one (I work on a solution in #62).
What you can do is to manually delete all reqnroll related packages from C:\Users\<your user>\.nuget\packages and then try again.

@gasparnagy thanks, I have done same tests, this scenaraio doens't work with mstest

The generated code has a different number of parameters
        [Microsoft.VisualStudio.TestTools.UnitTesting.TestMethodAttribute()]
        [Microsoft.VisualStudio.TestTools.UnitTesting.DescriptionAttribute("Another scenario")]
        [Microsoft.VisualStudio.TestTools.UnitTesting.TestPropertyAttribute("FeatureTitle", "Calculator")]
        [Microsoft.VisualStudio.TestTools.UnitTesting.DataRowAttribute("1", "", new string[] {
                "mytag"})]
        [Microsoft.VisualStudio.TestTools.UnitTesting.DataRowAttribute("2", null)]
        public async System.Threading.Tasks.Task AnotherScenario(string count, string notUsed6248, string[] exampleTags)
        {
            string[] tagsOfScenario = exampleTags;
            System.Collections.Specialized.OrderedDictionary argumentsOfScenario = new System.Collections.Specialized.OrderedDictionary();
            argumentsOfScenario.Add("count", count);
            Reqnroll.ScenarioInfo scenarioInfo = new Reqnroll.ScenarioInfo("Another scenario", null, tagsOfScenario, argumentsOfScenario, featureTags);
#line 25
this.ScenarioInitialize(scenarioInfo);
#line hidden
            if ((TagHelper.ContainsIgnoreTag(tagsOfScenario) || TagHelper.ContainsIgnoreTag(featureTags)))
            {
                testRunner.SkipScenario();
            }
            else
            {
                await this.ScenarioStartAsync();
#line 26
 await testRunner.GivenAsync(string.Format("there is {0} blue items", count), ((string)(null)), ((Reqnroll.Table)(null)), "Given ");
#line hidden
            }
            await this.ScenarioCleanupAsync();
        }

but I don't understand this, this code works with a .net48 test project
      [Microsoft.VisualStudio.TestTools.UnitTesting.TestMethodAttribute()]
      [Microsoft.VisualStudio.TestTools.UnitTesting.DescriptionAttribute("Another scenario")]
      [Microsoft.VisualStudio.TestTools.UnitTesting.TestPropertyAttribute("FeatureTitle", "Calculator")]
      [Microsoft.VisualStudio.TestTools.UnitTesting.DataRowAttribute("1", new string[] {
                "mytag"})]
      [Microsoft.VisualStudio.TestTools.UnitTesting.DataRowAttribute("2", null)]
      public async System.Threading.Tasks.Task AnotherScenario(string count, string[] exampleTags)
      {
         string[] tagsOfScenario = exampleTags;
         System.Collections.Specialized.OrderedDictionary argumentsOfScenario = new System.Collections.Specialized.OrderedDictionary();
         argumentsOfScenario.Add("count", count);
         // ....
      }

@livioc ah yes, thx. this is a bug and I know why. I'll fix. (I have no idea why this was working on .net48)

@livioc I fixed it now. I could only do the related test in my new "System Test" concept in #62 (a214d47), but that will be merged at some point.
If you have a little more time, please retest it. You can also have a look at my system test and give feedback if such testing would be useful/efficient for generator changes.


Information not up to date

Hi,
the information on this page is not up to date.
I just downloaded Version <IP_ADDRESS> and installed it. The latest release information on this page is for a version that is one year old.
Would be good to know what have changed since then.
Thanks in advance.
Best regards
Werner

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

ID: 5a5efdb8-5e17-6502-8e51-c77f8a6aa59c
Version Independent ID: e11ed733-2e65-7000-87b0-4adfcb16db21
Content: Azure AD Connect Health Version History
Content Source: articles/active-directory/hybrid/reference-connect-health-version-history.md
Service: active-directory
Sub-service: hybrid
GitHub Login: @zhiweiwangmsft
Microsoft Alias: billmath

@WernerGIT  Thanks for the feedback. We will engage the content development team for further investigation, they will verify and perform necessary changes in the document as needed.
@billmath  Can you have a look at this ?

@WernerGIT
Thank for submitting this. I have created a work item for it and will reviewing this in the coming weeks.  We have issues keeping this updated in the past and I think we are going to remove this page and re-direct to the AADC version history page here:  https://docs.microsoft.com/en-us/azure/active-directory/hybrid/reference-connect-version-history
Going forward I would use this doc for reference.  I am not sure what the timeline will be, so I am going to close this for now. But once I have an update I will drop it in here and let you know.
Thank you!
Bill
#please-close


table 39 "Purchase Line" - make local procedure InitHeaderDefaults() global

Describe the request
Hi,
Is it possible to change the now local procedure InitHeaderDefaults in table 39 Purchase Line into a global one?
The procedure is already global on Sales Line, so I hope it won't be an issue.
There is the extra TempPurchLine parameter, which is not used anywhere inside the function apart from the publisher, which is also not called from any Microsoft app (as far as I could find), so maybe a public overload without it would be favorable, but I'll leave that up to you.
We just need to reinit the defaults.
Thank you!
Additional context
@@ -4366,7 +4366,7 @@ table 39 "Purchase Line"
     local procedure InitItemAppl()
     begin
         "Appl.-to Item Entry" := 0;
     end;

-    local procedure InitHeaderDefaults(PurchHeader: Record "Purchase Header"; var TempPurchLine: Record "Purchase Line" temporary)
+    procedure InitHeaderDefaults(PurchHeader: Record "Purchase Header"; var TempPurchLine: Record "Purchase Line" temporary)
     var
         IsHandled: Boolean;
     begin
         CheckBuyFromVendorNo(PurchHeader);

         "Buy-from Vendor No." := PurchHeader."Buy-from Vendor No.";

Internal work item: AB#503095

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 17192.


Feature request - in AudioRecorder.onProgress - get audio level

Version
react-native-audio :  3.2.2
react-native : 0.44.0
react :  16.0.0-alpha.6
react-native-sound : No
Expected behaviour :
In the AudioRecorder.onProgress I was hoping to get audio level. If not level at the least I was hoping to get "its quiet" or "its loud". If you can help us with this that would be soooo awesome! :)

Duplicate of https://github.com/jsierles/react-native-audio/issues/156


Added init.py files in docs for custom management commands

Management commands still work if the directories management and commands do not contain __init__.py files, but I find it a bit odd that the docs don't tell the user to create them.
According to https://docs.python.org/3/tutorial/modules.html#packages:

The init.py files are required to make Python treat directories containing the file as packages.

By not creating the __init__.py files, the user is creating implicit namespace packages according to https://www.python.org/dev/peps/pep-0420/. I think this is not what we intend, and that we should keep the user on the normal path here.

It was removed in https://github.com/django/django/pull/9511.

It was removed in #9511.

Thanks Tim. I should have researched that before filing the PR. As far as I see, Aymeric's comment wasn't really addressed back then. IMO, getting rid of two empty files is not worth the potential for confusion and  surprising behaviour (even if very rare).
I remember that at some point, I read somewhere that "you don't need __init__.py on Python 3", and I got into the habit of not creating __init__.py files in Python 3 projects. Then, I got hit by some surprising bug because of this (I forgot the details, unfortunately). I think creating a namespace package should be an intentional decision and not be done just because it's possible. Also, someone reading the docs might wonder why polls has __init__.py but the subpackages don't.  I don't know myself, to be honest :)

Perhaps a mailing discussion could reach a consensus. I can't speak definitivley on the matter.

https://groups.google.com/d/msg/django-developers/GVHMH2ciAnk/7RzvMIzyAQAJ

So it seems if we move forward with this we should revert ccc25bfe4f0964a00df3af6f91c2d9e20159a0c2 (with possibly other __init__.py additions if any management and/or management/commands command directories have since been added).

So it seems if we move forward with this we should revert ccc25bf (with possibly other __init__.py additions if any management and/or management/commands command directories have since been added).

Ah, I see. So the scope of the change is a bit bigger than I thought at first. I think it would probably be a good idea to add the __init__.py files again. On the other hand, maybe this has more priority in the user-visible docs than in Django-internal code.

@timgraham Looking at the ML discussion, I think we have reached consensus that ccc25bf should be reverted. Would you like to do this yourself? Or should I force-push this here?

Please use this branch.

Merged in 578b3046e3b4adc43655bb5dd955054b7bf19f89.


[RNDIS][Open source][share code]

hi  everyone:
I have a question.  Does it have RNDIS open source sample code?  I want to learn this RNDIS driver.  I find such link  https://github.com/microsoft/Windows-driver-samples/tree/main/network/RNDIS  does not exist.
So  can you share this sample code to git?

@microsoft/network-driver-platform

hi  v-junyli:
can you show the detailed link? I can't find it through the key word "network-driver-platform". Thanks

@gg040431204 hi sorry "microsoft/network-driver-platform" is the alias of the owner team of NDIS driver samples, I tagged them to see if they have better information for you about this request.

Hi @v-junyli :
thanks for your reply. hope to get answer ASAP.

hi  @Shyamal-V :
Can you share RNDIS sample code? Thanks.

Hi @Shyamal-V :
Can you share RNDIS sample code?Thanks.


[Admin] (typo) Fix

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

File
Status
Preview URL
Details

docs/develop/persisting-add-in-state-and-settings.md
:white_check_mark:Succeeded

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


⚠ ClamAV has degraded performance

In 70b80b0, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 90 ms

Resolved: ClamAV performance has improved in b171cf0.


refreshInterval doesn't work when refreshWhenOffline = false

Bug report
Description / Observed Behavior
When using chrome, when refreshWhenOffline: false it appears that refreshInterval doesn't do anything
I've tested in chrome and firefox, firefox works as intended
Expected Behavior
refreshInterval to work as documented
Repro Steps / Code Example
https://codesandbox.io/s/busy-oskar-fsxky?file=/src/App.js
Additional Context
SWR version: 0.4.2
Chrome version: 88.0.4324.182 (Official Build) (x86_64)
Add any other context about the problem here.
Firefox:

Chrome

Seems related:https://bugs.chromium.org/p/chromium/issues/detail?id=678075

It appears to be a bug with chrome and not swr directly


[ERROR] sudo: unrecognized option '--preserve-env=EDITOR'

I'm trying to run pikaur in Arch as a normal user, eg pikaur -S foo, which calls sudo internally. But this fails with doas-sudo-shim with the error:
sudo: unrecognized option '--preserve-env=EDITOR'
Usage:
  sudo (-i | -s) [-n] [-u <user>] [<command> [--] [<args>...]]
  sudo [-ins] [-u <user>] <command> [--] [<args>...]
  sudo [-h]

Execute a command as another user using doas(1).

This is not the original sudo, but the doas shim for sudo. It supports only
a subset of the sudo options (both short and long) that have an equivalent in
doas, plus option -i (--login). Refer to sudo(1) for more information.

Please report bugs at <https://github.com/jirutka/doas-sudo-shim/issues>.

Distro: Arch Linux
App: pikaur v1.20.r6.g1821e8d

doas (OpenDoas) doesn’t have any equivalent for this option, so there’s nothing we can do about it.

doas (OpenDoas) doesn’t have any equivalent for this option, so there’s nothing we can do about it.

What about using the env command? It's not pretty, but converting the call to doas env EDITOR=$EDITOR does get the job done.


RTX 2080 issue with Tensorflow

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 16.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: NaN
TensorFlow installed from (source or binary): Both
TensorFlow version (use command below): 1.12.0-rc0-devel-gpu-py3
Python version: 3.5.2
Bazel version (if compiling from source): Latest
GCC/Compiler version (if compiling from source): 5.x
CUDA/cuDNN version: 9.0 / 10.0 (Tried Both)    | cuDNN : 7.2.x / 7.3.1
GPU model and memory: RTX Geforce 2080
Exact command to reproduce:  Running the most basic mnist dataset using keras with Tf backend

When I try running the code it starts and ends with a segmentation fault even without start of single epoch. The command nvidia-smi shows maximum usage of its volatile memory. I also restricted the gpu usage by a fraction of 0.5 still the problem persists.  I tried building the source files with cuda 9.0 / cudnn 7.2.1 in the docker environment yet the installed whl package produces the same error.

@tejavoo, Just to let you know, since 2080 is rather new, cuda 9.0 doesn't know anything about it. So try with cuda 10 installation and driver.

@tejavoo Is this still an issue?

@ymodak No its been solved and the link to how it was solved is here in a medium article. Thanks to one and all

Possibly related?  I have a system set up using Keras, TensorFlow, CUDA 9.0 on Ubuntu 18.0.  When running on 2080 Ti, I either get a validation score that is constant 5.0, or if I tweak a bit, inf.  Running the exact same code on a different machine with P100 works correctly (but slower, dammit).  The setup on the second machine is different, though, so could be a software version problem.


Change -d to -id for consistency with -od

https://github.com/AMP-SCZ/dpanonymize/blob/master/scripts/dpanon.py#L63

changed
https://github.com/AMP-SCZ/dpanonymize/commit/9db15a26ef60658ee173340240e19bdfdeab353f


Azure DevOps Lab -- CI -- Exceeded Limit

Hello user Simon Dickenson is reporting an issue running Continuous Integration and Continuous Development DevOps Labs which require require Pipelines.  Receiving below error:

Hi @bmoore1954 ,
Microsoft has temporarily disabled the free grant of parallel jobs for public projects and for certain private projects in new organizations. However, you can request this grant by sending an email to<EMAIL_ADDRESS>Existing organizations and projects are not affected.
To request the free grant for public or private projects, send an email to<EMAIL_ADDRESS>and provide the following information:

Your name
The Azure DevOps organization for which you are seeking the free grant
Whether you need the free grant for public projects or private projects

For more information https://docs.microsoft.com/en-us/azure/devops/pipelines/licensing/concurrent-jobs?view=azure-devops&tabs=ms-hosted
Thanks


Looking for dropdown menu examples

I am not a coder but can cobble together pieces of code to generally do what I want. I am looking for example sites and/or code using pico and drop down menus. I need samples plus the location of the files I need to edit. I have top navigation with upwards of 10 sub pages I want shown in a drop down style. Any help/pointers are appreciated.

The best way to create a recursive page menu is Pico's page tree feature. You can add the following Twig snippet to your themes/my_theme/index.twig:
{% macro tree(parent) %}
    {% import _self as utils %}
    {% for child in parent.children %}

            {% if child.page %}

{{ child.page.title }}

            {% endif %}
            {% if child.children %}

                    {{ utils.tree(child) }}

            {% endif %}

    {% endfor %}
{% endmacro %}

{{ pages["index"].title }}

    {% import _self as utils %}
    {{ utils.tree(pages["index"].tree_node) }}


Update chunk names to all chunks and small text changes

Almost all files are modified!
Minor impact on the pipeline ==> saved figures saved by the execution could be found under a new name.

I agree that I don't fit a single format.

However, I remember I've read that kebab-case is better suited for chunk names. I'm sorry, I can't find the source of it. And as the majority of the chunk names are in kebab-case, I would prefer to go for it, to not spend too much time on it.

Would you like to change these ?

get-topNmarkers, cluster-dimPlot-markers, cluster-dotPlot-markers, cluster-dotPlot-goi
topNmarkers is already a variable in the code.
DimPlot, FeaturePlot, ... are pre-existing functions from seurat.

Let's go with kebab-case then, I don't see the logic but since it's for file names it shouldn't be an issue.
For references to existing variables/functions, I'd say we can keep them as they are.


Unable to properly use Dexter

Data Provider
Blockfrost
DEX
All
Description
I'm having issues with how Dexter is set up.
I created a minimal reproduction of this error in a separate repository.
https://github.com/NickJorens/dexter
npx tsc
node build/src/index.js
Any help or insights are greatly appreciated.

@NickJorens Im assuming your issue here is running a project with Dexter, could you try running the project with the --es-module-specifier-resolution=node flag? So node --es-module-specifier-resolution=node build/src/index.js

@Sluder I'm still getting an error. I also defined that in my tsconfig.json
➜  dexter git:(main) node --es-module-specifier-resolution=node build/src/index.js
node:internal/modules/cjs/loader:1051
  throw err;
  ^

Error: Cannot find module '/Users/nick/dexter/build/src/index.js'
    at Module._resolveFilename (node:internal/modules/cjs/loader:1048:15)
    at Module._load (node:internal/modules/cjs/loader:901:27)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:83:12)
    at node:internal/main/run_main_module:23:47 {
  code: 'MODULE_NOT_FOUND',
  requireStack: []
}

Node.js v20.4.0

@NickJorens I pulled your project & got it running by adding "skipLibCheck": true in the tsconfig, could you try that out?

@Sluder I'm sorry but that didn't resolve the error. What version of Node are you running?
node:internal/errors:496
    ErrorCaptureStackTrace(err);
    ^

Error [ERR_MODULE_NOT_FOUND]: Cannot find module '/Users/nick/dexter/node_modules/@indigo-labs/dexter/build/constants' imported from<EMAIL_ADDRESS>    at new NodeError (node:internal/errors:405:5)
    at finalizeResolution (node:internal/modules/esm/resolve:225:11)
    at moduleResolve (node:internal/modules/esm/resolve:837:10)
    at defaultResolve (node:internal/modules/esm/resolve:1035:11)
    at DefaultModuleLoader.resolve (node:internal/modules/esm/loader:251:12)
    at DefaultModuleLoader.getModuleJob (node:internal/modules/esm/loader:140:32)
    at ModuleWrap.<anonymous> (node:internal/modules/esm/module_job:76:33)
    at link (node:internal/modules/esm/module_job:75:36) {
  code: 'ERR_MODULE_NOT_FOUND'
}

Node.js v20.4.0```

I updated the repo with your change.

Sorry forgot to add, I got it running by adding the skipLibCheck & using the --es-module-specifier-resolution=node flag when running the project. I am on node version v18.16

@Sluder So it seems this package is incompatible with v20.x
Works fine on v18.16 LTS
I'll switch to LTS for now.
Something to be aware of in the future :)
Thanks for your help!

Interesting find, appreciate you letting me know!
We've had this issue open to investigate the requirements as well https://github.com/IndigoProtocol/dexter/issues/28


Turn off debugging in dist scripts

Why are the changes needed?
set -x is currently used in dist scripts for debugging and the console looks a bit messy, it seems better to turn it off and users can enable it themselves if necessary.
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[x] Run test locally before make a pull request

Codecov Report

Merging #1368 (6373fe9) into master (ff1f78f) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #1368   +/-   ##
=========================================
  Coverage     79.07%   79.07%
  Complexity      110      110
=========================================
  Files           181      181
  Lines          7645     7645
  Branches        932      932
=========================================
  Hits           6045     6045
  Misses         1099     1099
  Partials        501      501

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 ff1f78f...6373fe9. Read the comment docs.

Thanks, merging to master


Add basic structure for asv performance measurement

Based on napari asv (https://asv.readthedocs.io/en/v0.6.1/) configuration, I have created initial configuration for this project.
Added benchmarks are only for adding structure. After review of the code, I'm not sure where I should start with benchmarks.

Ah no the problem is not with mamba, but that we need to run asv in main first. I'll revert my commit.

Ah no the problem is not with mamba, but that we need to run asv in main first. I'll revert my commit.

No problem is with lack of definition of release commit (it point to main, instead of commit hash).
When you plan to do next release? I will create separate PR to improve release process and fix lack of this information.

@Czaki I checked the code and it looks great to me! I have noticed that in a few places there seems to be some redundancy in some configuration that could lead to problems in the future if we are not careful when making changes.
For instance I see that Python 3.11 is specified manually in asv.conf.json, benchmarks.yml. I could imagine that forgetting to change all of them at the same time could lead to subtle errors. Instead of "centralizing" this information, I would suggest to address this by writing in this PR a explanation/checklist (it can be very small, like 5-10 lines of text) of how to use asv for our dev purposes and what to do when something needs to be changed.
Example checklist on how to add/edit benchmarks:

remember to split qt and non-qt tests
before committing, test locally with asv ...
if you change the Pyhton version, change it in asv.conf.json and benchmarks.yml

WDYT?

Probably we will be doing a release of spatialdata and spatialdata-plot within 2 weeks, in napari-spatialdata we could make it if needed.

Could you check readme now?

Looks great, thanks!


Move creation of /export/pvs directory into NFS server role

For an on-prem cluster, the value of the jump server is less clear to me than for the cloud providers. If using ansible to stage all the systems, you pretty much by definition have SSH access to all systems, so the jump server isn't required as an entry point into the cluster. Users may also have limited compute resources, so there could be a desire to eliminate the jump server if unneeded.
For my use cases, I'd like to eliminate this server. Simply not assigning an instance to the jump server role seems like the obvious way to do so, and works well enough for this specific project, but would cause complications when baselining with viya4-deployment. That project, when creating the sas storage class, expects /export/pvs to exist on the NFS server. When the cluster is created, the NFS server role creates /export:
https://github.com/sassoftware/viya4-iac-k8s/blob/main/roles/systems/nfs_server/tasks/main.yaml#L18
However, the jump server role creates /export/pvs:
https://github.com/sassoftware/viya4-iac-k8s/blob/main/roles/systems/jump_server/tasks/main.yaml#L88
It's not clear to me why it's done this way, other than for consistency with the external cloud provider viya4-iac-* projects. That being said, I don't know why it's done that way for those projects either, but perhaps there's good reason I'm unaware of.
I can see no technical reason that the jump server is responsible for creating /export/pvs, and to me it's unintuitive. It also makes the jump server basically required to complete the NFS server setup. With the proposed change, the jump server can still act as the entry point into the cluster, the mounts are still setup, but the NFS server role can stand alone and fulfill it's function for the sas and default SCs.

Would seem this PR was closed when the staging branch was deleted. Was that intentional? Otherwise I can re-open when/if the staging branch is recreated.

@AWSmith0216 I did not expect it to automatically close this PR when staging was deleted. I've recreated the staging branch, could you please re-open the PR?

Thanks. It's been re-opened.

For this code base this would work; however, when one chooses an enterprise level solution for their storage, i.e. other cloud providers this work flow breaks. There is not actual nfs server therefore I am closing this one.


Remove Rubidium recommendation

Rubidium is deprecated (as stated by one of the project maintainers in the ReforgedHub discord server) and won't receive new updates. They also recommend using Embeddium.

Thanks! They haven't seemed to mention this on their mod pages.


Validation & Error Bags

This doesn't seem to be implemented. Is anybody working on it? If not, I can submit a PR myself. I'm thinking something like this will work just fine:
from django import forms
from django.http import HttpRequest, HttpResponse
from django.views import View
from django.shortcuts import redirect

import inertia

class ExampleForm(forms.Form):
    name = forms.CharField(max_length=100)
    email = forms.EmailField()

class ExampleView(View):
    def post(self, request: HttpRequest) -> HttpResponse:
        form = ExampleForm(request.POST)

        # Option A, more explicit
        if not form.is_valid():
            raise inertia.ValidationError(form.errors)

        # Option B, automatically raise the error
        inertia.validate(form)

        # use form.data here...

        return redirect("some_route_name")

For this to work properly in all cases we need to know the route where the user is coming from. Django doesn't store the previous route in the session like Laravel does, so there's no equivalent of Laravel's return back()->withErrors(). You can emulate the return back() part using the HTTP Referer header as suggested here or by sending a next=URL query param in the request as suggested here.
However, the Referer header can't be trusted in all cases and the next query param requires changing the frontend code. So I'm thinking of 2 possible solutions, either implement Laravel's behavior of storing the previous URL in the session to provide a return back() function or just provide the route at validation time:
inertia.validate(form, error_url="some_route_name_or_url")

Second option is more verbose but easier to implement, and it can coexist with the first option since we could make error_url=None by default and grab the route from the session. I guess all this could be a separate middleware that basically catches the inertia.ValidationError, allowing the users to opt-in by adding the middleware to settings.py. Otherwise it could be integrated into the current inertia.middleware.InertiaMiddleware.
Edit:
Issue https://github.com/inertiajs/inertia-django/issues/21 is related to this.

https://github.com/inertiajs/inertia-django/issues/21

Hey @antoniosarosi thanks for raising this again. I'm very much a Rails/Laravel-minded developer who's built a few apps in Django so I'm sure I missed some of the intricacies of how Django's built in forms work.
My first thought is that explicitly providing the error url seems very pythonic to me; it's obvious and doesn't rely on magic to implement.
I also think it makes sense to integrate this into the existing middleware. It seems like this would be core functionality and, as a user, I wouldn't expect to have to add additional middleware to use it.

@BrandonShar I already have a basic working example using a separate middleware in my Django app. I'll copy-paste-refactor the code into the built-in Inertia middleware and submit a PR so we can discuss this further. I also love Laravel, but Django does things differently, particularly when validating forms they don't return back()->withErrors() like we would in Laravel, the example code that I linked doesn't even redirect, it just renders the form again with the errors in the POST response. That's fine if the URL is the same for GETing the form and submitting it, but it might not always be the case. The point is, I don't wanna make Django look like Laravel, but I think integrating Inertia requires some built-in functionality that Django doesn't provide.

I'm not using Laravel, but the Inertia documentation suggests flashing form errors into the session and then redirecting to the form page. Render could automatically extract errors from the session into page.props.errors. It could do the same with the session messages key, enabling the Django messages framework to work with Django-Inertia.
Having build_props in render to automatically extract errors and messages from request.session might be a naive implementation of this.
At the moment, I wrap inertia.render to do exactly this. Then, we just have to put form.errors in the session and redirect.


Create Wireframe for Website

Create basic layout of website visually showing functionality

Took care of the design of the wire frame.

this better be closed now

Finished with Wireframe


Options not being recognized as a valid property

Angular version: 15.1.2
ngx-slider-v2: ^16.0.2
It works properly but still got the error and the application won't build

Try using version 15 for Angular 15
npm install --save ngx-slider-v2@15


Implemented custom comparison auth function

Support supplying your own user/password comparison function and keep the original API the same.

Thanks for the PR. Have added a couple of comments. Breaking the API is my biggest concern.

Addressed by https://github.com/goji/httpauth/pull/15


MM-25363 Separate logic between running Cypress specs and saving/publishing test reports

Summary
Separate logic between running Cypress specs (run_tests.js) and saving/publishing test reports (save_report.js).  This is a step towards parallel testing in Cypress. Typical use would be running multiple test groups against its own test server, and then at the end, all reports will be processed all at once. Something like:
# Test run 1: against server 1
node run_tests.js --stage='@prod' --group='@messaging'

# Test run 2: against server 2
node run_tests.js --stage='@prod' --group='@enterprise'

# Test run 3: against server 3
node run_tests.js --stage='@prod' --group='@messaging,@enterprise' --invert

# Consolidate, save and publish all test data from all Test runs
node save_report.js

Ticket Link
Jira ticket - https://mattermost.atlassian.net/browse/MM-25363

@srkgupta Thanks for checking and yes, that makes sense.  I'll update the PR.

@srkgupta Updated, PTAL.

Some checks aren't passing, couldn't merge @saturninoabril

Thanks Prapti. It was due to recent update of ESLint rule on comma-dangle. Now fixed.  Also I added small update on S3 folder - replacing dot with dash. Will merge after passing the build.


[key binding] option + backtick, when bound, end up in editor

VSCode Version: 1.44.2
OS Version: macOS 10.15.4

I'm trying to use alt + 1 ~ alt + 9 to access sidebar views as alternative ways besides the default keys, given that on mac alt is the option key.
This works pretty well, except alt + `(backtick).  When the focus is not in an edit buffer, it works, otherwise the backtick character would be inserted into the active editor, as if the shortcut key is not bound.
Steps to Reproduce:

open keybindings.json
insert the following key binding:
{ "key": "alt+", "command": "workbench.action.toggleSidebarVisibility" }`
assuming the key was not yet bound
save and restart vscode to ensure the change take effect
close all editor buffer if any
type alt + `, observe it toggles the visibility of the sidebar.
now open an edit buffer and focus it, type the key again, it not longer works, and the editor is changed.

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

I'm having the same problem with the Option key. I'm trying to create an edit window/terminal toggle, so tried to set up the focus when using alt+ and when I'm in either the terminal or the edit window, it just type the '' as if it doesn't even recognize the Option key. I'm using the default macbook pro keyboard

We currently don't support keybindings which conflict with IME (alt+` insert accent characters).


Add GITHUB api url to allow usage on github enterprise

Hi @hestonhoffman , do you accept PRs?
I'd like to introduce api_url input, to allow using your github action in Github Enterprise servers.
I believe tests are passing: https://github.com/cvakiitho/changed-lines/actions/runs/9935750026/job/27442546701?pr=1
Thanks in advance

Thanks for the PR! I'll take a look ASAP 👀

Yes, you are right, https:// is inside api_url, sorry about that.


Scrolling screenshot support

Is there any way you can add support for scrolling or full page screenshots?

There's an option to define x/y ratio of the website to take screenshot. Prooly you'd need to manage that manually until i get any soln to make it work automatically.


Build Types

How can this be done in PHPCI, when i automatically build project when new commits are pushed i would like to run one set of Phing build files and if i manually hit build button i would run other set. This way i can have staging pack and when i would do manual build i would have production pack.
Is there a way to do that in phpci.yml?

Any time frame when this gets implemented?

It's quite a big change, as we'll need to change the phpci.yml format to support it.
We've been thinking about supporting different configurations based on which branch is being built, which I think would cover what you're trying to achieve.
This is certainly not going to make it into v1.6, so it'll be a few weeks at least, assuming someone can get it working in time for v1.7.

Yep that would be awesome! Thanks!


MUMMNG-2765 pull in url

Pull in url as a variable, rather than a literal

Coverage remained the same at 33.499% when pulling 5b8ac12001a6e35e69241ed6f9857b2768ea0f9e on urlFix into 01fe335d39d0f24399870f1090252f34be29ca32 on master.

Did this in #288 as well with some other changes.

Closing this pull request. The same functionality is in #288


More than just scroll

Scrolling is only one of the movements that could benefit from being natural; perhaps this library could be rewritten as naturalMove and let people apply it to any property.
Its API could be:
naturalMove(start, end);
-> {duration: ..., easingFunction: ...} // from what I understand

Once one has those you could just use one of the many many animation libraries

This sounds like a subject for a separate library (I have once started working on it, and the animation formulaes have actually been taken from there). Such a library would require more configuration options, and there are already a lot of them on the web. Eventually I came up with this minimal solution covering the particular scrolling-case.


Reduce differences of registration between execution machines

The current version (773fd20) produces different results on different machines, i.e., run the registration on my MacBook Pro and the desktop machine with Ubuntu 14.04 using:
#!/usr/bin/env python

import os
import socket
import mirtk

mirtk.subprocess.showcmd = False
mirtk.subprocess.onerror = 'exit'

repeat = 3
tgtuid = '1000'
srcuid = '1001'
imgdir = os.path.expanduser(os.path.join('~', 'Datasets', 'MAL35', 'N4'))
imgfmt = os.path.join(imgdir, '{id}.nii.gz')
tgtimg = imgfmt.format(id=tgtuid)
srcimg = imgfmt.format(id=srcuid)
params = dict(model='Rigid+Affine+FFD', bg=0, ds=5, be=.005, tp=.2)
refdof = None

def compare(refdof, outdof):
    """Compare result of registration to reference transformation."""
    refjac = refdof.replace(".dof", ".nii")
    if not os.path.exists(refjac):
        mirtk.evaluate_jacobian(tgtimg, refjac, refdof, float=None)
    outjac = outdof.replace(".dof", ".nii")
    if not os.path.exists(outjac):
        mirtk.evaluate_jacobian(tgtimg, outjac, outdof, float=None)
    mse = mirtk.calculate(refjac, '-', outjac, '-square', '-mean', onexit='output').strip()
    print("Mean squared difference of Jacobian determinants: {}".format(mse))

hostname = socket.gethostname()
print("Running {model} registration {n:d} times on {host}".format(model=params['model'], n=repeat, host=hostname))
outdofs = []
for i in range(repeat):
    outdof = '{prefix}_{host}_{i:d}.dof.gz'.format(prefix=os.path.splitext(__file__)[0], host=hostname, i=i+1)
    if os.path.exists(outdof):
        print("Exists: {}".format(outdof))
    else:
        print("Compute: {}".format(outdof))
        mirtk.register(tgtimg, srcimg, dofin='guess', dofout=outdof, **params)
    outdofs.append(outdof)

print("\nComparing all results to the first obtained transformation")
for i in range(1, repeat):
    compare(outdofs[0], outdofs[i])

if refdof:
    print("\nComparing all results to the reference transformation")
    for outdof in outdofs:
        compare(refdof, outdof)

The output with -model Rigid+Affine is the same on both machines (see also #300).

Actually getting the same results for each of the three iterations and each of the two machines with the original test script using -model Rigid+Affine+FFD, bending energy, and topology preservation.
It's possible I had just not recompiled on merapi (desktop) with the recent change of the Maximum length of steps for affine models of today.


[FEATURE REQUEST] Choose the neomake's maker to be run

I'm using neomake to run make in my current project, but I'm using a custom defined maker that looks like this:
let g:neomake_c_armgcc_maker={
    \  'exe': 'arm-none-eabi-gcc',
    \ 'args': [ '-Os', '-g', '-Wall', '-Wextra', '-Wno-main', '-pedantic', '-x', 'c', '-std=c99', '-DSTM32F4', '-DSTM32F407VG', '-DSTM32F4CCM', '-I.', '-Ilibopencm3/include', '-c', ],
     \ 'errorformat': '%f:%l:%c: %m', }

So I'm actually using :Neomake armgcc. I would like the option to tell neomake-autolint which of the neomake's makers to run.
Thanks in advance.

Could you try adding this to your vimrc: let g:neomake_c_enabled_makers = ['armgcc']

I have just tried using that line and it solved the issue. Funny thing is, I thought I read the neomake's documentation carefully. I guess there's nothing to be done here, so I'll close the issue.


selecting the first question from the list of questions

Description
Retreiving first question for a particular game instance from questions table
Fixes #152
How to test?
npm run dev
Checklist

[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have made corresponding changes to the documentation
[X] I have merged the latest version of the source branch (typically develop) into this branch before pushing my changes
[X] This PR is ready to be merged and not breaking any other functionality

Implemented the change in the PR title

Hey Swathi, Is this PR for the issue #152 ?


[SPARKR-203] load and save DataFrame

This PR also include HiveContext (but commented, depends on hive in spark-sql)

Cool. The thing with HiveContext is that there is build of Spark without Hive support. I'm not sure there is a way to detect that. Maybe that should be a build profile.

@felixcheung Yes, I didnot figure out how to deal with it. We could re-visit it later.

@davies -- Can't we write a Java function which checks if Spark has Hive bindings built in ? Either using reflection or by inspecting the class loader we should be able to figure this out ?

@davies - Can we do something like what we do in Spark's Scala REPL ? I'm pasting the function below which creates HiveContext if hive is available
  def createSQLContext(): SQLContext = {
    val name = "org.apache.spark.sql.hive.HiveContext"
    val loader = Utils.getContextOrSparkClassLoader
    try {
      sqlContext = loader.loadClass(name).getConstructor(classOf[SparkContext])
        .newInstance(sparkContext).asInstanceOf[SQLContext]
      logInfo("Created sql context (with Hive support)..")
    }
    catch {
      case cnf: java.lang.ClassNotFoundException =>
        sqlContext = new SQLContext(sparkContext)
        logInfo("Created sql context..")
    }
    sqlContext
  }

@shivaram I had fixed on R side.

The R side fix is good. LGTM.

@felixcheung Nice catch on the HiveContext name -- I will you send a PR to fix it

@felixcheung Thanks!


Simplify geo and weighted options in DNSPolicy

What
In the past we worked with OCM which brought the idea of a hub cluster and multiple views of the same gateway. We no longer support OCM but some of that legacy has remained in our API we should simplify this to reflect a policy can only target one gateway and even if it could target multiple they would be in the same cluster.
Currently we have
  loadBalancing:
    weighted:
      defaultWeight: 120
      custom: # <--- New Custom Weights being added
        - weight: 255
          selector:
            matchLabels:
              kuadrant.io/lb-attribute-custom-weight: AWS
    geo:
      defaultGeo: US #catch all geo

we then have to set attributes on the gateway in the form of annotations to set these custom weights. This no longer makes sense.
Proposed change is to have the values set directly on the policy and indicated if geo is a default or not
  loadBalancing: # each of these values would be required.
    weight: 100
    geo: U.S
    defaultGEO: true #indicates whether to use this as a the wilcard / catchall geo

[ ] Add RFC https://github.com/Kuadrant/architecture/pull/105
[ ] Update Kuadrant Opertor with new structure
[ ] Update Docs with new structure

@mikenairn any additionl thoughts here? If not I can look at putting together and RFC would ideally like this to happen before v1

@trepel @azgabur This is an important up-coming change we are going to prioritise.

@maleck13, do we want this change into v1alpha2, therefore also automatically in v1, of the DNSPolicy API (targeting Kuadrant v1)?

yes


[FEATURE] Prefix all commands with PnP or something similar

Is your feature request related to a problem? Please describe.
Having commands like Get-Site encourage non-namespaced cmdlets which may conflict or clobber with other modules. PnP doesn't have absolute domain over this space and should follow standard practices for this.
Describe the solution you'd like
By default only load prefixed commands (e.g. Get-PnPSite) and for compatability allow for a setting (global variable, cmdlet, etc.) that will enable aliases to the non-namespaced cmdlets like Get-Site
This will allow those who need the compatibility for legacy scripts to enable it, but encourage new scripts to use a more standard powershell convention.

We actually prefix all the files with PnP by means of the Prefix variable as set in the psd1 file included. It's not in the source code, but the released version (and the debug builds all have a PnP prefix). This is a deviation of our approach with the older modules for Windows PowerShell (Legacy) where we hard coded the prefix in the source code for the cmdlets. Making the prefix optional you will be allowed to do
Import-Module -Name PnP.PowerShell -Prefix MyPrefix

The cmdlets will then be available with the MyPrefix prefix, alike Connect-MyPrefixOnline
By default the prefix is set to PnP. The PSD1 file is generated in the build scripts that generate the nightly builds but it's also set in the build script that generates the debug builds: https://github.com/pnp/powershell/blob/4f2b2fddca6a6541fbb99a57a5057f9fa4f44729/build/Build-Release.ps1#L104

@erwinvanhunen then you have a bug because it's still exporting the nonprefixed commands as of 0.1.16.

What do you see when you do
Get-Command -Module PnP.PowerShell
And how did you import the module?

I use the module from the PowerShell Gallery on Windows, Linux and Azure Cloud Shell and everywhere I have prefixes.
If you import the module manually:
Import-Module PnP.PowerShell
That should automatically resolve the psd1 file and use that to load the cmdlets.

File this under "gremlins" because after a full reboot I can't reproduce it, sorry for the trouble, I'll re-inquire if I can consistently reproduce it.


add recipe back into setup_chef_server

lol :shipit: fast!

I don't know what I missed that.. Apologize @tduffield

No worries. I missed it too.


Shortcut key to open numbered Window Walker results

Summary of the new feature/enhancement
It would be nice to have a shortcut key to open a specific item in the window walker list, like Win+Digit.
E.g. if 4 results are shown then Ctrl+3 should open the 3rd item, without having to press Down 3 times then Enter.
Would be good to number the items in the list (maybe just while holding Ctrl), and it should work for the first 10 items (Ctrl+0 for the 10th).

WindowWalker will be merged with launcher. Based on the launcher spec here,

10
Users can use Ctrl+<insert number> to launch from the search result
2


Section "Binding Modes": Table needs to be updated

URL (minimal example if possible):
https://openui5nightly.hana.ondemand.com/docs/topics/68b9644a253741e8a4b9e4279a35c247.html
Steps to reproduce the problem:

Open the above documentation page
Scroll down to the table

What is the expected result?

Model
One-way
Two-way
One-time

OData V4 model
✔
✔(default)
✔

OData V2 model
✔(default)
✔
✔

OData meta model
❌
❌
✔(default)

JSON model
✔
✔(default)
✔

XML model
✔
✔(default)
✔

Resource model
✔(default)
❌
✔

ODataMetaModel is added
ResourceModel is updated

What happens instead?

Model
One-way
Two-way
One-time

OData V4 model
✔
✔(default)
✔

OData V2 model
✔(default)
✔
✔

JSON model
✔
✔(default)
✔

XML model
✔
✔(default)
✔

Resource model
❌
❌
✔

Missing ODataMetaModel
ResourceModel

Missing (default) indication
Incorrect supported modes

According to the API reference, ResourceModel supports one-time (not supported in async mode) and one-way (default):

This model supports the binding modes OneWay and OneTime, but not TwoWay. When the recommended asynchronous loading of the bundle is used, binding mode OneTime can't be used.

https://github.com/SAP/openui5/blob/669ce0236477ab0a3fec670c72c1e8bde9bd3b0f/src/sap.ui.core/src/sap/ui/model/resource/ResourceModel.js#L98-L102

Hello @boghyon
Thank you for sharing this finding. I've created an internal incident<PHONE_NUMBER>. The status of the issue will be updated here in GitHub.
Regards,
Petar

Hi @boghyon,
the documentation is adapted with the correct binding mode information. The ODataMetaModel and ODataMetaModel V4 are added to the table as well.
The update to date version can be seen here:
https://openui5nightly.hana.ondemand.com/#/topic/68b9644a253741e8a4b9e4279a35c247
Best regards,
Jiawei


Drop support for .NET 3.1

We currently support .NET 3.1 and .NET 6.0, but Microsoft no longer supports .NET 3.1 as of December 13, 2022.  We should likely drop support for .NET 3.1 as well, but will need to do so iteratively.  We'll need to ensure that all downstream codegen dependencies etc continue to function as desired after this change.

This has long since been addressed.


'detection of light stimulus involved in visual perception' should not be SubClassOf 'detection of visible light'

I think this is wrong and too human centric. Birds and insects with UV photoreceptors perceive images using those receptors. This is visual perception.  This is causing problems for CL definition of photoreceptors. See https://github.com/obophenotype/cell-ontology/issues/2253

Ooops - I accidentally did a hot fix to master !!  Sorry!!!
I didn't think that was possible on ODK repos.  Should probably set github to block.
Anyway - I think minor edit so should be harmless.  Would have like to look for any secondary effect though!

Better ti obsolete the grouping class than have an incomplete one?
On Fri, Aug 2, 2024 at 6:58 AM David Osumi-Sutherland <
@.***> wrote:

Closed #28657 https://github.com/geneontology/go-ontology/issues/28657
as completed via 68871a3
https://github.com/geneontology/go-ontology/commit/68871a3c5820d2876f50c6b6cdd4c1c7df8552fa
.
—
Reply to this email directly, view it on GitHub
https://github.com/geneontology/go-ontology/issues/28657#event-13742263368,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAMMOKYGZKTGMGLF5JEVLTZPOGBJAVCNFSM6AAAAABL4SWTIOVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJTG42DEMRWGMZTMOA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***
com>

I don't think any grouping classes need changing. While this does leave a singleton, there is definitely plenty of detection of visible light outside visual perception - e.g. in plants or by clock cells with cryptochromes

Passes QC at least


Allows regional admin to delete user via User Registration UI.

🗣 Description
Updates delete user handler to enable regional admins to perform this action that's available in the UI.
💭 Motivation and context
Resolves: https://maestro.dhs.gov/jira/browse/CRASM-320
🧪 Testing

Case 1. User with globalAdmin is able to deny a pending request in the User Registration UI
Cast 2. User with regionalAdmin is able to deny a pending request in the User Registration UI

✅ Pre-approval checklist

[x] This PR has an informative and human-readable title.
[x] Changes are limited to a single goal - eschew scope creep!
[x] All future TODOs are captured in issues, which are referenced
in code comments.
[x] All relevant type-of-change labels have been added.
[x] I have read the CONTRIBUTING document.
[x] These code changes follow cisagov code standards.
[x] All relevant repo and/or project documentation has been updated
to reflect the changes in this PR.
[x] All new and existing tests pass.

Looks like there is a minor linter issue to insert a newline: https://github.com/cisagov/XFD/pull/345/files#:~:text=api/users.ts-,GitHub,-Actions.
Aside from that it LGTM.

Lint issue resolved

Looks like there is a minor linter issue to insert a newline: https://github.com/cisagov/XFD/pull/345/files#:~:text=api/users.ts-,GitHub,-Actions.
Aside from that it LGTM.

@nickviola - can you re-review the linting update?


OPSEXP-590  Addition of cleanup and uninstallation playbook.

Cleanup and Uninstallation playbook added along with updated documentation.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

see https://github.com/Alfresco/alfresco-ansible-deployment/runs/7561678936?check_suite_focus=true#step:11:127

https://github.com/Alfresco/alfresco-ansible-deployment/pull/388#pullrequestreview-1055130
Here looks like TASK [Removing the Packages]  passed. Stopping the playbook at next play. Just trying to get why it is different for this instance.


SPARKC-527: Allow Split Count to be Passed to CassandraSourceRelation

Previous to this it was impossible to choose the target split count for
a CassandraSourceRelation (SparkSql, Dataframes). This patch allows
for case insensitive parameter passing as well as adding a new parameter
for splitCount. The extra lowerCase in consolidateConfs is for
making the forward merge easier.

Build against Scala 2.11 finished with success

Build against Scala 2.10 finished with failure

Build against Scala 2.10 finished with success

Build against Scala 2.11 finished with success

They would but you wouldn't be able to set it since it doesn't start with
spark.
On Fri, Feb 2, 2018, 1:50 AM Jaroslaw Grabowski<EMAIL_ADDRESS>wrote:

@jtgrabowski commented on this pull request.
In doc/reference.md
https://github.com/datastax/spark-cassandra-connector/pull/1161#discussion_r165600377
:

@@ -236,6 +236,14 @@ OSS Cassandra this should never be used.

64
Approx amount of data to be fetched into a Spark partition

+
+  splitCount
+  None
+  Specify the number of Spark partitions to
+read the Cassandra table into. This parameter is
+used in SparkSql and DataFrame Options.
This comment is a bit confusing. RDD like CassandraTableScanRDD also
respect this parameter, right?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/datastax/spark-cassandra-connector/pull/1161#pullrequestreview-93567364,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAZNYQerwZrDnx-KI8Y2d1DgIuwW9pwgks5tQtpbgaJpZM4R2TN5
.


Add workflow and group settings help texts

resolves #2352

please stick to https://github.com/OpenSlides/OpenSlides/wiki/Development-organization#pull-request-titles-and-commit-messages

Updated help text list. Please take care, most German and English tool tips have been changed.
2023-06-07_Sammlung-Tooltips.ods

As of now hyperlinks (or to be more specific html content) are not possible within the tooltips provided by Angular Material. Therefore we would need to implement something ourselves there if we want to use hyperlinks.
Would it be possible to provide text without hyperlinks or is this a requirement?

Hyperlinks would be nice to have, but if the implementation requires too much effort and time, we don't need to implement them. Just add the rest of the text without the brackets with hyperlinks.

Hyperlinks would be nice to have, but if the implementation requires too much effort and time, we don't need to implement them. Just add the rest of the text without the brackets with hyperlinks.

I removed the hyperlinks from the texts.


The README says it uses an algorithm by default

Hi, the README says it uses HS256 by default but since https://github.com/jwt/ruby-jwt/pull/184 you must provide an algorithm option
We should remove the word "default" from readme

This is also mentioned in #227 and explains this in more detail. Discussion will continue there.

You are correct, I mixed things up. #227 played tricks on my mind. Reopen.


Ensure that MS-build 64-bit is used with Ntools

While integrating ntools  into GitHub actions, I discovered that the msbuild.exe (32-bit) is being used instead of the 64-bit version.  The nbuild.targets files uses the import with $(ProgramFiles) in the path(<Import Project="$(ProgramFiles)\Nbuild\common.targets"/>).  Using the the mbbuild.exe (32-bit version) will expand the $(ProgramFiles) to "C:\Program Files (X86)" instead of "C:\Program Files".  The cause this error:
Project "D:\a\home\home\nbuild.targets" on node 1 (staging target(s)).
D:\a\home\home\nbuild.targets(3,2): error MSB4019: The imported project "C:\Program Files (x86)\nbuild\common.targets" was not found. Confirm that the expression in the Import declaration "C:\Program Files (x86)\nbuild\common.targets" is correct, and that the file exists on disk.
Done Building Project "D:\a\home\home\nbuild.targets" (staging target(s)) -- FAILED.

Acceptance Criteria

Modify the nbuild  to find the 64-bit version of the msbuild.exe so that $(ProgramFiles) expands to "C:\Program Files ".

Resolved in #51


Update broken link in README

The link where it says "Read the actual documentation here" goes to 404. Just updated the link to go to the start of the Docs.

Thanks @marykatefain! I think we want to link to the main docs page here](http://apostrophecms.org/docs/). If you'd like to update that I'll get it merged in.

There ya go! Should be good now. I wasn't sure if you meant to link to the same place twice (because the docs are linked directly above) haha.


New kli multisig continue command that can pick up an incomplete multisig event and finish the processing.

Signed-off-by: pfeairheller<EMAIL_ADDRESS>

Codecov Report

Merging #367 (76fb862) into development (b209e9a) will decrease coverage by 0.00%.
The diff coverage is 66.66%.

@@               Coverage Diff               @@
##           development     #367      +/-   ##
===============================================
- Coverage        92.69%   92.69%   -0.01%
===============================================
  Files              116      116
  Lines            36528    36530       +2
===============================================
+ Hits             33860    33861       +1
- Misses            2668     2669       +1

Impacted Files
Coverage Δ

setup.py
0.00% <ø> (ø)

src/keri/app/grouping.py
84.65% <50.00%> (-0.18%)
:arrow_down:

src/keri/__init__.py
100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


save checkpoint

when I use tutorial_train.py to train my own dataset, I notice that the checkpoint folder is empty, and hparams.yaml is {}, the code save a version_x every 300 steps, is the checkpoint folder empty because I haven't trained an epoch yet?

when I use tutorial_train.py to train my own dataset, I notice that the checkpoint folder is empty, and hparams.yaml is {}, the code save a version_x every 300 steps, is the checkpoint folder empty because I haven't trained an epoch yet?

I have the same problem, can you tell me how did you solve it?

when I use tutorial_train.py to train my own dataset, I notice that the checkpoint folder is empty, and hparams.yaml is {}, the code save a version_x every 300 steps, is the checkpoint folder empty because I haven't trained an epoch yet?

I have the same problem, can you tell me how did you solve it?

I just continued training and when finished, I found the ckpt file in the last version_xx folder.


[$500] Chat - Multiline code block (three back ticks) using makes the hyperlink broken

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.26-1
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Open Expensify App
Log in
Open any chat
In compose box enter any link for example google.com and send
Tap and hold on the sent link in the step 4
Select Edit comment
Enter three back ticks inside the square brackets for example [google.com]
Send the link

Expected Result:
The hyperlink should not be broken even if multiline code block (three back ticks) is used
Actual Result:
The hyperlink is broken if multline code block is used
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/1e0dc5a3-5d70-4594-8529-ec74ac9a0887

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~011a4dae0f313f46d1
Upwork Job ID:<PHONE_NUMBER>200806912
Last Price Increase: 2024-01-17

#vip-vsb

it's expected behavior

Issue
The issue is related when an edit action is triggered for an http link.
Cause
The cause is due to the getActionText function in file ContextMenuActions.tsx line 33.
This function retrieves the last message from the Onyx store.
A link is stored in following form at the moment:
[link](https://link)

The issue is not limited to editing the link within the [] by adding ''. Another issue occurs when the link within the () is edited which will cause an inconsistency between the link items between [] and () not matching anymore.
Solution
Implement a new feature for the getActionText function which handles link values accordingly by returning the link as shown in the chat window.
Additionally I suggest to add a sanitation feature which checks the link prior storing the result in the Onyx key/value store. The color of the text field can be changed to red in case a link is malformed like: htp://test.com.
I have the local development environment running and was able to reproduce the problem.
I feel confident to have a solution in a timely manner ready.

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~01d632f2317d732aa4

You can't make the whole multi-line codeblock a link. Why would anyone need that?

Thank you for your feedback, I wasn't referring to make it a link. I suggest to keep the entered content the same format like it was entered. In above example of the issue:
User entered: google.com

When edit action is triggered:
Show: google.com for editing without the braces.

Is this a bug really? And are we sure that its worth fixing?

Great question.
I understand:

User enters an address like. test.com
Edit shows : test.com
The entered link is not consistent with the edit link.
Therefor the issue with the editing was reported.
Thoughts?

#vip-vsb

I'm not sure what the context here is. I guess that we're talking about a concern raised by an important client.
If this weren't the case, it would be a no-brainer to close this issue immediately.
Still, the first action would be asking them what they actually expect here, as a multi-line codeblock being a giant link is something I've never seen. It makes little sense to me.

#vip-vsb

This is an internal project -- not tied to a client or customer! Agree that this is probably not worth fixing, let's close it!

For future reference, it's not only not worth fixing, but not a bug at all. Works as expected. ExpensiMark is inspired by MarkDown and is nearly compatible with it. MarkDown is currently specified in the CommonMark specification. CommonMark classifies multi-backtick code blocks as blocks, while links are inlines. You can never wrap a block with an inline in CommonMark; it's impossible by definition.

While the current edit feature is using the correct MarkDown syntax, is it expected that an end user has knowledge of how to use MarkDown to edit the field?

No, it's not. The users can follow the Expensify help pages.
Relating to MarkDown/CommonMark can help us decide what is an expected behavior and how we want to shape ExpensiMark, so it is based on the battle-tested standard solution unless we know we want to deviate.


Use rollup for bundling

Fixes #185
Lmk what you think about this @stalniy

I'd recommend to use @babel/preset-typescript as it allows to use babel to generate bundle and typescript to generate types. I commented about that solution in https://github.com/crcn/sift.js/issues/185#issuecomment-614023574
I use it in CASL and pretty much happy with that. There are some limitations how you can write TypeScript but nothing extraordinary

Thanks for the code review 👏. Updates should be published as 13.0.2.


Machine mac-6 lost external connection to iPhone device.

Machine mac-6 lost external connection to iPhone device.

External connection has been restored by rebooting machine mac-6.


[MM-30340] Update post height when comment box height change occurs

Summary
This is a recent regression from https://github.com/mattermost/mattermost-webapp/pull/6920
Ticket Link
https://mattermost.atlassian.net/browse/MM-30340
Fix 5.29

Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour.

Cherry pick is scheduled.


Example code only seems to work if I change the server to use ipv4

If I try to provide an ipv6 address in the client (say ::1:8080) the client refuses to accept the format.
I changed the code in the server to use ipv4:
let server_addr = (IpAddr::V4(Ipv4Addr::LOCALHOST), flags.port);
I can now connect with an ipv4 address from the client.
Am I doing this wrong, or is there a bug in the example code?

Thanks for the report! That's interesting — I can't reproduce this problem on my Macbook, but I can't say you're doing anything wrong either! At least, not without a bit more info :)

What's the specific error you're getting? Is it something like invalid IP address syntax?
Can you try --server-addr '[::1]:59061'?

Same experience on end. Would add using the tarpc module feels like working in the dark as there is no logs you can tap into.

Documentation can be incredibly better for launching a tcp transport server.
Logs can be better for the dev experience.
Example service doesn't tell you anything on fresh launch(execute client bin and it dosen't get a response log).

Still exploring for my use case and haven't managed to make an RPC call for 2 days now with pure pull from the example service.
Sorry.
Will contribute to making things better when am able to successfully make use of it and it's associated features. - A big blocker to contributions.
Sorry.

Hi, I'm sorry you've had a frustrating experience! Feel free to join the discord channel where others might help fix your problems faster.
There should be some logs available for the client binary, not sure what's going wrong.  Have you enabled log output with RUST_LOG environment variable? E.g. RUST_LOG=debug cargo run --bin client?

F##k yeah, it finally logged. Your command did it.
Would this be my knowledge of rust or it's how you get debug information from your setup?
- I learned rust fully a few days ago hence my curiosity
Most helpful thing I will do now is to write a beginners guide once am up to speed with what am doing.

Nice! RUST_LOG is not specific to tarpc, but it would still be good for the examples that use it to specify how to get output :)

Can see, my memory bank needs an update. Thanks for the insight.


Add Syntax Preprocessors Layer

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Motivation
Would it be possible to add specific rules of nonstandard markdown syntax to be preprocessed when built?
But Why would we?
This can allow much more diverse documentations that would've been previously impossible due to the technical limitations of Docusaurus. Examples include:
1. {{안녕^hello}}
This is a nonstandard md syntax used in the Eastern hemisphere for denoting pronunciations and side notes. This md syntax will convert to the following HTML:
<ruby>안녕<rp>(</rp><rt>hello</rt><rp>)</rp></ruby>

which will print as
안녕
2. [[wikilinks]]
This is a nonstandard md syntax. We can add a preprocessor that converts wikilinks to MD links when building.
3. [alt](image "figcaption")
This is also a nonstandard md syntax. When building, we can also add a preprocessor to convert them into HTML Figure-Figcaption.

figcaption value

Self-service

[X] I'd be willing to do some initial work on this proposal myself.

Hi! These are already triviallly implementable with custom Remark plugins. You just need to write the parser and transformer yourself. Docusaurus is not going to implement custom Markdown syntax unless there proves to be great demand—which there doesn't seem to be. Please file this as a Canny request instead, detailing what you want—do you want a custom preprocessor API? (In which case we already have the Remark plugin API) Do you want a specific one of these syntaxes?

I'll try to implement wikilinks and backlinking someday but not sure it will go in Docusaurus core.

For image figcaptions, there's likely already existing remark plugins you could use.
For example https://github.com/Microflash/remark-figure-caption
It's not exactly what you want but may be good enough?
Eventually, you can add a "figure" MDX component to parse the image URL caption directly in the client component


Update instructions for new nimbix trial access.  Issue #13.

Close issue #13.
Will open a new one for Korean version.

Forgot to update some of the numbering of the steps. Commit coming.


Limit PR checks to build only the modified images

Description
Followup to

https://github.com/opendatahub-io/notebooks/pull/543

that filters out the list of images to build during a PR check only to those that have some source files changed.
This will speed up the PR check build.
How Has This Been Tested?
Created a draft PR that changes one file in a single docker image

https://github.com/opendatahub-io/notebooks/pull/555

Checked that it correctly builds all images that depend on this one.
Merge criteria:

[x] The commits are squashed in a cohesive manner and have meaningful messages.
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has manually tested the changes and verified that the changes work

Review comments history

I see on this draft that you affect only the DS notebook, but i see that GH runs the trusty and some others, do i miss something?  https://github.com/opendatahub-io/notebooks/pull/555  stiill checking

actually, no, it is not nonsense, it truly is all images that 1) are at the end of the FROM: dependency chain and 2) they or one of the images in their FROM: chain is built in jupyter/datascience/ubi9-python-3.9 directory

for example, here's why it builds cuda; that cuda image builds in the jupyter/datascience/ubi9-python-3.9 directory

# Build and push cuda-jupyter-datascience-ubi9-python-3.9 image to the registry
.PHONY: cuda-jupyter-datascience-ubi9-python-3.9
cuda-jupyter-datascience-ubi9-python-3.9: cuda-jupyter-minimal-ubi9-python-3.9
	$(call image,$@,jupyter/datascience/ubi9-python-3.9,$<)

and here's why it builds trusty; trusty image depends on jupyter-datascience-ubi9-python-3.9

# Build and push jupyter-trustyai-ubi9-python-3.9 image to the registry
.PHONY: jupyter-trustyai-ubi9-python-3.9
jupyter-trustyai-ubi9-python-3.9: jupyter-datascience-ubi9-python-3.9
	$(call image,$@,jupyter/trustyai/ubi9-python-3.9,$<)

Since you affect the DS, I would expect to build the base, minimal and DS (base & minimal are prior DS on the build chain)

when it builds each of the three images, it does the full chain, so it starts from the base, https://github.com/opendatahub-io/notebooks/actions/runs/9498328161/job/26176822339?pr=555

It has to do the full chain because I don't have image registry available when building PRs on github actions

@atheo89 @harshad16 can you create tide/merge-method-squash label in this repo? It would be useful here. I want all commits squashed, but I don't want to cause any more notification spam by force-pushing.
If I could apply the label, tide would squash everything when it does its own thing.

Can we implement filtering to avoid building images for certain file types, such as *.md or *.yaml? :thinking: This idea came to me when I opened this PR:: https://github.com/opendatahub-io/notebooks/pull/573

@atheo89 future enhancement

https://github.com/opendatahub-io/notebooks/issues/581

/override ci/prow/runtimes-ubi9-e2e-tests
/override ci/prow/runtimes-ubi8-e2e-tests
/override ci/prow/anaconda-ubi8-e2e-tests
/override ci/prow/codeserver-notebook-e2e-tests
/override ci/prow/intel-notebooks-e2e-tests
/override ci/prow/notebooks-ubi8-e2e-tests
/override ci/prow/notebooks-ubi9-e2e-tests
/override ci/prow/rstudio-notebook-e2e-tests