[iOS] Keyboard is not synchronized with edge swiping gesture

Keyboard should be animated on edge swiping back gesture as it is on iOS apps, but it isn't. Indeed, it is instantly dismissed with a little touch, causing a bad user experience.
Expected results:

Actual results:

Hi @afnx
Thanks for fling the issue, keyboard is indeed not synchronized with a swipe gesture on iOS
When swiping back
https://user-images.githubusercontent.com/48603081/123433244-e747f300-d5d3-11eb-9e2e-755ad587c055.mp4
When navigating to screen with CupertinoTextField and immediately focusing
https://user-images.githubusercontent.com/48603081/123433251-e9aa4d00-d5d3-11eb-8c07-39e18da296b1.mp4
Used the following minimal code sample

minimal code sample
import 'package:flutter/cupertino.dart';
import 'package:flutter/scheduler.dart';

void main() {
  timeDilation = 12;
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return CupertinoApp(
      title: 'Cupertino App',
      home: Home(),
    );
  }
}

class Home extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return CupertinoPageScaffold(
      navigationBar: CupertinoNavigationBar(
        middle: Text('Cupertino App Bar'),
      ),
      child: Center(
        child: Container(
          child: CupertinoButton(
            onPressed: () {
              Navigator.push(
                  context, CupertinoPageRoute(builder: (_) => Second()));
            },
            child: Text('Go to second scree'),
          ),
        ),
      ),
    );
  }
}

class Second extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return CupertinoPageScaffold(
      navigationBar: CupertinoNavigationBar(
        middle: Text('Cupertino App Bar'),
      ),
      child: Center(
        child: Container(
          child: CupertinoTextField(),
        ),
      ),
    );
  }
}

stable
master

❌
❌

Check flutter doctor -v outputs for each channel below

flutter doctor -v
[✓] Flutter (Channel stable, 2.2.2, on macOS 11.4 20F71 darwin-x64, locale
    en-US)
    • Flutter version 2.2.2 at /Users/tahatesser/Code/flutter_stable
    • Framework revision d79295af24 (2 weeks ago), 2021-06-11 08:56:01 -0700
    • Engine revision 91c9fc8fe0
    • Dart version 2.13.3

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Volumes/Extreme/SDK
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Volumes/Extreme/SDK
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Volumes/Extreme/Xcode.app/Contents/Developer
    • Xcode 12.5, Build version 12E262
    • CocoaPods version 1.10.1

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

[✓] Android Studio (version 4.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] VS Code (version 1.57.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.23.0

[✓] Connected device (3 available)
    • iPhone 12 Pro (mobile) • 4819C924-80DB-4DE5-9093-71A234590ABB • ios
      • com.apple.CoreSimulator.SimRuntime.iOS-14-5 (simulator)
    • macOS (desktop)        • macos                                • darwin-x64
      • macOS 11.4 20F71 darwin-x64
    • Chrome (web)           • chrome                               •
      web-javascript • Google Chrome 91.0.4472.114

• No issues found

[✓] Flutter (Channel master, 2.3.0-17.0.pre.504, on macOS 11.4 20F71 darwin-x64,
    locale en-US)
    • Flutter version 2.3.0-17.0.pre.504 at
      /Users/tahatesser/Code/flutter_master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 102acbe39d (7 hours ago), 2021-06-24 22:51:02 -0700
    • Engine revision 4bf9d638c2
    • Dart version 2.14.0 (build 2.14.0-245.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Volumes/Extreme/SDK
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Volumes/Extreme/SDK
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Volumes/Extreme/Xcode.app/Contents/Developer
    • Xcode 12.5, Build version 12E262
    • CocoaPods version 1.10.1

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

[✓] Android Studio (version 4.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] VS Code (version 1.57.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.23.0

[✓] Connected device (3 available)
    • iPhone 12 Pro (mobile) • 4819C924-80DB-4DE5-9093-71A234590ABB • ios
      • com.apple.CoreSimulator.SimRuntime.iOS-14-5 (simulator)
    • macOS (desktop)        • macos                                • darwin-x64
      • macOS 11.4 20F71 darwin-x64
    • Chrome (web)           • chrome                               •
      web-javascript • Google Chrome 91.0.4472.114

• No issues found!

✅ : No Issue ❌: Issue reproduced

This may require thread merging. One experiment could be placing a platform view in the top-right corner, and seeing if that behaves correctly. /cc @cyanglaz

It's been almost 2 years and no update. The unsynchronized keyboard dismissal causes a strange experience that makes users feel there is something with the app. Fixing it is essential for those who target iOS devices.
This issue should be labelled as P2 or P3 at least. Flutter's goal of delivering apps that feel natural will never be achieved if the problems like this are not deemed important.

I don't know any way the keyboard position could be animated. A possible way that I can think of is to take a screenshot of of the keyboard and animate the screenshot with the navigation.

Still need this, because of viewInsets, which causes all widget tree to rebuild. Also have bugs with keyboard which breaks all screens after popping app from background with opened keyboard. Keyboard should be attached to only one screen view without affecting whole widget tree

is there any update for this?


Handle 304 responses?

Some APIs that support conditional requests, like the GitHub API, do not count it against your rate limit if you make a conditional request and get a 304 Not Modified response.
It might be convenient to add an option to not apply client-side rate-limiting for 304s.

Similar to handling 429 responses (#4) and the limit_statuses option (#34), it also might be useful to provide a no_limit_statuses option to specify additional status codes that don't count against the rate limit.
I'll wait until there's an actual use case for that before implementing, though.


Gemini stops in the middle of warnup with no apparent reason

Issue description
Gemini stopped sending out requests after less then an hour, without any indication in it's output that there's errors of any kind
this was the command:
2023-01-20 16:05:43.088: (GeminiStressEvent Severity.ERROR) period_type=end event_id=591f598c-7aec-42ed-825e-e44ec0569d3c duration=11h29m59s: node=Node gemini-1tb-10h-master-loader-node-f7d9883d-1 [<IP_ADDRESS> | <IP_ADDRESS>] (seed: False) gemini_cmd=gemini -d --duration 8h --warmup 2h -c 50 -m mixed -f --non-interactive --cql-features normal --max-mutation-retries 5 --max-mutation-retries-backoff 500ms --async-objects-stabilization-attempts 5 --async-objects-stabilization-backoff 500ms --replication-strategy "{'class': 'SimpleStrategy', 'replication_factor': '3'}" --oracle-replication-strategy "{'class': 'SimpleStrategy', 'replication_factor': '1'}" --test-cluster=<IP_ADDRESS>,<IP_ADDRESS>,<IP_ADDRESS> --outfile /gemini/gemini_result_7ac33800-9137-4a23-879e-d2de587c8215.log --seed 5 --request-timeout 180s --connect-timeout 120s --oracle-cluster=<IP_ADDRESS>

Gemini halted at 14.9M ops
eventfully SCT timeout the process, and there wasn't any output explaining the issue
Installation details
Kernel Version: 5.15.0-1028-aws
Scylla version (or git commit hash): 5.3.0~dev-20230119.aab5954cfb65 with build-id 5363e4cdfb489648423400b04906f5a8aff16578
Cluster size: 3 nodes (i3.4xlarge)
Scylla Nodes used in this run:

gemini-1tb-10h-master-oracle-db-node-f7d9883d-1 (<IP_ADDRESS> | <IP_ADDRESS>) (shards: 14)
gemini-1tb-10h-master-db-node-f7d9883d-5 (<IP_ADDRESS> | <IP_ADDRESS>) (shards: 14)
gemini-1tb-10h-master-db-node-f7d9883d-4 (<IP_ADDRESS> | <IP_ADDRESS>) (shards: 14)
gemini-1tb-10h-master-db-node-f7d9883d-3 (<IP_ADDRESS> | <IP_ADDRESS>) (shards: 14)
gemini-1tb-10h-master-db-node-f7d9883d-2 (<IP_ADDRESS> | <IP_ADDRESS>) (shards: 14)
gemini-1tb-10h-master-db-node-f7d9883d-1 (<IP_ADDRESS> | <IP_ADDRESS>) (shards: 14)

OS / Image: ami-076541a235f1dec31 (aws: us-east-1)
Test: gemini-1tb-10h
Test id: f7d9883d-60c4-47d8-8535-859f1acc65d7
Test name: scylla-master/gemini-/gemini-1tb-10h
Test config file(s):

gemini-1tb-10h.yaml

Logs and commands

Restore Monitor Stack command: $ hydra investigate show-monitor f7d9883d-60c4-47d8-8535-859f1acc65d7
Restore monitor on AWS instance using Jenkins job
Show all stored logs command: $ hydra investigate show-logs f7d9883d-60c4-47d8-8535-859f1acc65d7

Logs:

db-cluster-f7d9883d.tar.gz - https://cloudius-jenkins-test.s3.amazonaws.com/f7d9883d-60c4-47d8-8535-859f1acc65d7/20230120_161643/db-cluster-f7d9883d.tar.gz
sct-runner-f7d9883d.tar.gz - https://cloudius-jenkins-test.s3.amazonaws.com/f7d9883d-60c4-47d8-8535-859f1acc65d7/20230120_161643/sct-runner-f7d9883d.tar.gz
monitor-set-f7d9883d.tar.gz - https://cloudius-jenkins-test.s3.amazonaws.com/f7d9883d-60c4-47d8-8535-859f1acc65d7/20230120_161643/monitor-set-f7d9883d.tar.gz
loader-set-f7d9883d.tar.gz - https://cloudius-jenkins-test.s3.amazonaws.com/f7d9883d-60c4-47d8-8535-859f1acc65d7/20230120_161643/loader-set-f7d9883d.tar.gz

Jenkins job URL

This is due to the warmup took to long:
{"L":"INFO","T":"2023-01-20T04:35:50.188Z","N":"generator","M":"starting partition key generation loop"}
{"L":"INFO","T":"2023-01-20T14:35:50.150Z","N":"pump","M":"Test run stopped. Exiting."}

Defenitely some error with it that leads to warmup either being stuck or spinning indefinitely

This is due to the warmup took to long:
{"L":"INFO","T":"2023-01-20T04:35:50.188Z","N":"generator","M":"starting partition key generation loop"}
{"L":"INFO","T":"2023-01-20T14:35:50.150Z","N":"pump","M":"Test run stopped. Exiting."}

Defenitely some error with it that leads to warmup either being stuck or spinning indefinitely

There's a way to disable warmup in Gemini? In c-s we disabled them, since it's pointless with scylla that doesn't suffer from warmups like Cassandra

--warmup 0 should do the thing


Implement create process snapshot and attach

This pr is create at SELA hackathon with Sasha Goldshtein @goldshtn

To clarify this PR a bit: the Windows Process Snapshotting API provides a facility for creating a virtual address space clone of the target process. By creating a clone and then attaching the CLRMD live data reader to the clone we benefit from the best of both worlds:

The target process is not suspended at all (just like AttachMode.Passive), nor do we generate a dump file
We get a consistent snapshot of the target process (like AttachMode.NonInvasive or a dump file)

Creating the clone is a relatively cheap operation, however the cloned pages are copy-on-write. If the original process modifies a page, it is duplicated in RAM for the clone.
One final caveat is that Process Snapshotting is supported on Windows 8.1/Windows Server 2012 R2 and later.

Hi @leculver, I was wondering if we could get some feedback on this PR, or ideally get it merged. Thanks!

Overall this looks good.  I'd like to double check a few things but I plan to merge this later in the week.

@Alois-xx I don't think it's a good idea to change the default behavior of AttachToProcess to use a snapshot. Attaching in passive mode consumes virtually no resources on the target -- there is no clone, there is no extra memory usage. OTOH creating a process snapshot has the potential for using more resources. Also, using a snapshot would not support the Windows debugging interfaces, unlike AttachFlag.Invasive. All in all, I think process snapshotting warrants a separate mode -- it would be confusing otherwise.

@goldshtn: When I call ClrMD from a command line tool like MemAnalyzer I would always need to think if I can use PSS or not and pass a different command line flag to it. Procdump follows this route but what are the reasons that I would not want to use the faster approach always?
I have tried PSS years ago on Server 2012 machines which resulted in dumps which were not usable at all.  Since I want to reliably get full memory dumps I have never tried the PSS feature again. Is this now working reliably?
Yes it would be confusing if not supported functionality will not work on a newer OS because a PSS is now used. But that should be a decision of the caller of the API if he wants to use an AttachFlag  like InvasiveOrSnapshot if he can live without dbgeng support.

@Alois-xx Sorry, I disagree. It is already a decision of the API client to use the snapshot API. I don't think we should be making this choice automatically because of its potential cost and complications. And yes, PSS is more stable but the whole thing is not as thoroughly tested as CLRMD's main features, which have been around for years.

@AviAvni @goldshtn Really sorry about the delay.  I was out sick the second half of last week and wasn't able to get around to this.  Thanks for the submission.  I have a few modification requests to match the overall style of the code in ClrMD, but functionally it looks good.
@Alois-xx I do not think this should be the default for AttachToProcess with a fallback.  The user of the API should make a clear choice between using snapshots or attaching to a live process.    The LiveDataTarget can be used in places where a 'real' debugger is already attached and has the process paused (such as WinDbg/VS already attached to a process and you want to do extra inspection).  Trying to clone the process is just a waste in those scenarios.

@leculver all fixed the style issues thanks for the review

Merged.  Unfortunately the internal build sever where nuget packages are produced seems to be down, and most of CLR is out until Jan 2.  I'm going to try to put together a nuget release as soon as they are back and can take a look at why ClrMD builds aren't getting produced.  (Also hopefully adding strong name signing.)

@leculver did you publish it to nuget?

@AviAvni I pushed a new version to NuGet last week.  Unfortunately policies changed here and I had to go get re-approved for signing/shipping changes.  Sorry for the delay.
This build should contain the change:  https://www.nuget.org/packages/Microsoft.Diagnostics.Runtime/0.9.180305.1


add retrofit 2.0 basic example

Add a basic example used during Retrofit 2.0 hands-on event.

@rathodchintan please verify and merge it if it looks good :)

Can you please provide Write access to this repository.

Given, please check now!

Got it.. Thanks..

Thank you :+1:


php(-nts)-xdebug: Update to version 3.1.2-8.1

As #7526, updates xdebug to PHP8.1, since php version goes to 8.1 now.
Except this time from a branch, following feedback provided from @rashil2000, thanks!

/verify

The hash extraction seems to be broken, can you fix it?
See https://github.com/ScoopInstaller/Extras/runs/4515672536?check_suite_focus=true#step:3:221

The hash extraction seems to be broken, can you fix it?
See https://github.com/ScoopInstaller/Extras/runs/4515672536?check_suite_focus=true#step:3:221

I'll have a look, thanks.

/verify


SuperPaper won't start: OnInit returned false, exiting...

Hi, I'm running LXQt, dual screen, Void Linux, and Python 3.6.8. When I run SuperPaper in debug mode I get the following:
$ ./superpaper --debug
None
None
None
None
None
None
True
use_hotkeys: True
hkBinding_next: ('control', 'super', 'w')
hkBinding_pause: ('control', 'super', 'shift', 'p')
Traceback (most recent call last):
  File "superpaper.pyw", line 2526, in OnInit
  File "superpaper.pyw", line 2166, in __init__
  File "superpaper.pyw", line 775, in getDisplayData
  File "screeninfo/screeninfo.py", line 412, in get_monitors
  File "screeninfo/screeninfo.py", line 407, in _get_enumerator
NotImplementedError: This environment is not supported.
OnInit returned false, exiting...

The system tray briefly appears and then disappears. Thank you for creating this badly needed program!

Thanks for the debug output. Unfortunately it sounds like that the monitor data library (screeninfo) that Superpaper uses, does not know how to handle your system. Their stated supported Linux systems are:

GNU/Linux: X11 (through Xinerama)
GNU/Linux: DRM (experimental)

Does your system match either of these?
If you're unsure, to verify that this is the cause of your issue would you mind installing screeninfo and trying to get the monitor list from it? From their GitHub page: https://github.com/rr-/screeninfo
Install:
pip install screeninfo
and to get the monitor listing in Python:

from screeninfo import get_monitors
get_monitors()

Does this result in the same 'NotImplementedError: This environment is not supported' error message?

Hey, thanks for the kindly response. DRM is working just fine.
$ grep . /sys/class/drm/*/status
/sys/class/drm/card1-DP-1/status:disconnected
/sys/class/drm/card1-HDMI-A-1/status:disconnected
/sys/class/drm/card1-LVDS-1/status:connected
/sys/class/drm/card1-VGA-1/status:connected

However, I still get a NotImplementedError by screeninfo.

No problem! Thanks for verifying the cause of the issue.


Can't set up the dev environment.

Up on trying to do make run-server, I get the error as follows.
make run-server is causing issues.
11:18 PM
{"level":"info","ts":1548141418.725378,"caller":"utils/i18n.go:79","msg":"Loaded system translations for 'en' from '/Users/vineely/go/src/github.com/mattermost/mattermost-server/i18n/en.json'"}
{"level":"info","ts":1548141418.725854,"caller":"app/server_app_adapters.go:60","msg":"Server is initializing..."}
{"level":"debug","ts":1548141418.726004,"caller":"utils/html.go:31","msg":"Parsing server templates at /Users/vineely/go/src/github.com/mattermost/mattermost-server/templates"}
{"level":"info","ts":1548141418.7331588,"caller":"sqlstore/supplier.go:211","msg":"Pinging SQL master database"}
{"level":"error","ts":1548141418.7753248,"caller":"sqlstore/supplier.go:223","msg":"Failed to ping DB retrying in 10 seconds err=Error 1045: Access denied for user 'mmuser'@'localhost' (using password: YES)"}
{"level":"info","ts":1548141428.77999,"caller":"sqlstore/supplier.go:211","msg":"Pinging SQL master database"}

See also https://community-daily.mattermost.com/core/pl/36rpn6xztbrz9q9sjbfdax1foo


[Service Bus] Streaming Receiver Tests UnpartitionedSubscription failing in nightly runs

Service Bus nightly test runs are failing with:

Error message:
Expected 2, received 0 messages.: expected +0 to equal 2
Stack trace:
AssertionError: Expected 2, received 0 messages.: expected +0 to equal 2
at Object.should.equal (/Users/runner/work/1/s/common/temp/node_modules/.pnpm/chai@4.3.6/node_modules/chai/lib/chai/interface/should.js:98:35)
at testConcurrency (/Users/runner/work/1/s/sdk/servicebus/service-bus/test/internal/streamingReceiver.spec.ts:736:14)
at runNextTicks (node:internal/process/task_queues:61:5)
at processTimers (node:internal/timers:499:9)
at Context. (/Users/runner/work/1/s/sdk/servicebus/service-bus/test/internal/streamingReceiver.spec.ts:748:7)

For more details check here:

https://dev.azure.com/azure-sdk/internal/_build/results?buildId=1431689&view=results

Recent runs are green


[HIP] hipDeviceSetLimit not supported in ROCm

hipDeviceSetLimit is supported according to the Documentation but it is not defined in the /opt/rocm/bin/include
I have hipified my application and cudaDeviceSetLimit did not get converted

Thanks, it has not defined yet, but hipified.
@mangupta,  @gargrahul, Let's support hipDeviceSetLimit as long as hipDeviceGetLimit

@hpcresearch-ht, sorry for the lack of response.  Please try latest ROCm 6.0.2 (HIP 6.0.32831) to see if your issue still exists? If resolved, please close the ticket. Thanks.

Closing the ticket as it is stale. Please re-open if you still see the issue with latest ROCm 6.0.2 (HIP 6.0.32831).


Bump build tools version

This PR updates the build tools of the android project.

Merged #729 and published v6.5.0


Find all deleted files since last pull and delete them on servers

Currently deleting files will not be reflected on the server since it's only copying them over. We should store the last commit we were on upon each pull, then get the diff between that commit and the current (latest) one. Filter out only file deletions and return that list of paths to use within ansible. Python would probably be best for this.

Keepup also does this now :))


Support Poetry depenedency manager

Currently it looks like Pipenv is supported but not poetry. Is there plans to support this dependency manager as well?
Note:
Activating poetry environments do not work (at least not yet).
https://github.com/sdispater/poetry/issues/198
https://github.com/sdispater/poetry/issues/571

No specific plans at the moment, but if people begin to vote for this issue  with a 👍 reaction then we will definitely consider it.

We are going to wait until poetry hits version 1.0 before making a decision about this. People should continue to leave a 👍 reaction on the top comment until then to help us prioritize a decision and potential work compared to other feature requests.

@lig since we do not manage environment creation that will be out of scoped for this work.

We are looking at implementing this, so I want to clarify the desired behavior here: if Poetry is detected in the current environment, prompt user if they want to install packages (pylint, rope, etc) using poetry.
Is that correct? Please let us know if there's additional behavior desired here.

Till the time support for poetry gets added, I was thinking if there is any hacky way we can get through this. vscode installs any packages with following command:
<PYTHONPATH>python -m pip install -U pytest or <PYTHONPATH>python -m pip install -U pylint
Is there any way in vscode where we can map pip install to poetry add --dev?

@abhishekvm nope, we don't provide that kind of functionality. You can always do the install manually while people wait for this feature since as you noticed we aren't doing anything too magical.

Create PoetryInstaller similar to pipEnvInstaller etc (https://github.com/Microsoft/vscode-python/blob/ae1848a82f5af8983f663d5d4db2c4f77ad62c57/src/client/common/installer/pipEnvInstaller.ts)

It's great if the pybuilder dependency management is also supported.

@dclong please open a separate issue.

I was able to verify that poetry is used to install modules:

python -m pip install --user poetry
in project: python -m poetry init
in project: python -m poetry lock
workspace settings: "python.linting.flake8enabled": true
reload window

This caused the popup for installing flake8.  Then poetry was used to install flake8.


fix: add pass thru for some subdomains

This will be mostly a temporary workaround for now, to allow for some origins. We should rip this apart in our API migration to LB4.

I'm getting errors running this locally. @ojeytonwilliams mentioned that we could set HOME_LOCATION somewhere.

@scissorsneedfoodtoo if you set HOME_LOCATION='http://localhost:8000' (same as sample.env) it should be fine.

Thanks @ojeytonwilliams. This has something to do with my .env file. Checked on GitPod and everything seems to be working there.


Getting attachments with title, body and sender

Hello!
Thank you for this library, really helpful!
But anyway, my skills with promises is still low, so I want to ask.
How to get attachments with title, body and sender???
I wanted to do it myself, but I have problems with "merging all information as final value via promises".
Thanks

Do you have any code you can share? Maybe of what you have tried so far?
I'd be happy to help solve your problem.

I just depends on your example file of fetching attachments, I didn't any other changes to code!
Really thanks for this, maximum helpful for me 😊

Have you tried to solve it on your own already?

Yeah, but my promises skills is low and I doesn't how to on final return the attachments and everything in one array. Sorry for disturbing for banally issue but really I can't help for myself 😊

To combine the results of two promises into a single result, you'd use something like
Promise.all([
  promise1,
  promise2
]).then(function (results) {
  promise1Result = results[0];
  promise2Result = results[1];

  // do stuff with results
});

Alternatively, you could use a tool like Bluebird to get access to some helper functions for doing things like spreading the results into separate params in your callback instead of a single array param.

LMK if there's anything else I can help with.

@chadxz I have a problem when I saved downloaded attachments to a directory using
      const writable = fs.createWriteStream('public/resumes/' +  attachment.filename);

When i open the donwloaded attachment, I got the pics below.


Trash Dumpster

For use with the "Dumpster" Places preset

A more complicated symbol; followed some cues from the restroom symbol to optimize at small sizes.

16
24
32

Builder sizes:

12
18
24


CI - Add NPM publish

When build & tests succeed, NPM package should be published

This shouldn't be added for any commit, ONLY for commits tagged as a release


remove gitDir config option

close #271

Amazing work @camacho! Does tests cover all edge cases? I didn't see any tests where gitDir isn't process.cwd. Could you add some, please?

Those cases should be covered by the spec changes. I adjusted the old tests to account for a diff gitDir by mocking what is returned by the resolveGitDir and process.cwd methods instead of passing in the gitDir option. Those checks also including passing in the cwd option to execa if the gitDir is different than process.cwd and the bin is not git

@sudo-suhas updated based on feedback

@okonet Do you think it would be a good idea to publish this with the next tag?

@sudo-suhas why not do just breaking change? I feel like I'm not having enough time to batch changes into bigger releases and I don't want to postpone on new features from PRs.

@camacho could you please resolve the conflicts and update the branch?
Also I don't quite get your comments on tests: do you mean our tests aren't exhausting or what exactly?

@okonet sorry for not being more clear. The tests seem extensive enough. I was trying to highlight the places in the code where we test the gitDir being different than the process cwd. As far as I can tell, all cases are covered, but I am happy to add more tests if you have any cases in mind that are uncovered.

@okonet Breaking changes is a good idea. That will suffice.

@camacho one question that struck me before the merge: how this PR behaves with old configs that has the option? Will it output the deprecation message?

@okonet Good catch! I think it will print the Unknown option warning. We should update getConfig.js so that it prints the deprecation warning similar to https://www.npmjs.com/package/jest-validate#deprecation.

I've tested it and it displays a generic message right now:
● Validation Warning:

  Unknown option "gitDir" with value "." was found.
  This is probably a typing mistake. Fixing it will remove this message.

Please refer to https://github.com/okonet/lint-staged#configuration for more information...

which is confusing. Let's add a proper deprecation message, please.
See https://github.com/facebook/jest/tree/master/packages/jest-validate#deprecation

@camacho here is the patch you need to apply:
diff --git a/src/getConfig.js b/src/getConfig.js
index dc93440..fe8e684 100644
--- a/src/getConfig.js
+++ b/src/getConfig.js
@@ -30,6 +30,14 @@ const defaultConfig = {
   verbose: false
 }

+const deprecatedConfig = {
+  gitDir: config => `  Option ${chalk.bold('gitDir')} was removed.
+
+  lint-staged now automatically resolves '.git' directory.
+
+  Please remove ${chalk.bold('gitDir')} from your configuration.`
+}
+
 /**
  * Check if the config is "simple" i.e. doesn't contains any of full config keys
  *
@@ -121,6 +129,7 @@ function validateConfig(config) {

   const validation = validate(config, {
     exampleConfig,
+    deprecatedConfig,
     unknown: unknownValidationReporter,
     comment:
       'Please refer to https://github.com/okonet/lint-staged#configuration for more information...'

Requested changes pushed

@camacho could you please add a snapshot test for this?

updated

Had to amend from master and force push master :(
Lessons learned: don't merge breaking changes from the bed on the iPad 🤦♂

Thanks again for your work!

Happy to contribute @okonet

Had to amend from master and force push master :(

Good thing you caught it early.


Crash when running an edited scenario

I've got another crash on the VIP challenge. In 0.2.10a the scenario loads and runs unedited without problem, but after editing it, it crashes after applying the edits to run the simulation again. The log file: output.txt

I can't load your edits, because I think you reversed some roads around https://www.openstreetmap.org/node/53096970#map=19/47.64701/-122.31739 and one of the edit commands is now invalid. This means there are more bugs with applying edits in a stable order. Working on it.

At least one of the bugs involves stop signs where you remove all the travel lanes from one of the connecting roads. When the edits are validated, the number of roads connected to the stop signs doesn't match. Either the validation is too strict, or we need to apply ChangeRoad commands before attempting to validate the ChangeIntersection commands.


Request for Vim script/Vimscript/VimL track

What programming language should we add?
Vim's scripting language, variously known as "Vim script," "Vimscript," and "VimL."
What is the official website for the language?
Website for Vim. Vim script doesn't have its own website. Here is the chapter in the Vim documentation that introduces Vim script.
Is this a language that comes in many variants? If so, which variant should we support?
The version of Vim script in Vim isn't quite the same as the version in Neovim. The core language is consistent between the two, but there are differences in features like job control. However, Neovim hasn't yet reached 1.0 and its adoption is far below Vim's, so if it's ever an issue, we should support Vim rather than Neovim.
The real question here is which variant of Vim script's name to use: "Vim script," "Vimscript," and "VimL" all have their partisans.

"Vim script" is the name most frequently used in the documentation. GitHub uses the name "Vim script" for the repository language breakdown. A search for "Vim script" has 543 results at the moment on Stack Overflow, some of which are hits for "a Vim script."
"Vimscript" is quite commonly used. It never appears in documentation. 982 results on Stack Overflow.
"VimL" is also commonly used. It appears a couple times in documentation. GitHub uses the name "VimL" for repository cards. 232 results on Stack Overflow.

Is there a testing framework available for the language?
Yes, there are several. vader.vim and vim-vspec are probably the likeliest candidates here.
Who will be leading the effort to launch the track?
I'm happy to. I'd hope to enlist some other people to contribute as well.

@kevinlitchfield Thank you, this is great.

we should support Vim rather than Neovim

Agreed.

The real question here is which variant of Vim script's name to use

Hah. Yeah, there's a tricky one.
I would prefer to go with a name that is used by the documentation, which kicks out Vimscript.
I'm leaning towards "Vim script".
I took a look at github/linguist and it looks like the grammar is listed as "VimL" (https://github.com/github/linguist/blob/master/vendor/licenses/grammar/language-viml.txt) but the language is listed as "Vim script" (https://github.com/github/linguist/blob/master/lib/linguist/popular.yml). I can't tell if this is just inconsistent, or if it means something.

@kytrinyx Yeah, it's a tough call. "Vim script" doesn't have typical orthography for a programming language name, and its attestation in the documentation isn't overwhelming. But "VimL" isn't as widely used or, I'd guess, recognizable.
Re: the language's name in Linguist, I think it's just inconsistent. See this recent commit, which is described as changing "VimL" to "Vim script" (along with various other renamings). Not sure why there's a remaining reference to "VimL" in the grammar.
If we go with "Vim script,"  I'd suggest that the "code" version of the name be vimscript rather than vim-script or something else that represents the space in the name with some other glyph. (Unless, of course, Exercism already has an established convention for how to handle programming languages with spaces in their names. I noticed csharp and fsharp but those seem like special cases.)

I think I like vim.vader, because, well Star Trek.
Just checked for travis-ci and looks like that is covered as well.

Yeah, we don't have an established convention for spaces (this will be the first, apparently).
The commit you refer to suggests pretty hard that the name of the language is Vim script. I'm going to go with:
Name: Vim script
Slug: vimscript

[ ] Run bootstrap script
TRACK_ID=<id> LANGUAGE=<language> bin/bootstrap
[ ] Turn on Travis CI
[ ] If Windows-specific language, turn on AppVeyor
[ ] If Mac-specific language, turn on Circle CI
Add it as a Linux project, then switch it to OS X in Project Settings -> Build Environment
[ ] Add as a submodule to trackler
TRACK_ID=<id>; git submodule add https://github.com/exercism/x$TRACK_ID tracks/$TRACK_ID
[ ] Create new team for language
[ ] Add bootstrapped repository to team with write access
[ ] Invite requestor to team

Alrighty, the repo is ready for you. There's a gitter chat room where several track maintainers hang out: https://gitter.im/exercism/dev (probably other rooms, too).
The launch checklist is here:
https://github.com/exercism/xvimscript/issues/1
If you find things that are odd or confusing, holler. I may not have gotten all the bugs worked out of the bootstrapping script.
Thanks so much for getting this going :heart: :heart: :heart:

@kytrinyx Wonderful! Looking forward to getting started. 😀

Thank you much @kytrinyx .

@kevinlitchfield @kotp
FWIW, I started implementing the track recently, just in case you feel like reviewing. :]


Match the design of aria label with theme

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

VS Code Version: 1.83.0
OS Version: Windows 11

Steps to Reproduce:

move your mouse over some button on bottom panel (or over file)

This is completely different from other UI elements (should have round corner and grey border). It does not follow the theme (keep dark UI in light theme)
I hope they all follow the design below:

This may be a situation where we use both native and custom tooltips. @meganrogge would you have any insight here?

While I see the inconsistency here, I don't think it's an accessibility issue as the contrast is fine for both.

Seems like these are now using native hovers


Allow Bundle IDs that have a valid prefix

This allows iOS Extensions that have a Bundle ID of the format com.google.appName.extension

Closed in favor of #2515


Apps I have commit access to via a team do not show on My Flathub developer dashboard

I'm a member of the @flathub/endless team on GitHub, which has commit access to the manifests for Endless apps (e.g. Aqueducts) on GitHub. However, when logging into the My Flathub dashboard at beta.flathub.org, I don't see the Endless apps there. I've tried hitting the refresh button, as well.
@wjt has direct commit access to the same manifests—i.e. not just via a team membership—and sees the apps on his My Flathub dashboard.
Is it possible Flathub is only looking at direct user access and not accounting for team membership?

FWIW I worked around this by just doing the verification for those apps myself, but it think it would be nice to fix anyway!

@barthalion this appears to have been fixed by<PHONE_NUMBER>0f08aa7b7953c5b2485367755e1f83


How to use llama-index in alpaca-7b?

Can GPTListIndex be used in alpaca-7b?
I tried to use GPTListIndex in my own built model, but the log shows 0 token:

I also trie with OpenAI( and I set my OpenAI KEY),it shows the same logs:

when I query, it shows content;
pls tell me how to use llama-index in alpaca :)

building the list index doesn't consume any tokens! which is why that shows up
by the way, assuming this works, do you have an example notebook for showcasing llamaindex with alpaca? would love to feature this as a demo

@sa1utyeggs Do you have a complete example on how to do this? I see your example above but where does it get the model from? And how do you create the model using local Alpaca 7b model?

yes,I use the "Facico/Chinese-Vicuna" project, And it use the huggingface's package: transformer.
Transformer can down load the model from hugginface.com.
I succeed to use the llama-7b-hf and alpaca-7b model with GPTListIndex. But it runs slowly.
So I tried to Use GPTFaissIndex, then it failed to genereate index with my local model;
Just now, I tried to use ChatGPT to generate the faiss-index.json, then load from disk.
Unfortunately, it throw out a Exception about some node_key is not exist(sry i don't have screen shot)

finally, I trie to run it in mac silicon, and it works

It doesn't work to me. It says:
KeyError: 'llama'
When trying to download the files for the "decapoda-research/llama-7b-hf" model.
Any ideas?

@sa1utyeggs can you help me understand how you got Vicuna to run with LLAMA Index?
You just wrote a CustomLLM with HF pipeline (which pipeline, I mean text-generation or text2text or any other?) with Vicuna as the model_name?
And which Index did you build? GPTSimpleVectorIndex?
Thanks.

Also interested in running vicuna with llama-index context. Has anyone see or created a how-to guide for that? @Datta0, @sa1utyeggs ?
Thanks,


Added the sql tag for the code section to get syntax highlights

The code box for the TaxiView creation was not using the sql tag so that syntax highlights where shown.

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

Looks good; thanks.


Add deleteTag command, wrote some related tests and remove ab3 stuff

Codecov Report

Merging #84 (c3b8304) into master (8177e11) will decrease coverage by 0.49%.
The diff coverage is 61.22%.

@@             Coverage Diff              @@
##             master      #84      +/-   ##
============================================
- Coverage     74.84%   74.34%   -0.50%
- Complexity      484      493       +9
============================================
  Files            80       82       +2
  Lines          1447     1493      +46
  Branches        149      155       +6
============================================
+ Hits           1083     1110      +27
- Misses          316      333      +17
- Partials         48       50       +2

Impacted Files
Coverage Δ
Complexity Δ

src/main/java/seedu/module/MainApp.java
0.00% <0.00%> (ø)
0.00 <0.00> (ø)

...n/java/seedu/module/logic/commands/TagCommand.java
91.17% <ø> (ø)
11.00 <0.00> (ø)

...du/module/logic/parser/DeleteTagCommandParser.java
0.00% <0.00%> (ø)
0.00 <0.00> (?)

...va/seedu/module/logic/parser/ModuleBookParser.java
91.30% <0.00%> (-8.70%)
17.00 <0.00> (ø)

src/main/java/seedu/module/ui/UiManager.java
0.00% <ø> (ø)
0.00 <0.00> (ø)

.../seedu/module/logic/commands/DeleteTagCommand.java
93.33% <93.33%> (ø)
9.00 <9.00> (?)

...n/java/seedu/module/logic/commands/AddCommand.java
100.00% <100.00%> (ø)
8.00 <1.00> (ø)

src/main/java/seedu/module/model/tag/Tag.java
90.00% <100.00%> (ø)
7.00 <1.00> (ø)

... and 1 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8177e11...c3b8304. Read the comment docs.


Run Mendix on Kubernetes

Please use the form below, leaving the prefilled data to help us. Thank you.
Page link: Run Mendix on Kubernetes
Document link: run-mendix-on-kubernetes.md
My Issue/Suggestion
The documentation contains limited information on how persistence storage is attached to the app container despite it being mentioned as a topic the guide will teach you about.
"This how-to will teach you how to do the following: Attach persistence storage to the app container"
Can we include something such as the following to make this topic clearer. There doesn't seem to be any documentation how PVs work with the docker buildpack on any other pages:
"The current version of Docker Buildpack stores files in /opt/mendix/build/data/files. If a PV is mounted into this path in the app container, any uploaded files will be stored in that persistent volume.
Please note that the CF Buildpack will try to set file permissions in the volume before it starts the app, so it should have permissions to do that."
See internal discussion https://mendix.slack.com/archives/C0161BEMQ4B/p1658416780821589

Hi Elliot (@elliotclements-mendix )
Dmitrii has helped me tidy up this document and add the information about PVs.
Thanks for raising this issue - it also uncovered some additional issues so it was really helpful for that.
Please let us know if you spot other improvements which could be made to the documentation.
Mark


Does not recognize installed .net runtime

I have .net runtime installed with versions 3.5, 4, 5.0 and 6.0 but it still complains that .net is not installed.

Sorry I’m not supporting this project anymore.
Please use my latest project which has all the features and more: https://github.com/project-sbc/Power-Control-Panel-v2
Get Outlook for iOShttps://aka.ms/o0ukef

From: Marcel Ackermann @.>
Sent: Monday, October 3, 2022 3:53:40 AM
To: project-sbc/Power-Control-Panel @.>
Cc: Subscribed @.***>
Subject: [project-sbc/Power-Control-Panel] Does not recognize installed .net runtime (Issue #37)
I have .net runtime installed with versions 3.5, 4, 5.0 and 6.0 but it still complains that .net is not installed.
—
Reply to this email directly, view it on GitHubhttps://github.com/project-sbc/Power-Control-Panel/issues/37, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASZ4DJPRHMP3Y5ZLDHLYCNDWBKGIJANCNFSM6AAAAAAQ3KX67A.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

this software is not being supported anymore. find my latest here:
https://github.com/project-sbc/Power-Control-Panel-v2


Setting up the development environment

Setting up a development environment to create a website using React.

Node.js
React with Typescript
Json Sever (Only developer mode)

node.js 18.15.0 LTS
cra-redux with typescript
sass
react-router-dom
json-server


cellForItemAtIndexPath doesn't run but numberOfItemsInSection does

Hi,
I tried your solution, but cellForItemAtIndexPath doesn't run  but numberOfItemsInSection does.
What do you think is the issue?
thanks

Hmm, they should both be called as long as numberOfItemsInSection returns more than zero. Can you verify what it's returning?


glog compile error - can't execute "pod install" for iOS project

React Native version:
System:
    OS: macOS High Sierra 10.13.6
    Shell: 5.7.1 - /usr/local/bin/zsh
  Binaries:
    Node: 13.5.0 - /usr/local/bin/node
    Yarn: 1.21.1 - /usr/local/bin/yarn
    npm: 6.13.4 - /usr/local/bin/npm
    Watchman: 4.9.0 - /usr/local/bin/watchman
  SDKs:
    iOS SDK:
      Platforms: iOS 12.4, macOS 10.14, tvOS 12.4, watchOS 5.3
    Android SDK:
      API Levels: 17, 18, 19, 21, 22, 23, 24, 25, 26, 27, 28
      Build Tools: 19.1.0, 20.0.0, 21.1.2, 22.0.1, 23.0.1, 23.0.2, 23.0.3, 24.0.0, 24.0.1, 24.0.2, 24.0.3, 25.0.0, 25.0.1, 25.0.2, 25.0.3, 26.0.0, 26.0.1, 26.0.2, 26.0.3, 27.0.0, 27.0.1, 27.0.2, 27.0.3, 28.0.0, 28.0.1, 28.0.2, 28.0.3
      System Images: android-19 | Google APIs Intel x86 Atom, android-28 | Google Play Intel x86 Atom
      Android NDK: 20.1.5948944
  IDEs:
    Android Studio: 3.5 AI-191.8<IP_ADDRESS>10548
    Xcode: 10.3/10G8 - /usr/bin/xcodebuild
  npmPackages:
    react: 16.9.0 => 16.9.0
    react-native: 0.61.5 => 0.61.5
  npmGlobalPackages:
    react-native-cli: 2.0.1
    react-native: 0.61.5

Steps To Reproduce

react-native init app
cd app/ios
pod install

https://github.com/sryze/react-native-glog-compile-error
Describe what you expected to happen:
Expected the pods to install successfully.
Actual result:
Installing glog (0.3.5)
[!] /bin/bash -c
set -e
#!/bin/bash
# Copyright (c) Facebook, Inc. and its affiliates.
#
# This source code is licensed under the MIT license found in the
# LICENSE file in the root directory of this source tree.

set -e

PLATFORM_NAME="${PLATFORM_NAME:-iphoneos}"
CURRENT_ARCH="${CURRENT_ARCH}"

if [ -z "$CURRENT_ARCH" ] || [ "$CURRENT_ARCH" == "undefined_arch" ]; then
    # Xcode 10 beta sets CURRENT_ARCH to "undefined_arch", this leads to incorrect linker arg.
    # it's better to rely on platform name as fallback because architecture differs between simulator and device

    if [[ "$PLATFORM_NAME" == *"simulator"* ]]; then
        CURRENT_ARCH="x86_64"
    else
        CURRENT_ARCH="armv7"
    fi
fi

export CC="$(xcrun -find -sdk $PLATFORM_NAME cc) -arch $CURRENT_ARCH -isysroot $(xcrun -sdk $PLATFORM_NAME --show-sdk-path)"
export CXX="$CC"

# Remove automake symlink if it exists
if [ -h "test-driver" ]; then
    rm test-driver
fi

./configure --host arm-apple-darwin

# Fix build for tvOS
cat << EOF >> src/config.h

/* Add in so we have Apple Target Conditionals */
#ifdef __APPLE__
#include <TargetConditionals.h>
#include <Availability.h>
#endif

/* Special configuration for AppleTVOS */
#if TARGET_OS_TV
#undef HAVE_SYSCALL_H
#undef HAVE_SYS_SYSCALL_H
#undef OS_MACOSX
#endif

/* Special configuration for ucontext */
#undef HAVE_UCONTEXT_H
#undef PC_FROM_UCONTEXT
#if defined(__x86_64__)
#define PC_FROM_UCONTEXT uc_mcontext->__ss.__rip
#elif defined(__i386__)
#define PC_FROM_UCONTEXT uc_mcontext->__ss.__eip
#endif
EOF

# Prepare exported header include
EXPORTED_INCLUDE_DIR="exported/glog"
mkdir -p exported/glog
cp -f src/glog/log_severity.h "$EXPORTED_INCLUDE_DIR/"
cp -f src/glog/logging.h "$EXPORTED_INCLUDE_DIR/"
cp -f src/glog/raw_logging.h "$EXPORTED_INCLUDE_DIR/"
cp -f src/glog/stl_logging.h "$EXPORTED_INCLUDE_DIR/"
cp -f src/glog/vlog_is_on.h "$EXPORTED_INCLUDE_DIR/"

checking for a BSD-compatible install... /usr/local/bin/ginstall -c
checking whether build environment is sane... yes
checking for arm-apple-darwin-strip... no
checking for strip... strip
checking for a thread-safe mkdir -p... /usr/local/bin/gmkdir -p
checking for gawk... no
checking for mawk... no
checking for nawk... no
checking for awk... awk
checking whether make sets $(MAKE)... yes
checking whether make supports nested variables... yes
checking for arm-apple-darwin-gcc... /Applications/Xcode 10.3.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/cc -arch armv7 -isysroot /Applications/Xcode 10.3.app/Contents/Developer/Platforms/iPhoneOS.platform/Developer/SDKs/iPhoneOS12.4.sdk
checking whether the C compiler works... no
/Users/szx/Library/Caches/CocoaPods/Pods/External/glog/2263bd123499e5b93b5efe24871be317-1f3da/missing: Unknown `--is-lightweight' option
Try `/Users/szx/Library/Caches/CocoaPods/Pods/External/glog/2263bd123499e5b93b5efe24871be317-1f3da/missing --help' for more information
configure: WARNING: 'missing' script is too old or missing
configure: error: in `/Users/szx/Library/Caches/CocoaPods/Pods/External/glog/2263bd123499e5b93b5efe24871be317-1f3da':
configure: error: C compiler cannot create executables
See `config.log' for more details

It seems that the glog build script can't deal with spaces in Xcode path: my Xcode at /Applications/Xcode 10.3.app, after I moved it to /Applications/Xcode the build succeeded.


Christie - tell us how many assemblies from ARGOS project have been analyzed so far

Count the number of current ARGOS assemblies.
Further task:
Create a computational file that can count the number for us.

Has been done and will be updated as we continue QC assembly work. https://docs.google.com/document/d/1SQw2gYr6Yli376raKq4tm--IG0hIIRWHIeY9ShoukMw/edit


Update remote-fixtures, remote-robot to 0.11.16

Updates

com.intellij.remoterobot:remote-fixtures
com.intellij.remoterobot:remote-robot

from 0.11.15 to 0.11.16.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.intellij.remoterobot" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "com.intellij.remoterobot" }
}]

labels: library-update, early-semver-minor, semver-spec-patch, commit-count:1

Superseded by #342.


improve reduce default load stream per node to 2 for stream load #34065

cherry pick from #34065

run buildall


SetWriteDeadline "not implemented" panic

Hi,
We're using an older version of this in an AWS lambda and just today we started getting "not implemented" panics when closing DB connections.  Nothing was changed in our code.
I can see these "not implemented" panics have been removed in the latest version, so is there any reason not to upgrade?
Would you have any clue why this would just start happening now (also, they don't happen on all our environments, but do happen in production, which is using RDS etc.)
It's true we deployed new binary builds of our lambdas recently, but we certainly didn't change any source code.
Thanks!

Have seen #639?

Actually I did, but just skimmed through it, on re-reading it in more detail it makes sense now - it's very likely the go compiler got updated on our build servers since our last version was built.
Just annoying we only spotted it in production (but thankfully it didn't result in a customer-facing error).

Update your go.mod and make sure to use this version:
github.com/denisenkom/go-mssqldb v0.0.0-20210411162248-d9abbec934ba

Yeah have done so, seems fine.  Interestingly in AWS it only crashes when it's using RDS, vs an ec2 instance running SQL server (our non-profit set-up).

@wizofaus FWIW it also crashed with Azure SQL Server -- couldn't quickly verify whether it crashed with the Docker images because they won't run on M1 Macs.


Sliders display as floats in csv

In the app the slider display only shows integers, but once you export to a CSV the value is a double/float.

This is fixed in the next version.


The getTranslations and useTranslations methods will fail when the server component sets dynamic to force-static

Description
I used the "export const dynamic = "force-static"; " syntax in the server component to make the cache-control attribute of the header set in the middleware take effect. Unfortunately, at this time, using the "const t = await getTranslations();" or "const t= useTranslations();" methods in page.tsx will fail and the page will return a not found 404 page.
"next": "14.2.1",
"next-intl": "^3.11.1",

App Router with src dir
Verifications

[X] I've verified that the problem I'm experiencing isn't covered in the docs.
[X] I've searched for similar, existing issues on GitHub and Stack Overflow.
[X] I've compared my app to a working example to look for differences.

Mandatory reproduction URL
App Router
Reproduction description
Steps to reproduce:

Open reproduction
Click on …
See error: …
no more

Expected behaviour
Work well

force-static is not intended to be supported, this section should be linked to from an error you're probably seeing in your app. To implement static rendering properly, please follow the static rendering guide (theoretically, after you've set this up, force-static might even work).

to make the cache-control attribute of the header set in the middleware take effect

Is that actually the case? That's quite interesting, I wonder if that behavior is intended.

force-static is not intended to be supported, this section should be linked to from an error you're probably seeing in your app. To implement static rendering properly, please follow the static rendering guide (theoretically, after you've set this up, force-static might even work).

to make the cache-control attribute of the header set in the middleware take effect

Is that actually the case? That's quite interesting, I wonder if that behavior is intended.

The problem has been solved with your help, thank you very much.


feat: My Collection form location autocomplete

🔴 Do not merge before https://github.com/artsy/gravity/pull/17722 has run
The type of this PR is: Feature

This PR solves ONYX-860

Eigen PR: https://github.com/artsy/eigen/pull/10245
Gravity PR: https://github.com/artsy/gravity/pull/17680

Description
This PR adds autocomplete capability to the My Collection artwork form location input.
Screenshots
I couldn't get the auto-suggestions dropdown to work locally, so the screenshot doesn't show the dropdown.

Do you mind if you also add a short screen recording to show this in action, it might be more clear that way than using a screenshot

I would like to, but I couldn't get the dropdown working locally in a reasonable amount of time... :/ But it's working on Staging

thanks for explaining - no worries, I can try it out on staging.

thanks for explaining - no worries, I can try it out on staging.

It works like the location input in https://staging.artsy.net/settings/edit-profile


DLP Template issue - MPARR2

Hi, I have an issue with the DLPALL Data Details function on the template "MPARR - DLP General Overview Dic2023.pbit".

The error is "Expression.Error: The parameter is expected to be of type Text.Type or Binary.Type"

It appears to be this line which is causing the fest of the function to fail
" #"Parsed JSON" = Table.TransformColumns(#"Removed Other Columns",{{"PolicyDetails_s", Json.Document}}),"

Can you please go to the query DLPAll Data and show the column PolicyDetails_s, appears that the problem is with the data located in that column.

Please let me know if you find anything else on this.


member add command destroys group to group mappings

Hi,
I have a group that has a member - another group.
If I use the member add command to add a user the group to group mapping is lost.
Tested with UAA client 3.5.0

See https://cloudfoundry.slack.com/archives/C03FXANBV/p1493927873369874 for a workaround until bug is fixed.

This is fixed in the latest version of UAAC (3.13.1).


StableNormal + 2DGS

Hi, thanks for your great work!
Do you release the baseline method that uses stable normal to enhance the 2DGS?

Please keep patient! We are testing our model capacity, aiming at releasing more stable, accurate reconstruction pipeline.

@zhirui-gao This appears to be a duplicate of #15. Please refer to that issue for updates on the code of baseline methods using StableNormal with 2DGS. I'll close this issue to keep discussions consolidated. Please feel free to follow and participate in #15 for updates.


IMAGEDAM-223: TIFF images are visible and useable in BBC Images

What does this change?
It changes how tiffs files are handled. Before this update, only the transformed JPEG was kept, now the system keeps the transformed JPEG for cropping and other optimization but also keeps the original TIFF file. The TIFF file information is kept on "originalSource". Only non supported files, i.e. not browser friendly, like TIFF, will have this field. The correct file size of TIFF files are now being shown.
It adds a new filter for searching for specific file types (fileType). Now the user can select on the filter, "fileType" and put a
MIME or some alias (tiff, jpg, png).
It bumps the image_filter_buffer of the imgops nginx.conf file to 200M. This was needed becauses large TIFFs files are transformed into big JPEGs with the current setting of transformation(same resolution, quality set to 100).
How can success be measured?

Upload a TIFF with metadata
See if the same metadata appears on the screen
Search for TIFF files using the +fileType
Crop some TIFF files successfully
Download TIFF with fullsize (default download button)
Do the same tests with some large TIFFs (I have tested up to 150MB)

Screenshots (if applicable)

Who should look at this?
Tested?

[x] locally
[ ] on TEST

Hello! Just a tiny comment  about +mimeType. Could I suggest using +fileType instead as a more user-friendly and understandable name? (great to have this as a search!) Will the types values be offered as suggestions, too, similarly to eg.

Regarding

now the system keeps the transformed JPEG for cropping and other optimization but also keeps the original TIFF file

in our implementation, cropping always uses an original file (preferable, I think). Prerenders are used only as smaller intermediaries to send to imgOps (for display). They always had problems: we created them as PNG8 for all TIFFs irrespective if they contained transparency or not. This was never ideal (should be JPEGs for non-transparent imagery). But I cannot understand code, so will ask some colleagues around to explain to me how this PR changes things.
Great to see this! 🎉

Hi @gribeiro - just to let you know that both @paperboyo and I have dug into the behaviour of the Grid. The expectation has always been that the TIFF would be preserved as the original file, and would indeed be supplied when downloaded.
Unfortunately we introduced an inadvertent change in April this year that broke this behaviour. We'll look to fix this ourselves in the coming days but suffice to say that there is no need to introduce a secondary location to store the original file.

I'll must also add that the ability to search by file type is awesome 🎉

Hi @gribeiro - just to let you know that both @paperboyo and I have dug into the behaviour of the Grid. The expectation has always been that the TIFF would be preserved as the original file, and would indeed be supplied when downloaded.
Unfortunately we introduced an inadvertent change in April this year that broke this behaviour. We'll look to fix this ourselves in the coming days but suffice to say that there is no need to introduce a secondary location to store the original file.

Hi @sihil, thank you for your input. I thought that was odd that the original was thrown out. I'm glad that it was not intended design. I agree that only one TIFF should be stored on the system. Having said that, having a setting for storing a intermediate representation could be useful for future formats that the conversion is (way) more expensive, so the cropping and other operations could benefit from the optimized storage, and this storage just happens for files types that we require on the configuration. So we can easily turn this off for TIFFs in the near future and use your solution.

storing a intermediate representation could be useful for future formats that the conversion is (way) more expensive, so the cropping and other operations could benefit from the optimized storage

Cropping should always be done from the original image IMHO, if only because creating an intermediary takes as much time as cropping anyway but only applies if you actually crop (most pictures are unused, so – uncropped; creating intermediaries for all of them would be costly).
There is already a concept of optimisedFile. It’s used mostly for display purposes where the imgOps request would otherwise take ages. Logic when and how to create it (and how it’s named) is a bit broken, but we could and should improve on it. For some notes, see here and further below there. We will possibly take a wee look at some of it too as part of the aforementioned fix 🤞.

creating intermediaries for all of them would be costly).

Yes, creating for all would be costly, knowing that most are unused. But this is already on the codebase, it was not introduced by this PR, the feature introduced in this PR regarding the conversion is: when there are some images that are being converted to some intermediate format — for some reason that I’m not aware of — then store the original. I have to dig the past commits to see why this was conversion to tiff->jpeg implemented in the first place, but it was not the current intended scope to remove this conversion, but rather have also access to the original file.
Now, knowing that a fix is coming we can expand the scope on a future task and remove this past tiff->jpeg conversion.
Thank you for all the enlightenment 😅

@gribeiro - The intention is that at ingest time we: store the original unmodified file, a thumbnail to facilitate browsing and an optimised file (where necessary, e.g. TIFFs) for improving performance for cropping in the display (note that the crop is still generated from the original TIFF, but the user sees the proxy).
We've agreed to prioritise fixing this and will follow up here when we have PRs. I expect that we will have an upstream fix (and some improved tests) for this by the end of next week at the very latest and hopefully earlier.

Hey @paperboyo, @sihil
Thank you guys for all the explaining on the intended behavior of this process. Now I can see more clearly.
I changed the PR with a solution for https://github.com/guardian/grid/issues/3049 and removed our previous solution.


[Doc Fix][Video link fix]

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/webhooks-and-connectors/how-to/add-incoming-webhook.md
93
5
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Learn Build status updates of commit 611bcc6:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/webhooks-and-connectors/how-to/add-incoming-webhook.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit 5e0c57c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/webhooks-and-connectors/how-to/add-incoming-webhook.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/webhooks-and-connectors/how-to/add-incoming-webhook.md
93
5
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Learn Build status updates of commit 9de3506:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/webhooks-and-connectors/how-to/add-incoming-webhook.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Travis: add helpers for brew install and upgrade

brew install a formula if it is not installed. If it is, brew upgrade if it's outdated.

Line 44, is there any issue if/when brew upgrade returns an error?

Line 44, is there any issue if/when brew upgrade returns an error?

True, it's not necessary to the build so we could add || true just to be safe. Figured it couldn't hurt to be up-to-date, since I imagine most Homebrew users brew update/upgrade fairly frequently.

True, it's not necessary to the build so we could add || true just to be safe. Figured it couldn't hurt to be up-to-date, since I imagine most Homebrew users brew update/upgrade fairly frequently.


podman ps --pod <POD_NAME> doesn't work

Is this a BUG REPORT or FEATURE REQUEST? (leave only one on its own line)
/kind bug
Description
When I execute podman ps --pod my_pod it returns a obviously wrong and also unexpected error message: Error: `podman ps` takes no arguments
Steps to reproduce the issue:

podman create pod --name my_pod
podman ps --pod my_pod

Describe the results you received:
Error: `podman ps` takes no arguments
Describe the results you expected:
An empty list with an exit code of zero
Additional information you deem important (e.g. issue happens only occasionally):
None.
Output of podman version:
Version:            1.6.3
RemoteAPI Version:  1
Go Version:         go1.13.4
Git Commit:         9d087f6a766259ba53b224944f1b7b778035c370-dirty
Built:              Sat Nov 23 12:56:15 2019
OS/Arch:            linux/amd64

Output of podman info --debug:
debug:
  compiler: gc
  git commit: 9d087f6a766259ba53b224944f1b7b778035c370-dirty
  go version: go1.13.4
  podman version: 1.6.3
host:
  BuildahVersion: 1.12.0-dev
  CgroupVersion: v1
  Conmon:
    package: Unknown
    path: /usr/bin/conmon
    version: 'conmon version 2.0.3, commit: eb5fa88c26fde5ce1e3f8a1d2a8a9498b2d7dbe6'
  Distribution:
    distribution: arch
    version: unknown
  IDMappings:
    gidmap:
    - container_id: 0
      host_id: 1000
      size: 1
    - container_id: 1
      host_id: 100000
      size: 65536
    uidmap:
    - container_id: 0
      host_id: 1000
      size: 1
    - container_id: 1
      host_id: 100000
      size: 65536
  MemFree:<PHONE_NUMBER>
  MemTotal:<PHONE_NUMBER>
  OCIRuntime:
    name: runc
    package: Unknown
    path: /usr/bin/runc
    version: |-
      runc version 1.0.0-rc9
      commit: d736ef14f0288d6993a1845745d6756cfc9ddd5a
      spec: 1.0.1-dev
  SwapFree:<PHONE_NUMBER>
  SwapTotal:<PHONE_NUMBER>
  arch: amd64
  cpus: 4
  eventlogger: journald
  hostname: carbon
  kernel: 5.4.1-arch1-1
  os: linux
  rootless: true
  slirp4netns:
    Executable: /usr/bin/slirp4netns
    Package: Unknown
    Version: |-
      slirp4netns version 0.4.2
      commit: 69153b0d1cb82216d6782179ff7c3c5e91c731a1
  uptime: 3h 24m 34.71s (Approximately 0.12 days)
registries:
  blocked: null
  insecure: null
  search:
  - docker.io
  - registry.fedoraproject.org
  - quay.io
  - registry.access.redhat.com
  - registry.centos.org
store:
  ConfigFile: /home/tino/.config/containers/storage.conf
  ContainerStore:
    number: 38
  GraphDriverName: overlay
  GraphOptions:
    overlay.mount_program:
      Executable: /usr/bin/fuse-overlayfs
      Package: Unknown
      Version: |-
        fusermount3 version: 3.8.0
        fuse-overlayfs: version 0.7.2
        FUSE library version 3.8.0
        using FUSE kernel interface version 7.31
  GraphRoot: /home/tino/.local/share/containers/storage
  GraphStatus:
    Backing Filesystem: extfs
    Native Overlay Diff: "false"
    Supports d_type: "true"
    Using metacopy: "false"
  ImageStore:
    number: 133
  RunRoot: /run/user/1000
  VolumePath: /home/tino/.local/share/containers/storage/volumes

Package info (e.g. output of rpm -q podman or apt list podman):
> pacman -Q podman
podman 1.6.3-1

Additional environment details (AWS, VirtualBox, physical, etc.):
Tested on a up to date arch system

Sorry, I totally missed the purpuse of that flag -.-


perl configdata.pm --dump

What's the question?

What's the question?

Error on mac os when trying to install ruby (specfic command asdf install ruby 3.0.3)
Installing openssl-1.1.1...
Build Failed (MacOS 12.2.1 using ruby-build 20220218)

No error message?
I suspect this question would be better asked of the maintainers of the tool you are using to install (asdf?).


Path Interaction Between show-env, nextest, and report

I'm not quite sure what exactly is happening here, but here is the general flow I'm following:
## Build Machine Job 1 ##
source <(cargo llvm-cov show-env --export-prefix --no-cfg-coverage)
cargo build --bin wgpu-info --target ${{ matrix.target }}

## Build Machine Job 2 (env is reset) ##
cargo llvm-cov nextest-archive --all-features --no-cfg-coverage --archive-file tests.tar.zst --target ${{ matrix.target }}

## Transfer Binaries, Archive, etc ##

## Test Machine Job 1 ##
    # This has LLVM_PROFILE_FILE point to <snip>/target/wgpu-%p-%20m.profraw
source <(cargo llvm-cov show-env --export-prefix --no-cfg-coverage)

    # This command puts profraws in `target/` due to the above
./wgpu-info

    # This seems to expect profraws in `target/debug`, and hence finds nothing and errors.
cargo llvm-cov report --target ${{ matrix.target }} --lcov --output-path wgpu-info-lcov.info

## Test Machine Job 2 ##
    # These all work fine now!
cargo llvm-cov clean
cargo llvm-cov --output-path tests-lcov.info nextest --archive-file tests.tar.zst --extract-overwrite  --target ${{ matrix.target }}

The real CI config I'm running: build machine test machine
Aside
In my ideal world, I would be able to combine both the profraws from both the nextest and the wgpu-info run into a single lcov profile. I can do this if I'm running locally, but I don't think report would pick up on the nextest profraws.
I can live with two profraws - codecov lets you upload multiple files - would be nice though :)

# This seems to expect profraws in `target/debug`, and hence finds nothing and errors.

It expects both profraws and objects (binaries) to be in the target directory. Try putting the binary in the directory the same as the original location.

So I tried two different permutations:
Two Reports
source <(cargo llvm-cov show-env --export-prefix --no-cfg-coverage)

# This needs to match the command in xtask/tests.rs
chmod +x ./target/${{ matrix.target }}/debug/wgpu-info${{ matrix.suffix }}
./target/${{ matrix.target }}/debug/wgpu-info${{ matrix.suffix }} -vv
./target/${{ matrix.target }}/debug/wgpu-info${{ matrix.suffix }} --json -o .gpuconfig

cargo llvm-cov report --target ${{ matrix.target }} --lcov --output-path wgpu-info-lcov.info

## Job 2

cargo llvm-cov --lcov --output-path tests-lcov.info nextest --archive-file tests.tar.zst --extract-overwrite --no-fail-fast --retries 2 --target ${{ matrix.target }}

This worked fine for the windows and linux jobs, but the llvm-cov report returned the following on arm mac, after cross compiling
info: when --target option is used, coverage for proc-macro and build script will not be displayed because cargo does not pass RUSTFLAGS to them
warning: not found *.profraw files in /Users/runner/work/wgpu/wgpu/target/llvm-cov-target; this may occur if target directory is accidentally cleared, or running report subcommand without running any tests or binaries
error: no input files specified. See llvm-profdata merge -help
error: failed to merge profile data: process didn't exit successfully: `/Users/runner/.rustup/toolchains/1.71-aarch64-apple-darwin/lib/rustlib/aarch64-apple-darwin/bin/llvm-profdata merge -sparse -f /Users/runner/work/wgpu/wgpu/target/llvm-cov-target/wgpu-profraw-list -o /Users/runner/work/wgpu/wgpu/target/llvm-cov-target/wgpu.profdata` (exit status: 1)
Error: Process completed with exit code 1.
```

Full log [here](https://github.com/gfx-rs/wgpu/actions/runs/7514956188/job/20458449093?pr=5056)

## One Report

I also tried combining the nextest and info reports into one report by passing --no-report then calling

```
cargo llvm-cov report --target ${{ matrix.target }} --lcov --output-path lcov.info
```

This fails with the same error on all platforms

```
info: when --target option is used, coverage for proc-macro and build script will not be displayed because cargo does not pass RUSTFLAGS to them
warning: not found object files (searched directories: D:\a\wgpu\wgpu\target\llvm-cov-target\x86_64-pc-windows-msvc\debug); this may occur if show-env subcommand is used incorrectly (see docs or other warnings), or unsupported commands such as nextest archive are used
error: failed to generate report: process didn't exit successfully: `'C:\Users\runneradmin\.rustup\toolchains\1.71-x86_64-pc-windows-msvc\lib\rustlib\x86_64-pc-windows-msvc\bin\llvm-cov.exe' export -format=lcov '-instr-profile=D:\a\wgpu\wgpu\target\llvm-cov-target\wgpu.profdata' -ignore-filename-regex '\\rustc\\([0-9a-f]+|[0-9]+\.[0-9]+\.[0-9]+)\\|^D:\\a\\wgpu\\wgpu(\\.*)?\\(tests|examples|benches)\\|^D:\\a\\wgpu\\wgpu\\target\\llvm\-cov\-target($|\\)|^C:\\Users\\runneradmin\\\.cargo\\(registry|git)\\|^C:\\Users\\runneradmin\\\.rustup\\toolchains($|\\)'` (exit code: 1)
--- stderr
No filenames specified!
```

As for the issue about cargo llvm-cov report, I think the option added in https://github.com/taiki-e/cargo-llvm-cov/pull/355 is needed.
The other may have been solved in https://github.com/taiki-e/cargo-llvm-cov/pull/349.


Content cleaner could remove blank lines

As it is normalizing content it could also handle this easy thing

done


Layout broken in commit 11824e0e

Unfortunately my iPhone doesn't support 13 yet. So I build the project for iOS12. Which the latest commit compatible to that version is 11824e0e.
But I got this issue.

Is this worth to fix 😄? Can I make a PR for this?
Thanks in advance.

Hi @muizidn, so sorry for the long delay in getting back to you. Sure, I'd accept a pull request for those who want to build the project to a device which doesn't support iOS 13. I've created a branch called compatibility/ios-12, feel free to make a PR to that if you'd like. Unfortunately I don't think I'll be able to maintain this branch (won't get any bug fixes) but would be happy to have it around in case anyone wants to build the app in a working state for iOS 12. This is actually my fault for not bumping the version at the correct time.
Thanks!

Going to close this issue since I think the branch in the above comment should address the concern here. Feel free to reach out if I can help any more with this use case!


Complete NTRs & edits required for mapping of Gut Cell Atlas terms

[x] https://github.com/obophenotype/cell-ontology/issues/2580

[ ]  Enterocyte - Late (APOA1, APOC3, APOB, GUCA2A, CDHR5)

[x]  https://github.com/obophenotype/cell-ontology/pull/2465
`

[x]  https://github.com/obophenotype/cell-ontology/issues/2562 -  -Not added anymore -  the same as the existing term intestinal crypt stem call term

[ ]  https://github.com/obophenotype/cell-ontology/issues/2563

[x]  https://github.com/obophenotype/cell-ontology/issues/2476 This is a specialised, fenestrated endothelial cell - check with experts whether this is appropriate mapping for their capillary cell.

[x] https://github.com/obophenotype/cell-ontology/issues/2576

[ ]  https://github.com/obophenotype/cell-ontology/issues/2564

[x] Subepithelial fibroblasts and subclasses

[x] https://github.com/obophenotype/cell-ontology/issues/2586

[x] https://github.com/obophenotype/cell-ontology/issues/2587 - ticket is made but waiting for an uberon term follicle associated epithelium

[x] https://github.com/obophenotype/cell-ontology/issues/2586

[ ]  https://github.com/obophenotype/cell-ontology/issues/2565

[ ] https://github.com/obophenotype/cell-ontology/issues/2558

[x] https://github.com/obophenotype/cell-ontology/issues/2578

[x] https://github.com/obophenotype/cell-ontology/issues/2517

[x] Immature Pericyte https://github.com/obophenotype/cell-ontology/issues/2503

[x] Review mapping sheet for missing mappings

(almost all done - might be a few very recently added terms to map)

[x] Check for all cases where there is a validated CellGuide description

@Caroline-99 - work on simple script to lookup and report -  matched_validated_descriptions_gut_atlas

[x]  Review validated CellGuide descriptions for addition

check no contradiction with CL def

[ ]  Run GCA hierarchy through validator

(https://github.com/hubmapconsortium/validation-template) - requires reformatting. Try using ChatGPT to reformat. <-- stretch goal

Following cell terms removed as they are not in the updated version of the GCA hierarchy

[ ] Immune Organizing Fibroblast (mLN stroma)
[ ] Crypt Support fibroblast #2558
[ ] Villus Support fibroblast
[ ] Immune Organizing Fibroblast (mLN stroma)

Following cell terms added from updated GCA hierarchy:

[ ]Subepithelial fibroblasts (S2)
[ ]Lamina propria fibroblasts (S1)
[ ]Submucosal Fibroblasts (aka Crypt Support/Trophocytes/S3)
[ ]Fibroblastic reticular cells

There are 69 terms with a validated CellGuide description: matched_validated_descriptions_gut_atlas

Some of the cells are restricted in the intestinal compartment but contain the term 'gut' either in the label, the definition or both. The gut comprehends more structures than the intestine, such as the stomach or the esophagus, and the presence of intestinal cells in there might indicate pathology (e.g., Barret's esophagus). For example, transit amplifying cell of gut should be intestinal transit amplifying cell.
CC @emquardokus


"trace" like component?

I use this all the time. it's just for dev, so maybe something that can be stripped in prod. but seems like a useful addition here. thoughts?
(non-recompose simple version)
export default C => p => (
  console.info(
    `%ctracing ${C.displayName} props:\n==========================`,
    'color: rgb(22, 138, 96);font-weight: bold;',
  ), // eslint-disable-line
  console.log(p), // eslint-disable-line
  <C {...p} />
);

This is what I ended up with locally at the end of the day
/**
 *  Utility HOC designed to easily log received props.
*/

/* Recommended to be put on the global namespace in development mode:

global.trace = process.env.NODE_ENV === 'production'
  ? require(<path_to_trace>).noop
  : require(<path_to_trace>).default;

*/

/* usage:

  global.trace('Trying to fix issue #12')(Button)

  compose(
    global.trace() // string argument is optional.
  )

*/

export default (additionalMessage = '') => Component => props => (
  additionalMessage && console.log(additionalMessage),
  console.info(
    `%ctracing ${Component.displayName} props:\n${'tracing '.replace(
      /./g,
      '=',
    )}${Component.displayName.replace(/./g, '=')}=${'props:'.replace(
      /./g,
      '=',
    )}`,
    'color: rgb(22, 138, 96);font-weight: bold;',
  ),
  console.log(props),
  <Component {...props} />
);

export const noop = () => C => p => <C {...p} />;

Seems pretty interesting to me.

comment from colleague: "move noop branching choice inside the component via branch" which is a good idea. going to add it later
(additionalMessage = '') => branch(
  process.env.NODE_ENV === 'production',
  renderComponent(Component),
  Component => props => (/*...*/)
)

global.trace = require('@ncigdc/utils/trace');

This feature is quire useful sometime, but I think consumers may want use their own ways to log. I suggest we provide a generalized HOC for all side-effects, like what Ramda's tap() or RxJS's do() does.


Lai panics with: invalid variable type 2 in lai_write_filed on real hw

Closing because it is thesame as #88


Fix Python dtype conversion for int64 on Windows.

Fixes https://github.com/openxla/iree/issues/11080. The int64 and uint64 test cases here were failing on Windows as the element type mapping was routing via the code l, which is a "C long int" - not an explicitly 64 bit type. This changes the mapping to always use the explicit "type strings" (any string in numpy.sctypeDict.keys(), shown in this gist).
Relates to https://github.com/openxla/iree/pull/12872

Nice catch! Do we tests in place to fail if we regress? (I assume the test newly enabled on Windows)

Yeah, the newly enabled test covers this. We could still use some pure-python (no TF / other frameworks) tests that go through a more complete workflow with each data type though.


perf: Decrease bundle size for largest assets > 100kb

Description 📝
I'm going to be making incremental improvements to our bundle size starting with lodash.
Major Changes

We were importing the whole Lodash library when all we needed were a few methods

Preview 📷

Before
After

Testing

Run npx vite-bundle-visualizer

Nice. One thing about lodash is that we don't actually specify it in manager's package.json but we really should because we import it. I think the reason we haven't seen issues is because some other packages probably depend on it, therefore it is installed without us directly specify it. This problem can be addressed in this PR or we can worry about it another time.

Yea that's true, I'll do it in this PR 👍


larval vesA-C neurons aren't homologous between thoracic and abdominal segments

abdominal vesA-C innervate campaniform sensilla, but thoracic vesA-E innervate the Keilin's organ. Corresponding campaniform sensilla in thoracic segments are innervated by differently-named neurons.
It doesn't make sense to have abdominal and thoracic classes grouped under parent vesA, vesB and vesC classes (since the children are not the closest relatives of their sibling classes. These classes should be obsoleted, with 'larval segmental nerve sensory neuron' as suggested replacement (common superclass for ventral cluster neurons).

similar issue with v'esA and v'esB


is it possible to use cuda

Hi,
is it possible to use cuda for inference?

It is possible : refer to the GGML library.

yeah i know ggml supports cuda, what i mean is do you have any plan to add options for cuda?

I can add it if that's something you need. One issue is that i do not have a local GPU so it will be hard to test. would you like to test? you can also open a PR if you like!


Add bounding box/visualization output to nodes

Certain models return coordinates for displaying bounding boxes or other visualization options.
Models that provide this kind of information in their response should have the option to include an augmented image in the output from the model node, like is currently done with the object-detector node.
The nodes that still need this output are:

[x] facial age estimator
[ ] facial emotion classifier
[x] facial recognizer
[ ] image segmenter
[ ] human pose estimator

closed by #51


texture.bow_pulling_{0,1,2} not implemented/broken

Skipping sub cit: No such sub item model "bow_pulling_0" in Squishy's+Enchanted+Items.zip -> optifine/cit/0_bow/properties/flame.properties
Bow pulling item textures are throwing errors in this resource pack:
https://www.curseforge.com/minecraft/texture-packs/squishys-enchanted-bows

Can you provide the full logs please?

https://pastebin.com/9d8DF8jc

Running it in a clean setup with just that resourcepack running it works fine, I think I know what causes this and I already made #19 because of that. I'm closing this issue because a sub item rewrite is already a thing I'm working on. You might want to enable notifications for #19.