Minimal Code example html fails with "AudioContext suspended"

Facing the below error while trying the Minimal Code sample html.

https://goo.gl/7K7WLu --> suggests we might have to create some click activity before initializing AudioContext.
Amazing work with TalkingHead !!!

The first message is merely a notice or warning that can't be avoided. The warning appears because the TalkingHead class initializes the AudioContext before the document has received any user gesture. This type of initialization is permitted, but some browsers still give this notice to inform us about their policy, which restricts the app from playing audio before the first user interaction. The class follows the instruction and internally tries to call resume before initiating any audio playback.
 I couldn't reproduce that error with the Minimal code, and in the original code the only call to speakText is inside the click event handler, which is a user action.
Did you make any changes to the example code? - If you did, you have to make sure that the user has interacted with the app in some way before calling speakText or speakAudio.

I just changed the log message when the AudioContext resume fails, so that the reason for the error is a bit clearer. The change will be in the upcoming release. Thank you for bringing this to my attention. I will close this issue, but feel free to ask further questions, if any.


⚠ Bofrost.be has degraded performance

In d30fbd5, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 3026 ms

Resolved: Bofrost.be performance has improved in d342624.


speed-up testing

I have introduced a smaller test dataset which is only 2k samples from MNIST dataset which lower the testing time from about 20min just to 10min :smiling_imp:
I have also refactored imports in the test section

@Borda can we spend some time working on making the tests faster? maybe use a smaller dataset or something?

can we spend some time working on making the tests faster? maybe use a smaller dataset or something?

that would be perfect, (that is why I made this as WIP) since some contributors are creating PR with their first commit instead of developing in own branch and make PR when it is ready... the problem is that such approach takes almost all your CI workers so than all other PRs have to queue for hours...
Yes, in general the test should be as simple and straight forward as possible, so for example we can do:

make the test dataset smaller
have lower number of iteration
use higher stop threshold

the extended timeout helped as expected, but still, we have failing tests for 1.1 and 1.2

For the queuing problem: An alternative solution I've seen for bigger opensource projects would be to have a trigger for testing through a github bot. This way we don't run on every commit.

User creates WIP PR, asks for advice
Advice administered
User finishes PR, comments '/run-ci'
A bot triggers CI on the build
Build passes and PR can be merged.

@williamFalcon test time 20min -> 10min

@williamFalcon @jeffling @neggert @Ir1d  test time 20min -> 10min

@Borda great job!


Mark messages as unread page needs animated GIFs for mobile app workflows

Context
To help visual learners learn how to perform tasks using the Mattermost mobile app, add visuals and short animated GIFs on the Mark messages as unread page.
Parent Issue: #6657
Docs Request

Add/update appropriate visuals/GIFs for all the steps in the Mark messages as unread section under the mobile app tab.

Additional Notes

Use the latest Mattermost mobile app and the standard Mattermost Denim theme to capture all workflows.
Avoid drawing lines, arrows, boxes, or callouts on images.
Don't forget to include ALT Text descriptions for all new image assets.
Use existing docs pages with images as a starting point for code guidance. The Docs team can help provide RST syntax support for publication.
Save static image files in PNG format within the docs repository's /images directory.

QA Validation Criteria
In a generated docs preview:

Visiting the docs page results in appropriate new mobile-specific screenshots in each subsection from the Mobile tab.

@cwarnermm Please assign this issue to me. Thanks.


time cost

Hello, thanks for sharing your code!
I am wondering how much time does it cost for you to train your y-net model? And how many gpus do you use? I use one titian xp and it seems to cost me 2 days for training. Is that normal?

Yes, training takes time in the range of days. Especially the evaluation gets quite slow, since we need to do K_e*K_a forward passes.


marathon nproc and ulimit not respected

Hi,
Is there any reason Marathon would loose sysctl conf (nproc, limits...) when switching to another user ?
I have tested with a cassandra instance. After having configured /etc/sysctl.d/sysctl.conf, /etc/security/limits.conf and /etc/security/limits.d/20-nproc.conf,I can't get the cassandra process be launched with good pararmeters when I precise "user": "cassandra" in marathon json.
If I launch a task as root in marathon with "cmd": "su cassandra -c '..." everything is fine, I can get the right limits.
I start the app through the UI
{
  "id": "/cassandra",
  "cmd": "su cassandra -c '/opt/cassandra-3.0.8/bin/cassandra -f'",
  "cpus": 0.5,
  "mem": 2200,
  "disk": 0,
  "instances": 1,
  "constraints": [
    [
      "hostname",
      "UNIQUE"
    ]
  ],
  "portDefinitions": [
    {
      "port": 10000,
      "protocol": "tcp",
      "labels": {}
    }
  ]
}

It works good, the limits are fine.
{
  "id": "/cassandra",
  "cmd": "/opt/cassandra-3.0.8/bin/cassandra -f",
  "cpus": 0.5,
  "mem": 2200,
  "disk": 0,
  "instances": 1,
  "constraints": [
    [
      "hostname",
      "UNIQUE"
    ]
  ],
  "portDefinitions": [
    {
      "port": 10000,
      "protocol": "tcp",
      "labels": {}
    }
  ],
  "user": "cassandra"
}

Limits aren't set, only defaults are applying. The /etc/security/limits.conf isn't taken into account...

It looks like we have some band-aid code in our startup script. Sounds like
a more general solution is needed:
[jdefelice@ec101 marathon]$ cat bin/marathon-framework |grep -A3 -B3 -e
ulimit
local log_format='%2$s%5$s%6$s%n' # Class name, message, exception
if running as root and open file limit less than 8192 raise the limit
[ $EUID -eq 0 -a $(ulimit -n) -lt 8192 ] && ulimit -n 8192
export PATH=/usr/local/bin:"$PATH"
local vm_opts=( -Djava.library.path=/usr/local/lib:/usr/lib:/usr/lib64
On Tue, Oct 25, 2016 at 6:06 AM, elppc<EMAIL_ADDRESS>wrote:

Hi,
Is there any reason Marathon would loose sysctl conf (nproc, limits...)
when switching to another user ?
I have tested with a cassandra instance. After having configured
/etc/sysctl.d/sysctl.conf, /etc/security/limits.conf and
/etc/security/limits.d/20-nproc.conf,I can't get the cassandra process be
launched with good pararmeters when I precise "user": "cassandra" in
marathon json.
If I launch a task as root in marathon with "cmd": "su cassandra -c '..."
everything is fine, I can get the right limits.
I start the app through the UI
{
"id": "/cassandra",
"cmd": "su cassandra -c '/opt/cassandra-3.0.8/bin/cassandra -f'",
"cpus": 0.5,
"mem": 2200,
"disk": 0,
"instances": 1,
"constraints": [
[
"hostname",
"UNIQUE"
]
],
"portDefinitions": [
{
"port": 10000,
"protocol": "tcp",
"labels": {}
}
]
}
It works good, the limits are fine.
{
"id": "/cassandra",
"cmd": "/opt/cassandra-3.0.8/bin/cassandra -f",
"cpus": 0.5,
"mem": 2200,
"disk": 0,
"instances": 1,
"constraints": [
[
"hostname",
"UNIQUE"
]
],
"portDefinitions": [
{
"port": 10000,
"protocol": "tcp",
"labels": {}
}
],
"user": "cassandra"
}
Limits aren't set, only defaults are applying. The
/etc/security/limits.conf isn't taken into account...
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mesosphere/marathon/issues/4616, or mute the thread
https://github.com/notifications/unsubscribe-auth/ACPVLJVZsDpZre_HX04oultJiDNAke8Fks5q3dS_gaJpZM4KfytK
.

Oh wait, I think I misread. In this case your cassandra task is launching
on a slave. So .. did you change the ulimits conf on the slave? The
Marathon startup script only affects the Marathon process itself.
On Tue, Oct 25, 2016 at 9:00 AM, James DeFelice<EMAIL_ADDRESS>wrote:

It looks like we have some band-aid code in our startup script. Sounds
like a more general solution is needed:
[jdefelice@ec101 marathon]$ cat bin/marathon-framework |grep -A3 -B3 -e
ulimit
local log_format='%2$s%5$s%6$s%n' # Class name, message, exception
if running as root and open file limit less than 8192 raise the limit
[ $EUID -eq 0 -a $(ulimit -n) -lt 8192 ] && ulimit -n 8192
export PATH=/usr/local/bin:"$PATH"
local vm_opts=( -Djava.library.path=/usr/local/lib:/usr/lib:/usr/lib64
On Tue, Oct 25, 2016 at 6:06 AM, elppc<EMAIL_ADDRESS>wrote:

Hi,
Is there any reason Marathon would loose sysctl conf (nproc, limits...)
when switching to another user ?
I have tested with a cassandra instance. After having configured
/etc/sysctl.d/sysctl.conf, /etc/security/limits.conf and
/etc/security/limits.d/20-nproc.conf,I can't get the cassandra process
be launched with good pararmeters when I precise "user": "cassandra" in
marathon json.
If I launch a task as root in marathon with "cmd": "su cassandra -c '..."
everything is fine, I can get the right limits.
I start the app through the UI
{
"id": "/cassandra",
"cmd": "su cassandra -c '/opt/cassandra-3.0.8/bin/cassandra -f'",
"cpus": 0.5,
"mem": 2200,
"disk": 0,
"instances": 1,
"constraints": [
[
"hostname",
"UNIQUE"
]
],
"portDefinitions": [
{
"port": 10000,
"protocol": "tcp",
"labels": {}
}
]
}
It works good, the limits are fine.
{
"id": "/cassandra",
"cmd": "/opt/cassandra-3.0.8/bin/cassandra -f",
"cpus": 0.5,
"mem": 2200,
"disk": 0,
"instances": 1,
"constraints": [
[
"hostname",
"UNIQUE"
]
],
"portDefinitions": [
{
"port": 10000,
"protocol": "tcp",
"labels": {}
}
],
"user": "cassandra"
}
Limits aren't set, only defaults are applying. The
/etc/security/limits.conf isn't taken into account...
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mesosphere/marathon/issues/4616, or mute the thread
https://github.com/notifications/unsubscribe-auth/ACPVLJVZsDpZre_HX04oultJiDNAke8Fks5q3dS_gaJpZM4KfytK
.

Yes. Limits are set on all slaves. It's OK with "su Cassandra..."

I believe it depends on which user the cassandra tasks are running as. This sounds more like a mesos issue than a Marathon one. You could set the user to run the tasks as from marathon (probably root, but that'll depend on how you've configured mesos).


Automate GitHub app setup flow

The flow is very cumbersome with having to manually create the app, store its secrets to the cluster in multiple locations and "install" it into the relevant repos and orgs. The process also has to by-and-large be repeated every time the cluster is recreated.
Suggestions:

Automate the GitHub App creation process similarly to how tkn pac bootstrap githaub-app does it.
Store the app secrets to local files so that they can be reused if the cluster is reinstalled.

The link is broken.
The process is indeed very manual, but the steps which are most cumbersome, which is the smee channel, github app and quay org do not have to be repeated every time.
All in all, yes it would be great to have it automated. Not sure what it would take.

Fixed the link, perhaps, as a first step we can suggest using tkn pac bootsrtap in the readme files. That can already setup smee and GitHub for us. It was not very good at fixing partial or broken setups the last time I checked but perhaps it got improved since.
WRT repeating every time, if you lose your app secret, you will need to re-generate it and reconfigure the secrets in the cluster. Admittedly you can reuse the same app, which would save you some clicking in the GitHub UI.


Should the epsilon terminal node have an underlying significant token?

It will need to be put back because we will need to guard against left recursion.

No, and it has been put back.


HDDS-3981. Add more debug level log to XceiverClientGrpc for debug purpose

What changes were proposed in this pull request?
When I profile the s3g performance, i find in some log of XceiverClientGrpc, I cannot find the related datanode ip, so i use datanode instead of the UUID and networkFullPath.
Also, i add a debug level log to show the request has been executed and output the try counts.
What is the link to the Apache JIRA
HDDS-3981
How was this patch tested?
No need, all changes only related to log.

@xiaoyuyao Thanks to review my PR, i reply to your comments, please check it again, thanks.

@xiaoyuyao Thank you for the review suggestion, PTAL.

@bharatviswa504 What do we need to make it possible to merge this commit?

@xiaoyuyao Thanks @xiaoyuyao for the suggestion, I pushed a new commit to address your comment, I reply your comment and ask some question. PTAL.

Thanks @maobaolong  for the update. LGTM, +1. Will merge it shortly.


The motivation is discussed in #236 .
A new environmental variable CHAINERMN_FORCE_ABORT_ON_EXCEPTION is introduced.
The design and implementation are still subject to discuss.
This PR is tested to work properly using train_mnist.py .
The issue is reproduced by changing train_mnist.py as:
    # Some display and output extensions are necessary only for one worker.
    # (Otherwise, there would just be repeated outputs.)
    if comm.rank == 0:
        raise ValueError('error')    # <---- Inserted
        trainer.extend(extensions.dump_graph('main/loss'))
        trainer.extend(extensions.LogReport())
        trainer.extend(extensions.PrintReport(
            ['epoch', 'main/loss', 'validation/main/loss',
             'main/accuracy', 'validation/main/accuracy', 'elapsed_time']))
        trainer.extend(extensions.ProgressBar())

Thanks for the comment.
I checked my commit 0078a6b1, but I had no idea why the problematic change was made.
Anyways, I fixed the issue.

The make html issue is not yet fixed. I'm working on that.

Now I improved the function docstring to point to the FAQ entry, which explains the problem and two possible solutions (using -m mpi4py and using add_hook())


SyntaxError: Missing parentheses in call to 'print'. Did you mean print(octave, i, end, vis.shape)?

Noob with little experience here,
When trying to run any comand i get this same error
  File "E:\AITHINGS\DeepDreamAnim-master\dreamer.py", line 80
    print octave, i, end, vis.shape
          ^
SyntaxError: Missing parentheses in call to 'print'. Did you mean print(octave, i, end, vis.shape)?

Any help?
Thanks!

You are most likely running the script using python3, this script was made in python2.


Favorite sample using  and SVG icon

Feedback welcome.
The heart can only be toggled once for now. Will take another stab at it once this PR lands, so we can render another <input> in the <div submit-success> template.

Nice hack!
I filed this a while ago to make this use case even easier to implement. The approach described there doesn't require rendering multiple form inputs.

We should wait until #4272 is fixed before creating a sample for this. Will close this for now.


[Enhancement] Use isEmpty() to check whether the collection is empty or not.[ ShowListenClientByTopicHandler]

Search before asking

[X] I had searched in the issues and found no similar issues.

Enhancement Request

located at:
eventmesh-runtime/src/main/java/org/apache/eventmesh/runtime/admin/handler/ShowListenClientByTopicHandler.java line 71
analysis and explanation:
Using Collection.size() to test for emptiness works, but using Collection.isEmpty() makes the code more readable and can be more performant. The time complexity of any isEmpty() method implementation should be O(1) whereas some implementations of size() can be O(n).
Describe the solution you'd like
use Collection.isEmpty().
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

please assign to me

@KamToHung Welcome, you can submit a pr first.


HTML5 Support

I spoke with Zend about DomDocument not being able to support HTML5. They said it has to do with the DomDocument maintainers, not PHP.
https://www.php.net/manual/en/class.domdocument.php
Apparently it has to do with the libxml2 maintainers.
https://bugzilla.gnome.org/show_bug.cgi?id=761534
There appears to be a PHP fix here, which looks to be a more bloated version of what LivingMarkup is currently doing. It checks for weird things like multiple <head> tags. Consider using, rather than reinventing the wheel with the Page element. And then removing the Component part to it's own processor.
https://github.com/ivopetkov/html5-dom-document-php

Filed bug at Bugzilla for LibXML htmlpaser.


Added some PT-BR terms translations

Added some PT-BR terms translations and updated a wrong one (line 86)

Coverage remained the same at 94.629% when pulling 817e5b509d3aff4a1538404f2c09fe8c8b841c7b on Grohden:patch-1 into d40d80c4617324a7a78fc41c9b111df5767d764b on MarshallOfSound:master.


[Feature Request] Support for ALT - CMD - CTRL Keys

Hello @Qwyntex !!!!
As we talked here #5  i’m opening an item about the support for modifiers keys:
CRTL+SHIFT+SHIFT
CMD+SHIFT+SHIFT
ALT/OPT+SHIFT+SHIFT
Thanks for reading this!!!
Have a great day!!!!

AFAIK this functionality has been integrated as of the latest release. Thank you for requesting the feature. It's very helpful and expands the value of the plugin greatly! :smile:

@OGfrostbyte is it actually supported? (combinations such as CTRL+SHIFT+SHIFT)

No, for now this is not possible

Greetings @nhan000. I apologize for misreading your original post. 😞 I didn't realize you were specifying modifier key combinations. That'll teach me to try and read/write comments as important as software issues after midnight and more than 36 hours awake! 😆 Anyway, please accept my humble apology. And as Qwyntex stated, apparently it's not currently possible. But knowing how great this plugin's author is, I wouldn't be surprised if Qwyntex is able to pull something out of a hat to support it in the future.
Hope you're having a pleasant day and many more in the future!
-frostbyte


Get userID from token when using authorizeSSO method.

I want to get userID when using authorizeSSO method.
so I created a PR referring to the code below.
https://github.com/twitter-archive/twitter-kit-ios/blob/2cd78d3d5783d998f6b5fde242d1dd69d32e385d/TwitterCore/TwitterCore/Identity/TWTRSession.m#L44

Sorry about the delay, this LGTM 👍. Thanks for the contribution.


Reintroduce the "Add more photos" button in the photo section of the edit mode

Problem

Reported by @alexgarel today
Reintroduce the "Add more photos" button in the photo section of the edit mode
It was described as a pain point to have to scroll the carousel to Add more, especially given that the carousel will let you add one, but not 2 additional photos
@manon-corneille also mentionned that she'd like to be able to add extra recycling instruction photos. Possibly a add more photos in the Recycling Instruction edit screen (and possibly in the structured packaging section), both disguised as 'Take photos of any packaging/recycling information'

That can be split in 2:

being able to display and add "unconventional" pictures (i.e. not the main 4)
the UI about it

from which screens
which action widget (FAB, elevated button at the end of the page, icon button at the end of the carrousel)

Now that we are more relaxed about the process about the main 4, I can work on the first part (background task about additional pictures).
But it would be cool if you guys could add screenshots.
In addition to that, let me remind you that it's not that easy, because the main 4 are fields of the product we download and store, that's cool.
But the additional pictures are not in the downloaded product.

Should we add them as fields? (a lot of space, but less trouble)
Should we add them in a compact way? (e.g. downloading whatever the server gives but storing only the imgid)
or else, we are forced to make an extra "download additional images" step, which can be lousy, UX-wise

For instance, let's have a look at the images field only: https://fr.openfoodfacts.org/api/v3/product/3017620425035?fields=images
It takes 21551 bytes. and with tons of data we don't need in Smoothie, like:
"166": {
	"sizes": {
		"100": {
			"h": 100,
			"w": 75
		},
		"400": {
			"h": 400,
			"w": 300
		},
		"full": {
			"h": 4027,
			"w": 3024
		}
	},
	"uploaded_t":<PHONE_NUMBER>,
	"uploader": "thierryp"
},

As all we need to know is the list of image ids, much more compact in a shorter string (666 bytes, sorry about that):
"10","100","101","102","103","104","105","106","107","108","109","11","110","111","112","113","114","115","118","119","12","120","121","122","123","124","125","126","127","128","129","13","130","133","134","135","136","137","138","139","14","140","141","142","143","144","145","146","15","151","152","153","154","155","156","157","158","159","16","160","161","162","163","164","165","166","167","168","169","17","170","171","18","19","2","20","21","22","23","24","25","26","3","38","4","40","41","42","5","6","69","7","70","71","72","73","74","75","76","77","78","79","8","80","81","82","83","84","85","86","87","88","89","90","91","93","94","95","96","97","98","99"
@stephanegigandet @alexgarel Is there a compact image product field, that would for instance just return the imgids?

Alternate solution if off-server cannot provide a simple list of imgids: in off-dart we could create "virtual" or "computed" fields, where we download one field (in that case, images) but recompute its content (in that case, trimming the unwanted verbose data).
We still download too much data, but we store only a decent amount.
Besides, we never write the images field as a normal product field, therefore we cannot overwrite the full data with our light data.

@monsieurtanuki This issue is about uploading more images, but we do not have to display them at all. So it's just a button to send an "other" image like we have in the carousel.
In fact it would be very messy on some products to display all images as we may have hundreds on popular products.

@stephanegigandet I can understand that adding images and displaying them are different topics, and indeed it would be very easy just to add pictures, and very messy to display them all.
That said, I assume it would be a very frustrating experience for the user if those images are virtually lost forever. A bit like /dev/null.
Let's say I add an "OTHER" picture. I crop it, I confirm. And then? It's invisible in the app, it's invisible on the web. I cannot even specify "btw it was a packaging".
Is there something I'm missing?

We display only 4 images per language (front, ingredients, nutrition, packaging). It is useful to be able upload more images than those 4, but we won't display them. They might be used by robotoff to spot logos, or to crop some things in other languages.
e.g. whenever I add a product, I upload cropped images for the 4 things, but then I often also send an image with the full back of the packaging, plus other sides etc.
We could think of ways to show the extra images (e.g. a "show more pictures" button in the carrousel), but there are tons of other ways we could think of to do that, so I think we should really split the issue, and do the simple thing first: allowing users to upload more photos, even if they don't see them. Then we can think about the display, but it's probably a much lower priority feature, and it's very unclear what we should do (e.g. displaying pictures in view mode, making the pictures available in edit mode in an advanced edit mode etc.).

@stephanegigandet OK then. Still not functionally convinced, but at least I can create a button that takes/crops/uploads a picture that won't be visible for the user.

Still not functionally convinced, but at least I can create a button that takes/crops/uploads a picture that won't be visible for the user.

Right now it's principally for advanced users, so it's normal it's not that "complete" (it would be misleading for classic users). It enables users like me to send images where there are logos or other information that robotoff can create insights from.

One thing for the confusion on where the image went, we could add another column to an "All images" page.
There features as "select as x" and "report" could go, we should have open issues for those already

@monsieurtanuki doesn't your last comment belong to another issue ?

@manon-corneille also mentionned that she'd like to be able to add extra recycling instruction photos. Possibly a add more photos in the Recycling Instruction edit screen (and possibly in the structured packaging section), both disguised as 'Take photos of any packaging/recycling information'

@alexgarel I don't think so: I added a button in both pages.

@monsieurtanuki perfect, looks good to me! Thanks

@alexgarel I don't think so: I added a button in both pages.

Yes but it's not about "more photo" but packaging photo !
But ok that's related in some way :-)


Revert "jsk_common: 2.2.2-0 in 'indigo/distribution.yaml' [bloom]"

Reverts ros/rosdistro#13515

@k-okada I'm reverting this because it's causing a regression we have the fixed version of univeral_robots that we synced without last cycle so we can get in the updated version with the fix.
https://discourse.ros.org/t/preparing-for-indigo-sync-2016-12-21/1004/2


Enable entry of groupChar for Number data type

Enable entry of groupChar for number data type.
See #374

Add input text box showing default values, which user can change.
Ensure this is sent with validation,export, etc.
Put input box between type and format.

Hi @Stephen-Gates
Not 100% sure yet, but I'm going to issue a ticket in frictionless as I think it doesn't allow space, " ", as a valid group Char separator - as far as I can tell the frictionless spec should allow it. Can still work on this ticket and close off, but what it might mean is that numbers with spaces will just be allowed through (even when the default is set which is nothing or "").

Testing seems to confirm. Before I send this as issue to frictionless (and so I can get on with this sprint release), @Stephen-Gates , I've been looking at:
https://frictionlessdata.io/specs/table-schema/#number
https://www.w3.org/TR/xmlschema-2/#decimal
https://www.w3.org/TR/xmlschema-2/#rf-whiteSpace
From what I can see, number is an atomic type, so whitespace should apply (with property collapse)? But then it also says that number (decimal from W3 spec) is:
finite-length sequence of decimal digits (#x30-#x39)
I note also that frictionless departs from W3 spec a bit as it allows leading trailing zeros whereas W3spec doesn't, but insists there must be digit on either side of decimal char.
Just thought I should check with you first, before putting pull request in.

@mattRedBox I’d add an issue as well as a pr. You probably won’t get a reply until tonight and discussion may follow before pr is accepted. Is the pr just to support space? Alternatively we could propose non-space Character in spec but I like the idea of handling spaces often placed in phone numbers


Calculate d0 for step 6

We need to do the opposite of angleCalibEDD (step 2).
In step 2: select peaks, provide a list of d-spacing -> calculate the 2theta angle
To obtain the d0 values: select peaks -> calculate d-spacing

Try to implement it before Thursday.


Fix bert/roberta contrastive search tests

What does this PR do?
Fixes contrastive search tests for bert and roberta
RUN_SLOW=true  GAUDI2_CI=1 python -m pytest tests/transformers/tests/models/bert/test_modeling_bert.py -v -s -k test_contrastive_generate_dynamic_shapes
RUN_SLOW=true  GAUDI2_CI=1 python -m pytest tests/transformers/tests/models/roberta/test_modeling_roberta.py -v -s -k test_contrastive_generate_dynamic_shapes

Fixes # (issue)
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


[Bug] DICOM JSON - Invalid DICOM file, expected header is missing

Describe the Bug
Hello,
I am using your script "dicom-json-generator.js" to generate the configuration JSON file that allows the "viewer" to read my DICOM files.
I have carefully followed the instructions on the page https://docs.ohif.org/configuration/dataSources/dicom-json, but when I run the script, I encounter this error message for certain series: "Error: Invalid DICOM file, expected header is missing."
These DICOM files were downloaded from Google Cloud Healthcare, so they should normally be well-formed.
Would you have an explanation or a solution to prevent the script from completely halting? For instance, is there a way to skip only the problematic files without causing an error that stops the entire script?
Steps to Reproduce
It is not possible to reproduce without DICOM file with error.
The current behavior
I have carefully followed the instructions on the page https://docs.ohif.org/configuration/dataSources/dicom-json, but when I run the script, I encounter this error message for certain series: "Error: Invalid DICOM file, expected header is missing."
The expected behavior
The expected behavior is for the script to run to completion and generate the JSON file.
OS
Ubuntu
Node version
10.2.5
Browser
Brave v1.73.91

Did you try drag and drop it in other viewers to see if it works?

Excuse me, I just found the problem. My DICOM files were downloaded incorrectly and were corrupted.


NoDamBreak Mod

Mod uniqueName
likemauro.nodambreak
Mod name
No Dam Break
GitHub repository URL
https://github.com/LikeMauro/NoDamBreak-Mod
Parent uniqueName
No response
Utility mod
No response

Your thumbnail has a huge white part in the bottom, would probably fix that


Move Wdrls to Core

Description
Closes #3194
Checklist

[x] Commit sequence broadly makes sense
[x] Commits have useful messages
[x] New tests are added if needed and existing tests are updated
[x] Any changes are noted in the changelog
[x] Code is formatted with fourmolu (which can be run with scripts/fourmolize.sh
[x] Self-reviewed the diff

Thanks for the reviews! :raised_hands: I will make the final set of changes and merge soon!


PouchDB v5.4.5

I get an exception Cannot read property 'Promise' of undefined from pouch-utils.js when I upgrade to the latest version of PouchDB:
Promise = window.PouchDB.utils.Promise

Sorry yeah, PouchDB.utils was removed and this project still mistakenly contains references to it.

Awesome thanks, I'll see if that fixes it and close this issue if so.

1.4.4 is published and contains the fix. thanks for reporting!


[New Feature]: Create a Terraform script to copy ancillary data

Checked for duplicates
No duplicates exist.
Alternatives considered
The ancillary data could be copied via a shell command but this Terraform script would make the task easier.
Describe the feature request
Create a Terraform script that would copy data from an S3 bucket to an existing EFS partition, or that can run an arbitrary shell script. The script would be executed as a post-deployment step by a project.

@LucaCinquini i'm wondering how this scales. Making this a part of the terraform script only works if the deployment of the SPS is configurable by the project directly. In a world where we are creating a venue separate from the deployment of a workflow, how does this work?
Straw man: the algorithm package can have "dependencies" that are copied at the time of deployment (e.g. the WPS-T deploy process) so that subsequent executions would have this available. @hookhua is there anything in the WPS-T spec that would allow for that? or the Application Package spec?

It seems like there are a few different topics here.
(a) There are some built-in support for deployment at run-time in the OGC specs, but it is at the context of the algorithm steps. OGC WPS-T supports deployment of algorithms into an WPS-T endpoint (the ADES). And the Application Package has notions of data staging, but that is in the context of accessing and copying data into a local processing step / job.
(b) There is a separate topic of setting up a local "cache" of static ancillary/auxiliary files into a venue environment, which is reused across several jobs. This is more so for granules that are expected to have recurrent data access patterns.
(c) Separately still is the notion of setting up the venue resources, such as deploying an ADES cluster inside U-SPS for a tenant.
So overall, we can see the following steps:

U-SPS deploys a tenant-specific resource (e.g. ADES processing cluster) into a venue.
Pre-stage static ancillary/auxiliary files into S3 that is collocated with the ADES.
Deploy algorithm(s) into one of the ADES clusters.
Submits jobs to the ADES cluster.
When the job gets dispatched to run, it will do the stage-in, process, and stage-out steps local to each compute node. This may be referencing any local "cache" of the static files collocated to the compute.

If this ticket is referring to (2), then Terraform script could be used for this setup of a collocated set of data. This is useful for setting up a complete environment with resources and data. There could then be a similar Terraform script for the tear-down stage when a tenant is done.
Alternatively, if this ticket is referring to (5), then it can leverage the built-in data staging logic by U-DS. This prevents Terraform scripts from having to duplicate similar capabilities as the stage-in step. e.g. the logic of staging data from a DAAC via the TEA using EDL credentials. For some ADESes, the backend could be EFS as well, so the same could be done for stage-in into a shared EFS volume.

Hi guys, this ticket refers to step 2. above. Using a Terraform or shell script to stage the ancillary data (once only) was only meant to be a temporary solution, to show how a project could do it. I agree that building a general functionality into the system following an OGS spec would be a much better idea, if we can do it - but also a much longer effort.

Closing this ticket as "non-fix" for now... Copying ancillary data is for now a manual step executed by the project, from some S3 location to some EFS partition provisioned off band. The ticket can be re-opened when the use case is better defined.


Pending load

About this one, @matejsuchanek .
Something has gone wrong. "Get pages" is disabled also after pages are loaded.

Noticed, #58

Yes, I also noticed after pressing save :)


Added ppo hyperparams for 10 mujoco envs

Description
Using Optuna, I experimented with 150-200 different hyperparameters for 10 MuJoCo environments, and found the best one among them. Among the entire hyperparameters, I selected 10 best params and retrained them for 5-10 times to eliminate the biases. You can check out the optimization history graph and importances of each parameter for the environments here.
Motivation and Context

[ ] I have raised an issue to propose this change (required for new features and bug fixes)

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation (update in the documentation)

Checklist:

[x] I've read the CONTRIBUTION guide (required)
[ ] I have updated the changelog accordingly (required).
[ ] My change requires a change to the documentation.
[ ] I have updated the tests accordingly (required for a bug fix or a new feature).
[ ] I have updated the documentation accordingly.
[x] I have reformatted the code using make format (required)
[x] I have checked the codestyle using make check-codestyle and make lint (required)
[ ] I have ensured make pytest and make type both pass. (required)

Note: we are using a maximum length of 127 characters per line

Hello,
thanks you very much for this PR.
I will try to have a closer look soon.
After a quick look, is there a reason you did not try gSDE (use_sde=True and sde_sample_freq currently commented in the hyperparameter optimization code)?

Hi, thank you for the review.
Actually there was not a specific reason for not trying gSDE. I just wanted to experiment with current default settings as specified in the code, and just left commented lines commented out.
I'm not so sure, but do you think the gSDE parameters would largely affect the optimization result? In that case I have to include them for sure.

I'm not so sure, but do you think the gSDE parameters would largely affect the optimization result? In that case I have to include them for sure.

I don't think they would affect much the result (as gSDE is meanly meant for real robot training) but as I experienced with pybullet, it could lead to slightly better performance at the end (see Figure 3b in https://arxiv.org/abs/2005.05719).

@araffin so the reason we didn't include gSDE is that we ran this search for a paper we're submitting to ICLR that requires using very standard configs, and gSDE is not the standard thing everyone does for PPO. While we obviously could rerun these with gSDE, it would be very compute intensive and it's not really something we have the bandwidth for right now.


[MacOS] Argon can only be executed as root

I have installed the cli using the file downloaded from GitHub, it was placed in /usr/local/bin/argon and has the following permissions
-rwxr-xr-x@ 1 root  wheel  18683352 Aug  9 15:52 /usr/local/bin/argon*

Running argon serve -vvvv (File paths truncated)
WARN: Failed to load config file: No such file or directory (os error 2) [argon:51]
WARN: Failed to verify Argon installation: unsupported shell [argon:56]
TRACE: Initializing VFS [argon::core:40]
DEBUG: Update check already performed within the last hour [argon::updater:154]
INFO: Update check completed successfully! [argon:62]
TRACE: Snapshotting root project [argon::core:44]
TRACE: Creating snapshot of [...]/default.project.json [argon::middleware:146]
DEBUG: Stats already synced within the last hour or too few stats to sync [argon::stats:144]
INFO: Stat tracker initialized successfully! [argon:69]
TRACE: Creating snapshot of [...]/src/ReplicatedFirst [argon::middleware:146]
TRACE: Creating snapshot of [...]/src/ReplicatedStorage [argon::middleware:146]
TRACE: Creating snapshot of [...]/src/ServerScriptService [argon::middleware:146]
TRACE: Creating snapshot of [...]/src/ServerScriptService/MatchmakingTask [argon::middleware:146]
TRACE: Creating snapshot of [...]/src/ServerScriptService/MatchmakingTask/MatchmakingServer [argon::middleware:146]
TRACE: Creating snapshot of [...]/src/ServerScriptService/MatchmakingTask/MatchmakingServer/Match.luau [argon::middleware:146]
TRACE: Creating snapshot of [...]/src/ServerScriptService/MatchmakingTask/MatchmakingServer/KillerProbability.luau [argon::middleware:146]
TRACE: Creating snapshot of [...]/src/ServerScriptService/MatchmakingTask/.data.json [argon::middleware:146]
TRACE: Snapshot of [...]/src/ServerScriptService/MatchmakingTask/.data.json not created: no middleware matched [argon::middleware:154]
TRACE: Creating snapshot of [...]/src/ServerScriptService/BankingServer [argon::middleware:146]
TRACE: Creating snapshot of [...]/src/ServerStorage [argon::middleware:146]
TRACE: Creating snapshot of [...]/src/StarterGui [argon::middleware:146]
TRACE: Creating snapshot of [...]/src/StarterPack [argon::middleware:146]
TRACE: Creating snapshot of [...]/src/StarterPlayer/StarterCharacterScripts [argon::middleware:146]
TRACE: Creating snapshot of [...]/src/StarterPlayer/StarterPlayerScripts [argon::middleware:146]
TRACE: Creating snapshot of [...]/src/Workspace [argon::middleware:146]
TRACE: Creating snapshot of [...]/src/Workspace/.data.json [argon::middleware:146]
TRACE: Snapshot of [...]/src/Workspace/.data.json not created: no middleware matched [argon::middleware:154]
TRACE: Building Tree and Queue [argon::core:53]
TRACE: Starting Processor [argon::core:59]
TRACE: Core initialized successfully! [argon::core:69]
WARN: Port 8000 is already in use, using 8001 instead!
ERROR: Command execution failed: Permission denied (os error 13)

Running as root does work but it complicates my workflow since files that are synced back will be owned by root preventing write by my user.

Looks like this issue is caused by the actix-web crate, but I don't think this is the actual problem.
Argon automatically moves/copies its binary to the ~/.argon/bin/argon when it gets first launched. Could you verify that the proper executable is used? Just run whereis argon or to be sure remove ~/.argon/bin directory, run Argon, and check whether you get Installation completed! Do you want to remove this executable? prompt.

My argon never does that, I've just deleted the ~/.argon folder, and ran argon, all it does is say what commands are available. The folder is created again but it never moves itself there.

I just moved argon to the ~/.argon/bin folder and I think it now works fine, but installation is completely broken, if needed I can open another issue for that

Yeah fish is not supported, I'm going to add list of the supported shells to the wiki.
You can give zsh a try though bash should have worked just fine. Could you check if Argon throws the same unsupported shell error when using bash and zsh?

I will try it but I’d have to serve a project to serve a project to see the error since verbose is not available without a command.
Message ID: @.***>

I'm not sure what you mean. Just run argon serve -vvvv on any project.

I'm closing this issue as it's no longer relevant but feel free to re-open it.


fix typo causing undefined encoding in buffers

this typo prevents me from using writeStringImpl...

Oops, thanks!

I wonder if this should be a breaking change, since it technically changes behavior even though it's a bug fix.
cc @hdgarrood @garyb

I'm pretty confident this should be a patch level change. What kind of bug fix doesn't change behaviour?

Good point.


Can't run using nektos/act

Describe the bug
I'm trying to run my workflow locally using nektos/act and this error keeps coming up in the output when setting up PHP, tools and exts;
| /usr/bin/php: error while loading shared libraries: libsodium.so.23: cannot open shared object file: No such file or directory

It eventually fails completely with anything using the php executable;
| /usr/bin/php: error while loading shared libraries: libsodium.so.23: cannot open shared object file: No such file or directory
|<EMAIL_ADDRESS>line 134: ext_dir: parameter null or not set
[Testing Laravel/Laravel (PHP 8.0 on ubuntu-latest)]   ❗  ::error::The process '/usr/bin/bash' failed with exit code 1
[Testing Laravel/Laravel (PHP 8.0 on ubuntu-latest)]   ❌  Failure - Setup PHP, with composer and extensions
Error: exit with `FAILURE`: 1

Without tools and extensions defined I get this;
| /usr/bin/php: error while loading shared libraries: libsodium.so.23: cannot open shared object file: No such file or directory
[Testing Laravel/Laravel]   ❌  Failure - PHP version

I've used this action before for a package I made and it worked fine. I just ran it again (both projects using v2) and now I'm getting the same error there too.
Version

[x] I have checked releases, and the bug exists in the latest patch version of v1 or v2.
[x] v2
[ ] v1

Runners

[ ] GitHub Hosted
[ ] Self Hosted
[x] Local (nektos/act)

Operating systems
ubuntu-latest
PHP versions
8.0
To Reproduce
Use the setup-php action, create a step to output php version and run it locally using act.
Expected behavior
Run workflow locally without errors.
Screenshots/Logs
❯ act --secret-file ~/.config/act/secrets --container-architecture linux/amd64
[Testing Laravel/Laravel] 🚀  Start image=catthehacker/ubuntu:act-latest
[Testing Laravel/Laravel]   🐳  docker run image=catthehacker/ubuntu:act-latest platform=linux/amd64 entrypoint=["/usr/bin/tail" "-f" "/dev/null"] cmd=[]
[Testing Laravel/Laravel]   🐳  docker exec cmd=[mkdir -m 0777 -p /var/run/act] user=root
[Testing Laravel/Laravel]   🐳  docker cp src=/Users/andrew/Code/gruplet/gateway/. dst=/Users/andrew/Code/gruplet/gateway
[Testing Laravel/Laravel]   🐳  docker exec cmd=[mkdir -p /Users/andrew/Code/gruplet/gateway] user=
[Testing Laravel/Laravel] ⭐  Run Checkout
[Testing Laravel/Laravel]   ✅  Success - Checkout
[Testing Laravel/Laravel] ⭐  Run Setup PHP, with composer and extensions
INFO[0002]   ☁  git clone 'https://github.com/shivammathur/setup-php' # ref=v2
[Testing Laravel/Laravel] Non-terminating error while running 'git clone': some refs were not updated
[Testing Laravel/Laravel]   🐳  docker cp src=/Users/andrew/.cache/act/shivammathur-setup-php@v2/ dst=/var/run/act/actions/shivammathur-setup-php@v2/
[Testing Laravel/Laravel]   🐳  docker exec cmd=[mkdir -p /var/run/act/actions/shivammathur-setup-php@v2/] user=
[Testing Laravel/Laravel]   🐳  docker exec cmd=[node<EMAIL_ADDRESS>user=
| [command]/usr/bin/bash /opt/hostedtoolcache/linux.sh 8.0 /run/act/actions/shivammathur-setup-php@v2/dist
|
| ==> Setup PHP
| /usr/bin/php: error while loading shared libraries: libsodium.so.23: cannot open shared object file: No such file or directory
| /usr/bin/php: error while loading shared libraries: libsodium.so.23: cannot open shared object file: No such file or directory
| /usr/bin/php: error while loading shared libraries: libsodium.so.23: cannot open shared object file: No such file or directory
| /usr/bin/php: error while loading shared libraries: libsodium.so.23: cannot open shared object file: No such file or directory
| /usr/bin/ln: failed to create symbolic link '/Users/andrew/Code/gruplet/gateway/docker/8.0/conf.d/99-pecl.ini': No such file or directory
[Testing Laravel/Laravel]   ⚙  ::set-output:: php-version=
| ✓ PHP Installed PHP
|
| ==> Setup Tools
| /usr/bin/php: error while loading shared libraries: libsodium.so.23: cannot open shared object file: No such file or directory
| /usr/bin/php: error while loading shared libraries: libsodium.so.23: cannot open shared object file: No such file or directory
| ✓ composer Added composer 2.1.8
|
| ==> Support this project
| ✓ setup-php https://setup-php.com/support
[Testing Laravel/Laravel]   ✅  Success - Setup PHP, with composer and extensions
[Testing Laravel/Laravel] ⭐  Run PHP version
[Testing Laravel/Laravel]   🐳  docker exec cmd=[bash --noprofile --norc -e -o pipefail /Users/andrew/Code/gruplet/gateway/workflow/2] user=
| /usr/bin/php: error while loading shared libraries: libsodium.so.23: cannot open shared object file: No such file or directory
[Testing Laravel/Laravel]   ❌  Failure - PHP version
Error: exit with `FAILURE`: 127

Additional context
act version 0.2.24
Are you willing to submit a PR?

Here's a simplified workflow for reproducing.
# GitHub Action for Laravel
name: Testing Laravel

on: [push, pull_request]

jobs:
  laravel:
    name: Laravel

    runs-on: ubuntu-latest

    steps:
      - name: Checkout
        uses: actions/checkout@v2

      - name: Setup PHP, with composer and extensions
        uses: shivammathur/setup-php@v2 #https://github.com/shivammathur/setup-php
        with:
          php-version: '8.0'
#          extensions: mbstring, dom, fileinfo
#          coverage: xdebug #optional

      - name: PHP version
        run: php -v

Oh and I'm using Docker for Mac on a MBP M1 macOS 11.6. Error persists when using arch emulation too for act. (--container-architecture linux/amd64)

Please use shivammathur/node images for running setup-php on act.
https://hub.docker.com/r/shivammathur/node
act -P ubuntu-latest=shivammathur/node:latest

Refer to: https://github.com/shivammathur/setup-php#local-testing-setup

Sorry, I missed that in the readme. I've tried it and it's running fine now. Thank you for your help and the quick response.

act

what is act?


How to check if an alternative already exists for a given user?

Apparently, there is no API for this. Am I missing something?
Since I have multiple sixpack clients, I need to know whether a given user is already participating to the experiment or not. If yes, I am able to track conversions. If not, the API will return a 400 error.
It would be ok if the API returned a specific error code, so that I can handle this case without a specific API call. Semantically, a 404 HTTP status code might work. Does it make sense?
Thank you.

Here is my PR: https://github.com/seatgeek/sixpack/pull/212

:+1:
For us we're logging second page views as a fake KPI that gets ignored since the only way to fetch an existing alternative is to convert them.

Hello,
This feature could be very useful for us too.
We started to use sixpack a few months ago.
In our test paths, a user can go on multiple webpages, but triggers a a/b test experiment only on specific actions, which then lead to other behaviours later on.
We need to see if a user has been associated to an alternative or not, without triggering a unwanted association.
From what I see, the function already exists ('Experiment. existing_alternative') but has not been exposed through the API.
For now on, we will probably try the 'dirty way' by using the 'convert' API call.
Thanks for hearing me out !


Fix SPMe Potentials

delta_phi != phi_s - phi_e for the SPMe. Worth checking the analytic solutions or if this needs a corrector step.

A good test would be to implement a BasicSPMe and see if the bug still arises.


Add smoketest builds for openvpn3, openvpn3-linux and ovpn-dco

The openvpn3-linux smoketest builds include both OpenSSL and MbedTLS builds. If those builds pass then the rest of the builds are launched.
The openvpn3 and ovpn-dco smoketests include just one build, followed by the normal builds if all goes well.
I've tested each of these individually, but independent testing would be good.

I pushed three more commits that:

Remove openvpn3-linux mbedtls builds (smoketest and normal)
Remove unnecessary "combo" and "config_opts" parameters from openvpn3-linux, openvpn3 and ovpn-dco

I ran them through my private buildbot with one smoketest and one normal builder and all looks good afaics.


Support multiple content streams

Fixes #19

Codecov Report

Merging #20 (6cba545) into main (1f553fc) will decrease coverage by 0.02%.
The diff coverage is 66.66%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main      #20      +/-   ##
==========================================
- Coverage   61.32%   61.30%   -0.02%
==========================================
  Files          25       25
  Lines        1166     1176      +10
==========================================
+ Hits          715      721       +6
- Misses        451      455       +4

Files Changed
Coverage Δ

lib/src/model/pdf_page.dart
87.71% <66.66%> (-5.90%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


NNotepad: fix load(), add zeros(), improve tensor output display

The load() helper was broken when the scripts were converted to modules, and also when array parsing was updated. Oops! Fix it.
Add zeros() helper that creates a zero-initized tensor of the given shape. Useful for testing shape display.
Ensure tensor output spaces n-dimensions tensors the same as numpy - more spaces between each layer.
Ensure tensor output handles 0-sized dimensions.

@Honry - can you review this too? It started off as a quick fix to handle outputting 0-sized dimensions, and grew from there.


Remove tailoring of personal statement

Removes tailoring of personal statement by course choice
Splits personal-statement, subject-knowledge and interview into three separate fields, grouped under a ‘Personal statement and interview’ heading.

Can we write this up in design history, there's been much to and fro about this, we need to record our reasons for each change I think.


unsquashfs doesnt' extract files in squashfs-root

Tried to use binwalk on the attached file, but had strange results with unsquashfs. It soted files from squashfs into _P-330W_EE_V3.60(AMJ.5)D0_httpupgrade.bin.extracted, not in _P-330W_EE_V3.60(AMJ.5)D0_httpupgrade.bin.extracted/squashfs-root
If I try 'unsquashfs F8010.squashfs' in the _P-330W_EE_V3.60(AMJ.5)D0_httpupgrade.bin.extracted, I got an error message and no files at all. Why I got files from binwalk if manual run of unsquashfs fails?
P-330W_EE_V3.60(AMJ.5)D0_httpupgrade.zip

I don't know how files from the squashfs image would end up in the wrong directory. Was the squashfs-root directory created at all? Can you provide a directory listing of the _P-330W_EE_V3.60(AMJ.5)D0_httpupgrade.bin.extracted contents? When I run binwalk, it creates a squashfs-root directory and places all files there.
As for unsquashfs, it can't handle vendor-specific modifications, which this squashfs image apparently does have. If unsquashfs fails to extract an image, binwalk will attempt to use sasquatch (assuming it is installed on your system), which is a patched version of unsquashfs which handles these non-standard squashfs images better.
For debugging purposes, can you run the following command on your system and post the output (there will be a lot of debug stuff printed to screen):
$ python -O $(which binwalk) -e P-330W_EE_V3.60\(AMJ.5\)D0_httpupgrade.bin

Yes, squashfs-root was created, but extracted files are in _P-330W_EE_V3.60(AMJ.5)D0_httpupgrade.bin.extracted. The listing is attached.
I run the debugging command, the redirected output is attached too.
I'm using binwalk 2.1.1, squashfs-tools 4.3-r2 and I don't have sasquatch.
debug.txt
listing.txt

Ah, here is the issue, from debug.txt:
DEBUG: External extractor command "7z e -y '%e'" completed with return code 2 (success: False)
DEBUG: Running extractor '7z e -y '%e''
DEBUG: subprocess.call(7z e -y '20810.7z', stdout=None, stderr=None)

7-Zip [32] 16.02 : Copyright (c) 1999-2016 Igor Pavlov : 2016-05-21
p7zip Version 16.02 (locale=en_US.utf8,Utf16=on,HugeFiles=on,32 bits,1 CPU Intel(R) Pentium(R) M processor 1.73GHz (6D8),ASM)

Scanning the drive for archives:
1 file, 1816586 bytes (1775 KiB)

Extracting archive: 20810.7z

WARNINGS:
There are data after the end of archive

WARNING:
20810.7z
Can not open the file as [7z] archive
The file is open as [SquashFS] archive

--
Path = 20810.7z
Open WARNING: Can not open the file as [7z] archive
Type = SquashFS
WARNINGS:
There are data after the end of archive
Offset = 882696
Physical Size = 933888
Tail Size = 2
Headers Size = 7348
File System = SquashFS 3.0
Method = LZMA Spec
Cluster Size = 65536
Big-endian = +
Created = 2009-10-27 04:38:59
Characteristics = DUPLICATES_REMOVED

Everything is Ok

The fix is to upgrade to the latest binwalk from github which addresses this, but here is a more detailed explanation:
There is a block of LZMA data that precedes the SquashFS file system at offset 0x20810:
DECIMAL       HEXADECIMAL     DESCRIPTION
--------------------------------------------------------------------------------
32784         0x8010          bzip2 compressed data, block size = 900k
98304         0x18000         CSYS header, big endian, size: 131072
133136        0x20810         LZMA compressed data, properties: 0x5D, dictionary size: 8388608 bytes, uncompressed size: 1732656 bytes
1015824       0xF8010         Squashfs filesystem, big endian, version 3.0, size: 929841 bytes, 182 inodes, blocksize: 65536 bytes, created: 2009-10-27 01:38:59

Because the LZMA file format does not provide any information on how large the compressed data is, binwalk grabs everything from offset 0x20810 to the end of the firmware file and saves it to a file called 20810.7z. This means that the 20810.7z file also contains a copy the SquashFS file system which comes after the LZMA compressed data.
Since it is an LZMA file, binwalk then runs the 7z utility to try to decompress the contents of the  20810.7z file. However, some newer versions of the 7z utility examine trailing data after the actual LZMA compressed data, and will detect and extract SquashFS file systems too, rather than just decompressing the LZMA data. As you've seen, this extraction leaves a mess of files, without preserving directories and file system structure. I'm not sure if this is a shortcoming in 7z, or a side-effect of the SquashFS file system having seemingly been modified by whoever made the firmware image, but in either case it is undesirable.
Binwalk then finds the SquashFS file system, copies it out to a file named F8010.squashfs, and attempts to run the  unsquashfs utility to extract the contents of the file system to a directory named squashfs-root. The unsquashfs utility creates the squashfs-root output directory, however, due to the non-standard modifications apparently made to the SquashFS file system, unsquashfs fails to extract any files and bails.
Because unsquashfs failed, binwalk then attempts to run sasquatch against the F8010.squashfs file, but since sasquatch is not installed on your system this obviously fails as well.
The end result is that you have a bunch of extracted files from the F8010.squashfs file, but without any directory structure, and an empty squashfs-root directory.
Newer versions of binwalk no longer use 7z for LZMA decompression, thus avoiding variations in how different versions of 7z handle trailing data in LZMA files. If you install the latest binwalk, as well as the sasquatch utility, everything should work as expected and the file system contents will be extracted properly to the squashfs-root directory. Note: if you want LZMA files to also be properly decompressed with the latest version of binwalk, you will also need the python-lzma Python module (this module is included by default in Python3, so this extra installation is only necessary for Python2).

Wow, 7z supports the SquashFS? Didn't know about it! Thank you for explanation!

Newer versions of binwalk no longer use 7z for LZMA decompression,

As of today - 7z is enabled by default via config.
https://github.com/ReFirmLabs/binwalk/blob/cddfede795971045d99422bd7a9676c8803ec5ee/src/binwalk/config/extract.conf#L29-L30
Same problem as reported.
Commenting …
#^lzma compressed data:7z:7z e -y '%e':0,1
#^xz compressed data:xz:7z e -y '%e':0,1

e.g.
sed -i -E 's/^(\^.*:7z .+)$/#\1/' binwalk/src/binwalk/config/extract.conf

and it works as required.


Exception calling subscribe callback: 'float' object has no attribute 'slots'

Why do I get this on the ROS side bridge when I try to plot?
What does that mean? Any idea?

Can you provide some more details? Which version of ROS, what are you advertising on the topic, publisher side code, etc.

Ros Kinetic
Happens with a lot of kinds of topics but not all
For example if I try to plot latitude values from sensor_msgs/NavSatFix Message

Ah I think this should get fixed once we implement the webviz side of this PR: https://github.com/RobotWebTools/rosbridge_suite/pull/452 We'll have to make some time for that soon, so stay tuned!

I have the same issue when subscribing to /odometry /tf /joint_states messages. How's the fix doing? Btw, great tool you have here!

I have the same issue even if I subscribe to image topic.

This should be fixed now since #396. You'll have to use the latest version of the rosbridge now. Let me know if you still have any issues!


Android 1.1.x fix

Hi,
First thanks for this plugin. This pull request should solve the issue #37 . I didn't find any change regarding the testedVariant property which is null. As a fix I just use the testVariant instead of testedVariant to get 'outputs' directory. It fixes the problem in my current project. I've also updated the example app to the new version of android plugin. Let me know if It's fine for you.

I believe it can work only if tested app is already installed on a device or you are testing a library.
I raised a bug for the Android Tools team https://code.google.com/p/android/issues/detail?id=140267
Let's see what they say.

You're right with this fix It only works if tested app is already installed. I've made some tests and found that the testedVariant property is set when checked in an afterEvaluate closure. I've wrapped the apply body in an afterEvaluate block and It fixes this issue.

Any updates on this?

Not yet.

Closing this. It seems to work with Android plugin 1.1.0.


warning: field 'm_posix_timestamp_high' will be initialized after field 'm_posix_timestamp_low' [-Wreorder]

We cannot check for warnings in our header files and enable -Werror in part because some header files define constructors that attempt to initialize data members out of order:
/Users/jong/Source/turicreate/src/flexible_type/flexible_type_base_types.hpp:99:27: warning:
      field 'm_posix_timestamp_high' will be initialized after field
      'm_posix_timestamp_low' [-Wreorder]
  inline flex_date_time():m_posix_timestamp_high(0), m_posix_timestamp_low(0),
                          ^

We should always write member initializer lists in the actual order of initialization, then enable -Wreorder.

Fixed in #775


fix: open document side by side with chat panel

RE: https://sourcegraph.slack.com/archives/C05AGQYD528/p1699403470817339
fix: open document side by side with chat panel
Check current webview panel location before opening the document to show it side by side instead of replacing the chat panel.
Test plan

https://github.com/sourcegraph/cody/assets/68532117/bca885f8-8b09-4908-8500-707e5b3eeb91

This is better than the current behavior, but what if there are more than two columns?

good point. Updated to calculate the view column to open the file in based on the current webview panel's view column.
This will make sure to always open the file in the view column beside the current webview panel instead of hardcoded view columns.


Unicode glyphs in tree widget not showing

On the readthedocs page for some widgets my computer is not able to render certain characters.
For example for the TreeWidget the expansion "icon" is trying to render a character but all I see is boxes containing "23F7" for expanded and "23F5" for contracted. This occurs with IE/Chrome/Firefox. Is this expected behavior?
I'm on a pretty standard windows7 installation. Maybe there are some more common default choices.

Maybe charset meta tag / http header is not properly emitted by flexx?

Mmm, My guess is that the font being used does not contain these Unicode glyphs. I assumed that these glyphs would be available in all the common fonts. Guess I was wrong.
Perhaps we should set the fonts in these places to one that we know contains the glyphs (and are present on a lot of systems). Or maybe even ship a font? (e.g. FontAwesome).

Combobox should also be affected.

Yes, Combo box is affected. I'm supportive of shipping a font, it's the only way to absolutely ensure those glyphs are present, additionally should help create a common viewing experience.

Would like to mention that best practises on the web recommend sticking to safe characters and fonts, for legibility with screen readers or oddball devices. Anyhow,  the added font will work, although it's a pretty big overhead just for a glyph or two.

You'll find that supporting them crossbrowser is typically quite complicated.

It needn't be just a glyph or two. FontAwesome has awesome stuff in it. It was originally designed for bootstrap so it seems like it shouldn't be too hard to support cross browser, or at least it shouldn't be hard to find out.

Looking at the list of characters that font awesome has, http://fontawesome.io/cheatsheet/, it is very easy to see the wide range of applications it would have within Flexx. In fact, it seems like a perfect fit for a lot of cases.
Any of the fa-plus and fa-minus icons would be ideal for the the tree widget.

It's not that I don't see the value; I just think the custom font doesn't need to be a dependency by default. As in, my application doesn't need it, so why would I have to incur the page load time hit?

I agree that fontawesome is great, and see value in making it real easy to use from Flexx. But I agree that it should probably not be a default dependency. Also for the reason that it would remove the capability to export an app to a single HTML page.

Agreed.
Maybe this CSS based solution would work for the TreeWidget expand and contract "icons"?
http://jsfiddle.net/jlunzer/p9ghktfb/
It's not perfect but I think the aesthetic works. float and margins are only there for presentation.

That's a nice and robust solution for at least the collapse/expand button, thanks! I will look into the fonts a bit too, but need to fire up my win7 VM for that. BTW, does the checkbox look ok, and is this issue present in all browsers?

I tried in most recent Stable Firefox/Chrome and IE11, all displayed the same missing character glyph.

And yes, the checkbox class looks okay.

Just checked this out, the more common glyphs show up on my machine and are a suitable replacement.
I think there is still room for more pleasing aesthetics on the tree widget, but I'm sure you've got bigger fish to fry and what we've got now will is good enough. If I get some time I might try to implement the CSS solution in the tree widget.

Awesome! Thanks for reporting this, BTW.
I'm sure there are things we can do to make it look better, but such tweaks are not my strong suit :)


Limit Permit Generation

The bot should check for the Permitted label, or the comments in the issue, to see if a permit was already generated and disable generation for a second time.
In the future we should set the nonce to be related to the issue ID so that each issue is cryptographically verified to only be permitted once.

          ### [ **[ CLAIM 600 DAI ]** ](https://pay.ubq.fi)

0xf76F1AC...12DEA3f1
Originally posted by @ubiquibot[bot] in https://github.com/ubiquity/ubiquibot/issues/288#issuecomment-1551881478

In the future we should set the nonce to be related to the issue ID

Yes, we could use smth like nonce = keccak256(issueNodeId) (keeping in mind that issue's node id is unique)

Can we do this with our current setup or are there blockers? I prefer that as the bounty as it is a more robust solution.

Can we do this with our current setup or are there blockers? I prefer that as the bounty as it is a more robust solution.

We already issue.node_id as a nonce: https://github.com/ubiquity/ubiquibot/blob/development/src/handlers/payout/index.ts#L69
So each issue has a unique nonce and a single permit can not be claimed twice

Okay so then this bounty will be just so that the bot doesn't issue a new permit comment.

What is different about what we are doing currently and your comment?

I thought we don't have this feature (because I don't remember the corresponding task) but then I checked the codebase and this feature was implemented as a part of other issue

[ CLAIM 56.25 DAI ]
0x00868BB3...246b77E46

Looking forward to #341


UserWarning: Was asked to gather along dimension 0, but all input tensors were scalars

Describe the bug
Not sure if this is a bug. It shows me this warning at the beginning of training:
/home/adrian/research/envs/research/lib/python3.7/site-packages/torch/nn/parallel/_functions.py:61: UserWarning: Was asked to gather along dimension 0, but all input tensors were scalars; will instead unsqueeze and return a vector.
To Reproduce
The minimal MNIST example from the docs has this problem when trained on multiple GPUs. Attached the Python script:
import os
import torch
from torch.nn import functional as F
from torch.utils.data import DataLoader
from torchvision.datasets import MNIST
import torchvision.transforms as transforms

import pytorch_lightning as pl

class CoolModel(pl.LightningModule):

    def __init__(self):
        super(CoolModel, self).__init__()
        # not the best model...
        self.l1 = torch.nn.Linear(28 * 28, 10)

    def forward(self, x):
        return torch.relu(self.l1(x.view(x.size(0), -1)))

    def training_step(self, batch, batch_nb):
        # REQUIRED
        x, y = batch
        y_hat = self.forward(x)
        return {'loss': F.cross_entropy(y_hat, y)}

    def validation_step(self, batch, batch_nb):
        # OPTIONAL
        x, y = batch
        y_hat = self.forward(x)
        return {'val_loss': F.cross_entropy(y_hat, y)}

    def validation_end(self, outputs):
        # OPTIONAL
        avg_loss = torch.stack([x['val_loss'] for x in outputs]).mean()
        return {'avg_val_loss': avg_loss}

    def test_step(self, batch, batch_nb):
        # OPTIONAL
        x, y = batch
        y_hat = self.forward(x)
        return {'test_loss': F.cross_entropy(y_hat, y)}

    def test_end(self, outputs):
        # OPTIONAL
        avg_loss = torch.stack([x['test_loss'] for x in outputs]).mean()
        return {'avg_test_loss': avg_loss}

    def configure_optimizers(self):
        # REQUIRED
        return [torch.optim.Adam(self.parameters(), lr=0.02)]

    @pl.data_loader
    def tng_dataloader(self):
        return DataLoader(MNIST(os.getcwd(), train=True, download=True, transform=transforms.ToTensor()), batch_size=32)

    @pl.data_loader
    def val_dataloader(self):
        # OPTIONAL
        # can also return a list of val dataloaders
        return DataLoader(MNIST(os.getcwd(), train=True, download=True, transform=transforms.ToTensor()), batch_size=32)

    @pl.data_loader
    def test_dataloader(self):
        # OPTIONAL
        # can also return a list of test dataloaders
        return DataLoader(MNIST(os.getcwd(), train=False, download=True, transform=transforms.ToTensor()), batch_size=32)

if __name__ == '__main__':
    from pytorch_lightning import Trainer

    trainer = Trainer(
        gpus=[2, 3],
        distributed_backend='dp',
    )

    model = CoolModel()
    trainer.fit(model)

Expected behavior
There are no scalars involved in the forward pass, so the warning does not make sense and should not be shown.
Desktop (please complete the following information):

OS: Ubuntu 18.04
Version: the latest pip install

I'm using the newest pip version <IP_ADDRESS> and I'm still getting this warning (<IP_ADDRESS> also), do I have to toggle it somehow?

It happens also to me.

update  to 0.7.5

I'm currently on cutting edge.
When manually adding. Unsqueeze to the loss it vanishes of course
בתאריך שבת, 2 במאי 2020, 16:17, מאת William Falcon <
<EMAIL_ADDRESS>
update to 0.7.5
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/PyTorchLightning/pytorch-lightning/issues/249#issuecomment-622952331,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHM4A7ZFW2HGIDS7VB3NUULRPQMPFANCNFSM4I2MTP7Q
.

weird, can you post a colab with a minimal example? we added auto squeezing in dp for this. (i assume this is dp).
but on a much greater note, use DDP instead of DP as DP use is discouraged in general (by pytorch and us)


Removed Trial account screenshot

Please resolve conflict. Always do a fetch+pull before pushing anything to production to prevent conflicts.

Hi @MichaelCzcz ,
I have done as you suggested. Please check

There are still conflicts:

Please use the "Resolve conflicts" button to use the web editor.

Hi @MichaelCzcz ,
I resolved the conflict


RLOS: Observer Points parameter does not handle coordinates entered in Long Lat order (ArcMap)

The Coordinate Conversion tool was modified to allow users to enter in coordinates in both Lat Long and Long Lat order. All other tools that allow user to type in coordinates should allow the same.
Tool accepts coordinates to be entered in Lat Long...

Tool does not accept coordinates to be entered in Long Lat...

Addressed in #264 PR

I am not able to verify this due to this issue:
https://github.com/Esri/visibility-addin-dotnet/issues/265

Will be able to verify now since issue was fixed in this PR.

This is working correctly in ArcMap 10.3.1 MT 3 and Visibility add-in stand-alone build 119 (6/13/18).

Updating Visibility June 18 Release doc.

verified this works
verified the bullet has been added to what's new about supporting long/lat for input parameters


Different methods of authentication

Many companies have LDAP authentication protocol, so it would be very nice to have different authentication settings in drone like github does.

 Our Stash instance uses LDAP for authentication. From what I've read, it sounds like I would create a user / password pair in LDAP, make that a super-user in Stash and then connect using that?

@ahammond did you figure out how to hook up drone.io to bitbucket in house? Did you have to do what you said, create an ldap super user ?

I ended up building with Jenkins against a github repo. We are slowly
pushing people out of the legacy stash server and onto github.
On Tue, Apr 18, 2017 at 5:54 PM, JohnTheodore<EMAIL_ADDRESS>wrote:

@ahammond https://github.com/ahammond did you figure out how to hook up
drone.io to bitbucket in house? Did you have to do what you said, create
an ldap super user ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/drone/drone/issues/169#issuecomment-295029648, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAbNRKk1BjgZr1KfiINBM-MsW6U_Y6Piks5rxVtBgaJpZM4BnouN
.

did you figure out how to hook up drone.io to bitbucket in house? Did you have to do what you said, create an ldap super user ?

stash does not require a super user account to login to the web interface. You can login with your regular stash account using oauth. Perhaps @josmo can clear this up

@JohnTheodore,  @bradrydzewski is correct. You don't need a single super user to log in. You should be able to login as any normal user.  You do need need a user that has git clone access to the repos (it's configuration required settings in Drone, but should be seamless to users) since stash doesn't support oauth clone access :( Lastly to enabled builds the user doing the first time enable of the repo does need to have access to set hooks on the repo.

@josmo I'm cross-posting from gitter (since this is indexed by google and I don't think gitter is).
I want to use drone.io's bitbucketserver for my entire project (there are dozens of repos in there, it's basically my business unit). I will talk to the bitbucketserver admin tomorrow about what's possible and check the admin UI (since I'm not a superadmin).
I think I can make a service account in bitbucket server that can read all my repos in my project through https? Though for the oauth2 application, my bitbucketserver admin turned off oauth2 for individual users. Can oauth2 stay off in general, but still be enable for a drone application (or the service user) that is scoped to only my project (without write permissions to modify git commits)? How granular are the oauth2 application settings for bitbucket server, do you know what the minimal settings are for it to work with your remote bitbucketserver setup?
any help/advice is appreciated.

@JohnTheodore I'll respond in gitter as well but to keep the history on it.
I don't believe Bitbucket server supports oauth2 I think it's only oauth1.  Right now the drone implementation is based on oauth1 so all you should need to do is add an application link and then anyone that has bitbucket server should be able to log in. I'm not familiar with disabling oauth for individual users so YMMV on that one.  You do need to setup a generic service account to be able to clone repos but that's not related to user auth just cloning for builds.

@josmo ok, oauth1 seems fine. Are there any permissions around oauth1? or is it just 'an oauth1 application' that has all the permissions or unknown permissions?

It's just an oauth1 application and the permissions will be those of the user that logs in.  A user enabling the repo needs to have hook creation access (I think its the admin role for a repo).

yea, that seems bad. I wish the oauth1 application only had permission to read repos and update the build pass/failed. it seems like bad security practice to give drone permission to write changes to the repo, when it's not needed. least privileges required so to speak. This is a limitation of bitbucketserver though, not the drone implementation.

I'm not sure I'm following. I do dislike the service account for cloning. However the auth is based on the access the user has. I guess it could be limited by the user on the stash side but can I'm not sure what that would really buy you security wise. Anyways let me know if you need more help. I know the integration is pretty solid and works well, minus some documentation and the fact that oauth1 makes some things less simple.

So you can give a user 'admin', 'write' or 'read' permission to a repo. If a user has admin privileges to a project or repo (or even just write), the drone bitbucketserver oauth1 token also has those admin privileges. so in theory drone could write changes to the repo, or delete a repo (although I know there is no code for this in the remote for bitbucketserver on drone). It would be nice if you could give the least privileges necessary for the user's oauth1 token, so it can only read and update the build pass/fail status

Well sadly that's a limitation of stash.

yep, agreed it's a limitation of stash(bitbucketserver).
It seems like Atlassian hasn't put nearly as much development effort into bitbucketserver as they have into bitbucket cloud. and based on the release notes, there is probably only one developer left doing stuff to it occasionally (a small change every couple months).


South Africa rollover rules regarding holidays falling on Sundays don't appear to be correct

#392 changed South African holiday calculations to add observed holidays when the existing holiday falls on a Sunday, per:
The Public Holidays Act (Act No 36 of 1994) determines that whenever any public holiday falls on a Sunday, the Monday following on it shall be a public holiday.
However it doesn't just check if it can add it to the following Monday but continues checking for an "open" day until it can place it, leading to it incorrectly marking 27 December 2022 as a public holiday:
In [80]: list(sorted(holidays.ZA(years=2022).items()))
Out[80]:
[(datetime.date(2022, 1, 1), "New Year's Day"),
 (datetime.date(2022, 3, 21), 'Human Rights Day'),
 (datetime.date(2022, 4, 15), 'Good Friday'),
 (datetime.date(2022, 4, 18), 'Family Day'),
 (datetime.date(2022, 4, 27), 'Freedom Day'),
 (datetime.date(2022, 5, 1), "Workers' Day"),
 (datetime.date(2022, 5, 2), "Workers' Day (Observed)"),
 (datetime.date(2022, 6, 16), 'Youth Day'),
 (datetime.date(2022, 8, 9), "National Women's Day"),
 (datetime.date(2022, 9, 24), 'Heritage Day'),
 (datetime.date(2022, 12, 16), 'Day of Reconciliation'),
 (datetime.date(2022, 12, 25), 'Christmas Day'),
 (datetime.date(2022, 12, 26), 'Day of Goodwill'),
 (datetime.date(2022, 12, 27), 'Christmas Day (Observed)')]```

@pietervdw115 since you submitted the original change just want to check if there's something I'm missing here?

Issue fixed and released. Closing, thx!


Cache covariance matrix decomposition in frozen multivariate_normal

Currently, it seems that a frozen multivariate_normal distribution unnecessarily re-computes the root decomposition (and other properties such as evals for the logpdf) of the covariance matrix for each operation. For instance, when sampling scipy just calls the sampling of the underlying distribution with the full covariance matrix: https://github.com/scipy/scipy/blob/adc4f4f7bab120ccfab9383aba272954a0a12fb0/scipy/stats/_multivariate.py#L762
This is super wasteful, as most of the computation is in fact in computing this decomposition.
What should happen instead is that the frozen object should also have a factor or L attribute s.t. L @ L.T = cov, and then it would compute that only once (upon the first iteratio), and in later steps pass that into the sampling method instead of cov, avoiding a bunch of unnecessary compute.
Torch's mvn does this (though it doesn't allow singular matrices at this point): https://github.com/pytorch/pytorch/blob/master/torch/distributions/multivariate_normal.py#L146-L15

The rvs method indeed requires recomputing the square root of the covariance.
One approach to fix this could be adding an extra attribute to _PSD, say L as: self.L = np.multiply(u, np.sqrt(s)), where s, u is the eigendecomposition of the covariance matrix.
Then, the square root of the covariance is available in multivariate_normal_frozen as self.cov_info.L, and can be employed in rvs.
I think only rvs needs modifying. As far as I can tell, logpdf uses self.U, which is a square root of the precision matrix.
Happy to work on this if no one else is.

Might be nice to have A benchmark given that (AFAICS) this is a (very useful) speed up. Might be a good place to start.
Here is a good read if you’re not familiar with benchmarks in scipy
http://scipy.github.io/devdocs/dev/contributor/benchmarking.html
Sincerely,
-Lucas Roberts

On May 20, 2020, at 3:27 PM, siddhantwahal<EMAIL_ADDRESS>wrote:

The rvs method indeed requires recomputing the square root of the covariance.
One approach to fix this could be adding an extra attribute to _PSD, say L as: self.L = np.multiply(u, np.sqrt(s)), where s, u is the eigendecomposition of the covariance matrix.
Then, the square root of the covariance is available in multivariate_normal_frozen as self.cov_info.L, and can be employed in rvs.
I think only rvs needs modifying. As far as I can tell, logpdf uses self.U, which is a square root of the precision matrix.
Happy to work on this if no one else is.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

I think this has been addressed by the Covariance class. Do you agree @tirthasheshpatel @tupui? Shall we close this one?

@mdhaber Yup, Covariance should address the issue. Feel free to close the issue!
@Balandat FYI: Now, we can specify the covariance using the factory methods provided by the stats.Covariance class! Here's an example:
import numpy as np
from scipy import stats

cov = np.eye(3)
mu = np.zeros(3)

# only compute the eigen-decomposition once
w, v = np.linalg.eigh(cov)
cov_object = stats.Covariance.from_eigendecomposition((w, v))

# use the cov_object instead
dist = stats.multivariate_normal(mu, cov_object)
dist.pdf([0, 0, 0])

# also, notice that the covariance object is diagonal
# we can use the special property to make computation
# of pdf, ... more efficient
cov_object = stats.Covariance.from_diagonal(cov)
stats.multivariate_normal.pdf([0, 0, 0], mu, cov_object)  # uses the special property of the
                                                          # covariance matrix to compute pdf
                                                          # more efficiently.

Feel free to refer to the devdocs here for more examples.

Thanks! Let me use the opportunity to make a shameless plug for a related (PyTorch) project for structured linear algebra that can also encode various structured covariance matrices: https://github.com/cornellius-gp/linear_operator


Not working with chrome mobile

Hello, you have built a fantastic little script.. well done.
I have found though, This does not work with chrome mobile which is a must. I have tested it with chrome desktop and works great.
Thanks again.
Ben

Please see http://caniuse.com/#search=Web Notifications
Chrome for Android only supports push notifications, which is a different API.


Cannot update forum settings

[16-09-2018 12:53] ProxyMapper.GetOrbType: Failed to get OrbType for name 'Frigid Fossil'.
[16-09-2018 12:53] ProxyMapper.GetOrbType: Failed to get OrbType for name 'Primitive Chaotic Resonator'.
[16-09-2018 12:53] ProxyMapper.GetOrbType: Failed to get OrbType for name 'Primitive Chaotic Resonator'.
[16-09-2018 12:53] ProxyMapper.GetOrbType: Failed to get OrbType for name 'Potent Chaotic Resonator'.
[16-09-2018 12:53] ProxyMapper.GetOrbType: Failed to get OrbType for name 'Primitive Alchemical Resonator'.
[16-09-2018 12:53] ProxyMapper.GetOrbType: Failed to get OrbType for name 'Primitive Chaotic Resonator'.
[16-09-2018 12:53] ProxyMapper.GetOrbType: Failed to get OrbType for name 'Primitive Alchemical Resonator'.
[16-09-2018 13:04] Unable to save shop settings: System.ArgumentNullException: Wartość nie może być zerowa.
Nazwa parametru: value
w System.Xml.Linq.XAttribute..ctor(XName name, Object value)
w POEApi.Model.Settings.SaveShopSettings() w C:\Users\Maciek\Desktop\Procurement-master\POEApi.Model\Settings.cs:wiersz 261

Hey, @bombek92, sorry for not updating this bug for a while.  I created a pull request that I think will fix this bug.  In the meantime, if I understand the issue you're encountering, you should be able to work around this issue by making sure you've modified the "Thread Id" and "Thread Title" text boxes in the Trade Settings tab for each league.  You could even enter garbage text and then delete it (so Procurement thinks the text box is modified) and then click the Save Thread Settings button.
Alternatively, you could manually edit the Settings.xml file with initial values for each league's ShopSettings, but that's more complicated, so let's try the above fix first.
If this doesn't solve the issue, let me know.


03-reactive-statements console.log()

console.log(the count is ${count});
It does not work properly on localhost.
It should be:
$: count, console.log(the count is ${count});
PS. Sorry for my poor English.

Both work, so $: console.log(..) is the more concise solution. I'm not sure what you mean by "does not work properly on localhost"?

Thanks for your relpy. I misunderstood.


Sortable js template

Coverage decreased (-0.05%) to 65.098% when pulling 39c500606ee4c9c7895df8fae42ab0177e0c4923 on sortable-js-template into aa655ffa9439222ef62fd382cddb161e58589fcc on master.


ReferenceError: __dirname is not defined in ES module scope

Prerequisites

[X] I have written a descriptive issue title
[X] I have searched existing issues to ensure the bug has not already been reported

Fastify version
4.27.0
Plugin version
3.0.0
Node.js version
22.2.0
Operating system
macOS
Operating system version (i.e. 20.04, 11.3, 10)
14.5
Description
I'm trying to run the plugin with esbuild. At first, I had issues with the static folder. So I added a copy. My package.json looks like this:
{
  "scripts": {
    "build": "esbuild && cp -R ./node_modules/@fastify/swagger-ui/static dist",
    "start": "node dist/main.mjs"
  }
}

But I get this error
[node] file:///Users/me/service/dist/main.mjs:37584
[node]       const logoContent = await fsPromises.readFile(path.join(__dirname, "./static/logo.svg"));
[node]                                                               ^
[node]
[node] ReferenceError: __dirname is not defined in ES module scope
[node]     at fastifySwaggerUi2 (file:///Users/me/service/dist/main.mjs:37584:63)
[node]     at Plugin.exec (file:///Users/me/service/dist/main.mjs:38008:32)
[node]     at Boot._loadPlugin (file:///Users/me/service/dist/main.mjs:38518:14)
[node]     at process.processTicksAndRejections (node:internal/process/task_queues:82:21)

Do you plan to handle ESM? Is there any workaround?
From what I read, this is just
import { fileURLToPath } from 'url';
import { dirname } from 'path';

const __filename = fileURLToPath(import.meta.url);
const __dirname = dirname(__filename);

This looks simple but I might not have the whole picture.
Link to code that reproduces the bug
No response
Expected Behavior
No response

That sounds like a problem with your tooling. The source file in this project is not an ESM style file: https://github.com/fastify/fastify-swagger-ui/blob/758abaed570cce1a42e12e4b5cfd86b5e94d2efc/index.js#L1-L35
For what it's worth, on Node.js 20 and greater:
const dirname = import.meta.dirname

Thank you but that was my initial question: any plan to handle ESM or is it intended to stay in CommonJS?
Since fastify is ESM-ready, I was expecting this to be as well.

This is 100% esm ready. You can consume this library in any ESM project.
However, you are transpiling this code to esm, using esbuild, which does not support transpiling commonjs into esm.
My understanding is that rollup does this correctly. I recommend you to change your bundling strategy.


Create a client using only an AccessToken

Addresses https://github.com/rinukkusu/spotify-dart/issues/70 by adding a new constructor that only requires a valid access token.

Sorry made it on the wrong repo. I was testing out null safety.


Receiving SSL error

Receiving SSL error when connecting to Splunk instance over port 80.  BigIP Listens on port 80 forwards to splunk instance on 8089.  I am able to connect if I use urllib, urllib2.
Traceback (most recent call last):
File "info.py", line 32, in
service = client.connect(**opts.kwargs)
File "../splunklib/client.py", line 305, in connect
return Service(**kwargs).login()
File "../splunklib/binding.py", line 762, in login
password=self.password)
File "../splunklib/binding.py", line 1091, in post
return self.request(url, message)
File "../splunklib/binding.py", line 1108, in request
response = self.handler(url, message, **kwargs)
File "../splunklib/binding.py", line 1230, in request
connection.request(method, path, body, head)
File "/usr/local/Cellar/python/2.7.8_2/Frameworks/Python.framework/Versions/2.7/lib/python2.7/httplib.py", line 995, in request
self._send_request(method, url, body, headers)
File "/usr/local/Cellar/python/2.7.8_2/Frameworks/Python.framework/Versions/2.7/lib/python2.7/httplib.py", line 1029, in _send_request
self.endheaders(body)
File "/usr/local/Cellar/python/2.7.8_2/Frameworks/Python.framework/Versions/2.7/lib/python2.7/httplib.py", line 991, in endheaders
self._send_output(message_body)
File "/usr/local/Cellar/python/2.7.8_2/Frameworks/Python.framework/Versions/2.7/lib/python2.7/httplib.py", line 844, in _send_output
self.send(msg)
File "/usr/local/Cellar/python/2.7.8_2/Frameworks/Python.framework/Versions/2.7/lib/python2.7/httplib.py", line 806, in send
self.connect()
File "/usr/local/Cellar/python/2.7.8_2/Frameworks/Python.framework/Versions/2.7/lib/python2.7/httplib.py", line 1198, in connect
self.sock = ssl.wrap_socket(sock, self.key_file, self.cert_file)
File "/usr/local/Cellar/python/2.7.8_2/Frameworks/Python.framework/Versions/2.7/lib/python2.7/ssl.py", line 392, in wrap_socket
ciphers=ciphers)
File "/usr/local/Cellar/python/2.7.8_2/Frameworks/Python.framework/Versions/2.7/lib/python2.7/ssl.py", line 148, in init
self.do_handshake()
File "/usr/local/Cellar/python/2.7.8_2/Frameworks/Python.framework/Versions/2.7/lib/python2.7/ssl.py", line 310, in do_handshake
self._sslobj.do_handshake()
ssl.SSLError: [Errno 1] _ssl.c:510: error:140770FC:SSL routines:SSL23_GET_SERVER_HELLO:unknown protocol

Hi @httpstergeek,
Thanks for the bug report, we will look into this.

Hi @shakeelmohamed,
I am closing this bug.  After further investigation I believe this error is caused by a misconfigured virtual server.

Thanks @httpstergeek. On a diff note, love your github avatar :-)
On Fri, Apr 24, 2015 at 11:08 AM httpstergeek<EMAIL_ADDRESS>wrote:

Hi @shakeelmohamed https://github.com/shakeelmohamed,
I am closing this bug. After further investigation I believe this error is
caused by a misconfigured virtual server.
—
Reply to this email directly or view it on GitHub
https://github.com/splunk/splunk-sdk-python/issues/98#issuecomment-96017743
.


Align code sections to left in README.md

Align code sections to left in README.md.

Hi @hwhsu1231 - thanks for contributing, again :+1:
Yes, this looks like a good improvement to the readability (and ability to copy and paste from the example snippets). I'll take one more detailed look and then likely merge it in the weekend.


[TableCard] Threshold labels should be customizable or hidden

Describe the bug
Threshold labels should have customizable strings and the ability to hide.
Currently there is no way to do either.
To Reproduce
Steps to reproduce the behavior:

Open a table with threshold labels in a development environment
Change the label property to an empty string or other custom string
See that only the threshold column header changes

Expected behavior
Expect to either change the label or hide it - this is the label within the rows and not the column header label
Environment/versions:

OS: MacOS
Browser: chrome

Additional context
Current:

:tada: This issue has been resolved in version 2.58.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


This app can't run on your PC.

Hi, so I'm trying to use phantom to make my server show up as a lan so people watching my stream can connect to it, but when I try to do it I get this error.
https://imgur.com/a/tIwBWkP

Hey @trixxyz, a bit late but make sure you're using the correct 32-bit or 64-bit version for your PC. If you check the latest version on the Releases page here, you'll find both 32/64-bit downloads available.


d

What was wrong?
Issue #
How was it fixed?
Summary of approach.
To-Do

[ ] Clean up commit history

[ ] Add entry to the release notes

Cute Animal Picture

This is now ready to be taken apart into a few separate pull requests.


Combo Points element changes

This PR contains changes to the cpoints element.
These commits are compatible with both live and beta clients, this is not a PR specific to Legion, just something I found while iterating over all the elements while creating the Legion updates.

In Legion and post-#266, this element is not really needed any more and could be deprecated.

Hi,
i have changed both files elements/tags.lua and cpoints.lua -> the same result. No combo update on the target. Only deselect and targeting again show the actual combos, but no update. Do i make something wrong?
Thx and greetings.

Well, sorry, my fault. Combos are shown on the playerframe new ^^

I would highly suggest you try out classicons instead, they cover combo points as well.

@p3lim hi, thx for the answer, i will try it, but it works fine now for me.

It's Legion now, why dun we just delete CPoints for good? People are confused and try to do things wrong way.

Hi @ls- , yes, everything is fine, thx. It was a mistake by me, i thought the cp were on the target, but they are now on the playerframe. You can close this ticket.
Greetings

@daneras, you aren't the only one who asked about CPoints 😁


Add a shortcut to Travis build (bypass Check tab)

The problem
Before GitHub Checks, we had the possibility to directly go on Travis website by clicking on details here:

Now we should make 3 or 4 clicks and it's really boring:

Suggestion
The big issue here is to retrieve Travis build URL. I don't know if it's possible to do through GitHub v3/4 API, but we can do:

For each Travis Checks:

Extract a.status-actions[href] value
Make a XHR request
Parse the response body and find the <a> containg View more details on Travis CI, and extract href value

I have three suggestion at the moment.
Visual update
I have three ideas on how to exploit Travis url.
Completely replace Details link, easy and clean:

Make Travis CI - ... a link, less easy, would break CSS:

Or create a new link and put it aside Details link, hard and horrible to see, would break some CSS too:

Thanks

Can you add a link to a repo with checks? I haven't seen any yet

Ah yes, added in OP.

👍 I haven't seen one good thing from the introduction of Checks on GitHub. The URLs are strange, you cannot get back to the PR easily from some of the check pages. It's extremely frustrating you cannot get quickly to a failing build.
Until recently, we had both and we just needed to remember which name of the CI status was the old one without the Checks so we can click on it to go to the build URL. Now, when this is removed, I'm only using the CLI or nervously clicking 2-3 times to go to the build.
If it's quick to do a XHR request on hover/click, this could be become even better than before as we could extract the specific jobs from the Checks HTML (referencing Travis CI with one build and multiple jobs). However, the problem is the HTML of the Checks page is different for each service, but I guess supporting Travis CI would cover the majority.


delay problem

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.

What problem did you encounter?
basically ever since i downloaded bloxstrap i get this weird bug that makes everything i do on my computer have an almost 30 seconds delay but its worse when i try to play roblox i tried changing my settings or even remove them all but nothing seems to be working and roblox is basically unplayable with this and i now have alot of issues with m everyday quality of life with my computer

you clearly have other issues on your computer

been happening ever since i downloaded bloxstrap everything was fine beforehand
Doesn't change the fact you still clearly have other issues with your computer lol, Bloxstrap wouldn't really affect it.

ever since i downloaded bloxstrap i get this weird bug that makes everything i do on my computer have an almost 30 seconds delay

I don't think this has any correlation. This doesn't make any sense and there isn't anything I can do or am supposed to do about this.


Add default virtual destructor for Parser class.

This fixed clang [-Wdelete-non-abstract-non-virtual-dtor] warning:
delete called on non-final 'ManifestParser' that has virtual
functions but non-virtual destructor.
for subparser_.reset(new ManifestParser(state_, file_reader_, options_));.

Thanks, I think it would make more sense to add a virtual (default) destructor to the Parser base class instead. Can you do that?

Thanks, this looks good to me. @jhasse , wdyt?

thx :)