Setup a code linter

@LePips raises a good point that we should probably simplify some things around code review by setting up some basic rules with a linting tool.
I'm suggesting SwiftLint since I'm familiar with it, but open to other options too.
We should be able to run it locally and also it should run as a check on Pull Requests.

I guess to get started, what rules do we want? A first autocorrect run doesn't produce much, only Array<...> to [...] and some whitespace removed.

That sounds like a good start. If we can start with the default rules that are enabled and correct everything then that is probably already good enough.
Maybe we can take some examples from other public style guides beyond that? Some that I know of:

https://github.com/airbnb/swift/blob/master/Sources/AirbnbSwiftFormatTool/swiftlint.yml
https://github.com/raywenderlich/swift-style-guide/blob/main/com.raywenderlich.swiftlint.yml

I started with the raywenderlich config and just removed some options to get to this config: https://github.com/LePips/CreateAPI/blob/swiftlint/.swiftlint.yaml
You can see what changes it made with a first autocorrect: https://github.com/LePips/CreateAPI/commit/80551e91b57d8c2ac1dbe984827615ea38be236a
Overall, mainly fixes spacing and changes Array<type> declarations to [type].

Looks like a good start!
We’ll want to ignore the redundant default value in ConfigOptions.swift because that is used to generate documentation, and we’ll also want to ignore the DecodableWithDefault extension since that’s generated code

Disabled the redundant rule for ConfigOptions.swift and all rules for the generated files: https://github.com/CreateAPI/CreateAPI/commit/91af76e716342b5af746048029eac46a6f0bbf87


Unable to Render Code blocks

@coreyward  I am attempting to render a code block and have passed in a code prop to the serializer but keep receiving the following error: Error in function BlockSerializer in ./node_modules/react-portable-text/node_modules/@sanity/block-content-to-hyperscript/lib/serializers.js:23 and Unknown block type "code", please specify a serializer for it in the serializers.types prop
My Code:
// Component Rendering Portable Text
 <PortableText
        content={content._rawPortableBody}
        serializers={serializers}
      />

// serializers.js
import React from "react";
import imageUrlBuilder from '@sanity/image-url'

let baseHeaderStyles = 'py-5 px-48'
let baseParaStyles = 'py-2 px-48'
let baseListStyles = 'list-inside px-52'

const urlFor = source => imageUrlBuilder({ projectId: '', dataset: 'production' }).image(source)

const serializers = {
    code: props =>
{JSON.stringify(props, null, 2)}
,
    h1: props =>
{props}
,
    h2: props => <h2 className={`${baseHeaderStyles} text-3xl`} {...props} />,
    h3: props => <h3 className={`${baseHeaderStyles} text-2xl`} {...props} />,
    normal: props =>
,
    ul: ({ children }) => <ul className={`${baseListStyles} list-disc`}>{children}
,
    ol: ({ children }) => <ol className={`${baseListStyles} list-decimal`}>{children}
,
    li: ({ children }) =>
{children}
,
    imageSection: props => (

            {/* {JSON.stringify(props, null, 2)} */}

    )
    // someCustomType: YourComponent,
}

export default serializers;

As you can see I have a code prop created inside my serializer file but for some reason, the error keeps being thrown. Any assistance you could provide would be great. Thanks/

Also have tried adding a types prop to the serializers object:
const serializers = {
    types: {
        code: props =>
{JSON.stringify(props, null, 2)}
,
    },
    h1: props =>
{props}
,
    h2: props => <h2 className={`${baseHeaderStyles} text-3xl`} {...props} />,
    h3: props => <h3 className={`${baseHeaderStyles} text-2xl`} {...props} />,
    normal: props =>
,
    ul: ({ children }) => <ul className={`${baseListStyles} list-disc`}>{children}
,
    ol: ({ children }) => <ol className={`${baseListStyles} list-decimal`}>{children}
,
    li: ({ children }) =>
{children}
,
    imageSection: props => (

            {/* {JSON.stringify(props, null, 2)} */}

    ),
    // someCustomType: YourComponent,
}

Since code is one of the default mark types, your block type is conflicting with it. This is a fairly rare scenario where you want to use the context of a node to determine the rendering intent (i.e. if it's a mark, render it one way, and if it's a block, render it another way). Since react-portable-text is designed to stop worrying about distinguishing the two for the sake of lower cognitive overhead, dealing with this scenario is a bit trickier. You may be able to replace the span serializer, but it is likely easier to use @sanity/block-content-to-react directly for this case if you aren't able to use a unique type name for the block type in Sanity.

Going to close this out since it's been a couple weeks, but if you still need help feel free to reply!

@coreyward I have tried using PortableText without any serializer for the code block, it says
Unknown block type "code", please specify a serializer for it in the serializers.types prop
So wether I provide a serializer or don't it simply does not render out the code block...I don't wish to use other library because of the ease this library provides to define custom blocks, just stuck on this CODE thing rendering, any help will be appreciated...

@trulymittal Are you able to use a different name for your custom code block in Sanity? E.g. codeBlock. If so, you can change to that and avoid this issue altogether.

@coreyward yeah I can easily do that in sanity but I don't see any reason for that, it seems like a hack for that. Isn't it possible to fix without any change of names of code block ?

@coreyward ooooo, I got this, I thought defining a serialiser for the code block would override both of them, but my bad, thanks for the clarification, you rock 🫡

@trulymittal What was your solution to this ? I tried to change it in sanity and now getting an error with following
Error: Unknown block type "codeBlock", please specify a serializer for it in the serializers.types prop
This error happened while generating the page. Any console logs will be displayed in the terminal window.
@coreyward  If you have any example with Code Block, i can take a look at it.

@SandeepKumarYaramchitti or anyone else having this issue, this is the solution that I used and is suggested by @coreyward, works flawlessly...
Sanity schema looks like, just note that for type code i have given it a name codeBlock, which can be any name, and then you need to reference that name for serializing in Portable text renderer
{
  name: 'content',
  title: 'Content',
  type: 'array',
  of: [
    {
      type: 'block',
    },
    {
      type: 'image',
    },
    {
      type: 'code',
      name: 'codeBlock',
    },
  ],
},

Serializer looks like this:
<PortableText
  className='prose mx-auto mt-8'
  content={content}
  projectId={process.env.PROJECT_ID}
  dataset={process.env.DATASET}
  serializers={{
    code: props => {
      // console.log('Testing', props)
      return
{props.children}

    },
    codeBlock: props => {
      return <CodeBlock code={props.code} language={props.language} />
    },
  }}
/>

Code block component can be anyway you like,  a pre tag also, but I have used syntax highlighter like this:
import SyntaxHighlighter from 'react-syntax-highlighter'
import vs2015 from 'react-syntax-highlighter/dist/esm/styles/hljs/vs2015'

export default function CodeBlock({ code, language }) {

  return (
    <SyntaxHighlighter
      language={language}
      style={vs2015}
      showLineNumbers={language !== 'sh'}
      customStyle={{
        marginTop: 0,
        borderTopLeftRadius: 0,
        borderTopRightRadius: 0,
      }}
      wrapLongLines
    >
      {code}
    </SyntaxHighlighter>
  )
}


Adding AjaxControlToolKit getting error

The AJAX Control Toolkit <VERSION>
The installation method

[ ] Installer
[ X] NuGet package
[ ] A custom build from the source code

Minimal steps to reproduce the bug

add AjaxToolkit as install doc, add to web.config (same result as adding it to aspx page)

  <add assembly="AjaxControlToolkit" namespace="AjaxControlToolkit" tagPrefix="AjaxControl"/>

run the web page

Actual result
An unhandled exception was generated during the execution of the current web request. Information regarding the origin and location of the exception can be identified using the exception stack trace below.
[HttpException (0x80004005): The Controls collection cannot be modified because the control contains code blocks (i.e. <% ... %>).]
System.Web.UI.ControlCollection.Add(Control child) +11843103
AjaxControlToolkit.ToolkitResourceManager.RegisterCssReferences(Control control) +624
System.Web.UI.Control.LoadRecursive() +68
System.Web.UI.Control.LoadRecursive() +162
System.Web.UI.Control.LoadRecursive() +162
System.Web.UI.Control.LoadRecursive() +162
System.Web.UI.Control.LoadRecursive() +162
System.Web.UI.Control.LoadRecursive() +162
System.Web.UI.Control.LoadRecursive() +162
System.Web.UI.Control.LoadRecursive() +162
System.Web.UI.Control.LoadRecursive() +162
System.Web.UI.Control.LoadRecursive() +162
System.Web.UI.Control.LoadRecursive() +162
System.Web.UI.Control.LoadRecursive() +162
System.Web.UI.Control.LoadRecursive() +162
System.Web.UI.Control.LoadRecursive() +162
System.Web.UI.Page.ProcessRequestMain(Boolean includeStagesBeforeAsyncPoint, Boolean includeStagesAfterAsyncPoint) +3811
Expected result
no error
Browser(s) used
chrome
A site deployment method

[X ] IIS

Hi @hayhua
I could not reproduce this issue:

I created ASP.NET Web application for .NET Framework 4.5.2
Installed AjaxControlToolkit v17.1.1 NuGet package
Added AsyncFileUpload to the Default.aspx page.
Ran the application.

Here is the sample project that was made as a result:
AjaxControlToolkitTest.zip
Would you please modify it or provide your own so that the issue is reproduced and we can continue our research?

I am using control ComboBox, not the control AsyncFileUpload in the sample project. Maybe that is different?

Analizing the error and source code, i think the error is in code:
AjaxControlToolkit/ComboBox/ComboBox.cs
line 417: ToolkitResourceManager.RegisterCssReferences(this);
AjaxControlToolkit/ToolkitResourceManager.cs
line 199:         public static void RegisterCssReferences(Control control) {
line 203:              // Add the link to the page header instead of inside the body which is not xhtml compliant
which looks like the error mentioned in in link:
https://stackoverflow.com/questions/4995274/the-controls-collection-cannot-be-modified-because-the-control-contains-code-blo
adding code block to page.header causing such problem

Hi @hayhua
Would you please provide us with a complete sample project so that we can investigate the code which you put into the header?

We were expecting an answer from you, however we didn't receive one in the last 14 days.
We'll assume that the problem is resolved and automatically close the ticket.
If it hasn't, please do not hesitate to comment on this issue.


Create Module sample backups based on WWI Std to avoid HK

For anyone following this I will work on this post SQLBits

Create new backups available in the https://github.com/microsoft/sqlworkshops-sql2022workshop/releases and updated readme.md to point to these if needed. Orginal backups are left for the exercises as I've rarely seen a need to use the ones that don't include memory optimized tables.


failing to run exe file

Hi,
First, thanks for this script!!
As always, I download the most recent version.
I tried running:

ESET-KeyGen_v<IP_ADDRESS>_win64.exe --chrome --key
ESET-KeyGen_v<IP_ADDRESS>_win64.exe
python main.py --chrome --key

I run python 3.12 and the error message, for all cases is this:
[ FAILED ] Traceback (most recent call last):
  File "main.py", line 359, in main
    EsetKeyG.sendRequestForKey()
  File "modules\EsetTools.py", line 97, in sendRequestForKey
    uCE(self.driver, f"return {CLICK_WITH_BOOL}({GET_EBAV}('button', 'data-label', 'license-fork-slide-trial-license-card-button'))")
  File "modules\SharedTools.py", line 127, in untilConditionExecute
    raise RuntimeError('untilConditionExecute: the code did not return the desired value! TRY VPN!')
RuntimeError: untilConditionExecute: the code did not return the desired value! TRY VPN!

Try add this code to 209 LINE in /modules/SharedTools.py:
   driver.maximize_window()

And run main.py with --no-headless

Try add this code to 209 LINE in /modules/SharedTools.py:
   driver.maximize_window()

Just keep the indentation the same as the next line of code (210 LINE)

Where exactly to add this line?
After line 209 or instead of line 209?
This is line 209 :
driver_options.add_argument('--disable-dev-shm-usage')
From: Artem @.>
Sent: Friday, May 24, 2024 19:27
To: rzc0d3r/ESET-KeyGen @.>
Cc: karpiyon @.>; Author @.>
Subject: Re: [rzc0d3r/ESET-KeyGen] failing to run exe file (Issue #189)
Try add this code to 209 LINE in /modules/SharedTools.py:
driver.maximize_window()
And run main.py with --no-headless
—
Reply to this email directly, view it on GitHubhttps://github.com/rzc0d3r/ESET-KeyGen/issues/189#issuecomment-2129943812, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AHCZEKJBQZ4AC6O532V2J4DZD5S3PAVCNFSM6AAAAABIHR67VWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMRZHE2DGOBRGI.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>

I tried with --no-healess and it worked.
I then tried without it and it also worked.
This means that it started working without changing anything.

I tried with --no-healess and it worked.
I then tried without it and it also worked.
This means that it started working without changing anything.

That's where the difficulty lies in solving this bug in any way.

Where exactly to add this line?
After line 209 or instead of line 209?
This is line 209 :
driver_options.add_argument('--disable-dev-shm-usage')
From: Artem @.>
Sent: Friday, May 24, 2024 19:27
To: rzc0d3r/ESET-KeyGen @.>
Cc: karpiyon @.>; Author @.>
Subject: Re: [rzc0d3r/ESET-KeyGen] failing to run exe file (Issue #189)
Try add this code to 209 LINE in /modules/SharedTools.py:
driver.maximize_window()
And run main.py with --no-headless
—
Reply to this email directly, view it on GitHubhttps://github.com/rzc0d3r/ESET-KeyGen/issues/189#issuecomment-2129943812, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AHCZEKJBQZ4AC6O532V2J4DZD5S3PAVCNFSM6AAAAABIHR67VWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMRZHE2DGOBRGI.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>
Forget it.

solved by itself,
Thanks for your help


Build Orchestrator for amazonlinux 2

This PR adds the following

Base image that will be used to build Orchestrator for amazonlinux 2 https://gallery.ecr.aws/metrist/dist-amazon-linux
Change to the install script so that amazonlinux 2 users can run the install script to install metrist-orchestrator and start it through systemd

Testing
I was able to test this by spinning up a tiny EC2 instance running with Amazon Linux 2 and adding an invalid auth token, then enabling it through systemd
● metrist-orchestrator.service - Metrist.io Orchestrator service
   Loaded: loaded (/etc/systemd/system/metrist-orchestrator.service; enabled; vendor preset: disabled)
   Active: active (running) since Tue 2022-10-18 18:55:03 UTC; 31min ago
 Main PID: 9692 (start)
   CGroup: /system.slice/metrist-orchestrator.service
           ├─9692 /bin/sh /tmp/.metrist-bwcache/1D351DFA8BB0BC07418AB853B181909AC4DF6052/start start
           ├─9712 /tmp/.metrist-bwcache/1D351DFA8BB0BC07418AB853B181909AC4DF6052/erts-13.1.1/bin/beam.smp -- -root /tmp/.metrist-bwcache/1D351DFA8BB0BC07418A...
           ├─9746 erl_child_setup 65535
           ├─9761 sh -s disksup
           ├─9762 /tmp/.metrist-bwcache/1D351DFA8BB0BC07418AB853B181909AC4DF6052/lib/os_mon-2.7.1/priv/bin/memsup
           ├─9763 /tmp/.metrist-bwcache/1D351DFA8BB0BC07418AB853B181909AC4DF6052/lib/os_mon-2.7.1/priv/bin/cpu_sup
           ├─9765 /tmp/.metrist-bwcache/1D351DFA8BB0BC07418AB853B181909AC4DF6052/erts-13.1.1/bin/inet_gethost 4
           └─9766 /tmp/.metrist-bwcache/1D351DFA8BB0BC07418AB853B181909AC4DF6052/erts-13.1.1/bin/inet_gethost 4

Oct 18 19:25:09 ip-10-0-102-232.ec2.internal metrist-orchestrator[9692]: 19:25:09.055 [error] Got 403 posting https://app.metrist.io/api/agent/telemetr...trying
Oct 18 19:25:09 ip-10-0-102-232.ec2.internal metrist-orchestrator[9692]: 19:25:09.055 [info] IPA: We are sending (m=POST h=app.metrist.io p=/api/agent/… matches
Oct 18 19:25:09 ip-10-0-102-232.ec2.internal metrist-orchestrator[9692]: 19:25:09.126 [error] Got 403 posting https://app.metrist.io/api/agent/telemetr...trying
Oct 18 19:26:08 ip-10-0-102-232.ec2.internal metrist-orchestrator[9692]: 19:26:08.984 [info] Pulling configs...
Oct 18 19:26:08 ip-10-0-102-232.ec2.internal metrist-orchestrator[9692]: 19:26:08.984 [info] Fetching config for instance fake-dev-instance and run groups []
Oct 18 19:26:09 ip-10-0-102-232.ec2.internal metrist-orchestrator[9692]: 19:26:09.054 [info] IPA: We are sending (m=GET h=app.metrist.io p=/api/agent/r… matches
Oct 18 19:26:09 ip-10-0-102-232.ec2.internal metrist-orchestrator[9692]: 19:26:09.054 [info] - deltas: add=0 delete=0 change=0
Oct 18 19:26:09 ip-10-0-102-232.ec2.internal metrist-orchestrator[9692]: 19:26:09.125 [error] Got 403 posting https://app.metrist.io/api/agent/telemetr...trying
Oct 18 19:26:09 ip-10-0-102-232.ec2.internal metrist-orchestrator[9692]: 19:26:09.125 [info] IPA: We are sending (m=POST h=app.metrist.io p=/api/agent/… matches
Oct 18 19:26:09 ip-10-0-102-232.ec2.internal metrist-orchestrator[9692]: 19:26:09.194 [error] Got 403 posting https://app.metrist.io/api/agent/telemetr...trying

Deployment
The base image and the rpm file has been deployed and publish. Only left to do is review and merge this PR

We probably want to update the product docs at https://docs.metrist.io/en/latest/orchestrator-installation/ and the INSTALL.md to note we now support Amazon Linux v2. I'll add that tomorrow morning.

We probably want to update the product docs at https://docs.metrist.io/en/latest/orchestrator-installation/ and the INSTALL.md to note we now support Amazon Linux v2. I'll add that tomorrow morning.

Maybe we could a list of Linux distributions we support when running install.sh


Determine how to handle the 'proof' parameter for queries to the private store

In a discussion in #196, it was brought up the fact that some queries such as clients/{identifier}/connections that target the private store allows the proof parameter to be specified. The discussion was that if we need to set (hardcode) this parameter to false since we know that is the only valid value or if we should allow users to specify an invalid value (e.g. true) and get an error message (right now an error client connections query error store error: requested proof for privateStore path is shown). This is affects other queries too e.g. clients/{identifier}/clientState.
This is a UX decision too, need to think the best way to refactor the code for this. Hardcoding the value to false in these use cases is the quick solution, but there might be a need eventually some architecture decisions (e.g. know if the query targets a private or provable store and handle accordingly).
Reference: https://github.com/cosmos/ics/tree/master/spec/ics-024-host-requirements

Let's remove the -p option from QueryClientConnectionsCmd as there is no benefit to it, especially that the default is true which fails the query immediately.


Fix server routing

Server should return index.html on non-ajax requests on all requests
In case the request is triggered by an action on FE - return only data

Done


Propagate downstream timeout to upstream through multiple Envoys

When running Envoy on both egress and ingress, the client will provide a timeout header to the egress Envoy, which will propagate the expected upstream timeout in x-envoy-expected-rq-timeout-ms. The upstream Envoy will not read this header, so it will resolve a new timeout value that is set as the expected timeout for the upstream service. This means that the deadline expected by the egress Envoy is ignored in favor of the ingress Envoy, resulting in the upstream service having an incorrect view of the actual deadline.
It seems like either

inserting x-envoy-rq-timeout-ms with the expected timeout on egress
parsing x-envoy-expected-rq-timeout-ms as the deadline on the ingress side

would solve the issue (likely guarded by a config flag).
Thoughts?

Yeah agreed I think either would solve. I don't have a super strong opinion on which one.
This does bring up the general topic of deadline propagation which we haven't really tackled yet in a holistic way. I've thought at some point we may also want to stick the deadline in trace context baggage since this would also propagate through app calls, though that is a larger problem than you are trying to solve here.

+1. It would be very useful.  Inserting x-envoy-rq-timeout-ms on the ingress side may be good idea. That might require users to set use_remote_address to ensure that it is not sanitized?

@snowp would like to help out with this.

@nezdolik Great, I'll assign this one to you. Feel free to ping me with any questions


How to obtain email information in the msgraph sample uwp demo?

How to obtain email information in the msgraph sample uwp demo? I tried to obtain email information and it appears
Code: ErrorAccessDenied
Message: Access is denied. Check credentials and try again.
The code is as follows:
var events = await graphClient.Me.MailFolders["Inbox"].Messages.Request().GetAsync();

@xiakan this sample is setup with the appropriate permission scopes to access the user's calendar, not their email.
https://github.com/microsoftgraph/msgraph-sample-uwp/blob/8ee5d4a858d98ac5319f11c327d0cd936f4645a7/GraphTutorial/OAuth.resw.example#L123-L125
To read email, you'd need to add Mail.Read (or one of the other possible permission scopes applicable to that API). See the docs for a list of applicable permission scopes.

@jasonjoh Thank you very much!


Support export of modules using their "as" name

Original reporter: bjpop@
Haskell allows you to import modules using an "as" name, and re-export the module using that name:
module Language.Python.Common ( module Pretty )
import Language.Python.Common.Pretty as Pretty
It seems that Haddock 2.4.2 has trouble with this and complains that:
Warning: language-python-0.2:Language.Python.Common: Could not find documentation for exported module: Pretty
Changing the export to use the full qualified name of the module fixes the problem.

+1

should be resolved by #688


IE11 Performance

Bug report
Expected behaviour
svgPanZoom performed in a reasonable time (< 5sec)
Actual behaviour
svgPanZoom performed in > 45sec
Steps to reproduce the behaviour

use svgPanZoom() on an inline SVG with more than 30k nodes

Configuration

svg-pan-zoom version: 3.5.2
Browser(s): IE11
Operating system(s): Windows 7

More Explanations
I use SvgPanZoom on massive SVG (+30k nodes).
There is an expectable delay while SvgPanZoom performs its treatments. This delay is very short on FF or Chrome, but is more than 45sec on IE11!
I inspected the code : the delay is caused by the creation of the viewport and the deplacement of all the graphic nodes into it. (svg-utilities.js l.62).
I tried to display: none the svg parent element, without any improvements.
if (!viewport) {
      var viewportId = 'viewport-' + new Date().toISOString().replace(/\D/g, '');
      viewport = document.createElementNS(this.svgNS, 'g');
      viewport.setAttribute('id', viewportId);

      var parent = svg.parentNode;
      parent.style.display = "none";
      var svgChildren = svg.childNodes || svg.children;
      var length = svgChildren.length;
      // Internet Explorer (all versions?) can't use childNodes, but other browsers prefer (require?) using childNodes
      var start = performance.now();
      var k = 0;
      if (!!svgChildren && svgChildren.length > 0) {
        for (var i = svgChildren.length; i > 0; i--) {
          // Move everything into viewport except defs
          if (svgChildren[svgChildren.length - i].nodeName !== 'defs') {
            viewport.appendChild(svgChildren[svgChildren.length - i]);
            k++;
          }
        }
      }
      var end = performance.now();
      console.log("svg manipulation took " + (end - start) + " ms");
      console.log(k + " / " + length + " nodes manipulated");
      svg.appendChild(viewport);
      parent.style.display = "";
    }

Result is :
svg manipulation took 45050.880227233756 ms
30413 / 30413 nodes manipulated

Do you guys know a way to improving this ?
45sec is just not usable ;s

Hi @Kwoin,
Please check this comment on a related issue.
In short the solution is to create that viewport by yourself (say on the server or whenever your SVGs are generated). I don't know of any other solutions.
Hope this helps.


Use a mirror of the helm-test Docker image…

…instead of continuing to rely on quay.io.
Signed-off-by: Flynn<EMAIL_ADDRESS>
[ ] I made sure to update CHANGELOG.md.
[x] This is unlikely to impact how Ambassador performs at scale.
[ ] My change is adequately tested.
[x] I didn't need to update DEVELOPING.md.

What we really need to do is just stop running it in Docker.


v<IP_ADDRESS> is not avaiable on pypi

pip install tick==<IP_ADDRESS>
ERROR: Could not find a version that satisfies the requirement tick==<IP_ADDRESS> (from versions: 0.1.2013, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>)
ERROR: No matching distribution found for tick==<IP_ADDRESS>

pip install with version <IP_ADDRESS> returns not available. Would be possible to release this version (or current master branch) to pypi?

you're probably using python3.9 or higher I guess
https://pypi.org/project/tick/#files
these versions have no pip package as of yet
but I don't see a reason why the 3.8 version should not work for you (although it may not)


flowtype definitions

Are there flowtype definitions for alasql?

There is no flowtype definition for alasql at the moment.
If you would like to help me test, I would love to make one in cooperation with you...

@karubimania would you be interested in testing if I make a flowtype definition?

@mathiasrw I missed your original response, yes I could contribute some test scripts.  In the interim the easiest way to test is to run flow check.  It won't confirm for integration tests but it should pick up on small errors

We should make it work, and then make a PR to https://github.com/flowtype/flow-typed/ so its in the ecosystem.
I imagine we also place it in this repo so when build it gets placed in dist/together with the original lib file.
Ok - I have made a version - but not been able to setup the whole flow eco system, so I would be happy if you could test this out.

Download https://raw.githubusercontent.com/agershun/alasql/develop/dist/alasql.js
Make a file in your project called alasql.js.flow and insert the following:

declare namespace alaSQLSpace {
    interface AlaSQLCallback {
        (data?: any, err?: Error): void;
    }

    interface AlaSQLOptions {
        errorlog: boolean;
        valueof: boolean;
        dropifnotexists: boolean; // drop database in any case
        datetimeformat: string; // how to handle DATE and DATETIME types
        casesensitive: boolean; // table and column names are case sensitive and converted to lower-case
        logtarget: string; // target for log. Values: 'console', 'output', 'id' of html tag
        logprompt: boolean; // print SQL at log
        modifier: any; // values: RECORDSET, VALUE, ROW, COLUMN, MATRIX, TEXTSTRING, INDEX
        columnlookup: number; // how many rows to lookup to define columns
        autovertex: boolean; // create vertex if not found
        usedbo: boolean; // use dbo as current database (for partial T-SQL comaptibility)
        autocommit: boolean; // the AUTOCOMMIT ON | OFF
        cache: boolean; // use cache
        nocount: boolean; // for SET NOCOUNT OFF
        nan: boolean; // check for NaN and convert it to undefined
        angularjs: boolean;
        tsql: boolean;
        mysql: boolean;
        postgres: boolean;
        oracle: boolean;
        sqlite: boolean;
        orientdb: boolean;
    }

    // compiled Statement
    interface AlaSQLStatement {
        (params?: any, cb?: AlaSQLCallback, scope?: any): any;
    }

    // abstract Syntax Tree
    interface AlaSQLAST {
        compile(databaseid: string): AlaSQLStatement;
    }

    // https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/es6-promise/es6-promise.d.ts
    interface Thenable<T> {
        then<U>(onFulfilled?: (value: T) => U | Thenable<U>, onRejected?: (error: any) => U | Thenable<U>): Thenable<U>;
        then<U>(onFulfilled?: (value: T) => U | Thenable<U>, onRejected?: (error: any) => void): Thenable<U>;
        catch<U>(onRejected?: (error: any) => U | Thenable<U>): Thenable<U>;
    }

    // see https://github.com/agershun/alasql/wiki/User%20Defined%20Functions
    interface userDefinedFunction {
        (x: any): any;
    }
    interface userDefinedFunctionLookUp {
        [x: string]: userDefinedFunction;
    }
    // see https://github.com/agershun/alasql/wiki/User%20Defined%20Functions
    interface userAggregator {
        (value: any, accumulator: any, stage: number): any;
    }
    interface userAggregatorLookUp {
        [x: string]: userAggregator;
    }

    interface AlaSQL {
        (sql: any, params?: any, cb?: AlaSQLCallback): any;
        options: AlaSQLOptions;
        error: Error;
        parse(sql): AlaSQLAST;
        promise(sql: any, params?: any): Thenable<any>;
        fn: userDefinedFunctionLookUp;
        aggr: userAggregatorLookUp;
        autoval(tablename: string, colname: string, getNext?:boolean): number;
    }
}

declare var alasql: alaSQLSpace.AlaSQL;
declare module 'alasql' {
    export = alasql;
}

Let me know if it works. If now I will have to dig more into getting flow up and running.

@mathiasrw thanks for doing this!  There are some errors:

namespace isn't a thing in flow, I would just prefix alasql-internal stuff with _alasql_

the module export construct should be something like

declare module 'alasql' {
    declare var exports:AlaSQL;
}

You don't need to install anything to test, just go to https://flowtype.org/try/ and paste just the alasql.js.flow code

You are awesome!
Please run this version and verify that it works:
/* @flow */

interface _alasql_Callback {
    (data?: any, err?: Error): void;
}

interface _alasql_Options {
    errorlog: boolean;
    valueof: boolean;
    dropifnotexists: boolean; // drop database in any case
    datetimeformat: string; // how to handle DATE and DATETIME types
    casesensitive: boolean; // table and column names are case sensitive and converted to lower-case
    logtarget: string; // target for log. Values: 'console', 'output', 'id' of html tag
    logprompt: boolean; // print SQL at log
    modifier: any; // values: RECORDSET, VALUE, ROW, COLUMN, MATRIX, TEXTSTRING, INDEX
    columnlookup: number; // how many rows to lookup to define columns
    autovertex: boolean; // create vertex if not found
    usedbo: boolean; // use dbo as current database (for partial T-SQL comaptibility)
    autocommit: boolean; // the AUTOCOMMIT ON | OFF
    cache: boolean; // use cache
    nocount: boolean; // for SET NOCOUNT OFF
    nan: boolean; // check for NaN and convert it to undefined
    angularjs: boolean;
    tsql: boolean;
    mysql: boolean;
    postgres: boolean;
    oracle: boolean;
    sqlite: boolean;
    orientdb: boolean;
    autoExtFilenameOnRead: boolean;
    autoExtFilenameOnWrite: boolean;
}

// compiled Statement
interface _alasql_Statement {
    (params?: any, cb?: _alasql_Callback, scope?: any): any;
}

// abstract Syntax Tree
interface _alasql_AST {
    compile(databaseid: string): _alasql_Statement;
}

// From https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/es6-promise/es6-promise.d.ts
interface Thenable<T> {
    then<U>(onFulfilled?: (value: T) => U | Thenable<U>, onRejected?: (error: any) => U | Thenable<U>): Thenable<U>;
    then<U>(onFulfilled?: (value: T) => U | Thenable<U>, onRejected?: (error: any) => void): Thenable<U>;
    catch<U>(onRejected?: (error: any) => U | Thenable<U>): Thenable<U>;
}

// From https://github.com/agershun/_alasql_/wiki/User%20Defined%20Functions
interface _alasql_userDefinedFunction {
    (x: any): any;
}

interface _alasql_userDefinedFunctionLookUp {
    [x: string]: _alasql_userDefinedFunction;
}

// From https://github.com/agershun/_alasql_/wiki/User%20Defined%20Functions
interface _alasql_userAggregator {
    (value: any, accumulator: any, stage: number): any;
}

interface _alasql_userAggregatorLookUp {
    [x: string]: _alasql_userAggregator;
}

interface AlaSQL {
    (sql?: string | [], params?: any, cb?: _alasql_Callback): string | number | {} | [];
    options: _alasql_Options;
    error: Error;
    parse(sql: string): _alasql_AST;
    promise(sql: string | [], params?: any): Thenable<any>;
    fn: _alasql_userDefinedFunctionLookUp;
    aggr: _alasql_userAggregatorLookUp;
    autoval(tablename: string, colname: string, getNext?:boolean): number;
}

declare module 'alasql' {
declare var exports:AlaSQL;
}

Perfect with the website to test. I made a PR to add the definition: https://github.com/flowtype/flow-typed/pull/670
If you would like to make the test files better (or edit in the definition) you can:

Clone https://github.com/AlaSQL/flow-typed (dont rename the repo)
Edit the files in definitions/npm/alasql...
Test the changes with ./run_def_tests.sh alasql


courses page  stuck on loading page when logged out

This fix isnt quite right. Needs rethinking

@Zahra-8d  - I'd like your thoughts on this PR (even though ive merged it). I think the user.loading flag in the _app.tsx page does all the work. I think we can delete the piece of state .firebaseUpdateApplied as the user.token seems to indicate the same thing anyways. So confusing


Test Server and Service Frabric

Is it possible to use the test server to test a service fabric API ?

If you have a document issue, please open on the document page.


Delete private app

Delete private app
Delete unused code

Current dependencies on/for this PR:

main

PR #39   👈

This stack of pull requests is managed by Graphite.


Fix duplicate identifier reporting in classes

Fixes #13141 by updating the code from #8696 to understand MethodDeclaration as well.

@RyanCavanaugh mind taking a look since you wrote checkClassForDuplicateDeclarations?


[WIP] Remove macos target for builds and tests

Skip builds and tests if the PR name starts with [WIP].
Removed macos-latest from action target matrix.

@danielSanchezQ @jakubgs, please review.

Closing, as these changes are not required for public repos.


code release

Great work!
I'm looking forward to following your work，I wonder how soon you expect to release the open source code？
THX!

Hi @vhzy ,
Hopefully we will release the code this month ASAP. The implementation details, including the OpenAI API version and the prompts, are listed in our paper.
You can also refer to the other two projects PICa and Prophet for requesting OpenAI API.
If you encounter any urgent problems, feel free to send me an email.

Hi @vhzy , Hopefully we will release the code this month ASAP. The implementation details, including the OpenAI API version and the prompts, are listed in our paper. You can also refer to the other two projects PICa and Prophet for requesting OpenAI API. If you encounter any urgent problems, feel free to send me an email.

Thank you for your reply!
I'm curious whether you call OpenAI's LLM interface online and then train the filter model; or you call LLM first through offline operation, then collect all the data and train the filter model?

Thank you for your reply!
I'm curious whether you call OpenAI's LLM interface online and then train
the filter model; or you call LLM first through offline operation, then
collect all the data and train the filter model?
Also, the openai interface is blocked in China. How do you solve this
problem?
Finally, if I may, I am also curious about how much you spent on the API
for this article?
wangphoebe @.***> 于2023年12月1日周五 14:41写道：

Hi @vhzy https://github.com/vhzy ,
Hopefully we will release the code this month ASAP. The implementation
details, including the OpenAI API version and the prompts, are listed in
our paper https://arxiv.org/pdf/2311.11598.pdf.
You can also refer to the other two projects PICa
https://github.com/microsoft/PICa and Prophet
https://github.com/MILVLG/prophet for requesting OpenAI API.
If you encounter any urgent problems, feel free to send me an email.
—
Reply to this email directly, view it on GitHub
https://github.com/THUNLP-MT/FIIG/issues/1#issuecomment-1835552274, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AMR5ZONZIJNOFEH5DT65I2DYHF3Z7AVCNFSM6AAAAAA77JXLU2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMZVGU2TEMRXGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


handleOnXML tries to parse.xlsx files

The handleOnXML function attempts to parse responses with the content-type application/vnd.openxmlformats-officedocument.spreadsheetml.sheet. This is because the function looks for any mention of xml in the content type. This results in a parse error when xmlquery.Parse() is called (For example: `encoding/xml.SyntaxError {Msg: "illegal character code U+0003", Line: 1}).
XLSX files packaged as a zip - so can't be directly parsed as XML.
It would be ideal to not try and parse these files, possibly by being more explicit in which content-types we consider to be XML.

This doesn't only effect xlsx, but also docx, pptx etc.. type documents

To add to this it would be nice to able to have more granularity over what XML is parsed. For example, we use a OnXML handler to follow links in a XML sitemap, but our site contains many SVGs (image/svg+xml) and RFDs (application/rdf+xml) are also content types which are unnecessarily parsed.


'Rusto' figlet font

I noticed in xero_shell.gif and in aliases.zsh that you used a font called Rusto, but I can't find it anywhere, and it did not come installed with toilet on my system (Arch). Did you make it yourself? It's really pretty and I want it. :p

@ijks yes, custom. i have a lot more in the works. here ya go: http://git.io/.figletfonts
i added rusto and rustofat (the bold version).

┳━┓┏┓┓  ┳┏━┓┓ ┳┓
┣━ ┃┃┃┏ ┃┃ ┃┗┏┛┃
┻━┛┇┗┛┗━┇┛━┛ ┇ o

Awesome, thanks! :+1:


fix(CX-2138): login and signup screen, bottom buttons dont obey SafeArea

The type of this PR is: Bugfix

This PR resolves CX-2138
Description
Replaced View with Flex (yep, I committed, but didn't push.. Friday, I guess)
(PR where i should have pushed changes https://github.com/artsy/eigen/pull/6411)

PR Checklist (tick all before merging)

[x] I have included screenshots or videos to illustrate my changes, or I have not changed anything that impacts the UI.
[x] I have tested my changes on iOS and Android.
[x] I have added tests/stories for my changes, or my changes don't require testing/stories, or I have included a link to a separate Jira ticket covering the tests.
[x] I have added a feature flag, or my changes don't require a feature flag. (How do I add one?)
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added an app state migration, or my changes do not require one. (What are migrations?)
[x] I have added a changelog entry below or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

small layout improvement -daria

This PR contains the following changes:

Dev changes (small layout improvement -daria)

  Generated by :no_entry_sign: dangerJS against 46ff5b011fe4a8806a6ce9f71f52cb178b936f99


Check Definitions for Columns

What do the different columns in the CSVs mean? (e.g. FN, Active)
Transaction_train: Is every row a single article purchase?

https://www.kaggle.com/competitions/h-and-m-personalized-fashion-recommendations/discussion/305952


Docs: Better explain testing plugins in Grafana Cloud [UXR]

"Explain how plugins can be tested in Grafana Cloud when they are unsigned"
This is a request from a UXR interview. Is it actionable?

After discussion with Erik, I feel this issue is not of general importance to most plugin developers.


YATA -OC Error: [object Object]

I have the abovementioned error
YATA -OC
Error: [object Object]
on my faction page - shown above the OC list.

Thank you for the report. I can't really say what the underlying error is, but the newest version of the script I just released should have the error display fixed. Could you update it and tell me what it says?

Hey, thanks a lot! Now it says:
Error: error is not defined (from TornStats)
I have a TornStats account and it works but I can't find any options that
could be connected to this..
Am Mi., 17. Feb. 2021 um 13:40 Uhr schrieb TotallyNot <
<EMAIL_ADDRESS>
Thank you for the report. I can't really say what the underlying error is,
but the newest version of the script I just released should have the error
display fixed. Could you update it and tell me what it says?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/TotallyNot/yata-oc/issues/2#issuecomment-780529525,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AS35W5KXG5PDU3T5JIB6T7DS7O2NDANCNFSM4XWXDJUA
.

My bad, there was another bug in the error handling for torn stats API requests. That should be fixed in the newest version and it should now properly display the reason for the error.

I assume that this has to do with faction API access, but the error message should shed some light on that.


Ignore loader cache modification time during build

We ran into an issue using GitHub Actions where the cache action preserves modification times, but the checkout action does not, and so the loader sources always look newer than the cache, and hence the loaders run even though the cache is populated. So perhaps a better default during build is to ignore modification times, and simply check whether the files are present. This way it will work by default in GitHub Actions without workarounds such as
find docs/.observablehq/cache -type f -exec touch {} +

And you can always remove the cache before building if you want to force a clean build.

I believe I am running into an issue that is related to this one. I would like to build my dashboard periodically using npm run build and aggregate new data after fetching entries later than the cash from a database. Per documentation, I thought if I update the modification times of the data loaders using
find docs/data/*py -type f -exec touch {} +
before the build, this would enforce running the data loaders (in my case *.py files). This, however, is not the case.

@burakbudanur did you run find docs/data/*py -type f  separately to see if that gives you the expected list of files? (If you're using a glob pattern you could do touch docs/data/*py instead, it should be equivalent to what you describe above.)

Yes. Also confirmed that the command indeed updates the last modification time.

During build cache file modification times are ignored. So, delete the cache files you want to recompute instead of touching the data loaders. Touching is only something you do during preview.
(Also, file a separate issue or discussion if you want help rather that using a closed issue.)


ControllerVisualizationProfile is not multi platform capable

Overview
When running the MRKT on a mobile phone and touching the screen, an error is thrown:

No controller model available. Failed to add controller game object to scene

This happens, because the DefaultMixedRealityControllerVisualizationProfile is queried for controllers. By default, it provides 2 controllers for UWP basically.
It is queried from the DefaultMixedRealityRegisteredServiceProvidersProfile, which CAN distinguish between platforms and uses the UnityTouchDeviceManager, which actually fits the platform.
Expected Behavior

Touch devices do not try to instantiate a controller
Each platform provides its set of controllers

Actual Behavior

error because for every platform UWP Controllers are tried to be used
differentiation is currently only possible when exchanging the ControllerVisualizationProfile altogether.

Unity Editor Version
2018.3.9f1
Mixed Reality Toolkit Release Version
mrtk_development

This seems similar to https://github.com/Microsoft/MixedRealityToolkit-Unity/issues/3685
I will work on getting out a fix for this early next week.

yes I thought the same just forgot to link it

I removed the error message for controllers that have no visualization here https://github.com/Microsoft/MixedRealityToolkit-Unity/pull/3822


Remove old images

Summary
Remove images that are no longer relevant

Related issue : #68

Missing the fedcloud appliance as this one should be recreated properly

Setting as ready so we can start cleaning up. Will deal with the appliance soonish


Unable to start after horizon upgrade

exited: horizon (exit status 1; not expected)
I created the latest stellar-core image, re-run, horizon startup error
Container log is as follows：
running `/start'
pids are [6]
Starting Stellar Quickstart
mode: persistent
network: testnet (Test SDF Network ; September 2015)
postgres: config directory exists, skipping copy
supervisor: config directory exists, skipping copy
stellar-core: config directory exists, skipping copy
horizon: config directory exists, skipping copy
postgres: already initialized
core: already initialized
horizon: already initialized
starting supervisor
2019-03-07 11:27:37,976 CRIT Supervisor running as root (no user in config file)
2019-03-07 11:27:37,984 INFO RPC interface 'supervisor' initialized
2019-03-07 11:27:37,984 CRIT Server 'unix_http_server' running without any HTTP authentication checking
2019-03-07 11:27:37,985 INFO supervisord started with pid 6
2019-03-07 11:27:38,987 INFO spawned: 'postgresql' with pid 19
2019-03-07 11:27:38,990 INFO spawned: 'stellar-core' with pid 20
2019-03-07 11:27:38,992 INFO spawned: 'horizon' with pid 21
2019-03-07 11:27:38,992 INFO reaped unknown pid 16
2019-03-07 11:27:40,161 INFO success: postgresql entered RUNNING state, process has stayed up for > than 1 seconds (startsecs)
2019-03-07 11:27:40,161 INFO success: stellar-core entered RUNNING state, process has stayed up for > than 1 seconds (startsecs)
2019-03-07 11:27:40,161 INFO success: horizon entered RUNNING state, process has stayed up for > than 1 seconds (startsecs)
2019-03-07 11:27:40,232 INFO exited: horizon (exit status 1; not expected)
2019-03-07 11:27:41,231 INFO spawned: 'horizon' with pid 84
2019-03-07 11:27:41,323 INFO exited: horizon (exit status 1; not expected)
2019-03-07 11:27:43,233 INFO spawned: 'horizon' with pid 106
2019-03-07 11:27:43,327 INFO exited: horizon (exit status 1; not expected)
2019-03-07 11:27:45,331 INFO spawned: 'horizon' with pid 128
2019-03-07 11:27:45,423 INFO exited: horizon (exit status 1; not expected)
2019-03-07 11:27:48,427 INFO spawned: 'horizon' with pid 150
2019-03-07 11:27:48,521 INFO exited: horizon (exit status 1; not expected)
2019-03-07 11:27:48,521 INFO gave up: horizon entered FATAL state, too many start retries too quickly

Getting the same thing. Current docker images do not appear to be working

Can you send horizon logs from your docker container?

Hi @bartekn, I still got this log error after do horizon db migrate up
A database migration is required to run this version (v0.18.0) of Horizon. Run "horizon db migrate up" to update your DB. Consult the Changelog (https://github.com/stellar/go/blob/master/services/horizon/CHANGELOG.md) for more information.

Are there any instructions I must do?


Do not write to stream when given a nullptr in the writer

Submission Checklist

[x] Run unit tests: ./runTests.py src/test/unit
[x] Run cpplint: make cpplint
[x] Declare copyright holder and open-source license: see below

Summary
This is a simpler fix for https://github.com/stan-dev/cmdstan/issues/1055 where we just do a check if the output_ unique pointer is a nullptr and if so we don't write anything. For the diagnostics fix we just need to change line 502 in command.hpp so that if the diagnostic file to be written to is the default "" then we just insert a nullptr i.e.
    if (diagnostic_file != "") {
      auto diagnostic_filename
          = diagnostic_name + name_iterator(i) + diagnostic_ending;
      diagnostic_writers.emplace_back(
          std::make_unique<std::fstream>(diagnostic_filename,
                                         std::fstream::out),
          "# ");
    } else {
      diagnostic_writers.emplace_back(nullptr, "# ");
    }

Doing this and comparing 2.28 and 2.27 things look fine to me (in fact a bit faster but just a little so could be fuzz)
 # 2.27
 Elapsed Time: 7.075 seconds (Warm-up)
               11.757 seconds (Sampling)
               18.832 seconds (Total)

 2.28 with the linechange above and this branch
 Elapsed Time: 6.574 seconds (Warm-up)
               10.657 seconds (Sampling)
               17.231 seconds (Total)

Intended Effect
How to Verify
Side Effects
Documentation
Copyright and Licensing
Please list the copyright holder for the work you are submitting (this will be you or your assignee, such as a university or company):
By submitting this pull request, the copyright holder is agreeing to license the submitted work under the following licenses:

Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

Doing this and comparing 2.28 and 2.27 things look fine to me (in fact a bit faster but just a little so could be fuzz)

I am seeing the same thing. Thanks so much! Will just check the changes and then should be good to go.
Do you want to make the CmdStan PR as well? We are not going to do the patch release today anyways to no hurry.


Create a rake task to clean PARTNER_RECAP to a specific timeframe

@hackmastera and I worked on cleaning events and dumps for the PARTNER_RECAPS.
this is related to https://github.com/pulibrary/marc_liberation/issues/592
We tested on staging, locally and ran on production.
the rake task should accept one or two dates in case we would want to clean the events and their dumps in between a specific time frame.
dump_type_id_partner_recap=DumpType.where(constant: 'PARTNER_RECAP').first.id
event_in_timeframe = Event.where('start >  ?', "2020-01-08")
event_ids_in_timeframe = event_in_timeframe.ids
dump_recaps=Dump.where(dump_type_id: dump_type_id_partner_recap).where(:event_id =>      event_ids_in_timeframe).compact
dump_recaps_event_id = dump_recaps.pluck(:event_id)
events_to_delete = dump_recaps_event_id.map {|m| Event.find(m)}
events_to_delete.each(&:destroy)

👍 It seems likely there's a cleaner way to join these tables if we think about it a bit more. Regardless, if we make this a rake task it should call a service that has a test.


Feature/90-suggestions-quartermarks

In this PR I added a few features:

Quarter marks CRUD. The date of the marks should not be greater than today.
Suggestion model and CRUD. The text fields shouldn't be empty and the date shouldn't be greater than today.
Optional avatar attribute for students requested by the frontend.

You can test all of these features by performing all the CRUD operations.

Everything works as espected! Good work


App crashes when using ar on Android device.

There was no problem when I built and executed the Android environment for using viroreact in my existing project.
Afterwards, to run the code in App.js of the starter-kit, I imported only the code into my project and ran it, but there is no error log, and the app keeps forcibly shutting down.
Can you come up with a solution for this part? Minimum sdk is set. Is there anything else I should do?

same issue

Hi all, I've got a fix for this on #65. We've had one other user verify that this fixes things as I am currently unable to verify myself (my android device is in the mail). I'll keep this open until I personally verify it, but it would be great if you could try this PR out with an android device and comment on the PR saying that it either works or doesn't!

Please try this again with the latest version of NativeVision! I've published a new version https://www.npmjs.com/package/@viro-community/react-viro 2.41.0 this past week that fixes a lot of major issues.


fix(reactivity): no longer allows stop computed effect

Redo #10290, fix #10236

/ecosystem-ci run

https://github.com/vuejs/core/issues/10236#issuecomment-2222223511


[NEXTJS] Add configs folder for ts-nextjs

What this PR introduces?

Add configs folder for ts-nextjs
PR Checklist
Please check if your PR fulfills the following requirements:

[x] I have followed GitFlow pattern to create the branch
[x] I have added all the peerDependencies needed as devDependencies too
[x] I have added the new peerDependecies to the installation docs
[x] I have tested the updates and it works

PR Type
What kind of change does this PR introduce?
[ ] Hotfix
[ ] Bugfix
[x] Feature
[ ] Documentation update
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] CI/CD related changes

Does this PR introduce a breaking change?
[ ] Yes
[ ] No

Other information (Prints, details, etc)

Will be done in one single PR, "Update NextJs Configs"


Graceful handling of 409 Conflict when creating duplicate column

Hey,
Would it be possible to have the provider not treat HTTP 409s as an error? When I try to run terraform apply with an empty state and a dataset with columns already exists, I get this error:
╷
│ Error: 409 Conflict: Can't insert, column already exists with this key_name or alias.
│
│   on columns.tf line 15, in resource "honeycombio_column" "column":
│   15: resource "honeycombio_column" "column" {
│
╵

It'd be awesome if in this case it updated the existing column instead of having to manually import all of them.
Thanks!

Hi Tyler, thank you for your contributions! This request seems reasonable, I'm actually wondering why it doesn't already work that way... I'll check out your PRs in the next few days, at the latest end of next week.

Hi 👋.  Has there been a regression with this feature? I've hit the same 409 error, when Terraform tries to provision a column that exists already:
│ Error: 409 Conflict: Could not add or update the column: column already exists with this key_name or alias.
│
│   with honeycombio_column.myservice_columns["mycolumn"],
│   on honeycomb-dashboards.tf line 68, in resource "honeycombio_column" "myservice_columns":
│   68: resource "honeycombio_column" "myservice_columns" {

This is with v0.15.0 (June 2023) of the provider.

Hi, @mattjw!
The logic changed in v0.12.0 of the provider as it started doing column deletes and the behaviour of what amounted to "automatic import" is not a normal -- and arguably a dangerous -- Terraform workflow.
For the same reason, I'm pretty against any sort of allow_preexisting or create_if_missing options as I don't think they match the general Terraform workflow.

@jharley thanks for the reply. And that makes perfect sense. It did seem a little unusual to me too.
One a related topic, I was wondering if it's possible for the provider to validate the presence of required columns at plan-time? Rather than waiting to get a nasty surprise at apply?
Let me know what you think, and if there's appetite for this I'm happy to raise a feature enhancement request. (Presumably as an issue in this repo?)


UA and WA is inverted

@sinhat98,
I read the paper and found that UA and WA are inverted (both definition and coding in this report).
From the paper (page 3 left bottom):

The weighted accuracy (WA) is the average of the classification accuracy per class

WA should be the overall accuracy, not the average accuracy. The average accuracy (or balanced accuracy) should be named UA (unweighted accuracy) or UAR (unweighted average recall).
Reference (there are some, but I cited one below):
Xu, H., Zhang, H., Han, K., Wang, Y., Peng, Y., & Li, X. (2019). Learning Alignment for Multimodal Emotion Recognition from Speech. Proc. Interspeech 2019, 3569–3573. https://doi.org/10.21437/Interspeech.2019-3247
On page 3, the authors wrote:

weighted accuracy (WA) that is the overall classification accuracy and unweighted accuracy (UA) that is the average recall over the emotion categories.

So, in both the paper and this repo, UA showed WA and WA showed UA. Do you agree with this?

@bagustris
I agree that the common understanding between WA and UA aligns closely with our discussion. I apologize for any confusion.
I'd like to refer to a series of experiments detailed in a paper on data preparation, available at https://devblogs.microsoft.com/ise/2015/11/29/data-preparation-the-balancing-act/, which defines WA and UA as we've discussed.
It's important to note that while this paper maintains a consistent definition of WA and UA, I acknowledge that in many other papers, these terms might carry the interpretation you've highlighted.

@sinhat98
Thanks for making it clear.
Regarding the Microsoft article, I have already made an issue in their GitHub repo since last year (the link is on the bottom page of the article you shared).
https://github.com/ryubidragonfire/Emotion/issues/3


IE8 Bugfix

Reference #2

Make bower always fall back to angular 1.3.15 for attached libraries
Update tests to allow for new classes because of Angular 1.3x branch
Fix JSCS issues that stops the build

closes #2

Looks good Steven.


"After updateRecords(2468, "tracks", "Free"), tracks should have "1999" as the first element." still failing even with output correct.

Challenge Record Collection has an issue.
User Agent is: Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/52.0.2743.116 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

// Setup
var collection = {
    "2548": {
      "album": "Slippery When Wet",
      "artist": "Bon Jovi",
      "tracks": [
        "Let It Rock",
        "You Give Love a Bad Name"
      ]
    },
    "2468": {
      "album": "1999",
      "artist": "Prince",
      "tracks": [
        "1999",
        "Little Red Corvette"
      ]
    },
    "1245": {
      "artist": "Robert Palmer",
      "tracks": [ ]
    },
    "5439": {
      "album": "ABBA Gold"
    }
};
// Keep a copy of the collection for tests
var collectionCopy = JSON.parse(JSON.stringify(collection));

// Only change code below this line
function updateRecords(id, prop, value) {
  if (prop !== "tracks" && value !== "") {
        collection[id][prop] = value;
    } else if (prop == "tracks" && value !== "") {
        if (!collection.hasOwnProperty("tracks")) { // moved from outside loop
            collection[id]["tracks"] = [];
            collection[id][prop].push(value);
        } else {
          collection[id][prop].push(value);
        }

    } else { // removed (value === "")
        delete collection[id][prop];
    }

  return collection;
}

// Alter values below to test your code
updateRecords(5439, "artist", "ABBA");

@AeonBackflash there is an error in your code, namely that your nested if statement starting with  if (!collection.hasOwnProperty("tracks")) { does not correctly filter out the results. Please consider asking for help in the Gitter Help chat room.

As @dhcodes says, you have an error in your code. Please use the help chat as linked above for assistance with writing code when necessary. Issues are for reporting bugs or problems with the challenges or website.
Happy coding!

else if (prop == "tracks" && value !== "") {  // you said in this line my prop= "tracks" then let go inside the if
if (!collection.hasOwnProperty("tracks")) {  // the you said if tracks does not inside collection but with wrong syntax you should write if(!collection[id].hasOwnProperty("tracks"))
collection[id]["tracks"] = [];
collection[id][prop].push(value);
but totally you think wrong somewhere because if your value = null, you have to delete the item inside the object, so your first statement should be if(value = "") {delete collection[id][prop];

if(!collection.hasOwnProperty(“tracks”))  { // this line has error
collection[id][prop]=[];
}
instead of this you can write
if(!collection[id].hasOwnProperty(“tracks”))  { // we missed on which particular id's " tracks" we are updating
collection[id][prop]=[];
}
this works for me :)

Can someone explain the "!" before collection and its function, ty

@Xelepam
! is the logical NOT operator.
In this context, it would be the same as saying if the ID of the collection does not have the property "tracks" in human speak.


The identity error report not received anymore

Since January this year for our tenant, these reports are not received anymore, although we have recent sync errors shown on the O365 dashboard. We have a ticket with Microsoft, no solution until now.

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

ID: 6addd727-64bb-f71d-acf2-850220ba8973
Version Independent ID: 74342c05-c714-0cfc-8dd6-d29611cacaef
Content: Identity synchronization and duplicate attribute resiliency
Content Source: articles/active-directory/hybrid/how-to-connect-syncservice-duplicate-attribute-resiliency.md
Service: active-directory
Sub-service: hybrid
GitHub Login: @billmath
Microsoft Alias: billmath

@mihaibudurescu Thanks for your feedback! We will investigate and update as appropriate.

@mihaibudurescu Apologies for your trouble.  can you please send an email to azcommunity[at]microsoft[dot]com with your Case Number and the link of this thread so that I can check this internally.

Done, I've sent now the e-mail.
Thank you,

@mihaibudurescu We will now proceed to close this thread as we are discussing in over email. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Trakt.tv import doesn't import much info

Hi, finally with Lists added I started from scratch and tried to import my Trakt.tv info, but it seems that it doesn't import much info:
EDIT:
Peliculas = Movies
Programas = Shows
Temporadas = Seasons
Episodios = Episodes
Lista de seguimiento = Watchlist
Historial de visionado = Seen history
Puntuaciones = Scores

It doesn't import any episode watched nor any movie watched, while my trakt.tv profile has 2 pages of series progress and 678 movies watched:

In the logs I don't see anything relevant.
I'm using the last commit of the repository manually compiled, but after that I also tried with npm install -g mediatracker and in that version it has the same result.

I've saw in the Trakt.tv api that they send the information in pages, could be that MediaTracker is only reading the first page? I see that there are a few episodes and movies added as watched, and I checked my history and all the stuff before May 3 is not added, but all after May 3 included until yesterday (when I did the import) is added.
So I think the issue could be that is only reading the first page?

Thank you. Pagination is optional for watchlist and ratings, so I incorrectly assumed it's also optional for history, now I see it's not

not sure what exactly is going on with mine, as it's barely importing anything at all from trakt.. A couple things are just stuck at ?/{number} .. ontop of it not really importing fully or at all :/

This is running from source. I let it sit for a while in hopes that maybe it was still just importing, but nothing ever changed.

Hello, can confirm mine only gets 6 tv shows as well, and not much else.

Hi,
I am having problems importing from trakt too. I have a some personal list and nothing on my watchlist list and, of course, a bunch of watched movies/episodes.
When I try import from trakt I am able to successfully authenticate with trakt but then I am stuck on the importing page witch briefly states "exporting" to turn nearly immediately to "importing" without any other progress and no import ever happening.

Hi @bonukai , did you get any progress on this matter? After the data lose I would like to get all the info I can from Trakt to be sure that all these years of stats don't get lost.

I started working on that long time ago, it only needs some tests, I will try to finish it soon. I wish I could have more time to work on this project

I started working on that long time ago, it only needs some tests, I will try to finish it soon. I wish I could have more time to work on this project

Sure, I understand that. Thanks for the update!

@bonukai it is the same behavior with me. Only the contents of the watchlist are imported.
Thank you at this point for your efforts! I really appreciate it.
Can you tell when to test the revised import?

I am literally sick of Trakt. I mark things watched and later they don't show as watched.
I looked into the code, but I wouldn't know how to make the tests. I might be able to cobble the functionaility together, but you said you got that done already. Is there any way that I can help? Make some fake data file or something?

@phirestalker I've just published all the changes I was working on to a branch wip, including TraktTv import. There are few big changes there, if you could test it manually, that would be very helpful.

It seems that with that version its importing more info, but I have to execute several times the import to have all the media:
Second time of importing the same account (the first time I didn't made a screenshot, but it imported less items):

Third time:

Fourth time:

etc.
As you can see, on each import the quantity gets higher but never reach the maximum (for example, "Episodios" (Episodes) started on 1523 on the second import, and in the 4th reached 3603, but didn't reach the 6891 items that are on my trakt profile).

@bonukai I've found these little issues (the one on this issue and #488 & #489 ), but in general I'm really happy with this update, great job!

Fired up a docker from that branch. How do I know it is doing something? The numbers are not changing, but it says importing for the past 5 minutes.

I did the first one wrong. I had to edit the docker-compose.yaml to use the local Dockerfile. I see the same behavior as @BanCrash.
git show
git show
commit 99207bedd77a722d5056b607569dfa0ff7c5650f (HEAD -> wip, origin/wip)
Author: bonukai<EMAIL_ADDRESS>Date:   Sat Apr 22 15:27:25 2023 +0100

@phirestalker Could you please try again, I have just added some changes

I noticed it kept what it had before. Should I delete the db when I try again? If so, where can I find it?

You can use the same database, importing items again will add only those items that are not in your database

It almost imported everything on the first try. The second try it gave an error.
First time

Second time

@phirestalker this issue should be solved now

@BanCrash I've updated the how metadata is downloaded and this should fix the problem you are experiencing here, where you need to run the import multiple times

Now if item is not found by tmdb id it will also be search by imdb id, maybe this will help to import all items

Hi again @bonukai , I used the import on my previous database and it imported everything, but then for more testing I deleted the database and tried to import and I got this huge error: (I put in a .txt because it's so big)
Error.txt

@BanCrash I forgot there is a limit to how many records can be inserted at once, it should be fixed now, thank you.

I tried with the new updates. I did not delete my database, seeing if it would fill in the missing ones. I got an error that I have attached here.
MediaTracker error.txt

@BanCrash I forgot there is a limit to how many records can be inserted at once, it should be fixed now, thank you.

@bonukai Ok, now on the first attempt after deleting the database I got this:

Importing again doesn't have any change so it seems that this is the maximum I'm gonna get. Another suggestion arrives then, could you add to the log the items that are missing?
Debugging myself I achieve to print to the console four items:
Test1: Trakt 411426 Slug: vader-episode-i-2018
Tmbd: 560927
Test1: Trakt 185335 Slug: shrek-3d-the-story-continues-2003
Tmbd: 288306
Test1: Trakt 72222 Slug: huerfanos-electronicos
IMDB: tt3317336
Test1: Trakt 109068 Slug: overwatch
IMBD tt4332152
But there are a lot more missing and I don't know how to continue.
In any case thank you for taking time fixing this!

@phirestalker this have been now fixed.
@BanCrash Now after importing all not imported items will be displayed on the page. It looks like they are not in TMDb database and TMDb ids on Trakt are invalid for those items

@phirestalker this have been now fixed.
@BanCrash Now after importing all not imported items will be displayed on the page. It looks like they are not in TMDb database and TMDb ids on Trakt are invalid for those items

Just tested and it works like a charm, thanks!
For my side I think this issue can be closed, let's see what @phirestalker say.

I love it!
Quick question. For upcoming titles that are missing on TMDB, does it use the IMDB id and keep it?
Thank you for working on this.

No, the items the are not on the IMDb are simply omitted. After looking at some examples, I think this is the best solution.
For example Vader- Episode I has invalid TMDb id, just like Shrek +3D - The Story Continues. A Overwatch shows links to Overwatch game on IMDB, there are two separate entries for the same Tv Show Divorce and Divorce.
I think it's better to keep only TMDb as the only source

sounds right. Thanks again.

In any case when #56 is added we well be added to manually fix that so not a big issue. Closing this since it's already solved, thanks!


hass 2020.12.0 comes with TypeError

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 199, in _async_setup_platform
    await asyncio.shield(task)
  File "/config/custom_components/nsw_rural_fire_service_fire_danger/sensor.py", line 64, in async_setup_platform
    rest = RestData(
TypeError: __init__() missing 1 required positional argument: 'verify_ssl'```

This has already been fixed in the new repository: https://github.com/exxamalte/home-assistant-custom-components-nsw-rural-fire-service-fire-danger
I will add a link in this repository later, to point users to the new HACS-compatible one.


forms and ng-forms under ng-if directives do not get named field on $scope.

Not sure if this is intentional or not, but I have plenty of use cases where I need to create an entire form dynamically including the surrounding form or ng-form element/directive.
When I use ng-hide/ng-show, the $scope gets the usual 'nameoftheform' property on it.
That is however not the case when using ng-if.
In some cases, I MUST use ng-if so that the form (usually a nested ng-form) needs to be removed from the validation when it's not in view.

This is because ng-if creates a new child scope, and if your form is simple like myForm, then this will not be set on the parent scope (because of prototypical inheritance). A workaround for this is to handle form names like ng-models - always make them object properties, like obj.myForm, where obj is defined on your controller. This works because angular handles form names not as names, but rather as paths / expressions.
But I agree that this is unintuitive. I'd prefer ng-if to not create a child scope, but I don't know if this is possible.

The link is slightly out-of-date, but will help you understand scopes and why this is not an issue
https://github.com/angular/angular.js/wiki/Understanding-Scopes


Remove volatile keyword

Has been an error since 2014: https://github.com/D-Programming-Language/dmd/pull/3977

Auto-merge toggled on


I have a question, how can I change value interpolation from ${{ values.name }} to @@values.name@@ to do replace in the templates, it's possible ?
🎉 Proposal
For now, it's an informative question.
〽 Alternatives
No response
❌ Risks
No response
👀 Have you spent some time to check if this RFC has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Hey @astounds :wave:
Unfortunately, this isn't possible, and I don't think that we will ever want to make this configurable.
It introduces complexity that the template language isn't going to be standard in both the template.yaml or in the skeletons themselves for fetch:template. What's the use case, why do you want to change them, and specifically why @@ for beginning and end tokens?
If we lock this down and make it standard, theres better support for us to create migrations of templating syntax as we're also looking at moving away from nunjucks in future versions.


Broken in latest Firefox Nightly

Somehow the latest Nightly broke MaterialFox, it seems like all SVGs don't render anymore at all.
Since I don't really know anything about the Firefox-Theme-Development I can't really provide any more details or suggestions, sorry. But I'm definitely a huge fan of MaterialFox! I'm looking forward to use it again 😄

I think this issue somehow resolved itself 💯 🎉


Test Gradle plugin with AGP 8.2.0

Warnings

:warning:
This PR is approved with no milestone set. If merged, it won't appear in the detekt release notes.

  Generated by :no_entry_sign: dangerJS against e5c53481623b60bf334dec10e64ac122fdabf81f


PDB Error: Index Out of Bounds

Describe the bug
Trying to load debug symbols from Microsoft for mfc140.dll using the Universal loader. However during processing I receive an Index Out of Bounds error when processing the .pdb.
To Reproduce
Steps to reproduce the behavior:

Load MFC140.dll
Go to File -> Load PDB File...
Download or Load local symbols
See error

Expected behavior
PDB file loads successfully.
Screenshots

Attachments
log.txt
Environment (please complete the following information):

OS: Windows 10 Pro 22H2
Java Version: Oracle Corporation 17.0.6
Ghidra Version: 10.2.3
Ghidra Origin: Official Github

Additional context
Add any other context about the problem here.

Just to add more detail. I managed to get MSDIA support working and have successfully loaded that way. So seems to be something specific to the universal loader

Issue is due to member pointer being a 16-byte pointer, which is running up against some Ghidra limitations.  I need to discuss with team members short- and long-term solutions

Just to add more detail. I managed to get MSDIA support working and have successfully loaded that way. So seems to be something specific to the universal loader

MSDIA solution is probably not aware of the pointer size indicated in the PDB and thus throws it away.
One short-term solution I'm considering is to create an 8-byte pointer and pad with an additional 8-byte extension field; then, if possible, try to stage locations in non-PDB code that might detect if the extension field ever gets a non-zero value.  I'm not sure if this non-zero checking would be possible, but could server as a warning.
See this reference as well: https://stackoverflow.com/questions/29607359/size-of-pointer-to-member-function-varies-like-crazy

Thanks for the update ghizard, hope you manage to find a good solution.

So, we do not necessarily have a pointer that is larger than 8 bytes here (but I don't know for sure).  We do not have anything formalized in terms of the Pointer to Member Function (PMF) and Pointer to Data Member (PDM) types, and since the tool-chains get to define how these are implemented, these could vary greatly.  Given the parameters that I see in the PDB data, there could be dozens of varieties that are impacted by the memory/address model and the inheritance model of the underlying and containing classes.
I tried teasing some truth out of these for a couple of days, and realized that it will take a lot more study to get it right.  However, MSFT gives the size each PMF/PDM used, so the easiest thing I can do at this time is to define a structure of this size and not specify any of its members.  The members could be one or more pointers of whatever size is inherent to the binary along with a number of flags and attributes.  I could also try to put a comment on a member PMF/PDM of a class/struct and I can also place a description on the PMF/PDM struct type that I create (these comments and descriptions will likely change as the tool progresses).  I feel that this can at least allow users to make some sense of what these are and also enable further research towards formalizing these in the tool.  I have not put too much into looking at the Itanium C++ ABI and whether the referenced layouts provide any specific details for the tool-chains coming from that direction, but I'm interested in digging into this further from multiple directions as time permits.
Note, that the structure solution that I suggest here, would cause apparent regressions if some particular varieties of PMF/PDM are a simple pointer.  In these cases, we are already laying down a simple pointer, and will instead be placing a structure in its place.  Note, however, the the pointed-to-type for these pointers might not have been accurate.
Thoughts?


Update back/white logo per suggestion

Hi @benjdlambert could you check if this can work?  Thanks!

Thanks @benjdlambert for the reminder!  Hope I fixed it now.


Could not build module FBSDKCoreKit

I have the following setting but still not works

@EasonWang01  https://www.youtube.com/watch?v=mEKpjN1pm2s&t=1s&spfreload=10 , This is best tutorial so far, he has explained all the setup of Facebook SDK from scratch

@EasonWang01
Hi, I had this issue also. My dependencies in package.json:
"react": "16.5.0",
"react-native": "0.57.0",
"react-native-facebook-login": "1.6.0"

I have fixed it in the following way:

Please follow all instructions from https://github.com/magus/react-native-facebook-login#setup
Please note, that according to setup instructions you need to use the following path to Framework Search Paths:
$(SRCROOT)/../node_modules/react-native-facebook-login/FacebookSDK

Uncomment this line in the Podfile:
use_frameworks!

Run pod install again

It works for me.
My Podfile:
# Uncomment the next line to define a global platform for your project
platform :ios, '9.0'

target 'HopNBr' do
  # Uncomment the next line if you're using Swift or would like to use dynamic frameworks
  use_frameworks!

  # Pods for HopNBr
  pod 'GoogleSignIn'
  pod 'FBSDKLoginKit'

end

resolved this issue in the following way


add:hammadzafar05

Description
I only made changes in 2 files
hammadzafar05.json and ProfilesList.json
Related Issues
any changes in other files might have been caused by code formatting.

@hammadzafar05 Thank you so much for your contribution! Your efforts are greatly appreciated and will go a long way in improving our project. Please feel free to share it with others and help us grow even further!


Doesn't work with React 0.12.2

Since getDefaultProps() is now called by React.createClass(), and Sortable.getDefaultProps() tries to access this as if it was an element instance everything breaks miserably :(

Exactly the same problem here too! :D Should be using component lifecycle events and managing it's own state, not working from default props - that's for the component itself.

Anyone have a fix for this?

https://github.com/RubaXa/Sortable/blob/master/react-sortable-mixin.js

current version works with react 0.14


[BUG]: Affine version in requirements.txt doesn't exist

Thank you for helping us improve solaris!
Summary of the bug
In requirements.txt affine==2.2.3 doesn't exist
Steps to reproduce the bug
pip install -r requirements.txt

Buggy behavior and/or error message
Please describe the buggy behavior and/or paste output here.
Collecting affine==2.2.3 (from -r requirements.txt (line 18))
  ERROR: Could not find a version that satisfies the requirement affine==2.2.3 (from -r requirements.txt (line 18)) (from versions: 1.0, 1.0.1, 1.1.0, 1.2.0, 1.3.0, 2.0b1, 2.0b2, 2.0.0.post1, 2.1.0, 2.2.0, 2.2.1, 2.2.2, 2.3.0)
ERROR: No matching distribution found for affine==2.2.3 (from -r requirements.txt (line 18))

Expected behavior
I expected to be able to install the dependencies
Environment information

OS: Ubuntu
solaris version: master (cf372fe5302e62a4b96dbbf291dbbc228b9aa866)
python version: 3.7
version of any relevant dependencies (optional - we may ask for this information later if not provided)

Additional context
I was doing this inside of a docker file, but looking online it appears as though no 2.2.3 version of the affine library exists on pip.

@dannyjeck-matroid thanks for the catch. I'll fix this today.

Resolved with #264. Bumped the affine version to 2.3.0.


Coverage reports

Add coverage reports

Coverage increased (+9.5%) to 89.573% when pulling 66c15fa2bca616fd6f99f92449fb72d868757e37 on development into 87d6542c48c005d2d3252f89fac17998dea21af5 on feature/coverage.

Coverage increased (+9.5%) to 89.573% when pulling 66c15fa2bca616fd6f99f92449fb72d868757e37 on development into 87d6542c48c005d2d3252f89fac17998dea21af5 on feature/coverage.

Coverage increased (+9.5%) to 89.573% when pulling 66c15fa2bca616fd6f99f92449fb72d868757e37 on development into 87d6542c48c005d2d3252f89fac17998dea21af5 on feature/coverage.

Coverage increased (+9.5%) to 89.573% when pulling 66c15fa2bca616fd6f99f92449fb72d868757e37 on development into 87d6542c48c005d2d3252f89fac17998dea21af5 on feature/coverage.


Remove switch between Rucio and PhEDEx

Fixes #10095
Status
not-tested
Description
Remove environment variables check for Rucio, thus setting all the data management functionality to use Rucio by default.
Not all the unit tests were set to use Rucio - I guess many were still using Phedex mock data- so I had to make changes to the EmulatedUnittests module, mock rucio data and APIs, and some unit tests.
Note: even though we apply a mock patch to the PileupFetcher, it still doesn't use the mock Rucio data. My suspicious is that we do not explicitly import that module from the top of the module, but instead we use the StepFactory to import modules from arguments...
Is it backward compatible (if not, which system it affects?)
yes (but no in the sense that we can no longer switch between PhEDEx and Rucio, not that that matters anymore!)
Related PRs
Initial mocking logic performed in https://github.com/dmwm/WMCore/pull/10099
External dependencies / deployment changes
Deployment changes in:
https://github.com/dmwm/deployment/pull/1033
K8s deployment changes (transparent/useless):
https://gitlab.cern.ch/cmsweb-k8s/services_config/-/merge_requests/71
https://gitlab.cern.ch/cmsweb-k8s/services_config/-/merge_requests/70
cms-bot changes applied to the jenkins job (DMWM-WMCore-PR):
https://github.com/cms-sw/cms-bot/pull/1491

test this please

test this please

test this please

test this please

test this please

All right, Shahzad has updated the configuration file used by jenkins, giving it another try.

test this please

test this please

test this please

test this please

sigh... of course none of those tests fail on my docker container... keep looking!

Now testing without my cms-bot patch: https://github.com/cms-sw/cms-bot/pull/1491

test this please

and it does not work without having the correct rucio setup in jenkins. I'm re-enabling my cms-bot once again to confirm it.

test this please

Terrible effort estimation here. Something that was meant to take less than a morning, took me between 2 and 3 days of work...

cms-bot PR has been merged and this is good to go.


Update to latest sprig

Just want to use some newer functions in the latest sprig
/cc @technosophos

@binoculars Im not sure we want to use an arbitrary commit point from spring master. AFAIK we have been using official releases.

In that case, I would definitely appreciate someone with write access to the sprig repo cutting a release.
Other deps are pinned to SHAs, so I figured that it didn't matter much. Either way, the goal is to get the new/updated functions in helm.

I'll see what I can do.

@binoculars I just cut a sprig release. New version is v2.13.0. Can you update to that?

@mattfarina thanks so much. Got the test passing now. glide up was bumping the kube version to a beta and was causing build errors. Should be g2g now.

Resolves #2902

Anything else I should do to get this merged?


Inline Datetimelike accessor documentation to Pandas'

This PR targets to inline Datetimelike accessor documentation to Pandas'
The problem was that it seems like we can't access property's property (Series.dt.date for instance).
I mimicked Pandas' approach (class attribute -> property). Plus, it will cache once it's called.
Another problem was that the table is located in the center. I don't know why there's difference so I just edited CSS to make the tables alined left like Pandas'. I think it's hacky way to fix.
Pandas:

After this PR:

cc @icexelloss too

Codecov Report

Merging #404 into master will decrease coverage by 0.08%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #404      +/-   ##
==========================================
- Coverage   94.66%   94.58%   -0.09%
==========================================
  Files          42       42
  Lines        4615     4615
==========================================
- Hits         4369     4365       -4
- Misses        246      250       +4

Impacted Files
Coverage Δ

databricks/koalas/series.py
92.85% <100%> (ø)
:arrow_up:

databricks/koalas/datetimes.py
94.02% <100%> (ø)
:arrow_up:

databricks/koalas/__init__.py
89.28% <0%> (-3.58%)
:arrow_down:

databricks/koalas/tests/test_indexes.py
93.39% <0%> (-2.84%)
:arrow_down:

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 e5d7bd8...56161c2. Read the comment docs.

LGTM as well!
Although I am curious what's the difference between property and CachedAccessor that makes sphinx happy?

Another point is that I think this is not public pandas API. Maybe we want copy the class out to prevent issues in case it is moved (although probably unlikely). pandas also ported it from xarray

Although I am curious what's the difference between property and CachedAccessor that makes sphinx happy?

I am not sure .. :-). I am just guessing we can't go through something happens in runtime at property when generating docs. Property doc can be found but property.property doc can't be found statically (I guess?).

Another point is that I think this is not public pandas API. Maybe we want copy the class out to prevent issues in case it is moved (although probably unlikely). pandas also ported it from xarray

Nice point. Let's do that when something is found actually broken. Should be fine.

I merged this first. Please submit a separate PR to inline the accessor, since it's not a public API in pandas.


feat: stale issues and PR with not activity

Describe your changes
Add an action to close PR without activity in the lang/es

cc @CathPag @seokho-son @jihoon-seo @IdealUsrname how do you feel about this? Im mean the localization teams could handle their contribution workflow like they want to, but I feel like closing stale PRs is not the right way to go.

So this is something that automates that process, right? It would be a bummer to close a stale issue that has good content but am not sure what best practices generally are to handle these things.

cc @CathPag @seokho-son @jihoon-seo @IdealUsrname how do you feel about this? Im mean the localization teams could handle their contribution workflow like they want to, but I feel like closing stale PRs is not the right way to go.

This is the situation when the PR is stale, because the contributor does not answer the messages or apply the suggestions. We can discuss this in the glossary meeting, what will be the best approach in this specific situation?
cc: @electrocucaracha

Keeping PRs open for a while could reflect a low level of participation in the project and increase the technical debt on it. Having a third party actor who does that can alleviate frictions among the community members. Besides, any PR can be re-opened any time.

I'm ok with what the projects leads think is best for their process :)


nederob.nl - ##.featured-content filter

Sample URL: https://www.nederob.nl/2019/09/09/brave-browser-most-popular-in-japan/
The article at the above webpage won't appear due to the ##.featured-content filter. This can be remedied by adding the<EMAIL_ADDRESS>filter. This will whitelist the filter on nederob.nl.

Fixed in 3259b149962d778e04251cc84ed9c40e8bcc00c3


cargo vet is busted on mozilla-central

STR:
(1) fresh checkout of m-c
(2) cargo vet init
(3) cargo vet
Results:
Vetting Failed!

0 unvetted dependencies:

dependency tree:
  smoosh:0.1.0
    jsparagus:0.1.0
      jsparagus-ast:0.1.0
...

Very strange...

fixed in #47


add height sizing to detail view so that it renders all detail items

Description

In the UI, when a VM instance has more than one NIC, the NICs screen does not display all the details of the first NIC. The last few rows of text overlap with the second NIC's text. This is due to the incorrect placement of the second NIC's bar. The default height sizing does not allow enough head space to display all the details of the first NIC before displaying the second NIC's details.

Expected Behaviour:
When a VM instance has more than one NIC, the NICs screen must allow enough height size to display all the details of the first NIC before displaying the second NIC's details. The last few rows of text must not overlap with the second NIC's text.
Current Behaviour:
When a VM instance has more than one NIC, the NICs screen does not allow enough height size to display all the details of the first NIC before displaying the second NIC's details. The last few rows of text overlap with the second NIC's text.
To Reproduce:
In the UI, on the lefthand side, select the Instances link. On the Instances screen, select a VM instance that has more than one NIC and then select the NICs screen. Scroll down to where the second NIC bar is displayed. Notice that it does not display all the details of the first NIC correctly. The last few rows of text overlap with the second NIC's text.
Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

GitHub Issue/PRs

Screenshots (if appropriate):

How Has This Been Tested?
Manually (See screenshot above):

In the UI, on the lefthand side, select the Instances link. On the Instances screen, select a VM instance that has more than one NIC and then select the NICs screen. Scroll down to where the second NIC bar is displayed. Notice that it now displays all the details of the first NIC before displaying the details of the second NIC. The last few rows of text do not overlap with the second NIC's text.

Environment:
CloudStack version 4.11, KVM hypervisor.

Checklist:

[x] I have read the CONTRIBUTING document.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
Testing
[ ] I have added tests to cover my changes.
[ ] All relevant new and existing integration tests have passed.
[ ] A full integration testsuite with all test that can run on my environment has passed.

@dhlaluku @DaanHoogland please review.


Add Google Drive Persistence

See https://github.com/source-academy/cadet-frontend/pull/562

Postponed for now. Keep the label "_postponed" and re-visit after Sem 1 2019/20. See wiki on policy on postponed Issues.

We have this now.


Q: is it possible to ignore/skip directories with ZipDeploy?

I want to deploy two zips into the same AppService. The backend part should go to /home/site/wwwroot/api and the frontend part directly to /home/site/wwwroot.
This works fine when deploying the frontend first and then the backend with the path query param but when I deploy the frontend after the backend then the deployment deletes the api directory. I'm following the instructions from here.
Is there a workaround for my problem? I really like to deploy backend orfrontend without  touching the other parts. With msdeploy this is possible by using the -skip:Directory parameter.

@SeverinFriede we have used virtual directories to handle this in our use cases. https://docs.microsoft.com/en-us/azure/app-service/configure-common?tabs=portal#map-a-url-path-to-a-directory
You can have the URL mapping not necessarily follow the file & folder structure, and you should be able to deploy to specific virtual directories instead of the whole site. We use Azure Pipeline's built-in Azure App Service Deploy Task which has a VirtualDirectory parameter.

Hi
If the problem persists and is related to running it on Azure App Service, please open a support incident in Azure:
https://learn.microsoft.com/en-us/azure/azure-portal/supportability/how-to-create-azure-support-request
This way we can better track and assist you on this case
Thanks,
Joaquin Vano
Azure App Service


⚠ Bofrost.at has degraded performance

In d00483c, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1145 ms

Resolved: Bofrost.at performance has improved in 292029d after 7 minutes.


Adapt to new datascheme

With release V29 https://github.com/covidgraph/motherlode/releases/tag/V29 which is deployd to the dev database ( https://db-dev.covidgraph.org ) atm (as 07.06.2021) there are some minor changes in the BIOBASE loader datascheme.
Task to adapt:

[x] @mpreusse  needs to outline the changes
[ ] The app needs to be adapted to these changes
[ ] The app can be tested against the dev db
[ ] The DEV data and the app can be pushed to PRD ( when https://github.com/covidgraph/app-react/issues/21 is resolved)

Here is what changed in the data model:
There used to be the label :GeneSymbol which contained names for genes. This was a legacy model that made it into CovidGraph because we wanted to get started as soon as possible.
There is also the label :Gene which so far contained IDs for genes.
To make the model easier we renamed :GeneSymbol to :Gene and added a proeprty type='symbol' for all nodes that used to be :GeneSymbol.
For queries:
(g:Gene {type: 'symbol'}) == (g:GeneSymbol)

@motey @mpreusse - the refactoring seems to have worked, what's seems very strange is that if you search for ace2, now, you get two results (ACE2 and Ace2) - this is somewhat OK (though I guess they should be the same), however what's more questionable is that they do not seem to be related very closely - if you search for the related entities they seem almost disjoint, whereas I would have expected to find the same neighbors for these two "different" genes. Please investigate.

Once you want to move staging, to prod, I will have to update/redeploy the app, again. So let me know and give me some head-start.

@yGuy one is the gene in human and the other is the gene in mouse 😄
We added so much stuff I forgot about that ..
The genes have a property called taxid. That's a unique ID for the taxon a.k.a. the species. '9606' is human and '10090' is mouse.
We could add another filter to the queries to avoid additional complexity:
{g:Gene {type: 'symbol', taxid: '9606'})
In general it's incredibly interesting to be able to jump between mouse and human. There is so much evidence that wasn't reproduced in human tissue yet. So many ideas for the future 😄

OK - TBH, I don't know what more to do, now. I'd say this task is resolved, unless someone tells me that I need to somehow distinguish between the different species in the search functions and result tables (and in that case I would need clear instructions and input). From a completely objective point of view, the app has been adapted to the new data model and it works (and it's actually correct in displaying two different "ACE" genes and that they look unconnected because they are unconnected). So I think this can be closed.


todo: cruft cleanup and move theme into /themes directory

This theme lives at the top of the repo, rather than inside /themes as Hugo themes are expected to. It works because Hugo is flexible, but it makes it harder to keep theme and content clearly separated, or to change themes with the same content.
There's almost certainly other cruft in this repo I should have deleted, but haven't found yet

Fixed in https://github.com/misterbisson/securitysurvey.dev/pull/16


month/day/year date format for determinatons

Describe alternatives you've considered
Please change the DATE to this format mm/dd/yyyy
Managers need this date for correspondence with applicants.

Additional context
Add any other context or screenshots about the feature request here.
Priority

High (keeping you from completing day-to-day tasks)

Affected roles

Staff

Ideal deadline
January 2022

In the interests of good documentation on issues - this was discussed on a standup call the week of the 13th of December re. the uses of 'fuzzy time' and 'non fuzzy time' as well as process and usefulness from both a user pov and organisational (OTF) pov.


problem for SummaryGraph

I have a very simple net with shortcut structure.However , there is a loop if use summarygraph。What should I do to reduce this loop?Thanks
import torch
import torch.nn as nn
class Net(nn.Module):
def init(self):
super(Net,self).init()
self.conv1 = nn.Conv2d(3, 64, 7, stride=2, padding=3, groups=1)
self.layer1_0_conv1 = nn.Conv2d(64, 64, 1, stride=1, padding=0, groups=1)
self.top_level_op = nn.AvgPool2d(kernel_size=7)
self.last_linear = nn.Linear(16384, 1000)
def forward(self,x):
x_1 = self.conv1(x)
x_2 = self.layer1_0_conv1(x_1)
x_4 = x_2 + x_1
x_5 = self.top_level_op(x_4)
x_6 = x_5.view(x_5.size(0), -1)
x_7 = self.last_linear(x_6)
return x_7

Hi @wuzhi19931128 ,
Could you please add the code where you're using the SummaryGraph?
Thanks,
Lev

Sorry,  I found the reason.  I wrote a program to general pytorch model definition from distiller summary graph. So one layer of my net is named by "top_level_op" . When Summary graph analysis
this net , it's confused by layer and pytorch op . When I change the name , it acts normally.


Allow Serialize/Deserialize into a JsValue

We should have an option to convert an arbitrary object to a JsValue via Serialize an Deserialize. It may not be quick but it's often quite useful!

I want to take a stab at this and have a couple of questions.
Would you be in favor changing JsValue into an enum to make it easier to match based on its type during serialization?
It's not apparent how we deal with objects and arrays right now (if at all). Would you mind providing some instructions for how that would work?

I've posted an initial pass of this API at https://github.com/rustwasm/wasm-bindgen/pull/171