Reagan 33 rebranding

Rebrand from FABook to IBook.

Codecov Report
Base: 70.62% // Head: 70.62% // No change to project coverage :thumbsup:

Coverage data is based on head (5b74bb9) compared to base (00a1766).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master      #37   +/-   ##
=========================================
  Coverage     70.62%   70.62%
  Complexity      404      404
=========================================
  Files            73       73
  Lines          1280     1280
  Branches        131      131
=========================================
  Hits            904      904
  Misses          335      335
  Partials         41       41

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Did you remove FIM？

Can you share your basic results over these models?
I rememberd that there was a FIM implement. But you did not show the results.

Did you remove FIM?

Yes I found there are some errors in my implementation but I have no time to fix it. So I just removed it. You can view it in the commit history.

Can you share your basic results over these models?

Months ago, I tested all the methods on MIND small dataset and shown the results in README.md. However, I have made some small changes to the code and I began to use MIND large dataset. So I removed the outdated results. Sorry but I haven't tested on the new code and new dataset.

Hi, @yusanshi ,
Many thanks for your code! Can you share your finding of FIM? Is it comparable to other strong baselines,i.e., NRMS/NAML? I doubt its performance as my implementation of FIM cannot achieve the expected performance.

Hi @Veason-silverbullet
Yes, my implementation for FIM didn't work well too. Not knowing whether there are some errors with my code I just removed it  :(


chore: (vision) removed unsupported samples

Devsite update: cl/363499751 [publlshed]

PTAL @bshaffer

I do not agree that these samples should be removed so you will need to find someone else to approve and merge this


Refactor federation member votes from PoA miner

We can greatly simplify adding votes from the miner's perspective without going through all the join federation request monitor logic.

Will redo.


Minimum size for RSA key generation on Linux has increased.

Minimum size for RSAOpenSsl key generation has increased.
The minimum size for generating new RSA keys on Linux has increased from 384-bit to 512-bit.
Version introduced
3.0
Change description
The minimum legal key size reported by the LegalKeySizes property on RSA instances from RSA.Create(), new RSAOpenSsl() and new RSACryptoServiceProvider() on Linux has increased from 384 to 512.
Old behavior
RSA.Create(384) succeeds.
New behavior
RSA.Create(384) throws an exception indicating the size is too small.
Reason for change
OpenSSL, which performs the cryptographic operations on Linux, raised their minimum between 1.0.2 and 1.1.0.  .NET Core 3.0 prefers OpenSSL 1.1.x to 1.0.x, and the minimum reported version was raised to convey this new higher dependency limitation.
Recommended action

Ensure that keys are being generated of sizes no less than the provider minimum.
Note that 384-bit RSA is already considered insecure (as is 512-bit RSA). Modern recommendations, such as NIST Special Publication 800-57 Part 1 Revision 4 suggest 2048-bit as the minimum size for newly generated keys.

Category
CoreFx
Affected APIs

AsymmetricAlgorithm.LegalKeySizes
RSA.Create
RSAOpenSsl constructors
RSACryptoServiceProvider constructors

Issue metadata

Issue type: breaking-change

I've made some minor edits to this issue, @bartonjs. Please review them.

@rpetrusha I don't see an Edited marker on the top post, but if you edited it and I just can't see the edits: LGTM.

I can't see the edited marker either.

Nor can I. Possibly what I thought were edits were lost when my laptop dropped my wireless Internet connection, which it does all too frequently. I've made some very minor edits now.

Edited version looks good to me, too :smile:

@bartonjs, do we need to note the preview version in which this change was introduced?


The payload in watcher alarm is empty when alarm is raised

I tried installing sentinl tag - 6.2.4 on my kibana 6.2.4 version.
but I am observing that payload in the alarms is empty.

Hi @krishan-agrawal-guavus you need to explicitly enable the option to save payloads alongside alerts. By default, this is false.  Please reference our docs


fix: parse field lengths according by runes instead of bytes

Codecov Report

Merging #1269 (af23a4e) into master (eadd55b) will increase coverage by 0.04%.
Report is 3 commits behind head on master.
The diff coverage is 100.00%.

:exclamation: Current head af23a4e differs from pull request most recent head 5e8b0d5. Consider uploading reports for the commit 5e8b0d5 to get more accurate results

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1269      +/-   ##
==========================================
+ Coverage   88.52%   88.56%   +0.04%
==========================================
  Files          73       73
  Lines        7069     7094      +25
==========================================
+ Hits         6258     6283      +25
  Misses        477      477
  Partials      334      334


NotEmptyMutableList on all platforms

Description
Make the NotEmptyMutableList type available on all platforms.
Checklist

[x] Rename the type available for the JVM platform as NotEmptyJvmMutableList.
[ ] Create a NotEmptyMutableList type commonly available.
[ ] Make the new type serializable.
[ ] Migrate the builders from the JVM specific type to the common one.
[ ] Migrate the query operations from the JVM specific type to the common one.
[ ] Migrate the positional access operations from the JVM specific type to the common one.
[ ] Remove the JVM specific type.
[ ] Update Work in progress section in changelog.

A mutable collection can automatically be empty, so there is no need to introduce more complexity with a type that merges the mutability and the emptyness...


No sounds nor any updates on notification tray and status bar on android

Registration:
var push = PushNotification.init({
        android: {
            senderID: "my-id"
        },
        ios: {
            alert: "true",
            badge: "true",
            sound: "true"
        },
        windows: {}
    });

Registration successful
push.on('registration', function(data) {
        // data.registrationId
        console.log(data) // I get the data here and push it onto my server
    });

Receiving data is also successful
push.on('notification', function(data) {
        console.log(data) // I see my data in my logs!!
  });

Problem: I am not able to see any notification on my notification tray nor changes on my status bar nor can I hear any sound. The phone just stays still. I am testing on Android 5.0
I am testing it by pinging GCM servers:
curl --header "Authorization: key=my_key_here"
--header Content-Type:"application/json" https://gcm-http.googleapis.com/gcm/send
-d "{ \"data\" : { \"title\" : \"The App\", \"text\" : \"MessageText\", \"extra\":{\"url\":\"someurl.js\"}}, \"to\" : \"$device_registration_token_here\" }"

I also get a success response from the GCM server:
{"multicast_id":some_id_here,"success":1,"failure":0,"canonical_ids":0,"results":[{"message_id":"0:someIdHeer%AndSomeHere"}]}

What might be the issue?

@srameshr Sigh...I believe your problem is the app is in the foreground so the default behaviour is not to show the notification in the shade. You seem to be getting the on('notification') method called so that means the plugin is working fine. If you want the notification to always be shown then use the forceShow option (https://github.com/phonegap/phonegap-plugin-push/blob/master/docs/API.md#android). Forgive me if I seem a bit short, I've just answered this question many, many times.

@macdonst No! No! I can confirm. The app is in the background.

Confirming the same issue !

For me adding the android platform and removing it again with forceShow set to true in the register function did the job.

For me, been issue with Google's console project, fixed.


Remove subtle blip on waveform in analysis example

There's a subtle blip at the beginning of the waveform on the analysis example page due to a special case for the first point on the drawn line:

This PR fixes that blip:

This is my first time using this library and it is super duper - thanks so much for building it!

It looks like the Travis CI build failed due to timeouts unrelated to this change.

thank you!


Badwolf and LaTeX

Themes like molokai and your modified molokai correctly highlight, when editing .tex files, things like
\frac{A}{2}
Badwolf does not (maybe because he's bad :)

I don't use LaTeX, but I'd gladly take a pull request that adds support for it...


[Fix] the test for the package versions on images

In the second part of this test, we have a selected list of packages so that we expect these packages to be same version amongst all the tested images. We have to reduce this list as now there are more packages that are expected not to be in the same version as was in past.

CI:

The failed tests are expected and will be fixed once this fix opendatahub-io/notebooks#742 is propagated into the product.

Just rebased so that we can see the GHA are truly green.


Zipper Mystic empty folder

I used your Zipper in https://github.com/FAForever/faf-java-api
I have allways a magic empty folder in 7Zip:

In Window zip View I have NOT this empty folder:

File:
sludge_test.v0001.zip

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

May be related to #506

Fixed with next version.


Add search criteria and sort order

Overview

Type:

📚Docs

Ticket: 199284

Problem
The parameters for sorting search were not documented.
Solution
Suggested change to the docs: Describe the parameters sort and direction.

According to our backend developers we don't have sort and direction parameters for https://api.video.ibm.com/users/self/channels.json API endpoint.


Focus state: Button

Overview
closes https://github.ibm.com/Bluemix/design-system-website/issues/1049
Fix

fix(demo): update demo css + html to use theme vars

demo-all.dust and demo.scss were still using deprecated light-ui class and mixin, this has been updated to use new color theme vars.

Feat

updated focus states for Buttons

I think it looks good, once you have visual approval we can merge it!
👍 ✅

Had issues with deploying this to bluemix so I checked this with @bsonefeld and got her approval 👍 ✅


ContextMenus should not overlap edge of screen if it is opened near screen edge

Need to use screen API to check if we are near edge and then offset the popup so that it obeys limits of the screen.

Fixed by #1526.


bluetooth: Prevent btsak from repeatedly showing the same error.

"bt bnep0 scan start" causes a lot of error messages to be printed when the buffer is full.
btnet_scan_callback: ERROR: Too many scan results
btnet_scan_callback: ERROR: Too many scan results
btnet_scan_callback: ERROR: Too many scan results
btnet_scan_callback: ERROR: Too many scan results
btnet_scan_callback: ERROR: Too many scan results
btnet_scan_callback: ERROR: Too many scan results
btnet_scan_callback: ERROR: Too many scan results
After this change only one message will appear for each start command.
nsh> bt bnep0 scan start
nsh> btnet_scan_callback: ERROR: Too many scan results
nsh> bt bnep0 scan stop
nsh> bt bnep0 scan start
nsh> btnet_scan_callback: ERROR: Too many scan results
nsh> bt bnep0 scan stop
nsh>

please add a commit message
There is a commit message already.

@ldube what Mr. Alin is saying is you need to put this message in the git commit message, it is easir to see than looking at this PR (that will be closed after the merge)


Add Readme and Mini Docs

We need to add a small readme with the public API of the library and some additional details about the direction this mini library is taking.

https://github.com/vladnicula/proxy-live-document/pull/8


Quotes within argument string not supported

If value for a parameter contains escaped quoted, then the query fails because of failure in parsing the request. The error varies depending on what follows the quoted value, I think quotes are not handled properly:
https://github.com/pgutkowski/KGraphQL/blob/master/src/main/kotlin/com/github/pgutkowski/kgraphql/request/RequestPreProcessing.kt#L24
Example:
{
     hello(name : "Ted\" Mosby")
}

However, passing such strings using query variables works perfectly fine.

Hi @deepakyadav!
Thank you for reporting this issue, issue has been fixed in new version 0.2.8.


increase output size

Hi, thanks for great model!
Suffered a little with dependencies, but launched it on 11 cuda and 2.0.1 pytorch.
All works great, but I have a question: is it possible to increase the size of the output image, or it was set during training and cannot be changed?

Yes, the output resolution is fixed.
Maybe an upsampler cab be used like StableDiffusion or BSRGAN...


0-1 knapsack problem

Algorithm 0-1 knapsack problem

[ ] I have gone through the repository and made sure this algorithm has not been implemented

Given weights and values of n items, put these items in a knapsack of capacity W to get the maximum total value in the knapsack

Algorithm Name:0-1 knapsack problem

Programming Language:c++

Time complexity:O(nW) where n is the number of items and W is the capacity of knapsack.

Is this your first contribution to this repository?
Yes

May i take this one ?

Hey @afAditya are you working on this?

yeah I am working on this

@frextrite you can review my pr #12

Implemented in #12


Bump github.com/go-playground/validator/v10 to latest version

Motivation
Update the package to the latest compatible version to resolve related vulnerabilities.

We may want to wait for https://github.com/go-playground/validator/issues/1206 to be closed


Fixes and additions

Hi @arturdev,
Thanks for contributing your changes, it's much appreciated. I'm very busy currently, so I'm unable to take a look in detail just yet. What I've done is merged them into a development branch:
https://github.com/alskipp/ASScreenRecorder/tree/development
I'll update the README to notify people that you've made these additions and link to the new branch. When I have more time, I'll take a look at the changes in detail and merge them into master.
Cheers,
Al


docsUrl for latest version of zotero is failing

Another package that seems to be affected by the validation of package URLs is the latest version of zotero.
The documentation URL takes a bit long sometimes to view, so I assume there is a timeout on the validator due to this.
However, the URL is still valid.

@AdmiringWorm I have just re-run the package validator on this package, and all URL's contained within it have validated correctly.
I can only assume that this was a glitch in the matrix.
Going to close this out.


Update get_fingerprints.m

transpose a vector with appropriate dimension

On my machine both s(wstart:wend) and hamming(wlen) are column vectors, which is the correct format for element-wise multiplication. If you need to add a transpose sign to get it to work on your machine then it would be better to explicitly convert both arrays to columns.

Thank you for your reply,
However,* s(wstart:wend)* is a row vector in MATLAB (R2016a) on my machine.
Thus, I had to transpose the vector.
Thanks on your effort for this algorithm.
Sincerely yours,
Seungho Kuk
2018-04-17 6:56 GMT+09:00 Steven Van Vaerenbergh<EMAIL_ADDRESS>
On my machine both s(wstart:wend) and hamming(wlen) are column vectors,
which is the correct format for element-wise multiplication. If you need to
add a transpose sign to get it to work on your machine then it would be
better to explicitly convert both arrays to columns.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/steven2358/Shazam-Matlab/pull/1#issuecomment-381763023,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ASbWNqkPQ2h0Qt6LczwDjtCgVrKTgkj3ks5tpRN_gaJpZM4TWBNO
.

--
Seungho Kuk,
Wireless Data Communications Lab.
Korea University, Republic of Korea
Tel: +82-2-3290-3569

I added an explicit conversion of s into a column vector. That should work on both machines. Please check. Cheers!


Suggestion: Supporting additional slots?

I have a suggestion for improvement for the plugin.
How do you think about multiple slots for the PKCS plugin? For example, right now we have:
"kms_key_id": "......"
In the current case, it would default to slot 0
This is superseeded by kms_key_ids key (if set), in that case the kms_key_id is ignored and the kms_key_ids array key is used instead.
So, the config could look like this:
"kms_key_ids": [{
   "kms_key_id": "<slot0_kms_id>",
   "aws_region": "us-west-1"
 },
 {
   "kms_key_id": "<slot1_kms_id>",
   "aws_region": "us-west-2"
  },
  ...........
]

This way, it can use the array index as the slot index and we can pass an arbitrary number of slot items via PKCS11 plugin (upto the number of slots the pkcs11 protocol supports).
Ofcourse this can currently be supported by changing the config each time you want to use it, but I thought it could be nice to natively support it.

Yea, I thought about providing better support for using pkcs11 uris when I first put this together, but I really just wanted to get a MVP out the door. As opposed to having separate slots I was thinking of just representing the keys as objects in a single slot, so for tools that support a pkcs11 uri you could specify pkcs11:object=f9292d51-8d46-4de9-bd3d-c91f41b78c4d;objecttype=private where f9292d51-8d46-4de9-bd3d-c91f41b78c4d is the KMS key id. You would still need to either list all possible key ids that you would want to use in the config or else the module could fall back to doing a list keys call. That seems ideal since in cases where that's sufficient it would save you from having to make a config file at all.
The only real challenge is that I'd have to do some refactoring to be keep track of the getPublicKey calls and do it lazily (since right now it just eagerly fetches it when the session is started, but I wouldn't want to do that for all keys if it supports multiple keys).

Yea, I think doing it via objects also makes sense.
I think forcing to have the keys specified in the config is a good thing. It would mean that an arbitrary key couldn't be looked up unless it was specifically mentioned in the config. (one could argue that you should control this via IAM, but many people use very permissive IAM so an extra check is a good idea).

I just pushed a commit to master which should add support for this. I didn't expand the config to allow a region, per key, but otherwise it should look the same as above. So you can specify a list of key ids in your config, e.g. "kms_key_ids: ["kms_id_1", "kms_id_2"] and then use the pkcs11 uri to pick the key you want to use, e.g. pkcs11:object=kms_id_1. It seems to work properly for me based on some brief testing.

Actually, scratch that. I came around to your original suggestion. Multiple KMS keys are now represented as multiple slots. This makes it a lot easier to select a single KMS key from the list when you do a signing operation without having to do a GetPublicKey call for each one. This will also make it easier to add a certificate to the slot.

Ah awesome! Great work. I noticed you implemented label as well. I was going to suggest to add that ;) The new slots array is handy because it makes it easy to add additional items later if extra functionality is added.
I'm just testing it now (I generate the json dynamically from environment variables for my docker so just writing that script). Will post back with the results.

I think there is a bug in the code because key region and key id is getting mixed up in the code somewhere. You can see my config here:
{
  "slots": [
    {
      "kms_key_id": "xxxxxxxxxx",
      "label": "development",
      "aws_region": "us-west-1"
    }
  ]
}

And see the debug log here:
AWS_KMS_PKCS11_DEBUG=1 openssl
OpenSSL> engine pkcs11 -pre VERBOSE -pre MODULE_PATH:/usr/lib/x86_64-linux-gnu/pkcs11/aws_kms_pkcs11.so
(pkcs11) pkcs11 engine
[Success]: VERBOSE
[Success]: MODULE_PATH:/usr/lib/x86_64-linux-gnu/pkcs11/aws_kms_pkcs11.so
OpenSSL> pkeyutl -engine pkcs11 -sign -inkey 'pkcs11:token=development' -keyform engine -out foo.sig -in foo
engine "pkcs11" set.
PKCS#11: Initializing the engine
AWS_KMS: Debug enabled.
AWS_KMS: Attempting to load config from path: /root/.config/aws-kms-pkcs11/config.json
AWS_KMS: Skipping config because we couldn't open the file.
AWS_KMS: Attempting to load config from path: /etc/aws-kms-pkcs11/config.json
AWS_KMS: Configured slots:
AWS_KMS:   us-west-1
Found 1 slot
Loading private key "pkcs11:token=development"
Looking in slot -1 for key:
[0]                            no pin            (development)
Found slot:
Found token: development
AWS_KMS: Getting public key for key us-west-1
AWS_KMS: Got error from AWS fetching public key for key id us-west-1: Invalid keyId us-west-1
Loading private key "pkcs11:token=development"
Looking in slot -1 for key:
[0]                            no pin            (development)
Found slot:
Found token: development
No private keys found.
PKCS11_get_private_key returned NULL
cannot load Private Key from engine
140124731147584:error:80067065:pkcs11 engine:ctx_load_privkey:object not found:eng_back.c:858:
140124731147584:error:26096080:engine routines:ENGINE_load_private_key:failed loading private key:../crypto/engine/eng_pkey.c:77:
unable to load Private Key
pkeyutl: Error initializing context
error in pkeyutl

Specifically it shows lines like this. I'll trace the code by eye to see if I can see where it gets mixed up.
AWS_KMS: Getting public key for key us-west-1
AWS_KMS: Got error from AWS fetching public key for key id us-west-1: Invalid keyId us-west-1
AWS_KMS: Configured slots:
AWS_KMS:   us-west-1

Ah, looks like error on this line:
https://github.com/JackOfMostTrades/aws-kms-pkcs11/blob/4422ed3bb3ff49b451f6abbf2955f036470088a9/aws_kms_pkcs11.cpp#L149
if (json_object_object_get_ex(slot_item, "aws_region", &val) && json_object_is_type(val, json_type_string)) {
    kms_key_id = string(json_object_get_string(val));
}

Issued a pull request #6

D'oh. Thanks for the fix. Dangers of copy/pasting code late at night.

I just posted some instructions to setting up p11-tool which is a useful debug tool #7. I noticed that if no config is setup, this seems to list all available keys in my KMS account e.g.
p11tool --list-tokens. I'm not sure if this is intentional?
Token 0:
	URL: pkcs11:model=p11-kit-trust;manufacturer=PKCS%2311%20Kit;serial=1;token=System%20Trust
	Label: System Trust
	Type: Trust module
	Flags: uPIN uninitialized
	Manufacturer: PKCS#11 Kit
	Model: p11-kit-trust
	Serial: 1
	Module: p11-kit-trust.so
Token 1:
	URL: pkcs11:token=1d369ab6-a941-4b21-bd3e-6950f14b
	Label: 1d369ab6-a941-4b21-bd3e-6950f14b
	Type: Generic token
	Flags: uPIN uninitialized
	Manufacturer:
	Model:
	Serial:
	Module: /usr/lib/x86_64-linux-gnu/pkcs11/aws_kms_pkcs11.so
Token 2:
	URL: pkcs11:token=366f107f-887e-4832-b8a9-ba88e76c
	Label: 366f107f-887e-4832-b8a9-ba88e76c
	Type: Generic token
	Flags: uPIN uninitialized
	Manufacturer:
	Model:
	Serial:
	Module: /usr/lib/x86_64-linux-gnu/pkcs11/aws_kms_pkcs11.so
Token 3:
	URL: pkcs11:token=368836f7-e7e7-43cc-aa2a-e575855f
	Label: 368836f7-e7e7-43cc-aa2a-e575855f
	Type: Generic token
	Flags: uPIN uninitialized
	Manufacturer:
	Model:
	Serial:
	Module: /usr/lib/x86_64-linux-gnu/pkcs11/aws_kms_pkcs11.so
Token 4:
	URL: pkcs11:token=525414eb-f898-4769-ab99-2df70708
	Label: 525414eb-f898-4769-ab99-2df70708
	Type: Generic token
	Flags: uPIN uninitialized
	Manufacturer:
	Model:
	Serial:
	Module: /usr/lib/x86_64-linux-gnu/pkcs11/aws_kms_pkcs11.so
Token 5:
	URL: pkcs11:token=62f9d456-a336-439a-aa91-71dbc068
	Label: 62f9d456-a336-439a-aa91-71dbc068
	Type: Generic token
	Flags: uPIN uninitialized
	Manufacturer:
	Model:
	Serial:
	Module: /usr/lib/x86_64-linux-gnu/pkcs11/aws_kms_pkcs11.so
Token 6:
	URL: pkcs11:token=8e6426ad-dbd5-4b86-8446-b8adc503
	Label: 8e6426ad-dbd5-4b86-8446-b8adc503
	Type: Generic token
	Flags: uPIN uninitialized
	Manufacturer:
	Model:
	Serial:
	Module: /usr/lib/x86_64-linux-gnu/pkcs11/aws_kms_pkcs11.so
Token 7:
	URL: pkcs11:token=ddb28995-58d4-4a28-80b6-567aea09
	Label: ddb28995-58d4-4a28-80b6-567aea09
	Type: Generic token
	Flags: uPIN uninitialized
	Manufacturer:
	Model:
	Serial:
	Module: /usr/lib/x86_64-linux-gnu/pkcs11/aws_kms_pkcs11.so

The neat thing about this tool is you can also list the URIs p11tool --list-token-urls
pkcs11:model=p11-kit-trust;manufacturer=PKCS%2311%20Kit;serial=1;token=System%20Trust
pkcs11:token=1d369ab6-a941-4b21-bd3e-6950f14b
pkcs11:token=366f107f-887e-4832-b8a9-ba88e76c
pkcs11:token=368836f7-e7e7-43cc-aa2a-e575855f
pkcs11:token=525414eb-f898-4769-ab99-2df70708
pkcs11:token=62f9d456-a336-439a-aa91-71dbc068
pkcs11:token=8e6426ad-dbd5-4b86-8446-b8adc503
pkcs11:token=ddb28995-58d4-4a28-80b6-567aea09

After setting up a config, it appears to show more expected results:
Token 0:
	URL: pkcs11:model=p11-kit-trust;manufacturer=PKCS%2311%20Kit;serial=1;token=System%20Trust
	Label: System Trust
	Type: Trust module
	Flags: uPIN uninitialized
	Manufacturer: PKCS#11 Kit
	Model: p11-kit-trust
	Serial: 1
	Module: p11-kit-trust.so
Token 1:
	URL: pkcs11:token=devkernel
	Label: devkernel
	Type: Generic token
	Flags: uPIN uninitialized
	Manufacturer:
	Model:
	Serial:
	Module: /usr/lib/x86_64-linux-gnu/pkcs11/aws_kms_pkcs11.so
Token 2:
	URL: pkcs11:token=devrauc
	Label: devrauc
	Type: Generic token
	Flags: uPIN uninitialized
	Manufacturer:
	Model:
	Serial:
	Module: /usr/lib/x86_64-linux-gnu/pkcs11/aws_kms_pkcs11.so

With. urls looking normal too
pkcs11:model=p11-kit-trust;manufacturer=PKCS%2311%20Kit;serial=1;token=System%20Trust
pkcs11:token=devkernel
pkcs11:token=devrauc

if no config is setup, this seems to list all available keys in my KMS account e.g. p11tool --list-tokens. I'm not sure if this is intentional?

Yea, that's some functionality I added after the update to support multiple slots. The only overhead is a call to the KMS list-keys endpoint, so supporting key use without config seemed easy enough to do. It's using the KMS key id as the label by default, although labels can only be 32 characters and UUIDs are 37, so they're truncated. :shrug:
If you do have a configuration there isn't any behavior change (it doesn't do any list-keys call).

Thank you, this seems to be working well for me so far.


PDI-14840 - Having two syslog steps in transformation makes it running forever

Build Completed
:x: This pull request has errors. They will need to be addressed before it can be accepted. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
ant -Dtestreports.dir=bin/reports/unit-test -f engine/build.xml clean-all resolve jacoco && ant -Dtestreports.dir=bin/reports/integration-test -f engine/build.xml jacoco-integration checkstyle publish-local

Cleanup Commands
rm -r ~/.ivy2/local || echo no publish local to remove

Changed files
  engine/ivy.xml
  engine/src/org/pentaho/di/trans/steps/syslog/SyslogMessage.java
  engine/test-src/org/pentaho/di/trans/steps/syslog/SyslogMessageConcurrentTest.java

Newly Broken Tests:
org.pentaho.di.trans.steps.syslog.SyslogMessageTest.testDispose:
:red_circle: java.lang.NullPointerException
java.lang.NullPointerException: this.socket is null
	at org.pentaho.di.trans.steps.syslog.SyslogMessageTest$2.answer(SyslogMessageTest.java:70)
	at org.pentaho.di.trans.steps.syslog.SyslogMessageTest.testDispose(SyslogMessageTest.java:86)

Unit test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.job.entries.evalfilesmetrics.JobEntryEvalFilesMetrics

Branch Change:  + .39%
Complexity Change:  + .51%
Instruction Change:  + .08%

org.pentaho.di.job.entries.getpop.MailConnectionMeta

Branch Change: -1.00%:small_red_triangle_down:
Complexity Change: -1.41%:small_red_triangle_down:
Instruction Change: -.30%:small_red_triangle_down:

org.pentaho.di.job.entries.syslog.SyslogDefs

Branch Change:  + 23.53%
Complexity Change:  + 11.11%
Instruction Change:  + 5.64%
Line Change:  + 14.93%
Method Change:  + 33.33%

org.pentaho.di.trans.steps.calculator.CalculatorMetaFunction

Branch Change: -8.70%:small_red_triangle_down:
Complexity Change: -3.08%:small_red_triangle_down:
Instruction Change: -.75%:small_red_triangle_down:
Line Change: -.89%:small_red_triangle_down:

org.pentaho.di.trans.steps.constant.ConstantMeta

Instruction Change: -.10%:small_red_triangle_down:

org.pentaho.di.trans.steps.csvinput.CsvInputMeta

Instruction Change: -.05%:small_red_triangle_down:

org.pentaho.di.trans.steps.dbproc.DBProcMeta

Instruction Change:  + .11%

org.pentaho.di.trans.steps.groupby.GroupByMeta

Branch Change: -3.12%:small_red_triangle_down:
Complexity Change: -3.16%:small_red_triangle_down:
Instruction Change: -.14%:small_red_triangle_down:
Line Change: -.37%:small_red_triangle_down:

org.pentaho.di.trans.steps.insertupdate.InsertUpdateMeta

Instruction Change:  + .05%

org.pentaho.di.trans.steps.ldapoutput.LDAPOutputMeta

Branch Change: -1.22%:small_red_triangle_down:
Complexity Change: -.85%:small_red_triangle_down:
Instruction Change: -.17%:small_red_triangle_down:

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta

Branch Change: -2.35%:small_red_triangle_down:
Complexity Change: -2.70%:small_red_triangle_down:
Instruction Change: -.18%:small_red_triangle_down:
Line Change: -.48%:small_red_triangle_down:

org.pentaho.di.trans.steps.replacestring.ReplaceStringMeta

Instruction Change:  + .07%

org.pentaho.di.trans.steps.rowgenerator.RowGeneratorMeta

Instruction Change: -.08%:small_red_triangle_down:

org.pentaho.di.trans.steps.setvalueconstant.SetValueConstantMeta

Instruction Change:  + .14%

org.pentaho.di.trans.steps.symmetriccrypto.symmetriccryptotrans.SymmetricCryptoTransMeta

Branch Change:  + 3.33%
Complexity Change:  + 1.85%
Instruction Change:  + .28%

org.pentaho.di.trans.steps.syslog.SyslogMessage

Branch Change:  + 20.00%
Complexity Change:  + 4.76%
Instruction Change:  + 21.71%
Line Change:  + 23.16%
Method Change:  + 16.67%

org.pentaho.di.trans.steps.tableoutput.TableOutputMeta

Branch Change:  + .91%
Complexity Change:  + .82%

org.pentaho.di.trans.steps.uniquerows.UniqueRowsMeta

Instruction Change:  + .18%

org.pentaho.di.trans.steps.webservices.WebServiceMeta

Branch Change:  + 1.79%
Complexity Change:  + 1.19%
Instruction Change:  + .07%

Integration test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.trans.step.BaseStep

Branch Change:  + .29%
Complexity Change:  + .18%
Instruction Change:  + .05%
Line Change:  + .08%

Build Completed
:x: This pull request has errors. They will need to be addressed before it can be accepted. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
ant -Dtestreports.dir=bin/reports/unit-test -f engine/build.xml clean-all resolve jacoco && ant -Dtestreports.dir=bin/reports/integration-test -f engine/build.xml jacoco-integration checkstyle publish-local

Cleanup Commands
rm -r ~/.ivy2/local || echo no publish local to remove

Changed files
  engine/ivy.xml
  engine/src/org/pentaho/di/trans/steps/syslog/SyslogMessage.java
  engine/test-src/org/pentaho/di/trans/steps/syslog/SyslogMessageConcurrentTest.java

Newly Broken Tests:
org.pentaho.di.trans.steps.dimensionlookup.DimensionLookupMetaTest.testSerialization:
:red_circle: java.lang.RuntimeException
java.lang.RuntimeException: Error validating fieldUpdate
	at org.pentaho.di.trans.steps.loadsave.LoadSaveTester.validateLoadedMeta(LoadSaveTester.java:176)
	at org.pentaho.di.trans.steps.loadsave.LoadSaveTester.testXmlRoundTrip(LoadSaveTester.java:223)
	at org.pentaho.di.trans.steps.loadsave.LoadSaveTester.testSerialization(LoadSaveTester.java:182)
	at org.pentaho.di.trans.steps.dimensionlookup.DimensionLookupMetaTest.testSerialization(DimensionLookupMetaTest.java:145)
Caused by: org.pentaho.di.

Unit test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.job.entries.evalfilesmetrics.JobEntryEvalFilesMetrics

Branch Change: -.78%:small_red_triangle_down:
Complexity Change: -1.03%:small_red_triangle_down:
Instruction Change: -.16%:small_red_triangle_down:

org.pentaho.di.job.entries.syslog.SyslogDefs

Branch Change:  + 17.65%
Complexity Change:  + 11.11%
Instruction Change:  + 3.92%
Line Change:  + 8.96%
Method Change:  + 33.33%

org.pentaho.di.trans.steps.checksum.CheckSumMeta

Instruction Change:  + .11%

org.pentaho.di.trans.steps.constant.ConstantMeta

Instruction Change: -.10%:small_red_triangle_down:

org.pentaho.di.trans.steps.dimensionlookup.DimensionLookupMeta

Branch Change: -3.76%:small_red_triangle_down:
Complexity Change: -3.49%:small_red_triangle_down:
Instruction Change: -11.51%:small_red_triangle_down:
Line Change: -9.31%:small_red_triangle_down:
Method Change: -4.08%:small_red_triangle_down:

org.pentaho.di.trans.steps.groupby.GroupByMeta

Branch Change: -1.04%:small_red_triangle_down:
Complexity Change: -1.05%:small_red_triangle_down:

org.pentaho.di.trans.steps.ifnull.IfNullMeta

Branch Change: -2.00%:small_red_triangle_down:
Complexity Change: -1.89%:small_red_triangle_down:
Instruction Change: -.17%:small_red_triangle_down:

org.pentaho.di.trans.steps.ldapoutput.LDAPOutputMeta

Instruction Change:  + .06%

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta

Branch Change:  + 1.18%
Complexity Change:  + 1.35%

org.pentaho.di.trans.steps.rowgenerator.RowGeneratorMeta

Instruction Change:  + .08%

org.pentaho.di.trans.steps.setvalueconstant.SetValueConstantMeta

Instruction Change: -.14%:small_red_triangle_down:

org.pentaho.di.trans.steps.sort.SortRowsMeta

Instruction Change: -.08%:small_red_triangle_down:

org.pentaho.di.trans.steps.sortedmerge.SortedMergeMeta

Instruction Change: -.18%:small_red_triangle_down:

org.pentaho.di.trans.steps.syslog.SyslogMessage

Branch Change:  + 20.00%
Complexity Change:  + 4.76%
Instruction Change:  + 32.88%
Line Change:  + 33.77%
Method Change:  + 16.67%

org.pentaho.di.trans.steps.tableoutput.TableOutputMeta

Branch Change:  + .91%
Complexity Change:  + .82%

org.pentaho.di.trans.steps.textfileinput.TextFileInputMeta

Branch Change: -.53%:small_red_triangle_down:

org.pentaho.di.trans.steps.uniquerows.UniqueRowsMeta

Instruction Change: -.18%:small_red_triangle_down:

org.pentaho.di.trans.steps.webservices.WebServiceMeta

Branch Change: -1.79%:small_red_triangle_down:
Complexity Change: -1.19%:small_red_triangle_down:
Instruction Change: -.07%:small_red_triangle_down:

Build Completed
:x: This pull request has errors. They will need to be addressed before it can be accepted. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
ant -Dtestreports.dir=bin/reports/unit-test -f engine/build.xml clean-all resolve jacoco && ant -Dtestreports.dir=bin/reports/integration-test -f engine/build.xml jacoco-integration checkstyle publish-local

Cleanup Commands
rm -r ~/.ivy2/local || echo no publish local to remove

Changed files
  engine/ivy.xml
  engine/src/org/pentaho/di/trans/steps/syslog/SyslogMessage.java
  engine/test-src/org/pentaho/di/trans/steps/syslog/SyslogMessageConcurrentTest.java

Newly Broken Tests:
org.pentaho.di.trans.steps.dimensionlookup.DimensionLookupMetaTest.testSerialization:
:red_circle: java.lang.RuntimeException
java.lang.RuntimeException: Error validating fieldUpdate
	at org.pentaho.di.trans.steps.loadsave.LoadSaveTester.validateLoadedMeta(LoadSaveTester.java:176)
	at org.pentaho.di.trans.steps.loadsave.LoadSaveTester.testXmlRoundTrip(LoadSaveTester.java:223)
	at org.pentaho.di.trans.steps.loadsave.LoadSaveTester.testSerialization(LoadSaveTester.java:182)
	at org.pentaho.di.trans.steps.dimensionlookup.DimensionLookupMetaTest.testSerialization(DimensionLookupMetaTest.java:145)
Caused by: org.pentaho.di.

Unit test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.core.auth.core.impl.DefaultAuthenticationConsumerFactory

Branch Change: -7.14%:small_red_triangle_down:
Complexity Change: -7.69%:small_red_triangle_down:

org.pentaho.di.job.entries.sql.JobEntrySQL

Instruction Change:  + 3.28%
Line Change:  + 3.87%

org.pentaho.di.job.entries.syslog.SyslogDefs

Branch Change:  + 17.65%
Complexity Change:  + 11.11%
Instruction Change:  + 3.92%
Line Change:  + 8.96%
Method Change:  + 33.33%

org.pentaho.di.trans.step.StepMeta

Instruction Change: -.19%:small_red_triangle_down:
Line Change: -.27%:small_red_triangle_down:

org.pentaho.di.trans.steps.checksum.CheckSumMeta

Branch Change: -1.35%:small_red_triangle_down:
Complexity Change: -1.45%:small_red_triangle_down:
Instruction Change: -.21%:small_red_triangle_down:

org.pentaho.di.trans.steps.constant.ConstantMeta

Instruction Change:  + .10%

org.pentaho.di.trans.steps.csvinput.CsvInputMeta

Instruction Change: -.05%:small_red_triangle_down:

org.pentaho.di.trans.steps.datagrid.DataGridMeta

Instruction Change: -.09%:small_red_triangle_down:

org.pentaho.di.trans.steps.dbproc.DBProcMeta

Instruction Change: -.11%:small_red_triangle_down:

org.pentaho.di.trans.steps.dimensionlookup.DimensionLookupMeta

Branch Change: -3.76%:small_red_triangle_down:
Complexity Change: -3.49%:small_red_triangle_down:
Instruction Change: -11.51%:small_red_triangle_down:
Line Change: -9.31%:small_red_triangle_down:
Method Change: -4.08%:small_red_triangle_down:

org.pentaho.di.trans.steps.groupby.GroupByMeta

Branch Change: -1.04%:small_red_triangle_down:
Complexity Change: -1.05%:small_red_triangle_down:
Instruction Change: -.14%:small_red_triangle_down:
Line Change: -.37%:small_red_triangle_down:

org.pentaho.di.trans.steps.ldapinput.LDAPInputMeta

Branch Change: -1.61%:small_red_triangle_down:
Complexity Change: -.97%:small_red_triangle_down:
Instruction Change: -.09%:small_red_triangle_down:

org.pentaho.di.trans.steps.ldapoutput.LDAPOutputMeta

Instruction Change: -.06%:small_red_triangle_down:

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta

Branch Change: -1.18%:small_red_triangle_down:
Complexity Change: -1.35%:small_red_triangle_down:

org.pentaho.di.trans.steps.rowgenerator.RowGeneratorMeta

Instruction Change:  + .08%

org.pentaho.di.trans.steps.singlethreader.SingleThreaderMeta

Instruction Change:  + .08%

org.pentaho.di.trans.steps.sort.SortRowsMeta

Instruction Change: -.08%:small_red_triangle_down:

org.pentaho.di.trans.steps.sortedmerge.SortedMergeMeta

Instruction Change:  + .18%

org.pentaho.di.trans.steps.syslog.SyslogMessage

Branch Change:  + 20.00%
Complexity Change:  + 4.76%
Instruction Change:  + 32.88%
Line Change:  + 33.77%
Method Change:  + 16.67%

org.pentaho.di.trans.steps.systemdata.SystemDataMeta

Branch Change: -2.04%:small_red_triangle_down:
Complexity Change: -2.04%:small_red_triangle_down:
Instruction Change: -.32%:small_red_triangle_down:
Line Change: -.81%:small_red_triangle_down:

org.pentaho.di.trans.steps.tableoutput.TableOutputMeta

Branch Change:  + .91%
Complexity Change:  + .82%

org.pentaho.di.trans.steps.textfileinput.TextFileInputMeta

Branch Change:  + .53%

org.pentaho.di.trans.steps.uniquerows.UniqueRowsMeta

Instruction Change: -.18%:small_red_triangle_down:

org.pentaho.di.trans.steps.webservices.WebServiceMeta

Branch Change: -1.79%:small_red_triangle_down:
Complexity Change: -1.19%:small_red_triangle_down:
Instruction Change: -.07%:small_red_triangle_down:

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
ant -Dtestreports.dir=bin/reports/unit-test -f engine/build.xml clean-all resolve jacoco && ant -f engine/build.xml checkstyle publish-local

Cleanup Commands
rm -r ~/.ivy2/local || echo no publish local to remove

Changed files
  engine/ivy.xml
  engine/src/org/pentaho/di/trans/steps/syslog/SyslogMessage.java
  engine/test-src/org/pentaho/di/trans/steps/syslog/SyslogMessageConcurrentTest.java

Unit test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.core.auth.core.impl.DefaultAuthenticationConsumerFactory

Branch Change:  + 7.14%
Complexity Change:  + 7.69%

org.pentaho.di.job.entries.evalfilesmetrics.JobEntryEvalFilesMetrics

Branch Change:  + .39%
Complexity Change:  + .51%
Instruction Change:  + .08%

org.pentaho.di.job.entries.evaluatetablecontent.JobEntryEvalTableContent

Branch Change: -1.01%:small_red_triangle_down:
Complexity Change: -1.06%:small_red_triangle_down:
Instruction Change: -.17%:small_red_triangle_down:

org.pentaho.di.job.entries.syslog.SyslogDefs

Branch Change:  + 17.65%
Complexity Change:  + 11.11%
Instruction Change:  + 3.92%
Line Change:  + 8.96%
Method Change:  + 33.33%

org.pentaho.di.trans.step.StepMeta

Instruction Change:  + .19%
Line Change:  + .27%

org.pentaho.di.trans.steps.checksum.CheckSumMeta

Instruction Change: -.11%:small_red_triangle_down:

org.pentaho.di.trans.steps.combinationlookup.CombinationLookupMeta

Instruction Change: -.04%:small_red_triangle_down:

org.pentaho.di.trans.steps.constant.ConstantMeta

Instruction Change: -.10%:small_red_triangle_down:

org.pentaho.di.trans.steps.csvinput.CsvInputMeta

Instruction Change:  + .05%

org.pentaho.di.trans.steps.dbproc.DBProcMeta

Instruction Change:  + .11%

org.pentaho.di.trans.steps.groupby.GroupByMeta

Branch Change: -1.04%:small_red_triangle_down:
Complexity Change: -1.05%:small_red_triangle_down:

org.pentaho.di.trans.steps.insertupdate.InsertUpdateMeta

Instruction Change:  + .05%

org.pentaho.di.trans.steps.ldapoutput.LDAPOutputMeta

Instruction Change:  + .06%

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta

Branch Change: -1.18%:small_red_triangle_down:
Complexity Change: -1.35%:small_red_triangle_down:

org.pentaho.di.trans.steps.pgbulkloader.PGBulkLoaderMeta

Branch Change:  + 1.75%
Complexity Change:  + .93%
Instruction Change:  + .23%
Line Change:  + .28%

org.pentaho.di.trans.steps.rowgenerator.RowGeneratorMeta

Instruction Change:  + .08%

org.pentaho.di.trans.steps.setvalueconstant.SetValueConstantMeta

Instruction Change:  + .14%

org.pentaho.di.trans.steps.sortedmerge.SortedMergeMeta

Instruction Change:  + .18%

org.pentaho.di.trans.steps.synchronizeaftermerge.SynchronizeAfterMergeMeta

Instruction Change: -.04%:small_red_triangle_down:

org.pentaho.di.trans.steps.syslog.SyslogMessage

Branch Change:  + 20.00%
Complexity Change:  + 4.76%
Instruction Change:  + 32.88%
Line Change:  + 33.77%
Method Change:  + 16.67%

org.pentaho.di.trans.steps.systemdata.SystemDataMeta

Branch Change: -2.04%:small_red_triangle_down:
Complexity Change: -2.04%:small_red_triangle_down:
Instruction Change: -.32%:small_red_triangle_down:
Line Change: -.81%:small_red_triangle_down:

org.pentaho.di.trans.steps.tableoutput.TableOutputMeta

Branch Change:  + .91%
Complexity Change:  + .82%

org.pentaho.di.trans.steps.textfileinput.TextFileInputMeta

Branch Change:  + .53%

@mdamour1976, @pamval  could you please take a look


problem with optimize_for_inference

hi,
I'm using tensorflow 1.3.0 installed via pip with python 2.7.12 (Ubuntu 16.04, cuda 8, nvidia 1060)
when i try to optimize a custom model trained with tensorflow 1.3.0 with
python -m tensorflow.python.tools.optimize_for_inference --input saved_model.pb --output opt_model.pb --input_names=in --output_names=out
i get the following error:
Traceback (most recent call last):
  File "/usr/lib/python2.7/runpy.py", line 174, in _run_module_as_main
    "__main__", fname, loader, pkg_name)
  File "/usr/lib/python2.7/runpy.py", line 72, in _run_code
    exec code in run_globals
  File "/home/********/tensorflow/lib/python2.7/site-packages/tensorflow/python/tools/optimize_for_inference.py", line 146, in <module>
    app.run(main=main, argv=[sys.argv[0]] + unparsed)
  File "/home/*********/tensorflow/local/lib/python2.7/site-packages/tensorflow/python/platform/app.py", line 48, in run
    _sys.exit(main(_sys.argv[:1] + flags_passthrough))
  File "/home/**********/tensorflow/lib/python2.7/site-packages/tensorflow/python/tools/optimize_for_inference.py", line 83, in main
    input_graph_def.ParseFromString(data)
  File "/home/***********/tensorflow/local/lib/python2.7/site-packages/google/protobuf/message.py", line 185, in ParseFromString
    self.MergeFromString(serialized)
  File "/home/*********/tensorflow/local/lib/python2.7/site-packages/google/protobuf/internal/python_message.py", line 1069, in MergeFromString
    if self._InternalParse(serialized, 0, length) != length:
  File "/home/*********/tensorflow/local/lib/python2.7/site-packages/google/protobuf/internal/python_message.py", line 1105, in InternalParse
    pos = field_decoder(buffer, new_pos, end, self, field_dict)
  File "/home/*********/tensorflow/local/lib/python2.7/site-packages/google/protobuf/internal/decoder.py", line 633, in DecodeField
    if value._InternalParse(buffer, pos, new_pos) != new_pos:
  File "/home/*********/tensorflow/local/lib/python2.7/site-packages/google/protobuf/internal/python_message.py", line 1105, in InternalParse
    pos = field_decoder(buffer, new_pos, end, self, field_dict)
  File "/home/*********/tensorflow/local/lib/python2.7/site-packages/google/protobuf/internal/decoder.py", line 612, in DecodeRepeatedField
    if value.add()._InternalParse(buffer, pos, new_pos) != new_pos:
  File "/home/*********/tensorflow/local/lib/python2.7/site-packages/google/protobuf/internal/python_message.py", line 1105, in InternalParse
    pos = field_decoder(buffer, new_pos, end, self, field_dict)
  File "/home/*********/tensorflow/local/lib/python2.7/site-packages/google/protobuf/internal/decoder.py", line 743, in DecodeMap
    if submsg._InternalParse(buffer, pos, new_pos) != new_pos:
  File "/home/*********/tensorflow/local/lib/python2.7/site-packages/google/protobuf/internal/python_message.py", line 1095, in InternalParse
    new_pos = local_SkipField(buffer, new_pos, end, tag_bytes)
  File "/home/*********/tensorflow/local/lib/python2.7/site-packages/google/protobuf/internal/decoder.py", line 850, in SkipField
    return WIRETYPE_TO_SKIPPER[wire_type](buffer, pos, end)
  File "/home/*********/tensorflow/local/lib/python2.7/site-packages/google/protobuf/internal/decoder.py", line 799, in _SkipGroup
    new_pos = SkipField(buffer, pos, end, tag_bytes)
  File "/home/*********/tensorflow/local/lib/python2.7/site-packages/google/protobuf/internal/decoder.py", line 850, in SkipField
    return WIRETYPE_TO_SKIPPER[wire_type](buffer, pos, end)
  File "/home/*********/tensorflow/local/lib/python2.7/site-packages/google/protobuf/internal/decoder.py", line 814, in _SkipFixed32
    raise _DecodeError('Truncated message.')
google.protobuf.message.DecodeError: Truncated message.

i already did a ' export PROTOCOL_BUFFERS_PYTHON_IMPLEMENTATION=python'
without doing it i got
Traceback (most recent call last):
  File "/usr/lib/python2.7/runpy.py", line 174, in _run_module_as_main
    "__main__", fname, loader, pkg_name)
  File "/usr/lib/python2.7/runpy.py", line 72, in _run_code
    exec code in run_globals
  File "/home/*********/tensorflow/lib/python2.7/site-packages/tensorflow/python/tools/optimize_for_inference.py", line 146, in <module>
    app.run(main=main, argv=[sys.argv[0]] + unparsed)
  File "/home/*********/tensorflow/local/lib/python2.7/site-packages/tensorflow/python/platform/app.py", line 48, in run
    _sys.exit(main(_sys.argv[:1] + flags_passthrough))
  File "/home/*********/tensorflow/lib/python2.7/site-packages/tensorflow/python/tools/optimize_for_inference.py", line 83, in main
    input_graph_def.ParseFromString(data)
google.protobuf.message.DecodeError: Error parsing message

I'm saving the model like:
builder = saved_model_builder.SavedModelBuilder("model"))
builder.add_meta_graph_and_variables(sess,['serve'],signature_def_map= {"model": tf.saved_model.signature_def_utils.predict_signature_def(inputs= {"in" : X }, outputs= {"out": pred_Y })})
builder.save()

i had the same issue with 1.2.1, after this i did the upgrade to 1.3.0 via pip install --upgrade and a retrain.
i have no issues loading and using the model with tensorflow-rust in my test application, but to add some context i got the error
Caused by: java.io.IOException: Not a valid TensorFlow Graph serialization: Invalid GraphDef
at  org.tensorflow.contrib.android.TensorFlowInferenceInterface.loadGraph(TensorFlowInferenceInterface.java:392) at org.tensorflow.contrib.android.TensorFlowInferenceInterface.<init>(TensorFlowInferenceInterface.java:96)

when trying to load the model with tensorflow on android. first i thought this would be because of some unsupported ops, so i went the optimize_for_inference way. but i guess it looks like a protobuf issue
i would appreciate any hint to solve this issue.
with best regards

Did you close this because you found the solution or went another way? I'm having the same message using the SavedModel exported by the Estimator API. Not sure here if the input/output nodes were correctly chosen.

I'm having the same problem. Do you have the solution @kollapsderwellenfunktion ?

I am having the same problem, is there a solution for this?

any solution?


How can I pull an images or Run a container?

[root@VM_58_62_centos ~]# pouch pull centos
centos:latest: resolving      |--------------------------------------|
elapsed: 0.0 s total:   0.0 B (0.0 B/s)
failed to pull image: object required
[root@VM_58_62_centos ~]#
Because I failed to pull an images.I wandered how can i search for an images such as centos7 or ubuntu.
And the documents about the pouch-cli is null.So can guys help me to figure out this problems?

I figured out this problem by the https://github.com/alibaba/pouch/issues/123 . Duplicate #123 close it.
 And we will support pulling images without specify registry address. And we will make user have ability to config default registry.
Please wait a moment for this feature. Thanks again and closing this. @cai980273879


[MTSAT] current script give absurd values

MTsat value should be <1. Currently values are up to 10000 in the WM...

I used Simon MTSAT equation in this commit 989879e75e4494b609020f54cdef7967f07bcbe5
@BeliveauP can you check the equation. Note that there were an erratum in Helms paper.
You can ask @slevyrosetti if you have questions about his implementation

I think with his code we should be able to replace T1w with T1. An option could be added to enable this feature.

Great catch Tanguy! Once this bug is resolved, it would be nice if a test could be written that would have failed under the old method, and passes after the fix, in case it reappears in the future. Or did you discover it while already writing such a test for TRAVIS?

I'm going over all the examples and I saw many errors that I'm solving now. My idea would be to write a test that calls the batch of each Model and compare fitting results.

Has this been resolved? Thanks for catching it!

Ilana has also been looking into computing MT_sat from MP2RAGE. The problem
is that T1 maps generated from MP2RAGE are very different from SPGR T1
maps, so the MT_sat maps also look different. Not sure we have a good
solution for this, but understanding the absurd values is a start.
On Thu, Sep 28, 2017 at 12:27 PM, tanguyduval<EMAIL_ADDRESS>wrote:

I'm pretty confident that current script is correct.
However, that would be better if someone (e.g. Jen) could share a MTsat
dataset with MTsat already computed.
Also, @slevyrosetti https://github.com/slevyrosetti wrote an equation
for Mtsat using T1 from MP2RAGE (instead of T1w). That would be nice if we
could add this option
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/neuropoly/qMRLab/issues/62#issuecomment-332890648,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ab6A2wIUmYcCeLHbXDpnJ7E4kGtnjT0Lks5sm8jqgaJpZM4PWnjA
.

@tanguyduval @ileppe @nstikov
A branch has been created MTsatModif which includes initial modifications.

@tanguyduval @ileppe @nstikov
I've compared MTsat results between Ilana's script and the output of qMRLab, using a dataset she shared with me. The results are the same after changing for percent units in qMRLab: MTsat*100 (in branch MTsatModif). It would be ok to merge the branch.


Support for Veldhoven

Request
Hello, is it possible to add support for Veldhoven?
Information is available at veldhoven.nl/afvalkalender/
Additional information
Voorbeeld adres:
Postcode: 5502DD
Huisnummer: 1

Sorry, I have been quiet busy. I wlll respond asap!

Unfortunatly it's not possible to add Veldhoven currently. They don't provide a proper datasource.


Update Set-OrganizationConfig.md

Issues #8133 and #8131

Docs Build status updates of commit fb43509:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Set-OrganizationConfig.md
:bulb:Suggestion
View (exchange-ps)
Details

exchange/exchange-ps/exchange/Set-OrganizationConfig.md

Line 0, Column 0: [Suggestion-disallowed-html-tag] HTML tag 'CountryOrRegion' isn't allowed. Disallowed HTML poses a security risk and must be replaced with approved Docs Markdown syntax.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Mod won't fix snowlogged grass renders despite absence of Continuity

Hi! I just wanted to report that the mod wouldn't function with or without Continuity on a world with Wilder Wilds installed. I tried to also disable snowlogging in the other mod but it did more harm by breaking the textures so I would rather need this mod to work properly than to destroy a world file...
Below are the list of mods I installed. I am using 1.21.1 with Fabric Loader 0.16.5
modlist.txt
Screenshots:

better snow coverage 0.2.0 only works with sodium 0.6
you need to either use 0.1 of the mod (there weren't any major changes in 0.2 besides supporting sodium 0.6)
or upgrade sodium to 0.6
this should fix your problem, let me know if it doesn't!

Hello! Thanks for replying. I've been informed that the Indium mod will not be needed anymore for Sodium 0.6 and above. Tested out removing Indium and it seemed to have solved my Sodium 0.6 beta crashed and fixed the snowlogging textures as well!
So far, no issues were found during 5 hours of gameplay (without Indium and with Sodium 0.6) since I've done the steps.
Still, thank you so much for your time! I appreciate the help.

Hello! Thanks for replying. I've been informed that the Indium mod will not be needed anymore for Sodium 0.6 and above. Tested out removing Indium and it seemed to have solved my Sodium 0.6 beta crash issues and fixed the snowlogging textures as well!
So far, no issues were found during 5 hours of gameplay (without Indium and with Sodium 0.6) since I've done the steps.
Still, thank you so much for your time! I appreciate the help.


Directory sanity check

I'm submitting a

[x] bug fix
[ ] feature
[ ] documentation addition

What is the current behaviour?
It is reported that running the file copy against a directory on linux fails as it should, but it changes the directory permissions. As per issue #315
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem
Repro steps provided in #315
What is the expected behavior?
Fail action, but not modify behavior. It should also give a valid failure message that a file was expected but a directory was provided.
What is the motivation / use case for changing the behavior?
Bug report
Please tell us about your environment:
Version (comtrya --version): v0.8.8
Operating system: macOS apple silicon Sonoma and Debian Linux

Codecov Report
Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (877a41d) 28.59% compared to head (75ea7f7) 28.57%.

Files
Patch %
Lines

lib/src/actions/file/mod.rs
0.00%
2 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #395      +/-   ##
==========================================
- Coverage   28.59%   28.57%   -0.03%
==========================================
  Files          85       85
  Lines        2301     2303       +2
==========================================
  Hits          658      658
- Misses       1643     1645       +2

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Take a look at plan_walk.
https://github.com/comtrya/comtrya/blob/75ea7f7742e0bdbbd287bd1d68d20db7ff7008b4/lib/src/actions/file/link.rs#L84
We do expect this to work with a to that is a directory.
We should detect this during plan and append the existing filename.
We'll also need to ensure that the directory is correct, as plan() on link currently always grabs the parent.
Lastly, we have multiple fields to and target and from and source.
This could be cleaned up and a serde alias added to target and source to cover the older names


dephell project cd command

Store all projects (directories where dephell was runned with config) paths in the textfile in dephell's datadir and perform search for the best path by command
$ dephell project cd test

~/local/dephell/paths.txt:
/home/gram/Documents/lol/
/home/gram/Documents/test/
/home/gram/Documents/nani/

In this example dephell will cd into /home/gram/Documents/test/

I would like to take a shot at this for Hacktoberfest

Great! Keep going :)

For clarification, not having used it before, does dephell currently store paths.txt? If not, how would you want it formatted/where should it be stored within the dephell data dir?

Right, we have no it yet. Use, TextCache('paths'). It has load and dump methods that work with list of string. Exactly what you need :+1:


Tweets with Image

I am trying to show user tweets in tableview for iphone app.
Some tweets has image but I can not display them. (I can show text) How can I show it?
PS: I am getting tweet like:
 var tweetList : [String] = []

    let failureHandler: ((NSError) -> Void) = { error in
        self.alertWithTitle("Error", message: error.localizedDescription)
    }

    self.swifter.getStatusesUserTimelineWithUserID(userId, count: 5, success: { status in

        for item in status!{
            tweetList.append(item["text"].string!)
        }

        completionHandler(returnTweets: tweetList)

        }, failure: failureHandler)

In your current setup, you're only fetching the and returning the tweet text from the statuses JSON. If you look at the Twitter API documentation on Tweet Objects, you'll find out that the images are stored in what we call an entity as a URL (ref: Entities in Objects)
What you'll need it the media entity fetched from the JSON
if let urlString = status["entities"]["media"]["media_url_https"].string {
  imageURL = NSURL(string: urlString)!
}

After retrieving the URL, you can do the following to get the image:
let data = NSData(contentsOfURL: imageURL)
let image = UIImage(data: data)

On a sidenote, I'd recommend creating storing the tweetList as either an Array of JSONValue or create your own Tweet wrapper class. Additionally, your code for fetching tweets can be simplified (and improved) using the CollectionType method  .map :smile:
self.swifter.getStatusesUserTimelineWithUserID(userId, count: 5, success: { statuses in
    let tweetList = statuses.map { $0["text"].string! }
    completionHandler(returnTweets: tweetList)
    }, failure: failureHandler)

Thanks for your help  @meteochu


adjust equivalence tolerance in FUA comparison

This PR:

bumps FUA equivalence tolerances:

Auckland ---
Salt Lake City ---
Liège ---

resolves #75

Superseded by #78


Docs.MS CI Updates for Azure.Security.KeyVault.Keys

CI Update for release of Azure.Security.KeyVault.Keys

Docs Build status updates of commit 4c71bbc:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

bundlepackages/azure-dotnet-preview.csv
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Ability to use private registry

Is your feature request related to a problem? Please describe.
The current installation method requires internet access.
Describe the solution you'd like
Provide the capability to use a private registry when initializing the k3s server and agent.
Describe alternatives you've considered
Use crictl to pull the images from a private registry (unproven at this point).
Additional context
Add any other context or screenshots about the feature request here.

@Prodian0013 in general k8s can support private repos through pull secrets.  Are you looking for a way to either replicate the images need for k3s in your private repo or will https://github.com/rancher/k3s/issues/92 suffice?

@Prodian0013 in general k8s can support private repos through pull secrets. Are you looking for a way to either replicate the images need for k3s in your private repo or will #92 suffice?

@ibuildthecloud our current implementation uses docker and kubeadm to init the cluster.    Prior to the kube init each image is preloaded on each node from a private registry and tagged to the original image name.  Of course this is all automated.

It would be great to have this working with internal private registries, both with and without auth, for CI pipelines and such. I use registry:2 for these scenarios.

The support for private registry can be done from containerd template but its complicated so proposing this design to this feature:
Private registry Support Design #1

Adding couple of flags to support supplying of private registry address/auth info
k3s will use these flag to set up containerd template to support the private registry
k3s also should support docker mode and configure private registry for docker

Private registry Support Design #2

Adding a flag that points to  config file to support multiple private registries with their authentication, for example:

- address: x.x.x.x
  username: xxxxx
  password: xxxxxx
- address: y.y.y.y
  auth: xxxxxxxxx

k3s will use parse this config file and modify the containerd template accordingly
k3s also should support docker mode and configure private registry for docker

@galal-hussein Design 2 sounds better given that there are password/secrets involved.  I would just default the file to /etc/rancher/k3s/registries.yaml.

Verified using master.

So… how can we use this? there's no documentation for this feature and file format of that registries.yaml is unknown :(

This is documented here: https://rancher.com/docs/k3s/latest/en/installation/airgap/#create-registry-yaml
We'll look into documenting separately from an air-gap scenario. Registry cannot be insecure (containerd requires this).

What sort of mechanisms do we have to easily and automatically provision a private registry across all nodes?
I’m asking this because I was previously using an insecure private registry running on he master node, and all I had to do was copy the old configuration file across. Now to have an air-gapped cluster I have to do the entire CA rigmarole and copy across certificate files as well.
Are we sure that containerd does not support HTTP-only registries at all?

On 22 Nov 2019, at 17:05, David Nuzik<EMAIL_ADDRESS>wrote:

This is documented here: https://rancher.com/docs/k3s/latest/en/installation/airgap/#create-registry-yaml
We'll look into documenting separately from an air-gap scenario. Registry cannot be insecure (containerd requires this).
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

@davidnuzik thanks! I was able to recreate config from source code ;)
But yeah… I didn't know about that air-gap, still don't know what's that and separating is good idea!

@morsik Here is a sample registries.yaml content that works for me -
mirrors:   private.registry.com:     endpoint:     - https://private.registry.com configs:   private.registry.com:     auth:       username: <username>       password: <password>

It looks like this will break when you supply a registry password that is a json key, i.e _json_key for gcr. Any good options there?

Yeah, I'm trying to figure out how to give agent nodes access to a private GCR with a service account using docker as the container manager. Ideally it would there would just be a secret in the cluster that could be referenced via imagePullSecrets:. Otherwise, how does one add a GCP service account json file to a registries.yaml file?

Hi all,
I am trying to use docker registry with my k3s. I have docker registry running in a different server (dockerregistryip) and the k3s master in another (k3smasterip)
I have followed the instruction of setting up the docker login and certs as in the documentation and it worked.
I can push and pull the images. I did the following setup with docker login myregistrydomain.com:443.
This was done in k3smasterip
vi /etc/hosts
(dockerregistryip) myregistrydomain.com

vi /etc/docker/daemon.json
{
  "insecure-registries" : ["https://myregistrydomain.com:443"]
}

mkdir /etc/docker/certs.d
mkdir /etc/docker/certs.d/myregistrydomain.com

cp certs/domain.crt /etc/docker/certs.d/myregistrydomain.com/ca.crt
cp certs/domain.key /etc/docker/certs.d/myregistrydomain.com/
cp certs/domain.crt /etc/docker/certs.d/myregistrydomain.com/domain.cert

cp certs/domain.crt /usr/local/share/ca-certificates/myregistrydomain.com.crt
update-ca-certificates

sudo systemctl restart docker
sudo docker info

For k3s, I followed the instruction for WITH TLS and AUTHENTICATION
mirrors:
  myregistrydomain.com:443:
    endpoint:
      - "https://myregistrydomain.com:443"
configs:
  "myregistrydomain.com:443":
     auth:
       username: username
       password: password
     tls:
       cert_file: "/etc/docker/certs.d/myregistrydomain.com/domain.cert"
       key_file: "/etc/docker/certs.d/myregistrydomain.com/domain.key"
       ca_file: "/etc/docker/certs.d/myregistrydomain.com/ca.crt"

When I do a deployment with the image which is there in repository I get the following error
apiVersion: apps/v1
kind: Deployment
...
spec:
      containers:
      - name: imagename
        image: myregistrydomain.com:443/repo/imagename:tag

ERROR:
ImagePullBackOff
DETAILS:
Failed to pull image "myregistrydomain.com:443/repo/imagename:tag": rpc error: code = Unknown desc = failed to pull and unpack image "myregistrydomain.com:443/repo/imagename:tag": failed to resolve reference "myregistrydomain.com:443/repo/imagename:tag": failed to do request: Head "https://myregistrydomain.com:443/v2/repo/imagename/manifests/tag": x509: certificate relies on legacy Common Name field, use SANs or temporarily enable Common Name matching with GODEBUG=x509ignoreCN=0
Have I misconfigured k3s? because docker in k3master server can push and pull images from dockerregistryip. Your help will be really appreciated.

Hi all,
I am trying to use docker registry with my k3s. I have docker registry running in a different server (dockerregistryip) and the k3s master in another (k3smasterip)
I have followed the instruction of setting up the docker login and certs as in the documentation and it worked.
I can push and pull the images. I did the following setup with docker login myregistrydomain.com:443.
This was done in k3smasterip
vi /etc/hosts
(dockerregistryip) myregistrydomain.com

vi /etc/docker/daemon.json
{
  "insecure-registries" : ["https://myregistrydomain.com:443"]
}

mkdir /etc/docker/certs.d
mkdir /etc/docker/certs.d/myregistrydomain.com

cp certs/domain.crt /etc/docker/certs.d/myregistrydomain.com/ca.crt
cp certs/domain.key /etc/docker/certs.d/myregistrydomain.com/
cp certs/domain.crt /etc/docker/certs.d/myregistrydomain.com/domain.cert

cp certs/domain.crt /usr/local/share/ca-certificates/myregistrydomain.com.crt
update-ca-certificates

sudo systemctl restart docker
sudo docker info

For k3s, I followed the instruction for WITH TLS and AUTHENTICATION
mirrors:
  myregistrydomain.com:443:
    endpoint:
      - "https://myregistrydomain.com:443"
configs:
  "myregistrydomain.com:443":
     auth:
       username: username
       password: password
     tls:
       cert_file: "/etc/docker/certs.d/myregistrydomain.com/domain.cert"
       key_file: "/etc/docker/certs.d/myregistrydomain.com/domain.key"
       ca_file: "/etc/docker/certs.d/myregistrydomain.com/ca.crt"

When I do a deployment with the image which is there in repository I get the following error
apiVersion: apps/v1
kind: Deployment
...
spec:
      containers:
      - name: imagename
        image: myregistrydomain.com:443/repo/imagename:tag

ERROR:
ImagePullBackOff
DETAILS:
Failed to pull image "myregistrydomain.com:443/repo/imagename:tag": rpc error: code = Unknown desc = failed to pull and unpack image "myregistrydomain.com:443/repo/imagename:tag": failed to resolve reference "myregistrydomain.com:443/repo/imagename:tag": failed to do request: Head "https://myregistrydomain.com:443/v2/repo/imagename/manifests/tag": x509: certificate relies on legacy Common Name field, use SANs or temporarily enable Common Name matching with GODEBUG=x509ignoreCN=0
Have I misconfigured k3s? because docker in k3master server can push and pull images from dockerregistryip. Your help will be really appreciated.

Hi all,
I am trying to use docker registry with my k3s. I have docker registry running in a different server (dockerregistryip) and the k3s master in another (k3smasterip)
I have followed the instruction of setting up the docker login and certs as in the documentation and it worked.
I can push and pull the images. I did the following setup with docker login myregistrydomain.com:443.
This was done in k3smasterip
vi /etc/hosts
(dockerregistryip) myregistrydomain.com

vi /etc/docker/daemon.json
{
  "insecure-registries" : ["https://myregistrydomain.com:443"]
}

mkdir /etc/docker/certs.d
mkdir /etc/docker/certs.d/myregistrydomain.com

cp certs/domain.crt /etc/docker/certs.d/myregistrydomain.com/ca.crt
cp certs/domain.key /etc/docker/certs.d/myregistrydomain.com/
cp certs/domain.crt /etc/docker/certs.d/myregistrydomain.com/domain.cert

cp certs/domain.crt /usr/local/share/ca-certificates/myregistrydomain.com.crt
update-ca-certificates

sudo systemctl restart docker
sudo docker info

For k3s, I followed the instruction for WITH TLS and AUTHENTICATION
mirrors:
  myregistrydomain.com:443:
    endpoint:
      - "https://myregistrydomain.com:443"
configs:
  "myregistrydomain.com:443":
     auth:
       username: username
       password: password
     tls:
       cert_file: "/etc/docker/certs.d/myregistrydomain.com/domain.cert"
       key_file: "/etc/docker/certs.d/myregistrydomain.com/domain.key"
       ca_file: "/etc/docker/certs.d/myregistrydomain.com/ca.crt"

When I do a deployment with the image which is there in repository I get the following error
apiVersion: apps/v1
kind: Deployment
...
spec:
      containers:
      - name: imagename
        image: myregistrydomain.com:443/repo/imagename:tag

ERROR:
ImagePullBackOff
DETAILS:
Failed to pull image "myregistrydomain.com:443/repo/imagename:tag": rpc error: code = Unknown desc = failed to pull and unpack image "myregistrydomain.com:443/repo/imagename:tag": failed to resolve reference "myregistrydomain.com:443/repo/imagename:tag": failed to do request: Head "https://myregistrydomain.com:443/v2/repo/imagename/manifests/tag": x509: certificate relies on legacy Common Name field, use SANs or temporarily enable Common Name matching with GODEBUG=x509ignoreCN=0
Have I misconfigured k3s? because docker in k3master server can push and pull images from dockerregistryip. Your help will be really appreciated.

Solved it by adding insecure_skip_verify flag to true. I think this is needed for self-signed certs.

Hi all,
I am trying to use docker registry with my k3s. I have docker registry running in a different server (dockerregistryip) and the k3s master in another (k3smasterip)
I have followed the instruction of setting up the docker login and certs as in the documentation and it worked.
I can push and pull the images. I did the following setup with docker login myregistrydomain.com:443.
This was done in k3smasterip
vi /etc/hosts
(dockerregistryip) myregistrydomain.com

vi /etc/docker/daemon.json
{
  "insecure-registries" : ["https://myregistrydomain.com:443"]
}

mkdir /etc/docker/certs.d
mkdir /etc/docker/certs.d/myregistrydomain.com

cp certs/domain.crt /etc/docker/certs.d/myregistrydomain.com/ca.crt
cp certs/domain.key /etc/docker/certs.d/myregistrydomain.com/
cp certs/domain.crt /etc/docker/certs.d/myregistrydomain.com/domain.cert

cp certs/domain.crt /usr/local/share/ca-certificates/myregistrydomain.com.crt
update-ca-certificates

sudo systemctl restart docker
sudo docker info

For k3s, I followed the instruction for WITH TLS and AUTHENTICATION
mirrors:
  myregistrydomain.com:443:
    endpoint:
      - "https://myregistrydomain.com:443"
configs:
  "myregistrydomain.com:443":
     auth:
       username: username
       password: password
     tls:
       cert_file: "/etc/docker/certs.d/myregistrydomain.com/domain.cert"
       key_file: "/etc/docker/certs.d/myregistrydomain.com/domain.key"
       ca_file: "/etc/docker/certs.d/myregistrydomain.com/ca.crt"

When I do a deployment with the image which is there in repository I get the following error
apiVersion: apps/v1
kind: Deployment
...
spec:
      containers:
      - name: imagename
        image: myregistrydomain.com:443/repo/imagename:tag

ERROR:
ImagePullBackOff
DETAILS:
Failed to pull image "myregistrydomain.com:443/repo/imagename:tag": rpc error: code = Unknown desc = failed to pull and unpack image "myregistrydomain.com:443/repo/imagename:tag": failed to resolve reference "myregistrydomain.com:443/repo/imagename:tag": failed to do request: Head "https://myregistrydomain.com:443/v2/repo/imagename/manifests/tag": x509: certificate relies on legacy Common Name field, use SANs or temporarily enable Common Name matching with GODEBUG=x509ignoreCN=0
Have I misconfigured k3s? because docker in k3master server can push and pull images from dockerregistryip. Your help will be really appreciated.

Solved it by adding insecure_skip_verify flag to true. I think this is needed for self-signed certs.

Solved it by adding insecure_skip_verify flag to true. I think this is needed for self-signed certs.

No - the error message tells you what the problem is: x509: certificate relies on legacy Common Name field, use SANs or temporarily enable Common Name matching with GODEBUG=x509ignoreCN=0
You need to update the cert on your registry to include the hostname in its Subject Alternative Names. Go 1.15 no longer supports certs without a SAN.

Solved it by adding insecure_skip_verify flag to true. I think this is needed for self-signed certs.

No - the error message tells you what the problem is: x509: certificate relies on legacy Common Name field, use SANs or temporarily enable Common Name matching with GODEBUG=x509ignoreCN=0
You need to update the cert on your registry to include the hostname in its Subject Alternative Names. Go 1.15 no longer supports certs without a SAN.

Where do we add the insecure_skip_verify flag? Thanks!

@bvhoang2003 https://rancher.com/docs/k3s/latest/en/installation/private-registry/#configs


Error while executing yarn build

Following instructions exactly.

git clone --recurse-submodules https://github.com/ProjectOpenSea/seadrop && cd seadrop [success]
yarn install [success]
yarn build | failed with error below.

Generating typings for: 36 artifacts in dir: typechain-types for target: ethers-v5
An unexpected error occurred:

TypeError: Cannot convert a Symbol value to a string
    at Array.join (<anonymous>)
    at GlobSync._process (<root>\seadrop\node_modules\typechain\node_modules\glob\sync.js:113:53)
    at new GlobSync (<root>\seadrop\node_modules\typechain\node_modules\glob\sync.js:46:10)
    at globSync (<root>\seadrop\node_modules\typechain\node_modules\glob\sync.js:24:10)
    at <root>\seadrop\node_modules\typechain\src\utils\glob.ts:6:13
    at Array.map (<anonymous>)
    at glob (<root>\seadrop\node_modules\typechain\src\utils\glob.ts:5:35)
    at SimpleTaskDefinition.action (<root>\seadrop\node_modules\@typechain\hardhat\src\index.ts:69:26)
    at Environment._runTaskDefinition (<root>\seadrop\node_modules\hardhat\src\internal\core\runtime-environment.ts:308:35)
    at Environment.run (<root>\seadrop\node_modules\hardhat\src\internal\core\runtime-environment.ts:156:25)
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Same problem for me as well.
TypeError: Cannot convert a Symbol value to a string

For WIndows users, I recommend simply using WSL (Windows Subsystem for Linux).  I could not reproduce these issues in any Linux/Unix environment.

Install/Update WSL
Install NVM & Set Version to 16
Make sure you have your SSH key added to the SSH agent (so --recurse-submodules will work)
Go through README in this project.


New Release

Need to get a CHANGELOG record in.

Hold on merging until #97 is completed and included. This should be a minor release with the work done by @chadell

@jvanderaa are we still waiting on something for this except for review by @jmcgill298?

Let's do this today regardless. I'm hoping that we can get #56 across the finish line and included. But we should just get the new release out.

Let's' get this out, we can do another release. If you approve I'll get @jmcgill298 to put a check on it too.


[HUDI-5786] Add a new config to specific spark write rdd storage level

Change Logs
In BaseSparkCommitActionExecutor.java, This RDD is hardcoded as persist(MEMORY_AND_DISK_SER)
// TODO: Consistent contract in HoodieWriteClient regarding preppedRecord storage level handling
    JavaRDD<HoodieRecord<T>> inputRDD = HoodieJavaRDD.getJavaRDD(inputRecords);
    if (inputRDD.getStorageLevel() == StorageLevel.NONE()) {
      inputRDD.persist(StorageLevel.MEMORY_AND_DISK_SER());
    } else {
      LOG.info("RDD PreppedRecords was persisted at: " + inputRDD.getStorageLevel());
    }

But if we want to change its storage level, we have no way to use a parameter.
Impact
Add a new config: hoodie.spark.write.storage.level.
To specific the storage level of this RDD.

the config's default value is MEMORY_AND_DISK_SER

If set the config to another storage level
"hoodie.spark.write.storage.level": "DISK_ONLY"

If the storage level is wrong, It will throw exception
like set "hoodie.spark.write.storage.level": "DISKE_ONLY"
Caused by: java.lang.IllegalArgumentException: Invalid StorageLevel: DISKE_ONLY

Risk level (write none, low medium or high below)
Low
Documentation Update
Describe any necessary documentation update if there is any new feature, config, or user-facing change

The config description must be updated if new configs are added or the default value of the configs are changed
Any new feature or user-facing change requires updating the Hudi website. Please create a Jira ticket, attach the
ticket number here and follow the instruction to make
changes to the website.

Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

@hudi-bot run azure

LGTM.

Only config changed with the same default value as before.
All UT/IT passed.

Low impact. Merged now.


bug in navbar

In home section there is no effect or animations on navbar so do not add any interactivity to user without clicking but when we move to other pages it gives a hover effect of color change but not looks good also the font size is not matching with website

A clear and concise description of what the bug is.

To Reproduce

Steps to reproduce the behavior:

Go to homepage
Click on navbar items,you can see there is no interactivity
Moving to other pages like safety laws there is bug in emergency section which is not good looking gives a transparent white color.
Also font size is wrong and more is bolder than other heading on navbar

Expected behavior
There should be hover effect in icons and also in dropdown menu and remove bug in emergency section.
Possible solution
----I can change completely the navbar of homepage ,adds interactivity ,changes font size and also color combination themes,responsiveness,adding some hovering effects,removes bug of emergency section
Screenshots

Please assign this issue to me @swapnilsparsh

@Muskan-creator Go ahead.


Wallet error code: 51 Wallet error detail: You have not registered to use this API

I am getting following error for Android Pay in my Production App:
Wallet error code: 51
Wallet error detail: You have not registered to use this API.
This is happening while getting MaskedWallet class's instance from MaskedWalletRequest as mentioned in below code snippet:
MaskedWalletRequest maskedWalletRequest = MaskedWalletRequest.newBuilder()
.setPaymentMethodTokenizationParameters(parameters)
.addAllowedCardNetworks(allowedCardNetworks)
.setMerchantName(AppConstants.MERCHANT_NAME)
.setCurrencyCode(currencyCode)
.setEstimatedTotalPrice(totalPrice + "")
.setCart(cart = generateCart(currencyCode))
.build();
Wallet.Payments.loadMaskedWallet(googleApiClient, maskedWalletRequest, AppConstants.MASKED_WALLET_REQUEST_CODE);
After this the Android Pay Card chooser dialog appears for a moment and immediately it gets disappear after checking the logs, I am getting this : wallet error 51.
I have used Braintree as payment processor.
Is there anything I am missing? Any help is apprecieated

@mithilwcities please do not post the same comment on many issues.
Have you signed up to be an Android Pay merchant using this form: https://support.google.com/androidpay/merchant/contact/api_interest?rd=1 ?

yes

@mithilwcities I have reported this to the Android Pay team and we will try to find someone to help you with this issue.

I think it's possible you may not be clear on the use of that interest form.  That form expresses your interest in using Android Pay in production however the process is not instantaneous.  You should be notified when you are whitelisted.
In the future we hope to make this a quicker process.

I've got exactly the same error.
Something has changed here?

Thank you for taking the time to submit this issue, and I apologize that we never got around to solving it. I am closing this issue because the Android Pay API has been superseded by the Google Payments API.
This sample is deprecated and is no longer maintained. Please see the paymentsapi-quickstart sample for an up-to-date sample.
If you believe your issue is relevant to that sample as well, please feel free to open a new issue there and we can continue discussing.


fixes scroll restoration after native navigation to another sveltekit route return

Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[x] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[ ] Ideally, include a test that fails without this PR but passes with it.

Tests

[x] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[ ] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpx changeset and following the prompts. All changesets should be patch until SvelteKit 1.0

Closes #4386
Probably needs test.

I was about to look into this issue as well.
I think it would be far easier to change scroll_positions into an array and set the .length property to current_history_index + 1 when clicking a link or using goto.

@Rich-Harris what do you think about this?

@PatrickG I don't think that what you suggested would solve #4386 because we need to make collisions impossible, and that won't do it, this will because it uses random keys.

And in case I wasn't really clear in the issue, the issue goes like this:

go to some Kit app (the history.state would be like this: {sveltekit:index: 0}) and scroll to the end/bottom of the page
you click on some internal link in that app and for some reason(maybe router=false, maybe app version changed etc.) navigation is handled by the browser native navigation
after that the history.state would again be {sveltekit:index: 0} and that would cause overwrite of position state.
go back using browser back button, you would't be returned to the bottom of the page where scroll position should be because you left it there in step 1. because it was overwritten by the second navigation.

And in case I wasn't really clear in the issue, the issue goes like this:

go to some Kit app (the history.state would be like this: {sveltekit:index: 0}) and scroll to the end/bottom of the page
you click on some internal link in that app and for some reason(maybe router=false, maybe app version changed etc.) navigation is handled by the browser native navigation
after that the history.state would again be {sveltekit:index: 0} and that would cause overwrite of position state.
go back using browser back button, you would't be returned to the bottom of the page where scroll position should be because you left it there in step 1. because it was overwritten by the second navigation.

Alright, then i misunderstood the issue.
I thought the issue is:

start page (index 0): click link -> second page (index 1)
scroll down on second page (index 1)
go back to start page (index 0)
klick another link -> thrid page (index 1) -> page scrolls down unintentionally

I agree we should match the browser behavior and not share scroll state across tabs. FYI, Rich is on vacation this week

Your solution does fix that as well.

I thought that was working already, do you have a example where it wasn't?

@benmccann is it okay to merge this, it works locally, I tested it and it works, and it's really difficult to add test for this because it's pretty edge-case-y.

@Rich-Harris can you check this out?

@Rich-Harris can you merge this, or just say why it's not merged.

Can you add a test?

@Rich-Harris It's pretty hard thing to test for, because the bug happens in some specific conditions, that are hard to create in testing environment. But if you check out the code, you can see that there is not any way that this code could break anything, and I checked out the deployment https://kit-svelte-dev-zu82n3ukp-svelte.vercel.app/ and it works in 3 of my browsers(Edge, Firefox, Chrome)

@Rich-Harris but I can hope we can merge it anyway(without the test)

I haven't managed to write a test that fails on master, but I was able to simply the implementation somewhat. There's no need to maintain SCROLL_INDEX_KEY separately — all we need to do is ensure that cross-document navigations within the same app (or between two SvelteKit apps) don't corrupt the scroll position history, which is easy to do by offsetting the history index. Closing this in favour of #4640 — thanks


Science needs to be reworked in Reach

Description

This mostly pertains to the current artifact chamber. It is impossible to get the artifact in or out without destroying one of the thin plasma windows (preventing you from spacing the artifact) or destroying the table. This is because if you moved the thin plasma window to the inside of the artifact chamber, then it will still not allow the big artifacts to go onto the pad. The best way to fix this issue would be reshifting everything in the room.
Screenshots

could maybe move the tables and rack, but yeah, reach probably needs some love.

im probably gonna post more issues since ive been playing a lot of lowpop and noticed all of these really big issues


Add support for 2 additional licenses for Government

Add support for 2 additional licenses in the Licenses tab. Exchange Online Archiving for Exchange Online for Government & Project Online Professional for Government. See attached files for before & after change in the HTML output. I went with the exact names listed in Office365, but did notice that "Exchange Online Archiving for Exchange Online for Government" offsets the color in the charts due to length.

Awesome work!


How to properly retry workers if zeebe goes down?

I am running zeebe in my local docker container, and I am trying to figure out how to reset the worker long poll if zeebe were to restart. onConnnectionError doesn't get triggered when zeebe goes down, so I don't know what to listen to.

Version 0.23.3 of the client uses 1.0.4 of @grpc/grpc-js, to pull in https://github.com/grpc/grpc-node/issues/144

@jwulf  when is 0.23.3 expected to be released?

It's been out for a while now, and this issue should be resolved. I'm going to close this issue, and if it re-presents, please reopen it.


[FLINK-32539][tests] Update violations as a result of FLINK-27415 changes

What is the purpose of the change
The PR updates violations after changes done at https://github.com/apache/flink/pull/20006
the changes is a result of
mvn verify -pl flink-architecture-tests/flink-architecture-tests-production/ -Darchunit.freeze.store.default.allowStoreUpdate=true

Verifying this change
This change is a trivial rework
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): ( no)
The public API, i.e., is any changed class annotated with @Public(Evolving): ( no)
The serializers: ( no)
The runtime per-record code paths (performance sensitive): ( no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: ( no)
The S3 file system connector: ( no)

Documentation

Does this pull request introduce a new feature? ( no)
If yes, how is the feature documented? (not applicable )

@snuyanzin thanks for fixing this

I'm curious why the architecture tests didn't fail in #20006.

I guess because the PR was based on commits from Jun 2022 (more than 1 year ago)

that explains it :sweat_smile:

@snuyanzin thank you for fix it.


Estimate Gas Catch

First off, thanks for making this. I'm happy to throw back a commit to this project.

While attempting to use the estimateGas function and getting a failure, I noticed the esimateGas promise was not properly handling the error case.
~While attempting to fix the previous issue, I ran into a cross platform issue with truffle. Simple fix. You can read more here: https://ethereum.stackexchange.com/a/38120~

Hi, thanks! My initial reasoning was that gas estimation should never fail. Do you get an exception when a method in Solidity throws? I remember they changed geth behavior recently, before the change it returned max block gas on error in contracts.
As for truffle, I have it working on Windows via PowerShell and want to keep the default file name. Could you please revert the commit with this change and maybe create a separate issue/PR if you want to discuss?

Hey @buybackoff would you be able to release this fix to npm? I'm currently using a tar ball for this package in my project..

done


Building a resillient image generation pipeline

What does this PR do?
We are using a Gateway to build a Generative AI image generation pipeline that is resilient.
Please review let me know if you have any questions.
@MKhalusova @davanstrien

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

@MKhalusova @davanstrien can you please take a look at this one?

@aravindputrevu Hello, we actually encourage using open-source models only and through open-source libraries since this is Open-source AI Cookbook. Can you please make necessary changes to your notebook?


reorg rust section

Wanted to get an index of a bunch of the rust projects in one place. There's some duplication eg. CosmWasm is also in the VMs section and some stuff might be listed on the tendermint awesome but seems worth having an index into all the rust projects in one place for now ...
Sorry I pushed a few of these items to master first before this PR ...

I'm not entirely sure we want a rust ecosystem section. This will force us to create js and other lang sections. I would advocate against a language ecosystem section that causes duplication.

I suppose the alternative to a rust section would would be an indication after the item [rust]

I'd sya we should add it for now since it accounts for projects not currently included. This whole repo is getting converted to a spreadsheet that generates a JSON object. This will be possible to use in a build command to generate a markdown file that can accomodate a rust section with the use of a rust tag even if there are redundant entries (which is not a problem as long as they don't need to get updated separately)

I'd sya we should add it for now since it accounts for projects not currently included.

the projects cosmos based though. This goes back to a conversation we had around including projects that have forked the product or are not using the sdk in any way but are being included here.

the projects are not cosmos based though

what do we consider "cosmos based"? connected to the SDK somehow? seems too narrow and the items listed here are certainly part of the larger ecosystem.

what do we consider "cosmos based"? connected to the SDK somehow? seems too narrow and the items listed here are certainly part of the larger ecosystem.

Built using the cosmos-sdk is what I would categorize as cosmos based. Implementing IBC does not categorize as cosmos based as well, I would say. Crypto.com using tendermint does not mean in the cosmos ecosystem. If we are categorizing Tendermint based chains as cosmos based then this kind of goes against the idea of being able to use the sdk with other consensus.

Just going to merge this to get the information in. We can discuss anything else in an issue or slack, if need be


[FIXED] : Share Icons clashes with the Quotes Text

PULL REQUEST TEMPLATE
My PR closes #86
👨💻 What did you do ?
I aligned the share buttons horizontally so that it does not overlaps on Quotes Text.

Bugs fixed:

[x] Align the share buttons horizontally
[x] Align the quotes text with share buttons when horizontally aligned

📷 Screenshots

✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title and description of the PR is clear and explains the approach.
[x] I have read the contribution guideline properly and following the criteria.

did you get assigned for this?

your contribution was good and impressive, but pull request get closed due to this false acceptance as you didnt read


Refactor Perm Handler

https://github.com/apache/accumulo/blob/71fc7e77edbd73f21599a9d8b9b9254eb0dc74b9/server/base/src/main/java/org/apache/accumulo/server/security/handler/PermissionHandler.java

Closing.  No real design for this and this was really just for me to test github projects.


Su command for moderators

Allow moderators to run a command as the user they tag instead of wasting time repeatedly instructuing them how to run a command.

We need to understand how this will work with passing context to the command.

Won't add this. Instead we should consider separate commands for mods to quickly dump helpful information.

Won't add this. Instead we should consider separate commands for mods to quickly dump helpful information.


Glow chams & Envmaptint modulation

since someone asked for it, i'm not saying it's not pasted because it is.

this is the ugliest thing i ever seen:
 (*(void(__thiscall**)(int, float, float, float))(*(DWORD*)pVar + 44))((uintptr_t)pVar, col2[0], col2[1], col2[2]);
just use setVectorValue, it is index 11 in iMaterialVar class :)
pVar ->setVectorValue(col2[0], col2[1], col2[2]);

this is the ugliest thing i ever seen:
 (*(void(__thiscall**)(int, float, float, float))(*(DWORD*)pVar + 44))((uintptr_t)pVar, col2[0], col2[1], col2[2]);
just use setVectorValue, it is index 11 in iMaterialVar class :)
pVar->setVectorValue(col2[0], col2[1], col2[2]);

its a contestent for when i saw someone do if (!(!(!(!(!(!(!(!(!(!(!(!(booleanSomething))))))))))))) {}

He just proceeded to make it look even uglier lmao

why is the brace on a new line
why is there padding between the start of a block and the end (if you do a brace this is redundant)
this is being repeated in every block (*(void(__thiscall**)(int, float, float, float))(*(DWORD*)pVar + 44))((uintptr_t)pVar, literally just make it a variable.

lol