Binary / byte array messages broken.

Thanks for this awesome framework. I was excited to add the binary support to our product. The problem is I can't get the basic byte[] pushed down based on the demo.
I'm using 1.7.5 (and even tried snapshot). The BinaryEventLauncher and corresponding binary-event-index.html page.

The client on 'msg' doesn't get invoked unless I use Byte[] (instead of the demo code that has byte[]) on the server side. I wouldn't think this matters but for some reason it does. SERVER SIDE BELOW

Even when the client gets called back it throws an "Uncaught RangeError: Invalid typed array length" exception when constructing the Uint8Array(file). CLIENT SIDE BELOW

SERVER SIDE
server.addEventListener("msg", Byte[].class, new DataListener<Byte[]>() {
@Override
public void onData(SocketIOClient client, Byte[] data, AckRequest ackRequest) {
            //client.sendEvent("msg", data);
            ((ClientOperations) client).sendEvent("msg", data);
        }
    });

CLIENT SIDE (Copied from demo)
socket.on('msg', function(file) {
                  var arrayBuffer = new Uint8Array(file).buffer;
                  var blob        = new Blob([arrayBuffer]);

                  var imgList = $('ul#img-list');

                  var li = $('<li/>').appendTo(imgList);
                  $('<div/>').text(file.name).appendTo(li);
                  var img = $('<img/>').appendTo(li);

                  var reader = new FileReader();
                  reader.onload = (function(aImg) {
                    return function(e) {
                      aImg.attr('src', e.target.result);
                      aImg.attr('width', 150);
                    };
                  })(img);

                  reader.readAsDataURL(blob);
	});

Update. The callback handle in the on 'msg' isn't a blob / file. when I deref the .buffer var it is "undefined". I have tried sending Packet with subType = BINARY_EVENT and for some reason my chrome browser spins out of control until it crashes.
Was this perhaps fixed as in the discussion at  Binary Support, sending binary to client #178
but somehow not rolled forward to the latest?
Thanks

try to use byte[] not Byte[]

The default code in the example uses primitive byte array and doesn't end up generating a client side callback. Also I noticed that the client side code in the binary demo has the following in the msg handler:
new Uint8Array(file).buffer
That can't be right right? It should be (file.buffer)
Either way in no scenario does the client callback happen.
Can you try the checked in demo against snapshot and advise.
Thanks

You mean binary-event example is not working at all?

Correct. Using both the latest release as well wells snapshot. There is definitely something not right on the server side. However I don't see how the client could work either given what I mentioned about calling .buffer on the Uint8Array.

Yes, it's a bug, but only for websocket transport. For longpolling it works.

it works but you should add config.setMaxFramePayloadLength(200 * 1024); if you load big images

Are you saying that websocket works if you make make that setting? I I can't get even dummy simple byte arrays to go. Else are there plans to support websocket? Thanks

The correct behavior is added image on the screen after you push 'browse' button and select image. Do you see this?

No

Which browser you used? it was tested with latest firefox and chrome

Chrome 39 and FF 29. Same thing, client on 'msg' doesn't get invoked.

i updated socketio.js lib in netty-socketio-demo maybe it's a root of the problem

You can you "turn on" the logs on the server by switching log level for "com.corundumstudio" package to "TRACE" and check does server sends packet or not

I updated the socket.js in the demo code and set the max payload size (although my test image is only 1k)...Still no client side call back
Below is the "trace" level logging. The upload of the small image is there. I don't see the packet being written. I have debugging

Thoughts?
[2015-01-12 09:20:36,100] [TRACE] [handler.InPacketHandler] [nioEventLoopGroup-3-1] In message: 451-["msg",{"_placeholder":true,"num":0}] sessionId: 6656ec2a-b31c-491b-8624-57288578938e
[2015-01-12 09:20:36,101] [DEBUG] [websocketx.WebSocket08FrameDecoder] [nioEventLoopGroup-3-1] Decoding WebSocket Frame opCode=2
[2015-01-12 09:20:36,101] [DEBUG] [websocketx.WebSocket08FrameDecoder] [nioEventLoopGroup-3-1] Decoding WebSocket Frame length=1060
[2015-01-12 09:20:36,104] [TRACE] [handler.InPacketHandler] [nioEventLoopGroup-3-1] In message: ?PNG
[RAW PNG BYTES HERE]
[2015-01-12 09:20:43,305] [TRACE] [handler.EncoderHandler] [nioEventLoopGroup-3-1] Out message: 451-["msg",{"num":0,"_placeholder":true}] sessionId: 6656ec2a-b31c-491b-8624-57288578938e
[2015-01-12 09:20:43,305] [DEBUG] [websocketx.WebSocket08FrameEncoder] [nioEventLoopGroup-3-1] Encoding WebSocket Frame opCode=1 length=41
[2015-01-12 09:20:43,306] [DEBUG] [websocketx.WebSocket08FrameDecoder] [nioEventLoopGroup-3-1] Decoding WebSocket Frame opCode=1
[2015-01-12 09:20:43,306] [DEBUG] [websocketx.WebSocket08FrameDecoder] [nioEventLoopGroup-3-1] Decoding WebSocket Frame length=1
[2015-01-12 09:20:43,306] [TRACE] [handler.InPacketHandler] [nioEventLoopGroup-3-1] In message: 2 sessionId: 6656ec2a-b31c-491b-8624-57288578938e
[2015-01-12 09:20:43,307] [TRACE] [handler.EncoderHandler] [nioEventLoopGroup-3-1] Out message: 3 sessionId: 6656ec2a-b31c-491b-8624-57288578938e
[2015-01-12 09:20:43,307] [DEBUG] [websocketx.WebSocket08FrameEncoder] [nioEventLoopGroup-3-1] Encoding WebSocket Frame opCode=1 length=1

This issue has been introduced since netty-4.0.24 version. I make some changes and update it to latest version. Now all works fine!

Good to know. I just saw that PacketEncoder.encodePacket in the finally block looks like the following:
// we need to write a buffer in any case
if (!binary) {
buffer.writeByte(0);
int length = buf.writerIndex();
buffer.writeBytes(longToBytes(length));
buffer.writeByte(0xff);
buffer.writeBytes(buf);
            buf.release();
        }

If i comment out the !binary if then the packet goes out. However the client file.buffer is empty

no, you don't need to touch this code :)

so, is it works now?

I synced your change for #191 and I get the same result. In fact if the PacketEncode.encodePacket finally block doesn't have the change where I comment out the "if (!binary) {" if statement then the packet doesn't go out at all. If I comment that out I still don't a file.buffer that has anything on the client side.

which netty version do you use now?

netty-all-4.0.24.Final.jar
This was built using the POM and thus corresponding dependent jars

For the PacketEncoder, the comment in the finally block says "we need to write a buffer in any case". However if the argument "binary" is true then the buffer isn't written and released.
That being said even when I comment it out the file.buffer is non existent.
To summarize:

I have the latest server code (with #191 applied)
I have the updated socket.io for demo project
The demo code has bug in it re: new Uint8Array(file).buffer instead of new Uint8Array(file.buffer). Even when that is corrected the "file" ref doesn't have a buffer

Thanks for looking at this

@jimmyryan15
I have changed var arrayBuffer = new Uint8Array(file).buffer; to your code var arrayBuffer = new Uint8Array(file.buffer); and my code stops to work too. Please revert this change and try again

Interesting. If I
i) Apply the #191 change you posted
ii) I comment out the PacketEncoder.encodePacket finally block re: binary if
iii) Use the "var arrayBuffer = new Uint8Array(file).buffer" line in the client then the image on the client DOES work!

B) My chrome browser is showing the following errors (even though the image shows up now) which in turn disconnects the socket.io. Note: no exception is thrown on the server side.
POST http://<IP_ADDRESS>:9092/socket.io/?EIO=3&transport=polling&t=1421089896057-7&sid=d646898f-9780-4ed8-a07b-d42fd1575d2c 500 (Internal Server Error)
POST http://<IP_ADDRESS>:9092/socket.io/?EIO=3&transport=polling&t=1421089896065-8&sid=d646898f-9780-4ed8-a07b-d42fd1575d2c 500 (Internal Server Error)

500 (Internal Server Error)

Is there any server error in logs?

There aren't any exceptions that I see in trace level logging (per my previous NOTE: no exception is thrown on the server side). That being said I don't doubt that the browser is seeing the 500's but they must be happening at some internal layer (perhaps inside netty).
I'm sorry but I can't spend any more time on this. I'm going to leave the messages as base64 encoded and handle as String because I know that it works. I spent 4 days troubleshooting this and obviously there are plumbing level issues and the binary example doesn't work w/o changing the PacketEncoder.encodePacket finally block for me and even then I'm now getting the 500 exceptions that break the websocket.
I don't understand how yours would possibly work w/o the PacketEncoder writing the bytes. The change you made is based on a breaking change in netty but your POM file imports that version of netty!
If you can get the main and demo project to a place where the binary event works out of the box then please update here and I'll try it again.
I think your project is great and clearly you are attentive but I must say that I get the sense you aren't testing on the exact same context avail to us in the repos.

If you do figure out how to make this repeatably work please update the POM so that all dependencies are set to what you are using. I just realized that you said "This issue has been introduced since netty-4.0.24 version. I make some changes and update it to latest version. Now all works fine!"
Does this mean that we need the snapshot of netty? etc. etc.
It would be very clear if the project specifically declared what was needed.
Thanks

I must say that I get the sense you aren't testing on the exact same context avail to us in the repos.

Sorry for that. I tested demo with netty-4.0.23 in classpath
Here is my netty-socketio-demo project. https://yadi.sk/d/jBhpOBAddw9Dy

build latest netty-socketio 1.7.6-SNAPSHOT
build my netty-socketio-demo
run mvn exec:java -Dmain.class=com.corundumstudio.socketio.demo.BinaryEventLauncher
open the netty-socketio-demo\client\binary-event-index.html page in your browser

Now we are talking. I'll try this in the morning and see what we get. Thanks

have you try it?


Add UTM query parameters to GIT links

Context
The Get Into Teaching service would like to know more about the candidates that click on the links to their service from Apply. This would help their Google analytics.
Changes proposed in this pull request

Four UTM parameters - utm_source, utm_medium, utm_campaign, utm_content

An example set of query params would be:
From application page:
?utm_source=apply-for-teacher-training.service.gov.uk&utm_medium=referral&utm_campaign=candidate_interface/degrees/degree-new_enic& utm_content=apply_1
From email:
?utm_source=apply-for-teacher-training.service.gov.uk&utm_medium=referral&utm_campaign=eoc_deadline_reminder& utm_content=apply_1

Methods in view helper - different behaviour expected from links in emails vs links on application page

Guidance to review
Commit by commit.
Link to Trello card
https://trello.com/c/WFeDKbfZ/609-programmatically-tag-up-our-enyml-file-with-utms
Things to check

[ ] If the code removes any existing feature flags, a data migration has also been added to delete the entry from the database
[ ] This code does not rely on migrations in the same Pull Request
[ ] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[ ] If this code adds a column to the DB, decide whether it needs to be in analytics yml file or analytics blocklist
[ ] API release notes have been updated if necessary
[ ] Required environment variables have been updated added to the Azure KeyVault

 Programmatically tag up our en.yml file with UTMs

It looks like this is automatically including the request.path within the UTM params sent to Google Analytics? Whilst mostly these are consistent across all candidates (things like /candidate/application/personal-statement/edit) there’s a few places where we use an ID, eg /candidate/application/restructured-work-history/edit/12345. I don’t think we currently link to GIT from any of these pages, but if we did it’d make the data harder to analyse in Google Analytics (as they'd all be unique) and would potentially identify the candidate (when I think GA is meant to be anonymous?)
Don't think this a blocker, but something to consider whenever we add links to GIT in future?

Don't think this a blocker, but something to consider whenever we add links to GIT in future?

Yes this is a good point. Currently the utm_campaign method strips /candidate/application to shorten the url sent to GIT so in theory it can be amended to strip that id out as well (if this approach is sound).

Looking good, I think. Do we need to think about what happens in non-production environments? I guess UTM links should really be production-only.

Looking good, I think. Do we need to think about what happens in non-production environments? I guess UTM links should really be production-only.

Yes agreed, I've made them prod only in 3150316

@JonF1 Thanks for this - looks good!
We would definitely need to make sure that we can't identify anyone from the campaign parameter as that would go against GA terms, so stripping out ids is definitely needed.
I've had a chat with @MylesJarvis and we both think that putting / in the UTM campaign parameter might be problematic - might be better using unicode instead?
Would also be good to get our PA Shivani to have a look at this as well to make sure it all makes sense from a GA perspective - I'll ask her to review. We also might be able to test end to end (or it might be as easy to deploy and check in real-time GA).

@JonF1 Thanks for this - looks good!

Thanks Gemma, I've pushed up changes to encode the forward slashes, should be ready for Shivani to review. At the moment the utm params will only show in production, but I can change it to show locally if needed for the review.

Hi @JonF1 all looks good from what I can see,  When its rolled out I can go into GA QA the integration.


Add topics to the github repo

Nice new feature

Completely forgot about this until now haha. Added a few and was surprised to see that a bart-api topic already exists!


MPI naming conflicts for temporary files

I am running MMSeqs2 (the MPI version) on a cluster.
My batch script call looks like this:
RUNNER="mpirun --pernode" mmseqs search query_db target_db result_db tmp
An error occurs at the end of the prefiltering step. This is the end of the output file:
...
Starting prefiltering scores calculation (step 1 of 1)
Query db start  1 to 1370
Target db start  1 to 8162

0 k-mers per position.
32539 DB matches per sequence.
0 Overflows.
160 sequences passed prefiltering per query sequence.
Median result list size: 149
0 sequences with 0 size result lists.

Time for prefiltering scores calculation: 0h 0m 0s 776ms
Time for merging files: 0h 0m 0s 9ms
Could not move result tmp/14349913440832127037/pref_tmp_0 to final location tmp/14349913440832127037/pref!
--------------------------------------------------------------------------
mpirun has exited due to process rank 0 with PID 134942 on
node node218.cluster exiting improperly. There are two reasons this could occur:

1. this process did not call "init" before exiting, but others in
the job did. This can cause a job to hang indefinitely while it waits
for all processes to call "init". By rule, if one process calls "init",
then ALL processes must call "init" prior to termination.

2. this process called "init", but exited without calling "finalize".
By rule, all processes that call "init" MUST call "finalize" prior to
exiting or it will be considered an "abnormal termination"

This may have caused other processes in the application to be
terminated by signals sent by mpirun (as reported here).

It looks like the filenames in my tmp/ directory are the issue.
The list of files in tmp/14349913440832127037/ is
pref.index_tmp_0
pref_tmp_0.0
pref_tmp_0.1
pref_tmp_0.2
pref_tmp_0.3
pref_tmp_0.4
pref_tmp_0.5
pref_tmp_0.6
pref_tmp_0.7
pref_tmp_0.dbtype
searchtargetprofile.sh

This suggests that each thread (there are 8 threads) is saving a temporary file with its own suffix and mmseqs can't find the temporary filename it is looking for.
Have you had this problem before? Is there any other possible explanation or something I can try on my end?

Can you post the full output please?

+ '[' -z '' ']'
+ case "$-" in
+ __lmod_vx=x
+ '[' -n x ']'
+ set +x
Shell debugging temporarily silenced: export LMOD_SH_DBG_ON=1 for this output (/opt/lmod/lmod/init/bash)
Shell debugging restarted
+ unset __lmod_vx
+ RUNNER='mpirun --pernode'
+ mmseqs search query_dbs/1000562_0 mmseqs_dir/profile/Tenericutes_profile mmseqs_dir/search/1000562_0_search_Tenericute tmp
Program call:
search query_dbs/1000562_0 mmseqs_dir/profile/Tenericutes_profile mmseqs_dir/search/1000562_0_search_Tenericute tmp

MMseqs Version:                                                                 da6c1a5a7be5d82719f796b85ce9cbacf9f2162c-MPI
Sub Matrix                                                                      blosum62.out
Add backtrace                                                                   false
Alignment mode                                                                  2
E-value threshold                                                               0.001
Seq. Id Threshold                                                               0
Min. alignment length                                                           0
Seq. Id. Mode                                                                   0
Alternative alignments                                                          0
Coverage threshold                                                              0
Coverage Mode                                                                   0
Max. sequence length                                                            65535
Max. results per query                                                          300
Compositional bias                                                              1
Realign hit                                                                     false
Max Reject<PHONE_NUMBER>
Max Accept<PHONE_NUMBER>
Include identical Seq. Id.                                                      false
Preload mode                                                                    0
Pseudo count a                                                                  1
Pseudo count b                                                                  1.5
Score bias                                                                      0
Gap open cost                                                                   11
Gap extension cost                                                              1
Threads                                                                         8
Compressed                                                                      0
Verbosity                                                                       3
Sensitivity                                                                     5.7
K-mer size                                                                      5
K-score<PHONE_NUMBER>
Alphabet size                                                                   21
Offset result                                                                   0
Split DB                                                                        0
Split mode                                                                      2
Split Memory Limit                                                              0
Diagonal Scoring                                                                1
Exact k-mer matching                                                            0
Mask Residues                                                                   1
Minimum Diagonal score                                                          15
Spaced Kmer                                                                     1
Spaced k-mer pattern
Local temporary path
Rescore mode                                                                    0
Remove hits by seq.id. and coverage                                             false
Sort results                                                                    0
In substitution scoring mode, performs global alignment along the diagonal      false
Mask profile                                                                    1
Profile e-value threshold                                                       0.1
Use global sequence weighting                                                   false
Filter MSA                                                                      1
Maximum sequence identity threshold                                             0.9
Minimum seq. id.                                                                0
Minimum score per column                                                        -20
Minimum coverage                                                                0
Select n most diverse seqs                                                      1000
Omit Consensus                                                                  false
Min codons in orf                                                               30
Max codons in length                                                            32734
Max orf gaps<PHONE_NUMBER>
Contig start mode                                                               2
Contig end mode                                                                 2
Orf start mode                                                                  1
Forward Frames                                                                  1,2,3
Reverse Frames                                                                  1,2,3
Translation Table                                                               1
Use all table starts                                                            false
Offset of numeric ids                                                           0
Add Orf Stop                                                                    false
Number search iterations                                                        1
Start sensitivity                                                               4
Search steps                                                                    1
Run a seq-profile search in slice mode                                          false
Strand selection                                                                1
Disk space limit                                                                0
Sets the MPI runner                                                             mpirun --pernode
Force restart using the latest tmp                                              false
Remove Temporary Files                                                          false

[node218.cluster:134942] mca: base: component_find: unable to open /usr/mpi/gcc/openmpi-1.6.5/lib/openmpi/mca_mtl_psm: libpsm_infinipath.so.1: cannot open shared object file: No such file or directory (ignored)
MPI Init...
Rank: 0 Size: 1
Program call:
prefilter query_dbs/1000562_0 mmseqs_dir/profile/Tenericutes_profile tmp/14349913440832127037/pref --sub-mat blosum62.out -s 5.7 -k 5 --k-score<PHONE_NUMBER> --alph-size 21 --max-seq-len 65535 --max-seqs 300 --offset-result 0 --split 0 --split-mode 2 --split-memory-limit 0 -c 0 --cov-mode 0 --comp-bias-corr 1 --diag-score 1 --exact-kmer-matching 0 --mask 1 --min-ungapped-score 15 --add-self-matches 0 --spaced-kmer-mode 1 --db-load-mode 0 --pca 1 --pcb 1.5 --threads 8 --compressed 0 -v 3

MMseqs Version:                 da6c1a5a7be5d82719f796b85ce9cbacf9f2162c-MPI
Sub Matrix                      blosum62.out
Sensitivity                     5.7
K-mer size                      5
K-score<PHONE_NUMBER>
Alphabet size                   21
Max. sequence length            65535
Max. results per query          300
Offset result                   0
Split DB                        0
Split mode                      2
Split Memory Limit              0
Coverage threshold              0
Coverage Mode                   0
Compositional bias              1
Diagonal Scoring                1
Exact k-mer matching            0
Mask Residues                   1
Minimum Diagonal score          15
Include identical Seq. Id.      false
Spaced Kmer                     1
Preload mode                    0
Pseudo count a                  1
Pseudo count b                  1.5
Spaced k-mer pattern
Local temporary path
Threads                         8
Compressed                      0
Verbosity                       3

Initialising data structures...
Using 8 threads.
Could not find precomputed index. Compute index.
Touch data file mmseqs_dir/profile/Tenericutes_profile ... Done.
Substitution matrices...
Substitution matrices...
Use kmer size 5 and split 1 using Query split mode.
Needed memory<PHONE_NUMBER> byte) of total memory<PHONE_NUMBER>16 byte)
Target database: mmseqs_dir/profile/Tenericutes_profile(Size: 8162)
Index table k-mer threshold: 90
Index table: counting k-mers...

Index table: Masked residues: 0
Index table: fill...

Index table: removing duplicate entries...
Index table init done.

DB statistic
Entries:         293010879
DB Size:<PHONE_NUMBER> (byte)
Avg Kmer Size:   71.7443
Top 10 Kmers
        KKKKK           1557
        KKKNK           1519
        KKKKN           1416
        NNNNN           1398
        KKKNN           1359
        KNKNN           1349
        NKNNN           1320
        KNNNN           1306
        KKNKN           1210
        NNINN           1188
Min Kmer Size:   0
Empty list: 884115

Time for index table init: 0h 1m 32s 319ms
Query database type: Aminoacid
Target database type: Profile
Time for init: 0h 1m 32s 393ms
Query database: query_dbs/1000562_0(size=1370)
Process prefiltering step 1 of 1

k-mer similarity threshold: 90
k-mer match probability: 0

Starting prefiltering scores calculation (step 1 of 1)
Query db start  1 to 1370
Target db start  1 to 8162

0 k-mers per position.
32539 DB matches per sequence.
0 Overflows.
160 sequences passed prefiltering per query sequence.
Median result list size: 149
0 sequences with 0 size result lists.

Time for prefiltering scores calculation: 0h 0m 0s 776ms
Time for merging files: 0h 0m 0s 9ms
Could not move result tmp/14349913440832127037/pref_tmp_0 to final location tmp/14349913440832127037/pref!
--------------------------------------------------------------------------
mpirun has exited due to process rank 0 with PID 134942 on
node node218.cluster exiting improperly. There are two reasons this could occur:

1. this process did not call "init" before exiting, but others in
the job did. This can cause a job to hang indefinitely while it waits
for all processes to call "init". By rule, if one process calls "init",
then ALL processes must call "init" prior to termination.

2. this process called "init", but exited without calling "finalize".
By rule, all processes that call "init" MUST call "finalize" prior to
exiting or it will be considered an "abnormal termination"

This may have caused other processes in the application to be
terminated by signals sent by mpirun (as reported here).
--------------------------------------------------------------------------
Error: Prefilter died

Something seems to be wrong with your MPI setup:
MPI Init...
Rank: 0 Size: 1

This should show the number of nodes it runs on and its currently one. So its running separate non-connected MPI process on each node that overwrite each others results.
Maybe its connected to this warning:
[node218.cluster:134942] mca: base: component_find: unable to open /usr/mpi/gcc/openmpi-1.6.5/lib/openmpi/mca_mtl_psm: libpsm_infinipath.so.1: cannot open shared object file: No such file or directory (ignored)

I am afraid we can't really help you with MPI issues since they are usually very specific to the local environment.

Okay thanks. I'll follow up with the sys admins and see if they have any leads.

If your cluster has a module system, use that MPI and make sure no MPI packages from the OS package manager are installed. Thats one common problem.

I will close this issue for now. Please reopen if you still encounter problems


Out of order properties

I have the following configuration, However, I observe that my view the sections are coming in the wrong order. Can you give me any suggestions for a fix.
schema:{
            "title": "NGLA Schema",
            "type": "object",
            "options":{
                        "disable_collapse": true
                    },
            "properties":{
                 "project": {
                      "title": "Project",
                      "$ref": "#/definitions/project"
                  },
                 "sources": {
                      "title": "Source",
                      "$ref": "#/definitions/source"
                  },
                  "type1": {
                      "title": "Pattern Recognition",
                      "$ref": "#/definitions/type1"
                  },
                  "type2": {
                      "title": "Type 2 Parameters",
                      "$ref": "#/definitions/type2"
                  },
                  "type3": {
                      "title": "Sequence Analysis",
                      "$ref": "#/definitions/type3"
                  },
                  "type4": {
                      "title": "Volume Analysis",
                      "$ref": "#/definitions/type4"
                  },
                  "type5": {
                      "title": "Type 5 Analysis",
                      "$ref": "#/definitions/type5"
                  },
                  "type6": {
                      "title": "Type 6 Parameters",
                      "$ref": "#/definitions/type6"
                  },
                  "type7": {
                      "title": "Type 7 Analysis",
                      "$ref": "#/definitions/type7"
                  },
                  "type8": {
                      "title": "Type 8 Analysis",
                      "$ref": "#/definitions/type8"
                  },
                  "type9": {
                       "title": "Type 9 Analysis",
                       "$ref": "#/definitions/type9"
                  }
            },

The problem seems to come when I introduce an empty property. i.e. one in which I just want a section and no actual key-values to be entered:
"type5":
                {
                    "id": "type5",
                    "type": "object",
                    "format": "grid",
                    "options":{
                        "disable_collapse": true
                    }
                },

I am facing the same issue. The order of the view is not matching the schema. Any help !

Have you tried implementing property ordering?
https://github.com/jdorn/json-editor#property-ordering

Property ordering works but one would have liked the order to be same the either the schema order or the json order. Is it possible to do that ?

having the same issue, I have large schema, don't wanna use propertyOrder redundantly, since the order is expected to be the same as in schema..


Fix function signature

Fixing https://github.com/gijzelaerr/sqlalchemy-monetdb/issues/6
Hope this helps

ok, not sure if those failing tests are new but I'll just accept this PP.


Explore adding fuzzing tests in the SDK

TBD.

You can add this derive into fuel-types under the test-helpers feature, and it will be automatically propagated in all other crates because we use this feature.


Strange result

Hi,
My system is Ubuntu, when I run ./main, I got this result:

Unlike the paper's results, there are no trees or fire. Does this project achieve just a little part of what is mentioned in the paper?

Yes, unfortunately that's not the full implementation 😩
Le jeu. 20 juin 2019 à 03:55, zldtll1<EMAIL_ADDRESS>a écrit :

Hi,
My system is Ubuntu, when I run ./main, I got this result:
[image: Selection_001]
https://user-images.githubusercontent.com/50659824/59812327-22a5ec00-9340-11e9-9464-fb57d6a41db3.png
Unlike the paper's results, there are no trees or fire. Does this project
achieve just a little part of what is mentioned in the paper?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/art049/InteractiveWoodCombustion/issues/3?email_source=notifications&email_token=ADXKX2KOMTZKZMKIEJDXNXTP3LPQ3A5CNFSM4HZOUFO2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4G2RCFFA,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADXKX2JRFIZHLYZWDFP6XRDP3LPQ3ANCNFSM4HZOUFOQ
.

Ok, Thank you anyway.


Python agent and rust agent differently parse revocation scripts in action list

Description
There is a issue with parsing a revocation scripts in action list. Python keylime agent parse differently two and more scripts in action_list, than python rust agent. Python agent parse each script with comma and rust agent parse scripts by lines.
Expected behavior vs. actual behavior
Revocation scripts will be parse by same way for both type of agents vs. revocations scripts are parsed differently for each type of agents. From my point of view it should be parsed scripts by lines in action list file.
Steps to reproduce problem

Install tmt via sudo dnf install -y tmt-all
Clone specific branch where is test scenario for different algorithm git clone --branch script_sudo https://github.com/Koncpa/keylime-tests.git
Run test scenario with needed setup tasks, need to be inside cloned repo dir: tmt run -vvv plan --name distribution-keylime-tests-github-ci prepare discover -h fmf provision --how virtual -i Fedora-36 -c system execute --how tmt --interactive login finish

Relevant logs

Aug 24 08:28:04 ip-172-31-24-208.us-east-2.compute.internal keylime_agent[4559]: 2022-08-24 08:28:04.921 - keylime.cloudagent - WARNING - Exception during execution of revocation action local_action_modify_payload
Aug 24 08:28:04 ip-172-31-24-208.us-east-2.compute.internal keylime_agent[4559]: local_action_sudo_script: No module named 'local_action_modify_payload\nlocal_action_sudo_script'

ANOTHER LOGS

@Koncpa I think this should be moved to the rust-keylime repo as it seems to be a bug on their side. It's fine if the new agent does both cases (comma or newline) but it at least needs to support the old way.

Sure, I will close here and open new issue in rust.


Dynamic style model

Anders Sørby 15th of November 2018: Dynamic style model that lets us switch to a Halloween theme (for example) easily. Would be pretty cool.

Basically did this in #857
Not really a dynamic style model, but cool easter egg tho


refresh

Subject: refresh
Description:
refresh
Results:
Backend version: develop-7c33e18d
Frontend version:
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
Page: https://couchers.org/profile/edit/about
User: doruk (20006)

?


User should be able to pick a connection using the ADS connection dialog from Schema Compare

Repro:

Start new instance of ADS
Without opening a connection, launch Schema Compare (e.g. via database project)
Select Database as a source/target type

Expected: prompted to open a new connection
Reality: Get message saying no active connections, and have to exit dialog in order to connect
Note: still should be able to connect to a new connection even if the list isn't empty.

Looks like this is a duplicate of https://www.github.com/microsoft/azuredatastudio/issues/5222


[netdata] add new API otNetDataGetBorderRouterCount()

This API returns the total number of BRs added in network data, including both router and child roles.

Thanks @zesonzhang. The API in this PR can work, but in an earlier PR #10485, I added a new mechanism to not only track peer BRs but also their age. That PR implements public APIs for getting the list of peer BRs or their count, along with related CLI commands. It also include a test case validating the APIs.

Thanks @zesonzhang. The API in this PR can work, but in an earlier PR #10485, I added a new mechanism to not only track peer BRs but also their age. That PR implements public APIs for getting the list of peer BRs or their count, along with related CLI commands. It also include a test case validating the APIs.

Thanks @abtink , https://github.com/openthread/openthread/pull/10485 is great! I will close this one


Update Traction to 0.10.2

Image update, any library updates, test.

Note: Main branch (and dev env) now are running 0.10.2rc0 and that has been tested through.


DatePicker and TimePicker does not work on iOS?

Sorry if I totally missed something fundamental of Ace and have done something really wrong here. But as far I can't see that the DatePicker nor the TimePicker UI component works as expected on iOS?
Problem 1:
No time/date is reported back when time is changed.
Steps to reproduce:
Run the following code:
var p = new ace.Popup();
var stackPanel = new ace.StackPanel()
var dt = new ace.DatePicker();
stackPanel.getChildren().add(dt);
p.setContent(stackPanel)
p.show();

Then change the the date in the date picker.
Result:
dt.getDate() now returns "Invalid date".
After trying to figure out why i think this may be because no date is reported back from the native component.
In src/ios/DatePicker.mm it seams to me like nil is sent back to the JavaScript and not the actual date as it is in src/android/framework/DatePicker.java
src/ios/DatePicker.mm
    if (_dateChangedHandlers > 0) {
        [OutgoingMessages raiseEvent:@"datechanged" handle:_handle eventData:nil];
    }

src/android/framework/DatePicker.java
 void raiseDateChangedEvent(String dateString) {
    if (_dateChangedHandlers > 0) {
      OutgoingMessages.raiseEvent("datechanged", _dateChangedHandle, dateString);
    }
  }

Problem 2
As far as I can see the .setDate/.setTime doesn't work either.
Steps to reproduce:
Run the following code:
var p = new ace.Popup();
var stackPanel = new ace.StackPanel()
var dt = new ace.DatePicker();
stackPanel.getChildren().add(dt);
p.setContent(stackPanel)
p.show();
var d = new Date();
d.setDate(d.getDate()+10)
dt.setDate(d)

Result:
The date in the date picker is showing wrong date.
Don't have a clue why this doesn't work.

Thank you so much for the report!  I've just committed fixes for both issues, and I've published a new version with them included (0.0.17).


MGT1: Remove set_metric_as_preference deviation

Remove set_metric_as_preference deviation as it's no longer needed.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 55.5%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1983

Relevant Lines:
3573

💛  - Coveralls


Leave the power on by default

Upon completing discovery, leave the power for the
nodes on in order to support the eventual reboot skip
that will be possible once
https://review.openstack.org/#/c/635996/ has merged.

Build SUCCESS, see build http://<IP_ADDRESS>:8080/job/dev-tools/113/


chunk requests to broker during regional analysis enqueue; #38

Chunk broker requests in batches of 1000 (arbitrarily chosen, maybe there's a better size for this) to avoid memory issues during RegionalAnalysisManager::enqueue.

@landonreed How many requests were there in the instance it ran out of memory for you? HTTP requests internally are basically free, a lower max number with more requests might not be a bad idea.

Ooops. Checking out that ticket :)

This looks like the right approach to me. Sounds like you've seen it work locally, so we should try it out on staging.
Meta-note: it looks like including the issue number addressed by a pull request in the pull request title does not create a useable link. We should put it in the pull request description instead.


No configuration file "config.ini"

Fought for the last five hours with this error. Why? (It is still not working)
Raspberry Pi 4b, trying to run it on Docker.
(it's in the services/miflora-mqtt-deamon folder)
docker run -it --name miflora-mqtt-daemon -v /home/pi/IOTstack/services/miflora-mqtt-deamon:/config miflora-mqtt-daemon
Is where i quit for today. Please help!

daemon 🤢😭


Zooming editor view causes rendering issues with code annotations

VSCode Version: 1.7.1
OS Version: Windows 10 x64

Steps to Reproduce:

Open any C# project
Wait for intellisense/annotations to appear (ie, "references" annotation on class fields, methods, etc)
Zoom editor view (Ctrl++)

Result:
Annotations appear off to the left in the margin, instead of at the proper locations. They're still clickable out there, so it's functional.
See picture http://i.imgur.com/pUsklJd.png
I couldn't figure out if this was new, or not (there seems to be a variety of zoom issues already) so apologies in advance  if this is a dupe.

Closing as duplicate of #15389. The fix is already in the insiders build. You can use the insiders build until the next VS Code update


adding accessibilityLabelledBy to documentation

fixes https://github.com/facebook/react-native/issues/30846
was build by https://github.com/grgr-dkrk with PR https://github.com/facebook/react-native/pull/32470
Related https://github.com/facebook/react-native/pull/34371

@Simek Thanks. I'll add it to 0.68 next week. I have a couple of high-priority tasks now.


Adding global custom fields to access encoder.

LogstashAccessEncoder did not support global custom fields (see issue #139). This patch copies the logic in LogstashFormatter into LogstashAccessFormatter, making use of a GlobalCustomFieldsJsonProvider<IAccessEvent>. The following configuration then works:
<encoder class="net.logstash.logback.encoder.LogstashAccessEncoder">
    <customFields>{"foo": "bar"}</customFields>
</encoder>

Thanks for the contribution!


Segmentation

Hi
I want to know how can I use the segmentation feature?
Thanks

Read https://github.com/alumae/gst-kaldi-nnet2-online:

Full results can now inlude word alignment information for the best hypothesis. In order to activate this, set the word-boundary-file property to point to the word_boundary.int file typically found in the data/lang/phones/ subdirectory of a trained system.


Missing information about the default expiration policy

Hi,
in the remarks section is mentioned:

Links created using this action do not expire unless a default expiration policy is enforced for the organization.

What is the "default expiration policy", please add a link or info where I can find any info about it or explain it here. Does it have any relation to the expirationDateTime setting?

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

ID: 537290c1-f301-4874-d790-740fde5239d3
Version Independent ID: 31ddb53d-5b63-2d8e-06c3-e56125d22937
Content: driveItem: createLink - Microsoft Graph beta
Content Source: api-reference/beta/api/driveitem-createlink.md
Product: sharepoint
Technology: microsoft-graph
GitHub Login: @JeremyKelley
Microsoft Alias: MSGraphDocsVteam

As part of a repository clean up effort we are closing older issues. If this issue remains, please: open a new issue, reference this issue, and provide any additional details that may help in resolution. Thank you for your understanding as we work to improve our responsiveness.


MINIFI-156 - Windows script fixes from NiFi

Thank you for submitting a contribution to Apache NiFi - MiNiFi.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[x] Does your PR title start with MINIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[x] Has your PR been rebased against the latest commit within the target branch (typically master)?

[x] Is your initial contribution a single, squashed commit?

For code changes:

[x] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi-minifi folder?
[x] - N/A - Have you written or updated unit tests to verify your changes?
[x] - N/A - If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[x] - N/A - If applicable, have you updated the LICENSE file, including the main LICENSE file under minifi-assembly?
[x] - N/A - If applicable, have you updated the NOTICE file, including the main NOTICE file found under minifi-assembly?

For documentation related changes:

[x] - N/A - Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

reviewing

tested the scripts on a Windows system and worked as anticipated
thanks for fixing and will get this merged in


Hey Rafael, try setting KONGA_HOOK_TIMEOUT env var to 120000. Check out the FAQ section of the README.md

Hey Rafael, try setting KONGA_HOOK_TIMEOUT env var to 120000. Check out the FAQ section of the README.md
Originally posted by @pantsel in https://github.com/pantsel/konga/issues/284#issuecomment-425714143
I'm setting KONGA_HOOK_TIMEOUT,has error:
# ls -al | grep .env
-rw-r--r--.   1 root root    225 May 27 01:21 .env
-rw-r--r--.   1 root root    225 May 26 14:31 .env_example
# cat .env
PORT=1337
NODE_ENV=production
KONGA_HOOK_TIMEOUT=120000
DB_ADAPTER=postgres
DB_URI=postgresql://localhost:5432/konga
DB_USER=konga
DB_PASSWORD=konga123456
DB_DATABASE=konga
KONGA_LOG_LEVEL=warn
TOKEN_SECRET=some_secret_token

# node ./bin/konga.js  prepare --adapter postgres --uri postgresql://localhost:5432/konga
Preparing database...
(node:475025) Warning: Accessing non-existent property 'padLevels' of module exports inside circular dependency
(Use `node --trace-warnings ...` to show where the warning was created)
error: Failed to prepare database: Error: The hook `orm` is taking too long to load.
Make sure it is triggering its `initialize()` callback, or else set `sails.config.orm._hookTimeout to a higher value (currently 60000)
    at Timeout.tooLong [as _onTimeout] (/root/konga/node_modules/sails/lib/app/private/loadHooks.js:85:21)
    at listOnTimeout (internal/timers.js:557:17)
    at processTimers (internal/timers.js:500:7)

postgresql  DB is ok.
# telnet localhost 5432
Trying ::1...
Connected to localhost.
Escape character is '^]'.
^CConnection closed by foreign host.

Duplicate to #646, it can be solved by this PR.


A file "saved as" another file but not showing at the left panel in the file list

I am using
Release 1.6 build 1.6.0-16680 (release 3df0ac6fa)
build timestamp: Tue Jan 12 2016 05:35:00 GMT-0800
on Mac Mavericks.  If I have a file and then I use Save As, to save as another file, it really should show at the left panel in the file list, without we having to "refresh the tree".  I think you can observe on the top directory using some OS call and be notified when the tree structure changes.
(note that the file is under js folder in the top folder. the file will show if it is actually right in the top folder, but won't show if in js folder.)

Most likely fixed by #12145

A build refresh is scheduled by his week or next for 1.6 and after that the new installers posted will contain this fix.
Closing as fixed.
Please reopen if the  issue is not fixed.
Will inform when the new installers are avilable.


Split timeline and live view

Is your feature request related to a problem? Please describe.
I'm currently trying to redesign my camera dashboard for mobile and I see that your card is very good, it has a live view, timeline and clips which is perfect for me.
The way the timeline works now though is it looks like you have to click a separate button on the card to bring it up rather then always being shown underneath the card during live view.
Describe the solution you'd like
A I guess it is called "split view" option perfect for mobile, live view at the top and timeline below with the recent events:
Here is an example, I am not suggesting making it look like that picture below, the timeline right now you made is great I just want an option to keep it visible below the live view for easier access rather than having inexperienced family members with HA clicking random buttons or trying to figure out how to pull it up, especially after coming from the nest / google home app which is different.

Describe alternatives you've considered
Frigates experimental view, it's decent but just a website that you can't access externally embedded in the iframe HA card.

Another one is using your card of course and as you see the timeline below is what I want here, but it only shows when viewing clips, not all the timeline or on live view easily accessible.

Additional context
Add any other context or screenshots about the feature request here.

It's possible I misunderstand the request, but is this not what you're asking for:
type: custom:frigate-card
cameras:
  - camera_entity: camera.kitchen
live:
  controls:
    timeline:
      mode: below
media_viewer:
  controls:
    timeline:
      mode: below

Holy shit that's exactly what I was asking for! I'm sorry I did not realize it was possible to work with the live view!
Btw if I add multiple cameras will it work still and show all events from each camera on one timeline?
I still like the more mobile app look like above but this will work for now, thanks!

 I'm sorry I did not realize it was possible to work with the live view!
Btw if I add multiple cameras will it work still and show all events from each camera on one timeline?
I still like the more mobile app look like above but this will work for now, thanks!

 I'm sorry I did not realize it was possible to work with the live view!

Heh. Excellent!

Btw if I add multiple cameras will it work still and show all events from each camera on one timeline?

When you view the live for camera X, the timeline will show the events for camera X and any camera marked as a dependency of camera X.

@dnestico Probably. I made some tweaks to allow card_mod, although do not use it myself.

Alright cool, any example specifically for the timeline so I can figure out how to modify different parts of the card?

Alright cool, any example specifically for the timeline so I can figure out how to modify different parts of the card?

 There's one example in the docs on using card-mod, but as this is unsupported you're kind of on your own here. Good luck!

So far this doesn't do anything so tryna figure out how the whole CSS thing works rn...
card_mod:
  style:
    frigate-card-timeline-core $: |
      div.timeline {
        background: red !important;
      }


Have getPath() return the correct info in JAX-RS

In the context of a JAX-RS implementation in a Servlet container,
the implementation was incorrect because the servlet path itself
shouldn't be included in the URI.
getPathInfo() gives us the correct information in that context.
@leleuj, here is a PR for you to review because it is related to pac4j/pac4j#63 that we fixed together in the past.
I realised that in most of the pac4j j2e examples, the servlet path is part of the URL to match (e.g., /facebook/....), while with Jersey, the servlet path should NOT be included in the URL to match (if Jersey is deployed on say a servlet with path mapping /api/, then this should not be part of the URL to match since all the URLs in Jersey are defined relative to /api/ (and relative to the context path, which is before the servlet path ^^).
I think this may be of interest to you since the problem will maybe reappear in the future for other integrations…

After some thoughts, I wonder if it wouldn't be better to inverse the situation and:

use the actual getPath() (which includes the servlet path)
Give the possibility to tell jax-rs-pac4j what is the servlet path and prepend it to all urls entered by the user

Why? Because I feel like the norm should be the whole URL since:

some of the manipulated URLs are coming from the web clients (for example the url parameter of the logout logic or the pac4jRequestedUrl of the callback logic).
some of the URLs are used for redirects sent to the web clients.

In both case the web clients are not aware of the servlet path. Actually, I wonder if we should not even include the context path in the getPath() then if we want to above to be true… that point is a question for pa4j in general I guess @leleuj!

I remember the issue (you didn't reference the right one) and the fix. We now have a pretty long method in the getPath() method of the J2EContext class.
The getPath() returns the application path, whatever the framework. So for J2E, we have an implementation which includes the servlet path.
For JAX-RS, indeed, it doesn't seem a good idea to return the servlet path so we can have a new getPath() implementation. I have no issue with what you propose.
In fact, we use the getPath() method in only one place (yes really), in the ExcludedPathMatcher: https://github.com/pac4j/pac4j/blob/master/pac4j-core/src/main/java/org/pac4j/core/matching/ExcludedPathMatcher.java#L35
So I think the best solution is exactly your PR.

Yes, after some experiment, I arrived to the same conclusion indeed!
Still, I think there is something wrong about the way URLs are manipulated in pac4j with respect to this context/servlet path and URLs sent by the client. That's what I wanted to add to the roadmap for v2 :)


Radio Buttons Challenge, can't figure out what is wrong with my code

Challenge Create a Set of Radio Buttons has an issue.
User Agent is: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_3) AppleWebKit/601.4.4 (KHTML, like Gecko) Version/9.0.3 Safari/601.4.4.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

<link href="https://fonts.googleapis.com/css?family=Lobster" rel="stylesheet" type="text/css">

CatPhotoApp

Click here for cat photos.

Things cats love:

cat nip

laser pointers

lasagna

Top 3 things cats hate:

flea treatment

thunder

other cats

<form action="/submit-cat-photo">
  <lable><input type="radio" name="indoor-outdoor">Indoor</lable>
  <lable><input type="radio" name="indoor-outdoor">Outdoor</lable>
  <input type="text" placeholder="cat photo URL" required>
  <button type="submit">Submit</button>
</form>

Would you mind closing the input tags properly?
Admin staff

On Jan 2, 2017, at 1:18 PM, Tonytone916<EMAIL_ADDRESS>wrote:
Challenge Create a Set of Radio Buttons has an issue.
User Agent is: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_3) AppleWebKit/601.4.4 (KHTML, like Gecko) Version/9.0.3 Safari/601.4.4.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

CatPhotoApp
Click here for cat photos.

Things cats love:

cat nip
laser pointers
lasagna

Top 3 things cats hate:

flea treatment
thunder
other cats

Indoor
Outdoor

Submit

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Is not lable is label btw
Admin staff

On Jan 2, 2017, at 1:18 PM, Tonytone916<EMAIL_ADDRESS>wrote:
Challenge Create a Set of Radio Buttons has an issue.
User Agent is: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_3) AppleWebKit/601.4.4 (KHTML, like Gecko) Version/9.0.3 Safari/601.4.4.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

CatPhotoApp
Click here for cat photos.

Things cats love:

cat nip
laser pointers
lasagna

Top 3 things cats hate:

flea treatment
thunder
other cats

Indoor
Outdoor

Submit

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Wow! Thanks for the help! Can't believe I didn't see that.


Installation of Monitoring app "failed to wait for roles to be populated"

Rancher Server Setup

Rancher version: 2.6.5
Installation option Helm Chart:
Kubernetes Cluster and version: k3s

Information about the Cluster

Kubernetes version: v1.23.6+k3s1
9 nodes 2 Server 7 Agents
MySQL Database

User Information

User Account: Default Admin
Global Permissions: Administrator

Describe the bug
When installing the monitoring app I get the following error message "failed to wait for roles to be populated"
To Reproduce

Cluster Tools
Monitoring -> install
version 100.1.2+up19.0.3
cluster type = K3s (no other config changes)
install

Result
Error "failed to wait for roles to be populated"
Expected Result
Monitoring gets installed succesvol
Screenshots

@TheIronRock95 what did you do to resolve this? I'm seeing the same issue today

Never really solved it, I finally reinstalled my K3S cluster + Rancher. Then installed the monitoring stack first, after that it worked


landscape audit updates

corrects levels and adds appropriate dates :+1:

woops, forgot to rebase - let me take care of that quick


The unstable feature badge for items

It would be nice to add a badge unstable for items with #[stability::unstable]:

A common approach for this (e.g. used by tokio) is relying on the doc_cfg feature. Details. It works fine with the current docsrs.
So, #[stability::unstable] can additionally generate #[cfg_attr(docsrs, doc(cfg(feature = "unstable")))]. However, it still requires #![cfg_attr(docsrs, feature(doc_cfg))] at crate level and
[package.metadata.docs.rs]
all-features = true
rustdoc-args = ["--cfg", "docsrs"]

in Cargo.toml
The good news is that it doesn't fail even if these steps haven't added.
Will you accept the PR with an additional feature (docsrs-cfg?) that enables generation of #[cfg_attr(docsrs, doc(cfg(feature = "unstable")))]?

I'm not sure adding the additional feature is worth the complexity, especially since you can use the doc_auto_cfg nightly feature to have doc_cfg applied everywhere there is a cfg gated item. I use this in some of my crates to get the cfg badges:
#![cfg_attr(feature = "nightly", feature(doc_cfg))]
#![cfg_attr(feature = "nightly", feature(doc_auto_cfg))]

Then I enable the nightly feature in Cargo.toml like so:
[package.metadata.docs.rs]
features = ["nightly"]

There's probably other ways of doing this as well without needing to modify the stability macros.

doc_auto_cfg is much better than modifying stability. Thanks a lot)


TransitionMotion with staggered mount, unmount component and React router

Hi there. I wonder could I use TransitionMotion for disappearance of one route and appearance of another route one by one.
I have found excelent example for list of ellements https://github.com/mlewando/staggered-unmount-mount-react-motion by @mlewando.
I was trying to implement this but I couldn't do it,
Actually I hope to terminate position:absolute for wrapper of my routes

I have ended up with this code, it works but I am not sure that can be usefull. So I just leave it here and close the issue =)
class App extends Component {
    constructor(props){
      super(props);
      this.state = {
        animation:false,
        nextUrl:"/"
      };
    }
    getStyles = (previousStyles) => {
      let { children, location : { pathname }} = this.props;
          if (this.state.animation ) return [];
          return [{
            key:`${pathname}-key`,
            data: {
              handler:React.cloneElement(children)
            },
            style: {
              opacity:spring(1)
            },
          }];

    }
    didLeave = () => {
      this.props.router.push(this.state.nextUrl);
      this.setState({
        animation:false
      });
    }
    willLeave = () => {
      return {
        opacity:spring(0),
      };
    }
    willEnter = () => {
      return {
        opacity: 0,
      }
    }
    handleClickLink = (e, nextUrl) => {
      e.preventDefault();
      this.setState({
        animation:true,
        nextUrl: nextUrl
      });
    }
    render(){
      return(

            <Link onClick = {(e) => this.handleClickLink(e,"/")} to = "/">Page1</Link>
            <Link onClick = {(e) => this.handleClickLink(e,"/page2")} to = "/page2">Page2</Link>

          <Helmet {...config.app} />
          <TransitionMotion styles = {this.getStyles()} willEnter = {this.willEnter} willLeave = {this.willLeave} didLeave = {this.didLeave}>
            {interpolatedStyles =>

              {interpolatedStyles.map(({key, data, style}) => {
                return(

                          {data.handler}

);
               })
              }

            }
          </TransitionMotion>

      );
    }
}```

I change some parts of code and I think it could be usefull since it can be stugger transition between routes.


571 model instance unit test

@pkdash please revise.

Can one of the admins verify this patch?

@pkdash we have revised our tests according to your review. please verify our changes

@pkdash Done.

+1


[Design]Fixing 2340: ModelMetadata should recompute localizable properties.

Would like to see a test or two showing different descriptions or display names as the current locale changes. For example, might be a [Theory] unit test with a [Display] attribute in the model that uses a (faked) locale-sensitive resource.

:watch:

@dougbu there are similar tests in DataAnnotationsMetadataProviderTest already(GetDisplayMetadata_DisplayAttribute_NameFromResources GetDisplayMetadata_DisplayAttribute_DescriptionFromResources). let me know if you meant something else

updated.

there are similar tests in DataAnnotationsMetadataProviderTest

Those tests get a (single) value from a resource but don't demonstrate getting different values as the culture changes.

:shipit: from me once doug is happy

@dougbu updated.

:watch:  (close)

67d0bf880a2c584c08aa5fd82c94a0408cf8647c


The version of cuda?

Hello, I'd like to reproduce your excellent code. However, I'm a bit confused by the environment specified in the ReadMe ("cuda 11.6+pytorch 2.0.1"). On the website https://pytorch.org/get-started/previous-versions/#v201, I only see CUDA versions 11.7 and 11.8 listed. Could this be a mistake in describing the environment you are using?

As I remember, I used cuda 11.6 with torch 2.0.1. However, I think torch version is more important for reproduction. So I recommend using torch 2.0.1 and cuda 11.7 or 11.8.

Thank you for your reply, and I would also like to know if this is a necessary modification regarding the difference between your Refact version and the original version, and what is the reason for the change? Because I observed that this resulted in a slight decrease in the results.

There was replacemnet of my desktop and some code refactoring. However, the biggest reason for the performance change is due to the environmental change that occurred when I replaced my desktop. The perfrmance not only decreased but also increased at some metrics. For example, at<EMAIL_ADDRESS>on the ActivityNet Captions dataset, the performance improved significantly from 71.32 to 74.85. I provide both models before the change and after the change.

Sorry to bother you again, but after matching with your environment (which is probably only different from your cuda version), I think that my reproduction effect on the characters dataset is not satisfactory. Specifically, the result obtained by using config-react.json is "2024-04-11 23:41:52684- | Rank1, mIoU 0.4458 | Rank1."<EMAIL_ADDRESS>0.6786 | Rank1<EMAIL_ADDRESS>0.4956 | Rank1<EMAIL_ADDRESS>0.2562 | Rank5, mIoU 0.6863 | Rank5<EMAIL_ADDRESS>0.9829 | Rank5<EMAIL_ADDRESS>0.8610 | Rank5<EMAIL_ADDRESS>0.5279 |. The result obtained using config.json is "2024-04-12 05:00:30002- | Rank1, mIoU 0.4452 | Rank1<EMAIL_ADDRESS>0.6792 | Rank1<EMAIL_ADDRESS>0.4991 | Rank1<EMAIL_ADDRESS>0.2426 | Rank5, mIoU 0.6826 | Rank5<EMAIL_ADDRESS>0.9823 | Rank5<EMAIL_ADDRESS>0.8521 | Rank5<EMAIL_ADDRESS>0.5063 |.
I have provided the environment dependencies I am using as follows. Could you please share the dependencies you are using to help me approach the results reported in your paper(or the refact result)? If I receive your help, I would greatly appreciate it.
dependencies:

_libgcc_mutex=0.1=main
_openmp_mutex=5.1=1_gnu
bzip2=1.0.8=h5eee18b_5
ca-certificates=2024.3.11=h06a4308_0
ld_impl_linux-64=2.38=h1181459_1
libffi=3.4.4=h6a678d5_0
libgcc-ng=11.2.0=h1234567_1
libgomp=11.2.0=h1234567_1
libstdcxx-ng=11.2.0=h1234567_1
libuuid=1.41.5=h5eee18b_0
ncurses=6.4=h6a678d5_0
openssl=1.1.1w=h7f8727e_0
pip=23.3.1=py310h06a4308_0
python=3.10.8=h7a1cb2a_1
readline=8.2=h5eee18b_0
setuptools=68.2.2=py310h06a4308_0
sqlite=3.41.2=h5eee18b_0
tk=8.6.12=h1ccaba5_0
tzdata=2024a=h04d1e81_0
wheel=0.41.2=py310h06a4308_0
xz=5.4.6=h5eee18b_0
zlib=1.2.13=h5eee18b_0
pip:

antlr4-python3-runtime==4.8
bitarray==2.9.2
certifi==2024.2.2
cffi==1.16.0
charset-normalizer==3.3.2
click==8.1.7
cmake==3.29.1
colorama==0.4.6
cython==3.0.10
fairseq==0.12.2
filelock==3.13.4
h5py==3.8.0
hydra-core==1.0.7
idna==3.6
jinja2==3.1.3
joblib==1.4.0
lit==18.1.2
lxml==5.2.1
markupsafe==2.1.5
mpmath==1.3.0
networkx==3.3
nltk==3.8.1
numpy==1.26.4
nvidia-cublas-cu11==<IP_ADDRESS>
nvidia-cuda-cupti-cu11==11.7.101
nvidia-cuda-nvrtc-cu11==11.7.99
nvidia-cuda-runtime-cu11==11.7.99
nvidia-cudnn-cu11==<IP_ADDRESS>
nvidia-cufft-cu11==<IP_ADDRESS>
nvidia-curand-cu11==<IP_ADDRESS>
nvidia-cusolver-cu11==<IP_ADDRESS>
nvidia-cusparse-cu11==<IP_ADDRESS>
nvidia-nccl-cu11==2.14.3
nvidia-nvtx-cu11==11.7.91
omegaconf==2.0.6
pillow==10.3.0
portalocker==2.8.2
pycparser==2.22
pyyaml==6.0.1
regex==2023.12.25
requests==2.31.0
sacrebleu==2.4.1
sympy==1.12
tabulate==0.9.0
torch==2.0.1
torchaudio==2.0.2
torchvision==0.15.2
tqdm==4.66.2
triton==2.0.0
typing-extensions==4.11.0
urllib3==2.2.1

By the way, through the model file you provided, I can easily obtain the same results as the report in your paper. This is really a great job. Thank you for sharing!

Sorry to bother you again, but after matching with your environment (which is probably only different from your cuda version), I think that my reproduction effect on the characters dataset is not satisfactory. Specifically, the result obtained by using config-react.json is "2024-04-11 23:41:52684- | Rank1, mIoU 0.4458 | Rank1."<EMAIL_ADDRESS>0.6786 | Rank1<EMAIL_ADDRESS>0.4956 | Rank1<EMAIL_ADDRESS>0.2562 | Rank5, mIoU 0.6863 | Rank5<EMAIL_ADDRESS>0.9829 | Rank5<EMAIL_ADDRESS>0.8610 | Rank5<EMAIL_ADDRESS>0.5279 |. The result obtained using config.json is "2024-04-12 05:00:30002- | Rank1, mIoU 0.4452 | Rank1<EMAIL_ADDRESS>0.6792 | Rank1<EMAIL_ADDRESS>0.4991 | Rank1<EMAIL_ADDRESS>0.2426 | Rank5, mIoU 0.6826 | Rank5<EMAIL_ADDRESS>0.9823 | Rank5<EMAIL_ADDRESS>0.8521 | Rank5<EMAIL_ADDRESS>0.5063 |. I have provided the environment dependencies I am using as follows. Could you please share the dependencies you are using to help me approach the results reported in your paper(or the refact result)? If I receive your help, I would greatly appreciate it.
dependencies:

_libgcc_mutex=0.1=main

_openmp_mutex=5.1=1_gnu

bzip2=1.0.8=h5eee18b_5

ca-certificates=2024.3.11=h06a4308_0

ld_impl_linux-64=2.38=h1181459_1

libffi=3.4.4=h6a678d5_0

libgcc-ng=11.2.0=h1234567_1

libgomp=11.2.0=h1234567_1

libstdcxx-ng=11.2.0=h1234567_1

libuuid=1.41.5=h5eee18b_0

ncurses=6.4=h6a678d5_0

openssl=1.1.1w=h7f8727e_0

pip=23.3.1=py310h06a4308_0

python=3.10.8=h7a1cb2a_1

readline=8.2=h5eee18b_0

setuptools=68.2.2=py310h06a4308_0

sqlite=3.41.2=h5eee18b_0

tk=8.6.12=h1ccaba5_0

tzdata=2024a=h04d1e81_0

wheel=0.41.2=py310h06a4308_0

xz=5.4.6=h5eee18b_0

zlib=1.2.13=h5eee18b_0

pip:

antlr4-python3-runtime==4.8
bitarray==2.9.2
certifi==2024.2.2
cffi==1.16.0
charset-normalizer==3.3.2
click==8.1.7
cmake==3.29.1
colorama==0.4.6
cython==3.0.10
fairseq==0.12.2
filelock==3.13.4
h5py==3.8.0
hydra-core==1.0.7
idna==3.6
jinja2==3.1.3
joblib==1.4.0
lit==18.1.2
lxml==5.2.1
markupsafe==2.1.5
mpmath==1.3.0
networkx==3.3
nltk==3.8.1
numpy==1.26.4
nvidia-cublas-cu11==<IP_ADDRESS>
nvidia-cuda-cupti-cu11==11.7.101
nvidia-cuda-nvrtc-cu11==11.7.99
nvidia-cuda-runtime-cu11==11.7.99
nvidia-cudnn-cu11==<IP_ADDRESS>
nvidia-cufft-cu11==<IP_ADDRESS>
nvidia-curand-cu11==<IP_ADDRESS>
nvidia-cusolver-cu11==<IP_ADDRESS>
nvidia-cusparse-cu11==<IP_ADDRESS>
nvidia-nccl-cu11==2.14.3
nvidia-nvtx-cu11==11.7.91
omegaconf==2.0.6
pillow==10.3.0
portalocker==2.8.2
pycparser==2.22
pyyaml==6.0.1
regex==2023.12.25
requests==2.31.0
sacrebleu==2.4.1
sympy==1.12
tabulate==0.9.0
torch==2.0.1
torchaudio==2.0.2
torchvision==0.15.2
tqdm==4.66.2
triton==2.0.0
typing-extensions==4.11.0
urllib3==2.2.1

If you have any questions please feel free to ask :)
Are you checking the performance on the last epoch? It would be better to find the epoch with the best performance rather than the last epoch. Of course, there are many evaluation metrics, so it would be good to check which point is the best epoch by visualizing the results through wandb.
Due to changes in my workplace, the GPU was changed from A100 to RTX4090, resulting in slight differences in performance. This performance change is similar to what happened with our base code, which is CPL (https://github.com/minghangz/cpl). My guess is that this is due to the transformer code (transformer.py), but I am not sure.
I recommend checking the performance changes by adjusting the hyperparameters "alpha_ivc", "alpha_pull", "alpha_inter_push", and "alpha_intra_push" that control our Gaussian mixture proposal generation. In my case, when I changed these hyperparameters, I can reproduce the results (refactored version).
I do not know the exact versions of all dependencies because my development environment has changed now, but I remember that except for cuda and torch versions, there were no factors that affected performance at that time. I thinke matching the versions in the readme would be sufficient.

Thank you so much, I think I will try to adjust the hyperparameters.


Map modin category type to arrow dictionary type in omnisci backend

As omnisci finally supports arrow 2.0 we could explicitly map modin's category type to arrow's dictionary type.
This would allow us to distinguish those types in FSI.

https://github.com/intel-ai/omniscidb/tree/modin_test could be used as a source for omnisci backed wtih Arrow 2.0 support

I've implemented none encoded string support in the ArrowResultSetConverter.

Current intel-ai/omniscidb/modin_cats branch still has a problems with none-encoding strings. When such type of string occurs in the result set, the following exception may be thrown:
Exception: Columnar conversion not supported for variable length types

It's thrown from here and can be reproduced with this code:
import pyarrow as pa
import sys

prev = sys.getdlopenflags()
sys.setdlopenflags(1 | 256)  # RTLD_LAZY+RTLD_GLOBAL

from dbe import PyDbEngine

at = pa.Table.from_pydict(
    {"col1": ["str12", "str2", "str3"], "col2": [1, 2, 3]},
    schema=pa.schema({"col1": pa.string(), "col2": pa.int32()}),
)

server = PyDbEngine()
server.importArrowTable("test_name", at)

print(server.select_df("SELECT * FROM test_name ORDER BY col2"))    # OK
print(server.select_df("SELECT col1 FROM test_name ORDER BY col2")) # RuntimeError

The same queries in relation algebra notation:
 Calcite RA queries
 query 1
query_src: SELECT * FROM test_name ORDER BY col2
query_ra: {
  "rels": [
    {
      "id": "0",
      "relOp": "LogicalTableScan",
      "fieldNames": [
        "col1",
        "col2",
        "rowid"
      ],
      "table": [
        "omnisci",
        "test_name"
      ],
      "inputs": []
    },
    {
      "id": "1",
      "relOp": "LogicalProject",
      "fields": [
        "col1",
        "col2"
      ],
      "exprs": [
        {
          "input": 0
        },
        {
          "input": 1
        }
      ]
    },
    {
      "id": "2",
      "relOp": "LogicalSort",
      "collation": [
        {
          "field": 1,
          "direction": "ASCENDING",
          "nulls": "LAST"
        }
      ]
    }
  ]
}

 query 2
query_src: SELECT col1 FROM test_name ORDER BY col2
query_ra: {
  "rels": [
    {
      "id": "0",
      "relOp": "LogicalTableScan",
      "fieldNames": [
        "col1",
        "col2",
        "rowid"
      ],
      "table": [
        "omnisci",
        "test_name"
      ],
      "inputs": []
    },
    {
      "id": "1",
      "relOp": "LogicalProject",
      "fields": [
        "col1",
        "col2"
      ],
      "exprs": [
        {
          "input": 0
        },
        {
          "input": 1
        }
      ]
    },
    {
      "id": "2",
      "relOp": "LogicalSort",
      "collation": [
        {
          "field": 1,
          "direction": "ASCENDING",
          "nulls": "LAST"
        }
      ]
    },
    {
      "id": "3",
      "relOp": "LogicalProject",
      "fields": [
        "col1"
      ],
      "exprs": [
        {
          "input": 0
        }
      ]
    }
  ]
}

As we can see from the calcite json, the thing that triggers exception is the projection of sort result. Other cases when that exception may be thrown are not investigated yet.

So far it looks like problem is in Omnisci side, they are looking into it

So far it looks like problem is in Omnisci side, they are looking into it

Any updates here?

HDK engine is deprecated and will be removed in a future version.


Cask request: [Pritunel Client]

Cask details
(Please fill out as much as possible)
Name:  Pritunel Client
Homepage: http://client.pritunl.com/
Download URL: https://github.com/pritunl/pritunl-client-electron/releases/download/1.0.1210.1/Pritunl.pkg.zip
Description: Free and open source cross platform OpenVPN client

This is already available.


minor patches for cpplint

minor fixes, more explanation inside the patch messages. In short, these
provide:
- minor fixes
- py3k compatibility
- more default extensions
- some tests more robust

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 4 Jan 2015 at 1:29
Attachments:

0001-call-python-via-env.patch
0002-more-extensions-and-fix-check.patch
0003-header-check-by-first-extension-char.patch
0004-py3k-compatible.patch
0005-Allow-numbers-in-Header-guards-names-and-macros.patch
0006-Run-header-test-with-temporary-folder-and-file.patch

Active fork at https://github.com/theandrewdavis/cpplint

No reason to keep open both this issue and https://github.com/google/styleguide/pull/81. This issue can be closed.


Create unwinding tables

What does this PR do?
Ongoing work to check if the async profiler's libraries would make sense in terms of performance and object model
Your probably don't want to read this
Motivation
Checking performances of async profiler's unwinding

Closing in favour of : https://github.com/DataDog/ddprof/pull/275


Secret credentials has an incorrect instance_name (when externalName is used)

Hi colleagues,
We face an issue when creating a ServiceInstance with externalName, which according to the documentaton is the name for the service instance in SAP BTP and is what applications want to actually consume in their apps.
Steps to repoduce:

Create a service instance as shown below:

    apiVersion: services.cloud.sap.com/v1
    kind: ServiceInstance
    metadata:
        name: my-service-instance
    spec:
        serviceOfferingName: sample-service
        servicePlanName: sample-plan
        externalName: my-service-instance-external
        parameters:
          key1: val1
          key2: val2

Create a service binding:

  apiVersion: services.cloud.sap.com/v1
  kind: ServiceBinding
  metadata:
      name: my-binding
  spec:
      serviceInstanceName: my-service-instance
      externalName: my-binding-external
      secretName: mySecret
      parameters:
        key1: val1
        key2: val2

Check the secret details in the k8s cluster where this is executed.

kubectl get secrets mySecret

Issue:
The secret metadata always contains the instance name as the k8s service instance name. This is incorrect when an externalName is specified as the consumers (apps) expect to consume this e.g. via @sap/xsenv using instance name from BTP (e.g. as present in the service catalog)
apiVersion: v1
type: Opaque
kind: Secret
metadata:
  name: mySecret
stringData:
  type: sample-service
  provider: SAP
  instance_guid: xxxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx
  instance_name: my-service-instance # This should be my-service-instance-external
  ...

Issue seems to be in the service binding controller code here
IMO updating this to use externalName from instance (when it exists should solve the issue).
Please take over and provide a fix for this.
Thanks & Regards,
Pavan

Hi Pavan,
I don't think it's a bug, external name is the name that is being used outside of the cluster, it will be the instance name in Service Manager and the name that you will see in cockpit and SM api, but within the cluster the instance name is metadata.name
Why do you need to use the external name within the cluster?
Thanks,
Keren

Hi Keren,
As mentioned in my original description.. when the credentials are consumed say by a Pod within the cluster and an app wants to access the BTP service credentials via instance name - it would do so using the instance name in SM api and/or BTP cockpit and NOT via the k8s cluster metadata.name. Using the cluster specific name in the credential metadata is a bug as consuming apps do not care what the k8s instance name of the service is.. they need to know the actual instance name from SM / BTP cockpit.
The fix I proposed addresses that by updating the OSB credential metadata with the instance_name as present in BTP cockpit.

some applications might require the k8s name and some the BTP name, I will add another key to the secret "instance_external_name" that will contain the external name

Hi Keren,
That will not help as all consuming libraries need to adopt and support this new metadata. E.g. CAP, xsenv etc.. How should we/consumers now handle that with your proposal? (we do not want to create multiple mounts/secrets here to work around the issue).
See for instance: https://www.npmjs.com/package/@sap/xsenv#usage-in-kubernetes: as to how one must mount this in k8s.
Shouldn't your guidance instead be that applications that rely on k8s name - do not set any externalName? With that existing apps should work as they did before (i.e. use the same name for k8s and BTP service instance).
Regards,
Pavan

Hi,
Sorry the new property you introduce does not solve issue (as mentioned above). It instead causes all consumers to adopt and support this new metadata.
We think the instance_name property in the Secret metadata, never worked as intended and hence fixing it would be the proper solution here.

Actually, changing this would be incompatible, service tags might be a better option in general for selecting service credentials and why would users actually not use the instance name they created?
Adding external name as a property seems to be the best way forward, wouldn't it?

...why would users actually not use the instance name they created?

Exactly.. but if you try to consider the actual consumers here (who are trying to consume the BTP service credentials), the instance_name that one would expect to see in the service metadata would be the actual service instance name (from BTP) and not the "k8s instance name".
Especially if you want to put yourselved in the shoes of app developers.. they would wish to use the BTP service instance name (e.g. from BTP cockpit) - just like say they would when they'd create this manually -or- via any other mechanism on CF/K8s.

Adding external name as a property seems to be the best way forward, wouldn't it?

No. As mentioned before.. most of the consumers CAP, @sap/xsenv, any other library/consumers who have their existing way of consuming BTP service credentials - do not know what this "external name" is. To be frank, this would be something that is just invented here by the BTP service operator, which is NOT the only way to create service instances in BTP.
Moreover.. another discrepancy here that we mentioned already to the developers:
if instance_name is really meant for K8s instance name then shouldn’t instance_guid also point to the UId of the K8s instance(?).
In our opinion however it makes no sense do that, instead we should always point to the actual BTP service instance_name and instance_guid when it exists (e.g. when externalName is set). For existing consumers, who do not wish this behaviour they could just (continue to) not set any externalName and get the behaviour you seem to always mention (k8s instance name = BTP instance name = metadata credentials instance_name).
Basically the service metadata added to the secret by this operator was always incorrect and needs to be fixed.. even if it may be incompatible. Or do we stop fixing every issue claiming it's incompatible?

Hi. After re-evaluation, we accepted your argument that this is a bug.
A fix is delivered with [release v0.5.3]https://github.com/SAP/sap-btp-service-operator/releases/tag/v0.5.3 version.
In the created secret, the value of the instance name key is the BTP name, which is maintained in the external name property of the binding, not in the metadata name.

@avilupu Thanks for looking into this.
A small typo/issue in your comment above:
"In the created secret, the value of the instance_name key is the BTP name, which is maintained in the external name property of the ~binding~ instance (instance_name contains the external service instance name as in BTP when present), not the metadata name."
The change (#338) itself looks good though and there intance (external) name is used in secret instance_name property. 👍


(hiera/common.yaml) add rsyslog logrotate

are you constantly out of space on your disks? is your node always tainted with disk pressure? is your CNI complaining and your canal pods are unable to start? Well... logs are no longer going to be THAT problem, only if you click Approve 👌🏻

Was this tested on EL7 and EL8? It will probably work but I don't know how the existing /etc/logrotate.d.rsyslog are different.

this was tested on the foreman dev host with EL7 and some other EL8 cant recall the host 👍🏻


[AutoPR operationalinsights/data-plane] Add operationalinsights/data-plane/readme.typescript.md

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/4209

(message created by the CI based on PR content)


03/28/2018 22:54 : Create Pull Request for branch master -> prTesting1_2018032822

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestCon1.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Fix of bug 103

Q
A

Bug fix?
[x]

New feature?
[ ]

New sample?
[ ]

Related issues?
fixes #X, partially #Y, mentioned in #Z

What's in this Pull Request?
Fix for #103

Thanks @AJIXuMuK for this fix!


[plot] Infinit loop when the backend is specified with unicode in Python2

It affect silx 0.11 release.
I guess t is the same if we specify it in binary with python3.
Feed back from @woutdenolf
As silx view save the backend in a configuration file. When he reach it back, it is in unicode. Or maybe it was unicode because it was generated by another silx view installation using Python 3.
Anyway:
ERROR:silx.gui.qt._qt:<type 'exceptions.RuntimeError'> maximum recursion depth exceeded in cmp   File "/users/opid21/.conda/envs/pymcadev/lib/python2.7/site-packages/silx/gui/data/DataViewer.py", line 233, in __setDataInView
    self.__currentView.setData(self.__displayedData)
  File "/users/opid21/.conda/envs/pymcadev/lib/python2.7/site-packages/silx/gui/data/DataViews.py", line 579, in setData
    self.__updateDisplayedView()
  File "/users/opid21/.conda/envs/pymcadev/lib/python2.7/site-packages/silx/gui/data/DataViews.py", line 557, in __updateDisplayedView
    w = self.__currentView.getWidget()
  File "/users/opid21/.conda/envs/pymcadev/lib/python2.7/site-packages/silx/gui/data/DataViews.py", line 336, in getWidget
    self.__widget = self.createWidget(self.__parent)
  File "/users/opid21/.conda/envs/pymcadev/lib/python2.7/site-packages/silx/gui/data/DataViews.py", line 1391, in createWidget
    widget = ArrayCurvePlot(parent)
  File "/users/opid21/.conda/envs/pymcadev/lib/python2.7/site-packages/silx/gui/data/NXdataWidgets.py", line 78, in __init__
    self._plot = Plot1D(self)
  File "/users/opid21/.conda/envs/pymcadev/lib/python2.7/site-packages/silx/gui/plot/PlotWindow.py", line 805, in __init__
    roi=True, mask=False, fit=True)
  File "/users/opid21/.conda/envs/pymcadev/lib/python2.7/site-packages/silx/gui/plot/PlotWindow.py", line 114, in __init__
    super(PlotWindow, self).__init__(parent=parent, backend=backend)
  File "/users/opid21/.conda/envs/pymcadev/lib/python2.7/site-packages/silx/gui/plot/PlotWidget.py", line 222, in __init__
    self._setBackend(backend)
  File "/users/opid21/.conda/envs/pymcadev/lib/python2.7/site-packages/silx/gui/plot/PlotWidget.py", line 361, in _setBackend
    self._backend = self.__getBackendClass(backend)(self, self)
  File "/users/opid21/.conda/envs/pymcadev/lib/python2.7/site-packages/silx/gui/plot/PlotWidget.py", line 343, in __getBackendClass
    return self.__getBackendClass(b)
  File "/users/opid21/.conda/envs/pymcadev/lib/python2.7/site-packages/silx/gui/plot/PlotWidget.py", line 343, in __getBackendClass
    return self.__getBackendClass(b)
  File "/users/opid21/.conda/envs/pymcadev/lib/python2.7/site-packages/silx/gui/plot/PlotWidget.py", line 343, in __getBackendClass
    return self.__getBackendClass(b)
...
    return self.__getBackendClass(b)
  File "/users/opid21/.conda/envs/pymcadev/lib/python2.7/site-packages/silx/gui/plot/PlotWidget.py", line 343, in __getBackendClass
    return self.__getBackendClass(b)
  File "/users/opid21/.conda/envs/pymcadev/lib/python2.7/site-packages/silx/gui/plot/PlotWidget.py", line 340, in __getBackendClass
    elif isinstance(backend, abc.Iterable):
  File "/users/opid21/.conda/envs/pymcadev/lib/python2.7/abc.py", line 132, in __instancecheck__
    if subclass is not None and subclass in cls._abc_cache:
  File "/users/opid21/.conda/envs/pymcadev/lib/python2.7/_weakrefset.py", line 75, in __contains__
    return wr in self.data

In [1]: from PyQt5 import Qt
In [2]: qt =Qt

In [3]: app = qt.QApplication([])

In [4]: settings = qt.QSettings(qt.QSettings.IniFormat,
   ...:                             qt.QSettings.UserScope,
   ...:                             "silx",
   ...:                             "silx-view",
   ...: None)

In [5]: settings
Out[5]: <PyQt5.QtCore.QSettings at 0x7ff6f4098b98>

In [6]: settings.allKeys()
Out[6]:
[u'colormap/default',
 u'library/plot-image.y-axis-orientation',
 u'library/plot.backend',
 u'mainlayout/custom-nxdata-window-visible',
 u'mainlayout/spliter',
 u'mainlayout/spliter2',
 u'mainwindow/full-screen',
 u'mainwindow/pos',
 u'mainwindow/size',
 u'recent-files/path1',
 u'recent-files/path10',
 u'recent-files/path2',
 u'recent-files/path3',
 u'recent-files/path4',
 u'recent-files/path5',
 u'recent-files/path6',
 u'recent-files/path7',
 u'recent-files/path8',
 u'recent-files/path9']

In [8]: settings.value("library/plot.backend")
Out[8]: u'matplotlib'


DNN-9494: Do not track module deletion for hard delete and do not upd…

…ate module order if order not changed

Changes looks good and merged.


deepPopulate with model specified in refPath

I have 2 schemas, activities and notifications.
Model1:
var mongoose = require('mongoose');
var deepPopulate = require('mongoose-deep-populate')(mongoose);

var Schema = mongoose.Schema,
    ObjectId = Schema.ObjectId;

var model1schema = Schema({
    testModel2       : {
        type         :  ObjectId ,
        ref          :  'TestModel2'
    }
});

model1schema.plugin(deepPopulate);
module.exports = mongoose.model('TestModel1', model1schema);

Model2:
var mongoose = require('mongoose');
var deepPopulate = require('mongoose-deep-populate')(mongoose);

var schema = mongoose.Schema({
    actor_type      : String,
    actor           : {
        type        : mongoose.Schema.Types.ObjectId,
        refPath     : 'actor_type'
    }
});

schema.plugin(deepPopulate);
module.exports = mongoose.model('TestModel2', schema);

Queries are of the form:
    TestModel1
        .find({some condition})
        .deepPopulate('testModel2.actor')

I've tried different variations on this query.
When querying TestModel1, i'm able to populate the testModel2 field, but not able to populate sub documents, such as TestModel1.TestModel2.actor.
Yet, if i'm doing a find directly on TestModel2, I can populate those documents (just using populate(), not the deepPopulate plugin).
I'm not sure if this is a bug, missing feature, or something wrong with something else, but just wanted to check here I was not doing anything wrong.
Any help, even if telling me this is not supported by deepPopulate, would be great.
Thanks,

Was this ever resolved? I have the same question / issue.

Can you add a failed test?

I have the following models:
Notes:
`
var mongoose = require('mongoose');
var Schema = mongoose.Schema;
var ObjectId = Schema.ObjectId;
var deepPopulate = require('mongoose-deep-populate')(mongoose);
require('./chapter');
require('./user');
require('./activism_event');
require('./student');
var note_schema = new Schema({
user: {type: ObjectId, ref: 'User'},
parent: {
kind: String,
item: { type: ObjectId, refPath: 'parent.kind' }
},
date_created: Date,
date_modified: Date,
message: String
});
note_schema.plugin(deepPopulate);
module.exports = mongoose.model('Note', note_schema);
`
Chapter:
`
var mongoose = require('mongoose');
var Schema = mongoose.Schema;
var ObjectId = Schema.ObjectId;
var chapter_schema = new Schema({
school_name: String,
city: String,
state: String,
region: String,
status: String,
tier: String,
date_created: Date,
date_modified: Date,
reply_date: Date,
referral: String,
state_chair_assigned: String,
chapter_facebook_url: String,
president: {
first_name: String,
last_name: String,
email: String,
phone: Number,
street_1: String,
street_2: String,
city: String,
state: String,
zipcode: String,
facebook_url: String
},
free_speech: {
rating: String,
zone: Boolean,
target: Boolean,
status: String
}
});
module.exports = mongoose.model('Chapter', chapter_schema);
`
and Activism_Events:
`
var mongoose = require('mongoose');
var Schema = mongoose.Schema;
var ObjectId = Schema.ObjectId;
require('./chapter');
var activism_event_schema = new Schema({
chapter: {type: ObjectId, ref: 'Chapter'},
date_created: Date,
date_modified: Date,
date_executed: Date,
type: String,
status: String,
blog_post_url: String
});
module.exports = mongoose.model('ActivismEvent', activism_event_schema);
`
In notes, if parent.kind = "chapter" then it will refPath to the chapter model and populate the data with that id in the chapter collection. If parent.kind="activismevent" then it will populate from the activismevent collection.
This behavior works fine when I use the traditional "populate" methods but when I try using this deep-populate package it doesn't work.
Here's an example json output when I use deeppopulate:
{ "_id": "56f0a74796e6e1e13c6fbfbf", "user": { "_id": "56e5f0fe9c82097604be21ff", "first_name": "Kevin", "last_name": "Mulcrone", "phone":<PHONE_NUMBER>, "email"<EMAIL_ADDRESS>"facebook_url": "http://facebook.com/kaybesee", "date_created": "1967-01-02T05:00:00.000Z", "picture_url": "https://scontent.xx.fbcdn.net/hphotos-xtp1/t31.0-8/12829311_10207220825471340_897303930952526274_o.jpg", "region": "Midwest", "role": "State Chair", "__v": 0 }, "date_created": "2016-03-22T02:00:39.769Z", "date_modified": "2016-03-22T02:00:39.770Z", "message": "Test", "__v": 0, "parent": { "item": "56f0a70796e6e1e13c6fbfbc", "kind": "ActivismEvent" } }, { "_id": "56f0a78c0c8358873f24b385", "user": { "_id": "56e5f0fe9c82097604be21ff", "first_name": "Kevin", "last_name": "Mulcrone", "phone":<PHONE_NUMBER>, "email"<EMAIL_ADDRESS>"facebook_url": "http://facebook.com/kaybesee", "date_created": "1967-01-02T05:00:00.000Z", "picture_url": "https://scontent.xx.fbcdn.net/hphotos-xtp1/t31.0-8/12829311_10207220825471340_897303930952526274_o.jpg", "region": "Midwest", "role": "State Chair", "__v": 0 }, "date_created": "2016-03-22T02:01:48.493Z", "date_modified": "2016-03-22T02:01:48.493Z", "message": "This chapter is sweeeeeet.", "__v": 0, "parent": { "item": "56d701c9475c6206a205a908", "kind": "Chapter" } },

@KayBeSee From memory, and having had a look at the deep-populate source code, I seem to remember the issue being that the type of document/model of the field being deep populated is set at the collection level, not the document level, and would require a rewrite/refactor to account for document specific rewrites. This was a while ago, and someone more familiar with the code, i.e., @buunguyen, might like to advise.

I have the same problem. Not resolved yet.


Fix docx -> google drive rendering issues

Description
Issues fixes

Because of invalid nesting in our docx files XML, the google drive docx-renderer was failing to render lessons/threads
Threads overview table was too small when rendered in google drive
Optionality issue with whitespace

Screenshots

How to test

Go to https://deploy-preview-2502--oak-web-application.netlify.thenational.academy/teachers/curriculum/docx-poc/select
Generate a document
Upload the google drive and check issues described above are resolved

@rightsaidjames we'll have to investigate the size of the underline in google docs separately. We'll need to collect issues across editors first, so you can ignore this for now.


How to add a custom tfoot to the table?

As simple as that,
Just wanted to add a custom tfoot not a custom footer for each column!

hey did you end up finding a solution to this problem ?


Поправил поддержку опции family в http.request()

Just started reviewing :) :octocat: Sent from GH.

Thank you, pull request merged! See changes here. :octocat: Sent from GH.


Improve NavigationDrawer documentation

The documentation about NavigationDrawer is very complicated. The sample code should be easier to understand.
I can prepare a PR next week.

It probably wasn't a good idea of using the source code for my site as the example, but I was lazy :) I would appreciate a much simpler example for Navigation Drawers. If you have an idea of a nice way to demo that example as well, that would be great! I wasn't sure how to handle it since the NavigationDrawer is a full page layout component.
Thanks!


create and save err if new image file is not successfully created

What does this do?
Deals with Issue #65
Currently create/2 and save/2 fail silently if the file is not created as they don't check the result of the System.cmd call.
System.cmd returns a tuple with the result and the command exit status, so by pattern matching the result of cmd_mogrify and cmd_convert against a successful attempt ({_, 0}) these functions will now raise an error for unsuccessful exist statuses.
This is similar to the implementation for buffer in create/2.
How to test?
Try to save a file with an illegal file type, and error should be raised.
iex(3)> Mogrify.open("sample.png") |> format("jpg") |> save
%Mogrify.Image{
  animated: false,
  buffer: nil,
  dirty: %{},
  ext: ".jpg",
  format: "jpg",
  frame_count: 1,
  height: nil,
  operations: [],
  path: "/var/folders/hs/6c8zskxs7891h185psnyk3wm0000gn/T/18777-sample.jpg",
  width: nil
}
iex(4)> Mogrify.open("sample.png") |> format("doggos") |> save
** (MatchError) no match of right hand side value: {"mogrify: unable to open image 'doggos:': No such file or directory @ error/blob.c/OpenBlob/3537.\nmogrify: unrecognized image format `doggos' @ error/mogrify.c/MogrifyImageCommand/5062.\n", 1}
    (mogrify 0.7.4) lib/mogrify.ex:30: Mogrify.save/2

What needs to be considered ?
It may be nice to have it throw a better error, but I'd leave the shape of that open for discussion

Thanks. Agree that better error would be nice as a future improvement. Any error is better than silent failure so merging now.