upgrade spring security version 5.8.9

prepare for spring boot upgrade 3.x
see https://docs.spring.io/spring-security/reference/5.8/migration/index.html

Hi @strehle, I interpret this PR as doing the following 2 things:

allow us to explicitly define the versions of springframework and spring-security related libraries (before, it was just whatever versions springboot provides)
upgrade spring-security to 5.8.9.

Could you help me understand why these 2 changes are "[preparing] for spring boot upgrade 3.x"? Are we trying to upgrade spring-security before the springboot upgrade?

@peterhaochen47 yes, this PR includes  https://github.com/cloudfoundry/uaa/pull/2659
Reason:

https://github.com/cloudfoundry/uaa/pull/2659 is simply a preparation for the time where we have no spring boot updates anymore, but we will have spring framework and security updates, e.g. https://github.com/spring-projects/spring-framework/milestone/343 , same for spring security.

this update from 5.7x to 5.8.x is an extra PR to have a discussion about the step of upgrade to spring security to 5.8.x before going to spring boot 3. see https://docs.spring.io/spring-security/reference/5.8/migration/index.html#_update_to_spring_security_5_8 . the spring security update is no pre-requisit but I would do it as being prepared simply

@strehle I see. Makes sense. So I suppose that #2659 would be a temporary measure, and after spring boot 3 upgrade we can reverse this refactor (since we'll be getting the spring security updates from spring boot 3).


docs: add Ts.ED example usage

Hello :)
This PR is here to add the MikroOrm integration with our framework Ts.ED. Our integration is really recent but we'll improve the module and add more feature in future!
Tell me if any change is required :)
See you and great job for your ORM!
Romain

Hello @B4nan
thank for your review, I'll take a time to discuss with the author of this plugin to get your feedback.

i dont like that you alias the Property decorator to Column, we are in the MikroORM docs, so please alias your property decorator and keep the ORM one as is to reduce confusion

I don't totally agree, because this decorator is also necessary to properly take advantage of the integration of our two frameworks. We are precisely on a page that talks about an integration between Ts.ED and mikro ORM.
It's as if on my documentation page, I presented our annotations without describing yours. (https://tsed.io/tutorials/mikroorm.html)
In our example, there is nothing confusing because the decorators are clearly imported via their original packages.
We can add split the example in two part, if it's fine for your:

To begin, we need to define an Entity MikroORM like this:

import {Entity, Property, PrimaryKey, Property as Column} from "@mikro-orm/core";

@Entity()
export class User {
  @PrimaryKey()
  id: number;

  @Column()
  firstName: string;

  @Column()
  lastName: string;

  @Column()
  age: number;
}

We can also add Ts.ED decorators in order to reuse the model at controller level (mapping, validation and generation of the OS3 Spec aka Swagger):
import {Property, MaxLength, Required} from "@tsed/common";
import {Entity, Property, PrimaryKey, Property as Column} from "@mikro-orm/core";

@Entity()
export class User {
  @PrimaryKey()
  @Property()
  id: number;

  @Column()
  @MaxLength(100)
  @Required()
  firstName: string;

  @Column()
  @MaxLength(100)
  @Required()
  lastName: string;

  @Column()
  @Mininum(0)
  @Maximum(100)
  age: number;
}

would be good to verify things work with v5, the release will come soon so if we should add things like this to the docs, they should all work with v5 already

I'm not a Mikro ORM user so I don't know what are the change about the v5. I let the plugin author to answer to this point.
But I think it's not a blocker point to merge this PR (after fixing other points).
@derevnjuk I think the @Connection should be replaced by @Orm decorator. As @B4nan explained, it's not a connection in the sense that we mean it
I'll fix the other points :)
See you
Romain

I don't totally agree, because this decorator is also necessary to properly take advantage of the integration of our two frameworks. We are precisely on a page that talks about an integration between Ts.ED and mikro ORM.

I understand your point of view, but this is MikroORM docs we are talking about, so aliasing its decorators is a no-go for me - this would be the only place where we do such thing. My point is you should alias the Ts.ED decorator, as in the context of MikroORM docs, it is a 3rd party dependency.

In our example, there is nothing confusing because the decorators are clearly imported via their original packages.

If you are looking at MikroORM docs - where every single page except this one would use @Property decorator as something else - with different method signature and different options - to me that would be very confusing.

Ha ok you have a property decorator.
I didn't understand that you were following an identical decorator and why were talking about alias. and for information it’s not at all an alias to your decorator, its function is specific to Ts.ED to describe a jsonschema. ok now I get the point.
Required/MaxLength are a problem also?

and for information it’s not at all an alias to your decorator, its function is specific to Ts.ED to describe a jsonschema. ok now I get the point.

I know it is not alias for my decorator. I am talking about aliasing my decorator when importing it. import { Property as Column } from '@mikro-orm/core' - that is what I don't like. You should alias your Ts.ED decorator this way as we are in the context of MikroORM docs here.

haaa ok, really sorry for this confusing things, I haven't written the original example, and I haven't seen that.
My bad. I'm sorry for thank. Ok I fix the example.

Ok, I pushed all mandatory fixes:

[x] missing link from sidebar
please follow the code style this project uses (e.g. spaces in imports, trailing commas, single quotes ...)
[x] i would very much like to not suggest using __dirname in folder based discovery, it can easily break things
[x] i would rather have orm property than connection in that example, this "typeorm style" seems wrong, it is not a connection instance in the mikro orm context. (https://github.com/tsedio/tsed/pull/1742  is merged on our side)
[x] i dont like that you alias the Property decorator to Column, we are in the MikroORM docs, so please alias your property decorator and keep the ORM one as is to reduce confusion
[ ] would be good to verify things work with v5, the release will come soon so if we should add things like this to the docs, they should all work with v5 already

For the last point, there is migration note somewhere ? But regarding the Mikro ORM code used in this module, the risk seems to be limited. You can see the code here: https://github.com/tsedio/tsed/tree/production/packages/orm/mikro-orm/src/services . As I understand, the module create the Mikro ORM instance and provide a way to inject Orm/Em instance.
Tell me if you have more details for me or for @derevnjuk :)
See you
Romain

No issues have been encountered while testing this with v5.0.0-rc.0, especially when we speak about the main functionality. However, I have noticed you changed the fork method signature, which might lead to unexpected behavior while using the @Transactional() decorator.
Before:
orm.em.fork(true, true);

After:
orm.em.fork({ clear: true, useContext: true });

I will try to figure out how to bypass this.
@Romakita @B4nan fyi

Hello @B4nan
I hope your are fine. @derevnjuk and I, have done all requested changes on this PR :).
This PR will support mikro-orm v5 officially: https://github.com/tsedio/tsed/pull/1950
The documentation is up to date!
We should be good and waiting for your last ones if there are any :-)
See you
Romain


inset defaults by Plot.bin

The docu says:

Plot.binX changes the defaults for insetLeft and insetRight; Plot.binY changes the defaults for insetTop and insetBottom; Plot.bin changes all four.

If I am not mistaken it seems that it should be:

Plot.binX changes the default for insetLeft to 1 pixel; Plot.binY changes the default for insetTop to 1 pixel; Plot.bin changes insetLeft and insetTop to 1 pixel each.

In https://github.com/observablehq/plot/blob/068f8d8125a910ee1a528c8b299a79c27ca1a9a5/src/transforms/bin.js#L237 the defaults for (insetLeft, insetRight) and for (insetTop, insetBottom) where applicable, become either [1, 0] or [0.5, 0.5]; it's always a total inset of 1px but it might be distributed differently depending on the screen's DPR (https://github.com/observablehq/plot/blob/main/src/style.js#L3).
Reference commit https://github.com/observablehq/plot/pull/246/commits/154757f09c357a75347dd3735767207881720dcc


Ensure defaults are passed along for multiselect arrays when minItems…

… present (#1258)
Reasons for making this change
When minItems is present for multi select array fields, the default value returned was always an empty array.
Addresses issue #1258
Checklist

[ ] I'm updating documentation

[ ] I've checked the rendering of the Markdown text I've added

[X] I'm adding or updating code

[X] I've added and/or updated tests
[ ] I've updated docs if needed
[X] I've run npm run cs-format on my branch to conform my code to prettier coding style

[ ] I'm adding a new feature

[ ] I've updated the playground with an example use of the feature

@fsteger thanks for the PR -- can you add a test for the case in which default is an array of items (such as the example in issue #1258)? It looks like the test you added is a duplicate of an existing test.

Thanks!


Perform content negotiation

As per https://jsonapi.org/format/#content-negotiation

Currently working on this... how far should it go? Test for all the server responsibilities? What about the slightly tweaked 1.1 version?


使用的时候一直有这个警告，我该怎么办呢？Warning: Cannot update during an existing state transition (such as within render or another component's constructor). Render methods should be a pure function of props and state; constructor side-effects are an anti-pattern, but can be moved to componentWillMount.

本组件的本意就是不需要也不应该在render里使用


Current windows images do not allow the use of process isolation

In order to use process isolation in windows, the windows base image version number must match the version number of the container of the host. If this is not the case, only hyper-v isolation is possible, which is slower.
With only having one image available on 1809 you are forcing developers to choose between hyper-v mode only for most versions of windows or to deviate and build their own container.
I have submitted a pull request to create the docker files for each major current update of windows, this would need to be merged and the build process changed to build each image.

@cardinal252 Am I wrong to think that this issue is now solved ?

@mloiseleur I don't use traefik any more (windows or linux) it kept taking over the windows network stack entirely, so dropped it unfortunately.
I had not seen anything to suggest it had been when I last did in c 2022. Would need to talk to developers of a product like Sitecore to see if they still use it. I would suggest though - unless windows images other than 1809 are being built, it would not be solved. Happy to continue to assist in any way I can?

@cardinal252 Nowadays, images are built on LTS Server 2022

Truthfully - I haven't looked at process isolation compatibility for a long while on windows docker now - it would have to be chased down with a current windows based developer or from the MS documentation. I suspect just the 2 builds will limit it to hyper-v isolation mode for many use cases.

The documentation comes with a quite clear table:
On Windows Server 2019, one need to use 2019 image for process isolation
On Windows Server 2022, one need to use 2022 image for process isolation

Since official images only follow LTS version of Windows Server, I'll close this issue.
Feel free to re-open it if needed.


GEOMESA-1063 Load feature types and sfts from providers and expose API

Signed-off-by: Andrew Hulbert<EMAIL_ADDRESS>For comment...loading config and SFT from a URL

license headers failed the build...

FYI before this is merged i'm going to build it and test it with Nifi

looks good

@jahhulbert-ccri ready for merge?

cf3da940fa2891e8bc94a4a3b43b0acd7730444f


Add more statistics in SVG file

The feature
It would be nice to be able to see how many of each type of contact points are currently available.
Why?
22335 contactpoints  (as of 19/1/2024) it is just a number...  ¯|( ͡° ͜ʖ ͡°)/¯
Examples and inspiration
for example, a list (preferably sorted by the number of contact points)
Facebook: 2449 accounts
X/Twitter: 1902 accounts
Instagram: 1385 accounts
YouTube: 1277 accounts
LinkedIn: 1212 accounts
....
Fun fact those numbers are the actual numbers. I scraped the country pages. ;-)

Before improving the SVG, we should perhaps figure out how to use it. Currently, it is unused due to it was hard to get it to function well on mobiles.

The SVG is currently in use over in the Github profile: https://github.com/govdirectory

This is rather silly on my part.... In a weird brain twister, I thought the website stats were also an SVG. Apparently not 🤦🏼♂. So this is about the statistics page on the website.


Broken Link to AzCopy 7.3

The link to AzCopy 7.3 in the note below is broken.

If you want to copy data to and from your Azure Table storage service, then install AzCopy version 7.3.

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 15911160-5ba2-fb0a-0b24-16b3857e20ce
Version Independent ID: d925bc9f-f825-2f82-168c-6c30d716b9ce
Content: Copy or move data to Azure Storage by using AzCopy v10
Content Source: articles/storage/common/storage-use-azcopy-v10.md
Service: storage
Sub-service: common
GitHub Login: @normesta
Microsoft Alias: normesta

@ThomasPe
Thanks for your feedback! We will investigate and update as appropriate.

@ThomasPe
Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can take a look at it accordingly.

@ThomasPe  I was able to download the Azcopy 7.3 version. The article is UpToDate.   If the issue still persist, please try in different browser  Find where your browser is saving downloads
This article provides all version of azcopy download's files: https://github.com/Azure/azure-storage-azcopy/releases
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Publish children before the main object when new object is added

This eliminates the annoying moment on the client in which the main
object's data is inconsistent because its children objects are not
available yet.

GitHub won't allow me to merge this for some reason, but I've merged manually.


Updated AWS EKS regions to include eu-west-1

This update uses EKS worker AMI v23 for eu-west-1 (Ireland).
Other regions were not updated since the whole cluster would require an update for all worker nodes before adding new ones.
See note from Amazon:
https://docs.aws.amazon.com/eks/latest/userguide/eks-optimized-ami.html

Important
These AMIs require the latest AWS CloudFormation worker node template.
You cannot use these AMIs with a previous version of the worker node
template; they will fail to join your cluster. Be sure to upgrade any
existing AWS CloudFormation worker stacks with the latest template
(URL shown below) before you attempt to use these AMIs.

https://amazon-eks.s3-us-west-2.amazonaws.com/cloudformation/2018-08-30/amazon-eks-nodegroup.yaml

This is the same as #82.


New style im2col

New style im2col to support double backprop.

Rebased on top of master and adjusted tolerances.

LGTM, will merge after CI passes!


Debian install dua error

readme file  other linux version script error
raw sample:
curl -LSfs https://raw.githubusercontent.com/byron/dua-cli/master/ci/install.sh |
sh -s -- --git byron/dua-cli --target x86_64-unknown-linux-musl --crate dua
first: path lower byron.
can't find path 404
byron/dua-cli ====> Byron/dua-cli
second:  release version can't find latest path
 curl -LSfs https://raw.githubusercontent.com/byron/dua-cli/master/ci/install.sh | \
sh -s -- --git byron/dua-cli --target x86_64-unknown-linux-musl --crate dua

=====>OK
curl -LSfs https://raw.githubusercontent.com/byron/dua-cli/master/ci/install.sh |
sh -s -- --git Byron/dua-cli --target x86_64-unknown-linux-musl  --tag  v2.17.4  --crate dua

The instructions in the README changed to compensate:
curl -LSfs https://raw.githubusercontent.com/byron/dua-cli/master/ci/install.sh | \
    sh -s -- --git byron/dua-cli --crate dua --tag v2.17.2

I didn't get around fixing the install script after GitHub broke it apparently, and would love to have an alternative alltogether.

Closing as the issue is fixed in the README.

--git byron/dua-cli
--git Byron/dua-cli
repository name fist letter is Upper , when lower  url will happen 404

release version tag update latest also  ok
dua-latest-x86_64-unknown-linux-musl.tar.gz

Thanks a lot for pointing this out! It's amazing that this ever work, maybe it's a change on GitHub, who knows.
I have adjusted the script once again, but but --tag latest didn't work for me at all. Am I missing something? It would be great not to be able to avoid hardcoding a version.


Simplified and improved toolbar tutorial

Coverage decreased (-4.2%) to 84.594% when pulling cf9192660445543989830cec38252f31adc2dd67 on evildmp:docs-improve-toolbar-intro into 636dc6d928172b0fb13016c4aba0672d49892edd on divio:release/3.3.x.

@evildmp Please rebase. There was a new release of htmllib that broke the tests.

Coverage increased (+0.7%) to 89.422% when pulling 53ff1b19215bee7767584b92fb495a78431b46bc on evildmp:docs-improve-toolbar-intro into 1ac3e9212e275b6a350649af1d3c8ec6845dea98 on divio:release/3.3.x.

@czpython rebased, needs a quick review.

Coverage decreased (-4.3%) to 84.505% when pulling 8f66b0298f61dac957d94f27cad66943745c5f86 on evildmp:docs-improve-toolbar-intro into 1ac3e9212e275b6a350649af1d3c8ec6845dea98 on divio:release/3.3.x.

@czpython This will need to be ported forward into 3.4 and develop.

Coverage increased (+0.04%) to 88.809% when pulling 25bffefcaed4c3aafe667c07ff41b6f0e7709009 on evildmp:docs-improve-toolbar-intro into 1ac3e9212e275b6a350649af1d3c8ec6845dea98 on divio:release/3.3.x.


updated inscriptions.json merged but changes not reflected on website.

This commit was to fix the names of some ordinal inscriptions, but I'm not seeing this reflected on the website despite this update inscriptions.json with the correct ordinal names being merged.
Originally posted by @Bird-NZ in https://github.com/ordinals-wallet/ordinals-collections/issues/3630#issuecomment-1687221601

@Bird-NZ You don't do anything wrong. In some cases, there are sync issues from repo to website, in other cases it may take some time. Thank you for your patience.
@utxo-detective can you sync this collection?

I am still seeing the issue. Can this please be left open?


README.prod: Add documentation for how to create a self-signed SSL certificate.

This is just using the code from https://github.com/zulip/zulip/issues/78 with a longer key length.

Automated message from Dropbox CLA bot
@timabbott, it looks like you've already signed the Dropbox CLA. Thanks!

Merged.


User should see magic link explanation screen after email verification

For all new users passing wallet creation we should show a Magic link explanation screen after the last registration step ( entering email verification code) before the Welcome screen.
https://xd.adobe.com/spec/8f259bcd-550e-4f0c-40bb-bdce2ed36fe5-f914/screen/229c2efe-3550-4804-a109-eac96ee1b958/Sign-Up-23
Definition of Done

[x] The new Magic link explanation screen is shown as the last step of registration before the Welcome screen
[x] The Magic link explanation screen matches the design
[x] The screen looks OK on small devices

https://xd.adobe.com/spec/8f259bcd-550e-4f0c-40bb-bdce2ed36fe5-f914/screen/229c2efe-3550-4804-a109-eac96ee1b958/Sign-Up-23

@LiavGut I see there is a left error in the header. Do we need it on this screen?
If so, should it bring to the previous step? It makes no sense I guess.

@Nordwhale No, you're right, we don't need it here. I've updated the design.
Thanks :)

Checked on the QA environment.
The screen look as per requirement. The screen performing after user's verification and before "You're all set" screen.

Checked on QA environment. User get the magic screen explanation after email verification.
Work and look as per requirement.


Export a email as EML file

Checklist

[X] I have used the search function to see if someone else has already submitted the same feature request.
[X] I will describe the problem with as much detail as possible.
[X] This issue only contains a request for one single feature, not multiple (related) features.

App version
6.202
Problem you are trying to solve
I need EML export feature.
Suggested solution
I need EML export feature.
Screenshots / Drawings / Technical details
No response

Duplicate of #1498


Add additional installation information to the gather-logs data

We need to make sure we provide actionable usage information in the gather-logs command so that support can better engage with customers:

Number of nodes
Number of failing nodes
Number of nodes not checking in
Number of orgs
Number of users
Platform / version breakdown of nodes

Example: ubuntu_1604: 2013, windows_2019: 1043

Policyfiles in use
Is the server proxied
Can the server reach the internet

nodes, org, users, and policyfiles data ships in 14.8

14.8 will also include proxy and internet connection checks. I'm going to close this one out as done at this point. It's not practical given our schema to get node data directly out of the database so we will need to tackle that in a yet to be built tool.


Add naive approach to updating lineup, teardown and rerender

Closes #323
This PR adds a watcher for cellSize in the lineup component that redraws the whole lineup componenet. This comes with significant performance issues.
My goal is to explore lineup's documentation + implementation to see if it's possible to update the rowHeight parameter without a full re-render.

After exploring the docs and communicating with the Caleydo team, I found lineup could adjust row height without tearing down and re-rendering. The relevant function is dynamicHeight() and to trigger the re-draw in lineup you have to call lineup.update().
Small cell size:

Large cell size:


MNIST maxout example crashes

I am trying to reproduce the result stated here, namely, 0.45% on MNIST. I am running pylearn2 on Amazon EC2 from this image. I set up a g2.2xlarge instance, booted up and ran:
sudo -s
cd pylearn2/pylearn2/scripts/papers/maxout/
THEANO_FLAGS="device=gpu,floatX=float32" ~/pylearn2/pylearn2/scripts/train.py mnist.yaml

pylearn2 crashed with the following traceback (a more complete gist):
root@ip-172-31-44-36:~/pylearn2/pylearn2/scripts/papers/maxout# THEANO_FLAGS="device=gpu,floatX=float32" ~/pylearn2/pylearn2/scripts/train.py mnist.yaml
Using gpu device 0: GRID K520
error: XDG_RUNTIME_DIR not set in the environment.
error: XDG_RUNTIME_DIR not set in the environment.

(train.py:5031): Gdk-CRITICAL **: gdk_cursor_new_for_display: assertion 'GDK_IS_DISPLAY (display)' failed
Traceback (most recent call last):
  File "/home/ubuntu/pylearn2/pylearn2/scripts/train.py", line 252, in <module>
    args.verbose_logging, args.debug)
  File "/home/ubuntu/pylearn2/pylearn2/scripts/train.py", line 197, in train
    train_obj = serial.load_train_file(config)
  File "/home/ubuntu/pylearn2/pylearn2/utils/serial.py", line 524, in load_train_file
    return yaml_parse.load_path(config_file_path, environ=environ)
  File "/home/ubuntu/pylearn2/pylearn2/config/yaml_parse.py", line 379, in load_path
    return load(content, instantiate=instantiate, environ=environ, **kwargs)
  File "/home/ubuntu/pylearn2/pylearn2/config/yaml_parse.py", line 335, in load
    proxy_graph = yaml.load(string, **kwargs)
  File "/usr/lib/python2.7/dist-packages/yaml/__init__.py", line 71, in load
    return loader.get_single_data()
  File "/usr/lib/python2.7/dist-packages/yaml/constructor.py", line 39, in get_single_data
    return self.construct_document(node)
  File "/usr/lib/python2.7/dist-packages/yaml/constructor.py", line 43, in construct_document
    data = self.construct_object(node)
  File "/usr/lib/python2.7/dist-packages/yaml/constructor.py", line 90, in construct_object
    data = constructor(self, tag_suffix, node)
  File "/home/ubuntu/pylearn2/pylearn2/config/yaml_parse.py", line 479, in multi_constructor_obj
    mapping = loader.construct_mapping(node)
  File "/usr/lib/python2.7/dist-packages/yaml/constructor.py", line 208, in construct_mapping
    return BaseConstructor.construct_mapping(self, node, deep=deep)
  File "/usr/lib/python2.7/dist-packages/yaml/constructor.py", line 133, in construct_mapping
    value = self.construct_object(value_node, deep=deep)
  File "/usr/lib/python2.7/dist-packages/yaml/constructor.py", line 90, in construct_object
    data = constructor(self, tag_suffix, node)
  File "/home/ubuntu/pylearn2/pylearn2/config/yaml_parse.py", line 493, in multi_constructor_obj
    callable = try_to_import(tag_suffix)
  File "/home/ubuntu/pylearn2/pylearn2/config/yaml_parse.py", line 391, in try_to_import
    exec('import %s' % modulename)
  File "<string>", line 1, in <module>
  File "/home/ubuntu/pylearn2/pylearn2/models/mlp.py", line 26, in <module>
    from pylearn2.linear import conv2d
  File "/home/ubuntu/pylearn2/pylearn2/linear/conv2d.py", line 20, in <module>
    from pylearn2.packaged_dependencies.theano_linear.conv2d \
  File "/home/ubuntu/pylearn2/pylearn2/packaged_dependencies/theano_linear/__init__.py", line 9, in <module>
    from conv2d import Conv2d
  File "/home/ubuntu/pylearn2/pylearn2/packaged_dependencies/theano_linear/conv2d.py", line 3, in <module>
    from .imaging import tile_slices_to_image, most_square_shape
  File "/home/ubuntu/pylearn2/pylearn2/packaged_dependencies/theano_linear/imaging.py", line 9, in <module>
    from pylearn2.utils.image import Image, ensure_Image
  File "/home/ubuntu/pylearn2/pylearn2/utils/image.py", line 12, in <module>
    import matplotlib.pyplot as plt
  File "/usr/local/lib/python2.7/dist-packages/matplotlib/pyplot.py", line 109, in <module>
    _backend_mod, new_figure_manager, draw_if_interactive, _show = pylab_setup()
  File "/usr/local/lib/python2.7/dist-packages/matplotlib/backends/__init__.py", line 32, in pylab_setup
    globals(),locals(),[backend_name],0)
  File "/usr/local/lib/python2.7/dist-packages/matplotlib/backends/backend_gtk3agg.py", line 11, in <module>
    from . import backend_gtk3
  File "/usr/local/lib/python2.7/dist-packages/matplotlib/backends/backend_gtk3.py", line 55, in <module>
    cursors.MOVE          : Gdk.Cursor.new(Gdk.CursorType.FLEUR),
TypeError: constructor returned NULL

Why on earth are you running with sudo?
Seems like you have a problem with your matplotlib install, but it's
unclear to me why matplotlib should need to be imported, so I'll leave this
issue open for the time being.
On Mon, Feb 9, 2015 at 2:39 PM, Pastafarianist<EMAIL_ADDRESS>wrote:

I am trying to reproduce the result stated in
https://github.com/lisa-lab/pylearn2/blob/master/pylearn2/scripts/papers/maxout/README,
namely, 0.45% on MNIST. I am running pylearn2 on Amazon EC2 from this
image
http://thecloudmarket.com/image/ami-39296709--caffe-theano-pylearn2-torch7-cuda6-5-ubuntu14-04-gpu.
I set up a g2.2xlarge instance, booted up and ran:
sudo -s
cd pylearn2/pylearn2/scripts/papers/maxout/
THEANO_FLAGS="device=gpu,floatX=float32" ~/pylearn2/pylearn2/scripts/train.py mnist.yaml
pylearn2 crashed with the following traceback (gist
https://gist.github.com/Pastafarianist/9ccfb0dd10660eabc055):
ubuntu@ip-172-31-44-36:~/pylearn2/pylearn2/scripts/papers/maxout$
THEANO_FLAGS="device=gpu,floatX=float32"
~/pylearn2/pylearn2/scripts/train.py mnist.yaml
Using gpu device 0: GRID K520
(train.py:4883): Gdk-CRITICAL **: gdk_cursor_new_for_display: assertion 'GDK_IS_DISPLAY (display)' failed
Traceback (most recent call last):
File "/home/ubuntu/pylearn2/pylearn2/scripts/train.py", line 252, in
args.verbose_logging, args.debug)
File "/home/ubuntu/pylearn2/pylearn2/scripts/train.py", line 197, in train
train_obj = serial.load_train_file(config)
File "/home/ubuntu/pylearn2/pylearn2/utils/serial.py", line 524, in load_train_file
return yaml_parse.load_path(config_file_path, environ=environ)
File "/home/ubuntu/pylearn2/pylearn2/config/yaml_parse.py", line 379, in load_path
return load(content, instantiate=instantiate, environ=environ, **kwargs)
File "/home/ubuntu/pylearn2/pylearn2/config/yaml_parse.py", line 335, in load
proxy_graph = yaml.load(string, **kwargs)
File "/usr/lib/python2.7/dist-packages/yaml/init.py", line 71, in load
return loader.get_single_data()
File "/usr/lib/python2.7/dist-packages/yaml/constructor.py", line 39, in get_single_data
return self.construct_document(node)
File "/usr/lib/python2.7/dist-packages/yaml/constructor.py", line 43, in construct_document
data = self.construct_object(node)
File "/usr/lib/python2.7/dist-packages/yaml/constructor.py", line 90, in construct_object
data = constructor(self, tag_suffix, node)
File "/home/ubuntu/pylearn2/pylearn2/config/yaml_parse.py", line 479, in multi_constructor_obj
mapping = loader.construct_mapping(node)
File "/usr/lib/python2.7/dist-packages/yaml/constructor.py", line 208, in construct_mapping
return BaseConstructor.construct_mapping(self, node, deep=deep)
File "/usr/lib/python2.7/dist-packages/yaml/constructor.py", line 133, in construct_mapping
value = self.construct_object(value_node, deep=deep)
File "/usr/lib/python2.7/dist-packages/yaml/constructor.py", line 90, in construct_object
data = constructor(self, tag_suffix, node)
File "/home/ubuntu/pylearn2/pylearn2/config/yaml_parse.py", line 493, in multi_constructor_obj
callable = try_to_import(tag_suffix)
File "/home/ubuntu/pylearn2/pylearn2/config/yaml_parse.py", line 391, in try_to_import
exec('import %s' % modulename)
File "", line 1, in
File "/home/ubuntu/pylearn2/pylearn2/models/mlp.py", line 26, in
from pylearn2.linear import conv2d
File "/home/ubuntu/pylearn2/pylearn2/linear/conv2d.py", line 20, in
from pylearn2.packaged_dependencies.theano_linear.conv2d
File "/home/ubuntu/pylearn2/pylearn2/packaged_dependencies/theano_linear/init.py", line 9, in
from conv2d import Conv2d
File "/home/ubuntu/pylearn2/pylearn2/packaged_dependencies/theano_linear/conv2d.py", line 3, in
from .imaging import tile_slices_to_image, most_square_shape
File "/home/ubuntu/pylearn2/pylearn2/packaged_dependencies/theano_linear/imaging.py", line 9, in
from pylearn2.utils.image import Image, ensure_Image
File "/home/ubuntu/pylearn2/pylearn2/utils/image.py", line 12, in
import matplotlib.pyplot as plt
File "/usr/local/lib/python2.7/dist-packages/matplotlib/pyplot.py", line 109, in
_backend_mod, new_figure_manager, draw_if_interactive, _show = pylab_setup()
File "/usr/local/lib/python2.7/dist-packages/matplotlib/backends/init.py", line 32, in pylab_setup
globals(),locals(),[backend_name],0)
File "/usr/local/lib/python2.7/dist-packages/matplotlib/backends/backend_gtk3agg.py", line 11, in
from . import backend_gtk3
File "/usr/local/lib/python2.7/dist-packages/matplotlib/backends/backend_gtk3.py", line 55, in
cursors.MOVE          : Gdk.Cursor.new(Gdk.CursorType.FLEUR),
TypeError: constructor returned NULL
Error in atexit._run_exitfuncs:
Traceback (most recent call last):
File "/usr/lib/python2.7/atexit.py", line 24, in _run_exitfuncs
func(*targs, **kargs)
File "/usr/local/lib/python2.7/dist-packages/theano/gof/cmodule.py", line 1363, in _on_atexit
self.clear_unversioned()
File "/usr/local/lib/python2.7/dist-packages/theano/gof/cmodule.py", line 1332, in clear_unversioned
False)
File "/usr/local/lib/python2.7/dist-packages/theano/gof/cmodule.py", line 316, in module_name_from_dir
files = os.listdir(dirname)
OSError: [Errno 13] Permission denied: '/home/ubuntu/.theano/compiledir_Linux-3.13.0-35-generic-x86_64-with-Ubuntu-14.04-trusty-x86_64-2.7.6-64/tmp7HLozJ'
Error in sys.exitfunc:
Traceback (most recent call last):
File "/usr/lib/python2.7/atexit.py", line 24, in _run_exitfuncs
func(*targs, **kargs)
File "/usr/local/lib/python2.7/dist-packages/theano/gof/cmodule.py", line 1363, in _on_atexit
self.clear_unversioned()
File "/usr/local/lib/python2.7/dist-packages/theano/gof/cmodule.py", line 1332, in clear_unversioned
False)
File "/usr/local/lib/python2.7/dist-packages/theano/gof/cmodule.py", line 316, in module_name_from_dir
files = os.listdir(dirname)
OSError: [Errno 13] Permission denied: '/home/ubuntu/.theano/compiledir_Linux-3.13.0-35-generic-x86_64-with-Ubuntu-14.04-trusty-x86_64-2.7.6-64/tmp7HLozJ'
ubuntu@ip-172-31-44-36:~/pylearn2/pylearn2/scripts/papers/maxout$ sudo -s
root@ip-172-31-44-36:~/pylearn2/pylearn2/scripts/papers/maxout# THEANO_FLAGS="device=gpu,floatX=float32" ~/pylearn2/pylearn2/scripts/train.py mnist.yaml
Using gpu device 0: GRID K520
error: XDG_RUNTIME_DIR not set in the environment.
error: XDG_RUNTIME_DIR not set in the environment.
(train.py:5031): Gdk-CRITICAL **: gdk_cursor_new_for_display: assertion 'GDK_IS_DISPLAY (display)' failed
Traceback (most recent call last):
File "/home/ubuntu/pylearn2/pylearn2/scripts/train.py", line 252, in
args.verbose_logging, args.debug)
File "/home/ubuntu/pylearn2/pylearn2/scripts/train.py", line 197, in train
train_obj = serial.load_train_file(config)
File "/home/ubuntu/pylearn2/pylearn2/utils/serial.py", line 524, in load_train_file
return yaml_parse.load_path(config_file_path, environ=environ)
File "/home/ubuntu/pylearn2/pylearn2/config/yaml_parse.py", line 379, in load_path
return load(content, instantiate=instantiate, environ=environ, **kwargs)
File "/home/ubuntu/pylearn2/pylearn2/config/yaml_parse.py", line 335, in load
proxy_graph = yaml.load(string, **kwargs)
File "/usr/lib/python2.7/dist-packages/yaml/init.py", line 71, in load
return loader.get_single_data()
File "/usr/lib/python2.7/dist-packages/yaml/constructor.py", line 39, in get_single_data
return self.construct_document(node)
File "/usr/lib/python2.7/dist-packages/yaml/constructor.py", line 43, in construct_document
data = self.construct_object(node)
File "/usr/lib/python2.7/dist-packages/yaml/constructor.py", line 90, in construct_object
data = constructor(self, tag_suffix, node)
File "/home/ubuntu/pylearn2/pylearn2/config/yaml_parse.py", line 479, in multi_constructor_obj
mapping = loader.construct_mapping(node)
File "/usr/lib/python2.7/dist-packages/yaml/constructor.py", line 208, in construct_mapping
return BaseConstructor.construct_mapping(self, node, deep=deep)
File "/usr/lib/python2.7/dist-packages/yaml/constructor.py", line 133, in construct_mapping
value = self.construct_object(value_node, deep=deep)
File "/usr/lib/python2.7/dist-packages/yaml/constructor.py", line 90, in construct_object
data = constructor(self, tag_suffix, node)
File "/home/ubuntu/pylearn2/pylearn2/config/yaml_parse.py", line 493, in multi_constructor_obj
callable = try_to_import(tag_suffix)
File "/home/ubuntu/pylearn2/pylearn2/config/yaml_parse.py", line 391, in try_to_import
exec('import %s' % modulename)
File "", line 1, in
File "/home/ubuntu/pylearn2/pylearn2/models/mlp.py", line 26, in
from pylearn2.linear import conv2d
File "/home/ubuntu/pylearn2/pylearn2/linear/conv2d.py", line 20, in
from pylearn2.packaged_dependencies.theano_linear.conv2d
File "/home/ubuntu/pylearn2/pylearn2/packaged_dependencies/theano_linear/init.py", line 9, in
from conv2d import Conv2d
File "/home/ubuntu/pylearn2/pylearn2/packaged_dependencies/theano_linear/conv2d.py", line 3, in
from .imaging import tile_slices_to_image, most_square_shape
File "/home/ubuntu/pylearn2/pylearn2/packaged_dependencies/theano_linear/imaging.py", line 9, in
from pylearn2.utils.image import Image, ensure_Image
File "/home/ubuntu/pylearn2/pylearn2/utils/image.py", line 12, in
import matplotlib.pyplot as plt
File "/usr/local/lib/python2.7/dist-packages/matplotlib/pyplot.py", line 109, in
_backend_mod, new_figure_manager, draw_if_interactive, _show = pylab_setup()
File "/usr/local/lib/python2.7/dist-packages/matplotlib/backends/init.py", line 32, in pylab_setup
globals(),locals(),[backend_name],0)
File "/usr/local/lib/python2.7/dist-packages/matplotlib/backends/backend_gtk3agg.py", line 11, in
from . import backend_gtk3
File "/usr/local/lib/python2.7/dist-packages/matplotlib/backends/backend_gtk3.py", line 55, in
cursors.MOVE          : Gdk.Cursor.new(Gdk.CursorType.FLEUR),
TypeError: constructor returned NULL
—
Reply to this email directly or view it on GitHub
https://github.com/lisa-lab/pylearn2/issues/1391.

I need sudo because otherwise Theano fails to write a temporary file during some compilation.

Maybe specify a different base directory for compilation, by setting the base_compiledir flag.

@lamblin I would guess that TypeError should be handled in https://github.com/lisa-lab/pylearn2/blob/master/pylearn2/utils/image.py#L13. I have created a relevant pull request at https://github.com/lisa-lab/pylearn2/pull/1395

Fixed by gh-1395.

Unfortunately, this issue has to be reopened.
First, there's another place where matplotlib is imported in this example. It's in /pylearn2/pylearn2/packaged_dependencies/theano_linear/unshared_conv/localdot.py. The same fix applies (although this file handles the import failure a bit differently).
Second, even after localdot.py is patched, cuda_convnet compilation crashes with this output. I have no idea how to make any sense of it.

Regarding the crash, this is due to your Theano version being too old, and out of sync with Pylearn2.
In particular, your Theano version links with cublas via the legacy API, whereas the version of cuda-convnet packaged with pylearn2 uses the new API, cublas_v2.
You should use the bleeding-edge install of Theano.


Update compositiontarget_rendering.md

Addresses #314

updated links and removed out-of-date sample link
updated performance recommendations
clarified DirectX interop scenarios

@jwmsft : PR passed the acceptance checklist. Please evaluate further and approve/merge if appropriate. Thanks!


Unbreak gofmt

The gofmt logic in .ci/go-static-checks.go is incorrect.

Is that the old 'inverted check' debate - should we have it hard define what it should or should not check ??
I'll note, I'm sure I have an open issue to 'fix', that is more like unify, the static checkers across all the repos, ideally having a single static checker script, probably in the tests/.ci repo dir, that is then used for all the repos - one script to rule them all. I'm not expecting you to try and do that whilst fixing this though, but thought I'd mention it for good measure...

I think the approach of this script is pragmatic for the time being: slurp all go code excluding code under vendor and .git/. That's obviously the maximal approach and if it starts to somehow check "too much" and then falls over, we'll be notified so we can tweak the config. But better that than not checking enough I'd say.
/me looks forward to @grahamwhaley's static-checker-script-to-rule-them-all.sh :smile:


No image just black in square box

The header is coming out and a white border box put no image inside.  I tried browsing to img it gave a cert error, I just accepted and its up in browser but image still not popping in magic mirror.
Thanks for any help!

Hi, please make sure u have cloned the latest version, because the certificate problem is already fixed and it is functional.
if you still have a problem, please let me know if there are any errors loading it.

I cloned with the url that's in your GitHub instructions.  Is that the latest?


bug/segfault when using add_features_from and somewhat sparse data

How you are using LightGBM?
LightGBM component: Python package
Environment info

Operating System: Ubuntu 16.04.6 LTS
CPU/GPU model: Intel(R) Core(TM) i7-3840QM CPU @ 2.80GHz
C++ compiler version: gcc (Ubuntu 5.4.0-6ubuntu1~16.04.12) 5.4.0 20160609
CMake version: 3.5.1
Python version: 3.6.3
Comments: tried on other machines running Debian with newer python, gcc, and cmake

LightGBM version or commit hash:  I tried 1bc27939a43c414c3424f339994d0aa11f3aa3b1 and 3.0.0 from pypi
Error message and / or logs
this hash: 1bc27939a43c414c3424f339994d0aa11f3aa3b1
Set free_raw_data=False when construct Dataset to avoid this
  warnings.warn(err_msg)
/home/ilya/anaconda3/lib/python3.6/site-packages/lightgbm/basic.py:1973: UserWarning: Reseting categorical features.
You can set new categorical features via ``set_categorical_feature`` method
  warnings.warn("Reseting categorical features.\n"
[LightGBM] [Debug] Dataset::GetMultiBinFromSparseFeatures: sparse rate 0.800036
[LightGBM] [Info] Total Bins 10200
[LightGBM] [Info] Number of data points in the train set: 1000000, number of used features: 40
Segmentation fault (core dumped)

with 3.0.0 from pypi:
Traceback (most recent call last):
  File "lgb_test.py", line 28, in <module>
    m = lgb.train(pars, datasets[0], num_boost_round=3)
  File "/home/ilya/anaconda3/lib/python3.6/site-packages/lightgbm/engine.py", line 231, in train
    booster = Booster(params=params, train_set=train_set)
  File "/home/ilya/anaconda3/lib/python3.6/site-packages/lightgbm/basic.py", line 1991, in __init__
    ctypes.byref(self.handle)))
  File "/home/ilya/anaconda3/lib/python3.6/site-packages/lightgbm/basic.py", line 55, in _safe_call
    raise LightGBMError(decode_string(_LIB.LGBM_GetLastError()))
lightgbm.basic.LightGBMError: Bug. There should be only one multi-val group.

Reproducible example(s)

import lightgbm as lgb
import numpy as np

r = np.random.RandomState(42)
ROWS = 1000000
COLUMNS = 20
N = 2
tags = 'ab'

data = [r.rand(ROWS, COLUMNS) for _ in range(N)]
for i in range(N):
    data[i][data[i] < 0.8] = 0
label = r.rand(ROWS)

def construct(i):
    return lgb.Dataset(data[i], feature_name=[tags[i] + str(n) for n in range(COLUMNS)], label=label).construct()

datasets = [construct(i) for i in range(N)]

for i in range(1, N):
    datasets[0].add_features_from(datasets[i])

if lgb.__version__ == '3.0.0':
    datasets[0].feature_name = sum((d.feature_name for d in datasets), [])

pars = {'verbosity': 2, 'seed': 42, 'force_col_wise': True}

m = lgb.train(pars, datasets[0], num_boost_round=3)

Steps to reproduce
python <above code>
Thoughts and partial workaround
On pypi version, setting pars['force_row_wise'] = True instead makes it work but master tip still segfaults. My guess is that in 3.0.0 some bin appears more than once?
Separately, the warnings in master tip don't make sense to me: I think it is perfectly reasonable to free raw data, so why this check?
Many thanks for any help!

@shiyu1994 as you are changing the related codes, can you also fix this in your PR? (or after merge it).

I'll open another PR for this.

@shiyu1994 any progress for the fix?


KAFKA-17039: KIP-919 suports for describeDelegationTokens, describeUserScramCredentials and unregisterBroker

Use LeastLoadedBrokerOrActiveKController for describeDelegationTokens, describeUserScramCredentials, and unregisterBroker APIs.
Add --bootstrap-controller to DelegationTokenCommand.

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@frankvicky any update?

Hi @chia7712
Sorry there is no further update.
It's now blocking by missing server-side implementation

It's now blocking by missing server-side implementation

Could you please file a sub task for it?

Hi @chia7712
I have filed a jira ticket.
https://issues.apache.org/jira/browse/KAFKA-17651


Flake8 does not support some syntax in Python 3.10

should i remove match syntax or ignore it in flake8, or ci will always fail :(

Or I should remove pre-commit.ci if it caused more problems.
My local pre-commit runs fine.
You can use it locally:
pip install pre-commit
pre-commit
pre-commit install

If we have the same local pre-commit config, then pre-commit.ci is not really necessary.

I have removed pre-commit.ci
If needed, will try GitHub action in the near future.

My local’s pre-commit works fine too, seems pre-commit.ci maybe has some problems, going be me closed😃


feat(events): return the possible events in the invoked endpoint

Fixes: https://github.com/fabric8-launcher/launcher-planning/issues/167

Ike Plugins (test-keeper)
Thank you @gastaldi for this contribution!

It appears that no tests have been added or updated in this PR.
Automated tests give us confidence in shipping reliable software. Please add some as part of this change.
If you are an admin or the reviewer of this PR and you are sure that no test is needed then you can use the command /ok-without-tests as a comment to make the status green.
For more information please head over to official documentation. You can find there how to configure the plugin.


Python 3 Compatibility and Relative Module Imports

Hey Rodrigo
So your library doesn't work on Python 3 and you use both relative and absolute imports in your module. This might work for you since you have your library installed, but it won't work for people who are installing your package.
I've patched up both of these issues in this PR.
Also I might recommend some refactoring so there is less code duplication in your packages. That way, you only need to patch up one location rather than 3.

Hi @jjangsangy, if you are interested in tools for extraction, would you consider giving datalib/libextract a look? It is more extensible and modular than eatihit, and is the successor to it.

@eugene-eeo thanks for letting me know!
Mostly the reason I'm submitting the PR is due to some code that depends on eatiht and trying to get packages running on Python 3.
But in that case, I will let the package owners know so we try moving to libextract.

Also @eugene-eeo, i'm looking at libextract and there are also similar issues in terms of Python 2 / 3 as well as usage of absolute imports for module packages.
I might help you guys out and send a PR, but currently don't have time in terms or looking through the new codebase and building everything so nothing is broken. But I would definitely suggest looking at PEP 328: Imports: Multi-Line and Absolute/Relative as well as a blog article that will definitely better than I can explain.

@jjangsangy thanks, :+1: Will fix.

Does the latest build solve Python 2/3 compatibility? I have switched to relative imports for everything.

Wait, did you just push code??

Give me a sec, I'll check

Ugh.. so I'll go to libextract, I don't want to continue discussion on this thread since it's off topic

@eugene-eeo: thanks for taking of this!

[..] I will also let the package owners know so we try moving to libextract.

@jjangsangy please do! As I've said on eatiht's README, I'm moving most of my efforts to libextract.
Just fyi, comprehensive sphynx documentation still missing (aside from docstrings), but that's one of the next things on my priority list. Thanks again for the pr

Cool thanks!
Sphinx docs? I could help out setting up sphinx-build, and configuring the webhooks for building on readthedocs. The setup is pretty trivial in itself.
Hard part is documenting your API accurately, ensuring that the docs can be authoritative source of information. So that means figuring out which parts of your code to use as the public facing interfaces. I see that you guys are implementing strategy to generically swap algorithms. Each one of your strategies should have it's own interface and documentation. Actually I'm a little curious as to the implementations, if you want I could probably help build those interfaces if you show me how they're implemented ;)

Absolutely! Still in the middle of work I have to finish at my uni, but
let's get ahold of each other via email and/or a new libextract issue?
On Wed, Apr 15, 2015 at 12:13 PM Sang Han<EMAIL_ADDRESS>wrote:

Cool thanks!
Sphinx docs? I could help out setting up sphinx-build, and configuring
the webhooks for building on readthedocs https://readthedocs.org/. The
setup is pretty trivial in itself.
Hard part is documenting your API accurately, ensuring that the docs can
be authoritative source of information. So that means figuring out which
parts of your code to use as the public facing interfaces. I see that you
guys are implementing strategy to generically swap algorithms. Each one of
your strategies should have it's own interface and documentation. Actually
I'm a little curious as to the implementations, if you want I could
probably help build those interfaces if you show me how they're implemented
;)
—
Reply to this email directly or view it on GitHub
https://github.com/rodricios/eatiht/pull/20#issuecomment-93536237.

Sure thing, let me know.
Sang Han<EMAIL_ADDRESS>


scatterplot points completely distorted/invisible at higher zoom levels

To reproduce, just zoom in on a scatter plot view.
Possibly related to #45

We have already fixed this and will push the patch in two days.

@gnavvy Any pointers? I can apply it on my own fork and test today.

just published 2.4.1 to fix the distortion for the scatterplot and the hexagon layer.
want to give it a try? @contra
(the jittering on the centroids remains, unfortunately, will have to implement the high-precision simulation)

Any idea on when the jittering will be resolved? Thanks!

/cc @pessimistress did this land?
On Fri, Jul 14, 2017 at 4:08 PM John Leyden<EMAIL_ADDRESS>wrote:

Any idea on when the jittering will be resolved? Thanks!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/uber/deck.gl/issues/52#issuecomment-315488279, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABEzWzoYob-Yb0zeAa4rRspiyTKOSFKvks5sN_T7gaJpZM4JkFNP
.

@jleyden @ericsoco This landed a looong time ago. Use the fp64 flag.


.speed Modifiers throws an error

Error Info:
speed modifier throws an error. it doesnt depend upon the emote or the speed (fast, faster, hyper) it just keeps giving the same error.
Date of Error
I have gotten it multiple times, I even got it today.

Example

.speed- - For gifs, change delay between frames in hundredths of a second.
Meaning it should be a number, the lower the faster


Fix README.md example

In README example is stateless functional component but use this.state.value.
I think this is good for props.value

Thanks


SAF Export file limit is bypassed - these exports can consume arbitrary amounts of disk storage

Describe the bug
SAF exports can be arbitrarily large. We've seen cases in production where users were able to create 160GB SAF export files, blowing up the file storage of the server.
To Reproduce
Steps to reproduce the behavior:

Create a SAF export for a big collection
Check the size of the resulting file on disk & the available storage you have left.

Expected behavior
The Exporter should respect the limit defined in org.dspace.app.itemexport.max.size cfr https://github.com/DSpace/DSpace/blob/1b4f1cb49cce44618a357c10e35238b886b72472/dspace-api/src/main/java/org/dspace/app/itemexport/ItemExportServiceImpl.java#L643-L646
Related work
Link to any related tickets or PRs here.

Needs a volunteer to investigate & fix.  Pulling this over to the 7.6.x board though in the hopes that someone will claim it.

I'm afraid this is an conceptual problem. SAF shall contain all bitstreams of an item. If you export a lot of items with a lot of or large files, the archive get's large. One particular problem occurs, when you use S3 as backend storage, as the exports are stored on the server's disk and not in S3. This ends up in downloading a lot of data from S3 to your server. We should think if exporting large archives like this via UI makes sense at all and if we could create those in the assetstore instead of local directory.

I just notice, that my idea is already described in #7800

This issue is somehow related to #7800


Is this repo still accepting PR's ?

@Khaledgarbaya @stefanjudis I'd noticed a lot a room for improvement like migrating the application to the latest react version (including hooks) with a major upgrade. Are you both still maintaining the application? Can I proceed with the changes? I'd love to contribute to the project!

I'm not with Contentful anymore so I'd say it's up to @Khaledgarbaya :)

@gabrielreisn sure thing

Hey @gabrielreisn! Thanks so much for your interest in assisting with this example app. Unfortunately gallery-app-react isn't high priority for maintenance on our side (we have lots of other open source tools that keep us busy :) ) That said we would absolutely welcome your contributions! Feel free to make PRs and we will get around to them when we can. (And feel free to @ mention me here)


Dc

Delete the capital "DC" and "SMN" folders;
Added lowercase "dc", "vpn", and "smn" folders, including user guides and interface reference materials for the corresponding services


ebiten: forbid MaximizeWindow when a max size is specified

We have to forbid the maximizing button on Windows and Linux

I think this requires to modify GLFW. Let's focus on #2546 first.

We have to forbid the maximizing button on Windows and Linux

This is duplicated with #2289, so this issue is already fixed.


Path to Lazy Evaluation

I think that other than ergonomics (type-casting and APIs in general), I'm satisfied that a dataframe is possible to build.
What I'd like to focus on as early as possible is a path to lazy evaluation. A key limitation right now is that compute functions are just fn instead of general-purpose kernels. For example, implementing a cast operation is difficult as I need a few different functions to handle different types of casts.
Converting these functions to independent kernel members could allow generating a computation graph with a logical plan. After this, I would be able to leverage (hopefully) existing work that can take my logical plan and turn it into a physical plan.
Having recently worked on a JSON reader for Arrow Rust, I can see a way of supporting pushdown predicates to CSV and JSON data sources. This would be implemented as part of the physical plan to read the data source.
Definition of done
I want to ideally have a rough outline in a Markdown document, so this will take a while to complete.

The approach that I'm currently taking is to keep functions separate, but to instead create a way of expressing operations on data.
An initial implementation can work on scalar operations, but still needs more effort to complete the effort. I'd also need to create some graph structure that can model dependencies between dataframes.

Joins are now readyish as per #22


chart: support deploying additional kubernetes manifests

This change allows the Helm chart's user to include additional Kubernetes manifests to be deployed along with the chart. The incentive for this change was to allow overriding the default use of cluster-admin cluster role and also deploy a different cluster role and binding.
This is achievable by including the additional manifests as an array of multiline strings in field extraManifests. For example:
# values.yaml
extraManifests:
  - |
    apiVersion: v1
    kind: ConfigMap
    metadata:
      name: dummy-configmap
    data:
      key1: value1
      key2: value2
  - |
    apiVersion: v1
    kind: ConfigMap
    metadata:
      name: configmap-with-templated-data
    data:
      injectedKey: {{ .Release.Name }}

@knrt10 implemented the helm chart testing, and might be able to give some pointers on what's going wrong with the test failure.

@illume @knrt10 Apologies for the noise, test is fixed and all checks are passing. Are we OK to proceed? Please let me know if anything's missing.


Create a singleton property for class definition

Instantiate only one object (whatever the number of new)
All methods are automatically bound to this new object

Done with #285


add meter mainnet and testnet

Hi, please I would like to add Meter mainnet and testnet to Gnosis Safe.
Meter Mainnet
chainId = 82
rpcUrl: https://rpc.meter.io/
explorer: https://scan.meter.io/
Meter Testnet
chainId = 83
rpcUrl: https://rpctest.meter.io/
explorer: https://scan-warringstakes.meter.io/

recheck

I have read the CLA Document and I hereby sign the CLA

recheck

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
201 unchanged lines in 6 files lost coverage.
Overall coverage decreased (-5.9%) to 84.772%

Files with Coverage Reduction
New Missed Lines
%

gnosis/eth/typing.py
2
83.33%

gnosis/protocol/gnosis_protocol_api.py
2
89.25%

gnosis/safe/safe_tx.py
2
90.53%

gnosis/eth/multicall.py
9
88.35%

gnosis/eth/ethereum_client.py
47
79.75%

gnosis/eth/oracles/oracles.py
139
50.56%

Totals

Change from base Build<PHONE_NUMBER>:
-5.9%

Covered Lines:
2750

Relevant Lines:
3244

💛  - Coveralls


IDE User should experience minimal delay when viewing a new visualization.

This task is automatically imported from the old Task Issue Board and it was originally created by Michael Mauderer.
Original issue is here.

Why
As IDE user
**I want ** to be able to view data from new nodes with minimal delay
**So that ** I have a smooth experience of working with the IDE
Acceptance Criteria
Scenario:
Given I have an open project in the IDE
When I press tab to create a new node
Then The new node should be created and the visualisation should be visible.

Notes:
This user story is written aspirational, and delays should be as minimal as possible, but might not be instant in the end. The actual threshold for acceptance will need to be decided while investigating possible improvements.

Closing as do more detailed tasks anyway.


Look into pseudonymisation tool Spider.Erdri

In my Slack inbox:
Recently, for this domain [rare diseases], a pseudonymisation tool called [Spider.Erdri](https://eu-rd-platform.jrc.ec.europa.eu/spider/) was developed. The plus of the tool is that it generates pseudonyms for patients and allows linkage and transferring patient data across multiple registries (this is relevant because sometimes the same patient is in different registries). The downside is that registries need to upload data on erdri in order to use the tool - it's a downside because there are already different tools in place, and it makes them not interested in using yet another tool. I am texting you because from what I saw on the git repository, there isn't any "spider like" tool that could be incorporated with already existing methodologies, but I would love to know if you have knowledge of another alternative that I could give a look!
So to do:

Check out this specific tool
Look into other "spider-like" tools that would be an alternative to this tool

Personally, I feel like it's not really a tool in the traditional sense, maybe it's more like a platform? It's also quite specific to the ERDRI it seems? I'm not sure where to put it, but they call themselves a pseudonymisation tool, so I guess it goes there.


chore: increase bazel version limit to 0.24

This fails with --config=remote (and thus will likely fail on buildbot as well)
ERROR: /usr/local/google/home/shahms/.cache/bazel/_bazel_shahms/f0c8a49cbf1bfaa82de86be744427f4b/external/bazel_toolchains/configs/ubuntu16_04_clang/1.1/bazel_0.20.0/
default/BUILD:57:1: @bazel_toolchains//configs/ubuntu16_04_clang/1.1/bazel_0.20.0/default:cc-compiler-k8: no such attribute 'dynamic_runtime_libs' in 'cc_toolchain' r
ule
ERROR: /usr/local/google/home/shahms/.cache/bazel/_bazel_shahms/f0c8a49cbf1bfaa82de86be744427f4b/external/bazel_toolchains/configs/ubuntu16_04_clang/1.1/bazel_0.20.0/
default/BUILD:57:1: @bazel_toolchains//configs/ubuntu16_04_clang/1.1/bazel_0.20.0/default:cc-compiler-k8: no such attribute 'static_runtime_libs' in 'cc_toolchain' ru
le
ERROR: /usr/local/google/home/shahms/.cache/bazel/_bazel_shahms/f0c8a49cbf1bfaa82de86be744427f4b/external/bazel_toolchains/configs/ubuntu16_04_clang/1.1/bazel_0.20.0/
default/BUILD:87:1: @bazel_toolchains//configs/ubuntu16_04_clang/1.1/bazel_0.20.0/default:cc-compiler-armeabi-v7a: no such attribute 'dynamic_runtime_libs' in 'cc_too
lchain' rule
ERROR: /usr/local/google/home/shahms/.cache/bazel/_bazel_shahms/f0c8a49cbf1bfaa82de86be744427f4b/external/bazel_toolchains/configs/ubuntu16_04_clang/1.1/bazel_0.20.0/
default/BUILD:87:1: @bazel_toolchains//configs/ubuntu16_04_clang/1.1/bazel_0.20.0/default:cc-compiler-armeabi-v7a: no such attribute 'static_runtime_libs' in 'cc_tool
chain' rule
ERROR: /usr/local/google/home/shahms/.cache/bazel/_bazel_shahms/f0c8a49cbf1bfaa82de86be744427f4b/external/bazel_toolchains/configs/ubuntu16_04_clang/1.1/bazel_0.20.0/
default/BUILD:57:1: Target '@bazel_toolchains//configs/ubuntu16_04_clang/1.1/bazel_0.20.0/default:compiler_deps' contains an error and its package is in error and ref
erenced by '@bazel_toolchains//configs/ubuntu16_04_clang/1.1/bazel_0.20.0/default:cc-compiler-k8'
ERROR: /usr/local/google/home/shahms/.cache/bazel/_bazel_shahms/f0c8a49cbf1bfaa82de86be744427f4b/external/bazel_toolchains/configs/ubuntu16_04_clang/1.1/bazel_0.20.0/
default/BUILD:57:1: Target '@bazel_toolchains//configs/ubuntu16_04_clang/1.1/bazel_0.20.0/default:empty' contains an error and its package is in error and referenced
by '@bazel_toolchains//configs/ubuntu16_04_clang/1.1/bazel_0.20.0/default:cc-compiler-k8'
ERROR: /usr/local/google/home/shahms/.cache/bazel/_bazel_shahms/f0c8a49cbf1bfaa82de86be744427f4b/external/bazel_toolchains/configs/ubuntu16_04_clang/1.1/bazel_0.20.0/
default/BUILD:47:1: Target '@bazel_toolchains//configs/ubuntu16_04_clang/1.1/bazel_0.20.0/default:cc-compiler-armeabi-v7a' contains an error and its package is in err
or and referenced by '@bazel_toolchains//configs/ubuntu16_04_clang/1.1/bazel_0.20.0/default:toolchain'
ERROR: /usr/local/google/home/shahms/.cache/bazel/_bazel_shahms/f0c8a49cbf1bfaa82de86be744427f4b/external/bazel_toolchains/configs/ubuntu16_04_clang/1.1/bazel_0.20.0/
default/BUILD:47:1: Target '@bazel_toolchains//configs/ubuntu16_04_clang/1.1/bazel_0.20.0/default:cc-compiler-k8' contains an error and its package is in error and re
ferenced by '@bazel_toolchains//configs/ubuntu16_04_clang/1.1/bazel_0.20.0/default:toolchain'
ERROR: /usr/local/google/home/shahms/.cache/bazel/_bazel_shahms/f0c8a49cbf1bfaa82de86be744427f4b/external/bazel_tools/tools/cpp/BUILD:52:1: every rule of type cc_tool
chain_alias implicitly depends upon the target '@bazel_toolchains//configs/ubuntu16_04_clang/1.1/bazel_0.20.0/default:toolchain', but this target could not be found b
ecause of: Target '@bazel_toolchains//configs/ubuntu16_04_clang/1.1/bazel_0.20.0/default:toolchain' contains an error and its package is in error
ERROR: Analysis of target '//kythe/cxx/tools/fyi/testdata:basic' failed; build aborted: Analysis failed
INFO: Elapsed time: 6.682s

Actually, it works on buildbot because buildbot is installing an older version of bazel.  This will break those of us who use --config=remote to speed up compiles, though.

And will fail to work when buildbot updates to a newer version of Bazel.

And will fail to work when buildbot updates to a newer version of Bazel.

I'm confused, can you clarify your comments?
Data points:

This PR doesn't cause a failure in buildbot-launched buildbot builds, right, in particular if buildbot is on an older version.
This PR builds successfully locally.
You're saying that if we update buildbot to e.g. 0.24 that it will start failing? Or that because of some incompatibility between what buildbot is running and 0.24 that remote builds fail? If we updated buildbot to 0.24, presumably that would go away?

bazel build --config=remote fails with bazel 0.24.0 (and no further changes) due to the remote config using a toolchain config from 0.20.0.  0.22 is the oldest compatible release.  If buildbot is updated to 0.24 it, too, will fail due to incompatible toolchain definitions. #3652 includes the .bazelrc changes to fix the issues (I think) and updates buildbot.


For Documnetations

Documentations for the RustBoot

Documentations for the RustBoot

Just a few changes:

can you please fix some of the spellings.
remove the second header for now.

Documentations for the RustBoot

Just a few changes:
* can you please fix some of the spellings.

* remove the second header for now.

Removed the second header and corrected the spellings.

I have added a separate git workflow for pull requests. Can you update this pull request with an upstream fetch followed by a push?


Add a whitepaper

@cberner, what do you think about we create a whitepaper for redb? Explaning some concepts, strategies, etc...

I think the design doc is the right place for this: https://github.com/cberner/redb/blob/master/docs/design.md


📖 Updated README.md links

Summary
Fixed broken links in README.md

/ok-to-test
/cc @avinal looks like this was missed in #2219

Ah indeed missed 😅. Now that our documentation site is up, I was thinking about putting links to site pages instead of markdowns. WDYT @hardys?
In case you haven't checked the site yet, please do https://docs kcp.io/kcp/main

Ah indeed missed sweat_smile. Now that our documentation site is up, I was thinking about putting links to site pages instead of markdowns. WDYT @hardys?

My only concern with that is we seem to only publish docs for main to the site, so links in release branches will point to main docs, not the (potentially different) docs in each release branch.
Leaving the links to the markdown seems like the simplest solution unless there is a plan to publish per-release docs on the site?

@eranco74 the PR verifier is failing on this check - I think you'll have to make the PR description a bit more verbose

I have added the versioned docs mechanism in GitHub Actions. Since this is our first ever docs site so we have only the main ref. But with the next release we will have more versions.

/lgtm


🔑 Implement Cloud Platform Secret Architecture to Join GitHub Service

User Need
As a (type of user)
I want (some action)
so that (some result)
Value
Functional Requirements:

Non-Functional Requirements:

Acceptance Criteria:

Notes:

Not required


CheckNameAvailability on Microsoft.Web/Sites errors with {"The resource type could not be found in the namespace 'Microsoft.Web' for api version '2015-08-01'."}

Hi All,
Package id="Microsoft.Azure.Management.Websites" version="1.3.2-preview"
The following code always fails with "{"The resource type could not be found in the namespace 'Microsoft.Web' for api version '2015-08-01'."}" on CheckNameAvailabilityAsync call.
var cred = new TokenCredentials(token);
var client = new Microsoft.Azure.Management.WebSites.WebSiteManagementClient(new Uri("https://management.azure.com/"), cred)
{
SubscriptionId = "1885d885-......"
};
var checkName = await GlobalModelOperationsExtensions.CheckNameAvailabilityAsync(client.GlobalModel, new ResourceNameAvailabilityRequest
{
Name = "myWebApp",
Type = "Microsoft.Web/Sites",
});
In some thread - I had read that Azure team is working on consolidating Check Availability methods for all resources and the work is still in progress. Is this bug related to that work?
Interestingly the above code works in Azure PPE environment but not in Azure Production environment.
Thanks,
Sankaranarayanan MG

@Nking92 Hey Nicholas, would you mind taking a look at this issue?

Thanks for pinging me. We're looking into it.

@cormacpayne This should now be fixed. Please try again and let us know if you still run into issues.

Verified - its working now.
Thanks,
Sankaranarayanan MG
Date: Wed, 5 Oct 2016 11:38:39 -0700
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>CC<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Re: [Azure/azure-sdk-for-net] CheckNameAvailability on Microsoft.Web/Sites errors with {"The resource type could not be found in the namespace 'Microsoft.Web' for api version '2015-08-01'."} (#2366)
@cormacpayne This should now be fixed. Please try again and let us know if you still run into issues.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Thanks I will close the issue now.

@cormacpayne @Nking92 @naveedaz I got this error.
I used this package https://www.nuget.org/packages/Microsoft.Azure.Management.RecoveryServices (latest version)
{
  "error": {
    "code": "InvalidResourceType",
    "message": "The resource type 'locations/checkNameAvailability' could not be found in the namespace 'Microsoft.RecoveryServices' for api version '2016-06-01'. The supported api-versions are '2018-01-10'."
  }
}


Play an array sound list

Hi!
I have an array with some .mp3's id, for example:
$scope.list = ['bass', 'bass', 'bongo'];
And I want to play that sequence with a general form.
I tried a forEach, like this:
angular.forEach($scope.list, function(value, key) {
window.plugins.NativeAudio.play(value);
});
But that plays all the audios at the same time.
I tried using completeCallback and worked, but I had to put manually each element of the array.
Is there any function or method to play one by one?
Sorry my bad english.
Thanks!!

Finally a friend helped me.
If it helps someone the solution is inserting the function "play" into a $timeout, and asigning an incremental variable at delay.
Like this:
var delay = 0;
angular.forEach($scope.list, function(value, key) {
$timeout(function() {
window.plugins.NativeAudio.play(value);
}, delay);
delay += 1000; //1s
});
Sorry my bad english!!


Added a check for property bag in tag helper descriptor comparer.

aspnet/mvc#1051

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

:watch:

:up: :date:

:watch:

:up: :date:

:shipit:


fix TFE_OpReset doesn't clear inputs.

I've got below error when running c_api.TFE_Execute(op, retVals, ref num_retvals, status);.
NodeDef expected inputs 'int32, int32' do not match 4 inputs specified; Op<name=Sub; signature=x:T, y:T -> z:T; attr=T:type,allowed=[DT_BFLOAT16, DT_HALF, DT_FLOAT, DT_DOUBLE, DT_UINT8, DT_INT8, DT_UINT16, DT_INT16, DT_INT32, DT_INT64, DT_COMPLEX64, DT_COMPLEX128]>; NodeDef: {{node Sub}}

The op is from reused and reset op.
After investigating, I've found the op is not reset completed, at least the inputs is not clean.

It went through after I added op_to_reset->operation->Clear();
Could reproduce this issue from this example. I'm not sure if this is the appropriate fix for it.

Please open against master and then we can cherry-pick.
Also, please add unit test

I think you can cherry-pick from r2.2 to master as well.

No, our procedure is master -> r... branch. We prefer to not mix the branches

To give more context: code is integrated inside Google from the master branch. There is internal code at Google that only works with the master branch.
When we do a release, we cut a r... branch and only do cherry-picks to the branch to fix build/incomplete features.
Once release is done, we don't merge the branch back into master. Instead, we keep it there and if there is need for a patch release (usually only due to security vulnerabilities), we use the branch again.
All cherry-picks to the branch follow a strict process that involves both the branch and master.
Due to all of the above reasons, we cannot accept PRs targetting other branches if they are not cherry-picking things in master (or exceptional cases, such as r1.15 branch when code on that branch no longer exists on master)

Related to https://github.com/tensorflow/tensorflow/pull/37781

I'll close one and PR to master next time.


Bug: filestags.json doesn't export probably

Description
The exported file is missing the line "FileTagList": [ and the closing bracket at the bottom which stops it from importing and resets.
Steps To Reproduce

Create a tag
Export from Files settings > Advanced > Export
Import settings
Restart Files
See the created tag is missing

Requirements

Fix file formatting for export zip

Files Version
<IP_ADDRESS>
Windows Version
10.0.22621.3155
Log File

Hi @Josh65-2201, I would like to work on this issue. I found out that the ExportSettings method within the FileTagsSettingsService class is missing "FileTagList" line and the closing bracket in the exported JSON data.

Thanks, please open a PR when ready and link it to this issue


Is it possible to highlight with the "marked" markdown parser?

https://github.com/chjj/marked#highlight
I'd like to use prism together with marked, to render markdown in the browser. Any ideas?

It looks like it is, as long as the correct grammars are loaded first.
marked.setOptions({
	highlight: function (code, lang) {
		return Prism.highlight(code, Prism.languages[lang], lang);
	}
});

If someone writes markdown code block without specifying the language, it fails.

```
some unknown code here
```

Now I get rid of it by fallback to markup, any better ideas?
function (code, lang) {
  return prism.highlight(code, prism.languages[lang || 'markup'])
}

BTW, I found highlight.js provides an "auto" method to predict the language.

Prism has no way to autodetect the language currently. And it's not planned afaik.
I guess you could also fallback to doing no highlighting whatsoever when the language is not specified.

I am using the marked options key highlight function and it does not add the appropriate class to the <pre> tags for code blocks. <code> tags have the appropriate class, but not the <pre> tags that wrap them.
This is a problem for Prism because it claims that classes with no <pre> tag class is fine. From the Prism website:
If you use that pattern, the <pre> will automatically get the language-xxxx class (if it doesn’t already have it) and will be styled as a code block.

This is not occurring when I'm using Prism with marked. I am defining the behavior inside a React component .tsx file. Here is my code:
return (
    <Layout title={title}>
      <div
        className={postWrapper}
        dangerouslySetInnerHTML={{
          __html: marked(md, {
            highlight: (code, lang) => {
              // @ts-ignore
              return window.Prism.highlight(
                code,
                // @ts-ignore
                window.Prism.languages[lang],
                'typescript'
              )
            }
          })
        }}
      ></div>
    </Layout>

marked.setOptions({
  highlight: function(code, lang) {
    if (prism.languages[lang]) {
      return prism.highlight(code, prism.languages[lang], lang);
    } else {
      return code;
    }
  }
});

@chiefgunner This highlight function is dangerous. Prism will always return HTML source code meaning that all special HTML characters (<, &) will be escaped. In your case, this function can give you a good case of XSS depending on whether Prism supports a specific language.
You can use Prism.util.encode(someString) to escape all special HTML characters.

@aegatlin I had the same issue of the <pre> tag not getting the proper language class and breaking the whole code highlight thing. It happened with NuxtJS.
In order to fix it, just watch the change in the component and call the highlightAll method again once something has changed.
In my case, when a new component (the one that contains the text being rendered) was mounted, the code wasn't highlighted, though it would work everytime I would refresh the page, so I simply added a bit of code to watch the component being mounted:
// Watch the component mounted
mounted() {
    // Run PrismJS highlight again
    Prism.highlightAll()
}


wgpu 0.16

wgpu 0.16 has been released today: https://github.com/gfx-rs/wgpu/releases/tag/v0.16.0
Not much to do actually, just the type changes of NonZeroU32 to u32 and the replacement of TextureFormat::describe with is_srgb.

I don't think the right action here is to get us to downgrade, rather to get ash to upgrade. It was reverted due to Firefox not bring able to use windows-rs

libloading seems like a pretty small dependency, so we can whitelist it in deny.toml.
I wonder how it wasn't caught by the cargo deny check in the wgpu CI though 🤔

🥳


add proper error check for jobs not found

Description
Cronjobs that were deleted are now properly detected and Eunomia does not end up in an endless loop.
Fixes #241
Replace the unreliable string comparison for "not found" error checks, with the proper k8s function.
{"level":"info","ts":1578841086.369399,"logger":"gitopsconfig-controller","msg":"retrying the GetJob for CheckResult"}
{"level":"info","ts":1578841086.369462,"logger":"gitopsconfig-controller","msg":"Retry of GetJob for CheckResult unsuccessful: Job.batch \"gitopsconfig-hello-world-helm-cfsv6c\" not found"}
{"level":"info","ts":1578841086.369468,"logger":"gitopsconfig-controller","msg":"The job has been deleted"}

Type of change

Bug fix (non-breaking change which fixes an issue)

Checklist

[ ] Unit tests and e2e tests updated
[ ] Documentation updated

@Smiley73 the code that updates the .status section of the GitOpsConfig CR is being rewritten as part of #217. @akavel will be submitting a pull request soon. I'd like to hold on merging this until we see the code changes from @akavel.

@seanmalloy understood, but I really need this fixed asap. This is causing us quite some delays in our work. I was hoping to start using this today already. We can discuss in-person.


restart server : accept tcp <IP_ADDRESS>:9507: use of closed network connection

Simulate a time-consuming request, restart during the request process, and find that the old process cannot wait for the request to complete before exiting

Reason found

Reason found
I'm experiencing the same problem with the following error

accept tcp [::]:5001: use of closed network connection

I'm using the framework gin and this error occurs in restart.

I think you can change the restart signal
On Sun, 6 Aug 2023 at 11:51 PM bourne-3 @.***> wrote:

Reason found
I'm experiencing the same problem with the following error
accept tcp [::]:5001: use of closed network connection
I'm using the framework gin and this error occurs in restart.
—
Reply to this email directly, view it on GitHub
https://github.com/jpillora/overseer/issues/78#issuecomment-1666867429,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAE2X47QDA67I6HEXJ27ZJTXT6OM5ANCNFSM53RL45SQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

@dxxjing what was the solution?
@bourne-3 did you solve this in combination with gin?

Solved by implementing graceful restart with Chi: https://github.com/go-chi/chi/blob/master/_examples/graceful/main.go


Add showChessLeaderboard command using chess.com public api

Add new command /showChessLeaderboard
Please read the CONTRIBUTING.md carefully before getting started!

Add a new javascript file in the commands folder
Implement a API request with the help of axios to the chess.com API to get the current high rated players
Required categories are live_bullet, live_blitz and live_rapid
Format the data in a readable way and send it to the user
Use the discord.js docs to get started

Hi, can I be assigned for this issue?

Sure, please go ahead @jeremiaaxel!

Great contribution! Please check the requested change in #15

Merged PR #15
The hacktoberfest-accepted label has been assiged to your PR @jeremiaaxel
Great work ❤


[SUGGESTION] More clouds/weather phenomena

Space is way more interesting to look at now, but how about we bring our attention more down to Earth? Perhaps it isn't in this mod's scope, but given this mod does a lot of work with the sky, it's worth a shot. I have a few ideas for this.

"Fog" cloud layer - A cloud layer that moves around between y=68 and y=100, filling towns with dense cloudcover making it hard to see, or providing a damp humid atmosphere for jungle excursions.
More volumetric clouds - Each cloud tile grows and shrinks in height over time as it travels across the world, making them appear a bit more interesting. Look up videos on a game called Vintage Story to see what I'm talking about.
Cirrus clouds - An additional flat cloud layer high above, with a more wispy appearance. The halos of the sun and moon become more pronounced when they shine through them.
Rainsquares - Colorful squares of refracted light that can appear while it is raining, or just after it has stopped raining, during the evening or morning.

As you said, that is not really within the scope, since it doesn't fit the theme.
Could work for a separate mod that would be compatible, but the first line of StellarView description is:
"Stellar View helps your sky look just a bit better while keeping the Vanilla feeling."
Overcomplicating the clouds entirely removes the Vanilla feeling. Which is not what I'm going for.

And also, yes, these could definitely be added as something you can toggle on, while being turned off by default to keep the vanilla feeling, that would definitely work.
But I feel like it would be a waste for something cool like this to be off by default.
Basically, it's either this becomes its own thing, or I change the direction of StellarView away from Vanilla.

I feel as though these various cloud layers would be functionally identical to the normal cloud layer, just with different cloud maps, resolutions, and what not.

I think it would be more interesting to play around with this idea, with the volumetric clouds and I especially wanna see if I can use the information about rain buildup in the world to render clouds differently, to possibly have a smoother transition between clear and rainy weather, where clouds build up. Not sure yet, we'll see.
I've probably said it elsewhere, but I've changed my mind on this ever since I made Stellar View much more configurable thanks to resourcepacks, meaning that stuff that would have otherwise been relegated to configs barely anyone changes can instead be used as enhancements to resourcepacks.


Error when trying to use subscription

Describe the bug
I get an error now when I try to use subscriptions:
ALTAIR
An error occurred in subscription.
Error: Cannot use GraphQLSchema "[object GraphQLSchema]" from another module or realm. Ensure that there is only one instance of "graphql" in the node_modules directory. If different versions of "graphql" are the dependencies of other relied on modules, use "resolutions" to ensure only one version is installed. https://yarnpkg.com/en/docs/selective-version-resolutions Duplicate "graphql" modules cannot be used at the same time since different versions may have different capabilities and behavior. The data from one version used in the function from another could produce confusing and spurious results.

This is new, since I worked with subscriptions in previous versions.
To Reproduce
Try to use a subscription from a server.
Expected behavior
A clear and concise description of what you expected to happen.
Desktop (please complete the following information):

OS: macOS
Platform:  desktop app
Version: <IP_ADDRESS>15

Hey @Sytten, I can't reproduce this issue. Could you perhaps provide some more details about your usage?

Ha ok, it is a backend error. I have being changing the WS protocol to graphql-ws. This is weird but not an issue of altair.


Mixed-prefix unification

Consider the following examples:
(* mixed_prefix_1 *)
  let (x : ∀ a.((∀ b.  a → a) → int)) = λ(y:∀ b.  a → a). 42 in
  let (z : ∀ b.  b → b) =  λw. w in
  x (~z)
(* mixed_prefix_2 *)
  let (x : (∀ a.((∀ b.  b → a) → int)) = λ(y:∀ b.  b → a). 42 in
  let (z : ∀ b.  b → Int) =  λw. 42 in
  x (~z)

The first one is designed to force the unifier to solve the problem exists X. forall a. a -> a == forall a. X -> X which should fail because unifying X with a should not be allowed if a was not in scope when X was introduced.
The second one is designed to force the unifier to solve a problem exists X. forall a. a -> Int == forall a. a -> X which should succeed with X == Int.
Currently (4e19f88) both fail with implementation bugs.  The failures seem related to the lack of handling quantified variables of type ascriptions on lambda-bound variables (example poly_binding_2 in #16 ).

After fixing #16 mixed_prefix_2 works as expected.

mixed_prefix_1 doesn't work as expected. Here's what happens. We have:
let (x : ∀ a.(∀ b. a → a) → Int) = λ(y:∀ b. a → a). 1 in
let (z : ∀ b. b → b) = λw. w in
x (~z)

We should reject x (~z) application. We freeze z - its type is ∀ c. c → c, where c is a skolem. We then unify it with ∀ b. a → a and unification succeeds since there's no reason at this point why it should fail. One potential solution to test would be to skolemize all free type variables. It would work in this case but I'm not sure whether this is a universal solution.
I'm also concerned about System F term produced for this example:
Pretty-printing the System F term...
  Λ30. let x = Λ31. λ(y : ∀ [31]. 31 → 31). 1 in
  let z = Λ10. λ(w : 10). w in
  x [30] z

This doesn't look correct. Concretely, the quantifier in the signature on y should be fresh and not 31.

mixed_prefix_1 should fail since there is no type we can instantiate a with that will result in a type unifying with forall c. c -> c - by capture avoidance whatever type we substitute for a in forall b. a -> a will not be able to mention b (if b occurs in the substituted type then the quantifier has to be renamed).
So, the behavior of the System F term is irrelevant since we shouldn't be producing it in the first place.
In the paper this is handled by renaming the quantified names when we unify two polymorphic types to common names.  (In the paper we do this one at a time but we can also easily do this for all pairs of names simultaneously.)  I think/hope that some mechanism analogous to ranks could be used to detect when an existential variable is unified with a universal variable that was not in scope when the existential was introduced, but I suspect the details to be tricky.

So, the behavior of the System F term is irrelevant since we shouldn't be producing it in the first place.

Yes., exactly.

In the paper this is handled by renaming the quantified names when we unify two polymorphic types to common names.

Names (ids) don't play a role during unification in inferno, i.e. we don't unify by name.

Here's a very good example that @jamescheney came up with:
(λx. let y : ∀ a. a → a = λz.x in y) 1 true

This is now in the testsuite as mixed_prefix_3.

@jamescheney, I implemented the solution mentioned in our yesterday call and it works. The idea is to prevent unification of skolems and existential variables (the latter are easily identified by having rank > 0). mixed_prefix_2 does not currently work but for a different reason, unrelated to mixed-prefix unification (although mixed-prefix unification exposes the problem). I'm closing this ticket and will open another one for the problem triggered by mixed_prefix_2.


Honda Bosch Radarless:  check cruise faults

Dependent on https://github.com/commaai/opendbc/pull/807

Only found two instances of this signal being high in the field out of 1k segments (all we have locally for radarless Civic), and they look like by products of some CAN issue. CAN valid also drops while this is happening, and POWERTRAIN_DATA is spotty. Both from one dongle. Looks like the signal is good (and radarless cars don't fault often)!

thanks for starting the PR @royjr!


Fix Calc's "uuid" library function when run from GUI

Currently gives this error:
uuid ⇨ "Unrecognized option \"-g\"!\r\n\r\nUsage: uuid [-lower][-upper][-string][-bytes][-int][-nn]\r\n\r\n   Default is lower-case string; default for bytes is upper-case.\r\n   Aliases for options are:\r\n     -lower = -lowercase, -low, -l\r\n     -upper = -uppercase, -up, -u\r\n     -string = -str, -s\r\n     -bytes = -byte, -by, -b\r\n     -int = -integer, -i\r\n     -nn means output nn unique values (1..99)"

Works okay from command line.  So, how does the "-g" flag get passed down (presumably from the "-g" flag passed to Calc itself)??

That flag is in "CMD_ARGS" in the environment.  That's why "_find_and_run_class.bat" uses it for running "uuid".  So, the exec function needs to remove CMD_ARGS from the environment before starting the child process...

Suggested code change (which has been tested) (lines 5285 ff. in CalcObjectVisitor):
		RunCommand cmd = new RunCommand(args);
		// OMG!  This is messy, but our wrapper scripts sometimes define CMD_ARGS
		// which will still be present in the environment, and cause the scripts to
		// get screwed up. So, explicitly remove it here.
		Map<String, String> env = cmd.environment();
		env.remove("CMD_ARGS");
		StringBuilder result = new StringBuilder();

There could be a "removeStdEnv" method inside RunCommand, which is a "builder" type that returns "this".
RunCommand cmd = new RunCommand(args).removeStdEnv();


chore(v2): upgrade dependencies + require Node 12

Motivation
Follow-up of #4148
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
Preview/E2E/Local dev.
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/facebook/docusaurus, and link to your PR here.)

1:20:23 AM: error<EMAIL_ADDRESS>The engine "node" is incompatible with this module. Expected version ">=10.17.0".

Hmm, one of the execa v5 dependencies requires node >=10.17.0, maybe we should switch to this version as minimum node version for our packages (given the EOL of 10.x ends in April https://github.com/nodejs/Release#release-schedule)?

Hmm, one of the execa v5 dependencies requires node >=10.17.0, maybe we should switch to this version as minimum node version for our packages (given the EOL of 10.x ends in April https://github.com/nodejs/Release#release-schedule)?

Yes, I think it's a good time to move to Node 12 min

A strange bug on WIndows CI - all tests pass, but exit code 1 is returned (see https://github.com/facebook/jest/issues/9324).
Test Suites: 68 passed, 68 total
Tests:       499 passed, 499 total
Snapshots:   254 passed, 254 total

this is weird, don't know why.
Maybe it's Jest 26 behavior?
We have this line in logs.

ReferenceError: You are trying to import a file after the Jest environment has been torn down.

Also someone reported this here: https://github.com/facebook/jest/issues/9324#issuecomment-613108424
Maybe worth trying to fix this error and see?

Giving it a try. I think it's related to bad async tests in Jest that leak resources.
yarn test packages/lqip-loader/src/__tests__/index.test.ts --detectOpenHandles
Running Jest with --detectOpenHandles should not produce errors

@slorber awesome, CI passed, thanks!

Do we need to install the new version of immer via resolutions field to fix #4191? When react-dev-utils updates this package, we can remove this resolution. WDYT?

I don't think resolutions applies to dependencies, only our own repo, and it's yarn only so it's unlikely to solve any problem imho

Alright, so I think the PR is ready for a merge.

thanks, LGTM


Update the eICU pipeline and get it fully working.

Needs to include:

[ ] Add the extract_metadata stage, if any metadata sources exist.
[ ] Add the finalize_MEDS_data stage for MEDS compatability.
[ ] Add the finalize_MEDS_metadata stage for MEDS compatability.
[ ] Validate outputs
[ ] Get both local parallel and slurm parallel working.

Dupe of #53


implement a tree search UI on modifiers and other OntologyClasses

Definition of done
implement using the available https://www.primefaces.org/primeng/treeselect tree select component to search through modifiers, and other ontology classes terms.

Hi @belkassaby the trees look great. I think one last missing item to make it perfect is to remove the top-level tree elements.

for Onset, we should not show the top-level Onset term, but only the children: Adult onset, Antenatal onset, etc.
for Modifiers, we should not show the top-level Clinical modifier, but only the children: Clinical course, Position, etc.

linked to PR #81


Derby driver class name for 10.15.X.X issue #1835

Description
Newer versions of derby use a new driver name, and the old driver is no longer available.
Fix for Derby getDefaultDriver() method to load appropriate driver classes based on db version.

Code review and test results:
Things to be aware of

Behavior difference is derby 10.15 vs. earlier
Uses the same "look for the different possible classes in the classloader" trick that we do in MysqlDatabase for a similar problem
Only impacts derby

Things to worry about

Nothing


⚠ Processors has degraded performance

In 9ea112b, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 76 ms

Resolved: Processors performance has improved in 5dfd7bf.


shell: net: Switch to synchronous getaddrinfo call

This patch fixes an issue with the 'net dns query' shell command.  The
asynchronous nature of the dns_get_addr_info call makes this unsuitable
for the use inside the shell.  Instead, let's use the synchronous
getaddrinfo to acquire DNS records.
Signed-off-by: Andy Gross<EMAIL_ADDRESS>

This patch fixes an issue with the 'net dns query' shell command.

What issue?

The
asynchronous nature of the dns_get_addr_info call makes this unsuitable
for the use inside the shell.

Why it's unsuitable?

@pfalcon It crashes when issuing a 'net dns query'.
The asynchronous nature of the call makes it difficult to get it working in the current shell implementation.  By difficult, I mean that I've wasted countless hours dealing with it and I am not going to spend another minute when I can use a synchronous call that works 100% of the time without failure.

Codecov Report

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

@@           Coverage Diff           @@
##           master   #13191   +/-   ##
=======================================
  Coverage   48.86%   48.86%
=======================================
  Files         317      317
  Lines       46810    46810
  Branches    10818    10818
=======================================
  Hits        22873    22873
  Misses      19366    19366
  Partials     4571     4571

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 f2ef52f...e67062b. Read the comment docs.

It crashes when issuing a 'net dns query'.

So, what exactly crashes? Is it dns_get_addr_info() issue, or regression in the new shell implementation, which perhaps needs to be fixed by the shell folks? We had that command implemented long ago, much earlier than getaddrinfo(), and nobody reported and issues about it.
My concern is that getaddrinfo() is implemented for applications wanting POSIX API. It incurs additional code size overhead, and will incur only more going forward, when more and more POSIX options will be implemented. The shell has quite different requirements - offer as much functionality as possible with as little code size as possible, and itself it's quite large, so that all contradicts usage of getaddrinfo(), if dns_get_addr_info() served us well.

I am wondering the same thing as @pfalcon. I think we need to fix the resolver if it causing crashes in shell. One good thing about the async DNS call is that the shell is fully usable while the resolving is done, although the timeout when using the sync version is probably not very long.
Anyway, using async DNS code in shell is one of the few users of the resolver code so it is good to leave it here for testing purposes. Beside the getaddrinfo() is basically calling the async version of the resolver so I am suspecting that the reason for crashes is not in DNS code but in shell integration of it. I can look into this next week, the fix is probably quite simple one.
As a summary, I am not big fan of this change but if people really want to have this done this way we can certainly change it.

@jukkar Paul is being assigned the work.

Closed this ticket.  We'll look forward to other presented solutions.

@agross-linaro : Shell is not able to print asynchronous messages. Is it still an issue?

We'll look forward to other presented solutions.

For references, it's traced in https://github.com/zephyrproject-rtos/zephyr/pull/13260 .

Shell is now able to print asynchronous messages. Is it still an issue?

It seems to work ok in my testing of https://github.com/zephyrproject-rtos/zephyr/pull/13260 . Thanks for confirming it's intended usage, @jarz-nordic (I know we discussed all this stuff, but shell kept changing all this time either. With 1.14 going into RC, and shell apparently "complete", it's really good time to finally make a mind snapshot of its properties).


README: Use SVG badges

This PR changes the build badges to SVG for sharper readability ✨
Also drops Gemnasium badge. Closed service.

Coverage decreased (-0.007%) to 99.051% when pulling 8db3edfb79008536e353bd25317e9569e9cbdbae on olleolleolle:patch-1 into b14a1664ed08775c6be21e698e2ca27d3a534000 on adhearsion:develop.

Failures are irrelevant, merging. Thanks!


Simplify development for "only frontend" work

Expected Behavior
User wants to develop frontend without setting up backend each time he/she wants to edit JS/HTML files.
Actual Behavior
Backend needs to be set up and working for developing frontend.
Additional comment
Consider simplifying backend environment. It should be as light as possible if the developer doesn't want to upload/label/verify scans.

It's not easy and won't be done in the near future.


Improvements to danger init - fixes #341 #389 #239

Fixes a few minor annoyances around the init feature.

          1 Warning

:warning:
Tests were not updated

  Generated by :no_entry_sign: danger