:feet: Add preserveStaticIPs flag to the plan

We added new feature in https://github.com/kubev2v/forklift/pull/852 , to allow the perseverance of the VM static IPs from vSphere provider.
In order to do that, the user needs to specify he wish to preserve it by setting it within the plan:
preserveStaticIPs=true
e.g the plan will look like:
kind: Plan
metadata:
  annotations:
    populatorLabels: 'True'
  creationTimestamp: '2024-01-02T15:15:51Z'
  generation: 1
  name: testcluster
  namespace: konveyor-forklift
  resourceVersion: '7403505'
  uid: 53e13421-6590-49d9-84b0-735abf98e738
spec:
  archived: false
  description: ''
  map:
    network:
      name: testcluster-xzklj
      namespace: konveyor-forklift
    storage:
      name: testcluster-lmqbt
      namespace: konveyor-forklift
  preserveStaticIPs: true
  provider:
    destination:
      name: host
      namespace: konveyor-forklift
    source:
      name: ovirt
      namespace: konveyor-forklift
  targetNamespace: konveyor-forklift
  vms:
    - hooks: []
      id: 41f2988a-a579-40a8-ab25-4584b1f696dc
  warm: false

Implementation should be similar to https://github.com/kubev2v/forklift-console-plugin/issues/803

@sgratch hi, this is for 2.6.2 so it will take precedence over the 2.7 stuff

Notes:
a. this is vsphere only
b. when not defined it is false
@liranr23 please keep me honst here, not defined is false ?

Notes: a. this is vsphere only b. when not defined it is false
@liranr23 please keep me honst here, not defined is false ?

indeed

addressed by https://github.com/kubev2v/forklift-console-plugin/pull/1138

addressed by #1138 + #1139

@yaacov @sgratch missing backports?

@yaacov @sgratch missing backports?

should be in 2.6.2 (target is 2.6.2)

@liranr23
Am trying to add this , but it just doesnt allow me.
k apply -f /var/folders/7g/kjjlhgyn65d_zw5zb8f6c_6c0000gp/T/kubectl-edit-894711451.yaml
error: error validating "/var/folders/7g/kjjlhgyn65d_zw5zb8f6c_6c0000gp/T/kubectl-edit-894711451.yaml": error validating data: ValidationError(Plan.spec): unknown field "preserveStaticIPs" in io.konveyor.forklift.v1beta1.Plan.spec; if you choose to ignore these errors, turn validation off with --validate=false

% cat /var/folders/7g/kjjlhgyn65d_zw5zb8f6c_6c0000gp/T/kubectl-edit-894711451.yaml
# Please edit the object below. Lines beginning with a '#' will be ignored,
# and an empty file will abort the edit. If an error occurs while saving this file will be
# reopened with the relevant failures.
#
# plans.forklift.konveyor.io "copy-of-windows" was not valid:
# * <nil>: Invalid value: "The edited file failed validation": ValidationError(Plan.spec): unknown field "preserveStaticIPs" in io.konveyor.forklift.v1beta1.Plan.spec
#
apiVersion: forklift.konveyor.io/v1beta1
kind: Plan
metadata:
  annotations:
    populatorLabels: "True"
  creationTimestamp: "2024-10-18T09:02:46Z"
  generation: 2
  name: copy-of-windows
  namespace: konveyor-forklift
  resourceVersion: "617907626"
  uid: 5d79ebd5-3bd4-4cd7-ab3b-81f00361c354
spec:
  archived: false
  map:
    network:
      apiVersion: forklift.konveyor.io/v1beta1
      kind: NetworkMap
      name: copy-of-windows-7sj4h
      namespace: konveyor-forklift
    storage:
      apiVersion: forklift.konveyor.io/v1beta1
      kind: StorageMap
      name: copy-of-windows-7tq46
      namespace: konveyor-forklift
  preserveStaticIPs: true
  provider:
    destination:
      apiVersion: forklift.konveyor.io/v1beta1
      kind: Provider
      name: dev-kubevirt
      namespace: konveyor-forklift
      uid: 9869803a-4825-49cb-8bee-c0b9ba116030
    source:
      apiVersion: forklift.konveyor.io/v1beta1
      kind: Provider
      name: sbx-vc
      namespace: konveyor-forklift
      uid: 8df052e1-31e0-48bc-88c8-1d504e795889
  targetNamespace: forklift-migration
  vms:
  - id: vm-27411
    name: win-nextgen-01
  warm: false
status:
  conditions:
  - category: Required
    lastTransitionTime: "2024-10-18T09:08:36Z"
    message: The migration plan is ready.
    status: "True"
    type: Ready
  migration: {}
  observedGeneration: 2


Review comments not given for all changed files

Hi
anc95
PeterDaveHello
jeonbyeongmin
i changed 3 files and raised a PR https://github.com/vandnayadav1988/Hackathaon/pull/41
However, i am getting review comments only for 2 files and 3rd one is ignored. can you please help me out understand why this is happening? and if i can include the third file in the review?

anc95

@vandnayadav1988 was this fixed?

@vandnayadav1988 I have check https://github.com/vandnayadav1988/Hackathaon/pull/41. the review comments was sent by the cr-gpt bot. I guess the cause is that, the bot server is deployed on aws lambda. there has a runtime time limit on the function execution of lambda, it's 1min. if the time needed exceeds 1 min, the left files are not processed and not reviewed.

@anc95 : Thanks for explaining.


Use darker background color for .btn-default in #form-overview-draft

Right now the background color of .btn-default is the same as the background color of #form-overview-draft. #ccc is a little darker than the current color, #ddd.

This is what the new color looks like:


Unable to load asset / cached image

Hi Flutter community !
My App have a button to share images loaded from internet. The code is working fine on iOS.
But on Android, the first attempt never works. rootBundle throws an Exception(Unable to load asset).
The rootBundle only find the cached file after a hot restart. So release versions never work :(
==> If I re Run the app, the problem vanish until the next uninstall and reinstall of the app.
I'm using the flutter-cache-manager and esys-flutter-share packages.
And I tried Flutter Clean. Flutter Upgrade. on dev/beta/master channels.
shareMedia(int index) async {

    var file = await DefaultCacheManager().getSingleFile(imageLink);

    final ByteData bytes = await rootBundle.load(file.path);
    await Share.file('Hi !!', 'image.png', bytes.buffer.asUint8List(), 'image/png');
  }

$ flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel dev, v1.10.14, on Mac OS X 10.14.6 18G95, locale fr-FR)
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 11.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 3.5)
[✓] VS Code (version 1.38.1)
[✓] Connected device (3 available)
• No issues found!

Any ideas ?
Thanks a lot ! 🙏

I had same problem, change channel to stable will solved.

Hi @Chatenaisien
Please check this comment for solution
if you're still experiencing this problem, please create a new issue and provide the information required in the template.
Thank you


Implement i18 for internationalization

Hello,
Thank you so much for your work on this project. Hopefully the original maintainer merges all your PR soon.
In the meantime, I was wondering if there was a way to merge this PR into your fork: https://gitlab.com/commento/commento/-/merge_requests/153
I'll try to submit a PR here.

Thanks, me too! I think i18n is a great idea, and I've copied your PR from gitlab over here and refactored it to work with the latest commit :)

Okay I'm not sure what I did with git but it's merged now as of 23277f40e6d9806f03d3cbff51404331d0f8da9b !

Wonderful!
Thank you so much!
Sent from my mobile phone.
Le 26 déc. 2020 à 20:56, Souradip Mookerjee<EMAIL_ADDRESS>a écrit :

Okay I'm not sure what I did with git but it's merged now as of 23277f4https://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fsouramoo%2Fcommentoplusplus%2Fcommit%2F23277f40e6d9806f03d3cbff51404331d0f8da9b&data=04|01||cbcadf8034964e0dab8908d8aa023119|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637446273672937857|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=2v%2Fmv9z3K2X4B0DK1RbFVhzRa5U%2BeHvxnI0UfbjUbhE%3D&reserved=0 !
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fsouramoo%2Fcommentoplusplus%2Fissues%2F3%23issuecomment-751412452&data=04|01||cbcadf8034964e0dab8908d8aa023119|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637446273672937857|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=lLmYAZ4qdHp3xIUb%2FumShDLDlAWRxNcdgZYZaQeWiCg%3D&reserved=0, or unsubscribehttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAJCR5E3I645ROHSEGAYSAH3SW2A2LANCNFSM4TH3GPCQ&data=04|01||cbcadf8034964e0dab8908d8aa023119|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637446273672947856|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=RYSikWEZGDRyRIrAVtpYxxd4Zb4rjvk2zSRzwAtgy4I%3D&reserved=0.

Wonderful!
Thank you so much!
Sent from my mobile phone.
Le 26 déc. 2020 à 20:56, Souradip Mookerjee<EMAIL_ADDRESS>a écrit :

Okay I'm not sure what I did with git but it's merged now as of 23277f4https://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fsouramoo%2Fcommentoplusplus%2Fcommit%2F23277f40e6d9806f03d3cbff51404331d0f8da9b&data=04|01||cbcadf8034964e0dab8908d8aa023119|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637446273672937857|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=2v%2Fmv9z3K2X4B0DK1RbFVhzRa5U%2BeHvxnI0UfbjUbhE%3D&reserved=0 !
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fsouramoo%2Fcommentoplusplus%2Fissues%2F3%23issuecomment-751412452&data=04|01||cbcadf8034964e0dab8908d8aa023119|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637446273672937857|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=lLmYAZ4qdHp3xIUb%2FumShDLDlAWRxNcdgZYZaQeWiCg%3D&reserved=0, or unsubscribehttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAJCR5E3I645ROHSEGAYSAH3SW2A2LANCNFSM4TH3GPCQ&data=04|01||cbcadf8034964e0dab8908d8aa023119|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637446273672947856|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=RYSikWEZGDRyRIrAVtpYxxd4Zb4rjvk2zSRzwAtgy4I%3D&reserved=0.


Extension issue

Issue Type: Bug
Extension Name: python
Extension Version: 2020.12.424452561
OS Version: Windows_NT x64 10.0.14393
VSCode version: 1.52.0

:warning: We have written the needed data into your clipboard. Please paste! :warning:
{
	"messages": [],
	"activationTimes": {
		"codeLoadingTime": 275,
		"activateCallTime": 17,
		"activateResolvedTime": 11305,
		"activationReason": {
			"startup": false,
			"extensionId": {
				"value": "ms-python.vscode-pylance",
				"_lower": "ms-python.vscode-pylance"
			},
			"activationEvent": "onLanguage:python"
		}
	},
	"runtimeErrors": [
		{
			"name": "Error",
			"message": "request cancelled"
		},
		{
			"name": "Error",
			"message": "request cancelled"
		},
		{
			"name": "Error",
			"message": "request cancelled"
		},
		{
			"name": "Error",
			"message": "request cancelled"
		},
		{
			"name": "Error",
			"message": "request cancelled"
		},
		{
			"name": "Error",
			"message": "request cancelled"
		},
		{
			"name": "Error",
			"message": "request cancelled"
		},
		{
			"name": "Error",
			"message": "request cancelled"
		},
		{
			"name": "Error",
			"message": "request cancelled"
		},
		{
			"name": "Error",
			"message": "request cancelled"
		},
		{
			"name": "Error",
			"message": "request cancelled"
		},
		{
			"name": "Error",
			"message": "request cancelled"
		},
		{
			"name": "Error",
			"message": "request cancelled"
		}
	]
}

Thanks for the bug report, but unfortunately we need some more information from you in order to triage this issue appropriately. Please edit your opening comment by providing a filled-in copy of our issue template. Once you are done please leave a comment to that effect.


ARMS checklist

For the ARMS samples we will use the https://www.ebi.ac.uk/ena/browser/view/ERC000025 checklist but we will want to add our own things to do
This issue is for us to decide what should be added and where we can add values as URLs rather than strings

Related issues
https://github.com/emo-bon/emobon-ontology/issues/9
https://github.com/emo-bon/observatory-profile/issues/20


[web] what are differences between flutter run and simple http server of web build

I'm currently developing a flutter UI for a touch panel, and the app works great if I use "flutter run -d chrome".  If I run it though from a simple http server I get what look like minification lookup failures?  Here is how I'm starting a simple server:
python3 -m http.server 8000
from the 'build/web' path.  Here is how I'm doing the web build:
flutter build web
when I run it this way, I get a failure like this:

So my first question is, why would running this with "flutter run -d chrome" work better?  What's different about the build that way?
My next question, is there some better way to get failure logs from a web UI?  From the console I see this:

Here is my flutter info:
JustinMac:audacy_on_tap jeltoft$ flutter doctor -v
[✓] Flutter (Channel unknown, v1.10.15-pre.352, on Mac OS X 10.14.6 18G103, locale en-US)
    • Flutter version 1.10.15-pre.352 at /Users/jeltoft/flutter/flutter
    • Framework revision 01d55074af (4 days ago), 2019-10-31 10:10:25 -0500
    • Engine revision b16eab6aff
    • Dart version 2.6.0 (build 2.6.0-dev.8.2 d50c158d42)

[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at /Users/jeltoft/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 10.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 10.2.1, Build version 10E1001
    • CocoaPods version 1.7.5

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 3.5)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 40.2.2
    • Dart plugin version 191.8593
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)

[✓] Connected device (2 available)
    • Chrome     • chrome     • web-javascript • Google Chrome 78.0.3904.70
    • Web Server • web-server • web-javascript • Flutter Tools

• No issues found!

By default flutter run builds in debug mode, while flutter build runs in release mode. It looks like your app probably has a bug in it - if you run in release mode flutter run --release/--profile you should be able to see what is crashing with source maps.

Ok, thanks for the info.  Sure enough, if I run in --release or --profile modes I get the crash.
So with "flutter run --profile -d chrome" I get this:

let me see if anyone else has seen this.   One thing I would mention is that this same code base worked up to the point of 1.10.2.  I think 1.10.3 was the first time it started failing.

Is there a console stacktrace with more information?

Ah, found it.  Thanks for the help.  So my issue is with using plugins (again).  What confused me in this case though.  Why does it run in debug mode but not release mode?
https://pub.dev/packages/percent_indicator
This was the current cause of my issues.  I've already implemented this in a static non animated stack way of containers to show the progress bar.  But I'm also thinking there is one more plugin later in my app killing me (carousel) which will be harder to work around.
Any idea why these run in debug mode?

I'm not familiar with the plugin so I can't say. Is there more of a stack trace in the console that we could look at?

@jonahwilliams Thanks for asking.  So the plot thickens here a bit.  I'm sorry to have suspected that plugin a bit blindly.  What it looks like is this.  When I was using that plugin, I was using some Inherited Widget "getters" from my main.dart.  By removing my calls in a few places to the percent indicator plugin (I was using it as a progress bar in my setup screens) I ALSO removed a few calls to using inherited data.
So, my true issue is this I think:

I can run in --debug mode for some odd reason
when I run in --profile or --release, I get this error you see above
my app is now crashing at a screen further down the line but that's how I learned it's because of use of inherited widget data through a "getter"

Here is the failure console for another example of this same overall failure:

Here is are some code snippets that show how I'm using inherited widget data:

and then to get this failure (again, ONLY in release/profile mode) the code does this:

So my first question is why does this work in --debug mode?  And then I'm wondering, am I using inherited widget incorrectly somehow?  SessionInfo() in the above case is being placed at the root of my main app like this:

I started using this paradigm for my data when I watched this video many months ago and it's been working very well for me in apps.
https://www.youtube.com/watch?v=Zbm3hjPjQMk
@iapicca thanks for marking this
@filiph thanks for the youtube!  Love your videos!!!

One other clue to mention again here.  From my colleague's testing, this did not fail until flutter 1.10.3.  with 1.10.2 down to 1.9.x this same code base worked I believe.

I'll try to make a very simple example today some time that demonstrates this failure with just one or two dart files.  If it's just any time you use inherited then this shouldn't be hard

Ok, I've narrowed down to where the issue is a bit more.  The issue appears to be when I use inherited widgets in a util.dart file that's in a sub directory such as util/util.dart.  If I move the exact same util function into main.dart, then it works.  It also works if I make a util.dart at the main level instead of one level down.
Attached are the simplest files I could make to recreate this failure.

lib/pagetwo.dart
lib/main.dart
lib/launchpad.dart
lib/util/util.dart
pubspec.yaml

This is in a zip file and you should be able to reproduce the failure then by putting this in an example flutter app, doing the "flutter create ." to build web. And then use:
flutter run --profile -d chrome
This failure is caused by the use of this function in util/util.dart:

and if I move that util function completely as is into main.dart then I do not get the crash anymore.
This is the console log of the crash you should see:

So is this failure somehow a minification issue with path names and dart files being in sub directories??
simple_failure_reproduction.tar.gz

@jonahwilliams @iapicca Just FYI, I updated this with a very simple example for you to reproduce the failure on your setup.

@jonahwilliams Here is a screen shot of my full (non stripped down as above) app failing.  Again, it failing at the time of using inherited data from main.dart.  What's crazy still, it worked in 1.10.2, it works in debug mode and if I run on any AVD or real hardware, but it just fails here in flutter web release mode.

The other crazy thing is that I seem to have no issue accessing the same inherited data (SessionInfo.of(context)) from the parent "screen" (stateful widget) above the one that crashes.  So I found I can pass it down to this screen as a parameter!

So back to the original question, what's the difference between:
flutter run --profile -d chrome
and
flutter run --debug -d chrome
The first one fails, the second one doesn't.  And I looked at main.dart.js and it's 100% identical in both cases.
Attached here is the main.dart.js file that calls to
C.JSNull_methods.get$data(H.interceptedTypeCast(context.inheritFromWidgetOfExactType$1(C.Type__TapInfo_G37), "$is_TapInfo0"));
main.dart.js.zip

/cc @vsmenon what's strange about the generated code is that is attempts to call a method on something it knows is always null?

Ok, narrowing in, reviewing the generated js code. The first commit id (plus or minus 3) I see that failed is e045ef29691ea9f2ef3eafe3176f8d1da8a3a5fa.
This one passed: 7fb085ee04f87a527634ed2da8c7d2909eb2d209

@jdeltoft - with --debug it's using a very different compilation path.  Note, the actual file might have a different name.  You can look at what's loaded in the browser.
@sigmundch - any ideas?

@jonahwilliams Just FYI, this was the first commit that fails for me.
b2f38396f41cac707284c46d1ade7ae650fb3cad
Looks like it was reverted once and relanded here?  Any ideas if this was a major shift that could cause my problems? I haven't looked into the changes yet.

mmm - no ideas on my end, yet. As @yjbanov mentions, it appears that dart2js believes some values to always be null, so it generates those calls like JSNull_methods.*.
@jdeltoft one clue that might be useful to look into is how does the main.dart.js contents differ before and after b2f3839 - maybe that would clue us into whether dart2js is getting the same input or if some flags are different somehow.

I have the same exact problem here. When running in debug mode everything works fine. When I run it hosting the page or anything you already did it says there is no function called:
get$data

@vsmenon @sigmundch I'm labeling this as a Dart issue if you don't mind. As of right now I can see two possibilities:

There's a bug in the compiler: in this case the fix is also in the compiler
The bug is in the Dart code: in this case the stack trace should be better (e.g. if it's NSM due to null the message should say exactly what's null), i.e. the fix is still in the compiler

@vsmenon @sigmundch I'm labeling this as a Dart issue if you don't mind. As of right now I can see two possibilities:

There's a bug in the compiler: in this case the fix is also in the compiler
The bug is in the Dart code: in this case the stack trace should be better (e.g. if it's NSM due to null the message should say exactly what's null), i.e. the fix is still in the compiler

Thanks for the update.  Just as a reminder I point to the exact dart2js update from a flutter commit above and also have a simple flutter program with just two files that shows the failure.

This is probably related to 04 optimizations, turning them down might reveal a better error message

@jdeltoft - would you mind sharing the Dart code for buildLightRow in your screenshot example?
It does appear that the compiler believes you're indexing into a list (or similar collection) that is always null.  Note, one difference between debug and non-debug modes is that the former runs asserts and the latter doesn't.  So, you would see that exact error - release mode only - if you had something like:
void main() {
  List l;
  assert(() {
    l = [ 'hello' ];
  }());
  print(l[0]);
}

@vsmenon is there a way to disable assertions in DDC? Our tools should support debugging such errors in user code. Perhaps we should offer this mode in debug builds.

@vsmenon is there a way to disable assertions in DDC? Our tools should support debugging such errors in user code. Perhaps we should offer this mode in debug builds.

I don't think so.  You can open a request on Dart.
That said, should also try this at default optimization level (O1) in Dart2JS.

@yjbanov could you open a PR to expose the optimization level as a command line flag? Its currently defined here: https://github.com/flutter/flutter/blob/master/packages/flutter_tools/lib/src/build_system/targets/web.dart#L27

@jdeltoft - would you mind sharing the Dart code for buildLightRow in your screenshot example?
It does appear that the compiler believes you're indexing into a list (or similar collection) that is always null. Note, one difference between debug and non-debug modes is that the former runs asserts and the latter doesn't. So, you would see that exact error - release mode only - if you had something like:
void main() {
  List l;
  assert(() {
    l = [ 'hello' ];
  }());
  print(l[0]);
}

@vsmenon  Sure!  Thanks for asking.  Here is that function.  The SessionInfo.of() is my use of inherited widgets in main.  This has been how I've been dealing with data I want to be shared across the app's pages.  Not sure if you saw the zip I attached.  It reproduces this issue and shows all the details of how I was using inherited widgets.

Just a heads up, I just upgraded to the latest flutter, and retested.  Issue is still there, but now the "red screen of death" is just a blank white screen?  I ran with "flutter run -d chrome --profile"

@vsmenon @yjbanov Hi, is there a way for me to use the latest dart2js with optimizations turned lower?  Then I could see if there was a particular level that creates the failure?
Is there any other help I can give here?  I have shared the full source (2 files) to reproduce the failure 100% above, not sure what more I can do.

I took a deeper look and it appears the issue here is the same as #44600
It appears that there are some libraries imported by two different URIs: a relative import and a package import. When a file is imported via different URIs, we are required to treat them as separate libraries. So code like this gives surprising answers:
import 'package:foo/a.dart' as a1;
import 'a.dart' as a2; // same file as above

main() {
 print(a1.A() is a2.A); // prints false;
}

The code that is producing errors is code that contains casts like as _TapInfo, which then dart2js optimizes because it knows the cast will fail.
Because of common patterns of how flutter main lives in lib/, all of our tools have logic to convert lib/main.dart into package:my_package/main.dart. I wonder if that broke during b2f3839
@jonahwilliams - I am hoping that your fix for the other issue will also address this one.

@jonahwilliams Unfortunately, this is still crashing for me.  I just upgraded to the latest master flutter and I get this with the demo files I attached on this issue above.  I'll attach them here again:

Here's that same tar file with the lib/ folder (4 dart files total).  Let me know if I can somehow change the way I do my imports to work around this?
simple_failure_reproduction.tar.gz
Here is where I am in flutter:

@jonahwilliams let me know if I should make a new issue or do you re-open this one?

@jdeltoft - when I was playing with your example earlier, I was able to work around the issue by changing the imports as follows:

rename main.dart to main2.dart
create main.dart that only has these contents:

export 'package:my_package/main2.dart';

change all other relative imports from main.dart to main2.dart.

Basically this ensures that anyone seeing the contents of main2.dart will see it imported via a package URI. Let us know if that works for you.
Obviously we don't want you to have to use the workaround and we should figure out why @jonahwilliams change wasn't enough to fix your example.

Reopening the issue for now, since I only checked what I thought was a duplicate I didn't do any further verification here. I'll make sure I can repro this and take another look

@sigmundch Ok, I tried that work around and it worked but there are a few new concerns:

after some time, my whole web page stops taking all click input, but I know it's still running as I have a 15 min timer that starts a screen saver animation and that animation worked perfectly still
I'm unable to run my app now with that work around on an AVD tablet even if I point android studio run to main2.dart.  (Not a big deal, but was going to try this to debug the input)
I'm unable to run in chrome in --debug mode.  I can only run in --profile or --release

Here is the error I get when the input times out:

The instant that happens, all input ability stops.

So I looked at all my dispose calls up to this point, it's nothing there.  I printed out when those are happening and it's not near the failure.
I also noticed that when I get this failure, if I move my mouse pointer around the screen, then this failure count increases.  A still shot doesn't do it justice, but hopefully you understand my point.  I'm not aware of anything I'm doing special to work with mouse pointer location, but something is definitely happening with that which is this error.

Ok, sorry for the noise here.  I believe this was broken and now fixed in flutter release.  I just did another flutter upgraded, and I'm using your workaround for now still @sigmundch and now my app works great for long periods of time.
Here is where my head is now:

Thanks for confirming.  I just checked your tarball.  It appears to work for me in flutter dev and master channels (release/profile/debug) as well.

BTW - I didn't have to use a workaround either.  At least, not with the tarball.

@vsmenon Agreed!  I didn't actually notice that until just now.  I saw that other new hang problem, saw it was fixed by a flutter upgrade, and didn't undo the temporary work around yet.  Now I just did and indeed it's working without any work around.  Thanks again everyone for the fantastic help!


Including task description when adding tasks with quickadd

Is it possible to include task description when adding tasks with quickadd? I wasn't able to find the todoist docs https://todoist.com/help/articles/task-quick-add.

Yes - use the //description syntax.

@stacksjb this is a good thing to add to documentation as it's not currently mentioned.

Since quickadd is the most used feature we can describe some of the todoist syntax in the docs and link to one of their pages for more info.

Done!

You're welcome to swipe my quickadd documentation for the workflow (https://github.com/stacksjb/AlfredTodWorkflow/edit/main/README.md)
Examples

"Respond to Jim's marketing email" : creates a new task in the Inbox titled "Respond to Jim's marketing email"
"Clean my room on tuesday at 1pm p2" : creates a task named "Clean my room" due on Tuesday at 1pm, with Priority of 2
"Ship UPS Package #eBay @errands p2 tomorrow" : creates a task in the eBay project, anerrands label, priority of 2, due tomorrow.
"Purchase apples //Granny Smith" : creates a task in the Inbox named "Purchase apples" with a description of "Granny Smith"

I think I will!

Done, thank you


add centimeters to feet option to height converter project

add centimeters to feet option to height converter project
This option should in another html file
Please assign this to me I am working this feature

hey i have good know ledge in html please assign this project to me. You can trust me . I will definetely complete it.

please assign me under GSSOC-23


Checkbox using objects

I'd like to know how to do to use the md-checkbox with objects.
Basically I want to do the example of the Aurelia doc, but is not working properly.
app.html
<template>
  <form>

Products

      <input type="checkbox" model.bind="product" checked.bind="selectedProducts">
      ${product.id} - ${product.name}

    Selected products:

${product.id} - ${product.name}

  </form>
</template>

app.js
export class App {
  products = [
    { id: 0, name: 'Motherboard' },
    { id: 1, name: 'CPU' },
    { id: 2, name: 'Memory' },
  ];

  selectedProducts = [];
}

I'm trying this way below, but is not working.
<form>

      <md-checkbox md-checked="false" model.bind="product.id" checked.bind="selectedProducts ">
         ${product.name}
      </md-checkbox>

</form>

${ selectedProducts }

Sorry by the bad english

While your sample is a bit different from the one in the docs (not sure if binding to id would work in this case), I think that it is not the problem. I guess it's because the md-checkbox doesn't have a model binding.
That's bad. I'll implement it.

Sorry by the bad english

Don't worry about that. It's not my native language, either. :smiley:

Probably need to forward matcher , too.

Thank you. It will be great!
I didn't understand the second reply.

You can also bind checkboxes by using a "matcher", which is basically a compare function that checks if one element matches another. See here for details: http://aurelia.io/hub.html#/doc/article/aurelia/binding/latest/binding-checkboxes/5
And I think to have complete support, I would need to add this "matcher", too. :smiley:

Hmm... but this needs the model.bind too =(

Yes. I mean to add both.

Hahah ok
Thank you! I'll wait

This change works internally but it's a bit tricky. We didn't bind checked directly and I need to look up why. I don't remember.. This will take a little more time, I guess.

Okay, I didn't find it and my tests were all positive.
So I'm going to release this.. I hope I don't break anything. 😃

@williamfelippe http://aurelia-ui-toolkits.github.io/demo-materialize/#/samples/checkbox/binding-to-objects
😃

Thanks @Thanood!


Add way to get the item with the highest sale velocity on a server

This shouldn't be too difficult, but it's unclear what server resources the query will require.

I notice this feature is disabled in the API? Is the feature impossible to implement for some reason or still just awaiting testing?

Just needs testing, and probably rewriting after finding bugs. I don’t have an actual testing environment (which is a huge problem in its own right) so testing things means setting aside a big chunk of time to work and immediately roll things back if they break prod. Dealing with either of those will take a large chunk of time that I can’t currently set aside very often because of school.


 Bugs fixed.
Add final style touch to exercise. New features version 1.1.0 :
small_red_triangle Visual effect on click for add favorite
small_red_triangle Visual effect on list to markup favorites already added
small_red_triangle clean and organice dead code and project

Muy bien @marpri-17 buena iniciativa. Te lo miro a partir del miércoles!


[MultiInstanceHandler]: ERROR: FormatException: Unterminated string (at character 25)

The file name contains Chinese;

Flutter (Channel stable, 3.3.8, on Microsoft Windows [版本 10.0.19044.2251], locale zh-CN)
    • Flutter version 3.3.8 on channel stable at D:\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 52b3dc25f6 (5 days ago), 2022-11-09 12:09:26 +0800
    • Engine revision 857bd6b74c
    • Dart version 2.18.4
    • DevTools version 2.15.0
    • Pub download mirror https://pub.flutter-io.cn
    • Flutter download mirror https://storage.flutter-io.cn

[!] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at D:\Android
    X cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    X Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/windows#android-setup for more details.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop for Windows (Visual Studio Community 2022 17.4.0)
    • Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Community
    • Visual Studio Community 2022 version 17.4.33103.184
[√] VS Code (version 1.73.1)
    • VS Code at C:\Users\Administrator\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.52.0

[√] Connected device (3 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [版本 10.0.19044.2251]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 107.0.5304.107
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 107.0.1418.35

[√] HTTP Host Availability
    • All required HTTP hosts are available

Thanks for reporting, fixed in 0.1.5


Add selected facility marker

Overview
This PR changes the marker for a facility when it is selected, primarily by detecting the selected facility from the react-router details path.
One additional change here is that routes like http://localhost:6543/facilities/CN2019084R04PD8?name=shoe no longer load both the facility for the details page and the list of facilities for the search automatically on first arriving at the app from a URL entered in the browser. The details page still loads, but I updated the code to prevent the other facilities from loading, too, in order to simplify the number of cases that had to be handled for the imperative code set up around adjusting the marker styling.
Third commit is unrelated but fixes a linter complaint which I think issues from how react-router parses memoized components in the version we're using.
Connects #347
Demo

Testing

serve this branch then search for some facilities
verify that selecting a facility changes its marker to selected and also unselects other facilities
verify that you can navigate directly to a details page and see the selected facility marker
verify that multiple facilities at the same point show the selected maker when they are selected

TODO:

[ ] make sure marker shows on route like localhost:6543/facilities/TR2019080MJHCA7?name=shoe
[ ] figure out what to do with markers with multiple facilities at the same point

Looking at this now

I think we should update static-marker.png to be the selected marker color. It will help conceptually connect the point on the main map with the point on the mini map.

Good suggestion. I updated this in bf4ea56c4ef364cb99ab865d5c9daff6df1f1269

Testing it prior to staging will require:

serving /src/app/public dir from a static server
ngrok http
updating the window.location.origin text in the FacilityDetailsStaticMap component to be "https://whatever.ngrok.io"


KeyError: 'document'

Hi,
I need to work on the UI for an application I have built.
here are the steps I've done to try and get tkdesigner to work:
installed python3.9
added python to path

pip install tkdesigner
tkdesigner $FILE_URL $FIGMA_TOKEN

here is my file url:
https://www.figma.com/file/W3ekOTW6bFvGiUrndEpC4S/Untitled?node-id=4%3A41
The callback I get is:
Traceback (most recent call last):
File "C:\Users\Cai\AppData\Local\Programs\Python\Python39\lib\runpy.py", line 197, in _run_module_as_main
return _run_code(code, main_globals, None,
File "C:\Users\Cai\AppData\Local\Programs\Python\Python39\lib\runpy.py", line 87, in run_code
exec(code, run_globals)
File "C:\Users\Cai\AppData\Local\Programs\Python\Python39\Scripts\tkdesigner.exe_main.py", line 7, in
File "C:\Users\Cai\AppData\Local\Programs\Python\Python39\lib\site-packages\tkdesigner\cli.py", line 71, in main
designer.design()
File "C:\Users\Cai\AppData\Local\Programs\Python\Python39\lib\site-packages\tkdesigner\designer.py", line 29, in design
code = self.to_code()
File "C:\Users\Cai\AppData\Local\Programs\Python\Python39\lib\site-packages\tkdesigner\designer.py", line 21, in to_code
window_data = self.file_data["document"]["children"][0]["children"][0]
KeyError: 'document'
Any ideas what I've missed?

After reading through other issues trying to find one similar I tried running tkdesigner through the GUI which has worked, guessing there is an issue with CLI or maybe I was doing something wrong.
Either way got it to work!

the same error, even failed using GUI

Did anyone find a solution?

hey @aditdesai @MyGuyCai @jiuerbujie
i see you have python installed but your running in it wrong directory mate.
open a fresh terminal or cmd and run these commands in order:
git clone https://github.com/ParthJadhav/Tkinter-Designer.git
cd tkinter-designer
pip3 install -r requirements.txt
pip3 install tkdesigner
cd tkinter-designer (optional: only if u arent still in tkinter-designer directory after the pip3 steps)
cd gui
python3 gui.py

@AbdulkadirH5253
Can confirm this works, running the UI with pip gui.py allows me to get my designs

I found a Solution:
The cause of the error for me was an expired Figma Token
This is the error I got the error:
" tkinter KeyError: 'document' "
self.file_data.['document'] was undefined  because self.file_data was equal to an error object

To Fix the error, create a new Figma Token

I found a Solution: The cause of the error for me was an expired Figma Token
...
To Fix the error, create a new Figma Token

This essentially worked for me too. There was a typo in the Figma key and once corrected it worked fine


Go jump target output is confusing/wrong

Consider: https://godbolt.org/#g:!((g:!((g:!((h:codeEditor,i:(j:1,lang:go,source:'//+Type+your+code+here,+or+load+an+example.
//+Your+function+name+should+start+with+a+capital+letter.
package+io

func+Fun()+{
	for+i+:%3D+0%3B+i+<+10%3B+i%2B%2B+{

++++}
}

'),l:'5',n:'0',o:'Go+source+%231',t:'0')),k:50,l:'4',n:'0',o:'',s:0,t:'0'),(g:!((h:compiler,i:(compiler:gl1100,filters:(b:'0',binary:'1',commentOnly:'0',demangle:'0',directives:'0',execute:'1',intel:'0',trim:'0'),lang:go,libs:!(),options:'',source:1),l:'5',n:'0',o:'x86+gc+1.10+(Editor+%231,+Compiler+%231)+Go',t:'0')),k:50,l:'4',n:'0',o:'',s:0,t:'0')),l:'2',n:'0',o:'',t:'0')),version:4
The jump targets (jmp 7, jlt 4) don't seem to refer to anywhere in the file. Also the execution seems to fall off the end of the function. Seems the asm filterer is confused.

From the go tool directly:
go tool compile -S test.go
"".Fun STEXT nosplit size=14 args=0x0 locals=0x0
	0x0000 00000 (test.go:3)	TEXT	"".Fun(SB), NOSPLIT, $0-0
	0x0000 00000 (test.go:3)	FUNCDATA	$0, gclocals·33cdeccccebe80329f1fdbee7f5874cb(SB)
	0x0000 00000 (test.go:3)	FUNCDATA	$1, gclocals·33cdeccccebe80329f1fdbee7f5874cb(SB)
	0x0000 00000 (test.go:3)	XORL	AX, AX
	0x0002 00002 (test.go:4)	JMP	7
	0x0004 00004 (test.go:4)	INCQ	AX
	0x0007 00007 (test.go:4)	CMPQ	AX, $10
	0x000b 00011 (test.go:4)	JLT	4
	0x000d 00013 (<unknown line number>)	RET
	0x0000 31 c0 eb 03 48 ff c0 48 83 f8 0a 7c f7 c3        1...H..H...|..
go.info."".Fun SDWARFINFO size=32
	0x0000 02 22 22 2e 46 75 6e 00 00 00 00 00 00 00 00 00  ."".Fun.........
	0x0010 00 00 00 00 00 00 00 00 01 9c 00 00 00 00 01 00  ................
	rel 8+8 t=1 "".Fun+0
	rel 16+8 t=1 "".Fun+14
	rel 26+4 t=29 gofile../workspace/go/src/gopractice/io/test.go+0
go.range."".Fun SDWARFRANGE size=0
gclocals·33cdeccccebe80329f1fdbee7f5874cb SRODATA dupok size=8
	0x0000 01 00 00 00 00 00 00 00                          ........

Seems we need to treat each line as a label, and handle the ret with no file and line associated with it.

Do you mean we should be creating labels when these kinds of jumps are being used? How would we name them?

I think if we treated each address as a label with the bane being just "4" or whatever then the find used labels code might just work

We might have to rewrite then as addr4 etc I guess (and the jumps too to be JLT addr4). I'm thinking out loud here

the direct jumps are now rewritten to labels by #1194 and thus completes the fix


Add implementation for __builtin_popcountl for Windows on ARM

Window on Arm missing the __popcnt64 function.
Adding static implementation based on https://github.com/ngtcp2/ngtcp2/blob/b64f1e77b5e0d880b93d31f474147fae4a1d17cc/lib/ngtcp2_ringbuf.c, line 34-43

Thank you @sheldonrobinson !

Why is the parameter of type unsigned int rather than u64?
The function is called with a[i] ^ b[i] which is of 64-bit, while unsigned int is narrower than 64-bit on most platforms.


error on login

when login button is clicked the following error is generated :
Uncaught (in promise) Error: GraphQL error: Argument 'email' expected type 'String!' but got: {email: $email, password: $password}. Reason: String value expected (line 2, column 27):   authenticateUser(email: {email: $email, password: $password}) {                           ^ GraphQL error: Cannot query field 'id' on type 'AuthenticateUserPayload'. (line 3, column 5):     id     ^ GraphQL error: Field 'authenticateUser' argument 'password' of type 'String!' is required but not provided. (line 2, column 3):   authenticateUser(email: {email: $email, password: $password}) {   ^

I am experiencing the same error.

The authenticate user mutation is coded incorrectly.  Change Login.js as follows:
// const AUTHENTICATE_USER_MUTATION = gql`
//   mutation AuthenticateUserMutation($email: String!, $password: String!) {
//     authenticateUser(email: {
//       email: $email,
//       password: $password
//     }) {
//       id
//       token
//     }
//   }
// `

const AUTHENTICATE_USER_MUTATION = gql`
  mutation AuthenticateUserMutation($email: String!, $password: String!) {
    authenticateUser(
      email: $email,
      password: $password
    ) {
      id
      token
    }
  }
`

Hi @ardithastopalli, thanks a lot for pointing this out and sorry for the late reply! I just updated the repo and tutorial which should have fixed the issue. Please let me know if you're running into any further issues 🙂


Enable deletion of solutions from Portfolio

I've completed a couple of Bonfire's successfully and realized there was an issue that floated past the test. I'd love to be able to remove these solutions from my portfolio page.

+1

Thanks for the suggestion!
This is a duplicate of #1342, keep track of that for updates on this request.


POST /cart/{customerId}/items does not save itemId

When calling the endpoint, itemId is not retained in the store

case issue!


fix: docs/framework/wcf/feature-details/data-contract-schema-reference.md

xlms split across lines
trim trialing/leaading space
spacing around headings

Thanks for correcting the formatting of the XML block, @nschonni. I'll merge your PR now.


RigidBody2D is moving KinematicBody2d (how to prevent?)

Godot version:
v3.0.6 stable
Steam Version
Issue description:
This is my first post everyone on GitHub so if I am doing it wrong then I am sorry.
I am making a tetris-like where I need a RigidBody2D(ball) to bounce off of KinematicBody2D(the tetromino). I need to use the KinematicBody2D for the move_and_collide() so that it stops when it runs onto other pieces or the bottom platform(StaticBody2D). I honestly don't understand why there is no toggle for this. I can do this the long and hard way but would prefer to use Godot's built-in physics as I am new to programming to begin with.
If there is another way to approach this I am all ears but no matter how much I search for this topic I could not find an answer.
edit
This is a similar issue but is not current to the v3.0 and the problem was never resolved.
https://godotengine.org/qa/8820/disable-effecting-kinematic-collisions-forces-colliding

Please ask such questions on other community platforms, such as the Q&A, Discord, forums, etc.
This issue tracker is for engine bugs, not user support. If you establish with other users that there is indeed a missing feature to solve your use case, then you can open a feature request (or bug report if it's a bug that an existing feature doesn't work as advertised).


Create elementary graphics in SVG

Anna is working on that but I can eventually make it too by starting from the images.
Could you please provide here the link to the example mockup?

https://www.figma.com/community/search?model_type=hub_files&q=spotify wrapped

I closed this due #19


Fix Versions page closes #410, credits to EliasVal <3

Original author: https://github.com/EliasVal
See https://github.com/game-ci/documentation/pull/412
Changes

Fix versions page

Todo

[x] Slightly improve icon size, and dark vs light theme for icons

Did you wanna cherry-pick the changes to preserve the original author?

Did you wanna cherry-pick the changes to preserve the original author?

Oups, indeed, should have done that, but I missed your notification. My bad. Credits definitely goes to https://github.com/EliasVal here.
At least commit history shows it, but you're right, I should have done a cherry-pick in the first place.
I did it this way testing locally:
git checkout 62090d3 src/components/docs/versions/
git checkout 62090d3 src/components/molecules/tooltip/

Then I did commit. I'll make sure to do it properly next time.
Again, huge thanks to @EliasVal and sorry about history and stats. Hopefully, it's still there in history before the revert.


Update README.md

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=ru
Description of changes:

"An unexpected error has occurred, could you please contact admins. Your RequestID: 425f1852-cf5d-4eef-889d-011d604b3145"


Feature Request: Add end to end example starter module for AVM ALZ modules

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Description
Is your feature request related to a problem?

A stater module that leverages the new AVM ALZ  modules and can be leverages by customers and VBD deliveries in the future. It should leverage the existing yaml based config pattern as used in the complete starter module.
Describe the solution you'd like

Additional context

Started in #30

Resolved issues and merged the PR.


Random block refs invalidated after re-index

In the latest version (0.6.2) of logseq, after doing a full re-index, the references of random blocks created by Smart Blocks become invalid.
My use case is a simple one: it picks a random block with a certain tag. (The same as this video.)
After creating the block ref, I can jump to it via the ref, so the ID is correct. But after re-index, the ID changes. I don't know if this is a bug of Logseq, or a bug of Smart Blocks (forgetting to make the ID persist by adding it to the text file), or it is a change in logseq that Smart Blocks hasn't caught up with.

I should add that the block referenced has not been referenced before. So it's quite likely that Smart Blocks forgot to make the ID persist?

Yeah, that's likely the issue. I'll fix the issue today.

Should be fixed in the latest release


Unexpected sensitivity of CG2D convergence criteria with cg2dNormaliseRHS

For the 2-D pressure solver CG2D, the default is to normalise the RHS (cg2dNormaliseRHS=T)
and to provide the error tolerance without units, as "cg2dTargetResidual"
The RHS normalisation is function of the largest (in absolute value) RHS element but, in the current solve_for_pressure.F implementation, it is not directly related to the initial residual, i.e., to the magnitude of the problem to solve, as one could have thought: for a given "cg2dTargetResidual", Larger/Smaller initial residual could have lead to larger/smaller final residual once the target residual is reached.
But this is not the case, and in particular, with linear free-surface, if I shift the SSH initial field by a significant uniform increment, which does not affect the physical problem (same SSH gradient),
the RHS increases significantly, the norm (inverse of Max(|rhs|) decreases and the final solution is reached more quickly (fewer CG2D iterations) but is also less precise (larger error).
As a more concrete example, I tried, in tutorial_baroclinic_gyre, after turning off the wind-stress and just keep the SST relaxation, to add a 100.m initial SSH:
And here are some number from the first 5 time-steps:

cg2d: rhsMax =  1.23376801105470E+02 ; cg2d_iters(min,last) =      -1       0
cg2d: rhsMax =  1.23376801105470E+02 ; cg2d_iters(min,last) =      -1       5
cg2d: rhsMax =  1.23376801105470E+02 ; cg2d_iters(min,last) =      -1       7
cg2d: rhsMax =  1.23376801105470E+02 ; cg2d_iters(min,last) =      -1       9
cg2d: rhsMax =  1.23376801105470E+02 ; cg2d_iters(min,last) =      -1      10

which is very different from the reference case (etaN_ini ==0):

cg2d: rhsMax =  0.00000000000000E+00 ; cg2d_iters(min,last) =      -1       0
cg2d: rhsMax =  9.11436250326698E-06 ; cg2d_iters(min,last) =      -1      38
cg2d: rhsMax =  2.43007614611055E-05 ; cg2d_iters(min,last) =      -1      41
cg2d: rhsMax =  4.40770825601651E-05 ; cg2d_iters(min,last) =      -1      41
cg2d: rhsMax =  6.82090623486362E-05 ; cg2d_iters(min,last) =      -1      41

But if I skip the RHS normalisation (using cg2dTargetResWunit=2.E-16, instead of cg2dTargetResidual=1.E-7, so that cg2dNormaliseRHS=F) the results are independent of a uniform shift in etaN_ini.
Here the numbers from etaN_ini ==0 case:

cg2d: rhsMax =  0.00000000000000E+00 ; cg2d_iters(min,last) =      -1       0
cg2d: rhsMax =  9.11436250326698E-06 ; cg2d_iters(min,last) =      -1      27
cg2d: rhsMax =  2.43007480578522E-05 ; cg2d_iters(min,last) =      -1      28
cg2d: rhsMax =  4.40770605964644E-05 ; cg2d_iters(min,last) =      -1      29
cg2d: rhsMax =  6.82090943645262E-05 ; cg2d_iters(min,last) =      -1      31

For the 3-D solver, this might not be as sensitive but hard to tell as the cg3dNormaliseRHS=F option is not coded.
Planning to add a cg3dTargetResWunit option soon as this could be useful.

An explanation for this could be that the zeroth eigenvalue is in fact proportional to the mean of the RHS. I believe the smaller eigenvalues would be expected to converge faster than the smaller ones; so when the problem is shifted so that the zeroth is large and dominates the error, the solver iteration may converge more quickly than in the unnormalized case, where the eigenvalues are closer in magnitude such that improving the residual enough to reach convergence requires the longer process of improving many eigenvalues at once.
This is consistent with the result that the time-to-convergence is similar for a strict error criterion, since this stricter convergence requires finding the smallest eigenvalues in the problem, which are unaffected by the normalization.
If you can write down the exact system being solved we can deduce what the 0th eigenvalue should be (simply by taking the mean of the problem).


Update start.bat

bruh you might wanna change your bot token

I reset it after I push it out

On Apr 28, 2020, at 3:30 AM, iRey<EMAIL_ADDRESS>wrote:

bruh you might wanna change your bot token
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

This was clearly a mistake


Amazon s3 Stream Service opendir lists entire bucket when passed a sub directory.

After I've registered the s3 stream wrapper, when I call:
opendir('s3://bucket/part/of/a/key')

I get all (first 1000 anyway) objects in a bucket regardless of "path". Also because s3's default response is only 1000 objects I can't find the files I want after the fact. I suggest the stream wrapper use the prefix option when fetching the list via opendir. I have done this for my own uses so if this seems like a desired feature I can make a pull request.

@kkruit Zend_Service_Amazon wasn't updated for a while, so I recommend using the latest version of SDK maintained by Amazon - https://github.com/aws/aws-sdk-php


fixed broken override for site.variables

It is not possible to override the site.variables anymore because the line will always called after the override because @theme == @element.

You are misinterpreting this. @theme = @@element;, notice the two @@
This evaluates per-element differently, but assuming button,
@element = 'button' //defined in definition
@button = 'github';  // defined in theme.config

// theme.less
@theme = @@element;
@theme = @button;
@theme = 'github';

The order for site theme includes an extra level because it uses your specified @site theme as well.

Default theme site.variables
Selected theme's site variables
Current component's site variables
Your site theme's site variables

According you have the following theme config:
@site       : 'mysite';
@icon       : 'default';

if you override the @fontPath it will load the vars from default because in the icon less file, because:
@import (optional) "<EMAIL_ADDRESS>== @import (optional) "themes/default/globals/site.variables";
So if you want to override the fontPath you need to add it to the _site/globals/site.variables or use @icon       : 'mysite';


At step #5 sha256sum "no such file or directory" caused by linefeed compatability

Wow thanks for this awesome work. Exactly what I needed today. I've recently switched to Windows after almost a decade solely in Linux. Stepped into Windows 11 beta channel test this week. This project is the first Docker setup I've tried in this fresh instance.
In this new setup there seems to be an interoperability issue surrounding linefeeds in the old Unix vs Windows battle for how to do things right. When the dockerfile hit sha256sum checks across the packages it spits out an error no such file or directory. Might be dense of me but, you know, a decade's some time hey.
The only way I could resolve it to proceed with the build was to use the following in GIT config command before cloning this repo:
git config --global core.autocrlf false
A heads-up for you as it could go a little way to improving the documentation in your README for anyone not already tweaked GIT in this way and spare them a headache. Build just completed WAHOOOO!

Thanks for pointing this out. I've always disabled CRLF conversion, so I didn't know it was an issue. Then suddenly two of you trip over it a day apart (see #9). I've added a .gitattributes to disable line ending conversion, so that should fix it in future checkouts. This Git "feature" drives me nuts.


NullRef typing deparse()

Type deparse() in REPL. Null reference processing signature. VS survives

Broken RD with syntax errors. Need to harden against them
deparse(expr, width.cutoff = 60L,
        backtick = mode(expr)
            c("call", "expression", "(", "function"),
        control = c("keepInteger", "showAttributes", "keepNA"),
        nlines = -1L)

note comma missing after mode(expr)

Are you sure it's broken? in RS, the doc for deparse looks like this:
deparse(expr, width.cutoff = 60L,
        backtick = mode(expr) %in%
            c("call", "expression", "(", "function"),
        control = c("keepInteger", "showAttributes", "keepNA"),
        nlines = -1L)

Note the %in%. This looks like a valid expression to me - perhaps we're stripping that operator at some point before it's parsed?

Well there is no %in% in the RD and that causes R parser to parse function definition with errors and said named argument is missing value (looks like backticks=). RD is written by humans (it is basically LaTex) so it can contain errors anyway.

https://github.com/Microsoft/RTVS/commit/c15056c2fa91884f38d5b672002d84598d3be5e1


Add spamassassin

Adds spamassassin to inbound mail passing through the PyDis mailserver.
This adds a variety of headers which can be used by end user email clients or by
Sieve to filter mail according to the determined importance.

FIRE!


chore: minor refactor to eliminate code redundancy

Just an attempt made to eliminate code redundancy.

cc @juanpicado


How to register services which number of generic arguments dosen't equal the implementations.

This is the service interface:
public interface IRepository<TEntity> where TEntity : class, new()
{ ... }

This is the implementation:
public class Repository<TEntity, TContext> : IRepository<TEntity>
    where TEntity : class, new()
    where TContext : DbContext
{  ... }

This is how I register in a file named MyServiceCollectionExtensions.cs:
public static class MyServiceCollectionExtensions
{
    public static void AddMyFramework<TContext>(this IServiceCollection services)
        where TContext : DbContext
    {
        Type serviceType = typeof(IRepository<>).MakeGenericType(typeof(object));

        Type implementationType = typeof(Repository<,>).MakeGenericType(typeof(object), typeof(TContext));

        services.TryAddTransient(serviceType, implementationType);
    }
}

And in the Startup.cs file:
public void ConfigureServices(IServiceCollection services)
{
    ...
    services.AddMyFramework<ApplicationDbContext>();
    ....
}

Try it in a controller:
public class HomeController : Controller
{
    IRepository<ApplicationUser> _userRepository;
    public HomeController(IRepository<ApplicationUser> userRepository)
    {
        _userRepository = userRepository; // _userRepository is null
    }
}

The problem is the service IRepository<ApplicationUser> can not be resolved.
Is this kind of DI scenarios reasonable? And how can i make it work?
Thanks.
And sorry for my poor english :)

This issue is being closed because it has not been updated in 3 months.
We apologize if this causes any inconvenience. We ask that if you are still encountering this issue, please log a new issue with updated information and we will investigate.


Add optimization to minimize number of global loads

In order to minimize the number of global loads, we modify the expansion and allow more threads to load larger amounts of data. This PR adds this optimization pass.
Also, read nodes can now have multiple write dependencies and the expansion pass has been modified to handle this case. Other utilities have also been added such as delinearization and removing unused nodes.

Here is what the graph looks like before:

And this is the graph after:

Here we have a 4x reduction in global loads [8 -> 2]. I will add the code to generate the plots to the test.

Also refactored it a bit, so hopefully more clear.


fastlane init fails, missing company name, creating first app in app store connect

🔑

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description
When I run fastlane init:

using a team account
creating a new app in app store connect
with no other apps existing in app store connect

I get this message:
[22:11:56]: ["The request could not be completed because:", "You must provide a company name to use on the App Store. You must provide a company name to use on the App
 Store."]

From other issues and a scan of the docs for produce, it seems that I need to give it a company_name, but I'm not sure how to do that via fastlane init, which doesn't appear to take such params (based on fastlane init --help, at least).
Command executed

fastlane init
Complete output when running fastlane, including the stack trace and command used

➜ fastlane init
[22:11:11]: Detected an iOS/macOS project in the current directory: 'the-liturgists.xcworkspace'
[22:11:11]: -----------------------------
[22:11:11]: --- Welcome to fastlane 🚀 ---
[22:11:11]: -----------------------------
[22:11:11]: fastlane can help you with all kinds of automation for your mobile app
[22:11:11]: We recommend automating one task first, and then gradually automating more over time
[22:11:11]: What would you like to use fastlane for?
1. 📸  Automate screenshots
2. 👩✈  Automate beta distribution to TestFlight
3. 🚀  Automate App Store distribution
4. 🛠  Manual setup - manually setup your project to automate your tasks
?  2
[22:11:15]: -----------------------------------------------------------
[22:11:15]: --- Setting up fastlane for iOS TestFlight distribution ---
[22:11:15]: -----------------------------------------------------------
[22:11:15]: Parsing your local Xcode project to find the available schemes and the app identifier
[22:11:15]: $ xcodebuild -showBuildSettings -workspace the-liturgists.xcworkspace -scheme the-liturgists
[22:11:17]: $ cd /Users/brhiggins/src/theliturgists/app/ios && agvtool what-version -terse
[22:11:18]: --------------------------------
[22:11:18]: --- Login with your Apple ID ---
[22:11:18]: --------------------------------
[22:11:18]: To use App Store Connect and Apple Developer Portal features as part of fastlane,
[22:11:18]: we will ask you for your Apple ID username and password
[22:11:18]: This is necessary for certain fastlane features, for example:
[22:11:18]:
[22:11:18]: - Create and manage your provisioning profiles on the Developer Portal
[22:11:18]: - Upload and manage TestFlight and App Store builds on App Store Connect
[22:11:18]: - Manage your App Store Connect app metadata and screenshots
[22:11:18]:
[22:11:18]: Your Apple ID credentials will only be stored in your Keychain, on your local machine
[22:11:18]: For more information, check out
[22:11:18]:     https://github.com/fastlane/fastlane/tree/master/credentials_manager
[22:11:18]:
[22:11:18]: Please enter your Apple ID developer credentials
[22:11:18]: Apple ID Username:

[22:11:26]: Logging in...
Multiple App Store Connect teams found, please enter the number of the team you want to use:
1) "Brett Higgins"
2) "The Liturgists, Inc."
2
[22:11:32]: ✅  Logging in with your Apple ID was successful
[22:11:32]: Checking if the app 'com.theliturgists.app' exists in your Apple Developer Portal...
[22:11:32]: ✅  Your app 'com.theliturgists.app' is available in your Apple Developer Portal
[22:11:32]: Checking if the app 'com.theliturgists.app' exists on App Store Connect...
[22:11:33]: Looks like the app 'com.theliturgists.app' isn't available on App Store Connect
[22:11:33]: for the team ID '' on Apple ID ''
[22:11:33]: Would you like fastlane to create the App on App Store Connect for you? (y/n)
y
+----------------+-------------------------+
|       Summary for produce 2.130.0        |
+----------------+-------------------------+
| username       | (snip) |
| team_id        | (snip)              |
| itc_team_id    | (snip)               |
| platform       | ios                     |
| app_identifier | com.theliturgists.app   |
| skip_devcenter | true                    |
| sku            | (snip)              |
| language       | English                 |
| skip_itc       | false                   |
+----------------+-------------------------+
[22:11:37]: App Name: The Liturgists
[22:11:42]: Creating new app 'The Liturgists' on App Store Connect
[22:11:43]: ["The request could not be completed because:", "You must provide a company name to use on the App Store. You must provide a company name to use on the App
Store."]
[22:11:43]: It looks like something went wrong when we tried to create your app on the Apple Developer Portal
[22:11:43]: Would you like to try again (y)? If you enter (n), fastlane will fall back to the manual setup (y/n)
n
[22:11:56]: --------------------
[22:11:56]: fastlane init failed
[22:11:56]: --------------------
[22:11:56]: ["The request could not be completed because:", "You must provide a company name to use on the App Store. You must provide a company name to use on the App
Store."]
[22:11:56]: Something failed while running fastlane init
(loops after this. I stopped with Ctrl-C.)

Environment

✅ fastlane environment ✅
Stack

Key
Value

OS
10.14.6

Ruby
2.2.4

Bundler?
false

Git
git version 2.13.1

Installation Source
~/.fastlane/bin/bundle/bin/fastlane

Host
Mac OS X 10.14.6 (18G87)

Ruby Lib Dir
~/.fastlane/bin/bundle/lib

OpenSSL Version
OpenSSL 1.0.2g  1 Mar 2016

Is contained
false

Is homebrew
true

Is installed via Fabric.app
false

Xcode Path
/Applications/Xcode.app/Contents/Developer/

Xcode Version
10.3

System Locale

Variable
Value

LANG
en_US.UTF-8
✅

LC_ALL
en_US.UTF-8
✅

LANGUAGE
en_US.UTF-8
✅

fastlane files:
No Fastfile found
No Appfile found
fastlane gems

Gem
Version
Update-Status

fastlane
2.130.0
✅ Up-To-Date

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

slack-notifier
2.3.2

atomos
0.1.3

CFPropertyList
2.3.5

claide
1.0.3

colored2
3.1.2

nanaimo
0.2.6

xcodeproj
1.12.0

rouge
2.0.7

xcpretty
0.3.0

terminal-notifier
2.0.0

unicode-display_width
1.1.3

terminal-table
1.7.3

plist
3.2.0

public_suffix
2.0.5

addressable
2.5.1

multipart-post
2.0.0

word_wrap
1.0.0

tty-screen
0.7.0

tty-cursor
0.7.0

tty-spinner
0.9.1

babosa
1.0.2

colored
1.2

highline
1.7.8

commander-fastlane
4.4.6

excon
0.55.0

faraday
0.12.1

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20170404

http-cookie
1.0.3

faraday-cookie_jar
0.0.6

fastimage
2.1.0

gh_inspector
1.1.3

json
1.8.1

mini_magick
4.9.5

multi_xml
0.6.0

rubyzip
1.2.3

security
0.1.3

xcpretty-travis-formatter
0.0.4

dotenv
2.2.0

bundler
1.14.6

faraday_middleware
<IP_ADDRESS>

naturally
2.2.0

simctl
1.6.5

jwt
2.1.0

uber
0.0.15

declarative
0.0.10

declarative-option
0.1.0

representable
3.0.4

retriable
2.1.0

mime-types-data
3.2016.0521

mime-types
3.1

multi_json
1.12.1

signet
0.11.0

memoist
0.16.0

os
0.9.6

googleauth
0.6.7

httpclient
2.8.3

google-api-client
0.23.9

google-cloud-env
1.2.1

google-cloud-core
1.3.1

digest-crc
0.4.1

google-cloud-storage
1.16.0

emoji_regex
1.0.1

libxml-ruby
3.0.0

psych
2.0.8

generated on: 2019-08-31

Not sure why @fastlane-bot flagged this as related to code signing when the logs refer to init and produce, and don’t mention anything in the linked guide.

Workaround is to create the app in App Store Connect before running fastlane init, but this still seems like a bug.

Not sure what to do here other than repeat my comment from above. It would really help to have a human respond here.

This still appears to be an issue.  I will follow the workaround above but fastlane init errors out with the following.
+----------------+-----------------------------------+
|            Summary for produce 2.137.0             |
+----------------+-----------------------------------+
| username       |<EMAIL_ADDRESS>                  |
| team_id        | xxxxxxxxxx                        |
| itc_team_id    | xxxxxxxxx                         |
| platform       | ios                               |
| app_identifier | co.xxxx.xxxx.xxxx.xxxx            |
| skip_devcenter | true                              |
| sku            | xxxx                              |
| language       | English                           |
| skip_itc       | false                             |
+----------------+-----------------------------------+

[13:49:39]: App Name: AppNameHere
[13:49:47]: Creating new app 'AppNameHere' on App Store Connect
[13:49:47]: ["The request could not be completed because:", "You must provide a company name to use on the App Store. You must provide a company name to use on the App Store."]
[13:49:47]: It looks like something went wrong when we tried to create your app on the Apple Developer Portal
[13:49:47]: Would you like to try again (y)? If you enter (n), fastlane will fall back to the manual setup (y/n)

/me continues screaming into the void

I have never used fastlane init before, but I discovered this GitHub issue after receiving the error message The request could not be completed because:", "You must provide a company name to use on the App Store. You must provide a company name to use on the App  Store. when I ran fastlane produce on my machine.
I stopped receiving the error message when I ran fastlane produce --company_name "Company Name Here"
I am not sure why the CLI did not as me to enter my company name if it needed it and I did not provide it? The CLI does a great job at asking for other pieces of information that it does not know.

I give up. Best of luck to the next person who runs into this.

Yup, just ran into it.

Just ran into this today

Just ran into this today using:

fastlane 2.149.1

I also have never had an setup before and after having the failure trying fastlane produce --company-name "My Company Name" work and then re-running fastlane init it appears everything was/is setup correctly.
So it seems like there is just a bug in the fastlane init flow that doesn't aske for Company Name when its registering with App Store Connect for the first time ever with a fresh account without any previous apps.
Would be awesome to have someone look into this as it really gums up first time users new to iOS development.

Just encountered this today as well...

@brettdh This issue still exists. In fact, I have created a post on StackOverflow to resolve this issue but now I know it's a bug:
https://stackoverflow.com/questions/62705823/fatlane-is-unable-to-create-the-new-app-on-apple-developer-portal
I didn't know it was an issue related to the company name as in the terminal, it wasn't throwing an error. It was just asking whether I wanna start the process again. Later on, when I chose to switch back to manual setup, I got to know it was an issue related to the company name.
The issue was reported in Sep 2019 and I see very few are facing this issue. Is it only happening with the ones who have created a new account?

@rashtay please see the comment history above. I know this still exists; I argued with the bot for months trying to get some attention from maintainers, and then I gave up. My initial message details why it affects only a particular usage (which may be why it’s received no attention), but I hoped that the fact that potentially every new fastlane user (or new App Store developer at least) could encounter it would be compelling enough to warrant at least a response. Alas, here we are.
I’d be happy for you or anyone else to pick up the charge of trying to get a fastlane maintainer to look at this. As for me, I’m done trying.

It looks like there is a test for this condition and according to this https://github.com/fastlane-old/produce/issues/54#issuecomment-122854953 it only happens for new accounts and on the first app creation.
If you look at this https://github.com/fastlane-old/produce/issues/54#issuecomment-126267796 I will take a guess that what has happened is that the field was either renamed or moved and the selector no longer matches, since you have to have a new account to test this, it probably hasn't gotten much love:
    describe "#create! first app (company name required)" do
      it "works with valid data and defaults to English" do
        TunesStubbing.itc_stub_applications_first_create
        Spaceship::Tunes::Application.create!(name: "My Name",
                                              sku: "SKU123",
                                              bundle_id: "net.sunapps.123",
                                              company_name: "SunApps GmbH")
      end

      it "raises an error if something is wrong" do
        TunesStubbing.itc_stub_applications_broken_first_create
        expect do
          Spaceship::Tunes::Application.create!(name: "My Name",
                                                sku: "SKU123",
                                                bundle_id: "net.sunapps.123")
        end.to raise_error("You must provide a company name to use on the App Store. You must provide a company name to use on the App Store.")
      end
    end

@bsbodden Yep. I had looked at the comment too. And I agree, the issue is prevalent for newer account. What I atleast expect is for developers to show the errors upfront.
The command just keeps asking you to repeat the steps if something goes wrong rather than showing any error.
Should I create a new issue with steps to reproduce atleast to get their attention?

Probably a good idea. 5 years is a long time so I’m sure this has dropped
off the radar.
On Sun, Jul 5, 2020 at 10:49 AM Rahul Shetty<EMAIL_ADDRESS>wrote:

@bsbodden https://github.com/bsbodden Yep. I had looked at the comment
too. And I agree, the issue is prevalent for newer account. What I atleast
expect is for developers to show the errors upfront.
The command just keeps asking you to repeat the steps if something goes
wrong rather than showing any error.
Should I create a new issue with steps to reproduce atleast to get their
attention?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fastlane/fastlane/issues/15252#issuecomment-653918256,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAF4LINHI6PYZTML7ODNPTR2C4LBANCNFSM4ISVLMLQ
.

--
Cheers,
Brian
https://integrallis.com

I still have a fresh / new apple account with no apps and will test out the latest fastlane sometime this week and report back.

Got the same error right now on an empty account:
"It looks like something went wrong when we tried to create your app on the Apple Developer Portal"
The bundle/app_identifier was created fine in App Store Connect.
The app is not created in App Developer Portal.
I had to manually create the app in the portal to complete fastlane init


New Feature: Semantic Segmentation

I started to work on semantic segmentation with TF/Keras. i will mainly use segmentation models library, if you have any suggestions or if you want to help feel free to contact me!

Thx for your effort @erentknn! You are welcome to open a work in progress PR as soon as you like, so I can give feedback (mark with [WIP]).

Hey @erentknn, any progress on this so far? @The-ML-Hero was also thinking about working on segmentation (see #15), just want to coordinate so you don't do duplicate work :)

hey @jrieke I worked on for a while but currently i cant make any progress because of work and grad school. So he can work on Semantic Segmentation. I am closing this.


xwbtool command-line should support wildcards

Should be able to do:
xwbtool -o mybank.xwb *.wav

Support for wildcards in input filenames (along with -r for optional directory search) has been implemented for uvatlas tool, meshconvert, texconv, texassemble, and xwbtool.


I work at a school on a Reservation in Wyomin

We would love to add the Arapahoe and Shoshone Languages to Woolaroo as a schoolwide project, how can we assist?

Sorry it took so long to get this noticed - let me know if you are still interested in getting these included and I can hook you up with the team at Google.

@Wyomin if you are still interested in getting Arapahoe and Shoshone Languages into Woolaroo, please reach out to<EMAIL_ADDRESS>Thanks


I really should stop publishing empty npm packages

So, I keep publishing empty npm modules.  Accidentally of course.  Turns out that npm prepare failing to run doesn't actually cause a non-0 exit code.  See this for example:
https://circleci.com/gh/googleapis/nodejs-common/2847
#!/bin/bash -eo pipefail
npm publish
npm WARN lifecycle<EMAIL_ADDRESS>cannot run in wd %s %s (wd=%s<EMAIL_ADDRESS>npm run compile .
+<EMAIL_ADDRESS>
What you see right there is npm prepare failing, yet still letting npm publish continue with a 0 exit code ....  The result is the publishing of an empty npm module with no build directory.
Do folks have any thoughts on how we could test this before publishing empty modules?
@ofrobots @alexander-fenster @kinwa91 @GoogleCloudPlatform/node-team

If it's true that a failing npm prepare doesn't cause a non-zero exit code, maybe we should file a bug against npm?

This bit us again in https://github.com/googleapis/nodejs-storage/issues/741.

I think we're good now :)


Open to supporting alternative executable name for bat in this module?

Like the title suggests, I wanted to gauge whether there was interest on your side to support an alternative executable name for bat, in order to help people adopt the module without having to perform extra set up on their end.
The specific use-case I have in mind comes from Debian and its derivatives, which create the bat executable as batcat (possibly to avoid a conflict with another package):
% dpkg -L bat
/.
/usr
/usr/bin
/usr/bin/batcat

Right now it is possible to create a symlink manually in the PATH, or to fiddle with $commands, but given the popularity of Debian-like distros I think it could be convenient and relatively safe to fallback to batcat if bat doesn't exist.

Hallo @antoineco. First thank you for the support you give to Zim users in our issues and discussions!
Sure, I was not aware of bat also being installed as batcat in some distros. Fair enough, we could fallback to trying batcat if bat is not found. I can work on this.

Fantastic, thank you!
In case you're busy I can happily sent a PR, the change is trivial.

@antoineco, cool, please go ahead.


Trello-1731: Gor Replay ste header

The content-store application only supports the "Forwarded:" header
therefore the "X-Forwarded-Host:" header can be changed for the content
store.
NOTE: defaulting to the "X-Forwarded-Host:" header as the cache router
that also uses the gor replay class needs to support this.
Pair: @ronocg<EMAIL_ADDRESS>& @fredericfran-gds<EMAIL_ADDRESS>

fine my side. need someone to look at it.


Call showFirebaseLoginPrompt() if no user is logged in already

I want my app to display the login prompt when the user opens the app and if there is no currently logged in user. I do not want to call it on click of any button. I'm getting a Null pointer exception if i place showFirebaseLoginPrompt() directly in onCreate and the log cat reads:
Attempt to invoke virtual method FirebaseLoginDialog.show on a null object reference
Where should I call showFirebaseLoginPrompt() to show login dialog, only if there is no currently logged in user, without the user having to click any button?

Have you tried calling it in onStart()?

While the exact method showFirebaseLoginPrompt is not applicable to 0.4.0, this is now 'fixed' because you can do this in onStart():
if (FirebaseAuth.getInstance().getCurrentUser() == null) {
    // Not signed in, show login UI
    // ...
} else {
    // Signed in, continue
    // ...
}


gazebo_ros_pkgs: 3.8.0-1 in 'jazzy/distribution.yaml' only gazebo_msgs

Part of the works https://github.com/ros-simulation/gazebo_ros_pkgs/issues/1532

Holding for Jazzy sync


Siamese Network

A Simple Implementation of Siamese Network using Constrastive Loss.

@googlebot I signed it!

@8bitmp3 I made the changes  please review them.

@8bitmp3 I made the changes please review them.

@s-mrb will do, let me go through the remaining part and I'll get back to you. Thank you 👍

@fchollet i did all the changes you asked.

@rola93 I made separate validation_data as you said. You were right it is good for beginners to have separate val data
But I still want to understand what do you mean when you said

you are using the same images for validation and training, which is wrong. This is because of the nature of your sets: you build the pairs randomly.

I think having separate validation_data beforehand is important [only] if data is unbalanced or some particular classes are congested in the end of the data, because valid_split assign validation data from end of dataset. That said, this problem could have a simple solution, that is to shuffle data before fitting the model and without needing separate valid_data beforehand.
Info you would give might be new to me, so i am curious to know your point.

Thanks for the change, it is far better now.
Let me explain, based on your previous implementation.
suppose you have the following samples with its labels: [x1, x2, x3, x4], where xi includes its appropriated labels. Then, you add for each xi, a pair xj, (with an appropriated binary label to tell if label(xi) == label(xj)).
You do it randomly, so, for instance you can have the following pairs:
train_samples = [
    [x1, x2],
    [x2, x3],
    [x3, x2],
    [x4, x1]
]

Then, you take 10% of those pairs for validation, once again, randomly (note that this is implicitly done by keras with the validation parameter in fit function). For this sample, let's assume you take 50%, and what is more, take first two samples for training and the last two for validation:
training_pairs = [
    [x1, x2],
    [x2, x3]
]
validation_pairs = [
    [x3, x2],
    [x4, x1]
]

Now you can see that samples x1, x2 and x3 are in both splits: training and validation, and are not independent!. Additionally, as both towers share its weights (i.e. Siamese network), it doesn't matter the order of pairs, (xi, xj) is the same that (xj, xi).
I think this sample illustrates the point very well.
With respect to this:

I think having separate validation_data beforehand is important [only] if data is unbalanced or some particular classes are congested in the end of the data.

train, validation and testing sets must all be independent, and in particular validation and testing sets must respect real distributions.

@fchollet train_test_split removed!

thanks rodrigo @rola93  for your review, it is really great insight!! 💯

Please address this: https://github.com/keras-team/keras-io/pull/461/files#r632079235

@fchollet  Notebook for further review.

@fchollet generated files added

Can't wait to see this merged! great work!


Cannot upload cookbook to local supermarket.

Hi,
I'm trying to upload cookbook https://github.com/sethvargo/chef-sugar/ to local supermarket and it fails because of format of version metadata.rb. It's version Chef::Sugar::VERSION while number is expected. Do you know how to upload such cookbooks?
Thanks,
Yury.

Yes, do not use knife cookbook upload because it uploads uncompiled metadata which is both insecure and not guaranteed to be portable. Use a tool like Stove or compile the metadata.json yourself before uploading.


chore: remove duplicate theorem about lists

The theorem List.isEmpty_iff_eq_nil had already been moved to
Init.Data.List.Lemmas and renamed List.isEmpty_iff.

awaiting-review

Please add a deprecation.


Release 3.8 binary wheels

Issue: https://github.com/grpc/grpc/issues/20615
Build log id: 136e214a-f787-4938-b2c7-5a0bca1e2e49

As expected, the macOS and Windows Kokoro image do not support 3.8 yet.
New build log ID:  f49944b3-fd0a-459b-9c86-5b3ee5004c73

Unrelated RBE failure


Sortable transforms

Before I create a PR I'd like to talk over the design of the sortable transform, and transforms in general.
sortable currently takes as an argument the property, which both identifies the sorted column and tells the sorter how to access the data fields corresponding to them.
Pros: This makes it easy for users to make headers sortable ad-hoc, and allows reactabular's sorter function to stay relatively simple.
Cons: Potential for conflict when two columns use the same property and/or property isn't enough to extract the string to be sorted on – both of which may happen with nested data; also, it's not straightforward how multiple transforms can be composed.
One alternative approach – the one I've been taking, because I've abstracted a reusable table component away similar to what you've done in StatefulTable.jsx – is to call sortable with the column's index, instead of its property, and then to pass pre-flattened (read: resolved) data into the sorter, along with the properties of those columns whose indices are listed among the sortingColumns, like so:
    const filteredData = search.multipleColumns({ data: this.props.data, columns: this.state.columns, query: this.state.search });

    let sortedData = [];
    if (this.state.sortingColumns.length) {
      const flattenedData = cloneDeep(filteredData); // make a copy
      /* go through each of the sorting columns, loop through all values and resolve the data. */
      for (let j = 0, k = this.state.sortingColumns.length; j < k; j++) {
        const sortIndex = this.state.sortingColumns[j].property; // property is actually the column index
        const column = this.state.columns[sortIndex];
        const resolver = (column.cell.resolve && typeof column.cell.resolve === 'function') ? column.cell.resolve : (a => a);
        for (let i = 0, l = flattenedData.length; i < l; i++) {
          flattenedData[i][column.cell.property] = resolver(flattenedData[i][column.cell.property], {cellData: flattenedData[i]});
          flattenedData[i].index = i;
        }
      }

      /* throw the data into orderBy */
      const sortedFlattenedData = sort.sorter(flattenedData, this.state.sortingColumns.map(c => ({
        sort: c.sort,
        property: this.state.columns[c.property /* actually the column index */].cell.property
      })), orderBy);
      /* go through the sorted data and re-shuffle the original according to the new order of indices */
      sortedData = new Array(sortedFlattenedData.length);
      for (let i = 0, l = sortedFlattenedData.length; i < l; i++) {
        sortedData[i] = filteredData[sortedFlattenedData[i].index];
      }
    } else {
      sortedData = filteredData;
    }

Pros: Everything is uniquely identified and properly resolved before it gets sorted.
Cons: Can't really expect users to pass in the column index reliably, unless they do so in the context of a stateful table component that loops through columns and does so automatically.
So now I wonder … maybe it's possible to extend the transform API such that it calls ({label, cellData, index}). Then sortable could be used like
transform: ({index}) => sortable(index)

or, if multiple transforms are desired,
transform: ({label, cellData, index}) => ({
  ...sortable(index),
  ...otherTransform(label, cellData),
})

The sorter could then incorporate the pre-flattening steps, with the only necessary change to its API being that it now needs access to the columns as well.
I haven't really considered how this would affect other parts of the module, so I wanted to start a high-level discussion before submitting code. Penny fer ya' thoughts?

In fact, what'd be even nicer is if the transform field were called transforms and accepted arrays of functions, to be spread-composed internally. That would make it easier for stateful table components to supplement columns with extra transforms before rendering; e.g. I'd like for all my columns to be sortable by default.

Let's do transforms. That sounds sane to me. Do you want to PR something or should I give it a go?

Speaking of sorting, I did a little proof of concept today. It would be fairly easy to write a wrapper for Reactabular core that allows you to set sortable: true for a header at your data definition. The same could be done for search and so on.

Cool. To be honest I'm not 100% familiar with the codebase yet, so if you want to wait for a PR it'd probably take a few days. No hard feelings if you want to go ahead and put it in yourself.
Re: sortable: true ... one thing I awkwardly hacked together in my reusable component but forgot to mention was a way to define an initial sort order. Write it sortable: true, initialSort: 'asc', or sort: {initial: 'asc', locked: true}, or whatever you like, but I would find that very useful.

Ah, yeah. Initial sort order would definitely be handy.
Could you craft a little specification for transforms? Just having a couple of bullets would be enough for me to write proper test cases and the implementation. This way we understand it the same way.

All I was thinking was that transforms would get an array of functions, and each would be called with ({label, cellData, columnIndex}) resp. ({resolvedValue, cellData, rowIndex, columnIndex}) for the cells.
Each transform function returns an object of props, and they're all joined together into one object before application, i.e.
/* ... as you're going through all of the columns in table.jsx:61 ... */
const transformed = Object.assign(...transforms.map(t => t({label, cellData, i})));

Is that what you're asking?

Yeah, I'll go with that. Thanks. 👍

There's a prototype in the repository now. Should the results override or patch the existing data? Here's a test that explains the issue better.
Merging props like style would feel more natural alternative to me.

Nice, thank you!
I'm partial to style props and classNames being deep-merged as you said. At least you don't lose information that way, and if you really were in a position where you had to manually override earlier styles you still could. With classnames it's trickier than with style values actually, because their precedence depends on their order in the stylesheet, not on their order in the props. Still, worst-case frustration complexity is better for merging :+1:
What are your thoughts on changing the sortable to take an index instead of the property, and wrapping the sorter to deal with nested data?

I'm partial to style props and classNames being deep-merged as you said. At least you don't lose information that way, and if you really were in a position where you had to manually override earlier styles you still could. With classnames it's trickier than with style values actually, because their precedence depends on their order in the stylesheet, not on their order in the props. Still, worst-case frustration complexity is better for merging

Ok, deep merge is fine. I'll drop lodash merge there. To keep it predictable it probably has to go from right to left so that transforms: [foo, bar] is equal to foo(bar(... parameters ...)).
Good point about classnames. I have a special hack for that. I really don't like React's string based interface so we have to treat classes differently. This has already been done.

What are your thoughts on changing the sortable to take an index instead of the property, and wrapping the sorter to deal with nested data?

I will add nested data to the full demo and see what happens. That could motivate the change.

It's using the behavior now we discussed earlier.

I will add nested data to the full demo and see what happens. That could motivate the change.

The full demo actually uses nested data. Could you set up a standalone demo after the next beta so I can understand what's going on a little better?
I've thought about separating the ideas of data and order. Now these are intermingled and the approach works. Having separate sort/filtering index might be a direction worth investigating. Then we wouldn't have to mutate the original data and it would become more of a source for lookups. This is just a random idea, though.

The full demo actually uses nested data. Could you set up a standalone demo after the next beta so I can understand what's going on a little better?

My idea was that sorting should happen on the resolved string, not on whatever get(row, property) returns. When you sort the full table by country, for example, the order goes Germany, Finland, Greece ..., which isn't exactly alphabetic; what's worse, when get(row, property) is itself an object, the sorting fails entirely.
Using the index instead of the property is an unrelated suggestion; it makes sense to me because it's a) an easy way to avoid having conflicts between columns with identical property, and b) it lets you write columns[i] instead of having to use the finnicky and slow(er) find or findIndex etc. Since the transforms can be called with the column index as a parameter, it's not much of a burden on the users either to just have them write sortable(columnIndex). (And setting sortable: true would simply be a shortcut to having that same transform applied by core before rendering.)
I hope that makes sense?

Ah, yeah. I see the problem now. Avoiding find/findIndex would be definitely neat and getting resolved values to work in a sane way would be good.
I'll think about it and get something done.

You have access to columnIndex and rowIndex now. I happened to need them for an excel demo. They'll be available in the next beta so I suppose it's safe to close this.
I still have to walk through those find bits and simplify the demos.


[HeroSystem6eHeroic] README images updated. Minor sheet styling update.

Submission Checklist
Required

[X] The pull request title clearly contains the name of the sheet I am editing.
[X] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[X] The pull request makes changes to files in only one sub-folder.
[X] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

New Sheet Details

The name of this game is: <   >  (i.e. Dungeons & Dragons 5th Edition, The Dresden Files RPG)

The publisher of this game is: <   > (i.e. Wizards of the Coast, Evil Hat)

The name of this game system/family is: <   > (i.e. Dungeons & Dragons, FATE)

[ ] I have followed the Character Sheets Standards when building this sheet.

[ ] This sheet has been made on behalf of, or by, the game's publisher.
[ ] This game is not a traditionally published game, but a copy of the game rules can be purchased/downloaded/found at: <   >
[ ] This sheet is for an unofficial fan game, modification to an existing game, or a homebrew system.

Changes / Description

Character Sheet Info Roll20 Internal Use only.

@Villain1nGlasses I noticed in your preview image that the + in front of the inputs for Perception, Vision, Hearing, and Smell followed the input to the first row. You may want to put out a fast follow to put it back in front of the new bottom row

Thanks Nora, I’ll check it.
-Rob

On Jan 29, 2024, at 3:18 PM, Nora Cahill @.***> wrote:
@Villain1nGlasses https://github.com/Villain1nGlasses I noticed in your preview image that the + in front of the inputs for Perception, Vision, Hearing, and Smell followed the input to the first row. You may want to put out a fast follow to put it back in front of the new bottom row
—
Reply to this email directly, view it on GitHub https://github.com/Roll20/roll20-character-sheets/pull/12521#issuecomment-1915674153, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUP4VZTZQCAAUPEIVZLENZ3YRAN3VAVCNFSM6AAAAABCLI2EWOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJVGY3TIMJVGM.
You are receiving this because you were mentioned.

Hi Nora,
I can see how that would be confusing. The second line is the net value. There is space to add the base in front of the modifier. I’ll see how that looks and submit an update if it works out.
Thanks,
Rob

On Jan 29, 2024, at 3:19 PM, Robert C. Word @.***> wrote:
Thanks Nora, I’ll check it.
-Rob

On Jan 29, 2024, at 3:18 PM, Nora Cahill @.***> wrote:
@Villain1nGlasses https://github.com/Villain1nGlasses I noticed in your preview image that the + in front of the inputs for Perception, Vision, Hearing, and Smell followed the input to the first row. You may want to put out a fast follow to put it back in front of the new bottom row
—
Reply to this email directly, view it on GitHub https://github.com/Roll20/roll20-character-sheets/pull/12521#issuecomment-1915674153, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUP4VZTZQCAAUPEIVZLENZ3YRAN3VAVCNFSM6AAAAABCLI2EWOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJVGY3TIMJVGM.
You are receiving this because you were mentioned.


Property Based Tests

Description
adding property based tests (with streamdata) will improve the confidence that the test suite provides, and maybe expose bugs that are hard to find through case-based hand-written tests.
overall the existing code base (and test suite) should become easier to extend, and make the library more reliable as a dependency

i've started working on this over here

Feel free to open a draft PR if you'd like.