New fields in Serato DJ 1.6.x

It looks like Serato updated their session database format at some point. I was poking around with my files and just noticed this huge spew of messages at the top after -d'ing a session file:
$ php ~/Development/sslscrobbler/historyreader.php -d ~/Music/_Serato_/History/Sessions/2580.session
2014-06-27 08:15:17 INFO: SSLFileReader - reading structure of /Users/jason/Music/_Serato_/History/Sessions/2580.session...
2014-06-27 08:15:17 INFO: SSLFileReader - asking structure DOM to parse /Users/jason/Music/_Serato_/History/Sessions/2580.session...
2014-06-27 08:15:17 INFO: XOUPSSLTrackAdatUnpacker - Unknown field 70: 01                                                                              1
2014-06-27 08:15:17 INFO: XOUPSSLTrackAdatUnpacker - Unknown field 70: 01                                                                              1Unknown field 71: 4073 cdd8 4b3b 8f27                                                             @s!!K;!'
2014-06-27 08:15:17 INFO: XOUPSSLTrackAdatUnpacker - Unknown field 70: 00                                                                              .
2014-06-27 08:15:17 INFO: XOUPSSLTrackAdatUnpacker - Unknown field 70: 00                                                                              .
2014-06-27 08:15:17 INFO: XOUPSSLTrackAdatUnpacker - Unknown field 70: 00                                                                              .
2014-06-27 08:15:17 INFO: XOUPSSLTrackAdatUnpacker - Unknown field 70: 00                                                                              .
2014-06-27 08:15:17 INFO: XOUPSSLTrackAdatUnpacker - Unknown field 70: 00                                                                              .
2014-06-27 08:15:17 INFO: XOUPSSLTrackAdatUnpacker - Unknown field 70: 00                                                                              .
2014-06-27 08:15:17 INFO: XOUPSSLTrackAdatUnpacker - Unknown field 70: 00                                                                              .
2014-06-27 08:15:17 INFO: XOUPSSLTrackAdatUnpacker - Unknown field 70: 00                                                                              .
2014-06-27 08:15:17 INFO: XOUPSSLTrackAdatUnpacker - Unknown field 70: 00                                                                              .
2014-06-27 08:15:17 INFO: XOUPSSLTrackAdatUnpacker - Unknown field 70: 00                                                                              .
2014-06-27 08:15:17 INFO: XOUPSSLTrackAdatUnpacker - Unknown field 70: 00                                                                              .
2014-06-27 08:15:17 INFO: XOUPSSLTrackAdatUnpacker - Unknown field 70: 00                                                                              .
2014-06-27 08:15:17 INFO: XOUPSSLTrackAdatUnpacker - Unknown field 70: 01                                                                              1
2014-06-27 08:15:17 INFO: XOUPSSLTrackAdatUnpacker - Unknown field 70: 01                                                                              1Unknown field 71: 4072 2bd7 0a3d 70a4                                                             @r+!A=p!
2014-06-27 08:15:17 INFO: XOUPSSLTrackAdatUnpacker - Unknown field 70: 00                                                                              .
2014-06-27 08:15:17 INFO: XOUPSSLTrackAdatUnpacker - Unknown field 70: 00                                                                              .
2014-06-27 08:15:17 INFO: XOUPSSLTrackAdatUnpacker - Unknown field 70: 00                                                                              .
2014-06-27 08:15:17 INFO: XOUPSSLTrackAdatUnpacker - Unknown field 70: 00                                                                              .
2014-06-27 08:15:17 INFO: XOUPSSLTrackAdatUnpacker - Unknown field 70: 00                                                                              .
2014-06-27 08:15:17 INFO: XOUPSSLTrackAdatUnpacker - Unknown field 70: 00                                                                              .
2014-06-27 08:15:17 INFO: XOUPSSLTrackAdatUnpacker - Unknown field 70: 00                                                                              .
2014-06-27 08:15:17 INFO: XOUPSSLTrackAdatUnpacker - Unknown field 70: 00                                                                              .
2014-06-27 08:15:17 INFO: XOUPSSLTrackAdatUnpacker - Unknown field 70: 00                                                                              .
2014-06-27 08:15:17 INFO: XOUPSSLTrackAdatUnpacker - Unknown field 70: 00                                                                              .
2014-06-27 08:15:17 INFO: XOUPSSLTrackAdatUnpacker - Unknown field 70: 00                                                                              .
2014-06-27 08:15:17 INFO: XOUPSSLTrackAdatUnpacker - Unknown field 70: 00                                                                              .
2014-06-27 08:15:17 INFO: XOUPSSLTrackAdatUnpacker - Unknown field 70: 00                                                                              .
2014-06-27 08:15:17 INFO: XOUPSSLTrackAdatUnpacker - Unknown field 70: 00                                                                              .
2014-06-27 08:15:17 INFO: XOUPSSLTrackAdatUnpacker - Unknown field 70: 00                                                                              .
2014-06-27 08:15:17 INFO: XOUPSSLTrackAdatUnpacker - Unknown field 70: 00                                                                              .
2014-06-27 08:15:17 INFO: XOUPSSLTrackAdatUnpacker - Unknown field 70: 00                                                                              .
2014-06-27 08:15:17 INFO: XOUPSSLTrackAdatUnpacker - Unknown field 70: 00                                                                              .
2014-06-27 08:15:17 INFO: XOUPSSLTrackAdatUnpacker - Unknown field 70: 01                                                                              1

Warning: vsprintf(): Too few arguments in /Users/jason/Development/sslscrobbler/SSL/Logger/L.php on line 93
2014-06-27 08:15:17 INFO: XOUPSSLTrackAdatUnpacker -
2014-06-27 08:15:17 INFO: XOUPSSLTrackAdatUnpacker - Unknown field 70: 01                                                                              1
2014-06-27 08:15:17 INFO: XOUPSSLTrackAdatUnpacker - Unknown field 70: 01                                                                              1Unknown field 71: 4079 d999 9999 999a                                                             @y!!!!!!
2014-06-27 08:15:17 INFO: SSLFileReader - printing structure of /Users/jason/Music/_Serato_/History/Sessions/2580.session...
CHUNK<vrsn>:
    version => 1.0/Serato Scratch LIVE Review

Hey dude it seems like there are new fields again since I upgraded to SeratoDJ  1.8.0. Hope this is fixed asap.

Cool! - it's not a bug, it's just information that I might find useful at
some point. It won't affect the operation of the program.
You should really look at getting a 64-but PHP though.
On Fri, 13 Nov 2015 at 16:47, ByungHwa Ra<EMAIL_ADDRESS>wrote:

Hey dude it seems like there are new fields again since I upgraded to
SeratoDJ 1.8.0. Hope this is fixed asap.
[image: capture]
https://cloud.githubusercontent.com/assets/7411385/11151641/3339d1be-8a69-11e5-8da6-906ce2081f12.PNG
—
Reply to this email directly or view it on GitHub
https://github.com/ben-xo/sslscrobbler/issues/17#issuecomment-156484399.

Okay dude I'll try switching to a 64-bit version! Thank you!

Just one more thing dude. What does it mean "... passed scrobble point"? Has the song been successfully scrobbled? How do I know if it has been successful?

Cuz I noticed that the song had been scrobbled to last.fm even tho it said the song passed the scrobble point. So what does it mean exactly?

The scrobble point is the minimum length of time you listened to the song
before it's allowed to be scrobbled - more than half way, but no less
than 30 seconds and no more than 5 minutes.
But, the scrobble is not actually submitted until the song has finished
being played, usually after you change tracks (after it goes green in the
history).
On Fri, 13 Nov 2015 at 17:51 ByungHwa Ra<EMAIL_ADDRESS>wrote:

Cuz I noticed that the song had been scrobbled to last.fm even tho it
said the song passed the scrobble point. So what does it mean exactly?
—
Reply to this email directly or view it on GitHub
https://github.com/ben-xo/sslscrobbler/issues/17#issuecomment-156500252.

Ohh OK dude thank you so much for such detailed explanation!
On Nov 14, 2015 1:54 AM, "Ben XO"<EMAIL_ADDRESS>wrote:

The scrobble point is the minimum length of time you listened to the song
before it's allowed to be scrobbled - more than half way, but no less
than 30 seconds and no more than 5 minutes.
But, the scrobble is not actually submitted until the song has finished
being played, usually after you change tracks (after it goes green in the
history).
On Fri, 13 Nov 2015 at 17:51 ByungHwa Ra<EMAIL_ADDRESS>wrote:

Cuz I noticed that the song had been scrobbled to last.fm even tho it
said the song passed the scrobble point. So what does it mean exactly?
—
Reply to this email directly or view it on GitHub
<https://github.com/ben-xo/sslscrobbler/issues/17#issuecomment-156500252
.

—
Reply to this email directly or view it on GitHub
https://github.com/ben-xo/sslscrobbler/issues/17#issuecomment-156500908.

Dude I tried to insert the song into mysql database and it showed one unknown field and I found nothing has been inserted. What's the cause?
$dbConfig = array(
'dsn' => 'mysql:host=localhost;dbname=barsphere',
'user' => 'rabyunghwa',
'pass' => '12345678',
'options' => array(),
'sql' => "UPDATE track SET field=:track, artist=:artist, title=:title, album=:album WHERE         user=:key",
'empty_string' => '<no track is currently playing>',
);

my database name is "barsphere" and the name of the table which i want to insert the song data into is called "track". And the columns in the table are field, artist, title, album and user.

Here is the mysql cmd prompt: I don't think i'm missing anything here. I inserted that row of data beforehand so that later it can be updated when a new track is being played.

Okay I got it working now! The reason why it has taken me so long to know how to use this is that I thought that that update statement was the only one to use here. I changed it to an insert statement. And it's working now!

Very glad you got it working! :)
I've updated the very latest master to ignore the new fields.

Okay dude thank you so much!  :)

Hey dude! Just one more thing! Will this only work with MySQL database? Will it work with Oracle Database as well?

I've never tested it with any other database; but because it uses PHP's
"PDO" internally, it should work with any database that you can write the
correct DSN for. PHP does have an Oracle driver, but you'd have to enable
it in the .ini like you did for MySQL.
http://php.net/manual/en/ref.pdo-oci.connection.php
On Sun, 15 Nov 2015 at 13:17, ByungHwa Ra<EMAIL_ADDRESS>wrote:

Hey dude! Just one more thing! Will this only work with MySQL database?
Will it work with Oracle Database as well?
—
Reply to this email directly or view it on GitHub
https://github.com/ben-xo/sslscrobbler/issues/17#issuecomment-156810802.

Okay dude thank you very much!

Hey dude! I've been dealing with Oracle Database and it seems like I'm going the right direction but there is this error

. It says something like "could not resolve connection identifier". I've been googling and stackoverflowing for a while but haven't found anything helpful. Thanks in advance.

I'm sorry I really don't know - it could be any number of things. I've
never used this with oracle before. It does sort of sound like either the
DSN is wrong e.g has a typo, or perhaps it's just the wrong choice.
As you are on Windows, you could also try using the ODBC connector. You'll
have to look into setting up a database connection in ODBC system setting
first and giving it a name.
If you look in the config.php file, you'll find you can increase the log
level from INFO to DEBUG and it may give you better clues.
On Tue, 17 Nov 2015 at 08:52, ByungHwa Ra<EMAIL_ADDRESS>wrote:

Hey dude! I've been dealing with Oracle Database and it seems like I'm
going the right direction but there is this error
[image: capture]
https://cloud.githubusercontent.com/assets/7411385/11206655/72c45b94-8d4b-11e5-818e-0abb73137fd8.PNG
[image: capture2]
https://cloud.githubusercontent.com/assets/7411385/11206658/765ef80e-8d4b-11e5-846d-819a42b7c4c5.PNG
[image: capture3]
https://cloud.githubusercontent.com/assets/7411385/11206661/7c1d60fa-8d4b-11e5-9881-0f58e1f1c3fc.PNG
[image: capture2]
https://cloud.githubusercontent.com/assets/7411385/11206682/958ffafc-8d4b-11e5-908f-d4c6d65cffb9.PNG
. It says something like "could not resolve connection identifier". I've
been googling and stackoverflowing for a while but haven't found anything
helpful. Thanks in advance.
—
Reply to this email directly or view it on GitHub
https://github.com/ben-xo/sslscrobbler/issues/17#issuecomment-157311583.

Okay dude thank you very much!

Hey dude here I am again!
$dbConfig = array(
'dsn' => 'oci:dbname=//localhost:1521/clubsocialapp',
'user' => 'SYSTEM',
'pass' => '123456',
'options' => array(),
'sql' => "INSERT INTO playlists            (ID,field_name,artist_name,title_name,album_name,user_name) values (track_id.nextval, 'hi', 'hi', 'hi',   'hi', 'ByungHwa')",
'empty_string' => '<no track is currently playing>', );

This time it shows the much dreaded "missing or invalid option" error. Well does this at least show that the dsn has been set up correctly and that the php can connect to the oracle database since it says that it's sending the track into database? I've googled everywhere but still have't found anything helpful. Please help. P.S: I'm still using the PDO_OCI thing instead of the ODBC thing. Thanks in advance.

BTW dude will you ever consider adding support for Oracle Database or any other Relational Database?

And here is what the playlists table is like:

I notice that your fields are all in caps (ID, FIELD_NAME, etc) but your
query has them in lower case. You have to be consistent, oracle is picky.
On Fri, 20 Nov 2015 at 19:47, ByungHwa Ra<EMAIL_ADDRESS>wrote:

And here is what the playlists table is like:
[image: capture]
https://cloud.githubusercontent.com/assets/7411385/11310066/6eb978b8-9002-11e5-8b3c-ad80e9ff4bd1.png
—
Reply to this email directly or view it on GitHub
https://github.com/ben-xo/sslscrobbler/issues/17#issuecomment-158506815.

Hey dude. I actually used lower-cased letters when I was naming the columns but somehow Oracle converted them all into caps. However I changed the fields into caps but still it shows the same error. Geez this is driving me crazy. Anyways does this error mean that I got the dsn right? That the php has connected to the database but it just couldnt run the sql command due to a syntactic error? I changed the log level to debug but still its not showing much more info. I don't even know which part of the sql statement went wrong. So dude what else can I do? Thanks in advance.

So dude what do you think causes this error? Is it still the dsn or the sql statement?

Look at this point all I'm doing is googling the error codes from your
screen shot, and seeing what comes up. That's the best I can do to help.
Sorry!
On Sat, 21 Nov 2015 at 16:04, ByungHwa Ra<EMAIL_ADDRESS>wrote:

So dude what do you think causes this error? Is it still the dsn or the
sql statement?
—
Reply to this email directly or view it on GitHub
https://github.com/ben-xo/sslscrobbler/issues/17#issuecomment-158658920.

Okay dude! Well the main reason why I want to use Oracle DB instead of MySQL is its support of DB Change Notification Mechanism. Like when a new song is inserted into the DB, the server side(Java code) gets notified of the insert event.

If I can't insert anything into Oracle DB, then the only option left seems to be periodically polling data from MySQL DB to see if there is any new inserted record, which is far less efficient and ideal.

@VxJasonxV so i finally figured out that field 70 seems to represent that the row is an update to a previous row that was written… you often see pairs with 70 = 0 when you load the track, then 70 = 1 when you eject or load a new track. they have the same row ID for the same song, but with extra information in the updated packet. When serato is shut down, these are elided to remove the duplication, so mostly you see 70 = 1 in old sessions.

only 6 years later


"ReferenceError: Can't find variable: process" after update to 0.14.2

Hi,
I updated to version 0.14.2 from 0.13 and getting errors referring to variable "process" that does not yet exist.
First location it fails is here in the main.jsbundle:
var ReactPerf={
enableMeasure:false,
storedMeasure:_noMeasure,
measureMethods:function(object,objectName,methodNames){
if(process.env.NODE_ENV !== 'production'){
My package.json:
{
"private": true,
"scripts": {
"start": "node_modules/react-native/packager/packager.sh"
},
"dependencies": {
"art": "^0.10.0",
"flux-util": "0.1.1",
"immutable": "^3.7.4",
"is-promise": "^2.0.0",
"parse": "^1.6.9",
"parse-react": "^0.5.0",
"pure-render-decorator": "^0.1.0",
"react": "^0.14.2",
"react-dom": "^0.14.2",
"react-native": "^0.14.2",
"react-native-carousel": "^0.3.5",
"react-native-image-picker": "^0.9.0",
"react-native-linear-gradient": "^1.1.0-alpha",
"react-native-navbar": "^0.5.2",
"react-native-video": "^0.6.1",
"react-pure-render": "^1.0.2",
"react-timer-mixin": "^0.13.2"
}
}

I believe this is also #4022

This should be fixed in latest versions of React Native. Closing out for now!

The recommendations all say to upgrade react to 15.2.1. I just got this same error/crash with react=15.3.1.

Same problem here as GeoffreyPlitt. Receive the error with both react 15.2.1 and 15.3.1

This error is quite prevalent in the latest RN. It happens the most when a new npm package has been installed.

Downgrading to 15.3.2 solved this for me. Didn't figure out the issue with 15.4.0.

I've been having this same issue. I renamed one file and hit when to schittz.. I've tried downgrading my react-native package and also my react package. I can't find the right combination of react-native@0.3?.? with react-@15.??.? to get my app up and running. If anyone has any suggestions. Let me know please.

I had the same error<EMAIL_ADDRESS>react@15.3.2). I've tried virtually everything and what finally helped was a restart of my MBP. Sounds weird, i have no idea why, but if this is of help to you, i'm happy.

I had the same error after upgrade react-native(0.37.0) and react(15.4.0) , finally I downgrading the react to 15.3.2 solved this

I am getting this error too. With RN 0.37. I've tried everything now. Restarting, changing React version etc. Nothing works. Any help out there?

Do you use any custom babel plugins? I had one which tried to access process.env before proper initialization by RN.

@matttti No, but after 24h of trying a million things, finally upgrading to RN 0.38 solved it. No idea why though. But thanks for the reply!

Faced with similar issue on version<EMAIL_ADDRESS>Environment: Win 7. Do you have any proposals how to solve it?

@Gelassen Downgrade to<EMAIL_ADDRESS>

I use yarn instead of npm v5 to solves this problem(for anyone might stuck in here)

having this issue with RN 0.51. Is it only me?

@itinance I do have the issue too.


refactor: Capitalise all column and table names

Worth you all being aware of this. In theory, it shouldn't be breaking as I don't think SQLA cares about capitalisation (after all we have been using lower case when in fact the DB is all in upper) but I have run into issues with then matching them to entries from the RR_DATA_DEFINTION tables which list all table and column names in capital.
Hopefully you are all in agreement that the models should as much as possible reflect the tables. There are many areas where this is not the case but we can come back to that later.

In theory, it shouldn't be breaking as I don't think SQLA cares about capitalisation (after all we have been using lower case when in fact the DB is all in upper)

I think it's more correct to say that SQL Server doesn't care about capitalisation so will accept whatever it's given. When we tried the same on Oracle it didn't work ( See https://github.com/renalreg/ukrr_rda_extract/blob/6f765d6dd91c8eb6c08a711e5f6530d476dfd647/ukrdc/ukrr_extract/xml_utils.py#L20 ).

Yeah, I did notice this comment and was planning on removing it as I don't think any of our current databases care about capitalisation, but I do fear for time travelling George. I would assume though that the you that would arrive in the past having gone through the portal would do so with his memories intact.


[Bug] Dark theme does not theme menu (on Linux Mint?)

I've attached a screenshot to show the issue. The menu bar isn't themed at all, despise my selecting Dark. For now, I've decided to use the System theme, and it works well for me. See below for screenshot. (I dislike embedding without being able to use a spoiler wrapped around.)
Broken Dark Theme

@OdinVex can you try the latest version?

@Murmele, com.github.Murmele.Gittyup v1.1.2 stable, still occurs. Using Mint 21 at the moment.

Fixed by #260, closing.

@OdinVex Inga problem! :smile_cat:


db-sync freezes on mir rewards handling

One of the db-sync tests I'm currently writing https://github.com/input-output-hk/cardano-db-sync/pull/960/files#diff-81ed75e31cf74322027442ff4ea28726ba4ee68c26b7cc0006545c5b42d94989R235 freezes when the first block of the second epoch arrives. Debugging shows that this freeze happens in the stashMirRewards function, exactly at putTMVar. It seems the TMVar is not cleaned from the previous epoch. Probably we should assume no order between events LedgerRewardDist and LedgerMirDist.
Since it's a test, I'm not sure how realistic this scenario is.

In tests this was probably related to the configuration (k and f). So unlikely to happen in mainnet/testnet.

Probably we should assume no order between events LedgerRewardDist and LedgerMirDist.

If you look at the two stash* functions, you will see that there is not order assumed but that both must arrive or it hangs. Unfortunatley legder-specs make handling this stuff difficult.

#953 has been closed so this one should be too.


Adding support for insertion of tuned soft prompts

This is a MVP PR, the functionality works but needs some more polish to make production ready.
Just want to get thoughts + feedback on the current structure of the code before I go further.
Summary of Changes:

New prompt_adapter folder similar to the lora folder to create a LRU cache management system for multiple prompt adapters
New prompt_adapter_request parameter added to the generate functionality of the LLM_Engine
Current support for bloom and llama models, easily extensible for others
Simple test demonstrating that the prompt adapter works for bloom

Next steps:

Add tests for the prompt_adapter_manager LRU functions
Test functionality for async engine and api-server
Add support for other models
Add a prompt_config data object to specify max_prompt_adapters etc. similar to lora

Sorry for the fragmented review @SwapnilDreams100, just read through the whole thing and added some comments + noted some lora copy-paste errors


How to leave the Configuration screen

I'm sorry if this is a dumb question,
I'm following the "Creating Custom Plugins" wiki page.
I've started Davis, created an admin user with the web interface, connected it to my Dynatrace Tenant.
But how do I leave the configuration screen? I want to go to the "main" page where I can type questions, am I missing something obvious here?
Thanks for the great project!

Nevermind, I copied the package.json on the wiki page and that comes with davis version 0.4.1. Using the latest version of Davis fixed all my issues.

@dlopes7 Thanks for reporting this.


Note for Auth Method for Userhub

Central uses a different auth method that I'll author in a separate comment. Including link to how to identify a URI

Thanks for this! Are you thinking about a separate Connect-* function for Central? It'd be nice if we had a single Connect-IC function with a switch for the instance you're connecting to. e.g. Connect-IC -Central vs Connect-IC -Userhub (with the latter being the default).

Will work on this over the next few days -- will aim to keep it unified as you specified. Catch is, where Userhub requires the auth token central requires OAuth API creds as well as a username and password for auth. Will work on spitting useful errors out when we're missing params for "central".
Will put a pull request out there after I write it tomorrow, won't be offended if you look at it and decide that we should split it into two functions (or refactor it)
In the end, we end up with a bearer token all the same, just a different input to get there.
On Tue, Dec 7, 2021, at 4:22 PM, Matt Hamilton wrote:

Thanks for this! Are you thinking about a separate Connect-* function for Central? It'd be nice if we had a single Connect-IC function with a switch for the instance you're connecting to. e.g. Connect-IC -Central vs Connect-IC -Userhub (with the latter being the default).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/WodongaTAFE/PSInContact/pull/1#issuecomment-988271907, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAEX634K7JOY34AFVIWHYE3UPZ3JPANCNFSM5JSFNR3A.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Update to latest zig and simplify build.zig

installHeader's signature was changed recently, so a versioned function was added for it
srcdir isn't needed, just adding "src/" before each path is sufficient
cabarger's hack seems to not be needed anymore
process.Child.run and process.Child.exec was coerced into a single function

Tested on ubuntu x64, on both 0.12.0-dev.3580+e204a6edb and 0.11.0

@freakmangd I'm not an expert on Zig building but it looks good to me. Thanks for the review!

This breaks https://github.com/raysan5/raylib/issues/3834 again

@freakmangd please, could you review this issue?

@sykire Hi, sorry about the breakage, I neglected to test building raylib as a local dependency, I'll get that working asap

@sykire Could you try using #3913 and let me know if it resolves your issue? Thanks!

It builds successfully using zig build although if I do zig build run. It follows with:
INFO: Initializing raylib 5.1-dev
INFO: Platform backend: DESKTOP (GLFW)
INFO: Supported raylib modules:
INFO:     > rcore:..... loaded (mandatory)
INFO:     > rlgl:...... loaded (mandatory)
INFO:     > rshapes:... loaded (optional)
INFO:     > rtextures:. loaded (optional)
INFO:     > rtext:..... loaded (optional)
INFO:     > rmodels:... loaded (optional)
INFO:     > raudio:.... loaded (optional)
WARNING: GLFW: Error: 65544 Description: X11: Failed to load Xlib
WARNING: GLFW: Failed to initialize GLFW
Segmentation fault at address 0x0
???:?:?: 0x0 in ??? (???)
run 1: error: the following command terminated unexpectedly:
/home/username/Demos/Zig-Raylib/1/zig-out/bin/1
Build Summary: 5/7 steps succeeded; 1 failed (disable with --summary none)
run transitive failure
└─ run 1 failure
error: the following build command failed with exit code 1:
/home/username/Demos/Zig-Raylib/1/zig-cache/o/eff5a969d2db35df28286d86e61d4218/build /nix/store/c6ddcmrgzdhfh23y0j5pmqpj1ilfwi8a-zig-0.11.0/bin/zig /home/username/Demos/Zig-Raylib/1 /home/username/Demos/Zig-Raylib/1/zig-cache /home/username/.cache/zig run

I think that's due to an unrelated thing. I'm new to zig and raylib so don't much about them.
I'm using the configuration from this site: https://siddharthroy.com/posts/how-to-use-raylib-with-zig-without-any-bindings

@sykire I'm using the same configuration to build and run raylib in my testing.

I couldn't build it, I got until the error in the ticket https://github.com/raysan5/raylib/issues/3834 because of failed path resolution that was solved but then regressed. The other error I think I'm missing to build some library. I'm still in the process to run Raylib + Zig successfully once 😅.


python3Packages.winacl: init at 0.1.0

Motivation for this change
Platform independent library for interfacing windows security descriptors.
https://github.com/skelsec/winacl
Related to #81418
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 108816 run on x86_64-linux 1

3 packages built:

python37Packages.winacl
python38Packages.winacl
python39Packages.winacl

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 108816 run on x86_64-linux 1

3 packages built:

python37Packages.winacl
python38Packages.winacl
python39Packages.winacl

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 108816 run on x86_64-darwin 1

3 packages built:

python37Packages.winacl
python38Packages.winacl
python39Packages.winacl

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 108816 run on x86_64-darwin 1

3 packages built:

python37Packages.winacl
python38Packages.winacl
python39Packages.winacl


Confusion about configuring aliases in wiki

Aliases

[x] I would like to contribute to the project (code, documentation, advocacy, integration, ...)

Description
So I wanted to add some aliases for one of my mail servers (this image is great, currently migrating all of my mailservers to use this), said to myself RTFM, opened the wiki and there is an aliases page.
As the manual said I read the postfix documentation on virtual aliases, get some idea about how it works, and edit the /tmp/docker-mailserver/postfix-aliases.cf, adding this:
# Alias delivered to an existing account
<EMAIL_ADDRESS><EMAIL_ADDRESS>
I had created<EMAIL_ADDRESS>beforehand and<EMAIL_ADDRESS>didn't exist.
Now when I use another email trying to send an email to<EMAIL_ADDRESS>it always says:
said: 550
    5.1.1<EMAIL_ADDRESS>Recipient address rejected: User unknown in
    virtual mailbox table (in reply to RCPT TO command)

Then I went and RTFM again, trying to forward all emails temporary to<EMAIL_ADDRESS>using this:
@domain.tld<EMAIL_ADDRESS>
Doesn't work again.
Long story short, after three days struggling with it, I found my "best friend":
https://github.com/tomav/docker-mailserver/wiki/Setup-docker-mailserver-using-the-script-setup.sh#alias
And it worked perfectly.
So I don't know if aliases wiki is wrong and doesn't work the way I have implemented it or I should have used setup.sh, and aliases wiki is just for some particular cases which I don't know what.
I'm willing to edit the wiki if it needs to change, but if it's fine I suppose it leads to misunderstanding in someones like me.

Well, looking at the source code the setup script uses the file /tmp/docker-mailserver/postfix-virtual.cf, not /tmp/docker-mailserver/postfix-aliases.cf. I don't know why, the name in the wiki makes more sense but it does nothing.
Feel free to test manually editing that file and (if it works) update the wiki!

It uses postfix-virtual.cf indeed, I changed postfix-aliases.cf to postfix-virtual.cf in wiki and add a link to setup.sh wiki page at top Configuring Aliases wiki page in case someone doesn't want to mess with files manually.


Updated @see URL for GELF formatter.

The @see URL for GELF was broken so I found the new URL and updated it in the PHPDoc.

Thanks


[Bug] Add may work incorrectly with negative values

Lines like: value = np.full(img.shape, value, dtype=img.dtype) may give error.
For multuplication we may expect only positive values, but for addition, add_constant, add_vector and add_array negative values are quite valid

Fixed


Fix data partition error. Do not remove the last data.

When using data for training, the last data should not be removed.

Thanks for the quick fix!


Add minimal width for window for Linux dist

hey @Barraki,
LOL, fun bug :D Thanks for this catch.

hi @Barraki this issue is fixed by @ceiphr in latest release.
Thanks guys for the bug report and the fix ❤


BTable: Unable to sort by child object property with sortByFormatted

Describe the bug
When sorting by a column that is an object using a formatter function, I get the exception "aValue.localeCompare is not a function".  In trying to debug, it looks like an integer is being returned from the realVal function instead of a string, though I'm not sure how that's possible from looking at it.
My data looks like this: { id: 1, charges: { total: 100 } } and my formatter: charges => charges.total.  See StackBlitz for simplified example.
Edit: A workaround is to cast the total charges as a string in the formatter.  It would be very nice if I didn't have to do that everywhere, though.
Reproduction
https://stackblitz.com/edit/github-fwer37?file=src%2Fcomponents%2FComp.vue,src%2FApp.vue
Used Package Manager
npm
Validations

[X] Have tested with the latest version. This is still alpha version and sometime things change rapidly.
[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A, please open a GitHub Discussion instead.
[X] The provided reproduction is a minimal reproducible of the bug.

Considering the formatter function is always suppose to return a string (export type TableFieldFormatter<T> = (value: unknown, key: string, item: T) => string) it's a bit silly. But I suppose there's no harm to cast to a string again. Realistically those that would be disaffected would be doing things wrong already anyways


Add Direct Banking to Connectivity Apps

Added Appsource registration for Direct Banking NL from Idyn B.V.

@microsoft-github-policy-service agree company="IDYN B.V."

FYI, I will be move the code to line 227, so it's easier to tell the order when an app is added.
Will do it when resolving conflicts, since there is another Connectivity Apps PR

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 52045.

Availability update: We will publish a fix for this issue in the next update for release 21.
Build ID to track: 52214.


Error contract code couldn't be stored with PackagedApp on Infura mainnet

When running zos push, it occassionally crashes when trying to deploy the PackagedApp contract. However, when inspecting the transactions sent in etherscan, the PackagedApp is successfully deployed.
[AppDeployer] Deploying new UpgradeabilityProxyFactory...
[AppDeployer] Deployed UpgradeabilityProxyFactory REDACTED
[AppDeployer] Deploying new Package...
[AppDeployer] Deployed Package REDACTED
[AppDeployer] Deploying new AppDirectory...
[AppDeployer] Deployed AppDirectory REDACTED
[AppDeployer] Adding new version...
[AppDeployer] Added version 0.1.0
[AppDeployer] Deploying new PackagedApp...
[ZosNetworkFile] Successfully written zos.mainnet.json
[Error] Error: Error: The contract code couldn't be stored, please check your gas amount.
    at deploy (/usr/local/lib/node_modules/zos/node_modules/zos-lib/lib/utils/Transactions.js:55:70)
    at process._tickCallback (internal/process/next_tick.js:68:7)

Note that the PackagedApp requires that the version is registered in the package on the constructor: https://github.com/zeppelinos/zos/blob/936eec451c2919299171265c7ce48ab0077035b2/packages/lib/contracts/application/PackagedApp.sol#L26
This means that it requires that the previous transaction (the one that reports Added version 0.1.0) finishes successfully. It could be the case that the Infura nodes are out of sync, and thus the node that handles the PackagedApp tx has not yet seen the previous tx, and hence fails.
We need to try to reproduce the issue first, which we could do by trying to send a tx that depends on a previous one being run, before the first is actually mined. Once we know how to reproduce it, let's discuss how to fix (maybe retries on getTransactionReceipt, since the tx is actually successful?).

Apparently it's a problem with infura+mainnet+web3. In order to properly fix, we'd need to reimplement web3's contract#new, since the method that waits for a contract to be deployed cannot handle Infura's out-of-order responses.

Closing as issue was fixed in web3.js


soapysdr: fix CMAKE_INSTALL_LIBDIR for runtime module loading

this sets the CMAKE_INSTALL_LIBDIR to lib (relative-path)
instead of the default absolute path. this is due to soapysdr
concatenating it with the value of CMAKE_INSTALL_PREFIX for
module discovery and which also is an absolute-path, leading
to an invalid "double-absolute" path.
Motivation for this change
I recently encountered problems in using gqrx with my SDRplay RSP1A, wherein SoapySDR didn't find the libsdrPlaySupport module and failed with SoapySDR::Device::make() no match.
After inspection with strace i found, that SoapySDR looks for modules in /nix/store/p66f9qf80jlp9bcwvsri93vnbcwiim0i-soapysdr-0.8.0//nix/store/p66f9qf80jlp9bcwvsri93vnbcwiim0i-soapysdr-0.8.0/lib/SoapySDR/modules0.8
The path seems definitively wrong as the prefix to the package directory is duplicated.
Looking into the SoapySDR source shows, that - in the end - the library at runtime concatenates the values of CMAKE_INSTALL_PREFIX and CMAKE_INSTALL_LIBDIR, which both are absoulte paths in NixOS by default, if I understand that correctly, thus leading to the above invalid path.
Looking around in nixpkgs, I found multiple examples of setting CMAKE_INSTALL_PREFIX to lib, to avoid this problem. So I guess, this might be the correct solution to this problem.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

That is a work-around that will work but a proper fix is to change the code to join paths as paths instead of string concatenation.
See also https://github.com/jtojnar/cmake-snips#assuming-cmake_install_dir-is-relative-path for more information.

Ok. That's good to know / didn't know that. Thanks for the hint. I will try then to create a fix and propose it upstream.

Result of nixpkgs-review pr 129576 at a09d4566 run on aarch64-linux 1

4 packages marked as broken and skipped:

gnuradio3_7Packages.ais
gnuradio3_7Packages.gsm
gnuradio3_7Packages.limesdr
gnuradio3_7Packages.osmosdr

5 packages skipped due to time constraints:

gnss-sdr
gnuradioMinimal
gqrx
pothos
sdrangel

23 packages built successfully:

cubicsdr
gnuradio
gnuradio3_8Packages.ais
gnuradio3_8Packages.limesdr
gnuradio3_8Packages.osmosdr
gnuradioPackages.gnuradio
limesuite
python38Packages.soapysdr
python38Packages.soapysdr-with-plugins
python39Packages.soapysdr
python39Packages.soapysdr-with-plugins
qradiolink
rtl_433
soapyairspy
soapyaudio
soapybladerf
soapyhackrf
soapyremote
soapyrtlsdr
soapysdr
soapysdr-with-plugins
soapyuhd
urh

4 packages marked as broken and skipped:
gnuradio3_7Packages.ais gnuradio3_7Packages.gsm gnuradio3_7Packages.limesdr gnuradio3_7Packages.osmosdr
30 packages built:
cubicsdr gnss-sdr gnuradio gnuradio3_8Packages.ais gnuradio3_8Packages.limesdr gnuradio3_8Packages.osmosdr gnuradioMinimal gnuradioPackages.gnuradio gqrx limesuite pothos python38Packages.soapysdr python38Packages.soapysdr-with-plugins python39Packages.soapysdr python39Packages.soapysdr-with-plugins qradiolink rtl_433 sdrangel soapyairspy soapyaudio soapybladerf soapyhackrf soapyremote soapyrtlsdr soapysdr soapysdr-with-plugins soapysdrplay soapyuhd urh welle-io
It gave up due to sdrplay being unfree, oops. But dependencies seem to build fine :).

That is a work-around that will work but a proper fix is to change the code to join paths as paths instead of string concatenation.
See also https://github.com/jtojnar/cmake-snips#assuming-cmake_install_dir-is-relative-path for more information.

Upstream accepted a corresponding fix (pothosware/SoapySDR#321) and provided it in release v0.8.1 (PR #132090).

Thanks for reporting the problem upstream.


WIP Net 9.0 Support

Adds .NET 9 Targets to the Mapsui Nuget Packages.

[x] Needs SkiaSharp2.88 nuget Release (Is already fixed).
[x] Add .Net 9 Target to Blazor
[x] Add .NET 9 Target to Avalonia Browser
[x] Add .Net 9 Target to Uno Browser
[ ] Release Version of .NET 9

@pauldendulk when .NET 9 is released .NET 6 becomes end of life (should I remove the .NET 6 Targets in this pull request or in another one?)


VS Code Integrated terminal crashes with exit code: 259.

VSCode Version: 1.17.2
OS Version: Windows 10 build 17025

Steps to Reproduce:

Open integrated terminal.
Write a command between "bash", "node .", "react-native run-android", ecc.. (with some commands like "cls" it keeps working).
Press Enter.

Reproduces without extensions: Yes

Hmm.. What the error message means is that cmd.exe crashed, can you run bash without any problems in a regular cmd.exe window?
Could you also check the devtools console for any errors? (Help > Toggle Developer Tools)

Yes, I can run it in a regular cmd.exe. The devtools console catch the exact same error (messageService.ts:125 Il processo del terminale è stato terminato. Codice di uscita: 259).

I have to say that probably the issue came out when I updated my SO to Redstone 4 (builds 17xxx).

@FrancescoNegri do you mean an Insiders build of Windows? If so I can ping the console team as they might have changed something.

Yes, it's an Insider build, the last one relased in the "fast ring".

@bitcrazed any insight into what would cause a 259 exit code in bash.exe? This may just be a problem with winpty doing something it isn't supposed to.

No idea - not seen that! @zadjii - you see something like this?

Perhaps this is related to Microsoft/console#36? I haven't investigated it at all yet. I really doubt there's anything new in the 17017 build from us that would cause that, but hey who knows.
My bet would be something with the input stack, because that was the area with the most code delta over the weeks leading to 17017/17025.
Has anyone from winpty (@rprichard) or ConEmu(@maximus5) investigated or seen this?

No, I hadn't seen this problem before.

In the description I don't see where the error 259 appears.
But the code 259 usually means STILL_ACTIVE and this is rather strange.
You may try to use ProcessMonitor to determine which processes are started during your call and perhaps that gives you a clue.

I've haven't seen any reports of this before, but there have been 2 (this and https://github.com/Microsoft/vscode/issues/37764) over the last couple of weeks.

I updated my SO to build 17035 and the issue is still there

Yep, the same problem here.

Me too only on windows 10. Don't have "bash" do I need to install it?

@Ofer-Gal since @FrancescoNegri original reported this happening with Cmd as his default terminal shell, I doubt this is related to WSL.
Still trying to repro this issue, but have not been able to yet.
If anyone else has a reliable repro, please share detailed repro steps here, along with build number, VSCode version number, etc.
Also, please ensure you do NOT have Legacy Console enabled in your Console properties.

Thanks but somehow it does not crash anymore (maybe a windows update)
even though I have ,   "terminal.integrated.shell.windows": "C:\\WINDOWS\\System32\\cmd.exe" in the settings.json, the terminal starts PowerShell.
I can live with that.

I have just updated VS Code to 1.18.1, no changes about the issue.
Anyway I noticed that even the integrated terminal closes itself with error 259, the command is launched and keeps running. For example I ran  on the integrated terminal in VS, it exited with 259, then I tried to run  in a common cmd prompt and it said that I couldn't because it was already running in a different instance. Hope this can help.

Updating on the problem, currently running on:
VS Code version: Code - Insiders 1.21.0-insider (d28f096, 2018-02-09T05:14:50.066Z)
OS version: Windows_NT x64 10.0.17101
There are things that constantly bugging me:

The extensions does not work well with WSL.exe/ Bash.exe.

For example Go Exts constantly bugging, and wait for 15' least to even do a simple debugging test. Python linter doesn't work automatically, have to re choose what linter to start working, and only work once.

Everytime reloading Vscode. System starting to create, many duplicate init, bash, zsh, etc...

Exit code 259. Have to stop using it with vscode. The log doesn't show much of the help, trying to change to wsl.exe, assign to shortcut with custom commands at start did not help either. Tried using only bash, turning off everything, same result. The only thing left is to reinstall wsl. By the way, using it with Hyper.js or any other terminals work fine for me.

May be open the issue until this got fix?

I've just started seeing a similar crash of the integrated terminal with exit code 259.
I'm running Win 10 insiders Build 17723.rs5_release.180720-1452.
VS code is Version: 1.25.1
Commit: 1dfc5e557209371715f655691b1235b6b26a06be
Date: 2018-07-11T15:43:53.668Z
Electron: 1.7.12
Chrome: 58.0.3029.110
Node.js: 7.9.0
V8: 5.8.283.38
Architecture: x64
I've disabled plug-ins, closed everything, rebooted, reopened vs code and gave the Newly troublesome command: go test
The terminal crashes and give the error code.

Not sure what to do next to root cause this.
Thanks,
Will

Did a little more and found that I can, without issue, open a powershell and run the "go test." Also, the problem goes away if I reverted a specific file back to the last commit.

I seem to have lost the ability to make this happen. Not sure what I changed, so, for now I can't reproduce it or fix it.

@Tyriar I am seemingly hitting this issue constantly now. It seems that almost any webpack output that contains errors over a certain length (and its not that long maybe a 100 lines?) the terminals crash with this error.

@eamodio It's probably related to winpty https://github.com/Microsoft/vscode/issues/45693, which I'm looking into for 1.27. Maybe webpack changed something recently that triggers this in winpty?

Yeah, I guess -- its kind of nuts that I never saw this issue prior to a week or so ago and now it happens at least 10-20 times a day (constantly have to keep restarting watcher tasks).
Also I have just noticed this happening quite a few times over the last day or so too https://github.com/Microsoft/vscode/issues/26807. But probably related to me setting "terminal.integrated.experimentalRestore": true

Been happening to me a lot recently too, and very reliably, I provided more details on my issue I closed above, don't want to spam the same stuff.
If that helps at all.

This looks like an issue with our old terminal backend "winpty", going forward we are focusing our efforts on the new backend "conpty" which is build by the Console team at Microsoft and should fix many issues with the terminal under Windows. You can read more about getting ConPTY here.
/duplicate https://github.com/Microsoft/vscode/issues/45693


Support BIS modeling

New API

The new API is compatible with 0.2.0.
A new parent property of ElementNode and a new parentAttr property of ElementWithParentDMO for parent-child modeling.
A new node type, ModeledElementNode. This one is kind of weird. You can't stick DMOs into ModelNode's. A ModelNode represents exactly one partition and model in the iModel. ModeledElementNode's on the other hand support DMOs. Each IR instance becomes an element with its own submodel. You're much less likely to have many top-level models as you are modeled elements, so I think this is a good design.
An ElementNode can have either

Exactly one model (ModelNode). Use the model property. Note that this is a top-level model only, meaning ones with partitions, because subjects can only be in the repository model.
Exactly one parent. This is either an element with a submodel (ModeledElementNode) or a normal element (ElementNode). Use the parent property. The parent property allows submodeling to be abstracted away. You just draw "parent-child" relationships between element nodes and modeled element nodes, and PCF does the rest.

The major assumption above is that an element is in the same model as its parent. You can imagine a forest of trees rooted in a model. If this assumption isn't correct please let me know as soon as possible. For example, a folder owns a few repositories, but the repositories are in a different link model. That would be odd, because children are another kind of decomposition of an element in BIS.
Casey says:

BIS calls this a "parent Element"—essentially modeling an Entity as an aggregate. A parent Element is not redundant with its child Elements, i.e. the parent plus the children "add up" to represent the whole Object, rather than representing it twice, at two different granularities.

For further documentation about these API changes, please see the PCF wiki.
Other changes

nyc is a development dependency, shouldn't be a surprise at this point. (:
Deletion is rewritten to accomodate foreign-key dependencies in parent-child relationships. I've copied fir's code over until Sam Wilson gets ElementTreeDeleter merged into the iTwin libraries. It's not as efficient because we don't defer definition elements, but hopefully the simple traversal makes it maintainable and correct.
I've added another hack alongside the category property of ElementNode to allow for parent-child modeling. My concern is PCF has no way to support mandatory navigation properties. RelatedElementNode adds the navigation property after the target has been inserted, so it can't add navigation properties like bis:SubCategory's parent. If there's any others besides category and parent, PCF can't handle them at the moment. All the TypeScript library exposes to you is a bag of properties. There's no mechanism to get the ECInstanceId of an element in PCF, something like fir's put, because you'd have to construct its code value. I suspect this is why Zach added category to ElementNode. I'm trying to figure out a good design for this.

I think we need to update the TypeScript compiler in the Azure pipeline. Did Zach write that?

I think we need to update the TypeScript compiler in the Azure pipeline. Did Zach write that? I'm using this in Node.ts types and tsc doesn't seem to like that. The problem is npm ci relies on the lockfile, but I can build it locally off the lockfile.
edit: I've confirmed that the types in @itwin/ecschema-metadata have changed since 3.0.0, although I'm not sure when. Shouldn't be an issue because the API is exposed to the consumer of pcf but I'd like my patch to be correct. Not really familiar with EC.

Yes, Zach wrote the pipeline https://github.com/iTwin/pcf/commits/bis-modeling/azure-pipelines.yml

I think we need to update the TypeScript compiler in the Azure pipeline. Did Zach write that? I'm using this in Node.ts types and tsc doesn't seem to like that. The problem is npm ci relies on the lockfile, but I can build it locally off the lockfile.
edit: I've confirmed that the types in @itwin/ecschema-metadata have changed since 3.0.0, although I'm not sure when. Shouldn't be an issue because the API is exposed to the consumer of pcf but I'd like my patch to be correct. Not really familiar with EC.

Yes, Zach wrote the pipeline https://github.com/iTwin/pcf/commits/bis-modeling/azure-pipelines.yml

the only version I see specified is for node itself...
  - task: NodeTool@0
    inputs:
      versionSpec: '12.17.0'
    displayName: 'Install Node.js

Doesn't conform to what we have in package.json ...
"engines": {
"node": ">=14.17.0 <17.0"
},
We should bump that up.

I'm working on fixing the pipeline over at bis-modeling-pipeline. I think TypeScript isn't updating because of some engine constraint. I'm not sure.

I've made changes to the pipeline to get it to pass but I don't understand them. I bumped TypeScript because I was getting complication errors in the generated application, probably due to the declarations we provide.
Then, to satisfy the peer dependencies, the pipeline pulls from core instead of installing them like npm install [<@scope>/]<pkg>@<version range>. I added @itwin/common because the generated application was getting caught on types in Node.ts and not realizing they were the appropriate object type. I'm not sure why this wasn't happening before or what changed about TypeScript, because we didn't break a major version.


[Bug] Unable to locate Mansions of Madness on Mac

Hi everyone, I'm new to this world but I tried my best to not open a bug report that was already been sent.
Unfortunately I found different posts with the same issue but none of them seem to help me.
I have a Macbook Pro with MacOS Monterey.
I've downloaded and installed Valkyrie 2.5.8 last version and followed the instructions to:

put the .app file in the Applications folder;
give permissions to the app;
run chmod +x /Applications/Valkyrie.app/Contents/MacOS/Valkyrie in the Terminal app.
I also tried to give every single folder inside the Library folder writing/reading privileges.
The only thing I didn't do, but I'm not sure it is possibile, it is to run Valkyrie at elevated privileges.

Any of these points helped Valkyrie to self locate Mansions of Madness and even if I try to click on "Locate the game" and select the .app file of MoM nothing happens.
The game is installed using Steam.
I saw some other players attached some sort of log, but I didn't understand how to generate it.
Hope somebody can help me, because I'd really like to see what this app has to offer.
Thanks everyone.
Mauro

The only thing I didn't do, but I'm not sure it is possibile, it is to run Valkyrie at elevated privileges.

I tried to do that, using sudo from command line, but doesn't help either

https://github.com/NPBruce/valkyrie/issues/1605#issuecomment-1491433607 helped me

With Valkyrie 2.6.0 on MacOS 14.4.1 and MoM (Steam) 1.9.3 I had no issues importing the assets and running the custom scenario's, I think this is resolved in the current version?


Fix art/estimators/classification/tensorflow.py to work in TF2 compat.v1 mode

Is your feature request related to a problem? Please describe.
When testing things in armory, we are moving away from supporting tf1 directly in favor of using tf2 compatability mode.  As such we run into issue when we try to use: art/estimators/classification/tensorflow.py as it has a call (graph = tf.get_default_graph()) which is incompatible.
Describe the solution you'd like
It would be great if art supported the compat mode more directly, which would amount to changing graph = tf.get_default_graph() to graph = tf.compat.v1.get_default_graph() in my very specific problem, but it would be nice if that generalized across the entire art codebase.
Describe alternatives you've considered
To reproduce the issue, create venv with python3.7.12 and tensorflow 2.7.0 and run:
from art.estimators.classification import TFClassifier
import tensorflow.compat.v1 as tf
tf.disable_eager_execution()

input_ph = tf.placeholder(tf.float32, shape=[None, 28, 28, 1])
labels_ph = tf.placeholder(tf.int32, shape=[None, 10])
training_ph = tf.placeholder(tf.bool, shape=())

x = tf.layers.conv2d(input_ph, filters=4, kernel_size=(5, 5), activation=tf.nn.relu)
x = tf.layers.max_pooling2d(x, 2, 2)
x = tf.layers.conv2d(x, filters=10, kernel_size=(5, 5), activation=tf.nn.relu)
x = tf.layers.max_pooling2d(x, 2, 2)
x = tf.layers.flatten(x)
x = tf.layers.dense(x, 100, activation=tf.nn.relu)
logits = tf.layers.dense(x, 10)

loss = tf.reduce_mean(
    tf.losses.softmax_cross_entropy(logits=logits, onehot_labels=labels_ph)
)
optimizer = tf.train.AdamOptimizer(learning_rate=0.01)
train_op = optimizer.minimize(loss)
sess = tf.Session()
sess.run(tf.global_variables_initializer())

wrapped_model = TFClassifier(
    clip_values=(0.0, 1.0),
    input_ph=input_ph,
    output=logits,
    labels_ph=labels_ph,
    train=train_op,
    loss=loss,
    learning=training_ph,
    sess=sess
)

Additional context
@davidslater please feel free to add additional context.

It looks like several of the tests are using tf.compat.v1, especially tf.compat.v1.disable_eager_execution(). However, there seems to be a huge amount of tensorflow-related code in the codebase (I didn't realize there was so much in there), so trying to get this to work on everything would be quite challenging, I think.
adversarial-robustness-toolbox% grep -src tensorflow . | grep -v ':0$' | grep \.py: | wc -l
129

Not sure our best way forward.

Hi @shenshaw26 and @davidslater I think your proposal is good. We have made a step towards support for compatibility mode in the past which introduced the CI runs with tensorflow2v1 but I think it focused mainly on the object detection estimators. We'll try to update TensorFlowClassifier for ART 1.10.


Update gson version in carbon4 kernel

Update gson version from 2.3.1 to 2.7.0, to fix bundle duplication in product-is.

gson updated to 2.8.5 in 4.5.x


bug (docs): Horizontal scrolling in docs in Arabic/RTL (Firefox)

Provide environment information
Firefox 108.1 on MacOS 13. Doesn't happen in Chrome.
Describe the bug
When browsing the docs in Arabic (and presumably any other RTL language) in Firefox, there is additional space to the left, which creates a horizontal scroll bar. This happens at all sizes.
Large:

Medium:

Small:

To reproduce
Go to https://create.t3.gg/ar/introduction in Firefox
Additional information
No response

I've got the same issue in Firefox 108.0.1 on Windows 11. If you want to assign me I think I can fix it


Many build and test fixes

Summary:
Fixes the following:

Use GTest CMake integration — now, running make test, ctest, etc shows all individual gtests in CI and elsewhere!
Fix a linking issue with MemoryFrameworkTest where, with the vcpkg install, GTest::gmock is a separate target and needs to explicitly be linked to tests for gmock symbols to be included.
Adds Findcub.cmake to resolve an issue with NOT ${CUB_INCLUDE_DIRS} incorrectly being true even with the vcpkg build, which causes a re-download of CUB headers which conflict with the vcpkg Cub headers used to install ArrayFire
Refactors how gtest targets work to merge targets found by CMake's native FindGTest.cmake -- which returns GTest::GTest and GTest::Main, both of which (should) include gmock with GTest >= 1.10.0. Coalesces around GTest::gtest[_main] and GTest::gmock[_main].
Resolves some flakiness with the bias gradient computation in Autograd.LinearF16 test
Changes IMG_CLASS in CMake options and build configurations to be IMGCLASS to be consistent with the namespace, directory, etc.
Some readme fixes to vcpkg installation instructions, add glog as a dep
Fixes an issue where libgflags was being added as an include directory for flashlight-app-lm
Removes a use of tmpnam, which is dangerous

easter egg: does anyone actually read these diff summaries/know what any of this stuff means? If you see this note and mention it to me, you get a prize
Differential Revision: D25409790

I do read them :)


Performance issue: jitclass+typedlist more than 10 x slower than Julia

Hi! I was playing with jitclass and typedlist and get a bad performance result. Since the doc tells us to report performance problems with typedlist, I fill it!
Two codes doing the same thing in Julia and Numba. First the results:

Python

%timeit sum_few_x(points)
14 µs ± 19.3 ns per loop

Julia

  1.022 μs (1 allocation: 16 bytes)

Now the codes:

Python (Am I doing something stupid?)

from numba import njit
from numba.experimental import jitclass
from numba.typed import List as TypedList

jit = njit(cache=True, fastmath=True)

@jitclass
class Point:
    x: float
    y: float
    z: float

    def __init__(self, x, y, z):
        self.x = x
        self.y = y
        self.z = z

@jit
def sum_few_x(vec):
    s = 0.0
    for i in range(10, len(vec)):
        s += vec[i].x
    return s

points = TypedList()
for i in range(1000):
    points.append(Point(i, i, i))

Julia

using Printf
using BenchmarkTools

function sum_few_x(vec)
    result = 0.
    for i in 11:length(vec)
        result += vec[i].x
    end
    return result
end

struct Point
    x::Float64
    y::Float64
    z::Float64
end

zero(T::Type{Point}) = Point(0, 0, 0)

points = Vector{Point}(undef, 1000)

function init_vec(vec)
    x = 0.0
    for i in eachindex(points)
        points[i] = Point(x, 0, 0)
        x += 1.0
    end
end

init_vec(points)

@btime sum_few_x(points)

A related question: do you think there could be a good equivalent of points = Vector{Point}(undef, 1000) in Python? It would be very nice to be able to do points = Vector[Point].empty(1000).

@paugier thanks for reporting this, it will need some triage.

@paugier thanks for reporting this, it will need some triage.

@paugier without looking into the generate IR, my guess is that the for-loop is not vectorizing. Maybe you could try the following variant:
@jit
def sum_few_x(vec):
    s = 0.0
    for i in vec:
        s += i.x
    return s

It may bring you closer to the Julia example. Let us know you findings and thanks again for reporting this!

@paugier without looking into the generate IR, my guess is that the for-loop is not vectorizing. Maybe you could try the following variant:
@jit
def sum_few_x(vec):
    s = 0.0
    for i in vec:
        s += i.x
    return s

It may bring you closer to the Julia example. Let us know you findings and thanks again for reporting this!

Yes, it's much better when one iterates on all elements!
In [10]: %timeit sum_all_x(points)
1.87 µs ± 14 ns per loop

However, for the (nearly) "real life" code I'm working with (https://github.com/paugier/nbabel), it's important to be fast also when iterating on few elements:
The hottest loops look like this:
def compute_accelerations(accelerations, masses, positions):
    nb_particules = len(masses)
    for i0 in range(nb_particules - 1):
        mass0 = masses[i0]
        position0 = positions[i0]
        for i1 in range(i0 + 1, nb_particules):
            mass1 = masses[i1]
            position1 = positions[i1]
            delta = position0 - position1
            distance_cube = delta.norm_cube()
            accelerations[i0] -= mass1 / distance_cube * delta
            accelerations[i1] += mass0 / distance_cube * delta

Yes, it's much better when one iterates on all elements!
In [10]: %timeit sum_all_x(points)
1.87 µs ± 14 ns per loop

However, for the (nearly) "real life" code I'm working with (https://github.com/paugier/nbabel), it's important to be fast also when iterating on few elements:
The hottest loops look like this:
def compute_accelerations(accelerations, masses, positions):
    nb_particules = len(masses)
    for i0 in range(nb_particules - 1):
        mass0 = masses[i0]
        position0 = positions[i0]
        for i1 in range(i0 + 1, nb_particules):
            mass1 = masses[i1]
            position1 = positions[i1]
            delta = position0 - position1
            distance_cube = delta.norm_cube()
            accelerations[i0] -= mass1 / distance_cube * delta
            accelerations[i1] += mass0 / distance_cube * delta

@paugier perhaps it could be possible to slice the list before handing it into the jit compiled function? Also, perhaps you could use the new StructRef module:
https://numba.pydata.org/numba-doc/dev/extending/high-level.html#implementing-mutable-structures
Or maybe even a 3 x 1000 or 1000 x 3 NumPy array since you aren't attaching any functionality to the Jitclass? Maybe even try slicing the NumPy array instead of iterating with integers and then using __getitem__?

@paugier perhaps it could be possible to slice the list before handing it into the jit compiled function? Also, perhaps you could use the new StructRef module:
https://numba.pydata.org/numba-doc/dev/extending/high-level.html#implementing-mutable-structures
Or maybe even a 3 x 1000 or 1000 x 3 NumPy array since you aren't attaching any functionality to the Jitclass? Maybe even try slicing the NumPy array instead of iterating with integers and then using __getitem__?

There is already a pretty good implementation using Numba and Numpy arrays in https://github.com/paugier/nbabel. Here, my goal was to see if I could rewrite a very fast Julia implementation (https://github.com/paugier/nbabel/blob/master/julia/nbabel4_serial.jl) with Numba. I need to be able to "attach" methods to the jit class.
Unfortunately, "slice the list before" is slower, see:
@jit
def sum_few_x_alt(vec):
    s = 0.0
    for elem in vec[10:]:
        s += elem.x
    return s

In [12]: %timeit sum_few_x_alt(points)
63.6 µs ± 12.4 µs per loop (mean ± std. dev. of 7 runs, 1 loop each)

In [13]: %timeit sum_few_x(points)
13.9 µs ± 41.4 ns per loop (mean ± std. dev. of 7 runs, 100000 loops each)

In [14]: %timeit sum_all_x(points)
1.81 µs ± 10.9 ns per loop

There is already a pretty good implementation using Numba and Numpy arrays in https://github.com/paugier/nbabel. Here, my goal was to see if I could rewrite a very fast Julia implementation (https://github.com/paugier/nbabel/blob/master/julia/nbabel4_serial.jl) with Numba. I need to be able to "attach" methods to the jit class.
Unfortunately, "slice the list before" is slower, see:
@jit
def sum_few_x_alt(vec):
    s = 0.0
    for elem in vec[10:]:
        s += elem.x
    return s

In [12]: %timeit sum_few_x_alt(points)
63.6 µs ± 12.4 µs per loop (mean ± std. dev. of 7 runs, 1 loop each)

In [13]: %timeit sum_few_x(points)
13.9 µs ± 41.4 ns per loop (mean ± std. dev. of 7 runs, 100000 loops each)

In [14]: %timeit sum_all_x(points)
1.81 µs ± 10.9 ns per loop

Would namedtuplehelp here?
from numba import njit
from numba.experimental import jitclass
from numba.typed import List as TypedList
from collections import namedtuple
import time
jit = njit(fastmath=True)

@jitclass
class Point:
    x: float
    y: float
    z: float

    def __init__(self, x, y, z):
        self.x = x
        self.y = y
        self.z = z

@jit
def buf(vec):
    s = 0.0
    z = vec[10:]
    for i in z:
        s += i.x
    return s

points = TypedList()
for i in range(1000):
    points.append(Point(i, i, i))

ts=time.time()
for x in range(10):
    buf(points)
print(time.time()- ts)

Point2 = namedtuple('Point2', 'x y z')
points2 = TypedList()
for i in range(1000):
    points2.append(Point2(i, i, i))

ts=time.time()
for x in range(10):
    buf(points)
print(time.time()- ts)

gives me
0.7356758117675781
0.0016891956329345703

Would namedtuplehelp here?
from numba import njit
from numba.experimental import jitclass
from numba.typed import List as TypedList
from collections import namedtuple
import time
jit = njit(fastmath=True)

@jitclass
class Point:
    x: float
    y: float
    z: float

    def __init__(self, x, y, z):
        self.x = x
        self.y = y
        self.z = z

@jit
def buf(vec):
    s = 0.0
    z = vec[10:]
    for i in z:
        s += i.x
    return s

points = TypedList()
for i in range(1000):
    points.append(Point(i, i, i))

ts=time.time()
for x in range(10):
    buf(points)
print(time.time()- ts)

Point2 = namedtuple('Point2', 'x y z')
points2 = TypedList()
for i in range(1000):
    points2.append(Point2(i, i, i))

ts=time.time()
for x in range(10):
    buf(points)
print(time.time()- ts)

gives me
0.7356758117675781
0.0016891956329345703

I think it will be best to close this issue and move the discussion to Discourse at https://numba.discourse.group/ -- the rational is that this does not track an immediate issue with Numba per-se (such as a bug or a feature request) and we would like to improve on the signal-to-noise ratio of the issue tracker. @paugier what do you think?

I think it will be best to close this issue and move the discussion to Discourse at https://numba.discourse.group/ -- the rational is that this does not track an immediate issue with Numba per-se (such as a bug or a feature request) and we would like to improve on the signal-to-noise ratio of the issue tracker. @paugier what do you think?


Add support for GO15VENDOREXPERIMENT

In Go 1.5 the go command will read the environment variable GO15VENDOREXPERIMENT. If enabled, it will use the local "/vendor/" folder for dependencies.
Please add a command line flag that sets GO15VENDOREXPERIMENT=1 environmental variable in the build container.

I'll add a flag to allow passing arbitrary environmental variables. Though I'm across the continent on a conference so will only be able to address this next week.

@karalabe is this still something you are planning?

Dunno really, the vendor experiment flag is enabled by default on Go 1.6, so you can already use it by default with --go=1.6beta1. As the final Go 1.6 release should be done anyway in a month or so, I think there's not much benefit in complicating things with this experimental flag. Any great need for it that cannot be done with Go 1.6 (beta for now)?

Closing as xgo already uses Go 1.6, featuring this.


Support fzf-tmux when zoomed

Here's a slightly hacky approach to supporting zoomed windows:

Create temporary window and move the zoomed pane to the tmp window
Do the normal fzf-tmux thing
Move the zoomed pane back to the original window, kill the tmp window, and re-zoom

It's a bit fragile, for a couple reasons:

If the user starts moving around windows while fzf-tmux is open they'll notice that something is odd
If the user tries to select one of the hidden panes while fzf-tmux is open they'll notice that they are not there

I generally don't do much tmuxing around while fzf is open, though, so this seems good enough for my workflow.  Btw, the approach of creating a tmp window is based on this stack overflow answer:
http://superuser.com/a/357799

Also, for cross-reference this is an alternate fix to issues #196 and #303

Thanks, it's a clever hack and I like it.
Just a couple minor suggestions:
diff --git a/bin/fzf-tmux b/bin/fzf-tmux
index a5b51d0..db54ec7 100755
--- a/bin/fzf-tmux
+++ b/bin/fzf-tmux
@@ -87,11 +87,11 @@ if [ -z "$TMUX_PANE" ]; then
   exit $?
 fi

-# Handled zoomed tmux pane by moving it to a temp window
+# Handle zoomed tmux pane by moving it to a temp window
 if tmux list-panes -F '#F' | grep -q Z; then
   zoomed=1
   original_window=$(tmux display-message -p "#{window_id}")
-  tmp_window=$(tmux new-window -d -P -F "#{window_id}")
+  tmp_window=$(tmux new-window -d -P -F "#{window_id}" "bash -c 'while :; do for c in \\| / - \\\\; do sleep 0.2; printf \"\\r\$c fzf-tmux is running\\r\"; done; done'")
   tmux swap-pane -s $tmp_window \; select-window -t $tmp_window
 fi

So what do you think? If you think they're okay, please update your commit.
(I wonder why the test is failing though, I'll look into it)

Nice I like the indicator :smile:.  Strange that it's now passing the tests...

Thanks, one last request: we no longer use s:tmux_not_zoomed function in the vim plugin. So can you remove the function and squash the commits into one?

Yes, I had noticed that; wasn't sure if you wanted to keep it around for
potential future use.  Will remove and squash commits
On Mon, Dec 7, 2015 at 5:23 PM Junegunn Choi<EMAIL_ADDRESS>wrote:

Thanks, one last request: we no longer use s:tmux_not_zoomed function in
the vim plugin. So can you remove the function and squash the commits into
one?
—
Reply to this email directly or view it on GitHub
https://github.com/junegunn/fzf/pull/433#issuecomment-162725741.

Thanks :+1:

Of course!  Amazing tool, so happy to be able to help in some way :smile:


http://OriginProtocol.com/telegram returns a 404

It is very helpful in the chat and on social media to use the URL http://OriginProtocol.com/discord to send traffic to our Discord group.
Would it also be possible to use http://OriginProtocol.com/telegram to point to https://t.me/originprotocol to send traffic to our Telegram?
I believe it currently returns a 404.  Thank you.

Should be resolved by 67b1a982c4ab0fc54a8643b81aa3e8adba9754b6.

Thanks @micahalcorn !  https://originprotocol-com-micah.herokuapp.com/telegram appears to work correctly for me.


Dynamic footprint?

I would like to simulate a walking person (i.e. legs) with two circles, moving back and forward over time. Is it possible to have dynamic models with changing footprints? I didn't find the appropriate interface...

First of all thanks for the detailed reply.
My problems are the following:

If I set the position of the body-parts (right_leg, left_leg), the position isn't updated properly. Not just the marker integration breaks, but also the laserscan data represents the old initialized configuration of the body-parts.
Setting the velocity of the body-parts is also no option, because I work with weld-joints and finally the whole body moves, instead of just the leg.

I think, I might need another joint. If you have any more suggestions, please don't hesitate. But I might also dig into box2d.

I found a way to accomplish it. I didn't use any joints and set the velocity and position of each body separately. It's maybe not the nicest way and I still have to check if it slows down the simulation essentially. Thanks for your support so far!


Forward kwargs to compute to both get and optimize

Now
x.compute(some_kwarg_here='foo')

Gets forwarded to both x._optimize and x._get. This allows fine tuning of both steps of the compute process when using dask collections. Came up in discussion of #874.

Merging soon if no comment.


Support showing custom tags

The tags api allows users to tag rooms to better organize their room list. I'm pretty sure none of the Riot clients allow you to set these tags, but the tags can still be set through the API (via curl, for example).
Riot-web handles the tags nicely by just adding more groups on the side. Android however just mushes them together under 'Rooms', leaving a mess of rooms. It would be nice if the Android client at least supported the organization of the rooms and the UI to tag them coming at a later time.

We have a meta ticket for this so I close this one.
See https://github.com/vector-im/riot-meta/issues/11


Updated swift  syntax yaml file as missing keywords and some highlighting not working correctly.

Put most, if not all the keywords used in swift from the Apple manual online.
Tested by compiling micro with
make build-all
Then tested by trying the keywords in the editor.
Please give me feed back if it needs changing in any way and I will try my best to sort it out.
Tommy.

Thanks for the contribution. I don't think it's a good idea to have a test swift file in the root folder of the project, so if you could just remove that I'd be happy to merge this.

Deleted the swift test file, I forgot that was in when I did the pull request. (I was using it to find faults with highlighting) Keep up the good work. This is my new default cli editor now on Arch.

Thanks!


Adds optional support for tagging EC2 instances to rotate Splunk secrets

Adds cookbook dependency on ec2-tags-ohai-plugin to read EC2 tags as a secondary detection method for rotating Splunk secrets
Signed-off-by: Dang H. Nguyen<EMAIL_ADDRESS>
Description

In SecretsHelper::splunk_secret_inspect, it looks for a chef tag on instances that need to have certain Splunk secrets rotated. This change will extend that to support EC2 tagging of instances as an option.
Issues Resolved

Check List

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

seems there is a problem with ubuntu 20.04 tests unrelated to the changes (external issue). I'm going to merge this for now.


translations.js is missing

In the document says that:

But I return the following error

My development environment:
Rails version             <IP_ADDRESS>
Ruby version              ruby 2.6.3p62 (2019-04-16 revision 67580) [x86_64-darwin18]
c

If you haven't added the middleware, add it
If you have added the middleware, then I need more details for your setup
i.e. What did you change besides installing the gem
You can also run ::I18n::JS.export in console to see if it works (coz the middleware just runs it on every request)

@PikachuEXE I know the reason:
translations.js is not generate under public/javascripts/ directory
instead it generates under app/assets/javascripts/i18n/
so if I include: javascript_include_tag "translations", skip_pipeline: true (mentioned in doc)
it will break,
so now I just moved the generated file into public/javascripts/ directory,
it works!

resolved


Report arity errors on call target nodes

The proposed change is in the same vein as the one that got merged in https://github.com/microsoft/TypeScript/pull/52943
I believe that shorter error spans are better. Current arity errors might span a lot of lines, especially in calls that take callback arguments or object/array literals
closes https://github.com/microsoft/TypeScript/issues/7734

I looked the results again and: my opinion is still that this is an improvement. =)
I'll merge when tests finishing running.


fix: call stack size exceeded when paste large text (#6373)

fix #6373
There is no error after fix:

call stack image after fix:

sorry, I don't know what does spec means. comments? test case? or some description in PR? Thanks for the explanation.

sorry, I don't know what does spec means. comments? test case? or some description in PR? Thanks for the explanation.

Yes I mean test case.


Enable Stetho just on debug mode #813

This PR update CommonsApplication to enable Stetho just in DEBUG mode

Codecov Report

Merging #828 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff           @@
##           master    #828   +/-   ##
======================================
  Coverage    6.75%   6.75%
======================================
  Files         103     103
  Lines        5198    5198
  Branches      474     474
======================================
  Hits          351     351
  Misses       4820    4820
  Partials       27      27

Impacted Files
Coverage Δ

...n/java/fr/free/nrw/commons/CommonsApplication.java
30.43% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Some RO Crates raise a ValueError if there are multiple context URLs

Crates may have multiple top level context URLs, e.g.
{
  "@context": [
    "https://w3id.org/ro/crate/1.1/context",
    "https://w3id.org/ro/terms/workflow-run"
  ],
...
}

These currently raise a ValueError in cached_url_open.py. Example workflow ID that causes this error is 1046.

I could have a look, yes, in the next days. I assign the issue to myself then?
By the way, will we have regular meeting at some point to get updated a bit on the state and plans? It's been a while, I am not sure I am quite up to date.


feat: Implement mshv_install test to verify Linux Dom0 boot against fresh Hyper-V bins

Add Linux Dom0 boot smoketest to verify a set of Hyper-V binaries against Linux, the linux-mshv driver.

@LiliDeng mostly LGTM


Python thread may not be fully working

Context: http://www.indelible.org/ink/python-reloading/
Preparation:
pip install reloader
Write the file locally:
https://github.com/jparise/python-reloader/blob/master/monitor.py
Test:
import reloader
from .monitor import Reloader
reloader.enable(blacklist=['itertools', 'logging', 'math', 'matplotlib', 'numpy', 'scipy', 'sys', 'skimage'])
r = Reloader()

...

r.poll() # In functions calls

A strange thing is happening: after adding logger.critical() messages inside ModuleMonitor::_scan it looks like the Monitor thread is only running when a foreground function is called. As soon as that function is finished, the background thread looks to be frozen, without any activity.

If this turns out to be the case, the simplest way to deal with this I can image would be to create a new thread right away at start and to spawn the python interpreter there. All the calls would then be proxied to that thread.
This would allow us to have threads running in the background even when the main thread isn't running.

Simple reproduction:
Open Arma 3\python\threads_test.py and paste the following code:
import logging
import threading
import time

logger = logging.getLogger(__name__)

class BackgroundJob(threading.Thread):
    def __init__(self, interval=0.5):
        threading.Thread.__init__(self)
        self.interval = interval
        self.counter = 0

    def run(self):
        while True:
            time.sleep(self.interval)
            self.do_something()
            #return

    def do_something(self):
        logger.error('counting: {}'.format(self.counter))
        #print(self.counter)
        self.counter += 1

def main():
    bj = BackgroundJob()
    bj.start()
    time.sleep(1)

if __name__ == '__main__':
    main()

Do a tail -F pythia.log to look at the logs.
Run Arma 3 with @pythia, open the editor and start the game. While the game is running, open the console, and call:
["python.threads_test.main"] call py3_fnc_callExtension
and watch the log window.
What is expected:
The function will be called, will terminate, and the counting: x text will work all the time, logging the counter every half a second.
What happened:
The function stopped logging as soon as the main() function finished the sleep(1) call and exited.
Calling main() another time, will result in creating a new thread AS WELL AS waking up the old thread (you will see both "counting 0, 1" from the newly created thread AND "counting 2, 3" from the old thread.

Now, the strangest thing is that, after closing Arma completely, the logging starts again by itself!!!
To stop it, you have to kill arma3.exe that is a hanging process now.
This would suggest that Arma freezes the posix thread that is executing the extension after the call to the extension is done and that when closing Arma, theat thread is run again.

Check this stackoverflow answer: https://stackoverflow.com/a/25819019/6543759


Add note to enable aks-preview

Please add a note to run the below command in the "Set auto-upgrade channel" section until the feature is out of preview.
az extension add --name aks-preview

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

ID: 7010bd58-42b1-5c90-3253-05deed855be8
Version Independent ID: c890efb2-cdaf-a3eb-d54f-0e7ccbd4314d
Content: Upgrade an Azure Kubernetes Service (AKS) cluster - Azure Kubernetes Service
Content Source: articles/aks/upgrade-cluster.md
Service: container-service
GitHub Login: @mlearned
Microsoft Alias: mlearned

Thanks for the feedback! We are currently investigating and will update you shortly.

Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

Thank you for the feedback.
We've added a task to our backlog and will prioritize accordingly.
#please-close


HDDS-1666. Issue in openKey when allocating block.

We set size as below
final long size = args.getDataSize() >= 0 ?
args.getDataSize() : scmBlockSize;
and create OmKeyInfo with below size set. But when allocating Block for openKey, we use as below.
allocateBlockInKey(keyInfo, args.getDataSize(), currentTime);
I feel here, we should use size which is set above so that we allocate at least a block when the openKey call happens.
Current Code also works fine, but for readability I believe this should be good.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
33
Docker mode activated.

_ Prechecks _

+1
dupname
0
No case conflicting files found.

+1
@author
0
The patch does not contain any @author tags.

-1
test4tests
0
The patch doesn't appear to include any new or modified tests.  Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1
mvninstall
551
trunk passed

+1
compile
299
trunk passed

+1
checkstyle
89
trunk passed

+1
mvnsite
0
trunk passed

+1
shadedclient
896
branch has no errors when building and testing our client artifacts.

+1
javadoc
181
trunk passed

0
spotbugs
330
Used deprecated FindBugs config; considering switching to SpotBugs.

+1
findbugs
521
trunk passed

_ Patch Compile Tests _

+1
mvninstall
483
the patch passed

+1
compile
296
the patch passed

+1
javac
296
the patch passed

+1
checkstyle
96
the patch passed

+1
mvnsite
0
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedclient
688
patch has no errors when building and testing our client artifacts.

+1
javadoc
180
the patch passed

+1
findbugs
541
the patch passed

_ Other Tests _

-1
unit
158
hadoop-hdds in the patch failed.

-1
unit
146
hadoop-ozone in the patch failed.

+1
asflicense
54
The patch does not generate ASF License warnings.

5416

Reason
Tests

Failed junit tests
hadoop.ozone.container.common.impl.TestHddsDispatcher

hadoop.ozone.om.TestKeyManagerImpl

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/hadoop-multibranch/job/PR-943/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/943

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle

uname
Linux cc90a0e083ee 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / 101d5b5

Default Java
1.8.0_212

unit
https://builds.apache.org/job/hadoop-multibranch/job/PR-943/1/artifact/out/patch-unit-hadoop-hdds.txt

unit
https://builds.apache.org/job/hadoop-multibranch/job/PR-943/1/artifact/out/patch-unit-hadoop-ozone.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-943/1/testReport/

Max. process+thread count
1304 (vs. ulimit of 5500)

modules
C: hadoop-ozone/ozone-manager U: hadoop-ozone/ozone-manager

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-943/1/console

versions
git=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1

Powered by
Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
30
Docker mode activated.

_ Prechecks _

+1
dupname
0
No case conflicting files found.

+1
@author
0
The patch does not contain any @author tags.

-1
test4tests
0
The patch doesn't appear to include any new or modified tests.  Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1
mvninstall
471
trunk passed

+1
compile
241
trunk passed

+1
checkstyle
61
trunk passed

+1
mvnsite
0
trunk passed

+1
shadedclient
787
branch has no errors when building and testing our client artifacts.

+1
javadoc
149
trunk passed

0
spotbugs
345
Used deprecated FindBugs config; considering switching to SpotBugs.

+1
findbugs
558
trunk passed

_ Patch Compile Tests _

+1
mvninstall
464
the patch passed

+1
compile
256
the patch passed

+1
javac
256
the patch passed

+1
checkstyle
65
the patch passed

+1
mvnsite
0
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedclient
658
patch has no errors when building and testing our client artifacts.

+1
javadoc
159
the patch passed

+1
findbugs
586
the patch passed

_ Other Tests _

+1
unit
248
hadoop-hdds in the patch passed.

-1
unit
1689
hadoop-ozone in the patch failed.

+1
asflicense
36
The patch does not generate ASF License warnings.

6585

Reason
Tests

Failed junit tests
hadoop.ozone.client.rpc.TestOzoneRpcClientWithRatis

hadoop.ozone.client.rpc.TestOzoneAtRestEncryption

hadoop.ozone.web.client.TestKeys

hadoop.ozone.client.rpc.TestOzoneRpcClient

hadoop.ozone.client.rpc.TestSecureOzoneRpcClient

hadoop.ozone.om.TestKeyManagerImpl

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/hadoop-multibranch/job/PR-943/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/943

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle

uname
Linux cd17417896f4 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / 129576f

Default Java
1.8.0_212

unit
https://builds.apache.org/job/hadoop-multibranch/job/PR-943/2/artifact/out/patch-unit-hadoop-ozone.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-943/2/testReport/

Max. process+thread count
4169 (vs. ulimit of 5500)

modules
C: hadoop-ozone/ozone-manager U: hadoop-ozone/ozone-manager

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-943/2/console

versions
git=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1

Powered by
Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
86
Docker mode activated.

_ Prechecks _

+1
dupname
0
No case conflicting files found.

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

0
mvndep
45
Maven dependency ordering for branch

+1
mvninstall
642
trunk passed

+1
compile
361
trunk passed

+1
checkstyle
105
trunk passed

+1
mvnsite
0
trunk passed

+1
shadedclient
1186
branch has no errors when building and testing our client artifacts.

+1
javadoc
215
trunk passed

0
spotbugs
411
Used deprecated FindBugs config; considering switching to SpotBugs.

+1
findbugs
687
trunk passed

_ Patch Compile Tests _

0
mvndep
30
Maven dependency ordering for patch

+1
mvninstall
587
the patch passed

+1
compile
333
the patch passed

+1
javac
333
the patch passed

+1
checkstyle
93
the patch passed

+1
mvnsite
0
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedclient
887
patch has no errors when building and testing our client artifacts.

+1
javadoc
211
the patch passed

+1
findbugs
673
the patch passed

_ Other Tests _

+1
unit
391
hadoop-hdds in the patch passed.

-1
unit
3138
hadoop-ozone in the patch failed.

+1
asflicense
54
The patch does not generate ASF License warnings.

9915

Reason
Tests

Failed junit tests
hadoop.ozone.client.rpc.TestOzoneRpcClientWithRatis

hadoop.ozone.client.rpc.TestCommitWatcher

hadoop.hdds.scm.pipeline.TestRatisPipelineCreateAndDestory

hadoop.ozone.client.rpc.TestContainerStateMachine

hadoop.ozone.client.rpc.TestSecureOzoneRpcClient

hadoop.ozone.client.rpc.TestOzoneRpcClient

hadoop.ozone.client.rpc.TestCloseContainerHandlingByClient

Subsystem
Report/Notes

Docker
Client=18.09.7 Server=18.09.7 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-943/3/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/943

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle

uname
Linux 0d327d652801 4.15.0-52-generic #56-Ubuntu SMP Tue Jun 4 22:49:08 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / 64d4abf

Default Java
1.8.0_212

unit
https://builds.apache.org/job/hadoop-multibranch/job/PR-943/3/artifact/out/patch-unit-hadoop-ozone.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-943/3/testReport/

Max. process+thread count
4108 (vs. ulimit of 5500)

modules
C: hadoop-ozone/ozone-manager hadoop-ozone/integration-test U: hadoop-ozone

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-943/3/console

versions
git=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1

Powered by
Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.


Cannot iterate on Headers in Java 11

Trying to iterate over a Headers instance does not work when using plain Java 11 and JPMS.
The reason is that Headers implements Iterator<kotlin.Pair> and kotlin.Pair is not available to JPMS.
This works fine in Java 8 (as the Kotlin stdlib ends up in the classpath), but does not work when using Java 11 and proper JPMS modules, because the application class that iterates over the Headers would need to JPMS-read kotlin.Pair, which would require a directive in the application module-info.java.
Unfortunately the Kotlin stdlib does not support JPMS (no Automatic-Module-Name in the manifest, no module-info.class in the jar), so that leaves the application relying on automatic modules with unstable module names.
This can be worked around iterating using Headers.names() and Headers.values(name).
FTR, I encountered this problem in CometD 6 which is based on Java 11 and JPMS.

Is there a Kotlin bug filed for this? What action are you proposing OkHttp take here?

I know OkHttp 4 has been rewritten in Kotlin, so not sure what to suggest here.
Personally, I would like OkHttp classes that are APIs, such as Headers, to explicitly not have Kotlin classes in method signatures, class inheritance, type parameters, etc.
That would ensure maximum usability from Java 11 + JPMS, as the only dependencies an application would need to require in module-info.java would be Java-only ones.
I don't think it's a Kotlin issue as they need to remain Java 6 compatible and such, so for Kotlin JPMS modules are far far away.
However, removing implements Iterable<kotlin.Pair> may be an incompatible change, so I raised this issue just as a heads-up, as there is an easy workaround to iterate over Headers.
Trying to iterate over Headers using Iterable would not even compile with Java 11 + JPMS, so it's a hard error, but as I said easily workaroundable with a small code change.
I only found this example in my usage of OkHttp, but it may be possible that there are other cases where there is no work around.

Supplying an Automatic-Module-Name can be done regardless of what version they support though. Not to mention they could use the META-INF/versions/9/module-info.class trick, or provide a kotlin-stdlib-jdk9 variant of the standard library (like they already do for 7 and 8).
Let me search through for an issue about this on their tracker. I know it has come up in the past, but I don't know what the state of it is.

Thanks for reporting!
Kotlin folks should fix this for Kotlin 1.4.
https://youtrack.jetbrains.com/issue/KT-21266
No further action for us to take on this.


(maint) Unpin beaker

Stable?

Once this works.
On Thu, Dec 3, 2015 at 2:46 PM, Ken Barber<EMAIL_ADDRESS>wrote:

Stable?
—
Reply to this email directly or view it on GitHub
https://github.com/puppetlabs/puppetdb/pull/1773#issuecomment-161813122.

Refer to this link for build results (access rights to CI server needed):
http://jenkins-enterprise.delivery.puppetlabs.net/job/enterprise_puppetdb_init-multijob_githubpr-master/314/
Test PASSed.