Masters/slaves will block when entropy is low

Description of issue / feature request
On systems (particularly vms) without much uptime, total entropy on the system can be very low, meaning that calls to random will block. Locust when started makes calls to random that will block, causing the application to freeze with no particular indication to the user.
One option is to use, or have an option to use, urandom. This will not block, although it comes at the expense of being "less" random. It may also be slower, but I'm not 100% sure that it is, or it is in a way that matters.
Expected behavior
Expected behavior is that locust masters and slaves not block with no message on boot.
Actual behavior
It blocks, if the entropy pool is low enough at the time of process start. Running with strace will show that it is blocking on the following:
open("/dev/random", O_RDONLY)           = 8
poll([{fd=8, events=POLLIN}], 1, -1

Environment settings (for bug reports)

OS: Unknown
Python version: 3.5+ (confirmed in 3.5.3, 3.6.8, 3.7.2, presumably also applicable to 2.7+ and 3.whatever+)
Locust version: 0.9.0

Steps to reproduce (for bug reports)

Use a working locustfile of any sort (contents are mostly irrelevant, it just needs to not produce errors on locust boot.
Exhaust the entropy pool (this Python one-liner will do it: with open("/dev/random", "rb") as f: while True:  f.read(8)
Confirm that  cat /proc/sys/kernel/random/entropy_avail shows a very low number (0-9).
Start locust with strace locust <rest of command>. If entropy is low enough, it will hang on poll([{fd=<some number> after an open("/dev/random", O_RDONLY call with a return value the same as <some number>

Is using haveged or something else a good enough workaround for you? I dont think many users will run into this...

Using haveged is sufficient, although it took quite a bit of work (at the time) to realize that that was something that needed to be done. If locust wanted to solve the effects of this problem more generally for people so that investigation is a little easier, but changing its source of entropy sounds like more work than is worth it, it might be as simple as a note in documentation with good-enough SEO that it turns up in some common searches for this. Hell, it might even be enough that this issue exists and is visible, honestly.

Thanks! Closing this, hopefully people will be able to find it anyway!


Editing The gutenblock

I must be using the block boilerplate incorrectly....
I have copied "01-single-block" from the examples folder to my plugins folder, and done the following:

cd 01-single-block
npm start
Make changes in the file 01-single-block/src/block/block.js
^c
npm run build
but the changes I make in block.js do not appear on the page using the block.  What am I doing wrong?

Examples are old and outdated. Read the docs and build a new block for your plugin instead of copying from the examples.
And if that doesn't work file a bug report answering all the questions asked.


’ character causes publishing to fail

Hi,
Inclusion of the ’ character in the content of the Asciidoc document causes publishing to Confluence to fail with the following exception:
Caused by: org.sahli.asciidoc.confluence.publisher.client.http.RequestFailedException: 400  POST https://confluence.clm-comarch.com/rest/api/content
request: ''
response: '{"statusCode":400,"data":{"authorized":false,"valid":true,"allowedInReadOnlyMode":true,"errors":[],"successful":false},"message":"Error parsing xhtml: Unexpected character '’' (code 8217 / 0x2019) (expected a name start character)\n at [row,col {unknown-source}]: [1,13223]","reason":"Bad Request"}'
	at org.sahli.asciidoc.confluence.publisher.client.http.ConfluenceRestClient.lambda$sendRequestAndFailIfNot20x$7(ConfluenceRestClient.java:209)
	at org.sahli.asciidoc.confluence.publisher.client.http.ConfluenceRestClient.sendRequest(ConfluenceRestClient.java:220)
	at org.sahli.asciidoc.confluence.publisher.client.http.ConfluenceRestClient.sendRequestAndFailIfNot20x(ConfluenceRestClient.java:206)
	at org.sahli.asciidoc.confluence.publisher.client.http.ConfluenceRestClient.addPageUnderAncestor(ConfluenceRestClient.java:93)
	at org.sahli.asciidoc.confluence.publisher.client.ConfluencePublisher.addOrUpdatePageUnderAncestor(ConfluencePublisher.java:167)
	at org.sahli.asciidoc.confluence.publisher.client.ConfluencePublisher.lambda$startPublishingUnderAncestorId$1(ConfluencePublisher.java:124)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1540)
	at org.sahli.asciidoc.confluence.publisher.client.ConfluencePublisher.startPublishingUnderAncestorId(ConfluencePublisher.java:123)
	at org.sahli.asciidoc.confluence.publisher.client.ConfluencePublisher.publish(ConfluencePublisher.java:80)
	at org.sahli.asciidoc.confluence.publisher.maven.plugin.AsciidocConfluencePublisherMojo.execute(AsciidocConfluencePublisherMojo.java:131)
	... 22 more

Asciidoc:
= System parameters

[%header,format=dsv,separator=;]
|===
include::system_parameters.csv[]
|===

CSV:
Name;Description;Type;Category;Pattern;Value TEST01;Value UAT;Value PROD
<EMAIL_ADDRESS>
Edit:
Seems like the problem is not with ’ character on its own but rather combination of &’. I guess Confluence, seeing & is attempting to read HTML entity and fails. The strange thing is &{ that I have a bit later in the file is not causing such problem, but both &’ and &* are :/

Hi @GauntletPL, thank you for using the Confluence Publisher.
I did some testing and to me, it seems as if the initial ^ and [ characters in the pattern string cause the issue: the ^ sign is the superscript sign in AsciiDoc, the [  is used e.g. for macro parameters. Escaping both characters with a \ properly shows the CSV content. I was able to reproduce this issue with pure AsciiDoc, independent of the Confluence Publisher.
Does this help in any way?

Yes, it definitely helps, thank you for your explanation.
It seems to be even broader issue with how Asciidoctor deals with tables with delimiter-separated content.
Table defined like this:
[cols="2*l",format=dsv, separator=;]
|===
_a_;*b*
^c^;~b~
|===

renders every cell while taking into consideration all the formatting syntax, regardless of the "l" styling option being used, while:
[cols="2*l"]
|===
|_a_|*b*
|^c^|~d~
|===

renders everything in an expected, unformatted manner. I'll bring that issue, directly to the Asciidoctor team's attention. Meanwhile, I'm closing this ticket.
Thank you again.


Permission denied!

Hi!

I write simple test:

import { Application } from<EMAIL_ADDRESS>
const app = new Application()

const test = 'Test string'
const encoder = new TextEncoder()
const data = encoder.encode(test)

app.get('/', async (c) => {
    await Deno.writeFile('test.txt', data)
    return test
})

app.start({
    port: 8080
})

it allows you to understand how the file is being created and written.

Dockerfile

FROM hayd/alpine-deno:1.7.2
EXPOSE 8080
WORKDIR /app
USER deno
ADD . .
RUN deno cache app.js
CMD ["run", "--allow-net", "--allow-read", "--allow-write", "app.js"]

Run command in your example: docker build -t app . && docker run -it --init -p 8080:8080 app

Without writing files, everything works fine, but if create and write files, I see an error:

I test this example with Alpine Linux & CentOS
I'm doing this for the first time, maybe it's a simple task, but please tell me how to fix it?

By default:
USER deno

won't have access to write files. This is by design.
What you should do is either remove the USER deno or set the permissions of /app (or another directory) to be writable by the deno user e.g. something like chown -R deno /app.

thank you very much!
You've helped me a lot!


mechanism to clear cache?

Is there a way to adjust the cache expiration for data urls?
In general, in maplibre data resources come from an external url, which can make it a bit more difficult to update maps dynamically.  If I understand correctly, the typical solution (provided we can update the remote data source) is to expire the cache so that the new data loads, i.e. set
'Cache-Control: 'max-age=0'

(perhaps as discussed in https://github.com/mapbox/mapbox-gl-js/issues/2633#issuecomment-576050636 ? I'm a bit out of my depth on this one).

@cboettig In most case you do not need to update your data with Map.set_data but use filters instead which makes it very comfortable:
m = Map(...)
m.add_layer(...)

# Update your layer using a filter on the 'growth' property of your source
filter_expression = [">=", ["get", "growth"], 0.5] # only show features where growth >= 0.5

m.set_filter("your_layer_id", filter_expression)

Furthermore, the upcoming release come with seamless support for  geopandas data frames for vector data sources.
See also changelog for unreleased version.

oh fantastic! native support for geopandas is much simpler than cache exploration.
Thanks for mentioning the filters, I need to spend more time exploring the performance implications there. Do you know if maplibre js is able to execute those as a range request without transferring all the data?  For the moment I'm relying on duckdb to do my filters, it's quite good at filtering 150gb parquet file in seconds, and figured I that would be difficult purely client-side.  But in at least some cases I'm probably better off using the filters.
That filter syntax is really nice too. @giswqs -- is something like m.set_filter() possible in leafmap.maplibre wrapper?  Guess that feels a bit more pythonic than putting it all in the style dict (though I guess something about configuring the map entirely with a dict is also kinda elegant in it's own way).

@cboettig Yes, range requests are possible:
["all", [">=", ["get", "growth"], 0.2], ["<=", ["get", "growth"], 0.5]]

In the next release I also added some helper functions for creating these kind of expressions / filters as you can see in this example.
In general, for large datasets it is good practice to use url sources, because maplibre only loads / queries the feature in the current viewport and for the filters set, it never loads all of the data.

@cboettig I would the close this issue now if it is solved for you with the expressions!?

Thanks @crazycapivara .  Yes, filtering with range requests is good, and as you say scales considerably better than embedding the geojson directly like what we get from passing the geopandas to pydeck instead of a URL.


Undo Place Tile with a Token on the Tile does not restore the Token

When a Yellow Tile that has a Token is upgraded to a Green Tile, and then an Undo is performed, the Token is not restored to the Tile on either the Local or the Remote Client.

Updates to collect, store and use the Token Index (from the list of Tokens) to properly retrieve the token on Remote Clients for both the Undo and the Tile Upgrade


Test support

This PR implements support for emulating spago test from nix's building environment.
N.B. It's currently built on top of several other branches. It will be rebased and declared ready for a proper code review once those land. This PR is sent out to facilitate a concrete albeit preliminary discussion about alternatives.
The only relevant commit is ba1b2c12f5e33133d6be0cd2fc9e70e5195c5fd2.
Motivation
Being able to run an equivalent of spago test from within nix makes CI easy.

Writing CI pipelines is simplified because there is no need for special testing steps. They can just be part of a single nix flake check/nix build .#my_website.
CIs are more reliable. The alternative of building with nix and testing separately is brittle because it's not clear that the artifacts/dependencies match in the two cases.

Design

Compile the test modules in checkPhase of buildSpagoLock by invoking purs, similarly to how it's today invoked from buildPhase. This is the right place for it because the compilation cache is readily available. The nix package will refuse to build if tests fail.
Require user to explicitly turn on testing of the packages in their workspace by passing the names of those packages to buildSpagoLock.
Rely on slimlock to build npmDependencies for testing. Expose them by changing the user-facing API of buildSpagoLock to return  { jsArtifacts, npmDependencies }, where

jsArtifacts: map from package names to derivations with output/ directory. This is what buildSpagoLock returns today.
npmDependencies: the output of slimlock ran on $src/package-lock.json.

Considerations

It's good to expose npmDependencies to the user, so that they can readily use them in the bundling step. This increases confidence that the node module used in tests and in the bundle are identical.
If the user doesn't reference npmDependencies and doesn't enable tests, they won't be built due to nix's laziness.
OTOH, if they are to be built, the current implementation expects to find slimlock in scope, which requires the user to have installed it via overlay. This is to allow user not to depend on it, when it's not needed.
super.callPackage is changed to self.callPackage to allow the user to inject the slimlock. Seems correct to have it this way for other reasons too (e.g. allow user to override the node version), but I'm not 100% confident this won't lead to looping in some edge case.
Test.Main is currently hardcoded as the main module of the test because we have no way of using fromYAML on spago.yaml (it may contain comments). spago could be modified to include this information in the spago.lock.

Alternatives

Instead of calling slimlock allow the user to pass in the node dependencies explicitly.

Pros

More flexible for the user.
Doesn't break the API.
No confusion about errors stemming from slimlock missing from the overlays, when testing is enabled.

Cons

Requires more boilerplate.
Possible source of bugs, if the node modules are built erroneously (e.g. node modules only relevant for the test dependencies are missing for some reason). This is a contrived point.
The exact directory of the node modules becomes magical and brittle: e.g. slimlock puts them into js, node2nix uses lib.

Use backend-optimizer

Pros

Faster tests
May uncover bugs in backend-optimizer (unlikely)

Cons

Slower builds
Complexity

Call for reviewers! Per our preliminary conversation with @thomashoneyman the way npm dependencies are handled might need to change. This PR is currently producing the node dependencies internally using slimlock (which should be simple for the end user), but a more flexible alternative is to require the user to explicitly pass them to the buildSpagoLock function.
That being said, I sent the PR for review as is, to make sure I collect all the feedback before addressing the reviewer comments.


Proposal: Prevent mob spawning outside world border

A feature suggested by a player on my server; this proposed config toggle will allow WorldBorder to prevent mob spawns outside any border. Technically, it is only listening for CreatureSpawnEvent which affects:

Natural spawns
Spawn eggs
Forced spawns (e.g /spawnmob zombie 10)

Use cases

Better wither skeleton grinders - If a nether fortress is close to or intersects a world border, players would be unable to prevent mob spawns outside with half slabs or other methods. Thus, spawns outside the border would interfere with any grinders.
Optimization - Mobs that spawn outside of the border are most likely inaccessible to players; thus, it would be a waste of server resources
Matches vanilla behavior - 1.8's native worldborder prevents mob spawning outside the border, thus this config will allow WorldBorder to more closely match vanilla behavior

Implementation
I have implemented this config generally by duplicating code of the preventBlockPlace feature; some optimizations can be made (e.g. de-duplicate MobSpawnListener's code).
Like preventBlockPlace, it is false by default (meaning mob spawns are allowed outside the border) to prevent confusion for updating users. Included is a command and permission node for toggling this feature.
Out-of-scope changes

I hit a NPE in WorldBorder.java when disabling any of the prevent features, if they were already disabled. I changed the code to prevent this from happening.
I used IntelliJ to develop my changes; I added IntelliJ files to .gitignore for convenience of other IntelliJ contributors

Tests performed

Plugin compiles and runs on Spigot 1.8.3
/wb preventmobspawn on and off work opped
worldborder.preventmobspawn correctly blocks command unless granted
/wb preventblockplace on and off work with no regressions
When feature is on, mobs outside border are correctly blocked from spawning
Config is correctly updated when setting is changed

Looks great, thanks for the contribution.

No problem, thank you for the merge!


docs: populate Collection from entity constructor

I can't find this anywhere unfortunately. Following the example from the docs, how to populate set tags from within the constructor? It seems adding tags: Collection<BookTag> to the parameter list doesn't cut it; it stays undefined. I am using forceEntityConstructor: true.
@Entity()
export class Book {

  @PrimaryKey()
  _id: ObjectID;

  @Property()
  title: string;

  @ManyToOne()
  author: Author;

  @ManyToOne()
  publisher: Publisher;

  @ManyToMany({ entity: () => BookTag, inversedBy: 'books' })
  tags = new Collection<BookTag>(this);

  constructor(title: string, author: Author) {
    this.title = title;
    this.author = author;
  }

}

I don't understand what you mean, you can't "populate a collection from constructor", that is an async action, and constructors are synchronous.


Unable to query containers and kill executors on mesos-local when not on linux

https://github.com/ContainerSolutions/mini-mesos/blob/master/src/main/java/org/apache/mesos/mini/mesos/MesosContainer.java#L83
This line communicates directly with the docker daemon on mesos-local. Because Mac users are using docker-machine, this traffic needs to be proxied through docker machine.
See https://github.com/mesos/elasticsearch/blob/master/system-test/src/systemTest/java/org/apache/mesos/elasticsearch/systemtest/ExecutorSystemTest.java#L52
for an example

Obsolete


🐛(back) compute base static url from an existing file

Purpose
In commit b22477c we tried to fix how we compute the static base url.
In our tests it was working but we didn't test it in Production
environment. In the environment we tested it, the DEBUG mode was
activated and in this mode the S3Boto3Storage class storage always
return a result even if the requested file does not exist. To fix our
issue we choose to request an existing file: js/index.js
This file is the frontend entrypoint. It will always exist
until the frontend application choose to change it...
Proposal

[x] request an existing file to compute static base url.

This PR is closed because we choose to fix the issue another way in #538


fix: update deno docs to use correct preset value

Deno package has been removed from the repo. Thank you for you contribution regardless.


Target Visual Studio 2012

We are currently building only for VS2013, it should be a matter of changing definitions in .vsixmanifest and actually testing generated .vsix. Since both 2012 and 2013 version targets .Net 4.5 it won't affect the code.

This would be very handy to have.

This is not possible with the new project system. Closing as wontfix.


Error install autossub

Command "python setup.py egg_info" failed with error code 1 in /private/var/folders/md/qs2sj3q52qv9h4xjvtp_66bw0000gn/T/pip-install-6yo738md/progressbar/

I get the same error:
Collecting autosub
  Using cached https://files.pythonhosted.org/packages/f3/31/d3d677e97ec3e24ae6131df91eb5d20650309e3ed5122361613792180fb7/autosub-0.3.12.tar.gz
Collecting google-api-python-client>=1.4.2 (from autosub)
  Using cached https://files.pythonhosted.org/packages/0e/7b/0adbc5ee668e9006422c38ee6b7b92c018b03839fc7a11b18cc51054e88b/google_api_python_client-1.6.6-py2.py3-none-any.whl
Collecting requests>=2.3.0 (from autosub)
  Using cached https://files.pythonhosted.org/packages/49/df/50aa1999ab9bde74656c2919d9c0c085fd2b3775fd3eca826012bef76d8c/requests-2.18.4-py2.py3-none-any.whl
Collecting pysrt>=1.0.1 (from autosub)
  Using cached https://files.pythonhosted.org/packages/f6/33/16ad65a8973cb8bcb494af09ee1b9ab5ffdd6ff300bce5d3ac7d3cb1f2cc/pysrt-1.1.1.tar.gz
Collecting progressbar>=2.3 (from autosub)
  Using cached https://files.pythonhosted.org/packages/0f/e1/f30b72ecabba259c7c38dd0eb944a173effced3fd7de2c9c2902bd6f649a/progressbar-2.3.tar.gz
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/private/var/folders/xc/s0vn39m13lx1s4dcq4mx0kw00000gn/T/pip-install-i4csglhi/progressbar/setup.py", line 5, in <module>
        import progressbar
      File "/private/var/folders/xc/s0vn39m13lx1s4dcq4mx0kw00000gn/T/pip-install-i4csglhi/progressbar/progressbar/__init__.py", line 59, in <module>
        from progressbar.widgets import *
      File "/private/var/folders/xc/s0vn39m13lx1s4dcq4mx0kw00000gn/T/pip-install-i4csglhi/progressbar/progressbar/widgets.py", line 121, in <module>
        class FileTransferSpeed(Widget):
      File "/usr/local/Cellar/python/3.6.4_3/Frameworks/Python.framework/Versions/3.6/lib/python3.6/abc.py", line 133, in __new__
        cls = super().__new__(mcls, name, bases, namespace, **kwargs)
    ValueError: 'format' in __slots__ conflicts with class variable

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /private/var/folders/xc/s0vn39m13lx1s4dcq4mx0kw00000gn/T/pip-install-i4csglhi/progressbar/```

Environment:

1. MacOS 10.3.3
2. Python 3.6.4
3. Pip 10.0.1

I got the same error.
win10
python3.6

I got the same error on Ubuntu. Here's the work around:
git clone https://github.com/agermanidis/autosub.git
cd autosub
python2 setup.py install --user
autosub -h


Added localisation for the language of user

Currently language of user is not translated and is shown in english only . This PR added translation for the language of user .
Current behaviour:

Expected behaviour:

Apart from the language change in users view. I wonder if the double translation can be implemented in the dropdown following Sugarizer #293.

Yes , check #146 for this

Apart from the language change in users view. I wonder if the double translation can be implemented in the dropdown following Sugarizer #293.
done in #146


chrome can't upload and download with quiche nginx（php7.4 with yii2）

chrome:version 81,83
quiche:quiche-81bc973,quiche-98757ca
language:php7.4.6 with yii2 2.0.35
http3:h3-25,h3-27
export excel using kartik-v/yii2-grid
when post upload and some downoad,the server is not response.
it's happen on export data to excel file or post to upload file.

Does this problem still exist? We'd need more information in order to troubleshoot, such as a minimal reproduction, and client and server logs.

now is fine

I build nginx version: nginx/1.16.1 (quiche-f6608a9)
the problem again happen
2020/07/14 12:40:37 [warn] 32#32: *83 a client request body is buffered to a temporary file /usr/local/nginx/client_body_temp/0000000002, client: <IP_ADDRESS>

2020/07/14 12:40:37 [warn] 32#32: *83 a client request body is buffered to a temporary file /usr/local/nginx/client_body_temp/0000000002, client: <IP_ADDRESS>

Do you have proxy_request_buffering off; in your nginx.conf? See https://github.com/cloudflare/quiche/blob/master/extras/nginx/README.md#running

client_body_buffer_size is the same as client_max_body_size to upload, but not require with http2

proxy_request_buffering set fo off always exist problem,set client_body_buffer_size can solve.
i used
    client_max_body_size 50m;
    client_body_buffer_size 50m;
    proxy_request_buffering off;

The above is the chrome
than the firefox 78,No matter how you do it, you can't upload it

set proxy_request_buffering off,only proxy_pass is affected, not fastcgi_pass

I'm add fastcgi_request_buffering off;


transitionIssue example needed

I am trying to transition issue to done state using below code
jira.status.getAllStatuses({
   issueKey: task.ticket
}, function(err, statuses) {
   var doneStatus = (statuses.filter(function(status) {
      return status.name === 'Done';
   }))[0];
   jira.issue.transitionIssue({
     issueKey: task.ticket,
     transition: { id: doneStatus.statusCategory.id }
   }, function() {
     console.log(arguments);
  });
});

But this does not seem to be working. Any example would be very helpful.
Thanks in Advance.

@svapreddy I just hit the same issue; PR is in for a fix.

@greggilbert Thanks!  Merged and published as 1.7.0

Thanks guys :+1:


Incorrect su usage for FreeBSD

Using su results in hanging processes running:
su root -s /bin/sh -c stat /etc/installurl 1> /dev/null 2> /dev/null && (stat -c 'user=%U…
This is most likely because -s does not mean something like "shell", it means "Set the MAC label to the user's default label as part of the user credential setup" (so does not take an argument), so we end up with sh launched:
ec2-user@freebsd:~ $ su root -s /bin/sh -c stat /etc/installurl
root@freebsd:/home/ec2-user #

While this works:
ec2-user@freebsd:~ $ su root -s -c stat /etc/installurl
1895890688 94 crw--w---- 1 ec2-user tty 94 0 "Jul 23 12:36:27 2020" "Jul 23 12:36:27 2020" "Jul 23 12:36:27 2020" "Dec 31 23:59:59 1969" 4096 0 0 /dev/pts/0
ec2-user@freebsd:~ $ su root -c stat /etc/installurl
1895890688 94 crw--w---- 1 ec2-user tty 94 0 "Jul 23 12:36:27 2020" "Jul 23 12:36:27 2020" "Jul 23 12:36:27 2020" "Dec 31 23:59:59 1969" 4096 0 0 /dev/pts/0

@myfreeweb
I would like to see the example deploy that is causing this problem for you.  I agree the issue is su is handled differently in FreeBSD. However, I still feel you are using it incorrectly. That's why I want to see what your deploy looks like.

How can I use some.thing(…, su_user='root') incorrectly? And how would anything in the usage code affect how the command is generated?

Sorry, I wasn't clear.  I'm referring to your While this works section.  I'm assuming this is you running the su string on the command line.  If this assumption is incorrect, I'm sorry.
More detail on why I feel your While this works is incorrect (assuming this is you running on the cli)
-c stat seems like you are wanting to use the stat command on /etc/installurl.  However, you can see in your output the file is not /etc/installurl, instead the file is /dev/pts/0.
I understand the issue with su specifically in this case. However I would still like to see your deploy code so I fully understand how to reproduce this.

Oh. Yes, the assumption is correct. Right, I read this as "you are using it [the deploy] incorrectly" lol.

However, you can see in your output the file is not /etc/installurl, instead the file is /dev/pts/0.

hmmmm.
Oh okay so I understand it now.
https://www.freebsd.org/cgi/man.cgi?su#end
The -c at the end is supposed to pass to the shell, but the shell expects one argument, so quoting is required.
su root -c 'stat /etc/installurl'

so I fully understand how to reproduce this

pkg.packages(packages=['git'], su_user='root') is the first command (and the one that fails), I thought this would fail with anything with su_user.

I think what is happening here is the su command requires a password. There is no way to pass a password to the su command. Therefore it prompts for the password.  That's why the command is blocking. It's waiting for a password.
Instead of using su, you'll want to use sudo, which you can pass the password.

I confirmed this does work on a freebsd virualbox where the ssh user is osboxes.
pkg.packages(packages=['git'], sudo=True, sudo_user='root')

No, the EC2 AMIs do not have a root password! If I had to enter a password when trying the commands manually, I would've noticed that ;)
I already mentioned that the problem might be lack of quoting.

Instead of using su, you'll want to use sudo

sudo is not installed by default, so I'd need to su to install sudo.
Of course I could do that with an external ssh command before launching pyinfra, but that's not great :/

I understand your point on quoting.  I looked into that and it looks fine.  My testing leads me to believe the command is blocking for a password to the su command.
Try this for me.  Obtain a cli as your non-root user on the freebsd system. Then run:
su root
What do you see? I believe you'll see a prompt for a password.
This is what I see.
$ su root
Password:
root@osboxes:/usr/home/osboxes

This should now be fixed by https://github.com/Fizzadar/pyinfra/commit/0c7234be0c3d02172da63f843260851b7ff66794! I've released this immediately as 1.1.dev2 and, assuming no further issues, will release 1.1 this week :)

This has now been released in v1.1 🎉!


switch to io

ioutil is deprecated

Codecov Report

Merging #85 (dd082f9) into main (64d13bc) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #85   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           53        53
  Lines          909       909
=========================================
  Hits           909       909

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 64d13bc...dd082f9. Read the comment docs.


[Snyk] Security upgrade jest from 23.6.0 to 24.0.0

Snyk has created this PR to fix one or more vulnerable packages in the `npm` dependencies of this project.

Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

packages/ffe-formatters/package.json

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Breaking Change
Exploit Maturity

589/1000  Why? Has a fix available, CVSS 7.5
Prototype Pollution SNYK-JS-MERGE-1040469
Yes
No Known Exploit

494/1000  Why? Has a fix available, CVSS 5.6
Command Injection SNYK-JS-NODENOTIFIER-1035794
Yes
No Known Exploit

601/1000  Why? Proof of Concept exploit, Has a fix available, CVSS 5.6
Prototype Pollution SNYK-JS-YARGSPARSER-560381
Yes
Proof of Concept

506/1000  Why? Proof of Concept exploit, Has a fix available, CVSS 3.7
Regular Expression Denial of Service (ReDoS) npm:braces:20180219
Yes
Proof of Concept

(*) Note that the real score may have changed since the PR was raised.
Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Coverage remained the same at 82.009% when pulling b74540781130788f3fc5f0a8330e843b63811dc3 on snyk-fix-a1731c36bc21c7409176d1b788ac2e7a into d743d78bab0f6a4ce0bc199e8e419261247ca8b6 on develop.


fix(ramp sdk): On ramp not working

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type

[x] Bugfix
[ ] Feature
[x] Refactoring

Context
Just add the correct api key under RAMP_API_KEY

[x] Closes #6722
[x] Closes https://github.com/kodadot/nft-gallery/issues/6864

Did your issue had any of the "$" label on it?

[x] Fill up your DOT address: Payout

Community participation

[x] Are you at KodaDot Discord?

Screenshot 📸

[ ] My fix has changed something on UI;

Copilot Summary

🤖 Generated by Copilot at 01fdac7
This pull request improves the integration with the Ramp Instant SDK, a service that allows users to buy crypto with fiat. It introduces a new composable useRamp that handles the initialization of the SDK with the appropriate options and the API key. It also updates the existing components that use the SDK to leverage the new composable and the runtime config.

🤖 Generated by Copilot at 01fdac7

To buy crypto with fiat, we use Ramp
But our code was a bit of a tramp
So we made a composable
Called useRamp, so lovable
And refactored our code to be champ

@roiLeo  pls review again made the changes that you requested , also did you provide a fresh working api key ?

@roiLeo pls review again made the changes that you requested , also did you provide a fresh working api key ?

I provide the one we were using

I think then we have to wait , a new valid api key is needed https://github.com/kodadot/nft-gallery/pull/6727#issuecomment-1679375853

I've got reply they are already doing check on Kodadot, 🤞 everything will pass soon

checking your PR, seems it should work, but doesn't suspicious, meanwhile probably will look for other providers probably
https://github.com/kodadot/nft-gallery/pull/6727/files#diff-65d5d69bb59dcc81ad971719549cdee5018f0a240add8b7ef5c65bee3e73dae0R446

Pinged Ramp again, meanwhile we've been approached by another provider, will have updates tomorrow

#6930

So key is there..

I think bc we've got key to this only for now until we will pass KYB

https://docs.ramp.network/testing-environment

afk atm but this won't work url is not a param there gotta pass it here

afk atm i'd do it myself, but this won't work url is not a param there gotta pass it here

yes, I've checked it's the wrong place 😅 thank you for suggestion!

committing from PR like chad, lol.
Anyway gonna have look on KYBs, both operators are quite slow looks like that

ok, works!
but before merging we need remove that URL probably.

they are back from "long holiday", so we've implemented feedback they had

https://github.com/kodadot/nft-gallery/pull/7372

looking forward

@yangwao so now prod keys work ?

Hello can you please pull the latest changes an open pr against nuxt branch?

sure

sorry @roiLeo I didn't see you already approved

@yangwao resolved merge conflicts, also removed the   url: 'https://app.demo.ramp.network', now it should only need the new api keys

Nah, they are taking too long again to reply with keys.
I would merge it and then will update keys to production ones on nuxt
I believe we will migrate sooner to nuxt than Ramp will get back to me.
Thanks @hassnian !
pay 70 usd


Critical dependency warning, hot reload not working

monaco-editor version: 0.12.0
Browser: Chrome
OS:  Windows
Steps or JS usage snippet reproducing the issue:

Integrated monaco via ESM package by following the steps outlined in option 2 of: https://github.com/Microsoft/monaco-editor/blob/master/docs/integrate-esm.md
After loading the browser, I see the following warning. Looks like I am also not able to hot reload in the browser as well:

[HMR] bundle has 1 warnings
client.js:189 ./node_modules/monaco-editor/esm/vs/editor/common/services/editorSimpleWorker.js
378:12-387:17 Critical dependency: the request of a dependency is an expression
 @ ./node_modules/monaco-editor/esm/vs/editor/common/services/editorSimpleWorker.js
 @ ./node_modules/monaco-editor/esm/vs/editor/editor.worker.js

I don't know a lot about hot reloading and what could break that. I see the warning too when I execute webpack. But that is handled by creating those multiple entry points and loading works correctly for me (e.g. https://github.com/Microsoft/monaco-editor-samples/tree/master/browser-esm-webpack):
All those *.worker can be loaded at that location. Their names are dynamic because you get to choose the names.
entry: {
    "app": './index.js',
    // Package each language's worker and give these filenames in `getWorkerUrl`
    "editor.worker": 'monaco-editor/esm/vs/editor/editor.worker.js',
    "json.worker": 'monaco-editor/esm/vs/language/json/json.worker',
    "css.worker": 'monaco-editor/esm/vs/language/css/css.worker',
    "html.worker": 'monaco-editor/esm/vs/language/html/html.worker',
    "ts.worker": 'monaco-editor/esm/vs/language/typescript/ts.worker',
  },

Hi,
I also had the same issue and posted already a solutions here
new webpack.ContextReplacementPlugin(
	/monaco-editor(\\|\/)esm(\\|\/)vs(\\|\/)editor(\\|\/)common(\\|\/)services/,
	__dirname
)

Still suspecting win.base.js for it. 😤  Anyway, hope I could help!


Various assembly-related fixes

removed dependency on private EAP/WildFly modules
simplified the assembly files by sharing common config between both community and productized builds
removed productized WildFly 8 assembly as it not needed (not supported by product)
using different classifier for productized builds so that we can have unique coordinates and we can deploy both community and productized WARs in to single Maven repo

I checked the content of the WARs between and after the changes and apart from the expected changes (more jars in WEB-INF/lib for EAP war) the WARs look exactly the same.
See https://github.com/droolsjbpm/kie-wb-distributions/pull/55, https://github.com/droolsjbpm/kie-wb-distributions/pull/52 and https://github.com/droolsjbpm/kie-wb-distributions/pull/51 for similar fixes for KIE Workbench.

Please review, but don't merge yet. I have just found some issues with the ironjacamar dependencies for EAP/WildFly. I'll need to fix those before the PR can be merged.

I fixed the issue with ironjacamar by including the -core-api and -core-spi jars into the WAR.


add EL8 support

I added centos8 support. It works on my systems with selinux enabled.
I could not get tests for centos7/8 to run inside docker though.

any chance of getting this merged?

I look at it in the next days and merge it.


✨ fastapi webhook hmac validate dependency

close #165

add folder fastapi_extensions  instead of fastapi to avoid namespace conflict
add authentication.py module for shopify webhook hmac validation to be used by fastapi dependency
add tests

ignore the tests in tests/fastapi_extensions when fastapi is not installed

Ok I just saw that you are triggering an exception! My bad!
I guess we need to return something and the HMAC has no new info to return so bool is fine.

any comment @hillairet ?


create.t3.gg appears to be down. Can anyone else confirm?

Provide environment information
N/A
Describe the bug
N/A
Reproduction repo
N/A
To reproduce
N/A
Additional information

You can use https://create.t3.wtf/ temporarily. Tweet thread for more info: https://twitter.com/t3dotgg/status/1681386508559978497

Cool, thanks!

was an issue with the domain registrar. it's back now :)


fix(Resource Controller): re-gen service to get new Pagers

PR summary
This commit contains a re-gen of the Resource Controller service to leverage the new Pagers emitted by
the SDK generator.
Signed-off-by: Phil Adams<EMAIL_ADDRESS>PR Checklist
Please make sure that your PR fulfills the following requirements:

[x] The commit message follows the Angular Commit Message Guidelines.
[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

Current vs new behavior

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

:tada: This PR is included in version 0.29.3 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Please invite me to the GitHub Community Organization

Name
Callbackcat
Discord Username (if applicable)
No response
Additional Context
Explore

Check your email or organisation page
Thank you for contribution 🙂
Please contribute in Unique-Github-Profiles


do not show diff for credentials file

Currently credentials are being written in plaintext to syslog any time the credentials file changes, which is not ideal. This PR suppresses output when modifying credential files so that this will no longer happen.
I have a temporary solution in place using a resource collector to disable show_diff, but this seems like it should probably be the default behavior NOT to display the password in the log.

Funny, just ran into this and noticed I was too late for a PR :-)
@jdowning Would you mind tagging a bugfix release for this?
🍻

@mpdude 😬 Better late than never? 😎  https://forge.puppet.com/jdowning/awscli/2.2.0/readme


Add resourceType scalar in typespec-azure-core library

We also need a resourceType scalar to make a specific scalar to be the arm resource type.
DotNet SDK will generate resourceType into Azure.Core.ResourceType.
Other languages could simply generate it as string.
The definition could be
scalar resourceType extends string;

And the TrackedResource, ProxyResource template needs to be updated accordingly - their type property should be resourceType.

Also, this should change the type of the allowedResourceTypes array in the armResourceIdentifier template

est: 5


Tensorflow-gpu wont install Windows 10

Please make sure that this is a build/installation issue. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:build_template
System information

Windows 10  Home Ver 1803
TensorFlow installed from pip
TensorFlow version: 1.12.0 (tensorflow-gpu)
Python version:3.6.3
Installed using virtualenv? pip? conda?:pip
CUDA/cuDNN version: 10.0.132
GPU model and memory: GTX 1080

Describe the problem
Tensor flow-gpu is not properly installed.
Provide the exact sequence of commands / steps that you executed before running into the problem
This is the only code
import tensorflow
Thats it!!
If I install tensorflow-gpu and install tensor floe for the cpu it all works.
ImportError: DLL load failed: The specified module could not be found.
Failed to load the native TensorFlow runtime.
The entire traceback is listed below.
Installing the tensor flow
That is as basic as you get  it states
Any other info / logs
Python 3.6.3 (v3.6.3:2c5fed8, Oct  3 2017, 18:11:49) [MSC v.1900 64 bit (AMD64)] on win32
Type "copyright", "credits" or "license()" for more information.

RESTART: C:\Users\BillS\AppData\Local\Programs\Python\Python36\tensorgputest.py
Traceback (most recent call last):
File "C:\Users\BillS\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\pywrap_tensorflow.py", line 58, in
from tensorflow.python.pywrap_tensorflow_internal import *
File "C:\Users\BillS\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 28, in
_pywrap_tensorflow_internal = swig_import_helper()
File "C:\Users\BillS\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 24, in swig_import_helper
_mod = imp.load_module('_pywrap_tensorflow_internal', fp, pathname, description)
File "C:\Users\BillS\AppData\Local\Programs\Python\Python36\lib\imp.py", line 243, in load_module
return load_dynamic(name, filename, file)
File "C:\Users\BillS\AppData\Local\Programs\Python\Python36\lib\imp.py", line 343, in load_dynamic
return _load(spec)
ImportError: DLL load failed: The specified module could not be found.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:\Users\BillS\AppData\Local\Programs\Python\Python36\tensorgputest.py", line 1, in
import tensorflow
File "C:\Users\BillS\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow_init_.py", line 24, in
from tensorflow.python import pywrap_tensorflow  # pylint: disable=unused-import
File "C:\Users\BillS\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python_init_.py", line 49, in
from tensorflow.python import pywrap_tensorflow
File "C:\Users\BillS\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\pywrap_tensorflow.py", line 74, in
raise ImportError(msg)
ImportError: Traceback (most recent call last):
File "C:\Users\BillS\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\pywrap_tensorflow.py", line 58, in
from tensorflow.python.pywrap_tensorflow_internal import *
File "C:\Users\BillS\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 28, in
_pywrap_tensorflow_internal = swig_import_helper()
File "C:\Users\BillS\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 24, in swig_import_helper
_mod = imp.load_module('_pywrap_tensorflow_internal', fp, pathname, description)
File "C:\Users\BillS\AppData\Local\Programs\Python\Python36\lib\imp.py", line 243, in load_module
return load_dynamic(name, filename, file)
File "C:\Users\BillS\AppData\Local\Programs\Python\Python36\lib\imp.py", line 343, in load_dynamic
return _load(spec)
ImportError: DLL load failed: The specified module could not be found.
Failed to load the native TensorFlow runtime.

TensorFlow is tested against CUDA 9.0, to use CUDA 10.0 you have to build it yourself from the sources. The easier alternative would be to switch to CUDA 9.0 and build again. Thanks!

OK,
Any idea when 10 will be supported.
Will there be any issues with reverting to an older version om my system.
Should I unistall Tensorflow-gpu before switching to ver - 9


Designing Footer Component

Designing Footer Component

@codewithvk May I work on it?

@codewithvk May I work on it?

@codewithvk can I Work on this issue?

@Castella1313 & @vedikaag99  Go ahead!


Rework vformat to use variadic templates

Now scales better and does not perform unnecessary if statements

@hbina Is this still desired? If so, it needs to be rebased on the latest master branch.

@aaronfranke I have rebased both.

@hbina  Is this still desired? If so, it needs to be rebased on the latest master branch (again, I know...).

@hbina  Is this still desired? If so, it needs to be rebased on the latest master branch (again, I know...).

For the reference, I think this change is a good one. It came a bit early and couldn't be merged in 2019 as we do not use variadic templates or other modern features in the 3.2 codebase.
But in master Variant has been heavily rewritten to make use of variadic templates and vformat can definitely get the same treatment. This PR will need to be rebased indeed (or redone from scratch on latest master).

For the reference, I think this change is a good one. It came a bit early and couldn't be merged in 2019 as we do not use variadic templates or other modern features in the 3.2 codebase.
But in master Variant has been heavily rewritten to make use of variadic templates and vformat can definitely get the same treatment. This PR will need to be rebased indeed (or redone from scratch on latest master).

Discussed quickly on IRC to confirm, @reduz also thinks it's a good idea, and suggests doing the same for print_line (and likely other print_* debug statements).

Discussed quickly on IRC to confirm, @reduz also thinks it's a good idea, and suggests doing the same for print_line (and likely other print_* debug statements).

See also #35964

See also #35964

Ok, what about this? I just transformed the constant to a constexpr. Someone more knowledgeable said this
strangely enough static const int MAX_OCTAVES = 9; doesn't count as a definition there for the purpose of odr-use.

whatever that means

Ok, what about this? I just transformed the constant to a constexpr. Someone more knowledgeable said this
strangely enough static const int MAX_OCTAVES = 9; doesn't count as a definition there for the purpose of odr-use.

whatever that means

But why are you changing the constants? This PR is about vformat method, don't do unrelated changes.

But why are you changing the constants? This PR is about vformat method, don't do unrelated changes.

It doesn't compile, there are unresolved symbols (likely the void collect_args(const Array &); function, although not sure how it should be done).

It doesn't compile, there are unresolved symbols (likely the void collect_args(const Array &); function, although not sure how it should be done).


Upgrade to latest Consul version

Hi!  Would you mind upgrading to the latest Consul version in your project.json?

ok


[salvage]simplification & specialization for Sqlite Only

No more dependencies on SQLclient or Postgre

Sorry... bad manipulation... i thought it was on my forked repository ;-)
DO NOT MERGE.... I've salvagely delete all what referenced SQL and Postgre...


Docs broken about branch

The docs contains this:
// `hasLoaded()` is a function that returns whether or not the component
// has all the props it needs
const spinnerWhileLoading = hasLoaded =>
  branch(
    props => !hasLoaded(props),
    renderComponent(Spinner) // `Spinner` is a React component
  )

// Now use the `spinnerWhileLoading()` helper to add a loading spinner to any
// base component
const enhance = spinnerWhileLoading(
  props => props.title && props.author && props.content
)
const Post = enhance(({ title, author, content }) =>

{title}

By {author.name}

{content}

)

But branch doesn't support only 2 arguments. I saw that there were a PR about this but it's not merged. https://github.com/acdlite/recompose/pull/227
Please publish the new functionality or change the docs!

Merged,

not published yet.
Documentation updated according to PRs, so the problem in releases,
having that only @acdlite can publish this repo, I can't say much when, why, who.

PS: Until release just use latest tag for the documentation
https://github.com/acdlite/recompose/tree/v0.20.2

Oops,
now I can publish on npm,
so new version available on npm 0.21.0, please check.
;-)


Boot/reboot/shutdown splash screen

Steps to reproduce issue
Connect to the HDMI connected to a monitor
Boot the device
Expected result
Resin logo splash screen should be visible
Other information
Reported by: Florin Sarbu
Test configuration: VIA VAB820
Test case number: TC07
Test run: https://resinio.testlodge.com/projects/16238/runs/230391?tab=2&run_section_id=253458&executed_case_id=10655799#executed_case_10655799

No further development is planned for this board.


non-squre SVG with small coordinates are rendered distorted

typst code:
#figure(image(
	"./assets/p5-intersect.svg",
	width: 40%
), caption: [intersection is point E])

while ./assets/p5-intersect.svg is:
<?xml version="1.0" encoding="UTF-8"?>
<svg xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink" width="1.44pt" height="2.64pt" viewBox="0 0 1.44 2.64" version="1.1">
<circle style="fill-rule:evenodd;fill:#FF7043;fill-opacity:1;stroke-width:0.01;stroke-linecap:square;stroke-linejoin:miter;stroke:#616161;stroke-miterlimit:0.007;" cx="0.22" cy="0.32" r="0.1"/>
<circle style="fill-rule:evenodd;fill:#29B6F6;fill-opacity:1;stroke-width:0.01;stroke-linecap:square;stroke-linejoin:miter;stroke:#616161;stroke-miterlimit:0.007;" cx="1.22" cy="1.32" r="0.1"/>
<circle style="fill-rule:evenodd;fill:#FF7043;fill-opacity:1;stroke-width:0.01;stroke-linecap:square;stroke-linejoin:miter;stroke:#616161;stroke-miterlimit:0.007;" cx="0.22" cy="2.32" r="0.1"/>
<circle style="fill-rule:evenodd;fill:#29B6F6;fill-opacity:1;stroke-width:0.01;stroke-linecap:square;stroke-linejoin:miter;stroke:#616161;stroke-miterlimit:0.007;" cx="1.22" cy="0.32" r="0.1"/>
<circle style="fill-rule:evenodd;fill:g;fill-opacity:1;stroke-width:0.01;stroke-linecap:square;stroke-linejoin:miter;stroke:#616161;stroke-miterlimit:0.007;" cx="0.886667" cy="0.986667" r="0.1"/>
<path style="fill:none;stroke-width:0.01;stroke-linecap:butt;stroke-linejoin:miter;stroke:#616161;stroke-miterlimit:0.007;" d="M 0.287175 0.387175 L 0.819492 0.919491 "/>
<path style="fill:none;stroke-width:0.01;stroke-linecap:butt;stroke-linejoin:miter;stroke:#616161;stroke-miterlimit:0.007;" d="M 1.15282 1.25282 L 0.953842 1.05384 "/>
<path style="fill:none;stroke-width:0.01;stroke-linecap:butt;stroke-linejoin:miter;stroke:#616161;stroke-miterlimit:0.007;" d="M 0.262485 2.23503 L 0.844181 1.07164 "/>
<path style="fill:none;stroke-width:0.01;stroke-linecap:butt;stroke-linejoin:miter;stroke:#616161;stroke-miterlimit:0.007;" d="M 1.17751 0.404971 L 0.929152 0.901696 "/>
<text style="font-family:Ubuntu Mono;fill:white;font-size:0.100000px;text-anchor: middle;dominant-baseline: middle;" x="0.220000" y="0.320000">A</text>
<text style="font-family:Ubuntu Mono;fill:white;font-size:0.100000px;text-anchor: middle;dominant-baseline: middle;" x="1.220000" y="1.320000">B</text>
<text style="font-family:Ubuntu Mono;fill:white;font-size:0.100000px;text-anchor: middle;dominant-baseline: middle;" x="0.220000" y="2.320000">C</text>
<text style="font-family:Ubuntu Mono;fill:white;font-size:0.100000px;text-anchor: middle;dominant-baseline: middle;" x="1.220000" y="0.320000">D</text>
<text style="font-family:Ubuntu Mono;fill:white;font-size:0.100000px;text-anchor: middle;dominant-baseline: middle;" x="0.886667" y="0.986667">E</text>
</svg>

the SVG has been stretch, and the <text> labels are not rendered

I can't reproduce this, isn't this how it's supposed to look like? It looks fine for me.

That said, this is a pretty weird SVG you have there because it's only 1.44pt x 2.64pt in size...

Those should have be circles, not oval.
I've also noticed only small non-square SVGs suffer from this. Scaling the SVG by multiplying every number by 100 solves this problem. I guess Typst has incidentally casted some float to int or other float point peoblems.

Okay yeah, I guess the stretch shouldn't be there (although weirdly enough I see the same when opening it in an SVG editor), maybe it's due to some floating point rounding error or whatever. Not sure if there's much we can do here though, the best thing would be to just make the SVG bigger.

Firefox and Adobe Illustrator both can render this correctly.
This is probably a bug in usvg. I guess the best thing we can do is to mention this in the documention and wait for a fix from the upstream.

It was not an upstream bug after all. We just rounded stuff ...


Copy email signature to clipboard?

Hi;
I want to copy and paste"SOMETHING" in any email clients email signature box with the css we
apply to it. Can we copy to clipboard rendered html?can you make it so that the footer is in a copy box.
i.e when you select you just get the content of the box? The client wants me to add a "Copy Email Signature to Clipboard" feature.
< button id="copy-button" data-clipboard-text=''>Copy to Clipboard
Thanks

Can please show us what the value of $copy would look like?
Also, do you want to copy that value as plain text (HTML markup) or as rendered HTML?

Hello James;
Yes rendered HTML so users can just copy it in their email signature and links be actual links.
I can make $copy can be something like this (Please ignore white spaces, I just added them so Github doesn't modify the actual text):
Name: John Doe
www.website.com
< a h ref=“m ailto:obama<EMAIL_ADDRESS>Or basically, like this: http://grab.by/HJM4
Thank you

Is the text/rendered HTML displayed on the page in another element?

Not really. I generate it with php. I can do whatever is suitable for the clipboard.

Hi;
Sorry we are a little confused on how to copy the data. We have the item we want to copy rendered on a page that has  the copy button displayed, but currently we are copying the code generated directly from the php that generates the page. Having looked through the docs I cant see how we can copy the outputted html off the rendered page.

hi behnampmdg3, how did you solve it?

Sorry about the mess, but you get the idea...
ob_start();
foreach($profile_details as $val =>$row)
{
foreach($buttons as $button)
{
if($button == 'last_name')
{
$last_name_showed = true;
}
if($button == 'company')
{
$company_showed = true;
}
if(strlen($row->$button)>0)
{
$query = $this->db->get_where('content_items', array('card_details_value' => $button));
$res = $query->result_array();
$b = str_replace("_", " ", $button);
if($button!="last_name"){?><?php }
$link = $row->$button;
$query_link = $this->db->get_where('preview_buttons', array('card_details_column' => $button));
if($query_link->num_rows()>0)
{
$link = $query_link->result_array()[0]['link'];
$link = "$button)."">".html_escape($row->$button)."";
}
$copy .= $b.": ".$row->$button."\r\n";
if(in_array($button, $name))
{
echo "";
}
if(!in_array($button, $headers))
{
echo ucfirst($b).": ";
}
echo ucfirst($link);
if(in_array($button, $name))
{
echo " ";
}
                                                if($button!="first_name"){?>
<?php }
                                                if(strlen($logo)>4 && $logo_show==true && $last_name_showed==true)
                                                    {
                                                        $logo_show = false;?>

                                                        <img class="img-responsive" src="<?php echo $logo;?>" alt="">

                                                <?php }
                                                if($connect_with_me_show==true && $last_name_showed==true && $company_showed==true)
                                                    {
                                                        echo "
full_url."-e"."\">Connect With Me
";
                                                        $connect_with_me_show = false;
                                                    }
                                            }
                                        }
                                }
                  $copy = ob_get_clean();
                  echo $copy;
                        ?>

            <button id="copy-button" class="btn btn-primary"> Copy to Clipboard</button>

got it. thanks


sample: dfu: allow sample to be build on default platform

Remove platform_allow and allow sample to be build on
default platform but not on native posix.

You probably want to exclude all platforms that do not define image_0 partition.

@de-nordic and @PerMac could you please take a look?

@jfischer-no seems that the builtkite problems, at least with the mimxrt1064_evk, have existed prior to the PR but have been uncovered here.
I have tried to build the sample from master (f0352500b3e5c3aaecca17d03b1e07bb2e0ac973) and found that I can not even go through configuration phase to menuconfig.
Question is whether the fix for these problems should be part of this PR, or the problems should be filtered out and fixed as part of other PR.

@jfischer-no Is this still relevant?

@jfischer-no Is this still relevant?

@PerMac Can you also look at this?

Looks good. Just please change sanitycheck: to twister: in the commit title as there is no more "sanitycheck" :)


WIP: Visual Tests restructure

The visual tests have been restructured (#288). This isn't ready to merge, but I thought it'd be worth sharing at this stage - any feedback is welcome!
I'd recommend viewing here, or checking out this branch and running npm install && grunt visual-tests

Assemble is used to generate a set of pages, comprising of:

A page for each test
An index page which lists all tests and links to each test's own page

Each test requires a handlebars template, as shown in visual-tests/src/test-fixtures. The templates are built into HTML by the build process. Any other files in test-fixtures are copied to dist and the file hierarchy is unchanged, so additional files can be added and linked to (as shown in the current examples).
N.B. I've haven't moved many tests over yet, but it's easy to do.
New structure:
+-- visual-tests
|  +-- dist // contains the generated files to view in your browser (this should be ignored by git)
|  +-- src // edit files in here to change the output
|  |  +-- test-fixtures // tests go here; there are a few examples, hopefully fairly self-explanatory!
|  |  +-- site // contains the various layouts, includes and other files to build the site around

Updated; preview available here: http://jleft.github.io/d3-financial-components/
I'm hoping this is a solid foundation for the visual tests - we could add further features in the future, if we think they'd be useful - search, tag filtering, category pages etc.

LGTM other than the odd comment, thanks James!

LGTM other than the odd comment, thanks James!

No problem, thanks for looking over this :)


Not Working on iPad Mini 2?

Hello,
Thank you for this library, for the most part it's working well for us. For some reason, in iPad Mini 2 the blur effect is not visible, and instead a solid color is getting shown. It changes in shade when changing from "light" to "dark", and I have not been able to find anything online indicating that UIBlurEffect is not supported on iPad Mini 2. https://github.com/react-native-community/react-native-blur/issues/51, http://stackoverflow.com/questions/26411227/detect-if-device-properly-displays-uivisualeffectview seem to indicate that the blur effect is supported on iPad Mini, so I would assume iPad Mini 2 as well. I don't know Objective-C so I am not really equipped to investigate this myself. Any help would be appreciated.
https://forums.developer.apple.com/thread/50854
May (or may not) be helpful to look at.

Also, the iPad Mini 2 is running iOS 10.0.2


Emanote demonstration

[This is just a demo; feel free to adopt/discard however you see fit]
To run a live view of the docs,
cd ./docs
nix run

To build the statically generate site:
cd ./docs
nix build
# Test it out
nix run nixpkgs#nodePackages.http-server ./result

I had to fix the Markdown syntax a bit
Some links (to the source files in parent directory) are broken (hence allowBrokenLinks = true) because they point outside the nix store.

The site should look like this:

This is a 2-column layout; there is also a 1-column layout available with a nice inverted tree at top: https://emanote.srid.ca/demo/neuron-layout

Ctrk+K for search

For github actions / github pages CI, see
https://github.com/srid/emanote-template/blob/master/.github/workflows/publish.yaml

Oh, and about customizing HTML/CSS - you can copy the templates directory locally and modify the relevant file.
(You can also copy only the file being modified; with the rest of the template coming from the default layer linked above).


Make Numulus compile on macOS

BSD sed behaves differently than Linux sed. By adding a "-e" flag the
comand works in both systems.
See: https://unix.stackexchange.com/questions/101059/sed-behaves-different-on-freebsd-and-on-linux

This change is

Reviewed 1 of 1 files at r1.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @jianglai)

core/build.gradle, line 426 at r1 (raw file):

    }
    execInBash(
        'find . -name *.java -exec sed -i -e /\\*\\ \\<p\\>\\$/d {} +',

I know this is outside of the scope of this PR, but this regex is fairly incomprehensible and essentially does slash escaping at three different levels. I think we can improve upon it as:
"find . -name *.java -exec sed -i -e '/" + /\* <p>$/ + "/d' {} +"

This is a bit deceptive, as the slashes around the regex are actually string delimiters rather than regex delimiters (these are in the enclosing double-quoted strings), but at least in this case the the regex is just the regex.
I'll leave it to your discretion as to whether you want to go there now.

I'm curious, how does it actually work, with slashes being string delimiters?

Reviewable status: 0 of 1 files reviewed, 1 unresolved discussion (waiting on @jianglai and @mindhog)

core/build.gradle, line 426 at r1 (raw file):
Previously, jianglai (Lai Jiang) wrote…
It's a Groovy thing. Slashes are string delimiters for special string literals where there is no escape character and the backslash is always literal. It's the equivalent of a raw string in python: r'\* <p>$'
In the outer, double quoted strings (which are actually apparently not strings, BTW, though that's not important here) we then wrap the regex in slashes for the sed expression and single quotes for bash (backslashes are also not escape characters for single-quoted bash strings). So the expression above ends up sending the following command to bash:
find . -name *.java -exec sed -i -e '/\* <p>$/d' {} +

Aha, the part that I missed is “it’s a Groovy thing”. I was intrigued by how you can quote strings with slashes. Sorry that you have to write the whole thing down. Should have asked the question more directly.
https://www.baeldung.com/groovy-strings#slashy-string


Safeguard one model from failures in others

#1102 begs for another question -- why should a failure in one model (memory leak or otherwise), bring down other models served by the same TRTIS process? Can you add a mechanism to protect models from one another or, at the very least, restart failed models?

The models share the common GPU memory. If one model uses all the GPU memory and the other model needs to allocate some to perform an inference then that inference will fail. There isn't really anything TRTIS can do about not having enough GPU memory. We have thought about using UVM so that GPU memory could be oversubscribed or automatically managing models and/or failed inference requests... but such solutions are complex and potentially ineffective.
Retries could be handled at a higher-level. Model orchestration (the term we use for managing which models go on which servers/GPUs to avoid OOM and meet SLA/SLO requirements) could benefit from some additional tools that we do have on our longer-term roadmap. These tools would help you characterize and manage you model repositories so you could avoid OOM situations.


Storage Options

Hello,
I set it so that the minimap disappears as soon as I open the menu/equipment, but as soon as I open the storage options (Inventory, Wardrobe...) the minimap appears again.

Can you paste here what settings you've got for this addon? ashita/config/addons/minimapcontrol/<your character>/settings.lua

Thanks I'll give it a test with your settings and see if I can find what's wrong

Thanks. I don't think I recognise those menus. How do you get to them?

The first picture shows the storage options (equipment > storage options)
The second picture shows the window through which you can tell other players and invite them to the group.

Thanks!

OK I think I got the ones you reported plus spotted a few others. Give 1.2.1 a go, hopefully all fixed!

I tested it. Looks good so far. Many thanks for your effort.


Updates in wmcore for CRAB3

Thank you very much Marco, I included your DBS fix in #6463.
For pycurl fix we already have patch #6360 but integration test is not done. I will try to merge those as soon as integration test is done.  Could you check whether that fetch fixes the problem you have?

@mmascher @ticoann how about we follow up the pycurl changes in #6360 (after cmsweb production upgrade) and close this one?

@mmascher @ticoann how about we follow up the pycurl changes in #6360 (after cmsweb production upgrade) and close this one?

Hi @amaltaro , here is the PR https://github.com/dmwm/WMCore/pull/6916


feat: add for-the-badge style

Closes #29
📑 Description

added for-the-badge style.
added more dynamic styling options for the svg by using variables such as letter_spacing, font_size, height, title_anchor, gradient which are needed to modify the svg for this style. these are only defined, currently, within the for-the-badge style, and are not user customizable

✅ Checks

[x] My pull request adheres to the code style of this project
[x] My code requires changes to the documentation
[x] I have updated the documentation as required
[x] All the tests have passed

☢ Does this introduce a breaking change?

[ ] Yes
[x] No

ℹ Additional Information

(on the above image)

new for-the-badge style
image referenced on issue 29
square style

i wasn't sure if the for-the-badge style should force capitalized titles

Summary by CodeRabbit
Summary by CodeRabbit

New Features

Introduced a new style option for the progress bar: for-the-badge.
Enhanced styling capabilities with additional properties for progress bar customization.

Documentation

Updated the README.md to include the new for-the-badge style with an example preview and URL.

Hi @ztest95 👋,
Thank you so much for your pull request! 🙌
I appreciate the time and effort you put into this contribution.
I'll review it shortly, and if everything looks good, I'll approve it as soon as possible.
Thanks again for your valuable contribution! 🚀

@gstraccini review


ConfigureAppConfiguration not taken into account.

From @whizkidwwe1217 on February 25, 2018 2:27
I tried to put kestrel urls into a separate json file so I don't have to recompile the project every time I change urls but the problem is that the ConfigureAppConfiguration doesn't work. I don't want to use the UseConfigurationMethod because I can't find a way to get the IHostingEnvironment to set up different settings for different environment. Putting these configurations in the Startup class doesn't seem to work either.
hosting.json

Doing it this way works:

dotnet run:
Now listening on: http://[::]:4000
Now listening on: http://[::]:5000
Now listening on: http://[::]:5001
Now listening on: http://[::]:5002
Now listening on: http://[::]:5003

But doing this will not:

dotnet run:
Now listening on: http://localhost:5000

If I do the first method, how do I get the Hosting Environment?

Copied from original issue: aspnet/Home#2903

Duplicate of https://github.com/aspnet/Hosting/issues/1299


Update CDAP Kafka log directory

Move creation of kafka.log.dir to configure from start
Use built-in Ambari Directory method
Set proper permissions to cdap user

LGTM


Add SantiagoRodriguez2017 libs

Note that data is from MG-rast cringe, so there are no direct FTP links that I can find for the raw FQs...
Pull Request
This PR is for a

[ x] New Publication(s)
[ ] Correction

For the following list(s):

[ ] ancientmetagenome-anthropogenic (README)
[ ] ancientmetagenome-environmental (README, ask @thedir-team-dirt for advice)
[x ] ancientmetagenome-hostassociated (README, ask @thedir-team-bugparty for advice)
[ ] ancientsinglegenome-hostassociated (README, ask @thedir-team-pathopeeps for advice)

New Publication
Publication Information
This pull request is to add samples from the following publication(s):
This is to close
Checklist

[ ] Publication is published (preprints currently not accepted)?
[ ] Checked the publication is not already in the database?
[ ] Checked samples in this publication are not previously published data (newly re-sequenced metagenomes are OK!)?
[ ] Samples are shotgun metagenomes (hostassociated-singlegenome may also contain whole-genome enriched data)?
[ ] Checked the list follows conventions as described in the corresponding sample type's README file (e.g. using ERS/SRS accession codes for ENA/SRA)?
[ ] Changelog is updated to include the publication under 'Added'?
[ ] Pull request has passed validation checks (see automated comment from github-bot)?
[ ] Review requested from corresponding team (see list of lists above for each team)?
[ ] (If applicable) Made a separate PR to add new categories to controlled lists (stored under /assets/enums, e.g. material or archive)? If so the corresponding PR is here:

If you do not know how to check errors in failed validation checks, expand here

Correction
This PR is for

[ ] ancientmetagenome-hostassociated
[ ] ancientsinglegenome-hostassociated
[ ] ancientmetagenome-environmental
[ ] ancientmetagenome-anthropogenic

Reference
This pull request is to correct samples from the following publication(s):
This is to close #31
Description
The issue is:

Checklist

[ ] Checked the corrected entries follow conventions as described in the corresponding sample type's README file (e.g. using ERS/SRS accession codes for ENA/SRA)
[ ] Changelog is updated to include the publication under 'Changed'?
[ ] Pull request has passed validation checks (see automated comment from github-bot)?
[ ] Review requested from corresponding team (see list of lists above for each team)?

note to self: https://www.mg-rast.org/mgmain.html?mgpage=download&metagenome=mgm4629032.3

Problematic - not even host-removed data cannot be downloaded from MG-RAST anymore. Consider removing?

With agreement from @alexhbnr on slack we are removing as raw(-enough, for microbiome work) data is in this case not available.


MefContentLoader not called in MEF construction

I've followed the instructions in the Wiki for the MEF construction. My views code-behind look all like this:
    [Content("/View/EmployeeListView.xaml")]
    public partial class EmployeeListView : IContent
    {
        public EmployeeListView()
        {
            InitializeComponent();
        }

        [Import]
        public EmployeeListViewModel EmployeeListViewModel
        {
            get { return (EmployeeListViewModel)DataContext; }
            set { DataContext = value; }
        }

When debugging I can see the view is constructed, and also the view model is constructed by the MEF catalog. But the viewmodel is not set on the view, debugger does not break in setter of property EmployeeListViewModel. Also the MefContentLoader is not called.
Can anybody explain why this is not happening?
Workaround for now is that, in the constructor of the view, I provide access to the container (via MainWindow) to pull out the view model myself and set the DataContext
DataContext = Container.GetExportedValue<EmployeeListViewModel>()
which is not so nice :-/

I discovered some more details about the problem. From all the links I have on the main window, only the 'settings' link (which is a TitleLinks Link) triggers the LoadContent method of MefContentLoader.
Here are all the links on the main window:
<mui:ModernWindow.TitleLinks>
        <mui:Link DisplayName="Settings" Source="/View/SettingsView.xaml" />
    </mui:ModernWindow.TitleLinks>

    <mui:ModernWindow.MenuLinkGroups>
        <mui:LinkGroup DisplayName="Werknemers">
            <mui:LinkGroup.Links>
                <mui:Link DisplayName="Alle" Source="/View/EmployeeListView.xaml" />
            </mui:LinkGroup.Links>
        </mui:LinkGroup>
        <mui:LinkGroup DisplayName="Functies">
            <mui:LinkGroup.Links>
                <mui:Link DisplayName="Alle" Source="/View/EmployeeListView.xaml" />
            </mui:LinkGroup.Links>
        </mui:LinkGroup>
        <mui:LinkGroup DisplayName="Trainingen">
            <mui:LinkGroup.Links>
                <mui:Link DisplayName="Alle" Source="/View/EmployeeListView.xaml" />
            </mui:LinkGroup.Links>
        </mui:LinkGroup>
    </mui:ModernWindow.MenuLinkGroups>

Why is only that link calling the LoadContent method??

As far as I can tell, your code snippets looks fine. Things to double check;

I'm assuming EmployeeListView does derive from UserControl?
Any MEF conflicts resolve warnings in the output window?
Is the MefContentLoader correctly implemented and defined in App.xaml?
Check the value of the MefContentLoader.Contents property, does it contain all the exported content?

EmployeeListView  derives from UserControl.
No MEF conflicts resolve warnings in the output window.
The Contents property has three items (Settings, SettingsAppearance and EmployeeListView)
My App.xaml look like the following:
<Application x:Class="TrainingManagementTool.App"
             xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
             xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
             xmlns:view="clr-namespace:TrainingManagementTool.View"
             xmlns:mui="http://firstfloorsoftware.com/ModernUI">
    <Application.Resources>
        <ResourceDictionary>
            <ResourceDictionary.MergedDictionaries>
                <ResourceDictionary Source="/FirstFloor.ModernUI;component/Assets/ModernUI.xaml" />
                <ResourceDictionary Source="/FirstFloor.ModernUI;component/Assets/ModernUI.Light.xaml" />
                <ResourceDictionary>
                    <Style TargetType="mui:ModernFrame">
                        <Setter Property="ContentLoader" Value="{DynamicResource MefContentLoader}" />
                    </Style>
                    <Style TargetType="mui:ModernTab">
                        <Setter Property="ContentLoader" Value="{DynamicResource MefContentLoader}" />
                    </Style>
                    <Style TargetType="view:MainWindow">
                        <Setter Property="ContentLoader" Value="{DynamicResource MefContentLoader}" />
                    </Style>
                </ResourceDictionary>
            </ResourceDictionary.MergedDictionaries>
        </ResourceDictionary>
    </Application.Resources>
</Application>

And still I notice that the break point in LoadContent() of MefContentLoader is not hit when clicking the EmployeeListView link, but it does when clicking the Settings link.

How is the MefContentLoader instantiated? Is the Contents property properly imported?

My Bootstrapper class looks like this:
    public class Bootstrapper : MefBootstrapper
    {
        protected override DependencyObject CreateShell()
        {
            return Container.GetExportedValue<MainWindow>();
        }

        protected override void InitializeShell()
        {
            base.InitializeShell();

            Container.ComposeParts(this);

            //The MefContentLoader instance is retrieved from the container and added to the global application
            //resource dictionary. Once the loader is added, the DynamicResource reference used in the default styles can be resolved.
            var contentLoader = Container.GetExportedValue<MefContentLoader>();
            Application.Current.Resources.Add("MefContentLoader", contentLoader);

            // Add the imported resource dictionaries
            // to the application resources
            var views = Container.GetExportedValues<UserControl>();
            foreach (var r in views)
            {
                Application.Current.Resources.MergedDictionaries.Add(r.Resources);
            }

            ((MainWindow) Shell).MainWindowViewModel.Container = Container;
            Application.Current.MainWindow = (Window)Shell;
            Application.Current.MainWindow.Show();
        }

        protected override void ConfigureAggregateCatalog()
        {
            base.ConfigureAggregateCatalog();

            // Add this assembly to export views and viewmodels
            AggregateCatalog.Catalogs.Add(new AssemblyCatalog(typeof(Bootstrapper).Assembly));
        }
    }

Looks good. Not sure why you would add all user controls to the resource dictionary, but that should not conflict. Care to share the entire codebase?

Ok, here's a stripped version of the app. Notice when clicking on the menu's the content loader is NOT called, when clicking on settings (Instellingen) link it works as expected and calls the content loader.
SourceCode.zip

Found two issues;
First, you define a default style for mui:ModernWindow in App.xaml to set the ContentLoader. However, this needs to be the actual main window type TrainingManagementTool.View.MainWindow. So change the style to this (and define the xmlns:local="clr-namespace:TrainingManagementTool.View" );
<Style TargetType="local:MainWindow">
  <Setter Property="ContentLoader" Value="{DynamicResource MefContentLoader}" />
</Style>

Second, do not define the default style overrides in a merged dictionary. Add them to the main Application.Resources like so;
<Application.Resources>
  <ResourceDictionary>
    <ResourceDictionary.MergedDictionaries>
      <ResourceDictionary Source="/FirstFloor.ModernUI;component/Assets/ModernUI.xaml" />
      <ResourceDictionary Source="/FirstFloor.ModernUI;component/Assets/ModernUI.Light.xaml" />
    </ResourceDictionary.MergedDictionaries>

    <!-- style overrides here -->
    <Style TargetType="mui:ModernFrame">
      <Setter Property="ContentLoader" Value="{DynamicResource MefContentLoader}" />
    </Style>
    <!-- etc -->
  </ResourceDictionary>
</Application.Resources>

Hope this helps

Great, this works! Applying the ContentLoader in de app.xaml in the correct way solved the problem. Thanks!
(Prachtig framework, mijn apps zien er gelijk gelikt uit)

top!


SDK throws InnerExceptionCountExceededException while it should apply defaults

https://github.com/Microsoft/ApplicationInsights-dotnet/blob/develop/src/Core/Managed/Shared/DataContracts/ExceptionTelemetry.cs#L212

Exception is not thrown. It is collected and sent instead of inner exceptions that are cut to provide information for the user.


Add all that will be marked to all estimated tasks wiki page

Commit [3c7a23e] (https://github.com/olunet/SuperCap/wiki/All-estimated-tasks/3c7a23eff09f36ad9a56add0e389e3a4e9789e8e)

Work includes: making an organised page to later add the list of all tasks we currently foresee.
Estimated: 0.5 man-hours


Rename Payout => Potential Payout

This is not published yet, will published after merge

Coverage remained the same at 63.987% when pulling 3a50c8e682726259455764a1de42537b1238f018 on qingweibinary:qingwei/minor_fix into 7bf87aa5d717446455cdb756e69b6066b10529f7 on binary-com:master.

@borisyankov Merge this ?


benchmarks

Hello, I see in the release 7.1.1 snappy has an increase in performance of 2-3 times. But I did some local tests and I don't note the difference in performance between version 7.0.5 and 7.1.1. I note that you have changed the computer for the benchmarks, is it possible you have compared results of different hardware?

I'm genuinely curious about this too! I ran some benchmarks and the performance is great tho but I'm not seeing those x2/x3 mentioned in the release notes and was wondering if it was a typo!

Oh sorry. I forgot the performance improvement codes have been shipped in 7.0.4. So the performance in 7.1.x is basically identical with 7.0.4.
I have benchmarked the latest code aginst the 7.0.3 and here is the result:
Running "Compress" suite...
Progress: 100%

  snappy:
    2 438 ops/s, ±3.14%   | fastest

  snappy-7.0.3:
    1 862 ops/s, ±1.51%   | 23.63% slower

  snappy-v6:
    1 340 ops/s, ±1.84%   | slowest, 45.04% slower

I'll update the changelog in release notes, thanks.


Capitalization of error messages

We have mixed capitalization in error messages in youki e.g.
Failed to run post stop hooks
no linux in spec

I so far have used lowercase with no period because that is what is described in the api guidelines and also what the compiler itself uses. Personally I do not really care what we use as long as we do it consistently.  We do not need to change anything now, just something to keep in mind, when you touch code. Let me know what you want to do.
@utam0k @yihuaf @YJDoc2

I think we should follow the api guideline. I don't really have a preference either way, but I agree we should standardize it. The api guideline looks reasonable and something we can point to in the future.

I don't have a preference, but agree that we should have some consistent formatting.
I'd suggest rather than fixing messages in a dedicated commit, and possibly causing conflicts, let us update message in whichever files we change, and uphold this formatting in new PRs, that way eventually most/all messages will confirm to this.

You're right, it's better to unify. If we are going to unify, we want to follow the api guidelines. It would be great to have a tool to check them.
This is a bit off topic, but I'm also thinking a bit about introducing thiserror.

Great! Then the decision is to follow the rust api guidelines and use lower case for error messages.


Having issues updating

Hi!
Don't know if it's the proper place to raise the hand here, but I'm having trouble installing channel feat system. No matter what I do, install from marketplace, update it, or from the json, I always get version 0.0.7. I wanted to install the last one to get the catalan translation but no luck :(
Maybe there is some configuration json pointing to 0.0.7 by mistake or something?

Hi,
The 0.0.7 is the last version for Foundry v9. If you want to update the system, you must use Foundry v10.
If you're already using it, juste uninstall the system and install it again.

Thanks and sorry, I had to update and uninstall a couple of times until it worked. But I'm with the catalan version now. Thanks!

You're welcome! 👍🏻


Classpath has version 3.4.6 hard-coded

This line has version 3.4.6 hardcoded, it prevents any version other than 3.4.6 to work.

That’s… pretty important to fix. Thanks!

This is a big bug now that 3.4.6 doesn't appear to be available on any of the mirrors.
http://apache.mirrorcatalogs.com/zookeeper/zookeeper-3.4.7/
http://www.poolsaboveground.com/apache/zookeeper/

@spuder understood. I’ll get it sorted; it’s blocking us, too, now.

@spuder @patoarvizu alrighty, I’ve published v3.0.0 to Supermarket to solve this.


IP is not a value available when running via EKS

I’m not sure this change is correct by itself. @metral could you take a look?

Ah, yes this PR is correct, it should be hostname. I must've copy/pasted it by accident with GKE's tutorial. I'm glad this got caught, thanks @cwiggs!


Extend scancode-config.yml to ignore some vulnerabilities

The same way we can ignore paths and package scopes, it could be useful to ignore certain vulnerabilities in the context of a project. The ignore could be based either on a CVE or in a VCID. It could be useful to also have a comment that explain why we ignore this.

Merged and documented at https://scancodeio.readthedocs.io/en/latest/project-configuration.html#ignored-vulnerabilities


add launch gui via cli option

gui can now be launched with
siepic_testcreator --gui

Welcome to Codecov :tada:
Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


CommandArgumentInfo: new property defining valid values

First time using this framework and am porting a simple soundboard bot to ramp up...
One issue i'm facing is defining a command that only accepts a pre-defined list of arguments and having those arguments displayed elegantly when getting help on the command. I see ArgumentType.validate, which could inform the user of valid args on error, but was interested if you saw any value in adding a property to CommandArgumentInfo for this purpose?

There already are both validate and parse properties, which can be used to override the type's methods.


Validate MSAL MI API Integration (look for adoption blockers, breaking changes).

Validate MSAL MI Integration (from API and breaking changes POV).

Use the Msal beta release/package to test/validate API integration with Azure-Identity-JS package.
Check the existing features/functionality exposed to Azure-Identity-JS users for MI is supported by the Msal MI.
Run the automated MI tests with Msal MI Integrated in Azure Identity JS Package.

ETA Cost: (2-3 days).

Platform
System Assigned MI
User Assigned MI
Issues Reported

Azure VM Windows
Works
Works

Azure VM Linux
Works
Works
date parsing, managed identity logs

Azure App Service
Works
Works

Azure Functions
Works
Works

Azure Cloud Shell

Azure Service Fabric

Azure Arc

Reported and now fixed:

expiresOn calculated incorrectly
Log which managed identity sources were tried and which succeeded

Going to finish the last 3 in June

Closing for now as I validated most scenarios. Any potential issues can be worked through as part of the migration


Slider increase to 100% not working

I have the slider buttons set up to control light group entities.  The underlying hardware is Lightwave smart switches.
Sliding up and down works perfectly in every respect. However I find that if I increase the brightness to 100% nothing happens - the lights stay the same brightness as before.
If I give the button a tap (anywhere on the slider), then the 100% gets actioned. Not solved by using any particular slider settings, having been through them all. Only an issue with 100%. Slide up and down to 50% or 95% is works fine.

Further investigation reveals this is actually an issue confined to card as it operates the iOS app.  No similar problem in Chrome on a desktop.  Not sure therefore if it is really a card bug.

This is happening to me also in iOS, when the toggle is shown you can’t move to 100%, it works OK in Android.

This bug is triggered when the user's finger moves outside the boundary of the slider button card and is outside the boundary of the card when the user lifts their finger. This exhibits at the 100% end because the user generally has to move their finger off the end, outside the boundary, to reach 100%.


Change text/link from 'close case' to 're-open' case if the case is c…

…losed

Description

Checklist:

[ ] Have you documented your changes in the pull request description?
[ ] Does the change present any security considerations?
[ ] Is any gem functionality overloaded? Eg: Devise controller methods being overloaded.
[ ] Has acceptance criteria been tested by a peer?
[ ] Has the CHANGELOG been updated? (If change is worth telling users about.)

General testing (author)

[ ] Test without JavaScript
[ ] Test on small screen

Accessibility testing (author)

[ ] Works keyboard only
[ ] Tested with one screen reader
[ ] Zoom page to 400% - content still visible
[ ] Disable CSS - does content make sense and appear in a logical order?

Codecov Report

Merging #1961 (cc82e38) into master (6780429) will decrease coverage by 32.99%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master    #1961       +/-   ##
===========================================
- Coverage   88.88%   55.88%   -33.00%
===========================================
  Files         398      398
  Lines        8780     9408      +628
  Branches     1277     1143      -134
===========================================
- Hits         7804     5258     -2546
- Misses        781     3896     +3115
- Partials      195      254       +59

Impacted Files
Coverage Δ

app/forms/alert_form.rb
0.00% <0.00%> (-100.00%)
:arrow_down:

app/jobs/send_alert_job.rb
0.00% <0.00%> (-100.00%)
:arrow_down:

app/forms/remove_user_form.rb
0.00% <0.00%> (-100.00%)
:arrow_down:

app/forms/reset_password_form.rb
0.00% <0.00%> (-100.00%)
:arrow_down:

app/forms/remove_business_form.rb
0.00% <0.00%> (-100.00%)
:arrow_down:

app/controllers/teams_controller.rb
0.00% <0.00%> (-100.00%)
:arrow_down:

app/forms/notifying_country_form.rb
0.00% <0.00%> (-100.00%)
:arrow_down:

app/controllers/account_controller.rb
0.00% <0.00%> (-100.00%)
:arrow_down:

app/controllers/searches_controller.rb
0.00% <0.00%> (-100.00%)
:arrow_down:

app/models/audit_activity/alert/add.rb
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 203 more

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 6780429...cc82e38. Read the comment docs.


[release-1.23] Address CVE-2022-41717

Bump golang.org/x/net to v0.4.0
The issue was addressed:

on master with https://github.com/kubernetes/cloud-provider-openstack/pull/2064
on v1.25 with https://github.com/kubernetes/cloud-provider-openstack/pull/2078
on v1.24 with https://github.com/kubernetes/cloud-provider-openstack/pull/2080

Which issue this PR fixes(if applicable):
CVE-2022-41717
Special notes for reviewers:
The issue was addressed:

in v1.26 with https://github.com/kubernetes/cloud-provider-openstack/commit/52858db8131938f661f5201be2e3418094eed62b
in v1.25 with https://github.com/kubernetes/cloud-provider-openstack/commit/702d53ad9c7c7a2a1b4b1e054ef44e886b5d7b3b

Release note:

NONE

/ok-to-test


Login

This allows you to login using github and save your token and rooms using a db, this also adds a db for saving the rooms

This has been locked due to it not being done

I will unlock when its not a draft

We will fix conflicts soon (when merging)

They aren't big

We will fix conflicts soon (when merging)

They aren't big


[backport] Fix DictDataset to work in Python 3.

backports #3237

All Jenkins tests have been passed.

LGTM!

Closing, as we won't backport to v2 any more.


Add MPI Hostfile support to NnfDataMovementCommandStatus

Why is this needed?

For testing, for the user to understand what is contained inside the hostfile without having to shell into a pod and for being able to delete the hostfile when the NnfDataMovement is deleted.

Why would the host file be anything besides the list of Rabbits and slot count? What value is this adding? These are resources stored in a database; while a host file isn't megabytes in size, any amount makes a difference.
Why would the host file need to linger any longer than the mpirun command? Because it is deterministic, it isn't required for debugging (it can be easily reconstituted from existing resources).

Why would the host file be anything besides the list of Rabbits and slot count? What value is this adding? These are resources stored in a database; while a host file isn't megabytes in size, any amount makes a difference.
Why would the host file need to linger any longer than the mpirun command? Because it is deterministic, it isn't required for debugging (it can be easily reconstituted from existing resources).

Because sometimes debugging is worth it's weight in gold. Since this can impact performance, I'll remove it.


DS-4201 Do not pass starts_with if browsing an item index.

Fixes DS-4192.

I tested this PR to see if it could also resolve https://jira.duraspace.org/browse/DS-4208.  The issue is similar.  Unfortunately, this PR does not also resolve that bug.  If I find a solution to DS-4208, I will add a comment to this PR.

merged, thanks @J4bbi


fix(ui): disable app password on windows platform

Description:
Disable app password feature on windows

How Has This Been Tested?
Manually

Types of changes:
Security adjustment

Checklist:

[x] My code follows the code style of this project.
[x] I have reviewed and updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes where needed.
[ ] All new and existing tests passed.
[x] My commits have been squashed into a concise set of changes.

Coverage decreased (-0.02%) to 22.729% when pulling 31d1695e35e9f1e05ad6013b0df4554d0c785ce6 on korhaliv:fix/disable-pwd-win into 9d1d07be51e63e08753a10059d2d48ea967f4b23 on LN-Zap:next.


addresses the logging issues

Modifies the logging configuration to top level instead of root level in ludwig/init.py
User's desired logging level is set at the project level instead of module level

Verified that the user's setting is reflected across the project by trying different levels.
However, I couldn't suppress tensorflow's warning messages. It's probably using tf.logging or something.

@Lothiraldan I addressed all the comments. Please take a look.

LGTM, let's way for Boris to confirm before merging.

@w4nderlust fixed some more logging issues.

Merging this, will improve if afterwards Boris finds issues with it.

Looks good! Sorry for the time for review.


Use webpack to build the source code

Switch to a common.js module system which will also give better npm integration.

Done


Craco build fails when using native webpack 5 plugin in craco.config.js

Issue
I am trying to use a native webpack 5 plugin NormalModuleReplacementPlugin.
Expected Behaviour
NormalModuleReplacement can be used to modify module names based on environment variable.
Actual Behaviour
Building with craco build fails with the error:
Creating an optimized production build...
Failed to compile.

Entry module not found: TypeError: Cannot read property 'resource' of undefined

craco.config.js
const webpack = require('webpack')

module.exports = {
    webpack: {
        plugins: [
            new webpack.NormalModuleReplacementPlugin(/--env/, function() {})
        ]
    }
}

Package.json
{
  "dependencies": {
    "@craco/craco": "^6.3.0",
    "@material-ui/core": "^4.9.8",
    "@material-ui/icons": "^4.9.1",
    "@material-ui/lab": "^4.0.0-alpha.47",
    "@types/jest": "24.0.23",
    "@types/react": "16.9.2",
    "@types/react-dom": "16.9.0",
    "babel-loader": "^8.0.4",
    "connected-react-router": "^6.5.2",
    "csso": "^3.5.1",
    "react": "^17.0.0",
    "react-app-polyfill": "^1.0.1",
    "react-dom": "^17.0.0",
    "react-scripts": "^4.0.0",
    "slick-carousel": "^1.8.1",
    "typeface-roboto": "^0.0.54"
  },
  "devDependencies": {
    "script-ext-html-webpack-plugin": "^2.1.5",
    "webpack": "^5.54.0",
    "webpack-dev-server": "3.11.1"
  }
}

It seems this was my bad. I re-installed all dependencies using webpack 4.x, babel-loader 8.10 and eslint ^7.11 and this does not produce the issue.


RuntimeError： cudaErrorIllegalAddress

Hi, when I run the train.sh, I encountered such problem"RuntimeError: merge_sort: failed to synchronize: cudaErrorIllegalAddress: an illegal memory access was encountered". Does anyone have any idea?
my environment is:
Ubuntu 18.04
Cuda 11.1
gpu 3090
pytorch1.8.1

To be honest, I do not encounter this issue. Maybe you can provide more details shown in error log.

Thanks a lot . I think it is the reason of cuda.  When I train it with cuda 10.2,I do not encounter this issue .

The reason maybe here
when you run import torch_scatter it will look for cuda 10.2 but not cuda 11.1
As you can test like this:
>>> import torch_scatter
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python3.6/dist-packages/torch_scatter/__init__.py", line 14, in <module>
    f'{library}_{suffix}', [osp.dirname(__file__)]).origin)
  File "/usr/local/lib/python3.6/dist-packages/torch/_ops.py", line 104, in load_library
    ctypes.CDLL(path)
  File "/usr/lib/python3.6/ctypes/__init__.py", line 348, in __init__
    self._handle = _dlopen(self._name, mode)
OSError: libcudart.so.10.2: cannot open shared object file: No such file or directory

The reason maybe here when you run import torch_scatter it will look for cuda 10.2 but not cuda 11.1
As you can test like this:
>>> import torch_scatter
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python3.6/dist-packages/torch_scatter/__init__.py", line 14, in <module>
    f'{library}_{suffix}', [osp.dirname(__file__)]).origin)
  File "/usr/local/lib/python3.6/dist-packages/torch/_ops.py", line 104, in load_library
    ctypes.CDLL(path)
  File "/usr/lib/python3.6/ctypes/__init__.py", line 348, in __init__
    self._handle = _dlopen(self._name, mode)
OSError: libcudart.so.10.2: cannot open shared object file: No such file or directory

and finally I found the solution, you need modify Dockerfile totally:
FROM nvidia/cuda:11.1-cudnn8-devel-ubuntu18.04

# Just in case we need it
ENV DEBIAN_FRONTEND noninteractive

RUN apt update && apt upgrade -y
RUN apt install -y python3 python3-dev python3-pip
RUN pip3 install --upgrade pip
RUN apt install -y git wget unzip
RUN apt install -y build-essential
RUN apt install -y ffmpeg libsm6 libxext6

# get newer version of cmake
RUN wget https://apt.kitware.com/kitware-archive.sh
RUN chmod +x kitware-archive.sh && ./kitware-archive.sh

RUN pip3 install torch==1.8.2+cu111 torchvision==0.9.2+cu111 torchaudio==0.8.2 -f https://download.pytorch.org/whl/lts/1.8/torch_lts.html
RUN pip3 install pyyaml==5.4.1
RUN pip3 install cython==0.29.24
RUN pip3 install nuscenes-devkit==1.1.6
RUN pip3 install numba==0.53.1
RUN pip3 install strictyaml==1.4.4

# Torch scatter
RUN pip3 install --no-index torch-scatter -f https://pytorch-geometric.com/whl/torch-1.8.1+cu111.html

The things modify here:

FROM nvidia/cuda:11.1-cudnn8-devel-ubuntu18.04
RUN pip3 install torch==1.8.2+cu111 torchvision==0.9.2+cu111 torchaudio==0.8.2 -f https://download.pytorch.org/whl/lts/1.8/torch_lts.html
RUN pip3 install --no-index torch-scatter -f https://pytorch-geometric.com/whl/torch-1.8.1+cu111.html

Although the torch-scatter cannot correspond to torch 1.8.2 but I found that it can work like here:

https://github.com/taesungp/contrastive-unpaired-translation/issues/83
this can help.