"toBeAccessible()" test never passes

What will this achieve?
Passing an accessibility unit test.
How will success be measured?
A failing test will pass.
Additional information
I could not pass the specific accessibility test below for the Non Linear Reading component.
Adding both of these extra tests passed, but did not solve the problem, and so were finally not included:

Focus on body > tab > button has focus? Test passed.
Button is visible? Test passed.

Debug
<body>
  <div>
    <span> // <-- Component starts here.
      <button>
        Some text...
      </button>

Full report (link)
NonLinearReading › has no accessibility violations
Message: Confirm the element should be tabbable, and is visible on the screen when it has keyboard focus
Level: potentialviolation
Snippet: <button class="..." type="button">

Following the link, the documentation for this issue includes a "needs review" notification, but no details.

I have noticed that some other components skip() this test. See also AboutModal.test.js, CreateFullPage.test.js, and others.

Typical. The problem was between the keyboard and the chair.
This problem was solved last week with a similar issue.
Until last week, I had been using the default React test command yarn test path/to/COMPONENT_NAME.test.js and this was always throwing errors.
As it has been recently pointed out, I should have been using IBM Products' specific yarn test:c4p COMPONENT_NAME instead. The tests pass using this command.
False alarm.

Thank you, Paul. Classic PEBKAC. 😹


Add a testing example

It's more an improvement than an issue, but it would be nice to have the boilerplate for the testing of the components in the project.

Great suggestion. I would welcome a PR if you have gotten around to this in the meantime.


Invalid Javascript is generated causing Uncaught SyntaxError

Description
The javascript that is injected into the HTML does not seem to be valid resulting in a syntax error.
Steps to Reproduce

gatsby new newrelictest
cd newrelictest
yarn add https://github.com/newrelic/gatsby-plugin-newrelic.git
add configuration

      resolve: 'gatsby-plugin-newrelic',
      options: {
        instrumentationType: 'lite', // also tried pro and proAndSPA
        configs: {
          production: {
             accountId: '<some integer>',
             trustKey: '<some integer>',
             agentID: '<some integer>',
             licenseKey: '<the license key>',
             applicationID: '<some integer>',
             beacon: 'bam.nr-data.net',
             errorBeacon: 'bam.nr-data.net'
          },
        },
      },
    }

GATSBY_NEWRELIC_ENV=production yarn build
Run website
:sob:

Expected Behaviour
NewRelic JS to load and report information
Relevant Logs / Console output
Uncaught SyntaxError: Unexpected token ')'
Your Environment
Chrome 83.0.4103.116 & Firefox 77.0.1
Additional context

Hey @shawncarr. Thanks for reporting this. We'll look into it ASAP.

#6 closes this issue. Thanks again, @shawncarr!


Support operation within existing monorepo

The current blueprint does not allow creating new v2 addon within an existing monorepo easily, unlike what the Readme says. Although we have customization options like --addon-location, these only allow creating the addon structure in a temporary path and then copying things over to the actual monorepo. But ideally we would be able to do...
cd existing-repo
ember addon my-addon -b @embroider/addon-blueprint --addon-location=packages/my-addon/addon --test-app-location=packages/my-addon/test-app --test-app-name=my-addon-test-app --skip-git

This would:

omit all root files, especially package.json
put README.md et al directly into the adodn folder, and not copy-on-build

This fails though:

ember addon forcefully creates a new my-addon directory here, so we would end up with the addon being in existing-repo/my-addon/packages/my-addon/addon instead of existing-repo/packages/my-addon/addon. EmberCLI does not seems to have a way to prevent that.
another approach would be to use the simpler generate command, which does not have this behaviour, as e.g. ember g in-repo-addon my-addon -b @embroider/addon-blueprint ..., but this also fails due to

the generate command expecting to be run inside an EmberCLI project, which existing-repo is not
the custom blueprint option -b not being supported here

How can we solve this?

copy stuff from existing-repo/my-addon/* to .. on afterInstall? 🤨
Create the missing functionality in EmberCLI? Would require all users to use that version of EmberCLI...
Use our own blueprint infrastructure not related to EmberCLI (or something of the shelve from npm)

copy stuff from existing-repo/my-addon/* to .. on afterInstall? 🤨

This is what I'm doing for now in #88, to get us unblocked...


Create a separate test for each layout configuration in HomePageConfigurationTests

Task/Issue URL:
Tech Design URL:
CC:
Description:
Main reasons for this change:

Having a separate test case for each test configuration improves the readability
Using a dedicated 'assertLayout' function with '#file' and '#line' macro parameters show test failures nicely in the Xcode
The test cases were doubled, the duplicates were removed

Steps to test this PR:

code review

Orientation Testing:
Not needed
Device Testing:
Not needed
OS Testing:
Not needed
Theme Testing:
Not needed
Internal references:
Software Engineering Expectations
Technical Design Template

@bwaresiak thanks for your suggestions! Should be good now


Sorting function in member page

Adding sorting function to member page.

closed because #70 is merged to the master successfully
the number of boy or girl would be put into our enhancement list


rangeCheck internal method is slow

Currently super.rangeCheck is used a lot in the code to throw RangeError before writing into a buffer whenever a bound will be crossed (in comparison without this check we may have partial writes meaning that the buffer may contain invalid data).
This "correctness" comes with a small performance penalty for each class that inherits SizedType.
For single types the penalty is not much but for compound types like arrays, tuples and struct it can become quite a hassle.
With rangecheck

Without rangecheck

I think it may be a good idea to either remove the rangecheck entirely and just declare the written buffer as not good to read from.
Alternatively we can remove a bit of the penalty in compound types and only let the top level type do the range check (all types under the top level will do a bound check that is less than the total size of it's parent)

@eliassjogreen @load1n9 What do you two think about this?


Add pow method

Add the pow method that returns the number raised to a specified exponent.
Ex.:
root.new_value = this.value.pow(2)

# In:  {"value":2}
# Out: {"new_value":4}

# In:  {"value":5}
# Out: {"new_value":25}

root.new_value = this.value * 10.pow(-2)

# In:  {"value":2}
# Out: {"new_value":0.02}

@mfamador can you move this into the pure package. I'm hoping to eventually have all bloblang methods and functions outside of the query stuff.

I moved it to pure. Thanks


[Auto-update][PT Settings] Updated UI

Summary of the Pull Request
What is this about:

Added Downloading... text.
Mute the Download and install button after downloading started.
Removed Read more for the error state.
Changed button text to Try to download and install in the error state.

What is include in the PR:
How does someone test / validate:
Quality Checklist

[ ] Linked issue: #xxx
[ ] Communication: I've discussed this with core contributors in the issue.
[ ] Tests: Added/updated and all pass
[ ] Installer: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Docs: Added/ updated
[ ] Binaries: Any new files are added to WXS / YML

[ ] No new binaries
[ ] YML for signing for new binaries
[ ] WXS for installer for new binaries

Contributor License Agreement (CLA)
A CLA must be signed. If not, go over here and sign the CLA.

How is ux looking when downloading in background (auto update) has begun before opening settings ux?
Can we show installed version number every time and use the last search text line for update status information?
Can we define error text as ErrorBrush?
Can we make the update status bold if update is available/ready?


Chrismwendt support hole fill via auto

This is really cool, and the implementation looks good, but I have to say that auto provides such poor suggestions that I wonder if it's even worth using at all.

About the only good it did on a real-ish project was suggest a variable in scope:

After about a dozen of these, my system started stuttering and locking up, and I saw this:
$ ps auxww | grep ghc
chrismw+   524 77.6 18.3 3291740 2992192 pts/1 Rl   15:14   0:17 /home/chrismwendt/.local/bin/ghc-mod -g -fno-warn-type-defaults -g -fno-warn-unused-binds -g -fno-warn-unused-do-bind -g -fno -warn-unused-imports -g -fno-warn-missing-signatures --map-file /home/chrismwendt/sandbox/slow- ghc-mod-auto/app/Main.hs auto /home/chrismwendt/sandbox/slow-ghc-mod-auto/app/Main.hs -- 24 52
chrismw+   717  0.0  0.0  14224  1036 pts/1    S+   15:14   0:00 grep --color=auto ghc
chrismw+ 32387 86.0 22.8 6090332 3716912 pts/1 Rl   15:13   1:13 /home/chrismwendt/.local/bin/g hc-mod -g -fno-warn-type-defaults -g -fno-warn-unused-binds -g -fno-warn-unused-do-bind -g -fno -warn-unused-imports -g -fno-warn-missing-signatures --map-file /home/chrismwendt/sandbox/slow- ghc-mod-auto/app/Main.hs auto /home/chrismwendt/sandbox/slow-ghc-mod-auto/app/Main.hs -- 24 13
chrismw+ 32462 86.6 29.0 6083164 4733092 pts/1 Dl   15:13   1:10 /home/chrismwendt/.local/bin/g hc-mod -g -fno-warn-type-defaults -g -fno-warn-unused-binds -g -fno-warn-unused-do-bind -g -fno -warn-unused-imports -g -fno-warn-missing-signatures auto /home/chrismwendt/sandbox/slow-ghc-mo d-auto/app/Main.hs -- 24 13
chrismw+ 32749 92.3 17.6 3332700 2868160 pts/1 Rl   15:13   0:48 /home/chrismwendt/.local/bin/g hc-mod -g -fno-warn-type-defaults -g -fno-warn-unused-binds -g -fno-warn-unused-do-bind -g -fno -warn-unused-imports -g -fno-warn-missing-signatures --map-file /home/chrismwendt/sandbox/slow- ghc-mod-auto/app/Main.hs auto /home/chrismwendt/sandbox/slow-ghc-mod-auto/app/Main.hs -- 24 22

Also, autocomplete doesn't pop up until auto has either finished or been killed, which sometimes takes a few seconds.
I think auto has awesome potential, but it just doesn't seem worth the performance penalties right now. I actually have more hope for the existing autocomplete behavior in terms of providing useful suggestions.

After about a dozen of these, my system started stuttering and locking up, and I saw this

That's weird, processes should be killed after a couple seconds, but this doesn't seem to be the case here. Maybe issuing SIGKILL instead of SIGTERM would do it?..

Also, autocomplete doesn't pop up until auto has either finished or been killed, which sometimes takes a few seconds.

Yes, this is quite a major concern tbh. But I can't add suggestions after showing them, obviously, and auto isn't the fastest thing in this world, to say the least.

I think auto has awesome potential

TBH, it wasn't properly maintained for a while. Last meaningful commit is more than a year ago and I don't think many people actually use it. Besides, djinn (used as backend for auto) is highly experimental software with no apparent activity lately. So I'm not sure if this potential you see will ever be realized.

I actually have more hope for the existing autocomplete behavior in terms of providing useful suggestions.

Well, there is a lot of space for improvement on current autocompletion. For one, it's a long-standing issue that local definitions are not suggested (there are two ways to deal with this actually, one Atom-side, and one ghc-mod-side). Hole completions also were written in an afternoon or two and then promptly forgotten about -- I don't even remember half of the implementation details :) So feel free to hack on it if you think you can improve something. Or file issues. Can't promise I will do something fast, but I usually do something eventually.

it just doesn't seem worth the performance penalties right now

Well, we might as well put this on ice then. Might come in handy one day, if not today, right?

feel free to hack on it if you think you can improve something

It's worth exploringn, so I just might take a look at it in my spare time :smiley:
But yeah, let's shelve this for now. Thanks for all the thoughtful comments and for prototyping this out!


Added support for logging Scopes

Hi Rory,
your tool has been most helpful over the last months. Thank you for the time you put into it!
I just wanted to be able to see logging scopes (i.e. BeginScope) in my test output as well.
This is my first ever pull request so I hope I did this the right way.
Best regards,
Sebastian

Thanks Sebastian. This looks good. I'm going to spend a bit of time to review it.

I've had a play with the code you have put together. I like the outcome but want to see a cleaner relationship between TestOutputLogger, scope tracking and CacheLogger. I'm trying to come up with a cleaner way of using the disposal of scopes to clean up the tracking of scopes.
I've also be thinking about how the scopes are written to the output. I would prefer the scope to write their scope value at the beginning and end of the scope but only write the latest scope value rather than the complete collection of scopes. I'm also thinking about using simple nested whitespace in the output to visually identify the relationship/stack of scopes among the log outputs. Would this still suit your purpose?
I'll play with these ideas over the next few days and push a branch with what I'm thinking.

I was (somewhat) following Nicholas Blumhardt's post about how they do it in Serilog but as long as the scopes show up somewhere it'll work for me.

First change is done. I renamed Scope to CacheScope because it is really just a wrapper around the scope from the ILogger in CacheLogger. CacheLogger itself is just a wrapper. The naming of the Scope class caused a little confusion. I also passed an Action to it so that the CacheScope was not directly coupled to the private _scopes field of CacheLogger.
There was also a potential bug where the innerScope would be null if the _logger.BeginScope returned null. This would have caused a NullReferenceException when the CacheLogger.Scope instance was disposed. Changing that code from
_logger != null ? _logger.BeginScope(state) : NoopDisposable.Instance;
to
_logger?.BeginScope(state) ?? NoopDisposable.Instance;
covers both the scenarios where _logger is null or the scope returned from its BeginScope is null.

I think I'm quite happy with this. A good example of the changes I've made to your excellent pull request is in the following test.
        [Fact]
        public void LogEntryContainsSnapshotOfActiveScopes()
        {
            var logger = _output.BuildLogger();

            logger.LogCritical("Writing critical message");
            logger.LogDebug("Writing debug message");
            logger.LogError("Writing error message");
            logger.LogInformation("Writing information message");
            logger.LogTrace("Writing trace message");
            logger.LogWarning("Writing warning message");

            using (logger.BeginScope("First scope"))
            {
                logger.LogInformation("Inside first scope");

                using (logger.BeginScope("Second scope"))
                {
                    logger.LogInformation("Inside second scope");
                }

                logger.LogInformation("After second scope");
            }

            logger.LogInformation("After first scope");
        }

This results in the following being written to the xunit test output.

Critical [0]: Writing critical message
Debug [0]: Writing debug message
Error [0]: Writing error message
Information [0]: Writing information message
Trace [0]: Writing trace message
Warning [0]: Writing warning message
<First scope>
   Information [0]: Inside first scope
   <Second scope>
      Information [0]: Inside second scope
   </Second scope>
   Information [0]: After second scope
</First scope>
Information [0]: After first scope

Does this still satisfy what you were trying to achieve?

It gives a nice structured view of the log entries including indentation of messages inside the scope. I like it.

@webseb Thank you for the pull request. I have pushed 2.0.0 with your logging scopes onto nuget.org.

FYI - https://www.neovolve.com/2019/03/25/supporting-scopes-with-xunit-ilogger/


Task 1 and 2 completed

Issue: #1

Issue: #2

helloWorld.txt

Create two different pull requests for task 1 and task 2. You can make branches and send PRs independently


Keep anchor's href attribute

Keeping the href attribute means visited styles can be applied to the tab link.
This builds on top of #3

Hi there,
I'm afraid I have to refuse this PR, the href value is removed by the script to fix a bug in some speech synthesis (Jaws). So we can't keep it.
Anyway, thanks for the proposal and your patience. 👍

Time flies!
It’s strange that a common, standard attribute causes such a problem.


Add openssh-client package to base image

As many of use have private package dependencies sitting on private git repositories, it would be helpful to have openssh-client preinstalled so we wouldn't have to fetch those repositories overly complicated.

To support my case, see:
https://github.com/larruda/google-cloud-sdk/issues/93
Where said problem is described and one of the solutions mentioned is using git+ssh to get the private repo, yet that seems to be overly complicated as openssh-client is no longer available to install after the sources have been replaced

If you have a working patch please make a pull request.

@sgjesse Done, see #22


Replace @SubCommands SumType!(TYPES...) with SubCommand!(TYPES...)

Closes #80

struct A
{
    int[] arr;
}

immutable one = SubCommand!A(A([1]));
SubCommand!A mut;
mut = one;
mut.match!(_ => _.arr[0] = 2);
assert(one.match!(_ => _.arr[0]) == 2); // Successfully modified an immutable object!

Still have to investigate if it’s a SubCommand’s fault or SumType’s.

Still have to investigate if it’s a SubCommand’s fault or SumType’s

It seems also related to int[] array - this fails as expected:
struct A
{
    int arr;
}

immutable one = SubCommand!A(A(1));

SubCommand!A mut = one;
mut = one;
mut.match!(_ => _.arr = 2);

assert(one.match!(_ => _.arr) == 2);

struct None {}

struct S(T)
{
    SumType!(None,T) a;
}

struct A
{
    int[] arr;
}

immutable one = S!A.init;
immutable two = SumType!(None,A)(A([1]));
immutable three = SumType!(A,None)(A([1]));
immutable four = SumType!(A)(A([1]));

one and two do compile and three and four do not (cannot implicitly convert expression)

Asked question on forum.

May be adding None to SumType was not good solution so I'd like to re-think my proposal a bit:

There are two use cases: with default subcommand and without.
Public API for both cases should be the same although internal implementation might not.
I think the following public API is useful:

Check whether any subcommand is chosen. This case always be true for a case with default subcommand.
Check whether specific subcommand is chosen.
Ctor and assignment operator that take subcommand object as parameter (without Default!).

In case of default subcommand, init value should be set to that default subcommand's init.

Pushed changes and confirmed that this does not compile:
    struct A
    {
        int[] i;
    }

    immutable sub = SubCommand!(A)(A([1]));

Error:

Error: cannot implicitly convert expression SubCommand(Nullable(DontCallDestructorT(SumType(Storage(A(null)), cast(ubyte)0u)), true)).this(A([1])) of type SubCommand!(A) to immutable(SubCommand!(A))

@SirNickolas Is this now good from your point of view?


Fix PyInstaller hooks to work with v6.x.x #8653

Maintainer merge checklist

[X] Title is descriptive/clear for inclusion in release notes.
[ ] Applied a Component: xxx label.
[ ] Applied the api-deprecation or api-break label.
[ ] Applied the release-highlight label to be highlighted in release notes.
[ ] Added to the milestone version it was merged into.
[ ] Unittests are included in PR.
[ ] Properly documented, including versionadded, versionchanged as needed.

Question (from someone not very into Windows development):

Do we need to support PyInstaller<6.0.0 ? (Consider Kivy is going to have a major release, so we can break things, if well documented)

I would say considering the code to allow this is a few lines,  let the support for older pyinstaller be present for now. Maybe add a deprecated warning.. Can always remove/deprecate the support for older pyinstaller.

@misl6

Do we need to support PyInstaller<6.0.0

Since Pyinstaller introduce a couple of breaking chance that might impact some application, I would consider preferable to keep compatibility with Pyinstaller v5 and v6

@misl6

Do we need to support PyInstaller<6.0.0

Since Pyinstaller introduce a couple of breaking chance that might impact some application, I would consider preferable to keep compatibility with Pyinstaller v5 and v6

Makes sense!


Image sizes in datasets

What are the image sizes in datasets? It would be nice to know since torchmeta tries to standardise few-shot learning datasets and this info isn't readily available from looking at sources.

This information is indeed not available right away unfortunately. If you are using the datasets with torchmeta.datasets.helpers, then this information can be found by looking at the Resize transforms, although it is a bit hidden. For example, in Omniglot the images have size 28. Otherwise if you are not using torchmeta.datasets.helpers, the image sizes are the default ones from the extracted dataset (e.g. 105 in Omniglot).


Sonoff SWV goes direct OFF-status after putting ON

I’m new to Zigbee and using a Sonoff ZB Bridge-P with a Sonoff SWV (Smart Water Valve). I’ve set up the SonoffLAN integration via HACS, connected through the ‘AUTO’ method.
The valve appears as a device in Home Assistant after adding it through the eWeLink app and refreshing HA.
The problem:

When I turn the valve ON, the status in HA immediately switches to OFF, but the valve physically turns ON.
Turning the valve ON again, the status in HA stays ON.
Turning the valve OFF works correctly - both the status and the physical valve turn OFF.
Through the eWeLink app, it seems to be working fine.

Any suggestions on how to resolve this status mismatch?
Thanks in advance for any assistance!

https://github.com/AlexxIT/SonoffLAN?tab=readme-ov-file#issues

For the moment i've added a small rule to put it a second time ON. This works.
alias: SonOffKraanStatusCheck
description: ""
trigger:
  - platform: device
    type: turned_on
    device_id: 68f418251d408947863c809f26d032ed
    entity_id: 1ef333dd8c261fbe01156022326615bf
    domain: switch
condition: []
action:
  - delay:
      hours: 0
      minutes: 0
      seconds: 1
      milliseconds: 0
  - type: turn_on
    device_id: 68f418251d408947863c809f26d032ed
    entity_id: 1ef333dd8c261fbe01156022326615bf
    domain: switch
mode: single

I didn't found any issues in the logs that relate to this.
The diagnostics:
sonoff-01J73N995EM1AQ88AF83B0VQ5J-KraanSonoff1-68f418251d408947863c809f26d032ed.json

What of this params do you need in Hass? Looks like switch and battery. Maybe todayWaterUsage.
"device": {
      "uiid": 7027,
      "params": {
        "fwVersion": "1.0.3",
        "subDevRssi": -38,
        "battery": 100,
        "trigTime": "1725626515000",
        "supportPowConfig": 1,
        "switch": false,
        "realIrrigationVolume": 3,
        "realIrrigationVolumeGal": 0,
        "controlMode": "manual",
        "todayWaterUsage": 8,
        "todayWaterUsageGal": 2,
        "runningMode": 2,
        "exceptionReport": [],
        "hasException": false,
        "only_device": {
          "ota": "success",
          "ota_fail_reason": 0
        },
        "subOtaInfo": {
          "state": 0,
          "reason": 0,
          "stateTime": "1727005559000"
        },
        "endIrrigationTime": "1727011383000",
        "lastIrrigationTime": "1727010803000",
        "sledOnline": "off"
      },
      "model": "SWV",
      "online": true,
      "local": null,
      "localtype": null,
      "host": null,
      "deviceid": "a4800a275a"
    }

I got the same problem with SonOff Minir4M but I can´t find a way to fix it :(

I'm having a similar issue with the SWV. I can turn it on from HA using cloud but the switch state never stays ON. And then I can't turn it OFF, even calling the service directly. I have to turn it off from the ewelink app 🤔

Same behavior. I'm just going with the suggested workaround.
Ty for your time.

Same issue here. Will try workaround


FileDialog doesn't show button shadows correctly

This is caused by the fact FileDialog doesn't take full advantage of Pos/Dim for layout.
Generally, the fix is

use Pos.AnchorEnd for any buttons that are supposed to be at the bottom/right.
dont position views with absolute values, such as the "up" dir button

What is the easiest way to turn on shadow buttons in UICatalog?

What is the easiest way to turn on shadow buttons in UICatalog?

Select the UI Catalog Theme.
Or in any config.json
{
  "SOME THEME": {
        "Dialog.DefaultShadow": "Transparent",

Hot Dog Stand has it enabled too

The main table view is hosted in a tile.  It needs to Fill up to the buttons.  But the buttons are now of variable height (i.e. Pos.AnchorEnd).
What is best approach here @tig ? some kind of Dim.Func that looks at the current Y of the buttons above and below?
_splitContainer = new TileView
{
    X = 0,
    Y = Pos.Bottom (_btnBack),
    Width = Dim.Fill (),
    Height = Dim.Fill (1/*TODO needs to be 2 for shadow buttons, how best?*/)
};

Bellow the Up and Right buttons it's a space? If affirmative you can remove it and add it bellow before the buttons and search text.

Bellow the Up and Right buttons it's a space? If affirmative you can remove it and add it bellow before the buttons and search text.

There is no space, it looks like there is only because of the use of shadow buttons.
The issue is how to declare the Height property of the main table tile.  At the moment it is Fill(1) which works for normal theme but when buttons become height 2 then it needs to be Fill(2).  I'm looking for suggestions on smart way to handle that.

I think the best solution is including the Shadow adornment as part of the View class, as the Margin, Border and Padding and included in the layout for calculation of the Frame size, based on his thickness. So, any view that want to used it only needs to set the thickness as desired.

I'd love to come up with an elegant, easy to use, solution for this. It's a common case.
One of the reasons I was thinking of putting dialog buttons in Padding was to address this (I've backed away from that idea).
I've solved this several times differently in places. Later today when I'm at a 'puter I'll remind myself and share.
I also have an idea for a new Dim type along the lines of "table.Height = Dim.Track (okBtn)".

What about Dim.FillTo(Pos a).
Or does this create a circular dependency?

What about Dim.FillTo(Pos a). Actually I guess that would just be a Pos overload of the regular Fill method.
Or does this create a circular dependency?

Yeah, maybe something like that.

I think the best solution is including the Shadow adornment as part of the View class, as the Margin, Border and Padding and included in the layout for calculation of the Frame size, based on his thickness. So, any view that want to use it, only needs to set the thickness as desired.

Shadow is in Margin.

This works:
Height = Dim.Fill () - Dim.Func (() => IsInitialized ? Subviews.First (view => view.Y.Has (typeof(PosAnchorEnd), out _)).Frame.Height : 1),

And...
With #3716, this works:
Height = Dim.Fill (Dim.Func (() => IsInitialized ? Subviews.First (view => view.Y.Has<PosAnchorEnd> (out _)).Frame.Height : 1)),

#3716 has been merged.
@tznind you can do
table.Height = Dim.Fill (Dim.Func (() => IsInitialized ? Subviews.First (view => view.Y.Has<PosAnchorEnd> (out _)).Frame.Height : 1));

Shadow is in Margin.

Makes sense. Good choice.

@tznind you can do [...]

Thanks, I have referenced view directly rather than use linq but the Dim overload for Fill is super helpful
Height = Dim.Fill (Dim.Func (() => IsInitialized ? _btnOk.Frame.Height : 1)),


V13 deprecated

Hello.
can you change to v15 in your GoogleAds.php File?
public function __construct(
private string $clientId,
private string $clientSecret,
private string $developerToken,
private int $sdkVersion = SupportedVersions::VERSION_13,
private string $transportProtocol = 'rest',
private ?LoggerInterface $logger = null,
private string $logLevel = 'INFO'
) {
Regards, DOminik

Wrong repo – joelbutcher/googleads is the one you're looking for :)

Sorry, will you change this there?
Von: Joel Butcher @.>
Gesendet: Freitag, 9. Februar 2024 10:53
An: joelbutcher/laravel-googleads @.>
Cc: dominikraff @.>; Author @.>
Betreff: Re: [joelbutcher/laravel-googleads] V13 deprecated (Issue #8)
Wrong repo – joelbutcher/googleads https://github.com/joelbutcher/googleads  is the one you're looking for :)
—
Reply to this email directly, view it on GitHub https://github.com/joelbutcher/laravel-googleads/issues/8#issuecomment-1935624184 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AGQCZYT43F77P4LGJM3MPOLYSXWWFAVCNFSM6AAAAABDA56MUGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMZVGYZDIMJYGQ .
You are receiving this because you authored the thread.  https://github.com/notifications/beacon/AGQCZYRK6PWWNSOJOGNLMXDYSXWWFA5CNFSM6AAAAABDA56MUGWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTTTL5D7Q.gif Message ID: @.*** @.***> >

Please update your composer to use v7.0.0 of this package, which should use v8.0.0 of the base Google Ads SDK


Create Select Field

Another field we need to create. It should accept an array of label/value objects and all of the common properties of a select (check the input field I added for reference).

@KovaCro one thing I'm noticing on this one. It should be possible to assign a different name and value to the options. In other words the options should be an array of {name: string; value: string} not a string array.

select should open an overlay and display the menu there because if somebody puts a select element inside a block with overflow: hidden; the menu will definitely get cut off.
same thing with #6
i'll be working on this.


Replace "Admin" terminology in permissions

Continuation of #28451

Rename the ADMIN permission flag. The only suggestion that I've been able to think of is PRIVILEGEDUSER
Rename IsAdmin to something like IsPrivilegedUser and create an obsoleted IsAdmin function that just redirects to the renamed one to avoid breaking forks. Ensure upstream uses are renamed to avoid creating tons of warnings.
Modify the dead chat "ADMIN(username)" display. Some options:

Replace "ADMIN" with "STAFF"
Make it a new flag control it

The purposes of these changes are:

To avoid contributors mistakenly using IsAdmin or the ADMIN flag when intending to give a permission to game admins. My understanding is that admin was not intended to refer to game admins
To reduce in-game confusion between staff with elevated permissions and game admins

I don't think this is a good idea, because in all forks, any player who has received any rights is an administrator. It doesn't matter if he's a game admin or just a staff member. This is equivalent. And rather, this change will create confusion in terminology.

I'm not sure what you mean.

in all forks, any player who has received any rights is an administrator.

There is no way that wizden is the only server of all SS14 servers where there are people who have some in-game permissions but are not game admins. I think I've been told that there are some russian servers which give people ahelp permissions to act as mentors.

And rather, this change will create confusion in terminology. The exception does not work that every admin is a staff, but not every staff is an admin. Think about forks.

I don't understand what you're trying to say here. All game admins are staff, but not all staff are game admins. Calling all staff admins is inaccurate if any one of those people is not an admin, but calling all admins staff is still accurately labeling them

All members of the administration are called admins. Regardless of whether they have admin access to the game. That's the way it's done.. Because it is literally called the "admin staff". We do not take cases of illiterate management when admin rights are distributed to everyone just like that. Yes, there are helpers who have nothing but the rights to write to ahelp. But they are admins anyway.

In my experience, most players and contributors assume that when someone says "admin", they mean "game admin". I have never heard a non-game admin refer to themselves as an admin or administrator in the context of SS14, and I would consider it misleading if anyone who was not a game admin, project manager, or host did so. That's the primary reason why I think the change needs to be made.
Responding specifically to what you wrote: I cannot imagine how roles like "helper" fall under the umbrella of "administration". A person who only answers questions but has no ability to do anything else is not in an administrative role. A person who only has access to vv and debugging tools for the purposes of debugging is not in administrative role. Arguably, not even someone who has access to a wide set of admin permissions for the sole purpose of running events on the servers is an administrator
https://dictionary.cambridge.org/dictionary/english/administrator

This is the main difference of the Russian-speaking segment. For most, any member of the administration is an admin. And if you have any rights in the game, then you are definitely an admin. Admin is not only about having any rights, it is about the attitude towards the server administration.

well the flags in the code are written in English and aren't player facing, so presumably changing the word used as the flag won't negatively impact forks. The deachat should be a localization string so russian forks should be able to make it say whatever they want regardless of what we do here. While I don't know that they do, I assume that russian forks already translate it to say admin in russian rather than in english

What about adding localization to flags in that case?

by the way i don't understand your logic in light of the fact that in my pull request you said that every admin has all the rights to solve problems and now you say that not every player with rights is an admin

by the way i don't understand your logic in light of the fact that in my pull request you said that every admin has all the rights to solve problems and now you say that not every player with rights is an admin

At least in Wizden, In Wizden ADMIN is used to EXCLUSIVELY refer to people that are Game Admins that punish problem players according to policy and rules, and sometimes host events, only when they are a full admin and not a trial admin. An ADMIN is somebody that has Volunteered by putting forth an Application to become an Administrator to deal with player facing administration issues.
Maintainers are CODERS who do not have any expectation nor have they put in an application to deal with player facing administration issues.
SOME Project Managers are Admins, SOME Project Managers are Maintainers, SOME Project Managers are both.

by the way i don't understand your logic in light of the fact that in my pull request you said that every admin has all the rights to solve problems and now you say that not every player with rights is an admin

At least in Wizden, ADMIN is used to EXCLUSIVELY refer to people that are Game Admins that punish problem players according to policy and rules, and sometimes host events, only when they are a full admin and not a trial admin. An ADMIN is somebody that has Volunteered by putting forth an Application to become an Administrator to deal with player facing administration issues.
Maintainers are CODERS who do not have any expectation nor have they put in an application to deal with player facing administration issues.
SOME Project Managers are Admins, SOME Project Managers are Maintainers, SOME Project Managers are both.

In the ru segment, any member of the administration is considered an admin even if he has no rights in the game. And I did not understand what you are getting at if in my pull request to add a position in ahelp, they told me the opposite, that it is not necessary and everyone with rights can help. And now it turns out that not all of you are admins and should help directly in ahelp.

What about adding localization to flags in that case?

None of the flags have any localization. Can you explain what burden it would cause for forks if the flag was named something like PRIVILEGEDUSER instead of ADMIN in the code?

by the way i don't understand your logic in light of the fact that in my pull request you said that every admin has all the rights to solve problems and now you say that not every player with rights is an admin

Can you provide a link to where I said this please? Also those things aren't in conflict, even if admins have all rights it doesn't mean that everyone with any rights is an admin.

they told me the opposite, that it is not necessary and everyone with rights can help. And now it turns out that not all of you are admins and should help directly in ahelp.

On Wizard's Den, everyone with the ahelp permission has equal ability to warn a player or tell them that something is against the rules. There are things that game admins aren't allowed to do, but in the context of an ahelp, none of that is something that is both affected by the game admin's rank and that we expect or want players to enforce.
This is the same situation with every fork that I'm personally familiar with.
The only instance that I'm aware of where there is any difference in what someone can do in ahelp and a server would want to communicate rank for that purpose is if someone is in a mentor type position so only has ahelp permissions. In that situation, my understanding is that ahelp already changes the color of their name to communicate this

Many forks have their own sponsorship (privilege) systems. Therefore, such names of flags can be misleading. I've just reread the comments and I'm sorry, it wasn't you who wrote it, but another admin. In any case, our dispute lies in the cultural differences between the understanding of the word "admin" and those who relate to this word.


Add native view config for collapsableChildren

Summary:
collapsableChildren was introduced in D56226241
Changelog: [Internal]
Reviewed By: javache
Differential Revision: D56849868

Merged as https://github.com/facebook/react-native/commit/ef1e84e425a783aaae372b9938d6bb31505ac9dd


Generating Data Matrix code

Hi,
Is there a way to generate a Data Matrix code from this library?
Like this one in Image.

No, this library is not compatible with data matrixes. Sorry. :s


[MRG] Add a new step to circle CI to run a command

Tim-learns-circle-workflows
This adds a step to the circle workflow that will run a simple command
in the container to check it will run and do something minimally useful.

This could be merged too.


i18n(es): Update core-concepts/routing.md

What kind of changes does this PR include?

[ ] Minor content fixes (broken links, typos, etc.)
[x] New or updated content
[ ] Translated content
[ ] Changes to the docs site code
[ ] Something else!

Description

Updates core-concepts/routing.md to match #608 and #611

LGTM!


⚠ Bofrost.de has degraded performance

In 369d58c, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1192 ms

Resolved: Bofrost.de performance has improved in 08fc2a0.


I think it would be good if we provide some basic overview of JavaScript before challenges, we can provide resources to learn too. Challenges are amazing!

It is a nice idea. You can start working on it. I am assigning this to you. Thanks for your contribution.

I'm glad you like the idea. I will start Working on it.


[v10] installations fails, missing styles folder

Describe the bug
The reinstallation under v10 fails because the styles folder and the files in it are missing.
To Reproduce
Steps to reproduce the behavior:

open a v10 installation
go to modules
open the console
install ATL

Screenshots

Environment:

Hosting - Molten Hosting (also tested it local)
Browser [e.g. chrome, safari] Chrome
Foundry Version [e.g. 0.8.8] 10.284
Module Version [e.g 0.1.42]  0.4.01

Same Issues here, it doesn't even "Install" and the error persists even after deleting manually the folder in modules.
I'm on a Ubuntu machine, NGINX reverse proxi, Foundry 10.285, ATL 0.4.01

Workaround to install the module
For the less tech-savvy people (like myself)...
After the install fails, you'll find a partial installation consisting of the following directories and files in foundrydata/Data/modules/:

ATL/README.md
ATL/module.json
ATL/packs/
ATL/packs/premade
ATL/src/
ATL/src/activeLighting.js
ATL/src/updateManager.js

If you manually create the ATL/styles/ directory; ATL/styles/ATL.css file; and copy and paste the CSS file contents from the repository into the local ATL.css file you created, the module will appear as installed and you can lock it to prevent updates. I can't speak to whether or not there are any issues with the module itself once its running in v10, but it seems to work fine for me so far.

If you manually create the ATL/styles/ directory; ATL/styles/ATL.css file; and copy and paste the CSS file contents from the repository into the local ATL.css file you created, the module will appear as installed and you can lock it to prevent updates. I can't speak to whether or not there are any issues with the module itself once its running in v10, but it seems to work fine for me so far.

When i attempt this it still throws the error and just deletes the file and the directory. Any thoughts?

If you manually create the ATL/styles/ directory; ATL/styles/ATL.css file; and copy and paste the CSS file contents from the repository into the local ATL.css file you created, the module will appear as installed and you can lock it to prevent updates. I can't speak to whether or not there are any issues with the module itself once its running in v10, but it seems to work fine for me so far.

When i attempt this it still throws the error and just deletes the file and the directory. Any thoughts?

It's not a fix to install...

Install the module (This will seem to fail, but in your folder for modules, you should see a folder called ATL [...AppData\Local\FoundryVTT\Data\modules])
Create a folder called styles
Download this CSS file _https://raw.githubusercontent.com/kandashi/Active-Token-Lighting/main/styles/ATL.css_ into your styles folder that you just created
Restart Foundry
Go to your Add-on Modules
Click the Lock icon on the right side of Active Token Effects (this will stop you from accidentally blowing away what you just did)
Check back here often until a v10 release is done,
Once(if) a v10 update is completed unlock and update

@NateThompson114 Thanks for the tips until @kandashi update (happy dev).

@NateThompson114  THANK YOU SIR!  This worked perfectly!

@NateThompson114 Confirmed that this also works to eliminate the error in 10 build 288
Thanks so much!

thanks a ton @NateThompson114 !

fixed in #113 and available in v0.5.0


Add staking event to chain event

add new staking event + change propagate chain event to use a bytes slice instead of the event message

This closes #13


Update 2023-04-24-submissions-2023.md

Add details about the submission of new proposals

Gracias!


autoTrust not working anymore for chrome and firefox on linux

Looks like chrome and firefox have updated how they store certificates on Linux.
Maybe there is just a new url to put there: https://github.com/jsenv/https-local/blob/ef072a154d2f1dafdb2e00cd91d95224739b5649/src/internal/linux/chrome_linux.js#L40-L47

Since chrome 105, chrome uses his own certificate store on windows and Linux. Still relying on system for macos due to their policy.
For windows and Linux, until there is a programmatic solution found, trusting cert must be done manually.

The manual process is easy and described for instance here:
https://doc.webradiocontrol.tech/user-guide/installing-tls-certificate/chrome-linux/
But is there still a programmatic way to achieve that? That would be great


Android-Development-Java 2.1.5 DrawerLayout in Android

Description
Explain -

What is DrawerLayout
DrawerLayout with NavigationDrawer
How to sync toolbar with NavigationDrawer

Domain
Android Dev (Java)
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

/assign


Node with randomly added by CB Column.from_vector to select_nodes renders weirdly and behaves like a part of a vector editor

Discord username
No response
What type of issue is this?
Permanent – Occurring repeatably
Is this issue blocking you from using Enso?

[ ] Yes, I can't use Enso because of this issue.

Is this a regression?

[ ] Yes, previous version of Enso did not have this issue.

What issue are you facing?
When CB added Column.from_vector to select_columns ["id"] node it started behave weirdly (like it was a part of a vector) and had problems with rendering.
Expected behaviour
Column.from_vector should not be part of a widget.
How we can reproduce it?
No response
Screenshots or screencasts
https://github.com/enso-org/enso/assets/12892578/8b996367-261e-43f6-a8e0-39a1d0a4c64b
Enso Version
nightly 18.05
Browser or standalone distribution
Standalone distribution (local project)
Browser Version or standalone distribution
standalone
Operating System
MacOS
Operating System Version
No response
Hardware you are using
No response

Part of this issue is related to the list editor being used on a syntax tree that it didn't expect, and part is about the component browser duing a weird thing.

component browser edit
When I have a node with expression:

operator3.select_columns ["id"]

Then I enter its edit mode, the cursor is placed at its end and the component browser shows some suggestions. I select one with Column.from_vector, which modifies the node code to the following:
operator3.select_columns ["id"]Column.from_vector


Update various bits of the Cython FAQ for Cython 3

The Cython FAQ has been slowly bit rotting for years, but is now getting a huge blow by the developments for Cython 3.0.
https://github.com/cython/cython/wiki/FAQ
There are various bits of information sprinkled in there that are no longer true or for which better solutions exist now. This can be done iteratively, question by question, and whenever someone looks into it, but some effort needs to be put into it to make it more helpful again.

It's probably time to remove most of the comparisons to Pyrex. It's obviously worth acknowledging as the foundation of Cython, but I don't think anyone is trying to decide whether to use Cython or Pyrex at the moment

Instead, most people will probably wonder how it is different to MypyC.

There's a bit of comparison to mypyc (and other tools) in the readme file I believe (i.e. information spread in loads of different places)

Now that #5374 is merged, should we delete https://github.com/cython/cython/wiki/FAQ from the wiki?

Now that #5374 is merged, should we delete https://github.com/cython/cython/wiki/FAQ from the wiki?

Just done that (deleted the text and replaced it with a link)

I think this is done, mostly, so let's close the ticket.


NIFI-8081 Added new Listing Strategy to ListFTP and ListSFTP

https://issues.apache.org/jira/browse/NIFI-8081
New Listing Strategy is Adjusted Time Window. User can specify the time zone or time difference (compared to where NiFi runs) of the system hosting the files and based on the calculates the current time there. Lists files modified before this adjusted current time (and after the last listing).
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[x] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[x] Has your PR been rebased against the latest commit within the target branch (typically main)?

[x] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[x] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[x] Have you written or updated unit tests to verify your changes?
[ ] Have you verified that the full build is successful on JDK 8?
[ ] Have you verified that the full build is successful on JDK 11?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check GitHub Actions CI for build issues and submit an update to your PR as soon as possible.

Tested it with an SFTP server but it did not work for me at all. It does not list anything if the server is running in EST (NiFi in CET). If the server is in HKT (GMT+8), it lists the files only in the first round.
@tpalfy Could you please check it?

Tested it with an SFTP server but it did not work for me at all. It does not list anything if the server is running in EST (NiFi in CET). If the server is in HKT (GMT+8), it lists the files only in the first round.
@tpalfy Could you please check it?

@tpalfy Your changes have conflicts with NIFI-7263. Please rebase your branch onto main and resolve the conflicts.

@tpalfy Your changes have conflicts with NIFI-7263. Please rebase your branch onto main and resolve the conflicts.


help with field

Tell me how to filter the Item table so that only the Code and Description are in the field?  I don't need all the columns

@elhola you should use the yammer group https://www.yammer.com/dynamicsnavdev/ for partners
the twitter #bcalhelp or other dynamics nav / business central communities (https://community.dynamics.com/nav/b/navug) for these kind of questions.

This repository no longer handles general questions and is only dedicated to issue reporting for the latest preview version of the AL compiler and developer tools.
To get help for your question please use one of the following resources:
Business Central Community
mibuso forum
Dynamics User Group


Indented string display

Summary
Fixes https://github.com/Qiskit/qiskit-aqua/issues/1099
Current results
ComposedOp([
	SummedOp([
		ComposedOp([
			OperatorMeasurement(AbelianSummedOp([
	0.18093119978423136 * Z,
	-1.052373245772859 * I
])),
			     ┌───┐
q_0: ┤ H ├
     └───┘
		]),
		ComposedOp([
			OperatorMeasurement(AbelianSummedOp([
	0.18093119978423136 * Z,
	-2.104746491545718 * I
])),
			I
		])
	]),
	     ┌───┐
q_0: ┤ H ├
     └───┘
])

My current implementation (adding another display function passing the indentation level through its calls) is working for ListOp
ComposedOp([
	SummedOp([
		X,
		Z
	])
])

but making it work for more cases requires a lot of changes including changes to QuantumCircuit to make StateFn display properly.
Did you have another implementation in mind?
Details and comments

Hi, I had not really thought too much about the issue as it was raised quite recently. Here is a thought. How about a method that all the opflow entities can use if they wish in OperatorBase where I see you put something. Lets call it say indent(lines: str) -> str so given an string it will simply return an (one level) indented version back. I.e it will turn it into a list of strings at newline boundaries, prepend some indent to each string in the list, and the join them all back to a single string to return.
Now if any opflow entity is including other parts that it needs indenting, such as the string from a QuantumCircuit you can now do indent(str(qc)) for example. If each opflow entity does its own indent of what it needs in its str then it should all workout I think.
One concern with indenting in general might be that the output marches off to the right quite fast. Guess we'll have to see with how much indenting/nesting goes on in the most common sort of usage. Limiting the indent to say 2 or 4 spaces may still aid readability but keep it from marching to the right to fast. Since the indent would be in one place it would be easy enough to adjust. We could even set up OperatorBase to allow the indent level to be set, so as a user could override whatever default we choose.
Anyway hope this thought helps.

These are really interesting thoughts, thank you!
The indent approach is a great idea, I'll work on that! Now that you've suggesting it, it feels quite natural to pull the indentation from the callee to the caller to reduce changes downstream / on end dependencies and I wish I thought about that ^^
I'll add the centralized indentation + override capability as indentation could indeed be quite harmful to visualization...
Thank you for your good feedback. Not sure I'll have time to work on it this week but the issue said

This is a just requirement for nice-to-have.
so hopefully this is fine.

@woodsp-ibm thanks again for your previous comment. I worked on implementing this logic and it seems like having it in ListOp is enough as I felt like all Operators with nested elements inherit from it.
As of now, I put indentation as a parameter to __str__ but I don't think this is an easy way to set it up because it doesn't allow global setup. I couldn't find any class variables (ListOp.indentation) or global variables (INDENTATION) for any other operators so wasn't sure about your preferred / consistent way to go.

@romainfd The output certainly looks a lot nicer now than before. So none of the primitive operators need any form of indent, or whatever indent level they might use for their own formatting of content is fixed and looks ok I guess. The indenting you are adding is just around the listops then to ensure each element comes out indented. I had imagined some class or module level setting, maybe in OperatorBase class or the module its in, that could be set to control the indent. At that level it could be read by any of the opflow that wanted to allow control over the indenting - maybe at present only listop would use it. The operator_globals file might also be a place for it - maybe a better place - it has some global flag in there already around precision. Maybe a fixed level of indent is good enough, and some global control is not needed - the output seems way more readable already.

Can you please add unit test case(s) that create some operator(s) and check the output string meets expectation.

Hey, thanks for the good comments. I am on holidays this week but will work on that when I come back!

The operator_globals file might also be a place for it

Adding it there caused problems while importing it in Operators (I think it creates import cycles) so I moved it to OperatorBase. With this approach, it can also be updated for a subclass only (doing ListOp.INDENTATION = '\t') so I think this is quite good.
As asked, I also added a few unittests.
Let me know if there is something else I can improve.

No worries, my pleasure!
I added a comment based on what you suggested.
I marked it ready for review but indeed forgot to remove the [WIP], sorry!


Added node.send for when no files are found that match the pattern.

This node will hang if there are no files that match the pattern. This returns an empty payload array to indicate no files found. This guarantees the node will move on to the next one.

Hi, many thanks for submitting this. However, this changes the expected behaviour of the node.
As such, this really needs to be an option and to default to off so that existing flows are not compromised if the node is updated.
I you can add that, I'd be more than happy to accept the PR.

Julian,
I can make the option update if this is the only way to go.
I’m not sure how not responding to a file listing query is the expected behavior. It basically froze my Node-Red flow because the node didn’t match any files and then just sits there. The node has an output connector and so I would expect to always get an output from this node. Be it a list of files or an empty list; something so a flow could continue.
A returned empty array should be a valid output for this node. Users of this node should not always expect to get a populated array list returned. And they should also validate this array by checking the array length before attempting to use anything in the array.
Do you know of the current flow workaround to overcome this issue? It doesn’t throw an exception I can catch that I am aware of.
Thanks
John Malich
On Mar 20, 2019, at 8:52 PM, Julian Knight <notifications<EMAIL_ADDRESS>wrote:
Hi, many thanks for submitting this. However, this changes the expected behaviour of the node.
As such, this really needs to be an option and to default to off so that existing flows are not compromised if the node is updated.
I you can add that, I'd be more than happy to accept the PR.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/TotallyInformation/node-red-contrib-fs/pull/8#issuecomment-475081798, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ADUxcx_o5_8Jonr9qEnTH7FX5DPNIeiuks5vYtfSgaJpZM4b1NSz.

I’m not sure how not responding to a file listing query is the expected behavior

I am sure that you are correct that this would be better behaviour, it is just that it isn't the current behaviour and therefore changing it might break someone elses flows. If you can convince me that it wouldn't break any existing flows then I will let it through.
The alternative is that we bump the version to v2 so that people understand this is a breaking change.

Do you know of the current flow workaround to overcome this issue? It doesn’t throw an exception I can catch that I am aware of.

No, I'm afraid not. The whole idea of the node is that it doesn't generate errors.
To be honest, this node hasn't received a lot of love since it was created so if you fancy doing any other work on it, I would welcome it and would be happy to add you as a collaborator to the repo so that you can merge PR's yourself. Let me know. Unfortunately, I don't have a lot of time at the moment and what little I do is focused on node-red-contrib-uibuilder as that is a much more heavily used node.

Hi, apologies that this has taken so long to get to.
I have decided that, since I'm doing some other changes as well, I would add your suggestion.
You will always get something returned now. Will shortly be published as v1.1.0.

I appreciate that. Thanks!
John Malich
Jpmalich<EMAIL_ADDRESS>On Sep 21, 2019, at 12:51 PM, Julian Knight <notifications<EMAIL_ADDRESS>wrote:
Hi, apologies that this has taken so long to get to.
I have decided that, since I'm doing some other changes as well, I would add your suggestion.
You will always get something returned now. Will shortly be published as v1.1.0.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/TotallyInformation/node-red-contrib-fs/pull/8?email_source=notifications&email_token=AA2TC4ZUNCD3OQTKVTF2I7TQKZGH5A5CNFSM4G6U2SZ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7IVNMI#issuecomment-533812913, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AA2TC4YICQ3EMHXONTBCIDLQKZGH5ANCNFSM4G6U2SZQ.


Add plotNumber field to 'Plot Layout Details' of a study.

Plot number field is missing from Plot Layout Details of a study.

Plot Layout Details by study doesn't exist in V2.0 any more after an attempt to decouple the concepts of Study and Observation Unit a little bit and move them into separate modules. To retrieve the same information in V2.0 use
GET /observationunits?studyDbId={studyDbId}
this will retrieve all the positional information for each observation unit, including plotNumber


Share button is missing in app

Description:
User reported that he is missing share sensor button in Ruuvi Station Android app.
Issue was reproduced with these preconditions:

Gateway was offline, user was in free mode
User checked sensor sharing and didn't see button during this time
Subscription was updated to pro, still no sharing
Gateway was connected back online
Now user was able to see sensor sharing after clean installing app, but not before reinstallation

Confirmed doesn't occur anymore


System.MissingMethodException for IReadOnlyCollection after updating from JsonSchema.Net.Generation 4.3.0 to <IP_ADDRESS>

Nuget Package
JsonSchema.Net.Generation
Package Version
<IP_ADDRESS>
Operating System
Windows
.Net Target (if relevant, please specify the version in the description)
.Net (5 or after)
A clear and concise description of the bug.
When using AsyncAPI generation via the Neuroglia asyncapi package, which relies on JsonSchema.Net.Generation, in combination with the latest version of said package, the following exception occurs:
Unhandled exception. System.MissingMethodException: Method not found: 'System.Collections.Generic.IReadOnlyCollection1<System.Text.Json.Nodes.JsonNode> Json.Schema.JsonSchemaExtensions.GetExamples(Json.Schema.JsonSchema)'.`
This looks like a breaking change in the library?
What did you expect?
For the above call to succeed.
Please add test code or steps to reproduce the behavior.

Is there any other information you'd like to share regarding this bug?
No response
Code of Conduct & Contributor Guidelines

[X] I agree to follow this project's Code of Conduct and Contribution Guidelines.

Crap, I thought so.  cc: @SGStino
I'm going to revert this and unlist these packages.

Reversion pushed.  New package v6.1.2 should work for you.

@gregsdennis I can confirm the latest version fixed the issue. Thanks for the quick fix!

Crap, I thought so. cc: @SGStino (#704)
I'm going to revert this and unlist these packages.

We did the test in that issue: https://github.com/gregsdennis/json-everything/issues/704#issuecomment-2048227375
It requires a recompile, but in this case, of an intermediate package. So yes, queueing it for the next major might be a good thing.

@SGStino I have included these changes in v7.


Add bos_token to lm-zoo for gpt2 (and any other models that need it)

We determined that lm-zoo's gpt2 implementation matches HuggingFace's implementation to a good number of significant digits, but only when the beginning-of-sequence token bos_token is omitted from the sequence passed to HuggingFace. We need to confirm whether including or omitting bos_token is the best practice (most likely, including it is the best practice); if including it is the best practice, we should modify our invocation of gpt2 in lm-zoo to match this.

there's really no ground truth about the BOS token. In fact .. the original OpenAI implementation is even internally inconsistent, using BOS in its unconditional sampling code but not in its conditional sampling code. We don't have access to the original training code/dataset, but the model seems to do just fine anyway in both regimes.
But I think matching with Huggingface is a good rule of thumb, since that implementation is far more popular than the original.
I'll switch over lm-zoo / document / release accordingly today.

Just to complicate this -- I played with the vanilla Huggingface GPT2 and it actually doesn't add BOS, even with add_special_tokens=True. Plenty of confusion about this in the community e.g. here.
I'd still say incorporating BOS is better for our purposes / the purposes of lm-zoo. I'll just document this clearly on the site.

Not totally fixed yet -- looks like deps are breaking upon rebuilding transformers-base. Will look into this tomorrow.
There is also something breaking in unkify. Both show up in integration test failures below.
INFO:__main__:Loading tokenizer
INFO:__main__:Reading sentences from <_io.TextIOWrapper name='/tmp/tmpxpk_v855' mode='r' encoding='utf-8'>
INFO:__main__:Loading tokenizer
INFO:__main__:Reading sentences from <_io.TextIOWrapper name='/tmp/tmpxpk_v855' mode='r' encoding='utf-8'>
Traceback (most recent call last):
  File "/opt/transformers/get_surprisals.py", line 19, in <module>
    from transformers import AutoModelForCausalLM, AutoTokenizer
ImportError: cannot import name 'AutoModelForCausalLM' from 'transformers' (/opt/conda/lib/python3.7/site-packages/transformers/__init__.py)
E.
======================================================================
ERROR: test suite for <class '__main__.LMProcessingTest'>
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/opt/conda/lib/python3.7/site-packages/nose/suite.py", line 210, in run
    self.setUp()
  File "/opt/conda/lib/python3.7/site-packages/nose/suite.py", line 293, in setUp
    self.setupContext(ancestor)
  File "/opt/conda/lib/python3.7/site-packages/nose/suite.py", line 316, in setupContext
    try_run(context, names)
  File "/opt/conda/lib/python3.7/site-packages/nose/util.py", line 471, in try_run
    return func()
  File "/test.py", line 88, in setUpClass
    cls.surprisals_output = subprocess.check_output(["get_surprisals", text_f.name]).decode("utf-8")
  File "/opt/conda/lib/python3.7/subprocess.py", line 411, in check_output
    **kwargs).stdout
  File "/opt/conda/lib/python3.7/subprocess.py", line 512, in run
    output=stdout, stderr=stderr)
subprocess.CalledProcessError: Command '['get_surprisals', '/tmp/tmpxpk_v855']' returned non-zero exit status 1.
-------------------- >> begin captured stdout << ---------------------
== tokenize /tmp/tmpxpk_v855
<|endoftext|> This Ġis Ġa Ġtest Ġsentence . <|endoftext|>
<|endoftext|> This Ġis Ġanother Ġtest Ġsentence . <|endoftext|>
<|endoftext|> This Ġis Ġa Ġthird Ġsentence Ġwith Ġan Ġa ors nar nt Ġtoken . <|endoftext|>
<|endoftext|> This Ġis Ġa Ġsentence Ġwith Ġa Ġspecial ĠâĢ ľ Ġtoken . <|endoftext|>

== unkify /tmp/tmpxpk_v855
1 0 0 0 0 0 0 1
1 0 0 0 0 0 0 1
1 0 0 0 0 0 0 0 0 0 0 0 0 0 1
1 0 0 0 0 0 0 0 0 0 0 0 1

== get_surprisals /tmp/tmpxpk_v855

--------------------- >> end captured stdout << ----------------------

----------------------------------------------------------------------
Ran 1 test in 5.270s

FAILED (errors=1)


Video duration trimming to incorrect time length

Hello, I have a video setup as the following:
ffmpeg(path)
    .format('mp4')
    .size('?x500')
    .duration(10)
    .screenshots({
      timemarks: ['50%'],
      filename: name+'.png',
      folder: 'uploads',
      size: '?x700'
    })
    .on('error', function(err) {
      console.log('error', err.message);
    })
    .on('end', end)
    .save('uploads/'+name+'.mp4');

It seems as though if I upload a 10 second mp4 to my server and pass it through this function, I randomly get end up with videos between 5 - 8 seconds. Please let me know if I can give any more information that can help. My setup is:
node: 6.0.0
host: heroku
"fluent-ffmpeg": "^2.1.0"

I think the problem comes from ffmpeg. Can you try it from command line (using the command line reported by the 'start' event) and see if it behaves the same ?


chore: Tag WebRTC tests as flaky

copilot:all

Current dependencies on/for this PR:

main

PR #3453   👈

This comment was auto-generated by Graphite.


Fix broken link in case studies.

I fixed a broken link.

@jontybrook had to revert merge request as link pointed to an old md file.


Add traits to S57 Rational sequence topology

Are you sure about P87?  That seems to be a justification for P82.

P26 (separable) is now missing.


Store Outline until Example block is given

This fixes #12 and lets us put examples after the outline - as in the feature file spec.
Unfortunately, this means we lost the @noescape flag on the Outline method, which has a side effect of needing to add self. before all the Given / When etc. Sigh. We should chat though this to see if the benefit of conforming more closely to the Gherkin spec is worth all that extra cruft!
S

After some discussion, we decided this wasn't worth all the extra code


[TASK] Use Firebase Crashlytics and remove old fabric sdk

Stuff for Friday 2020/09/11 - 4.2
Can you look at Enabling Crash Reporting in Safer 2.6 please, Firebase?

Described in email.


Add Solo to scVI tools

Hi all,
We discussed this a while ago, and I finally have a bit of bandwidth.
Goal: Have solo be a model directly within scvi-tools
This still cool with you guys?
Best,
Nick

@adamgayoso You mentioned I should not use the scVI classifier module for solo, and instead code up my own vanilla NN. Is that still the recommendation? I see the classifier module seems to be up and running.

Hi @njbernstein
Yes we have a classifier, though haven't really completed adding a TrainingPlan class for the classifier. So as of now there's not yet a way to train just a classifier.
That said, I think to add solo to the codebase it's more about API decisions.

Should solo accept a pre-trained scvi model object as input? (with checks that it was trained on one lane)
If solo gets a dataset with multiple batches (lanes), it trains scvi per batch, right?
Are solo users tending to be on CPU/GPU? Maybe a sklearn MLP classifier is sufficient, since there's unlikely to be more than 20,000 observations and n_latent + 1 features, with the +1 being the library size. A sklearn MLP may be surprisingly fast, as if I recall now, you're re-encoding cells to train the classifier.
What needs to be saved and loaded? The scvi model? The classifier?

How do you envision the workflow?

Yes
Yes
It seems most use CPUs, but I'm hesitant to switch to sklearn. For scalability, I think GPUs will be necessary as datasets get larger. Also, it's been quite a while since I've used the MLP in sklearn, but I remember not having great luck with it.
Not sure what would be necessary, but I think both the scVI model and the classifier would be nice to save. That might be partially/completely a decision for a wrapper script in a separate repo.

I think your description when this first came up makes sense:
adata = sc.read...
scvi.data.setup_anndata(adata)
vae = scvi.model.SCVI(adata)
vae.train()
solo = scvi.model.SOLO(vae)
solo.train() # simulates doublets, trains classifier
solo.predict(adata)

Please checkout the PR.
I think it's easiest for us if the workflow follows the one presented in the docstring examples. The decision to run SCVI/SOLO for multiple batches is something that should be handled by your CLI.
Saving and loading the SOLO object works automatically, but only saves the classifier.
We can move this conversation over to the PR.

This is great! Thank you! I was still working my around understanding the repo structure.


💡 [REQUEST] - Lazy loading / splitting icon fonts css

Describe the feature
Hey, thanks for creating this plugin, it's been very helpful when transitioning to nuxt.
We use a lot of the mdi-icons notice our initial page render spit out like 600kb of icon declarations.  This is understandably very slow. I was wondering if we could edit the plugin / add a config option that would move these into a seperate stylesheet which could be heavily cached, we could save a lot of time on inial load.
Perhaps this feature already exists? any info would be greatly appreaciated.
Thanks

A LOT of these mdi declarations are plonked at the top of index.html

Additional information

[x] Would you be willing to help implement this feature?
[ ] Does this feature have a potential to break existing features?

You can use inlineStyles option to disable inlining all entry css. But this will externalize all globally defined css in nuxt.options.css.
export default defineNuxtConfig({
  features: {
    inlineStyles(id) {
      return !(typeof id === 'string' && id.endsWith('nuxt/dist/app/entry.js'))
    },
  },
}

But if you still want to be able split the font icons, I can think of something else. But since that will alter the order of css, it will not be enabled by default.

Thank you!
This helped our homepage to go from 600kb to 17kb. Lifesaver.


Filter stories by contributor

What
Filter a stories search by contributor
in relation to Stories search#5681

Long id appearance 🐛
No issue:
Selecting top 20 contributor with a format combination. Either selection can be removed without the label changing.
Issue replication:

Select a format which will reveal contributors beneath the top 20 listed contributors.
Select a contributor from this beneath top 20 category
Remove the format filter
Their long id will now be displayed.

A good way to see this in practice is to choose a top 20 contributor and a beneath top 20 contributor and then remove the format filter. The top 20 contributor label will stay the same, the beneath top 20 contributor label will transform into the long id.
Screenshot gifs below
One contributor

Two contributors

Above issue sorted with API corrections - https://github.com/wellcomecollection/content-api/pull/59 + https://github.com/wellcomecollection/content-api/pull/60
PR - https://github.com/wellcomecollection/wellcomecollection.org/pull/10035 + https://github.com/wellcomecollection/wellcomecollection.org/pull/10038


CI fails with nuget restore

For example: CI #1196 Attempt #2
https://github.com/y-iihoshi/ThScoreFileConverter/runs/6432659306?check_suite_focus=true#step:6:8
Run nuget restore ThScoreFileConverter.sln
  nuget restore ThScoreFileConverter.sln
  shell: C:\Program Files\PowerShell\7\pwsh.EXE -command ". '{0}'"
  env:
    pythonLocation: C:\hostedtoolcache\windows\Python\3.10.4\x64
MSBuild auto-detection: using msbuild version '<IP_ADDRESS>01' from 'C:\Program Files\Microsoft Visual Studio\2022\Enterprise\MSBuild\Current\bin'.

MsBuild timeout out while trying to get project to project references.
Error: Process completed with exit code 1.

https://docs.microsoft.com/en-us/nuget/consume-packages/package-restore#restore-using-msbuild

Use the msbuild -t:restore command to restore packages listed in the project file (see PackageReference) and starting with MSBuild 16.5+, packages.config projects.


nohup command causes bats-format-tap-stream hang

Hi,
I was stuck in this issue few days. Seems nohup command in following test code would cause bats-format-tap-stream hang and never exit except Ctrl+C to interrupt. Anyone know how to fix it?
#!/usr/bin/env bats

load test_helper
fixtures bats

@test "a passing test" {
  nohup /bin/sleep 30000000 &
  echo 'hello'
}

$ bin/bats -t test/case.bats
1..1
ok 1 a passing test
              <<<==== hang here

I suffered the same problem. In general when a daemon is running in background and test fails, then it's not recognized until CTRL-C.
 I'm having another solution that let $formatter read input from file, not pipeline. Bcoz I have other code to analyze the test result with pretty format.
Anyway, hope bats can consider how to handle this scenario. In mostly test case, using nohup to start/stop daemon is often case.


Package is broken on Mac OS

When following the instructions in https://www.sphinx-doc.org/en/master/usage/markdown.html, I get the following error from Sphinx:
>make html
Running Sphinx v2.0.1

Exception occurred:
  File ".venv/lib/python3.6/site-packages/recommonmark/parser.py", line 9, in <module>
    from commonmark import Parser
ModuleNotFoundError: No module named 'commonmark'

Looking in the site-packages directory, there is a subdirectory called CommonMark, but no commonmark.

Upon further investigation, this appears to happen whenever upgrading in an existing environment with an older installation of commonmark.


Shift-click on a tag to add it to excluded tags

It would be very nice to be able to shift-click on a tag in the tag cloud, and it would be added to the excluded tags rather than included tags filter! 👍

Cool idea!


How to start typing from left aligned?

Example is here
It's totally original example.

The syles.css file is applied to the whole App, which includes the editor. To fix, remove the text-align: center; line or add text-align: initial; styling to the editor (for example by using the jodit-container class)

style: {
textAlign: "left",
}
add this in your config


Expose ProtocolHandlerPlugin so it's possible to call 'handleEventURL' for multiple plugins

We've run into the issue that we have multiple plugins that depend on setting an event handler through NSAppleEventManager.shared().setEventHandler, the next one always overwriting the previous one.
We are looking into exposing the handler methods on those plugins so we can achieve something like this:
    public func handleURLEvent(_ event: NSAppleEventDescriptor, with replyEvent: NSAppleEventDescriptor) {
        HandlerA.instance.handleURLEvent(...)
        HandlerB.instance.handleURLEvent(...)
       // other handlers
    }

Please let me know what you think of this approach (or if you have any better ideas for our issue)

LGTM


No Data in Data Preview after upgrade from 2.2.5 to 2.2.6 or 2.2.7

Describe the bug:
No Data in Data Preview, but Approx Entity Count show the correct number.
After downgrade back to 2.2.5, data appeared again.
Also while open app show the following error info if the connection was not closed last time.
TypeError: Cannot read properties of undefined (reading 'showCollections')
Then click close connection
TypeError: Cannot read properties of undefined (reading 'closeConnection')
Steps to reproduce:

Upgrade from 2.2.5 to 2.2.7
Open app
Open the Data Preview page of the old collection
Close Connection

Attu version:
2.2.6,2.2.7
Attu version:

what is your milvus version

what is your milvus version

v2.2.5

what is your milvus version

v2.2.5

I will check soon.

我也遇到了，我的 milvus 版本是 2.2.6
我的 attu 版本是 2.2.8
既不然预览数据，也无法查询数据

ok, I will test it on v2.2.6.

我也遇到了，我的 milvus 版本是 2.2.6
我的 attu 版本是 2.2.8
既不然预览数据，也无法查询数据

已解决：

等待不解决问题
升级 attu 到最新的 2.2.8 不解决问题
把 attu 降级到 2.2.5 解决问题

我的 milvus 版本是 2.2.6


Feature: Auto refresh icons on sidebar when icons are changed

What feature or improvement do you think would benefit Files?
A way to refresh the icons on the favorites bar. I really like to use folder icons with my favorited folders so that it is easy to identify them, but when I change an icon for a favorited folder, I have to remove the favorite and add it back again.
Requirements

Add file watcher for items in the sidebar to reload icons when they're changed

Files Version
2.3.12
Windows Version
Windows 11 22000 1098
Comments
No response

What happens if you restart the app, does the correct icon load?

Yes, it does.

FWIW, there's a Windows Shell API to detect shell folders and normal folders but I'm not sure we can do with WinUIEx because notifications are set to the registered window (in this case Files window) via WndProc. That's said, I believe we can create a hidden window object for us to be notified of changes.
LRESULT WndProc(HWND hWnd, unit message, WPARAM wParam, LPARAM lParam)
{
    switch (message)
    {
        case WM_SHCHANGENOTIFY:

            LPITEMIDLIST** ppidl;
            long* lEvent;
            var hLock = SHChangeNotification_Lock((nint)wParam, (uint)lParam, ppidl, lEvent);

            if (!hLock)
                break;

            if (*lEvent == SHCNE.SHCNE_UPDATEIMAGE)
                UpdateSidebarIcons();

            SHChangeNotification_Unlock(hLock);
            break;

        //...
    }

    return (LRESULT)0;
}

We have a method for this but we don't use this as of now.
https://github.com/files-community/Files/blob/db3491617d7e8d46fef01d798006ab96d3be88e7/src/Files.App/Data/Items/WindowEx.cs#L266-L293

FileSystemWatcher won't work for virtual shell folders such as Recycle Bin and Quick Access because it uses a raw Win32 API that requires a full Win32 namespace path.


Teleporter problem

The Teleporter is not working correctly when entering it from left or from top. It works perfectly from down and right.

I guess you are just surprised how the teleporter works. I guess LogicDeluxe invented it, so I just describe what I have observed:

Dirt/boulders etc. block a teleporter. (If the target teleporter has dirt at its exist, no teleporting)
Entering source teleporter from the left => exit target teleporter on the right (vice versa)
Entering source teleporter from the top => exit target teleporter on the bottom (vice versa)
The target teleporter is determined this way: Starting from Rockford's location the next teleporer is searched row by row. When the level end is reached, it is started from the beginning. Only a teleporter which is not blocked on the appropriate exit is considered. And even the source teleporter might be the target teleporter!

In doubt, check C64 Crazy Light Engine!
I guess you have two teleporters which have spaces on their sides.
Entering from the left: Looks like walking through? Yes, according to spec this is correct.
Entering from the top? See yourself ;-)

The teleporter behavior is certainly well-conceived but on the other side it’s not fully logical for regular players.

When I played around with the teleporter I was surprised, too. Then I checked some of the original C64 caves and I saw that the behavior is intended. I can only speculate why the teleporter behaves like this: Maybe easier to implement on C64? And if the most exists are blocked with steel walls, you may not notice it.


update functional tests and associated resources

Gets the functional tests working again.

They're comment based, so I need to merge this into main first. It was tested here: https://github.com/dominodatalab/hephaestus/pull/155.

all the runs have failed.


Set default data attributes of marks to []

The default value of the principal data traits of marks (x and y for Lines) is None, even though they do not allow None.
We should be able to first create the Mark instance, and then set the data attributes programmatically.
This depends on #291

Thanks @rmenegaux This looks good.


Updated pricing section.

Regarding Issue #177
Made shadow appear expanded when mouse hovers on the cards. This makes the card seems to hover up.
(Look at the third card in the screenshot.)

Please inform me if the changes I made are good or not. And please merge it @abhay-raj19

LGTM @Rakshit-gen


WFLY-16451 [primary/secondary] Problematic Language usage deprecation  and replacement of Java docs/variable names

Issue: https://issues.redhat.com/browse/WFLY-16451

@yersan pr is ready for review.
Thanks

Superseded by https://github.com/wildfly/wildfly/pull/16192

Closing in favor of #16192. Thanks @parsharma!


Can we draw a structure (During the runtime) using VTS ?

Hi,
I just wanted to know if we can draw any objects or structures like a cube or something after the scene has been rendered on the browser. Is there any functionality available on VTS which will enable me to do so? Please Provide me with some insights regarding that.
Thank You,

Hi,
yes, you can render additional objects. There are examples which can be interesting for you:
https://jsfiddle.net/Lzb2o5uj/
https://jsfiddle.net/2uubdxa7/
More examples can be found there:
https://github.com/Melown/vts-browser-js/wiki/Examples
or there:
https://github.com/Melown/vts-browser-js/tree/master/demos


ledger/contracts: Pretty instead of Show

Use Pretty instead of Show for all the debug output that's printed from the test predicated
Add a number of Pretty instances to ledger types
Also replace two occurrences of Either with proper datatypes

Hang on, some more changes are coming

Should be OK now, sample output: https://gist.github.com/j-mueller/43232b729120e5c45e900fa9016bcb36

It might be good to have a test that shows an error output.

I'll do that as well.


exitstat.bpf.o missing?

First up: below looks awesome! As a big atop fan I am excited to see development in this area. Great job.
On my pretty vanilla Arch box:
$ podman run --privileged --cgroupns=host --pid=host -it below/below:latest
Nov 02 12:02:39.065libbpf: failed to open /usr/facebook/below/bpf/exitstat.bpf.o: No such file or directory
 WARN Failed to initialize BPF: Failed to increase rlimit. Data collection will be degraded. You can ignore this warning or try to run with sudo.
Nov 02 12:02:39.071 ERRO%

I'm assuming that something is supposed to compile https://github.com/facebookincubator/resctl/blob/master/resctl/below/src/bpf/exitstat.bpf.c and put it in the path mentioned above but there's no such location in the image (in fact there's no /usr/facebook at all). I had a snoop about but I'm woefully unfamiliar with building Rust projects and have no idea how this is supposed to work.

Hi Mark,
Right now the exitstat bpf prog isn't packaged for open source yet. Internally it's shipped with the RPM. Once I finish up https://github.com/libbpf/libbpf-rs/pull/24 the bpf prog will be embedded inside of below so we won't need to worry about packaging anymore.
For now, the warning is harmless -- below will just have slightly incomplete data.

The bpf skeleton stuff is done now


Release region viewer update

#48 made breaking changes to region-viewer. When that is released, all track packages will also have to be updated to allow region-viewer version 1 or 2 in peer dependencies.

Done in 3474b6a834e462795eb32938437c9a7729fe4183 and 7443b69505051e98b719d3367e4e519a03b53bc2.


Compile Errors

RuntimeError: Error building extension 'enclib_gpu': [1/4] :/usr/local/cuda-9.0/bin/nvcc -DTORCH_EXTENSION_NAME=enclib_gpu -I/home/ayx/anaconda3/envs/pytorch/lib/python3.6/site-packages/torch/lib/include -I/home/ayx/anaconda3/envs/pytorch/lib/python3.6/site-packages/torch/lib/include/TH -I/home/ayx/anaconda3/envs/pytorch/lib/python3.6/site-packages/torch/lib/include/THC -I:/usr/local/cuda-9.0/include -I/home/ayx/anaconda3/envs/pytorch/include/python3.6m -D_GLIBCXX_USE_CXX11_ABI=0 --compiler-options '-fPIC' -std=c++11 -c /home/ayx/ShelfNet-citys/encoding/lib/gpu/encoding_kernel.cu -o encoding_kernel.cuda.o
FAILED: encoding_kernel.cuda.o
:/usr/local/cuda-9.0/bin/nvcc -DTORCH_EXTENSION_NAME=enclib_gpu -I/home/ayx/anaconda3/envs/pytorch/lib/python3.6/site-packages/torch/lib/include -I/home/ayx/anaconda3/envs/pytorch/lib/python3.6/site-packages/torch/lib/include/TH -I/home/ayx/anaconda3/envs/pytorch/lib/python3.6/site-packages/torch/lib/include/THC -I:/usr/local/cuda-9.0/include -I/home/ayx/anaconda3/envs/pytorch/include/python3.6m -D_GLIBCXX_USE_CXX11_ABI=0 --compiler-options '-fPIC' -std=c++11 -c /home/ayx/ShelfNet-citys/encoding/lib/gpu/encoding_kernel.cu -o encoding_kernel.cuda.o
/bin/sh: 1: :/usr/local/cuda-9.0/bin/nvcc: not found
[2/4] :/usr/local/cuda-9.0/bin/nvcc -DTORCH_EXTENSION_NAME=enclib_gpu -I/home/ayx/anaconda3/envs/pytorch/lib/python3.6/site-packages/torch/lib/include -I/home/ayx/anaconda3/envs/pytorch/lib/python3.6/site-packages/torch/lib/include/TH -I/home/ayx/anaconda3/envs/pytorch/lib/python3.6/site-packages/torch/lib/include/THC -I:/usr/local/cuda-9.0/include -I/home/ayx/anaconda3/envs/pytorch/include/python3.6m -D_GLIBCXX_USE_CXX11_ABI=0 --compiler-options '-fPIC' -std=c++11 -c /home/ayx/ShelfNet-citys/encoding/lib/gpu/syncbn_kernel.cu -o syncbn_kernel.cuda.o
FAILED: syncbn_kernel.cuda.o
:/usr/local/cuda-9.0/bin/nvcc -DTORCH_EXTENSION_NAME=enclib_gpu -I/home/ayx/anaconda3/envs/pytorch/lib/python3.6/site-packages/torch/lib/include -I/home/ayx/anaconda3/envs/pytorch/lib/python3.6/site-packages/torch/lib/include/TH -I/home/ayx/anaconda3/envs/pytorch/lib/python3.6/site-packages/torch/lib/include/THC -I:/usr/local/cuda-9.0/include -I/home/ayx/anaconda3/envs/pytorch/include/python3.6m -D_GLIBCXX_USE_CXX11_ABI=0 --compiler-options '-fPIC' -std=c++11 -c /home/ayx/ShelfNet-citys/encoding/lib/gpu/syncbn_kernel.cu -o syncbn_kernel.cuda.o
/bin/sh: 1: :/usr/local/cuda-9.0/bin/nvcc: not found
[3/4] :/usr/local/cuda-9.0/bin/nvcc -DTORCH_EXTENSION_NAME=enclib_gpu -I/home/ayx/anaconda3/envs/pytorch/lib/python3.6/site-packages/torch/lib/include -I/home/ayx/anaconda3/envs/pytorch/lib/python3.6/site-packages/torch/lib/include/TH -I/home/ayx/anaconda3/envs/pytorch/lib/python3.6/site-packages/torch/lib/include/THC -I:/usr/local/cuda-9.0/include -I/home/ayx/anaconda3/envs/pytorch/include/python3.6m -D_GLIBCXX_USE_CXX11_ABI=0 --compiler-options '-fPIC' -std=c++11 -c /home/ayx/ShelfNet-citys/encoding/lib/gpu/roi_align_kernel.cu -o roi_align_kernel.cuda.o
FAILED: roi_align_kernel.cuda.o
:/usr/local/cuda-9.0/bin/nvcc -DTORCH_EXTENSION_NAME=enclib_gpu -I/home/ayx/anaconda3/envs/pytorch/lib/python3.6/site-packages/torch/lib/include -I/home/ayx/anaconda3/envs/pytorch/lib/python3.6/site-packages/torch/lib/include/TH -I/home/ayx/anaconda3/envs/pytorch/lib/python3.6/site-packages/torch/lib/include/THC -I:/usr/local/cuda-9.0/include -I/home/ayx/anaconda3/envs/pytorch/include/python3.6m -D_GLIBCXX_USE_CXX11_ABI=0 --compiler-options '-fPIC' -std=c++11 -c /home/ayx/ShelfNet-citys/encoding/lib/gpu/roi_align_kernel.cu -o roi_align_kernel.cuda.o
/bin/sh: 1: :/usr/local/cuda-9.0/bin/nvcc: not found
ninja: build stopped: subcommand failed.

/bin/sh: 1: :/usr/local/cuda-9.0/bin/nvcc: not found
Looks like your cuda is not correctly configured

/bin/sh: 1: :/usr/local/cuda-9.0/bin/nvcc: not found
Looks like your cuda is not correctly configured

but,my nvcc has been installed correctly:
nvcc --version
nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2017 NVIDIA Corporation
Built on Fri_Sep__1_21:08:03_CDT_2017
Cuda compilation tools, release 9.0, V9.0.176

Not sure what happened. But it's definitely a problem with cuda and nvcc. Maybe your cuda or nvcc is not installed exactly in this path /usr/local/cuda-9.0/bin/nvcc.  Also please check ninja version, I'm using ninja 1.8.2, and have not tested on other versions.
An alternative choice, you can abandon building this repo, instead, you can replace all synchronized batch norm layer with the default batch norm layer in PyTorch.

@aiyanxiao Did you solve this problem?

@aiyanxiao @celabi check this: gedit ~/.bashrc --> export CUDA_HOME=/usr/local/cuda  As my experience you might have one more : here, similar like this "export CUDA_HOME=:/usr/local/cuda".
Alternative: cat :/usr/local/cuda-9.0/bin/nvcc You will find it does not exist. :P


getting closer to actual header

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix #89
Test URLs:

Before: https://main--shredit--stericycle.aem.page/en-us/about/press-releases
After:

https://header--shredit--stericycle.aem.page/en-us/about/press-releases
https://header--stericycle-shared--aemsites.aem.page/en-us/about/press-releases

you'll need to override the button css that the email links is inheriting


Support onion invites again

Not sure when this broke, not everyday that I use it :) but people are interested in this and I have just checked that this makes invites work again (together with a minor patch in scuttlebot).

Unless anyone objects, I'll merge this soon, and add some documentation about tor.
Related:
https://github.com/ssbc/secret-stack/pull/18
https://github.com/ssbc/scuttlebot/pull/450

merged into 2.7.2 - sorry, my github notifications got really messy, but I got them tidied up so I should see things relatively quickly now.

oops, 2.8.1

Thanks. What about the two linked issues? One of them is on secret stack, where this one is also open? https://github.com/ssbc/secret-stack/pull/16


Create a QC dashboard that will show daily number of detectors by its health status

We have already created mart level data  named 'DIAGNOSTICS__DETECTOR_DAILY_BY_STATION' that contains the details breakdown count of detectors health. Use two weeks dynamic detector status to develop dashboard that will track the details health status breakdown and ensure data quality over time. This model has county code, make sure county full name reflects before using the data in dashboard p[reparation.
Following visualization can be developed-
a.  two weeks Good and bad detectors time series daily count
b. Details Breakdown of daily detectors count by health status such as constant count, card off, down, high value issues, insufficient data, and intermittent count
c. Health status Breakdown by District, County, Route, and station type

This is dependent on the county name PR #443  to be reviewed/completed before this can start.

A draft dashboard of detector daily health has been established, here is the link
https://svgcmdl02:5601/app/dashboards#/view/e5133f00-9afa-11ef-8f47-a5289c4fae81?_g=(filters%3A!()%2CrefreshInterval%3A(pause%3A!t%2Cvalue%3A0)%2Ctime%3A(from%3Anow-15d%2Cto%3Anow))


ADDED CODECHEF JUNE LONG CHALLNEGE DIV 3 SOLUTION IN C++

Hello @AkashSingh3031 ,
Kindly Review this PR !

@codeaholic-shub
After changing the code again send a Pull Request

Please check your solution of the 3 problems Bella ciao, Bitwise Tuples, Shortest Route

I have already checked , its already submitted solutions of codechef from my account

Hello @AkashSingh3031 ,
I have changed the solution and its correct,
U may check my PR !


pritunl-client: refactor, add service and electron app

Description of changes
Refactor of pritunl-client:

cli is split in a subpackage
service subpackage added, building the pritunl-client-service binary, similar to the official deb package structure.

includes a systemd .service, which can be enabled with:

systemd.packages = [ pkgs.pritunl-client ];
systemd.targets.multi-user.wants = [ "pritunl-client.service" ];

electron app is copied to lib/pritunl_client_electron/, with icons and .desktop, ran using electron package
pull shell completion from #231292
add myself as maintainer

The service and electron app should work on Darwin, but I don't have a machine to test. This is working great though on Linux
Closes #233793
Includes/supersedes #231292
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Hi @andrevmatos, thanks for the great works. I'm still learning how to create service in Nix, can't get my head around it. So please add other reviewer who capable of doing it. Thanks.

Result of nixpkgs-review pr 238857 run on x86_64-linux 1

1 package built:

pritunl-client

@minizilla thanks for the heads-up; in this case, there's no actual [module] service added to the package; I went the route of simply adding the service file to the packages, and letting user add it through systemd.packages if desired, so no module file/config to maintain for something so simple as a 3-line service file.
I'll try to ask somebody else to review as well then.
Thanks

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2360

@SuperSandro2000 would the current version be acceptable? Can you give another look, please?

Rebased after #231292 was merged.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2421


add dinosaur that breaks everyone's period.split(' ').last == value code

I like to be mean. More importantly, the "all observed period values are two words, the latter of which is the actual period" is a really brittle approach. Add a dinosaur that makes clear that this approach doesn't work well on real datasets.

I like it.
Are we looking for a .in? solution?

.in?, .include?, =~. There are still lots of options, but they all recognize the limitations of the previous approach.

:+1:


option to use file name as the file label

no need for a specific flag, the -l flag is already optional, so if no labels were provided the file paths should be used. and common parts of the paths should be replaced by ".."

added in commit https://github.com/MouslihAbdelhakim/Quick/commit/23f59ae4ac1698103818d648b525f798f6fa151d