Nolyfills for @stdlib?

I'm seeing huge graphs in stdlib with deps which are absolutely unreal.
Examples:

@stdlib/assert-is-boolean

@stdlib/assert-is-function

On further inspection it looks like they have accidentally included @stdlib/cli-ctor and @stdlib/fs-read-file in dependencies instead of devDependencies (tracked at https://github.com/stdlib-js/stdlib/issues/2469), but assert-is-function / assert-s-array are still way overboard with poisonous dep chains.

I noticed that assert-is-function behaves differently than typeof, so probably we can't simplify it to typeof x === 'function'.

assert-is-function / assert-is-array are still way overboard with poisonous dep chains.

@hfhchan-plb We don't need to litigate on this issue, but this is a mischaracterization. stdlib simply splits out requisite functionality into separate well-defined APIs. It does not bring in "poisonous" external dependencies. The reason for modularity is to allow code reuse across multiple packages. This, in turn, allows users who depend on more stdlib functionality to benefit from smaller, not larger, eventual bundle sizes, as utilities and helpers are not duplicated across packages.
If you want standalone modules without dependency chains, that is fine; we provide that. Just use our UMD builds (see https://github.com/stdlib-js/assert-is-boolean#installation).

@kgryte Apologies.
While cleaning up deps in a project I am on, I found many packages from @stdlib, many with names looking suspiciously similar to other packages that nolyfill replaces.
This turns out to be due to @segment/tsub lib, which was included in our project recently, resulting in over 98 packages being imported from @stdlib, with many packages referencing each other. I couldn't even load the dep graph without the page lagging every time I scrolled on my mbp M1.
In my initial investigation, I believe some of these deps to be false positives, hence I opened the other issue https://github.com/stdlib-js/stdlib/issues/2469 instead.
Based on your response I believe the core issue is that @segments/tsub (coming from @segment/analytics-next) is pulling in severely outdated versions of libraries. I can confirm the newest version of @stdlib/math-base-special-ldexp no longer has deps on CLI tools.
Unfortunately it looks like segment has paused development for @segment/tsub, while still using it as a dep, and there is already an open ticket requesting the lib to be updated to 0.2.1 (https://github.com/segmentio/tsub-js/issues/45).
Based on that, I doubt upstream will be able to change to UMD builds in a short amount of time.

@SukkaW
There are still a significant amount of indirect deps with the newest version of @stdlib/math-base-special-ldexp. E.g. @stdlib/assert-is-function, @stdlib/assert-is-plain-object, @stdlib/utils-get-prototype-of, @stdlib/math-base-assert-is-infinite etc still have huge chains.
Looking at assert-is-function, which is basically a thin wrapper over utils-type-of, behaviour is changed only to cover Chrome 1 - 12, Safari 8, and PhantomJS (see: https://github.com/stdlib-js/utils-type-of/blob/main/lib/check.js), all three of which are long dead, and deader than Node 12. I suppose it should be safe to replace this with `typeof  x === 'function'.


Document how to wire boards to battery modules (3.4)

This should now be covered by the Electromechinal Subteam.


Fix deselect all

Fixes https://github.com/nt4f04uNd/sweyer/issues/98
Problem was on this line
https://github.com/nt4f04uNd/sweyer/blob/fix/98/playlist-deselect-all/lib/routes/home_route/search_route.dart#L619
Also, here this code couldn't pick up the proper runtimeType
I refactored both places for double-bulletproof + other place where we were checking runtimeType, instead of contentType
Opened an issue to remove all usages of runtimeType
https://github.com/nt4f04uNd/sweyer/issues/137

Also, should we add a test for this to avoid regressing this in the future?

I am trying to write a test for this, but I can't get the last pumpAndSettle to wait for the animation when deselecting everything. Maybe you have an idea what's wrong here.
group('common selection actions on all tabs', () {
    final album0 = albumWith(id: 0);
    final album1 = albumWith(id: 1);
    final album2 = albumWith(id: 2);
    final artist0 = artistWith(id: 0);
    final artist1 = artistWith(id: 1);
    final artist2 = artistWith(id: 2);
    final song0 = songWith(id: 0, title: 'Song 0', albumId: album0.id, artistId: artist0.id);
    final song1 = songWith(id: 1, title: 'Song 1', albumId: album1.id, artistId: artist1.id);
    final song2 = songWith(id: 2, title: 'Song 2', albumId: album2.id, artistId: artist2.id);
    final playlist0 = playlistWith(id: 0, songIds: [song0.id]);
    final playlist1 = playlistWith(id: 1, songIds: [song1.id]);
    final playlist2 = playlistWith(id: 2, songIds: [song2.id]);

    for (var (tabName, contentType) in [
      ('tracks', ContentType.song),
      ('album', ContentType.album),
      ('playlists', ContentType.playlist),
      ('artists', ContentType.artist),
    ]) {
      testWidgets('allows selecting all and deselecting all on the $tabName tab', (WidgetTester tester) async {
        await setUpAppTest(() {
          FakeSweyerPluginPlatform.instance.songs = [song0, song1, song2];
          FakeSweyerPluginPlatform.instance.albums = [album0, album1, album2];
          FakeSweyerPluginPlatform.instance.playlists = [playlist0, playlist1, playlist2];
          FakeSweyerPluginPlatform.instance.artists = [artist0, artist1, artist2];
        });
        await tester.runAppTest(() async {
          HomeRouter.instance.goto(HomeRoutes.tabs);
          await tester.pumpAndSettle();

          // Select tab
          if (contentType != ContentType.song) {
            await tester.tap(find.byIcon(contentType.icon).first);
            await tester.pumpAndSettle();
          }

          // Select first content
          await tester.longPress(find.byType(ContentTile).first);
          await tester.pumpAndSettle();

          var selectionController = ContentControl.instance.selectionNotifier.value!;
          expect(selectionController.inSelection, true);
          expect(selectionController.data.length, 1);
          expect(find.descendant(of: find.byType(SelectionCounter), matching: find.text('1')).hitTestable(),
              findsOneWidget);

          // Select all
          await tester.tap(find.byType(SelectAllSelectionAction).last);
          await tester.pumpAndSettle();

          int numElements = ContentControl.instance.getContent(contentType).length;
          expect(selectionController.inSelection, true);
          expect(selectionController.data.length, numElements);
          expect(find.descendant(of: find.byType(SelectionCounter), matching: find.text('$numElements')).hitTestable(),
              findsOneWidget);

          // Deselect all
          await tester.tap(find.byType(SelectAllSelectionAction).last);
          await tester.pumpAndSettle();

          expect(selectionController.inSelection, false);
          expect(selectionController.data.length, 0);
          expect(selectionController.animationController.isCompleted, true); // This fails
          expect(selectionController.animation.isCompleted, true); // This fails

          expect(find.byType(SelectAllSelectionAction), findsNothing); // This fails
          expect(find.byType(SelectionCounter), findsNothing); // This fails
        });
      });
    }
  });

A have created #142 that adds the tests for this, so we can merge this. 👍

Thanks!


[SGX] Build Migration Updates

[ ] CMake migration
[ ] NNVM example migration

cc @nhynes now that nnvm is merged into tvm, we need to merge nnvm/examples/sgx to apps/sgx as a single example

@nhynes can you act on this issue?

hrm.. the current best way to use TVM in SGX is via Rust. The C++ is still segfaulting for unknown reasons. I could try to update the C++ version, but it's not the preferred interface imho. How would you like to proceed?

Let us update the c++ interface to make sure it is complete

closed by https://github.com/dmlc/tvm/pull/1763


How to change the Background Pic?

How to change the Background Pic?
请问怎样更改背景图？
どのように背景図変更しますか？

css/style.css 第 313 行

thanks
发自网易邮箱大师
On 10/15/2017 20:18, metheno wrote:
Closed #2.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


temp: postgres 14 (instead of 10)

1 surprising snapshot difference:
FAIL src/__tests__/integration/postgresql/introspection.test.ts (8.384 s)
  ● introspection: findUnique where composite PK with foreign key

    expect(received).toMatchSnapshot(hint)

    Snapshot name: `introspection: findUnique where composite PK with foreign key: datamodel 1`

    - Snapshot  - 1
    + Received  + 1

    @@ -18,8 +18,8 @@

      model b {
        id  Int @id @default(autoincrement())
        one Int
        two Int
    -   a   a   @relation(fields: [one, two], references: [one, two], onDelete: NoAction, onUpdate: NoAction, map: "b_one_fkey")
    +   a   a   @relation(fields: [one, two], references: [one, two], onDelete: NoAction, onUpdate: NoAction)
      }
      ↵

      [20](https://github.com/prisma/prisma/actions/runs/6713932243/job/18246392877?pr=21721#step:8:21)1 |       states[scenario.name] = state
      202 |
    > 203 |       expect(introspectionResult.datamodel).toMatchSnapshot(`datamodel`)
          |                                             ^
      204 |       expect(introspectionResult.warnings).toMatchSnapshot(`warnings`)
      205 |
      206 |       await teardownScenario(state)

      at toMatchSnapshot (src/__tests__/__helpers__/integrationTest.ts:203:[45](https://github.com/prisma/prisma/actions/runs/6713932243/job/18246392877?pr=21721#step:8:46))

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Chore docs crawl

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
What is the current behavior? (You can also link to an open issue here)
What is the new behavior (if this is a feature change)?
Does this PR introduce a breaking change?
Please check if the PR fulfills these requirements

[ ] The commit message follows our guidelines
[ ] Fix/Feature: Docs have been added/updated
[ ] Fix/Feature: Tests have been added; existing tests pass

Other information:

I've now allowed es6 for all our node js scripts

@gkalpak @mgol thanks for the reviews!


Not allowing to enter custom value and only showing values configured for the field

When I am configuring the custom control backed by 'Operating System' field which is a multi select string picklist, in the work item I can only see and select values configured for the field. When I click on the 'Add Item' it expands the control to select the existing value but no way to enter new value.
In other hand if I put the field 'Operating System' also in the layout it allows to enter values separated by the semi columns. In this case I get two fields in the Layout.

The control today is designed to only let you pick from the existing list of configured values. The intent is that you update the list to add new values you want to support into it.

When checked and a search string that does not match any of the given options will presented as its own option.
ie if your options are
qwer
asdf
zxcv
and you search
poiu
poiu will be presented as new option to select.


Browser Back/Forward Button Actions Don't Animate

I'm experiencing the consistent issue of <html lang="en-US" class="swup-enabled"> not animating on browser history actions (back and forward buttons, are these referred to as popstates?)
In-page links do work to animate the content by adding the classes of is-changing, is-animating, etc. to the html tag. When I observe what's going on within Chrome element panel I can watch the html tag remain static with no classes added/removed when using the browser buttons. It should be noted that the title and body classes DO change via swup when using browser buttons.
I don't know how to fix this. I've tried the options and I can get no solution to work. Thoughts?

Hi, this is intended behavior by default. Did you check out Animate History Browsing option?

I will explore this and report back.

This info is now here but the point stays. This functionality is implemented, but not used by default.
Please, let me know if there are any problems. Closing


torch.fx of torch==2.5.1 seems to have some limitation with which ThunderFX is not content

Observed in #1456.
This could fall into a category of "won't get fixed" because the nightly looks fine.
🐛 Bug

To Reproduce

Start pytorchlightning/lightning-thunder:ubuntu22.04-cuda12.1.1-cudnn-fe1.5.2-py3.10-pt_2.5.1-dev
Clone crpa/inplace-test-real-adamw branch, more specifically, https://github.com/Lightning-AI/lightning-thunder/pull/1456/commits/f94892c7b8dace84c971b0549b56f1f118eba4da.

Yes, it looks to be the same as https://github.com/pytorch/pytorch/issues/138207. Fix will be available in next release.

Also, currently we have this the skip below for thunderFX optim test which fails for the same reason
https://github.com/Lightning-AI/lightning-thunder/blob/60f3ee1ec536ee8d6fdef503af54525e0a3978a4/thunder/tests/test_dynamo.py#L449-L456

ah, I forgot about the test case. thank you both


missing check-model-parents-schema

I got this error when I try to run this pre-commit. I think it can fix the error

Sorry @miki-lwy . I was solving some family issues. Will look into this tomorrow!

Thanks @miki-lwy! Well spotted!


Pentest - Outdated Dependencies

In our latest Pentest Apexcharts library came up as one of the chief contributor of outdated dependencies thanks to svg.*.js in Version 2+. I have found this reply by Ryan Maffey in response to another topic: https://github.com/apexcharts/apexcharts.js/issues/3278#issuecomment-1361630629
Since we would like to keep on using the Apexcharts library, I would like to know if there's any plans of upgrading to svg.js 3+ (I take that's a no due to complexity involved?) or baking the dependencies directly into Apexcharts (which would also allow solving the other Issue). The way I see it, svg.js 2+ and plugins were not updated for years.

Hey @agaluf
I understand your concern. I already have the Svg.js core library baked into ApexCharts, but the plugins are imported from their respective repositories. Upgrading the Svg.js 3+ would break a lot of things, so I don't plan to update to the latest SVG.js unless they upgrade their plugins to make it Svg.js 3+ compatible.


Fixes #5017

Reference Issue
This PR fixes #5017
What does this implement/fix? Explain your changes.
patchPandasrepr() patches pd.io.formats.html.HTMLFormatter._write_cell(), then it restores the original function before returning. However, while doing so, it also sets an _rdkitpatched attribute on pandas.io.formats.html.HTMLFormatter.
The next time patchPandasrepr() is called, the patch is not applied anymore due to the present of the _rdkitpatched attribute, even though the patch has actually been reverted.
This explains why the DataFrame displays molecules as PNG images only the first time.
The fix is removing the usage of _rdkitpatched attribute entirely as it is not necessary, since the patch is applied and then reverted. Instead, a try..finally block is used to make sure that the patch is reverted before returning even if an exception is raised.
After this PR, running the test described in #5017 yields the desired behavior:
from rdkit import Chem
from IPython.display import HTML
from rdkit.Chem import PandasTools
import pandas as pd

import rdkit
rdkit.__version__
'2022.03.1pre'

df = pd.DataFrame(dict(a=[1], s=['CCN']))
PandasTools.AddMoleculeColumnToFrame(df, smilesCol='s', molCol='m')
df

df

PandasTools.RenderImagesInAllDataFrames(False)
df

df

PandasTools.RenderImagesInAllDataFrames(True)
df

df

The macOS_x64_java failure was caused by Avalon failing to download from SourceForge, nothing to do with this PR.

@ptosco : I merged this without actually trying it, and it doesn't seem to actually work in jupyter:

I've tried it in both jupyter notebook and jupyterlab (v3.4.4) and get the same behavior in both of them
Which version/OS did you test this on?

@greglandrum The fix works, the problem is that I inadvertently reverted it in PR #5132:
https://github.com/rdkit/rdkit/pull/5132/files#diff-4d3e0e17d686a9de85fb42b78a37f8fe69e95258f4e6be45899873fa292a2090L200
because I did not merge #5103 into my rdkit-structure-renderer branch before submitting #5132.
Sorry for the inconvenience, I have just submitted #5381 to restore the #5103 fix.

@ptosco : sorry, I should have caught that in the review... the PandasTools code is a real brain bender though.
It would be super shiny if we could figure out some way to actually automatically test this stuff, but I don't see how that would work.


Create starter / library to support TLS communication between Spring apps (with certs in key vault)

Support loading server certificate for server side TLS Spring app from key vault.
Support loading trust certificate (CA) for TLS client side Spring app from key vault.
#21463
#21464

Closing this issue. Use this issue instead: https://github.com/Azure/azure-sdk-for-java/issues/21566


Refactor executable_cmd macro from attribute into a derive macro

Currently boilerplate code is avoided by using the executable_cmd macro.
This macro is implemented as attribute which is not ideal to say the least.
It should be implemented as a Derive macro as it implements the ExecutableCommand trait.

Inline documentation references attribute macro.
As it's not longer the case it should be changed to reflect the changes.


Question: What is nodesСontract in RouteLayerTestActivity?

Hello, thanks for this project.
I don't understand - What is nodesСontract in RouteLayerTestActivity? How calculate it for my own map?

it's the relation between your nodes. check the following example:
nodesID: 1, 2, 3, 4, 5
nodeContract: (1, 5), (5, 1), (2, 3), (3, 2)
this means that you reach node #5 from node #1 and vice versa, same goes for the #2 and #3; but you can't reach #3 from #1 "it's not in the nodesContract array".
hope that helps :)

Thanks man! You saved me


feat: Implement EventStream using mio

This branch changes the implementation of Stream for EventStream
to use mio and tokio_reactor to register interest in events on
the Inotify file descriptor with the reactor, rather than using
task::current().notify() to unpark the task in the executor when
polling the fd would block.
Since the reactor is intended to drive all IO resources (such as the
inotify FD) and to be the bridge between tasks in the executor and
events from the OS, using PollEvented is a more correct way to
implement the stream.
This branch shouldn't result in any noticeable changes in behaviour,
but will hopefully allow tokio to manage Inotify's IO events more
intelligently.

Thanks for the PR! I'll take a closer look once #104 has been merged.

Now that #104 is merged, this PR is no longer blocked.
@hawkw I'm a bit clueless about mio/futures/tokio, so I'm not too confident in my ability to provide a good review. That said, I can't find anything wrong with this PR.
There's one thing that's keeping me from merging this right now: There's an open PR (#101) that is a follow-up to the initial futures work (#99). It was blocked back then (may still be?), so I couldn't merge right away. It's my impression that your PR supersedes #101. Is that correct?
@mathstuf This PR seems to supersede your open PR #101. Do you have any thoughts on this PR? I'm considering merging this PR and closing #101. Do you see any reason not to do that? Please note that this is based on a previous PR, and the first 4 commits are already merged. As of this writing, only fa6bf8a and 2c63299 are relevant.

First, just FYI, I've rebased this branch onto master, so the diff should no longer include diffs from PR #104.
@hannobraun

It's my impression that your PR supersedes #101. Is that correct?

Yeah, that's correct. I haven't looked too closely at #101, but here are some of the important differences from my perspective:

It looks like #101 uses the old version of tokio (i.e. the tokio-core and tokio-io crates), while this branch uses the new tokio crate. These are incompatible, and IMO since a majority of the ecosystem has migrated to the new tokio, it's better to use that.
On master, Inotify::event_stream method returns a concrete EventStream type. On #101, this changes to impl Stream<...>, while on this branch, we still return a named EventStream type. Personally, I think that while impl Trait is nice to use in applications or in internal code that's not part of a library's public API, it's not ideal for library methods to return an impl Trait, as it limits the way users of the library can use that returned type. For example, if I wanted to put the stream returned by Inotify::event_stream in a struct field, or use it as a type parameter to some other type, I'd have to box it. This could be a potentially breaking change.
#101 depends on the tokio_file_unixcrate, while my branch doesn't. I'm not familiar with that crate, and this isn't a value judgement, just a difference I noticed.

Thank you for the explanation, @hawkw, this is great! I'll give @mathstuf a few days to respond before making a decision, but so far everything looks great.

For what it's worth: we just found that switching to this branch from master fixed an issue where inotify-rs was causing our application to pin CPU use at 100%.

In particular, this line is what causes this stream to burn CPU, since the executor can't leave this stream as parked (since the task is always notified).

@olix0r Thank you for the info. If I remember correctly, we decided to land a preliminary version of futures support, as the more efficient version was blocked for various reasons.
Looking back, this seems like a mistake on my part. We should have documented this extensively and hidden it behind a Cargo feature, at the very least. I'm sorry for the trouble this caused you!

@hannobraun No worries! Luckily we caught this in testing ;)
Thanks for putting together this very useful library.

@hannobraun I think releasing the futures support experimentally was a good choice on your part, given that otherwise, we wouldn't have found this issue at all.
Honestly, I'm mostly just surprised nobody else hit this issue earlier, as I'm fairly sure could occur any time the EventStream was used (rather than being triggered by some particular uncommon condition). I guess nobody was using the futures support seriously enough to care about its performance until us.
By the way, thanks for being so willing to work with me on landing all the patches I've submitted, I really appreciate it!

@olix0r @hawkw Thank you both for your kind words!

I guess nobody was using the futures support seriously enough to care about its performance until us.

Yeah, probably. Judging from the downloads numbers on crates.io, this library must see a fair bit of use. Unfortunately, not a lot gets back to me.

By the way, thanks for being so willing to work with me on landing all the patches I've submitted, I really appreciate it!

No problem. Thank you for the patches!

I'm merging this. Sorry for the additional delay.
@hawkw Thank you again for your contributions!

@hannobraun

Sorry for the additional delay.

No problem at all!

@hawkw Thank you again for your contributions!

My pleasure! It was fun. :D Thanks for all your reviews.

@mathstuf This PR seems to supersede your open PR #101. Do you have any thoughts on this PR?

Sorry, was on an extended vacation. Superseding is fine. This does look more idiomatic than my PR. Thanks @hawkw!


Bug: Sometimes, an error happens on loading screen

Expected Behavior
Redirection to the IAM login screen
Current Behavior
Blank "loading" page, unhandled exception strip, and a stack trace in browser logs
Steps to Reproduce

Context (Environment)
Browser
Mozilla Firefox 91.1.0esr (32-bit)
Container Operating System
Linux containers
Docker
N/A
Logs

Additional information
There are no verified circumstances for when it happens, but in my case it was after coming back from a week end break.

Hi @Metal-Mighty ,
When this error occurs, could you provide us the previous Rest request response ?

@Metal-Mighty , any update ?

@Metal-Mighty and I did not encounter this problem for a while.
@hocinehacherouf @kbeaugrand you can close this issue.


Ensure Unique Work - Wrong URL for REPLACE, KEEP and APPEND docs

There's a typo in the URL the user is sent to when clicking on either REPLACE, KEEP or APPEND.
Currently:
http://d.android.com/eference/androidx/work/ExistingWorkPolicy#replace
Should be:
http://d.android.com/reference/androidx/work/ExistingWorkPolicy#replace

Thanks for reporting this.
We've fixed the links in the codelab document.
Please reopen if you still think this is an issue with the codelab.


update recipe

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

@isuruf if you have a few minutes yo spare can you take a quick look at the Windows failures here. I'm completely lost b/c this used to work :-/


feat(module:modal): update global settings for the modal

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/NG-ZORRO/ng-zorro-antd/blob/master/CONTRIBUTING.md#commit
[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Application (the showcase website) / infrastructure changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: #7458
What is the new behavior?
The NzModalModule now has a forRoot method, where you can set default settings for all kinds of modal.
The settings object can be changed as it is not clear which settings can be global and which can't.
Does this PR introduce a breaking change?
[ ] Yes
[x] No

I think you can use NZ_CONFIG instead of setting these properties with forRoot()

I think you can use NZ_CONFIG instead of setting these properties with forRoot()

Yes, you're right. I have updated pr, added a new field to the nz config.

I think you can use NZ_CONFIG instead of setting these properties with forRoot()

Yes, you're right. I have updated pr, added a new field to the nz config.

Please make sure all tests work as expected

@hsuanxyz Please take time to review :)

@volvachev are you able to add more tests for lines that have been marked as not covered?
https://github.com/NG-ZORRO/ng-zorro-antd/pull/7461/checks?check_run_id=6652702744


Delete duplicated constant

Hi! ☀
I deleted duplicated constant in VisionAPI FeatureTypes.

ordered :+1:

Thanks @nownabe!
This LGTM. @daspecster do we need these hard-coded values or can you lean on protobufs more? (Not part of this PR, but this PR clearly indicates that writing this by hand is problematic.)

@dhermes the goal down the road is to really use the protobufs almost entirely. So that's definitely a good direction to head IMHO.


Make x509 altnames and extkeyusage optional

The defaults of empty arrays are generating invalid openssl configs
Example of what's generated:
[ v3_req ]
extendedKeyUsage  =
subjectAltName    = @alt_names

This throws all sorts of errors:
[root@rocky-8 /]# /opt/puppetlabs/puppet/bin/openssl req -new -key /etc/xdmod/simplesamlphp/cert/xdmod.key -config /etc/xdmod/simplesamlphp/cert/xdmod.cnf -out /etc/xdmod/simplesamlphp/cert/xdmod.csr
Error Loading request extension section v3_req
46912522284800:error:2206D06C:X509 V3 routines:X509V3_parse_list:invalid null name:crypto/x509v3/v3_utl.c:386:
46912522284800:error:22097069:X509 V3 routines:do_ext_nconf:invalid extension string:crypto/x509v3/v3_conf.c:93:name=extendedKeyUsage,section=
46912522284800:error:22098080:X509 V3 routines:X509V3_EXT_nconf:error in extension:crypto/x509v3/v3_conf.c:47:name=extendedKeyUsage, value=

Sorry for the fly by comment, but you may have a look on https://github.com/voxpupuli/puppet-openssl/commit/50317c20da3eabce73436996cd233ac853061cec which is fixing the config template too, I hope.

@rtib What about the updated change to use Optional[Array[String[1]]] ?  That way it's either Undef or a non-empty Array so no need to check if empty as Puppet will validate the parameters first.

The semantics of the empty() function is covering all you need. It is built-in with Puppet, so there is no additional dependency.
Though notions like Optional[Array[String[1]]] seem pretty popular these days, IMO, they are kinda crabbed and still allow undef which you need to check for later. It seems more simple and easier to understand to have an Array or Array[String] with an empty array as default.

The root cause of the problem is lying in the template used to generate the config, not in type signature of the parameters. This is part of what rendered release 3.0.0 unusable (see #178). I'd prefer to not change the module API.

I agree partly with both: we have to fix the template to only set the values (and corresponding keys) in the config, if they are present. Checking for undef is easy enough in a template, default boolean behaviour will take care of that (even without empty).
I also agree with using Optional[Array[String[1]]]. That way only non empty Arrays with non-empty Strings can be passed in. The undef for the default value we can deal with easily in the config template.

... using Optional[Array[String[1]]]. That way only non empty Arrays with non-empty Strings can be passed in. The undef for the default value we can deal with easily in the config template.

Look at it:
Optional[Array[String[1]]] $altnames = undef,
or
Array $altnames = [],
Now imagine you are new to this module, probably new to Puppet, you want to use the module trying to understand its parameters. Which one you'd prefer?

From the software engineering point of view, the real difference between
if $somearray {

and
unless $somearray.empty {

is that the first one is using a hidden, implicit behaviour of the language, while the semantics of the later is fully explicit.

I look at data types as a way to represent what's valid.  If an empty array is not valid, it should not be allowed as a parameter irregardless of how it might look to someone new to Puppet.  Data types should validate the inputs, if empty array is not a valid input, it shouldn't be allowed.  Similar for empty strings.  If empty strings are not a valid input, data types need to prevent them.
I'd imagine a new Puppet user would rather be prevented from invalid inputs or unexpected behavior rather than having an easier time looking at the code.  If they go to look at the code cause they passed empty array and it broke X509 cert creation, it will be a lot harder to debug (as I recently found out) than if they are prevented from that issue to begin with.

Correct, empty array is a valid default in these cases.  @rtib Do you want to open PR with template fixes or do you want me to handle with this PR?

Correct, empty array is a valid default in these cases. @rtib Do you want to open PR with template fixes or do you want me to handle with this PR?

I think this PR is the right place for that. But, we need to test that thoroughly, that commit is just a first take.

I found some time to look deeper into this and did some refactoring:

e1367b92dce56efe6a1508c6b2d3247e31a47fa7 did some code cleanup of x509.pp
798f6746eaf43e98ef39ee1f73e165bf9f9b93fd move the handling of distinguished name attributes a bit towards specification as described in section-<IP_ADDRESS> of RFC5280

Those look pretty good, would you like to provide them in a PR? :)

Opened #179

I think with the merge of #179 and #180, this is no longer needed? @treydock,. can you check that?


Change go version to 1.12.7

Coverage decreased (-0.3%) to 91.9% when pulling 9bfafb476e1cc421eb095b5065394d262cccb591 on go-1.12.7 into 9ecfd62ac540e948e8afbcc815f621687cb25fc0 on master.


Looking forward to script and cfg_file on training a Lookup-FFN transformer.

As I try to run the training script following the command line in README
"""
python3 main.py --config cfgs/roberta/small-512/prenorm.py
"""
A bug occurs: There is no "RobertaEmbeddings" and there is no file "postnorm.py"
Traceback (most recent call last):
File "/home/ma-user/work/lookup_ffn/main.py", line 139, in
main(config)
File "/home/ma-user/work/lookup_ffn/main.py", line 85, in main
model = import_from_string(config.model.pl_module)(config, data)
File "/home/ma-user/work/lookup_ffn/src/roberta/tasks/mlm.py", line 90, in init
self.model = RobertaForMLM(self.model_config)
File "/home/ma-user/work/lookup_ffn/src/roberta/tasks/mlm.py", line 37, in init
self.roberta = import_from_string(config.encoder_type)(config)
File "/home/ma-user/work/lookup_ffn/src/args.py", line 35, in import_from_string
exec(f"import {location}")
File "", line 1, in
File "/home/ma-user/work/lookup_ffn/src/roberta/models/prenorm.py", line 5, in
from .postnorm import RobertaEmbeddings
ModuleNotFoundError: No module named 'src.roberta.models.postnorm'

So I copy the "class RobertaEmbeddings" from https://github.com/mlpen/mra-attention/blob/main/src/roberta/models/postnorm.py#L6, and now it runs normally.
But the training script and config file are for a standard transformer.
It's NOT for a transformer with Lookup-FFN.
Could you please upload the script and cfg_file on how to run a Lookup-FFN transformer (on GPU and CPU) ?

Hello, thanks for interesting at this work and pointing out the bugs. I updated the code and uploaded the config files for lookups.
Thank you!

Thanks for the update, now I can train from scratch normally using the new cfg file.
The C code for bh4 and fast_hadamard_transform can be compiled by ninja on a computer with cuda-11.8.
However, the ninja failed to compile the C code on a computer with cuda-10.2 version.
May I ask how to select the specified cuda version when running the "torch.utils.cpp_extension.load" part ?

Sorry for the late response.
I am not sure how to change the cuda version for compilation. Are there any specific reasons for this specific version? Can't the code be run on cuda-11.8?

I update the cuda to 12.1 and torch to 2.0.1, and the codes run smoothly ~ Appreciate


[FEATURE] Add User/Group to project independent after the project is created

Is your feature request related to a problem? Please describe.
Right now, when we create users, we create the users before the project and then assign the users we have created to the project. One of my users is looking to automate this at a much higher scale and I cannot offer him that because terraform right now only allows member association during project creation.
Describe the solution you'd like
I'm eagerly looking for a project_user_association, very similar to how sonarqube does: https://github.com/jdamata/terraform-provider-sonarqube/blob/master/sonarqube/resource_sonarqube_qualitygate_usergroup_association.go
Describe alternatives you've considered
I have no other option besides providing dozens of system account references back to the project admin and he has to add one by one.

project_user_association and project_group_association looks like it make more sense.

terraform right now only allows member association during project creation

@EvertonSA If users are already created prior to the project creation, these users can be part of the project configuration, no? I am not seeing the problem from your description.
Also, you can add/remove users from the project after it is created by altering the users attribute.

@EvertonSA Now I see the use case. Thanks for the explanation.
I haven't tested this yet, but I think if the access token has the project admin permission set correctly (i.e. with ability to manage users for the project) then the 'team01' can still use project resource to manage the users/groups attribute for that project.
Of course, as you mentioned, that team uses a separate configuration file/state, so they will need to define the project configuration and import the resource first.
The side effect is that the project states are now in 2 places and synchronization may become an issue. I'm not sure if introducing project_usergroup_association resource will solve this issue as this is an inherit trait in your use case.

this decision I cannot take because it seems to be a MAJOR change. If jfrog decides that it makes sense, I would be very happy and my internal customer as well. if not, then we will see internally how to proceed. thanks for the insights!


Bullet & Numbering in Timeline item Model

how i can show cardDetailedText as bullet or numbering text?

hi @ranggabadrus . welcome to react-chrono.
have  you tried using the custom rendering option. does this help
https://github.com/prabhuignoto/react-chrono#rendering-custom-content
https://codesandbox.io/s/react-chrono-vertical-custom-qepnm?file=/src/App.js


Providing properties for wrapping Admonition

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Motivation
This is part of a bigger effort that I want to invest some time in if anyone here is interested in this direction. I want to incrementally make more of the theme-classic components safe for wrapping. As a first step, I decided to look into Admonition and wanted to share my thoughts.
My main motivation here is to provide a a way to use tailwind css for all styles while still using the Admonition components and logic provided by the docusaurus.
Proposal:
Currently, when I create wrappable version of Admonition, I get the following:
import React from 'react';
import Admonition from '@theme-original/Admonition';

export default function AdmonitionWrapper(props) {
  return (
    <>
      <Admonition {...props} />
    </>
  );
}

We can provide a className prop here but this prop does not replace the infima class names that are stored in the class. So, my suggestion here is to provide a method or an object that maps to different class names:
<Admonition {...props} classMap={{
  note: "bg-white-200",
  tip: "bg-yellow-200",
  info: "bg-blue-200",
  caution: "bg-orange-200",
  danger: "bg-red-200"
}} />

Self-service

[X] I'd be willing to do some initial work on this proposal myself.

If we go into this direction, then we could also pass the icon and label as a map, and entirely get rid of the sub components.
That somehow assumes that all admonitions are normalized to a std layout, which in practice may not always be the case considering users can implement their own admonitions.

The root <Admonition/> component is just the entry point that has the responsibility to redispatch props to the appropriate underlying admonition component. Those components should be able to stand alone and be self-contained.
If you want to use Tailwind instead of Infima, you should eject instead of wrap, and change the implementation details you don't like (ie usage of Infima). Wrapping is for "enhancing", not "overriding" internal impl details. <AdmonitionTypeNote/> should be able to render correctly alone, even when not used through the root <Admonition> component, so it should contain the Infima class.


about id loss

I'm curious about your IDloss  looks like different from the original paper which trained with triplet loss using the BatchHard strategy for sampling triplets.

I'm curious about your IDloss looks like different from the original paper which trained with triplet loss using the BatchHard strategy for sampling triplets.

i am sorry,it's just a quick implementation of the network structure,training strategy and loss i didn't refer the paper.


Overlay icons appearing only in shell debugger

I followed all the steps but the overlay icons not appearing, only if I using Shell Debugger. Can someone tell me what I miss?
https://drive.google.com/file/d/0B8lAfI0XqQq7VXBHcjBIeGU2eWs/view?usp=sharing

I downloaded your solution, replaced the content of your CanShowOverlay method with just return true;, compiled in my Windows 7 32bit/Visual C# 2010, loaded the resulting TestSharpIconOverlayHandler.dll in SharpShell Server Manager <IP_ADDRESS>, clicked Install server (x86) then Register server (x86), then started a new Windows Explorer window.
Result: The overlay shows up on every file.
Could you please give more version details about your Windows, compiler, and SharpShell Server Manager? Thanks for the feedback!

I have this exact same problem.
And I can reproduce the issue using the above mentioned code by @Almis90.
My setup:

Windows 10
Microsoft Visual Studio Community 2015
Version 14.0.24720.00 Update 1
Microsoft .NET Framework
Version 4.6.01038
Installed Version: Community
SharpShell: 2.2.0 installed using NuGet
After building the project, I have installed & registered it using SharpShell ServerManager.exe(<IP_ADDRESS>).
The icons showed in debugger once, but now it's not working either.
I'd appreciate some help in debugging this issue. How can I make sure the registration has happened properly and my handler is being invoked?
Regards,
Sandeepa

I'm having the same problem. Even with the sample one on Windows 10.
It shows in the Open Dialog of Server Manager, but does not show. Likewise with a fully patched Windows Server 2012 R2 Server. I'm unable to get icon overlays functioning.
One thing I have not tiered is placing the assemblies in the gac as I've been using code base.

I'm hitting the same problem, on windows 10.
any help?

Double check:
https://github.com/dwmkerr/sharpshell/blob/master/docs/troubleshooting.md
I think what is most likely going on is that you have dropbox or something else installed which has taken over all of the available slots.


Change output window to

Might be worth doing this so that we can embed URLs inside easier.

Not doing this in the end.


fix(onboarding): link to the correct AWS IAM console page

closes #539
//cc @lcouzens

Codecov Report

Merging #579 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #579   +/-   ##
=======================================
  Coverage   82.51%   82.51%
=======================================
  Files         193      193
  Lines        2963     2963
  Branches      492      492
=======================================
  Hits         2445     2445
  Misses        461      461
  Partials       57       57

Impacted Files
Coverage Δ

...c/pages/onboardingModal/iamPolicy/instructions.tsx
100% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Thank you @dlabrecq :raised_hands:


New package: Samsung.RecoverySolution version <IP_ADDRESS>

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] 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/111978&drop=dogfoodAlpha

#111634

Supporting custom hardware in validation is "Not Planned".

https://github.com/microsoft/winget-pkgs/issues/199504


Including jetson.utils in a systemd service file results in core-dump

I can execute my python script and have it run perfectly with
sudo python3 <filename>.py
but with a service file that is setup like
[Unit]
Description=Python Boot Service File

[Service]
WorkingDirectory=/home/<project root>
ExecStart=/usr/bin/python3 /home/<project root>/main.py

[Install]
WantedBy=multi-user.target

results in the following errors
Main process exited, code=dumped, status=4/ILL
Failed with result 'core-dump'

I also know that the problem is including jetson.utils since a made a test.py file that will just do and print some math operations. It worked using the same service file setup until I included jetson.utils.
Is there some special pathing that I am missing to be able to use jetson.utils and jetson.inference in a python file that is setup as a service file?

Have you find the solution? I guess I've met the same problem.

I can execute my python script and have it run perfectly with sudo python3 <filename>.py
but with a service file that is setup like
[Unit]
Description=Python Boot Service File

[Service]
WorkingDirectory=/home/<project root>
ExecStart=/usr/bin/python3 /home/<project root>/main.py

[Install]
WantedBy=multi-user.target

results in the following errors
Main process exited, code=dumped, status=4/ILL
Failed with result 'core-dump'

I also know that the problem is including jetson.utils since a made a test.py file that will just do and print some math operations. It worked using the same service file setup until I included jetson.utils.
Is there some special pathing that I am missing to be able to use jetson.utils and jetson.inference in a python file that is setup as a service file?

Have you found the solution? I guess I've met the same problem.

Why is this closed? I encountered the same problem today. Did anyone find a solution?

I did not find a solution to this problem. This was a school project that I was working that it wasn’t a requirement to get working. It would have been nice to get working. Not sure why it’s closed though.


format=2 transactions without data are invalid

The library computes a data_root even when there is no data. A transaction with data_size=0 and a non-empty data_root is invalid.

This is fixed in v1.8.0


Release version 0.23.0

Just a few small comments.  You can resolve in these release or next.  If next please have a tracking issue.
LGTM

I've run all benchmarks except Bonnie++ (failing, I need to look into it) and SpecCPU2006 (still running) on Google Cloud Platform. If anyone is available to test other providers, that'd be great.

Hi
I have tested the following benchmarks on our Kubernetes cloud (each of them executed twice - using local or Ceph storage):

cluster_boot
copy_throughput
cassandra_stress
cluster_boot
fio
perf
mesh_network
mongodb_ycsb
netperf
redis
sysbench_oltp

They all pass except from one - mongodb_ycsb fails on local storage because of a bug in our version of Kubernetes (https://github.com/kubernetes/kubernetes/issues/2630 - partial fix is already done and will be available in 1.1 release). I haven't tried the rest of the benchmarks so I don't have the results.
These benchmarks were run on the version of code from PR https://github.com/GoogleCloudPlatform/PerfKitBenchmarker/pull/574 - it would be great if it was merged to master in this release as it brings some improvements and stability to K8S provider.

Marcin - One feature of PKB is the ability to create "groups" or as they
are called in PKB sets of workloads.
For example there is an all_set, google_set, etc in this file -
https://github.com/GoogleCloudPlatform/PerfKitBenchmarker/blob/master/perfkitbenchmarker/benchmark_sets.py
The intel_set current is defined as:
'intel_set': { MESSAGE: 'Intel benchmark set.', BENCHMARK_LIST: ['fio', '
iperf', 'unixbench', 'hpcc', 'cluster_boot', 'redis', 'cassandra_stress', '
object_storage_service', 'sysbench_oltp'] },
You may want to update it with the kubernetes workloads.  Then you can run
them like this "./pky.py --benchmarks=intel ..."

@mateusz-blaszkowski - thanks!  I can pull in #574, but would you mind rebasing against master? Right now it has a merge which complicates cherry-picking.

Bonnie++ works.
SpecCPU2006 works.

@voellm - I have created kubernetes_set - I think a set of benchmarks per cloud is a better idea. In case of Mesos we may want to choose another set of benchmarks which we will use for testing.
@cmccoy - done, thanks

@mateusz-blaszkowski: cherry-picked #574 in d8bd9f10f8aec820a573238cb9eb6c82eac18ae. Thanks!


Tooltip text unreadable

Describe the bug
Some users have seen the tooltip text change to other colors, sometimes to a white or other light color making the text unreadable.
TheVastator#5275 gave me details on Discord.
Workaround
In the module folder for Illandril's Token Tooltips (located inside the Foundry data folder), you should see a styles.css file (among other things). Add these lines to the end of that file:
.illandril-token-tooltips--tooltip {
    color: #000;
}

Then close and reopen Foundry. That should fix it temporarily while I work on a permanent solution.

The conflict for TheVastator was the darksheet 5e module, which set the body color to white.


Filter problems

I have added the possibility to filter the problems you want to get, for instance to make some tests.
You simply have to define a logical condition with the combination of symbols and three operators: !, | and &, respectively for the negation, the disjunction and the conjunction.
Here is an example to get the problems whom description contains :toto but not the pair :exponential and :energy.
@Problems !(:exponential & :energy) | :toto

Would it be better to define the following operators?

!
||
&&

Something else?

👍🏽 since there is no short circuit semantics, & and | seem fine


add evaluate code

In addressing #1, the R code is so slow and hard to understand (for me), so I need a review to double-check the math of my implementation. I'm pretty sure we can get expected weights for each time series from M5Data.get_aggregated_ma_dollar_sales() method. So I only need to review that given weight, wrmsse are wspl are computed as expected.
There are some issues which I will comment below.
TODO

[x] add a smoke test

@fritzo Things look good to me. I'm going to construct a model now.


Browser gem not working in Sinatra

Gem is not detecting chrome browser not it is detecting my platform i.e linux.
browser = Browser.new(:accept_language => "en-us")
browser.platform => returning empty string
browser.name => always returns other
any help would be appreciated.

You're not passing the user agent. Try something like this:
browser = Browser.new(ua: request.user_agent, accept_language: request.accept_language)

Thank you for your prompt response.
Yes it works like a charm.
Kindly change the documentation on the read me page.
browser = Browser.new(:ua => "some string", :accept_language => "en-us")
Kindly change it to
browser = Browser.new(ua: request.user_agent, accept_language: request.accept_language)


chore(deps): update ffi to >=1.1.2 <3.0.0

Would really like this change too.

Thanks so much -- I missed this project! Will publish now.

https://pub.dev/packages/win32_registry 1.0.2 is now live. Thanks so much for the pull request, @Zekfad!


Restrict wiki editing to collaborators only

Duplicates

[X] I have searched the existing issues

Current behavior 😯
Random GitHub users are able to edit the wiki.
Expected behavior 🤔
They shouldn't be able to do that 😆
Steps to reproduce 🕹
No response

There should be a setting "Restrict editing to collaborators only"

done 👍

Just reversed https://github.com/o2sh/onefetch/wiki/Installation/_compare/dd6be88^...dd6be88 🙂

Just reversed https://github.com/o2sh/onefetch/wiki/Installation/_compare/dd6be88^...dd6be88 🙂

:scream:

Hey, I'd like to commit some changes to my script in the wiki, but I can't find a way to create a pull request for it.
Can you point me in the right direction to do that?
Thanks for featuring me btw, I noticed it just now and I felt I had to make some adjustments since it's officially in your project.

@quazar-omega Funny, I literally just created this repo before viewing your comment :laughing:
Assuming we don't want to wait for me to write the initial implementation, this action looks promising (but unmaintained). @o2sh What do you think?
TBH we don't even need an action (I was going to write it as a pure YAML composite action) if we wanted to do this.  That action looks pretty good!
Either way you want to tackle this, I think I'll be able to wait to publish my changes to do so properly, I'll have to keep my hands tied in the meantime (  ⌣̀_⌣́)
I think we can move this conversation to a new issue at this point

#925


narrow: Expand descriptive text when starred and mentioned views are empty.

Expands the descriptive text for both the starred messages and mentions views when they are empty.
Notes:

Because this text is much longer than what's normally in these empty narrow banners, the max-width for the empty narrow feed is now set to 600px. See relevant CZO design discussion.
I edited the text for the starred view from what's specified in the issue (see fixes note below). Specifically, I changed the instruction for how to star a message to better match what we say in the help center.
Instead of creating a new hbs template for these empty narrows, I put the longer text in some constants at the top of web/src/narrow_banner.ts so it was easier to read in the file.

Fixes #29378.

New strings to be translated in translations.json:

"Starring messages is a good way to keep track of important messages, such as tasks you need to go back to, or useful references. To star a message, hover over a message and click the <star-icon></star-icon>. <z-link>Learn more</z-link>"
"This view will show messages where you are mentioned."
"To call attention to a message, you can mention a user, a group, topic participants, or all subscribers to a stream. Type @ in the compose box, and choose who you'd like to mention from the list of suggestions. <z-link>Learn more</z-link>"

Screenshots and screen captures:

Updated empty starred messages view

Updated empty mentions view

Updated empty mentions view with wider browser window

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Looks good to me!

Looks good to me in manual testing! I tried super-wide windows (including using full width), and everything looked reasonable.

Merged, thanks @laurynmm!


using API for current POST every 10 seconds needed?

I am using Smart EVSE V3 Version: SERKRI-1.4.4 in combi with Home Assistant.
And in stead of the p1-cable I am using the API with POST/currents.
In my configuration:
laadpaal_share_p1:    url: http:///currents?L1={{ states('sensor.current_phase_l1')|float10  }}&L2={{states('sensor.current_phase_l2')|float10  }}&L3={{  states('sensor.current_phase_l3')|float*10  }}
method: POST
In my automation:
action:
service: rest_command.laadpaal_share_p1
This works fine.
But I have learned that I have to POST at least every 10 seconds otherwise I will get an "Error: Communication Error" which will be solved with one (or some) new POST(s). My intention was to POST only when the current changes and the smart EVSE is <> OFF. I can understand that the Smart EVSE wants an update regularly. Is this 10 seconds maximum (and always needed even when it is OFF) or is there an option to optimize and mimimize the number of POSTs?

That change is implementing a safety feature: if you cannot provide accurate current information every 10 seconds, SmartEVSE cannot do what it promises (guard currents in Smart mode, dynamic charging in Solar mode).
Homeassistant is ideal to read/write non-timecritical data, but (AFAIK) not meant for these kinds of update intervals. So better have some script load your P1 data directly into SmartEVSE (it monitors every 100ms), and from there load it into Homeassistant ....


Search_Algorithms folder in Algorithms

Submission Title
Exponential Search/Sublist_Search
Submission Type
Algorithm/Search_Algorithms
Submission Description
Exponential Search : We first find the range where the element can be found and then apply binary search in that region. The idea to find the range is to start with subarray size 1, compare its last element with x, then try size 2, then 4 and so on until last element of a subarray is not greater. Once we find an index i (after repeated doubling of i), we know that the element must be present between i/2 and i (Why i/2? because we could not find a greater value in previous iteration)
Sublist Search : We first take first node of second list, start matching the first list from this first node, if whole lists match return true, else break and take first list to the first node again and take second list to its second node. Repeat these steps until any of linked lists becomes empty. If first list becomes empty then list found else not.
Submission Directory
Algorithms => Search_Algorithms => Exponential_Search => exponential_search.cpp
Algorithms => Search_Algorithms => Exponential_Search => exponential_search.php
Algorithms => Search_Algorithms => Sublist_Search => sublist_search.cpp
Algorithms => Search_Algorithms => Sublist_Search => sublist_search.java
Algorithms => Search_Algorithms => Sublist_Search => sublist_search.py
Contributors

[x]  C++ -
[ ]  C -
[x]  Python -
[x] Java -
[ ] JavaScript -
[ ] TypeScript -
[ ] Golang -
[ ] Ruby -
[x] php -
[ ] Swift -
[ ] C# -

Can I work on it ?

@RgnDunes Please don't just add all the labels that don't relate to the issue at all. Please remove the labels that shouldn't be here.

@RgnDunes Please don't just add all the labels that don't relate to the issue at all. Please remove the labels that shouldn't be here.

How to delete labels, I don't have much idea, this is my first Hactoberfest.

Hi! Can I take up this issue in C++ language?

@RgnDunes Please don't just add all the labels that don't relate to the issue at all. Please remove the labels that shouldn't be here.

How to delete labels, I don't have much idea, this is my first Hactoberfest.

That's OK! To the right of the issue, where you see all the labels. There should be a gear. Click on the gear and you will see a menu pop up with all the labels. Un-check the ones that aren't necessary. I would recommend getting rid of all the labels that are a programming language and keeping everything else.
Here is what the gear should look like next to the labels

I just saw that a lot of other issues have a bunch of language tags too. You don't want to include a tag unless the question is actually specifically related to that. If it's just "do it in any programming language" then it does NOT specifically relate to those programming languages and thus shouldn't have that as a label.

@RgnDunes Please don't just add all the labels that don't relate to the issue at all. Please remove the labels that shouldn't be here.

How to delete labels, I don't have much idea, this is my first Hactoberfest.

That's OK! To the right of the issue, where you see all the labels. There should be a gear. Click on the gear and you will see a menu pop up with all the labels. Un-check the ones that aren't necessary. I would recommend getting rid of all the labels that are a programming language and keeping everything else.
Here is what the gear should look like next to the labels

No gear option bro,

image

Can you please assign it to me for Java?
@ahampriyanshu

Assign this to me in Python please.

@ahampriyanshu I would like to contribute for this issue in C#.Please assign it to me in C#.

@ahampriyanshu exponential search for C++ has already been completed by someone. Should I implement this in python?

thanks everyone :v:


Investigate using Apache Arrow (or other interchange formats)

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/2616
Originally created by @tayloramurphy on 2021-03-01 12:25:03

As mentioned in https://news.ycombinator.com/item?id=26284253#26300348, it would be interesting to see how the Singer Spec could evolve to support other interchange formats such as Apache Arrow.

View 4 previous comments from the original issue on GitLab


terminal gives error when up arrow key is pressed

whenever up arrow key is pressed more than 10 times in a row , terminal gives out following error -
System.ArgumentOutOfRangeException: The value must be greater than or equal to zero and less than the console's buffer size in that dimension.
Parameter name: top
Actual value was -44.

Please do not remove the issue template. Which version of Windows Terminal are you using? Are you using CMD, PowerShell or WSL?

If this is in PowerShell, please try upgrading PSReadLine as recommended in https://github.com/PowerShell/PSReadLine/issues/1306.

Please do not remove the issue template. Which version of Windows Terminal are you using? Are you using CMD, PowerShell or WSL?

Windows Terminal version 1.2.2381.0
I'm using powershell

As the error message indicates, you must report the issue on the PSReadline repository.
For now, /dup https://github.com/PowerShell/PSReadLine/issues/1306


Help message per commands for Storm client

Updated bin/storm:

usage message update
help message per commands (e.g. bin/storm help ui)
simple error handling
fix some Python misc

Sent. Actually, I don't know if my signature is enough, but I can't find easy way to put my own one.

HelloFax lets you draw your signature by clicking the "Add signature" button when editing the document. It would be great if you could resend it with an actual signature. Sorry for all the hassle!

Resent. It was very interesting adventure :)


Add a "ledger-app" collection of nix code, containing the common code

I am skeptical we want to do this sort of deduplicatation. ledger-app just happens to be how we do things today, and a bit all-singing all dancing more "all singing, all dancing framework entry-point" than "does one thing well library function". This creates something which is a compatibility hazard going forward.
What I instead had in mind is that the stencil app would be merged into the history of the actual apps. If there was a stencil app improvement we wanted to "broadcast" to each app we can just merge a later version of the stencil app into each app that can take the change without bad conflicts.
This keeps reflex-platform less opinionated---more "mechanism" than "policy" in addition to being more "library" than "framework"---while still allowing us to not struggle completely manually sinking fixes back and forth.
If you look at a few of the apps git histories today, you will see that do have a proto-stencil-app common ancestor.

@Ericson2314 Lets consider a scenario, we bump the nixpkgs in ledger-platform and this necessitates some minor changes to the extraRustcOpts.
Should this change go via the stencil app -> merge stencil app in apps -> bump ledger-platform, or would it be better to have this tied to the ledger-platform?

@dfordivam
overridesForRootCrate =  attrs: let
    sdk = lib.findFirst (p: lib.hasPrefix "rust_nanos_sdk" p.name) (builtins.throw "no sdk!") attrs.dependencies;
  in {
    preHook = gccLibsPreHook;
    extraRustcOpts = attrs.extraRustcOpts or [] ++ [
      "-C" "link-arg=-T${sdk.lib}/lib/nanos_sdk.out/script.ld"
      "-C" "linker=${pkgs.stdenv.cc.targetPrefix}clang"
    ];
  };

is a function that does one thing that I would be happy to see put in ledger-platform for that purpose. We can "bottom up" find duplicated bits of code that are self-contained and become nice single purpose functions, but I don't think we shouldn't "top down" deduplicate a wide variety of things all at once as this ledger-app function does.


Run two parallel simulations of a gz-sim-yarp-plugins-enabled robot

Closes #147

In https://github.com/robotology/gz-sim-yarp-plugins/pull/153/commits/b460db8597023f5573ad90c78a11f28540919827, two simualtions of the same world are instantiated and launched together, using TestFixture class provided by gz-sim.
Notes:

In this preliminary attempt the model does not contain any gz-sim-yarp-plugin;
Running the server with the blocking parameter set to false allow to run each simulation in a separate thread.
I'm having difficulties to extend the method using a vector of TestFixtures, because when I try to call the emplace_back passing the world path, I get a compilation errors. I'm trying to debug this.

With https://github.com/robotology/gz-sim-yarp-plugins/pull/153/commits/8886369671e3827384dcd95bbd7f6994aa05e89b the unit test ControlBoardOnMultipleGazeboInstances uses a coupled pendulum model with a controlboard plugin.
Now the unit test fails, since there is an error when trying to set two devices in DeviceRegister  with the same key.
It is then time to make a modification to the way the keys are generated.
@traversaro if I recall well you suggested prefixing the current device key with the memory address stored into the pointer in the call to:
DeviceRegistry::setDevice(std::string deviceDatabaseKey, yarp::dev::PolyDriver* device2add)

Moreover, I think that all the plugin singletons key should be updated in the same way, since for example, we should be able to store the data for the same controlboard device belonging to the same model used in two parallel simulations.
Do you foresee any troubles in progressing on this path?

In the last 5 commits I've restructured the data management mechanisms of the plugins that previously made use of the singletons (see https://github.com/robotology/gz-sim-yarp-plugins/issues/147#issuecomment-2049366584). Now the only singleton in the codebase is constituted by the DeviceRegistry class.

With https://github.com/robotology/gz-sim-yarp-plugins/pull/153/commits/c345e4cf30d89a92a37a45c3802db81fa877803a I updated the unit test to use a double pendulum, such that the reading of the second joint would present numerical differences pretty quickly.
Here's a video of the simulation of such model:
https://github.com/robotology/gz-sim-yarp-plugins/assets/57228872/7e1bb59d-a7eb-494b-9908-24f3e222299b
Now to make the unit test pass I used the uglier way possible, but also the only one that seems to work at the moment, i.e. using the ECM memory address in the device id (along the scoped name of the device and the yarpDeviceName property).
I'm restructuring the code and the unit test in order to make them pass with the new modifications introduced, thing that is taking a bit of time.

There are still three tests failing:

ControlBoardTorqueControlTest (SEGFAULT)
ControlBoardPositionDirectControlTest (Subprocess aborted)
ControlBoardPositionControlTest (SEGFAULT)

What was causing the strange errors (really difficult to debug) the unit test https://github.com/robotology/gz-sim-yarp-plugins/actions/runs/8718567060/job/23916088302#step:12:217 was the use of the TestHelpers method https://github.com/robotology/gz-sim-yarp-plugins/blob/9dc1044dc307742d4c5e7ed45d31392931f2440a/tests/test-helpers/TestHelpers.hh#L14-L32
I created this to help retrieve only a vector of devices already casted to the interface needed in the test. For example, if the models has different gz-sim-yarp-plugins defined, I want to perform tests only on ControlBoardDriver devices, but the way in which I implemented it does not work and make the tests crash in unexpected ways.

This is the outcome of the unit test in which in one simulation the reference position is 10 degrees, and in the other is 5 degrees:
Start 10: ControlBoardOnMultipleGazeboInstancesTest

10: Test command: /home/acroci/repos/gz-sim-yarp-plugins/build/bin/ControlBoardOnMultipleGazeboInstancesTest
10: Working Directory: /home/acroci/repos/gz-sim-yarp-plugins/build/tests/controlboard
10: Environment variables:
10:  GZ_SIM_SYSTEM_PLUGIN_PATH=/home/acroci/repos/gz-sim-yarp-plugins/build/lib
10: Test timeout computed to be: 1500
10: Running main() from /home/acroci/repos/gz-sim-yarp-plugins/build/_deps/googletest-src/googletest/src/gtest_main.cc
10: [==========] Running 1 test from 1 test suite.
10: [----------] Global test environment set-up.
10: [----------] 1 test from ControlBoardOnMultipleGazeboInstances
10: [ RUN      ] ControlBoardOnMultipleGazeboInstances.StartConcurrentGazeboInstances
10: [WARNING] File specified with a relative path:  ../../../tests/controlboard/conf/gazebo_controlboard_multiple_instances.ini , resolved to:  /home/acroci/repos/gz-sim-yarp-plugins/build/tests/controlboard/../../../tests/controlboard/conf/gazebo_controlboard_multiple_instances.ini . It is recommended to use a URI.
10: [INFO] Yarp configuration file loaded:  /home/acroci/repos/gz-sim-yarp-plugins/build/tests/controlboard/../../../tests/controlboard/conf/gazebo_controlboard_multiple_instances.ini
10: [INFO] gz-sim-yarp-controlboard-system: configuration of device  controlboard_plugin_device2  loaded
10: [DEBUG] gz-sim-yarp-controlboard-system : robot scoped name:  model/triple_pendulum
10: ==================== ecm address for generated id: 0x58490cc5e5e8
10: [DEBUG] gz-sim-yarp-controlboard-system : deviceId:  0x58490cc5e5e8/model/triple_pendulum/controlboard_plugin_device2
10: [DEBUG] gz-sim-yarp-controlboard-system: initialConfiguration:  0.0
10: [DEBUG] |yarp.dev.PolyDriver|gazebo_controlboard| Parameters are (LIMITS (jntPosMax (10000.0)) (jntPosMin (-10000.0))) (POSITION_CONTROL (controlUnits metric_units) (controlLaw joint_pid_gazebo_v1) (kp (3000.0)) (kd (2.0)) (ki (0.100000000000000005551)) (maxInt (99999)) (maxOutput (99999)) (shift (0.0)) (ko (0.0)) (stictionUp (0.0)) (stictionDwn (0.0))) (VELOCITY_CONTROL (controlUnits metric_units) (controlLaw joint_pid_gazebo_v1) (velocityControlImplementationType integrator_and_position_pid) (kp (500.0)) (kd (2.0)) (ki (0.100000000000000005551)) (maxInt (9999)) (maxOutput (9999)) (shift (0.0)) (ko (0.0)) (stictionUp (0.0)) (stictionDwn (0.0))) (controlBoardId "0x58490cc5e5e8/model/triple_pendulum/controlboard_plugin_device2") (device gazebo_controlboard) (initialConfiguration "0.0") (jointNames (joint1)) (yarpDeviceName controlboard_plugin_device2)
10: [INFO] |yarp.dev.PolyDriver|gazebo_controlboard| Created device <gazebo_controlboard>. See C++ class ControlBoardDriver for documentation.
10: [INFO] Found 1 joints from the plugin configuration.
10: [INFO] Found 2 joints from the model description.
10: [INFO] Joint  joint1  added to the control board data.
10: [WARNING] Group TRAJECTORY_GENERATION not found in plugin configuration.  Defaults trajectory generation reference speed and accelerations will be applied
10: [DEBUG] Joint  joint1  trajectory generation reference speed:  10  [deg/s]
10: [DEBUG] Joint  joint1  trajectory generation reference acceleration:  10  [deg/s^2]
10: [INFO] Initial configuration found, initializing trajectory generator with it
10: [INFO] Registered YARP device with instance name: 0x58490cc5e5e8/model/triple_pendulum/controlboard_plugin_device2
10: [WARNING] File specified with a relative path:  ../../../tests/controlboard/conf/gazebo_controlboard_multiple_instances.ini , resolved to:  /home/acroci/repos/gz-sim-yarp-plugins/build/tests/controlboard/../../../tests/controlboard/conf/gazebo_controlboard_multiple_instances.ini . It is recommended to use a URI.
10: [INFO] Yarp configuration file loaded:  /home/acroci/repos/gz-sim-yarp-plugins/build/tests/controlboard/../../../tests/controlboard/conf/gazebo_controlboard_multiple_instances.ini
10: [INFO] gz-sim-yarp-controlboard-system: configuration of device  controlboard_plugin_device2  loaded
10: [DEBUG] gz-sim-yarp-controlboard-system : robot scoped name:  model/triple_pendulum
10: ==================== ecm address for generated id: 0x58490e649a88
10: [DEBUG] gz-sim-yarp-controlboard-system : deviceId:  0x58490e649a88/model/triple_pendulum/controlboard_plugin_device2
10: [DEBUG] gz-sim-yarp-controlboard-system: initialConfiguration:  0.0
10: [DEBUG] |yarp.dev.PolyDriver|gazebo_controlboard| Parameters are (LIMITS (jntPosMax (10000.0)) (jntPosMin (-10000.0))) (POSITION_CONTROL (controlUnits metric_units) (controlLaw joint_pid_gazebo_v1) (kp (3000.0)) (kd (2.0)) (ki (0.100000000000000005551)) (maxInt (99999)) (maxOutput (99999)) (shift (0.0)) (ko (0.0)) (stictionUp (0.0)) (stictionDwn (0.0))) (VELOCITY_CONTROL (controlUnits metric_units) (controlLaw joint_pid_gazebo_v1) (velocityControlImplementationType integrator_and_position_pid) (kp (500.0)) (kd (2.0)) (ki (0.100000000000000005551)) (maxInt (9999)) (maxOutput (9999)) (shift (0.0)) (ko (0.0)) (stictionUp (0.0)) (stictionDwn (0.0))) (controlBoardId "0x58490e649a88/model/triple_pendulum/controlboard_plugin_device2") (device gazebo_controlboard) (initialConfiguration "0.0") (jointNames (joint1)) (yarpDeviceName controlboard_plugin_device2)
10: [INFO] |yarp.dev.PolyDriver|gazebo_controlboard| Created device <gazebo_controlboard>. See C++ class ControlBoardDriver for documentation.
10: [INFO] Found 1 joints from the plugin configuration.
10: [INFO] Found 2 joints from the model description.
10: [INFO] Joint  joint1  added to the control board data.
10: [WARNING] Group TRAJECTORY_GENERATION not found in plugin configuration.     Passed   12.39 sec

I want to iterate a bit on the DeviceRegistry data structure, I will put again this PR into a draft.

The problem was probably some kind of consequences of us breaking the one definition rule (that is the reason why you always include .h header files with declarations or inline definitions and never .cpp with non-inline definitions). Anyhow, if this now works that is great, is this ready for review?

The problem was probably some kind of consequences of us breaking the one definition rule (that is the reason why you always include .h header files with declarations or inline definitions and never .cpp with non-inline definitions). Anyhow, if this now works that is great, is this ready for review?

Yep, I agree with you, I had an unpleasant meeting with "undefined behavior"  in C++ 🤣
Yep it's ready for review!

The unit test failing on the CI is quite scary:
ODE INTERNAL ERROR 1: assertion "!colliders_initialized" failed in dInitColliders() [collision_kernel.cpp:168]

gz-sim related issue: https://github.com/gazebosim/gz-sim/issues/18
The unit test failing is launching two parallel Gazebo instances of different models (one sphere and one box bouncing on the ground). This error seems related to the ODE collision detector having issues due to some global variable. However it seems to happen only on the apt CI workflow on Ubuntu@Debug.

Ah, that is an old foe. For parallel workflows we should either use another collision detection algorithm when using dart (dart supports several of them if I recall correctly), or using another physics engine. cc @diegoferigo as probably we experienced something similar when working on gym-ignition.

Now the models used by concurrent tests use dart both as physics engine and collision detector. All the tests are passing.
Merging 🚀

That is nice for the tests, but it also means that users will start getting the problem if they start using our APIs for parallel computation. We should at least document this once we will document the possibility of running multiserver. See also the discussion in https://github.com/gazebosim/gz-sim/issues/18#issuecomment-2067252636 .


fix(cli): address snapshot bugs

Description

Print gas diff percentage when lower than 1%, addresses cases where it was erroneously displaying "gas used is 0% more"
Switches new with old to reverse the comparison order as asked by Fridrik
Fixes Exit code not being 0 when the gas usage is different
Prints the new gas usage after the percentage difference from the .gas-snapshot

Addresses #96 and #98

I was wondering if maybe it would be better to always print absolute value differences along with percentages ? Something along the lines of:

"Target.wasm::TestMethodParameter: gas used is 17% less ( 15345634 > 124523 )"

yup makes sense, done :)


New feature: locale for messages

Hi,
is it possible to include locale for messages? or some way to customize them.

Yes, you just have to use your own indicators.


Feature: Publish clip names to companion as variables

Hi
This feature publishes the clip names of playoutbee to companion. The names could then be used on buttons by specifying the variable in the title of the button.
In addition to mention that I was not able to test it on a version of Playoutbee 0.9.4.
Companion: 2.2.0 (2.2.0-86711173-3681) --> Experimental build
Playoutbee: 0.9.3
Regards
André

VLC has a configuration option to specify how many clip name variables to reserve. They preset to blank or '·'. This enables you to make a page with 10 Play # buttons that will show the actual clip name from VLC or blank instead of $NA if there are fewer clips.


Roll Flutter Engine from 15959feeacd6 to 040655beb617 (1 revision)

https://github.com/flutter/engine/compare/15959feeacd6...040655beb617
2022-07-08<EMAIL_ADDRESS>Roll Skia from fb1fe33fc2f6 to 663ec474bbb8 (2 revisions) (flutter/engine#34539)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Approving goldens because its different by a single pixel


Increase robustness of MVN commands in pre-merge  scripts [databricks]

Signed-off-by: Peixin Li<EMAIL_ADDRESS>part of #6289 for premerge test. Add retry logic to avoid instant fail w/ mvn repo service or unstable networking
NOTE: this update would not help if above issues would not be resolved in a short time. We still need to manually try rebuild after the service is back to normal
FYI update of nightly build and deploy will come later w/ some more testing

build

build

build

build


Update ips.md

Description of the change: Removed 'Dedicated' from description of IP Addresses section in the UI
Reason for the change: Was inaccurate
Link to original source: ?
@ksigler7

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Hello @lweinig,Thanks again for the PR! We want to show our appreciation by sending                                              you some swag. Could you please fill out                                              this form so we can send it to you? Thanks!                                             Team SendGrid DX


Server-side rendering test script

Description
Created a package.json script to see if particular Covalent components will render successfully server side. If it fails the terminal error will indicate what issues need to be resolved. Currently the script tests a single Covalent component, <td-message>, just to get us started. The messaging component correctly fails. It fails because we are referencing window inside @covalent/core which is not available server side. Adding additional components to test in the future is straight forward.
Note: I tested the script against the material button component and it successfully passed/rendered server-side.
What's included?

New script npm run combat-training

Test Steps

[ ] npm run reinstall
[ ] npm run combat-training
[ ] Verify this error appears:

covalent/node_modules/@covalent/core/core.umd.js:7
...
ReferenceError: window is not defined
...

General Tests for Every PR

[ ] ng serve --aot still works.
[ ] npm run lint passes.
[ ] npm test passes and code coverage is not lower.
[ ] npm run build still works.

Screenshots or link to CodePen/Plunker/JSfiddle

Great work！


Make a polling task timeout configurable

An internal task poller currently hardcodes the polling task timeout to 70 seconds. However, it should be set to dynamicconfig.MatchingLongPollExpirationInterval + some delta to account for the full round trip to matching.dynamicconfig.MatchingLongPollExpirationInterval (default is 60 seconds) is the maximum wait time for a new task, after which the server will return empty.
This change will make the polling task timeout configurable when the poller is created.

Can you share the use case for altering the polling timeout? If we do allow customizing this option, we have to do this for all SDKs, not just Go.

Connectivity between Temporal workers and the Temporal cluster goes through a few additional layers. One of them that is a gateway has configured a timeout setting that is less than the default value of 70s. This results in a 504 Gateway timeout error when there are no tasks in queue. I believe deploying workers behind a gateway is a common pattern.

thanks @cretz for your clarification and raising this concern to the team. I completely agree with your points that we should prioritize the platform consistency across SDKs and also between self-host/cloud users. I'll discuss alternative solutions with the team internally. I'll close this PR as well.


RetinaNet: ValueError when using tf.distribute.MirroredStrategy()

The error message is as follows:  ValueError: SyncOnReadVariable does not support assign_add in cross-replica context when aggregation is set to tf.VariableAggregation.SUM.
I'm using tensorflow 2.5 and I'm trying to run RetinaNet as in the Keras tutorial using the provided COCO dataset. The error occurs when calling model.fit().
I would appreciate any insight on this.

Found the problem. Not only model.compile() has to be placed within strategy.scope(), also the model instantiation, as follows:
with strategy.scope():

    resnet50_backbone = get_backbone()
    loss_fn = RetinaNetLoss(num_classes)
    model = RetinaNet(num_classes, resnet50_backbone)

    adam_optimizer = tf.keras.optimizers.Adam()

    model.compile(loss=loss_fn, optimizer=adam_optimizer)


Add popup for all promotions and demotions

Sadly, no desktop notification will be created by user.popup().
Might add that, too, if I find a good way to do it.

still missing a space in one of the messages - I'll amend it in a bit.

you need to rebase(there's a guide on the roomintro of the room)

Do I? Looks like just updating it was fine?

so it's not 4 commits for what could easily fit in 1

I think you meant fixup

Please do not create new pull requests; just force-push to the pull request you already have open. Creating new pull requests spams all 156 subscribers to this repository and is completely unnecessary.

This is also a bad idea in private rooms and groupchats, where people could promote/demote you to spam you.

Ahem:

This is also a bad idea in private rooms and groupchats, where people could promote/demote you to spam you.

Make it such that only if the user is in the room, will you get a popup

I'm fine with the above change.

If all concerns (see below) have been addressed, this should be ready to merge(?)
--

Zarel commented 16 hours ago
This is also a bad idea in private rooms and groupchats, where people could promote/demote you to spam you.

Changed it to work in public, non-personal, non-battle rooms only. (That's what I meant to do anyway, but when I got to it yesterday, I forgot.)
Note that this was indeed a problem for demotions in private rooms, which would produce popups before and could be used to spam.
--
@sirDonovan: room.id and testing for (!targetUser) have been added.

Other than those two, this looks good!

Can I get an MIT license from you? Repeat after me:

I, ____, hereby license my contributions to the Pokémon Showdown project, past and future, under the MIT license, available in the LICENSE file in this repository.

I, Daniel Kilian, hereby license my contributions to the Pokémon Showdown project, past and future, under the MIT license, available in the LICENSE file in this repository.


Render error on /variant/MCNV_1_81?dataset=gnomad_sv_r2_1

What you did:

What happened:

Hi @vbpastor. Thanks for the bug report. This should be fixed now.


payload_stage1 missing or exceeds max size

i get this error and im on 2.1, help me please!

Did you prep your SD card according to this step:
Copy the a9lh folder from data_input_v3.zip to the root of your SD Card


non-default builds seem to be really hard

I'm trying to hack my way around it.
I just want to build a local copy of the library etc.
All the use of cmake envirables is making it difficult.
e.g.:
CMAKE_INSTALL_PREFIX=~/tools/rocprofiler/install CMAKE_PREFIX_PATH=/opt/rocm/include/hsa:/opt/rocm cmake ..
end up with
CMake-install-prefix: /usr/local
CPack-install-prefix: /usr/local
Also it doesn't find libhsakmt.so apparently not in 4.5.2:
/opt/rocm-4.5.2/lib64/cmake/hsakmt
/opt/rocm-4.5.2/lib64/cmake/hsakmt/hsakmt-config.cmake
/opt/rocm-4.5.2/lib64/cmake/hsakmt/hsakmtTargets-release.cmake
/opt/rocm-4.5.2/lib64/cmake/hsakmt/hsakmtTargets.cmake
/opt/rocm-4.5.2/lib64/cmake/hsakmt/hsakmt-config-version.cmake
/opt/rocm-4.5.2/lib64/libhsakmt.a
/opt/rocm-4.5.2/include/hsakmttypes.h
/opt/rocm-4.5.2/include/hsakmt.h
/opt/rocm-4.5.2/share/doc/hsakmt
/opt/rocm-4.5.2/share/pkgconfig/libhsakmt.pc
Is it on the github somewhere?
Thanks.
Hacked around it:
diff --git a/CMakeLists.txt b/CMakeLists.txt
index 7e1df75..0551493 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -23,7 +23,7 @@
 cmake_minimum_required ( VERSION 2.8.12 )

 # Install prefix
-set(CMAKE_INSTALL_PREFIX "/opt/rocm" CACHE PATH "Install path prefix default")
+#set(CMAKE_INSTALL_PREFIX "/opt/rocm" CACHE PATH "Install path prefix default")

 ## Verbose output.
 set ( CMAKE_VERBOSE_MAKEFILE TRUE CACHE BOOL "Verbose Output" FORCE )
diff --git a/cmake_modules/env.cmake b/cmake_modules/env.cmake
index 30e86c1..c8ca290 100644
--- a/cmake_modules/env.cmake
+++ b/cmake_modules/env.cmake
@@ -108,12 +108,15 @@ elseif ( ${CMAKE_SYSTEM_PROCESSOR} STREQUAL "x86" )
 endif ()

 ## Find hsa-runtime headers/lib
-find_file ( HSA_RUNTIME_INC "hsa/hsa.h" )
-find_library ( HSA_RUNTIME_LIB "libhsa-runtime${NBIT}.so" )
+#find_file ( HSA_RUNTIME_INC "hsa/hsa.h" )
+#find_library ( HSA_RUNTIME_LIB "libhsa-runtime${NBIT}.so" )
+set ( HSA_RUNTIME_INC "/opt/rocm/include/hsa/hsa.h" )
+set ( HSA_RUNTIME_LIB "/opt/rocm/lib/libhsa-runtime64.so" )
 get_filename_component ( HSA_RUNTIME_INC_PATH "${HSA_RUNTIME_INC}" DIRECTORY )
 get_filename_component ( HSA_RUNTIME_LIB_PATH "${HSA_RUNTIME_LIB}" DIRECTORY )

-find_library ( HSA_KMT_LIB "libhsakmt.so" )
+#find_library ( HSA_KMT_LIB "libhsakmt.so" )
+set ( HSA_KMT_LIB "/opt/rocm-4.5.2/lib64/libhsakmt.a" )
 get_filename_component ( HSA_KMT_LIB_PATH "${HSA_KMT_LIB}" DIRECTORY )
 get_filename_component ( ROCM_ROOT_DIR "${HSA_KMT_LIB_PATH}" DIRECTORY )

Hi @lfmeadow, apologies for the lack of response. The updated build instructions can be found at Build environment. Could you please give these a try and confirm if they resolve the issue you are encountering?

The issue is probably fixed with the upcoming rocprofiler SDK, so it is fine to close this one.

From: harkgill-amd @.>
Sent: Tuesday, August 13, 2024 8:17 AM
To: ROCm/rocprofiler @.>
Cc: Larry Meadows @.>; Mention @.>
Subject: Re: [ROCm/rocprofiler] non-default builds seem to be really hard (Issue #67)
Hi @lfmeadowhttps://github.com/lfmeadow, apologies for the lack of response. The updated build instructions can be found at Build environmenthttps://github.com/ROCm/rocprofiler/blob/amd-staging/README.md#build-environment. Could you please give these a try and confirm if they resolve the issue you are encountering?
—
Reply to this email directly, view it on GitHubhttps://github.com/ROCm/rocprofiler/issues/67#issuecomment-2286509911, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABH5YK5ZY2OHPFNKULDA7FLZRIPRLAVCNFSM6AAAAABMOQB5YSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOBWGUYDSOJRGE.
You are receiving this because you were mentioned.Message ID: @.***>


Add a setup script to change/adopt the templates

a quick attempt to automate the tasks i have to do when using your great template ;)

Thanks a lot for your pull request. This looks really nice!
Please give me a few days to review and test this. I'm quite busy at the moment but I will make sure I don't forget about this one. ;)

no hurry! ;)

Thank you very much again for your pull request. The script works great. :)
Would you mind adding a (mandatory) option for changing the package name as well? It can be found in the files manifest.json and share/forum-app/main.qml and didn't change when running the script. I noticed that the template still uses the old naming scheme, so I'll change that now and we'll only need an option asking for the developer's application namespace.
Otherwise the script looks fine and I'd be happy to merge it with these changes. Good job! :)

Here's the change: https://github.com/nikwen/ubuntu-go-qml-template/commit/e1248e38f3b11ba8bab2d1dd7e232237490b3fcd ;)

sure - will have a look at it today...
;)

... guess this should do the trick now...  :yum:

Thanks! This would probably work most times now. However, it might break when the entered email address contains the phrase "nikwen" (like mine). ;)
I'd suggest adding the following code before line 118 (and removing line 122):
if fname == 'manifest.json' || fname == 'share/forum-app/main.qml'
  changed = text.gsub(/ubuntu-go-qml-template.nikwen/, "#{options.name}.#{options.namespace})
end

As I've never coded in Ruby before, please check that code for correctness. It could be that we have to escape the dot in the gsub string for example.

my fault - checked against wrong option name - but for sure nikwen can be used ;)

finally - cleaned up.

Looks good, thanks! :)
Just let me test it once again. ;)

Works perfectly fine. Thank you very much again! :+1: :smile:


How to configure handlebars?

When I have js-beautify format my handlebars templates, it takes things like this:
{{ember-flatpickr
    altInput=true
    altFormat="m/d/Y"
    appendDataInput=false
    dateFormat="Y-m-d"
    enableTime=false
    onCloseAction=(action 'enableValidation' 'hireDateValidationEnabled')
    placeholder="Choose a date"
    value=(mut user.hireDate)}}

And turns them into one long line instead:
{{ember-flatpickr altInput=true altFormat="m/d/Y" appendDataInput=false dateFormat="Y-m-d" enableTime=false onCloseAction=(action 'enableValidation' 'hireDateValidationEnabled') placeholder="Choose a date" value=(mut user.hireDate)}}
Is there a way to preserve those line breaks?

I believe that the preserve newlines option should keep the formatting, but it does not. I filed an issue about that here https://github.com/Glavin001/atom-beautify/issues/1102 , but they think it is an issue with JS Beautify itself.

Handlebars is not supported yet. We have done minimal work to make it not completely break, but have not had time to do more.  Any help you can provide is much appreciated.

@bitwiseman unfortunately, this is not my forte, but I can try to give it a look. I know the WebStorm formatter works as I would expect. Do you know if there is somewhere we can find what they are using and copy some of it?

@rwwagner90 - I do not know.  It seems pretty unlikely, as WebStorm is not open-source.

@bitwiseman I was just curious if they used a library or wrote it all themselves. Thought maybe you guys knew, since you probably researched other formatters to build this.

When I ran this on v1.8.1 the actual output was:
{{ember-flatpickr altInput=true
    altFormat="m/d/Y"
    appendDataInput=false
    dateFormat="Y-m-d"
    enableTime=false
    onCloseAction=(action 'enableValidation' 'hireDateValidationEnabled')
    placeholder="Choose a date"
    value=(mut user.hireDate)}}

It isn't exactly as preferred in the "Expected Output" but could be considered "fixed." @rwwagner90  do you want to double-check and let me know if you disagree?

@MacKLess that seems closer, but altInput should be on a new line as well.


wsl --update invalid command line option

Environment
Windows build number: [run `[Environment]::OSVersion` for powershell, or `ver` for cmd]
Your Distribution version: [On Debian or Ubuntu run `lsb_release -r` in WSL]
Your WSLg version: [Open 'Apps and Features' and check the version of 'Windows Subsystem for Linux GUI app support', e.g: <IP_ADDRESS>]

Steps to reproduce

WSL logs:

Attach WSLg logs from  /mnt/wslg

You can access the wslg logs using explorer at: \\wsl$\<Distro-Name>\mnt\wslg (e.g.: \\wsl$\Ubuntu-20.04\mnt\wslg)

puseaudio.log
weston.log
versions.txt

Expected behavior

Actual behavior

I suspect you may be trying to do this from an older version of Windows. Could you type the following in a Windows command prompt
C:\WINDOWS\system32>ver

Microsoft Windows [Version 10.0.21367.1000]

You need to be on Windows insider build 21364+ for WSLg to be available. You can join Windows Insider program here: https://insider.windows.com/en-us/.
Steve

Windows build lower than 21364.


Ability To Validate Against Multiple Types

I am using AntD's forms to validate some user inputs and I am running into a case where I want to validate against multiple types for a single rule, akin to the behaviour described in async-validate.
It would be nice to be able to define rules in a manner akin to:
rules: [
  {
    type: ['integer', 'number']
    message: 'Not a number.'
  },
  {
    type: ['integer']
    message: 'Not an integer.'
  }
]

I'm looking forward to it.


Nexus

What was changed

Added Nexus Endpoint CRUD commands
Added Pending Nexus Operations and Callbacks to workflow describe command.

There's a TODO I left to fix the describe task queue command due to it referencing the deleted tqname package from the server repo. I'll let @antlai-temporal address that as part of his work on versioning.
Also had to point the server at my local commit due to an unmerged small fix.
DO NOT MERGE until I fix the TODO and the server and SDK dependencies.

The fix for tqname is already in the versioning-2 branch but it may take a while (blocked on go SDK merge) until we merge it. It is just two lines, and I don't want to block your PR,  do you want to go ahead and add them?


bug: Opening Trouble crashes Neovim

Did you check docs and existing issues?

[X] I have read all the trouble.nvim docs
[X] I have searched the existing issues of trouble.nvim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
v0.10.0-dev-629+gb2e8c0df2
Operating system/version
Mac Ventura 13.4.1
Describe the bug
When opening Trouble on nightly, Neovim crashes and a "Process exited with error code 134" error is displayed in the terminal.
Steps To Reproduce

Open any file with the latest nightly release of Neovim.
Open Trouble.

Expected Behavior
For Trouble (and Neovim) to not crash :)

After several hours of debugging, I discovered that this was caused by my folding ranges configuration. Adding ft_ignore = { 'trouble' } to my statuscol settings fixed, although I'm quite confused as to what changed, since I haven't recently updated those extensions...

Plugins can't really crash Neovim, that's always due to bugs in Neovim itself. So maybe due to a recent Neovim change?

Plugins can't really crash Neovim, that's always due to bugs in Neovim itself. So maybe due to a recent Neovim change?

Yeah, I can only reproduce it on nightly when statuscol is enabled.

I’m now paranoid of this mysterious issue appearing again :(


'ui' command - eseless

Add config instead of 'ui' command. json/ini

Since Windows uses registry and is pretty volatile I like work with it. :D. Beside is Windows only app and no need to port settings to other file.


Adding ARM community builder support

List of Changes Proposed
Added support for the Arm Packer builder.
Testing Evidence
$ pytest tests/packerlicious/community/test_builder_arm.py
================================================== test session starts ==================================================
platform linux -- Python 3.6.7, pytest-4.1.1, py-1.7.0, pluggy-0.8.1 -- /home/user/packerlicious/venv/bin/python
cachedir: .pytest_cache
rootdir: /home/user/packerlicious, inifile: setup.cfg
collected 1 item

tests/packerlicious/community/test_builder_arm.py::TestArmBuilder::test_required_fields_missing PASSED            [100%]

=============================================== 1 passed in 0.01 seconds ================================================```

Thanks you for this project.