Fixing failed counter test for unsupported paths

Commented out the lookup for unsupported paths as its confirmed that the subinterface discard counters are not supposed to be supported

Added the await check before the interface config (ConfigureDUTIntf) to make sure interface is in operational state before fetching the counters.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 70.423%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1050

Relevant Lines:
1491

💛  - Coveralls

Creating new PR with CLA authentication


Added RuntimeIdentifiers property for non-UAP projects for PackageReference

Added RuntimeIdentifiers property for non-UAP projects for PackageReference scenarios inside Microsoft.NuGet.Targets file. For UAP projects, It's already being handled by UAP team.
Fixes https://github.com/NuGet/Home/issues/4030
@rrelyea @emgarten @jasonmalinowski @tmeschter

Hi @jainaashish, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;


MessagesDeliveredAsync not triggering (F&F=false)

There is one test scenario which is hanging until Timeout due to MessagesDeliveredAsync not being triggered.
The scenario works as follows: I have implemented FireAndForgetDelivery and I am trying to test it. The GrainsQueueAdapter.QueueMessageBatchAsync() won't return until the MessagesDeliveredAsync triggers and marks the message as completed.
// F&F=false
When_Sending_Compound_Message_One_To_Many
-> EmitterGrain.SendAsync(text, data)
    -> CompoundReceiverGrain.OnNext()
        -> BlobReceiverGrain.OnNext()
        -> SimpleRecevierGrain.OnNext()

The messages are passed all the way to the BlobReceiverGrain.OnNext() but unfortunately it hangs after finishing that method execution. SimpleRecevierGrain.OnNext() is never triggered in the CompoundReceiverGrain.OnNext() due to awaiting the completion of the BlobReceiverGrain.OnNext(), which executes successfuly.
All other tests (F&F on and off) work fine. This one doesn't and I guess its due to two consecutive OnNext invocations in the CompoundReceiverGrain.OnNext().
Please help me trace this issue by either reviewing the code of the PostAsync or by debugging the test case in the .devcontainer.

I have noticed that if I don't await inside the CompoundReceiverGrain.OnNext() I will get MessagesDeliveredAsync triggered for all messages in the test. I really do need to await them so this is not a good workaround but maybe someone can explain this behaviour to me.

cc @benjaminpetit @ReubenBond

Solved by implementing custom Queue Balancer for Tests only


Time zone identifier string (Tzid) should be significant when checking equality between events

Right now the comparison seems to look at start and end times. In this case, Toronto and New York have the same times. If I change one of the values to something completely different (Europe/Amsterdam for example), this behaves as you would expect.
Have a look at GetHashCode and Equals.
Failing unit test:
var now = DateTime.Now;
var later = now.AddHours(1);

var firstEvent = new Event
{
    DtStart = new CalDateTime(now, "America/New_York"),
    DtEnd = new CalDateTime(later, "America/New_York"),
};

var calendar = new Calendar();
calendar.Events.Add(firstEvent);

var secondEvent = new Event
{
    DtStart = new CalDateTime(now, "America/Toronto"),
    DtEnd = new CalDateTime(later, "America/Toronto"),
};

var same = firstEvent.Equals(secondEvent);
Assert.IsFalse(firstEvent.Equals(secondEvent));

Actually, the problem is in CalDateTime, and there's even a note I helpfully left myself by the Equals implementation:
//ToDo: Time zone equality should include time zone values. Perhaps we should separate the idea of "equal" with "equivalent"
/// <summary>Equality is determined by the unambiguous UTC representation of the time. Time zone string values are ignored.</summary>


Fix matcher doc typo

Fixing typo and discussing documentation
:memo: Checklist

[x] All user-facing changes have changelog entries.
[x] The changes are reflected on vast.io, if necessary.
[x] The PR description contains instructions for the reviewer, if necessary.

:dart: Review Instructions
NA

https://github.com/tenzir/vast/blob/0226c38a946c34c326eb2cb9d41e2b46ee6737f0/web/docs/use-vast/detect/match-threat-intel.md#L342-L343
@mavam why couldn't we update threat intel from files through the client?

Ohter questions:

why is the schema + indicator concept provided for Pulsedive in the matcher plugin and not the Feodo tracker?
why are the matchers created client side not persisted like server side ones?

https://github.com/tenzir/vast/blob/0226c38a946c34c326eb2cb9d41e2b46ee6737f0/web/docs/use-vast/detect/match-threat-intel.md#L290-L293
This section should not be called Context Usablility (this is a copy/paste from the section above. Also the trailing ::: is a typo

@mavam why couldn't we update threat intel from files through the client? The Replace state at the server feature looks like an anti-pattern to me.

The idea behind state replacement is that it allows for shipping matcher "storage" easily. For example, one matcher backend is DCSO's bloom. Anyone can generate Bloom filters with bloom and directly load them into a matcher.
This use case of shipping intel "packages" in binary form is not yet that common, but the more we go into the edge, the more relevance such a feature has.

@rdettai and I discussed this out-of-band.

@mavam why couldn't we update threat intel from files through the client? The Replace state at the server feature looks like an anti-pattern to me.

The documentation does a bad job at showing a good use case for this feature, which was intended for large (> GBs) updates that would be expensive to push through the client, and typically arrive on the node out-of-band through the filesystem. I'll rephrase this in the docs.
I looked at the code again and wanted to see if the following warning is correct:

You must ensure manually that VAST does not write to the matcher state file while exchanging the underlying state file.

At a first glance, I actually think the statement is wrong and it is safe, given that a matcher actor operates entirely sequentially.
@dominiklohmann if we go from self->request to self->await in the implementation, would you agree that this is safe?

Also the trailing ::: is a typo

I fixed this in a work-in-progress PR.

This section should not be called Context Usablility

Also fixed.

why is the schema + indicator concept provided for Pulsedive in the matcher plugin and not the Feodo tracker?

This is an oversight. For a complete example, we should do both.

why are the matchers created client side not persisted like server side ones?

Because that would entail rewriting the server-side configuration. In my opinion, the ephemeral matchers are the ones that would be expendable. I see them as an interactive feature. What's missing is perhaps the ability to dump a matcher configuration so that it's easy to create a persistent matcher from an ephemeral one. But so far we haven't gotten a lot of feedback on this feature.

You must ensure manually that VAST does not write to the matcher state file while exchanging the underlying state file.

At a first glance, I actually think the statement is wrong and it is safe, given that a matcher actor operates entirely sequentially.
@dominiklohmann if we go from self->request to self->await in the implementation, would you agree that this is safe?

We already do self->request(filesystem, ...).await(...) when matchers reload their state for exactly this reason.

We already do self->request(filesystem, ...).await(...) when matchers reload their state for exactly this reason.

Ah right, so then should I update the docs? I think they make it sound more dangerous than it actually is.


Implement speed progress message

$ art init s3://art-vcs/datasets/foobarbar
$ art push
upload objects: (125/125), speed: 1.07MB/s
0 modified(M), 125 added(+), 0 deleted(-), 0 renamed(R)
create commit: c1ff40606ec9d01eb3d33bddfc049f672eab5d69
update ref: latest -> c1ff40606ec9d01eb3d33bddfc049f672eab5d69

There is a performance impact about 40% when i download data from s3. I will look into the root cause.

LGTM


enable dependabot for GitHub actions

This allows to get updates for GitHub actions automatically. I have used this for my own packages, the Trixi.jl framework, and the SciML organization. After merging this, you could also enable other Dependabot actions in 'Settings -> Code security and analysis -> Dependabot alerts' and '... -> Dependabot security updates'.
See https://github.com/SciML/MuladdMacro.jl/pull/37

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5

Relevant Lines:
5

💛  - Coveralls


Some UI improvement is needed in the Hacktoberfest2022.html

Description
I've noticed some UI irregularities , The screenshots are attached below.
Screenshots

Can I work in this issue?

Go for it!

All of the previous issues that i've identified are fixed by the update 1.2.2 . I am updating this issue with some new ui irregularities that i've noticed.


Generate chunks around the player in a spiral-motion

The chunks should be generated in a spiral rather than a cube, to ensure that the chunks closest to the player will be generated before the ones that are further away.

Instead of a spiral-


GVRAndroidResource private constructor with InputStream

Hi,
Thank your for your great sdk !
I do not understand why GVRAndroidResource(String, InputStream) is private ? Our application has already is own assets manager (SDcard, resource, assets, Urls...) and can return a FileDescriptor or an InputStream so we don't have to worry about the source. Could we move this constructor public or is it possible to add a public constructor GVRAndroidResource(FileDescriptor fd) ?
Thanks for your reply

For our purposes, it's currently used only by the ZipLoader according to comments:
 * Note that this constructor is package private and is only used by the {@link ZipLoader}
 * (for now).

This could change though before any such decision you are welcome to extend/modify this class and add your own functionalities

@jeanpul Historical reasons. I find it all very convoluted (even somewhat broken) but is not likely we will refactor soon.
You might be able to do something like the following; have not verified it as working:
    private class YourResourceVolume extends GVRResourceVolume {
        private InputStream mYourStream;

        public YourResourceVolume(GVRContext context, InputStream yourStream) {
            super(context, VolumeType.INPUT_STREAM);
            mYourStream = yourStream;
        }

        @Override
        public GVRAndroidResource openResource(String ignored) throws IOException {
            return super.addResource(new YourResource(mYourStream));
        }

        private class YourResource extends GVRAndroidResource {
            private InputStream inputStream;

            public YourResource(InputStream inputStream) throws FileNotFoundException {
                super("");
                this.inputStream = inputStream;
            }

            @Override
            public synchronized void openStream() throws IOException {
                setStream(inputStream);
            }
        }
    }

....

        InputStream yourStream = null; // get it from somewhere
        YourResourceVolume volume = new YourResourceVolume(getGVRContext(), yourStream);
        GVRAndroidResource androidResource = volume.openResource(null);

Or just use reflection and mark the package private ctor as accessible.
And finally, considering how much detail we already expose, I will just make this one public too.

Hi @bohuang3d ,@liaxim , thanks for your replies.
Using your "GVRResourseVolume" is not working, I have errors "Cannot set the stream of an open resource" and "Input stream doesn't support mark" so finally I've used reflection as described bellow :
// get private GVRAndroidResource(String,InputStream) reference from reflection
Class[] args = new Class[2];
args[0] = String.class;
args[1] = InputStream.class;
Constructor<GVRAndroidResource> constructor = GVRAndroidResource.class.getDeclaredConstructor(args);
                constructor.setAccessible(true);

// then later create any GVRAndroidResource with inputstream as you want
GVRAndroidResource res = constructor.newInstance("my 360 3D 4k photo!", myInputStream);

Regards

@jang7875 Yeah. Forgot you have to also wrap your InputStream in something like the following. Then pass that. Much better to use reflection though.
    static class BufferedInputStream extends InputStream {
        private BufferedInputStream in;
        private InputStream realStream;

        public BufferedInputStream(InputStream realStream) throws IOException {
            this.realStream = realStream;
            in = new BufferedInputStream(realStream);
        }

        public boolean markSupported() {
            return true;
        }

        public void reset() throws IOException {
            this.in = new BufferedInputStream(realStream);
        }

        public int read() throws IOException {
            return in.read();
        }

        public int read(byte[] buffer, int byteOffset, int byteCount) throws IOException {
            return in.read(buffer, byteOffset, byteCount);
        }
    }


Add setStorage support

Functional changes:

added function withSetStorage which wraps the model into (model, setStorage model)
run this in every update case where we change model

Cool, thank you! :D


Make user and extra context merge order match tags

Followup to #322.
Tags passed when capturing an event override tags in the request context. For consistency, user and extra context should behave the same way.

Thanks!


IGNITE-15759 Remove LOCAL caches

Thank you for submitting the pull request to the Apache Ignite.
In order to streamline the review of the contribution
we ask you to ensure the following steps have been taken:
The Contribution Checklist

[ ] There is a single JIRA ticket related to the pull request.
[ ] The web-link to the pull request is attached to the JIRA ticket.
[ ] The JIRA ticket has the Patch Available state.
[ ] The pull request body describes changes that have been made.
The description explains WHAT and WHY was made instead of HOW.
[ ] The pull request title is treated as the final commit message.
The following pattern must be used: IGNITE-XXXX Change summary where XXXX - number of JIRA issue.
[ ] A reviewer has been mentioned through the JIRA comments
(see the Maintainers list)
[ ] The pull request has been checked by the Teamcity Bot and
the green visa attached to the JIRA ticket (see TC.Bot: Check PR)

Notes

How to Contribute
Coding abbreviation rules
Coding Guidelines
Apache Ignite Teamcity Bot

If you need any help, please email<EMAIL_ADDRESS>or ask anу advice on http://asf.slack.com #ignite channel.

whats the purpose for : org.apache.ignite.internal.client.GridClientCacheMode#LOCAL ?

@zstan @anton-vinogradov @isapego @ptupitsyn @NSAmelchev
Folks, I've fixed all your comments. Please, take a final look.


Suggestion

The brunysixl discord also has a link generator so I was wondering if u could add it to the "what happened to the proxies" thing
link:https://dsc.gg/bruny
It generates bruny websites which have built in proxies

ok


Add GenieJS example

What:
Add GenieJS example.

Why:
Increase number of examples that people can see and play around with.

How:
Codesandbox link

Checklist:

[x] Documentation
[ ] Tests
[x] Ready to be merged
[x] Added myself to contributors table

Codecov Report

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

@@           Coverage Diff           @@
##           master      #96   +/-   ##
=======================================
  Coverage   15.93%   15.93%
=======================================
  Files           3        3
  Lines         251      251
  Branches       58       58
=======================================
  Hits           40       40
  Misses        158      158
  Partials       53       53

Continue to review full report at Codecov.

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

Hi @zsid :wave:
I took the time to make my own example integrating this library with geniejs, but I didn't do too much to style it very well. If you want to take my example and add some nice styling (and maybe even the functionality to make it appear and disappear, as well as other "wishes" and even deeper integration with context and things like that, that would be awesome.
I'll go ahead and close this issue for now though. Thanks again!

Looks really good @kentcdodds! I have been busy at work in the last few days and didn't have time to look at it. I am glad you have picked it up as I had a different implementation in my mind and yours is very elegant and easy to follow!
I will try and create different examples as I have a few ones from projects I have been involved in. I would really love to bring downshift to my current project but we are using typescript with React there. I saw there was a PR to add types. I hope you can approve it soon so that I can show how good downshift is at my project 😄


IPDB test net usable by any 3rd parties

(WIP)
Abstract
foo
Motivation
foo
Goals

foo goal 1
foo goal 2

Challenges
Foo

foo challenge 1?
foo challenge 2?

Architecture
foo
Examples
foo
Tasks
Friday, foo date 1:

[ ] foo task 1
[ ] foo task 2

Friday, foo date 2:

[ ] foo task 3
[ ] foo task 4

Copied to https://github.com/bigchaindb/org/issues/3 so closing this issue here.


Can't fully create a new API under an existing group

Files seem to be getting generated (including a controller though it's explicitly disabled in the command below), but some functions are missing. The error is the following:
% operator-sdk create api --group ocs --version v1 --kind someKind --controller false
FATA[0003] failed to create API: unable to run post-scaffold tasks of "go.kubebuilder.io/v2": exit status 1

A solution is suggested in #2191

Hi Racheli, The fix is merged you create a scaffolding now.


javascript 30%-50% slower Chromium vs Chrome

Hello,
This is not specific to ungoogled-chromium but does affect it like all other Chromium builds i've tried. For some reason javascript runs 30-50% slower than official Chrome builds of the same version. I find this difficult to understand since everything (including V8) is compiled from 95% the same sources, right? Are there some V8 optimizations or GC aggressiveness flags that are set differently in all Chromium builds?
😕

Have you tried builds from Woolyss? They customize their build flags beyond what most Chromium projects do, like Whole Program Optimization.
If those builds have comparable performace to that of Chrome, then we could try implementing their changes in our builds.

Ok, i tried to carefully reproduce the exact thing in all versions using this bench [1]. Turns out this difference is due to x32 (mean settles @1.00ms) vs x64 (mean settles at 1.50ms).
Quite the per hit for x64. Sorry for the noise.
[1] https://rawgit.com/leeoniya/domvm/2.x-dev/demos/bench/dbmonster/index.html

Hm, so there is quite a bit of a performance benefit using x32. Interesting. Thanks for the info.

i re-tested on my Thinkpad T440S Win10 x64. the x64 overhead is ~8% in all builds. think it depends on what V8's JIT can cobble together for any given CPU's features. i've seen exactly the same code perform differently on different CPUs and platforms using the same V8 version.

It is also possible that some fieldtrial testing code is active in one built but not in the other, which would explain the performance differences.
You can see the active fieldtrial modifications in chrome://version as Variations.
More info on this: http://superuser.com/questions/541466/what-is-the-variations-section-in-the-output-of-aboutversion-or-chrome-v

@gcarq I've set fieldtrial_testing_like_official_build=true, which I think disables all of the fieldtrial testing.


polymer init still not working

Windows 10 Home 64bit,
polymer --version
1.9.2
npm --version
6.4.1
git --version
git version 2.19.1.windows.1

Init script stuck at
polymer init
? Which starter template would you like to use? polymer-3-application
info:   Running template polymer-3-application...

There is gitignore (without dot at the begin) at \node_modules\polymer-cli\templates(application|elements)\polymer-3.x. I also tried to add .gitignore and .npmignore

On Windows 10, Polymer version 1.9.7 still hangs when running the polymer init command and making a selection.
Can we get an update this?

Polymer CLI hangs and doesn't do anything on windows. Is there any fix?

It works; Instead of going through polymer cli menu if you directly use polymer init polymer-3-starter-kit in terminal,

polymer init still doesn't work for me on Windows, even when providing a full command like polymer init polymer-3-element. However, polymer init polymer-3-starter-kit was fine. My environment is as follow:

Windows version: Version 1803 (Build 17134.765)
Npm: 6.4.1
Polymer: 1.9.0-pre.2

Do we have an update? Or what can be missing?

Looks like I can get it to work on Windows 10 by using the Git Bash shell, but still not working from a native Windows Command Prompt.

npm init still not working for

Windows version: Version 1903 (Build 18362.535)
Node v12.13.1
Npm 6.12.1

Polymer CLI is deprecated and it will not be further developed. There was an article about this on March-2019. Polymer team has developers to move something else like Webpack CLI. But there is no guide on how to start a project.
From my personal experience, I would recommend you to use the following command instead.
vue create hello-world

Polymer CLI is deprecated and its not further developed. There was an article about this on March-2019. Polymer team has developers to move something else like Webpack CLI. But there is no guide on how to start a project.
From my personal experience, I would recommend you to use the following command instead.
vue create hello-world

For newer projects open-wc is recommended by the community that provides both webpack and rollup support

Good day 🐱👤,
Recently I had this issue and my way to solve it was the following. Take into account this "error" is because the node version is not supported for Polymer:

Install NVM to manage your node version. If you want you could use this guide https://linuxhint.com/downgrade-node-version-windows/
Change your node version to 14 or similar with NVM.
Implement polymer init 😊.
Enjoy your Polymer coding!!

Installling node 14 worked.
Thanks mate @WilliamCastr0


fix(routes): graphiql redirection if a prefix have been set

The redirection is incorrect if the plugin is registered with a prefix.
This PR fixes the issue ;)

Can you add a unit test?

app.register(GQL, {
    prefix: '/test'
  })

This example is not managed, maybe we should handle it ?

This example does not work, maybe we should handle it ?

What do you mean?

app.register(async function (app, opts) {
    app.register(GQL, {})
  }, { prefix: '/test' })

In order to add a prefix to fastify-gql you have to wrap the module like the example above. But maybe this is an intended behavior.

It depends if a plugin is encapsulated or not. Likely fastify-swagger  should support prefix, maybe you can open a bug there.

Edit: We could handle prefix in GQL options by adding prefix: opts.prefix here

Definitely! Would you like to amend this PR or send a new one?

It's WIP, I'm adding tests for prefixed /graphql

We should be good now, prefix is handled both from a wrapper and from opts, tested and documented ;).


EncryptedDirectMessage class; simplify Event

This one probably should have been separated out into 3-4 small PRs. I can pull these apart if need be.
EncryptedDirectMessage class
Derived from the parent Event class to provide a more human-friendly way to construct a DM:
from nostr.event import EncryptedDirectMessage

dm = EncryptedDirectMessage(
  recipient_pubkey=recipient_pubkey,
  cleartext_content="Secret message!"
)
private_key.sign_event(dm)
relay_manager.publish_event(dm)

The recipient's 'p' reference tag is automatically inserted for us.
'cleartext_contentprovides a clean way to differentiate the cleartext content from the ultimate encryptedcontent`.
Trying to instantiate a DM by specifying content throws an exception.
PrivateKey.sign() encrypts the message for us and populates the content field.
Explicit PrivateKey.encrypt_dm() convenience method added while preserving the more general encrypt_message() in case that finds other uses(?).
The DMs are still Event types and so can be passed straight on to the RelayManager.
Relevant tests added.
Example added to README.

Event.__init__ boilerplate removed via dataclass
Boilerplate bad. DRY good.
Simplified Event instantiation
We're going to sign an Event with our PrivateKey anyway so there's no need to populate the Event.public_key field on instantiation.
event = Event("Hello Nostr")
private_key.sign_event(event)
relay_manager.publish_event(event)

This same simplification is also demonstrated in the EncryptedDirectMessage example above.
README examples updated accordingly.
Event explicit 'e' and 'p' tag support
Similar to the discussion in #12:

Add explicit add_event_ref(event_id) to generate 'e' tags
Add explicit add_pubkey_ref(pubkey) to generate 'p' tags
README example added for replying to a note.

ACK
Type annotations good

@jeffthibault great notes! All my updates are now committed.

Thanks @kdmukai. This looks great!
Tested ACK.


Adding multi-modal endpoints with API key security checks. refactorin…

…g config variables.

Requires these environment variables:
DBUSER=
DBPASS=
DBHOST=
DBNAME=omega

API_KEYS=[]

I checked in validator-api/_generate_api_key.py as a tool that can generate API keys to be put in the API_KEYS environment list variable.


pb2nc prepbufr data dir exception

Bug:
Exception occurs when the defined PREPBUFR_DATA_DIR is the parent directory of the prepbufr data.
Currently, the path must be one level above the parent directory of the prepbufr data.
Add support so PREPBUFR_DATA_DIR can be defined as the parent directory of the data.
Explained:
ie. lets say this is the structure of our prepbufr path and data files on disk.
.../blahdir1/blahdir2/blahdir3/<prepbufr_data files>
IF ... in the conf you define either of the following ...
that is, the last path element to be the parent of the data files,
you get an exception ...
PREPBUFR_DATA_DIR = .../blahdir1/blahdir2/blahdir3
PREPBUFR_MODEL_DIR_NAME =
OR ...
PREPBUFR_DATA_DIR = .../blahdir1/blahdir2
PREPBUFR_MODEL_DIR_NAME = blahdir3
The Exception....
CRITICAL: master_metplus  failed:
...
File "/mnt/lfs1/projects/dtc-hurr/James.T.Frimel/metplus/metplus.hrrre/ush/pb2nc_wrapper.py", line 522, in retrieve_pb_time_info
if len(date) == 8:
TypeError: object of type 'NoneType' has no len()

Note: in the future, PREPBUFR_MODEL_DIR_NAME may be deprecated.

This issue should be resolved with the pb2nc refactor.


setTimeout after rendererWindow.close() fails to fire

It seems that if I call setTimeout after closing a renderer window, the setTimeout never fires.
To reproduce: grab hello-atom from https://github.com/dougnukem/hello-atom and add a few lines at the end of the app.on('ready', function() { callback in main.js so it looks like this:
 app.on('ready', function() {
  // Create the browser window.
  mainWindow = new BrowserWindow({width: 800, height: 600});

  // and load the index.html of the app.
  mainWindow.loadUrl('file://' + __dirname + '/index.html');

  // Emitted when the window is closed.
  mainWindow.on('closed', function() {
    // Dereference the window object, usually you would store windows
    // in an array if your app supports multi windows, this is the time
    // when you should delete the corresponding element.
    mainWindow = null;
  });

/////add these lines:
  setTimeout( function() {
        console.log( 'about to close window' );

        mainWindow.close();

        setTimeout( function() {
              console.log( 'after window close' );
        }, 3000 );
  }, 3000 );
////end added lines

});

The first setTimeout is just there to trigger the problematic code (on my project it's triggered by a menu).
On atom-shell 0.22.3 in windows 7 the "after window close" never gets logged (and in my experience with my project nothing in the setTimeout callback ever gets executed).
If you comment out mainWindow.close(); then "after window close" appears as expected in the console.

Hi teleclimber,
I think the problem you're running into is that the window-all-closed event handler (here: https://github.com/dougnukem/hello-atom/blob/master/hello-app/main.js#L12) is firing and quitting the application immediately after the last window is closed. You should be able to keep your application running on Windows/Linux by disabling that event handler.

Ooohh. Of course. I guess I never realized it quit because atom.exe keeps running after the application quits.
Thanks!


Test on Python 3.5 in addition to 3.6 and 3.7

Do we still work with Python 3.5?

The tests failed, so making the requirement formal in #641


Alternative naming of HistCounts variants. Additional documentation.

Not the most beautiful names, but they're clear 😀.

I'm thinking more like docs to our future selves on the weirdness involved here. Not public.


publish library on pub.dev

done. Added GitHub action to publish automatically as well. Just create a tag with the version number and v as a prefix (for example v1.1.9). The versions in pubspec.yaml and the tag have to match


Exception during interpolate when using nested vars

What do I have:
test.yml
a: ((cf.i_am_ok))
b: ((cf.private))

var.yml
cf:
  i_am_ok: some_value

When i run: bosh-cli int test.yml --vars-file var.yml I got:
Finding variable 'cf.private':
  Expected to find a map key 'private' for path '/private' (found map keys: 'i_am_ok')

Exit code 1
exit status 1

What I expect to get:
a: some_value
b: ((cf.private))

@alex-slynko I would prefer not to have nested vars at all, since there are systems that does not support them. For instance, right now we use Vault to store all vars and we fetch vars easily with the command vault read --format=yaml secret/some-path > vars.yml, in this case vars.yml is not able to have nested vars  at all. That's why I do not support nested vars in manifests right now. (this will clearly change after switching to config server).
I also not sure how you are going to reference nested vars in variables section.
But you are right, since BOSH allows to have nested vars, it should not fail in this case.

@alex-slynko thx for the repro, will take a look.

Closing this due to inactivity. At the moment fixing this class of errors is not an immediate priority, but please feel free to reopen if you are still running into this issue.


if a disabled input is re-enabled, cascade the updated flag on across…

… the whole input chain to ensure that any and all updates will be pushed through the chain
bug reproduction:
http://vizor.io/reiska/revolver-inactive-chain-input-bug-reproduction

lgtm


Adds progress and completion handlers. Support for adding folders and subfolders to ZIP packages.

Thanks @boherna for your contributions.


Cannot push data to individual time series dataset

chart.js Version 2.7.3
chart.js zoom plugin
I have a chart with 4 time-series line plot and would like to do the following:

Receive an array of 4 elements which correspond to a new datapoint for a plot respectively.
Push the data to the dataset and update graph periodically.

Issue:
I am unable to push data to individual dataset following the documentation.
function getSolarOutputData(){     xdata=[36.36, 64.44, 92.43, 12.55];     let tick =moment();   window.g1.data.datasets.forEach((dataset,index) => {     dataset.data.push({x:tick,y:parseFloat(xdata[index])});       updateGraph(window.g1);   }); };;
Each and every element is written to every dataset, resulting in all dataset to contain the same data.
It appears that every dataset addresses the same array.
 window.g1.data.datasets[0].data.push({t:moment(),y:parseFloat(data.data)});
above line causes "data.data" to be pushed into all datasets.
Is there a work around?

Can you please provide a jsfiddle that reproduces your issue?

It appears that using empty array in config to initialize datasets resolves the issue.


UI enhancement of footer

There is a UI bug in the web page the download buttons collide with the headings of the the footer. It makes it inconvenient for the user to read. It will give bad user experience as it will be difficult to navigate through.
The download links need to be separated from  the footer links.
@khushi-joshi-05 I have experience of fixing such issues. I can solve this bug to enhance the UI of the website. I am willing to contribute to this project. Kindly assign me this project under gssoc'24

I would like to work on this issue, Please assign it to me under GSSOC 2024

@GarimaSingh0109 Ihave assigned u this issue

@GarimaSingh0109 Ihave assigned u this issue

@khushi-joshi-05 I raised this issue three days ago, #403  yet it has not been assigned to me. Instead, it has been assigned to someone else who raised the issue today. Given that we operate on a first-come, first-serve basis, this situation seems unfair.
I would appreciate it if you could review this matter and consider reassigning the issue to me, as I have been awaiting its allocation since I first raised it.


TN-2193 indefinite QB in adherence report

Add "none of the above" Questionnaire Bank, and extended classification enum with "other".
TN-2303 Prevent repeated QNR->QB lookup with marker for n/a.
TN-2304 Add indefinite to adherence report

Hello @pbugni! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file portal/migrations/versions/da6506c5899a_.py:

Line 16:1: E402 module level import not at top of file
Line 33:80: E501 line too long (80 > 79 characters)
Line 62:80: E501 line too long (80 > 79 characters)

Looks good!
Question- why not change the QB lookup code to tolerate null QBs when attempting to find the QB for a QuestionnaireResponse?(instead of assigning QNRs to a "special" QB)

The lookup code does tolerate null QBs - but we need a clear way to distinguish those QNRs that have yet to be sync'd with the appropriate QBs, and those that just don't have one.
Anytime we want to force a (re)evaluation of which QB a given QNR should be attached to, we set QNR->qb_id = None and then the next cache / lookup will evaluate.  Post of a new QNR, or change of consent or new procedure or ... all trigger such invalidation.


Server Capability get by name - returns blank response array when the requested item doesn't exist

I'm submitting a ...

[X] bug report
[ ] new feature / enhancement request
[ ] improvement request (usability, performance, tech debt, etc.)
[ ] other

Traffic Control components affected ...

[ ] CDN in a Box
[ ] Documentation
[ ] Grove
[ ] Traffic Control Client
[ ] Traffic Monitor
[X] Traffic Ops
[ ] Traffic Ops ORT
[ ] Traffic Portal
[ ] Traffic Router
[ ] Traffic Stats
[ ] Traffic Vault
[ ] unknown

Current behavior:

response code: 200
returns
{
    "response": []
}

Expected / new behavior:

response code 404
should return alert
Text: "no server capability with that key found"
							Level: "error"

Minimal reproduction of the problem with instructions:

When GET https://{{TO_BASE_URL}}/api/{{api_version}}/server_capabilities?name=DOESNT_EXIST
where DOESNT_EXIST - is  sever capability which is not in the list.
Anything else:

Also the return struct is an array of response while we expect only one to be returned.

This is a matter of opinion. The URI in question does exist, it's just that a list of all the server capabilities filtered by a non-existent name is an empty list.
If a change was made to consider identifying fields special cases of filters that would cause a 404 error to be returned, it would need to be made globally, as this is the behavior of many (all?) endpoints.
I think that'd be a fine change to make - so that it would match the behavior of other request methods - but I also don't think that what we're currently doing is extremely wrong.

I don't think this is a bug, it's a fundamental change to the API.
For a 404 vs an empty array for objects that don't exist, I don't think either is the "right" solution. A 404 is probably more expected. But to say "the API returns an array of objects, and if your filter didn't have any, you get a successful empty array" is not a violation of HTTP.
Moreover, the TO API has returned 200 [] for ages. It isn't documented behavior, but it's most certainly a specific behavior, that clients almost certainly rely on in places. Even if it doesn't violate the SemVer API Promise, we can't reasonably change this without breaking tons of people.
Likewise, IMO we should continue to make new endpoints follow the same pattern as everything else.
It wouldn't be a bad idea to do this in the next major version of the API (2.0), but we simply can't safely do it to 1.x.

For what it's worth, to me this makes sense:
GET /foos/{id-does-not-exist} returns a 404 as the URI does NOT exist
and
GET /foos?id=id-does-not-exist returns an empty array as i see the query parameter as a filter of a perfectly valid URI as @ocket8888 mentioned.

IMO this should simply be closed as this is expected behavior.

It's not a bug but it can stay open as an enhancement request


Use of undeclared type 'JTAppleCell' Cocoa pod 7.1.5

IMPORTANT - BEFORE CREATING YOUR ISSUE PLEASE READ THE FOLLOWING:

PLEASE STATE THE VERSION NUMBER you are using.
Are you using the latest version of JTAppleCalendar? Latest version is currently 7.1.6
PROVIDE ENOUGH INPORMATION SO I CAN RECREATE THE PROBLEM.

import JTAppleCalendar
Above line cell file fixed my issue


Stack deploy errors cut off, disappear quickly

Bug description
When deploying a stack via the Portainer web UI and there's an error, there's a z-index overlapping issue with the editor scrollbar covering part of the alert:

Also the error toast alerts disappear quickly, so much so that the stack has to be deployed again just to read / capture the error in cases.
Expected behavior

Other content on the page like the stack editor doesn't overlap the toast alerts
Error alerts stay up longer and/or the error can be viewed on the page after

Portainer Logs
Logs are not applicable here - this is more client web UI / UX.
Steps to reproduce the issue:

Match environment info under Technical details
Deploy a stack within Portainer where it will generate an error

Technical details:

Portainer version: 2.0
Docker version (managed by Portainer): Docker version 19.03.12, build 48a66213fe
Platform (windows/linux): Portainer running in Linux ubuntu but client browser on Mac
Command used to start Portainer (docker run -p 9000:9000 portainer/portainer): Swarm startup similar to portainer.io
Browser: Microsoft Edge (Chromium) Version 85.0.564.44 (Official build) (64-bit) on Mac

Additional context
Add any other context about the problem here.

Thanks for reporting this, I'll close at it is already reported here #3545


Added Haar random function

I find myself writing these lines a lot in some demos I'm making so might as well make it a function.

I don't know. I switched to double precision. However, I don't want to use that norm function because its not very explicit and I'd prefer to use something that I can easily tell what it is doing. "norm" is ambiguous and can mean many things. So I think my way is more readable.

The word "norm" is ambiguous but numpy.linalg.norm is unambiguous. Anyway, I'll approve.

It is totally ambiguous. I'd need to look up numpy.linalg.norm to know what it does!


[FLINK-19655][flink-table-runtime-blink]  add super.open() and write unit test for temporal process join

What is the purpose of the change
The ultimate goal of this PR is to fix a bug where NPE is thrown when using blink planner and TemporalTableFunction after setting IdleStateRetentionTime .TemporalProcessTimeJoinOperator should initialize timer service for processing time.
See #FLINK-19655
Brief change log

Add super.open() in LegacyTemporalProcessTimeJoinOperator#open() to solve NPE
Write two unit test for LegacyTemporalProcessTimeJoinOperator to verify the code worked fine

Verifying this change
This change added tests and can be verified as follows:

Added test that validates that:

1.test proctime temporal join when non state clear.
2.test proctime temporal join when set state clear.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving):  no
The serializers:   no
The runtime per-record code paths (performance sensitive):  no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper:  no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature?  no
If yes, how is the feature documented?  not documented

@seunjjs Thanks for the update, the code looks fine, but the code check style is a little missy maybe you're using windows system.
I add a commit to help format that, you can cherry-pick this https://github.com/leonardBang/flink/commit/cd333bf537031e7ff5049b3a59da38d7859f416b directly.

Thank you. My system is win10, and I have followed this guide https://ci.apache.org/projects/flink/flink-docs-release-1.11/flinkDev/ide_setup.html#checkstyle-for-java to set Checkstyle-IDEA plugin. But if I run ctrl+alt+l, then the code style is different from yours. Do you know how can I get the right checkstyle plugin config on windows?😥

Thank you. My system is win10, and I have followed this guide https://ci.apache.org/projects/flink/flink-docs-release-1.11/flinkDev/ide_setup.html#checkstyle-for-java to set Checkstyle-IDEA plugin. But if I run ctrl+alt+l, then the code style is different from yours. Do you know how can I get the right checkstyle plugin config on windows?😥

I also followed this guide, I'm using mac, I think you've set right check style, the only difference between us is using single tab or double tabs before a new line, both the two formats exits in Flink codebase and can pass the code style check well.
I also improve a little about the note, parameter name and redundant blank lines, that's why I add the format commit.


Show the system version on the Bootstrap screen

It's needed for the right version of node to be installed.
Didn't have time to dive into directly comparing versions, but this is better than talking about a niche DA concept with no reference to the docs.

Does this do something special compared to get_config() of another arbitrary configuration option, or does upgrading the container always bump this config setting?

Yeah, it should  It's the same way that docassemble shows you the system version on the package management screen. I hadn't looked internally to see how it works but I can.
No rush. Happy holidays!

Just to document a bit more how this works, get_config() is a wrapper over the internal daconfig dictionary. When loading, daconfig reads a version file that is installed at /usr/share/docassemble/webapp and sets system version to it's value.


[#200] Add newsletter popup

I added simple newsletter popup with validation.
After successful sign up the newsletter bar in footer will disappear.
#200 #318

I corrected some styles to follow css rules. @filrak plase take a look at these changes

Now it looks great, thank you! ;)


Update User Guide

Updates:

Add to /order add example
Add to quick start
Add /home command plus explanation on dashboard
Update UI picture

Pull Request Test Coverage Report for Build 284

13 of 14 (92.86%)  changed or added relevant lines in 3 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.03%) to 87.637%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/main/java/seedu/address/ui/Display.java
4
5
80.0%

Files with Coverage Reduction
New Missed Lines
%

src/main/java/seedu/address/ui/Display.java
1
87.41%

Totals

Change from base Build 280:
0.03%

Covered Lines:
2800

Relevant Lines:
3195

💛  - Coveralls


Proposal: Disable dynamic function dispatching for native or arch-specific builds

My vision of PR #1626 (@nmoinvaz)
Based on my PR #1643
For X86, PPC, RISC-V, s390 the conditions are correct.
For ARM, they require verification.
Microsoft Visual Studio does not support compile time detection of CPU features for /arch before AVX. Missing predefined macros (__SSE*__).

It will be good to see what it looks after PR #1643 and #1644 lands.

How do we address any of the comments by @Dead2 here https://github.com/zlib-ng/zlib-ng/pull/1626#issuecomment-1875230082?

@nmoinvaz, @Dead2
I agree with @Dead2 (https://github.com/zlib-ng/zlib-ng/pull/1626#issuecomment-1875230082) if we use -march=*** and dynamic dispatching.
If we use static/compile-time dispatch (this PR), then functions for this architecture will be selected but on compile time. This PR essentially disables newer and older instruction sets.
I'm thinking of adding restrictions on the compilation of those functions that will not be called during static dispatch.

Rebased onto develop.

Conditions have been updated for PPC (without changes), RISC-V, s390.
Verification of the conditions for ARM is required.

@nmoinvaz, please remove the "PR Dependency" label.

Rebased.

All compile time CPU features checks is implemented.
I think PR #1659 is more promising.


Not returning partial results

Expected Behavior
When I have a model like this:
// pseudocode:
#[Object]
impl Query {
    async fn foo(&self) -> Result<String> {
        Ok("Hello from foo()".into())
    }

    async fn bar(&self) -> Result<String> {
        Err("Some error".into())
    }
}

And I make a GraphQL query like this:
query {
    foo
    bar
}

I'd expect to get the value of foo even though bar returned an error.
Actual Behavior
Instead, I see "data": null, (as well as the expected "errors" array).
Steps to Reproduce the Problem

Implement the #[Object] above
Query a field that returns an Ok and one that returns an Err
Observe that the Ok value is not available in the output.

Specifications

Version: 6.0.11
Platform:  macOS
Subsystem:  ???

Well, today I learned that the "bug" behavior I'm seeing is actually defined in the GraphQL specification:

If an error was raised during the execution that prevented a valid response, the data entry in the response should be null.

And ascync_graphql is advertised as:

a high-performance graphql server library that's fully specification compliant

So, I suppose this is not a bug. It just differs than multiple GraphQL server implementations I've used before, so was surprising.

Oops, nevermind, the spec goes on to say:

If a field error is raised, execution attempts to continue and a partial result is produced (see Handling Field Errors). The data entry in the response must be present. The errors entry should include all raised field errors.

and:

If a field error is raised while resolving a field, it is handled as though the field returned null, and the error must be added to the "errors" list in the response.

So, async_graphql is, AFAICT, not currently "fully specification compliant".

If you wrap your resolver response in Option I believe this works.
async fn bar(&self) -> Option<Result<String>> {
    Some(Err("Some error".into()))
}

At least it has been working for my project.

I tried that as well, and got the same error.  Maybe I'm on an older version.  I'll update and try again.

As @ashmarchington said, if you want to have a null here it would means that the type of foo and the type of bar should be Nullable, but in async-graphql to have a Nullable type, you must return Option<T>.
When you return Result<String> it means that the inferred type should be a Non-Nullable<String> for GraphQL, but it may fail and if it fails, it'll bubble up the error.
So here, if you change it by Option<Result<String>> it will work.

Came back to test this.  Created a repository where I can reproduce this issue:
https://github.com/NfNitLoop/async-graphql-bug-1449/commit/5563f45e2ebed13b44d04bba6803a380796baafc
But then I re-read @Miaxos's comment, suggesting Option<Result<T>> and I tried that.  Sure enough, it works:
https://github.com/NfNitLoop/async-graphql-bug-1449/commit/e1531a52f1bf2c959e5fbcd3f1e5701c1672ac1f#diff-42cb6807ad74b3e201c5a7ca98b911c5fa08380e942be6e4ac5807f8377f87fcR20-R23
But -- why would that work and not Result<Option<T>>? That seems to be the more "correct" type in my mind. It also allows you to use ? for error handling, where Option<Result<T>> does not.

If you were to use Result<Option<T>> aren't you inherently stating that the resolver should not fail and if it does then there is an error with the request?
All the Option<T> in this instance is stating is that the Ok result may be None but the result should still be Ok for the request to be valid and an Err result is invalid.
With Option<Result<T>> you are stating that the resolver may return Some or None and the Some case may contain an Err but this is still valid for the request as the result of the resolver itself can be None as opposed to Ok in your first version.
I may have the wrong understanding here but that is what seems to make sense to me.


[nodemon] app crashed

When running gatsby develop I'm getting:
success run page queries - 0.029s - 2/2 69.82/s
[nodemon] app crashed - waiting for file changes before starting...

You can now view fullstack_app in the browser.

  http://localhost:8000/

View GraphiQL, an in-browser IDE, to explore your site's data and schema

  http://localhost:8000/___graphql

Note that the development build is not optimized.
To create a production build, use gatsby build

success Building development bundle - 2.814s

App is working otherwise. Getting the same message when making changes to the server and have to manually restart it. Any thoughts? Thanks!

What version of Gatsby are you using, and what operating system do you have?

Thanks for getting back to me and sorry if this is not of your concerns.
OS: Ubuntu 18.04.5 LTS
"gatsby": "2.24.52"

Well, I've quickly bootstrapped a new project and I can't see that error. It looks like it comes from an another library or something... Be sure to place the server in the server/index.js :)

ok, thanks anyway!


Resume training

Hi and thanks for the nice project!
I am looking for a way to resume the training, because my machine got crashed and I had to restart it.
Is it possible to resume the training?
Best regards,
Krasimir

@bagaturchess if you have a saved checkpoint, you can do as follows
let schedule = TrainingSchedule {
    ...
    start_superbatch: resume_from_here_superbatch
    ...
}

trainer.load_from_checkpoint("path/to/checkpoint");

trainer.run(&schedule, &settings);

Thanks a lot for the answer!
The solution provided looks working.
Best regards,
Krasimir


Design System: DropDown a11y updates

Summary
The DropDown base PR was getting unruly so I decided to separate out the updates to accessibility to make them easier to focus on. The DropDown component in the design system aggregates a lot of the functionality (and aria) or the other custom select menus that already exist in the editor and dashboard. This PR just tests things and fixes some of the redundancies.
We have passing AXE and lighthouse scores (the only thing either of them complained about was color contrast for the dark theme hint text, which I documented here in my list of questions for the design team after the holidays).
Relevant Technical Choices

Remove redundant aria-roles on expanded "menu". This was a little tricky because sometimes we have grouped lists that create all of the options.  I see these as their own lists, all of which are under the button's given labeling which is passed to the menu's container. A watered down example would be a dropdown labeled "Pick a favorite creature", where we have a list of options grouped by type of creature. So we'd have lists of mammals, reptiles, and insects. I also did it this way because it gives our html scaffolding a less complex structure.
Added a memoized uuid based id to the select button to tell the menu that the select button can act as a label if needed.
Sets aria up in dropDown/index and passes to select button as ...rest because we need the same ariaLabel vs dropDownLabel for the menu, so keeping that set up together.

To-do

User-facing changes

None, storybook only

Testing Instructions
Run storybook locally and try using a screen reader.

Do you get appropriate context to use the menu?
You can also try setting up an 'ariaLabel' in the storybook knobs and see that that a value is spoken as opposed to the dropDownLabel (that's printed on the screen) when set.

Addresses #4950

Noticed a separate bug unrelated to this change that there's an off by one error for the screen reader. If you're at the last item in a list it says "item 11 of 12" or whatever the total num items are.

thank you! I know what this is, i'll do a separate pr today.

Noticed a separate bug unrelated to this change that there's an off by one error for the screen reader. If you're at the last item in a list it says "item 11 of 12" or whatever the total num items are.

thank you! I know what this is, i'll do a separate pr today.

just kidding, fixed it here (needed to update tests that I forgot about). using the optionInset to set aria-posinet and it's an index of 0 starting, so just need to add 1 to the aria value.


Note on double encoding of Storage Key

in K8S Secrets..
Added If you place the storageAccessKey parameter (under the spec/metadata path) into the secret store, it needs to be Base64 encoded AGAIN in the yaml manifest file, (even though it is already in a base64 encoded format).
Thank you for helping make the Dapr documentation better!
If you are a new contributor, please see the this contribution guidance which helps keep the Dapr documentation readable, consistent and useful for Dapr users.
If you are contributing a new authored "How To" article, this template was created to assist you.
In addition, please fill out the following to help reviewers understand this pull request:
Description
Please explain the changes you've made
Issue reference
Please reference the issue this PR will close: #[issue number]

Thanks for opening the issue @WillEastbury.
I tried reproducing this and couldn't.
I am closing for now, feel free to reopen if you can provide exact reproduction steps.


Comment referencing search shows not found when it is

Also reported by @obigriffith

@susannasiebert had actually already fixed it and we just rolled it out to prod now. It should be working now!


[Do not merge] Add two factor authentication for Super Admin users

https://trello.com/c/s3l4mIVJ/30-enable-2fa-for-govwifi-admin-login-at-least-for-super-admins
Enforces 2FA setup and authentication for super admin users.
The setup step is activated once for super admin users as they log in.

They are then asked to authenticate via TOTP on every login (this is configurable).

Do not merge until https://github.com/alphagov/govwifi-build/pull/367 and https://github.com/alphagov/govwifi-terraform/pull/269 have been applied.

LGTM but I've observed that after entering the correct TOTP, I can still go back to http://localhost:8080/users/two_factor_authentication route for entering TOTP again. This doesn't seem right as I'm already logged in. Ideally, I would expect it to redirect me to the dashboard.

OK thanks @robinmitra that's a really helpful review, a few things to fix up here.

@robinmitra having looked at the two_factor_authentication gem I can see nothing would prevent a user explicitly navigating to the 2FA page and re-entering a TOTP code after setup or auth. We would need to patch gem functionality to avoid this. My feeling is that if someone wants to manually navigate to a 2FA page after authenticating and re-authenticate, we shouldn't worry too much, I'm not sure it's worth the additional effort in preventing this.


generate statistics/ plots on AU status

@ValentineHerr , would it be easy to replicate the files here for the AU status? If so, could you please do so?
This will be helpful for @irdangler (SMSC student).

Will try this now

@teixeirak, I added the tables and figures.
I understood to do the same calculations as for mortality (status A or AU becomes D) but looking at trees that were A and became AU at next census.
(also, I added a plot looking at mort rates and rates of A to AU for each size class, across years and species).

Thanks, this is great!
Although, could we also do this as just the fraction of living trees that are unhealthy (AU/(A+AU))?
The one you calculated is very valuable, but I think is rather hard to interpret given that definitions of AU were pretty subjective up until 2021.


[redbull.com] Not working  Failed to parse JSON

Checklist

[x] I'm reporting a broken site support
[x] I've verified that I'm running youtube-dl version 2021.04.07
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar issues including closed ones

Verbose log

youtube-dl -v -F https://www.redbull.com/gb-en/films/kriss-kyle-out-of-season
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'-F', u'https://www.redbull.com/gb-en/films/kriss-kyle-out-of-season']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2021.04.07
[debug] Python version 2.7.16 (CPython) - Darwin-20.3.0-x86_64-i386-64bit
[debug] exe versions: none
[debug] Proxy map: {}
[RedBull] kriss-kyle-out-of-season: Downloading JSON metadata
[RedBullEmbed] rrn:content:films:6d72aae4-d724-40ff-b627-912a2be1fb7d:en-GB: Downloading JSON metadata
ERROR: rrn:content:films:6d72aae4-d724-40ff-b627-912a2be1fb7d:en-GB: Failed to parse JSON  (caused by ValueError('No JSON object could be decoded',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 906, in _parse_json
    return json.loads(json_string)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/json/__init__.py", line 339, in loads
    return _default_decoder.decode(s)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/json/decoder.py", line 364, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/json/decoder.py", line 382, in raw_decode
    raise ValueError("No JSON object could be decoded")
ValueError: No JSON object could be decoded
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 806, in wrapper
    return func(self, *args, **kwargs)
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 827, in __extract_info
    ie_result = ie.extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 534, in extract
    ie_result = self._real_extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/redbulltv.py", line 143, in _real_extract
    }''' % (rrn_id, self._VIDEO_ESSENSE_TMPL % 'LiveVideo', self._VIDEO_ESSENSE_TMPL % 'VideoResource'),
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 899, in _download_json
    expected_status=expected_status)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 883, in _download_json_handle
    fatal=fatal), urlh
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 910, in _parse_json
    raise ExtractorError(errmsg, cause=ve)
ExtractorError: rrn:content:films:6d72aae4-d724-40ff-b627-912a2be1fb7d:en-GB: Failed to parse JSON  (caused by ValueError('No JSON object could be decoded',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Description

Test link:
https://www.redbull.com/gb-en/films/kriss-kyle-out-of-season

I'm having the same issue with 2021.05.16. The request to https://edge-graphql.crepo-production.redbullaws.com/v1/graphql gets HTML as response. I fired mitmproxy to snoop at the requests, and the request headers include Accept: text/html, ... where it should be Accept: application/json. With a mitmproxy script I forced the Accept header to application/json and it works perfectly.
For reference, the mitmproxy script is
def request(flow):
    flow.request.headers['accept'] = 'application/json'

which then is loaded with mitmproxy -s script.py, and then ytdl can pick up the proxy + cert with SSL_CERT_FILE=~/.mitmproxy/mitmproxy-ca-cert.pem youtube-dl --proxy http://localhost:8080 https://www.redbull.com/int-en/live/...
I'm not familiar enough with python or the ytdl codebase, but I hope some other dev can pick this one up. Thanks!


[$500] Tapping download on download banner should deeplink to App and dismiss banner

We recently added a download banner as part of our sign-up flow for upcoming conferences, though oddly, if you tap download and we correctly deeplink you to the App Store, the download banner still appears.

Solution: We should dismiss the download banner no matter which option a user choose, either ignore or `download1.
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~015d3134e73e0c4471
Upwork Job ID:<PHONE_NUMBER>257160704
Last Price Increase: 2023-09-04

Proposal
Please re-state the problem that we are trying to solve in this issue.
Tapping download on download banner should deeplink to App and dismiss banner
What is the root cause of that problem?
We we click on download we're just opening the app using deeplink but not closing the modal.
What changes do you think we should make in order to solve the problem?
We should close the modal by setting state here. in this function
https://github.com/Expensify/App/blob/1e0a249ba030b779d749ba07774a896b0f2c931f/src/components/DownloadAppModal.js#L46
setshouldShowBanner(false)

What alternative solutions did you explore? (Optional)
NA

I agree with @allroundexperts, he proposed the full solution (setShowDownloadAppModal) includes this. It took me sometime to verify my solution and update it.

Proposal
What is the root cause of that problem?
In the handleOpenAppStore function, we're not closing the modal by not calling the handleCloseBanner function.
What changes do you think we should make in order to solve the problem?
We need to make sure we first call handleCloseBanner function inside handleOpenAppStore function so that visually UI/UX wise it represents the closing modal and then visits the external app Url
const handleOpenAppStore = () => {
    handleCloseBanner()
    Link.openExternalLink(link, true);
};

What alternative solutions did you explore? (Optional)
1st >>
const handleOpenAppStore = () => {
setShowDownloadAppModal(false);
setshouldShowBanner(false);
Link.openExternalLink(link, true);
};
2nd >>
We can remove custom logic being set from our DownloadAppModal component and move it to the upper-level parent component
so shouldShowBanner and onClose be propped to our DownloadAppModal and we can utilise it anymore with more controls in the future with callback
const handleOpenAppStore = () => {
    if (onClose) {
        onClose();
    }
    Link.openExternalLink(link, true);
};

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/pvaghani

Proposal from @allroundexperts looks good to me https://github.com/Expensify/App/issues/26706#issuecomment-1705624927
🎀 👀 🎀 C+ reviewed

@allroundexperts Mind getting us kick off with the bug zero checklist?

In the meantime, here's the payment summary:

Contributor: @allroundexperts $750 (incl. urgency bonus)
Contributor+: @narefyev91 Agency

The PR that introduced the bug has been identified. Link to the PR: - it's a fully additional functionality which was not coming from any regressions
The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment: N/A
A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion: N/A
Determine if we should create a regression test for this bug. N/A

@JmillsExpensify @allroundexperts what's the status for te payment here?

Alright, @JmillsExpensify might be delayed with some chores due to the traveling over the weekend.

It seems that @JmillsExpensify was ooo last days. He'll hopefully take care of this one soon.

$750 payment approved for @allroundexperts based on BZ summary.


CPP/error/toggle.cpp

Find out the error in the code
NOTE:- Screenshot of the code and output is required

can I work on this issue

go ahead
@chiragcode11

I can Do this task

is this closed ?


MAINT: update notebooks for Altair 4.0

This PR just updates the notebooks using Altair 4.0. I opened each notebook and did "Restart Kernel and Run All..."
There is no editing for content.

Awesome!
Part of this change is moving to Altair 4.0's default HTML renderer, which means charts show up with interactions in nbviewer! https://nbviewer.jupyter.org/github/altair-viz/altair_notebooks/blob/82e5e841acad68db3778785d964e6a60701628fe/notebooks/02-Tutorial.ipynb

Nice! I plan on going back later and reviewing the content of each notebook, but the first step was brining the old ones up to date.

Cool - I've only skimmed, but this looks great.


Invariant Violation: ListView has been removed.

Invariant Violation: ListView has been removed from React Native. See https://fb.me/nolistview for more information or use deprecated-react-native-listview.
pleach help me, how can fix?

Hi @Traced are you using the latest version of this library? There shouldn't be any more traces of ListView

Closing as inactive

I got the same issue with latest version of this library.
"react-native-swipe-list-view": "^3.2.5"
Please help me check it?


Class 'ParagonIE\ConstantTime\Base64' not found

Class 'ParagonIE\ConstantTime\Base64' not found in C:\wamp64\www\password_lock\src\PasswordLock.php on line 26

This is a Composer-specified dependency. Run composer install to install dependencies.
https://github.com/paragonie/password_lock/blob/31de911f8ba99e8f45139c683bbae5b570ef129f/composer.json#L33


Home and End keys do not work when connected through OpenSSH and ZSH

Environment
Windows build number: 10.0.19041.0
Windows Terminal version: 1.3.2651.0
OpenSSH: OpenSSH_for_Windows_7.7p1, LibreSSL 2.6.5
Ubuntu: 20.04.1 LTS
ZSH: zsh 5.8 (x86_64-ubuntu-linux-gnu)

Steps to reproduce
Connect from Windows to an Ubuntu server through the builtin OpenSSH:
{
        "guid": "{2c4de342-38b7-51cf-b940-2309a097f519}",
        "name": "Ubuntu (SSH)",
        "commandline": "ssh -o ServerAliveInterval=240 root@ubuntu-desktop"
},

Expected behavior
The Home and End key to properly work.
Actual behavior

The Windows build of OpenSSH 7.7 has its own “input generator” that does not properly handle Home and End. You can upgrade to a newer version of use the SSH client from WSL. :smile:

In that case, I wonder how it works in VSCode with the Remote - SSH extension. I don't face this problem in VSCode integrated terminal and I doubt it uses WSL's OpenSSH.
Do you know anything about it? Perhaps the Remote - SSH extension carry its own OpenSSH?

The same issue persists even with the latest version of OpenSSH. I believe this is something else since as I said, it works properly in VSCode integrated terminal.
/cc @DHowett

Glad it works. :smile:

By the way, if others run into this problem, this is the steps to fix it currently:
choco install openssh --version 8.1.0-beta

Then, make sure you have C:\Program Files\OpenSSH-Win64 before C:\Windows\System32\OpenSSH\ in your PATH environment variable:

I get this same issue but just with git bash.

Sorry, I mean I'm not even using ssh, just git bash by itself is doing this.

Here I do not have this problem, and just out of curiosity, this is my Git Bash profile in Windows Terminal:
{
  "commandline": "\"%PROGRAMFILES%\\git\\usr\\bin\\bash.exe\" -i -l",
  "guid": "{00000000-0000-0000-ba54-000000000002}",
  "icon": "%PROGRAMFILES%\\Git\\mingw64\\share\\git\\git-for-windows.ico",
  "name": "Git Bash",
  "startingDirectory": "%USERPROFILE%"
}

And I'm using the latest version of both Git Bash and WT.

Seems like windows terminal preview doesn't flicker.

export TERM=xterm-256color on server works, Ubuntu ssh to Centos

Same issue with Home/End keys in Preview 1.16.3463.0, but without SSH, just plain Git Bash.


About include hybrid_property in QueryAjaxModelLoader fields parameter

Dear,
I found that it's not possible to  include hybrid_property in QueryAjaxModelLoader fields parameter,
Is there any way that I could do an AJAX look up based on hybrid property using the out of the box QueryAjaxModelLoader?
Thanks.

I could look into it. Can you provide and example that you would expect to work?

Thanks for your help @vToMy
For example, I have a model: Customer, it has two fields: first_name and last_name.
Then a hybrid properties name is created as first_name + last name.
For example in Chinese, one person's first name is 张，and last name is 三，then the hybrid property name will be 张三
Ideally it's good if we could filter by "张三" when do Ajax searching, because search by "张" or "三" is both too wide and there will be too many results return back.
While on current version, when we search by 张三, no record will be returned because does not match "张" or "三".
Thanks.

I meant like a sample code I could run.
I've added general support for hybrid properties in flask-admin, but I'm not familiar with QueryAjaxModelLoader or how to use it.

Actually I just tried this and it works with hybrid_properties, so if something's not working for you I'd need an example.


Bluetooth: controller: Fix Conn Param Req procedure stall issue

Fix an issue wherein local or remote initiated Connection
Parameter Request procedure would stall without generation
of LE Connection Update Complete HCI event because a local
or remote initiated PHY Update procedure has overwritten the
currently active Link Layer Control Procedure type.
Signed-off-by: Vinayak Kariappa Chettimada<EMAIL_ADDRESS>

Codecov Report

Merging #12139 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master   #12139      +/-   ##
==========================================
- Coverage   48.05%   48.05%   -0.01%
==========================================
  Files         284      284
  Lines       43368    43369       +1
  Branches    10397    10398       +1
==========================================
- Hits        20840    20839       -1
- Misses      18388    18389       +1
- Partials     4140     4141       +1

Impacted Files
Coverage Δ

subsys/bluetooth/controller/ll_sw/ctrl.c
42.56% <0%> (-0.04%)
:arrow_down:

Continue to review full report at Codecov.

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

recheck

I can confirm that this PR fixes the issue which was seen in our device. Great work @cvinayak !


The Switch component breaks if it contains arbitrary elements

This code will throw an error:
  <Switch>
    Hello, world!
  </Switch>

  is valid react element.


create the base requirement for our project

create the composer json with the autoloader.
and require the first libraries who will be used for our project.

We'll surely need to add other library to our project but for now this issue is closed.


Can't run a Waku 0.17.0 application on Windows 11 with Node v18.14.0

Step taken:

npm create waku@latest
Template: async-template

Setting up project was fine
PS C:\Temp> npm create waku@latest
√ Project Name ... waku-project
√ Choose a starter template » async-template
Setting up project...

Done. Now run:

cd waku-project
npm install
npm run dev

Next cd into project and try to start the development server with npm run dev
Resulting console output when opening http://localhost:3000/ in the browser:
(node:14372) ExperimentalWarning: Custom ESM Loaders is an experimental feature and might change at any time
(Use `node --trace-warnings ...` to show where the warning was created)
(node:14372) DeprecationWarning: Obsolete loader hook(s) supplied and will be ignored: getSource, transformSource
Error: Only URLs with a scheme in: file and data are supported by the default ESM loader. On Windows, absolute paths must be valid file:// URLs. Received protocol 'c:'
    at new NodeError (node:internal/errors:399:5)
    at throwIfUnsupportedURLScheme (node:internal/modules/esm/resolve:1059:11)
    at defaultResolve (node:internal/modules/esm/resolve:1135:3)
    at nextResolve (node:internal/modules/esm/loader:163:28)
    at ESMLoader.resolve (node:internal/modules/esm/loader:838:30)
    at ESMLoader.getModuleJob (node:internal/modules/esm/loader:424:18)
    at ESMLoader.import (node:internal/modules/esm/loader:525:22)
    at importModuleDynamically (node:internal/modules/cjs/loader:1159:29)
    at importModuleDynamicallyWrapper (node:internal/vm/module:429:21)
    at importModuleDynamically (node:internal/vm:102:46)
    at importModuleDynamicallyCallback (node:internal/process/esm_loader:35:14)
    at preloadModule (C:\temp\waku-project\node_modules\react-server-dom-webpack\cjs\react-server-dom-webpack-client.node.unbundled.development.js:98:41)
    at resolveModule (C:\temp\waku-project\node_modules\react-server-dom-webpack\cjs\react-server-dom-webpack-client.node.unbundled.development.js:1915:17)
    at processFullRow (C:\temp\waku-project\node_modules\react-server-dom-webpack\cjs\react-server-dom-webpack-client.node.unbundled.development.js:1986:9)
    at processBinaryChunk (C:\temp\waku-project\node_modules\react-server-dom-webpack\cjs\react-server-dom-webpack-client.node.unbundled.development.js:2134:7)
    at PassThrough.<anonymous> (C:\temp\waku-project\node_modules\react-server-dom-webpack\cjs\react-server-dom-webpack-client.node.unbundled.development.js:2207:5)
    at PassThrough.emit (node:events:513:28)
    at addChunk (node:internal/streams/readable:324:12)
    at readableAddChunk (node:internal/streams/readable:297:9)
    at Readable.push (node:internal/streams/readable:234:10)
    at node:internal/streams/transform:182:12
    at PassThrough._transform (node:internal/streams/passthrough:46:3)
    at Transform._write (node:internal/streams/transform:175:8)
    at writeOrBuffer (node:internal/streams/writable:392:12)
    at _write (node:internal/streams/writable:333:10)
    at Writable.write (node:internal/streams/writable:337:10)
    at PassThrough.<anonymous> (file:///C:/temp/waku-project/node_modules/waku/dist/lib/middleware/rsc/ssr.js:370:16)
    at PassThrough.emit (node:events:513:28)
    at addChunk (node:internal/streams/readable:324:12)
    at readableAddChunk (node:internal/streams/readable:297:9)
    at Readable.push (node:internal/streams/readable:234:10)
    at node:internal/streams/transform:182:12
    at PassThrough._transform (node:internal/streams/passthrough:46:3)
    at Transform._write (node:internal/streams/transform:175:8)
    at writeOrBuffer (node:internal/streams/writable:392:12)
    at _write (node:internal/streams/writable:333:10)
    at Writable.write (node:internal/streams/writable:337:10)
    at file:///C:/temp/waku-project/node_modules/waku/dist/lib/middleware/rsc/worker-api.js:420:29
    at Worker.<anonymous> (file:///C:/temp/waku-project/node_modules/waku/dist/lib/middleware/rsc/worker-api.js:245:121)
    at Worker.emit (node:events:525:35)
    at MessagePort.<anonymous> (node:internal/worker:233:53)
    at [nodejs.internal.kHybridDispatch] (node:internal/event_target:735:20)
    at exports.emitMessage (node:internal/per_context/messageport:23:28)
    at MessagePort.callbackTrampoline (node:internal/async_hooks:130:17) {
  code: 'ERR_UNSUPPORTED_ESM_URL_SCHEME'
}
Error: The destination stream errored while writing data.
    at Transform.<anonymous> (C:\temp\waku-project\node_modules\react-dom\cjs\react-dom-server.node.development.js:11660:20)
    at Transform.emit (node:events:513:28)
    at emitErrorNT (node:internal/streams/destroy:151:8)
    at emitErrorCloseNT (node:internal/streams/destroy:116:3)
    at process.processTicksAndRejections (node:internal/process/task_queues:82:21)

Environment:
PS C:\Temp\waku-project> node -v
v18.14.0

I would like to help on this

related: https://github.com/nodejs/node/issues/31710

@dai-shi this should be closed


report-umfds:0.1.0

I am submitting

[x] a new package
[ ] an update for a package

Description: A template for writing reports for the Faculty of Sciences of the University of Montpellier.

I have read and followed the submission guidelines and, in particular, I

[x] selected a name that isn't the most obvious or canonical name for what the package does
[x] added a typst.toml file with all required keys
[x] added a README.md with documentation for my package
[x] have chosen a license and added a LICENSE file or linked one in my README.md
[x] tested my package locally on my system and it worked
[x] excluded PDFs or README images, if any, but not the LICENSE

[x] ensured that my package is licensed such that users can use and distribute the contents of its template directory without restriction, after modifying them through normal use.

Hi, can you please choose a more "unique" name for your package, so that other alternative package can potentially be proposed? You can simply add a pun or an adjective to the current name.

Hi, can you please choose a more "unique" name for your package, so that other alternative package can potentially be proposed? You can simply add a pun or an adjective to the current name.

Thank you for your feedback, I hope this is okay :D

Yes, the new name works fine? Thank you :)


DO NOT MERGE: Testing the MacOS wheel failures

Test for https://github.com/apache/arrow/issues/39562

@github-actions crossbow submit wheel-macos

So it's clearly not related to the logging fs wrapper around the filesystem ..

@github-actions crossbow submit wheel-macos

@github-actions crossbow submit wheel-macos

@github-actions crossbow submit wheel-macos

@github-actions crossbow submit wheel-macos

So I could nail down the failure to the following:
https://github.com/apache/arrow/blob/ac50918a63f13088a5ea9e6c27a9fadbbb19d53f/cpp/src/arrow/dataset/file_parquet.cc#L814-L823
In the above snippet, sometimes manifest_->descr->num_columns() returns -1, and so then we do a vector resize with -1 which triggers the std::length_error crash.
But I have no idea why that would sometimes return -1, and only on MacOS when running the test from an installed wheel (not any of the other Mac builds where we build Arrow directly)

Fixed in #39632. We can close now.


cannot launch node of type

Problem:
I have built vive_ros from source and I got some sort of path error or node error. Not sure, pretty new to ROS.
ERROR: cannot launch node of type [vive_ros/launch_servervr.sh]: vive_ros
ROS path [0]=/opt/ros/kinetic/share/ros
ROS path [1]=/opt/ros/kinetic/share
so where is the problem? I've strictly followed the installation instrucions on the install.md data.
I've read a lil bit about the catkin workspace..is it possible the reason of the problem?

did you source your workspace?


Proposal: custom state name

Is this a bug report?
No
The proposal itself
My proposal is to add ability to set a custom representation of the state (for sagas). It can be a state's friendly name. Also it can be a kind of localization of a state. With configuration something like:
    State(() => SomeState, state => state.Name = "Friendly name on my native lang");

Would the ability to add a title to the state work for you? Or description? It wouldn't change Name, because honestly that would be entirely broken IMHO, since it's an identifier and not a string.
State(() => Running, "Job is running");

Console.WriteLine($"State: {job.State.DisplayName}");

What i wanted is to have this display name to be persisted as state name in sagas persistent store automatically. So instead of seeing "Running" i want to see "Job is running".
Yes, i think it would be suffice to implement your option. Then i can add another string property to the saga instance and map it to State.DisplayName.

Honestly, that doesn't seem like a great idea. You'd be wiser to update a description table and key of an int state, and then use the description table to map the int -> display name for UI purposes.

I already dropped the idea to store display name automatically :). But what about to have a display name for states as general idea?

Probably not going to happen, just FYI.


[physac] Collision handling and performance

I am not sure how to respond to collisions. Found this issue https://github.com/victorfisac/Physac/issues/31 recommending to check collision seperately. Doesn't that lead to duplicated checks?
From a quick look, it checks every one against every other. Thats fine for a few objects but I can see that being a issue with more objects.
Example of what I am trying to do
My player can shoot bullets at enemies.

Hi @ChrisDill, not sure what's the performance for lots of objects (maybe several hundreds?), probably @victorfisac can give us some numbers?

@ChrisDill That doesn't sound like an absurd number of collision checks. I feel like you would probably be fine with a very ignorant collision system unless you were doing checks between hundreds, or even thousands of objects. I should know because I made a very ignorant collision system called Resolv, haha.
It probably would be best to try following the recommendation on using Physac and see if it fills your needs efficiently. If not, then you can see about refactoring and optimizing.

@SolarLune Thanks thats a good point. It is easy to overthink it. I like your physics library design!
I guess my main issue is responding to collisions. Maybe a example could be useful here.

Update on this issue. Testing using physac_demo, performance lags after 15-20 bodies. I think its because the cost of all the manifolds being generated leading to alot of costly collision checks.
One solution I have seen with other libraries is using axis alligned bounding boxes first to see if a object might be colliding then checking for a more detailed collision afterward. The axis alligned check is cheaper and would allow for more objects.

Closing it. I'm afraid I can not give support to physac library...


Update free-programming-books-langs.md

Add a wiki book for data mining in R
What does this PR do?
Add resource(s)
For resources
This is a free wiki book for data mining in R
Why is this valuable (or not)?
It is a great resource for data mining
How do we know it's really free?
It is free wiki book
For book lists, is it a book? For course lists, is it a course? etc.
It is a book
Checklist:

[] Read our contributing guidelines.
[ ] Search for duplicates.
[ ] Include author(s) and platform where appropriate.
[ ] Put lists in alphabetical order, correct spacing.
[ ] Add needed indications (PDF, access notes, under construction).
[ ] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Thanks, and welcome to free-programming-books!


Support for "for..of" statement

The following valid code is highlighted with many errors in Consulo:
function fireListener(listeners, callbackArgument) {     for (let listener of listeners)     {         listener.call(callbackArgument);     } }

Using JavaScript plugin 2796 (from latest nightly build) — for..of is still highlighted as errorneous.

I pushed some notificator about change js version. But there an option for changing JS version in Project Structure (client js extension) - select EcmaScript 6
If Consulo drop this extension - can you say how u use it(which framework, etc). here

Thanks, after selecting Default Version and updating to latest nightly build it worked.
About usage. I use it with just plain JavaScript — no React/Angular/etc.


add event library stubs

Official Document:
http://docs.php.net/event
Reference files:
https://github.com/KazuakiM/extension-phpdoc/tree/master/src/remote_other

Okay, I remove it. But I have a doubt about the .idea folder. Why add .idea folder to phpstorm-stubs repository?

@zqhong it contains some project specific shared settings for PhpStorm Multiple classes declaration detection setting in Undefined class inspection is disabled.


Incorrect parsing of optionals and no warning that a critical parameter missing

Save this in a file main.tf in a new folder:
variable "config_wrong_cost" {
  type = object({
    db_engine = optional(string, "postgres")
  })

  default = {}
}

variable "config_correct_cost" {
  type = object({
    db_engine = string
  })

  default = {db_engine = "postgres"}
}

resource "aws_db_instance" "postgres" {
  engine         = var.config_wrong_cost.db_engine // **
  # engine         = var.config_correct_cost.db_engine // **
  instance_class = "db.t3.2xlarge"
}

Cd to the folder. This passes terraform validate and requires terraform 1.4.4 and infracost v0.10.19. A terraform plan shows that the engine type is as expected ("postgres").
Now run infracost breakdown --path . on the above: will show 0 cost for the DB:
$ infracost breakdown --path .
Evaluating Terraform directory at .
  ✔ Downloading Terraform modules
  ✔ Evaluating Terraform directory
  ✔ Retrieving cloud prices to calculate costs

Project: .

 Name                                                        Monthly Qty  Unit   Monthly Cost

 aws_db_instance.postgres
 ├─ Database instance (on-demand, Single-AZ, db.t3.2xlarge)          730  hours         $0.00
 └─ Storage (general purpose SSD, gp2)                                20  GB            $2.30

 OVERALL TOTAL                                                                          $2.30
──────────────────────────────────
1 cloud resource was detected:
∙ 1 was estimated, it includes usage-based costs, see https://infracost.io/usage-file

Finally switch which of the two // ** lines is commented out, and re-run infracost. This time it shows correctly the cost of DB instance is non-zero:
 ├─ Database instance (on-demand, Single-AZ, db.t3.2xlarge)          730  hours       $422.67

Note how for the cost=0 case, there is no indication whatsoever that a parameter required (ie critical) to determine cost was not found by infracost. In my original code, the instance class is similarly incorrectly computed, and shows as db_instance_class-mock in the output. No indication of any problem there either, if you didn't notice the odd name you would have no way of knowing.

Thanks @schollii, this will be resolved when https://github.com/infracost/infracost/pull/2389 is merged

The fix for this is now live in the v0.10.20 release


Customizable FORMFIELD_OVERRIDES

Would it be useful addition to move default FORMFIELD_OVERRIDES and formfield_callback to  floppyforms.future.defaults and allow providing alternate formfield_callback function with settings, ie: FLOPPYFORMS_FORMFIELD_CALLBACK='myapp.formfield_callback'?
Use case is when user want to customise globally floppyforms fields within modelforms (ie: not to use HTML5 inputs).
If this seems reasonable, I can work on it.

workaround for now is to just put something like this on top of your form.py
import floppyforms.__future__ as fforms_f
from floppyforms import fields

# Your custom fields
from stripped_fields.models.fields import CharFieldStripped, TextFieldStripped

fforms_f.models.FORMFIELD_OVERRIDES[CharFieldStripped] = {
    'form_class': fields.CharField, 'choices_form_class': fields.TypedChoiceField}
fforms_f.models.FORMFIELD_OVERRIDES[TextFieldStripped] = {
    'form_class': fields.CharField, 'widget': Textarea,
    'choices_form_class': fields.TypedChoiceField}

You can find more examples here https://github.com/gregmuellegger/django-floppyforms/blob/master/floppyforms/future/models.py#L38-L67
Also do not forget on 'choices_form_class': fields.TypedChoiceField which is automaticly patch to all theese dicts like here https://github.com/gregmuellegger/django-floppyforms/blob/master/floppyforms/future/models.py#L71-L72

I would also suggest as enhancement to function https://github.com/gregmuellegger/django-floppyforms/blob/master/floppyforms/future/models.py#L75-L78 to look into db_field.__bases__ if the field is not just inherited from type we know perhaps this might be even better http://stackoverflow.com/a/2611939/2629036


[Update Request]: CodecGuide.K-LiteCodecPack.* 18.3.5

What type of update are you requesting?
A new version of an existing package
Current Package Identifier
CodecGuide.K-LiteCodecPack.Full
Package Version
18.3.5
Please describe the changes you would like to see
Requesting all K-Lite codec packs be updated:

ID
URL

CodecGuide.K-LiteCodecPack.Basic
https://files3.codecguide.com/K-Lite_Codec_Pack_1835_Basic.exe

CodecGuide.K-LiteCodecPack.Standard
https://files3.codecguide.com/K-Lite_Codec_Pack_1835_Standard.exe

CodecGuide.K-LiteCodecPack.Full
https://files3.codecguide.com/K-Lite_Codec_Pack_1835_Full.exe

CodecGuide.K-LiteCodecPack.Mega
https://files3.codecguide.com/K-Lite_Codec_Pack_1835_Mega.exe

#156138


kernel 5.19 requires bash to build

This dependency can be removed with this patch. It's been submitted upstream.
https://0x0.st/o2_n.patch

Thank you I also noticed that when building my kernel

https://codeberg.org/kiss-community/repo/issues/79


如何源代码启动？

python ./NEMbox/init.py 无法启动
Traceback (most recent call last):
File "init.py", line 19, in
from .menu import Menu
ValueError: Attempted relative import in non-package

python -m NEMbox.init
On Mon, Apr 24, 2017, 22:14 jack chen<EMAIL_ADDRESS>wrote:

python ./NEMbox/init.py 无法启动
Traceback (most recent call last):
File "init.py", line 19, in
from .menu import Menu
ValueError: Attempted relative import in non-package
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/darknessomi/musicbox/issues/533, or mute the thread
https://github.com/notifications/unsubscribe-auth/AD_RShOPCcNgmQjdTefNYu2y7UF_0Mzyks5rzK5SgaJpZM4NGONT
.

在项目根目录下，
cd NEMbox; python -m NEMbox.__init__

Detailed explanation can be found here http://stackoverflow.com/questions/6630822/running-a-python-package


mk-python-derivation: fix config.doCheckByDefault being ignored

Description of changes
make buildPythonPackage respect config.doCheckByDefault.
Don't know if you know this: { ... } @ attrs syntax won't include default arguments in attrs (!!). So despite the , doCheck ? config.doCheckByDefault or false in the parameter list, attrs.doCheck will not exist if doCheck is not explicitly passed. And because doCheck is only accessed through attrs, the default argument is meaningless there.
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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

[ ] 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/)
24.05 Release Notes (or backporting 23.05 and 23.11 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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Perhaps the intention is indeed to enable check by default unless disabled, in that case the default argument config.doCheckByDefault in the parameter list is confusing and should be removed.


Seems that "timeui" crash on start on macOS Ventura 13.1, built with Xcode 14.2

macOS: Ventura 13.1
Xcode: 14.2
timeui: 0.0.3
$ swift run timeui
Building for debugging...
Build complete! (0.17s)
Fatal error: The operation couldn’t be completed. (Swift.String error 1.)

Same error appears when install using install.sh script (ignoring the fact that it's broken as well, it needs 'sudo' to copy anything to /usr/local/bin - maybe recent macOS hardened this location?)

Oh yes, that's a bug — the app crashes when there's no argument given. Try running ./run-demo.sh to see how to use. Ideally, I'll also accept a PR to add a proper error message :)

Hmm, ./run-demo.sh works indeed, but I tried to run it with other apps - and it didn't work 🤔
For example: ./.build/release/timeui sleep 5 or ./.build/release/timeui ping -c 5 www.google.com - both fails with the same error.
Am I still missing something?

I see, the first argument is an executable path (as in run-demo.sh), I think to run sleep or ping you need to try timeui /usr/bin/env sleep 5 (can't try that now myself but it's a valid point maybe timeui should try that automatically if the argument isn't a path

Indeed, I browsed the code a little bit and indeed you first try to figure out path of the executable, not the command itself. Is there a reason for that? Or maybe there's some technical difficulty for that?

Nah, not really — my use case is mostly timing running actual binaries that I'm working on, probably never needed to time one of the system commands myself. I think if you need it, you can easily add a PR for it

It's not only for system commands per-se, but you can run, for example some scripts via command something like that python3 foo.py or some build systems, like make. Closer to iOS dev, it could be fastlane command, etc.
I'll try to contribute in my spare time as this is a nice little tool 🙂 You can mark this issue as feature for example...


Add community-cc-list.md file

Description
This adds the initial wider PHP_CodeSniffer community cc list file as proposed in #227.
Suggested changelog entry
N/A
Related issues/external references
Closes #227

@jrfnl please add me in as well, thanks ❤

@michalbundyra Added!


ERR_OPENING MEDIA

When i try to install i am getting this error on my Windows Stremio:

All is running fine at :7000 for so far I can see

I found something

while using <IP_ADDRESS>:7000, i can perfectly add the addon
while using my IP of my home device 192.168.100.x:7000 I am getting this error above

I am facing the same error maybe it needs to run with a https link? I'm just saying that as I have seen some user shared his, which it is using SSL certificates and such.

I am facing the same error maybe it needs to run with a https link? I'm just saying that as I have seen some user shared his, which it is using SSL certificates and such.
EDIT: Yeah I was able to add it to my Stremio using a reverse proxy with a Let's encrypt certificate, still it was not able to scrap anything... I'm stuck at that.

I was able to get things working after setting up reverse proxy with lets encrypt on port 443.   The scrapper is working till it gets to 1337, where it throws errors.   I am getting options to stream for the content that succeeded.

Sorry for some reason I did not get the notifications for these issues. Yes, Stremio will allow <IP_ADDRESS> to run in plain http but anywhere else needs to be with a valid certificate. In this case, the easiest is to have a reverse proxy with Let's Encrypt.

I downloaded Nginx Proxy Manager and created a proxy host, unfortunately i am not be able to redirect 443 > 7000.
I am getting a 502 Bad Gateway.
I am sure I did the right thing, cause when redirecting to 81, i am getting successfully login screen of Nginx Proxy Manager
Any idea?

Thank you for the prompt response, that worked like a charm!