Corner case for which higher kinded equality does not work properly

While working on paradoxes/InjectiveTypeFormers.fst, I've discovered a corner case in which higher kinded equality does not work properly. The following slightly simplified code should type-check, but it doesn't:
// even if the definition of P shouldn't matter below, it actually does
// (everything works if we assume P or if we define it in a simpler way)
type P (a:Type) = (cexists (fun (x:a) -> ctrue))

// all this works fine if we change to h2':(P p -> Tot unit)
val aux : p:Type -> h:P p -> a:(Type->Type) -> h2':(a p -> Tot unit) ->
            Pure unit (requires (a == P)) (ensures (fun _ -> True))
let aux (p:Type) (h:P p) (a:(Type->Type)) h2' = h2' h
// bug351b.fst(16,37-16,38): Subtyping check failed;
// expected type (a p); got type (P p)

Experimented quite a bit, but didn't manage to find a simpler counterexample illustrating the problem. Higher kinded equality works well in all other cases I've tried.

The shrunk counterexample works now, but the original problem in inj.fst is still there.

The example in 351.fst works fine.
The example in inj.jst does not complete because of the below error.
.\examples/paradoxes/inj.fst(13,5-13,6) : Error
Invalid identifer 'I'; expected a symbol that begins with a lower-case character

@catalin-hritcu  can you verify ?

@vkanne-msft: that's an unrelated error due to a syntax change.
Reproducing this requires a means to denote universes beyond Type0, which is still pending in #505.
See also #349 for the discussion on injectivity of type constructors.

After porting inj.fst to universe syntaxt (and renaming it to InjectiveTypeFormers.fst), I think I'm hitting the error @s-zanella was talking about:
[hritcu@detained paradoxes]$ fstar.exe InjectiveTypeFormers.fst
./InjectiveTypeFormers.fst(14,2-14,4) : Error
Expected expression of type "(_:Type -> Type)";
got expression "f" of type "(_:Type -> Type)"

@s-zanella @kyoDralliam: Kenji has just implemented explicit universe syntax, so we should be able to reproduce this now? https://github.com/FStarLang/FStar/wiki/Explicit-universes

InjectiveTypeFormers.Explicit.fst typechecks and we're able to demonstrate that assuming injectivity (via an axiom) leads to a contradiction.


Many errors running it locally.

Describe the bug
npx unlighthouse --site unlighthouse.dev --debug
(node:27916) UnhandledPromiseRejectionWarning: SyntaxError: Unexpected token '??='
at Loader.moduleStrategy (internal/modules/esm/translators.js:145:18)
(Use node --trace-warnings ... to show where the warning was created)
(node:27916) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag --unhandled-rejections=strict (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
(node:27916) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
npx unlighthouse --site keysaver.app
(node:66616) UnhandledPromiseRejectionWarning: SyntaxError: Unexpected token '??='
at Loader.moduleStrategy (internal/modules/esm/translators.js:145:18)
(Use node --trace-warnings ... to show where the warning was created)
(node:66616) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag --unhandled-rejections=strict (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
(node:66616) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
Reproduction
No response
System / Nuxt Info
System:
    OS: Windows 10 10.0.23606
    CPU: (16) x64 AMD Ryzen 9 5900HX with Radeon Graphics
    Memory: 1.41 GB / 15.42 GB
  Binaries:
    Node: 14.18.0 - C:\Program Files\nodejs\node.EXE
    npm: 9.7.2 - ~\AppData\Roaming\npm\npm.CMD
  Browsers:
    Edge: Chromium (121.0.2277.4), ChromiumDev (121.0.2277.4)
    Internet Explorer: 11.0.23606.1000

node --trace-warnings ...
internal/modules/cjs/loader.js:905
throw err;
^
Error: Cannot find module 'C:\Users\aylac\OneDrive - ModMana\Desktop\KeySaver\M.M_Key_Saver...'
at Function.Module._resolveFilename (internal/modules/cjs/loader.js:902:15)
at Function.Module._load (internal/modules/cjs/loader.js:746:27)
at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:76:12)
at internal/main/run_main_module.js:17:47 {
code: 'MODULE_NOT_FOUND',
requireStack: []

Hi, node 14 is not supported. I'd recommend node 18

@harlan-zw Readme is outdated as it states Node >= 14.x.


Where to store additional configuration information?

where should I store additional details related to API's like developer contact etc? Can I use same database that Kong is configured to use or use a different database? What are the pros and cons with each approach?
Thank you.

I think you could use the same database, biggest issue I can think of is migrating to newer versions or migrating to a new cluster or something like that.
Another database is cleaner, but requires more management

In the future, we'll likely provide a mechanism to tie metadata into consumer (and other) objects in the admin API; for now, as @Tieske mentioned you could manually use either the existing, or a separate, data store to manage this metadata.
Feel free also to continue this discussion on the mailing list, or Gitter. Thank you!

Thank you @p0pr0ck5 . Is this on the road map or is this something that you want to do at some point of time?


New version: Microsoft.DevHome version <IP_ADDRESS>

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.4 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/110640&drop=dogfoodAlpha

Manual validation needed 😴

Looks like this depends on WindowsRuntime 1.3 which I've opened a PR for in #110644

https://github.com/microsoft/winget-pkgs/pull/110644
[Policy] Needs-Attention

/AzurePipelines run

Requires manual validation like previous version #108247

x64 Automatic Validation ended with:

2023-06-27 04:06:47.972 [CORE] Deployment operation #0: Windows cannot install package Microsoft.Windows.DevHome_<IP_ADDRESS>_x64__8wekyb3d8bbwe because this package depends on a framework that could not be found. Provide the framework "Microsoft.WindowsAppRuntime.1.3" published by "CN=Microsoft Corporation, O=Microsoft Corporation, L=Redmond, S=Washington, C=US", with neutral or x64 processor architecture and minimum version 3000.820.152.0, along with this package to install. The frameworks with name "Microsoft.WindowsAppRuntime.1.3" currently installed are: {}
2023-06-27 04:06:47.972 [FAIL] D:\a_work\1\s\external\pkg\src\AppInstallerCommonCore\Deployment.cpp(161)\WindowsPackageManager.dll!00007FFC2AC59A45: (caller: 00007FFC2ABE3515) Exception(1) tid(1e38) 80073CF3 Package failed updates, dependency or conflict validation.

x86 Automatic Validation ended with:

2023-06-27 04:05:43.809 [CORE] Deployment operation #0: Windows cannot install package Microsoft.Windows.DevHome_<IP_ADDRESS>_x86__8wekyb3d8bbwe because this package depends on a framework that could not be found. Provide the framework "Microsoft.WindowsAppRuntime.1.3" published by "CN=Microsoft Corporation, O=Microsoft Corporation, L=Redmond, S=Washington, C=US", with neutral or x86 processor architecture and minimum version 3000.820.152.0, along with this package to install. The frameworks with name "Microsoft.WindowsAppRuntime.1.3" currently installed are: {}
2023-06-27 04:05:43.810 [FAIL] D:\a_work\1\s\external\pkg\src\AppInstallerCommonCore\Deployment.cpp(161)\WindowsPackageManager.dll!776813F2: (caller: 77613932) Exception(1) tid(5b0) 80073CF3 Package failed updates, dependency or conflict validation.

Also had this output at some point, but can't find its source:

Failure to parse win 10, attempting win 11 Installation failed with exit code -1978334972 2023-06-27 04:06:47.972 [FAIL]

This package requires Windows 11, and I'm not sure that the Automated Validation pipeline is performing this "upgrade" correctly. In a Windows 11 VM, the x86 version installed successfully, but the x64 had an issue with the dependency which I'm troubleshooting.

@wingetbot waivers Add Validation-Installation-Error


[WIP] Producev and datetime

Fix producev binding, producing timestamp now works correctly
I only added one API (for easy testing)
Adding a parameter on all the produce api generates really many method.
Should we do like the java producer and have a ProducerRecord<TKey, TValue>?
Then we would have only the producerRecord and the blockIfQueueFull
Producev and produce doesn't return the same thing, had to duplicate the code. I think the goal is to only use producev in future (from what i saw in commentary but not sure. @edenhill ?)

That's right @treziac, producev() is the future.

This is awesome - would be so happy to get this one in.
We discussed this previously and arrived at using overloads. I think it's manageable for this many parameters (but not any more) and it's the most straightforward to use from a user perspective (no need to create another object).
Feel free to argue against if you strongly object

I completly removed produce for showing what it may look like.
Might be better to keep both, and perhaps just a boolean to choose which implementation we want to use at producer creation (and also for #115 : do we want to keep it working with librkdaka which doesn't provide producev?)
Note that i couldn't remove SafeKafkaTopicHandle because of getmetadata. It misses a method to call it with in char * rather than an rkt for get metadata on a topic only @edenhill

Regarding overloads, what happens if in the future if we add a bool or int ? We won't be able to make an overload which works with only this parameter, or have to mess with parameter order. It's still ok with two optional parameters (partition and blockifqueuefull) but grow exponentially (adding datetime add 4 methods, adding an other parameter will add 20 more methods...)
I would prefer to go to two methods:
ProduceAsync(string topic, TKey key, TValue value) //keep the most used one
ProduceAsync(ProduceRecord<TVKey, TValue> produceRecord)

And add a structure
public struct ProduceRecord<TKey, TValue>
    {
        public ProduceRecord(string topic, TKey key, TValue value)
        {
            Topic = topic;
            Key = key;
            Value = value;
            Partition = Producer.RD_KAFKA_PARTITION_UA;
            Datetime = Timestamp.NO_PRODUCE_DATETIME;
            BlockIfQueueFull = true;
        }

        public string Topic { get; }
        public TKey Key { get; }
        public TValue Value { get; }
        public int Partition { get; set; }
        public DateTime Datetime { get; set; }
        public bool BlockIfQueueFull { get; set; }
    }

producing then looks like:
var deliveryReport = producer.ProduceAsync(topicName, key, val);
var deliveryReport2 = producer.ProduceAsync(new ProduceRecord<string, string>(topicName, key, val) {Partition = 1, BlockIfQueueFull = true });

This make:

less methods, so easier to maintain
closer to java API
more complicated for user only if he doesn't use the simple method (so not a problem for a user discovering the lib)
no problem if we add parameter. Just add a field in the struct. The struct might me big, but we pass all the parameter anyway so I assume it's ok
break previous API (optionally, we could just mark them obsolete for the next release)

Ok. Headers are coming along very soon as well which will make all combinations of parameters unworkable. So I agree with you, I think we should have a ProducerRecord based method and a small number (possibly only one, the one you suggest) of commonly used overloads. We should mark any uncommonly used ones as depreciated.
@edenhill @ewencp - if I recall correctly, you already agree with the above but feel free to comment if not.

Close in favor of #121


[BUG] import app.postcss failed in +layout.svelte

Bug report
Report for:
lucia-sveltekit

Current behavior
failed to import app.css
Expected behavior
should import app.postcss
Possible cause

Additional info
Version

Reproduction

Additional notes

Do you have a full error, and where you are getting this error? Lucia doesn't use postCSS?

I'm using this add-on : "pnpx svelte-add@latest tailwindcss"
It creates an app.postcss which contains the three import of tailwind and this file is imported in +layout.svelte

This seems nothing to do with Lucia?


Penalize ways with bicycle=discouraged

See discussion here: https://github.com/chairemobilite/transition/pull/950
With the function way:get_value_by_key("bicycle")
Possible example (untested!)
function safety_handler(profile, way, result, data)
  -- convert duration into cyclability
  if profile.properties.weight_name == 'cyclability' then
    local safety_penalty = profile.unsafe_highway_list[data.highway] or 1.
    local is_unsafe = safety_penalty < 1

    -- primaries that are one ways are probably huge primaries where the lanes need to be separated
    if is_unsafe and data.highway == 'primary' and not data.is_twoway then
      safety_penalty = safety_penalty * 0.5
    end
    if is_unsafe and data.highway == 'secondary' and not data.is_twoway then
      safety_penalty = safety_penalty * 0.6
    end

    local forward_is_unsafe = is_unsafe and not data.has_cycleway_forward
    local backward_is_unsafe = is_unsafe and not data.has_cycleway_backward
    local is_undesireable = data.highway == "service" and profile.service_penalties[data.service]
    local forward_penalty = 1.
    local backward_penalty = 1.
    if forward_is_unsafe then
      forward_penalty = math.min(forward_penalty, safety_penalty)
    end
    if backward_is_unsafe then
      backward_penalty = math.min(backward_penalty, safety_penalty)
    end

    if is_undesireable then
      forward_penalty = math.min(forward_penalty, profile.service_penalties[data.service])
      backward_penalty = math.min(backward_penalty, profile.service_penalties[data.service])
    end

+   -- Add penalty for bicycle=discouraged tag
+   local bicycle_tag = way:get_value_by_key("bicycle")
+   if bicycle_tag == "discouraged" then
+     forward_penalty = forward_penalty * 0.5
+     backward_penalty = backward_penalty * 0.5
+   end

    if result.forward_speed > 0 then
      -- convert from km/h to m/s
      result.forward_rate = result.forward_speed / 3.6 * forward_penalty
    end
    if result.backward_speed > 0 then
      -- convert from km/h to m/s
      result.backward_rate = result.backward_speed / 3.6 * backward_penalty
    end
    if result.duration > 0 then
      result.weight = result.duration / forward_penalty
    end
  end
end

Originally posted by @davidmurray in https://github.com/chairemobilite/transition/issues/950#issuecomment-2122688947

This is fixed by #1159, we can close


Fetch recently starred pages from storage

This adds a recentBookmarks field to state, populates it from storage, makes the bookmark objects richer (title, location, etc.), and thus gives us a working bookmarks menu.

Coverage increased (+0.1%) to 39.984% when pulling 6c777340c22cafce3f8ac211d94121d01ff888db on rnewman/recently-starred into 9d6a40623974507b9516570ee12b5bc211aadd01 on master.

@rnewman One real request, and then nits.  Also, remember to assign someone (me!) for reviews.


Does not work in docstring position

Normal f-strings work when put as docstrings.
However this codec rewrites them to expressions, which aren’t automatically assigned to thing.__doc__ when in a docstring position (first item in a function/class).
To fully support f-strings, this package should transform e.g.
def foo():
    f"""Hai
    {thar}
    """
    pass

to
def foo():
    pass

foo.__doc__ = textwrap.dedent(
    """Hai
    {}
    """.format(thar)
)

(Or so, I didn’t check if dedent really does what we think it does)

the dedent is incorrect / unnecessary, but otherwise you're correct -- are you willing to try a stab at this? shouldn't be too difficult

I have quite a bit on my plate, so I can’t say if I will find the time. Probably not anytime soon, sorry :cry:

that's fine, it's a pretty severe edge case that shouldn't really impact execution anyway :man_shrugging: -- if someone's depending on that they're already broken by python -0

Interesting, could you please elaborate? How does -O come into play here?

err, I guess I mean -OO
def f():
    """doc"""

print(f.__doc__)

$ python t.py
doc
$ python -OO t.py
None

Ah I see. Well, I don’t think anyone is depending on that in the sense that their application breaks once the docstrings are gone, but e.g. Sphinx of course relies on them!

usually it's a good idea to generate docs from the latest supported python version anyway so that's probably not an issue either

True! But at the moment readthedocs uses 3.5 by default still.

actually, are you sure f-strings are valid docstrings?
$ python3
Python 3.6.7 (default, Oct 22 2018, 11:32:17)
[GCC 8.2.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> def f():
...     f'{f.__name__} hello world'
...
>>> f.__doc__
>>>

actually, are you sure f-strings are valid docstrings?
$ python3
Python 3.6.7 (default, Oct 22 2018, 11:32:17)
[GCC 8.2.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> def f():
...     f'{f.__name__} hello world'
...
>>> f.__doc__
>>>

actually this issue is invalid, I should have checked :laughing:
https://docs.python.org/3/reference/lexical_analysis.html#formatted-string-literals

Formatted string literals cannot be used as docstrings, even if they do not include expressions.

That’s a lie, it works fine!

show me! what version are you using?

hm, this implies that it never worked: https://bugs.python.org/issue28739
probably I’m just wrong…

looks like in 3.6.0 it "worked" for f-string-docstrings without expressions:
root@673f92e13a74:/# python3.6
Python 3.6.0 (default, Feb 28 2017, 22:20:46)
[GCC 4.9.2] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> def f():
...    f'''hi'''
...
>>> f.__doc__
'hi'
>>> def f():
...     f'{f.__name__} hi'
...
>>> f.__doc__

This was removed in 3.6.1:
/ # python
Python 3.6.1 (default, Jun 28 2017, 21:57:03)
[GCC 5.3.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> def f():
...    f'''hi'''
...
>>> f.__doc__
>>>

here's the relevant bpo issue: https://bugs.python.org/issue28739


Show interclass relationships in mermaid diagrams in addition to improved heirarchy

The class mermaid diagrams have improved by showing both parent and child hierarchy. Thanks!
Let's show all inter-class relationships in the Mermaid diagrams, so we would also see the relationship between Biosample and Study.
This may be a dupe. Feel free to move or close.

see also https://github.com/microbiomedata/nmdc-schema/pull/441

see also:

https://github.com/microbiomedata/NMDC_documentation/issues/10


RelationManager toggaleable() with many columns renders offscreen.

Package
filament/filament
Package Version
3.0
Laravel Version
10.10
Livewire Version
3.1
PHP Version
8.1
Problem description
Note: I had to make a new ticket as the gitbot closed my other one without a repo link and I cant seem to reopen it.
https://github.com/filamentphp/filament/issues/9468
When using a relation manager with many columns that use toggleable(), the widget draw sometimes renders vertically off the screen and is unusable. It appears to only be an issue when the widget button is close the the bottom of the page so it tries to render the draw upward and off screen.
Expected behavior

The top of the draw should never go above the top of the page (or the bottom of the header if using the ->topNavigation()
If the panel is taller than the viewport, split into multiple columns.
Allow for the toggles to appear in a modal window with multiple columns.

Steps to reproduce
If using Sail:

Download the repo
Change directory to the root
Create a sail/docker instance: ./vendor/bin/sail up -d
Start docker: ./vendor/bin/sail start
Add ./vendor/bin/sail to the start of each command below in 'the main part'

If not using sail:

Setup your own database
Setup your own host/ports
Setup your own application directory
Download the repo

The main part:

Create an .env file
Generate an application key php artisan key:generate
Install the composer dependencies: composer install
Install the NPM dependencies:  npm install
Run the builder: npm run build
Create database: phpl artisan migrate:fresh --seed
In your browser go to http://localhost/admin
Log in with<EMAIL_ADDRESS>'password'
Navigate to http://localhost/admin/roles/1/edit
Click the toggle columns button on the relation manager.
NOTE: This will work correctly as there are many records in the relation
Navigate to http://localhost/admin/roles/2/edit
Click the toggle columns button on the relation manager.
NOTE: This will show the breakage as there are few records but many columns. The toggle columns button will not snap to it's lower position and the top controls are inaccessable.

Reproduction repository
git@github.com:uqjohart/filamentphp-issue-9468.git
Relevant log output
No response

This is a tricky issue, and not much we can do I think.
The best solution is probably to set a max height on the columns dropdown if you're expecting it to be too tall, using columnToggleFormMaxHeight().
Give it a try, and let me know if you still run into any issues.


"dart info" reports "No Dart processes found"

Running dart info says there are no Dart processes:
danny@Dannys-MacBook-Pro sdk % ~/Dev/Dart\ SDKs/nightly/bin/dart info

If providing this information as part of reporting a bug, please review the information
below to ensure it only contains things you're comfortable posting publicly.

#### General info

- Dart 3.0.0-edge.85d1a7752eb0196ceff40f8d3a9db843db5205c2 (be) (Tue Feb 14 09:01:13 2023 +0000) on "macos_x64"
- on macos / Version 13.1 (Build 22C65)
- locale is en-GB

#### Process info

No Dart processes found.

However I have a VS Code window open that certainly has an analysis server running. If I run ps -x | grep dart I see a few:
danny@Dannys-MacBook-Pro ~ % ps -x | grep dart
10714 ??         0:05.31 /Users/danny/Dev/Google/flutter/bin/cache/artifacts/engine/darwin-x64/flutter_tester --vm-service-port=0 --start-paused --enable-checked-mode --verify-entry-points --enable-software-rendering --skia-deterministic-rendering --enable-dart-profiling --non-interactive --use-test-fonts --disable-asset-fonts --packages=/Users/danny/Dev/Google/flutter/dev/automated_tests/.dart_tool/package_config.json --flutter-assets-dir=/Users/danny/Dev/Google/flutter/dev/automated_tests/build/unit_test_assets /var/folders/b1/ftm2whhj4mb3_c4nfjhq23z00000gn/T/flutter_tools.S4PH0b/flutter_test_listener.hqexdE/listener.dart.dill
11811 ??         0:05.29 /Users/danny/Dev/Google/flutter/bin/cache/artifacts/engine/darwin-x64/flutter_tester --vm-service-port=0 --start-paused --enable-checked-mode --verify-entry-points --enable-software-rendering --skia-deterministic-rendering --enable-dart-profiling --non-interactive --use-test-fonts --disable-asset-fonts --packages=/Users/danny/Dev/Google/flutter/dev/automated_tests/.dart_tool/package_config.json --flutter-assets-dir=/Users/danny/Dev/Google/flutter/dev/automated_tests/build/unit_test_assets /var/folders/b1/ftm2whhj4mb3_c4nfjhq23z00000gn/T/flutter_tools.0LQlQj/flutter_test_listener.gwbO2M/listener.dart.dill
37260 ??         0:16.93 /Users/danny/Dev/Dart SDKs/nightly/bin/dart language-server --protocol=lsp --client-id=VS-Code --client-version=3.59.20230214 --instrumentation-log-file=/Users/danny/Dev/Dart-Code-Logs/server-project-analyzer.instrum.txt
37328 ttys008    0:00.00 grep dart

(FYI @devoncarew)

Thanks for the report! Can you run this:
ps -eo "rss,pcpu,etime,args"

and just paste in the lines for the process you think we should be reporting (dart, flutter_tester, ...). I want to verify that our parsing isn't going wrong, and / or the filtering we apply to make sure we're just reporting dart+flutter related processes.

If I run that through grep dart I see this:
danny@Dannys-MacBook-Pro ~ % ps -eo "rss,pcpu,etime,args" | grep dart
111880   0.0 04:18:56 /Users/danny/Dev/Google/flutter/bin/cache/artifacts/engine/darwin-x64/flutter_tester --vm-service-port=0 --start-paused --enable-checked-mode --verify-entry-points --enable-software-rendering --skia-deterministic-rendering --enable-dart-profiling --non-interactive --use-test-fonts --disable-asset-fonts --packages=/Users/danny/Dev/Google/flutter/dev/automated_tests/.dart_tool/package_config.json --flutter-assets-dir=/Users/danny/Dev/Google/flutter/dev/automated_tests/build/unit_test_assets /var/folders/b1/ftm2whhj4mb3_c4nfjhq23z00000gn/T/flutter_tools.S4PH0b/flutter_test_listener.hqexdE/listener.dart.dill
111424   0.0 04:15:20 /Users/danny/Dev/Google/flutter/bin/cache/artifacts/engine/darwin-x64/flutter_tester --vm-service-port=0 --start-paused --enable-checked-mode --verify-entry-points --enable-software-rendering --skia-deterministic-rendering --enable-dart-profiling --non-interactive --use-test-fonts --disable-asset-fonts --packages=/Users/danny/Dev/Google/flutter/dev/automated_tests/.dart_tool/package_config.json --flutter-assets-dir=/Users/danny/Dev/Google/flutter/dev/automated_tests/build/unit_test_assets /var/folders/b1/ftm2whhj4mb3_c4nfjhq23z00000gn/T/flutter_tools.0LQlQj/flutter_test_listener.gwbO2M/listener.dart.dill
103032   0.0    17:14 /Users/danny/Dev/Google/flutter/bin/cache/dart-sdk/bin/dart --disable-dart-dev --packages=/Users/danny/Dev/Google/flutter/packages/flutter_tools/.dart_tool/package_config.json /Users/danny/Dev/Google/flutter/bin/cache/flutter_tools.snapshot daemon --show-web-server-device
 63932   0.0    17:14 /Users/danny/Dev/Google/flutter/bin/cache/dart-sdk/bin/dart devtools --machine --try-ports 10 --allow-embedding --port 9237
605388   0.0    17:14 /Users/danny/Dev/Google/flutter/bin/cache/dart-sdk/bin/dart language-server --protocol=lsp --client-id=VS-Code --client-version=3.59.20230214
   716   0.0    00:00 grep dart

However without grep, it seems like each line is truncated:
danny@Dannys-MacBook-Pro ~ % ps -eo "rss,pcpu,etime,args"
   RSS  %CPU  ELAPSED ARGS
 27064   0.0 06:49:23 /sbin/launchd
 44628   0.0 06:49:19 /usr/libexec/logd
// SNIP
605388   0.0    17:54 /Users/danny/Dev/Google/flutter/bin/cache/dart-sdk/bin/dart lang
   648   0.0    17:31 /usr/bin/script -t 0 /dev/null xcrun xcdevice observe --both
  2104   0.0    09:31 /System/Library/PrivateFrameworks/ToneLibrary.framework/Versions
  6080   0.0    09:31 /System/Library/Frameworks/AudioToolbox.framework/XPCServices/co
 88992   0.3    01:08 /System/Applications/Utilities/Terminal.app/Contents/MacOS/Termi
158608   0.0    00:28 /Applications/Google Chrome.app/Contents/Frameworks/Google Chrom
 41528   0.0    00:27 /Applications/Google Chrome.app/Contents/Frameworks/Google Chrom
  2156   0.0    17:53 /bin/zsh -il
 34044   0.0 06:15:27 /Applications/Xcode.app/Contents/Developer/usr/bin/xcdevice obse
 34160   0.0 06:14:20 /Applications/Xcode.app/Contents/Developer/usr/bin/xcdevice obse
 33932   0.0    17:31 /Applications/Xcode.app/Contents/Developer/usr/bin/xcdevice obse
 33796   0.0 04:42:09 /Applications/Xcode.app/Contents/Developer/usr/bin/xcdevice obse
 34000   0.0 04:22:19 /Applications/Xcode.app/Contents/Developer/usr/bin/xcdevice obse
 33820   0.0    57:41 /Applications/Xcode.app/Contents/Developer/usr/bin/xcdevice obse
 33952   0.0 04:12:32 /Applications/Xcode.app/Contents/Developer/usr/bin/xcdevice obse
  4184   0.0    01:08 login -pf danny
  2584   0.0    01:08 -zsh
  1340   0.0    00:00 ps -eo rss,pcpu,etime,args
danny@Dannys-MacBook-Pro ~ %

Not sure why it's truncated, or if that's what you're getting. I can try to debug locally if this seems odd.

Hmm, I do see the truncation - perhaps something it does for an interactive process? I don't see if when piped to a file, and don't think we see it when shelling out to that command from dart.
I'll see if I can repo the issue now w/ your output from above.

OK, gotcha - I think w/ that I'll be able to repro this issue.

Yep, confirmed. I ran the command locally and changed this:
bool _isProcessDartRelated(ProcessInfo process) {
  return process.command.toLowerCase() == 'dart' ||

And now that one shows up (because my folder with a space happens to have "Dart" before the space). You might need to find a way to get ps to output in a way that doesn't have ambiguous spaces 🙃

Yeah, that doesn't look easy. From https://unix.stackexchange.com/questions/485999/how-can-i-parse-the-output-of-ps-to-different-fields, the parsable flags of ps aren't available on the mac (ps -o '%p,%a,%c,%y').

In man ps I have:
      -c      Change the “command” column output to just contain the executable name, rather than the full command line.

If I add this, then the paths with the problematic spaces are gone:
danny@Dannys-MacBook-Pro ~ % ps -eco "rss,pcpu,etime,args" | grep dart
112044   0.0 05:04:45 listener.dart.dill
111592   0.0 05:01:09 listener.dart.dill
 63596   0.0    08:36 dart devtools --machine --try-ports 10 --allow-embedding --port 9237
608572   0.0    08:36 dart language-server --protocol=lsp --client-id=VS-Code --client-version=3.59.20230214
119120   0.0    08:11 test.dart
130892   0.0    08:02 listener.dart.dill
   520   0.0    00:00 grep dart

If you do need to get those paths back, you could always grab the PIDs from here and then re-run without the -c and look it up there?

Actually, that output seems to capture the last argument for some things :/ For example those showing listener.dart.dill look like this in the full output:
112056   0.0 05:07:28 /Users/danny/Dev/Google/flutter/bin/cache/artifacts/engine/darwin-x64/flutter_tester --vm-service-port=0 --start-paused --enable-checked-mode --verify-entry-points --enable-software-rendering --skia-deterministic-rendering --enable-dart-profiling --non-interactive --use-test-fonts --disable-asset-fonts --packages=/Users/danny/Dev/Google/flutter/dev/automated_tests/.dart_tool/package_config.json --flutter-assets-dir=/Users/danny/Dev/Google/flutter/dev/automated_tests/build/unit_test_assets /var/folders/b1/ftm2whhj4mb3_c4nfjhq23z00000gn/T/flutter_tools.S4PH0b/flutter_test_listener.hqexdE/listener.dart.dill

I would say the process here should've been flutter_tester? 🤔


Add basic telemetry?

During the sidekick launch I noticed that we do not know how many people have the sidekick open at a time. It may be useful to add some basic telemetry logging so that we know when and how much sidekick is used, where outdated versions are used, and which features are used and unused.
I'd re-use the /.rum/ endpoint for logging and extend the RUM collector to write to a different data sink if the URL is /.rum/sidekick/.

Depends on #23


Add fetch queries for redeem flow

Add fetch queries to support redeem & unstake flow.

It's fixed in PR #177


sentry: add PKCS8 support

Signed-off-by: Long Dai<EMAIL_ADDRESS>Description
Add PKCS8 support.
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: #3240
Checklist
Please make sure you've  completed the relevant tasks for this PR, out of the following list:

[ ] Code compiles correctly
[ ] Created/updated tests
[ ] Unit tests passing
[ ] End-to-end tests passing
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Specification has been updated / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Provided sample for the feature / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

@artursouza please help review

@artursouza all tests pass.


CLB: search for names containig square bracket (or other non-standard characters)

Frontend version: 1bb80f5 February 11, 2022 1:19 PMBackend version: c94dbef February 15, 2022 12:00 AM
Present release of CLB does not allow me to do a search in Workbench for names which contain square bracket or other non-standard characters.
I need this option in the Workbench, for example, to get a list of names similar to  [Culex] stercoreus and mark them as Provisionally Accepted (case of Systema Dipterorum, https://github.com/CatalogueOfLife/testing/issues/127#issuecomment-1045199243)

There is no interpreted version of [Culex] stercoreus:
https://data.catalogueoflife.org/dataset/1101/names?q=stercoreus
The genus has been removed, probably due to the unexpected curly brackets, don't know.


[core] Profiles are no longer hardcoded in help example / supported-profile tests.

Closes #8957

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

[ ] I adhere to the Command Guidelines.

@tjprescott, the release is done and this is good to merge right?

Yes, and thanks for the fix @adewaleo.


Add a footer menu with the items we do not want in the main navbar

Meet the Team
Terms of Service (page ready)
Privacy Policy
I can't think of anything else right now

Added footer links. Configurable via _config.yaml > footer-links in the same way navbar-links work

Created Privacy Policy page and added it to footer menu.
I'll close this - thanks @ScreamingHawk !


When will you release 2.0 FlagData?

I've not seen 2.0 FlagData in your release card.

Any update here?


optionally append into existing files with hog, allow manual reconnect calls on HLOG groups

This PR will resolve this issue raised recently: https://github.com/Morgan-Stanley/hobbes/issues/161

It looks like the travis-ci builds for OSX are failing again due to a failure to download llvm-3.7 on the Mac images that they have.  This is not relevant to this change but obviously it does interrupt the full cross platform testing.  I will watch it and kick it as necessary but for now please note that the Linux builds are passing without incident.

Darn, it looks like homebrew has hosed our expectations with these osx builds:
https://github.com/Homebrew/homebrew-core/pull/31762
I will have to update this osx build on my laptop when I get a chance.


Create SHU token contract on Optimism

Create a contract on Optimism that implements the interface needed for bridging SHU.
Resources:

Optimism docs
Forum post

I will update this issue tomorrow cause today I had a major challenge with the OP team (still ongoing), in short words:

The L2 to L1 bridge cannot be easily made from a contract perspective, only with EOA using the OP SDK

This means that in theory, we will need a lot of time and effort to allow a multi-sig to burn SHU on L2 and unlock on L1. Relaying to a team maybe be an easier path. I will sync with the team tomorrow and try to ping some contacts on OP that might be able to help.

@0xneves  Test withdraw on mainnet this friday (16th Aug)


Fixed Address Sanitizer issues with bench_search

Fixed Address Sanitizer errors when running bench_search in debug mode. These were caused due to the conversion from a character set (i.e. sz::newlines()) to std::string resulting in strings with an incorrect length.

Address Sanitizer output
StringZilla. Starting search benchmarks.
Parsed the dataset with:
- 1048576 words of mean length ~ 5.12 bytes
- 32768 lines of mean length ~ 128.54 bytes
Benchmarking for a newline symbol:
- std::string_view.find          0.3006 GB/s      27902428.1 ns          0 errors in        360 iterations
- sz_find_serial                0.1551 GB/s      54083324.5 ns          0 errors in        188 iterations
- sz_find_avx2                  0.4209 GB/s      19930871.6 ns          0 errors in        504 iterations
- strstr                        0.0001 GB/s<PHONE_NUMBER>00.0 ns          0 errors in          4 iterations
- std::search<>                 0.0217 GB/s     386564882.1 ns          0 errors in         28 iterations
- std::search<BM>               0.0081 GB/s<PHONE_NUMBER>.7 ns          0 errors in         12 iterations
- std::search<BMH>              0.0134 GB/s     627206156.2 ns          0 errors in         16 iterations
- std::string_view.rfind          0.1094 GB/s      76695712.9 ns          0 errors in        132 iterations
- sz_rfind_serial               0.1537 GB/s      54575531.0 ns          0 errors in        184 iterations
- sz_rfind_avx2                 0.4247 GB/s      19749949.8 ns          0 errors in        508 iterations
- std::search<R>                0.0173 GB/s     486119066.7 ns          0 errors in         24 iterations
- std::search<R, BM>            0.0075 GB/s<PHONE_NUMBER>.7 ns          0 errors in         12 iterations
- std::search<R, BMH>           0.0096 GB/s     874787233.3 ns          0 errors in         12 iterations
Benchmarking for an [\n\r] RegEx:
=================================================================
==15944==ERROR: AddressSanitizer: global-buffer-overflow on address 0x7ff632faeec8 at pc 0x7ff9aa6f681a bp 0x005bd8cfe160 sp 0x005bd8cfd910
READ of size 10 at 0x7ff632faeec8 thread T0
==15944==WARNING: Failed to use and restart external symbolizer!
    #0 0x7ff9aa6f6819 in _asan_wrap_strlen+0x2c9 (C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.39.33519\bin\Hostx64\x64\clang_rt.asan_dynamic-x86_64.dll+0x180036819)
    #1 0x7ff632f92c82 in std::_Narrow_char_traits<char,int>::length C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.39.33519\include\xstring:398
    #2 0x7ff632f79ac1 in std::basic_string<char,std::char_traits<char>,std::allocator<char> >::basic_string<char,std::char_traits<char>,std::allocator<char> > C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.39.33519\include\xstring:2557
    #3 0x7ff632f51dc6 in main C:\Users\ashbo\Documents\StringZilla\scripts\bench_search.cpp:298
    #4 0x7ff632f9d868 in invoke_main D:\a\_work\1\s\src\vctools\crt\vcstartup\src\startup\exe_common.inl:78
    #5 0x7ff632f9d7bd in __scrt_common_main_seh D:\a\_work\1\s\src\vctools\crt\vcstartup\src\startup\exe_common.inl:288
    #6 0x7ff632f9d67d in __scrt_common_main D:\a\_work\1\s\src\vctools\crt\vcstartup\src\startup\exe_common.inl:330
    #7 0x7ff632f9d8dd in mainCRTStartup D:\a\_work\1\s\src\vctools\crt\vcstartup\src\startup\exe_main.cpp:16
    #8 0x7ff9f7c67343 in BaseThreadInitThunk+0x13 (C:\WINDOWS\System32\KERNEL32.DLL+0x180017343)
    #9 0x7ff9f7e026b0 in RtlUserThreadStart+0x20 (C:\WINDOWS\SYSTEM32\ntdll.dll+0x1800526b0)

0x7ff632faeec8 is located 0 bytes to the right of global variable '`ashvardanian::stringzilla::newlines'::`2'::all' defined in 'stringzilla.hpp:214:27' (0x7ff632faeec0) of size 8
SUMMARY: AddressSanitizer: global-buffer-overflow (C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.39.33519\bin\Hostx64\x64\clang_rt.asan_dynamic-x86_64.dll+0x180036819) in _asan_wrap_strlen+0x2c9
Shadow bytes around the buggy address:
  0x11705a875d80: 00 00 f9 f9 f9 f9 f9 f9 f9 f9 f9 f9 f9 f9 f9 f9
  0x11705a875d90: 00 00 00 00 00 00 00 00 01 f9 f9 f9 f9 f9 f9 f9
  0x11705a875da0: f9 f9 f9 f9 f9 f9 f9 f9 00 00 00 00 01 f9 f9 f9
  0x11705a875db0: f9 f9 f9 f9 f9 f9 f9 f9 00 00 00 00 f9 f9 f9 f9
  0x11705a875dc0: f9 f9 f9 f9 f9 f9 f9 f9 06 f9 f9 f9 f9 f9 f9 f9
=>0x11705a875dd0: f9 f9 f9 f9 f9 f9 f9 f9 00[f9]f9 f9 f9 f9 f9 f9
  0x11705a875de0: f9 f9 f9 f9 f9 f9 f9 f9 00 00 00 00 00 00 00 00
  0x11705a875df0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
  0x11705a875e00: 00 00 06 f9 f9 f9 f9 f9 f9 f9 f9 f9 f9 f9 f9 f9
  0x11705a875e10: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
  0x11705a875e20: 00 00 00 00 00 02 f9 f9 f9 f9 f9 f9 f9 f9 f9 f9
Shadow byte legend (one shadow byte represents 8 application bytes):
  Addressable:           00
  Partially addressable: 01 02 03 04 05 06 07
  Heap left redzone:       fa
  Freed heap region:       fd
  Stack left redzone:      f1
  Stack mid redzone:       f2
  Stack right redzone:     f3
  Stack after return:      f5
  Stack use after scope:   f8
  Global redzone:          f9
  Global init order:       f6
  Poisoned by user:        f7
  Container overflow:      fc
  Array cookie:            ac
  Intra object redzone:    bb
  ASan internal:           fe
  Left alloca redzone:     ca
  Right alloca redzone:    cb

Also increased the number of characters for bench name output, since there were names longer than 20 chars.

:tada: This PR is included in version 3.6.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[NEXT] Babel preset throws when nativewind tailwind preset loaded indirectly

Describe the bug
When the nativewind tailwind preset is loaded indirectly, such as may be the case when using a shared configuration in a monorepo workspace (like in this example from the create-t3-turbo template) the babel plugin throws an error:

NativeWind preset was not included

This is caused by this check to see if the nativewind preset is in the current tailwind config:
https://github.com/marklawlor/nativewind/blob/0e285148072ea0ab381fedb64185b80ad64493df/packages/nativewind/src/babel/index.ts#L76-L86
To Reproduce
Minimal reproduction:
shared-tailwind.config.js:
const nativewind = require('nativewind/tailwind');

module.exports = {
  presets: [nativewind],
  content: ['./**/*.{js,jsx,ts,tsx}'],
  theme: {
    extend: {},
  },
  plugins: [],
};

tailwind.config.js:
module.exports = {
  presets: [require('./shared-tailwind.config.js')],
};

Workaround
This check can be bypassed by tricking the hasPreset check into thinking the shared preset is the nativewind preset 😅
shared-tailwind.config.js:
const nativewind = require('nativewind/tailwind');

module.exports = {
  presets: [nativewind],
  content: ['./**/*.{js,jsx,ts,tsx}'],
  theme: {
    extend: {},
  },
  plugins: [],
+  nativewind: true,
};

Fixed in 3.0.0-next.33


Typo?

https://github.com/lusaxweb/vuesax/blob/master/src/components/vsSlider/main.styl#L92
Typo?
Or Spanish?
I think spelling isEquals .

it's a mistake the idea was isEquals, can you make a PR fixing this?

OK.
I make a PR when workspace arrive.


Add additional link to doc

Category

[x] Content fix
[ ] New article

Related issues

closes #6557

What's in this Pull Request?

add link to steps on creating an Azure AD application

Docs Build status updates of commit 46248ad:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/general-development/how-to-avoid-getting-throttled-or-blocked-in-sharepoint-online.md
:bulb:Suggestion
View
Details

docs/general-development/how-to-avoid-getting-throttled-or-blocked-in-sharepoint-online.md

Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.

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 in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


WAN Download sensor - Starts at 1.44GB

The problem
When extracting daily download statistic or looking in the history log, the WAN download sensor is expected to start at 0GB of data. But for some reason the amount of data always starts at 1.44GB.
Your device model
AX86U
Firmware type
Merlin
Firmware version
3004.388.6_2
Integration version with the issue
0.30.0
Method of the integration installation
HACS
What version and type of Home Assistant installation do you use
2024.3.3
Is there anything useful in the logs?
No response
Diagnostics information
No response
Additional information
type: statistic
entity: sensor.rt_ax86u_wan_download
period:
calendar:
period: day
offset: -1
stat_type: change
name: Yesterday

Hello,
Sorry to hear you are experiencing issues with AsusRouter.  Upload, LAN, WLAN and so on

Hello, thanks a lot for so useful integration!
I have a similar problem on 0.34.0 RT-AX86U Merlin <IP_ADDRESS>.388.8_2 with Upload calculation.
For several days I have been faced with the problem of huge upload traffic.
Traffic Analyzer - Statistic - App show on top HA with Google APIs(SSL) ~ 10Gb daily - can't realize exactly what happened..
but the issue is about total calculation (currently):
WAN Upload - 921.688 GB
2.4 GHz Upload - 5.036 GB
5 GHz Upload - 12.597 GB
Bridge Upload - 38.163 GB
Wired Upload - 143.980 GB
USB Upload - 0.000 GB
The amount is not even close to 921 GB ?
Also Traffic Analyzer - Statistic shows the different numbers??


Missing PinRef and PinMut map etc APIs

Ref and RefMut both have some APIs for "projecting" through them, like map. I've written the signatures of APIs like these for PinRef and PinMut in some commented out code, but I haven't implemented them.
I didn't really try to implement them, it may be necessary to add new APIs to std to implement these APIs without dark magic like transmutes. I'm not certain.

I decided to give it a go.
First a failed attempt which type checks. due to panic!() and #[allow(unreachable_code)]. Why it fails, featuring lines 34 & 35:
let _inner: RefMut<'a, T> = panic!(); // Cannot get RefMut out of Pin which is needed because RefMut::map takes it by value
//let _inner: RefMut<'a, T> = ::core::mem::transmute(_orig); // error[E0512]: transmute called with types of different sizes

The second attempt was somewhat more succesfull. I am not aware of a way to encode the "does implement Deref, but not DerefMut" constraint into the type system in order to achieve the (bare minimum) level of soundness required to make these kind of changes to the Pin API.
Here is the commit message:

Things that should should all be improved before merging:

Make rust-lang/rust PR for Pin API changes
1.1) Remove Pin::into_ref, calling Pin::map_mut_unchecked::<U, _> and manually turning it into a &'a U is still an option, right?
2a) Make NotDerefMut into a auto trait of sorts
2b) Or, Wait for GAT and create a family (see Niko's 2nd blog post about GAT) with some way to go from the mutable type state to the shared state so that we can reintroduce Pin::into_ref... Actually, I am not sure why I thought this is an alternative to being able to create a type bound that excludes DerefMut without some sort of auto trait intrincis thingy. But a family would compliment it...?
2c) Or, something clever I haven't thought of
Docs. "meow" and "woof" won't do.

Now it is time to address why I copied pin.rs and changed it the way I did: RefMut::map and RefMut::map_split take self, named orig, by value.
Which isn't a problem for Ref because that provides Ref::clone. We need some way to make sure Pin::get_ref is not available for Pin<P> where P: DerefMut.

Pay special attention to 2c!

I am not aware of a way to encode the "does implement Deref, but not DerefMut" constraint into the type system in order to achieve the (bare minimum) level of soundness required to make these kind of changes to the Pin API.

Can you explain this comment? You've written a lot about how to implement this constraint, but its not clear to me what problem it is trying to solve.

Looking at the code it seems to me like adding unsafe Pin<P> -> P function would be sufficient to implement all of this.

I am probably just overthinking stuff. Which I kinda enjoyed... But is not always helpfull.
unsafe Pin<P> -> P does seem like an obvious solution that I wanted to avoid for some reason. When I come back from seeing Venom I'll make some proper PRs. (Can the pinning API changes be done in a plain old PR or do need a separate RFC?)


Fix moving text in script editor with no selected character on last line

Fixes #54362
Issue
In script editor, when moving selection with alt+up/down, an extra line at the end or at the beginning can be moved.
It happens when selection begins at the end of a line or finishes at the beginning of a line (with no character of those lines selected).
Fix proposal
Check if we are in such a case and prevent those line to move with the selection.
(I also rename a variable from_col to from_column for consistency reason.)
Before
You can see that in some case a line is moved whereas it shouldn't happen.

After
Behaviour is the expected one (as in Visual Studio / VS Code)

Tested out other editors seems as though we should only be skipping the last line if to_col == 0. As it still moves the line if the from_col is at the end of the text.

After checking with VisualStudio, you're right. But there is a visual difference :

In VS Studio, VSCode and SublimeText, you see that the carriage return is selected so the selection on that line is not empty.
In godot editor, nothing seems to be selected at the end of the line.

@jmb462 are you going to continue the work on this? If not, I think I could finish it. This problem has been my biggest issue with script editor since forever, and now I'm at last capable of fixing it (I believe).

@jmb462 are you going to continue the work on this? If not, I think I could finish it. This problem has been my biggest issue with script editor since forever, and now I'm at last capable of fixing it (I believe).

Feel free to work on this :)

Thanks, I'll give it a shot :)

Superseded by https://github.com/godotengine/godot/pull/72672.


stl format

Not sure if this is wanted but.... this patch allows reading of STL files using the stl_io crate.

created big match blocks to detect "obj" or "stl" extension, and load
added models/cube.stl models/part.stl (less than 60 kb)
change OBJ INPUT to INPUT FILENAME
ran rustfmt on main.rs and geometry.rs

Well done! :)


[AutoPR cosmos-db/resource-manager] Add short url for all api resources

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/6884

(message created by the CI based on PR content)


refactor(grpc-sdk): way of declaring fields and descriptions changed

Way of declaring descriptions in route fields changed.
Many thanks @kon14  for his help.
Current implementation:
field: {
  type: ConduitType.Required,
  description: 'some description'
}

PR's Implementation
field: ConduitType.Required.Description("Some description")

Also you the way that fields are declared without description remains the same
field: ConduitType.Required

What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update
[x] Refactor
[ ] Build-related changes
[ ] Other (please describe)

Does this PR introduce a breaking change?

[ ] Yes
[x] No

The PR fulfills these requirements:

[x] It's submitted to the main branch
[ ] When resolving a specific issue, it's referenced in the PR's description  (e.g. fix #xxx, where "xxx" is the issue number)

The implementation is completely wrong


Aion Integration?

Request to add Aion to the tool.
Novice coder here, but some of this code might be helpful(?)
https://github.com/aionnetwork/aion_ledger/blob/5ab1e434257ee54d028ada0c9ff298213abe480b/src/main.c
https://github.com/aionnetwork/aion_ledger/blob/5ab1e434257ee54d028ada0c9ff298213abe480b/test/test_cases.py
https://github.com/aionnetwork/aion/blob/ebbb7a982228bac0fc0590cd2bc7ecf8fa221046/modGui/src/org/aion/wallet/util/CryptoUtils.java

Please ask the Aion community to submit a pull request adding this coin to the tool.


Skip init assertion for gdk::set_allowed_backends

To my understanding gdk::set_allowed_backends() should be called (when needed) before the init()
https://docs.gtk.org/gdk3/func.set_allowed_backends.html

This call must happen prior to gdk_display_open(), gtk_init(), gtk_init_with_args() or gtk_init_check() in order to take effect.

The way I added the assertion skip might not be the way intended.
Please let me know.

Would you mind sending a similar PR for gtk4-rs?

Would you mind sending a similar PR for gtk4-rs?

Sure, https://github.com/gtk-rs/gtk4-rs/pull/1183


future announcements/ outreach

ontology clean ups

[x]  tRNA metabolism (this is done, will announce soon)
[ ]  transcription .  This is nearly done. I need to describe the simplifications
Other
[ ] community curation update/push - (might as well wait for the paper)
[ ] slimming

Add suggestions. I think I have other possible, i need to find my list

moved to my outreach doc


UI refactor from SwiftUI to UIKit

Refactored to UIKit, still needs some work on adding features but it's refactored


refactor(infrastructure): Image service methods converted to async and added rule for file extension control

Looks good to me. Thanks for your contribution. 🚀


Packager and live reload not picking up changes in node_modules

Description
I just updated my project from RN 0.34.1 to 0.39.0.  We have a shared code base that we use between several projects that we include in the project.  When we make changes to that shared codebase we have a watch script which copies the transpiled files over into our project's node_modules folder.  With RN 0.34.1 and earlier this would trigger a live reload and the packager would pick up the file changes.  Now with 0.39 there is no live reload and if I manually reload it doesn't pick up any of the changes.
DOESN'T RELOAD
Folder to shared code:
project/node_modules/business-logic/lib/constants/session.js
Imported into project via:
import * as sessionConstants from 'business-logic/lib/constants/session';
If I make a change directly in my project it works.  And if I make a change in core RN files in node_modules it works okay too.
What is causing this?  Is there some new caching of node_modules files somewhere?
Additional Information

React Native version: 0.39.0
Operating System: MacOS
watchman 4.5.0

Duplicate: https://github.com/facebook/react-native/issues/11301

Thanks.


BUG: file names are truncated when relative line numbers are enabled

Did you check docs and existing issues?

[X] I have read all the docs.
[X] I have searched the existing issues.
[X] I have searched the existing discussions.

Neovim Version (nvim -v)
v0.9.5
Operating System / Version
Arch Linux
Describe the Bug
File names are truncated when relative line numbers are enabled.
Screenshots, Traceback

Steps to Reproduce

Adds to config event handler:

  event_handlers = {
      event = "neo_tree_buffer_enter",
      handler = function()
        vim.opt_local.relativenumber = true
      end,
   },

Open neo-tree by hotkey e.

Expected Behavior
file names are not truncated
Your Configuration
-- DO NOT change the paths and don't remove the colorscheme
local root = vim.fn.fnamemodify("./.repro", ":p")

-- set stdpaths to use .repro
for _, name in ipairs({ "config", "data", "state", "cache" }) do
  vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end

-- bootstrap lazy
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({ "git", "clone", "--filter=blob:none", "https://github.com/folke/lazy.nvim.git", lazypath, })
end
vim.opt.runtimepath:prepend(lazypath)

-- install plugins
local plugins = {
  "folke/tokyonight.nvim",
  -- add any other plugins here
}

local neotree_config = {
  "nvim-neo-tree/neo-tree.nvim",
  dependencies = { "MunifTanjim/nui.nvim", "nvim-tree/nvim-web-devicons", "nvim-lua/plenary.nvim" },
  cmd = { "Neotree" },
  keys = {
    { "<Leader>e", "<Cmd>Neotree<CR>" }, -- change or remove this line if relevant.
  },
  opts = {
    -- Your config here
    -- ...
    event_handlers = {
      event = "neo_tree_buffer_enter",
      handler = function()
        vim.opt_local.relativenumber = true
      end,
    },
  },
}

table.insert(plugins, neotree_config)
require("lazy").setup(plugins, {
  root = root .. "/plugins",
})

vim.cmd.colorscheme("tokyonight")
-- add anything else here

This is not a bug I guess. Your filename is bigger than the width of neo-tree buffer. So some of the name is hidden. You can use your mouse(click and hold, then drag) to change the width of neo-tree to see the full name. Check this video...
Screencast from 2024-01-25 10-32-42.webm

I forgot to add the option auto_expand_width = true to the bug report description from my config (now added it).
I think the problem with auto_expand_width option is that it does not check that relativenumber is on and calculates the width less than necessary

As far as I know, there is no way to know the real available width of a window (ref).
API calls such as nvim_win_get_width(0) returns a number that includes the width of nu, rnu, signcolumns so we cannot detect how long the line should be if you enable those columns, hence the content overflows.
Due to this limitation we force set nonu nornu for neo-tree windows and forget about this difficult problem.
If you happen to know any good ways to detect the actual buffer width of a window, please please let me know.
But until then labeling this as wontfix for now.
@cseickel Can we close this FR?

Is there any news? Would love to use relative line numbers and the expand option.
Same but different: It would be awesome if I could at least jump to the end of the line / file name using $ to see the file extension, does it need to be truncated or could it just be a long line outside of the current viewport / buffer width? 🤔


Added feedback mechanism

Updated to include the new user feedback mechanism as part of beta testing the new system.

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

File
Status
Preview URL
Details

docfx.json
:white_check_mark:Succeeded

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


Footer "Quick Links" tag hover effect improved

#851
Here I improved the Footer "Quick Links" section where I added hover effect as well animation , text-shadow effects on the Footer "Quick Links" tag also I added keyFrames blink for animation.
#851
BUG: Footer "Quick Links" ul tag word gap is not good
Video -
https://github.com/Suchitra-Sahoo/AgriLearnNetwork/assets/136844498/052da0f8-e8b2-4f46-bfd5-3fa320cc6999
Type of PR

[ ] Bug fix
[x] Feature enhancement
[x] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
[Attach any relevant screenshots or videos demonstrating the changes]
Checklist:

[x] I have performed a self-review of my code
[x] I have read and followed the Contribution Guidelines.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[x] I have commented my code, particularly in hard-to-understand areas.

Additional context:
[Include any additional information or context that might be helpful for reviewers.]

@Suchitra-Sahoo
dear admin as your suggestion I only added hover with animation effects on the Quick links tag in the footer.
Thank you !

Dark green is not suitable it barely visible use some shade of light green or something else.

@Suchitra-Sahoo
ma'am you give level 1 in this PR ,
ok then my question is when I will I got level 2.


Moving data fetch in the browse all page to the client

Currently the initial data fetch for the browse all page is happening on the server. In cases where there are high latencies, it takes a while for the page to load. This might cause confusion for the user on if the page is loading.

Loading data on the client for browse data page might impact SEO.


Update graph-introduction.md

Remove extra double quote

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

Looks good; thanks @lestersr.


1.5.0-RC testing

The following e2es are not passing on dev12. Please add your name next to the test that you will be performing. Check the box once you verify the test cases manually. Post a message on the release channel if you find a problem.
Backend: Dev12
Frontend: Dev12 (or local, using 1.5.0-RC branch)
Mobile

[ ] added-to-cart-modal-mobile (Kate)
[ ] cart-mobile @qjimmy
[ ] consignment-tracking-mobile @Pucek9
[ ] coupon
[ ] my-account/my-coupons
[ ] mobile/my-coupons
[ ] save-for-later

Regression

[ ] added-to-cart-modal (GC)
[ ] anonymous-consents-config-flow (Kate)
[ ] anonymous-consents-flow (Kate)
[ ] applied-promotions
[ ] billing-address
[ ] cart @qjimmy
[ ] consignment-tracking @Pucek9
[ ] coupon
[ ] my-coupons
[ ] my-account/payment-methods (GC)
[ ] product-details (GC)
[ ] save-for-later (GC)

Asterix & Obelix already took care

anonymous-consents-config-flow (Kate) ❌
works in https://github.com/SAP/cloud-commerce-spartacus-storefront/pull/6415


Benchmark driver fix

Added some fixes for benchmark driver:
Support skipping failed queries, and capture error messages if any
Sort list of files for benchmark driver

Please start commit messages with a capital letter and follow the standard format: http://tbaggery.com/2008/04/19/a-note-about-git-commit-messages.html
The second commit message is too long and doesn't need a comma.

Can shorten to "Support skipping failed queries and capture error messages"

David, fixed issues you mentioned. Also, added custom exception type BenchmarkDriverExecutionException to make it clear when we deal with failed queries.

Let me know when the changes are integrated

Merged

Merged


docs(tutorials/jokes.md): update jokeListItems type using ts Pick.

I'm going to close this one as we're working on updating the docs with removing all ActionData  & LoaderData
Thanks for taking the time to update this though! 🙏


Send release and queue metadata back as JSON in the response body

This is to allow API users to get metadata back such as the release version and next development version when these parameters are not supplied in the request (i.e. when allowing the API to use the defaults). This also returns the queue item id as its own field so it can be used explicitly rather than parsed from the Location header.
The Location header and HTTP 201 response code is left as-is to conform to the vanilla build API.

This has been open for a while now, is there anything you would like me to improve for this to be merged?

My apologies for keeping this PR waiting and thank you for submitting it @reversefold.
We'll review it during the next few days and reply back soon.

Thank you for this @reversefold!
HAP-971 tracks this enhancement.


Refactor member variable transformer

This removes the member variable transformer logic from the class
transformer in favor of reusing the old ES6ClassTransformer.

@johnjbarton PTAL
The diff looks a lot larger because of the file renames. I can rename it back the old file names for the sake of the review if you prefer?
@vicb FYI

What would help the most is more/different commit description. For example, "reusing the old ES6ClassTransformer" seems to conflict with that file being deleted.

PTAL
I split it into 2 commits. The second one is pure rename and can mostly be ignored when reviewing.
I changed the commit message slightly:
Refactor member variable transformer

This removes the member variable transformer logic from
ClassTransformer. Instead we use the alrady existing
ES6ClassTransformer which does this as a stand alone transformer.

LGTM but the change to the test case seems suspicious.


Tag the repo when the version is changed

When a change is made to the version file we should also create a tag on this repository with the new
version number.
This means that breaking changes can be made, users can then choose whether or not to update to the latest version.
This will mean adding a jenkins.sh file, one like this exists for govuk elements.
cc. @fofr, @dsingleton, @timpaul.

It'll need the jenkins.sh and a job in Jenkins to execute it / push the version (There's one for Elements, which is a good template.

We've recently had a few FE failed builds, that we've not noticed for a while. It would be good to tweak jenkins.sh (here and on Elements) to mark commits with a failed status, if the build fails. There's some prior art on some GOV.UK apps showing how to do this. Not a blocker, but would add some robustness.


ITIndexerTest.testGetLockedIntervals seems to be flaky

ITIndexerTest.testGetLockedIntervals seems to be flaky. Here is one sample run - https://app.travis-ci.com/github/apache/druid/jobs/556115018
Affected Version
Master branch
Description
The test has likely failed because of a reason unrelated to the change in PR. I can't get the actual reason since we get another exception while unloading the data. This other exception is masking the original problem.

@abhishekagarwal87 thank you for making this issue. I have the same problem in https://app.travis-ci.com/github/apache/druid/jobs/556115018 for https://github.com/apache/druid/pull/12169.

this has been fixed already.


Global response to DM6 not observed in Test 7.7

Global response to DM6 not observed in Test 7.7.  See #949 for parallel example in DM2.
...
Both the Global and DS Queries were Supported... in Part 3.  Yet a failure was in Part 7.
  The following is filtered:
12:26:57.5150 18EAFFFE [3] EB FE 00
12:26:57.5182 18ECFF00 [8] 20 3F 00 09 FF EB FE 00
12:26:57.5782 18EBFF00 [8] 01 49 4E 54 20 20 2A 37
12:26:57.6382 18EBFF00 [8] 02 35 37 31 30 33 31 37
12:26:57.6983 18EBFF00 [8] 03 31 37 2A 31 32 34 4B
12:26:57.7584 18EBFF00 [8] 04 4D 32 59 34 35 30 33
12:26:57.8182 18EBFF00 [8] 05 32 39 37 00 00 00 2A
12:26:57.8407 18EAFFF9 [3] CF FE 00 (TX)
12:26:57.8435 18FECF17 [8] 00 FF 00 00 00 00 FF FF
12:26:57.8784 18EBFF00 [8] 06 30 30 30 30 30 30 30
12:26:57.9382 18EBFF00 [8] 07 30 30 30 2A 19 19 19
12:26:57.9985 18EBFF00 [8] 08 19 19 19 19 19 19 19
12:26:58.0581 18EBFF00 [8] 09 19 19 19 19 19 19 00
12:26:58.4468 18EA00F9 [3] CF FE 00 (TX)
12:26:58.4482 18FECF00 [8] 00 FF 00 00 00 00 FF FF

Application behaves as expected.  There was no BUSY NACK and reporting was handled correctly.


NameError: name 'pcl_gt_1K_scaled' is not defined

Hi, I want use the evaluation to see the cd, emd etc. But it report the issue. I see some file but didn't find the detail. I miss something here?

I changed the parameter to 'pcl_gt_scaled', it worked. But another problem appeared, in the 'feed_dict={pcl_gt:batch_gt, img_inp:batch_ip}' showed the problem is 'ValueError: cannot feed value of shape(0,) for tensor u'pcl_gt:0',which has hsape '(24,1024,3) '
What should I do ?

I changed the parameter to 'pcl_gt_scaled', it worked. But another problem appeared, in the 'feed_dict={pcl_gt:batch_gt, img_inp:batch_ip}' showed the problem is 'ValueError: cannot feed value of shape(0,) for tensor u'pcl_gt:0',which has hsape '(24,1024,3) '
What should I do ?

Hello,
did u find the solution ?

I changed the parameter to 'pcl_gt_scaled', it worked. But another problem appeared, in the 'feed_dict={pcl_gt:batch_gt, img_inp:batch_ip}' showed the problem is 'ValueError: cannot feed value of shape(0,) for tensor u'pcl_gt:0',which has hsape '(24,1024,3) '
What should I do ?

I changed the parameter to 'pcl_gt_scaled', it worked. But another problem appeared, in the 'feed_dict={pcl_gt:batch_gt, img_inp:batch_ip}' showed the problem is 'ValueError: cannot feed value of shape(0,) for tensor u'pcl_gt:0',which has hsape '(24,1024,3) '
What should I do ?

Hello,
did u find the solution ?

Sorry for bothering,
Do you guys find any solution?

I slove the above two problem by changing lines: 282,  283, 285, 286, 290 in metrics.py:
273: batches = len(train_pair_indices)
274: calculate_metrics(train_models, batches, pcl_gt_1K_scaled, reconstr_img_scaled, train_pair_indices)
276: batches = len(val_pair_indices)
277: calculate_metrics(val_models, batches, pcl_gt_1K_scaled, reconstr_img_scaled, val_pair_indices)
281: batches = len(models)

into
import math
273: batches = int(math.ceil(len(train_pair_indices) / BATCH_SIZE))
274: calculate_metrics(train_models, batches, pcl_gt_scaled, reconstr_img_scaled, train_pair_indices)
276: batches = int(math.ceil(len(val_pair_indices) / BATCH_SIZE))
277: calculate_metrics(val_models, batches, pcl_gt_scaled, reconstr_img_scaled, val_pair_indices)
281: batches = int(math.ceil(len(models) / BATCH_SIZE))

I think the author forgot to deal with batch size problems. It seems more reasonable to divide the length of the dataset with batch size since it later iterates through the batches in function: calculate_metrics.

As for changing pcl_gt_1K_scaled to pcl_gt_scaled, I not sure what the author's intention here and how will changing it does to the performance. At least the Evaluation part in README.md can run smoothly now.

If there is any misunderstanding of the code, please let me know. Thz!


Improvements to the register_new_resolver() API

Fixes #370
Note: there is no additional news item because there is a already an item referring to #426, which also includes a pointer to #370 (at the end). Since these changes are pretty small I'm not sure it's worth making them more prominent in the news, so @omry let me know how you feel about that.

Actually - we need to update the docs a bit right?

Can't hurt :) Done in c953789


VS Code "and still" won't open in a Linux xrdp session

I still can't believe that twice this issue has been closed without any resolution:
https://github.com/Microsoft/vscode/issues/12233
https://github.com/Microsoft/vscode/issues/27369
In my own opinion, this still an open issue!
So, if a Windows Admin decide to remote connect to a Linux box, so they can't use VS Code.

VSCode Version: 1.17.0
OS Version: Ubuntu 16.04.3

Steps to Reproduce:

Setup *xrdp:

sudo apt-get install tightvncserver
sudo apt-get install xrdp
sudo apt-get install xubuntu-desktop
echo "xfce4-session" > ~/.xsession
sudo gedit ~/.xsession
sudo service xrdp restart

*note: It's known that xrdp works only on 2D desktops, so either MATE and xubuntu need to be installed.

From Windows use RDP app to connect to Linux.

Open Code, but won't open and there are no errors displayed.

Workaround:
This time around I resolved the issue by narrowing the following Hack in two steps:

Copy the system file 'libxcb.so.1.1.0' to the VS Code Insider folder:

copy /usr/lib/x86_64-linux-gnu/libxcb.so.1.1.0 /usr/share/code-insider

Last execute the sed stream editor command to alter value:

sudo sed -i 's/BIG-REQUESTS/_IG-REQUESTS/' /usr/share/code-insider/libxcb.so.1.1.0

After these step were completed, I was able to open VS Code Insider on a xrdp session.
must important I didn't have to alter the system file "libxcb.so.1.1.0" as in previous posts was done in the "/usr/lib/x86_64-linux-gnu/" location.
(Below image shows implemented hack)

Reproduces without extensions: Yes/No

We seem to be tracking this as #3451 as you might be aware.

Hi @chrmarti,
Thanks for the heads up!
:)

I'm closing this as a duplicate of #3451. This is not to say that it is fixed, but that there already is an issue tracking it.


Warning about using ${buildType} in ${cmake.buildDirectory} endlessly pops up

This warning

It is not advised to use ${buildType} in the cmake.buildDirectory settings when the generator supports multiple build configurations.

continuously pops up even though I am using Ninja which DOES NOT support multiple configurations.

That's strange... Could I get some more information about your platform and setup?

Windows 10
VSCode 1.7.1
CMakeTools 0.7.4

Do you ever manually do set(CMAKE_CONFIGURATION_TYPES ...) in your CMake files? That's the heuristic I'm using to determine if the generator is multi-conf or not, and I'm guessing it is not working very well in this case. I may need to come up with a better way to detect multi-conf generators without just enumerating them all.

Should be fixed in next release

Close this issue if the 0.7.5 release fixes this for you. Thanks!

Yes it does thansks!


Add lyra/mode/orderly mainnet superchain configs

Description
A clear and concise description of the features you're adding in this pull request.
Tests
Please describe any tests you've added. If you've added no tests, or left important behavior untested, please explain why not.
Additional context
Add any other context about the problem you're solving.
Metadata

Fixes #[Link to Issue]

hi @kahuang , CI is failing on this PR, can you fix it and re-request a review from me? thanks!
e.g.
  Checking /root/project/superchain/extra/addresses/mainnet/lyra.json
Error:
script failed: no matching value found at ".SystemConfigProxy"

Should be resolved now, looks like some rate limit errors on main, and unspecified on codegen?

yay ok got them to pass

@kahuang when this gets merged, could you update the superchain registry dependency in op-geth & in the monorepo to ensure that this configuration gets used. At some point in the next couple weeks, we'll do a release which will include these chains by flag.

@kahuang when this gets merged, could you update the superchain registry dependency in op-geth & in the monorepo to ensure that this configuration gets used. At some point in the next couple weeks, we'll do a release which will include these chains by flag.

If you can ping me when this gets merged I can take a look


VideoRecorder throwing exception on a virtual machine

The following exception occurs sporadically when running the VideoRecorder on a virtual machine.
build	30-Jun-2020 14:38:50	 System.AggregateException: One or more errors occurred. ---> System.InvalidOperationException: Pipe hasn't been connected yet.
build	30-Jun-2020 14:38:50	    at System.IO.Pipes.PipeStream.CheckWriteOperations()
build	30-Jun-2020 14:38:50	    at System.IO.Pipes.PipeStream.Flush()
build	30-Jun-2020 14:38:50	    at FlaUI.Core.Capturing.VideoRecorder.WriteLoop()
build	30-Jun-2020 14:38:50	    at System.Threading.Tasks.Task.Execute()
build	30-Jun-2020 14:38:50	    --- End of inner exception stack trace ---
build	30-Jun-2020 14:38:50	    at System.Threading.Tasks.Task.ThrowIfExceptional(Boolean includeTaskCanceledExceptions)
build	30-Jun-2020 14:38:50	    at System.Threading.Tasks.Task.Wait(Int32 millisecondsTimeout, CancellationToken cancellationToken)
build	30-Jun-2020 14:38:50	    at FlaUI.Core.Capturing.VideoRecorder.Stop()
build	30-Jun-2020 14:38:50	 ---> (Inner Exception #0) System.InvalidOperationException: Pipe hasn't been connected yet.
build	30-Jun-2020 14:38:50	    at System.IO.Pipes.PipeStream.CheckWriteOperations()
build	30-Jun-2020 14:38:50	    at System.IO.Pipes.PipeStream.Flush()
build	30-Jun-2020 14:38:50	    at FlaUI.Core.Capturing.VideoRecorder.WriteLoop()
build	30-Jun-2020 14:38:50	    at System.Threading.Tasks.Task.Execute()<---
build	30-Jun-2020 14:38:50
build	30-Jun-2020 14:38:50	 One or more errors occurred.
build	30-Jun-2020 14:38:50	 System.AggregateException: One or more errors occurred. ---> System.InvalidOperationException: Pipe hasn't been connected yet.
build	30-Jun-2020 14:38:50	    at System.IO.Pipes.PipeStream.CheckWriteOperations()
build	30-Jun-2020 14:38:50	    at System.IO.Pipes.PipeStream.Flush()
build	30-Jun-2020 14:38:50	    at FlaUI.Core.Capturing.VideoRecorder.WriteLoop()
build	30-Jun-2020 14:38:50	    at System.Threading.Tasks.Task.Execute()
build	30-Jun-2020 14:38:50	    --- End of inner exception stack trace ---
build	30-Jun-2020 14:38:50	    at System.Threading.Tasks.Task.ThrowIfExceptional(Boolean includeTaskCanceledExceptions)
build	30-Jun-2020 14:38:50	    at System.Threading.Tasks.Task.Wait(Int32 millisecondsTimeout, CancellationToken cancellationToken)
build	30-Jun-2020 14:38:50	    at FlaUI.Core.Capturing.VideoRecorder.Stop()
build	30-Jun-2020 14:38:50	 ---> (Inner Exception #0) System.InvalidOperationException: Pipe hasn't been connected yet.
build	30-Jun-2020 14:38:50	    at System.IO.Pipes.PipeStream.CheckWriteOperations()
build	30-Jun-2020 14:38:50	    at System.IO.Pipes.PipeStream.Flush()
build	30-Jun-2020 14:38:50	    at FlaUI.Core.Capturing.VideoRecorder.WriteLoop()
build	30-Jun-2020 14:38:50	    at System.Threading.Tasks.Task.Execute()<---

The setup of the VideoRecorder is as follows:
m_videoRecorder = new VideoRecorder(new VideoRecorderSettings { VideoFormat = VideoFormat.xvid, VideoQuality = 8, ffmpegPath = ffmpegPath, TargetVideoPath = videoFilePath }, r =>
{
    var testName = testTitle;
    var img = Capture.Screen();
    img.ApplyOverlays(new InfoOverlay(img)
    {
        RecordTimeSpan = r.RecordTimeSpan,
        OverlayStringFormat = @"{rt:hh\:mm\:ss\.fff} / {name} / CPU: {cpu} / RAM: {mem.p.used}/{mem.p.tot} ({mem.p.used.perc}) / " + testName,
        OverlayTextColor = Color.Red
    },
    new MouseOverlay(img));

    return img;
});

and the teardown
if (m_videoRecorder != null)
{
    Trace.WriteLine($"Recorder saving to -> {m_videoRecorder.TargetVideoPath}");

    filepath = m_videoRecorder.TargetVideoPath;
    m_videoRecorder.Stop();

    m_videoRecorder.Dispose();
    m_videoRecorder = null;
}

We also found that running with VideoFormat.x264 resulted in no videos being saved, whereas VideoFormat.xvid works more reliably.
System info:
Windows 10 Enterprise (x64) Build 18363
16GB RAM
Running through Hyper-V (Microsoft Virtual Machine Manager)
Tests are running through Specflow using the MSTest runner.
A desktop session is available and the actual UI tests run. The problem is related simply to the VideoRecorder.

That is kind of strange as I have a WaitForConnection on the pipe. I suppose it happens when the test is finished and it tries to write the video file?

@Roemer, you are correct - it fails when the test finishes. Any thoughts?

Very strange. Do you have some way to reproduce the issue?

Sorry for the delay, we have been trying to track down the issue a bit more.
We have a test that causes FlaUI to throw the following exception
Failed applying overlay 'FlaUI.Core.Capturing.MouseOverlay'
System.Runtime.InteropServices.ExternalException (0x80004005): A generic error occurred in GDI+.
   at System.Drawing.Image.FromHbitmap(IntPtr hbitmap, IntPtr hpalette)
   at System.Drawing.Image.FromHbitmap(IntPtr hbitmap)
   at System.Drawing.Icon.BmpFrame()
   at FlaUI.Core.Capturing.CaptureUtilities.CaptureCursor(Point& position)
   at FlaUI.Core.Capturing.MouseOverlay.Draw(Graphics g)
   at FlaUI.Core.Capturing.CaptureImage.ApplyOverlays(ICaptureOverlay[] overlays)

Once this exception occurs, no further videos are recorded. We are now able to replicate this issue on our local machines so it is not just on virtual machines (good news 👍)


GPU AMIs not available with explicit amiSelector

Description
Observed Behavior:
When AWSNodeTemplate does not specify amiSelector, ATM 3 unique images are autodiscovered (here is log from us-east-1 cluster):
2023-10-22T11:25:17.852Z    DEBUG    controller.awsnodetemplate    discovered amis    {"commit": "34d50bf-dirty", "awsnodetemplate": "", "ids": "ami-0a68d6081e5e0d388, ami-067fb3752f43c5ef8, ami-067fb3752f43c5ef8, ami-0fe6f9f7224693
AWSNodeTemplate object status after autodiscovery:
amis:

id: ami-0a68d6081e5e0d388
name: amazon-eks-node-1.26-v20231002
requirements:

key: karpenter.k8s.aws/instance-accelerator-count
operator: DoesNotExist
key: kubernetes.io/arch
operator: In
values:

amd64

key: karpenter.k8s.aws/instance-gpu-count
operator: DoesNotExist

id: ami-067fb3752f43c5ef8
name: amazon-eks-gpu-node-1.26-v20231002
requirements:

key: kubernetes.io/arch
operator: In
values:

amd64

key: karpenter.k8s.aws/instance-accelerator-count
operator: Exists

id: ami-067fb3752f43c5ef8
name: amazon-eks-gpu-node-1.26-v20231002
requirements:

key: kubernetes.io/arch
operator: In
values:

amd64

key: karpenter.k8s.aws/instance-gpu-count
operator: Exists

id: ami-0fe6f9f7224693928
name: amazon-eks-arm64-node-1.26-v20231002
requirements:

key: kubernetes.io/arch
operator: In
values:

arm64

key: karpenter.k8s.aws/instance-gpu-count
operator: DoesNotExist
key: karpenter.k8s.aws/instance-accelerator-count
operator: DoesNotExist

When I put these AMI ids explicitly into amiSelector.aws::ids, GPU AMI is thrown away silently.
2023-10-22T11:31:14.749Z    DEBUG    controller.awsnodetemplate    discovered amis    {"commit": "34d50bf-dirty", "awsnodetemplate": "", "ids": "ami-0a68d6081e5e0d388, ami-0fe6f9f7224693928", "count": 2}
AWSNodeTemplate object status after setting amiSelector:
status:
amis:

id: ami-0a68d6081e5e0d388
name: amazon-eks-node-1.26-v20231002
requirements:

key: kubernetes.io/arch
operator: In
values:

amd64

id: ami-0fe6f9f7224693928
name: amazon-eks-arm64-node-1.26-v20231002
requirements:

key: kubernetes.io/arch
operator: In
values:

arm64

Expected Behavior:
GPU-compatible AMIs should be available with explicit amiSelector.aws::ids
Reproduction Steps (Please include YAML):
Set amiSelector.aws::ids explicitly (below AMI ids in us-east-1):
amiSelector:
aws::ids:  ami-0a68d6081e5e0d388,ami-067fb3752f43c5ef8
Versions:

Chart Version: v0.31.1
Kubernetes Version (kubectl version): Server Version: version.Info{Major:"1", Minor:"26+", GitVersion:"v1.26.7-eks-2d98532", GitCommit:"c9c3fc1fdf677f38b8d26568c10b921a1235fa43", GitTreeState:"clean", BuildDate:"2023-07-28T16:51:21Z", GoVersion:"go1.20.6", Compiler:"gc", Platform:"linux/amd64"}

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

I have same issue. GPU Ami can't make node.

I also encountered same issue

@ellistarn is this behavior documented anywhere? I'm a bit surprised you relabeled it as a feature...

Sorry I misread and thought this was related to a similar ask someone had DMed me. You're right -- this is odd. Will dig into it when I get a chance.

@pgackiewiczlcloud Are you seeing GPU instances unable to be launched after aws::ids are set?

To be honest I did not have a need to run GPU node, but I bet it would be not possible in that case

The reason this was re-labeled as a feature is that we treat the amiSelector as a full override of the default AMIs that are discovered with SSM when no AMI selector is used. I'm proposing a clarifying statement here #5412.
We add additional requirements to the default AMIs that we select on because we know specific things about these AMIs e.g. this SSM alias always maps to an AMI that has the GPU device plugin while this one doesn't and works for instance types that don't have GPU hardware. We don't have a way to infer this knowledge from the pure selection logic that we have today, all we can infer is the architecture of the image.
There was some thinking for a bit about adding a requirements stanza to the amiSelectorTerm so that it looks like this
amiSelectorTerms:
- id: ami-1234
  requirements:
  - key: karpenter.k8s.aws/instance-gpu-count
    operator: Exists

There hasn't been strong enough use-cases for doing something like this today since most users that are using custom ami selection that we have seen are also creating separate, dedicated NodePools for their GPU amis and instance types.


SoundComponent streaming

It would be great to be able to mark certain slots to enable streaming if available.
Possible consider adding stream flag to SoundSlot. Which by default is false.
Every time play is called, it will check if asset it is loaded. If it is loaded, it ignores streaming. If asset is not loaded it will start streaming it.
It should handle well cases where asset gets loaded eventually, as streaming for every time it plays audio, it re-downloads data, and not caches it (that is the way browsers work).

Related request: live streamed audio (https://developer.mozilla.org/en-US/docs/Web/Guide/Audio_and_video_delivery/Live_streaming_web_audio_and_video)

Regarding initial "streaming" terms, for sound component: it was about downloading while playing instead of preloading and then playing.
Live streaming - is another story I believe, but might relate.

Hey,
would be great to get this implemented, i am currently working on a project where i am trying to convert my voice chat to an positional voice chat.  Here are some more infos:
https://forum.playcanvas.com/t/assign-audio-htmlelement-to-soundslot/19179/8
BR
Sinlyu.


where is "FlagPole.obj" file?

Hi, nice project.
I have built your project under Windows, and by using codeblocks+msys2.
See:
asmwarrior/ClothSimulator: Cloth Simulator
I see such code:
void Controller::OnMenuFileAddObject(wxFrame* frame)
{
	wxLogDebug("OnMenuFileAddObject");
	wxTextFile objfile("Objects/FlagPole.obj");
	bool success = objfile.Open();
	wxLogDebug("opened: %i", success);

	for (wxString str = objfile.GetFirstLine(); !objfile.Eof(); str = objfile.GetNextLine())
	{
		if (str.StartsWith("o ")) { // new object
			wxString name = str.SubString(2, 0);
			wxLogDebug(name);
		}
	}

	objfile.Close();
}

I see this lines in the code, but where is "FlagPole.obj" file?

Hi,
thanks!
It's been a while since I worked on this. I may have used the flagpole.obj (some simple object made in blender) as a test.
This part of the code doesn't look complete as well. You might as well do your thing.


Normalize device names in multi_gpu_model

added _normalize_device_name function to training_utils.py --> convert "/device:GPU:0" to "/gpu:0"
call _normalize_device_name() after obtaining available devices in multi_gpu_model() before verifying against target_devices.
Successfully run tests in tests/keras/utils/multi_gpu_test.py
Should work in both nightly and stable versions of tensorflow as all device names are shortened.

Related Issues:
fixes #8213  : converts all to "/xpu:0" representation
fixes #8220
fixes #8222   : This PR suggests converting to "/device:GPU:0" format for gpu and "/cpu:0" for cpu.
Source:
Uses code snippet suggested by @fchollet in https://github.com/fchollet/keras/pull/8222

The problem of this solution is that in the future, if tensorflow changes /device:CPU:0 notation into something else, we should change these codes as well.

Yes of course, but in the meantime we can use this to deal with shared notations. Once tensorflow completely shifts both gpu and CPU to "/device/XPU:0" notation then keras could also transition to it. Also since a lot of people use tensorflow from pip this will allow them to use multi gpu training now instead of waiting.

LGTM. Can you confirm it works on a setup that would previously raise an error?

we should change these codes as well.

Yes, if TF makes breaking changes, we will adapt the Keras code as well, in a way that is compatible with both the old version and the new version. But I don't expect breaking changes at this point.

@fchollet I ran the multi_gpu_application_folder_generator_benchmark() function in multi_gpu_test.py in the tests folder. I printed the normalized and un-normalized device names before training started.  It ran on both tf-1.3.0 and tf-1.4.0rc1.

tensorflow 1.3.0 : obtained from "pip install tensorflow-gpu".
unnormed_names: /cpu:0, /gpu:0, /gpu:1
normed_names:   /cpu:0, /gpu:0, /gpu:1

tensorflow 1.4.0rc1 : obtained from "pip install tensorflow-gpu==1.4.0rc1". This was released 2 days ago so probably reflects the most recent changes.
unnormed_names: /device:CPU:0, /device:GPU:0, /device:GPU:1
normed_names:   /cpu:0, /gpu:0, /gpu:1

Let me know if any other versions need to be tested.


Add make sure

Fixes https://github.com/hypothesis/via3/issues/165

I guess this answers my question: https://github.com/hypothesis/via3/pull/179/files


Styling of container

Hi!
I would love to be able to style the container of the dropdown menu. Specifically, I would like to change the default CSS for min-width. Would it be an option for you to maybe add a property for us to set specific CSS classes for the container?

Hi,
sorry for late reply. Did you look at the demo (https://v-dropdown-menu.now.sh/)
you can apply custom style to the dropdown-menu. bind a class to the v-dropdown-menu component then write your style
ex.
.v-dropdown-menu {
  $this: &;
  &.custom-style & {
    &__container {
      min-width: 400px;
...

....

Hi!
I would love to be able to style the container of the dropdown menu. Specifically, I would like to change the default CSS for min-width. Would it be an option for you to maybe add a property for us to set specific CSS classes for the container?

Hi,
sorry for late reply. Did you look at the demo (https://v-dropdown-menu.now.sh/)
you can apply custom style to the dropdown-menu. bind a class to the v-dropdown-menu component then write your style
ex.
.v-dropdown-menu {
  $this: &;
  &.custom-style & {
    &__container {
      min-width: 400px;
...

Ah, I see. No, I expected to find that kind of information in the README/documentation area. You might want to consider doing that. Thank you for the reply though! That was helpful.


Test removing VCLibs dependency

DO NOT MERGE. this is a test PR

Microsoft Reviewers: Open in CodeFlow

@wingetbot run

[Policy] reset feedback


Slide transitions with 2 slides

Hello! Thanks for the plugin, it's very good. However, I ran into a bit of an issue when I just had two slides. When the slide transitions, the slide that was just displayed immediately disappears and goes to the right of the next slide. This makes for a very jarring transition. I also can't drag the slide to the right and get the slide that was just displayed, I only get whitespace. Is there a way for me to fix this?
Thanks!

Hi. If you're using only two or three slides, you should set the loop option to false. It has been discussed here: https://github.com/bqworks/slider-pro/issues/5
Cheers!


Takes context as a parameter but doesn't do anything with it

Looks like a false trap that context that Take takes as a parameter.

Nevermind, I see that Take does not wait at all so the ctx is useless in this case.

True. I thought Take waits until it can "give" as in other packages and I was looking for a way to stop it with a context in case it waits too much but your implementation does not wait so it's all good.

Ah - yes. Take immediately returns. The boolean returned indicates whether the take was successful. This is documented in the godoc: https://github.com/sethvargo/go-limiter/blob/main/store.go#L34-L35


Interpose by means of a filter

The goal is to remove the need for a user to preload, ldscope or attach. This could be done automatically by means of a list of processes to interpose and possibly a list of processes not to interpose.
Contextually there are 2 paths; preload when a process starts and attach after a process has started. It is likely that both are needed at some level.
Initial research efforts will focus on an ability to interpose any and all processes and apply a filter as provided. The ability to attach to a process after it has started is fairly well understood. There are numerous paths from which this can be accomplished.

Configured an alpine container to preload libscope for every process started after login.
docker run --name musl --privileged -v /home/ubuntu/appscope/:/opt/appscope -it alpine:latest ash
docker exec -it musl ash -l

The "-l" is needed as it causes the profile to be executed. It is not needed for ubuntu.
added to /etc/profile:
export LD_LIBRARY_PATH=/tmp/libscope-1.1.1/
export LD_PRELOAD=/opt/appscope/lib/linux/x86_64/libscope.so

We could document this as a way to enable preload of everything in a docker musl libc env. There are no init.d scripts defined in this container. Probably need to create an alpine VM in order to investigate openrc init in alpine.

A design approach:
We want to start with the context of giving preference to preload on process start over attach after process start. When attaching after process start we lose visibility of start up and a number of limitations. Delineating these is probably better defined in a different communication.
Precedence to preload implies that preload from the loader is used only to load libscope.so. Currently the loader effectively performs loading of libscope.so and interposition of functions because libscope.so is loaded before dependent libs. That behavior will need to change. We currently support the ability to interpose by means of GOT hooking. That is used in an attach operation. It is not a new capability. We will need to disable interposition from the loader and adopt a GOT hooking mechanism for most interpositions.
This shift in interposition approach enables the ability to load libscope.so and choose, based on a user supplied filter, which process will be interposed.
The pathname of the process to be started, as supplied to an exec function, will be compared to a list of processes that should be scoped. There are suggestions that an enable and a deny list should be provided. At this point we are not addressing specifics of the config. Suffice to say that a list of processes is available to libscope. We expect this list to be provided by Edge. It has yet to be determined what default behavior to take if a process list is not available. Coming soon.
At lib constructor time, before the app starts, the executable path is compared to a list and if the process to be started matches, then all the existing functions defined by libscope will be interposed, connections established and events emitted.  This reflects current libscope behavior.
It is expected that Edge will provide a default config in a path that libscope checks. That is beyond the scope of this design discussion at this point. The AppScope team will provide Edge with the definition of an appropriate default config.
At lib constructor time if the app does not match an entry in a process list, functions will not be interposed. The use of GOT hooking makes this possible. In this case libscope will interpose execve (and possibly execv as needed. same behavior). One function is interposed. This enables the capability to track child processes created by a process that is not scoped. No connections for events, metrics or logs will be made. No data is emitted. No activity is tracked. The library exists, but nothing except execve is interposed or in any way interacted. This enables the scenario where a process does not match a process list entry but a child process does match.
Note that a distinction is now being made between preloading and interposing or in colloquial terms scoping. A process can be preloaded and not scoped. Where a process list matches, a process can be preloaded and scoped.
At a high level, there are two categories to be addressed in order to preload potential processes defined in a process list; 1) services and 2) interactive apps. Services are those processes started when the OS boots and controlled by a startup mechanism including systemd or initd. Interactive processes are those started by a user having logged in to a system and executing commands.
There is existing capability in the scope CLI, the service command, to enable systemd and initd services to be scoped. A new CLI command will be added to accept and parse a process list. The process list entries will be compared to services defined on the host. Where there is a match, for example /usr/lib/systemd/system/cron.service matches a process entry for cron, the service will be scoped by means of the CLI service capability. As part of the new CLI command the user will be presented with the option to restart a newly scoped service, attach to an existing service or take no action at that point. The new CLI command will require root perms. It is expected that Edge will execute the new CLI command when it provides a process list.
Interactive apps will be preloaded by one of a few potential mechanisms. Note that a process is not scoped unless there is a match with a process list entry. Also note that there are at least three scenarios to consider when preloading interactive apps; 1) local login, 2) remote login and 3) container start. It is not clear at this point if a single approach is effective in all cases. There is a historic glibc capability to define an /etc/ld.so.preload file that causes all processes to preload an entry in that file. This is not supported by musl libc. Therefore, this is not considered a viable approach. The act of scoping the sshd service works well and supports remote logins. All processes started by that logged in session will be preloaded. There is a great demo of this. Processes started from a local login can be preloaded in one of a couple ways. We are investigating scoping the login service. We have had success in adding two lines to an /etc/profile file. This has proven to enable preloading of libscope for all processes started from a local login and from a container start. Research and POC efforts are in progress to define specific approach.
For reference, for future consideration, the approach defined here enables the ability to consider several states of a preloaded process. Up to now there is one state when scope is enabled; a process has libscope loaded, functions are interposed, connections are established for events, metrics and logs and a periodic thread is executing. The approach delineated here defines a second state; preloaded, one function interposed, no connections established and no periodic thread enabled.
We can now start to consider various states. For example a process could be preloaded and a connection established to Edge or other remote source. In this manner no events are emitted, yet the process can be remotely managed such that a user could request that data start being emitted. Likewise, a process could be preloaded, all functions interposed, a connection to remote management established and no data emitted. A user could then request that data start to flow, which will include all network activity currently in progress, for example. Conversely, a process could be preloaded, no connection established, with a periodic thread executing. This would allow the periodic thread to respond to changes in confg from the local file system in lieu of a remote connection. Numerous options as we move forward.


Complete Overhaul of Non Dynamic Pages

Summary
Non dynamic pages should be re-designed from scratch
Current
Pages look quite old/classical and aren't the most welcoming to new users
Expected
Pages should be completely resigned to look more modern and inviting
Must include a fancy welcome page with easy to read information and search bar

First Part: #218