Add vector + scaled vector math

A feature request here.
I know, we don't want to bloat the vector math with convenience methods. However, this one I found I am using too often. Almost every time I do any vector math - when I am trying to add a scaled vector (without changing the original) to another vector.
Normally, one would do something like:
// a + b*2;
const temp = new pc.Vec3();
temp.copy(b).mulScalar(2).add(a);

It requires you to allocate a temp vector, if you don't want to change the original. The only way to avoid allocation, you'd need to do a component wise writeout, which is verbose and would better be as part of the vector math methods:
a.addScaled(b, scalar);

// where
a.x += b.x * scalar;
a.y += b.y * scalar;
a.z += b.z * scalar;

Definitely could be useful.
You can also add a function to your Vec3 prototype, but agree, not as convenient.

Often it's even more "specialized": offset an existing point with a scaled normal
Example:
pc.Vec3.prototype.mulAdd = function (a, s, b) {
    this.x = a.x + s * b.x;
    this.y = a.y + s * b.y;
    this.z = a.z + s * b.z;
}
const start = new pc.Vec3(100, 200, 300);
const normal = new pc.Vec3(0, 1, 0);
const end = new pc.Vec3();
end.mulAdd(start, 20, normal);

(the engine code itself is a good "test suite" for testing generic methods, so the extra methods can show how they shorten existing code in a potential PR)

@kungfooman I think your example is similar to my first snippet, where you are using a third vector to store the product. I personally prefer the chain variant, though, as it is self-explanatory of the operands in use just by following the chain. Not sure there is a benefit over current API:
const temp = new pc.Vec3();
temp.copy(normal).mulScalar(20).add(start);

I think addScaled would be more generic in this case, as it will be up to developer to decide if a third vector storing the product to be allocated or not.

So now you prefer the chain variant and apparently have no problem using a temporary vector, why are you proposing a change?
Or what exactly does it make simpler? Like a real-life example of situations you encounter too often.

Sorry for not being clear :D What I meant is I do prefer the chain one over your suggested .mulAdd(vec, scalar, vec), because both require a third vector, but a chain one is easier to read.
My proposal is to add a .addScaled(vec, scalar). Which modifes the target vector in place, but doesn't create a temporary vector that holds a scaled one.
If we take your example, where I'd like to change the position of a point by some scaled normal, it would be:
position.addScaled(normal, 20);

The benefit here, is I only use 2 vectors to end up with the same result as in your example, which uses 3.

No problem, I just want to hash it out a bit and that's why I asked for real life examples.
Because in most cases I feel that you usually have a starting point, so your example "in full" would still require chaining aswell:
position.copy(start).addScaled(normal, 20);

But yes, I see your point, addScaled is a little bit "smaller" which by nature should apply to more situations.

You are correct there, it depends on case by case. It would be up to the developer to decide, whether a temp vector is needed. At least we give a choice. For example, this is a valid case as well:
const pos = entity.getPosition();
pos.addScaled(normal, 20);
entity.setPosition(pos);


add print statements to preprocess

Added several print statements to give us an idea of the size of the dataset (using nrow) at various stages throughout the script so that we can check logs inside L4.
@RochelleKnight @venexia I have assigned both of you, feel free to tweak anything!

It all looks good to me, I can't think of anything else to add


Submit signed Plutus-generated transactions

[x] Research Yoroi to determine if there's components we can use prior to cardano-wallet support

Removed from scope since submitting a transaction is a generic operation. Client-side behaviour to be implemented in https://github.com/input-output-hk/cardano-js-sdk


Transformime replacement

We need to replace the transformime dependency since we want to be able to return phosphor widgets, not just DOM elements.

CC @afshin, @sccolbert

@jasongrout, I sent a PR against your PR with some enhancements.

@blink1073, @afshin - ready for review.

Needs a rebase

(sorry)

too fast for me :). give me a second...

Hang on, let me add a bit more documentation...

Okay, docs committed.

I was going to suggest including the katex CSS, but it is not included in the npm package, bummer.

This css? https://github.com/Khan/KaTeX/blob/a81c4fe78df1b19401883cff4f8af2b1c0f5b033/static/main.css
That seems to be the only css file in the repo...

ah, never mind, they have a less file: https://github.com/Khan/KaTeX/blob/a33fa4fd646994484ce9d4335e5d2f8d4ccc274d/static/katex.less

Right, but see https://github.com/Khan/KaTeX/issues/148.

I am actually about to send them a PR  ;)

Darn, PR not needed, we just need them to release (its been 7 months).

I'm inclined to just say that we expect mathjax to be on the page, otherwise things don't render.

All I'm saying is this:
class LatexWidget extends Widget {
   constructor(latex: string);
}

class HTMLWidget extends Widget {
   constructor(html: string);
}

And leave everything else as an implementation detail.

I'd say leave the katex in there for now, if folks want to use it they need to provide the CSS from the CDN.

@blink1073 - better?

Yep!  Let's wait for the conversion on where to put this stuff :wink:

agreed!

Closed in favor of #136.

Small question, I don't know where to put it. 1or 2 years ago, I noticed a few discussions/idea about having latex mathjax into something smaller, that could be shippable offline. I lost track since then.
Did something happen ? is it Katex ? is it Mathjax 3.0 ?

Hi @stonebig, we looked into using katex, but it is not as fully featured as MathJax, and appears to have lost its primary contributor.  MathJax is now packaged on npm, so could be shipped offline if desired.

ok, thanks.

@stonebig mathjax is included in notebook-4.x like the rest of our js dependencies, so it is available offline now.

ah thanks...  my number-of-files did only incressa by 9 000 instead of 20 000 expected. Did you do some "mathjax huge number of file " reduction ?

@stonebig yup. It's mainly removing the image fonts, which is the bulk of it, but a few other output options are removed as well.

thanks a lot, I was hoping for this feature.


use client-go instead of kubectl for deploy operations

Deploy was implemented by calling external kubectl binary https://github.com/surajssd/kapp/pull/72
It will be bettter to use client-go to communicate with cluster. Once we start doing some more complicated operation it might become tricky to parse kubectl output, using client-go should be much better.

I think this is a feasible thing now! We don't need to rely on kubectl with new client go is in.


Package 'gpg' has no installation candidate

Section Install point 1 produces error on Ubuntu 16.04.4 LTS:
> Package 'gpg' has no installation candidate
Change 'gpg' to 'gnupg' or make a note of it.

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

ID: fe1526a9-7f22-504e-484c-0f3edc11b178
Version Independent ID: c5dec108-efcd-aa77-04b5-803b6b894dba
Content: Install the Azure CLI on Linux with apt
Content Source: docs-ref-conceptual/install-azure-cli-apt.md
Product: azure
Technology: azure-cli
GitHub Login: @sptramer
Microsoft Alias: sttramer

Will be fixed on the next site publish.


Found 2 elements with non-unique id #first_name

Found 2 elements with non-unique id #first_name on https://lonestarcargo.org/profile/edit
First Name and Last Name Fields both share the #first_name value.
Not essential, just something my inspector found on accident.

Can you provide more detail on this? A cursory look shows that the form is correctly populating the name fields.

Found it. It is in the profile form for the disabled field.
Good catch! This does need updated, however this is a display only (disabled) field so it is relatively low priority as it does not affect functionality.
Thanks for the report!

Both inputs on the profile page for first name and last name have the same ID.

First Name
<input type="text" name="first_name" id="first_name" value="" disabled="disabled" class="form-control">

Contact staff to make name changes

Last Name
<input type="text" name="first_name" id="first_name" value="" disabled="disabled" class="form-control">

Yeah, Like I said. Not important at all, just saw a red light in my inspector. haha

Indeed, found it in the profile _form partial.
Thanks!


Add DWOC in the Open Source Program Section in readme.

Is your feature request related to a problem? Please describe.
It's important to add this Project's participation in DWOC in its readme file as well.
Please assign me to work on this if interested. Thank you :)

Go ahead @IshetaBansal 👍

Hello sir,
I would like to work on this . Can you assign this task to me?

Hello sir,
I would like to work on this . Can you assign this task to me?

Hey @Khimit , This task is already assigned to isheta so you can checkout other issues.

hii
i would like to work on this issue

Hey, I would like to take this work

This issue is already assigned : )


After switching from vstest v1 to v2, ASCII bell character in test causes vsts-agent to fail to publish results.

After opening this in the vsts-agent repo, it occurred to me that this is probably something that should be handled in the new version of vstest:
https://github.com/Microsoft/vsts-agent/issues/899
I apologize that I don't know the appropriate way to handle this cross post.

@nigurr

@GaTechThomas thanks for raising this issue.
Error message that shows up is "Failed to publish test results: '�', hexadecimal value 0x07, is an invalid character.". @Faizan2304 is this handled in the work we did for escaping special characters (for invalid xml)?

@codito : Yes we have fixed it but fix is in Visual Studio 2017.
In vstest v1 they were taking care of invalid xml character when they read trx file but in vstest v2 it seems they are not. I talked with @nigurr and they want to go with this flow only and expecting trx file generator to give sanitized trx file.

@GaTechThomas : I am not able to repro the issue with given repro test method shared in bug https://github.com/Microsoft/vsts-tasks/issues/3932 . I used below test method and got the repro
public static TheoryData TestValues
            => new TheoryData<IEnumerable<KeyValuePair<string, string>>>
            {
                { new []{ new KeyValuePair<string,string>("key", "string with \a in it") } },
                { new []{ new KeyValuePair<string,string>("key2", "second string with \0 in it") } }
            };

        [Theory]
        [MemberData(nameof(TestValues))]
        public void Test2(IEnumerable<KeyValuePair<string, string>> values)
        {
        }

We have sanitized the invalid xml character in trx file in Visual studio 2017. I have verified and issue is not getting repro if we are generating trx with visual studio 2017.

Thank you. I have been testing VS2017, but not yet on VSTS. It has a few bugs that our team will have to wait before we can switch. However, I may be able to build on the server with VS2017. Probably won't be able to prioritize it for this week.
-Thomas

From: Faizan2304<EMAIL_ADDRESS>Sent: Tuesday, April 25, 2017 7:20 AM
To: Microsoft/vstest
Cc: GaTechThomas; Mention
Subject: Re: [Microsoft/vstest] After switching from vstest v1 to v2, ASCII bell character in test causes vsts-agent to fail to publish results. (#683)
@GaTechThomashttps://github.com/GaTechThomas : I am not able to repro the issue with given repro test method shared in bug Microsoft/vsts-tasks#3932https://github.com/Microsoft/vsts-tasks/issues/3932 . I used below test method and got the repro
public static TheoryData TestValues
=> new TheoryData<IEnumerable<KeyValuePair<string, string>>>
{
{ new []{ new KeyValuePair<string,string>("key", "string with \a in it") } },
{ new []{ new KeyValuePair<string,string>("key2", "second string with \0 in it") } }
};
    [Theory]
    [MemberData(nameof(TestValues))]
    public void Test2(IEnumerable<KeyValuePair<string, string>> values)
    {
    }

We have sanitized the invalid xml character in trx file in Visual studio 2017. I have verified and issue is not getting repro if we are generating trx with visual studio 2017.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/vstest/issues/683#issuecomment-297012563, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AIvtiyyicsKkCbX_tRSc7mRjMPRQ9Jtdks5rzeUigaJpZM4MyOOv.

@GaTechThomas
To get you unblock we need to have fix in two places:

Fix in TestPlatform side ( i'e vstest.console.exe, the one who is creating trx file)
Fix in TestAgent (who is reading trx file and uploading test).

We have made fix in TestPlatform side in VS 2017 but to consume fix you need pass custom vstest.console.exe path which is VS-Insallation-Directory\Common7\IDE\Extensions\TestPlatform\vstest.console.exe in your vstest V2  task.
@nigurr Can you please a bug on agent side and add link here for the same.
I m closing this bug from our side as we don't have any action item here.

when i use this version of vstest.console.exe I get a bunch of UI unit test failures that I don't get with the default one used.  I am stuck because the tests pass with the default version, but that has the exception described in this issue when trying to publish results, this one will publish but these tests fail... any reason why these are failing in the version of vstest.console.exe in this folder: VS-Insallation-Directory\Common7\IDE\Extensions\TestPlatform\vstest.console.exe?
Test method .. threw exception:  System.InvalidOperationException: The calling thread must be STA, because many UI components require this.

@santoorman the support for STA threads in Extensions\TestPlatform\vstest.console.exe was recently added (PR: https://github.com/Microsoft/vstest/pull/969). Documentation for this is at https://github.com/Microsoft/vstest-docs/blob/master/docs/configure.md#execution-thread-apartment-state.
This fix will ship as part of VS 2017.5 releases.


Build time transforms cannot use shorthands

Thanks for making a very good library.
I tried using "@griffel/webpack-extraction-plugin" in "Nextjs".
almost everything works perfectly, but "makeStyles" using "shorthands" is not bundled in css

Hey, can you please create a repro on https://stackblitz.com/ or https://codesandbox.io/? So we can check in which conditions and which styles are broken.

I made a sample with codesandbox
Please check.
https://codesandbox.io/s/cranky-black-h2ofbr

Closing as a duplicate of #154. We will get to it and fix it.


Tileset tiles outside texture cannot be removed

Godot version
4.2.stable
System information
Godot v4.2.stable - Windows 10.0.19045 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 3050 (NVIDIA; <IP_ADDRESS>67) - 11th Gen Intel(R) Core(TM) i7-11700KF @ 3.60GHz (16 Threads)
Issue description
It's possible for a tileset to end up with tiles outside of the texture when the Separation and Margin values are changed. This is fine temporarily, the user might want to tweak the settings until everything lines up.
But, if those tiles are stuck outside of the texture when the user is done, there is no quick way to remove them. The drop-down button for removing tiles outside the texture is always greyed out. The button above it does not affect invalid tiles. Tiles outside the texture cannot be selected or erased.

The only quick ways to remove the invalid tiles, are to:

Remove the spritesheet from the tileset and try again, setting the proper values before creating any tiles.
Reload the project and reopen the tileset. Errors are printed to Output about invalid tiles. The option to remove invalid tiles is now enabled, even though the invalid tiles do not show up visually. Click it, save, and the issue is resolved.

Possibly an issue with #77986 not detecting these invalid tiles, and enabling the drop-down button when atlas settings are changed.
Steps to reproduce
Assuming you're using the provided spritesheet: (spritesheet.zip)

Create a spritesheet with separated tile sprites, or with margins around the edges
Create a tileset, and pull the spritesheet into it
Agree to the automatic tile creation (this can happen with manually created tiles too, this is just for speed)
Set Margin and Separation X/Y values to 32

You will end up with invalid tiles outside the texture that cannot be removed, until the project is reloaded
Minimal reproduction project
A project reload detects the invalid tiles and hides (but does not actually remove) them, so a full project MRP is not really useful for clear, accurate reproduction.
You can use the spritesheet below for testing, though. The tiles are 64x64, with a 32px margin and 32px separation.
spritesheet.zip

That button is active when there are tiles outside the texture (you should also be getting errors in the output). Seems like in this case these tiles are not detected as such.

In my case it started working after test-run the project.


Does this plugin create jib-classpath-file?

Hi,
does this plugin generate the same layout for the /app as plugins for other build systems? My images don't seem to contain /app/jib-classpath-file and /app/jib-main-class-file that I rely on when using Gradle for instance. If these files are not supposed to exist in images built by sbt-jib, what is the canonical way to define custom entry point scripts for it that would be stable between potential future jib layout changes?

Try playing with jibJavaDockerBuild (in contrast to jibDockerBuild) sbt-jib task. It is different by using the functionality from jib-core to assemble the class files and jars together (jibDockerBuild uses sbt-jib proprietary logic). Maybe the files will get created.
But if you want to pick a specific main class, just use the sbt key Compile / mainClass -- that should work out of the box.

@sideeffffect when I try using java counterpart commands I get the following error:
[error] Expected ':'
[error] Not a valid key: jibJavaDockerBuild (similar: jibDockerBuild, jibTarImageBuild, jibImageBuild)
[error] myProject / jibJavaDockerBuild
[error]                           ^

The same is with jibJavaTarImageBuild vs jibTarImageBuild that I use regularly instead of jibDockerBuild. I depend on this version of the plugin: addSbtPlugin("de.gccc.sbt" % "sbt-jib" % "1.0.0")

I've updated the plugin to v1.3.4 and I'm now able to run the java versions of the commands, but unfortunately the layout is still the same and it's missing the classpath and main class meta files.

What is it that you're trying to achieve?
Maybe the jibJavaAddToClasspath sbt-jib setting could help you?

@sideeffffect at this point it's more about image layout parity between Gradle/Maven and sbt plugins. I'm in the process of migrating a project from Gradle/Java to sbt/Scala and it'd help substantially if I could guarantee drop-in replacement quality of the resulting images, hence my attention to these extra metadata files that are being generated by jib's gradle plugin.
I realize that for the most part I could get around a custom entrypoint with a help of jibJvmFlags and other options as long as customizations are related to JVM runtime options. But overall I'd like to know whether avoiding these meta files was a conscious design consideration on the part of the sbt plugin authors that motivated them to diverge from the official Gradle/Maven plugin behaviour.

As far as I understand the Gradle plugin's motivation for these meta-files, they allow custom user-defined entrypoints to remain stable and resistant to any potential future layout changes that jib may choose for library and class files. As per their documentation, the files are guaranteed to be present at all times, so that the following commands will be able to launch your apps via your custom entrypoints at all times:
(Java 9+) java -cp @/app/jib-classpath-file @/app/jib-main-class-file
(with shell) java -cp $( cat /app/jib-classpath-file ) $( cat /app/jib-main-class-file )

I'm not sure about the history, maybe @schmitch would know? But it doesn't sound too complicated to add this feature. Could you give us specific examples how these files look like?

Sure, /app/jib-classpath-file looks like a colon-separated list of image-included libraries and resources that the project depends on:
/app/resources:/app/classes:/app/libs/commons-logging-1.2.jar:/app/libs/xml-apis-1.4.01.jar:/app/libs/ion-java-1.0.2.jar:/app/libs/joda-time-2.8.1.jar[: ...]

and /app/jib-main-class-file contains the module path to the main entry point, like this:
com.myorg.mypkg.MyMainClass

And it seems like this is the PR that introduced these files into the plugin, serving the need of determinism in loading order: https://github.com/GoogleContainerTools/jib/pull/3280

Unfortunately, jib-plugins-common is not published :(
https://github.com/GoogleContainerTools/jib/issues/3578

Oh, that's a bummer, especially the lack of technical clarity on why not publishing it.


styled-jsx FOUC with custom Document

Bug report
Describe the bug
When using a custom document _document.tsx, styled-jsx does not SSR styled-jsx and there is a flash of unstyled content unless I duplicate the static getInitialProps method.
To Reproduce
FOUC
import React from 'react';
import Document, { Head, Main, NextScript } from 'next/document';

import fonts from '../styles/fonts.css';
import reset from '../styles/reset.css';

export default class MyDocument extends Document {
  public render() {
    return (
      <html lang="en">
        <Head>
          <style dangerouslySetInnerHTML={{ __html: reset + fonts }} />
        </Head>
        <body>
          <Main />
          <NextScript />
        </body>
      </html>
    );
  }
}

NO FOUC
import React from 'react';
import Document, { Head, Main, NextScript } from 'next/document';
import flush from 'styled-jsx/server';

import fonts from '../styles/fonts.css';
import reset from '../styles/reset.css';

export default class MyDocument extends Document {
  public static getInitialProps({ renderPage }) {
    const { html, head, buildManifest } = renderPage();
    const styles = flush();
    return { html, head, styles, buildManifest };
  }

  public render() {
    return (
      <html lang="en">
        <Head>
          <style dangerouslySetInnerHTML={{ __html: reset + fonts }} />
        </Head>
        <body>
          <Main />
          <NextScript />
        </body>
      </html>
    );
  }
}

System information

OS: macOS
Version of Next.js: 7.0.0

Just tried your reproduction and in both cases it works fine.

@timneutkens Interesting. I'm using typescript with next 7 and it didn't work yesterday. We are running a custom server too. Can you think of another reason getInitialProps would not get called from the inherited class?

Could you point me to where _document gets used so I can try debugging a couple more things?

https://github.com/zeit/next.js/blob/canary/server/render.js#L159

Works fine if <Head /> exists. 🤔


Kwargs don't work for "big_model" that isn't OpenAI based

When Bloom, for example, is the big model:
response = client.complete(prompt = "Answer question Q. ",context="Q: What is the currency in myanmmar",
                           openai_key=settings.openai_api_key,
                           temperature=0.0,
                           data_synthesis=True,
                           finetune=True,)

This call returns an error, because the temperature param cannot be processed.
  File "/opt/conda/lib/python3.10/threading.py", line 1016, in _bootstrap_inner
    self.run()
  File "/opt/conda/lib/python3.10/threading.py", line 953, in run
    self._target(*self._args, **self._kwargs)
  File "/opt/conda/lib/python3.10/site-packages/llm_vm/completion/optimize.py", line 43, in new_thread
    t[0] = foo()
  File "/opt/conda/lib/python3.10/site-packages/llm_vm/completion/optimize.py", line 254, in promiseCompletion
    best_completion = self.call_big(prompt, **kwargs)
  File "/opt/conda/lib/python3.10/site-packages/llm_vm/client.py", line 100, in CALL_BIG
    return self.teacher.generate(prompt, max_len,**kwargs)
TypeError: BaseOnsiteLLM.generate() got an unexpected keyword argument 'temperature'

issue duplicate of #328
resolved by #332


How to add custom Table Styles

Hello,
Under Add-WordTable -Design parameter has option "Custom", Is there an  example on how to use the -Design Custom ?
Add-WordTable -WordDocument $WordDocument -DataTable $DataTable -Design Custom
Also if using Custom is possible, how do i define the Custom design?

Hi,
This is one enum values so not really sure how to use it. I'm not a word expert. But what you can do is use an example: https://github.com/EvotecIT/PSWriteWord/blob/master/Examples/Example-Tables4.ps1 but building your own custom stuff is going to be pain :-)

Hi @PrzemyslawKlys , Thanks!,
Can I use custom Hex or RGB code for color properties as stated in Example you posted?

Using my commands probably not. Overall, to be honest I don't know. I have created this like 2 years ago and haven't touched it since. Probably you would need to dive into sources and see what I do there. I don't think you can use custom colors but I may be wrong here.
Here's the C# code that I "translate" to PowerShell:

https://github.com/xceedsoftware/DocX/blob/master/Xceed.Document.NET/Src/Table.cs
https://github.com/xceedsoftware/DocX/blob/master/Xceed.Document.NET/Src/_Enumerations.cs
https://github.com/xceedsoftware/DocX/tree/master/Xceed.Words.NET.Examples/Samples/Table

You will be most interested in examples from there. Any reason why you want to waste time building your own custom design ? :)
From a quick look custom colors are no go.

So basically I am trying to automate some documentation, however the pre-design Word template has specific format for Heading Styles which I am able to apply using the Word template.
However it seems the Table styles doesn't work the same way how Heading Styles do.
Long story short, looking for custom colors so i can match the table style which the template uses.
Thanks, appreciate your help :)

Well, you can probably overwrite just header rather than doing the whole table from scratch. It's possible to add a table and edit it post-factum. Basically, once you create $table with add-wordtable you can get that table and display it's settings/change stuff around etc.

Thanks @PrzemyslawKlys for your help once again...

Not sure you can 'create' a custom table design with PSWriteWord but I did eventually work out how to do this by going back to the xceed site.
Process is:

Have a word document that has all of your styles (including custom tables types) already in it. Make sure the custom table design name does not include spaces.
Delete all the content so you still have all of the styles and then save
$CustomTableDesign = 'Name-Of-The-Custom-Table-Design'
$wordDocument = Get-WordDocument -FilePath $newDoc.FullName to open the doc
$table = Add-WordTable -WordDocument $wordDocument -DataTable $objData -Design Custom
$table.CustomTableDesignName = $CustomTableDesign
$wordDocument.Save()

Thanks @mdcaddic . Can you please let me know how can I get the Custom Table Design which I have created manually ?

A big Thank you to @mdcaddic .
After couple of brainstormings, trial and errors -- it is found that the table style name is CASE SENSITIVE and now it is working like a charm for me.

Glad this was resolved. Just to clarify - what @mdcaddic proposed was a simple .docx creation even where you create stuff, delete content, and just use it as a template. There are no special requirements here or anything that Xceed does differently.

@mdcaddic and @PrzemyslawKlys the above solution is not appropriate.
when we edit existing default table style ( Plain table) in word document and save it as template then only it is applying the style. when we defined new Table style is not applying. Could you help me in resolving this issue.
seems some issue wit

I assume when you say save it as a template you are referring to the table style. Absolutely make sure that the table style name you are using does not have spaces. That caused me a bunch of heartache for a while. For the document ensure you are saving the document as a docx and then open using Get-WordDocument.

@mdcaddic,
I can confirm the the style name i have created in word document is same as we have in following script. refer attached word document where i have created table style and imported data in to table.
the style is not applied what we have specified in script.
New_TEMP.docx
$FilePath = "C:\ps\New_TEMP.docx"
#Call the document from above filepath
$WordDocument = Get-WordDocument -FilePath $FilePath
#Technical Data
$hotfixes = Get-HotFix | select HotFixID, InstalledOn
$CustomTableDesign='Test'
$table = Add-WordTable -WordDocument $wordDocument -DataTable $hotfixes -Design Custom
$table.CustomTableDesignName = $CustomTableDesign
$wordDocument.Save()
#Invoke-Item $FilePath

@Nizamc I ran your code in ISE and got the table you were expecting (white writing on green in header). Sorry, I can't produce an error.


The last declared handler is applied to all stories in a file.

I'm trying to use the MSW add-on with React Query. Am experiencing an issue wherein the last declared handler is what's used for all stories in a file.
// LoginForm.stories.tsx
import { LoginForm } from ".";
import { Meta, Story } from "@storybook/react";
import { rest } from "msw";

export default {
  title: "components/molecules/LoginForm",
  component: LoginForm,
  decorators: [
    (Story) => (
      <div className="flex flex-col justify-center py-12 sm:px-6 lg:px-8 min-h-screen bg-gray-50">
        <Story />
      </div>
    ),
  ],
  parameters: {
    // opt out of decorators rendering in code preview
    docs: {
      source: {
        type: "dynamic",
        excludeDecorators: true,
      }
    },
  },
} as Meta;

const Template: Story = () => <LoginForm />;

export const MockedDefault = Template.bind({});
MockedDefault.parameters = {
  msw: [
    // Loading state
    rest.post("/auth/login", (req, res, ctx) => {
      return res(ctx.delay(1000 * 60 * 60 * 60), ctx.status(200));
    }),
  ],
};

export const MockedError = Template.bind({});
MockedError.parameters = {
  msw: [
    // Error state
    rest.post("/auth/login", (req, res, ctx) => {
      return res(
        ctx.status(400),
        ctx.json({ message: "Incorrect email and/or password. xxxxx" })
      );
    }),
  ]
};

The "Error state" handler is what's applied to all stories in the file. If I swap "Loading state"  and "Error state", it's the "Loading state" that's applied to all stories in the file. As an additional check, I changed the endpoint of MockedDefault's handler to some/other/endpoint, I still receive the response from the "Error state" handler. My assumption is MockedDefault should throw an error, since the story has an unhandled request?
This is what my .storybook/preview.js looks like:
import "../src/index.css";
import { BrowserRouter as Router } from "react-router-dom";
import { QueryClient, QueryClientProvider } from "react-query";
import { initialize, mswDecorator } from "msw-storybook-addon";

// Initialize MSW
initialize();

export const parameters = {
  actions: { argTypesRegex: "^on[A-Z].*" },
  controls: {
    matchers: {
      color: /(background|color)$/i,
      date: /Date$/,
    },
  },
  options: {
    storySort: {
      order: ["Introduction", ["Intro", "Installation"], "components"],
    },
  },
  layout: "fullscreen",
};

const queryClient = new QueryClient();

export const decorators = [
  mswDecorator,
  (Story) => (
    <QueryClientProvider client={queryClient}>
      <Router>
        <Story />
      </Router>
    </QueryClientProvider>
  ),
];

Packages:

Package
Version

MSW
0.35.0

msw-storybook-addon
1.4.1

storybook/react
6.3.12

react-query
3.33.1

I changed nothing with my code. Only thing I did was update to Storybook 6.4.7 and issue has been resolved.

For anyone else who encounters this, it took me too long to realise that this works fine when you're looking at the individual canvases but not when you're looking at them all at once on the Docs tab.


Patch for deb-pkg v1.6.1

stlink (1.6.1+ds-3)

Corrected install paths & changelog files
Corrected settings for deb pkg distribution

--> This patch addresses several points from a previous discussion and user feedback from the source project.
Notes:

Though it may not be a good practise in general, the changelog has been updated as some old entries have turned out to be wrong which have been corrected in the project's own changelog in the meanwhile. To avoid this in the future the Debian changelog should cover only package related changes, while the former will only focus on software-related changes (and will not list any distribution-related issues).
The Debian copyright file will no longer be updated with new contributors (there are too many and possibility of text formatting is very limited). To comply with the license terms we have added a separate listing within the project which is explicitly mentioned in the copyright file. This may be transferred to a markdown list in the future. For privacy reasons we also avoid to further disclose e-mail addresses publicly where possible.

This diff looks wrong? It's not just changing debian/ content, it's merging from the upstream master branch too. Was it a merge gone wrong on your local fork?

Notes:

Though it may not be a good practise in general, the changelog has been updated as some old entries have turned out to be wrong which have been corrected in the project's own changelog in the meanwhile. To avoid this in the future the Debian changelog should cover only package related changes, while the former will only focus on software-related changes (and will not list any distribution-related issues).

As mentioned already multiple times, the changelog will contain anything that is deemed relevant, as per debian policy. This includes non-packaging changes. And older entries will not be changed retroactively, as they are already part of the stable distribution.

The Debian copyright file will no longer be updated with new contributors (there are too many and possibility of text formatting is very limited). To comply with the license terms we have added a separate listing within the project which is explicitly mentioned in the copyright file. This may be transferred to a markdown list in the future. For privacy reasons we also avoid to further disclose e-mail addresses publicly where possible.

As mentioned already multiple times, the preferred format of the FTP team is the current format, and thus it is not going to change, to avoid rejections on future binary-new uploads and RC bugs.

This diff looks wrong? It's not just changing debian/ content, it's merging from the upstream master branch too. Was it a merge gone wrong on your local fork?

No, it is a patch for the latest release on our upstream master branch and was meant to be exclusively for the distributed deb package. That's why it will not be pushed to the projects' master branch currently. I addressed the points you mentioned earlier and corrected the tool/package-descriptions to align with the source project as they were outdated.
I will no longer care about the other points cited above, to be honest and will not add anything to the /debian files anymore too.
You may accept or change or close this PR - do whatever you want.

As per the debian workflow,changes outside of the Debian directory cannot be committed directly. The build will break. They have to be done as out-of-tree quilt-format patches. This is not a matter of personal preference - it's how 3.0 quilt format packages work.

So I am closing this then...
Proposed changes will remain on the debian branch of the stlink source repo for a while.
However this branch will be deleted again, shall there be no interest in working with it.


Update AArch64 features to Linux 5.17.

Adds three new features: ecv, afp and rpres.

for the record, in 5.18-rc1 added mte3

@gchatelet, @Mizux please review PR, approved from my side

Thx for the PR @tamaszarm and thx for the review @Mizux and @toor1245 .


Running MatlabStan

I tried to run the rats example but I get the following error message:
Undefined variable "mstan" or class "mstan.stan_home".
Error in StanModel (line 170)
p.addParamValue('stan_home',mstan.stan_home);
Error in stan (line 108)
model = StanModel();
Error in rats (line 20)
fit = stan('file','rats.stan','data',rats_dat,'verbose',true);
Anyone know how to get it to run? I know cmdStan is working as I was able to run the Bernoulli example.
Thanks

Looks like the matlab package +mstan folder is not in your path.

I've made sure the folder is in the path and modified the stan_home.m to point to my CmdStan installation. The error message changed to the following:
Error using StanModel/set.file (line 332)
File does not exist
Error in StanModel/set (line 275)
self.file = p.Results.file;
Error in StanModel/sampling (line 730)
self.set(varargin{:});
Error in stan (line 138)
fit = model.sampling(p.Unmatched);
Error in rats (line 20)
fit = stan('file','rats.stan','data',rats_dat,'verbose',true);

Looks like the rats.stan file is not in your current directory. You can download it here:
https://raw.githubusercontent.com/brian-lau/MatlabStan/master/Examples/rats.stan

This is not the issue. I already had the file rats.stan in the Examples folder

Is your current working directory the Examples folder? The error you are getting is simply saying the rats.stan file cannot be found (using Matlab's exist function). So, either the file is not in your current directory, or Examples is not in your path, in which case you should specify the absolute path to the file.


Spike three way merge using diff3

I've been testing this with a branch of my test prototype which includes commits for various scenarios: https://github.com/lfdebrux/test-prototype/compare/test-patching-package-json

So I have a couple of outstanding questions:

does it work on Windows if you have Git Bash installed? I'm assuming that that comes with diff3, but I haven't tested that yet
should we download just the package.json, or the whole release archive?
in what scenarios will this break the users package.json?

More generally, there is the question of does it follow the behaviour we want? How do we handle cases where it does the wrong thing?  Depending on the options you give to diff3, it will try to handle conflicts on its own or not. I've gone with the option -e which seems to be the most aggressive with resolving conflicts, and I have yet to find a scenario where it puts merge markers in the package file, or does behaviour that is obviously bad. But I haven't done an exhaustive search. So far I think broadly it does, and we could assume that it is unlikely, and just tell users to come to us if they have problems. But it might be worth discussing with others.


[OPENJPA-2915] properties of type Duration can be set as expected

@solomax don't think so, we can do it lazily if there is any request


List exchanges without historical api

[kraken]
[liqui.io]

@hleb-albau @vitalylvov
there is information about exchanges
https://docs.google.com/spreadsheets/d/1SQtFmfn5b2Kp2T9X6BF7dQhnx4ybq7dVTIihBWyXjbo/edit?usp=sharing


Missing checks in order.go

See the shipping checks done here. Emptiness checks for postalCode and country are missing.

The countrycode needs a nil check. @drwasho didn't you say you wanted them to have the option to not enter a postal code in shipping?

Only name and country should be required, as I recall.

Yeah some locations don't have a postcode.

The nil check is actual there when shipsTo is checked.


BSB LAN Adapter

Short Review-Result:

onUnload should clear the timeout
please test compact mode, especially that stopping really stops everything
if onObject/state changes are not needed please do not register it

@hacki11 Did you find time to check my comments?

Not yet, but thanks for your review I will address the issues!

all three issues are fixed in current master 0.0.6

updated from master and resolved conflicts

welcome in the repo


state missing from List notifications REST API response schema

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
https://docs.github.com/en/rest/activity/notifications?apiVersion=2022-11-28#list-notifications-for-the-authenticated-user
What part(s) of the article would you like to see updated?
The API response schema is missing an optional field called state within the subject object.
This might also be missing from the response schemas for other Notification API endpoints...
Additional information
No response

@setchy Thank you for opening this issue! I'll get this triaged for review ✨

@setchy It looks like the team will need to address this one internally, so I've copied this issue over to our internal repo. I'll go ahead and close this out 💛 thanks again for raising this issue!

@nguyenalex836 - i've done some more research and it looks like I was mistaken and raised this in error.  No further action is required :)

@setchy No problem - appreciate the update here! 🟡


Corrected faulty example on doc for Enable-PnpDevice

The command which uses -match to validate the error code does not work as intended. Changing it to -eq solved the error.

Docs Build status updates of commit 81597d3:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/windows/pnpdevice/Enable-PnpDevice.md
:white_check_mark:Succeeded

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

@dariomws @get-itips @tseward
Peer review needed for this PR. Thanks.

Thanks for the contribution however,
PnP cmdlet's documentation changes should not be done on this repo.
They should be done here https://github.com/pnp/PnP-PowerShell/pulls against the DEV branch.
PnP cmdlet's doc is directly reflected from PnP cmdlet's source code a process is then run to sync the documentation.
This should be closed.