Problem with larger tables

Look like there's a problem with larger sheets (~150k rows), since there's document metadata update every 1000 rows.
Is there a way how to tune it?
In previous version (1.2.1) same configuration took 4-5 minutes. In current version it timed out.

Mě tady https://connection.keboola.com/admin/projects/2117/jobs/380482277?q=%2Bparams.component%3Akeboola.wr-google-sheets %2Bparams.config%3A372058201 jede load 18k řádku 3/4 hodiny #fyi

Related to #16

FIXED in https://github.com/keboola/google-sheets-writer/pull/17


How to get contact list from icloud?

Hi @tmyroadctfig ,
Please help me out, I am start working on project that require the access contact groups from icloud. so how can I access the contact groups from icloud use by this lib?.

It looks like this is possible since it's supported by Pyicloud, but I haven't implemented it here yet. If you port across their work and submit a pull request I'll add it in.
Here are their current docs:
https://github.com/picklepete/pyicloud/tree/ced8e83a40764438b9f0de7f441b932411aaac97#contacts
And this is the implementation class:
https://github.com/picklepete/pyicloud/blob/ced8e83a40764438b9f0de7f441b932411aaac97/pyicloud/services/contacts.py

Thanks @tmyroadctfig


Allow partial phrase match for command palette search

I would like to be able to type "field" and have the results show "Create a field". Right now, you need to type the first word of a command palette phrase to trigger a match.
https://recordit.co/ZOsftNyPP6

should be as simple as using "includes" instead of "startsWith"


Clean up and refactor digital asset template

Clean up and refactor digital asset code, folder structure, file names

Did you run all this through npm run fmt? I feel like some of these lines are longer than 120.

yes, it passes. we also have a fmt check on CI


Create quickSort.c

PR accepted @tushar-kumar
Successfully merged to HacktoberFest_22 repo.
Need to Star this repo incase not done.
🚀🚀👍


link change to readme

just a minor change to the readme (no review needed)

Codecov Report

Merging #103 (0422ef8) into main (ce1ea76) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #103   +/-   ##
=======================================
  Coverage   95.71%   95.71%
=======================================
  Files           5        5
  Lines         257      257
=======================================
  Hits          246      246
  Misses         11       11

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 ce1ea76...0422ef8. Read the comment docs.


The custom hotspot position is not animated when the panorama is moving

Hello.
I want to display a frame for an object on the panorama and change its position and transformation via matrix3d depending on the panorama position.
If I use viewer.setUpdate(true), then the frame does not move at all, and if without viewer.setUpdate, then the frame moves only after the movement is completed.
I can't use the custom hotspot from the examples because I want to apply matrix3d transformation.
What is the correct event to use for movement?
How to make a transformation with animation?
Here is an example of code:
const CONFIG = {
    default: {
        firstScene: '1',
        sceneFadeDuration: 1000,
        autoLoad: true,
        showControls: false,
        autoRotate: false,
        backgroundColor: [255, 255, 255],
        hotSpotDebug: true,
    },
    scenes: {
        '1': {
            hfov: 100.0,
            pitch: 0,
            yaw: -43,
            type: 'multires',
            multiRes: {
                basePath: '/vt/output',
                path: '/%l/%s%y_%x',
                fallbackPath: '/fallback/%s',
                extension: 'jpg',
                tileResolution: 512,
                maxLevel: 4,
                cubeResolution: 3360
            },
        },
    }
};

viewer = pannellum.viewer('tour', VIEWER_SETTINGS);

const element = document.createElement('div');
element.classList.add('custom-hotspot');
const container = document.getElementsByClassName('pnlm-render-container');
container[0].append(element);

viewer.on('animatefinished', function () {
    const coord = [
        Math.floor(Math.random() * 100),
        -1 * (Math.floor(Math.random() * 400) + 200)
    ];

    const transform = 'translate(' + coord[0] + 'px, ' + coord[1] +
        'px) translateZ(9999px) rotate(0deg)';

    element.style.webkitTransform = transform;
    element.style.MozTransform = transform;
    element.style.transform = transform;

    viewer.setUpdate(true);
});

The viewer.setUpdate() method is for handling dynamic content as the panorama input. This is for supporting, e.g., <video> and <canvas> elements as input. This only works for equirectangular images, not multires like you're using, and it's irrelevant for anything that isn't the panorama image source, anyway.

then the frame moves only after the movement is completed

What did you expect was going to happen? The documentation for the animatefinished event states that it fires "when any movements / animations finish, i.e. when the renderer stops rendering new frames". As per the documentation, it fires when the movement stops, not during the movement.

If I use viewer.setUpdate(true), then the frame does not move at all

Normally, the viewer doesn't render new frames when there's no movement, since that would be wasteful as nothing is changing. To support dynamic content, the viewer has to render new frames even when the view is stationary. Since the viewer continues to render frames, the animatefinished event isn't fired. This is arguably a bug, since it breaks the intent of the event (although it's not relevant for what you're trying to do), but it is consistent with the documentation.

What is the correct event to use for movement?

There is currently no such event. This is tracked by https://github.com/mpetroff/pannellum/issues/310#issuecomment-264591495, which I should really revisit at some point but have not gotten around to. There I discuss a render event, although it's probably better to also have a viewchanged event, to better handle the nuances of dynamic content.


Handle no session properly

This PR addresses issue #722. When no session is find in recordedit it continues on rather than throwing a not found error. This will now properly error out when checking the ACLs for the returned reference.
I also removed the error code numbers from the error messages that show up in the modal.

@jrchudy Aside from the the small typo I commented on, everything else looks good to me.


untested bug squash and add test

Fixed a bug I noticed when adjusting indices when start_index=1 for cases with multiparts. The multipart break value within the array was being inadvertently modified. I also added a test for this, and I added a test for multipolygons with multiple holes since we have a WKT for it.

Fixed a bug I noticed when adjusting indices when start_index=1 for cases with multiparts.

Yup. :+1:


Feature Request: Configurable retry limit/delay time when checking deployment status.

We have a deployment pipeline that runs serverless deploy for about 100 functions as well as autoscaling rules bringing it up to 200 pods.
I think this load during deployment causes the plugin to fail to find new pods within the 3 retry limit here: https://github.com/serverless/serverless-kubeless/blob/b0376313e7607d90a99036bc8c34a024c547e12f/lib/deploy.js#L215
Retrying the deployment pipeline gets further and it takes around 5 retries to get through everything.
It would be great for us to be able to configure the retry limit and/or the delay time between retries to something more reasonable for our use case.

Serverless: Deploying function alchemy-auth-get-batch...
Serverless: Unable to find any running pod for alchemy-auth-add-cluster-admin. Retrying...
Serverless: Unable to find any running pod for alchemy-auth-get-batch. Retrying...
Serverless: Unable to find any running pod for alchemy-auth-add-cluster-admin. Retrying...
Serverless: Function alchemy-auth-list-public-entities successfully deployed
Serverless: Function alchemy-auth-get-entity successfully deployed
Serverless: Function alchemy-auth-list-child-entities successfully deployed
Serverless: Function alchemy-auth-calculate-usage successfully deployed
Serverless: Function alchemy-auth-create-alchemy-user successfully deployed
Serverless: Unable to find any running pod for alchemy-auth-get-batch. Retrying...
Serverless: Function alchemy-auth-list-candidate-entities successfully deployed
Serverless: Function alchemy-auth-create-entity successfully deployed
Serverless: Function alchemy-auth-calculate-created successfully deployed
Serverless: Function alchemy-auth-ping successfully deployed
Serverless: Function alchemy-auth-create-session successfully deployed
Serverless: Function alchemy-auth-get-entity-by-name successfully deployed
Serverless: Function alchemy-auth-unimpersonate-user successfully deployed
Serverless: Function alchemy-auth-create-company successfully deployed
Serverless: Function alchemy-auth-delete-session successfully deployed
Serverless: Function alchemy-auth-create-service-provider successfully deployed
Serverless: Function alchemy-auth-enable-entity successfully deployed
Serverless: Function alchemy-auth-impersonate-user successfully deployed
Serverless: Function alchemy-auth-get-session successfully deployed
Serverless: Function alchemy-auth-execute successfully deployed
Serverless: Function alchemy-auth-update-entity successfully deployed
Serverless: Function alchemy-auth-disable-entity successfully deployed
Serverless: Function alchemy-auth-patch-entity successfully deployed
Serverless: Function alchemy-auth-set-roles-on-session successfully deployed
Serverless: Unable to find any running pod for alchemy-auth-add-cluster-admin. Retrying...
Serverless: Unable to find any running pod for alchemy-auth-get-batch. Retrying...
Serverless: Unable to find any running pod for alchemy-auth-add-cluster-admin. Retrying...
Serverless: Unable to find any running pod for alchemy-auth-get-batch. Retrying...
Serverless: Giving up, unable to retrieve the status of the alchemy-auth-add-cluster-admin deployment.
Serverless: Unable to find any running pod for alchemy-auth-add-cluster-admin. Retrying...
Serverless: Giving up, unable to retrieve the status of the alchemy-auth-get-batch deployment.

  Error --------------------------------------------------

  Error: Found errors while processing the given functions:
  Unable to retrieve the status of the alchemy-auth-add-cluster-admin deployment
  Unable to retrieve the status of the alchemy-auth-get-batch deployment
      at Object.checkFinished (/srv/node_modules/serverless-kubeless/lib/helpers.js:211:14)
      at /srv/node_modules/serverless-kubeless/lib/deploy.js:564:21
  From previous event:
      at PluginManager.invoke (/usr/local/lib/node_modules/serverless/lib/classes/PluginManager.js:490:22)
      at /usr/local/lib/node_modules/serverless/lib/classes/PluginManager.js:525:24
  From previous event:
      at PluginManager.run (/usr/local/lib/node_modules/serverless/lib/classes/PluginManager.js:525:8)
      at /usr/local/lib/node_modules/serverless/lib/Serverless.js:133:33
      at processImmediate (internal/timers.js:439:21)
      at process.topLevelDomainCallback (domain.js:130:23)
  From previous event:
      at Serverless.run (/usr/local/lib/node_modules/serverless/lib/Serverless.js:120:74)
      at /usr/local/lib/node_modules/serverless/bin/serverless.js:75:30
      at /usr/local/lib/node_modules/serverless/node_modules/graceful-fs/graceful-fs.js:111:16
      at /usr/local/lib/node_modules/serverless/node_modules/graceful-fs/graceful-fs.js:45:10
      at FSReqCallback.oncomplete (fs.js:146:23)
  From previous event:
      at /usr/local/lib/node_modules/serverless/bin/serverless.js:75:8
      at processImmediate (internal/timers.js:439:21)
      at process.topLevelDomainCallback (domain.js:130:23)
  From previous event:
      at Object.<anonymous> (/usr/local/lib/node_modules/serverless/bin/serverless.js:64:4)
      at Module._compile (internal/modules/cjs/loader.js:955:30)
      at Object.Module._extensions..js (internal/modules/cjs/loader.js:991:10)
      at Module.load (internal/modules/cjs/loader.js:811:32)
      at Function.Module._load (internal/modules/cjs/loader.js:723:14)
      at Function.Module.runMain (internal/modules/cjs/loader.js:1043:10)
      at internal/main/run_main_module.js:17:11

  Get Support --------------------------------------------
     Docs:          docs.serverless.com
     Bugs:          github.com/serverless/serverless/issues
     Issues:        forum.serverless.com

  Your Environment Information ---------------------------
     Operating System:          linux
     Node Version:              12.14.1
     Framework Version:         1.60.5
     Plugin Version:            3.2.7
     SDK Version:               2.2.1
     Components Core Version:   1.1.2
     Components CLI Version:    1.4.0

Serverless: Unable to find any running pod for alchemy-auth-get-batch. Retrying...

Hey @andresmgot
After some investigation it seems that the root of the problem is actually the time taken by the install init container, it takes a while to run npm install with so many other pods trying to deploy at the same time.
Since the serverless-kubeless plugin is only looking for "Running" pods it can easily push past the timeouts while npm downloads and compiles packages.
I have a branch with these config options implemented let me know if you'd like this as a PR, but for us I think the real solution is to improve the install time. Happy to close this issue otherwise.
(https://github.com/d0x2f/serverless-kubeless/commit/9823fa8ce606114fcd40708a27afc95a2a89cfc2)

Thanks for the report and the link to the commit! Let's leave as it is then and check if someone else finds the same problem, we can re-evaluate then.


Add RegexPriority support to /routes endpoints

Resolves #21

@andres-rojas looking at the documentation I don't believe this property is on the request? It comes back in the response and it is already on there or am I missing something?

I'll be adding a test to confirm this functionality as per our response / conversation in the linked issue.

closing as this is now supported


Working with Docker

When using the Docker image the client will fail while setting up the interface. The client is successfully retrieving data from the index I filled out, CORS is working as far as I can tell.
What goes wrong is a call made to https://accapi.appbase.io/user. In my logs I can see
XMLHttpRequest cannot load https://accapi.appbase.io/user. No
'Access-Control-Allow-Origin' header is present on the requested resource.
Origin '<my-dev-domain>' is therefore not allowed access.
The response had HTTP status code 401.

The content of the response is
{"message":"Not Logged In, No Token"}
Would be awesome if anyone could give an answer to why this call is made and either how to disable it from doing so or how I can make it work. At first I thought this client was to be used separately from Appbase, is that not the case?
Thanks!

@timhuijser Which port are you using dejavu from?

@siddharthlatest Thank you for your fast reply. I am using dejavu from port 1358, I'm accessing it on <my-dev-domain>:1358/. I'm using the browser view and the issue is after filling out the index name and the ES target.
See screenshot: http://imgur.com/a/Q9tvX

@timhuijser I tested with appbaseio/dejavu:0.13.0 and do not see that request being made.

I was using appbaseio/dejavu:latest, when trying appbaseio/dejavu:0.13.0 I'm getting the same error. While digging around I saw this code, which looks like the cause of the call. Any idea why this call is made?

We added an import data functionality recently, where we need to log in a
user to show the app to import to. It seems there is a side effect of that
in the data browser view, which shouldn't require a login.
Thanks for pointing this out, I would recommend trying 0.12.0 version on
docker in the meanwhile.
On Thu, Jul 6, 2017 at 3:36 AM Tim Huijser<EMAIL_ADDRESS>wrote:

I was using appbaseio/dejavu:latest, when trying appbaseio/dejavu:0.13.0
I'm getting the same error. While digging around I saw this code
https://github.com/appbaseio/dejavu/blob/40e0ceac3261dcdebef23f9a4af4bc9817cb69fd/live/src/js/features/Importer.js#L31,
which looks like the cause of the call. Any idea why this call is made?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/appbaseio/dejavu/issues/86#issuecomment-313332860,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAmdGsmAw9ipEykjXb6GRpIv86uFxosaks5sLJyZgaJpZM4OOcKL
.

--
written in the spirit of three.sentenc.es

Thank you for the explanation, I will try the previous version!

Update: Able to replicate the issue, will be fixed in v0.14.0. Thanks for pointing out @timhuijser.

This has been fixed in the dev branch.


incorrect peer dependency

<EMAIL_ADDRESS>has incorrect peer dependency "next@^4".
although it works fine with next 6

see PR
https://github.com/BDav24/next-url-prettifier/pull/15


Proliferation of forecasting base classes?

@mloning, @big-o (creditef as contributor in the base classes):
I wanted to work on #464 and looked at the base classes in forecasting, amongst others.
I looked at the base classes and feel there is some unnecessary proliferation of classes going on.
Why: we seem to have two nested template base classes, BaseForecaster and _SktimeForecaster.
The (undocumented) extension pattern seems unnecessarily complicated.
To give one example, implementation of prediction (can you confirm that this is the intended pattern, see new post in #464):
The BaseForecaster expects from extender to inherit and implement predict.
However, if you implement a sktime native forecaster (which most extenders will be concerned with), you have to inherit from _SktimeForecaster, which inherits BaseForecaster and implements predict, but now you have to implement _predict.
I would strongly favour there being only one base class - with all private functions - and a clean docstring that states what to implement.
Only vendor adaptors and similar overrides should have nested template schemes.
Mixins (like for the forecasting horizon) are ok i.m.o.
That way, for example, one wouldn't have to track overrides through two parent classes...
What do you think, @mloning, @big-o? Next major version to refactor this?
Considering use of abc since it's getting a little complex?

I think my contribution to those classes was when I tidied up the prediction intervals API, so I only have one very specific perspective on this. That said, I felt very much the same at the time I worked on it - I was unsure what the justification was for two separate classes (there was probably a good reason at the time, but it seems less clear now). It was difficult to figure out where new methods should live.
Something I liked in earlier versions of sktime that perhaps we could return to in this refactor is a simplification of the docstrings. At the moment, the documentation is spread across both private and public methods. Not only does this clutter up the codebase a little (and make it less readable), but it doesn't play as nicely with documentation tools like sphinx. I think it could be better to keep all of the meaningful user-focussed documentation in the public methods, and the private methods can contain very light documentation notes for developers (if absolutely necessary), or be undocumented (comments where they are called in the public methods can explain what they are being used for if needed). For example:
def predict(self, fh, X=None, return_pred_int=False, alpha=DEFAULT_ALPHA):
    """Meaningful, generic, napoleon-formatted user documentation."""
    # Call class-specific implementation.
    return self._predict(fh=fh, X=X, return_pred_int=return_pred_int, alpha=alpha)

@abstractmethod
def _predict(self, fh, X=None, return_pred_int=False, alpha=DEFAULT_ALPHA):
    "A light, informal note *iff* it's useful for developers."
    raise NotImplementedError()

I think my contribution to those classes was when I tidied up the prediction intervals API, so I only have one very specific perspective on this. That said, I felt very much the same as you at the time I worked on it - I was unsure what the justification was for two separate classes (there was probably a good reason at the time, but it seems less clear now). It was difficult to figure out where new methods should live.
Something I liked in earlier versions of sktime that perhaps we could return to in this refactor is a simplification of the docstrings. At the moment, the documentation is spread across both private and public methods. Not only does this clutter up the codebase a little (and make it less readable), but it doesn't play as nicely with documentation tools like sphinx. I think it could be better to keep all of the meaningful user-focussed documentation in the public methods, and the private methods can contain very light documentation notes for developers (if absolutely necessary), or be undocumented (comments where they are called in the public methods can explain what they are being used for if needed). For example:
def predict(self, fh, X=None, return_pred_int=False, alpha=DEFAULT_ALPHA):
    """Meaningful, generic, napoleon-formatted user documentation."""
    # Call class-specific implementation.
    return self._predict(fh=fh, X=X, return_pred_int=return_pred_int, alpha=alpha)

@abstractmethod
def _predict(self, fh, X=None, return_pred_int=False, alpha=DEFAULT_ALPHA):
    "A light, informal note *iff* it's useful for developers."
    raise NotImplementedError()

I think it could be better to keep all of the meaningful user-focussed documentation in the public methods, and the private methods can contain very light documentation notes for developers (if absolutely necessary), or be undocumented (comments where they are called in the public methods can explain what they are being used for if needed).

Hm, I don't like that - I think even abstracts and privates should be well-documented in-place, i.e., directly in the code.
More docstrings do not necessarily mean more readability, but in this case (and more often than not) I think it does - as the base classes would be a common place for advanced users to look for an explanation of how sktime works internally.

Hi @fkiraly, yes I agree, the forecasting API needs to be simplified, this is still mostly my first attempt.
I'm not sure what the best solution is in terms of base classes. Many implementations at the moment are in fact interfaces to other packages. I agree that the use of _predict saves us from some code duplication but makes it hard to understand and extend.
In hindsight, I'd also remove the update_predict functionality from the estimator class and implement in a separate stand-alone function. It adds a lot of complications inside the class for handling state changes.

High-level, I see two major options - both resulting in a merge of the two base class templates:

make it more lean and adopt the pattern where fit, predict, etc are overridden in concrete implementation descendants
make it nested and have fit and _fit and predict and _predict, where utility functionality and other plumbing sits in the public methods, while extenders consistently implement the private ones

An "outsider" option but perhaps more elegant is to do the first, and mimick behaviour of the second via decorators on the methods.

In hindsight, I'd also remove the update_predict functionality from the estimator class and implement in a separate stand-alone function. It adds a lot of complications inside the class for handling state changes.

Regarding update_predict: I'd advocate to leave it in, because it's a very common deployment mode that users might expect and actually use (I suspect without concrete evidence but from my view of the field - opinions anyone?).
We just need to very carefully think about the state model here.

In deployment, you'd use update and predict separately or what we call update_predict_single - of course they need to stag.
But update_predict does temporal CV which creates lots of complications - I don't see any reasons why that should live in the class but open to other thoughts?

From #941 on the refactoring of _set_fh:
I've thought about this a bit more and I prefer the option where we pass the name of the calling method to _set_fh, e.g. _set_fh(fh, method="fit"). For me, the main reasons are:

it keeps the functionality for checking/setting the fh contained to that function and does not assume state changes elsewhere in the code, like at the beginning of fit,
it keeps it closer to the implementation of other estimator types where we do not re-set the state at the beginning of fit,
it avoids the redundancy of having to set _is_fitted=False in the constructor and at the beginning of fit.

solved now via #912


C#: support serialization to IBufferWriter

Allow writing to SerializationContext through IBufferWriter<byte> interface.

allows writing directly to a native slice buffer, eliminating overhead of copying chunks of memory between byte[] and grpc_byte_buffer
allows slice-by-slice serialization of bigger payloads
current way of serializing data requires user to allocate a new byte array each time they serialize a message, which can cause a lot of GC pressure especially if the payload is big. The new API allows to overcome this limitation
grpc-dotnet implementation should be able to provide access to the http response in form of IBufferWriter as well.

Work remaining:

make sure the default protobuf serialization uses the more efficient API when possible.

@apolcyn  this is ready for review.
@JunTaoLuo  @JamesNK  feel free to review.

@apolcyn  addressed the comments.
@JamesNK  @JunTaoLuo  can you please review the additions to the SerializationContext API and confirm that this is going to work well with grpc-dotnet?

For grpc-dotnet's perspective, we only really care about the two new virtual methods on SerializationContext. Is that correct? Looks ok to me.

Yes, I wanted to know if you're happy with the newly proposed methods and their behavior.
(and if the Complete() semantics makes sense to you).

I guess there isn't much way to performance test without a Protobuf serializer that supports IBufferWriter.


Should wrap all elements in p tags when removing, except for existing p tags

For example, given a table row

something
somethingelse

The following extraction occurs
somethingsomethingelse

Which then just renders
somethingsomethingelse

fixed in #6


alerts for grouped pre liquidity

Create a market with orders. User is directed to the portfolio page. In the My created markets section, there's a button to bring up pre-liquidity modal.
The orders are batch sent for creation. 0x mesh they all will be sent. We are assuming the same log will come in for 0x order creation.
Toast can say orders have been placed as this is done a group transaction
Bell message should show each individual order
spreadsheet doesn't differentiate pre-liquidity Order Placed creation from regular place order alert. toast message is different.
Toast Message:
Order Placed
[Market Title]
Your orders have been added

see excel sheet
https://docs.google.com/spreadsheets/d/1Q18_TKEg9lcSrZX8eWcKt26IjTAc-KNMfBn8oaKBrYc/edit?ts=5d7969a8#gid=0v


Replace Tab's options to getOptions(key: Any? = null)

Sometimes we need to display different icons or titles according to different situations, such as whether the tab is selected or not, so it is good to provide such interface.

What is this key? The screen key or a custom key to be used on remember?
The  options has a @Composable get, so you can get the TabNavigator and check if it is selected:
val options: TabOptions
  @Composable
  get {
    val tabNavigator = LocalTabNavigator.current
    return if (tabNavigator.current.key == key) {
      ...
    } else {
      ...
    }
  }


Python user documentation: fix code highlighting, use valid Nix expressions

Fix code syntax highlighting by specifying language in every code block and adding some context to Nix code blocks to make them valid expressions.  Use the same markup style for all code blocks.

The latest version uses
    preConfigure = ''
      export LDFLAGS="-L${pkgs.fftw.out}/lib -L${pkgs.fftwFloat.out}/lib
-L${pkgs.fftwLongDouble.out}/lib"
      export CFLAGS="-I${pkgs.fftw.dev}/include
-I${pkgs.fftwFloat.dev}/include -I${pkgs.fftwLongDouble.dev}/include"
    '';

On Sun, Mar 5, 2017 at 7:21 PM, Alexey Muranov<EMAIL_ADDRESS>wrote:

@alexeymuranov commented on this pull request.
In doc/languages-frameworks/python.md
https://github.com/NixOS/nixpkgs/pull/23535#discussion_r104319731:

LDFLAGS="-L${pkgs.fftw.dev}/lib -L${pkgs.fftwFloat.out}/lib -L${pkgs.fftwLongDouble.out}/lib"
CFLAGS="-I${pkgs.fftw.dev}/include -I${pkgs.fftwFloat.dev}/include -I${pkgs.fftwLongDouble.dev}/include"
'';

FIXME!

LDFLAGS="-L${pkgs.fftw.dev}/lib -L${pkgs.fftwFloat.out}/lib -L${pkgs.fftwLongDouble.out}/lib"
CFLAGS="-I${pkgs.fftw.dev}/include -I${pkgs.fftwFloat.dev}/include -I${pkgs.fftwLongDouble.dev}/include"
'';

There is no LDFLAGS in https://github.com/NixOS/
nixpkgs/blob/master/pkgs/top-level/python-packages.nix
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/NixOS/nixpkgs/pull/23535#discussion_r104319731, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACB872QoGieMkkQsangBIWINzsbM7ItLks5riv0fgaJpZM4MTeGV
.

Sorry, i do not know where to look for the "latest version".  Anyway, i see now that apparently it is
preConfigure = ''

that is missing, thanks.

Its not really missing. The way it is currently in the docs is valid as well.

The way it is currently in the docs is not even valid Nix.

 At least I fail to figure out what would be non-valid in there...

I suppose that
{
  LDFLAGS="-L${pkgs.fftw.dev}/lib -L${pkgs.fftwFloat.out}/lib -L${pkgs.fftwLongDouble.out}/lib"
  CFLAGS="-I${pkgs.fftw.dev}/include -I${pkgs.fftwFloat.dev}/include -I${pkgs.fftwLongDouble.dev}/include"
  '';
}

is then also valid Nix, right?

Ah, now I see what you mean... Yep, sorry, this is not valid Nix, you are right!

Yes, and for a good reason.  Should i add preConfigure = '' or remove ''; and add a ;?  (I am not sure.)

The former would have the preference.


Failing unit tests on master branch

Issue description
It seems that after last merged pull request one of the test suites is failing.
Is it something wrong on my local environment or is this happening to others as well?
Steps to reproduce and a minimal demo of the problem

Clone repo to a new directory
Run 'npm install'
Run 'npm run test'

Current behavior
One of the test suites is failing due to not recognized imports.
After checking out one of the previous commits and running 'npm install' and 'npm run test' all tests pass.
Expected/desired behavior
All test suites should pass

Correct, because the repo has to be built before running tests.
Do
npm install
npm run build
npm run test

Additionally, we are using yarn for building agm, not npm, so really it should be
yarn
yarn build
yarn test


Unable to start counterparty-core on testnet on 10.5

I seem to be unable to bring up counterparty-core testnet. When updating to 10.4.6 last week, I was suddenly unable to bring testnet back up. I asked at the time, and was told there were still some issues on testnet. I have waited a week and am now trying to bring up 10.5.0 on testnet and still having issues.
I realize this is probably just a local issue with my docker container, so I am curious how one can blow away/reset the docker containers for testnet and have them re-created, as the docker compose --profile testnet up -d command is not working to bring up the testnet instance.

docker container prune removed the stopped containers and resolved the issue.


Support User Data / Cloud Init for AWS Node Templates

What kind of request is this (question/bug/enhancement/feature request): feature request
The title says it all.
However, specifically since Rancher has Rancher OS and the primary way of configuring it is to use UserData/Cloud-Init, I believe this should be a first class supported item.

I don't know if you noticed but you can send some userdata to AWS iinstances, however :

It is not supported in the UI so you need to use the API
For AWS, the userdata has to be in a file on the Rancher instance. (We use a configmap to do so)

I think the later is because Rancher uses Docker Machine behind the scene to provision the AWS instances, and Docker Machine only supports the use of files for userdata.
This is not the easiest nor the cleanest way, but it works.


Allow MLflow URI with scheme

MLflow can load many schemes. However, get_model_uri accepts only local file paths.
This PR enables MLflowRuntime to load models by URIs such as s3://... and file:....

we try to shy away from downloading artefacts in MLServer, as we see it as more of an infrastructure concern

Your policy is reasonable. Downloading artifacts introduces complexity such as cache control.


Can't get bearer token from request in C# function

In the JS example for custom role assignment at https://github.com/staticwebdev/roles-function/blob/main/api/GetRoles/index.js the bearer token for calling graph is retrieved by setting a const user = req.body and then getting the token from user.accessToken
The request body in C# is a stream that does not contain anything. What is the C# equivalent to the js example code?

The format of accessing fields in the request body is the same for this context as for normal HTTP trigger C# functions (https://docs.microsoft.com/en-us/azure/azure-functions/functions-bindings-http-webhook-trigger?tabs=csharp).
The following code snippet can be used to access the access token for example:
string requestBody = string.Empty;
using (StreamReader streamReader =  new  StreamReader(req.Body))
{
     requestBody = await streamReader.ReadToEndAsync();
}

dynamic data = JsonConvert.DeserializeObject(requestBody);
string accessToken = data?.accessToken;


Login Issue

Hi.
Firstly this is a great app. so kudos for making it
one problem i have is ive added my alt name under local.php
i try to login and i get

no issues in the logs so no idea

Hi
Are you able to login with you main char you added to the local.php as a
'admin'?
You may look into the log files in "./data/log/". In this directory you
should have a log file for each day. In the log you should see any login
attempt. In case of problems please send me the last log and your
local.php file.
Kind regards & fly safe,
Rtg Quack
Am 19.11.2017 um 16:00 schrieb CokkocZateki:

Hi.
Firstly this is a great app. so kudos for making it
one problem i have is ive added my alt name under |local.php|
i try to login and i get
image
https://user-images.githubusercontent.com/16074209/32991807-bfb6d664-cd42-11e7-89b9-b524ded6255b.png
no issues in the logs so no idea
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/chrRtg/eve-skylizer/issues/1, or mute the thread
https://github.com/notifications/unsubscribe-auth/AP-JwHym0MD0g1xmEN3XXyPl5KACZTvRks5s4EKogaJpZM4QjaMH.

Hi.
Yes ive added my alt as the admin. this is my log below
X's are omitted for sensitve info
2017-11-19T14:52:07+00:00 DEBUG (7): Auth indexAction
2017-11-19T14:52:07+00:00 DEBUG (7): Auth indexAction
2017-11-19T14:52:11+00:00 DEBUG (7): Auth indexAction
2017-11-19T14:52:11+00:00 DEBUG (7): Auth indexAction
2017-11-19T14:52:11+00:00 DEBUG (7): token: League\OAuth2\Client\Token\AccessToken Object
(
    [accessToken:protected] => xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
    [expires:protected] =><PHONE_NUMBER>
    [refreshToken:protected] => xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
    [resourceOwnerId:protected] =>
    [values:protected] => Array
        (
            [token_type] => Bearer
        )

)

2017-11-19T14:52:11+00:00 DEBUG (7): EveSSO, checkCredentials for user: __Cokkoc Zateki__  belongs to: __Space Mutts__
2017-11-19T14:52:11+00:00 NOTICE (5): login attempt failed for Cokkoc [Space Mutts]
2017-11-19T14:53:06+00:00 DEBUG (7): Auth indexAction
2017-11-19T14:53:09+00:00 DEBUG (7): Auth indexAction
2017-11-19T14:53:12+00:00 DEBUG (7): Auth indexAction
2017-11-19T14:53:12+00:00 DEBUG (7): token: League\OAuth2\Client\Token\AccessToken Object
(
    [accessToken:protected] => xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
    [expires:protected] =><PHONE_NUMBER>
    [refreshToken:protected] => Sxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
    [resourceOwnerId:protected] =>
    [values:protected] => Array
        (
            [token_type] => Bearer
        )

)

2017-11-19T16:27:13+00:00 DEBUG (7): EveSSO, checkCredentials for user: Cokkoc Zateki  belongs to: Space Mutts
2017-11-19T16:27:13+00:00 NOTICE (5): login attempt failed for Cokkoc Zateki [Space Mutts]

Problem Solved. I needed to clear my Cache


[webview_flutter_android] Fix throwing StateError when onShowFileChooser was nonnull

Fix to https://github.com/flutter/plugins/pull/6881
The release of 3.2.0 has not finished yet, so I wanted to try and sneak this fix in and release webview_flutter_android manually.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the relevant style guides and ran the auto-formatter. (Unlike the flutter/flutter repo, the flutter/plugins repo does use dart format.)
[x] I signed the CLA.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I listed at least one issue that this PR fixes in the description above.
[ ] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy, or this PR is exempt from version changes.
[ ] I updated CHANGELOG.md to add a description of the change, following repository CHANGELOG style.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Merging on red to fix a bug before a release.


Remove PEP8 violations and Use strip() to remove quotes

Do PEP8 check and make strip_outer_quotes to use strip() to remove single or double quotation marks.

@jesserayadkins

Thanks!


how the heck is my guild in this scrapey bakey

meow

Well, it's done by fetching data from algolia's api and discord's api.
Take a look:
https://github.com/xhyrom/discord-list/blob/b6e1223956fafdd65d8a4dee104e7608f175d2a8/scripts/update/guilds.mjs#L65-L110

wow nice, so you bypass rate limits using dns?


fix: allow certain keychain operations without a password

Listing, removing, renaming etc keys do not require a password so
the user should not be required to provide one.
This means we don't have to prompt the user to create a password
when they aren't going to do any operations that require a password.

I've update this to not require a password. I'll create an issue for supporting a password rotate feature. We should be able to set new passwords on they keychain. This will give us the ability to start without one and then "rotate" to one later.

@achingbrain is this enough for js-ipfs to use libp2p's keychain without a password?
I believe that this needs to be changed: https://github.com/libp2p/js-libp2p/blob/master/src/index.js#L85

I believe that this needs to be changed: https://github.com/libp2p/js-libp2p/blob/master/src/index.js#L85

Good catch, I rebased with master and added a test for libp2p creation without a keychain pass.

FYI I verified it fixes js-ipfs, https://github.com/ipfs/js-ipfs/pull/3212. I'm working on bubbling it up there.

Thanks for pulling this across the line!


Cannot click on "Task" (Tick mark) in iOS safari

Version
v1.4.7
Test Environment
iOS safari
Current Behavior
In WYSIWYG mode, when user is typing (showing keyboard)
for the "Task" item

[ ]

cannot not click on it to see the "Tick" mark
Expected Behavior
After click on it should show tick-ed

[x]

@kelvinkoko
I checked the issue. The editor doesn't fully support mobile but.. This issue seems to be caused by focus on mobile safari. I've tested it on version 13.1.2, iPhone XR, and the click behavior was different depending on the focus position.
The checkbox is not clicking when the focus is right behind the checkbox. The gap between the checkbox area and the next focus is too narrow, so the checkbox area is focused instead of being clicked.

Correct behavior

Wrong behavior

If you are overriding the style of the editor, you might solve the problem by adjusting the spacing between the checkbox and the text that follows.
.tui-editor-contents .task-list-item {
  padding-left: 22px; /* original value */
}

Can you check it? :)

@seonim-ryu Thank you for your investigation, you are right. I think it is the cause.
Thanks for you guys great work and your support!~

I'm closing this issue because it seems to be resolved.


Android X update for Exo Player?

Im trying to build on bitrise but I got the following error:
/Users/vagrant/git/node_modules/react-native-video/android-exoplayer/src/main/java/com/brentvatne/exoplayer/VideoEventEmitter.java:3: error: package android.support.annotation does not exist import android.support.annotation.StringDef;
Looking at the code in exoplayer/ I see that there are still some old references to android.support. Is there a reason this hasnt been updated to support Android X yet?
Running the latest "react-native-video": "^5.0.0",

This was fixed in 5.0.2. Closing the issue for now but feel free to open a new one if this persists after 5.0.2.


saved models are not displayed well.

use_crabnet.py has a function "list_saved_models".
This function lists models, but they aren't usable when you copy them. We should clean this up so you can double click on a model and copy it directly into the predict_crabnet() function.

Fixed with https://github.com/anthony-wang/CrabNet/commit/c44b48d711e894b451eba088013c818aafdacd82


Added satNOGS

I saw that the repo didn't have info on any Ground Stations. Added SatNOGS. Open Source. Anyone can use the network and also self-host a part of it

Looks great! Thanks for contributing ✨


Add Wine install

Pull Request
Title
Add Wine install
Type of Change

[x] New feature

Description

Adding Wine install on Arch, Fedora and Ubuntu / Debian

Testing

Tested on Ubuntu, Fedora and Arch the install of Plain Wine

Impact

No dependencies added, no impact in performance.

Issue related to PR

Resolves https://github.com/ChrisTitusTech/linutil/issues/207

Additional Information

Checklist

[x] My code adheres to the coding and style guidelines of the project.
[x] I have performed a self-review of my own code.
[x] My changes generate no errors/warnings/merge conflicts.

I thought this was part of the gaming dependencies? Maybe I'm mistaken.

@sevu11 Yeah, just took a look, you're spot on.

@prschorn This functionality is already included in the gaming-setup, and as such this PR is redundant.

@prschorn This functionality is already included in the gaming-setup.

Okay, that makes sense, sorry for not seeing it beforehand. Closing this PR.


Added commonjs support

I'm not really familiar with CommonJS, is that compatible with RequireJS?! What is the impact of this PR on users that don't use CommonJS? Are all users that use the main attribute of the package.json using CommonJS?
I think this PR looks good to me though, unlike #301 it doesn't add anything for users that don't use CommonJS.

The index.js file will make the angular module into a CommonJS module, and
exports the name of that angular module. This makes it possible to import
in nodejs/webpack/es6 and so on. For users that don't use CommonJS, the
angular-drag-and-drop-lists.js is unchanged, so it will not make a
difference.
Other modules use the same way to support CommonJS, for example:
https://github.com/danialfarid/ng-file-upload/blob/master/index.js
https://github.com/angular-ui/bootstrap/blob/master/index.js
On Tue, Jan 17, 2017 at 12:51 AM, Marcel Juenemann<EMAIL_ADDRESS>
wrote:

I'm not really familiar with CommonJS, is that compatible with RequireJS?!
What is the impact of this PR on users that don't use CommonJS? Are all
users that use the main attribute of the package.json using CommonJS?
I think this PR looks good to me though, unlike #301
https://github.com/marceljuenemann/angular-drag-and-drop-lists/pull/301
it doesn't add anything for users that don't use CommonJS.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/marceljuenemann/angular-drag-and-drop-lists/pull/350#issuecomment-272987624,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARep5Ldir_LF4SfwP755B7DSDmsWiFaJks5rTAJogaJpZM4KsZy8
.

Thanks! Can you resolve the merge conflict? Then I'll merge the PR

@marceljuenemann @emiluen I've rebased this for you in #388

Hey guys , any trouble to accept this PR ?

any trouble with this PR?


Remember all settings in the Lists tab.

Remember settings like

Keep Folders
Tool Tip Regex
All

Didn't remember following settings, because it probably better to have save defaults for them.

Target Folder
Import Folder
Collections
Copy or Move


Board Review: Microsoft.Azure.WebJobs.Extensions.EventHubs GA

Contacts and Timeline

Responsible service team: ADP
Main contacts: @pakrym @JoshLove-msft

About the Service

Link to documentation introducing/describing the service: https://docs.microsoft.com/en-us/azure/azure-functions/functions-bindings-event-hubs
Link to the service REST APIs: N/A

About the client library

Name of the client library: Microsoft.Azure.WebJobs.Extensions.EventHubs
Languages for this review: .NET

Artifacts required (per language)
.NET

APIView Link: https://apiview.dev/Assemblies/Review/baa01eaa859b4d169a781df9517f80e9
Link to Champion Scenarios/Quickstart samples: https://github.com/Azure/azure-sdk-for-net/blob/master/sdk/eventhub/Microsoft.Azure.WebJobs.Extensions.EventHubs/README.md

scheduled for 2/23

Recording[MS INTERNAL ONLY]


Hooking up bonding/unbonding to wallet

Still needs unbonding, which we can't do yet since the data about which candidates we have bonded to doesn't seem to be working properly in gaia.

This PR is functional now: users can go through the staking flow, restart the app, and then edit their existing delegations. But before we merge, it would be nice to rethink the current UX.
Until now, we made the staking flow only keeping in mind the situation where a user has never bonded before. With the changes in this PR, users can go back to PageBond and see their current delegation amounts, edit them (increasing or decreasing delegations) and hit 'Bond' again. Any amount decreased from previous delegations are now in the unbonding process and aren't liquid for a month.
So basically what I'm thinking is the language in PageBond just needs to be changed to explain about coins going into the unbonding period, and that the amounts aren't the amount being deducted but the balance of bonded coins after the user submits.
@jolesbi @nylira @faboweb Any ideas about how to communicate this clearly?

I think we need to show Atoms in their tristate like: Total [Available, Bonded, Unbonding]

For current error 'Cannot set property '_error' of undefined' merge #270

i think we should merge this (if it's ready) and make another ticket to improve the UX.

This PR is ready

tests for PanelSort too!

Now PageBond has 96% coverage, this PR is ready (sorry for the delay).
I'll revert back to using the cosmos-sdk for the API calls in a later PR (
see #301)


Show due message

Pull Request Test Coverage Report for Build 386

20 of 20 (100.0%) changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 87.903%

Totals

Change from base Build 382:
0.03%

Covered Lines:
2209

Relevant Lines:
2513

💛  - Coveralls


CI: ConformanceEKS: failed to fetch cilium runtime config: error dialing backend: remote error: tls: internal error`

CI failure
⚠  Unable to detect Cilium version, assuming v1.13.2 for connectivity tests: unable to parse cilium version on pod "cilium-7pkkz": unable to fetch cilium version on pod "cilium-7pkkz": error dialing backend: remote error: tls: internal error

This indicates that cilium-cli is unable to talk to the K8s apiserver or kubelet on the node. It seems like the sysdump was able to succeed tough, so this might have been an ephemeral error.
https://github.com/cilium/cilium/actions/runs/5188573844/jobs/9352416199
logs_1565015.zip
Unfortunately the sysdump already expired.

For everybody passing by: I fixed it by disabling my Wireguard client.


RHINENG-8654: Prevent duplicate or empty remediation name during PATCH

The purpose of this is to prevent users from using duplicate or empty/null names on an account basis. This PR updates the API endpoints POST remediations/ and PATCH remediations/{playbook_id} to make sure we're checking for duplicates and null values when these are used.
The parent ticket contains more info on why this is necessary: https://issues.redhat.com/browse/RHINENG-4942
## Secure Coding Practices Checklist GitHub Link
- https://github.com/RedHatInsights/secure-coding-checklist

## Secure Coding Checklist
- [ ] Input Validation
- [ ] Output Encoding
- [ ] Authentication and Password Management
- [ ] Session Management
- [ ] Access Control
- [ ] Cryptographic Practices
- [ ] Error Handling and Logging
- [ ] Data Protection
- [ ] Communication Security
- [ ] System Configuration
- [ ] Database Security
- [ ] File Management
- [ ] Memory Management
- [ ] General Coding Practices

Converted to draft because I'm going to combine this with the changes for the POST operation too: https://issues.redhat.com/browse/RHINENG-8655
There's a lot of overlapping code so I think it makes more sense.

Consider removing this as well: src/remediations/controller.read.js:131

Still in progress but just wanted to push what I have so far. Need to test the migration still and make sure the queries in the migration do what they're supposed to do.

The sequelize model object in src/remediations/models/remediation.js needs to be updated too.

Tests are passing besides 2 tests that are failing intermittently. Looking into this now.


Use kebab-case for evidence path

Proposed changes
What changed
Use kebab-case for evidence path

Why did it change
Issue tracking

PYI-XXXX

Checklists
Environment variables or secrets

[x] No environment variables or secrets were added

Other considerations

[ ] Update README with any new instructions or tasks

Please can we tie this to the issue on JIRA


for Mono use Miguel's components.md text

Build seemed stuck, Closed and reopened to try again. @tdykstra, do we need @terrajobst review before merging?

@mairaw No - already reviewed for the components doc.


Lunr Languages intergration

Hiya
I think this project would be better server as a plugin for
https://github.com/MihaiValentin/lunr-languages/
You can then utilise the power of Lunr 2+ and provide the tokenisation and stemmer required for Chinese. There is already a Japanese version on lunr languages and I know people are after chinese
Tokeniszer
https://github.com/yishn/chinese-tokenizer
Others
http://linfuyan.com/add-chinese-support-to-lunrjs/
https://www.ranks.nl/stopwords/chinese-stopwords
Would be great to assist on this as I need for my website also

Hi @iwarner
Thanks for the info, and it is the best way to integrate with Lunr.js. I have already put it into my to-do list, hope it won't take too long :p

Hi @ole3021
Is there any update about this topic?

use this https://github.com/repairearth/lunr-languages


Feature: by default, ask for confirmation of next date when spawning a new recurrent todo

Rationale: it's actually a bit tricky, especially when the done mark is made before or after the due date. Safer to confirm.

done (but didn't merge the branch right)


Stepwise LR-Scheduler not working across epochs

Bug description
Description
I'm training a model based on number of iterations instead of a number of epochs. The same model trains on datasets of different sizes, hence one epoch differs in the number of iterations. Let's say I want to train e.g. a model for 900 iterations which corresponds to 90 epochs on one of the datasets and want to have a stepwise lr scheduler on iteration 300 & 600. To my understanding this is not natively possible in the pytorch lightning environment.
I know that I can change the lr scheduler interval to "step" and then set the frequency, like so:
'lr_scheduler': {"scheduler": sched,                            "interval": "step",                            "frequency": 300}
However this only applies the steps within one epoch. If I set the frequency larger than the number of iteration per epoch no scheduler step is applied. I would assume that the expected behaviour is to call the scheduler.step() every n frequency across multiple epochs.
What version are you seeing the problem on?
v2_0
How to reproduce the bug
import os

import torch
from torch.utils.data import DataLoader, Dataset
from pytorch_lightning import LightningModule, Trainer

class RandomDataset(Dataset):
    def __init__(self, size, length):
        self.len = length
        self.data = torch.randn(length, size)

    def __getitem__(self, index):
        return self.data[index]

    def __len__(self):
        return self.len

class BoringModel(LightningModule):
    def __init__(self):
        super().__init__()
        self.layer = torch.nn.Linear(32, 2)

    def forward(self, x):
        return self.layer(x)

    def training_step(self, batch, batch_idx):
        loss = self(batch).sum()

        for param_group in self.optimizers().optimizer.param_groups:
            lr = param_group['lr']
        self.log('lr', lr, prog_bar=True, on_step=True, on_epoch=False)
        return {"loss": loss}

    def configure_optimizers(self):
        opt = torch.optim.SGD(self.layer.parameters(), lr=0.1)
        scheduler = torch.optim.lr_scheduler.StepLR(opt, 1)
        return {"optimizer": opt, 'lr_scheduler': {"scheduler": scheduler,
                                                   "interval": "step",
                                                   "frequency": 10}}

def run():
    train_data = DataLoader(RandomDataset(32, 32), batch_size=8)

    model = BoringModel()
    trainer = Trainer(
        accelerator='cpu',
        default_root_dir=os.getcwd(),
        num_sanity_val_steps=0,
        max_epochs=-1,
        max_steps=30,
        log_every_n_steps=1
    )
    trainer.fit(model, train_dataloaders=train_data)

if __name__ == "__main__":
    run()

same problem

I think it is a reasonable ask for the frequency parameter to apply across epoch boundaries. This is an easy change, here in this line of code
https://github.com/Lightning-AI/lightning/blob/af852ff5908e9a99917eeeff05bb4536dbb1cade/src/lightning/pytorch/loops/training_epoch_loop.py#L363C33-L363C33
the self.batch_idx would have to be changed to self.total_batch_idx, that's all. Anyone from the community is free to contribute this change.


unifi6: 6.1.71 -> 6.2.25

Motivation for this change
Update unifi 6 to latest release: https://community.ui.com/releases/UniFi-Network-Application-6-2-25/d9cb0897-3ef4-4dda-a5d6-c07530ff8a86
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.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

[x] Fits CONTRIBUTING.md.

Ah, just noticed this one: would you mind amending your commit message to start with unifi6? 🙂


Object Texture

Describe the feature
Add a texture to the object.

git fetch origin
git checkout 11-object-texture


Running Logshark in command line

Hi ,
I recently came across this tool and wanted to get started . I have successfully installed the standalone Postgres instance and trying to run logshark in command line with full admin rights. I  am using this after opening cmd - C:"C:\Program Files\Logshark\Logshark.exe" but this says No logset target specified.
I tried several combinations like  C:\Program Files\Logshark>Logshark C:\Users\User1\Desktop\Tableau Ziplog\UAT Logs.zip -- plugin Apache.
But this doesn't give me any output file. I am sure that I am missing something here but couldn't get it working as I read the User guide.
I will really appreciate if you can help me with this trivial thing.
Thanks!

Hi simi676,
If there are spaces in the path to your logset, you will need to wrap the path with double quotes.  Can you please try executing the following:
logshark "C:\Users\User1\Desktop\Tableau Ziplog\UAT Logs.zip" --plugins Apache
Thanks for trying Logshark, let me know how it goes!

Thank you  Justin ! It worked like a magic. I will let you know if I face any further problems or need some clarification for understanding. Thanks for this awesome product.

Awesome -- glad to hear it's working for you simi676! 👍 Closing this issue out.


Rating is missing keyboard focus

@NateBaldwinDesign is there a design spec for the keyboard focus for Rating?

There is a design spec, and it was implemented in the prototype @lazd built for this. Larry may have insight into this.

On keyboard focus, the entire control receives the focus ring. The user can arrow left/right to change the value

@GarthDB the CSS contains an is-focused class that applies the focus style. This is not shown in the docs because, in reality, the <input> within the rating gets the focus, though we could add something to enhancements.js to enable it.

It would be nice to have some way to preview the focus state.

Hi,
I am a first-time contributor and would like to work on this issue. Can I start working on this issue, please?

@thnat1234 absolutely. Go ahead and send a PR adding an example with the .is-focused class. Thanks!


Support for MySQL utf8mb4 charset

Just trying to build a new wagtail site on MySQL 5.5 and ran into a few errors on the initial migrations. If the MySQL database is setup for utf8mb4 (MySQL speak for a proper UTF-8 charset) the max length it can handle for a char/varchar index is 191 (since the maximum column size is 767 bytes).
I ran into the error on a few of the initial migrations with Wagtail 1.5.1:

wagtailimages.0001_initial
wagtailcore.0001_squashed_0016
wagtailimages.0004_make_focal_point_key_not_nullable
wagtailimages.0005_make_filter_spec_unique
wagtailcore.0021_capitalizeverbose
wagtailcore.0024_collection
wagtailembeds.0001_initial (having URLFields)
wagtailredirects.0001_initial
wagtailredirects.0004_set_unique_on_path_and_site
wagtailredirects.0005_capitalizeverbose
wagtailsearch.0001_initial

Wordpress last year started converting all of their hosted sites to utf8mb4 (see here), and it's been an outstanding Django issue for quite a while (#18932)
It may not be possible to fix past migration files, but a note in the docs regarding utf8mb4 might be a good idea, and at least consideration to making sure future migrations either limit char/varchar to 191 characters would be helpful. One of the solutions for Django proper is to add a field to let the user specific the max length globally (either 191 or 255). This seems to have been employed by django-allauth and a few other apps as a workaround to this problem.

Thanks for the report! I'm in favour of the idea of adding a setting variable to specify the max length.
I don't think there's anything in Wagtail that specifically depends on a CharField larger than 191 characters - normally, in the cases where people have run into size limits (image filenames, Page.url_path...) we've responded by converting the field into a TextField instead.

Quite welcome - that really seems like the simplest solution for this problem. There really aren't very many fields that would be impacted, and the only people needing the shorter length wouldn't be losing data (since they can't have more stored anyway). Having modified all of the fields in question (and previous migrations), there really doesn't seem to be anything dependent on it. Mostly it's just either key storage or file paths.
MySQL always adds to the fun.
Thanks for the quick reply!

Reviewed today – assigning to @cnk who has sites running utf8mb4, to review whether this is outdated and can be closed – or if not further triage.

Might be related to https://github.com/wagtail/wagtail/issues/9477, and might be fixed by setting the collation in the database settings as per https://github.com/wagtail/wagtail/issues/9477#issuecomment-1823599286


Updating framework versions for reference code

Should this be done? At what cadence?

SWG:
We think should be decided and discussed by the owners. Also, additional reasons could be helpful to understand this. We recommend this is brought up among the owners.

Converting to reference issue.


✨ Create CI workflow that updates contributor and package metadata from this repo [CI infrastructure]

Right now to run this workflow, you have to locally install pyosmeta and run the updates.
It would be nice to update the data periodically via a ci workflow that runs and updates the pyopensci.github.io repository.
the data files live there in
_data
   packages.yml
   contributors.yml

It might be that this includes install this package from github in a action and then running the scripts.
Because the scripts hit an API it might need to somehow be broken up in a way that we don't always hit API limits. so some thought will need to be put into this.
It could be that to start someone works on  brainstorming what might work in a CI environment given API limits.
Then someone else could implement.

this has been completed via the pycon sprint!!


Permissions when running as another user

I encountered an issue with permissions on shared volumes, which eventually boils down to :
$ docker run -it --rm -p 8888:8888 -u 1002:1002 pangeo/base-notebook:latest ls -ltrn
total 40
-rwxr-xr-x 1 1000 1000   303 Mar  7 18:21 start
-rw-r--r-- 1 1000 1000  2993 Mar  7 18:21 packages.txt
-rw-r--r-- 1 1000 1000   166 Mar  7 18:21 environment.yml
-rw-r--r-- 1 1000 1000 21054 Mar  7 18:21 conda-linux-64.lock
-rw-r--r-- 1 1000 1000    30 Mar  7 18:21 Dockerfile

This prevents running the image with another user (1002 here) which is needed to satisfy permissions when accessing shared volumes owned by that (uid != 1000) user
From my understanding, this comes from https://github.com/moby/moby/issues/7198
I am aware of two possible fixes :

https://github.com/boxboat/fixuid
https://github.com/tianon/gosu

I was wondering if someone encountered the same issue ?
If so, what would be the appropriate fix here ?

@asmodehn, yes these images are really intended to run on jupyterhub and as far as I'm aware all users have gid:uid 1000:1000, so our base dockerfile currently hard-codes things like the /home/jovyan home directory and uid 1000.
https://github.com/pangeo-data/pangeo-docker-images/blob/a1a014addeed4deea977fa6d2b37c2ac614ad662/base-image/Dockerfile#L9-L10
https://github.com/pangeo-data/pangeo-docker-images/blob/a1a014addeed4deea977fa6d2b37c2ac614ad662/base-image/Dockerfile#L62
perhaps @consideRatio has some guidance for workarounds or links to relevant jupyterhub issues on this topic. Of course you're welcome to fork this repository and modify the base Dockerfile to suit your needs


[SPARK-38132][SQL] Remove NotPropagation rule

What changes were proposed in this pull request?
This is a follow-up PR to mitigate the bug introduced by SPARK-36665. This PR removes NotPropagation optimization for now until we find a better approach.
Why are the changes needed?
NotPropagation optimization previously broke RewritePredicateSubquery so that it does not properly rewrite the predicate to a NULL-aware left anti join anymore.
Does this PR introduce any user-facing change?
No
How was this patch tested?
Existing tests

cc @aokolnychyi @viirya @cloud-fan @allisonwang-db @HyukjinKwon

For unblocking https://github.com/apache/spark/pull/35395

cc @sunchao too since he was on the original PR.

Thank you @dongjoon-hyun @viirya for the feedback. Added NotInSubqueryEndToEndSuite.scala

Thanks for pinging me. Just checking for my own understanding - so is this technically a revert of SPARK-36665?

Late LGTM. It seems to me that this rule won't make a big deal to the overall performance, so it's better to make it very simple, otherwise it's not worthwhile.

Thanks for pinging me. Just checking for my own understanding - so is this technically a revert of SPARK-36665?

@HyukjinKwon It is actually not a full revert. SPARK-36665 introduced two classes(objects), and reverted only one of them


Pool Resource-type to allot from set

The semaphore mechanism for resource identification should be augmented with a pool-allotment.
The pool token will be injectable into execution wrappers.

Fixed with #1873


Helpful warning messages in functions

E.g. pivot_ requires austraits$traits but if user supplies austraits only perhaps a prompt "pivot_trait_XX only works for the trait data table. Try austraits$traits in the data argument"

this was my comment:

could refer to the website with the trait names


[UT] Enable more test_dot variants after driver update

Build and test A770 for this branch: https://github.com/intel/intel-xpu-backend-for-triton/actions/runs/10041500810/job/27749720480

Thanks Pavel, it passes.


Initial support for a "journeyman" CLI command.

> journey find booga booga

I don't know a skill named "booga booga"

> journey find pulverize

You can learn "Pulverize" after 4 turns in Cobb's Knob Laboratory.

> journey find springy fusilli

You can learn "Springy Fusilli" after 8 turns in The VERY Unquiet Garves (which is not currently accessible to you).

> journey find advanced sauce

You already learned "Advanced Saucecrafting" after 12 turns in The Valley of Rof L'm Fao.

> journey find sing

The "Sing" skill is not available to Journeymen.

> journey find SA saucegeyser

Journeymen Saucerors can learn "Saucegeyser" after 16 turns in Cobb's Knob Laboratory.

> journey find all saucegeyser

Journeymen Seal Clubbers can learn "Saucegeyser" after 24 turns in The Obligatory Pirate's Cove.
Journeymen Turtle Tamers can learn "Saucegeyser" after 16 turns in Infernal Rackets Backstage.
Journeymen Pastamancers can learn "Saucegeyser" after 16 turns in The Haiku Dungeon.
Journeymen Saucerors can learn "Saucegeyser" after 16 turns in Cobb's Knob Laboratory.
Journeymen Disco Bandits can learn "Saucegeyser" after 8 turns in The Dire Warren.
Journeymen Accordion Thiefs can learn "Saucegeyser" after 12 turns in Cobb's Knob Menagerie, Level 1.

I also have "journey zones" which prints a table showing all 180 skills, from 30 zones, each with 6 possible.
These two features are pretty much everything I wished I'd had during my first 3 days of a lazy Journeyman run.
With these, no more looking at spreadsheets or data files to find out where to go for desired skills.
I think this is ready for review.

Any chance of a test covering the command? I think the lines you pasted in the first post would give a decent amount of coverage.

I will do that.

Test Coverage for the "journey find" command is now comprehensive.
None for "journey zones"; not so easy to check content of massive HTML tables.

yes, good job!


instantiateImageCodecWithSize is not disposing intermediates on web

Steps to reproduce

Run tests below on a non-web platform
Tests should pass
Run tests below on web platform
instantiateImageCodecWithSize will fail

Expected results
Both tests should pass on all platforms. instantiateImageCodecWithSize on web is not disposing of the original imagery following conversion to the target size. cc @polina-c - this is similar to an issue I opened previously, but here are tests demonstrating the leak.
Actual results
Both tests pass on non-web platforms. The second test, which evaluates if instantiateImageCodecWithSize is leaking, fails on web.
Code sample
Code sample
import 'dart:typed_data';
import 'dart:ui';
import 'package:flutter_test/flutter_test.dart';

void main() {
  late final Uint8List bytes;
  final loaded = <Image>{};

  setUpAll(() async {
    final image = await createTestImage(width: 500, height: 500);
    final data = await image.toByteData(format: ImageByteFormat.png);
    bytes = data!.buffer.asUint8List();
    image.dispose();
    Image.onCreate = loaded.add;
    Image.onDispose = loaded.remove;
  });

  tearDownAll(() {
    Image.onCreate = null;
    Image.onDispose = null;
  });

  tearDown(loaded.clear);

  test('instantiateImageCodec', () async {
    expect(loaded.length, 0);
    final codec = await instantiateImageCodec(bytes);
    final frame = await codec.getNextFrame();
    expect(loaded.length, 1);
    frame.image.dispose();
    codec.dispose();
    expect(loaded.length, 0);
  });

  test('instantiateImageCodecWithSize', () async {
    expect(loaded.length, 0);
    final codec = await instantiateImageCodecWithSize(
      await ImmutableBuffer.fromUint8List(bytes),
      getTargetSize: (_, __) => const TargetImageSize(width: 250, height: 250),
    );
    final frame = await codec.getNextFrame();
    expect(loaded.length, 1);
    frame.image.dispose();
    codec.dispose();
    expect(loaded.length, 0);
  });
}

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
No response
Flutter Doctor output
Doctor output
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 3.19.5, on Microsoft Windows [Version 10.0.19044.4170], locale en-US)
[√] Windows Version (Installed version of Windows is version 10 or higher)
[√] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
[√] Chrome - develop for the web
[√] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.3.5)
[√] Android Studio (version 2021.3)
[√] VS Code (version 1.88.1)
[√] Connected device (3 available)
[√] Network resources

• No issues found!

Thanks for the report. I was able to replicate the reported behavior on web.

 stable, master flutter doctor -v
[!] Flutter (Channel stable, 3.19.5, on macOS 12.2.1 21D62 darwin-x64, locale
    en-GB)
    • Flutter version 3.19.5 on channel stable at
      /Users/dhs/documents/fluttersdk/flutter
    ! Warning: `flutter` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/flutter, which is not inside
      your current Flutter SDK checkout at
      /Users/dhs/documents/fluttersdk/flutter. Consider adding
      /Users/dhs/documents/fluttersdk/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/dhs/documents/fluttersdk/flutter.
      Consider adding /Users/dhs/documents/fluttersdk/flutter/bin to the front
      of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 300451adae (4 days ago), 2024-03-27 21:54:07 -0500
    • Engine revision e76c956498
    • Dart version 3.3.3
    • DevTools version 2.31.1
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

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

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

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

[!] Flutter (Channel master, 3.22.0-14.0.pre.32, on macOS 12.2.1 21D62
    darwin-x64, locale en-GB)
    • Flutter version 3.22.0-14.0.pre.32 on channel master at
      /Users/dhs/documents/fluttersdk/flutter
    ! Warning: `flutter` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/flutter, which is not inside
      your current Flutter SDK checkout at
      /Users/dhs/documents/fluttersdk/flutter. Consider adding
      /Users/dhs/documents/fluttersdk/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/dhs/documents/fluttersdk/flutter.
      Consider adding /Users/dhs/documents/fluttersdk/flutter/bin to the front
      of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision d4b42780ba (2 hours ago), 2024-04-19 00:21:25 -0400
    • Engine revision b6234dd198
    • Dart version 3.5.0 (build 3.5.0-76.0.dev)
    • DevTools version 2.35.0-dev.8
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/dhs/Library/Android/sdk
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for
      more details.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 13C100
    • CocoaPods version 1.11.2

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

[✓] IntelliJ IDEA Ultimate Edition (version 2021.3.2)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 65.1.4
    • Dart plugin version 213.7228

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

[✓] Connected device (3 available)
    • Darshan's iphone (mobile) • 21150b119064aecc249dfcfe05e259197461ce23 • ios
      • iOS 15.3.1 19D52
    • macOS (desktop)           • macos                                    •
      darwin-x64     • macOS 12.2.1 21D62 darwin-x64
    • Chrome (web)              • chrome                                   •
      web-javascript • Google Chrome 109.0.5414.119

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

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

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

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.


Error: Cannot read property '0' of null On Piped Results

Version: 4.0.0
OS: Win 10 X64
Node: 6.10.1
In the process of migrating to 4.0.0.  We are successfully retrieving results back via pipe, however we very quickly get a TypeError Cannot read property '0' of null.
It appears the recordsets is null @node_modules\mssql\lib\base.js:1253
recordset: recordsets[0],
Simple Usage Example:
sql.connect(config)
.then(pool => {
  request = pool.request();
  request.query(query(date));
  return request.pipe(new CustomStream());
})
.then(stream => {
 doStuffWithStream(stream);
})

Error:
TypeError: Cannot read property '0' of null
    at _query (...\node_modules\mssql\lib\base.js:1253:32)
    at Request.tds.Request.err [as userCallback] (...\node_modules\mssql\lib\tedious.js:614:15)
    at Request._this.callback (...\node_modules\tedious\lib\request.js:47:27)
    at Connection.message (...\node_modules\tedious\lib\connection.js:1401:27)
    at Connection.dispatchEvent (...\node_modules\tedious\lib\connection.js:687:45)
    at MessageIO.<anonymous> (...\node_modules\tedious\lib\connection.js:602:18)
    at emitNone (events.js:86:13)
    at MessageIO.emit (events.js:185:7)
    at ReadablePacketStream.<anonymous> (...\node_modules\tedious\lib\message-io.js:102:16)
    at emitOne (events.js:96:13)
    at ReadablePacketStream.emit (events.js:188:7)
    at readableAddChunk (...\node_modules\tedious\node_modules\readable-stream\lib\_stream_readable.js:217:18)
    at ReadablePacketStream.Readable.push (...\node_modules\tedious\node_modules\readable-stream\lib\_stream_readable.js:176:10)
    at ReadablePacketStream.Transform.push (...\node_modules\tedious\node_modules\readable-stream\lib\_stream_transform.js:123:32)
    at ReadablePacketStream._transform (...\node_modules\tedious\lib\message-io.js:70:16)
    at ReadablePacketStream.Transform._read (...\node_modules\tedious\node_modules\readable-stream\lib\_stream_transform.js:159:10)
    at ReadablePacketStream.Transform._write (...\node_modules\tedious\node_modules\readable-stream\lib\_stream_transform.js:147:83)
    at doWrite (...\node_modules\tedious\node_modules\readable-stream\lib\_stream_writable.js:345:64)
    at writeOrBuffer (...\node_modules\tedious\node_modules\readable-stream\lib\_stream_writable.js:334:5)
    at ReadablePacketStream.Writable.write (...\node_modules\tedious\node_modules\readable-stream\lib\_stream_writable.js:271:11)
    at Socket.ondata (_stream_readable.js:555:20)
    at emitOne (events.js:96:13)
    at Socket.emit (events.js:188:7)
    at readableAddChunk (_stream_readable.js:176:18)
    at Socket.Readable.push (_stream_readable.js:134:10)
    at TCP.onread (net.js:551:20)

Thank you for reporting this, fixed in 4.0.1.

Great thanks again @patriksimek!


Don't import modules from devDependencies in app/ code

Fixes https://github.com/mozilla/tofino/issues/420

LGTM


DSM protocol fix

Passes unit tests, needs HW test.

Tested successfully with DX9.

@kd0aij This fixes DSM.


Better support freeform and pasting into the simple file picker

Context: https://github.com/microsoft/vscode/issues/121280
Things to support:

Pasting a full path into the input box

Items should update
If a file is included in the path it should be selected.

Better handling of invalid paths

Support ~ as an "escape hatch" from bad paths

Thanks - just clarifying one element:

Items should update

That update should check for non-existent directories, including (apparent) sub-directories and files in non-existent directories - that is certainly not working for files yet:

Removing the items shown for non-existent directories should also fix the incorrect highlight of dir.tmp (which resides in /home/bers!) at /home/bers/this/d.

Another thing to fix:

In this example, I am using Remote SSH to work on a remote Linux server. Starting from the original path,
/tmp/sub/sub/sub/sub/sub/sub/sub/sub/sub/sub/sub/sub/sub/sub/sub/sub, I do nothing else but hold Ctrl (on my Windows client keyboard) and hammer backspace. Two things are of note:

Stuff keeps reappearing
Even if not, I need two backspaces to remove one sub/ (one for /, one for sub).

Here's another one, potentially opening the wrong file when pasting a full path:

Remote SSH/WSL into a Linux box.
Open /tmp as a folder in VS Code

mkdir /tmp/bar
echo 1 > /tmp/foo
echo 2 > /tmp/bar/foo

Put the file name "/tmp/bar/foo" into clipboard

Ctrl-O - note that /tmp/ is prefilled.

Ctrl-A, V to paste "/tmp/bar/foo", Enter

File /tmp/foo is opened instead of /tmp/bar/foo.

Pasting already works, just need to add the ~ escape hatch.


Fix for expo autolinking

Expo SDK 44 requires importing react headers with #import <React/*.h> due to swift compatibility reasons.
For more information: https://github.com/expo/expo/issues/15622#issuecomment-997141629
You can see people changing this all over the place:

https://github.com/expo/expo/pull/15299
https://github.com/expo/expo/issues/15622
https://github.com/RevenueCat/react-native-purchases/pull/316

I think this will still work, I confirmed example still builds & seems to work (I am seeing a RCTBridge required dispatch_sync to load RCTDevLoadingView. This may lead to deadlocks warning but I think that is unrelated and was happening before?).
Will also update once I import this version and see if it fixes my build errors in my actual app.

merged thanks @imownbey


Add a lockable quickhook retainer

I like quickhooks but wish they would stay in place better.
This is an attempt at addressing with the addition of a new retainer. It is compatible with all quickhooks (it's the same geometry, just wider).
This extra retainer has an orientation and it must be aligned with a panel hole.
To secure the retainer to the panel, a small peg is used. It is removable without the need for tools (it uses a snap fit design).
The two new STLs are oriented in their ideal print direction.
I can provide a Fusion360 file for the source if that's worthy of a contribution.

That looks great!  Ready to merge this when you are, I bet some folks would love this.

Related to your comment regarding a rubber mallet in your README update, I found a $6 jeweler's hammer from Harbor Freight works great for final assembly. I'll use vice grip pliers initially to try and set a pin, then just set on a flat surface to hammer down using the nylon side. Also works well to lock in pieces on the board itself if it's a really tight fit.
https://www.harborfreight.com/hand-tools/hammers-pry-bars/mallets/jewelry/double-sided-mallet-with-wooden-handle-98285.html

@aderusha Ready to merge! Added the source file for posteriority.


Support Elixir 1.7 Logger

The logger is back and pretty awesome. Can we hook into the Logger.Translate?

For what is worth https://github.com/ForzaElixir/rollbax/issues/126#issuecomment-2327219615.


InvalidOperationException: IDX10803: Unable to obtain configuration from: 'https://DevIDPServer/sts/Metadata.xml'

I get this error when start the application. it is not able to complete the configuration from Federation Metadata. if I enter the metadata url on the browser I am able to see the xml, but only issue I see is that there is  a certificate error, which I override on the browser. would that be an issue causing this?
my config:
` public void ConfigureServices(IServiceCollection services)
{
services.AddMvc();
services.AddAuthentication(sharedOptions =>
{
sharedOptions.DefaultScheme = CookieAuthenticationDefaults.AuthenticationScheme;
sharedOptions.DefaultSignInScheme = CookieAuthenticationDefaults.AuthenticationScheme;
sharedOptions.DefaultChallengeScheme = WsFederationDefaults.AuthenticationScheme;
})
.AddWsFederation(options =>
{
options.Wtrealm = @"http://localhost:3230/";// Configuration["wsfed:realm"];
options.MetadataAddress = @"https://DevIDPServer/sts/Metadata.xml";// Configuration["wsfed:metadata"];
})
.AddCookie();
    }

`
 app.UseAuthentication();
----------------------Errors-----------------------------
System.InvalidOperationException: IDX10803: Unable to obtain configuration from: 'https://DevIDPServer/sts/Metadata.xml'. ---> Microsoft.IdentityModel.Xml.XmlReadException: IDX13004: Security token type role descriptor is expected.    at Microsoft.IdentityModel.Protocols.WsFederation.WsFederationMetadataSerializer.ReadEntityDescriptor(XmlReader reader)    at Microsoft.IdentityModel.Protocols.WsFederation.WsFederationMetadataSerializer.ReadMetadata(XmlReader reader)    at Microsoft.IdentityModel.Protocols.WsFederation.WsFederationConfigurationRetriever.<GetAsync>d__4.MoveNext() --- End of stack trace from previous location where exception was thrown ---    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at System.Runtime.CompilerServices.ConfiguredTaskAwaitable1.ConfiguredTaskAwaiter.GetResult()
at Microsoft.IdentityModel.Protocols.ConfigurationManager1.<GetConfigurationAsync>d__24.MoveNext()    --- End of inner exception stack trace ---    at Microsoft.IdentityModel.Protocols.ConfigurationManager1.d__24.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.AspNetCore.Authentication.WsFederation.WsFederationHandler.d__3.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.AspNetCore.Authentication.AuthenticationHandler1.<ChallengeAsync>d__53.MoveNext() --- End of stack trace from previous location where exception was thrown ---    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at Microsoft.AspNetCore.Authentication.AuthenticationService.<ChallengeAsync>d__11.MoveNext() --- End of stack trace from previous location where exception was thrown ---    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at Microsoft.AspNetCore.Mvc.ChallengeResult.<ExecuteResultAsync>d__14.MoveNext() --- End of stack trace from previous location where exception was thrown ---    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at Microsoft.AspNetCore.Mvc.Internal.ResourceInvoker.<InvokeResultAsync>d__19.MoveNext() --- End of stack trace from previous location where exception was thrown ---    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at Microsoft.AspNetCore.Mvc.Internal.ResourceInvoker.<InvokeFilterPipelineAsync>d__17.MoveNext() --- End of stack trace from previous location where exception was thrown ---    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at Microsoft.AspNetCore.Mvc.Internal.ResourceInvoker.<InvokeAsync>d__15.MoveNext() --- End of stack trace from previous location where exception was thrown ---    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at Microsoft.AspNetCore.Builder.RouterMiddleware.<Invoke>d__4.MoveNext() --- End of stack trace from previous location where exception was thrown ---    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at Microsoft.AspNetCore.Diagnostics.DeveloperExceptionPageMiddleware.<Invoke>d__7.MoveNext()

oops..looks like I posted it in the wrong place...

@brentschmaltz any idea why you'd get a XmlReadException for an https failure?

@Tratcher  thank you for your response.
I added the BackchannelHttpHandler,  but that did not make any difference, I get the same error. It does not look an issue due to the certificate error. it show a different error related security when I return false in the ServerCertificateCustomValidationCallback
this is the new code:
`public void ConfigureServices(IServiceCollection services)
{
services.AddMvc();
services.AddAuthentication(sharedOptions =>
{
sharedOptions.DefaultScheme = CookieAuthenticationDefaults.AuthenticationScheme;
sharedOptions.DefaultSignInScheme = CookieAuthenticationDefaults.AuthenticationScheme;
sharedOptions.DefaultChallengeScheme = WsFederationDefaults.AuthenticationScheme;
})
.AddWsFederation(options =>
{
options.Wtrealm = @"http://localhost:3230/";// Configuration["wsfed:realm"];
options.MetadataAddress = @"https://DevIDPServer/sts/Metadata.xml";// Configuration["wsfed:metadata"];
options.BackchannelHttpHandler = new HttpClientHandler() {
ServerCertificateCustomValidationCallback =
(req, cert, er, t) =>
true
};
})
.AddCookie();
    }

`
I created a test 4.5 app pointing to the same IDP and it works without any issue on the same machine. here is my web config on the 4.5 machine. We are using SecureAuth as our identity provider.
<system.identityModel>     <identityConfiguration>       <audienceUris>         <add value="http://localhost/testAuth/" />       </audienceUris>       <certificateValidation certificateValidationMode="None" />       <issuerNameRegistry type="System.IdentityModel.Tokens.ConfigurationBasedIssuerNameRegistry, System.IdentityModel, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089">         <trustedIssuers>           <add thumbprint="22 33 s3 57 cd aa d3 a3 07 ea 32 7s e6 b9 ec e3 84 7d 91 34" name="https://DevIDPServer/sts/" />         </trustedIssuers>       </issuerNameRegistry>     </identityConfiguration>   </system.identityModel>   <system.identityModel.services>     <federationConfiguration>       <cookieHandler requireSsl="false" hideFromScript="true" persistentSessionLifetime="1:2:0" />       <wsFederation passiveRedirectEnabled="true" persistentCookiesOnPassiveRedirects="true" issuer="https://DevIDPServer/sts/" realm="http://localhost/testAuth/" reply="http://localhost/testAuth/" requireHttps="false" />     </federationConfiguration>   </system.identityModel.services>

@arc2018 can you post the wsFed metadata xml.
This statement
Microsoft.IdentityModel.Xml.XmlReadException: IDX13004: Security token type role descriptor is expected.
indicates the xml is not as expected. We are expecting at least one 'RoleDescriptor'

@brentschmaltz   thank you for your help. I am attaching metadata.xml
Metadata_xml.txt

@arc2018 the issue is our code expects to find a <RoleDescriptor xsi:type="fed:SecurityTokenServiceType"  ... which is where we will look for keys.
We could relax this, as it's optional

fixed commit https://github.com/AzureAD/azure-activedirectory-identitymodel-extensions-for-dotnet/commit/79be3ea38fc5f5e18c4f4e4a90857162f456ebb8

@brentschmaltz I'm also getting this error: IDX13004: Security token type role descriptor is expected. Is there an updated build I can grab to test your fix? Thanks.

Is it possible for me to add a <RoleDescriptor> tag to my metadata.xml file to bypass the Security token type role descriptor is expected. error I'm getting until a fix is released?

@arc2018 were you able to get your implementation working with the preview2 release?: https://www.nuget.org/packages/Microsoft.AspNetCore.Authentication.WsFederation/
I'm able to get past the Security token type role descriptor is expected. error now, but still not successful in getting a cookie.
My 4.5 web.config looked similar to yours with the persistentCookiesOnPassiveRedirects="true" and  passiveRedirectEnabled="true". I'm using PortalGuard. I didn't know if/how I need to account for those in a .NET Core implementation.
4.5 web.config
<system.identityModel>
	<identityConfiguration>
		<securityTokenHandlers>
			<remove type="System.IdentityModel.Tokens.SessionSecurityTokenHandler, System.IdentityModel, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=abc123" />
			<add type="System.IdentityModel.Services.Tokens.MachineKeySessionSecurityTokenHandler, System.IdentityModel.Services, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=abc123" />
			<add type="System.IdentityModel.Tokens.Saml2SecurityTokenHandler, System.IdentityModel, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=abc123">
				<samlSecurityTokenRequirement mapToWindows="false"></samlSecurityTokenRequirement>
			</add>
		</securityTokenHandlers>
		<audienceUris>
			<add value="https://auth.domain.com/" />
		</audienceUris>
		<issuerNameRegistry type="System.IdentityModel.Tokens.ConfigurationBasedIssuerNameRegistry, System.IdentityModel, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089">
			<trustedIssuers>
				<add thumbprint="abc123" name="https://saml.domain.com:1234/sso/" />
			</trustedIssuers>
		</issuerNameRegistry>
	</identityConfiguration>
</system.identityModel>
<system.identityModel.services>
	<federationConfiguration>
		<wsFederation persistentCookiesOnPassiveRedirects="true"passiveRedirectEnabled="true" issuer="https://saml.domain.com:1234/sso/go.ashx" realm="https://auth.domain.com/" requireHttps="false" />
		<cookieHandler requireSsl="false" path="/" domain="domain.com" persistentSessionLifetime="0:8:0:0"></cookieHandler>
	</federationConfiguration>
</system.identityModel.services>

metadata.xml
<EntityDescriptor xmlns="urn:oasis:names:tc:SAML:2.0:metadata" entityID="https://saml.domain.com:1234/idp/identifier">
	<IDPSSODescriptor protocolSupportEnumeration="urn:oasis:names:tc:SAML:2.0:protocol">
		<KeyDescriptor use="signing">
			<KeyInfo xmlns="http://www.w3.org/2000/09/xmldsig#">
				<X509Data>
					<X509Certificate>abc123=</X509Certificate>
				</X509Data>
			</KeyInfo>
		</KeyDescriptor>
		<NameIDFormat>
urn:oasis:names:tc:SAML:1.1:nameid-format:emailAddress
		</NameIDFormat>
		<NameIDFormat>
urn:oasis:names:tc:SAML:2.0:nameid-format:persistent
		</NameIDFormat>
		<SingleSignOnService Binding="urn:oasis:names:tc:SAML:2.0:bindings:HTTP-Redirect" Location="https://saml.domain.com:1234/sso/go.ashx"/>
		<SingleSignOnService Binding="urn:oasis:names:tc:SAML:2.0:bindings:HTTP-POST" Location="https://saml.domain.com:1234/sso/go.ashx"/>
	</IDPSSODescriptor>
	<ContactPerson contactType="support"/>
</EntityDescriptor>

.NET Core Startup.cs
services.AddAuthentication(sharedOptions =>
{
	sharedOptions.DefaultScheme = CookieAuthenticationDefaults.AuthenticationScheme;
	sharedOptions.DefaultSignInScheme = CookieAuthenticationDefaults.AuthenticationScheme;
	sharedOptions.DefaultChallengeScheme = WsFederationDefaults.AuthenticationScheme;
})
.AddWsFederation(options =>
{
	options.Wtrealm = "https://auth.domain.com";
	options.MetadataAddress = "https://saml.domain.com:1234/sso/metadata.xml";
})
.AddCookie(options =>
{
	//options.LoginPath = "/api/login"; // not sure if needed?
	//options.LogoutPath = "/api/logout"; // not sure if needed?
});

However, when I load my project, I get a 404 and the URL is: https://auth.domain.com/myappdirectory?wtrealm=https%3A%2F%2Fauth.domain.com%2F&wa=wsignin1.0&wreply=https%3A%2F%2Fauth.domain.com%2Fmyappdirectory%2Fsignin-wsfed&wctx=GUIDHere
The logs have the following warnings:

warn: Microsoft.AspNetCore.DataProtection.Repositories.EphemeralXmlRepository[50]
Using an in-memory repository. Keys will not be persisted to storage.
warn: Microsoft.AspNetCore.DataProtection.KeyManagement.XmlKeyManager[59]
Neither user profile nor HKLM registry available. Using an ephemeral key repository. Protected data will be unavailable when application exits.
warn: Microsoft.AspNetCore.DataProtection.KeyManagement.XmlKeyManager[35]
No XML encryptor configured. Key {12076a57-7f06-4639-9865-965be6d5df32} may be persisted to storage in unencrypted form.
The sign-out redirect URI [url here] is malformed


⚠ Samsung has degraded performance

In 20dba3c, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2373 ms

Resolved: Samsung performance has improved in be2caf7.


Unmanage spec_helper_acceptance.rb until it gets fixed in modulesync_config

The spec_helper_acceptance.rb template is currently missing from modulesync_config. The short term workaround seems to be to unmanage that file in .sync.yml until we can get it added back to modulesync.

Hi. I'm going to close this Issue. We deprecated the module and the new version is available here: https://forge.puppet.com/puppetlabs/iis