Question: how to a POST with form-data correctly

Hi,
So my question is how I would implement the following (with request) using isomorphic-fetch
var options = {
    url: authUrlRoot + 'auth',
    followRedirect: false,
    form: {
      client_id: config.APP_KEY,
      redirect_uri: config.REDIRECT_URL,
      response_type: "code"
    }
  };

  request.post(options, function(error, httpResponse, body){
    // Do something here..
  });

I've tried something like
var FormData = require('form-data');
var form = new FormData();
form.append('client_id', config.APP_KEY);
form.append('redirect_url', config.REDIRECT_URL);
form.append('response_type', 'code');
fetch(authUrlRoot + 'auth', { method: 'POST', body: form })
    .then(function(response) {
        if (response.status >= 400) {
            throw new Error("Bad response from server");
        }
        return response.json();
    })
    .then(function(data) {
        console.log(data);
    });

But that gives me "Bad response from server" - status code is 400 Bad request. I might be totally blind, but I cannot seem to figure out what is missing?

The spec says you pass the FormData instance as the body...
fetch("/login", {
  method: "POST",
  body: form  //just pass the instance
})

You may need additional parameters, depending on your use...
fetch(url, {
  credentials: 'include', //pass cookies, for authentication
  method: 'post',
  headers: {
  'Accept': 'application/json, application/xml, text/play, text/html, *.*',
  'Content-Type': 'application/x-www-form-urlencoded; charset=utf-8'
  },
  body: form
});

Or you can manually encode your data, example below will take an object... (NOTE: ES6 syntax)
//post-as-form.js

// npm module - safe deep cloning
import clone from 'safe-clone-deep'

export {postAsForm as default};

async function postAsForm(url, data) {
  var response;
  try {
    //safe clone of data
    data = clone(data);

    //array that holds the items that get added into the form
    var form = [];

    //special handling for object/array data, arrays will use "model" as the container ns
    addItemsToForm(form, typeof data == 'object' ? [] : [options.name || 'model'], data);

    //you'll need to await on response.text/blob/json etc
    return await fetch(url, {
		  credentials: 'include', //pass cookies, for authentication
		  method: 'post',
		  headers: {
		  'Accept': 'application/json, application/xml, text/play, text/html, *.*',
		  'Content-Type': 'application/x-www-form-urlencoded; charset=utf-8'
		  },
		  body: form.join('&')
	  });

    //return plain text as-is, don't parse it here
    return {
      response: response,
      text: await response.text()
    }
  } catch(err) {
    //console.error("Error from server:", err && err.stack || err);
    err.response = response;
    throw err;
  }
}

function addItemsToForm(form, names, obj) {
  if (obj === undefined || obj === "" || obj === null) return addItemToForm(form, names, "");

  if (
    typeof obj == "string"
    || typeof obj == "number"
    || obj === true
    || obj === false
  ) return addItemToForm(form, names, obj);

  if (obj instanceof Date) return addItemToForm(form, names, obj.toJSON());

  // array or otherwise array-like
  if (obj instanceof Array) {
    return obj.forEach((v,i) => {
      names.push(`[${i}]`);
      addItemsToForm(form, names, v);
      names.pop();
    });
  }

  if (typeof obj === "object") {
    return Object.keys(obj).forEach((k)=>{
      names.push(k);
      addItemsToForm(form, names, obj[k]);
      names.pop();
    });
  }
}

function addItemToForm(form, names, value) {
  var name = encodeURIComponent(names.join('.').replace(/\.\[/g, '['));
  value = encodeURIComponent(value.toString());
  form.push(`${name}=${value}`);
}

Would you please explain why we clone first ?

Hi, im having strange problem, my code is like this:
export function signDocument(uri, jobid, securitycode, docid, signature) {
  var form = new FormData()
  form.append('signature', 1)

  return fetch(uri+jobid+'/documents/'+docid+'/signNextSigPlaceholder?securitycode='+securitycode, {
      method: 'POST',
      header: {
        'Accept': 'application/json, application/xml, text/plain, text/html, *.*',
        'Content-Type': 'application/x-www-form-urlencoded; charset=utf-8'
      },
      body: form,
    })
    .then(function(response) {
      return checkStatus(response)
    });
}

But on the Api side, i don get signature. but if i use jquery like this:
$.ajax({
          url: postUrl+'/documents/'+ this.state.currentDocument.docId+'/signNextSigPlaceholder?securitycode='+code,
          dataType: 'json',
          method: 'POST',
          contentType:"application/x-www-form-urlencoded",
          data: { signature:podpis },
         cache: false,
          success: function(data) {
            ....
            });
    });

Then i get signature in  API.. Event if i try it with Advanced REST client chrome extension, and i pass signature as RAW or FORM, i get the signature.
What could be wrong?

So what helped me when i was trying to replicate a request that I got working not using fetch was to look at the request in chrome dev tools in networking. Look and see whats different  when you click on the same request URL .ALl the request and form data should match up. My realization was that I needed to send over cookies as well because my API was giving me a 403.  I added credentials: 'include', or  credentials: 'same-origin', to the fetch and that added the missing cookies . IDK if this is your issue but i was in a similar postion where i had a working old school style request I wanted in fetch

url-search-params not work in Chrome 44:
    import URLSearchParams from 'url-search-params';
    const searchParams = new URLSearchParams();
    searchParams.set('userid', billUser);
    return fetch(`/sscpub/sckmsg/usermsghistory`, {
      credentials: 'include', //pass cookies, for authentication
      method: 'POST',
      headers: {
        'Accept': 'application/json'
      },
      body: searchParams
    })

So I use the method provided by stevensanborn , it works perfect.
    let billUser = '123';
    let bodyStr = "userid=" + encodeURIComponent(billUser);
    return fetch(`/sscpub/sckmsg/usermsghistory`, {
      credentials: 'include', //pass cookies, for authentication
      method: 'POST',
      headers: {
        'Accept': 'application/json',
        'Content-Type': 'application/x-www-form-urlencoded; charset=utf-8'
      },
      body: bodyStr
    })

I've been struggling with this too.
The issue seems fairly random with some requests needing the credentials hack and others not. Below is one of my recent configs with some dummy data.
It seems what was tripping up my code is that OPTIONS responses are coming through the promise, I would expect only POST responses to do this? Please forgive me if I'm wrong.
My current work around is to catch the 204 (no content) response code sent by express and prevent it interfering with the flow of my app. Any ideas on how to improve this are most welcome.
fetch(${apiServer}/v1/some/path, {       method: 'POST',       headers: {         Accept: 'application/json',         'Content-Type': 'application/json',       },       body: JSON.stringify({         someBodyProperty,       }),       credentials: 'same-origin',     })   .then(response => {       if (response.status >= 400) {         throw new Error('API Server Error')       }       if (response.status === 204) {         return {           status: 'options'         }       }       return response.json()   })   .then(({ status, data }) => {       if (status === 'success') {         dispatch({ type: types.SUCCESS, data })       } else if (status !== 'options') {         dispatch({ type: types.ERROR })       }     })

Regarding @tracker1 response I think you should use URLSearchParams instead of FormData when using application/x-www-form-urlencoded;charset=UTF-8 as Content-Type header, as indicated in the fetch documentation or the fetch specs

mark

const  dat = {
'grant_type': 'authorization_code',
          'code':code ,
           "redirect_uri" : redirect
           };

const URLSearchParams = Object.keys(dat).map((key) => {
return encodeURIComponent(key) + '=' + encodeURIComponent(dat[key]);
}).join('&');
fetch('https://polarremote.com/v2/oauth2/token',
{
method: 'POST',
headers: {
'Authorization':Basic ${creds},
'Content-Type': 'application/x-www-form-urlencoded',
 'Accept':'application/json; charset=utf-8'

},
data: URLSearchParams
}).then(function(res) {
return res.json();
}).then(function(body) {
console.log(body);
});

When body typeValue=%Y-%m the value %Y-%m can't send to server
in Chrome network  formData typeValue: (unable to decode value)


(fix) Widget::getId() generates same identifiers when ajax request

Q
A

Is bugfix?
✔

New feature?
❌

Breaks BC?
❌

Tests pass?
✔

Fixed issues

EN: Widget::getId() generates same identifiers when ajax request, because counter was reset to zero every request. This creates conflicts when JavaScript/jQuery linking to widgets identifiers, bu they already exists on page.
RU: Widget::getId() генерирует одинаковые идентификаторы при AJAX-запросах, так как при каждом запросе счётчик сбрасывается в ноль. Это создаёт конфликты, когда JavaScript/jQuery завязывается на идентификаторы виджетов, но они уже есть на странице.

The issue is known. In case of asynchronous calls you have to set IDs explicitly. We can not merge the pull request into Yii 2 because that would break too many applications relying on sequences in auto-generated IDs (I know it's not a good idea in first place but there are many).

So, if I has ajax-page with hundred widgets (buttons, sliders, etc), I need to specify ids all of that?
Why do not mark this bugfix as breaking BC?

I think that rely to an auto-generated ID is a bad development style.
So, such developers will have a good chance to fix this.

Why do not mark this bugfix as breaking BC?

Because Yii 2.0 doesn't accept breaking changes anymore (for about 2 years) except security or PHP 7 compatibility.

I think that rely to an auto-generated (!) ID is a bad development style.

It certainly is but that's how it is any many many projects are to be broken if the change is applied.


On Styled-Components 3, ServerStyleSheet.collectStyles creates a bunch of duplicate classes that crash browser

System:

OS: macOS Sierra 10.12.6
CPU: x64 Intel(R) Core(TM) i7-4770HQ CPU @ 2.20GHz
Free Memory: 510.73 MB
Total Memory: 16.00 GB
Shell: /bin/bash - 3.2.57

Binaries:

Node: 9.5.0
Yarn: 1.5.1
npm: 5.6.0
Watchman: Not Found

npmPackages:
babel-plugin-styled-components:

wanted: ^1.5.1
installed: 1.5.1

styled-components:

wanted: ^3.2.1
installed: 3.2.3
`

Reproduction
https://github.com/zeit/next.js/tree/canary/examples/with-styled-components
Steps to reproduce
--Clone above
--Upgrade to styled-components 3+
--npm install or yarn install
--Run app and examine the DOM, the same class will be repeated over and over again
The relevant lines are:
    const sheet = new ServerStyleSheet()
    const page = renderPage(App => props => sheet.collectStyles(<App {...props} />))
    const styleTags = sheet.getStyleElement()

Expected Behavior
Classes are only inserted once
Actual Behavior
Classes repeat over and over again. There must be some limit as it isnt an infinite loop but if you have enough elements on a page it will eventually crash your browser and take > a minute to load. This happens on production as well as on development.
This doesn't happen on styledComponents 2, only on 3.
Thanks, and let me know if I can provide any more information!

I modified one of our tests and can not reproduce this behaviour unfortunately

  it('should not spill ServerStyleSheets into each other', () => {
    const A = styled.h1`color: red;`
    const B = styled.h1`color: green;`

    const sheetA = new ServerStyleSheet()
    renderToString(sheetA.collectStyles(<A />))
    const cssA = sheetA.getStyleTags()

    const sheetB = new ServerStyleSheet()
    renderToString(sheetB.collectStyles(<B />))
    const cssB = sheetB.getStyleTags()

    expect(cssA).toContain('red')
    expect(cssA).not.toContain('green')
    expect(cssB).not.toContain('red')
    expect(cssB).toContain('green')
+
+    const sheetC = new ServerStyleSheet()
+    renderToString(sheetC.collectStyles(<B />))
+    const cssC = sheetC.getStyleTags()
+
+    expect(cssB).toEqual(cssC)
  })

I'm also running a Next.js setup on a fresh project (created yesterday) and am also not seeing this behaviour there.
Can you please provide some code snippets of how you're using styled-components and maybe some of those accumulating SSR CSS tags/rules you're referring to?
I think it might be a usage issue? Let's have a look :smile:

Ok thanks, I'll get a better reproduction going and post it here

@realalexhomer any news? :)

Hey thanks, I was super busy this weekend but I will get it going today
hopefully!
On Tue, Mar 20, 2018 at 10:04 AM, Phil Plückthun<EMAIL_ADDRESS>wrote:

@realalexhomer https://github.com/realalexhomer any news? :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/styled-components/styled-components/issues/1615#issuecomment-374609327,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIvoQdINwKEqskaFCXV3Ectp8aYAn9-6ks5tgQxdgaJpZM4SsyXz
.

Not sure which one did it but this bug is gone after I upgraded:

babel-plugin-styled-components from 1.4.0 to 1.5.1
styled-components from 3.2.1 to 3.2.5.

This problem can be solved by upgrading styled-components to latest version (fixed in 3.2.2).
We can close this issue.


[Feature] Phylogenetic tree

What problem does this feature solve?
Tree charts now cannot have a distance information for the particular edge. To draw not usual but phylogenetic tree such an information is used to depict distance between different sequences of the same gene in different organisms. Ideally such a tree should also be able to be constructed from a file in Newick format.
What does the proposed API look like?
myChart.showLoading();
$.get(ROOT_PATH + '/data/asset/data/phylogenetic-tree.nwk', function (data) {
  myChart.hideLoading();
  myChart.setOption(
    (option = {
      tooltip: {
        trigger: 'item',
        triggerOn: 'mousemove'
      },
      series: [
        {
          type: 'tree',
          data: [data],
          left: '2%',
          right: '2%',
          top: '8%',
          bottom: '20%',
          symbol: 'emptyCircle',
          orient: 'vertical',
          expandAndCollapse: true,
          label: {
            position: 'top',
            rotate: -90,
            verticalAlign: 'middle',
            align: 'right',
            fontSize: 9
          },
          leaves: {
            label: {
              position: 'bottom',
              rotate: -90,
              verticalAlign: 'middle',
              align: 'left'
            }
          },
          animationDurationUpdate: 750
        }
      ]
    })
  );
});

Hi, I see that there are many designs in Phylogenetic_tree. Which one is required? Like this one?

Would graph series be useful in this case? But you have to provide the position of each node.

It could be like this:

At least usually people get used to see such design. But the design itself can be different. For instance:

Or like this:

The main idea not to have different layout but rather have additional information on distance of a node from each other. That is why different nodes placed at different distances from each other on such trees.

The second point is to be able to load a tree from newick format.

@Ovilia This is something my organization would also find super valuable. Any thoughts on pursuing this in echarts?


Mac OS Monterey

Not an issue but I thought I'd let you know that all runs fine under MacOS Monterey 12.4

Thanks for the feedback!


account for richtext or keytext for contributor names

Turns out the prismic model is the issue. .name can either be RichText or Text. Once we change the type in the source data, the type system catches these errors.
The implementation around it isn't great, but when reading union RelationField types, seems it's all a bit fiddly as it greats a union of the type field and the data field but doesn't allow you to differentiate on it.
e.g.
if (agent.type === 'organisations') {
  agent.data // <= this isn't of type `Organisation['data']`.
}

For https://github.com/wellcomecollection/wellcomecollection.org/issues/7438


Add move base pos and ori to simulator

Add a function that allows users to call the resetBasePositionAndOrientation to move an entire URDF object without physics.

Completed with last release.


Testing template validations [Do not review]

Required items, please complete
Change(s):

See guidance below

Reason for Change(s):

See guidance below

Version Updated:

Required only for Detections/Analytic Rule templates
See guidance below

Testing Completed:

See guidance below

Checked that the validations are passing and have addressed any issues that are present:

See guidance below

Guidance <- remove section before submitting

Before submitting this PR please ensure that you have read the following sections and filled out the changes, reason for change and testing complete sections:
Thank you for your contribution to the Microsoft Sentinel Github repo.

Details of the code changes in your submitted PR.  Providing descriptions for pull requests ensures there is context to changes being made and greatly enhances the code review process.  Providing associated Issues that this resolves also easily connects the reason.

Change(s):

Updated syntax for XYZ.yaml

Reason for Change(s):

New schema used for XYZ.yaml
Resolves ISSUE #1234

Version updated:

Yes
Detections/Analytic Rule templates are required to have the version updated

The code should have been tested in a Microsoft Sentinel environment that does not have any custom parsers, functions or tables, so that you validate no incorrect syntax and execution functions properly.  If your submission requires a custom parser or function, it must be submitted with the PR.

Testing Completed:

Yes/No/Need Help

Note: If updating a detection, you must update the version field.

Before the submission has been made, please look at running the KQL and Yaml Validation Checks locally.
https://github.com/Azure/Azure-Sentinel#run-kql-validation-locally

Checked that the validations are passing and have addressed any issues that are present:

Yes/No/Need Help

Note: Let us know if you have tried fixing the validation error and need help.

References:

Guidance for Detection checks
General contribution guidance
PR validation troubleshooting

Hi @devikamehra @rushriva @vakohl, please provide feedback on this. Thanks

Hi @devikamehra @rushriva @vakohl, could you please provide your feedback on this. Thanks

Hi @Amitbergman, as this is a test PR and you have mentioned do not review can we label it as a Draft. Thanks

Hi @Amitbergman, as this is a test PR and you have mentioned do not review can we label it as a Draft. Thanks

Hi @Amitbergman, could you please provide your comment on above suggestion. Thanks

Hi @Amitbergman, waiting for your response. Thanks

Hi @Amitbergman, as this is a test PR and you have mentioned do not review can we label it as a Draft. Thanks

Hi @Amitbergman, waiting for your response. Thanks

Hi @Amitbergman, as this is a test PR and you have mentioned do not review can we label it as a Draft. Thanks

Hi @Amitbergman,  we wanted to check on the status of PR(https://github.com/Azure/Azure-Sentinel/pull/7395). PR is pending for more than 30 days. Please let us know if you need any assistance to review this PR. Per our standard operating procedures if no response is received in the next 7 business days, we will close this PR. Thank you for your cooperation.

Hi @Amitbergman, we wanted to check on the status of PR(https://github.com/Azure/Azure-Sentinel/pull/7395). PR is pending for more than 33 days. Please let us know if you need any assistance to review this PR. Per our standard operating procedures if no response is received in the next 4 business days, we will close this PR. Thank you for your cooperation.

Hi @Amitbergman, we wanted to check on the status of PR(https://github.com/Azure/Azure-Sentinel/pull/7395). PR is pending for more than 36 days. Please let us know if you need any assistance to review this PR. Per our standard operating procedures if no response is received in the next 1 business day, we will close this PR. Thank you for your cooperation


Blazor: nullable issues

Describe the bug
Converting null literal or possible null value to non-nullable type. (CS8600)
To Reproduce
Steps to reproduce the behavior:

dotnet new blazor
enable NullableContextOptions
insert the code:

@inject HttpClient httpClient

Warning: Converting null literal or possible null value to non-nullable type. (CS8600)

Expected behavior
Nothing happen
Additional context
Add any other context about the problem here.
Include the output of dotnet --info
.NET Core SDK (reflecting any global.json):
 Version:   3.0.100-preview6-012264
 Commit:    be3f0c1a03

Runtime Environment:
 OS Name:     Windows
 OS Version:  10.0.18922
 OS Platform: Windows
 RID:         win10-x64
 Base Path:   C:\Program Files\dotnet\sdk\3.0.100-preview6-012264\

Host (useful for support):
  Version: 3.0.0-preview6-27804-01
  Commit:  fdf81c6faf

.NET Core SDKs installed:
  2.1.700 [C:\Program Files\dotnet\sdk]
  2.2.300 [C:\Program Files\dotnet\sdk]
  3.0.100-preview6-012264 [C:\Program Files\dotnet\sdk]

.NET Core runtimes installed:
  Microsoft.AspNetCore.All 2.1.11 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.All 2.2.5 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.App 2.1.11 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 2.2.5 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 3.0.0-preview6.19307.2 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.NETCore.App 2.1.11 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.2.5 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 3.0.0-preview6-27804-01 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.WindowsDesktop.App 3.0.0-preview6-27804-01 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]

To install additional .NET Core runtimes or SDKs:
  https://aka.ms/dotnet-download

Thanks for contacting us, @hez2010.
We'll look into this after 3.0 release, given it's not a blocker.

This is similar to https://github.com/aspnet/AspNetCore/issues/10919 but for the @inject directive. Very possible that fixing the other will fix this. I won't close as duplicate for now.

@inherits is also displaying CS8600. Does this issue fixes that as well?

This is fixed in preview 9 which is not available yet. Sorry for the confusion, due to an odd set of circumstances, the team is working a few weeks ahead of the public releases.


Can't clear Output

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

VS Code Version: 1.94.2
OS Version: Windows 11 24H2

Steps to Reproduce:

Clear the Output panel
Restart VSCode
As you can see in the video bellow, the old outputs still remain.

https://github.com/user-attachments/assets/af3eb715-1c8e-4717-aa48-8b9ecdda8ff3

/designed
Clear command clears the output for the session.


Support Z-Stack 2.6.x firmware

The new versions of Z-Stack firmware do not work with zigbee4java.  This is because the firmware does not support the ZDO_xxx_RSP commands.  This code translates from the ZDO_MSG_CB_INCOMING command to the ZDO_xxx_RSP commands.  It also fixes a few other problems that were exposed when using the new firmware.
Tested with the Z-Stack 2.6.2 ZNP firmware that is bundled with Z-Stack Home 1.2.2.  Support for the older firmware remains.

I have just added support for the CC2531ZNP-Pro-Secure_LinkKeyJoin.hex firmware in addition to the already supported CC2531ZNP-Pro-Secure_Standard.hex firmware.


When can it be compleately ready for deployment.

FEATURE REQUEST

Is there an open issue addressing this request? If it does, please add a "+1" reaction to the
existing issue, otherwise proceed to step 2.

Describe the feature you are requesting, as well as the possible use case(s) for it.

Indicate the importance of this feature to you (must-have, should-have, nice-to-have).

BUG REPORT

What were you trying to achieve?

What are the expected results?

What are the received results?

What are the steps to reproduce the issue?

In what environment did you encounter the issue?

Additional information you deem important:

Question is not clear, Mainflux is used today in commercial products and professional. You need to contact Mainflux Labs for commercial inquiries.


fix: dummy to trigger release (#2410)

Backport
This will backport the following commits from 2.x to 1.x:

fix: dummy to trigger release (#2410)

Questions ?
Please refer to the Backport tool documentation

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 2411

Questions ?
Please refer to the Backport tool documentation


Evaluate unlearned images.

Hello.
I want to Evaluate unlearned images(my own image).
What should I do?

Hello @Heidisnaps
Me too, any progress on that?


Change to not translate unique names

Same as #23964
The proper noun is translated in localized version. So, I want you to use a code fence not to translate.

@changeworld : Thanks for your contribution! The author, @bwren, has been notified to review your proposed change.

#sign-off


Can shiny app run on Azure Databricks?

It is not clear on documentation whether you can run a shiny app in azure databricks? I know we can develop shiny app on azure databrick, but it's not clear where to host or publish our apps after development? Do we need a separate server to host the shiny apps?
https://docs.microsoft.com/en-us/azure/databricks/spark/latest/sparkr/shiny

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

ID: 92c85ecd-0a81-ca06-7c61-cee928433e4d
Version Independent ID: 40e9f75f-dc87-9a81-cd80-c29b15d1533a
Content: Shiny on Azure Databricks - Azure Databricks
Content Source: databricks/spark/latest/sparkr/shiny.md
Service: azure-databricks
GitHub Login: @mamccrea
Microsoft Alias: mamccrea

@pmahmoudzadeh
Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

#reassign:mssaperla

Came across this issue for the same reason. It is unclear to me if shiny applications can actually be published, and shared from Azure Databricks.

https://stackoverflow.com/questions/64573528/sharing-a-shiny-application-with-the-members-of-my-workspace-in-databricks

We have filed a ticket in the Databricks bug tracking system: DOC-2994. We will update this Git issue when the ticket has been resolved. #label:"in-progress".

Yes, you can share Shiny apps from Azure Databricks. For considerations, see https://docs.microsoft.com/en-us/azure/databricks/spark/latest/sparkr/shiny-notebooks#share-shiny-app-url.
This issue is resolved in the Databricks source files and will be published soon. #please-close.

Only Microsoft employees can close issues or pull requests via commenting in this repo.

#please-close

I dont think this is the correct solution. The issue is not about sharing the app. It is about using the shiny app parallely and enabling multiple user access at the same time by deploying it in the databricks cluster


Remove CLIF umulhi and smulhi instructions

Instead of supporting these instructions natively, which for example most source languages don't have direct support for (nor wasm), pattern-match these instructions during lowering. This commit replaces backend-specific lowerings of umulhi and smulhi with patterns in each backend in respective places. This helps detect when this happens from WebAssembly for example and helps simplify producers as well. The intent of this commit is that when desired the same instructions are emitted as before.
The downside of this commit is that CLIF is losing the ability to implement umulhi and smulhi on vector types. This was only implemented in the riscv64 and s390x backends and was not used by WebAssembly for example.

The main motivation for this is that in thinking about https://github.com/WebAssembly/128-bit-arithmetic I'm hoping to be able to add a "multiply wide" instruction for 64x64->128 but also wanted to retain the ability to generate just umulh on aarch64 for example. Without adding new opcodes to wasm that requires pattern matching at the underlying lowering level anyway, so my hope was to go ahead and start moving in that direction.

Cg_clif uses these instructions: https://github.com/rust-lang/rustc_codegen_cranelift/blob/7e4cafb653980b4dc1ca95eb67df8ac4608eab61/src/num.rs#L270

Thanks for pointing that out! While it should still be possible to get the same performance/semantics with this PR, the more I think about this the more I've started to sour on the idea. One of the other main motivations for this PR was that the ISLE optimization today to pattern-match an open-coded umulhi doesn't work for i128 when isplit is used to acquire the top bits instead of a shift-then-reduce. It might be best to work on that instead and keep these instructions as-is for produces to use and from wasm it'd mostly be the ISLE optimizations that create the instructions.

Ok I'm going to close this in favor of https://github.com/bytecodealliance/wasmtime/pull/9215 since I now think that's the better approach.


added simple support for browser platform (always has permissions)

I was missing the support for browser during development, so I did this very simple version that always give us any requested permission.

Can you also add iOS to use this?

Hello @NeoLSN, I have added that always-granted behavior to iOS too. But be aware that it's not tested yet.

Thx. I'll try to merge this this weekend.

Many thanks. Now, it is already in v1.0.0


Rich output

Shows #step and processing time on the output in the training.
It will help estimating ETA.

This is nice to have. Thank you!


feat: update auro-library to 3.0.1

Alaska Airlines Pull Request
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Resolves: #202
Summary:
This updates to the latest auro-library version and ensures the documentation paths are updated in index.md and api.md to match!
Type of change:
Please delete options that are not relevant.

[x] New capability
[ ] Revision of an existing capability
[x] Infrastructure change (automation, etc.)
[ ] Other (please elaborate)

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team
Summary by Sourcery
Update auro-library to version 3.0.1 and adjust documentation paths to align with the new directory structure.
New Features:

Update auro-library to version 3.0.1.

Documentation:

Update documentation paths in index.md and api.md to reflect new directory structure.

:tada: This PR is included in version 2.2.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


App Crashes

New Issue Checklist

[x] Use the latest version of CalendarKit
[x] I've read the Contribution Guidelines
[x] I've searched for existing GitHub issues

Issue Description

Hi @richardtop  I found a bug (maybe not) when I add invitee from ios calendar and go to calenderkit view and click that event app directly crashes or when I click add invitee in the app in calender kit view it directly crashes. Is there a way to solve it or a way to remove that section that says add invitee? Even I remove the add invitee in the calender kit view if a user has added an invitee from ios calender it will keep crushing. do you have a solution for that? Any document or walktrough video smth?
Code I'm using with CalendarKit

<!-- INSERT CODE HERE -->

Result I am trying to achieve

Could you please share a code and a video?

https://user-images.githubusercontent.com/109156022/185733637-63a2d634-45d1-4e8f-ad68-0c97ff767a5a.MP4
@richardtop I added my calenderview class all codes(only addition is uploading calender events considering the code you shared on youtube) and here is record

I cannot see the record, looks like it's an image. Also looks like the right repository to add this issue is to CalendarApp and not CalendarKit

It's a video?  Could you please confirm you see it?

Yep, I can see it. It's a CalendarApp bug, not CalendarKit's


feat: precise positioning and configurable up-axis

Fixes the issues related to imprecise or jittery positions and adds support for selectable up-axis.
The old version of the lat/lng to world-space conversion functions only worked properly when the
origin of the scene was positioned at {lat:0,lng:0}. But that leads to numeric problems (distorted
meshes and jittery motion) due to excessively large values for the world-space x/y coordinates.
The implementation of latLngToVector3Relative() that could be used to compute positions relative
to the anchor-point didn't account for the mercator-scale factor, which leads to very imprecise
positions in higher latitudes.
Additionally, we want the user to be able to choose the up-axis they want to use, as there are
good reasons for both the y-up and z-up orientation, depending on the use-case.
In order to handle the coordinate translations properly at all latitudes and for any world-space
orientation, the conversion functions have to be aware of the current anchor-position and upAxis.
For this reason, the conversion function latLngAltitudeToVector3 needs to be a method of the
overlay-class instead of a utility function. The utility functions in util.ts should be
considered internal and no longer be used outside the library.
This commit also changes the way the orientation is specified. While before the raw Float32Arrays
for the CoordinateTransformer are specified to the constructor, we now only use a single option,
upAxis that can be specified as 'Y' or 'Z', or a Vector3 to describe the axis to be used as "up".
BREAKING CHANGE: removed, renamed and refactored exported functions from utils.ts, changed shape of options (removed and replaced parameters)

[x] Ensure the tests and linter pass
[x] Code coverage does not decrease (if any source code was changed)
[x] Appropriate docs were updated (if necessary)

Fixes #618 🦕

Note, this PR depends on #635 and #637, and might need to be rebased before merging once the other two have been merged. To review it independently, you can use the single commit view: https://github.com/googlemaps/js-three/pull/638/commits/0de02f1cb00fba28f44b6831d49cd48e6c9b63f6


[Feature Request] Customization to enable full-height AppShell

Usage example, including component, action, motion, or utility API
AppShell currently generates some intermediary components that can't be easily influenced to make it work for a fairly common full height admin page usecase, requiring me to add global styles.
To get an AppShell that properly takes up the whole page (which is what I assume people want 90%+ of the time) I need the following global CSS:
 /* Target the first inner wrapper container inside the AppShell */
 .app-shell > div {
   height: 100%;
   display: flex;
   flex-direction: column;
 }

 /* Target the wrapper div *for content* which is two layers down inside the AppShell */
 .app-shell > div > div.body {
   flex-grow: 1;
 }

Maybe I've missed what the proper way to do this?
Another pain point was that the AppShell sample code wasn't quite correct (it would be much better to write the actual content that produces exactly that output rather than using the <NavbarContent /> component as shorthand)
Possible implementation - describe how the feature can be implemented
Adding some extension points to the Box used in AppShell, allowing it to be full height (or more generally have classes attributable to it)
Do you want to contribute this feature and create a pull request
No

At this point I think I have a good idea of how to fix this, but I'd like to get some input on how this project prefers to solve this -- there's a little bit of style machinery that is a bit intense.
Seems like this is as easy as adding some props to the Box, but would like some input

Hi @t3hmrman, thank you for this issue, you raise some valid problems and some good solutions.
The current style API is not yet final, there are still some kinks we have to work through. You can always open a PR with your proposed solution, and we can discuss that there. Meanwhile, I'll also explore the problem.
Regarding the documentation, it's ongoing, we simplified the more verbose code so that it is easier to understand the component usage, however you can always consult the full code in the demos package. Perhaps it would be a good idea to add the link to the respective demo in each example block, so that there is a faster way to see the full code

Hey @BeeMargarida thanks for the note -- I'll dig around the demos package a bit more.
I was able to get by with the global code above (I forgot to include the global attribute, will update the original ticket) -- so I think I'm OK for now I think. I'm glad there wasn't anything I missed!

Until the styling API is finalized (or even after?) maybe it's reasonable to include this styling workaround (global style + accurate CSS tareting) as a part of the documentation? I wouldn't mind contributing that if it's a good idea.

Good idea, it could be added to the "Override Styles" section as one of the possible solutions. Would really appreciate if you want to contribute to that part of the documentation 😄

I seem to be confused as to what the issue is here. There are props available that allow you to modify the height of the AppShell, granted the props available aren't immediately available in the documentation (something we need to get done soon). Usually in a dashboard project the fixed prop will be your best friend on the AppShell component's. Maybe I'm missing something here though 🤔

What I was trying to get wasn't keeping the header fixed, but actually getting the AppShell to expand to be full height of the page, no matter the size of the content. It's a classic sticky footer/app layout problem.
height=100% works if the container is also 100% (going up to body & html), but the problem was there is a <div> in between the outer div that is the AppShell and the actual content (pretty sure it's this Box on line 37.
Here's what the HTML looks like in the browser for me:

That in-between div *does not* have any classes or styles applied to it by default (and it cannot, with the way the code is currently written), but that div also needs `height: 100%` for the heights to cascade through properly.

In my case I found that `div` produced by the `Box` inside the `AppShell` component.

I'd love to know if I did something wrong or there was a way to achieve it *without* these overrides -- if you just try and create an `AppShell` that takes up the full height of a page (with the footer being at the bottom of the page, not being half-way up due to lack of content to push it down), then I think you might see what I ran into.


-recursion only works with URLs that end with FUZZ

I am trying to fuzz the site with a few directories and subdirectories. The directories return 404 if you visit them without the trailing slash:
example.com/configs - 404
example.com/configs/ - 200

Could it be possible to add ability to recursively fuzz URLs that end with a slash?

Unfortunately this is a corner case that would require specific logic applied to it. While it's possible to do, it would be confusing for some users and error prone (as we'd have to treat 200 answers as we treat 3XX redirects in the typical scenario).
I'm try to avoid creating complex rules within the ffuf features to make it as generic as possible. That said, it's possible to wrap ffuf within a few lines of shell script to achieve the same functionality without modifying the internals for this corner case. It would actually be a good fit for https://github.com/ffuf/ffuf-scripts/
If you (or anyone else) would like to tackle trying to create it, it would be most welcome addition. If not, I'll try to do it when I find the time for it.


WFLY-9869 Server to server EJB over HTTP invocations do not work due …

https://issues.jboss.org/browse/WFLY-9869

retest this please


New Lua errors since last update on all toons and Yes they are level 110

4x Nerdpack-Zylla\classes.lua:307: attempt to index local 'spec' (a nil value)
Nerdpack-Zylla\classes.lua:307: in function <Nerdpack-Zylla\classes.lua:305>
(tail call): ?
Nerdpack-Zylla\fakeunits.lua:6: in function <Nerdpack-Zylla\fakeunits.lua:5>
Nerdpack-Zylla\fakeunits.lua:92: in function ?' NerdPack\system\fakeunits.lua:28: in function <NerdPack\system\fakeunits.lua:24> NerdPack\system\fakeunits.lua:53: in function <NerdPack\system\fakeunits.lua:41> NerdPack\system\fakeunits.lua:65: in function Process'
NerdPack\system\fakeunits.lua:50: in function <NerdPack\system\fakeunits.lua:41>
NerdPack\system\fakeunits.lua:65: in function Process' NerdPack\system\fakeunits.lua:74: in function Filter'
NerdPack\system\parser.lua:95: in function <NerdPack\system\parser.lua:93>
(tail call): ?
NerdPack\system\parser.lua:111: in function `func'
NerdPack\system\parser.lua:101: in function <NerdPack\system\parser.lua:93>
(tail call): ?
NerdPack\system\parser.lua:195: in function <NerdPack\system\parser.lua:180>
SharedXML\C_TimerAugment.lua:16: in function <SharedXML\C_TimerAugment.lua:14>
Locals:
InCombatSkipped
I get errors on all toons that I use Zylla for CR Like this one. I just downloaded the update today a few min ago.

Debugging as we speak :)

This is happening because you're using one of the "FakeUnits" in Settings, and a function within WoW that should report your SpecID is not returning your SpecID, but it's returning 0.
I've made a ticker that checks this value every second.
Which should fix the problem.
If it happens again, just /reload
I'll see if i can add another better fix in the future.

Ok ...will do

Ok I just was playing my DH. I have not played it since I posted the lua error a few days ago and I am getting the same lua error that I got before....I dont see in the options in the Zilla's combat Routines an option to toggle FakeUnits on of off, where do I turn it off. I also looked in the nerdpack settings and dont see an option there either. I love the CR you have created. I Just wish I could use it.
this is the lua error I am getting
7x Nerdpack-Zylla\classes.lua:307: attempt to index local 'spec' (a nil value)
Nerdpack-Zylla\classes.lua:307: in function <Nerdpack-Zylla\classes.lua:305>
(tail call): ?
Nerdpack-Zylla\fakeunits.lua:6: in function <Nerdpack-Zylla\fakeunits.lua:5>
Nerdpack-Zylla\fakeunits.lua:92: in function ?' NerdPack\system\fakeunits.lua:28: in function <NerdPack\system\fakeunits.lua:24> NerdPack\system\fakeunits.lua:53: in function <NerdPack\system\fakeunits.lua:41> NerdPack\system\fakeunits.lua:65: in function Process'
NerdPack\system\fakeunits.lua:50: in function <NerdPack\system\fakeunits.lua:41>
NerdPack\system\fakeunits.lua:65: in function Process' NerdPack\system\fakeunits.lua:74: in function Filter'
NerdPack\system\parser.lua:95: in function <NerdPack\system\parser.lua:93>
(tail call): ?
NerdPack\system\parser.lua:111: in function `func'
NerdPack\system\parser.lua:101: in function <NerdPack\system\parser.lua:93>
(tail call): ?
NerdPack\system\parser.lua:195: in function <NerdPack\system\parser.lua:180>
SharedXML\C_TimerAugment.lua:16: in function <SharedXML\C_TimerAugment.lua:14>
Locals:
InCombatSkipped

I think I may have found error If I select nearest in the targeting I get the error if I select normal I dont get an error. so That must be the toggle you mentioned....One quick question...When I go to a target dummy and try to test my dps It wont attack on either of my toons that are 110. I have selected the Pref tallents and nothing. If I go out and attack a mob in game it is fine it auto attacks. Is this working as intended or is it a bug.

Those different targets you can select are the FakeUnits.
Using "Normal" will simply attack your current target.
I've been testing this quite thoroughly on both my Mage and Demon Hunter now.
I've also sent another update, that hopefully will fix the problem you had with the LUA error.
When you're trying to attack a dummy; are you using "Normal" as the selected target in the Settings?

Yes I was. I checked and was on normal....Do I need to change that?

Nah Normal is the same as using "target". Which should work on dummies as it doesn't use any custom unit stuff.


startingZIndex only works as global setting

For me at least, setting the startingZIndex only takes effect if it's done in the global settings e.g. in my main.js:
  aurelia.use.plugin('aurelia-dialog', (settings) => {
    settings.startingZIndex = 1032; // Go above Bootstrap navigation.
  });

Setting it in my specific DialogController doesn't work, although other settings such as settings.lock do work in that location.
If this is not a bug could you adjust the docs slightly to point this out?

Hmm I'm not 100% sure which is better.  Basically right now we stack everything on the starting globalSetting and just increment after that.  Might be best to document it for now.

Thanks!


another problem related to 'above' and 'below' virtual text

Steps to reproduce
I still see another problem, related to: https://github.com/vim/vim/issues/11959#issuecomment-1427589736

cursorline is displayed on both 'above' and 'below' virtual lines as well as actual one.
colorcolumn is displayed 'above' virtual line, not on actual one.

number and relativenumber are on and cursor is moved line 5 to line 1, the number is not properly redrawn.

Expected behaviour

cursorline is not displayed above and below virtual lines.
colorcolumn is not displayed 'above' virtual line.
the number is properly redrawn.

Version of Vim
9.0.1313
Environment
Windows 10
Logs and stack traces
No response

I noticed another problem. When an line includes non-ASCII character and "below" virtual line is drawn, screen looks OK but cursor can not be moved properly.
Left window is "above" and cursor can be moved to all lines. No problem.
Right window is "below" and cursor can NOT be correctly moved to line 2 and 3.
In this case, 0xA9 copyright sign is used but problem happens on other multibyte character as well.

Please use a separate issue for each problem.   Otherwise it is very hard to keep track of what has been fixed and what hasn't.
That 'cursorline' also highlights the "above" and "below" lines might be OK.  It also highlights the space after the text.
I'll see how to fix 'colorcolumn'.

Problem with multi-byte character fixed with patch 9.0.1327

Thank you for your quick fix and sorry for your confusion. I found all fixed except one. I will post it on another issue.


metagraph loading fails with 'No op named ImageProjectiveTransform' message

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Ubuntu 16.04
TensorFlow installed from (source or binary):
binary, installed with pip3
TensorFlow version (use command below):
v1.8.0-0-g93bc2e2072
Python version:
3.5

Describe the problem
I'm trying to load a metagraph file saved with tf-v1.4.0-rc1-11-g130a514. Loading fails with the following error:
  File "/home/vyal/.local/lib/python3.5/site-packages/tensorflow/python/training/saver.py", line 1955, in import_meta_graph
    **kwargs)
  File "/home/vyal/.local/lib/python3.5/site-packages/tensorflow/python/framework/meta_graph.py", line 743, in import_scoped_meta_graph
    producer_op_list=producer_op_list)
  File "/home/vyal/.local/lib/python3.5/site-packages/tensorflow/python/util/deprecation.py", line 432, in new_func
    return func(*args, **kwargs)
  File "/home/vyal/.local/lib/python3.5/site-packages/tensorflow/python/framework/importer.py", line 460, in import_graph_def
    _RemoveDefaultAttrs(op_dict, producer_op_list, graph_def)
  File "/home/vyal/.local/lib/python3.5/site-packages/tensorflow/python/framework/importer.py", line 227, in _RemoveDefaultAttrs
    op_def = op_dict[node.op]
KeyError: 'ImageProjectiveTransform'

I thought it's the result of some backward incompability so I retested on python2.7 and tf-v1.4.0-rc1-11-g130a514. Getting very similar message:
In [3]: tf.train.import_meta_graph('00000000000001107000.meta')
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-3-6d533993d6fe> in <module>()
----> 1 tf.train.import_meta_graph('00000000000001107000.meta')

/home/marin/tensorflow/lib/python2.7/site-packages/tensorflow/python/training/saver.pyc in import_meta_graph(meta_graph_or_file, clear_devices, import_scope, **kwargs)
   1808                                       clear_devices=clear_devices,
   1809                                       import_scope=import_scope,
-> 1810                                       **kwargs)
   1811   if meta_graph_def.HasField("saver_def"):
   1812     return Saver(saver_def=meta_graph_def.saver_def, name=import_scope)

/home/marin/tensorflow/lib/python2.7/site-packages/tensorflow/python/framework/meta_graph.pyc in import_scoped_meta_graph(meta_graph_or_file, clear_devices, graph, import_scope, input_map, unbound_inputs_col_name, restore_collections_predicate)
    658     importer.import_graph_def(
    659         input_graph_def, name=(import_scope or ""), input_map=input_map,
--> 660         producer_op_list=producer_op_list)
    661
    662     scope_to_prepend_to_names = "/".join(

/home/marin/tensorflow/lib/python2.7/site-packages/tensorflow/python/framework/importer.pyc in import_graph_def(graph_def, input_map, return_elements, name, op_dict, producer_op_list)
    283       # Set any default attr values that aren't present.
    284       if node.op not in op_dict:
--> 285         raise ValueError('No op named %s in defined operations.' % node.op)
    286       op_def = op_dict[node.op]
    287       for attr_def in op_def.attr:

ValueError: No op named ImageProjectiveTransform in defined operations.

Similar error when trying to load a meta file using C++ interface
FYI: https://github.com/tensorflow/tensorflow/issues/17942
           MetaGraphDef graph_def;         auto session = NewSession(SessionOptions());         status = ReadBinaryProto(Env::Default(), GRAPH_PATH, &graph_def);         status = session->Create(graph_def.graph_def());         if (!status.ok()) {                 **log_err("Error creating graph: %s", status.ToString());**                 return -1;         }
System information
TensorFlow build using make
built as a static library. **libtensorflow-core.a**

TensorFlow version (use command below):
v1.8.0-0

Below are the error message
Apr 10 16:17:22  E tensorflow/core/framework/op_kernel.cc:1242] OpKernel ('op: "MutableDenseHashTable" device_type: "CPU" constraint { name: "key_
dtype" allowed_values { list { type: DT_INT64 } } } constraint { name: "value_dtype" allowed_values { list { type: DT_DOUBLE } } }') for unknown op: MutableDenseHashTable
Apr 10 16:17:22: E tensorflow/core/framework/op_kernel.cc:1242] OpKernel ('op: "HashTableV2" device_type: "CPU" constraint { name: "key_dtype" all
owed_values { list { type: DT_STRING } } } constraint { name: "value_dtype" allowed_values { list { type: DT_DOUBLE } } }') for unknown op: HashTableV2
pr 10 16:17:22  E tensorflow/core/framework/op_kernel.cc:1242] OpKernel ('op: "LookupTableImportV2" device_type: "CPU"') for unknown op: LookupTa
bleImportV2
Apr 10 16:17:22: E tensorflow/core/framework/op_kernel.cc:1242] OpKernel ('op: "LookupTableImport" device_type: "CPU"') for unknown op: LookupTabl
eImport
Apr 10 16:17:22 : E tensorflow/core/framework/op_kernel.cc:1242] OpKernel ('op: "LookupTableExport" device_type: "CPU"') for unknown op: LookupTabl
eExport
Apr 10 16:17:22 : E tensorflow/core/framework/op_kernel.cc:1242] OpKernel ('op: "LookupTableSize" device_type: "CPU"') for unknown op: LookupTableS
ize
Apr 10 16:17:22 : E tensorflow/core/framework/op_kernel.cc:1242] OpKernel ('op: "LookupTableInsertV2" device_type: "CPU"') for unknown op: LookupTa
bleInsertV2
Apr 10 16:17:22 : E tensorflow/core/framework/op_kernel.cc:1242] OpKernel ('op: "LookupTableInsert" device_type: "CPU"') for unknown op: LookupTabl
eInsert
Apr 10 16:17:22E tensorflow/core/framework/op_kernel.cc:1242] OpKernel ('op: "LookupTableFindV2" device_type: "CPU"') for unknown op: LookupTabl
eFindV2
Apr 10 16:17:22 E tensorflow/core/framework/op_kernel.cc:1242] OpKernel ('op: "LookupTableFind" device_type: "CPU"') for unknown op: LookupTableF
ind
Have checked the tensorflow/contrib/makefile/tf_op_file.txt and all the necessary operation files are available.

I found the culprit. It was call to tf.contrib.image.rotate. Importing tf.contrib.image before loading the metagraph helps.
Looks to me like quite unintuitive behavior.

Thank you very much @akamaus for sharing the workaround!

Is it OK to close this issue? I'm not sure what TF would do differently to avoid the problem since ops are loaded on demand.

A warning saying something like 'perhaps you forgot to import contrib module' would have saved a day for me.

Nagging Assignee @michaelisard: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

This will be fixed once we automatically load op libraries from a directory instead of relying on (lazy) python imports to load them. I'm ok with adding a warning as you describe. Happy to approve a PR.

Please remove the assignee, as this issue is inviting external contributions. Otherwise, remove the contributions welcome label. Thank you.

@akamaus,
Could you please take a look at this comment from the developer and provide the PR for the same. Thank you!


Update task: Auto-tag products when another tag is added

https://tasks.mechanic.dev/auto-tag-products-when-another-tag-is-added
Add a manually triggered mode that looks at all existing products

fixed by: https://github.com/lightward/mechanic-tasks/issues/151


Stefan Heren- Final Project

Merging my final project to your repository.

This is a well-written project, overall. But I did have a hard time ploughing through all the unfamiliar terminology. It seemed at times that you went into some unnecessary detail in the write-up. For example: "a geostationary satellite has no inclination, so it sits at the same node all of the time” Huh? Why do we need to know this? My assessment is that the work would  have benefited from revision and editing (i.e., it reads like a good draft, not a final revision).
“As mentioned above, low earth orbits are determined by an orbital altitude of …” —> I don’t think it was mentioned above.
“we gain the benefit of having stability even with larger step sizes”—> compared to what? (you don’t mention explicit method until the end of the paragraph)
“a time step that large would certainly produce large amounts of error over time” —> in this passage, you’re mixing stability and accuracy
“and the next time step will be given by:” You have some LaTeX there that is not rendered correctly. I think you need a line break before the \begin{equation} command.
In your equations, use a backslash with trigonometric functions: \sin, \cos, etc., so they will be typeset in roman font and distinguishable from variables.
In Eq. (4), one of your denominators with cos(nu) is missing the backslash on \nu.
In Eq. (7), compared with Eq. (4), you have switched zeta into k. You do mention this in the passage above, but why not just use k all the time and avoid this?
Under Eq. (7), there is another bit of LaTeX that is not rendered correctly.
"I really wasn't expecting an oscillation in the error curves…” Well, your quantities of interest are periodic, and your error definition is the absolute difference. Is the error proportional to the magnitude of the quantity of interest? What if you were to normalize the error by that magnitude? (a possible problem would arise when the magnitude approaches zero, though).
The section on the System Tool Kit is interesting, but it did not connect with the rest of the project. You gave a nice video of the satellite’s orbit, but is there a way to get the state variables from STK so that you could’ve compared with the results obtained with implicit Runge Kutta? I was left wondering what the point was of this section.
Typos, grammar, etc.
… will destroy a satellite, which are typically multi-million dollar investments—> a satellite is
factors in it's design —> its
See:
http://www.its-not-its.info
Below is the orbital elements —> Below are
If one were to … they'd see —> agreement: “one” is singular
two body problem —> two-body problem
fourth order accuracy method —> fourth-order accurate method
lets set up —> let’s
occasional —> occasional
lets get —> let’s
an 313 Euler rotation –> a 313 Euler rotation
we aren't going to anything here —> do anything
lets get into —> let’s
lets plot —> let’s
everything was coded it properly —> delete “it”
high precision propagators —> high-precision (compound adjective)
tool to use the two-body problem —> on the two-body problem

Thank you for the feedback, Prof. Barba!
Should I update my project and commit it again?
Thank you,
Stefan
On Fri, Dec 25, 2015 at 4:41 PM, Lorena A. Barba<EMAIL_ADDRESS>wrote:

This is a well-written project, overall. But I did have a hard time
ploughing through all the unfamiliar terminology. It seemed at times that
you went into some unnecessary detail in the write-up. For example: "a
geostationary satellite has no inclination, so it sits at the same node all
of the time” Huh? Why do we need to know this? My assessment is that the
work would have benefited from revision and editing (i.e., it reads like a
good draft, not a final revision).
“As mentioned above, low earth orbits are determined by an orbital
altitude of …” —> I don’t think it was mentioned above.
“we gain the benefit of having stability even with larger step sizes”—>
compared to what? (you don’t mention explicit method until the end of the
paragraph)
“a time step that large would certainly produce large amounts of error
over time” —> in this passage, you’re mixing stability and accuracy
“and the next time step will be given by:” You have some LaTeX there that
is not rendered correctly. I think you need a line break before the
\begin{equation} command.
In your equations, use a backslash with trigonometric functions: \sin,
\cos, etc., so they will be typeset in roman font and distinguishable
from variables.
In Eq. (4), one of your denominators with cos(nu) is missing the backslash
on \nu.
In Eq. (7), compared with Eq. (4), you have switched zeta into k. You do
mention this in the passage above, but why not just use k all the time and
avoid this?
Under Eq. (7), there is another bit of LaTeX that is not rendered
correctly.
"I really wasn't expecting an oscillation in the error curves…” Well, your
quantities of interest are periodic, and your error definition is the
absolute difference. Is the error proportional to the magnitude of the
quantity of interest? What if you were to normalize the error by that
magnitude? (a possible problem would arise when the magnitude approaches
zero, though).
The section on the System Tool Kit is interesting, but it did not connect
with the rest of the project. You gave a nice video of the satellite’s
orbit, but is there a way to get the state variables from STK so that you
could’ve compared with the results obtained with implicit Runge Kutta? I
was left wondering what the point was of this section.
Typos, grammar, etc.
… will destroy a satellite, which are typically multi-million dollar
investments—> a satellite is
factors in it's design —> its
See:
http://www.its-not-its.info
Below is the orbital elements —> Below are
If one were to … they'd see —> agreement: “one” is singular
two body problem —> two-body problem
fourth order accuracy method —> fourth-order accurate method
lets set up —> let’s
occasional —> occasional
lets get —> let’s
an 313 Euler rotation –> a 313 Euler rotation
we aren't going to anything here —> do anything
lets get into —> let’s
lets plot —> let’s
everything was coded it properly —> delete “it”
high precision propagators —> high-precision (compound adjective)
tool to use the two-body problem —> on the two-body problem
—
Reply to this email directly or view it on GitHub
https://github.com/numerical-mooc/assignment-bank-2015/pull/15#issuecomment-167263708
.

Updating the project based on the post-deadline feedback is entirely optional.

Oh okay, well I'll update it anyway just to make it look better. Thanks!
On Sun, Dec 27, 2015 at 1:28 PM, Lorena A. Barba<EMAIL_ADDRESS>wrote:

Updating the project based on the post-deadline feedback is entirely
optional.
—
Reply to this email directly or view it on GitHub
https://github.com/numerical-mooc/assignment-bank-2015/pull/15#issuecomment-167432711
.


⚠ Samsung has degraded performance

In 63bd054, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2208 ms

Resolved: Samsung performance has improved in b8a3d94.


Added few Movie Quotes

@ChrisIIx branch has conflicts, please solve and pr again.

Hi @NikhilNamal17, sorry for conflict, should be resolved now. My Bad.


Correct copyright notices in NOTICE and source headers

The legal acceptance criteria had previously stated that all contributed code needed to represent that the Symphony Software Foundation was the copyright holder, but this was an unfortunate mistake - in virtually all cases, copyright remains held by the contributor of the project, and (via the CCLA / ICLA) is licensed to the Foundation.
In preparation for a v1.0 release of the Hubot ↔ Symphony integration, it would be ideal if the copyright notices in the NOTICE file and headers of all source code files were updated to show the correct copyright holder of the contribution.

That's done in https://github.com/symphonyoss/hubot-symphony/commit/d9a4dcc5c31077db1710a23c6dca3c0a2b50c774

Thanks @jonfreedman - love the speedy turnaround!  And apologies for the instructions previously being wrong.


kustomize overlay for running mac workloads

This is a setup I've been using for e2e tests with mac1.metal. Probably we should wait with merging this one until we have other changes in kip & itzo merged, so we can use non-dev versions of those two.

fyi @henryprecheur @ldx
manifest I'm using for testing:
apiVersion: v1
kind: Pod
metadata:
  name: buildkite-agent-mac1-metal
  labels:
    role: buildkite-agent-mac1-metal
  annotations:
    pod.elotl.co/instance-type: mac1.metal
spec:
  restartPolicy: OnFailure
  nodeSelector:
    type: virtual-kubelet
  tolerations:
   - key: virtual-kubelet.io/provider
     operator: Exists
  containers:
    - name: test-runner
      image: localhost:8089/3b1ecf0e-3441-44d2-8dc7-8635fbb390ed

image: localhost:8089/xcode12.2 probably will work as well, but haven't tested it yet.


Hotfix pong and Fixed hud

Updated HUD rendering, which fixed #123.
Also added a line to the pong demo to ensure it scales on resizing the window.
Feel free to review and merge @paked @matiwinnetou

Tahnks!


Codechef2

@1dharode ohh thanks sir, Sure I'll sub to your channel...and please help me for hactoberfest ...i have pulled 15+ requests from which 13 are showing excluded project...and 2 of them are showing...pull request not merged yet! Please help me to contribute in some good prs to make 3pull requests more..
THANK YOU


Makefile: install docs and add uninstall

extend the Makefile to install README & LICENSE as well and add uninstall
requested by a user of the AUR package - i'd like to have this upstream instead of messing around in the PKGBUILD ;)

Wonderful! Thanks for working on this. In the future a man page would be awesome too.


Agent

I've removed the -baseaspect option in javamopagent, and only keep that option in javamop.
I've inlined the contents of the default BaseAspect.aj in the .aj file generated by javamop if there is no user defined BaseAspect is provided.
I also modify the name of the aspect generated by javamop in -merge option so that it is consistent with the library generated by rv-monitor.
I've passed the mvn verify, and maybe @seriousamlqz we  can discuss how to compare the contents of agent.jar generated by previous javamop's -agent option and the one generated by our separate javamopagent tool in order to increase our confidence.

Thanks He, I will take a look at your changes.

Thanks He, I will take a look at your changes.

@xiaohe27 I madea some minor comments, overall it looks good to me.

@seriousamlqz  I've compared the agent jars generated by -agent option and javamopagent, and I found that only the aspect classes are different. However, if I use Intellij to view the java code from these class files, their contents are exactly the same. In addition to this, I also notice that in my test case, both the BaseAspect.class and HasNextMonitorAspect.class (generated by javamopagent) are 26 bytes less than their counterparts generated by -agent option. I'll do more experiments to understand the reason.

@seriousamlqz I've found the reason why the byte code of aspects differ in the agent.jars generated by javamop's -agent option and javamopagent. The directory from which we compile the .aj file can affect the generated byte code of the aspects. After using -d option in javamop and put the .aj file in exactly the same directory when I execute javamopagent, I made the contents of the two jars (unzip first) exactly the same (The binary jar files still differs though).

@xiaohe27 You can go ahead and remove the old agent generator, and make sure everything is OK (commandline option, document, etc). Then we can merge it.

The latest newAgent PR contains all the stuff in this PR.


Make the grid responsive

Initially, columns should drop rows at certain sizes. For example, a four-column row may become two two-column rows at mobile/tablet size. All of this should be easily-configurable through LESS variables.

Hi Rowan, Excuse my ignorance as I'm a little less than up to scratch on what's going on with all the git branches, but I was just having a snoop around and wondered why this ticket's here? Sounds exactly like what I commited days ago. :)

You did, and it's great! This is a personal project; nothing whatsoever to do with work, just me having a play :)

ha ha. that's less confusing :D
On 7 January 2012 17:57, Rowan Manning <
<EMAIL_ADDRESS>
wrote:

You did, and it's great! This is a personal project; nothing whatsoever to
do with work, just me having a play :)

Reply to this email directly or view it on GitHub:
https://github.com/rowanmanning/frag/issues/1#issuecomment-3397054


Remove PhysicalPressEventRouter

We have gotten feedback that
Interactable + InteractablePressEventRouter + Pressable Button is confusing for developers.
Specifically, they don't understand how the components work together, and often don't know they need to add components such as PhysicalPressEventRouter.
One way to simplify this is to remove one of the components: PhysicalPressEventRouter.
The PhysicalPressEventRouter is just a component that looks for an Interactable object and calls "OnClick" on the interactable.
Instead, we propose the following changes to
Add a "Forward clicks to interactable" checkbox to PressableButton
Add the dropdown to specify when to forward clicks to PressableButton
Remove PressableButtonEventRouter.
Note, that instead of actually looking for the router and calling OnClick directly, we can just raise the PointerClicked event from the button. Not sure yet what implementation is better.

Comment from Jonathan Palmer:
I agree this is a architecture conveyance problem and removing the overloading that the EventRouter causes.
I do not think removing it in entirety is the best step. If we do, we'd lose that level of abstraction of a dev wanting to have PressableButton sending events to non-Interactables, or if a dev wants to have it process in a different way then On Touch/Click/Press. Those were the main reasons for the EventRouter's original creation.
I think we want to fold the EventRouter into PressableButton's behavior as a private variable handler rather than a component on the gameobject. It continuing to be a component is a needless gotcha moment.
Devs could select a router through a custom inspector dropdown, defaulting to something like our current one.
This solves the overloading problem while maintaining the layer of abstraction which makes PressableButton more flexible.
This reduces inspector overload, especially if the UnityEvents could become foldouts and populated with the default routing cases at runtime.

Ported from internal database.

@killerantz do you have any thoughts/concerns on the proposal above? Do you think we need more information?

I like the PhysicalPressEventRouter because it shows a good pattern of setting state and pushing events through Interactable. I also agree that it is another component to add when there are others that sound like that functionality would already exist, like the PressableButton or the NearInteractionTouchable.
I want to consider the different paths one might build a button. we could start with PressableButton, add the NearInteractionTouchable (should this be required with pressableButton), then add Interactable to handle feedback for far interaction. It seems like a lot already. We could add the router functionality to PressableButton or NearInteractionTouchable, depending on which one is more UI feedback focused. My vote it pressableButton.
The other route is having a button that already works for far interaction or in MR, then we want to add PressableButton functionality. Again, adding the pressableButton (should also add NearInteractionTouchable), setup the visual feedback for press and everything else should continue to work.

I actually started moving some of that functionality to Interactable with PR #4357 , but that means some of the functionality is duplicated with pressableButton. The easiest thing would be to create a public method for click on Interactable and you can call it from the event of your choice on PressableButton. There are a couple public methods already, SendOnClick(IMixedRealityPointer) or OnPointerClicked(MixedRealityPointerEventData), but they both have parameters not supported by the UnityEvent parameter field.

Super old at this point, but wanted to tag some people as we're looking at what buttons look like in v3: @CDiaz-MS @Zee2 @RogPodge
Might be some interesting context in here for why things are the way they are and ideas for how we might improve. Might be something of relevance; might not!

A good direction would be taking all the View functionality, like themes and feedback, out of Interactable. Have a base button class that is only events, then we can add states, themes, pressable button triggers and other visuals on top. Drive button states based on the types of handlers the button processes, like grab, physical press, focus.
The state system (which drives themes) is a stub of what it could have been, the idea was to have something that's super configurable. If it's separate from the button, then we can route the events into the theme system to set any number of states, even non-button related states like collision, data updates. In the current setup the states drive the button events, maybe it should be the other way around to allow more flexibility.
The pressable button is a good template for setting up complex interactions. It just seems weird to have the PhysicalPressEventRouter when it seems like you should be able to link events from Pressable button directly into Interactable.
My wish list:

What types of events do we consume?
This could be done with the current pattern of adding components like NearInteractionTouchable or using check boxes - though the later would require code updates if a new type of interaction is added.

Are they pointer only events?
Speech commands can break Interactable if they are set to global and pointer events are expected to be local. (maybe worth removing from the button and use the voice command components.)
    - Though I would like to get events or state changes to show feedback like it was pressed by the speech command.
Global events are nice for designers to setup a click anywhere button. Maybe that's a different thing?

We can add tools/components to configure rules for interactions - like PressibleButton. Even a simple button has rules, like focus is required for press, some buttons have a time limit to trigger a click and allow a drag off to cancel.

The base button has states based on the handled events and rules. We can use these to drive feedback

Designers and engineers can wire up button events. OnFocus, OnClick, OnToggle (*more on toggles below)

A Theme system to provide feedback through animation, sound and visual changes.

Interactable tried to put all that into a single inspector component, maybe that was too ambitious, plus it also broke the pattern of adding components to change functionality. I was trying to balance designer vs. engineering needs and figured one complex component would be less of a learning curve than many simpler components even if we might have sacrificed some flexibility. After lessons learned, I would err on the side of consistent patterns and flexibility.
What is a button in MR? It could be a flat UI button on a slate that should react to near and far interactions or it could be a space capsule in orbit. I can grab it to inspect it, let it go or even throw it. It would be nice to add a throw state and throw event without having to write a new button system.
*A toggle could have more than two dimensions, though be mostly use selected and unselected. A speed control toggle could have 1x, 2x, or 4x as dimensions.
My favorite parts of Interactable are the themes, there are some good button rules (toggles) and the event/receiver system. Though for the event/receivers I would stick with the receiverBaseMonoBehaviour instead of the others ReceiverBase classes that can be detected by the Receiver system. It's nice to have an easy way to attach a new event type to the Interactable.

We have ended up with a pattern very similar to what @killerantz discussed in the previous comment, enabled through XRI in MRTK3.

Data/model-only component: PressableButton (or, the base class StatefulInteractable for non-buttony-things)
View/visualization: StateVisualizer

All inputs (physical touch, ray, etc) all route through the same "interaction model" component (interactable), and all visuals are displayed/rendered by StateVisualizer.


Check for build directory and create if necessary in  make build

Right now the build directory must be created manually.

Fixed


Microsoft.Git: update to 2.33.0.vfs.0.0

null
Microsoft Reviewers: Open in CodeFlow

Approved https://github.com/microsoft/winget-pkgs/pull/25387 since it blazed trough validation


/vscode no longer works

It requires a trailing slash where it did not before (was optional before).

Fixed via https://github.com/cdr/code-server/pull/4522

@TeffenEllis feel free to pick this one up if you want

Resolved via https://github.com/cdr/vscode/commit/43af6fc0c92bb211dbea9f8357698be330e749b1


Differentiate read and unread issues

It would be nice to see which issues have had new comments, and to mark them as read when it's the case, like it already exists in the web interface.
And to have it mark the issue as read on the web of course.

+1

Moved to #196


question

Hi , i m using cassandra.
Entity A{name:string,surname:string}
How can search a where name=x and surname=y in a huge big data table.
I found secondary index. If i set index for name and surname,can i use this indexes also together? (name and surname).
In the case i want make a fulltext seach , how can i ?

@publicocean0
Please refer PersonCassandraLuceneTest for info on querying.
Check the entity PersonLuceneCassandra for info on annotations.
Also, make sure you add the following line in your persistence.xml to use lucene as indexer:
<property name="index.home.dir" value="lucene" />

You can also use elasticsearch as an indexer.
Hope this answers your queries. Let us know if you have more queries.
-Karthik

lucene in the application can work just if you have a unique instance working. In my case i could   have multiple instances so i thought to use https://github.com/Stratio/cassandra-lucene-index.
I configured kundera with client datastax.
I will add in server lucene plugin so i can use lucene index in cassandra.
For this reason i d like add a annotation for adding CustomIndex in the entity.

@publicocean0
Can you explain more about your usecase?
We can achieve this indexing through createNativeQuery (Which will execute the CREATE CUSTOM INDEX query on Cassandra client). Since this indexing is one time thing, you can create this manually.
Let us know why you want a client specific annotation.
-Karthik

In all my app i always developped autoconsistent executable. All the info for executing it is inside in the code. Anuway now i have 2 more urgent issues in my app:
org.springframework.dao.InvalidDataAccessApiUsageException: Not an entity, {class:class com.itiboss.app.model.Language}; nested exception is java.lang.IllegalArgumentException: Not an entity, {class:class com.itiboss.app.model.Language}
app created schema in cassandra automatically but then gives this problem.

i m not expert in cassandra but this sounds strange:
cassandra@cqlsh:example> select * from example.LANGUAGE ;
InvalidRequest: Error from server: code=2200 [Invalid query] message="unconfigured table language"
cassandra@cqlsh:example> select * from example."LANGUAGE" ;
key | code | enabled | name | tcode
-----+------+---------+------+-------
(0 rows)

i found the reason kundera create schema as
CREATE TABLE example."LANGUAGE" instead of
CREATE TABLE example.LANGUAGE

about first bug Not an entity ... it is a problem with classloader.

select table_name from system_schema."tables" and select table_name from system_schema.tablesworks .... uhmmm what is the logic in this ?

In Cassandra, you have to specify case sensitive names in quotes. It will consider all names as small case if quotes are not used. So, LANGUAGE will be considered as language.

Hi @publicocean0,
Did you check https://github.com/impetus-opensource/Kundera/wiki/Using-Kundera-with-Spring?
-Dev


on main branch: command yarn workspace "@dm3-org/dm3-messenger-web"  start:dev : It no longer works

I tried to run  dm3-messenger-web : so i execute commands yarn & yarn build and i add .env file and type command
yarn workspace "@dm3-org/dm3-messenger-web"  start:dev
but i faced this error :

Uncaught TypeError: Cannot read properties of null (reading 'useMemo')
    at useMemo (react.development.js:1650:1)
    at Home (Home.js:38:1)
    at renderWithHooks (react-dom.development.js:16305:1)
    at mountIndeterminateComponent (react-dom.development.js:20074:1)
    at beginWork (react-dom.development.js:21587:1)
    at beginWork$1 (react-dom.development.js:27426:1)
    at performUnitOfWork (react-dom.development.js:26557:1)
    at workLoopSync (react-dom.development.js:26466:1)
    at renderRootSync (react-dom.development.js:26434:1)
    at recoverFromConcurrentError (react-dom.development.js:25850:1)

but when I  tried to run dm3-messenger-widget , using  command  yarn workspace "@dm3-org/dm3-messenger-widget"  start:example and saame .env file ,it run perfectly:

fixed by #1090

Thanks for reporting! @enghibrahem


typings: make MockFunction generic

this allows esmock to be used in a more convenient way in typescript setups
import esmock from 'esmock';
import type * as moduleUnderTest from './moduleUnderTest.js';

const { func } = await esmock<typeof moduleUnderTest>(
  './moduleUnderTest.js',
  import.meta.url,
  {
    //...
  },
);
const actual = await func();

Btw thank you so much for creating this module @bumblehead! :)

@uwinkelvos thanks for the thanks --that makes me glad to read that.
Its fine for me if @koshic publishes, but I'm going to publish now since it is not published. one moment...

@iambumblehead just want you do decide - it's not an emergency case, so )


Fix crash(es) in RSA_padding_add_PKCS1_PSS_mgf1

Checklist
Description of change
This patch adds a number of checks that ought to ensure that there is not a single addition or subtraction operation in RSA_padding_add_PKCS1_PSS_mgf1 that results in unwanted behavior.
For example, before this fix
openssl dgst -sigopt rsa_padding_mode:pss -sign private.pem  ....

results in a crash if private.pem is a serialization of an RSA structure whose "N" parameter is "1".

This affects both 1.0.2 and 1.1.0.

ping @levitte and @bernd-edlinger .  status?

fixed on master by 108909d30e
fixed on 1.1.0 by e653b6cd7
fixed on 1.0.2 by 04cf39207

Thanks for the info!  Closing.


[Peloton] Last cadence seen by Gymnasticon never clears if it is non-zero

Reported to me over FB Messenger, trying to dig in.
We appear to have an edge case where if the bike is no longer sending updates (i.e. ride screen left), if the last frame contained a non-zero cadence, we emit pedal stroke events at that given rate until the service is restarted.
Thinking through this, we already have the means to account for this -- the PingInterval.  We currently reset the clock on the pingInterval with every PedalEvent, but we might be able to move the reset into the onBikeStats handler rather than onPedalEvent since they are being generated synthetically.
The other option is to try to detect the frames sent (if there is anything unique) when the serial handler on the Peloton Tablet is started and destroyed, and match that lifecycle within Gymnasticon, but it feels more fragile than adjusting our pingInterval logic.

@ptx2 I'm happy to make these changes, but I don't have a way to test on non-Peloton hardware, and we don't have any real branching logic at that point in the application.
Before I go start crafting a PR, I'd love to make sure the intended fix is in line with your mental model of the situation as well.

@ptx2 I'm happy to make these changes, but I don't have a way to test on non-Peloton hardware, and we don't have any real branching logic at that point in the application.
Before I go start crafting a PR, I'd love to make sure the intended fix is in line with your mental model of the situation as well.

Hey @jeremydk thanks for finding this! I'll try this on the Flywheel today.

Hey @jeremydk thanks for finding this! I'll try this on the Flywheel today.


Update external-domain-name-system-records.md

Remove SFB online from the note. as the DNS records also apply for teams and SFB hybrid scenario

Issue#https://github.com/MicrosoftDocs/OfficeDocs-Enterprise/issues/936

@JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks

@kelleyvice-msft Please merge this PR. Thanks.


Change curriculum link on maths page

Trello card
https://trello.com/c/FQgwn44F
Context
Changing the link to the national curriculum to go to the top of the page rather than the key stage 3 section, to be consistent with other subject pages

Closing this - not using latest maths page


fix(paperwallet): fix order limit for buy order

What have you changed and why?
The paper wallet buy logic is inverse. A buy order limit is executed if the price is bellow the limit.
Related Issues
Closes I was confusing the order with SideSell but it's SideBuy

Codecov Report

Merging #133 (f0f9d42) into main (1d1614f) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #133   +/-   ##
=======================================
  Coverage   49.57%   49.57%
=======================================
  Files          15       15
  Lines        2255     2255
=======================================
  Hits         1118     1118
  Misses       1073     1073
  Partials       64       64

Impacted Files
Coverage Δ

exchange/paperwallet.go
66.04% <100.00%> (ø)

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 1d1614f...f0f9d42. Read the comment docs.


Change express requirement to <v4

The example app's express config is not compatible with Express v4+. I have modified the package.json to pull up to the latest v3. This should be considered a temp fix. Plan to update app.js to be compatible with the most recent express version.

Fixed


[Enhancement] MultiConverter

Summary
This is a converter that allows for multiple converters to be applied in series. I already made some code for this in the multiconverter branch. I would, however, like to have a discussion on whether or not this is useful or how we can improve upon it.
API Changes

Add a MultiConverter

Intended Use Case
Use it as follows:
<xft:MultiConverter x:Key="myMultiConverter">
    <xft:IntToBoolConverter />
    <xft:BoolToObjectConverter />
</xft:MultiConverter>

And in XAML:
<MyElement Property="{Binding X, Converter={StaticResource myMultiConverter}, ConverterParameter='Y'}" />

Code
public class MultiConverter : List<IValueConverter>, IValueConverter
{
        public object Convert(object value, Type targetType, object parameter, System.Globalization.CultureInfo culture)
            => this.Aggregate(value, (current, converter) => converter.Convert(current, targetType, parameter, culture));

        public object ConvertBack(object value, Type targetType, object parameter, System.Globalization.CultureInfo culture)
            => throw new NotImplementedException();
}

Discussion
The problem I have for now is that you can only pass in one parameter on the MultiConverter level that gets pushed down into the separate converters. Other than that the chaining of values would work fine in this scenario.

I believe x:Array can be passed as a parameter to MultiConverter.

I like this idea very much

I believe x:Array can be passed as a parameter to MultiConverter.

So you would provide an array with 4 items if you chain 4 converters together and we grab them by index? There's not really a different way to indicate which parameter goes into which converter otherwise. Not sure if that is too much magic spaghetti for my liking though 😓  I will experiment a bit with some options.

@AndreiMisiukevich How do you feel about something like the following:
<ResourceDictionary>
    <xct:MultiConverter x:Key="myMultiConverter">
        <xct:IntToBoolConverter x:Name="intToBool" />
        <xct:BoolToObjectConverter x:Name="boolToObject" />
    </xct:MultiConverter>
    <x:Array x:Key="multiParams" Type="xct:MultiConverterParameter">
        <xct:MultiConverterParameter Converter="intToBool" Value="{Binding x}" />
    </x:Array>
</ResourceDictionary>

Then in the converter we can pass in this array as the root parameter and the converters can go grab the parameter from the array if their name exists in there.

Upon further exploration I might need to do a type-based check as the x:Name is obviously not available in the converter.

@AndreiMisiukevich How do you feel about something like the following:
<ResourceDictionary>
    <xct:MultiConverter x:Key="myMultiConverter">
        <xct:IntToBoolConverter x:Name="intToBool" />
        <xct:BoolToObjectConverter x:Name="boolToObject" />
    </xct:MultiConverter>
    <x:Array x:Key="multiParams" Type="xct:MultiConverterParameter">
        <xct:MultiConverterParameter Converter="intToBool" Value="{Binding x}" />
    </x:Array>
</ResourceDictionary>

Then in the converter we can pass in this array as the root parameter and the converters can go grab the parameter from the array if their name exists in there.

hmmm, can we working :)

We do need a sample for this in the sample app. And ideally tests (for all the things in here right now for that matter)

This one is resolved, I guess

Opened a new issue to add tests/sample for this so we can close this one


Cannot bind dropdown list value to a non primitive type with kendo-ui

Hello,
When you select a label in a DynamicSelectModel, the associated value can be either a primitive type (string, number, null) or a complex type (any object).
The Kendo-Ui implementation of ng2-dynamic-form only allows to return primitive types using this:
http://www.telerik.com/kendo-angular-ui/components/dropdowns/dropdownlist/value-binding/#toc-primitive-values-from-object-fields
If we have a model form such as DynamicSelectModel[], you will have an error at runtime when you select something in the dropdownlist ("Expected initial value of primitive type"). This is normal since in the current code, the field "[valuePrimitive]" is always set to true: https://github.com/udos86/ng2-dynamic-forms/blob/master/modules/ui-kendo/src/dynamic-form-kendo.component.html#L155.
Best regards,
Sébastien.

@SebLz Thanks for your message! Could you provide a sample model?
Until now I've not been aware that the primitive binding can cause problems since DynamicSelectModel expects DynamicFormOption[] as options which are bound under the hood.

@udos86 :
Here is how i initialize my form model (starting from a const myObject1FormModel):
ngOnInit() {         let options = sampleMyObject1s.map(v => {return {value: v, label: v.name}; }) as DynamicFormOption<MyObject1>[];         this.formModel = myObject1FormModel;         this.formModel.forEach(c => c.options = options);     }
For this to work with DynamicSelectControls and Kendo UI, I had to change [valuePrimitive] from "true" to "false" in https://github.com/udos86/ng2-dynamic-forms/blob/master/modules/ui-kendo/src/dynamic-form-kendo.component.html#L156. This is because I am not trying to return one of the property (that is [valueField]) of the object, but the whole object itself. This is again a behavior specific to Kendo UI.

@SebLz Thanks for giving some details on that!
I'll have to examine it further? Maybe you have a proposal on how to fix this?

@udos86 unfortunately, I don't :(. Although I haven't compared with all the other UI components, I suspect this option "valuePrimitive" is kind of specific to KendoUI. Therefore I'm not too sure how to "generalize" this at the level of the DynamicSelectControl.
As a quick fix, I changed the default value of [valuePrimitive] in the template file because I believe it's more common/convenient to return the whole object than only one of its property. For the next update and until a better solution can be found, I suppose the value of [valuePrimitive] should be set to either true or false depending on what is the most used popular choice among KendoUI users.

@SebLz Ok, but isn't it possible for you to just set a primitive as value property?
let myOptions = [
    {
        label: "Option 1",
        value: "option-1",
    },
    {
        label: "Option 2",
        value: "option-2"
    },
    {
        label: "Option 3",
        value: "option-3"
    },
    {
        label: "Option 4",
        value: "option-4"
    }
];

new DynamicSelectModel<string>(
    {
        id: "kendoDropDownList",
        label: "Kendo DropDownList",
        options: myOptions
        value: myOptions[2].value
    }
)

Whenever another value is selected by the user you can then remap it to the original options object:
mySelectModel.valueChanges.subscribe(value => {
    let selectedOption = myOptions.filter(option => option.value === value);
});

Of course it's a little overhead but I can't see any further problems here.

@udos86 You're right, it's probably the best solution thanks.


ensure noise and sky level is implemented correctly in images

For DC2 i-band coadds, if you plot the variance in each pixel to the corresponding image flux, you get approximately a straight line. And pixels far from bright objects have image flux values around 0. This corresponds to a simple image generation model where
a) all noise is Poisson-distributed in each pixel and uncorrelated between pixels,
b) the image flux has a uniform sky level subtracted off, so that all that remains is astronomical light + sky noise + shot noise, and
c) the remaining image flux is everywhere equal to a constant scale factor (the "gain") times the number of input photons to each pixel.
Under these assumptions, the number of astronomical photons per pixel should be photons = inst_flux / m, and the number of sky photons is sky_level = b / (m**2), where m and b are the slope and intercept of a straight line fit of variance vs. instrumental flux in each pixel. This allows one to generate a reasonable noise level per pixel (given a noiseless GalSim image that's been scaled to instrumental flux counts) using the following code:
rng = galsim.BaseDeviate(seed) # seed = whatever
image = image / m
noise = galsim.PoissonNoise(rng=rng, sky_level=b/(m**2))
image.addNoise(noise)
image = image * m

For a specific i-band coadd in DC2, the values of m and b are
m = 0.00015247276
b = 0.003510862


Periodic jobs not working/saving settings

Describe the bug
Periodic jobs not working when they should (everyday) or they don't even get their settings saved.
Screenshots (optional)
This one doesn't keep/save settings. If i select 'Monday' at 0900 and click save job, it keeps blank
GUI

Settings

This one saves/keeps settings but it is not triggered everyday. It runs fine manually.
GUI

Settings

Environment (please complete the following information):

OS: dietpi
Browser: any

Used app version:

docker-edge-bullseye 2.0.3 built at mar 12, 2024

Hit this bug also.  Running edge-debian-bullseye on Unraid.  config file is updating, but not saving correctly.  Same symptoms as @papayavice mentioned.  This is keeping indexing (and the rest of the normal jobs) from running daily.

I do have the same issue. The periodic scheduling is unfortunately not working. The setting is not getting applied after saving the jobs. I am currently running v2.0.3-edge built at 28.5.24.

I'm having the same bug and am using a temporary sort of fix by just running the tasks as an API request to the server, I used F12 in firefox to see the exact HTTP POST request and just copied it's curl command into a shell script that runs daily with crontab. It's not ideal but it does work so far.

Apologies for my late reply, I'll try to expand on the steps below:

Open your pigallery URL and go to the settings screen
Press F12 in firefox
Go to the network tab to activate it
Press start on the scheduled task you want to start
You'll see there is a POST and several GET's, you right click the POST and select copy as curl (POSIX)

In Linux you do crontab -e and add it at the bottom with a time cron schedule like here

Let me know if it worked or if you need more help


The damage calc does not match what is on Pokemon Showdown

Using the damage calc here: https://pokemonshowdown.com/damagecalc/
252 SpA Choice Specs Tapu Koko Grass Knot (120 BP) vs. 252 HP / 252+ SpD Groudon-Primal: 244-288 (60.3 - 71.2%) -- guaranteed 2HKO
252 SpA Choice Specs Tapu Koko Grass Knot (120 BP) vs. 252 HP / 252+ SpD Groudon: 244-288 (60.3 - 71.2%) -- guaranteed 2HKO
Note they are the same percentages for both Primal-Groudon and normal Groudon
Here is a replay showing this situation in an AG game during the first 2 turns: https://replay.pokemonshowdown.com/gen7anythinggoes-878492172
Here is a replay of the testing I did after: https://replay.pokemonshowdown.com/gen7anythinggoes-878501709
You can see that the damage is not the same at all. The damage calc is bugged.
Interestingly by selecting a non Primal Groudon set and modifying it you can see the calc display Groudon-Primal (even though this should be a calc for a normal groudon) and give the correct damage calculation.

This is not the repository for the damage calculator. You're looking for: https://github.com/Zarel/honko-damagecalc


Fix fill randomly not working in geom_bar + geom_histogram

This code randomly produces one of two graphs:
ggplot(diamonds, aes(x='clarity', fill='cut')) + geom_bar()

randomly good
randomly bad

Problem: geom._get_plot_args renames keys in mpl_params according to
_aes_renames, but not in a deterministic order. In particular,
geom_bar and geom_histogram both include renames:

color -> edgecolor
fill -> color

which when done in that order are fine but in the other order causes
color to get dropped.
Solution: use OrderedDict in geom_bar and geom_histogram to ensure
that the color -> edgecolor rename happens before the fill ->
color rename.

Good call, but there are two other occurrences of .items() in this file that this change doesn't touch, and this change only turns one existing .items() into another, so what do you think about calling six.iteritems out of scope for this PR to keep the diff minimal (e.g. easier rebasing)?

let us see what yhat people say. Opened issue #578.


Add cursor-project-manager extension

Description
This is a fork off the popular search in project manager and open with visual studio code extension. The goal is to alter this to open any project from the project manager in cursor. cursor is a fork off of the vscode repo rebuilt for AI coding assistant. Cursor
Screencast
https://github.com/user-attachments/assets/9ee18125-c9dd-4cab-acc8-ef6875b40268

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Instead of having a fork of the extension it's better to enhance the current one by @MarkusLanger - we do the same for the main Visual Studio Code to ensure that we don't have to update them both since they can do the same
Does that make sense?

We can look at an enhancement, however the one created was built for visual studio code specifically, I believe it could lead to a confusing user experience to alter the existing one.

In my extension it is possible to open the project in Cursor. In the configuration it is possible to switch to Cursor.

We could potentially improve the naming and descriptions to make them clearer. Having two identical problems will cause further complications.