"Runtime check on non-ground type T may throw StrongModeError"

I got the strong-mode analyzer message:
Runtime check on non-ground type T may throw StrongModeError
...but what does it mean? When may it throw that? Why? How do I fix it?

Please re-open this issue on the dev_compiler project.

This is primarily relevant to DDC generated JS code, but it is a general soundness issue in Dart.
What this means is that we can't guarantee that the is check is sound.  E.g., the following:
var list = ["hello", "world"];
print(list is List<int>);

prints true.  list is actually considered a List<int> per the spec.  Function type checks have similar problems.
In general, type checks on String or List are fine, ones on List<Foo> are problematic and usually a bug in the program.
For type parameters (T is one here?), we don't necessarily know at compile time which one it is.  Do you have a pointer to your code?

Isn't that a List<dynamic>?
Here are the actual errors I get, with pointers to the code online:
[hint] Runtime check on non-ground type T may throw StrongModeError (https://github.com/flutter/engine/tree/master/sky/packages/sky/lib/src/widgets/drag_target.dart, line 158, col 9)
[hint] Runtime check on non-ground type ValueKey<T> may throw StrongModeError (https://github.com/flutter/engine/tree/master/sky/packages/sky/lib/src/widgets/framework.dart, line 34, col 9)
[hint] Runtime check on non-ground type StatefulComponentElement<dynamic, T> may throw StrongModeError (https://github.com/flutter/engine/tree/master/sky/packages/sky/lib/src/widgets/framework.dart, line 113, col 9)
[hint] Runtime check on non-ground type T may throw StrongModeError (https://github.com/flutter/engine/tree/master/sky/packages/sky/lib/src/widgets/framework.dart, line 331, col 39)

Yes, but List<dynamic> can act as any List type.
So, it's both a List<dynamic> and a List<int> per the Dart spec (as far as is checks go).  As well as a List<String> or a List<Map<Foo, Bar>>.
In general, list is List or list is List<dynamic> is a meaningful check.  list is List<int> is usually a bug.
To take the first example on your list, if you do something like:
var state = new DragTargetState<List<int>>();
var result = state.didEnter(["hello", "world"]);

then didEnter may accept the argument and return true.  The is check on line 158 will return true.

Is the runtimeType of a List<dynamic> the same as the runtimeType of a List<int>? Or is it a superclass of List<int>? I'm trying to figure out what the relationship is. Is is special-casing , or is it just walking a class hierarchy that I can't see?
I guess I'm saying it depends on what the definition of is is.
It sounds to me like what you're saying is that the bug is using [] without explicitly specifying the type. I'd certainly support a warning that flagged any code that implicitly typed something using dynamic.

The actual runtimeType is different.  The is check, however, is a subtype check - e.g., x is Object asks if x's type is a subtype of Object (which is alway true in Dart).
The funny thing about Dart's subtype relationship is that it is cyclic in a few corners.  List<int> is a subtype of List<dynamic> and List<dynamic> is a subtype of List<int>.  Strong mode is basically disallowing the latter.
A no implicit dynamic would be useful, but perhaps not really usable until we have generic methods.  (E.g., Iterable.map always returns Iterable<dynamic>.)
Another (complementary) possibility is to annotate classes like DragTargetState that effectively say the they can only be instantiated on 'safe' types (what we're opaquely calling ground types).  So, no error on x is T inside the class.  Instead, warn on new DragTargetState<List<int>>() but allow new DragTargetState<List>().

What I'd ideally want is to allow DragTargetState to differentiate List from List, and disallow List. Is there a way to do that?

We might be able to make that work.  To do it statically, we may need to annotate both DragTargetState and Foo to denote that DragTargetState requires a 'safe' type param and that Foo is always safe (no dynamics inside the runtime type).  The latter might be something like:
@noDynamic
class Foo<T> { ... }

which would allow new Foo<int>() but disallow new Foo() or new Foo<dynamic>().

DragTargetState can't know about Foo, and Foo can't know about DragTargetState. Foo could be List, for instance. Or some generic class from some random pub package.
I'm fine with waiting for generic methods if it would allow us to warn on implicit dynamics.

Anyway, it sounds like this error message is actually useful (even outside strict mode). The bug is that it's very unclear what it means or what action should be taken to address it.

I believe @munificent recently fixed this?

I don't think so. The underlying behavior is unchanged as far as I know. I definitely think we can improve the error message, and I'd be happy to do that.

merging https://github.com/dart-lang/sdk/issues/26100, CC @hermanbergwerf

Now that more folks are hitting this, I wonder if we can add a document somewhere to describe why as List<int> and as int are treated differently. It's definitely something folks wonder about.
@hermanbergwerf -- the way to handle that in your example would be something like:
factory fromData(Map<String, dynamic> data) {
  List<int> field;

  if(data.containsKey('field') && data['field'] is List/*OLD: <int>*/) {
    // OLD: field = data['field'] as List<int>;
    field = new List<int>.from(data['field']); // NEW
  } else {
    throw new ArgumentError.value(data, 'data', "no valid data field for 'field'");
  }

  return new MyClass(field);
}

... the copy will ensure that each item is checked that it's actually an integer.
An alternative would be to supply a checked wrapper of some sort, e.g. new CheckedList<int>(list), there's another bug requesting that feature.

An alternative would be to supply a checked wrapper of some sort, e.g. new CheckedList(list), there's another bug requesting that feature.

I think @nex3 is working on a wrapper class like that for the collections package.

https://codereview.chromium.org/1840573002/

I think it's worth considering where to put the blame. Vijay said

In general, list is List or list is List<dynamic> is a meaningful check. list is List<int> is usually a bug.

The underlying issue is that a reference of type List<int> xs justifies conclusions like "xs[0] has type int" (unless xs == null || xs.length < 1, of course, but then we'll be thrown out of the context where that question is relevant).
But if xs is allowed to refer to an instance of List<dynamic> then there is no such guarantee. The spec subtype rule for dynamic and type argument covariance conspire to allow this, presumably in order to let dynamic have an effect that may be described as "just make the type checker stop complaining". ;-)
From the point of view of soundness this is an expensive design choice. In particular, navigation in the object graph is unsound even in checked mode if we encounter an object whose runtime type is a reified-dynamic type (that is, a generic class whose type arguments include dynamic at any level, e.g., List<List<dynamic>>).
I'd like to be able to guarantee that such objects just don't exist, in which case xs is List<int> is not a bug, it's a perfectly appropriate runtime check which will justify conclusions like "xs[0] has type int". This means that generic types can be recovered (re-discovered) at runtime as needed, just like non-generic types (because, luckily, we don't have any objects whose runtime type is dynamic, so nothing will claim to have a non-generic type like MyClass unless it is an instance of a class that actually implements/extends MyClass).
Strong mode in the analyzer uses a different approach, where reified-dynamic type instances are allowed to exist, and dynamic type checks for instantiations of generic classes are frowned upon. This means, of course, that a List<dynamic> may be accessible via references of type List<int> xs and List<Object> ys at the same time, so when getting an element from xs it may be a String that someone added via ys:
List<Object> f(List<int> xs) => xs;

main() {
  List<int> xs = [];
  List<Object> ys = f(xs);
  ys.add("Hello, world!");
  int x = xs[0];
  print("$x");
}

This program has 'No issues' in strong mode analysis, and still it stops with a runtime error in checked mode.
It's because of this kind of phenomenon that strong mode cannot have exactly the same semantics as the one that the spec describes. I'd love to see strong mode and the spec converge, but it is not trivial (especially as long as we allow all these reified-dynamic instances to exist ;-).

@eernstg Just a quick note:
For soundness, strong mode still requires a runtime type check on generic class method arguments where the parameter type is a type parameter (or based on the type parameter).
In DDC,
ys.add("Hello, world!");

would fail at runtime.  The alternative is to disallow covariance on generics, but that's a bigger change.

Indeed, but you're not being entirely explicit about what is going on here:
If you want a runtime check to fail at ys.add("Hello, world!") when ys is an instance of List<dynamic> then you can't use the  actual value of the type argument to make that decision: A String certainly does have type dynamic. So you're not just referring to a normal checked mode check. It's not a check based on the static type, either: If you perform the dynamic check based on the type annotation of ys then we will simply require that the String is an Object -- and succeed.
So the only way you can get the run time failure you mention is by making sure that the given list is an instance of something else than List<dynamic> in the first place, and this relies on inference from the declaration of xs. So you'll let List<int> xs = [] mean List<int> xs = <int>[], and then you'll have an int list at the invocation of add.
This is the kind of thing I was referring to when I said 'cannot have exactly the same semantics'.

For what it's worth, over in Flutter land, we don't really care that <dynamic>["Hello"] matches List<int>.
We're designing our framework to handle two cases:

people who care very much about types and want everything to be typed and type-checked.
people who don't care about types at all.

For people in the first category, we have configured the analyzer to complain about any missing types, and (once the analyzer supports this) this will include complaining about untyped literals. So there will never be a List<dynamic> to conflict with unless the author is making a very specific choice, which can be an informed choice whose effects we would document.
For people in the second category, all their lists are <dynamic>. It's their job to make sure that they don't send the wrong kind of list to the our APIs. They are explicitly saying that they want to trade source-level brevity for greater responsibility.
We try to write our code to handle both. This means we don't do checks like "is this a List<int> or a List<String>" when we could be getting a List<dynamic>.
As a side note, for people in the first bucket, I'm totally fine with the analyzer and the runtime using types on the LHS to imply types on the RHS or vice versa. (I think I'd prefer explicit RHS types implying LHS types, but that's another story.) For people in the second bucket, though, there's no types at all in the source, and it's not ok to turn this:
   var xs = [0];

...into a List<int>. There's not enough information there to say if it's a List<int>, List<Object>, or List<num>. (Or indeed, a List<double> where someone has assumed that 0 is compatible with double, which is something we see a lot. But that's also another story.)

@Hixie could you please file a separate bug on this?  There is a general question of how strong (magical?) inference should be for generic methods and generic class constructors (and list literals are just a case of the latter).

For people in the second bucket, though, there's no types at all in the source, and it's not ok to turn this:
    var xs = [0];

...into a List<int>. There's not enough information there to say if it's a List<int>, List<Object>, or List<num>. (Or indeed, a List<double> where someone has assumed that 0 is compatible with double, which is something we see a lot. But that's also another story.)

Happy to file bugs -- can you clarify which of my various passive aggressive complaints in my last comment you'd like bugs for? :-)

We currently infer / instantiate:
var xs = [0];

as List<int>.
That means you'll get a static error if follow that with:
xs.add("hello");

I think you're objecting.  :-)

Type-asserting wrapper methods have landed in collection 1.5.0.

Duping this against https://github.com/dart-lang/sdk/issues/28988 ... but the good news is, we're going to remove this error.

(also, that bug is about DDC, the analyzer message was removed quite a while ago)


[GSSoc'23] Added Wault Master Game

PR Description 📜
Wault-Master
A TailwindCSS based Javascript game in which the user must guess the completely random vault code in order to win.
Features
Fully Responsive (Cross-platform)
Written in Tailwind and Pure JS.
IOS-Like design.
Dark mode. (no toggle, yet.)

Fixes #1992

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for website preview in assets/images
[x] I have added entries for my game in main README.md
[x] I have added README.md in my folder
[ ] I have added working video of the game in README.md (optional)
[x] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

Thank you @Vedansh-777 ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

Thank you  @Vedansh-777 , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


Camera: app passed NULL surface

Camera: app passed NULL surface on Android Nougat Google Pixel in Log Console.
There is no more message provided.

@TonyTangAndroid are you setting the surface view as explained in the readme ? Also check if the sample app also throws the same error ?

Yes. I was using the exact sample app that gets this error.

@nisrulz +1 I am having the same issue. It occurs when I read a QRCode with a video a url, launch the intent to play the video and come back to the fragment that has the qrreader. Using Lollipop on Moto G 1st gen.

Let me look into it and update it here.

@nisrulz I might be having the same issue. The surface view turns black and doesn't update whenever the activity resumes. This happens with the sample app as well.
To replicate, simply leave the app by going to home, then resume the app.

Hello sir ,i have the same issue as of Mr Joelclaudio.And i really need the solution.Please help.When i open the mail client by intent than on coming back to activity i m not able to scan and surface view shows black screen.Please help as soon as possible

Ok I managed to find the problem and fix it.
When QReader.initAndStart() is called in onResume(), it adds a GlobalLayoutListener on surfaceView which calls QReader.start(). In here, it checks the boolean surfaceCreated and if it is false, it adds surfaceHolderCallback to the holder.
Now the issue is that surfaceHolderCallback removes itself from surfaceView when it is destroyed. It might be a change in the latest Android but when the view is rebuilt, the same surfaceView is used and re-created. The problem is that the re-adding of surfaceHolderCallback occurs after the surfaceView has been created, thus the surfaceCreated listener event is never called, resulting in the camera not starting.
The way to fix this is to not remove the callback when surfaceView is destroyed. This can be done by removing line 106 of QReader.java: surfaceHolder.removeCallback(this);

Helo sir,
Thank you for your precious reply.but m not able to identify line 106!!
where can i add "QReader.java: surfaceHolder.removeCallback(this);" .
On Thu, Jan 5, 2017 at 4:11 AM, Wai Kay<EMAIL_ADDRESS>wrote:

Ok I managed to find the problem and fix it.
When QReader.initAndStart() is called in onResume(), it adds a
GlobalLayoutListener on surfaceView which calls QReader.start(). In here,
it checks the boolean surfaceCreated and if it is false, it adds
surfaceHolderCallback to the holder.
Now the issue is that surfaceHolderCallback removes itself from
surfaceView when it is destroyed. It might be a change in the latest
Android but when the view is rebuilt, the same surfaceView is used and
re-created. The problem is that the re-adding of surfaceHolderCallback
occurs after the surfaceView has been created, thus the surfaceCreated
listener event is never called, resulting in the camera not starting.
The way to fix this is to not remove the callback when surfaceView is
destroyed. This can be done by removing line 106 of QReader.java:
surfaceHolder.removeCallback(this);
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/nisrulz/qreader/issues/28#issuecomment-270630610, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AXx0NJG22Xiy_YDL9IQNzj77e5UiS7U3ks5rPN3dgaJpZM4LKwVZ
.

I got the line bt its the read only file ,so cant remove the line
On Tue, Jan 10, 2017 at 2:48 AM, Malti Devnani<EMAIL_ADDRESS>wrote:

Helo sir,
Thank you for your precious reply.but m not able to identify line 106!!
where can i add "QReader.java: surfaceHolder.removeCallback(this);" .
On Thu, Jan 5, 2017 at 4:11 AM, Wai Kay<EMAIL_ADDRESS>wrote:

Ok I managed to find the problem and fix it.
When QReader.initAndStart() is called in onResume(), it adds a
GlobalLayoutListener on surfaceView which calls QReader.start(). In
here, it checks the boolean surfaceCreated and if it is false, it adds
surfaceHolderCallback to the holder.
Now the issue is that surfaceHolderCallback removes itself from
surfaceView when it is destroyed. It might be a change in the latest
Android but when the view is rebuilt, the same surfaceView is used and
re-created. The problem is that the re-adding of surfaceHolderCallback
occurs after the surfaceView has been created, thus the surfaceCreated
listener event is never called, resulting in the camera not starting.
The way to fix this is to not remove the callback when surfaceView is
destroyed. This can be done by removing line 106 of QReader.java:
surfaceHolder.removeCallback(this);
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/nisrulz/qreader/issues/28#issuecomment-270630610,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXx0NJG22Xiy_YDL9IQNzj77e5UiS7U3ks5rPN3dgaJpZM4LKwVZ
.

It is read only because you are currently "linking" to the package in GitHub. What you need to do is clone the source out and then add it as a module to your project. Then you can edit the source.

@pottiepie if you think that you have fixed the issue, could you send a PR. I haven't been able to pick   the library code for sometime because of work.

@pottiepie  i did as per your instructions bt it gives me error on resuming that "Fail to connect to camera service".So help me in that

Closing as issue is outdated. Please open a new issue if the issue still persists.
The library will be undergoing some maintenance soon.


Game gomoku tested model update

name: Pull Request
about: Create a pull request
Description
updated the tested wrappers/models for the example [game_gomoku](https://github.com/modelscope/agentscope/tree/main/examples/game_gomoku)
Checklist
Please check the following items before code is ready to be reviewed.

[ ]  Code has passed all tests
[ ]  Docstrings have been added/updated in Google Style
[x]  Documentation has been updated
[ ]  Code is ready for review

Please see inline comments.

I think the version I did modification on was taken from an out-of-date version of the notebook. I just updated it with the latest version from the official main branch, but it seems there are still many changes, and I'm not sure why is that the case..


🚀 Feature Request: Allow arbitrary unauthorized self-signed certificates

Describe the solution
Hi there! Not sure if this should be a bug or a feature request, but here's my need:
I'm trying to build a Gemini proxy (an HTTP proxy for the Gemini protocol).
More info about Gemini is here: https://geminiprotocol.net, but it's similar to Gopher, and I've been able to use the same sample code for Gopher (in the TCP documentation) with little changes.
The main difference between them however is that Gemini enforces the use of TLS, but not through CAs, but instead through self-signed certificates (on the server-side). This means that building a proxy requires that I accept any valid certificate, without checking it against a CA.
Yes, that allows MITM attacks and such, but this is the way Gemini is designed.
For now, I get this error:
✘ [ERROR] workerd/jsg/util.c++:276: error: e = kj/compat/tls.c++:221: failed: TLS peer's certificate is not trusted; reason = self signed certificate

But I can't seem to find a way to basically skip certificate checks. In Node.js this can be done using the NODE_TLS_REJECT_UNAUTHORIZED=0 env var, but it doesn't work in Workers.
Thanks in advance.

Here's the repository, just in case: https://github.com/Skaldebane/gemini-proxy-worker

Also need this feature


Make it clearer that non-github projects are included, and can be included

Currently, the language of the statusboard implies that only GitHub projects are included. While that's the case for the majority of brigades, it is not inherently true, as any CSV or JSON file could be an organization's source of truth.
The statusboard should include some description about how the data is gathered so that people can understand where it's coming from, what its limitations are, and what they need to do if they want to see non-coding projects show up in the index. This might beg some questions about how we determine the last updated date of projects listed in a CSV or JSON file.
I wrote up a description for the general documentation push last spring that may be helpful to reference, but has perhaps been overwritten recently? This is the relevant PR where it was added.

I wrote up a description for the general documentation push last spring that may be helpful to reference, but has perhaps been overwritten recently? This is the relevant PR where it was added.

Isn't it moved? https://github.com/codeforamerica/brigade-project-index/blob/master/docs/projects/crawler/contributing/get-indexed.md

Yes Jonatan, you are right, that is the same description on how to list projects.
Maybe we can add a link from the Dashboard description (in the same set of docs) to the "Crawler" relevant section.

Proposed solution: Add a link "Add project" which links to the page on github which describes how to add projects.

Instructions are already there, we have a 2 hops from ProjectIndex page to detailed instructions.
https://brigade.cloud/projects/crawler/contributing/get-indexed/


the epg has dissapeared. suspected epg download cooldown from provider but not sure.

xteve  2.5.0 for windows
epg has dissapeared from emby and plex for most channles but not all. i believe there is some sort of cooldown from my iptv provider when downloading the epg too many times in a short period of time. however, i dont know why theres no cooldown with tivimate. does anyone know some things i can try? i used to be able to wait about 6 hours for my epg to come back but this time i have waited 24 hours and it still has not come back into emby or plex from xteve or my provider.

If you are using an external tool like guide2go to parse your EPG you have to do a manual update, it'll fix itself

I'm only using xteve with emby. I have nothing external.
On Mon, 23 May 2022, 6:24 pm Ge082, @.***> wrote:

If you are using an external tool like guide2go to parse your EPG you have
to do a manual update, it'll fix itself
—
Reply to this email directly, view it on GitHub
https://github.com/xteve-project/xTeVe/issues/315#issuecomment-1135196300,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AODRSPOEYUV3KG4STKARRF3VLQASTANCNFSM5WRLWMLQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

Provider EPG is terrible. Consider adding a Zap2it free account for Premium EPG, although this EPG will require manual mapping.


Error when filtering with no username in Orders List

Hi, we are trying out Homs using a fresh install of the Docker images, with only a few changes made (a few new users, new orders etc.) for testing purposes.
Steps to reproduce
Remove all the tags in the username field of the Orders List. Press search.

Error

Thanks for your help.

Hello, @mchesterkadwell
Thanks for your report! Bug is reproduced and confirmed. We are going to fix it soon.
BTW: To show orders with empty user field you need to select Empty user in User name field and click Search button.

Thanks for your quick reply!
When a user has removed the Empty tag, it does not seem to be possible to get it back again:

You have to reload the page to get it back again.

Actually Empty is not a real user. It's a special value to represent an empty user. That's why it's not found when you type Empty. You can add Empty tag if your click on Not set value. This is the only way and is not a bug.

leads to

BTW we are going to change Not set text to Empty.


test(circuits): fix ust proof, add with/without replacement tests

Pull request checklist
Please check if your PR fulfills the following requirements:

[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been reviewed and added / updated if needed (for bug fixes / features)
[x] Build (yarn build) was run locally and any changes were pushed
[x] Lint (yarn lint --check) has passed locally and any fixes were made for failures

Pull request type

Please check the type of change your PR introduces:

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: fix #368
What is the new behavior?

Does this introduce a breaking change?

[ ] Yes
[x] No

Other information

I think this is fixed in #382, i added this.

Closing as fixed


Fixing invalid SASS-Loader paths

I have found wrong path in file webpack.config.dev.js and webpack.config.prod.js in plugin sass-loader. Take a look at line 116 of this file. The problem should be in includePath parameter. I'm guessing there should be option "styles" in third parameter instead of "scss". If I'm wrong just let me know why. Thank you.
{
  loader: 'sass-loader',
  options: {
	includePaths: [path.resolve(__dirname, 'src', 'scss')],
	sourceMap: true
}

Issue: https://github.com/coryhouse/react-slingshot/issues/594

Coverage remained the same at 91.453% when pulling e53aa7aba270caa758606b1fda2ef3b834ce3a59 on vbatrla:master into 7a7d886d7a6dfe05d412c5fb7b16ccaf482f4152 on coryhouse:master.

Thanks for the PR! I'm thinking we can be less specific and point it simply at the src directory so people can place Sass anywhere in there. Thoughts?

I've seen your email but couldn't respond earlier. I think there should be
one configuration file where could be placed all specific paths and
variables. So user could have a possibility to set custom paths and
variables.
pá 2. 11. 2018 v 12:43 odesílatel Cory House<EMAIL_ADDRESS>napsal:

Thanks for the PR! I'm thinking we can be less specific and point it
simply at the src directory so people can place Sass anywhere in there.
Thoughts?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/coryhouse/react-slingshot/pull/595#issuecomment-435352864,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANKuj88k1phcHO0itZlG9UI_610Gdn-Aks5urC_QgaJpZM4YLdSN
.

Just following up on this again @vbatrla.


Socket Error while working

I started bot after new fix, and now he works, but i have error and he don't draw:
Starting with parameters: {"constantWatch":false,"customEdgesMapImagePath":"","ditherTheImage":true,"doNotOverrideColors":[null],"fingerprint":"d072436afbe1ef7657f3d401411bd46071570951437","imgPath":"M.png","xLeftMost":13900,"yTopMost":-10900} Reading the picture... Done reading. 70 x 70 currentlyMaking.png <- Contains the current edges map we're working with. expectedOutput.png <- Contains the final expected image. Initializing... Ok, let's go! TypeError: Cannot read property 'status' of undefined     at axios_1.default.catch (c:\Users\Пользователь\Desktop\pixelplanet.fun-bot-master\dist\chunkCache.js:236:40)     at process._tickCallback (internal/process/next_tick.js:68:7) Socket encountered error, closing socket connect ETIMEDOUT <IP_ADDRESS>:443 Socket was closed. Reconnecting... Socket encountered error, closing socket connect ETIMEDOUT <IP_ADDRESS>:443 Socket was closed. Reconnecting... TypeError: Cannot read property 'status' of undefined     at axios_1.default.catch (c:\Users\Пользователь\Desktop\pixelplanet.fun-bot-master\dist\chunkCache.js:236:40)     at process._tickCallback (internal/process/next_tick.js:68:7) Socket encountered error, closing socket connect ETIMEDOUT <IP_ADDRESS>:443 Socket was closed. Reconnecting... TypeError: Cannot read property 'status' of undefined     at axios_1.default.catch (c:\Users\Пользователь\Desktop\pixelplanet.fun-bot-master\dist\chunkCache.js:236:40)     at process._tickCallback (internal/process/next_tick.js:68:7) Socket encountered error, closing socket connect ETIMEDOUT <IP_ADDRESS>:443 Socket was closed. Reconnecting... TypeError: Cannot read property 'status' of undefined     at axios_1.default.catch (c:\Users\Пользователь\Desktop\pixelplanet.fun-bot-master\dist\chunkCache.js:236:40)     at process._tickCallback (internal/process/next_tick.js:68:7) Socket encountered error, closing socket connect ETIMEDOUT <IP_ADDRESS>:443 Socket was closed. Reconnecting... TypeError: Cannot read property 'status' of undefined     at axios_1.default.catch (c:\Users\Пользователь\Desktop\pixelplanet.fun-bot-master\dist\chunkCache.js:236:40)     at process._tickCallback (internal/process/next_tick.js:68:7) Socket encountered error, closing socket connect ETIMEDOUT <IP_ADDRESS>:443 Socket was closed. Reconnecting...
Maybe i need open firewall ports or something else?

I guess, that it's conflict with Cloudflare protection, but i can't really say.

Hmm.. If you have some custom protection, it might be causing this, blocking requests from other apps other than browser or something along those lines. I'm not sure...

Like antivirus programm?

Lol, ok.
So, i'm live in Russia, and PixelPlanet blocked here, so i need use VPN.
I used VPN, but it just Chrome extension, so Windows Shell didn't worked anyway.
Now it's started work.
If you have same problem, install VPN client.

But now i'm have infinity captcha check :/
Starting listening for changes via websocket Initialization complete Capthca was requested.  Press any key to continue Pixel updated received: 13900:-10898, color: 31 Capthca was requested.  Press any key to continue Pixel updated received: 13901:-10896, color: 31 Capthca was requested.  Press any key to continue Capthca was requested.  Press any key to continue Capthca was requested.  Press any key to continue Capthca was requested.  Press any key to continue Pixel updated received: 13942:-10870, color: 31 Pixel updated received: 13943:-10872, color: 31 Pixel updated received: 13948:-10870, color: 31 Pixel updated received: 13944:-10866, color: 31 Capthca was requested.  Press any key to continue Pixel updated received: 13945:-10869, color: 31 Pixel updated received: 13946:-10868, color: 31 Pixel updated received: 13944:-10868, color: 31 Pixel updated received: 13945:-10868, color: 31 Pixel updated received: 13945:-10867, color: 31 Capthca was requested.  Press any key to continue

I bet it's not Russia who is blocking pixelplanet it's the other way round. Since there were massive "griefing attacks" from Russia a while ago. Did not know that that could be the case.
About this catcha thing, if you enabled global vpn client, disable one on your chrome. I bet it gives different ips then to your browser instance and the "bot client". Then it would not work

Ok, i try it.

Wow, started working, thanks.
About different IP's, you're right.

Glad it helped. Have fun :D

Thanks


Edge Model: how will APIs be offered?

received from @PrashantGDT
"How will these APIs will be offered to customers. Through one common single platform or there will multiple platform owned by different operators?"

In my understanding any operator who want to provide these services as exposed by the APIs should be able to independently offer them to their customers. In case of GSMA Operator Platform based implementation, the APIs can also be federated between the two operators and in that case one operator can front end the APIs from its customers and forward the APIs to other operators responsible for that request provided the other operator via their operator platform also provide support for these APIs.

How they are offered is out of scope of CAMARA, but as @gunjald says this is being discussed in other groups. But basically it will be flexible: operators can expose themselves, via partners, via a federation etc,.


Example jar has invalid main class

This was introduced by issue gh-14 when packages were renamed.
We should probably just remove the main class and allow users to chose which example class to run on the command line.

Migration Steps
To run the examples on the command line you now need to specify which example to run by providing the class name as an argument and using 'java -cp' instead of 'java -jar'
java -cp example-jar-with-dependencies.jar gaffer.example.gettingstarted.analytic.LoadAndQuery1

Pull request approved and merged in to develop branch.


Delete osmomath.BigDec.ApproxRoot

Summary
As far as I can tell, this is only used to derive some constants in a stableswap test. We can just hardcode those constants, and get this code out of the repo.
Problem Definition
.
Proposed Feature
Delete code

used here too

Ah thanks! Maybe we should just move that solve_cfmm_direct function to testing and approx root to testing in stableswap then.


Reloading Airplay player results in flickering display

What version of Music Assistant has the issue?
2023.6.b32
The problem
When doing some testing I disabled the DLNA player and enabled the Kodi Airplay player. This stayed unavailable so I reloaded the AirPlay provider which then resulted in flickering on various parts of the UI. @marcelveldt see this video https://drive.google.com/file/d/18AXWpaepi18sWxwG9J9tg-COQBotbU_Q/view?usp=drive_link
A restart of the addon is required to fix it
How to reproduce
As above
Relevant log output
No response
Additional information
No response
What version of Home Assistant Core are your running
2023.5.4
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
Raspberry Pi

Should be fixed in beta36, if not please let me know.

Just happened to do a reload of the slimproto provider and got the flickering thing again. Video is on the Google Drive

Appears fixed as at beta88


Fix a small leak in generate

This fixes #7560
I still don't understand exactly why the regular java leak detection code didn't pick this up, but this does fix the leak.

build


[BUG] - International Calendars not working in documentation

NextUI Version
2.4.2
Describe the bug
Documentation (incorrect behaviour): https://nextui.org/docs/components/calendar#international-calendars

Storybook (correct behaviour): https://storybook.nextui.org/?path=/story/components-calendar--international-calendars&globals=locale:da-DK

Your Example Website or App
No response
Steps to Reproduce the Bug or Issue
Compare the difference between documentation and storybook for International Calendars
Expected behavior
The locale should be applied
Screenshots or Videos
No response
Operating System Version
macOS
Browser
Chrome

Can you explain more about the issue?

@sac1hin Sure. If you look at international calendars in documentation. You can see the locale is set to zh-CN-u-ca-chinese. However, in the preview, it still shows in English while it should show in Chinese. In the storybook, you can see it renders correctly.
<I18nProvider locale="zh-CN-u-ca-chinese">
  <Calendar aria-label="Date (International Calendar)" />
</I18nProvider>

Hi, Can you assign this to me ?

@Mihir867 There's a PR already. Please check out other issues.

@Mihir867 There's a PR already. Please check out other issues.

Ohh sure!!
Could you guide me with something I can work on, I'm actually quite new to this


Remove.sh

remove.sh: 101: remove.sh: Syntax error: "(" unexpected (expecting ";;")

Which command did you use to execute the script?

Sudo ./remove.sh

However not being sure, last time I had similar issue and I overcome by using either sudo sh remove.sh or ./remove.sh. You can also try uninstalling by using RPi_Cam_Web_Interface_Installer.sh.

I solved it by removing the folder and the apache2 folder and then its fixed,but i wanted to allert you with the problem.


[bug]: Autocomplete Suggestions

Describe the bug
Can't scroll if there are too many options in the Autocomplete Suggestions Popover.
How to reproduce
Add many tags to the autocompleteOptions where it will overflow
Link to reproduction
https://github.com/JaleelB/emblor
Additional information
No response

You can specify a max-size for the popover and add overflow auto.
This is the snippet from the code I'm using:
styleClasses={{
  autoComplete: {
    popoverContent: "max-h-[150px] overflow-y-auto",
  },
}}

I tried adding the styleClasses, but it still seems to be unscrollable. It is currently within a shadcn stepper component within a form.
https://github.com/user-attachments/assets/bee38de8-3ac7-47cb-9e53-477de85cb0d4
                              <TagInput
                                      {...field}
                                      styleClasses={{
                                        autoComplete: {
                                          popoverContent:
                                            "max-h-[200px] overflow-y-auto z-100",
                                        },
                                      }}
                                      placeholder="Enter a tag"
                                      tags={tags}
                                      className="sm:min-w-[450px]"
                                      activeTagIndex={activeTagIndex}
                                      setActiveTagIndex={setActiveTagIndex}
                                      enableAutocomplete
                                      autocompleteOptions={TagChoices}
                                      restrictTagsToAutocompleteOptions
                                      setTags={(newTags) => {
                                        setTags(newTags);
                                        setValue(
                                          "tags",
                                          newTags as [Tag, ...Tag[]]
                                        );
                                      }}
                                      truncate={5}
                                      clearAll
                                    />


Add decoding benchmark plus benchmark for GZIP-compressed CSV files

Refs https://github.com/clue/reactphp-ndjson/pull/16, https://github.com/clue/reactphp-zlib/pull/24 and others
Resolves #11

For the reference, here's what the output looks like on my machine:
$ php examples/91-benchmark-count.php < IRAhandle_tweets_1.csv
243891 records in 4.812s => 50683 records/s

$ php examples/92-benchmark-count-gzip.php < IRAhandle_tweets_1.csv.gz
243891 records in 4.789s => 50929 records/s

Interestingly, this means that decompressing and parsing a large file was slightly faster than parsing an uncompressed CSV file on my system (I/O overhead vs CPU usage). Decoding NDJSON is somewhat faster than decoding CSV in my benchmarks (https://github.com/clue/reactphp-ndjson/pull/16):
me@me-in:~/workspace/clue-reactphp-ndjson$ php examples/91-benchmark-count.php < IRAhandle_tweets_1.ndjson
243891 records in 1.021s => 238835 records/s

Comment incited by this tweet.

Running the above benchmark against this CSV file (careful when on restricted traffic volume, 90MB file), I get roughly 25% improvement on PHP 8.0, with another ≈4% on top when enabling JIT.
The running machine is a MacBook Pro, late 2019, with a 2.3GHz 8-Core Intel Core i9 CPU.
PHP 7.4
php -v

PHP 7.4.14 (cli) (built: Jan  8 2021 13:20:04) ( NTS )
Copyright (c) The PHP Group
Zend Engine v3.4.0, Copyright (c) Zend Technologies
    with Zend OPcache v7.4.14, Copyright (c), by Zend Technologies

php examples/91-benchmark-count.php < IRAhandle_tweets_1.csv

243891 records in 2.685s => 90838 records/s

Records per second roughly settle between 86k and 92k.
PHP 8.0
php -v

PHP 8.0.1 (cli) (built: Jan  8 2021 12:43:54) ( NTS )
Copyright (c) The PHP Group
Zend Engine v4.0.1, Copyright (c) Zend Technologies
    with Zend OPcache v8.0.1, Copyright (c), by Zend Technologies

Without JIT
php examples/91-benchmark-count.php < IRAhandle_tweets_1.csv

243891 records in 2.146s => 113654 records/s

Records per second roughly settle between 105k and 120k (a plus of about 25%).
With JIT
php -d opcache.enable_cli=1 -d opcache.jit_buffer_size=100M examples/91-benchmark-count.php < IRAhandle_tweets_1.csv

243891 records in 2.051s => 118918 records/s

This is not a lot faster than without JIT, but shows way less variance with records per second settling between 115k and 120k.

Comment incited by this tweet.

Running the above benchmark against this CSV file (careful when on restricted traffic volume, 90MB file), I get roughly 25% improvement on PHP 8.0, with another ≈4% on top when enabling JIT.
The running machine is a MacBook Pro, late 2019, with a 2.3GHz 8-Core Intel Core i9 CPU.
PHP 7.4
php -v

PHP 7.4.14 (cli) (built: Jan  8 2021 13:20:04) ( NTS )
Copyright (c) The PHP Group
Zend Engine v3.4.0, Copyright (c) Zend Technologies
    with Zend OPcache v7.4.14, Copyright (c), by Zend Technologies

php examples/91-benchmark-count.php < IRAhandle_tweets_1.csv

243891 records in 2.685s => 90838 records/s

Records per second roughly settle between 86k and 92k.
PHP 8.0
php -v

PHP 8.0.1 (cli) (built: Jan  8 2021 12:43:54) ( NTS )
Copyright (c) The PHP Group
Zend Engine v4.0.1, Copyright (c) Zend Technologies
    with Zend OPcache v8.0.1, Copyright (c), by Zend Technologies

Without JIT
php examples/91-benchmark-count.php < IRAhandle_tweets_1.csv

243891 records in 2.146s => 113654 records/s

Records per second roughly settle between 105k and 120k (a plus of about 25%).
With JIT
php -d opcache.enable_cli=1 -d opcache.jit_buffer_size=100M examples/91-benchmark-count.php < IRAhandle_tweets_1.csv

243891 records in 2.051s => 118918 records/s

This is not a lot faster than without JIT, but shows way less variance with records per second settling between 115k and 120k.

@loilo Thank you very much for sharing your results here!
Really interesting to see how PHP 8 improved this somewhat and with JIT enabled may be able to achieve even better results. I haven't toyed around with this yet, but I wonder if different JIT settings (e.g. https://stitcher.io/blog/php-8-jit-setup)  may yield even better results 🤘
By the way, if you're into benchmarking: https://github.com/clue/reactphp-ndjson includes a very similar benchmark script that can be executed on the same input data set. It was ~5 times faster than this one last time I checked. https://github.com/clue/reactphp-tsv includes a similar benchmark script that was ~10 times faster than this one. 🔥

@loilo Thank you very much for sharing your results here!
Really interesting to see how PHP 8 improved this somewhat and with JIT enabled may be able to achieve even better results. I haven't toyed around with this yet, but I wonder if different JIT settings (e.g. https://stitcher.io/blog/php-8-jit-setup)  may yield even better results 🤘
By the way, if you're into benchmarking: https://github.com/clue/reactphp-ndjson includes a very similar benchmark script that can be executed on the same input data set. It was ~5 times faster than this one last time I checked. https://github.com/clue/reactphp-tsv includes a similar benchmark script that was ~10 times faster than this one. 🔥


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-main" workflow

Codecov Report

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

@@           Coverage Diff           @@
##             main      #18   +/-   ##
=======================================
  Coverage        ?   96.59%
=======================================
  Files           ?        9
  Lines           ?      176
  Branches        ?       15
=======================================
  Hits            ?      170
  Misses          ?        6
  Partials        ?        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 917f112...c1b0d70. Read the comment docs.


Populate users to appropriate groups

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.
GitHub Users Who Are Not Authorized To Contribute
The following GitHub users do not appear to have signed a CLA:

@FlorentFlament

Please sign the CLA here.

Hi. Your friendly Curry bot here. Just letting you know that all commit authors have become authorized to contribute. I have added the "Signed CLA" label to this issue so it can easily be found in the future.

Any chance CI fails can be reviewed and corrected (looks like syntax only problems only)? If this could be merged that would be perfect.

@rmldsky Sorry for the delay, I didn't notice that the tests were failing. Hopefully, this is now fixed.

Merged with PR #133


gpkg: invalid filepath

I can't set filepath for tegola_lambda for gpkg file.

Can you please provide the [[providers]] block of the config so I can see how you're referencing it?
Btw, that's great you're using tegola_lambda and Geopackage. I think it's such a cool implementation.

Hi!
[[providers]]
name = "ne"
type = "gpkg"
filepath = "s3://{{bucket}}/natural_earth_vector.gpkg"
#srid = 3857
[[providers.layers]]
name = "ne_10m_admin_0_label_points"
tablename = "ne_10m_admin_0_label_points"
fields = ["sr_subunit","scalerank"]

......
Other providers looks like first

I see, the issue there is that the geopackage provider can't pull the file from S3 right now (though that could be added, cool idea). Depending on the size of the geopackage, try zipping it up with tegola_lambda and deploying it as part of the Lambda function. The Lambda limits page shows you can do 50mb zipped and 250mb unzipped.

Thanks, but for the moment I wouldn't like it, the gpkg weigh about 3Gb each, I think it will play with postgres (with geopackage it's faster)

Got it. I'm not sure if Lambda would allow for that large of a file to be downloaded, but then again I have never tried.
If you're going to play with Postgres, maybe check out Postgres Aurora too. I put together a blog on a fully serverless architecture that might be helpful: https://medium.com/@alexrolek/the-serverless-vector-map-stack-lives-22835d341d7d

Got it. I'm not sure if Lambda would allow for that large of a file to be downloaded, but then again I have never tried.
If you're going to play with Postgres, maybe check out Postgres Aurora too. I put together a blog on a fully serverless architecture that might be helpful: https://medium.com/@alexrolek/the-serverless-vector-map-stack-lives-22835d341d7d

Could you explain to me how to download the file with tegola?

I was referring to if tegola_lambda supported downloading from S3. I'm not sure lambda could download a 3GB file.


Various issues in Josharias Survival

Tested around for a while and found some issues to document - some may relate to recent changes, unsure. Ping @Josharias :-)

[x] Picking blueberries gets no blueberries :-(
[ ] Trees look a little ... odd (1)
[ ] Some issue gets logged every time you open the inventory / assembly table / click 'i' again with it already up. Seems benign though (2)
[x] With the inventory (and all the text) visible some sort of entity creation loop kicks in. I was stable at 200 without, then opening it made hundreds of entities generate and then get garbage collected a la memory stats (which were also impacted). Closing the inventory makes the entity count go back to normal (after a moment)
[x] One time a severe performance issue triggered while auto-saving. Unsure if it relates to the entity count (both screenshots I got only show 200-300 entities, but the lag was causing UI updates to be rare). During saving the game would slow to a crawl, with the save process taking longer than normal (30 seconds?). After completion game went back to normal. Had the inventory screen up. (3)

(1)

(2)
17:27:36.871 [main] ERROR o.t.w.b.internal.BlockManagerImpl - Family not available: ManualLabor:Sifter
17:27:36.871 [main] ERROR o.t.w.p.i.InventoryOutputComponent - EntityRef{id=0} DisplayNameComponent not found

(3)

I hope to resolve some of these soon.  Day job has us on crunch mode.
The trees are indeed odd.  I already had one usuccessful hunt for the cause of the always-inside-out tree trunks. :(

Ouch, sorry to hear! But hang on there, RL as usual comes first. Thanks for the updates :-)

Picking blueberries gets no blueberries :-(
Fixed in https://github.com/Terasology/SimpleFarming/commit/a0642a8a9bd15d26ac3063864ff5a0ada07cf83d

One time a severe performance issue triggered while auto-saving

Possibly fixed by using non-persistent entities in https://github.com/Terasology/Workstation/commit/a78e4c9c7342b63bfcb916957ac801c9967d9dad

With the inventory (and all the text) visible some sort of entity creation loop kicks in.

Should be an ok level of entity creation by only checking valid inputs every 200ms in https://github.com/Terasology/Machines/commit/728c915a41bb000766bd3e1316e307c0da9f412c

Thanks for the attention and tweaks, none were crazy bad and good enough for the rest to know you're aware. Going to close this as fixed enough and try to release this weekend :-)


Support for greater/less than comparison in test for array sizes

Forked from issue #3 by @gitblit.
Cannot use $gt, $gte, $lt, $lte operators on array size.
Arrays can only be asserted for exact sizing. Ideally I'd like more flexibility on testing array sizes.
$.items: { $exists: true, $type: array, $size: { $gt: 5 } }

Even I have felt the need for these. Currently the tests are process only one level deep. We will need to add the next level. We will take this up.

zzapi v1.2.0 and zzapi-vscode v2.1.0 address this issue.


CPU measurement tags are not returned

Tags are  not loading at all for “CPU” measurement.. for other measurements its loading properly.
Upon investigating , found that Chronograf posts following metadata query to InfluxDb when we select CPU metric in chronograf UI and it goes to pending state as there is no response back from InfluxDB for longer duration and then times out
SHOW TAG VALUES FROM "autogen"."cpu" WITH KEY IN ("GlobalMon.Site", "__auth_key", "_resourcegroup_", "_subscription_", "_team_", "application", "cluster_owner", "cpu", "cynosure.isCynosure", "dc", "dspa.xdm application", "dspa.xdm environment", "env", "host", "hostname", "logfile", "month", "namespace", "weekday", "year")"
This metadata query works fine when posted without Tag key "CPU".
SHOW TAG VALUES FROM "autogen"."cpu" WITH KEY IN ("GlobalMon.Site", "__auth_key", "_resourcegroup_", "_subscription_", "_team_", "application", "cluster_owner", "cynosure.isCynosure", "dc", "dspa.xdm application", "dspa.xdm environment", "env", "host", "hostname", "logfile", "month", "namespace", "weekday", "year")"
TAG value cardinality for key "CPU "-   in CPU measurement is 418
Tried posting following query manually in Influx data node command line  and it takes longer duration and never loads.
SHOW TAG VALUES ON "vc" FROM "autogen"."cpu" WITH KEY IN ("cpu")
Any pointers how to troubleshoot this ?

Are you using 1.8 or 2.x OSS or Cloud2 InfluxDb?


Android issue for multiple modals

my current version: v2.4.0
Attached the screenshots, on testing with the example.
If we close the modal by pressing android back button instead of the close button, warning message will be shown when we open another modal.
After quick checking on source code, I made the following workaround, but I am not sure if it will break other behaviour.
  _close = async () => {
    if (this.state.isVisible) {
      this.backdropRef.transitionTo({ opacity: 0 }, this.props.backdropTransitionOutTiming);
      this.contentRef[this.props.animationOut](this.props.animationOutTiming).then(() => {
        this.setState({ isVisible: false });
        this.props.onModalHide();
      });
    }
  };

Hey @ccycecilyip! Thank you for opening the issue and inspecting for the error.
I'm on vacation right now, so I can't investigate much but... why do you think the issue is arising only on back button press? 🤔

hi @mmazzarolo
i logged the function call, and the _close function was called twice
i am thinking about if onRequestClose is the reason that leads to the issue, as the official doc said that it will be called when user tapped the hardware back button

I do not think this is directly related to the back button. Rather if isVisible gets toggled quickly while transitioning occurs some pretty bad things happen.

The refs gets nulled out temporarily which can cause the backdropRef and/or contentRef to be null which causes the promise rejection seen above. Semi-related is this issue https://github.com/facebook/react/issues/4533 but the real issue is just that _close is called while already closing and/or opening.

The visible prop may be set to false on the native Modal component while transitioning which causes a setState was called on an unmounted component error inside the animated View.

I've opened https://github.com/react-native-community/react-native-modal/pull/74 for this.

@ccycecilyip could you give a shot to @mmerickel solution? Thanks!


loss of active materials by stresses

Description
A new loss of active material submodel in pybamm. It is based on the LAM model (Eq. 25) in Reniers et al.
Reniers, J. M., Mulder, G., & Howey, D. A. (2019). Review and performance comparison of mechanical-chemical degradation models for lithium-ion batteries. Journal of The Electrochemical Society, 166(14), A3189.
This is easier than I thought, so I pick it up before the Christmas break.
Fixes #1245
Type of change
Please add a line in the relevant section of CHANGELOG.md to document the change (include PR #) - note reverse order of PR #s. If necessary, also add to the list of breaking changes.

[x] New feature (non-breaking change which adds functionality)
[ ] Optimization (back-end change that speeds up the code)
[x] Bug fix (non-breaking change which fixes an issue)

Key checklist:

[x] No style issues: $ flake8
[x] All tests pass: $ python run-tests.py --unit
[x] The documentation builds: $ cd docs and then $ make clean; make html

You can run all three at once, using $ python run-tests.py --quick.
Further checks:

[x] Code is commented, particularly in hard-to-understand areas
[x] Tests added that prove fix is effective or that feature works

Additional notes for actions

Add loss of active material model
Add LAM parameters in graphite_Ai2020 and Lico2_Ai2020
Add notebooks

Hi @weilongai! If you merge the develop branch this should fix the MacOs tests.

Hi @weilongai! If you merge the develop branch this should fix the MacOs tests.

Hi @weilongai! If you merge the develop branch this should fix the MacOs tests.

Thanks! I thought that was from the GitHub maintaince last December. Now this branch has been updated.

Hi @weilongai! If you merge the develop branch this should fix the MacOs tests.

Thanks! I thought that was from the GitHub maintaince last December. Now this branch has been updated.


Devise clean header case strategy

Currently, incoming headers on all requests and and outgoing headers on OPTION requests are cleaned to lower case by the CORS handler. At least, implement it so that all outgoing headers are cleaned as well.
However, it might be better to move such cleaning to the most basic handler (currently only the NodeHttpRequestResponseHandler), and devise a strategy for making sure this is always present in the lowest header; or, alternatively, let every handler that relies on headers do the cleaning itself (e.g. with a util function).
┆Issue is synchronized with this Wrike Task by Unito

Personally, I'm more of a fan of make sure it's always done at the entry and exit point. Doing it for individual headers in individual handlers leaves us prone to mistakes. It's quite easy to forget about this.

See https://developer.mozilla.org/en-US/docs/Web/API/Headers for inspiration


Add install_xcode_cli_tools variable

This pull request adds the ability to disable the installation of the Xcode CLI tools by setting the install_xcode_cli_tools variable to false.

Thanks! Rebased and committed as a1decc3c8f6f3c2cfdff0761086d1553e9c93eed.


Status of project?

Hey Brian,
I tried to use this project a while ago and maybe it was right in the midst of firmware changes, so I couldn't get anything working reliably. If I have the latest iOS app, and the newest firmware, does your library work against it? Are there any limitations?
I know I could try it and see for myself, but I'm hoping you don't mind filling me in
Thanks a bunch

It's been a while since I've used it myself, and I don't use apple devices, so I'd encourage you to try it yourself and report back if you run into issues! I'm not sure what limitations you had in mind, but I can't think of any.

has anyone successfully used this with the switchmates? I am considering buying some but I want to confirm I can control via raspberry pi first

@Thomasbomb I found it to be unreliable.  If anyone wants to buy them from me, hit me up lol

Yeah @jlippold, I'll bite. Are they brights?

@lavahot I have 10 rocker switch models: https://www.amazon.com/Switchmate-Wire-Free-Activated-Assistant-Controlled/dp/B07K1JJSGR/ref=pd_sbs_60_img_2/133-2386734-4904120?_encoding=UTF8&pd_rd_i=B07K1JJSGR&pd_rd_r=10fdcb5b-cb3f-408c-8907-5ace35589345&pd_rd_w=nwTqc&pd_rd_wg=vyghb&pf_rd_p=5cfcfe89-300f-47d2-b1ad-a4e27203a02a&pf_rd_r=8H7QYS1F1KS306E5MKD5&refRID=8H7QYS1F1KS306E5MKD5&th=1

Oh, was looking for toggle. Thanks anyway, @jlippold

@jlippold When reliability issues did you run into with the script? What type of computer were you running the script on, a Raspberry PI? Sorry to hear it didn't work out for you, but it would be helpful to share more details about the problems that you ran into. Perhaps the script can be improved.

I was running a rPi3, so maybe it was the bluetooth range? I'm not 100% sure. I had it wired it up to a web UI and alexa, and I found it took a while to toggle, which wasn't ideal but bearable. The bigger problem was that they sporadically stopped working.
One day, I'll try again since I have 10 of these things collecting dust, lol. When I do, i'll get better debug logs

Thanks. If you think bluetooth range was the main issue, I'd suggest trying an external bluetooth adapter connected to the PI with a short USB extension cable. I've seen range issues in the past as well. There might be ways to have the script boost the bluetooth power, but there still might be limits especially if you've got walls in the way.
Perhaps you could do a simple reliability test by keeping the switches in short range over the course of a day and switch them on/off periodically.

I finally got around to testing the script with my devices. I can confirm that it still works with the latest firmware updates from switchmate on both the original and the slim models.
I am running into inconsistencies with status checks, but the switch commands seem to be more reliable. I'm not sure if there's much I can do on that front. My guess is the underlying bluepy library is buggy, but it hasn't been updated in over a year. I'm open to suggestions if people have specific ideas. For now I will close this issue, but feel free to open new ones.


Add Releaser app (source code and half of unit tests)

This PR adds code to generate a Go Cobra application used for flexible module releasing for Go repos with multiple Go Modules. Currently, the script works as intended in testing with the OpenTelemetry-Go repo, and is intended primarily for maintainers of repos.
The following are completed and included in the PR:

[x] support app subcommands

[x] verify
[x] prerelease
[x] tag

[x] update README with usage
[x] unit tests have been added for all functions in common and verify

There are several TODO items to clean up the code which will be implemented shortly in subsequent PRs, including a full testing suite to help maintain the tool. In addition, several refactoring items and additional features are currently in the works, such as:

prevent script from deleting tags already in remote upstream
use Go-Git for Git functions rather than CLI (in progress)
enhance error messaging in verify to support slices of errors and update syntax
add ability to specify multiple module sets
add verification to tag script to check that the module set specified is the same as the one updated in the prerelease commit

Disclaimer: Opinionated comment
I think it might be worth considering using https://github.com/goyek/goyek for making build pipelines in Go.
As an example I am trying to make some reusable tasks/pipelines here: https://github.com/signalfx/go-pipeline and here is a  sample PR for adoption: https://github.com/signalfx/splunk-otel-go/pull/71
I think about creating a separate issue in https://github.com/open-telemetry/opentelemetry-go regarding it.
However, maybe it would be good just to use this PR to discuss if such an approach could help?


Consider renaming services in the sample to partially match Order Fulfillment processes

See
https://en.wikipedia.org/wiki/Order_fulfillment

Product Inquiry
Order Booking
Order Monitoring
Realtime Notifications
Shipment

low prio


New error handling, isClientSafe always false?

Hey,
I'm trying to upgrade from v0.9.14 to v0.11.2 and the only issues I'm having are how the new Errors are handled.
I want to display some errors in the response, so I've tried to throw them like so:
throw new \GraphQL\Error\UserError( 'My custom error message' );
But that still outputs:
"errors": [
    {
      "message": "Internal server error",
      "category": "internal",
      "locations": [
        {
          "line": 2,
          "column": 2
        }
      ],
      "path": [
        "post"
      ]
    }
  ]

So I tried to implement my own error class, as outlined here: https://github.com/webonyx/graphql-php/blob/master/docs/error-handling.md#default-error-formatting
And I still get the Internal server error output when using that new class to throw errors.
I did a bit of digging and it seems like this (https://github.com/webonyx/graphql-php/blob/master/src/Error/Error.php#L164) is causing all errors to have isClientSafe set to false. . .not sure if that's a bug or if I'm just doing something wrong still.
If I set Line 164 to true, then I do get a message instead of Internal Server Error but it's a long string containing the call stack, the message, etc. . .not just the message string that I'm throwing, and the category is still set to internal instead of the category I set in my custom error class.
Here's an example (truncated):
  "errors": [
    {
      "message": "GraphQL\\Error\\UserError: My custom error message in /Users/jasonbahl/sites/my-site/my-graphql-project/src/Data/DataSource.php:160\nStack trace:\n#0 /Users/jasonbahl/sites/my-site/my-graphql-project/src/Type/PostObject/PostObjectQuery.php(59):"
    }
]

I would expect message to just be My custom error message instead of the full trace.
Any insight into what I might be doing wrong / what else I'm missing?

It's hard to say without test case. But https://github.com/webonyx/graphql-php/blob/master/src/Error/Error.php#L164 is executed only if error does not implement ClientAware interface.
If you could build a simple reproducible test case - I could take a look. But before you do, please make sure that you don't catch and re-throw these exceptions somewhere in your code.

Ok, I've got the errors working in the Blog example of your repo, so something is definitely missing in my implementation. We can consider this closed and I'll dig into what's different in my codebase vs. the example. Thanks!

Just wanted to follow up. The issue was indeed another instance of a try/catch that I had in a separate piece of connected code.
I have tracing setup per the Apollo Tracing spec, and I had some try/catch logic around using the Fields resolver, or falling back to the defaultFieldResolver. . .removing the try/catch here fixes the issue.
Thanks so much!


Support for lowR in BitcoinJsWalletProvider

bitcoinjs-lib added the ability to always have low R values during transaction creation: https://github.com/bitcoinjs/bitcoinjs-lib/pull/1442
This means signature size is at most 71 bytes, and maxWitnessLen goes from 108 to 107
It also has other benefits, such as saving several bytes on txs for multisigs: https://github.com/bitcoinjs/bitcoinjs-lib/issues/1175#issuecomment-471114180
Would be great if there were a way to force low R values during transaction creation, or default to this for CAL.

This could be as simple as modifying the lowR value in ECPair class when returning keyPair from BitcoinJsWalletProvider: https://github.com/bitcoinjs/bitcoinjs-lib/blob/533d6c2e6d0aa4111f7948b1c12003cf6ef83137/src/ecpair.js#L19

@monokh any actions here?


Preview a serialized Graphviz graph in Visual Studio

Is Your Feature Request Related to a Problem? Please Describe.
Hi, i am currently using the QuikGraph library to create a bidirectional graph in Graphviz format using this approach. When debugging, I am able to get the serialized graph in the correct format and able to display the entire graph in another external visualizer, so i am wondering if it's also possible to use your visualizer to display the same graph in Visual Studio 2022 as well?
Describe the Solution You'd Like
Able to preview the serialized Graphviz graph using GraphvizVS visualizer.
Describe Alternatives You've Considered
Currently, I am using this online visualizer by copying out the string.

Hi, basically how this extension works is, if you have a window open with .dot file extension (and have the preview window open from the View tool menu) and hit save, the graph appears in the preview window.

Ah, ok that works. I didn't know that i have to open the .dot file and hit save once again. I'm not familiar with Visual Studio Extensions but would be a really nice enhancement if we could have visualized the dot graph string as well.

At least at the moment, i could still export the .dot file first and open it in a window, just require a few more steps to save the file onto disk and reopen it again.

Great idea! If I have time, I will add that.


This fork in behind by 40 commits and it can't be automatically merged

We need to review if these 40 commits have anything we need to merge.

@marwaneltoukhy Done
FYI, some of rule decks in google main have been updated in efabless version.
Cc @atorkmabrains


Inconsistency of test-dev result

Hi, thanks for your great work!
I tested your pre-trained model (R-101 3x) on test-dev2017 in the coco evaluation server.
when I extract only mask results and save to json files, the segmentation score is matched the results reported on the paper. However, when I save mask results with box results (generated by 511~514 lines in sotr.py) to json files,  the score (AP_s, AP_m, AP_l) is different from the paper.
This is the result from json file which is consist of only mask information
overall performance
 Average Precision  (AP) @[ IoU=0.50:0.95 | area=   all | maxDets=100 ] = 0.402
 Average Precision  (AP) @[ IoU=0.50      | area=   all | maxDets=100 ] = 0.612
 Average Precision  (AP) @[ IoU=0.75      | area=   all | maxDets=100 ] = 0.434
 Average Precision  (AP) @[ IoU=0.50:0.95 | area= small | maxDets=100 ] = 0.102
 Average Precision  (AP) @[ IoU=0.50:0.95 | area=medium | maxDets=100 ] = 0.590
 Average Precision  (AP) @[ IoU=0.50:0.95 | area= large | maxDets=100 ] = 0.731
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=  1 ] = 0.328
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets= 10 ] = 0.512
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=100 ] = 0.536
 Average Recall     (AR) @[ IoU=0.50:0.95 | area= small | maxDets=100 ] = 0.301
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=medium | maxDets=100 ] = 0.590
 Average Recall     (AR) @[ IoU=0.50:0.95 | area= large | maxDets=100 ] = 0.733

This is the result from json file which is consist of  mask information with box information
overall performance
 Average Precision  (AP) @[ IoU=0.50:0.95 | area=   all | maxDets=100 ] = 0.402
 Average Precision  (AP) @[ IoU=0.50      | area=   all | maxDets=100 ] = 0.612
 Average Precision  (AP) @[ IoU=0.75      | area=   all | maxDets=100 ] = 0.434
 Average Precision  (AP) @[ IoU=0.50:0.95 | area= small | maxDets=100 ] = 0.194
 Average Precision  (AP) @[ IoU=0.50:0.95 | area=medium | maxDets=100 ] = 0.440
 Average Precision  (AP) @[ IoU=0.50:0.95 | area= large | maxDets=100 ] = 0.552
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=  1 ] = 0.328
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets= 10 ] = 0.512
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=100 ] = 0.536
 Average Recall     (AR) @[ IoU=0.50:0.95 | area= small | maxDets=100 ] = 0.301
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=medium | maxDets=100 ] = 0.590
 Average Recall     (AR) @[ IoU=0.50:0.95 | area= large | maxDets=100 ] = 0.733

As can be seen, (AP_s, AP_m, AP_l) of first result and second result are (0.102, 0.590, 0.731) and (0.194, 0.440, 0.552), respectively.
I don't know the detailed process of the coco evaluation server, so I wonder why there is a difference in segmentation ap due to the presence or absence of box information.

I think https://github.com/cocodataset/cocoapi/blob/8c9bcc3cf640524c4c20a9c40e89cb6a2f2fa0e9/PythonAPI/pycocotools/coco.py

Thanks very much for your issue. By reading the code, we found that these two calculation methods do lead to differences in area calculation. We have added a note in readme to illustrate this problem.


⚠ Develop API Endpoint has degraded performance

In 8ce34a9, Develop API Endpoint (https://develop.roblox.com/v1/toolbox/items?category=Hat&keyword=Hat) experienced degraded performance:

HTTP code: 200
Response time: 1671 ms

Resolved: Develop API Endpoint performance has improved in 5edb333.


Create a Consistent Design and Colour Scheme

Make the app more visually appealing by finalising a consistent design across all pages in the app.

Also, improve the design of the Splash Page.


Backend timeout looks like a request timeout

The timeout defined in the backend section seems to be a timeout for the entire request. If I configure it lower than the time required to get the full response, but fast enough to get the template from the backend I get errors like:
Timeout exceeded, failed to respond in 100ms. Failed Fragment: http://XXX/account/
The fragment timeout however is set much higher in the template:
<div cx-url="{{server:status}}/account/" cx-timeout="4s">
</div>

Well spotted, it is the overall response timeout, makes much more sense if this is done as an aggregation of the fragment timeouts.
I notice in the PR that you are updating the timeout to whatever it is in the fragment.  Would it make more sense to only increase the timeout if the current timeout is lower than the fragment?  e.g. have the backend timeout remain the default overall response time, but bump it up if you find a fragment that is willing to wait longer.

Sounds good, I'll change the PRs

I updated both PRs

Looking at this some more, I wonder why the timeout in parxer state exists to begin with. This is the handling of a timeout that must occur after the initial backend template has been fetched (as it's part of parsing that template). These timeouts are already dealt with by the request in reliableget.
With the changes I get less timeouts, but if I force them (by adding sleep calls to my services), I sometimes get the error in the fragment part (which I expect), but also sometimes the error from parxer.
I think the callbacks checking reliableget's request timeout and parxer state timeout both get called, sometime resulting in a timeout in the request (where it should IMO) and sometimes in parxer, which results in an empty page instead of just the missing fragment (either with or without an error depending on the quietFailure setting in the backend section).
What is the purpose of the timeout in parxer, is it safe to just delete it or would it be better to allow enabling / disabling when fetching fragments (or something else).

From memory it was a safety net, a way to ensure it never got stuck or took too long (which under load would stack requests up and take the whole thing down).
It may be better to have the backend timeout be for the request to the backend, and perhaps add an optional parserTimeout that could be added if you wanted the larger timeout to be enabled?

Thanks, merged and published today.


GitHub Actions' native run

Hi peeps.!
Many thanks to you all for making and maintaining Deployer. ♥🙂
Just a quick question here. Is it possible to just use GitHub Actions' native run to run Deployer as opposed to using this action?
I'm trying to understand the purpose of this action vs the above, as assuming I have dealt with SSH authentication in my workflow, does this action offer any benefits?
Ta

Yes, it is totally fine to just use run:.  ;)


New PR for README changes, discarding old PR

Closing https://github.com/opentensor/btwallet/pull/26 and using this new PR.

See https://github.com/opentensor/btwallet/pull/31


Add Brew distance algorithm with tests

I read the issue thread. I'm not sure to rename the algorithm name since at this point creating another name for it would add more confusion. Besides, although the Brew edit distance and Wagner-Fischer algorithm are both used to compute the edit distance between two strings, there are some differences between them:

Cost of operations: In the Wagner-Fischer algorithm, the cost of insertion and deletion operations is the same, while in the Brew edit distance, you can have separate costs for deletion (DEL) and insertion (INS) operations. This allows you to differentiate between the two operations and assign different weights to them if needed.
Substitution operation: The Brew edit distance algorithm includes a separate substitution (SUBST) operation, making it distinct from the match (MATCH) operation. This allows you to assign a different cost to substitutions compared to matches. In the Wagner-Fischer algorithm, a substitution is treated as a combination of deletion and insertion.
Edit operation description: The Brew edit distance algorithm provides the description of the edits needed to transform the first string into the second one. This feature allows you to understand the specific edit operations that were performed to achieve the final edit distance. The Wagner-Fischer algorithm does not provide this information by default, although it can be modified to track the edit operations as well.

In summary, the Brew edit distance offers more flexibility in terms of operation costs and provides a description of the edit operations, while the Wagner-Fischer algorithm is a more straightforward approach to calculating the edit distance between two strings.
I could add this information in the class comment.

This seems good to me.
Are you ready to get this merged hernan?

Yep, you can merge it as requested changes were pushed.


Add snapshot remove

@alena1108 updated vendor.conf with latest rke and types. Ready for review and merge.


Persian providers

I've added persian local providers, Including Person and Internet.

The text generating is the same as ar_JO just the regular expression has changed to match persian characters.

OK for the text, sorry I thought the first method contained the text.
Also OK for the lastName, let's hope that someone can add more names in the future.

Added more Ascii first and last names will add more persian last names in a second to.

Ok, I think all issues mentioned above are resolved now.

Is there anything else I should do to get this merged?

Thanks!


searching =:= results to a failure

steps

http://scalex.org/?q=%3D%3A%3D

problem
[1.1] failure: string matching regex `\z' expected but `=' found

=

^

expectation
it finds =:=.

fixed http://scalex.org/?q=%3D%3A%3D

Cool. Thanks!


[18.03] buildbot: fix CVE-2019-7313

Motivation for this change
https://github.com/buildbot/buildbot/wiki/CRLF-injection-in-Buildbot-login-and-logout-redirect-code
Fix for master comes as bump to 1.8.1 #52536
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg test buildbot

buildbot failed on Hydra after new evaluation: https://hydra.nixos.org/eval/1503570
I can't reproduce it locally and it was fine on OfBorg before I merged.


Enhanced dark mode toggler button

Resolved the issue #198

light mode button not matching with the image shown

I understand but when I'm inserting clouds some technical issues are happening.
I believe this view of toggler button is cute too.


add tutorial document

Type of Change
documentation change
Description
This PR is used to add tutorial document for NeuralChat

This is an initial version. some contents need to add.
@XinyuYe-Intel  The OptimizationConfig definition and usage are not clean enough. pls follow this doc to revise API

@ftian1  you can create the PR against master.

Shall we create jupyter notebook (using docker file to simplify the setup)?

Shall we create jupyter notebook (using docker file to simplify the setup)?

the problem is the code is quite simple... and has been pasted in the tutorial doc. creating a new jupyter notebook looks duplicated.

Shall we create jupyter notebook (using docker file to simplify the setup)?

the problem is the code is quite simple... and has been pasted in the tutorial doc. creating a new jupyter notebook looks duplicated.

Env setup is not that simple. we'll anyway need notebook esp. Gaudi2/client. so we can create notebooks in parallel. My understanding is for tutorials, we can explain more details from API level, but not necessarily deployment level.

This is an initial version. some contents need to add.
@XinyuYe-Intel The OptimizationConfig definition and usage are not clean enough. pls follow this doc to revise API

Current implementation is for the cases like combining AMP and weight only quantization, use the way in the doc won't be able to realize such cases.

This is an initial version. some contents need to add.
@XinyuYe-Intel The OptimizationConfig definition and usage are not clean enough. pls follow this doc to revise API

Current implementation is for the cases like combining AMP and weight only quantization, use the way in the doc won't be able to realize such cases.

no, AMPConfig is purely for bfloat16 or float16 support. if user wants to do weight only quant and specify some ops to be float16, the WeightOnlyConfig should contain such parameter. The idea is each config means a quantization scheme on the model. it's self-contained and self-described. not need to combine with other config.


Use walletPublicId/username instead of walletName

In the GrahpQL API, the new field is exposed as WalletId
Remove the WalletName field and restore the Username field where appropriate
Add 2 new queries, usernameAvailable and userWalletId, the latter
to be used to get the public wallet ID from a username

$ make check-code currently failing... it looks like we're still trying to call walletName on the Wallet object in a few places

I'm collecting a few related comments here about figuring out how we treat with the user -> account -> wallet relationships. These comments all seem to come back to the question:

Can a user have context about a wallet or should it always refer to an account which would have the wallet context?

https://github.com/GaloyMoney/galoy/pull/602#discussion_r718506512

https://github.com/GaloyMoney/galoy/pull/602#discussion_r718770649

https://github.com/GaloyMoney/galoy/pull/602#discussion_r718840593

https://github.com/GaloyMoney/galoy/pull/602#discussion_r718507362

https://github.com/GaloyMoney/galoy/pull/602#discussion_r718741085

https://github.com/GaloyMoney/galoy/pull/602#discussion_r718745591

https://github.com/GaloyMoney/galoy/pull/602#discussion_r718798163

https://github.com/GaloyMoney/galoy/pull/602#discussion_r718802101

I agree not having a direct relationship from user <-> account.
We are aiming for
user 1-* account 1-* wallet and should refactor accordingly - even if it is only logical for the time being. IE in code but not reflected (yet) in the db schema.


gulp-watch bug?

Hi,
I have a setup where I use gulp-watch like so:
        watch(glob)
            ... \\ A
            .pipe(typescript({ module: 'commonjs'}))
            ... \\ B
            .pipe(gulp.dest(dir));

Everything works fine in (A) but nothing gets through the pipe with typescript to (B).
Gulp.watch is not enough for my case and I'm stumped as to why this should not work with gulp-watch.
I have other identical targets using different gulp compilers which work fine. Can you confirm or reject the issue?
Cheers =)

That's probably because of the way gulp-watch is designed: gulp-watch is an endless stream. gulp-typescript needs to have all the files before it can start compilation, since files contain types that other files are using, thus it waits for the end event. gulp-watch doesn't emit that event, since a stream can only emit that once, and it needs to push new files to the stream if something changes. The only solution would be to not use gulp-watch here.
Why are you using gulp-watch here? gulp-typescript needs to have all files, not just the ones that are changed, because unchanged files contain type info that TypeScript needs to know, so you probably don't want to pass only the changed files to gulp-typescript.

Fair enough thanks for the explanation! I guess that makes sense.
I am moving from a setup where I compiled coffeescript files,
and after the initial compile step, I used to just compile the changed files.
I was using gulp-watch to add a filter that caught delted source files and made sure the corresponding output files where deleted too.
Thank you. <(")


Move file to folder does not seem to work as expected

Describe the bug
I try to organize my project using folders. I can create a file directly in the right folder by providing the relative path with the file name. But if I want to move the file to another folder, it does not seem to work with the UI. The file is un-referenced from the fsproj, but not referenced back in the correct location.
Here's the log from Forge:
~/.vscode/extensions/ionide.ionide-fsharp-4.9.0/bin_forge/Forge.dll move file -n ~/src/Releasy/home/jerome/rehia/design-all-the-things/releasy/src/Releasy/FeatureManagement/ACL/Test.fs -d --no-prompt
Project not found

It seems that the target folder in its absolute form is treated as a relative path, and concatenated to the root directory.
The only workarounds are to change manually the file path in fsproj file, or to recreate it in the right folder by providing it with the name.
To Reproduce
Steps to reproduce the behaviour:

Add a file to project
Move file to folder
Choose a target folder

Expected behaviour
The file is moved to the right folder
Screenshots

Environment (please complete the following information):

OS: Fedora 31
Ionide version: 4.9
VSCode version: 1.44
dotnet SDK version: 3.1.201
mono / .Net Framework version:

Additional context
none

Issue transferred to Forge repo


tslint

finnnnally new circleci stuff works. split out to work a bit faster and include both lint and tslint

Ah, this merges into https://github.com/textileio/textile-mobile/pull/464 :)
needed one before the other


mediago bid adapter doc

🏷 Type of documentation

[ ] new bid adapter
[x] update bid adapter
[ ] new feature
[ ] text edit only (wording, typos)
[ ] bugfix (code examples)
[ ] new examples

📋 Checklist

[ ] Related pull requests in prebid.js or server are linked
[ ] For new adapters check submitting your adapter docs

hello，when will our doc update？（https://docs.prebid.org/dev-docs/bidders/mediago.html）

hello，when will our doc update？（https://docs.prebid.org/dev-docs/bidders/mediago.html）

Our bid adapter has already merged.


Initial plugin setup

Overview
This is the first working version of a standalone Phone Home plugin.

Extracts the Phone Home feature from our Viget Base Module
Uses Notion instead of Airtable

Why are we doing this?
Previously this functionality was tied to our Viget Base module. But a few things prompted the idea of breaking this into it's own plugin.

Airtable's API was changing, and we use Notion more these days.
Having a third-party API tied to our base module felt a little brittle.

Forcing us to make updates when we didn't want to.

A smaller simpler plugin would be easier to share with the broader Craft community.

Nice work on this, @joshuapease!


Document what customizations are supportable

Is your feature request related to a problem? Please describe.
We have several places where admins can change the behavior of the system. We should document which ones are safe and to what extent they can be changed.

Are we talking about these?

load balancers (traefik, nginx etc)
S3 backends (minio, aws, etc)
registries (built-it, dockerhub etc)
builder images during staging
app charts
catalog services
other?

Yes. I think what likely needs to be written is what features of each boundry that we need (that might not be generic).
For example, could likely say that "We use the networking v1 Ingress and work with any ingress that consumes it" and be done with ingress but might need a little more language around app charts and catalog services. For those, I'd imagine it doesn't need to be longwinded, just enough to give an impression of what can be changed without breaking (e.g. "needs to have an ingress and deployment for the api to pull the right data back.").


editor option to turn off automatic reformatting of statements to individual lines

Please briefly describe what you were doing that led to the issue, if applicable:
When i try to enter multiple steps on one line separated by semi-colon, e.g.
    dataTidy <- copy(data); str(dataTidy); colnames(dataTidy); unique(dataTidy$eventType)
the editor automatically moves each statement to an individual line whenever you enter a semi-colon.  At that point you have to manually make use of backspace/del to get everything back on one line again in cases where that is the formatting you want.  In the c#/core ide experience it doesn't do this, e.g. i can enter the following without it automatically breaking everything out onto separate lines.
    var i = 1; var j = 2; var k = 3;
    if (i == 2) { i++; j++; k++; }
It would be nice to have a way to disable that automatic breaking up of semi-colon delimited statements onto individual lines w/o having to turn off all automatic formatting behaviors in the r tools | editor options settings.
The reason i often put a bunch of r statements on a single line is for exploratory data analysis reasons that i later want to comment out of script using a single # prefix.  I find it useful when i want to put a set of related statements or simple if or function stack frame on a single line versus stretching out length of script by having each statement get its own line.
If you have any screenshots demonstrating the issue, please include them as well to help us diagnose it better.
Additional information:
OS Information
Version:       Microsoft Windows NT 10.0.15063.0
RTVS Information:
Assembly: Microsoft.VisualStudio.R.Package, Version=1.1.30523.1233, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a
Active R URI:
R 3.3.2: C:\Program Files\R\R-3.3.2

Gnenerally LintR doesn't recommend this but it can be done. By the way, you can comment multiple statements in a single keystroke. Select them and ^K^C comments and ^K^U uncomments.

k, if this is considered bad practice then i can live with manually adjusting when i care for this this layout of essentially exploratory only script comments.   Thanks for heads up that keyboard shortcuts ctrl+k+c and ctrl+k+u work to comment and uncomment selected lines in this language environment, that's an option i can move to as well.

. . . circling back on this as i'm now further along in the aka.ms/dsJhu data science specialization and am running into wanting to stuff multiple statements on a single line, using ';' statement separator,  with almost every assignment.  This arises from assignment requirements to keep Rmd output under a certain number of pages.  The c# style checker / fxcop i believe throws warnings in cases where you put multiple statements on same line, using ';' statement separator, but as noted earlier doesn't auto-reformat them on you.  Was watching some build 2017 talks on typescript and was noticing editor experience for it, vscode, doesn't auto-reformat multiple statements onto individual lines even though i'd expect jslint to also suggest that you keep everything on its own line.  So perhaps providing this auto-reformat multiple statements into separate lines would make sense as  settings option to align with how other language editing experiences handle this coding style related matter.

On related note i'm finding that editor will auto-reformat this multiple statement line into separate lines when you hit enter at the end of it
install.packages("caret");  library(caret)
but it does not, thankfully, auto-reformat this multiple statement line into separate lines when you hit enter at the end of it.
if (!require("caret")) { install.packages("caret"); library(caret) }
Not sure if having the first example not auto-reformat, and relying on R lint processing to highlight these matters unless rule is suppressed, so as to align with behavior in latter case and like noted earlier how c# and typescript editors handle this style matter.

Multiple statements on one line is not recommended by R style guide so that's why formatter breaks them. Similarly, LintR flags semicolons (and upcoming RTVS 1.2 linter has similar functionality). Might add an option though.
However, formatter does not drill into the complete scope as above.

Added option

Thanks, looking forward to using it to do away with the backspace and del key workarounds I've been using.  Do you expect it to be in next update pushed out for vs17 v15.2 (26430.16)?

No. Also, 15.3 is already closed. 1.2 will appear in 15.4 starting with preview channel https://www.visualstudio.com/vs/preview/ some time early August.

Thanks for clarification.  I'll look at switching over vs17 preview install in mid-august so i can start exercising this and other fixes you've made before they make their way to release update.

https://github.com/Microsoft/RTVS/commit/30b7dcc86b562c3678d27ea21130cbbb4c34a3b8


Reformat codebase with Eslint

This PR:

add the new linter rule from stylistic plugin
reformats the whole codebase to comply with the Eslint rules
adds lint check for src directory on build

This one is a huge one but merging that ASAP will probably make our lives easier :smile:


offer a way to make one-time contributions

I wanted to pay, but here is why I don't use gittip
https://github.com/gratipay/gratipay.com/issues/5#issuecomment-97075087

Don't worry, it's fine!
What are you interested in at the moment, some particular missing feature?
Feel free to write me an email!

pherhaps with Paypal?


Set the default value of SECURITY_REGISTERABLE to False

Don't allow registrations by default.

Coverage remained the same at 56.556% when pulling 2a898615a766d5c34bc27a403bb73a2cdbfea26d on sysboy:set_SECURITY_REGISTERABLE_default_to_false into 1b269954d977fbb770440e208ad43e2298773d5d on Netflix:develop.


Still get a collapse result from wav2vec2

❓ Questions and Help
Before asking:

search the issues. Already. Tested some solutions from other issues but don't work.

What is your question?
I tried some advice from #3718 and #3661 which are mainly asked to decrease the learning rate and set the optimize frequency. I deceased LR to 1e-5 and optimize freq to 32(2GPUs). The result now still looks bad and here is my result. I observed that contrastive loss just fell to decrease and perplexity went randomly as #3735 showed.

The red one is valid loss and green one is training loss.
What have you tried?
I'm now training VCTK dataset which has only 50h data and maybe it's too small for a self-training task, however, it seems to be something could be fixed by a lower batch token or smaller code num. But I still want to know, what exactly the A Proper Loss Curve looks like? I didn't see any properly functioned loss from those issues. it will be appreciated if anyone share some good results. Thanks!

I guess it just don't work with this few amount of data.


WordPress Blocking Bridgy

I have been trying to install Bridgy on my self-hosted WordPress blog. After install the suite of plugins, I go to sign in to Bridgy and keep getting the following message:

Can anyone help?

Hmm, perhaps are you clicking "Wordpress.com" on the Bridgy homepage? If so, sorry that that's misleading -- you actually want to enable services from the first row (Facebook, Twitter, etc.).
The Wordpress.com service in Bridgy is actually only for people using wordpress.com who can't install the indieweb plugins themselves.

Thank you for that. I was.

@mrkrndvs In case you haven't seen it, you should be aware of http://indiewebcamp.com/wordpress. Let us know if we can be of help: http://indiewebcamp.com/WordPress_Outreach_Club

@chrisaldrich can you please elaborate on the Indieweb issues or direct me to more information? Can you use Brid.gy without running the Indieweb plugin?

You could start with http://indiewebcamp.com/Getting_Started_on_WordPress which will walk you through most of the process. You should start with the Webmention Plugin and Semantic Linkbacks to send/receive webmention support and to help comments look better respectively. Brid.gy connects your blog to responses from sites such as Facebook, Twitter, et al.
Keep in mind that some of the other plugins you'll run across may have small bugs or quirks as they're being actively developed. Most of the plugins in the WordPress.org repository are usually more stable than some you'll run across on GitHub, but those also are dated in terms of development work, so you'll have a better idea how current they are.  You can usually find one or more of us via IRC, or in social media if you need help.


Interior Gravity Settings

Several interior locations do not match the exterior Gravity value

Details
https://discord.com/channels/916686954421157930/916692282021916702/1166106689649516544
Game Version
<IP_ADDRESS>
Language
English (EN)
New Game
Yes
Reproduction Steps
Visit any location in the patch.

InteriorGravityFix.zip

Related to https://www.starfieldpatch.dev/issues/63 #63


Iss 113

#113  Add code for api transitions data and ensure that the data was part of the django migrations and testing.

This looks good to me! Many of the changes look likee formatting using black, but the changes regarding the db transitions, models, and API I took a closer look at and all appear good. I don't believe I can approve or merge, though.


Rating component

Provide a general summary of the feature here
https://spectrum.adobe.com/page/rating/
🤔 Expected Behavior?
Displaying a star rating widget and allow users to click on the stars to give a rating
😯 Current Behavior
Currently it is not available in React Spectrum
💁 Possible Solution
No response
🔦 Context
To display a rating with a star widget and allow users to submit a rating
💻 Examples
No response
🧢 Your Company/Team
No response
🕷 Tracking Issue
No response

Note that this can use a radio group for the semantics: https://www.w3.org/WAI/ARIA/apg/patterns/radio/examples/radio-rating/
The labeling may need some tweaking so that the total number of stars is announced.


Typo in developing.adoc causes improper rendering

The closing block of the NOTE admonition contains an extra = which causes all of the subsequent content to be rendered inside of the NOTE.

Fixed by #83