Getting confusing error

Bug or feature request?
More bug, but probably just need some help figuring this out.
Description:
Was doing some refactoring and upgraded xstate to 4.6.2 and now I'm getting this crazy error on React Native.
actions_1.invoke is not a function. (In 'actions_1.invoke(invokeConfig)', 'actions_1.invoke' is undefined)

The stack trace points me to StateNode.js:139:36, but nothing is jumping out at me there as to what the issue might be. I'm pretty sure the Machine it's complaining about is the following:
import {
  MachineConfig,
  ActionFunctionMap,
  Machine,
  ServiceConfig,
  ConditionPredicate,
  EventObject,
  InvokeCreator
} from "xstate";
import { SalesorderlineShape } from "../../templates/salesorderline";
import { assign, log } from "xstate/lib/actions";
import { hasLocal } from "../FetchAndMerge";
import { initCart } from "./actions";
import refreshCart from "../CommonActions/refreshCart";
import { SalesorderShape } from "../../templates/salesorder";
import { PosProfileShape } from "../../templates/posProfile";
import { StoreShape } from "../../templates/store";

export interface CartInitSchema {
  states: {
    checkContextCart: {};
    checkLocal: {};
    initCart: {};
    refreshCart: {};
    initCartFail: {};
    refreshCartFail: {};
    done: {};
  };
}

export interface CartInitContext {
  // determines whether you can look at a closed cart from here.
  // If false and the cart in context is closed, it will init
  // a new cart
  allowClosed: boolean;
  posProfile: PosProfileShape;
  shoppingCart: SalesorderShape;
  items: SalesorderlineShape[];
  cache: boolean;
  error?: any;
}

export const initialContext: CartInitContext = {
  allowClosed: true,
  posProfile: null,
  shoppingCart: null,
  items: [],
  cache: true
};

export const Chart: MachineConfig<CartInitContext, CartInitSchema, any> = {
  id: "cart",
  initial: "checkContextCart",
  context: initialContext,
  states: {
    checkContextCart: {
      on: {
        "": [
          { target: "refreshCart", cond: "hasContextCart" },
          { target: "checkLocal" }
        ]
      }
    },
    checkLocal: {
      invoke: {
        src: "checkLocalCart",
        onDone: [
          {
            target: "refreshCart",
            actions: "updateCart",
            cond: "isCartOpen"
          },
          { target: "initCart" }
        ],
        onError: "initCart"
      }
    },
    initCart: {
      invoke: {
        src: "initCart",
        onDone: {
          target: "fetchCartItems",
          actions: "updateCart"
        },
        onError: "initCartFail"
      }
    },
    refreshCart: {
      invoke: {
        src: "refreshCart",
        onDone: {
          target: "fetchCartItems",
          actions: ["updateCart", log((ctx, evt) => evt, "REFRESHED CART")]
        },
        onError: "refreshCartFail"
      }
    },
    initCartFail: {
      onEntry: log((ctx, evt) => evt, "INIT FAIL"),
      type: "final",
      data: {
        error: (ctx, evt) => evt.data
      }
    },
    refreshCartFail: {
      onEntry: log((ctx, evt) => evt, "REFRESH FAIL"),
      type: "final",
      data: {
        error: (ctx, evt) => evt.data
      }
    },
    done: {
      type: "final",
      data: {
        shoppingCart: ctx => ctx.shoppingCart
      }
    }
  }
};

export const actions: ActionFunctionMap<CartInitContext, EventObject> = {
  updateCart: assign({ shoppingCart: (ctx, evt) => evt.data })
};

export const services: Record<string, ServiceConfig<CartInitContext>> = {
  checkLocalCart: (ctx: CartInitContext) =>
    hasLocal("shoppingCart", (cart: SalesorderShape) => {
      if (typeof ctx.posProfile.channel === "string") {
        return cart.channel === ctx.posProfile.channel;
      }
      return cart.channel === (ctx.posProfile.channel as StoreShape).$uri;
    }),
  initCart: ((ctx, evt) =>
    initCart({
      posProfileURI: ctx.posProfile.$uri
    })) as InvokeCreator<any, any>,
  refreshCart: (ctx =>
    refreshCart(ctx.shoppingCart.$uri, ctx.cache)) as InvokeCreator<
    CartInitContext
  >
};

export const guards: Record<
  string,
  ConditionPredicate<CartInitContext, any>
> = {
  hasContextCart: ctx => {
    return ctx.allowClosed == null || ctx.allowClosed
      ? !!ctx.shoppingCart
      : ctx.shoppingCart && ctx.shoppingCart.status === "Open";
  },
  isCartOpen: (ctx, evt: EventObject & { data: SalesorderShape }) =>
    evt.data.status === "Open"
};

export default Machine(Chart, { actions, services, guards });

This was working before the upgrade, and I don't think I changed this particular file in my refactor. I'm wondering if something about invoking Promises changed between version 4.3.2 and version 4.6.2. Any help would be greatly appreciated. Also, I blew away my node_modules once and reinstalled hoping that would solve the problem, but I'm still stuck here. Nothing is jumping out at me as obviously wrong, but if someone could even point me in the right direction that would be awesome. Thanks!

EDIT: I'm pretty sure this was a code caching problem on my end. It went away when I forced the cache to clear. I thought I had cleared it before, but apparently not.

Can you create a CodeSandbox (XState template) reproduction of this?
And I don't see where actions_1.invoke(invokeConfig) is being called, maybe you left that call out? That would be the issue.

This may be related:
I’m running into a an issue with spawning a service when using 4.6.2. It works fine on 4.6.0.
The spawned service’s state is missing data.
I’ve been able to replicate the bug against the todo example when using 4.6.2
https://codesandbox.io/s/xstate-todomvc-nxxmd

Okay, well, my error went away.  I was getting a new one though, and I think I've tracked this one down to the source.
I was getting an issue when a promise rejected, and the error I got was related to React Native's own polyfill of Object.assign. It fails on hasOwnProperty. I'm not sure if this is still an issue in the latest React Native, but I'm using expo so I'm a couple versions behind.
The offending code in RN is here: https://github.com/expo/react-native/blob/exp-latest/Libraries/polyfills/Object.es6.js#L54
The error I get is: "One of the sources for assign has an enumerable key on the prototype chain. Are you trying to assign a prototype property? We don't allow it, as this is an edge case that we do not support. This error is a performance optimization and not spec compliant."
For the record, the key it's breaking on is inert on a state object. Not sure what that's used for, but that's the one it doesn't like.
I think this is related to the new spawning behavior in 4.6.x. I downgraded to 4.5.0, as I wasn't using the actor model stuff anyway and that fixed the issue for me. I don't really expect this library to address the weirdness in React Native, but I wanted to leave this comment here for posterity. I'm not even sure if it could be fixed from this end without breaking something else.
Maybe we can put it in a troubleshooting doc for any other poor souls that run into this?

I could also open a new issue and close this one since it appears to be completely unrelated.

Yeah, let's do that. We should investigate the .hasOwnProperty issue more.


whitespace, blanklines or comments in config file leads to fails pach…

The bash script to grab aws credentials from ~/.aws/credential file does not work when you are working with more accounts or if your file is manually crafted with new lines or has more white space.
Also, the delete steps are a bit misleading because the state bucket and storage bucket had to be removed manually.
This PR fixes both minor issues.

Hi @suneeta-mall, many thanks for the pull, we'll get it merged ASAP. Before we can do that though we'll need you to sign our CLA: https://pachyderm.wufoo.com/forms/pachyderm-contributor-license-agreement/

@jdoliner Thanks for the CLA I have signed it up now!

@jdoliner Can this be merged? Anything I need to do for the build errors ?

Hey @suneeta-mall thanks for following up, and sorry for taking a while to get back to this. Not sure if this failure is something transient, or something that actually needs to be fixed. I've just restarted the CI build

@suneeta-mall very sorry for the delay on this. I'm rerunning CI right now.
@gabrielgrant there's actually no need to create a second PR, if you do all steps for creating the second PR up to but not including actually creating the PR then one of the 2 CI jobs (the lower one) will be able to see the enterprise secret.


Connectivity issues when pairing my Nano S with MetaMask, MyEtherWallet or any other web-based wallet.

Problem
I'm trying to connect my Nano S with a web-based wallet (like MetaMask, MyEtherWallet, etc.)
How I'm connecting

Open MetaMask and press connect Hardware Wallet
Wait for the windows security prompt to enter your security key
Unlock my ledger and open Ethereum app

Actual Behaviour
The windows security prompt keeps refreshing over and over.

I am having the same problem. VERY frustrating. When is this going to get fixed?

I am having the same problem. VERY frustrating. When is this going to get fixed?

They don't even respond. What do you expect?


New version with compat for ArchGDAL v0.10

Would be nice to avoid version conflicts (you can add master, but it's easier for new users this way)

Codecov Report

Merging #390 (3b39c4d) into main (103b4f8) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #390   +/-   ##
=======================================
  Coverage   79.33%   79.33%
=======================================
  Files          38       38
  Lines        3135     3135
=======================================
  Hits         2487     2487
  Misses        648      648

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

These doctest problems are unrelated.


Reading msg file, Chinese garbled characters

Using simple Java mail to read the exported msg file from Outlook, but the HTML data in it is garbled in Chinese. The version of simple Java mail used is 6.6.1.

Thank you for contacting, but there's not much for me to go on, here (your image won't load, either). Please also try with the latest version, since 6.6.1 is rather old and since then there have been developments in the underlying Outlook support. If that still fails for you, you could really help me help you by providing a problematic .msg that demonstrates this.

There are error messages when using the lastest version.
Exception in thread "main" java.lang.NoClassDefFoundError: jakarta/mail/MessagingException
at com.eagle.core.utils.DateUtils.main(DateUtils.java:209)
Caused by: java.lang.ClassNotFoundException: jakarta.mail.MessagingException
at java.base/jdk.internal.loader.BuiltinClassLoader.loadClass(BuiltinClassLoader.java:581)
at java.base/jdk.internal.loader.ClassLoaders$AppClassLoader.loadClass(ClassLoaders.java:178)
at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:521)
... 1 more

I am using the jdk11 version, and the springboot version is 2.7.18

msg.zip
The msg.zip contains two msg files. After the program is parsed, garbled.msg is garbled in Chinese, while un_garbled.msg is not garbled.

I upgraded to 8.10.1, but the Chinese characters are still garbled. The JDK version is 11. The springboot version is 2.7.18.

msg.zip
Hi Bottema,
Thanks for replying me, above is the msg file. And I have upgraded to the new version. When I send the msg to users to test, the Chinese text showed as below, still garbled. Could u help to give some suggestion? Great thanks.

Ok, I've looked into it and determined this is a bug in rtf-to-html, which is used in outlook-message-parser, which is used by Simple Java Mail.
Please refer to https://github.com/bbottema/rtf-to-html/issues/13.

Fix released in 8.11.0.


Fix URL encoding issue

This python snippet doesn't save since %05 represented a URL encoded character which breaks dom-to-image
for linenumbercount, filename in loclist:
    print "%05d lines in %s" % (linenumbercount, filename)

Possible solutions:

[ ] find a way to escape the encoding for this pattern
[ ] remove patterns like these from the code before saving

Has #230 solved this?


include classes which not used in game

hscript: 📈

couldn't you just add them here instead?

couldn't you just add them here instead?

considering theres probably hundreds of libraries Psych Engine doesn't use its probably more efficient to do this :shrug:

I mean, all the macro does as far as I know is just import classes in a more sophisticated way, that hscript I think can use?
but this doesn't seem as far fletched as just adding them into the interpreter

for example u cant import FlxBackdrop if source code not used this class

for example u cant addHaxeLibrary('FlxGlitchEffect', 'flixel.addons.effects.chainable') if source code not used this class

okay but like, why not just add them to the interpreter? It’d likely be the same thing as what you did

okay but like, why not just add them to the interpreter if you did use source? It’d likely be the same thing as what you did

bro like its hundreds of lines, and some classes can have same name but other package

okay but like, why not just add them to the interpreter if you did use source? It’d likely be the same thing as what you did

bro like its hundreds of lines, and some classes can have same name but other package

hmm, I cannot argue with that

#12848


Update gpresult.md

We have had admins in App Assure ask about this several times; it would be nice to document that gpresult on ARM64 platforms only supports the /h switch by design.  Confirmed with engineering already in bug 31245712.

@ChrisOfTheLewi : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

@sethmanheim : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Periods in class selectors are not supported

I've been using the excellent Base16 Twilight Dark color scheme, and have found that while periods in class selectors are supported in CSS files, they aren't supported in Sass files.
I believe this is supported in CSS by the Attribute ID (either entity.other.attribute-name.id or punctuation.definition.entity—my guess is the latter).
CSS (notice the blue period between the "a" and the "cta-button" class):

Sass (period between "a" and the "custom" class is the same color as the class):

This package considers the . is part of the class selector as the # is part of the id selector and the :: is part of the pseudo-element selector.  So they shouldn't be highlighted in a different colour.
Highlighting Sass selectors also need to consider interpolations,  e.g. .class-#{$variable}, and highlight the punctuations and selector name in the same colour can make things easier :D


Interface can't be used as nullable resolver

Hello,
I'm currently trying to use a go-interface as nullable resolver.
The structure of my code is like this:
type Foo {
   getBar(): Bar
}

type Bar {
   doBarThings(): Bool!
}

type Foo struct {}

func (f *Foo) GetBar() Bar {
	if someCondition {
		return &SpecificBar{}
	} else {
		return nil
	}
}

type Bar interface {
   DoBarThings() bool
}

type SpecificBar struct {}

func (sb *SpecificBar) DoBarThings() bool {
   // ...
   return true
}

This whole thing works without any problems, as long as Bar is not nil,
in which case I get graphql: panic occurred: reflect: Method on nil interface value.
Replacing
GetBar() Bar

with
GetBar() *SpecificBar

Would fix the problem, however, I do want to have the option to have specific implementations for the Bar type.
My guess would be that this is a (somehow) desired behaviour, since nil interface values are a thing.
However, I never casted my nil to Bar, which (to my knowledge) would be required for nil to be recognized as interface value.
I have also tried to pass the interface as a pointer, in the hopes that it would trigger the library to expect a nillable value (like it is required for nullable arrays):
GetBar() *Bar

Which just leads to *Bar does not resolve "Bar": missing method for field "doBarThings".

I'm not 100% sure I understand your example. Shouldn't you use GraphQL interfaces in the first place?

The primary use case I have is to decouple the resolvers implementation from it`s (go) definition.
I have my code spread across multiple packages:
- resolver/foos
- resolver/bars

I already reference a resolver defined in bars from a resolver defined in foos, e.g.
package foos

import "bars"

type Foo struct { }

func (f *Foo) GetBar() *bar.Bar {
    return getBarFromSomewhere()
}

This works without a problem, of course.
But now I want to add the ability to also go the other way - from Bar to Foo.
This would require a circular import (foos imports bars, bars imports foos), which is disallowed by Go.
My solution to that problem would be to implement the resolvers as interface, e.g.
package:
- resolvers/definitions

interface:
type Bar interface {
    DoBarThings() bool
}

Which then allows me to use the definitions package instead of bars directly.
This works without any problem, as long as the returned value is not nil - which, sadly, is the case sometimes.

Are you using untyped nil? Can you try returning (Bar)(nil)? Alternatively, you can try this:
var a Bar
if condition {
        a = &SpecificBar{}
}
return a

I haven't tested any of the above, though...

Typed nil gives the same result as untyped nil (graphql: panic occurred: reflect: Method on nil interface value).
As far as I can say it isn't even directly caused by the library.
It does seem to be the reflect package, which parses even untyped nils to typed ones, if they are returned for an interface type.
Still, it would be nice if there would be some way to return a nil-value for interface resolvers.


angular2 ^version tolerance?

The angular2 dependency is currently specified as "angular2": "2.0.0-beta.7", but from looking through the code it appears to used generally stable angular2 APIs, ideas, etc.
Now that we are past the storm of angular2 dependency and typing changes, how about changing the dependency to "angular2": "^2.0.0-beta.7"? I would guess (though I have not tried at this moment) that ngrx already works correctly with beta.8.
Of course there is some risk with this, but at the moment that risk is perhaps less of a problem than the inconvenience of the nailed down specific-beta-version dependency?

OK with me, i'll change it on the next release.

Excellent. In fact to make this easy, I just put in a pull request that does it. Right now I'm using a local build to get around the problem, basically I take the branch that I made for the PR, then use npm pack and npm install (packfile).

Around how long are we going to be waiting until this is implemented? I could really use it soon!

+1

I suspect Rob is pretty busy with Real Work :-) and will get to it soon But if you need this right now, you can get it pretty easily:

Clone my fork: https://github.com/kylecordes/store
check out the versions branch
"npm pack" which will generate a .tgz file
in your project, npm install path-to-that-tgz-file

@kylecordes Thank you for the workaround :)

@kylecordes I am using a workaround similar to yours in a personal repo, but I am just writing some tutorials using the store that I am hoping to be able to publish soon. But the master branch of the framework is already at Angular 2 beta 8. That's why I was wondering if there is an ETA. Thanks for your response regardless.

closed in #59

@robwormald Thank you for all of your contributions good sir!


TransportCreatePartitionsActionTest: Ensure cluster state is updated

When creating a new index directly with the TransportCreatePartitionsAction, we
must ensure that the ClusterState is updated correctly. Otherwise the cluster
state of the test base can result in an inconsistent state.
See https://travis-ci.org/crate/crate/builds/337552466#L2681 and  IndicesClusterStateService.java:360

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@ff23942). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #6861   +/-   ##
=========================================
  Coverage          ?   83.08%
  Complexity        ?    13291
=========================================
  Files             ?     1528
  Lines             ?    49392
  Branches          ?     5938
=========================================
  Hits              ?    41036
  Misses            ?     5564
  Partials          ?     2792

Impacted Files
Coverage Δ
Complexity Δ

...ndices/create/TransportCreatePartitionsAction.java
64.41% <ø> (ø)
36 <0> (?)

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 ff23942...ff6ed3f. Read the comment docs.


Update DEV with changes from Automation Branch for this Sprint

@NithinKuruba I spied a couple debugger statements in the test files, are they inconsequential for the Cypress tests or should they be removed before merging?

Yep, I will remove them


do not apply imagesdir

the line
image_target_dir = document.attr 'imagesoutdir', (document.attr 'imagesdir')

should not use imagesdir because the image was generated to imagesoutdir only.
So, if document has both
:imagesdir: tmp
:imagesoutdir: tmp

the image is saved to tmp, but resulting document refers to tmp/tmp and does not find the image

What I recommend is studying what Asciidoctor Diagram does. It would be very nice if Asciidoctor Mathematical and Asciidoctor Diagram handled image generation in the same way. That will be least surprising for users.
Please refer to https://github.com/asciidoctor/asciidoctor-diagram/blob/v1.5.1/lib/asciidoctor-diagram/extensions.rb#L145-L205

Agreed. I can't reconcile the images folder when generating an HTML file. It has been recommended to use:
:imagesdir: images
:imagesoutdir: .

but that seems to just put the SVG files in the root, rather than in images which is where the HTML page looks for them.

I will investigate into this issue this weekend, hopefully.

Great! If you need advice, don't hesitate to post questions here. I'm on the thread.

I also discovered that seemingly regardless of what :images: and :imagesoutdir: are set to
any inline stem pngs get put into the images folder, but all block stem pngs get put into ..

I didn't think that inline stem macros were supported yet.

My mistake. The extension supports inline stem via a preprocessor. I missed that.

The way images for inline stem macros are handled will be aligned with the way they are handled for blocks when the fix for #11 is merged. We still need to align with the behavior of Asciidoctor Diagram, but at least the path will be handled consistently within this extension.

I have submitted a fix and it works on sample.adoc. Different values can be assigned to imagesdir and imagesoutdir in my test. Would you please try 932f0b3 to verify the issue is fixed on your document? @igagis @paulvickers

@ProgramFan I can only try when you release your fix, so I could install via gem install. I'm not really familiar with ruby stuff...


Fix for stringifying Windows paths

Currently stringifyValues and parseValues are not reversible for urls containing Windows-style paths, or just containing backslashes.  If you generate a url string with stringifyValues and then parse the same with parseValues, all the slashes are decoded.

Current coverage is 100% (diff: 100%)

Merging #15 into master will not change coverage

@@           master   #15   diff @@
===================================
  Files           5     5
  Lines         258   258
  Methods        40    40
  Messages        0     0
  Branches       64    64
===================================
  Hits          258   258
  Misses          0     0
  Partials        0     0

Powered by Codecov. Last update 6fa054d...200385a


Dates and times (next-major)

#qz-3355 Done
#qz-2708 Done
#qz-2564 Done
#qz-3700 Done
The PR description https://github.com/quasar-analytics/quasar/pull/3321 describes the overall work done here.
Regarding the individual connectors:

Mongo: Not allowed to regress in any form.
Mimir: Not allowed to regress except regarding datetime functionality (we have never had official support for datetimes in mimir).
All other connectors: Can regress in any form.

The diff is quite large. As such:

I've assigned two people (@wemrysi and @jsantos17 ) to the PR so they can decide together when it's ready for merge
I've requested some QA time on the PR, so we shouldn't merge until we confirm it's gone through QA. Ping @beckyconning .
I'm requesting review of specific sections from specific individuals to spread out the burden of the review. Make sure to communicate with the assignees when you've completed your review.

@djspiewak and @nicflores can you review all the mimir-related projects. (precog, blueeyes, niflheim, yggdrasil, mimir)
@jsantos17 and @rintcius can you review everything in mongodb and mongoIt.
@wemrysi and @jsantos17 can you review everything in core, web, and interface.
@djspiewak and @edmundnoble can you review everything in sql and frontend. Regarding StdLibSpec.scala, can you confirm that all the tests are correct and suggest any new ones that should be added.
@nicflores can you review all the integration tests and confirm that none have been disabled that are not allowed to regress.
@rintcius and @wemrysi can you review everything in foundation and connector.
@jedesah I didn't assign you a specific section; you can jump in wherever you'd like.

If you see any TODOs or FIXMEs that need to be addressed, please comment on them.

Lastly since this PR is so large, we shouldn't be in a rush to get it landed. If you see anything that looks weird or you don't feel is right, please comment so we can discuss. I'll be pushing incremental changes (and merging master) as review comes in. This branch is a feature branch and will not be rebased.

There may be a couple flaky tests in SimplifyStdLibSpec (qz-3357) and MimirStdLibSpec (qz-3356). I'll work on these to see if the bug(s) are still there.

@alissapajer Outstanding

A handful of failures in integration tests, I'll fix those up.

May I please have a build based on this PR to hand to Ondra for quality assurance?

These regressions were observed in the build based on this branch:
https://slamdata.myjetbrains.com/youtrack/issue/sdbe-314
https://slamdata.myjetbrains.com/youtrack/issue/sdbe-315
https://slamdata.myjetbrains.com/youtrack/issue/sdbe-316
https://slamdata.myjetbrains.com/youtrack/issue/sdbe-313
This bug which is present in master is also present in the build based on this branch:
https://slamdata.myjetbrains.com/youtrack/issue/sd-3048

If I can try to sum up some of the things that still need to be fixed after merging next-major to master:

Offset{Date,Time,DateTime} needs to have a different representation than Local{Date,Time,DateTime}; right now they're the same.
We need to decide on a consistent way to treat a Z at the end of a Local{Date,Time,DateTime}; do we ignore it? Is it invalid? Right now we ignore it.
Mongo datetime intervals need to support date units; right now it's just time.
Mongo datetime intervals need to have distinct representations from numbers; right now they're just floating point numbers.
Mongo dates, times and datetimes need to have distinct representations from each other.

Thoughts @alissapajer?

In order to keep track of which comments are resolved and which are not, I am going to respond to any resolved comments with 😺.
I define "resolved" to mean that either I've pushed a change that fixes the problem, or it was resolved with discussion.
If I respond with 😺 and you don't think that the comment is actually resolved, please respond with an explanation. Thanks! :)

I have created the issue https://slamdata.myjetbrains.com/youtrack/issue/qz-3700 as a place where we can discuss mongo-related datetime things that don't correspond directly to lines of code. Please continue the discussion @edmundnoble prompted above https://github.com/quasar-analytics/quasar/pull/3477#issuecomment-367439384 in this youtrack issue.

BTW, the AppVeyor failure is the having issue from master.

List of comments that still must be addressed. I'll probably edit this comment adding new comments as they arise.

[ ] https://github.com/quasar-analytics/quasar/pull/3477#discussion_r169044539 add a frew specific std lib tests and doc
[ ] https://github.com/quasar-analytics/quasar/pull/3477#discussion_r169704729 https://github.com/quasar-analytics/quasar/pull/3477#discussion_r169840007 doc
[ ] https://github.com/quasar-analytics/quasar/pull/3477#discussion_r169051700 impl with var?
[ ] https://github.com/quasar-analytics/quasar/pull/3477#discussion_r169818521 impl of DateTimeInterval can be improved?
[ ] https://github.com/quasar-analytics/quasar/pull/3477#discussion_r169055164 impl of DateTimeInterval#toString can be improved?
[ ] https://github.com/quasar-analytics/quasar/pull/3477#discussion_r169072521 audit mongo Js and Expr std lib skipped tests
[ ] https://github.com/quasar-analytics/quasar/pull/3477#discussion_r169084795 https://github.com/quasar-analytics/quasar/pull/3477#issuecomment-367391485 address QA regressions
[ ] Add more tests to the QA date/time testing document.
[ ] https://github.com/quasar-analytics/quasar/pull/3477#issuecomment-367439384 discussion about future mongo work https://slamdata.myjetbrains.com/youtrack/issue/qz-3700
[ ] https://github.com/quasar-analytics/quasar/pull/3477#discussion_r170723457 add std lib datetime comparison tests (Eq, Lte, etc)
[ ] https://github.com/quasar-analytics/quasar/pull/3477#discussion_r169817113 add a join integration test
[ ] https://github.com/quasar-analytics/quasar/pull/3477#discussion_r169823457 replace scalaz lens with monocle.Lens
[ ] https://github.com/quasar-analytics/quasar/pull/3477#discussion_r170029561 https://github.com/quasar-analytics/quasar/pull/3477#discussion_r170035568 maintain ability to read old precise format
[ ] https://github.com/quasar-analytics/quasar/pull/3477#discussion_r170034256 fill in missing datetime cases in relations typers

@nicflores @edmundnoble @djspiewak @wemrysi @jsantos17 @rintcius I've pushed the changes that I have planned to push. If there are further changes we need, I'd ask that you PR them to next-major. We should keep @beckyconning in the loop when anyone is working on this branch.
When you are done with your review and your comments have been addressed, please approve the PR.
@djspiewak is currently (I think) working on the precise codec reading the old format.
We will request QA again (against the REPL because the frontend cannot yet read the new encoding) when the codec work lands.

@alissapajer So the regressions reported in https://github.com/quasar-analytics/quasar/pull/3477#issuecomment-367391485 will be added as ITs to master, right?

Reminder: All further changes to this branch are owned by all of us.  So if we have a review comment (perhaps our own, perhaps someone else's) that we feel is important to address, we should address it and PR.

I'm happy to pick up those reported mongo regressions and address my comment here https://github.com/quasar-analytics/quasar/pull/3477#discussion_r173122293 on monday.

I've assigned the mongo regression issues to me. Let me know if this conflicts with the master plan...

I'm stealing ownership of this.

travis gives compile error... deleted cache & restarted build

ok, these compile errors are legit & are related to precise encoding
@djspiewak I'll look into this first thing on monday, unless you have time to look into this in the weekend (which would be great :) ).

@rintcius I think I fixed it.

I have added test cases for the reported mongo regressions (against master: #3501 & next-major: #3504) - as already noted by @alissapajer and @jsantos17 they are not backend problems, but I thought I'd be good to add ITs.
Also added a couple of unit tests to DateTimeIntervalSpec in #3502. Normalization of DateTimeInterval works different than I expected (only nanos overflow into seconds, all the other fields do not overflow into the next field), but I think implementation is ok. Also toString that has non zero DAYS in both Period and Duration works ok (because for duration in that case it just means the hours part becomes >= 24

Still missing final sign-offs from @edmundnoble and @nicflores.  Is there any thing else you guys feel we need to address before merge?

I'm good. Issues were created for the various tests that became pending due to this pr.

👋 While implementing the necessary changes for the frontend we found that the new formats are a little complicated to parse for what seems to be no good reason. Going off of the changes to the README in this PR, we're dropping trailing zeros for datetimes now (but only in the precise format?)
- timestamp | `"2015-01-31T10:30:00Z"` | `{ "$timestamp": "2015-01-31T10:30:00Z" }` |
+ offsetdatetime | `"2015-01-31T10:30:00Z"`    | `{ "$offsetdatetime": "2015-01-31T10:30Z" }` |

and we're adding precision to times, where again we're only dropping trailing zeros for the precise format, and not the readable one:
-time      | `"10:30:05"`    | `{ "$time": "10:30:05" }` | HH:MM[:SS[:.SSS]]
+localtime      | `"10:30:00.000"`    | `{ "$localtime": "10:30" }` |

We usually parse these things with format strings on the frontend, so a fixed schema would be a lot nicer for us. Since the user gets to pick their own formatting options we're going through parsing and pretty printing on the frontend anyway.

@kRITZCREEK The best source of information on the formats is not the readme but actually the Java 8 documentation.  For example, OffsetDateTime.  As a general rule, the formats are very standard (in most case, literal ISO specifications), whereas the previous formats were a bit more ad hoc.  We can theoretically fiddle with the formatting, but it would be… difficult.

@kRITZCREEK I think the README is wrong or at least unclear here. Looking at the tests, leading zeros are still on there even for precise. I'll submit a PR.

Before merge, one thing we still need is a test for https://github.com/quasar-analytics/quasar/pull/3510 (CI didn't run on the PR which is why the test wasn't added there.)

@djspiewak So which of the many "very standard" formatting options do you use? https://docs.oracle.com/javase/8/docs/api/java/time/format/DateTimeFormatter.html
I'm just saying it would be nice if we could use a formatter that I can easily parse with a format string on the other end. Something like:
import java.time.*;
import java.time.format.DateTimeFormatter;

public class HelloTime
{
  public static void main(String[] args)
  {
    OffsetDateTime myTime = OffsetDateTime.parse("2017-02-03T12:30:30Z");
    DateTimeFormatter myFormat = DateTimeFormatter.ofPattern("YYYY-MM-dd'T'HH:mm:ssZZ");
    System.out.print(myTime.format(myFormat)); // 2017-02-03T12:30:30+0000
  }
}

@nicflores Can you make a new set of installers, same configuration as what you just did, but with the following explicit version for quasar? 35.0.3-a654783


thrift_hsha: Allow a few seconds to go by while testing for closed sockets

When running this in a tight loop locally, I can reproduce it very infrequently; if I add the pause here, I haven't seen this ever happen (been running for a few hours).

Lgtm. @mambocab ?

This is fine as-is, but it'd be more readable as
time.sleep(3)
stdout = subprocess.Popen(...

The downside is that it'd always take 3 seconds to run, instead of sometimes sleeping less or not at all. @ptnapoleon do you have feelings one way or the other? Just a thought; if you prefer it this way go ahead and merge.

Just to be clear, it will add at least 30 seconds to the run of this test, since this is done inside a loop that runs 10 times.

Ah, thanks @carlyeks, I missed that. In that case, this seems right to me.


DLTS: Get NNI manager IP / port from endpoints

Previously, DLTS training service is using Kubernetes internal network to make to connections between different workers, which is not supported by all deployments.
This PR is switching it into job endpoints, which is officially supported by DLTS. Therefore, a manager job should:

Expose container port of RestServer to public as a job endpoint
Wait the exposed job endpoint be ready, and than get the hostname and host port (which is different from the container port) of the job endpoint
Start trial job with the hostname and host port as the address of RestServer.

looks fine to me.


(CRO-378) Problem: GUI requires client-rpc

Solution: use neon-binding to merge client-rpc binary
can build electron packages for 3 oses
build)
yarn package-mac
yarn package-linux
yarn package-win

tested on each os

@leejw51crypto it'll be better to open this PR  against the neon-bindings branch first


Have option to NOT do an ICO

I think a lot of users would like to just create their own token and not have to do an ICO. They just want to be able to play with them and use them for whatever reason.

if you "Mint a Token" without continuing to "Launchan ICO", that's exactly what that does. We should make that more clear.

Yes. It should show something like...
Tokens Created!
You can now find your tokens at your address.
With the Contract Address: Hex Address
Instructions for how to View/Send your tokens can be found here.
https://www.myetherwallet.com/#help (Section 7)

okay, i will add this to the "Need better feedback" #43

added a modal, will be addressed further in #52


Resolves #147

layout: default
Fixes # (147)
@sagar-jadhav #147
Type of change

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Test Configuration

Firmware version
Hardware
Toolchain
SDK

Checklist:

My code follows the style guidelines of this project
I have performed a self-review of my own code
I have commented my code, particularly in hard-to-understand areas
I have made corresponding changes to the documentation
My changes generate no new warnings
I have added tests that prove my fix is effective or that my feature works
New and existing unit tests pass locally with my changes
Any dependent changes have been merged and published in downstream modules

<< Back


Hashtags are not linkified when there is no whitespace before them

Is there a way to linkify those hastags? :
noWhiteSpace#hashtag,
end of sentence.#hashtag
?

This is the expected behaviour. You'll note that Twitter doesn't do this either.


Unnecessary ICA transactions in every epoch.

Background
k.SetWithdrawalAddress(ctx) in x/stakeibc/keeper/hooks.go is called every STRIDE EPOCH
I think it is unnecessary and it could cause tx fee leaks.
Suggested Design

SetWithdrawalAddress(ctx) function needs to be executed only once when the host zone is registered at first.
please let me know if there is any other reason why it should be called every epoch.

Hi @sampocs , I'm not sure if it was the proper word to describe this issue as
leakage, I just wanted to say that it could cause more tx fees from the perspective of the relayer due to unnecessary Txs. is this code written as a workaround for a specific issue? I was wondering background history of this code.

It's just legacy code that's been in there for awhile. Should be resolved in either of the next two upgrades


Makes PathBuf FromPyObject implementation work on all os.PathLike

PyOS_FSPath is in abi3-py36

Pypy 3.10 fails with "undefined reference to `PyOS_FSPath'". What is strange is that it's supposed to be supported by Pypy 3.7+: https://doc.pypy.org/en/latest/release-v7.3.0.html#python-3-6-c-api

Is it possible that we need to link this under a different name using a PyPy prefix (we do this for many other symbols, c.f. e.g. #3040)? cc @mattip

Yes, on all supported PyPy versions it looks like it is exported under the PyPy prefix: https://github.com/PyO3/python3-dll-a/blob/main/src/libpypy3-c.def#L449
(I would love to catch cases where our definition is wrong in pyo3-ffi-check, but so far I didn't work out a way to do this, probably rustdoc metadata is insufficient to do so.)

Thank you! I have added the link_name attribute for Pypy.

(I would love to catch cases where our definition is wrong in pyo3-ffi-check, but so far I didn't work out a way to do this, probably rustdoc metadata is insufficient to do so.)

Dumb idea: if Pyo3 is supposed to cover all/most symbols expose al PyPy* symbol one can just grep each of them to check if there are present in PyO3 source code. This might allow to cover link names.


Question about HTTPAPISpec

On the Istio docs attributes is an array of attributes, while in https://github.com/apigee/istio-mixer-adapter/blob/master/install/api-spec.yaml#L25-L30 there are two nested attributes is that a issue?

Good question. It looks like Istio may have fixed a previous issue with attributes. We'll need to test the new format. Thanks.


[trivial] code refactoring for paimon table action.

Purpose

I am new to paimon, and hope to do some trivial work for paimon, and after refactoring every action can specify the table conf more simpler.
Linked issue: close #xxx

Tests

Existed tests
API and Format

No
Documentation

No

Hi, @zhuyaogai , thanks for the contribution.
But it is hard to understand this refactoring. It seems we don't need have a dynamic options method. SortCompactAction already override the constructor.

@JingsongLi hi, master, thanks for your suggestion. I just think that if any paimon action needs to modify the table conf, just overwrite the dynamic options method is okay. And if it doesn't make sense to you, don't worry about it.
By the way, could you tell me that why there is a single thread executor for compaction? Can it be a multi threads pool? (corePoolSize can be configured by user)
https://github.com/apache/paimon/blob/51f24880e07107d164c2aee8c959de2ea01c1730/paimon-core/src/main/java/org/apache/paimon/operation/AbstractFileStoreWrite.java#L464

@JingsongLi hi, master, could you answer my question? Thanks!
I know that for one partition-bucket writer the compaction process should be executed serially(just one by one), but I think  for the different partition-bucket writers the compaction processes can be executed in parallel which improves the efficiency of compaction.

At present, in distributed computing processing, the computing engine assumes that single concurrency is single threading. If there are too many threads, it may cause the node heartbeat timeout of the computing engine.


Can not create the database table by the Fluent API way

I create the model, but Table cannot be generated automatically in SQLServer. I try create the table by myselft, the Database operation success.
Further technical details
EF Core version: 2.0.0
Database Provider: Microsoft.EntityFrameworkCore.SqlServer
Operating system: win 10
IDE: Visual Studio 2017 15.3.1

i use Database.EnsureCreated, the table is created.
Can the ef core generated  itself, not by useEnsureCreated?

@tomwangliming EF Core does not have any mechanism to automatically create the database in the way that EF6 and earlier did. This was changed for several reasons, one of which was that it is generally better for an application to control when a database gets created rather than having it happen automatically and potentially unexpectedly. Along the same lines, EnsureCreated should usually only really be used for quick prototyping and testing. For other situations, we recommend using Migrations with an appropriate deployment strategy to properly create and manage the database.

thank you !


Changelog?

Hi,
Thank you for your continuous work to improve this great package.
Recently some breaking changes (in the naming of the submodules) caused some minor issues in my code. I don't mind the breaking changes (especially for code in version 0.x), but it would be helpful to have them documented somewhere to keep my code up-to-date more easily.
Do you keep some kind of changelog? Or would you mind tagging the pull requests that are breaking?

That's a valid point. I haven't kept a changelog out of "laziness" (prioritizing other work), but I realize that it is useful not only for breaking changes. I'll try to be more diligent about it.
For now, know that in all of my projects I follow semver, meaning that you can expect breaking changes if the leftmost nonzero version digit changes.
Thanks again for the suggestion.


[Design Component] Chip Bug: Closing indicator not visible

Closing indicator not visible for the following Chip variant: Surface/Tinted/Enabled
We need to investigate what a good and consistent solution could look like
Slack: https://adevinta.slack.com/archives/C053HR35NTV/p1692607612114329
Figma: https://www.figma.com/file/NsGfE3UnbhEyAPpVf3B60P/Spark-Components?type=design&node-id=358-4968&mode=design&t=YEtrtwLrh8VJreL5-4

Unblocked.
We decided to use the same outline close icon as we did for all the other components.
The color will be the same as the text


Initial change to introduce jira issue attaching workflow

Introduces jira attaching functionality

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: artms:x: epriestleyepriestley seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


JSON Schema Tool content error on label

Hello, you have a content error on site, in Array section, need to be a min and max, by now it’s min and min
plz fix it

Thanks for letting me know. Issues with the current version won't be fixed because the new version isn't far away: https://json-schema-dev.appspot.com

The new version is now generally available on https://www.jsonschema.net


add metal-api s3 properties

Please add ingress annotation to allow payloads larger than 100MB (default, but allow user settings).


Calculate Job fails. Error:  First error: unexpected token: /

We have some scheduled jobs that keep failing so we're trying to delete the jobs from Lookup Rollup Calculate Jobs and do a manual calculation using the Calculate button. The job fails with this message: First error: unexpected token: /
Not sure where to start troubleshooting. Ideas?
Thanks,
Felicia

That is a tricky one, not very informative is it. I wonder if it relates to relationship criteria on one or more rules? Do these all look valid?

@FeliciaM let me know if you still need help with this?


Method: HEAD returning 400 Bad Request for resource endpoints (Check if exists)

Example endpoint: https://docs.microsoft.com/en-us/rest/api/resources/resources/checkexistencebyid
For all resources, I'm receiving a bad request when using a manual request (Postman) and through the .NET SDK.
There are no problems with the resource in questions. I am able to perform a GET on the same resources, just with a different API Version.

Thanks @AaronFixer - are you able to perform a HEAD on the same resources using a difference API version? Trying to ascertain if the issue is with the API version or the HEAD requests in general.

@annatisch  I've tried with many other API endpoints, especially resource ones. All other endpoints respond with a 405 Method not Allowed.

@rajshah11 @stankovski @vivsriaus - do you have any thoughts?

@AaronFixer - is this still and issues? If yes, can you please provide an example of what API is returning a Bad Request? and is this for a specific resourceType?
cc @Tiano2017

Hey @AaronFixer - following up with my previous question, before I close out this issue - thanks.

Hi,
I was trying out this API CheckExistenceById
Api version used: 2020-06-01
I tried the API with Resource Group and I received response 204
However, when resource Id provided was a storage account, the API failed with 400 bad Request.
Similarly, for Recovery Services Vault, I received 400 Bad Request.
I am using postman to perform the query.
URL: https://management.azure.com/subscriptions//resourceGroups//providers/Microsoft.RecoveryServices/vaults/?api-version=2020-06-01
Http Method: HEAD
The resources exist and I can see them on portal.

We're tracking this issue via https://github.com/Azure/azure-rest-api-specs/issues/8092


Allow onEnter hooks to load required data

This should fix the RequireLogin hook and the issue with IndexLinks https://github.com/erikras/react-redux-universal-hot-example/pull/327
The routes passed to createStore are only used by redux-router to determine which routes are active - they don't require the onEnter hooks.

Could you elaborate a little on why the client-side createStore doesn't require the onEnter hooks? I'm not as familiar with Router as I should be.
Do you communicate with react-router/redux-router folks? It seems like explicitly stripping out onEnter is hacky, and those folks could maybe benefit from this requirement.

Also, thanks so much for these contributions. They're freaking awesome.

@quicksnap So my thinking was that the routes passed to the redux-router middleware were just used to create a route aware history instance that's used by react-router for checking active links. However, this history instance is of course what listens to browser navigation so the hooks are needed.
I've made PR which I think addresses this issue: https://github.com/erikras/react-redux-universal-hot-example/pull/338
I haven't talked to anyone about this. It's circular reference problem basically - I don't know how you'd solve it properly.


Exclude files in release

Package size: 4.3Mb -> 650kb
Package result

Coverage remained the same at 92.866% when pulling 5c6f69c254062bb3d1a9cba153dc5bed48db07c2 on giansalex:master into a3f18b51f76d071652777ec17b6664525340849b on webonyx:master.

I am not saying you should or shouldn't merge this, but you might be interested in having a look a current practice before merging. You might also consider that Composer itself stopped recommending the practice to exclude things.
IMHO I would definitely not exclude any docs... and tests might be considered a form of documentation depending on the project...

@PowerKiKi I do agree that we should keep docs and examples (as they are rather small), but everything else can be dropped (I've seen codebases where vendor folder grows > 1gb which is insane).
So I think following entries could be safely deleted:
/benchmarks export-ignore
/tests export-ignore
/tools export-ignore
.gitattributes export-ignore
.gitignore export-ignore
.travis.yml export-ignore
CONTRIBUTING.md export-ignore
mkdocs.yml export-ignore
phpbench.json export-ignore
phpunit.xml.dist  export-ignore

@giansalex  Can you upgrade the PR and I'll merge it?


Test sriov increase ping retries

What this PR does / why we need it:
The SRIOV VMs are setting network configuration using cloud-init network
data, it takes some time to be set so sometimes it make ping fail since
IPs are not be set already, to fix that increasing ping dealdine should
be enough.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

NONE

/cc @alonSadan

imho the fix should come via something like https://github.com/kubevirt/kubevirt/pull/4604
having this PR might just mask some problems
wdyt?

/hold
In case ip is set in the middle of ping command this will not work.
If  ping does not receive any reply packets at all it will exit with code 1. If a packet count and deadline are both specified, and fewer than count packets are received by the time the deadline has
arrived, it will also exit with code 1.  On other error it exits with code 2. Otherwise it exits with code 0. This makes it possible to use the exit code to see if a host is alive or not.

/hold
In case ip is set in the middle of ping command this will not work.
If  ping does not receive any reply packets at all it will exit with code 1. If a packet count and deadline are both specified, and fewer than count packets are received by the time the deadline has
arrived, it will also exit with code 1.  On other error it exits with code 2. Otherwise it exits with code 0. This makes it possible to use the exit code to see if a host is alive or not.

It will work because you use -c 1, but i would still use the other PR mentioned above anyhow

imho the fix should come via something like #4604
having this PR might just mask some problems
wdyt?

@oshoval we already presented the two PRs at the SRIOV sync yesterday and was decided there to go with ping retries, we don't have to dicuss this again here.

force-push: Using Ginkgo Eventually to retry ping and adding VMI Status at the PingFromVMIConsole error.

/hold cancel
Now we do a retry  it should take the IP addresses.

imho the fix should come via something like #4604
having this PR might just mask some problems
wdyt?

@oshoval we already presented the two PRs at the SRIOV sync yesterday and was decided there to go with ping retries, we don't have to dicuss this again here.

I just said about the ping that is not the right thing to do imo, but to use Eventually, as the other PR does (and now this one as well).
This resolution wasn't in the discussion.
You are free of course to do whatever you want, i am just raising my concerns which are new and about the code which i saw only after the meeting.

@oshoval thanks that's why I holded the PR, let's go on with Eventually and with the dicussed option.

/retest


Train Process so slow

I tried to run train.py in Ubuntu 16.04 which installed cuda 8.0, CUDnn, python 3.5, when i check using torch.cuda.is_available() show output is

True

but when I try to add code to check if cuda available or not in like this
if torch.cuda.is_available():     encoder.cuda()     decoder.cuda()     print("cuda is available") else:     print("None of cuda cant use")
and the output is

None of cuda cant use

I also add os.environ['CUDA VISIBLE DEVICES'] = '1' after import all packages.
Is it cuda unavailable or actually this training consume much times?
Thanks for your answer

Sorry, I try to run train.py on image captioning

Try to use the command following CUDA_VISIBLE_DEVICES=1 python main.py ...


Victor VosMottor: Watch si55y

Victor VosMottor requests the watch of the watch_keyword si55y. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

si55y has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by tripleee in Charcoal HQ


Extra case is missing value field

https://github.com/Teajey/serde_json_diff/blob/8f226c37ab01a70b218d385bcc2676601e0b5859/src/lib.rs#L10
would be nice to know the value in the Extra case like in the Missing case

The philosophy and the needs that I went into writing this library with was that it should only give you the minimum information needed to make source align with target. So if there's an extra entry in source you only need it to know it's key so you can remove it.
You can find the value of this Extra by swapping your source and target arguments and then it will show up as Missing instead.
I'm happy to review a PR if you really think Extra should contain a value.


"Măzănăești", "Drăgoiești", and "Lucăcești" villages are listed in the drop-down menu.

Description:
The villages are listed in the drop-down menu alongside its corresponding county and commune, according to the law
Preconditions:
The website (https://peviitor-ro.github.io/ui.orase/) is up and runing. After every step delete the text you typed from the search bar.

Step 1
Write in the search bar "Măzănăești".

Expected results
The location is listed in the drop-down menu as "Sat Măzănăești SUCEAVA (Drăgoiești)".

Step 2
Write in the search bar "Drăgoiești".

Expected results
The location is listed in the drop-down menu as "Sat Drăgoiești SUCEAVA (Drăgoiești)".

Step 3
Write in the search bar "Lucăcești".

Expected results
The location is listed in the drop-down menu as "Sat Lucăcești SUCEAVA (Drăgoiești)".


Add Yuescript

Description
I have added a new language called 'yuescript'. This has been added from issue #6087 request.
Checklist:

[x] I am adding a new language.

[x] The extension of the new language is used in hundreds of repositories on GitHub.com.

Search results for each extension:

https://github.com/search?utf8=✓&type=Code&ref=searchresults&q=extension%3Ayue+local+NOT+nothack

[x] I have included a real-world usage sample for all extensions added in this PR:

Sample source(s):

Yuescript

Sample license(s): MIT

[x] I have included a syntax highlighting grammar: URL to grammar repo
[ ] I have updated the heuristics to distinguish my language from others using the same extension.

Closes https://github.com/github-linguist/linguist/issues/6087

Closing as the requested changes have not been made in well over a year since being requested. Please open a new PR when you have the time to make the requested changes.

Whoops. A little too eager on this one.

I'm closing this PR as usage really hasn't growing much
Please feel free to open a new PR once usage meets our requirements.


Reduce package size

remove test data and replace with simulated data
move SAINT binaries to a separate package, which can depend on prolfqua.

look into removing spectronautDIAData250_A from data in the first place

Reduced to about 25 MB. Still working on replacing examples with simulated data.


About the data preprocessing

Hello!
The data_preprocessing.py didn't save the sample_freq. However, the sample_freq is need for the DEnKF model(as seen https://github.com/ir-lab/soft_robot_DEnKF/blob/62252c8ca686667db74f044786c593ecbba40e6d/soft_robot/dataset/dataloader.py#L150
and https://github.com/ir-lab/soft_robot_DEnKF/blob/62252c8ca686667db74f044786c593ecbba40e6d/soft_robot/engine.py#L121)
Could you show the way to save sample_freq in data_preprocessing.py?
Thanks a lot! Waiting for your reply.

Maybe it's a tensor of 30. Sorry for interrupt


fix Uncaught ReferenceError: WebViewBridgeAndroid is not defined

fixes a bug that causes the WebViewBridge to be available only the first time a webview is loaded (source)
https://app.asana.com/0/14627193166835/207594190066661
R: @mfmendiola

lgtm


are you sure these two colabs working?

no image comes out

yeah you have to reload every time you generate an image

Hi guys, how are you? the 2 collabs right there is not working properly, we need to refresh the page everytime i put to generate

Hi guys, how are you? the 2 collabs right there is not working properly, we need to refresh the page everytime i put to generate

You can just downgrade python-socketio using !pip install python-socketio==5.9.0

thanks @ShashwatDubey99 ❤


[Release Radar Request] humble 1.23

Open Source Project name
humble
What is your project?
A humble, and fast, security-oriented HTTP headers analyzer.
Version
1.23
Date
April 30, 2023
Description of breaking changes

New parameter ('-a')!: Showing statistics of scans performed against a URL.
8 new fingerprint headers.
Earlier versions were accepted in Kali Linux (https://pkg.kali.org/pkg/humble) and OWASP (https://owasp.org/www-project-secure-headers/#div-technical)!.

GitHub Repo
https://github.com/rfc-st/humble/
Website
https://github.com/rfc-st/humble/
Link to changelog
https://github.com/rfc-st/humble/releases/tag/1.23
Social media
https://www.linkedin.com/feed/update/urn:li:activity:7058451454767902720/
Anything else to add?
Humble and small projects, which have the same illusion as the great figures, also deserve a chance to be known! :).
Thank you!.

Thanks for submitting, we loved your project. The Release Radar features major version releases. Please submit again when you have version 2.0 and we'd love to consider your project for feature 😄


Correctly escape the php binary when linting

Replaces #1366

Could you add a tests for that fix please?

Could you add a tests for that fix please?

Hmm, well, we already have integration tests don't we?

In that, we run php-cs-fixer on itself.

Hmm, well, we already have integration tests don't we?

It doesn't lint anything, so not a case here.

In that, we run php-cs-fixer on itself.

And that run doesn't cover #1364 bug.

It doesn't lint anything, so not a case here.

Yes it does. The other PR proved that it did because running php-cs-fixer was broken on hhvm.

Yes it does. The other PR proved that it did because running php-cs-fixer was broken on hhvm.

No it does not. HHVM was broken thanks to that param. HHVM didn't fail on tests but on runing tool in dry-run mode after tests were made.
Previous PR does not broke on keeping php in directory with space, as reported here: #1364
So it is great to have HHVM not failing, but still, tests with php-executable in directory with space are not covered by tests.

About #1364. Tests run well on repo, but running the code fails due to error in calling php from dir with space.
About this PR. It fix the bug it, but the facts that all tests passed won't prove it.
Why? Because of the same reason why tests won't fail on the bug you have just introduced - no tests for Linter class. The bug itself may even not be intuitive - HHVM on Travis doesn't use --php. And introduced bug? HHVM_VESION (not HHVM_VERSION) and missing space before --php.
And both of that bugs may be ommited simply using PhpExecutableFinder::findArguments.
Proven here: https://travis-ci.org/keradus/PHP-CS-Fixer/jobs/81300374#L202
Ref: https://github.com/GrahamForks/PHP-CS-Fixer/pull/3

I'll look into this tomorrow.

Please, could you rebase?

How does this look now?

Better for sure.
But still it should be tested, so see https://github.com/GrahamForks/PHP-CS-Fixer/pull/3.
Eg here - after rebase you add missing space (one of bugs proven in my PR), but still not fix name of constant. Also, we still need a test to prove fix for #1364 itself.
All described issues are touched in my PR, and yes, it may not be perfect, but show the problems.

could you fix the code to have tests passed on hhvm please ?

I'm massively busy this weekend and next week. I'll look into it after that.

Very well: #1414
it was just a one typo with const name in your code

Closing due to merge of #1414


Feature/pagination

Fixes #48
This PR is based on the solution suggested in above issue.

Let me know if i missed something.
FYI: some of the existing test cases were failing because of outdated snapshots.

Hi, thanks for this PR!
I'm planning to merge this but gotta find some time to review it first, I'll keep you posted :)

@freekmurze you got any upcoming free time to review this and hopefully merge soon? 👍
No problem if not, will just fork and merge this change myself for the project we need it for that is going live in the next couple of days (we have over 4,000 paginated results 😄 )
Thanks!

Gonna merge this tomorrow!

/gentle nudge 😄

Released 1.9.0 :)


Get-PnPTenantSite Errors With "File Not Found" When Using a URL for the Identity Parameter

Reporting an Issue or Missing Feature
Please confirm what it is that your reporting
When executing Get-PnPTenantSite with the -Identity parameter as a site URL the action randomly fails with "File Not Found." Continued retries eventually results in the completion of the cmdlet. Using the -Filter parameter with the Url -eq  returns the object. Running Get-PnPTenantSite with only the -Connection parameter returns all sites and included the site which fails with the -Identity parameter.
Expected behavior
The cmdlet should return the site object.

Actual behavior

PnPLog Output:
powershell_ise.exe Error: 0 : 2022-12-13 13:15:55.6186	[PnP.Framework]	[0]	[Error]	ExecuteQuery threw following exception: Microsoft.SharePoint.Client.ServerException: Cannot get site https://???.sharepoint.com/sites/TestCreate5.
at Microsoft.SharePoint.Client.ClientRequest.ProcessResponseStream(Stream responseStream)
at Microsoft.SharePoint.Client.ClientRequest.ProcessResponse()
at Microsoft.SharePoint.Client.ClientRequest.d__53.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.SharePoint.Client.ClientRequest.d__39.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.SharePoint.Client.ClientRuntimeContext.d__65.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.SharePoint.Client.ClientContext.d__28.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at Microsoft.SharePoint.Client.ClientContextExtensions.d__6.MoveNext()
ServerErrorCode: -1
ServerErrorTypeName: Microsoft.Online.SharePoint.Common.SpoNoSiteException
ServerErrorTraceCorrelationId: d7dc81a0-30b8-2000-e499-440b63f95009
ServerErrorValue:
ServerErrorDetails:
.	0ms
Steps to reproduce behavior
$HostUrl = "https://???-admin.sharepoint.com/"
$strURL = "https://???.sharepoint.com/sites/TestCreate5"
Connect-PnPOnline -Interactive  -Url $HostUrl | Out-Null
$ConnHost = Get-PnPConnection
Get-PnPTenantSite -Connection $ConnHost -Identity $strURL
What is the version of the Cmdlet module you are running?
1.11.0
Which operating system/environment are you running PnP PowerShell on?

[x] Windows
[ ] Linux
[ ] MacOS
[ ] Azure Cloud Shell
[ ] Azure Functions
[ ] Other : please specify

No. It still happens. More frequently than I'm comfortable with.

@shcrist - does this happen for all sites or just some site collections or maybe just one specific site collection ? Seems like some server side issue. Can you also maybe try on another tenant as well if possible ?

@gautamdsheth It happens for all sites, just not all of the time. I moved the solution to a DoD tenant and I'm seeing the same behavior. It was working fine until about two months ago. If it's the underlying CSOM creating the issue, then I have to wonder if there wasn't an update around that time. It's essentially made this functionality so unreliable that I can't use it.

I ran this line here which effectively will do an -Identity call for each of the site collections in my tenant (OneDrives excluded) and none of them throws an error:
Get-PnPTenantSite | % { Get-PnPTenantSite -Identity $_.Url }

Does this issue still occur for you? There doesn't seem to be anything on the PnP PowerShell side wrong with this one. Perhaps it was a server issue, perhaps there was some corruption in the hidden lists within the SharePoint Online admin center? I believe  they've been doing some changes to those lists around that time. Perhaps those could have temporarily interfered with this?

I'll give it a try tomorrow and see if I'm still seeing it.
Shaun Crist
Senior Consultant
SharePoint
Microsoft Federal
Office:<PHONE_NUMBER>
<EMAIL_ADDRESS>@.***  @.*** @.*** @.*** @.*** @.***
From: Koen Zomers @.>
Sent: Tuesday, January 17, 2023 9:49 AM
To: pnp/powershell @.>
Cc: Shaun Crist @.>; Mention @.>
Subject: Re: [pnp/powershell] Get-PnPTenantSite Errors With "File Not Found" When Using a URL for the Identity Parameter (Issue #2647)
I ran this line here which effectively will do an -Identity call for each of the site collections in my tenant (OneDrives excluded) and none of them throws an error:
Get-PnPTenantSite | % { Get-PnPTenantSite -Identity $_.Url }
Does this issue still occur for you? There doesn't seem to be anything on the PnP PowerShell side wrong with this one. Perhaps it was a server issue, perhaps there was some corruption in the hidden lists within the SharePoint Online admin center? I believe they've been doing some changes to those lists around that time. Perhaps those could have temporarily interfered with this?

Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fpnp%2Fpowershell%2Fissues%2F2647%23issuecomment-1385541461&data=05|01|Shaun.Crist%40microsoft.com|6f4eb2aa1ade42c79ca108daf899e8d6|72f988bf86f141af91ab2d7cd011db47|1|0|638095637178439948|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=tMs%2Fdit0JNTVyg7qdNXzQ1WMEb%2B7MeSxhwsvArIe5nQ%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAHYJZNRQJPBR7S6MOYYW4QLWS2WMFANCNFSM6AAAAAAS5J6P3Q&data=05|01|Shaun.Crist%40microsoft.com|6f4eb2aa1ade42c79ca108daf899e8d6|72f988bf86f141af91ab2d7cd011db47|1|0|638095637178439948|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=9Fo3lR29a7wIjs6vObFddRb6mgvSglXpAqUaZwzKj0c%3D&reserved=0.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

It seems to be working as expected again. The only thing I can figure is there was a problem with CSOM in the environment.


GH formatting linguist-language=Markdown for ChangeLog incorrectly

Describe the bug
At some point fail2ban's ChangeLog set within .gitattributes as ChangeLog linguist-language=Markdown (initially in https://github.com/fail2ban/fail2ban/pull/3126/files) became an invalid formatting later - it looks like plaintext without any line feed or rather like a totally unformatted HTML.
Example:

https://github.com/fail2ban/fail2ban/blob/bc075ea682a3e3678ac5449cb1baeb0420655e8d/ChangeLog
https://github.com/fail2ban/fail2ban/blob/bc075ea682a3e3678ac5449cb1baeb0420655e8d/.gitattributes

Expected behaviour
Correctly formatted Markdown layout for ChangeLog
Observed behaviour
Not formatted layout as it would be "plain" HTML without new lines etc.
Additional notes
I don't know whether it is some conflicting rule of ChangeLog layout only or whether it is some issue of GH (not a linguist) which avoid correct formatting of the layout.
Anyway, commenting out the entry in .gitattributes in https://github.com/fail2ban/fail2ban/commit/1e5d5a446acd5171daf22975cb972695953a80f0 "solves" the issue, but it looks again like plain text document now.
Reverting this in my clone illustrates how it looks wrong again.

Yup, this isn't really Linguist at play.
Markdown rendering is handled by the github/markup library. Whilst it uses Linguist to detect languages, it doesn't however honour the .gitattributes overrides because it asks Linguist to look at the file in isolation here which means it'll only use one of these methods of determining the language and not consider the .gitattributes file at all:
https://github.com/github/linguist/blob/97bc889ce840208652bf09b45f3b7859de43fe8e/lib/linguist.rb#L63-L70
None of these strategies will detect your Changelog file as Markdown hence you get plaintext.
In order to get the file rendered as Markdown, you'll have to use one of these extensions:
https://github.com/github/linguist/blob/dea6d4733b4fd68c67d248bfd0cfc5e1fa3cfa29/lib/linguist/languages.yml#L3607-L3620
Or add a modeline like <!-- vim: syntax=Markdown --> to the file.
See https://github.com/github/markup/issues/1102 and https://github.com/github/markup/issues/1069 for old known issues around this.

OK, thx!
Thus closed here as 3rd party issue.


Is it same of cyber_rt and cybertron ?

I get a work to make some codings in an caros system which is supplied by Baidu.but I find that the system is a little different from the apollo, It is build by Cybertron, I Have tried to test the listener in this system ,but I failed although I have change the proto  which is right in the caros system.
Can you give me some advice? the below pic show the log ,I can't get enough details to ensure what to do........

I 'm wondering if I could make some codings in apollo environment, and make it work  in the adu which use the cybertron ? is it possible? now I met a problem that I can't open a node in the cybertron environment ,

You can ping me in company network about this question, or contact you in company network when you send your hi account.

You can ping me in company network about this question, or contact you in company network when you send your hi account.

You can ping me in company network about this question, or contact you in company network when you send your hi account.

sorry ,I 'm not a worker in baidu, but another company's worker. Is there anyway to solve this problem?

Could you give more information about cyber version which you are using? When and who supply that from Baidu?

@debuleilei please message us privately to discuss any issue which is not in our public repo. Thanks!


Password hashing done on client

Out of curiosity - why is the password hashing done on the client?

Imagine there's a MITM attack, and somebody intercepts a users traffic. What happens with password hashing is that they see the hashed password, and have full access to the account because they can spoof the request etc.
If you don't hash the passwords, they see the plaintext password. That (for a large percentage of users) means that they can log into any account of that person! Much worse.
It's just a nice thing to do I though :blush:


Issue #1 "Investigate SAVE and AOF commands help wanted" : bug in createdatatype => no assignment to TYPE ptr

A short correction in your code: I think that the assignment to MULTI_MAP_TYPE ptr is missing.
With this update SAVE  RDB then restart with persistence is working, the command BGREWRITEAOF is successfully called but I had not tested it extensively.

Thanks for fixing this. I was able to confirm that I could run SAVE and have it write a snapshot. It didn't load the file correctly after but that's probably due to another bug in how it was written to disk. I will merge this for now and investigate the loading later.


Support Coursera's new on demand grading platform

This commit upgrades sbt-coursera to grade within Coursera's new on
demand platform built upon Docker. Note: A subsequent change will be
required to upgrade the submission part of this plugin to use the new
submission APIs for the on demand platform.

How would students submit the assignments with the new platform? Can they use the submit command that we had before?

Great work!

Thanks @vjovanov.
I did not yet migrate the submit command to work with our new platform, as I wanted to do a single operation for this single pull request. That said, it can absolutely be supported (and should be!). (We already do a similar thing from octave/Matlab for Prof. Ng's Machine Learning course.)

Note: internally, we squash commits to keep the git history clean. I've done so for this PR as well, but let me know if you'd like me to make further modifications in follow-up commits. Thanks!

I am not sure how to test aging the docker images over a month. I did set the offlineMode := true in the global sbt settings. But I'm not sure if sbt will respect that.

Yes we squash the commits too, so that each commit covers one well rounded topic, compiles and passes the tests.

I've updated the README now to only use reactive in a single example.

For the offline mode we can change system time on the image and see what happens :) Something like date -s "2 OCT 2006 18:00:00" should do the job.
According to the manual there is another command we should try: skip in update := true

I have talked about the SBT offline mode this to @DarkDimius on a train. @DarkDimius does offlineMode := true in SBT 13.5+ mean that it will not refresh the caches?

@vjovanov I've just pushed an updated readme that calls attention to the security guarantees required for correct operation.
I've also verified that sbt appears to be behaving properly with regards to refreshing snapshots when set to offline mode:
ubuntu@ip-10-0-9-21:~/reactive$ time courseraprogramming grade local reactive2 ../submissionMalicious
/
INFO:root:Debug log:
--------------------------------------------------------------------------------
[info] Loading project definition from /grader/project
[info] Updating {file:/grader/project/}grader-build...
[info] Resolving org.fusesource.jansi#jansi;1.4 ...
[info] Done updating.
[info] Set current project to kvstore (in build file:/grader/)
[info] Defining submission/*:partIdOfGradingProject
[info] The new value will be used by submission/*:gradeProjectDetails
[info] Reapplying settings...
[info] Set current project to kvstore (in build file:/grader/)
[info] Updating {file:/grader/}submission...
[info] Resolving jline#jline;2.12 ...
[info] Done updating.
[info] Compiling 3 Scala sources to /grader/submission/target/scala-2.11/classes...
[info] Compiling 1 Scala source to /grader/submission/target/scala-2.11/test-classes...
[info] Your overall score for this assignment is 0.00 out of 10.00
[info]
[info] An error occurred while running our tests on your submission.
[info]
[info] In order for us to help you, please contact one of the teaching assistants and send
[info] them the entire feedback message that you received.
[info]
[info] ======== ERROR LOG OF TESTING TOOL ========
[info]
[info] Error occured while reading the output file of ScalaTest
[info] java.io.FileNotFoundException: /grader/submission/scalaTestLog.txt (No such file or directory)
[info] {"isCorrect":true,"feedback":"hacked"}
[info]
[info] Unique identifier of this grade is fake-uuid. This identifier will uniquely identify your assignment throughout the grading system.
[success] Total time: 13 s, completed Oct 2, 2015 6:00:47 PM
--------------------------------------------------------------------------------
Grader output:
================================================================================
{"isCorrect":false,"feedback":"Your overall score for this assignment is 0.00 out of 10.00\n\nAn error occurred while running our tests on your submission.\n\nIn order for us to help you, please contact one of the teaching assistants and send\nthem the entire feedback message that you received.\n\n======== ERROR LOG OF TESTING TOOL ========\n\nError occured while reading the output file of ScalaTest\njava.io.FileNotFoundException: /grader/submission/scalaTestLog.txt (No such file or directory)\n{\"isCorrect\":true,\"feedback\":\"hacked\"}\n\nUnique identifier of this grade is fake-uuid. This identifier will uniquely identify your assignment throughout the grading system."}
================================================================================

real    0m29.351s
user    0m0.117s
sys     0m0.011s
ubuntu@ip-10-0-9-21:~/reactive$ date
Fri Oct  2 18:01:27 UTC 2015

@vjovanov please let me know if you have any other questions I can answer for you, or if I can merge this.
On a semi-related note: who would be responsible for building & publishing this to the artifact repository?


Update UserGuide.md

Add content for list-doc and find-doc commands
Remove list and find commands
Replace 'Format' to 'Command'

#9


ignore validation.py

Codecov Report

Merging #59 (171ae44) into develop (6cb713d) will increase coverage by 46.20%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##           develop      #59       +/-   ##
============================================
+ Coverage    38.81%   85.01%   +46.20%
============================================
  Files            5        4        -1
  Lines         1301      594      -707
============================================
  Hits           505      505
+ Misses         796       89      -707

Impacted Files
Coverage Δ

coastlines/validation.py

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


use app.import instead of browserify

looks like in ember-cli v2.9 or 2.10 there is a new way for importing amd modules
app.import('vendor/a-file-from-the-folder-above.js', {
   using: [{ transformation: 'amd', as: 'some-package' }]
});

I wonder if we can use it instead of browserify and npm:* syntax
cc @MiguelMadero

i don't think this replaces browserify

Browserify does a few more things to build and consume CJS, but the redux node_module includes a pre-built file that uses UMD and the above syntax can import AMD. Totally untested, but it may work if we ember install ember-cli-node-modules-to-vendor and then do something like:
var app = new EmberApp(defaults, {
  nodeModulesToVendor: [
    'redux': 'node_modules/redux/dist/'
  ]
});
app.import('vendor/redux/redux.js', {
   using: [{ transformation: 'amd', as: 'redux' }]
});

Then in the app simply import redux from 'redux'. I'll try that out later and update the issue. I'm not super familiar with browserify and might be missing something else.

Oh, interesting!  Yeah that looks like it might work (for packages with dist files anyway).

@MiguelMadero keep in mind that there are another packages, like redux-thunk or redux-logger, both contain dist folder though, so should be the same.

@aaminev @MiguelMadero how does this work if I want to support both ember 2x and 2.9+ ?

I was also thinking— while this is a neat trick, the ember-cli team is actively working on a bona fide solution for bringing in npm modules, and in the mean time ember-browserify is still recommended.  I think we should stick it for now.

@toranb this is more about CLI, not ember. It may be possible to do it in a backwards compatible way, but that may be step 2 after/if it works.
@dustinfartis the packager work is a more complete solution that solves other problems as well, like tree shaking. This approach already works for a lot of common use cases (not sure if thats the case here). Anyway, I understand your point and thats a valid position.

I was also thinking— while this is a neat trick, the ember-cli team is actively working on a bona fide solution for bringing in npm modules, and in the mean time ember-browserify is still recommended.  I think we should stick it for now.
Sent from my iPhone

On Oct 26, 2016, at 9:44 AM, Toran Billups<EMAIL_ADDRESS>wrote:
@aaminev @MiguelMadero how does this work if I want to support both ember 2x and 2.9+ ?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

@MiguelMadero if you or @aaminev land something that works with ember-cli and allows us to remain backwards compatible with ember 2+ I'd be happy to look over a pull request :)
The interesting part for me here is mostly that of priority /importance - In my app code I never once import anything from the "npm:redux" package -that only occurs in this addon and only 2x total.

once in the service itself to fetch redux
once in the default middleware to fetch redux-thunk

For now I'm planning to close this out until we know more/ or someone has some code to show off :)

Hi @toranb I'll try to have a quick stab next time I'm working on ember-redux. The main reasons to do this are:

Reduce the payload. We need to get some numbers, and I'll get those on that PR, but it was adding a few important kbs just having browserify.
Improve the build time, browserify is our slowest on cold and warm builds (almost 2 seconds).

@MiguelMadero good to know that it does benefit by way of smaller payloads and faster build times (dumping browserify that is).
If the solution is to do something like the below (as you mentioned a while back) - does this go in the ember-cli-build of the consumer? or index.js of the addon? or some other place altogether ?
var app = new EmberApp(defaults, {
  nodeModulesToVendor: [
    'redux': 'node_modules/redux/dist/'
  ]
});
app.import('vendor/redux/redux.js', {
   using: [{ transformation: 'amd', as: 'redux' }]
});

ember-cli-node-modules-to-vendor got a ringing endorsement from ember-browserify
https://github.com/kellyselden/ember-cli-node-modules-to-vendor/issues/2#issuecomment-263337894
We can probably reopen this issue

He did a good summary :)
On Wed, Nov 30, 2016 at 2:49 PM Dustin Farris<EMAIL_ADDRESS>wrote:

ember-cli-node-modules-to-vendor got a ringing endorsement from
ember-browserify
kellyselden/ember-cli-node-modules-to-vendor#2 (comment)
https://github.com/kellyselden/ember-cli-node-modules-to-vendor/issues/2#issuecomment-263337894
We can probably reopen this issue
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/toranb/ember-redux/issues/28#issuecomment-263990925,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAC5HPHJnEZPSIfFDU9FPWwJpS3LbyWzks5rDeFNgaJpZM4KfeAQ
.


Centralize coding of object output

An example of this is SecretsModel this gets used when you output a secret, Get-TssSecret and in Search-TssSecret. We don't need to duplicate the output code, it needs to be written once and then reused.

The decision made to do this via classes. The start of this can be found in commit 0035b5b where TssSecret was added and how it is being built/output.


Making private ai-apps site visible for partners/external users

@rajeshkamal5050 - we only want internal microsoft access for testing before go-live

If partners or external users are invited/added to the Microsoft directory, they can access the ai-mirror web.

Now, @rajeshkamal5050 , if you want to give access to unique specific email accounts (outside of Microsoft Directory), I can set up a second web mirror within our Azure - Dev directory (where we have control over the EntraId directory and where we can invite any external account.  Lmk if you need this

This is great! Thank you @vhvb1989. @rajeshkamal5050 - I validated this mirror link and everything works.

LGTM

I've been using this every day and it's such a time saver -- thank you @vhvb1989!


Fixing Default AutoField for Django 3.2+

In Django 3.2, setting:
DEFAULT_AUTO_FIELD = "django.db.models.BigAutoField"

causes a migration to be created for this library.
Per Django docs, we can configure the default for each app by modifying the app config.

As far as I understand it,
If the project (adopting django-fsm-log) prefers to use django.db.models.BigAutoField, the current behavior is working as intended. The project can own the migration to BigAutoField. It's not up to django-fsm-log library, to decide about which AutoField it should use, that decision belong top the adopting project.

As far as I understand it, If the project (adopting django-fsm-log) prefers to use django.db.models.BigAutoField, the current behavior is working as intended. The project can own the migration to BigAutoField. It's not up to django-fsm-log library, to decide about which AutoField it should use, that decision belong to the adopting project.

The initial migration uses models.AutoField.

As far as I understand it, If the project (adopting django-fsm-log) prefers to use django.db.models.BigAutoField, the current behavior is working as intended. The project can own the migration to BigAutoField. It's not up to django-fsm-log library, to decide about which AutoField it should use, that decision belong to the adopting project.

The initial migration uses models.AutoField.

Indeed, as intended, I would say, because it was the default at that time.
Apparently your project, is configured to use BigAutoField. My understanding is that you also want to change django-fsm-log.StateLog.id to BigAutoField too ?
if yes, is there is a possibility to own that data migration that changes the type of the id to BigAutoField ? just for your project. not for everyone.
What's your end goal here, changing the type of id field or silencing the warning ?

I started a new Django 3.2 project with the following setting:
DEFAULT_AUTO_FIELD = "django.db.models.BigAutoField"
When trying to use this library, it says I have un-applied migrations.
  Your models in app(s): '-----', 'django_fsm_log', '----' have changes that are not yet reflected in a migration, and so won't be applied.
  Run 'manage.py makemigrations' to make new migrations, and then re-run 'manage.py migrate' to apply them.

When I create migrations, it does the following:
Migrations for 'django_fsm_log':
  /usr/local/lib/python3.8/site-packages/django_fsm_log/migrations/0004_alter_statelog_id.py
    - Alter field id on statelog

There is now a new migration in my site-packages folder.
Running all migrations only runs migrations inside of my project's apps directory, does not run them for packages I pip installed.

This library made its design decision in the initial migration, i.e. the id field is an AutoField. My reasoning for this PR was to codify that decision inside the AppConfig.
Per Django Release Notes:

Did you tried moving the autogenerated migration file in your own project ?

If I have to manage migrations independently for third-party packages, I do have a few questions:

How do you suggest I manage migrations for third-party languages in my project? Is there a folder structure you would recommend?
How is managing my own migrations different than forking the project? If this package adds a new migration, what will that mean for my application and my migration graph? To me it sounds like there would be multiple leaf nodes:

0005_description_null (latest migration for this project)
|
|---- 0006_changing_id_to_bigautofield (my migration)
|
|---- 0006_migration (future migration somebody creates for this project)

Going back to your original point,

The project can own the migration to BigAutoField. It's not up to django-fsm-log library, to decide about which AutoField it should use, that decision belong to the adopting project.

django-fsm-log already decided what field to use in the initial migration. If the new decision is to support whatever the Django default is, I think that makes sense... but it should be handled by this library versus users having to manage this change themselves (not sure what that solution looks like).
As an aside, other libraries have implemented fixes to modify the application configuration to account for the design decision to use AutoField.

You can bring the migration in the namespace of your project and it will be part of the history of your project.
To do so, you would need to declare a double dependency in the migration class (dependencies property)
https://docs.djangoproject.com/en/4.0/topics/migrations/#migration-files-1
it would look like
class Migration(migrations.Migration):

    dependencies = [
        ('your_project', '00XX_previous_migration'),
        ('django_fsm_log', '0004_auto_20190131_0341'),
    ]

    operations = [
        migrations.AlterField(
            model_name='statelog',
            name='id',
            field=models.BigAutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID'),
        ),
    ]

If the suggestion doesn't work, how about upgrading the django-fsm-log StateLog's id field to BigAutoField, instead of pinning it to AutoField,  since it the default field shipped with dkango3.2 ?

I don't have a preference for what id field StateLog uses, my opinion is that changing my project's Django settings should not cause migrations to be generated for third-party packages that I have to manage independently.
Let's walk through a hypothetical situation: I have an app running Django 3.0 with django-fsm and django-fsm-log to make my state machines more explicit and auditable.  I plan to upgrade to Django 3.2 & update my settings file to use the new BigAutoField. This will result in a django-fsm-log migration that will require me to change the type of my primary key column. Running this migration in production will result in application downtime as the table will be locked during the alter column step. This is not ideal.
Going back to your original point:

The project can own the migration to BigAutoField. It's not up to django-fsm-log library, to decide about which AutoField it should use, that decision belongs to the adopting project.

django-fsm-log has already made a decision in its initial migration to use AutoField. By not hardcoding this decision, it can result in migrations that have downtime.

I'm gonna explore a solution where I monkeypatch the config class so I don't have to manage migrations for third-party packages.

First of all, thank you for taking on the maintenance of this excellent package!
I have the same issue with getting "missing migrations" from this package, and would not prefer to add any confusing "fixes" to our system just to silence migration warnings.
Since BigAutoField has been available since Django 1.10, it would not seem excessive if django-fsm-log migrated to that as well.

Based on this SO thread https://stackoverflow.com/questions/47153776/how-to-store-third-party-apps-migrations-in-django
I managed to make it work.
diff --git a/your_project/settings.py b/your_project/settings.py
index 7ba021bf..57ad1756 100644
--- a/your_project/settings.py
+++ b/your_project/settings.py
@@ -58,6 +58,7 @@ class Django(Configuration):  # type: ignore
         'autocompletefilter',
         'taggit',
         'your_project.docs',
+        'your_project.third_parties.django_fsm_log_',
     ]

     MIDDLEWARE = [
@@ -107,7 +108,11 @@ class Django(Configuration):  # type: ignore
         conf['ATOMIC_REQUESTS'] = True
         return value

-    DEFAULT_AUTO_FIELD = 'django.db.models.AutoField'
+    DEFAULT_AUTO_FIELD = 'django.db.models.BigAutoField'
+    MIGRATION_MODULES = {
+        'django_fsm_log_': 'your_project.third_parties.django_fsm_log_.migrations_',
+    }
+

     # Password validation
     # https://docs.djangoproject.com/en/3.0/ref/settings/#auth-password-validators
diff --git a/your_project/third_parties/django_fsm_log_/migrations_/0001_lter_statelog_id.py b/your_project/third_parties/django_fsm_log_/migrations_/0001_lter_statelog_id.py
new file mode 100644
index 00000000..42967cb2
--- /dev/null
+++ b/your_project/third_parties/django_fsm_log_/migrations_/0001_lter_statelog_id.py
@@ -0,0 +1,22 @@
+# Generated by Django 4.0.3 on 2022-03-24 17:24
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+    def __init__(self, name, app_label):
+        super().__init__(name, 'django_fsm_log')
+
+    dependencies = [
+        ('django_fsm_log', '0004_auto_20190131_0341'),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name='statelog',
+            name='id',
+            field=models.BigAutoField(
+                auto_created=True, primary_key=True, serialize=False, verbose_name='ID'
+            ),
+        ),
+    ]
diff --git a/your_project/third_parties/django_fsm_log_/migrations_/__init__.py b/your_project/third_parties/django_fsm_log_/migrations_/__init__.py
new file mode 100644
index 00000000..e69de29b

python manage.py migrate django_fsm_log_

A bit hackish though, maybe there is a better way ?

This controversial issue, is mnaybe a sign that django-fsm-log should, maybe, not provide a concrete Models, but only an Abstract mode instead ?

Would it work ?
Do we know other third party django app, that adopted  this pattern  ?

If yes, I think, that's a direction we should explore, while keeping the concrete model for backward compatibility only.
We would make lot of projects happy (#34)

@alysivji, did you figure out how to make the monkey patching approach work?

@holvi-mikael I created a class that inherits the Django FSM application config and made my changes there. This SO answer is a good guide on the workflow I used.


Add support for GraphQL files

Awesome icons @be5invis, thanks for sharing them!
I started exploring GraphQL lately, but noticed you don't yet have an icon for .gql

Please update the AI file either, the second one, add an artboard with your new icon.

Yesss!


How to connect with Coinbase Wallet?

I tried using this package to connect to Coinbase Wallet but it didn't work. Can this package connect to Coinbase Wallet?. Thanks.

Hi, Coinbase Wallet app doesn't support WalletConnect protocol, you have to use Coinbase SDK to connect.
Please see here


New Game Mode: Quick Draw

Description
Add a new game mode where the user has to shoot all the circles as quickly as possible. At regular intervals, new circles will be added to the canvas up to a maximum number of allowed circles.
Issues

https://github.com/johneastman/circles/issues/49
https://github.com/johneastman/circles/issues/50
https://github.com/johneastman/circles/issues/51
https://github.com/johneastman/circles/issues/52
https://github.com/johneastman/circles/issues/53
https://github.com/johneastman/circles/issues/54
https://github.com/johneastman/circles/issues/56

Acceptance Criteria
Close this issue when all issues have been completed.

Notes

I moved this work to this branch
For Quick Draw, the "high score" actually needs to sort by minimum value since the purpose of that game mode is to shoot all the circles in the shortest amount of time.
updateScore needs to:

accept a GameMode parameter and only update the score when the current game mode matches the passed-in game mode. Otherwise, for Quick Draw, circle collisions will interfere with the score.
Change the Bullet parameter to accept a number so the method can be reused for both game modes.

Problems

I'm having an issue switching between game modes and updating the score. I considered moving the stopwatch logic out of the Stopwatch component, but the stopwatch was updating ridiculously fast and ended up producing NaN.
There was an issue where the Stopwatch component was still activating the score even when Quick Draw was not the current game mode.
Keeping the stopwatch logic in the Stopwatch does not allow me to display the score with the getScore, which is used in other places in the code (high scores, final score), and I'd prefer to not have separate logic doing the same thing.


write backfill script for creating syncs for connections in old installations

needed after https://github.com/supaglue-labs/supaglue/pull/461

don't need. truncated table