convert tf to caffe error

Platform (like ubuntu 16.04/win10):ubuntu 16.04
Python version:2.7
Source framework with version (like Tensorflow 1.4.1 with GPU):tensorflow 1.8.0 cpu
Destination framework with version (like CNTK 2.3 with GPU):caffe
Pre-trained model path (webpath or webdisk path):https://github.com/xubin19939/tftocaffe
Running scripts:
$ mmconvert -sf tensorflow -in model199000model.ckpt.meta -iw model199000model.c
kpt --dstNodeName Softmax -df caffe -om tf_resnet
Parse file [model199000model.ckpt.meta] with binary format successfully.
Tensorflow model file [model199000model.ckpt.meta] loaded successfully.
Tensorflow checkpoint file [model199000model.ckpt] loaded successfully. [19] variables loaded.
2019-01-07 15:06:55.798600: I tensorflow/tools/graph_transforms/transform_graph.cc:264] Applying fold_constants
2019-01-07 15:06:55.823933: I tensorflow/core/platform/cpu_feature_guard.cc:140] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX2 FMA
Tensorflow has not supported operator [NoOp] with name [Dropout/cond/dropout/Shape/Switch:1].
Traceback (most recent call last):
File "/usr/local/bin/mmconvert", line 11, in
load_entry_point('mmdnn==0.2.3', 'console_scripts', 'mmconvert')()
File "/usr/local/lib/python2.7/dist-packages/mmdnn/conversion/_script/convert.py", line 102, in _main
ret = convertToIR._convert(ir_args)
File "/usr/local/lib/python2.7/dist-packages/mmdnn/conversion/_script/convertToIR.py", line 115, in _convert
parser.run(args.dstPath)
File "/usr/local/lib/python2.7/dist-packages/mmdnn/conversion/common/DataStructure/parser.py", line 22, in run
self.gen_IR()
File "/usr/local/lib/python2.7/dist-packages/mmdnn/conversion/tensorflow/tensorflow_parser.py", line 359, in gen_IR
func(current_node)
File "/usr/local/lib/python2.7/dist-packages/mmdnn/conversion/tensorflow/tensorflow_parser.py", line 436, in rename_Placeholder
IR_node.attr['shape'].shape.dim[0].size = -1
IndexError: list index out of range
below is my definition of placeholder:
height=64;
width=64;
inputs = tf.placeholder(tf.float32, shape=[None, height, width, 3], name='inputs')
is there something wrong with placeholder?how should I fix it?
thanks o lot for help
and I have tried on gpu version tf1.8.0 ,error remains

Hi@xubin19939,I am also same the issue.can you share the solution of th problem ?


Bring up to date with Nimble 4.0.0

Nimble 4.0.0 supports Swift 2.2 in XCode 7.3 and in order for Guanaco to
carry on working we need to drop FullMatcherFunc as it was removed in
the PR referenced below:
https://github.com/Quick/Nimble/pull/253

@modocache I might fork this project over to the SwiftGit2 as you seem to have an awful lot on your plate. OK with that?

Gah!! Sorry to have left this open for so long. Yes, that sounds great, or I can give you commit access to this repository--whichever works best for you.

+1


Fix the build

We broke the master build merging things earlier today. This fixes the issue.

Closing, this made it in as #112


Requires administer permission to test connection with slack.

[x] Jenkins version: 2.204
[x] Plugin version latest at the moment
[x] OS jenkins: lilnux, client: win7x64, chrome

Description

configure post-build notifications
click "Test Connection"
expand "ERROR" link:

Not sure how you're hitting this, I was unable to reproduce it on 2.204
Can you please specify the actual plugin version and not latest?
The permissions I used to test it was:

What type of job are you configuring?

Unable to reproduce, if you can provide more details please re-open with them


Append/Remove specific entries from secure variables

Current working is any update of secure variable completely replaces the previous value. For example, if the KEK currently contains the ESL's X, Y and Z then using secvarctl  or writing to the sysfs locations to update the KEK with a new auth file containing the ESL's A and B will set the updated KEK to just contain A and B . So, appending and removing specific entries is currently not supported.
This is the request for feature/enhancement to support append/remove of specific entries from the secure variables.
Thank you.

Hey Nageswara, thanks for your patience! I finally got around to implementing half of your request. I will get around to secvarctl remove soon

Sorry, forgot to update this. Pull request #23 should have fully implemented the requested features (insert and remove and test cases). Just waiting on review.


selenium.common.exceptions.SessionNotCreatedException: Message: Expected browser binary location, but unable to find binary in default location

I get this error
selenium.common.exceptions.SessionNotCreatedException: Message: Expected browser binary location, but unable to find binary in default location, no 'moz:firefoxOptions.binary' capability provided, and no binary flag set on the command line

any update on this issue? got the same issue.

Hi, same issue:
The rendering of the video is perfect but I am stuck on the upload.
WIN10
I am getting:
selenium.common.exceptions.SessionNotCreatedException: Message: Expected browser binary location, but unable to find binary in default location, no 'moz:firefoxOptions.binary' capability provided, and no binary flag set on the command line
Everything is installed well. The logs end with Video is done rendering! so that is not of much help but I tried whatever I could. I added the firefox to PATH (folder and exe). I have retried everything again, admin, different config, headless=False, etc. but nothing works. Everything at the latest version except for Python.
Thank you in advance if you have any pointers.


Updated qrc files to remove spaces from paths

Modified all C++ and QML samples to no longer use spaces in their qrc paths.
This PR needs to be merged with corresponding pull request in the qt-sdk repo.
https://devtopia.esri.com/runtime/qt-sdk/pull/3194
@JamesMBallard please review/merge.

:+1:


Sign event data would be more helpful with additional properties

When receiving a 'sign' event, the response object has the signatureId which cannot be used for anything useful.
Please include the signature_request_id or the entire signature request as a response object.

ftfy.

I'll run this up the chain to try get it slotted in for a later sprint, but at the moment this is likely pretty low priority. I don't disagree, more properties in the event data would be helpful :) There are some properties though that the frontend just doesn't have access to without some retrofitting, which is why I suspect the data we send over right now is pretty limited.


Fullcalendar 3.1.0

@gr8bit Do you have time to merge this? 🙂

Whoooooa sorry David, forgot about it.

No worries 🙂 Wasn't sure if you had seen it or not !🙂

Pushed. :)


Inconsistent output compared with other JS engines

Hello,
The following code behaves strangely (inconsistent with other engines).
v1 = new (Float64Array)();
v2 = {
    valueOf : function () {
        v3.y = "bar";
        return 42; }
};
v3 = v1;
v3[0] = v2;
print(JSON.stringify(v1));

In Edge, output is
{"y":"bar"}
However, in V8 and Firefox, output is
{}
BT group
2018.6.27

In this case Chakra is spec-compliant (along with JSC).
See https://tc39.github.io/ecma262/#sec-integerindexedelementset
ToNumber should be called on the assignment RHS before the length check is performed on the TypedArray. It appears that V8 and SM are incorrectly performing ToNumber after the length check.
Test case:
(new Float64Array())[0] = { valueOf() { print('ToNumber'); return 42; } };

Output:
#### ch
ToNumber

#### jsc
ToNumber

#### v8

#### sm

#### xs
ToNumber


Create subscriptionGroup through the command line and set the maximum number of retries for retryMaxTimes, which does not take effect：RocketMQ4.9.4

The issue tracker is used for bug reporting purposes ONLY whereas feature request needs to follow the RIP process. To avoid unnecessary duplication, please check whether there is a previous issue before filing a new one.
It is recommended to start a discussion thread in the mailing lists or github discussions in cases of discussing your deployment plan, API clarification, and other non-bug-reporting issues.
We welcome any friendly suggestions, bug fixes, collaboration, and other improvements.
Please ensure that your bug report is clear and self-contained. Otherwise, it would take additional rounds of communication, thus more time, to understand the problem itself.
Generally, fixing an issue goes through the following steps:

Understand the issue reported;
Reproduce the unexpected behavior locally;
Perform root cause analysis to identify the underlying problem;
Create test cases to cover the identified problem;
Work out a solution to rectify the behavior and make the newly created test cases pass;
Make a pull request and go through peer review;

As a result, it would be very helpful yet challenging if you could provide an isolated project reproducing your reported issue. Anyway, please ensure your issue report is informative enough for the community to pick up. At a minimum, include the following hints:
BUG REPORT

Please describe the issue you observed:

Create subscriptionGroup through the command line and set the maximum number of retries for retryMaxTimes, which does not take effect

What did you do (The steps to reproduce)?
create new group
./mqadmin  updateSubGroup -b localhost:10911 -n localhost:9876 -r 1 -w 1 -s true -i 0 -c DefaultCluster -g test100

//consumer failed
DefaultMQPushConsumer consumer = new DefaultMQPushConsumer("group003",getAcl(),new AllocateMessageQueueAveragely());
consumer.setNamesrvAddr("localhost:9876");
consumer.subscribe("topic003","*");
consumer.registerMessageListener(new MessageListenerConcurrently() {
@Override
public ConsumeConcurrentlyStatus consumeMessage(List list,
ConsumeConcurrentlyContext consumeConcurrentlyContext) {
System.out.println("接收到的消息内容为：+"+ list);
return ConsumeConcurrentlyStatus.RECONSUME_LATER;
}
});
consumer.start();

What is expected to see?

Retry consumption 2 times

What did you see instead?
Retry consumption 16 times

Please tell us about your environment:
Rocketmq 4.9.4

Other information (e.g. detailed explanation, logs, related issues, suggestions on how to fix, etc):

FEATURE REQUEST

Please describe the feature you are requesting.

Provide any additional detail on your proposed use case for this feature.

Indicate the importance of this issue to you (blocker, must-have, should-have, nice-to-have). Are you currently using any workarounds to address this issue?

If there are some sub-tasks involved, use -[] for each sub-task and create a corresponding issue to map to the sub-task:

sub-task1-issue-number: sub-task1 description here,
sub-task2-issue-number: sub-task2 description here,
...

After looking at the source code, it should be judged wrong here

Let's move to GitHub Discussion, see #6153.


Rainbow(Chroma) fps option

Hi,
I added an option for rainbow fps. With rainbow fps I mean, that the fps have a chroma effect, for example green to red to blue...
It isn't really a crazy thing but maybe some people want this little feature.


HDDS-10029. Improved logs for SCMDeletedBlockTransactionStatusManager

What changes were proposed in this pull request?
Improved logs for SCMDeletedBlockTransactionStatusManager.
SCMDeletedBlockTransactionStatusManager sometimes outputs some redundant WARN logs or unclear logs.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-10029
How was this patch tested?
Existing Test

@swamirishi can you please take a look?

Thanks @xichen01 for working on this.
These messages include the same information with different prefix label. E.g. SCM command ID appears as:

scmCmdId {}
SCM Command: {}
SCM Command ID: {}
ScmCommandId {}

which makes it difficult to filter the log for this information.
Similar problem with datanode ID.
I suggest standardizing it.

Standardized LOG format to SCM Command ID: {}.

@SaketaChalamchala would you like to take another look?

Thanks @xichen01 for the patch, @SaketaChalamchala, @sumitagrawl for the review.


openssl incorrect allowed maximum domain length

An error occurs when trying to get a CSR request for an SSL certificate:
ASN1_mbstring_ncopy:string too long:crypto/asn1/a_mbstr.c:109:maxsize=64

Domain name: the-very-long-domain-name-in-the-internet-for-generate-csr-strg.uz
The length of this domain, given the zone, is 66 characters, and without zone 63.
If you reduce the length of the domain by two characters, the error does not occur.
For example: the-very-long-domain-name-in-the-internet-for-generate-csr-st.uz
Please support longer domain names or consider the length without a zone.

That's an illegal domain name.  No single component can be more than 63 characters long.

But the component is exactly 63 characters long.

Ah, I miscounted (the period).  Okay, seems like a bug.

Not necessarily a bug.  How is that domain name used?  Is it perhaps used as the subject CN?  The subject CN has a limit of 64 characters...

(lesson to learn here is to give complete info, not just an error line)

There is domain кодекс-административных-правонарушении.рф.
whois xn-----7kcabalclcij6aabgatc7bwedcerywgfkh7ac1cxdzgoe.xn--p1ai
% By submitting a query to RIPN's Whois Service
% you agree to abide by the following terms of use:
% http://www.ripn.net/about/servpol.html#3.2 (in Russian)
% http://www.ripn.net/about/en/servpol.html#3.2 (in English).

domain:        XN-----7KCABALCLCIJ6AABGATC7BWEDCERYWGFKH7AC1CXDZGOE.XN--P1AI
nserver:       ns1.hosting.reg.ru.
nserver:       ns2.hosting.reg.ru.
state:         REGISTERED, DELEGATED, UNVERIFIED
person:        Private Person
registrar:     REGRU-RF
admin-contact: http://www.reg.ru/whois/admin_contact
created:       2020-04-09T11:11:20Z
paid-till:     2021-04-09T11:11:20Z
free-date:     2021-05-10
source:        TCI

Last updated on 2020-04-13T05:51:32Z

But we can't convert it's name and have same problem.
error:0D07A097:asn1 encoding routines:ASN1_mbstring_ncopy:string too long:crypto/asn1/a_mbstr.c:109:maxsize=64

What should we do to fix the problem? When the problem would be fixed globally?

The problem will never be fixed globally, not everyone on the Internet will update their software or systems.

@richsalz I don't understand your meaning.  The 61 bytes of punycode decode to 79 bytes of UTF-8, which is permitted by the spec (RFC 5890):
                                                                     Second, expansion of
   the A-label form to a U-label may produce strings that are much
   longer than the normal 63 octet DNS limit (potentially up to 252
   characters) due to the compression efficiency of the Punycode
   algorithm.  Such extended-length U-labels are valid from the
   standpoint of IDNA, but caution should be exercised as shorter limits
   may be imposed by some applications.

The bug report here seems to be that we should modify our tools to be better participants in the IDNA ecosystem and not be one of those applications that imposes shorter limits.

RFC 5280 is irrelevant here.  RFC 3280 is.  At the end of Appendix A, you can find a slew of sizes, among others this:
ub-common-name INTEGER ::= 64

That's why I asked how the domain name was being used, a couple of years ago

I feel like we're talking past each other -- I said RFC 5890, not the one we usually talk about that starts with 5, ends with 0, and has an 8 in the middle.  You can choose between the A-label (punycode) and the U-label (native UTF-8) for representing DNS labels.
If we then consider certificate encoding, RFC 5280 becomes relevant (n.b. that 5280 obsoletes 3280), and a very quick read suggests that both PrintableString and UTF8String are admissible.  So it would be okay to put the A-label in the certificate field, which in this latest example does comply with the 64-byte length limitation, and then ask for some local processing (e.g., display) to convert to the U-label.  Because of that conversion step, some buffers would need to be larger than 64 bytes.

And, getting back to the main point of my original comment above, even if OpenSSL does something, the whole rest of the Internet might not and some parts never will.

5280 was a mistype on my part.
Do we do internal conversions between A-label and U-label? I do recall a punycode decoder and encoder, but have forgotten the details. Previously, I believe openssl let you enter whichever form you wished... it would make sense if this was still the case for commonName.

Do we do internal conversions between A-label and U-label?

#9456 introduces some, though I don't remember a direction.

Uhm, care to try again, @beldmit? That PR number led to a surprise...

Oops. Sorry, it was a hard day...
#9654

Marking as inactive, to be closed at the end of 3.4 dev barring further input


File Upload

I got it installed successfully and am trying to upload a .csv file by clicking on the browse button, but it's not getting uploaded as the data is not displayed on the left panel. Please help. I am using python 3.6. TIA

Error logs would be printed in the console. Can I get an error logs?

Hi, thanks for replying. Below is the console output after launching knotter:
Serving at port 8000 Server started at http://localhost:9000 [7250:7288:0213/172337.296268:ERROR:browser_gpu_channel_host_factory.cc(121)] Failed to launch GPU process. Created new window in existing browser session. <IP_ADDRESS> - - [13/Feb/2018 17:23:37] code 404, message File not found <IP_ADDRESS> - - [13/Feb/2018 17:23:37] "GET /bower_components/paper-listbox/paper-listbox.html HTTP/1.1" 404 - /usr/local/lib/python3.6/dist-packages/knotter/server.py:419: RuntimeWarning: coroutine 'WebSocketResponse.send_str' was never awaited   self.ws.send_str(json.dumps(content))
Please let me know if this is useful.

I reinstalled knotter, now it is showing the following output in the console:
Serving at port 8000 Server started at http://localhost:9000 [26792:26830:0214/023032.467238:ERROR:browser_gpu_channel_host_factory.cc(121)] Failed to launch GPU process.

Thanks a lot :), this is working like charm. And thanks a lot for this project.


Refactor query to get all exchange events of user

Now uses ethindex_db to get all events within one query instead of
one query per exchange

Codecov Report

Merging #503 into master will decrease coverage by 0.07%.
The diff coverage is 19.35%.

@@            Coverage Diff             @@
##           master     #503      +/-   ##
==========================================
- Coverage   61.83%   61.75%   -0.08%
==========================================
  Files          55       55
  Lines        4284     4286       +2
==========================================
- Hits         2649     2647       -2
- Misses       1635     1639       +4

Impacted Files
Coverage Δ

src/relay/ethindex_db.py
22.94% <12.00%> (-0.44%)
:arrow_down:

src/relay/relay.py
31.77% <50.00%> (-0.12%)
:arrow_down:

Continue to review full report at Codecov.

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


devin/fastlane/metadata/android/en-US/images/icon.png should be 512x512

512x512 is the standard image size for app store icons, e.g. _ devin/fastlane/metadata/android/en-US/images/icon.png_  The current one is 8192x8192, and is too large to be included in f-droid.org

Oops, I fixed this a while ago and never pushed it... :clown_face:


Added netlify reCaptcha div before submit for spam fight

Added netlify reCaptcha div before submit for spam fight

Hi @dwaiba I'm not sure this is needed in the default theme, as it's quite easy to add on a user-needs basis. Netlify already has filtering on forms, and I've got a honeypot field in the form.


d3 error

Hi,
I receive a d3 error:
Error: Invalid value for  attribute d="null"
when running c46 in the graph.root example.
Nevertheless, a graph is shown (should it be a polar graph? i see an euclidean plane), so maybe it is not a critical issue.

Hi,
This was because of TGraphPolargram class.
I add simple protection against it.
But as you could see TGraphPolar drawing not yet supported.
Thanks for reporting the problem.
Regards,
Sergey

Does the sqroot graph of the fillrandom.root file work for you with require.js? I receive a require error:
Uncaught ReferenceError: JSROOT is not defined

Yes, I see an error with require.js. Will investigate it....

When loading modules with require.js, no global variables like JSROOT or JSROOT.Math are defined.
Therefore I need to keep them locally to use in TF1::Eval expression.
Should work now.
Thanks for reporting it.

Great, thanks.


Cannot activate compression in production server

Hello,
In my developer computer, i can activate using :
app = express();
app.use(compression());
I can see in the http headers response:
Content-Encoding:gzip
But in my production server it does not activate. I have the exact same version in both computer and the code is exactly the same:
node 7.9.0
compression 1.6.2
express 4.15.2
The only diferrence is that the server is ubuntu server 16.04.02 LTs and my developer computer is Ubuntu desktop 16.10.
I have other appliction in a difference server and there i can also use the compression without a problem. I do not understand why in this case it does not work.
Can you help me? Thank you

I made this small app for testing:
var express = require('express'),
app = express(),
compression = require('compression');
app.use(compression());
app.get('/', function (pedido, resposta) {
resposta.status(202).send(pedido.headers);
});
app.listen(8080);
If i run in my local computer, show that my browser is sending to the server (both local server as remote server):
"accept-encoding":"gzip, deflate"
But the response is without gzip from local server and remote server. What am i doing wrong?

I have remove all node modules and even removed express. Then i only installed node e expressjs e compression. The problem remains.

If i cnahge the testing application to:
var compression = require('compression'),         express = require('express'),         app = express(); app.use(compression()); app.get('/', function (pedido, resposta) {     resposta.sendFile('p.html', {         root: __dirname + "/../p/d"     }); }); app.listen(8080);
then in my local computer the transfer uses gzip but in my production server keeps not using gzip.

I made some changes in the testing application:
var compression = require('compression'),
        express = require('express'),
        compressible = require("compressible"),
        app = express();
app.use(compression({filter: shouldCompress}));

function shouldCompress(req, res) {
    var type = res.get('Content-Type');
    console.log('Content-Type:' + type);
    if (type === undefined || !compressible(type)) {
        debug('%s not compressible', type);
        return false;
    }
    return true;
}

app.get('/', function (pedido, resposta) {
    resposta.header({"Content-Type": "text/plain", "Cache-Control": "public, max-age=0", "Last-Modified": "Fri, 07 Apr 2017 11:00:37 GMT"});
    resposta.send("olá");
});
app.get('/lixo', function (pedido, resposta) {
        resposta.sendFile('p.html', {
     root: __dirname + "/../p/d"
     });
});
app.listen(8080);

Now in my local computer the curl -i --compressed localhost:8080/lixo return the html file compressed. The curl -i --compressed localhost:8080/ returns olá uncompressed.

Hi @FgfdCBVCbnsebtgf the reason why curl -i --compressed localhost:8080/ is not compressed in your example is because the result is only 3 bytes, below the cut-off threshold of 1kb (https://github.com/expressjs/compression#threshold).
As for your other route, I don't have your p.html file to try out, but I created my own and it's compressed both locally and on a variety of production servers I tried it on. Something is definitely going on, and there isn't a lot of information provided to figure out what is different between our setups, as I tried
(1) A fresh, brand new install of Ubuntu 16.04.02
(2) An install of node 7.9.0 from nodejs.org
(3) compression 1.6.2 and express 4.15.2
(4) your exact code above
I then called it from another machine on the same network as the Ubuntu machine and then again from the other side of the globe (with the machine in Tokyo and the caller in London) and it was compressed every time.
Perhaps can you gather some packet captures of the response at various points in the flow to pinpoint what may be happening?

One difference: i use Ubuntu server 16.04.02 LTS
I installed the testing script here: http://<IP_ADDRESS>:8080/ (keeps not sending in gzip). Made a wireshark capture of my ethernet while making  the http request:
www.op3racional.eu/capture.pcap
Thank you.

Hi @FgfdCBVCbnsebtgf thanks for putting up that server. Both responses are not compressed, because as I said above the reason why curl -i --compressed localhost:8080/ is not compressed in your example is because the result is only 3 bytes, below the cut-off threshold of 1kb (https://github.com/expressjs/compression#threshold). It looks like your p.html file is also less than 1kb in size, which is why it's also not compressing from your server.
I hope that helps!

I confirm. The file was too small. But in my true project (https://sementes.gratis) the default html file is 1.4Kb and it gets no gzipped.


implement repack output straight to error dataset

Repack jobs stageout directly to merged for files larger than 2GB. If these files are larger than 12GB we do not want them to be included in the regular RAW output dataset and we also don't want them to be prompt reconstructed. Therefore implement an override in the JobAccountant that changes the DBS Primary Dataset for such files and also skips adding them to the filesets used for PromptReco.
And yes, it's a bit hacky, but it's the only way I could think of to make this work in a half-way sane way.
@ticoann @amaltaro please review

So this big file still need to uploaded to dbs with new dataset (-Error)?

Yeah, we don't want to completely drop it (users should still be able to find it), we just don't want it in the normal output dataset as the PhEDEx subscriptions are different.


kpt: 0.35.0 -> 0.36.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/kpt/versions.
meta.description for kpt is: "A toolkit to help you manage, manipulate, customize, and apply Kubernetes Resource configuration data files"
meta.homepage for kpt is: "https://googlecontainertools.github.io/kpt/"
meta.changelog for kpt is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 0.36.1 with grep in /nix/store/pwvnhfj70p27pk5v9i1f96q5x4pysqzg-kpt-0.36.1

directory tree listing: https://gist.github.com/5f76aa52131642586ca455e10a99fd2c

du listing: https://gist.github.com/b5787f2c90d0baa62bd0a999ae3ddbd9

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
kpt

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/pwvnhfj70p27pk5v9i1f96q5x4pysqzg-kpt-0.36.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A kpt https://github.com/r-ryantm/nixpkgs/archive/e0dfa4f786a6c1b9e3aef758f912ff1231955b26.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/pwvnhfj70p27pk5v9i1f96q5x4pysqzg-kpt-0.36.1
ls -la /nix/store/pwvnhfj70p27pk5v9i1f96q5x4pysqzg-kpt-0.36.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review 1

1 package built:

kpt

Maintainer pings
cc @mikefaille @kalbasit for testing.

Result of nixpkgs-review pr 102492 run on x86_64-darwin 1

1 package built:

kpt


How to serve API spec from multiple services

Hi Team,
We have a API Gateway and we want to serve the documentation for multiple microservices behind it.
We are using Spring Boot 2.2.x.
We tried the springdoc.swagger-ui.urls[0].url but the missing CORS headers block us.
Is there any other way to do that.
May you please point us some other way/example how to achieve that.
All the best

Hi @tzieleniewski,
This is sample code, how to serve multiple microservices behind spring cloud gateway:

https://github.com/springdoc/springdoc-openapi-demos/tree/master/sample-springdoc-openapi-microservices


Fails to remove ml extension on Windows

This is autogenerated. Please review and update as needed.
Describe the bug
When remove ml extension, pvectorc.cp310-win32.pyd file can't be deleted.
After this command fails, az extension list raises error:
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/extension/__init__.py", line 308, in get_extension_modname
AssertionError: Expected 1 module to load starting with 'azext_': got []

After some investigation, I think pyrsistent might be the reason.
Create a new virtual env and install pyrsistent. This script also fails with same error:
import os

import pyrsistent
os.remove('venv/Lib/site-packages/pvectorc.cp310-win_amd64.pyd')

Once pyrsistent is imported, this file can't be deleted.
I guess the file is opened by Python, but I can't find a way to release it.
Command Name
az extension remove
Errors:
Failed to delete 'C:\Users\kk\.azure\cliextensions\ml': [WinError 5] Access is denied: 'C:\\Users\\kk\\.azure\\cliextensions\\ml\\pvectorc.cp310-win32.pyd'. Retrying ...
Failed to delete 'C:\Users\kk\.azure\cliextensions\ml': [WinError 5] Access is denied: 'C:\\Users\\kk\\.azure\\cliextensions\\ml\\pvectorc.cp310-win32.pyd'. Retrying ...
Failed to delete 'C:\Users\kk\.azure\cliextensions\ml': [WinError 5] Access is denied: 'C:\\Users\\kk\\.azure\\cliextensions\\ml\\pvectorc.cp310-win32.pyd'. Retrying ...
Failed to delete 'C:\Users\kk\.azure\cliextensions\ml': [WinError 5] Access is denied: 'C:\\Users\\kk\\.azure\\cliextensions\\ml\\pvectorc.cp310-win32.pyd'. You may try to delete it manually.

To Reproduce:
az extension add -n ml
az extension remove -n ml

Expected Behavior
Environment Summary
Windows-10-10.0.22621-SP0
Python 3.10.5
Installer: MSI

azure-cli 2.41.0

Extensions:
account 0.2.5
alias 0.5.2
interactive 0.4.5
ml Unknown
rdbms-connect 1.0.3

Dependencies:
msal 1.20.0b1
azure-mgmt-resource 21.1.0b1

Additional Context

Good finding！


Intermittent failed deployments due to short timeouts when downloading manylinux packages

Summary
Timeouts for downloading manylinux packages are very short (2 seconds initially then 1.5 seconds for chunks).
Normally this works ok but sometimes repo's can respond slowly, which leads to failed deployments.
Current values may be suitable for default values but it would be great to be able to override these in settings to improve reliability of deployments. Alternatively, allow retries.
Steps to reproduce
Tricky to reproduce consistently, it's only an intermittent problem. Any method for delaying the connection to repo for more than 2 seconds. Perhaps route the download URL through a local proxy with a delay?
Expected Behavior
Add a setting to increase the timeout for downloading manylinux packages, say 30 seconds.
Actual Behavior
Sometimes downloads fail if delayed by more than 2 seconds.
Possible Fix
Configurable timeouts (or retries) in core.py for get_manylinux_wheel_url() and download_url_with_progress() to prevent failed deployments due to slow repo's.
Your Environment

Zappa version used: 0.43.1
Operating System and Python version: OSX / 2.7

This has affected me recently (usually when I'm on a busy/slow network) and results in broken deployments being made. I think Zappa should abort the deployment completely if a package cannot be downloaded.

Duplicated by #1235.


AID attack roll not working for players

Should be working now although I don't think it had any issue that I noted. Perhaps there was a crash happening under the hood?


Door Access Camera Feed?

Is there a way to access the camera feed off the Intercom or G2 Reader Pro through the API?

As said in the top of the README, Camera feeds are currently unsupported.


Emmet Support Instructions

Added instructions for using emmet with .heex file extensions

I added this PR request to the wrong branch. My apologies.


Add workload tenant id support

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-identity


Reproduce 2003 spectral amge paper example

In particular, we consider the Poisson equation discretized with bilinear functions
on a 32 × 32 square grid of square elements. Homogeneous Dirichlet boundary conditions
are assumed, and they are eliminated from the matrix during discretization.
Standard 2 × 2 agglomeration is forced on each level.

Unstaggered elements

metric

number of levels
2

convergence rate
0.04

grid complexity
1.47

operator complexity
1.9

Here is a relevant paragraph from the paper:
Consider Poisson’s equation, discretized using bilinear functions on rectangular
elements. The use of 2 × 2 agglomerates on the finest level necessitates $m_{\tau} = 1$ in
order to maintain $c_{op} < 1$, but this is far too restrictive for rectangular elements with large aspect ratios. In such cases, the assumption, implicit in the estimate, that $c_{op}$ grows at the same rate on all coarser grids is not generally warranted. Unfortunately, more realistic growth rate estimates for $c_{op}$ are difficult to derive. To avoid having separate complexity measures on different levels, we employ a compromise in which the number of levels in the estimate is fixed. For small p, measure (3.10) does not
restrict $m_{\tau}$ excessively; hence, we use small p in practice. In fact, empirical results indicate that a 3-level cost measure (p = 3) produces the most satisfactory results.
To reflect the true cost of the algorithm without resorting to an arbitrary choice of
p, we intend to refine the operator complexity estimate in future research. We also
observe that, if we really wish to restrict $c_{op} < 1$ on all levels, larger agglomerates
could likely be used, allowing more freedom in the choice of $m_{\tau}$.

In the header to the results table, they have "tests used $\mu_{op}$ with $p = 3$". I think this very well means that they used $\mu_{\tau} > 1$, most likely 2. This could explain the 1.47. This also means that we may need to introduce an automatic determination of the number of eigenvectors.

Unfortunately, mfmg breaks right now with 2 eigenvectors. The backtrace shows that it's inside the determination of the Trilinos sparsity pattern.

We reproduced the complexities with #34, but still need to fix the convergence.

Using symmetric Gauss-Seidel instead of Gauss-Seidel change the convergence rate from 0.10 to 0.08

Sad. Btw, did you make a mistake at minimum degree Gauss-Seidel? 0.995 does not look right.

Yeah a zero is missing. Any idea what we can do to fix this?

Not sure what's wrong yet. Here are some hypotheses:
Check Partition of Unity
Can you check (and maybe add a test) that if you sum up all the weights from all the agglomerates, you'll get a vector of all ones? I think one issue is that using global matrix diagonal may not be fully correct as it may be affected by the boundary conditions (not sure if that happens in deal.ii), while in the paper (pg.19) they divide by the sum of agglomerate contributions and not the global diagonal matrix (they are the same for all internal nodes, but I wonder if there are variations for boundary).
Check matrix entries
They say:
Homogeneous Dirichlet boundary conditions are assumed, and they are eliminated from the matrix during discretization.

So maybe when they say 32x32 system, it was originally 34x34 and with eliminated b.c. it has the same entries on the diagonal (while I think deal.ii diagonal may have different entries on the diagonal).
Eigenvectors
We know that the eigenvector corresponding to the smallest eigenvalue is $e = (1, ..., 1)^T$. However, the second eigenvalue spans a subspace (as we have a standard Poisson problem) from which we pick up an eigenvector. We may choose a different eigenvector from the paper. However, I don't see how that could affect the convergence for 2-levels (I could see it affecting 3+ levels due to smoothing on the coarse level).
I would also like to know if the eigenvectors picked for different agglomerates are the same, or if the second eigenvector vary due to some random ARPACK initializations.

RHS and LHS
I don't think we know what they used as the initial guess and rhs for the run. It may be that it was randomly faster than average.

They are using a random initial guess and the rhs is zero. It is described just below equation 4.2

I don't think so because they say the grid is 32x32 not the system.

But the diagonal element they have is probably still the same for all entries. Is our deal.ii matrix the same?

The second vector is different for different agglomerates

Hmm, OK. I wonder if that is done that way, is the global vector still a good approximation? If they were independent, that would have been fine. But with shared dofs, I'm not so sure. Maybe a good idea would be to try to run an experiment where we set all the the local eigenvectors corresponding to the 2nd eigenvalue to the same vector. The intuition says that nothing should be affected.

This is not easy to test. I can easily scale the original matrix but I don't think this is sufficient. The agglomerate matrices also need to be scaled by the same scaling factor as the global one...

Hmm, you may be right. Let me think about it.

Using the same eigenvectors for all the agglomerates improves the convergence rate by 0.01 So the choice of the second eigenvector definitely has an effect.

Hmm, you may be right. Let me think about it.

If $Ax = \lambda x$, then $D^{-1/2}Ax = \lambda D^{-1/2} x$. Let $y = D^{-1/2}x$ ($x = D^{1/2} y$), then $D^{-1/2}AD^{1/2} y = \lambda y$. Therefore, eigenvalues are going to be the same (as matrices $A$ and $D^{-1/2}AD^{1/2}$ are spectrally equivalent), and $y$ eigenvectors are going to need to be scaled by $D^{1/2}$ globally. Therefore, there is no need to scale local matrices, just need to scale the rows of restrictor after building it the usual way. I think this is correct.

Per @Rombur:

I was trying to use different geometries and I found a problem.  To use arpack, we need to invert the local operator. The problem is that Laplace without Dirichlet is singular (you can add an arbitrary constant to the solution). When we use umfpack right now we are lucky and because of the round-off error we are able to invert the local matrix but if I change the geometry it doesn't work anymore. This make me think that we may get the second eigenvector wrong and this could explain the convergence. I am going to see if I can use lapack instead so I don't have to invert the matrix first but we need to understand how this is supposed to work. I saw in saamge that they use arpack so they should have had the same problem.

Per @Rombur:

So if I use lapack, I can use the preconditioner on distorted meshes and on a circle. However, there was a big surprise. Like we could expect lapack and arpack give the same eigenvector for the first eigenvalue but they have different eigenvectors for the second eigenvalue (there are two eigenvectors with this eigenvalues).  Now if we use one or three eigenvectors, we could the same results for both arpack and lapack. However, the convergence rate with two eigenvectors is a lot worse with lapack compared to arpack. Since we don't have the full subspace with only two eigenvectors, I guess it is a matter of luck if we have the right vectors or not. When we have three eigenvectors, we have the full subspace and we are good. So with n=2, there is always some luck involved in reproducing their results. Now, even with n=3, I can't get a convergence rate of 0.04. Depending on the parameters, I get a convergence between 0.05 and 0.08.

Per @Rombur:
Using Arpack
Hyper-cube, number of eigenvectors = 1

Smoother
Ordering
Convergence rate

Gauss-Seidel
default
0.267

Symmetric Gauss-Seidel
default
0.197

Gauss-Seidel
Reverse  Cuthill-McKee
0.269

Symmetric Gauss-Seidel
Reverse Cuthill-McKee
0.191

Gauss-Seidel
King
0.269

Symmetric Gauss-Seidel
King
0.191

Gauss-Seidel
Reverse minimum degree
0.262

Symmetric Gauss-Seidel
Reverse minimum degree
0.218

Hyper-cube, number of eigenvectors = 2

Smoother
Ordering
Convergence rate
Same Eigenvectors

Gauss-Seidel
default
0.105
0.095

Symmetric Gauss-Seidel
default
0.081
0.077

Gauss-Seidel
Reverse  Cuthill-McKee
0.108
0.098

Symmetric Gauss-Seidel
Reverse Cuthill-McKee
0.073
0.066

Gauss-Seidel
King
0.108
0.099

Symmetric Gauss-Seidel
King
0.068
0.061

Gauss-Seidel
Reverse minimum degree
0.0995
0.092

Symmetric Gauss-Seidel
Reverse minimum degree
0.087
0.078

Hyper-cube, number of eigenvectors = 3

Smoother
Ordering
Convergence rate

Gauss-Seidel
default
0.075

Symmetric Gauss-Seidel
default
0.058

Gauss-Seidel
Reverse  Cuthill-McKee
0.079

Symmetric Gauss-Seidel
Reverse Cuthill-McKee
0.051

Gauss-Seidel
King
0.079

Symmetric Gauss-Seidel
King
0.051

Gauss-Seidel
Reverse minimum degree
0.071

Symmetric Gauss-Seidel
Reverse minimum degree
0.057

Using Lapack
Hyper-cube, number of eigenvectors = 1

Smoother
Ordering
Convergence rate

Gauss-Seidel
default
0.267

Symmetric Gauss-Seidel
default
0.197

Gauss-Seidel
Reverse  Cuthill-McKee
0.269

Symmetric Gauss-Seidel
Reverse Cuthill-McKee
0.191

Gauss-Seidel
King
0.269

Symmetric Gauss-Seidel
King
0.191

Gauss-Seidel
Reverse minimum degree
0.262

Symmetric Gauss-Seidel
Reverse minimum degree
0.218

Hyper-cube, number of eigenvectors = 2

Smoother
Ordering
Convergence rate

Gauss-Seidel
default
0.213

Symmetric Gauss-Seidel
default
0.147

Gauss-Seidel
Reverse  Cuthill-McKee
0.216

Symmetric Gauss-Seidel
Reverse Cuthill-McKee
0.136

Gauss-Seidel
King
0.216

Symmetric Gauss-Seidel
King
0.142

Gauss-Seidel
Reverse minimum degree
0.208

Symmetric Gauss-Seidel
Reverse minimum degree
0.165

Hyper-cube, number of eigenvectors = 3

Smoother
Ordering
Convergence rate

Gauss-Seidel
default
0.075

Symmetric Gauss-Seidel
default
0.058

Gauss-Seidel
Reverse  Cuthill-McKee
0.079

Symmetric Gauss-Seidel
Reverse Cuthill-McKee
0.051

Gauss-Seidel
King
0.079

Symmetric Gauss-Seidel
King
0.051

Gauss-Seidel
Reverse minimum degree
0.071

Symmetric Gauss-Seidel
Reverse minimum degree
0.057

Diagonal scaling may not help with this matrix. All rows corresponding to Dirichlet dof have 1.333 on the diagonal, and 0 offdiagonals. All other rows have 2.666 on the diagonal, and 0 or -0.333 offdiagonal. Thus, scaling would not change anything as Dirichlet rows will converge in a single iteration, and all others will be scaled exactly the same.


Sfr/test debugging

Is this a work in progress PR? What is this for?

This will show you that not all tests pass the compliance test. #16 have me showed that something is wrong with the compliance test. With this PR I mean to show which test are going wrong.

I synced up with the latest compliance tests in https://github.com/mtdowling/jmespath.rs/commit/9e4824b18a2d9f92e9d477b27eb94c15219f6b55, and I was able to pass every test. Resolving as I can't reproduce.

Is #16 fixed?


Add https_redirect_response_code variable

Adds a https_redirect_response_code to allow the HTTP status code to be chosen for the HTTPS redirect. The variable defaults to "MOVED_PERMANENTLY_DEFAULT" so that there is no change in functionality unless a different value is explicitly set.
A use case for this is when you want the request method to be retained on the HTTPS request.

Thanks for the PR! 🚀✅ Lint checks have passed.

Closing this while we discuss #236.

@morgante could we please reconsider the decision made in #236 and this PR?
I appreciate your thoughts on how http-https redirect should look, but at the same time the GCP product itself, i.e. External HTTP LB, offers configuring of the http status code for the redirect. I'd love to see consistency among GCP products - either it's a TF module, or SDK based tool, or API/Web Console.
Keeping the current value as the default one, will help with backward compatibility.
Thank you in advance!

@andrewmackett / @mqmr - do you mind recreating the PR since the originating repo has been deleted. We're happy to review the PR for you.


BlueMap-Chat's web assets sometimes don't load

Sometimes bluemap-chat.js and assets/bluemap-chat.css are missing from bluemap/web/settings.json after the server starts up, so the chat interface is missing completely.
When this happens running bluemap reload causes settings.json to be updated correctly, and everything works without issue (until the next restart).
Of course on my development and testing instances this never happens; however, it happens every time on my production server.
Broken settings.json:
{
  "defaultToFlatView": true,
  "enableFreeFlight": false,
  "hiresSliderDefault": 100,
  "hiresSliderMax": 500,
  "hiresSliderMin": 0,
  "lowresSliderDefault": 2000,
  "lowresSliderMax": 7000,
  "lowresSliderMin": 500,
  "maps": [
    "world",
    "world_the_end",
    "world_nether"
  ],
  "maxZoomDistance": 100000,
  "minZoomDistance": 5,
  "resolutionDefault": 1,
  "scripts": [
    "assets/clod-mc.js",
    "assets/auth-integration.js"
  ],
  "styles": [
    "assets/clod-mc.css"
  ],
  "useCookies": true,
  "version": "4.1"
}

Then after bluemap reload:
{
  "defaultToFlatView": true,
  "enableFreeFlight": false,
  "hiresSliderDefault": 100,
  "hiresSliderMax": 500,
  "hiresSliderMin": 0,
  "lowresSliderDefault": 2000,
  "lowresSliderMax": 7000,
  "lowresSliderMin": 500,
  "maps": [
    "world",
    "world_the_end",
    "world_nether"
  ],
  "maxZoomDistance": 100000,
  "minZoomDistance": 5,
  "resolutionDefault": 1,
  "scripts": [
    "assets/clod-mc.js",
    "assets/bluemap-chat.js",
    "assets/auth-integration.js"
  ],
  "styles": [
    "assets/bluemap-chat.css",
    "assets/clod-mc.css"
  ],
  "useCookies": true,
  "version": "4.1"
}

Interestingly auth-integration.js is always present.
At first I thought BlueMap-Chat was missing a dependency on BlueMap, but this isn't the case.  Looking at my startup logs I see BlueMap being loaded first:
[13:52:04] [Server thread/INFO]: [BlueMap] Loading server plugin BlueMap v4.1
[13:52:04] [Server thread/INFO]: [BlueBridgeCore] Loading server plugin BlueBridgeCore v2.1
[13:52:05] [Server thread/INFO]: [BlueMap-Auth] Loading server plugin BlueMap-Auth v0.1.0
[13:52:06] [Server thread/INFO]: [BlueBridgeGP] Loading server plugin BlueBridgeGP v2.1
[13:52:07] [Server thread/INFO]: [BlueMap-Chat] Loading server plugin BlueMap-Chat v0.2.0
[13:52:07] [Server thread/INFO]: [BlueMapFloodgate] Loading server plugin BlueMapFloodgate v3.2
[13:52:13] [Server thread/INFO]: [BlueMap] Enabling BlueMap v4.1
[13:52:13] [Server thread/INFO]: [BlueMap] Saving all worlds once, to make sure the level.dat is present...
[13:52:14] [BlueMap-Load/INFO]: [BlueMap] Loading...
[13:52:14] [Server thread/INFO]: [BlueBridgeCore] Enabling BlueBridgeCore v2.1
[13:52:15] [BlueMap-Load/INFO]: [BlueMap] Loading resources...
[13:52:16] [BlueMap-Load/INFO]: [BlueMap] Loading textures...
[13:52:16] [Server thread/INFO]: [BlueMap-Auth] Enabling BlueMap-Auth v0.1.0
[13:52:16] [Server thread/INFO]: [BlueMap-Auth] BlueMap Auth Integration enabled
[13:52:18] [BlueMap-Load/INFO]: [BlueMap] Baking resources...
[13:52:19] [BlueMap-Load/INFO]: [BlueMap] Resources loaded.
[13:52:19] [BlueMap-Load/INFO]: [BlueMap] Initializing Storage: 'file' (Type: 'bluemap:file')
[13:52:19] [BlueMap-Load/INFO]: [BlueMap] Loading map 'world'...
[13:52:19] [BlueMap-Load/INFO]: [BlueMap] Loading map 'world_the_end'...
[13:52:19] [BlueMap-Load/INFO]: [BlueMap] Loading map 'world_nether'...
[13:52:19] [BlueMap-Load/INFO]: [BlueMap] WebServer bound to all network interfaces on port 8100
[13:52:19] [Thread-11/INFO]: [BlueMap] WebServer started.
[13:52:19] [Server thread/INFO]: [BlueBridgeGP] Enabling BlueBridgeGP v2.1
[13:52:19] [Server thread/WARN]: [BlueBridgeGP] "BlueBridgeGP v2.1" has registered a listener for me.ryanhamshire.GriefPrevention.events.ClaimModifiedEvent on method "public void de.mark225.bluebridge.griefprevention.addon.listener.GriefPreventionListener.onClaimModified(me.ryanhamshire.GriefPrevention.events.ClaimModifiedEvent)", but the event is Deprecated. "ClaimModifiedEvent will be removed in favor of ClaimResizeEvent"; please notify the authors [Mark_225, Aurelien30000].
[13:52:19] [BlueMap-Load/INFO]: [BlueMap-Auth] Webserver bound to /<IP_ADDRESS>:8400
[13:52:19] [BlueMap-Load/INFO]: [BlueMap] Loaded!
[13:52:20] [Server thread/INFO]: [BlueMap-Chat] Enabling BlueMap-Chat v0.2.0
[13:52:20] [Server thread/INFO]: [BlueMap-Chat] Webserver bound to /<IP_ADDRESS>:8800
[13:52:20] [Server thread/INFO]: [BlueMap-Chat] BlueMap Chat enabled
[13:52:20] [Server thread/INFO]: [BlueMapFloodgate] Enabling BlueMapFloodgate v3.2
[13:52:20] [Server thread/INFO]: [BlueMapFloodgate] BlueMap Floodgate compatibility plugin enabled!

I'm a bit stumped on the next step to diagnose this; any pointers?

I'd imagine it would at least throw an error if something failed... absolutely no idea what is happening here. Never have had this happen and I don't see a reason for it to happen...

🎉  yay.
Thanks for the super quick response!  That you can't think of a reason why this could happen is useful.
I suspect it's best if I close this issue and poke around some more.  I'll report back if I find anything.

Alright apparently some weird BlueMap behaviour with saving the settings.json, I may be able to work around this one

You may find success with the recent CI build artifact of BlueMap-Chat https://github.com/Chicken/Auth/actions/runs/8950375983/artifacts/1473055765
Registering of assets was moved to onLoad

Thanks; I've tested it once and it went from a 100% failure rate to 100% success.


[FEATURE] Support multiple port_mappings for the primary container

Is your feature request related to a problem? Please describe
Primary container in the ECS Service module only supports 1 port/protocol.
Describe the solution you'd like
ECS Service module support port_mappings map. This will allow multiple ports to be exposed on the primary container.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
N/A
Additional context
Add any other context or screenshots about the feature request here.

Describe the solution you'd like
ECS Service module support port_mappings map. This will allow multiple ports to be exposed on the primary container. ECS blueprints currently doesn’t support this feature.
Describe alternatives you've considered
ECS terraform  module supports multiple port mappings on the primary container.
https://github.com/umotif-public/terraform-aws-ecs-fargate/issues/54
https://towardsaws.com/multiport-fargate-ecs-service-with-terraform-
Additional context
This feature allows applications to expose multiple ports ex: 3000 for http connectivity and 5000 for websocket connectivity.
More information on ECS Portmappings: https://docs.aws.amazon.com/AmazonECS/latest/APIReference/API_PortMapping.html


Breaking change for 2.14 - SQL command collection off by default

More details here https://github.com/microsoft/ApplicationInsights-Announcements/issues/28 but the gist of it is to add this to the ApplicationInsights.config for ASP.NET.
<Add Type="Microsoft.ApplicationInsights.DependencyCollector.DependencyTrackingTelemetryModule, Microsoft.AI.DependencyCollector">,
<EnableSqlCommandTextInstrumentation>true</EnableSqlCommandTextInstrumentation>
</Add>

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

ID: 35ac4c4b-6534-7751-1aa9-51d631f1c7e4
Version Independent ID: 32f21b4f-5f9b-5706-1410-8e9c4861142e
Content: Azure Application Insights Agent troubleshooting and known issues - Azure Monitor
Content Source: articles/azure-monitor/app/status-monitor-v2-troubleshoot.md
Service: azure-monitor
Sub-service: application-insights
GitHub Login: @TimothyMothra
Microsoft Alias: tilee

@kiwiant Thanks for the question! We're investigating this and will get back to you shortly.

Hello, I'm an engineer on the C# SDK.
Yes, we introduced this breaking change, it was necessary to fix a privacy bug.
As of today, this does not affect the Status Monitor v2 product because they are not on the latest version of our SDK.
We've been in communication with that team so that they make this change when they update.
@ank3it, @mrbullwinkle

Understand the necessity of the change and agree with it.
My goal was to communicate to the community this change to avoid hours of lost time investigating. A document on troubleshooting known issues on Azure Monitor seems like an ideal place to communicate this breaking change.
We are using Az.ApplicationMonitor 1.1.0 on-prem with version 2.14 of the Nuget packages and were directly affected using latest release code until I found the EnableSqlCommandTextInstrumentation setting solution.

I'm sorry to hear you were affected by our change. In the future, feel free to ask us questions on our github! We try to have conversations about topics like this publically so they get indexed by search engines.
I'm curious to hear more about your use case.
The Az.ApplicationMonitor product will back off if your application is already instrumented with our SDK.
I assume you're using Az.ApplicationMonitor to install the profiler specifically for SQL dependencies?

Correct Az.ApplicationMonitor to collect SQL command text. We were using the Status Monitor but after the upgrade we lost the SQL command text. More interestingly this was a partial loss as some (not all) store procedure calls showed.
In hindsight this may have coincided with the the use of the 2.14 Application Insights packages. The eventually solution 2.15 beta and the use of the EnableSqlCommandTextInstrumentation setting but haven't diagnosed further.
Affected solution is using ASP.NET 4.8 running on-prem.
Happy to take a call if the team needs more detail.

@kiwiant Thanks for bring this to our attention. I updated the Status Monitor v2 troubleshooting doc to reflect what people need to add to their ApplicationInsights.config file for SQL collection. The change should go live later this weekend.
Thanks again
#please-close


feat: Add /nodes/{nodeLabel}/dependencies endpoint

Depends on https://github.com/Sage-Bionetworks/sage-monorepo/pull/1903
fixes fds-782
This PR:

Adds the /nodes/{nodeLabel}/dependencies endpoint (formerly  /explorer/get_node_dependencies)
Adds the node, nodesPage, and nodeLabel schema
Adds the component and componentsPage schema (not needed for this PR, but kept for future ones)
Adds returnDisplayNames and returnOrderedBySchema parameters

Files to Check:

Everything in libs/schematic/api-description/src/
Everything in apps/schematic/api/schematic_api/test/
apps/schematic/api/schematic_api/controllers/schema_controller_impl.py

@mialy-defelice This is ready to re-check. Thanks!


tdlib 1.8.2: Changed behaviour for deleting messages?

Hello,
we found out that with tdlib 1.8.2 the flags if a message can be deleted for either self or all users is making problems for group admins. It seems they get a "false" on all messages except their own. Was there a change in what Admins can do?
BR

There were no changes. Do you use 7da29e17d636f7dc8609ed88f3e915710daa4bd6 as 1.8.2? Generally, for old versions you need to use the last commit before the "Update layer to ..." commit.

@levlam yes exactly that commit. Investigating further...

But as a fact, Admins should be able to delete everything, no?

Yes.
There was a bug between d97d20a4209b6974371141a2ecd87c0a1e55e92f and 68db9a03cccbe3973c3d741b9a9f26f446716061, which resulted in the bug. You can downgrade 1.8.2 to 163bfbcc7a86f3faea373ae70aec9aa7b51b71eb to avoid it.


chore(all): Remove the box_syntax feature gate.

I can't merge this? Did you start from an older master?

Possibly, let me rebase.

I've included this in #403


Text on login screen should have better contrast

Much of the text on the login screen doesn't satisfy the W3C WCAG Level AA guidelines for sufficient contrast from the background.
You can identify which text has this issue via the Web Developer or Accessibility Developer Tools browser extensions or by entering the page URL at http://wave.webaim.org/.  As of the time of writing this, the elements at the following CSS selectors are flagged as having insufficient contrast:

.split-view .info-box .organization-path
.split-view .left-side .description a (2 links)
.portico-page .or
.login-google a
.login-github a
.login-page-container .right-side .actions (2 links)
.footer .footer-navigation a (7 links)
.footer .footer-navigation span (6 separator bullets)

The browser extensions and online accessibility audit tool mentioned above can all help pick color pairs with more suitable contrast ratios.

@zulipbot label "accessibility"

@zulipbot claim

This can be closed! Forgot to add the issue in the commit body.


Add ability to cancel Gossip Sync

In integrating Speedloader into ZEUS, we discovered that some users were facing frustrations being with sync times in some situations due to poor network connections. In order to skip the sync, they would have to disable Speedloader (called Express Graph Sync in ZEUS) in Settings and then restart the app.
This PR adds a new function, CancelGossipSync that cancels the GossipSync function using a newly added global context.
Related ZEUS issue

Looking pretty good!
nit: fmt.Println -> proper logger


[SPARK-43979][SQL][FOLLOWUP] transformUpWithNewOutput should only be used with new outputs

What changes were proposed in this pull request?

This is a followup of https://github.com/apache/spark/pull/41475 . It's risky to use transformUpWithNewOutput with existing attribute ids. If the plan contains duplicated attribute ids somewhere, then we will hit conflicting attributes and an assertion error will be thrown by QueryPlan#transformUpWithNewOutput.
This PR takes a different approach. We canonicalize the plan first and then remove the alias-only project. Then we don't need transformUpWithNewOutput anymore as all attribute ids are normalized in the canonicalized plan.
Why are the changes needed?

fix potential bugs
Does this PR introduce any user-facing change?

No
How was this patch tested?

existing tests

thanks, merged to master and 3.5

Hi, @cloud-fan, @yaooqinn and all. Unfortunately, this follow-up commit broke the original test case of SPARK-43979 in branch-3.5. Could you make a follow-up to recover the branch?

https://github.com/apache/spark/actions/runs/5808343145/job/15744942830 (this PR backport)
https://github.com/apache/spark/actions/runs/5826647221/job/15801070612

[info] SQLQuerySuite:
22:39:37.462 WARN org.apache.hadoop.util.NativeCodeLoader: Unable to load native-hadoop library for your platform... using builtin-java classes where applicable
[info] - SPARK-43979: CollectedMetrics should be treated as the same one for self-join *** FAILED *** (686 milliseconds)
[info]   org.apache.spark.sql.catalyst.ExtendedAnalysisException: [DUPLICATED_METRICS_NAME] The metric name is not unique: my_event. The same name cannot be used for metrics with different results. However multiple instances of metrics with with same result and name are allowed (e.g. self-joins).;

Let me revert this to unblock other testings. I verified locally that the revert recovers the branch-3.5.
$ git log -n1
commit 898f3d8a0bdc044838a576789c9f26e96e73bc60 (HEAD -> branch-3.5)
Author: Dongjoon Hyun<EMAIL_ADDRESS>Date:   Thu Aug 10 23:04:07 2023 -0700

    Revert "[SPARK-43979][SQL][FOLLOWUP] transformUpWithNewOutput` should only be used with new outputs"

    This reverts commit b8204d1b89eea0c32e5269fb155651157e2c96e8.

$ build/sbt "sql/testOnly org.apache.spark.sql.SQLQuerySuite -- -z SPARK-43979"
[info] SQLQuerySuite:
23:05:24.559 WARN org.apache.hadoop.util.NativeCodeLoader: Unable to load native-hadoop library for your platform... using builtin-java classes where applicable
[info] - SPARK-43979: CollectedMetrics should be treated as the same one for self-join (1 second, 9 milliseconds)
23:05:34.829 WARN org.apache.spark.sql.SQLQuerySuite:

===== POSSIBLE THREAD LEAK IN SUITE o.a.s.sql.SQLQuerySuite, threads: rpc-boss-3-1 (daemon=true), shuffle-boss-6-1 (daemon=true) =====
[info] Run completed in 13 seconds, 695 milliseconds.
[info] Total number of tests run: 1
[info] Suites: completed 1, aborted 0
[info] Tests: succeeded 1, failed 0, canceled 0, ignored 0, pending 0
[info] All tests passed.
[success] Total time: 69 s (01:09), completed Aug 10, 2023 11:05:35 PM

This is reverted via https://github.com/apache/spark/commit/898f3d8a0bdc044838a576789c9f26e96e73bc60 from branch-3.5.

Thanks @dongjoon-hyun

Thank you @dongjoon-hyun ! let me resubmit the PR to 3.5 and investigate the test failures.

Thank you so much, @yaooqinn and @cloud-fan .


How to test OSPRAY_MODULE_BILINEAR_PATCH ?

I am building ospray manually (not via superbuild)
I see this option.
It builds the module so file
I noticed that when I do make install, the *.so file is not installed in the installation location.
Is there are particular test that will exercise this module ?
Cheers

The main purpose of the bilinear patch is the demonstration of how to write a module, a plugin to extend OSPRay. We test its compilation to catch that this module needs to adapt to internal changed, but not more.
I just realize that its README outdated regarding loading it in ospExampleViewer... I think it's rather easy to adapt some of the builders in apps/ to serve some vertex arrays to the bilinear geometry. Many of the features of former ospExampleViewer are now found in OSPRay Studio, but not yet the bilinear patch.

superbuild lacks control rules to build this module. Can you add it?

In the superbuild the bilinear patch module is activated via BUILD_OSPRAY_CI_EXTRAS.

Ok. It is built. But not installing. Just not copied from
building directory to CMAKE_PREFIX_INSTALL/bin
Win10/MSVS2019/tbbH18.5.672
пт, 17 сент. 2021 г. в 14:15, Johannes Günther @.***>:

In the superbuild the bilinear patch module is activated via
BUILD_OSPRAY_CI_EXTRAS.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ospray/ospray/issues/493#issuecomment-921714375, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAQD2CPE3ZCRUAOQCTMDAEDUCMPMHANCNFSM5DYH6PNA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Yes, as said, the module servers as example how to write modules. The bilinear patch geometry is not indented as first class geometry.
Is this rather a feature request, i.e., do you have use cases for bilinear patches and like to have that geometry included in the native build?

No, I seem, there no need to put this implementation to production, really it may be used just as a template.
To be true - it is one of hardest expansion module development process I've deal with.

To install your own module you could use ospray_install_library, see e.g. modules/multiDevice/CMakeLists.txt.
We are happy to improve the dev process based on your suggestions. Please note that when writing custom modules we cannot generally guarantee that used internal APIs stay stable (which is why we encourage checking versions during module init).


Is this a bug? Exposure warning coming 6 days too late

My understanding of the corona-warn-app is that it will try to warn contacts AS EARLY AS POSSIBLE.
So is this a bug?  The following has happened:
On December 12th a group of six people met indoor in a room for more than one hour.
One of them had a covid-19 infection only 2 weeks ago, his positive test had been registered in his cwa app on November 29th. So on December 12th he was already out of quarantine, but in the warn app he was still "red".
All other members of the group got no warning on December 12th, 13th, 14th, 15th, 16 th, 17th. But on December 18th and 19th ALL members of the group got a warning of a high risk contact for December 12th.
Why would the app "wait for 6-7 days" before it warns other people. This seems to be a bug. Or is it a feature? Am I missing out something?
Was the cwa warning service ever "interrupted" due to LOG4J issues ?

The app was not waiting it will publish keys after 2h waiting period. Sometimes it took some until the infectious person recognize the symptoms, take a PCR test and share the keys with other CWA user.
After submitting the keys then also other person 6 days in the past get warned, depending on the onset of symptoms.

There seems to be a basic misunderstanding on how the app works here.
If a user enters their positive test in the app, the app warns all contacts who were met in the last 14 days. But the app does not warn the contacts which were met after the positiv test was entered into the app.
Thus, the increased risk warning you received wasn't from the person who registered their infection in the CWA on 29.11.21.

Can this issue be closed?

Yes it can be closed. Thanks to everyone.


Kpi minor enhancements

Coverage remained the same at 91.99% when pulling fd097fc2342dcd996ac34f89be7f3985678e6101 on comzeradd:kpi-spinner into 2b6a42b2b53caa9b7fbca319cf163d9e0258c49f on mozilla:master.

I suggest to use the unminified d3.js and compress it like we do with the rest of the libs.
https://github.com/mbostock/d3/releases/download/v3.5.5/d3.zip

Thanks @johngian. PR updated.

Coverage remained the same at 91.99% when pulling 592eea9cb8bd69dfb70e8493df78d6e9647a9932 on comzeradd:kpi-spinner into 2b6a42b2b53caa9b7fbca319cf163d9e0258c49f on mozilla:master.

Please move this fix to another commit: https://github.com/comzeradd/remo/commit/592eea9cb8bd69dfb70e8493df78d6e9647a9932#diff-8883a863906ee9ab9e6b8f44d4ad13e1R413

You can squash the minor fixes to a single commit but this is just a suggestion (feel free to choose)

I' we pushed some changes here: https://github.com/johngian/remo/tree/pr-998

Feel free to cherry pick the jshint fixes

I am not sure if the second fix is required. Please r? and apply if we want to enable multiple filters.

r+wc

Thanks. Squashed and Cherry-picked :gift:


Not able to choose different fanart

Hi!
I've been using Artwork Beef with Kodi Leia without issues and last night I finally switched to Matrix because I read about your Matrix version!
I can't really scrape my artwork with the default Kodi scrapers because there's no option to limit the amount of artwork it scrapes. My limited storage on the Shield always runs low when scraping artwork with TMDb.
So I completely wiped my install and started building my library from scratch. (I have setup a virtual library using TMDb Helper). After that I installed Artwork Beef, set every artwork type to 'one' (like I did in the Leia version) and started scraping. After it was done I checked my library and found that a lot of fanart had titles in them!
I double checked the settings and 'Prefer fanart with no title' was enabled.
So I didn't understand the problem until I read somewhere that you kind of combined the fanart and landscape images. I thought no big deal I'll just switch the fanart manually, so I clicked the context menu; manage; AB: Select artwork. It then shows the menu with all the different artwork, it says 'x amount' available, but when I click on fanart, nothing shows and I can't select anything.
(See screenshots below)
So I thought maybe if I disable the language in the settings, change the amount of scraped artwork to 'three' and try again.. So I deleted all the artwork and scraped again.. But still the same problem: Titles in fanart and no way to manually change the artwork.
I also noticed I didn't like some artwork that came from fanart.tv so I set the preferred scraper to TMDb, which is only an option for movies, would be great if that option would be available for tv shows as well.
Another question is regarding the rating of the scraped artwork, from 0-10, what does that mean? Cause I can't see ratings on the sites, I went to fanart.tv and TMDb.. Obviously I want the best artwork for my media and noticed if I set the rating to 10 almost nothing gets scraped, but if I set to 0 I get pretty bad fanart while my size is set to 1920x1080.. I've been at it for hours trying to built my library and finally gave up so I thought maybe I should open an issue on GitHub lol.
Hope you can help me!

Ok I had a look at the show Dark on my setup with Artwork beef, and I got artwork showing up at my end.
I made the change to artwork beef to use fanart/landscape art for shows where no fanart/landscape is actually returned.  I'd much rather have some slighty "wrong" art than none at all.
So yes this would no longer respect the "titles" setup, which is pretty hit and miss whether or not the original fanart uploader acutally set it up correctly.
Without full logs im not sure if I can help you.  Even with full logs im not sure I can help you, i ported it over to kodi 19 and made some changes but there is a bunch of stuff in the addon im not 100% sure how it works.
Ratings and stuff, im not sure about either.  I dont even know how you rate stuff on fanart tv so again I wouldnt trust ratings at all.
So adding an additional scraper for TV shows, probably not going to happen.
Basically my experience with kodi fanart is that you will probably need to manually fix a bunch of stuff and just plug the gaps in your library artwork yourself with artwork beef/kodis builtin as a jumping off point

Thank you for your feedback! I did indeed download artwork I wanted to use and changed it inside kodi using the default 'choose art' function in information. Everything is good now so I'll close this issue :)


Added template cookbook property, updated readme and spec tests

Signed-off-by: Dustin Lactin<EMAIL_ADDRESS>Description
Added the template_cookbook parameter to the motd custom resource. Updated readme and added spec tests for new functionality.
Check List

[ ] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[x] New functionality includes testing.
[x] New functionality has been documented in the README if applicable
[x] All commits have been signed for the Developer Certificate of Origin. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD

Released as 5.1


[Unblock Request]

repository.eset.com is a false positive. Eset is an AV company.

Domain added to OSINT-DS white list => c8282d951009fd21dc8d8bd036a3ac3d082dba6e
Thanks for the advice.
Regards


Access denied

I have following error:

events.js:2817: Uncaught Error: Opening \\.\COM1: Access denied

my program is:

var SerialPort = require("serialport").SerialPort;

var serialPort = new SerialPort('COM1',{baudrate: 9600}, true);

serialPort.on ('open', function () {
    console.log("Open");
    serialPort.write(0x05);
    serialPort.on ('data', function( data ) {
        console.log("data" + data.toString());
    });
});
What could be the reason?
I am trying to test the COM port working with cash register device.

Same here, win7x64, It solves restarting the PC


Feature Request: Theme Support

Ability to change theme of application (including a dark mode/theme)

Hi there! Thanks for creating this issue. The app already has an official dark mode, but I think it would definitely be great to allow theming in a similar way to Google Play Music Desktop Player. I'll definitely research this.

Hi! how to activate the dark theme?


Jupyter Notebooks do not work out of the box

When following the installation instructions in the README file, the necessary libraries or modules (e.g., libbasic) are not being built. This is the output I get when trying to execute the first part of index.ipynb:
jupyter notebook ./share/jupyter/index.ipynb
[...]
[NbConvertApp] Converting notebook basic.ipynb to notebook
[NbConvertApp] Executing notebook with kernel: python3
Collecting package metadata: done
Solving environment: failed

PackagesNotFoundError: The following packages are missing from the target environment:
  - libbasic

CommandNotFoundError: To use 'conda build', install conda-build.

[NbConvertApp] ERROR | Error while converting 'basic.ipynb'
Traceback (most recent call last):
  File "/home/user/miniconda3/envs/fp17/lib/python3.7/site-packages/nbconvert/nbconvertapp.py", line 407, in export_single_notebook
    output, resources = self.exporter.from_filename(notebook_filename, resources=resources)
  File "/home/user/miniconda3/envs/fp17/lib/python3.7/site-packages/nbconvert/exporters/exporter.py", line 178, in from_filename
    return self.from_file(f, resources=resources, **kw)
  File "/home/user/miniconda3/envs/fp17/lib/python3.7/site-packages/nbconvert/exporters/exporter.py", line 196, in from_file
    return self.from_notebook_node(nbformat.read(file_stream, as_version=4), resources=resources, **kw)
  File "/home/user/miniconda3/envs/fp17/lib/python3.7/site-packages/nbconvert/exporters/notebook.py", line 32, in from_notebook_node
    nb_copy, resources = super(NotebookExporter, self).from_notebook_node(nb, resources, **kw)
  File "/home/user/miniconda3/envs/fp17/lib/python3.7/site-packages/nbconvert/exporters/exporter.py", line 138, in from_notebook_node
    nb_copy, resources = self._preprocess(nb_copy, resources)
  File "/home/user/miniconda3/envs/fp17/lib/python3.7/site-packages/nbconvert/exporters/exporter.py", line 315, in _preprocess
    nbc, resc = preprocessor(nbc, resc)
  File "/home/user/miniconda3/envs/fp17/lib/python3.7/site-packages/nbconvert/preprocessors/base.py", line 47, in __call__
    return self.preprocess(nb, resources)
  File "/home/user/miniconda3/envs/fp17/lib/python3.7/site-packages/nbconvert/preprocessors/execute.py", line 354, in preprocess
    nb, resources = super(ExecutePreprocessor, self).preprocess(nb, resources)
  File "/home/user/miniconda3/envs/fp17/lib/python3.7/site-packages/nbconvert/preprocessors/base.py", line 69, in preprocess
    nb.cells[index], resources = self.preprocess_cell(cell, resources, index)
  File "/home/user/miniconda3/envs/fp17/lib/python3.7/site-packages/nbconvert/preprocessors/execute.py", line 378, in preprocess_cell
    raise CellExecutionError.from_cell_and_msg(cell, out)
nbconvert.preprocessors.execute.CellExecutionError: An error occurred while executing the following cell:
------------------
subprocess.call('conda remove -y libbasic', shell=True)
CONDA_RECIPE = os.path.join(GIT_ROOT, 'bin', 'conda', 'libbasic')
subprocess.check_call('conda build --python=' + PYTHON_VERSION + ' ' + CONDA_RECIPE + ' -c statiskit -c defaults --override-channels',
                      shell=True)
subprocess.check_call('conda install -y libbasic --use-local -c statiskit -c defaults --override-channels',
                      shell=True)
------------------

---------------------------------------------------------------------------
CalledProcessError                        Traceback (most recent call last)
<ipython-input-7-059b9ae9f86d> in <module>
      2 CONDA_RECIPE = os.path.join(GIT_ROOT, 'bin', 'conda', 'libbasic')
      3 subprocess.check_call('conda build --python=' + PYTHON_VERSION + ' ' + CONDA_RECIPE + ' -c statiskit -c defaults --override-channels',
----> 4                       shell=True)
      5 subprocess.check_call('conda install -y libbasic --use-local -c statiskit -c defaults --override-channels',
      6                       shell=True)

~/miniconda3/envs/fp17/lib/python3.7/subprocess.py in check_call(*popenargs, **kwargs)
    345         if cmd is None:
    346             cmd = popenargs[0]
--> 347         raise CalledProcessError(retcode, cmd)
    348     return 0
    349

CalledProcessError: Command 'conda build --python=3.7 /home/user/Documents/Projects/Python/AutoWIG-FP17/FP17/bin/conda/libbasic -c statiskit -c defaults --override-channels' returned non-zero exit status 1.
CalledProcessError: Command 'conda build --python=3.7 /home/user/Documents/Projects/Python/AutoWIG-FP17/FP17/bin/conda/libbasic -c statiskit -c defaults --override-channels' returned non-zero exit status 1.

Some quick info on how to build it (I guess I have to use scons?) would be very helpful.
Maybe the conda build package is missing in the provided meta package?
This is my environment:
environment.txt

Did you install conda-build in your base environment ?
conda install -n base conda-build

This is required but since you can only install it in the base environment, this has to be done manually.
I will add it to the documentation.

Note that I'm not sure that the install documentation of FP17 is up to date.
Using the autowig-toolchain environment with jupyter installed, should be enough.

I just installed conda-build. But the libbasic package still cannot be build.
I think the cause of the error lies here:
subprocess.call('conda remove -y libbasic', shell=True)
CONDA_RECIPE = os.path.join(GIT_ROOT, 'bin', 'conda', 'libbasic')
subprocess.check_call('conda build --python=' + PYTHON_VERSION + ' ' + CONDA_RECIPE + ' -c statiskit -c defaults --override-channels',
                      shell=True)
subprocess.check_call('conda install -y libbasic --use-local -c statiskit -c defaults --override-channels',
                      shell=True)

CONDA_RECIPE does not point to a valid directory. There is no bin folder in the Git repository and thus conda cannot build anything.

Well, l wanted to update notebooks for a long time but did'nt have time.
Today (as you can see) and tomorow I have some.
I will try to repair all notebooks. The major problems are that:

the repository structure have been changed.
the PyBind11 generator has been implemented and files related to SCons are ot up-to-date with the new version of scons-tools.

Note that this will take a little time (at least to replay all notebooks). I will let you know when all notebooks are ready.

Thank you so much!

Hy, it's been a while but I was working on other projects.
I just committed few changes that repair three out of four notebooks.
The only one not working is the one with dependent libraries.
This is due to the fact that repositories for libraries used are now missing.
I will transform this notebook in a little time but you can play with the others.
Note that this requires to update the fp17 environment (using AutoWIG 1.1.0)


Skip serializing review part if it's == Default::default()

Especially for advisories that would cut on the size of the proof.

That would break previous versions. Worth it?

Done.


Implement moving platforms

https://github.com/RodZill4/godot-platform2d
This plugin has moving platforms. Maybe this could be implemented in the platform_template

Can you add a .gif of the moving platforms? I've implemented some a while ago, but they were really bad implemented, And for horizontal movement they didn't work.
Also, I think will be better to wait 3.1 for that, they made a better implementation of KinematicBody physics.
https://godotengine.org/article/godot-31-will-get-many-improvements-kinematicbody

Oh! I remember 3.1 is gonna get this.
I agree that it's better to wait.

I think this is fixed in https://github.com/pigdevstudio/platform_template/commit/910b0e7e0fe818062f4b41c41bac8700498df87c I won't add a smoothing snap...snap is snap, it teleports stuff to the snapping point, not tweens :P
But now we have the option to mode on X and Y axis while in the ladder, so I think is good now


Wrong ScriptPath variable for in memory execution

Hi,
first of all you really did some nice work on this in the last months. Thank you for that, this is my favorite Privesc-Script atm.
I found that by loading your script not from disk but from a webserver or decoding it at runtime the global $ScriptPath variable is -  wrong. For example:
IEX (New-Object Net.WebClient).DownloadString('https://raw.githubusercontent.com/itm4n/PrivescCheck/master/PrivescCheck.ps1')
Invoke-PrivescCheck -Extended -Report PrivescCheck2 -Format CSV,HTML,txt -Verbose
Results in a $ScriptPath variable with the content of the last command which is IEX (New-Object Net.WebClient).DownloadString('https://raw.githubusercontent.com/itm4n/PrivescCheck/master/PrivescCheck.ps1') in this case.
So the Plugin-Loading results in some error messages:

My changes just check for a correct path set the Current Working directory as $ScriptPath if its not found.

Greetings

Thanks for reporting this issue. I forgot to test this use case after all the changes I've made.
The plugin loading feature is essentially meant for local usage so I simply added a Test-Path check as you suggested to make sure $MyInvocation.MyCommand.Definition corresponds to a local path.
I preferred to update the script myself because of the other changes I've made in the meantime. :)
Next time you spot this kind of issue, don't hesitate to open an issue. It will be less frustrating for both of us.
You won't waste your time fixing the code yourself.  Next time i´ll open up an pull request.


Help - scipy.optimize.curve_fit doesn't accept native data and mathematical functions

For some reason, curve_fit does not accept data and fit functions in their canonical form, but insists on getting the independent data for N predictors as an NxM meshgrid called xdata, the dependent data as a flattened 1xM array, and the fit function as numpy.ravel(f(xdata, params)) instead of the native f(x1, ..., xN, *params). The burden to convert and repack all data and functions is effectively passed on to the user, which turns out to be a huge productivity killer. I would therefore like to write a short function that does the job for me (taken from a suggestion on Stack Overflow):
def repack_for_curve_fit(f, xdata):
    '''
    This function unpacks the NxM-array with independent data for N predictors into N separate 1xM-arrays,
    passes them to the fit function as the first N arguments, and returns the flattened fit function for
    further processing with the scipy.optimize.curve_fit package.

    Parameters
    ----------
    f : callable
        N-dimensional fit function, f(x1, ..., xN, *params), must take N independent variables as first N
        arguments and the fit parameters as remaining arguments
    xdata : array_like
        NxM-array with independent data for N predictors
    '''
    import numpy as np
    return np.ravel( f(*(x.reshape(1, -1) for x in xdata)) )

Unfortunately, this throws an error and it's not clear where it comes from:
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-39-51aaf66438c3> in <module>()
     26     # Do nonlinear curve fit using scipy.optimize.curve_fit, write fit results to TXT file
     27     params_init = (-7, -7, 0, 7)    # Specify initial guesses for fit parameters (default is 0)
---> 28     params_opt, params_cov = curve_fit(repack_for_curve_fit(fit_function, xyGrid), xyGrid, imgArrayFlattened, params_init)    # Run the fit, return optimized parameters and covariance matrix
     29     params_err = np.sqrt(np.diag(params_cov))    # Standard deviation of optimized parameters
     30     print(params_opt)

/home/davidra/Desktop/repack_for_curve_fit.py in repack_for_curve_fit(f, xdata)
     14     '''
     15     import numpy as np
---> 16     return np.ravel( f(*(x.reshape(1, -1) for x in xdata)) )

TypeError: fit_function() missing 4 required positional arguments: 'x0', 'y0', 'Imax', and 'FWHM'

What is the issue here?
PS.: This is the fit function used above, it's well-defined and proven to work with other curve-fitting packages:
def fit_function(x, y, x0, y0, Imax, FWHM):
    '''
    Parameters
    ----------
    x, y : float
        X and Y coordinates
    x0 : float
        X offset
    y0 : float
        Y offset
    Imax : float
        Peak intensity
    FWHM : float
        Full width at half maximum
    '''
    import numpy as np

    return Imax * np.e * 4 * np.log(2) * ((x+x0)**2 + (y+y0)**2) / FWHM**2 * np.exp(-4 * np.log(2) * ((x+x0)**2 + (y+y0)**2) / FWHM**2)

Hello, is anybody maintaining still SciPy?

Yes, but we don't guarantee an immediate response to every new issue. In this case, you are asking a question about a bug in your own code rather than presenting a bug in scipy, even though your code is related to a feature that you would like in scipy. Please ask such coding questions on the scipy-user mailing list. You are more likely to get a helpful response from the wider community than just the scipy maintainers who are looking at Github issues for triaging scipy bugs.
I don't know who suggested that repack_for_curve_fit() function, but look at what you are actually passing to the f() call; just the unpacked xydata, not the parameters that you are trying to fit. These are the arguments that the error message is complaining about. Note that you are calling repack_for_curve_fit() before curve_fit() executes. Instead of a function that calls your fit_function(), you want a function that wraps fit_function() and returns that wrapper function. Something like the following might work:
def wrap_for_curve_fit(f):
    def g(xdata, params):
        return f(*xdata, *params)
    return g

I'm closing this issue because it is primarily a coding question. If, after discussing the issue on the scipy-user mailing list, you want to make a specific proposal for changing or adding to scipy, please go ahead and open a new issue or PR with the feature request. Thanks!

Thanks!!!!!!!
Gotta read about wrappers, this concept is new to me...

In your solution,
def wrap_for_curve_fit(f):
    def g(xdata, params):
        return f(*xdata, *params)
    return g

won't doing *xdata "explode" the NxM-array down to its single elements? I already tried star expressions on nested NumPy arrays and this does not seem to work. E.g. with an NxM-array, starring doesn't return N separate length-M arrays, but N times M separate arrays of length 1.

As the error message says, it takes 2 arguments xdata and params, but 5 were given.

Sorry, forgot that curve_fit() does expand the params vector. It should be easy enough to fix, though, with one more * in the g() function signature:
def wrap_for_curve_fit(f):
    def g(xdata, *params):
        return f(*xdata, *params)
    return g

won't doing *xdata "explode" the NxM-array down to its single elements?

No, just the first axis (N values of M-vectors). If you are getting different results, please show an example.
[~]
|5> def f(x, y):
..>     print(x)
..>     print(y)
..>

[~]
|6> xy = np.arange(6).reshape((2, 3))

[~]
|7> xy
array([[0, 1, 2],
       [3, 4, 5]])

[~]
|8> f(*xy)
[0 1 2]
[3 4 5]

And you don't have to pack xdata into a single array. Any object suffices. You could use a tuple (x, y) of the x and y arrays that you would pass to fit_function() when calling it directly.

@rkern: Thank you so much for your quick help!
Possibly I'm just having trouble understanding the curve_fit documentation right. In the docs it says:
Parameters:

    f : callable
        The model function, f(x, …). It must take the independent variable as the first argument and the parameters to fit as separate remaining arguments.

    xdata : array_like or object
        The independent variable where the data is measured. Should usually be an M-length sequence or an (k,M)-shaped array for functions with k predictors, but can actually be any object.

    ydata : array_like
        The dependent data, a length M array - nominally f(xdata, ...).

f : The model function "must take the independent variable as the first argument". => So a tuple (e.g. (x, y)) also counts as a single element? Also, in all the related Stack Overflow/Stack Exchange code examples, the function is flattened using numpy.ravel() before being passed to curve_fit. Is this really necessary? See also docs for ydata below...
xdata: The argument "can actually be any object". => As you already mentioned, a tuple (x, y) also works. But then, what are the rules/limitations for the independent data, can it really be just any object as long as it has the same data structure as the dependent data (i.e. a one-to-one correspondence between coordinates and values is preserved)?
ydata: It says, that the dependent data must be "a length M array". => Same as with the model function f (see above), is it correct that this needs to be flattened before it's passed to curve_fit?

xdata doesn't have to have any specific structure to it at all. It just needs to be the case that f(xdata, *params) returns an array the same shape as ydata. Typically, some aspect of xdata is of length M to match ydata, but it really just depends on how you want to write your function. In your case, you seem to want to work with a fit_function() such that ydata = fit_function(x, y, *params), so I would choose to have all x, y, and ydata to be of shape (M,). If you are evaluating this on a meshgrid, you might need to flatten these. I don't know what examples you are looking at, but presumably that's why they are calling ravel(). That's not intrinsic to curve_fit(), per se, just that they were deriving their xdata from meshgrid arrays. It's easiest, in my mind, to do the ravel()ing outside of the function or the wrapper.
xgrid, ygrid = np.meshgrid(np.arange(3), np.arange(5))
zgrid = np.zeros_like(xgrid)
assert xgrid.shape == ygrid.shape == zgrid.shape == (3, 5)
x = xgrid.ravel()
y = ygrid.ravel()
ydata = zgrid.ravel()
xdata = (x, y)
curve_fit(wrap_for_curve_fit(fit_function), xdata, ydata, ...)

Maybe, just to explain and motivate why I'm bothering you with all these questions...up until now, I always had to do this lengthy repacking process even with rather simple data. E.g. this is what I'm currently doing in order to feed curve_fit some simple gray value images:
# Open image file
img = Image.open(inputPath + filename)

# Repack independent variables into 2xM-array for scipy.optimize.curve_fit
width, height = img.size
xRange = np.arange(1, width+1)
yRange = np.arange(1, height+1)
xGrid, yGrid = np.meshgrid(xRange, yRange)
xdata = np.vstack((xGrid.ravel(), yGrid.ravel()))

# Flatten 2D pixel data into 1xM-array for scipy.optimize.curve_fit
imgArray = np.array(img)
imgArrayFlattened = imgArray.ravel()

# Then comes the wrapper for the model function itself...
wrapped_model_function = wrapper_for_curve_fit(model_function)

# ...which also needs to be flattened (I think)
flattened_wrapped_model_function = wrapped_model_function.ravel()

# NOW I can start fitting
params_init = (-7, -7, 0, 7)
params_opt, params_cov = curve_fit(flattened_wrapped_model_function, xdata, imgArrayFlattened, params_init)

Whoops, you were faster...

Looking at the function that you gave, you would not have to do anything like flattened_wrapped_model_function. Just flatten the inputs and ydata and fit_function() will return an array of an appropriate shape.

Just flatten the inputs

By which I mean, flatten each of xGrid and yGrid separately, and put them in a tuple.

That said, if you wanted to let the wrapper do more of the work:

def ravel_wrap(f):
    def g(xdata, *params):
        return np.ravel(f(*xdata, *params))
    return g

xGrid, yGrid = np.meshgrid(...)
ydata = imageData.ravel()  # This one still needs to be explicit.
curve_fit(ravel_wrap(fit_function), (xGrid, yGrid), ydata, ...)

xdata doesn't have to have any specific structure to it at all. It just needs to be the case that f(xdata, *params) returns an array the same shape as ydata. Typically, some aspect of xdata is of length M to match ydata, but it really just depends on how you want to write your function.

Thanks for this clarification. So xdata really can have any mathematically sensible structure, while f(xdata, *params) and ydata have to be of identical shape. What about f(xdata, *params) itself, can curve_fit handle an array-like input for both f and ydata or do they have to be flattened before being passed?

Without knowing the inner workings of how calculations are done inside Python, I imagine that having an array-like xdata as input, which gets squared inside f(xdata, *params), will result in a "flat" array. Whereas having a linear function in xdata will result in an array of the same shape as xdata - in this case, I wonder whether curve_fit will complain if the input is not flattened first...

PS.: The reason why I'm asking this is simple - if it's possible, I would just omit the ravel()ing step altogether and pass everything to curve_fit in their "native" form (i.e. as arrays or in whatever form f and ydata exist after reading in e.g. the experimental data)

curve_fit() places no restrictions on how you interpret xdata inside your function f. That's up to how you decide to write (and maybe wrap) f.
As for ydata and the return value of the (possibly-wrapped) f, yes, it must be flattened. My last example showed how you can wrap your fit_function so that only needs to be done explicitly once, when passing the ydata to curve_fit().

Hi, thank you so much for the really detailed information and help! I think I've now understood 100% how to work with curve_fit.
On a side note, it might help others if this crucial information is appended to the curve_fit docs, i.e. if you point out explicitly that ydata needs to have the same shape as f(xdata, params). This ties everything together.


Fixed #156

Fixes #156
Changes Proposed in this Pull Request:

Bug fix #156

Editor is not updated even to save the new file :write

I have written tests for: lib/ex.coffee

[ ] New features introduced
[x] Bugs fixed
[ ] Neither (I'm just enhancing tests!)

@jazzpi @LongLiveCHIEF

This doesn't quite work, it always makes :w behave like :saveas. If I'm editing a file foo, modify it and do :w bar, the editor should not be updated.

Closed in favor of #158


Modded models being rendered by Create causes the game to crash

Describe the Bug
Create mod seems to have rather specific issues when regarding modded textures, immediately crashing in some instances.
One of these instances is when trying to select a grass block with the creative world shaper item while playing with the Lambda's Better Grass mod, where it'll Immediately crash.
The crash that this report refers to, however, relates to Colorful Enchantments crashing when an enchanted book is rendered by the ponder system.
The issue above can be reproduced by a game with Create Enchantment Industry and Colorful Enchantments running alongside Create, as the default mod contains no listed ponder with the item (or so i believe)
Reproduction Steps

start game
enter ponder index
navigate to experience related ponders
ponder the enchanting guide
move to the second scene: Enchanting
watch the game immediately crash once the ponder system attempts to render the Enchantment book
view logs
restart the game
repeat the process without Colorful Enchantments to see how the ponder behaves with the vanilla model
...

Expected Result
Both the ponder Index and the creative World Shaper should render modded textures normally OR fallback to a default texture, such as the vanilla models or a template missing texture to avoid crashes
Screenshots and Videos

Normal expected behaviour
Crash reproduction

Crash Report or Log
https://mclo.gs/NfPTYak
Operating System
Ubuntu 22.04.2 LTS
Mod Version
0.5.1b
Minecraft Version
1.19.2
Other Mods

Colorful Enchantments
Create

Create Enchantment Industry

Debugify
Fabric Api

Additional Context
This issue, as pointed out by the crash logs, seems to be directly related to the mechanisms behind how Create decides to render items. Due to the nature of it, it may cause plenty of unintended mod incompatibilities that could be avoided with proper methods.

LBG issue is known: #631