s3fs and server side encryption

Hi again, I'm hoping you can point me in the right direction on something. My company IT has applied some kind of global IAM policy where you have to specify a KMS key to upload anything, even if the bucket encrypts everything going in with the same key by default. In Lambda, I can use boto3 to upload like this:
ExtraArgs={'ServerSideEncryption':'aws:kms','SSEKMSKeyId':'arn:aws:kms:[region-keystring]'}

boto3.Session().resource('s3').Bucket([bucket name]).Object([prefix]).upload_file([file here], ExtraArgs=ExtraArgs)

Since awswrangler uses s3fs under the hood, I have been trying to figure out if there is a way to pass the KMS ExtraArgs data as above either through awswrangler session initiation or when calling a method that includes writes to s3 like session.pandas.to_parquet()? These are erroring out when I try to use them and I suspect that is the reason.
I get "Invalid KeyID" in the log output from calling session.pandas.to_parquet() with valid pandas dataframe:
t/python/s3fs/core.py", line 952, in _call_s3
    **kwargs)
  File "/opt/python/s3fs/core.py", line 182, in _call_s3
    return method(**additional_kwargs)
  File "/opt/python/botocore/client.py", line 357, in _api_call
    return self._make_api_call(operation_name, kwargs)
  File "/opt/python/botocore/client.py", line 661, in _make_api_call
    raise error_class(parsed_response, operation_name)
botocore.exceptions.ClientError: An error occurred (KMS.NotFoundException) when calling the CreateMultipartUpload operation: Invalid keyId [name of KMS key]
...

I've also tried doing basic writes from Lambda to S3 just using pure s3fs like this and still get Access Denied error.
s3 = s3fs.S3FileSystem(s3_additional_kwargs=ExtraArgs)
with s3.open(['same bucket and prefix as above/f.txt', 'wb') as f:
    f.write('lalala')

output:
{
  "errorMessage": "Access Denied",
  "errorType": "PermissionError",
  "stackTrace": [
    "  File \"/var/task/lambda_function.py\", line 17, in lambda_handler\n    f.write('lalala')\n",
    "  File \"/opt/python/fsspec/spec.py\", line 1157, in __exit__\n    self.close()\n",
    "  File \"/opt/python/fsspec/spec.py\", line 1123, in close\n    self.flush(force=True)\n",
    "  File \"/opt/python/fsspec/spec.py\", line 995, in flush\n    self._initiate_upload()\n",
    "  File \"/opt/python/s3fs/core.py\", line 966, in _initiate_upload\n    raise translate_boto_error(e)\n"
  ]
}

Any ideas appreciated - thanks!

Hi @mjh7!
I've just runned your s3fs example in a Lambda here and everything worked fine!
import s3fs

def lambda_handler(event, context):
    ExtraArgs={'ServerSideEncryption':'aws:kms','SSEKMSKeyId':'arn:aws:kms:...'}

    s3 = s3fs.S3FileSystem(s3_additional_kwargs=ExtraArgs)
    with s3.open("s3://...", 'w') as f:
        f.write('lalala')

I'm think that you only have some permission missing on your IAM Role, maybe some kms or s3 multipart related action.
I'm suspecting of some multipart permission because following your logs we can track that probably this is the line throwing the exception.
Btw, I will add the ExtraArgs option as a parameter in the Wrangler Session. This way we will able to encrypt our outputs on S3 using KMS for example.
I'm planning to launch a new version with that in the next 2 days.

Hey @mjh7!
I just released the new version 0.0b32 here and in pypi as well.
In this version we have s3fs 0.3.3 and s3_additional_kwargs on Wrangler's Session, so you can simply run:
extra_args = {
    "ServerSideEncryption": "aws:kms",
    "SSEKMSKeyId": "YOUR_KMY_KEY_ARN"
}
session = awswrangler.Session(s3_additional_kwargs=extra_args)
session.pandas.to_parquet(
    path="s3://..."
)

I also figured out what is your permission issue. Basically you need more privileges to run a multipart upload encrypted with KMS (What s3fs uses behind the scenes) than a simples put_object encrypted.
Let me link you with some official documentation:
https://docs.aws.amazon.com/AmazonS3/latest/dev/mpuAndPermissions.html
"To perform a multipart upload with encryption using an AWS KMS key, the requester must have permission to the kms:Encrypt, kms:Decrypt, kms:ReEncrypt*, kms:GenerateDataKey*, and kms:DescribeKey actions on the key. These permissions are required because Amazon S3 must decrypt and read data from the encrypted file parts before it completes the multipart upload."
So you will need to have permission over these action referencing your KMS Key.
"Action": [
    "kms:Decrypt",
    "kms:Encrypt",
    "kms:ReEncryptTo",
    "kms:GenerateDataKey",
    "kms:DescribeKey",
    "kms:ReEncryptFrom"
]


VQA_RAC preprocessing dictionary dimension

Hi,
After following the instruction you provided in 'QCR_PubMedCLIP' folder, we got the following error
/content/PubMedCLIP/QCR_PubMedCLIP loading dictionary from ./data/data_rad/dictionary.pkl loading DAE image data from file: ./data/data_rad/images128x128.pkl loading CLIP image data from file: ./data/data_rad/images250x250.pkl loading DAE image data from file: ./data/data_rad/images128x128.pkl loading CLIP image data from file: ./data/data_rad/images250x250.pkl Traceback (most recent call last):   File "main/main.py", line 85, in <module>     question_classify.load_state_dict(pretrained_model)   File "/usr/local/lib/python3.7/dist-packages/torch/nn/modules/module.py", line 1483, in load_state_dict     self.__class__.__name__, "\n\t".join(error_msgs))) RuntimeError: Error(s) in loading state_dict for classify_model: 	size mismatch for w_emb.emb.weight: copying a param with shape torch.Size([1178, 300]) from checkpoint, the shape in current model is torch.Size([1260, 300]).
And we found this bug is caused by the newly generated dictionary. The size is different from your saved_model. Could you help me with it?

Thanks for reporting this issue.
If by the saved_model you mean the type_classifier models, then you can re-run the https://github.com/sarahESL/PubMedCLIP/blob/main/QCR_PubMedCLIP/main/type_classifier.py script with respect to the datasets you want to use and save the models.
I will also update these models some time soon. But meanwhile, you can go ahead and re-train the models.

@Liang-Di Please look at the comment and discussion at https://github.com/sarahESL/PubMedCLIP/issues/9 .


chore: bdd-nh-import-1576804790 to 0.0.1

chore: Promote bdd-nh-import-1576804790 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Updated Aviral's information

Added my information to the ice breaker template

Nice self introduction! 🎉

@Aviral190694 Could you sign the CLA first?

@yangshun I already did :D


Unexpected token nuxt:3

Hi today i try nuxt by running this command
vue init nuxt/starter nuxt
cd nuxt
yarn install
npm run dev

And getting this error
$ npm run dev

><EMAIL_ADDRESS>dev /home/ferri/Developments/web/nuxt
> nuxt

/home/ferri/Developments/web/nuxt/node_modules/nuxt/bin/nuxt:3
const { join } = require('path')
      ^

SyntaxError: Unexpected token {
    at exports.runInThisContext (vm.js:53:16)
    at Module._compile (module.js:373:25)
    at Object.Module._extensions..js (module.js:416:10)
    at Module.load (module.js:343:32)
    at Function.Module._load (module.js:300:12)
    at Function.Module.runMain (module.js:441:10)
    at startup (node.js:139:18)
    at node.js:974:3

This is my system
$ npm --version
3.10.9

$ yarn --version
0.15.1

OS Freya / Ubuntu 14.04 64 bit

Thanks

+1.
I ran:
vue init nuxt/starter myproject
cd nuxt
npm i
npm run dev

Error:
const { join } = require('path')
      ^

Env:
$ npm --version
4.0.3

Windows 10 Pro x64

Thanks guys for trying Nuxt.js
What is your version of node.js?

Updated nodejs from v4 to v6.x, it worked.

I am getting the same error... using Node v5.12.0 on Mac OS X 10.12

mine
$ node -v
v4.6.0

Should i upgrade to v6?

In the meantime upgrade to v6, I will work on make it compatible starting from node v4

After upgrade to v6 LTS, then its working..
$ node -v
v6.9.1

So the solution just upgrade nodejs :+1:
closed .. Thanks

I wouldn't consider this "closed" as long as it's not working with Node v4.x (which is stated as the minimum required version for nuxt.js)

You're right. I did not close this issue but the todo is on the list :)

It's now fixed in the v0.8.8

I'm getting an error
C:\Users\Jerick\Desktop\WEB-APPS\vue-nuxt\vue-nuxt-app\node_modules\nuxt\lib\core\module.js:14
async ready() {
^^^^^
SyntaxError: Unexpected identifier
at createScript (vm.js:56:10)
at Object.runInThisContext (vm.js:97:10)
at Module._compile (module.js:542:28)
at Object.Module._extensions..js (module.js:579:10)
at Module.load (module.js:487:32)
at tryModuleLoad (module.js:446:12)
at Function.Module._load (module.js:438:3)
at Module.require (module.js:497:17)
at require (internal/module.js:20:19)
at Object. (C:\Users\Jerick\Desktop\WEB-APPS\vue-nuxt\vue-nuxt-app\node_modules\nuxt\lib\co                                                                                re\index.js:2:16)
steps to reproduce:
vue init nuxt-community/starter-template vue-nuxt-app
npm install
npm run dev
Node version 6.11.3 Windows 10

@jeck5895  Nuxt needs Node.js  >= 8


Missing reference (node)

https://github.com/linkedtales/scrapedin/blob/master/src/login.js#L32 where does node come from

Hey @seiflotfy , thanks for reporting! It was probably just wrong code, it should reference element instead, I fixed that and created a new version of this fix, please update scrapedin to v1.0.2 , thank you!


Change distribution actions to use FL_CHANGELOG value by default

if no other release notes or notes path is specified

I added some small comments before it's ready to merge, thank you :+1:

Yeah, that's great :rocket:


refactor: jest configuration

allows Jest to be run from the root of the project.  As Jest supports projects we can use that to run Jest from root of the project folder. "This is great for monorepos".

Also allows the usage of VSCode Extension Jest to work without custom configuration.

removed what appears to be a legacy node version logic in jest.config.base.js. We can use the projects tsconfig.json. Perhaps we need to define the package.json engines for what version of node & yarn this project supports.

generates coverage only on root yarn test and not in each package folder

As yarn workspaces is used in the project there should be no need to reference Jest from the node_modules directory for the test:watch cli. node ../../node_modules/jest/bin/jest.js becomes jest.

Removed some config and arguments that are already defined as the defaults for jest. e.g --env=jsdom

Added root cli yarn test:watch so you can watch all test files in project.

fix jest displayName to match package name.

Looks great to me. The simpler the better 😄
@hadihallak and @christianalfoni do you guys mind taking a look at this one? Just wanna make sure that we're not removing something that was there for some edge case reason.


Since octokit/routes is archived, should this use the OpenAPI Description instead?

:wave: @swinton!
I recently imported this and noticed there were some missing routes. Reading up on https://github.com/octokit/routes, it's deprecated (latest: v30.0.3) in favor of the OpenAPI Description:
https://github.com/github/rest-api-description/
I'm wondering if this repo should use that instead? Not quite sure how much work it would take to port over, but thought to raise this so others who rely on this are aware!

Update: I discovered that OpenAPI is supported by Insomnia (docs) and it's possible to import it directly. For example, the OpenAPI Description for api.github.com is available, and Insomnia will do the Right Thing:tm: by importing its raw version.
At this point, I'm wondering if we should just point folks to this method instead? 💭

@imjohnbo started to PR this: https://github.com/swinton/github-rest-apis-for-insomnia/pull/72
We felt this project was still helpful, since it also generates environment variables.


[HFLM]Use Accelerate's API to reduce hard-coded CUDA code

This PR mainly includes two modifications:

Replace the CUDA harded-coded code cuda:{self.accelerator.local_process_index} with accelerator.device
Add a new parameter gpus to reduce the repeated calls to torch.cuda.device_count()

The following is a log based on evaluations conducted through three different methods since this PR was passed. Thank you for your check. @haileyschoelkopf @lintangsutawika
lm_eval --model hf --model_args pretrained=Qwen1.5-0.5B --tasks arithmetic --device npu:0 --batch_size 64

accelerate launch -m lm_eval --model hf --model_args pretrained=Qwen1.5-0.5B --tasks arithmetic --batch_size 8

lm_eval --model hf --model_args pretrained=Qwen1.5-0.5B,parallelize=True,device='npu:7' --tasks arithmetic --batch_size 32

The following is a log based on evaluations conducted through three different methods since this PR was passed. Thank you for your check. @haileyschoelkopf @lintangsutawika
The following are my command and log screenshot:
lm_eval --model hf --model_args pretrained=Qwen1.5-0.5B --tasks arithmetic --device cuda:0 --batch_size 64

accelerate launch -m lm_eval --model hf --model_args pretrained=Qwen1.5-0.5B --tasks arithmetic --batch_size 64

lm_eval --model hf --model_args pretrained=Qwen1.5-0.5B,parallelize=True --tasks arithmetic --batch_size 64

Test failures unrelated, see #1859


How to get IV that generated by RNNCryptor?

Can i get the IV after generating? because i need IV for using in Laravel : Crypt::encrypt('123456');  , i need it for simulate laravel encryption in iOS Swift 3. and you can see this:  Check it and this: Check it

See FAQ: Can I use RNCryptor to read and write my non-RNCryptor data format?, and following questions.
If you need a PHP implementation of the RNCryptor format, see RNCryptor/RNCryptor-php.


Not working

Support & Bug Report
https://t.me/androidryukimodsdiscussions/2618
If you don't do above, issues will be ignored and closed immediately


Better tests

Added attributes to some std.range.package unittests so they have a better chance to catch regressions.

We need more enforcements like this. Thanks!

There are already some in place - see the failing CircleCI message:
Enforce whitespace before opening parenthesis
grep -nrE "(for|foreach|foreach_reverse|if|while|switch|catch)\(" $(find . -name '*.d') ; test $? -eq 1
./std/range/package.d:668:    catch(AssertError unused)
./std/range/package.d:2159:    catch(UTFException unused)
./std/range/package.d:2163:    catch(Exception unused)
./std/range/package.d:2823:    catch(UTFException unused)
./std/range/package.d:2827:    catch(Exception unused)
./std/range/package.d:3049:    catch(UTFException unused)
./std/range/package.d:3053:    catch(Exception unused)

(you can run the style tests locally with make -f posix.mak style).

starting to fix now...

how I test for style at windows? make -f win[32/64].mak style compains:
Error: don't know how to make 'style'

how I test for style at windows? make -f win[32/64].mak style complains:
Error: don't know how to make 'style'

At the moment it's Posix only:
https://github.com/dlang/phobos/blob/master/posix.mak#L497
However, I am trying to move the "simple grep" style checks over to Dscanner, s.t. it's truly cross-platform, but this is pending since quite a while.

What does that codecov/patch test failure mean I did wrong? After all, I didn't add any new code outside unittests so I don't see how I worsened unittest coverage ratio, if it means that.

What does that codecov/patch test failure mean I did wrong? After all, I didn't add any new code outside unittests so I don't see how I worsened unittest coverage ratio, if it means that.

If you click on the link or install the browser extension you will see the reason:

See this discussion for an explanation.
tl;dr: the DMD compiler doesn't ignore unittest blocks and it won't do so in the foreseeable future.

Thanks, I see. Trough, I'm not sure how I should act in blocks like this...

Hmm one never knows how long a PR will take to land in master, so I suggest to mark the "ugly" try /catch bits with a  // @@@BUG@@@ 12647 comment, s.t. they are easy to find once the fix for the bug is there.
-> afterwards squash your commits into one & we should be good to go :)

Thank you very much, again. I probably have to exercise first with a branch I've not made a pr yet of.

@dukc We have a pretty good getting started guide. It also gives instructions on how to rebase
Nice job with the PR

Nice job with the PR

Looking forward to more!


Replace async_setup_platforms

This replaces async_setup_platforms with async_forward_entry_setup. https://github.com/home-assistant/core/pull/73806 increased the verbosity from DEBUG to WARNING.

Resolves #15

Thanks!


Skip upgrade from latest minor release tests for docker

Issue #, if available:
Description of changes:
Testing (if applicable):
Documentation added/planned (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/test eks-anywhere-presubmit


Feature Request - Option to disable Panel View in MOBI+EPUB and MOBI/AZW3

I really appreciate EPUB (Calibre KFX) output format but can you please add an option to disable Panel View completely in MOBI+EPUB and MOBI/AZW3 because sometimes while reading it triggers due to accidental touch. It will be nice if there was an option to disable this feature in those modes completely.
Thanks in Advance :-)

Disabling it has some issues. https://github.com/ciromattia/kcc/pull/499#issuecomment-1500921345


On Ctrl+H it searches on all languages, not the currently used one

On MacOS Sierra, when I hit ctrl-H it runs dash, but the search shows all languages.

Hi @kraklin
Will note the about this issue. I haven't upgraded my OS to Sierra due to some incompatible softwares like office and screenflow in my end.
Thank you

It's just a minor issue. :) Keep up great work on this plugin ;)

@kraklin Hi Kraklin,
Do you remember the extension of file that you were searching in?

Hi,
it was Elm or Javascript. I tried them both now, JS is opening the right search within JS docs, although it is not searching for array.fill on [].fill(). It searched for animation something .fill method. But this is not the problem of plugin, dash behaves the same when I try to search it for myself.
Elm goes right to Google search inside Dash.

Hi @kraklin
I just released v1.3.0 that supports elm. 🎉 🎉
https://marketplace.visualstudio.com/items?itemName=deerawan.vscode-dash
Make sure you install elm vscode plugins first
https://marketplace.visualstudio.com/items?itemName=sbrink.elm
Let me know if it solves your issue 😃

Hi, I totally forgot to mention that... It works!!! :champagne: 👍
Thank you

glad to hear that 👍


[dart:js_interop] Add a way to internalize undefined values and pass it back to JS

jsObject.prop = undefined;

Alas, this is one of the few remaining sharp corners to solve. I mention this in https://github.com/dart-lang/sdk/issues/49353.
I don't have a great recommendation here as undefined is always reified to Dart null in dart2wasm, so this will require leveraging some JS code to do the assignment until we resolve this.

Just ported a rather big dart interop lib (https://pub.dev/packages/mapbox_gl_dart/) to the new js_interop
Ran into trouble with a constructor that constructs an invalid object if null is (explictly) passed. I fixed it by passing a valid default value (in this case 0 and a good default).
I would much rather be able to set undefined, as there are certainly libs where no sane defaults are possible and only undefined is an options. Also couples the dart code much closer to the js code as one has to check what defaults make sense for the current version of the js lib.

Thanks for migrating! That's exciting to see.
I agree undefined is still needed. I think the one complication I haven't really resolved is how to expose an undefined value such that it fits into the JS type hierarchy. If we had union types, this would be easier. Without union types, we could imagine a JSAny? singleton e.g.
JSAny? get undefined => // some compiler internals

Which resolves the case where you pass it to JS, but doesn't resolve the case where it's passed back as dart2wasm won't know to treat that JSAny? different from other JSAny?s and therefore will convert it to Dart null. We could maybe use a typedef e.g.
typedef JSAnyOrUndefined = JSAny;

that statically tells the compiler to not convert the undefined to null when used on a return value. Then we can make the singleton above typed as this typedef as well. However, this means you don't have a way of expressing something like "JS string or undefined" as you're stuck with JSAny.
Another way of maybe resolving this is using an annotation on an external member to tell the compiler not to convert undefineds, but annotations are not great.

Also couples the dart code much closer to the js code as one has to check what defaults make sense for the current version of the js lib.

Can you explain a bit more what you meant by this?

I have to replicate the defaults of JS values in the dart interop code. A new version of the js libary might change the defaults, but the dart interop would keep whatever was defined in the dart side. If i can pass undefined the JS libary should use its defaults - if i (implicitly) pass null if often gets used as the acutal value.

Sometimes I'm write code like this, which is not sound:
@JS('fn')
external void _fn(
  JSAny first, [
  JSAny? second, // JS side checks if it's null or undefined
]);

void fn(JSAny first, [Object? second = const Object()]) {
  if (identical(second, const Object())) {
    _fn(first);
  } else {
    _fn(first, second as JSAny?);
  }
}


fix: Bolding on Default Response

:tada: This PR is included in version 2.2.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


not floating

By "Float on the right alongside content", I'm guessing this means it is supposed to stay visible as you scroll down? This is not working for me.

Nope! Just floating on the right at the top of the document.

Ah, sorry for this misconception then!


Update innersource learning path link

This PR updates the link pointing to the InnerSource learning path on lab.github.com
Reported by: John Papa

Fix updated and verified 🚢


Run bjorn without display

Are there any plans to make this work without a screen attached similar to pwnagotchi and its bluetooth pairing option?
Or am I missing a configuration parameter to make this work already?

Are there any plans to make this work without a screen attached similar to pwnagotchi and its bluetooth pairing option?
Or am I missing a configuration parameter to make this work already?

I am working on this mode 👍

looking very forward for björn to be able to run without an ink display; this is currently the only bottleneck I think avoiding this repo to spread in masses :)
however I already doing budget to buy an 2.5 inch screen from aliexpress to build it very much great effort is given here; appreciated 👍

Already working, multiple users are using it without screen

what do i need to set for this?


modify run_events and deadheads, service/run/piece/block_id

This propsal makes the service_id, run_id, piece_id, and block_id fields more consistent.
It's based on the runs.txt proposal, but could be done independently of it.
It's a backwards in-compatible change to add required fields and make a required field optional. It's not totally necessary because you could work around these fields by querying through runs.txt or maybe runs_pieces.txt, but it will be more convenient and consistent, and allow representing some new edge cases.
Currently:
run_events.txt has a required piece_id field. It does not have service_id or run_id.
deadheads.txt has a required service_id field and a required block_id field. It does not have run_id or piece_id.
Proposed:
run_events.txt

Field name
Type
Required
Description
Change

service_id
ID referencing calendar.service_id
Required
Identifies a set of dates when the event is scheduled to take place.
New

run_id
ID referencing runs.run_id
Required
Identifies the run during which the event takes place.
New

piece_id
ID referencing runs.piece_id
Optional
Identifies the piece during which the event takes place. May be left null for events that take place outside of a piece, such as a break.
Optional

deadheads.txt

Field name
Type
Required
Description

service_id
ID referencing calendar.service_id
Required
Identifies a set of dates when the deadhead is scheduled to take place.
Unchanged

run_id
ID referencing runs.run_id
Optional
Identifies the run during which the deadhead takes place.
New

piece_id
ID referencing runs.piece_id
Optional
Identifies the piece during which the deadhead takes place.
New

block_id
ID
Optional
Identifies the block to which the deadhead belongs.
Optional

Benefits:

Can query run_events by date/service_id without going through runs.txt/runs_pieces.txt.

Deadheads can be used by agencies that don't otherwise publish run_ids or block_ids, or can represent deadheads that are not associated with a run or block. (This is just hypothetical to me, but it seems like the restriction isn't necessary.)

trips.block_id is optional, so deadheads.block_id should be too.

Makes querying run_events.txt, deadheads.txt, and trips.txt very similar, which is nice since they're the three types of things that can be in a run.

Obsolete after #55 and #60


nodePackages.rust-analyzer-build-deps: fix path

Path to package.json has been changed at PR (#221878).
Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

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

[ ] Fits CONTRIBUTING.md.

Adding new packages to nodePackages is highly discouraged because of its size and unmaintainable status. Please use buildNpmPackage instead which is easier and significantly faster to use and does not rely on codegen.


Multiple language support

This pull request presents a working version of behe-keyboard which support multiple language layouts. The "qwerty keyboard" is now called "standard keyboard" and a new field in the preferences allow users to chose between multiple layouts (azerty or qwerty for the moment). The way I implemented it, is flexible. It will be easy from now on to add new layouts such as Dvorak or Bépo or any local layout. I am French, so I added the azerty layout (xml file made by @seigneurfuo).

Amazing work. Going to merge this right away and make some tweaks so that the space key will have a background on any layout (currently it has a background only on the qwerty layout). Thanks for working on this, going to credit you in the app.


Writing to Sql Anywhere 16 doesn't appear to be working

I don't have much experience with Sql Anywhere and am trying to hook into a legacy system that runs on it. I am able to connect to the database and read from it but my update queries do not seem to be having any effect. For instance, if I run the following query:
UPDATE Party INNER JOIN Names ON Party.party_id = Names.names_id SET Names.first_name = 'Lindsay 5', Names.aka_first = 'Lindsay 4' WHERE (((Party.our_client)='Y') AND ((Party.case_id)=353178))

The values for first_name and aka_first remain unchanged. However, if I run that exact query inside Microsoft Access the query works and the values for first_name and aka_first are updated.
Database version is <IP_ADDRESS>4
Any idea why this may be happening?

I'm assuming you are checking the error code from the query. If there's no error, then the update should be succeeding. Are you doing a conn.commit() afterwards?

 I didn't realize that I had to run conn.commit() afterwards. I just gave it a try and it worked. In what circumstances do you need to run conn.commit()? Every query besides a SELECT?

A commit is a command to the database server to tell it that the transaction you are working on should be considered permanent. Once you've done the commit, you can disconnect and your changes will persist. The opposite is a rollback, which will "undo" any delete, insert, or update statements you've made (a) since the last commit or rollback or (b) since you connected. A commit / rollback isn't technically necessary after selects since you haven't changed anything but it's a good idea anyway since it releases locks held by the connection.

That's good to know. I appreciate your help. Thanks for the quick response!


Update changelog for v7.2 and v7.3

Update changelog for v7.2 and v7.3

Markdown lint and link-test passed, so I will merge without waiting for the rest CIs.


nodejs 4.4.5 - RangeError: Invalid status code: 0

When running on node 4.4.5 or higher I get this error in a stacktrace, 4.4.4 and below work normally.
POST done desktop-animation2.gif 130620-desktop-animation2gif
_http_server.js:192
    throw new RangeError(`Invalid status code: ${statusCode}`);
    ^

RangeError: Invalid status code: 0
    at ServerResponse.writeHead (_http_server.js:192:11)
    at ServerResponse._implicitHeader (_http_server.js:157:8)
    at ServerResponse.OutgoingMessage.end (_http_outgoing.js:573:10)
    at ServerResponse.send (/home/media/git/flow.js/samples/Node.js/node_modules/express/lib/response.js:204:10)
    at /home/media/git/flow.js/samples/Node.js/app.js:23:31
    at /home/media/git/flow.js/samples/Node.js/flow-node.js:122:33
    at FSReqWrap.cb [as oncomplete] (fs.js:212:19)

➜ media@MEDIA-PC  ~/git/flow.js/samples/Node.js git:(master) ✗ npm version
{ npm: '2.15.5',
  ares: '1.10.1-DEV',
  http_parser: '2.5.2',
  icu: '56.1',
  modules: '46',
  node: '4.4.5',
  openssl: '1.0.2h',
  uv: '1.8.0',
  v8: '<IP_ADDRESS>',
  zlib: '1.2.8' }

The error happens right after uploading a file, the client says file could not be uploaded but the file is successfully saved on the server before it crashes. It just seems to fail to let the client know it worked and the server crashes because the demo doesn't catch the exception.  I've tested it on my Ubuntu 14.04 media PC and my Ubuntu 16.04.
--- Edit ----
It looks like the problem is that the example passes the text status ('done', 'partially_done', ---) to express as an HTTP status code. I don't know why that worked in previous versions of node.
// Handle uploads through Flow.js
app.post('/upload', multipartMiddleware, function(req, res) {
  flow.post(req, function(status, filename, original_filename, identifier) {
    console.log('POST', status, original_filename, identifier);
    if (ACCESS_CONTROLL_ALLOW_ORIGIN) {
      res.header("Access-Control-Allow-Origin", "*");
    }
    //res.status(status).send()
    res.status(/^(partly_done|done)$/.test(status) ? 200 : 500).send();
  });
});

This appears to fix it. I'd probably prefer that the node module returned null on success to follow the callback(err,result) convention. I'd submit a PR to do that would likely be a breaking change so I'm not sure if people would be interested in it or not.

I had the same experience and fixed it the same way.  That allowed the second and after chucnks to downlaod,  but now my chunks arent getting put back together into the file nor are they being cleaned up after.  Did you have that problem too?


Question about save_many

Why are the ids stored in models_to_insert?
    for idx, inserted_id in enumerate(result.inserted_ids):
        models_to_insert[idx].id = inserted_id

They are not used anywhere and not returned by the function.

Ah, ok. Now I get it, completely missed that the list is referencing the original models. Thanks!


urllib3.connection.HTTPSConnection error for Azure Resource Graph API

We are using python SDK for azure.mgmt.resourcegraph for querying the assets. But SDK most of the time giving us follwoing error stating: <urllib3.connection.HTTPSConnection object at 0x3ed9892d49a0>: Failed to establish a new connection: [Errno 110] Connection timed out.
package version of azure.mgmt.resourcegraph is 7.0.0
Its running on google's cloud function V1 with Python version: Python 3.9

Hello team, I am from MS support team, [tsaindane-cloudwiz], reached us on this particular issue. Can we get an update. Appreciate your support and understanding.

Hello team, I am from MS support team, [tsaindane-cloudwiz], reached us on this particular issue. Can we get an update. Appreciate your support and understanding.

@visingla-ms can you see this net connect error! Thanks


[Bug] [SQOOP Task] data target type is only HDFS

Search before asking

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

What happened
Windows11, Drag a SQOOP task, select SQL as the data source modeltype, the data target type contains HIVE and HDFS.
Then I switch the data source modeltype from SQL to Form, and then switch to SQL, the data target type is only HDFS.

What you expected to happen
Switch data source modeltype from SQL to Form, and then switch to SQL, the data target type should contains HIVE and HDFS, but now is only HDFS.

How to reproduce
1、Drag a SQOOP task, select SQL as the data source modeltype
2、Then  switch the data source modeltype from SQL to Form, and then switch to SQL
Anything else
No response
Version
3.1.x
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

can you try to fix this?

sure, my pleasure

can you try to fix this?

Sure, my pleasure. I have fixed it at local


Extract storage layer + cs3impl

@refs thanks for your input! Adjusted the PR and rebased to current master. Could you re-review please? :-)


Add MATE support for dbus-launch (linux)

Thank you for your contribution!

My pleasure. Cheers

Jun 2017 10:52 by<EMAIL_ADDRESS>

Thank you for your contribution!
—
You are receiving this because you authored the thread.
Reply to this email directly, > view it on GitHub> , or > mute the thread> .


option to add extra labels

port of https://github.com/Activiti/example-runtime-bundle/commit/8c16709a47ad33fdefda0cee06b9fcdf104cf9eb

Not really important now we have https://github.com/Activiti/activiti-cloud-charts/commit/b9e580ef3502788a315785d70d1490a65ff63fc3


Pausing Service when you have multiple in the same login

Hi this looks a really handy script.
I have a connection that I only activate on weekends but i have several connections in the same login.
I found the Service ID by inspecting the Pause button on the launtel portal.
I just wondered if any further work has been done on this.

No, I haven't done any further work on it.


Handle => HandleReq

@Rick-Anderson @rachelappel @blowdart

Hi @HaoK, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

Merged thanks d4a241a39ebe424c6e77b314eaed72a3dfdbf565


ddl: Fix an issue where a query may throw an error during a remote read and the precision of a duration data type is changed (release-6.5)

This is an manual fix of #8671 in release-7.1
What problem does this PR solve?
Issue Number: close #8601
Problem Summary:
What is changed and how it works?
see https://github.com/pingcap/tiflash/issues/8601#issuecomment-1878393488. The different in release-7.1 and earlier branches is because the fsp change of time column type is ignored
Check List
Tests

[ ] Unit test
[x] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

Fix an issue where a query may throw an error during a remote read and the precision of a duration data type is changed.

/run-all-tests

/run-unit-test


Blank screen on start

I'm trying to run the 1.3.2 binaries on my Windows 7 x64 desktop and upon opening YakYak I'm greeted with a blank window. I've tried running both the 32 and 64-bit versions for good measure but both do the same thing. I also double checked the application was being allowed through the firewall. If you need any information just let me know what you need and where to get it.

1.3.2 (both the binaries and building it myself) is giving me a blank screen as well — it shows me the Google Account login page, lets me enter a password, then shows a blank screen instead of progressing to asking for my TOTP code. Output:
hconnect
reconnecting 0
resolving proxy http://plus.google.com
resolving proxy https://plus.google.com
resolved proxy DIRECT
resolved proxy DIRECT
asking for login credentials
2016-09-17 1:17:51 WARN no SAPISID cookie
2016-09-17 1:17:51 WARN no SAPISID cookie
login: did-finish-load https://accounts.google.com/ServiceLogin?passive=1209600&continue=https://accounts.google.com/o/oauth2/programmatic_auth?top_level_cookie[...]
login: programmatic auth
login: did-finish-load https://accounts.google.com/signin/challenge/totp/2?continue=https%3A%2F%2Faccounts.google.com%2Fo%2Foauth2%2Fprogrammatic_auth%3Ftop_level_cookie[...]

URLs are trimmed because I'm not entirely sure of the security issues with their content, I can provide more if necessary.

I'm having the same issue as OP on Windows 10 64-bit, when compiling the latest commit (https://github.com/yakyak/yakyak/commit/fbcc26736c9a11fd7f7d7805fff8782c8c4c9b61). I tried running it from the windows commandline but there was no commandline output. I'm not sure where the log files are but I can post them if someone knows how to find them.

I also just compiled it under the above commit in Arch Linux (Kernel 4.7) 64-bit and also got a blank screen on startup.

The latest commit problem is from a typo in the menu generation in the last PR merge.
#403 has the solution for that one.

To debug the application when there is a blank interface the console is a great place, but if no error/warnings are there you need to debug YakYak using the developer's tools.
Available through the links in the menus or by shortcut using Ctrl+Alt+I
In case the shortcut does not work it is usually the sign that a problem happened during menu generation or before. In Ubuntu I can always access the Developer's Tools via the menu, not sure about other environments though.

@averissimo thank you for the prompt response and debugging tips. I'm still getting a white screen after applying the fixes from your PR. The console spams the message in the screenshot below indefinitely, and Ctrl+Alt+I doesn't bring up anything.

Can you access the menus if you try to press Alt key or via the menu bar?

getting info from developer's console would be really great

Are you authenticating an account in this process or is that already saved in your profile?
Have you done npm install recently just to make sure you have all packages?

this only apply for who is compiling from git

I cannot replicate this :\

install electron-dev
$ npm install --save electron-debug

add this below app = require...in main.coffee
require('electron-debug')({showDevTools: true})

And this opens dev tools automatically.

ps. screencast shows me running without the PR fix, that's why I get the blank window.

Hi @averissimo ,
I gave it another try just now with your new instructions. First I ran npm install to make sure I had all the packages. Then I ran npm install --save electron-debug. See screenshot:

Then I added the line in /src/main.coffee : require('electron-debug')({showDevTools: true}) as in the screenshot:

Then I tried running the app using npm run electron app like from the readme, but it's been giving me the following error:

So I compiled it with npm run deploy and ran the resulting windows binaries on Windows 10 64-bit.  I'm not sure if these errors are related to the white screen and lack of console, but I thought I'd mention them in case they're relevant.
I really appreciate the help thus far!

I'm guessing it is using the previously compiled version as you did not run
$ npm run gulp

Then the npm run electron app should work and the deploy will also rebuild with the recently compiled version
PS. Nice screen cast :)

Thank you sir!
npm run gulp gives the following output:
pablo@elemio:~/yakyak$ npm run gulp

><EMAIL_ADDRESS>gulp /home/pablo/yakyak
> gulp

[13:07:49] Requiring external module coffee-script/register
[13:07:52] Using gulpfile ~/yakyak/gulpfile.coffee
[13:07:52] Starting 'package'...
[13:07:52] Starting 'coffee'...
[13:07:52] Starting 'html'...
[13:07:52] Starting 'images'...
[13:07:52] Starting 'icons'...
[13:07:52] Finished 'icons' after 22 ms
[13:07:52] Starting 'less'...
[13:07:52] Starting 'fontello'...
[13:07:52] Finished 'html' after 305 ms
[13:07:52] Finished 'images' after 305 ms
[13:07:53] Finished 'less' after 564 ms
[13:07:53] Finished 'package' after 910 ms
[13:07:54] Finished 'coffee' after 1.49 s
[13:07:54] Finished 'fontello' after 1.63 s
[13:07:54] Starting 'default'...
[13:07:54] Finished 'default' after 3.79 μs

And I still get the max depth reached errors from above, leading to the blank screen. I can't run it in Linux because npm run electron app gives:
pablo@elemio:~/yakyak$ npm run electron app
npm ERR! Error: ENOENT, open '/home/pablo/yakyak/node_modules/electron/package.json'
npm ERR! If you need help, you may report this log at:
npm ERR!     <http://github.com/isaacs/npm/issues>
npm ERR! or email it to:
npm ERR<EMAIL_ADDRESS>
npm ERR! System Linux 3.19.0-66-generic
npm ERR! command "/usr/bin/nodejs" "/usr/bin/npm" "run" "electron" "app"
npm ERR! cwd /home/pablo/yakyak
npm ERR! node -v v0.10.25
npm ERR! npm -v 1.3.10
npm ERR! path /home/pablo/yakyak/node_modules/electron/package.json
npm ERR! code ENOENT
npm ERR! errno 34
npm ERR!
npm ERR! Additional logging details can be found in:
npm ERR!     /home/pablo/yakyak/npm-debug.log
npm ERR! not ok code 0

Does npm run electron app work for you? what is the content of your electron package.json?

I dug a bit and found that ~/yakyak/node_modules contains an electron-debug and electron-prebuilt folder. It seems that it can't find the electron folder for some reason, which is why it's erroring out.

you are using a really old version of node and npm, can you updatee to v4 or v6 (i use this one) and then npm -g install npm
After delete the electron folder inside modules and run npm install (you can also delete all contents of node_modules, and run the install command)

Not sure if it is necessary though (it is not for running the binaries, but it should be for starting the dev tools, when like you cannot call it via shortcuts or menu)
i'm not that experienced with node and started playing with yakyak recently, if other people want to chip in please do :)

Not a fix, but a workaround, have you tried running with --disable-gpu parameter? Fixed the problem for me (on Linux)

just though of a slightly better solution to debug YakYak when cases like this appear and it is already on pull request #418 for discussion.
With this change the original menu is visible and can access the DevTools directly.
@pmcanseco If you really want to debug you don't have to build everything (sorry, but I did not know better), just change file resources/app/main.js in released binary to add above the line containing loadAppWindow();:
mainWindow.webContents.openDevTools();
loadAppWindow();

It will open the DevTools when running YakYak and you can see the error preventing the interface from loading

@averissimo that's really cool! I'll give it a shot.

@pmcanseco @MarkusMcNugen (cool name) I'm closing this issue
Right now YakYak has better debugging capabilities (as it shows errors in terminal), and when pull request #521 is merged it will have a cool error window with a descriptive message
If you guys are still having problems please open a separate issue
thanks!


[WIP] JSONAPI Content Negotiation

 Purpose
We are changing the formatting of JSON responses in the API to match the JSONAPI v1 spec http://jsonapi.org/format/.  References Issue #3074.
This PR sends JSON data with the header Content-Type: application/vnd.api+json.
 Changes
Adds custom content negotiation for when application/json in acceptable media types. Enforces JSONAPIRenderer, which has the media_type application/vnd.api+json.   Custom content negotiation helped get around issues with Swagger docs. https://github.com/marcgibbons/django-rest-swagger/issues/271
New Response Headers

Old Response Headers

I don't think we're going to return anything that's not JSON or HTML, but it might be better to return the JSON if the request type is a json requesting type, so if we added, I dunno, yaml or XML or something, it won't send the json type. That being said, if it looks like it's going to be A Thing to do that, then we can mark it as a potential future improvement.

Also, if you get this out of WIP before anyone else reviews the other related issues, we might as well just roll them all up into that one PR.

Custom content negotiation is now the same as DefaultContentNegotation except when application/json in acceptable media types. Then, the JSONAPIRenderer with media_type application/vnd.api+json is enforced.
Otherwise, it checks acceptable media types against the renderer, same as always.

Woo woo!

Merged with #3683


chore: Update .codegen.json with commit hash of codegen and openapi spec

Pull Request Test Coverage Report for Build<PHONE_NUMBER>9
Details

10 of 55 (18.18%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 53.952%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

Box.Sdk.Gen/Schemas/AiLlmEndpointParamsAwsOrAiLlmEndpointParamsGoogleOrAiLlmEndpointParamsOpenAi/AiLlmEndpointParamsAwsOrAiLlmEndpointParamsGoogleOrAiLlmEndpointParamsOpenAi.cs
9
31
29.03%

Box.Sdk.Gen/Schemas/AiLlmEndpointParamsAws/AiLlmEndpointParamsAws.cs
0
23
0.0%

Totals

Change from base Build<PHONE_NUMBER>9:
-0.1%

Covered Lines:
8379

Relevant Lines:
16211

💛  - Coveralls


Link to latest mantis-example and update commands

Context
The git repository mantis-examples has been deprecated and have been moved to the central mantis repository (this). However, example code and links in documentation are still pointing to the old mantis-example repository. That could be confusing when people follow the link only to find the stale code.
This patch fixes the deprecated links, and also updates the commands to run those example jobs.
NOTE: there could be other stale links as we have deprecated multiple standalone repositories and moved them here. We should file new pul requests to fix that. This one is mainly for mantis-examples.
Checklist

[X] ./gradlew build compiles code correctly
[ ] ~Added new tests where applicable~
[X] ./gradlew test passes all tests
[X] Extended README or added javadocs where applicable
[ ] ~Added copyright headers for new files from CONTRIBUTING.md~

thank you for this change!


Missing pg_ivm extension in v0.1.6/latest Docker tag

When creating a new pg_timeseries project using the latest tag, and executing the CREATE EXTENSION timeseries CASCADE; instruction, I get the following Postgres error:
NOTICE:  installing required extension "columnar"
NOTICE:  installing required extension "pg_cron"
NOTICE:  installing required extension "pg_ivm"

Query 1 ERROR at Line 1: : ERROR:  extension "pg_ivm" is not available
DETAIL:  Could not open extension control file "/usr/share/postgresql/16/extension/pg_ivm.control": No such file or directory.
HINT:  The extension must first be installed on the system where PostgreSQL is running.

However, if I revert to the v0.1.5 Docker tag, it works correctly:
NOTICE:  installing required extension "columnar"
NOTICE:  installing required extension "pg_cron"
NOTICE:  installing required extension "pg_partman"

Query 1 OK: CREATE EXTENSION

Thanks for raising the issue @kaysersoze . We added a dependency on pg_ivm (Tembo's fork of it) in the latest release. This was an oversight on our part that it didnt make it into the image.
We'll get the build updated and a new image published.

@kaysersoze thanks again for letting us know about the issue. We rebuilt the latest and v0.1.6 tags


too busy

regardless of the specified user-agent, virtually every call is "too busy"; can we just pay for API that isn't going to be "too busy" virtually every time its called? out of the 5 coins I am monitoring, Helium is the only one failing constantly; If you do not want people aggregating real-time stats just let us know. Don't write a document telling people they can use the API.
I implemented a wait for "comeBackInMs" (which is not always present.) immediately after waiting, it says again "comeBackInMs" which an even longer timestamp and still no data. This is silly.

I'm having the same issue... The API response sometimes gives 429 error, and sometimes 503... And sometimes work as expected. Hopefully this will work better in the future


Fixed live update EoF exception

Just fixing the frequent EoF error from reloading a Aseprite file while it's being written by the editor.
This fix retries after a 1 second delay, up to 3 times.

Nice! Thanks :)


Set client state to idle after error

If clients fail to checkout a connection from the pool or lose a connection they had, we don't change the state of the client so it will be stuck in state active or waiting. This state is not persistent as the client state will be corrected the next time the client attempts to perform a query.
This PR fixes this by setting client state to idle after errors. It also adds tests that cover that case.

Neat!


Trigger Heroku Deploy Action on merge of elixirschool/elixirschool repo.

This likely needs to wait till the rest of this is complete and move the entire "legacy" repo to the new content. When changes to a lesson or blog post are merged, we need to trigger a deploy of this repository.

https://github.community/t/triggering-by-other-repository/16163

#61 and https://github.com/elixirschool/elixirschool/pull/2603 should do the trick. I'll be testing that today.

This is working now. There is a separate issue with the newest post having some bad markup which failed the build. I'll sort that out 👍


Upgraded to 8.30, lost BT and now can't lock/unlock

So I upgraded to 8.30 from 8.27 and lost the Bluetooth pairing while upgrading.
So I

unpaired the lock
paired it again

Now I always get the following on the starting page:

Now everytime I enter the PIN, the nukihub restarts and the same message keeps popping up - and still no lock/unlock command goes through.
What can I do?
The configuration is like this:
NUKI Hub version: 8.30
run: true
deviceId: 39xxx
deviceIdOp: 38yyy
mqttbroker: <IP_ADDRESS>
mqttport: 1883
mqttuser: ***
mqttpass: ***
mqttlog: true
lockena: true
mqttpath: devices/NukiHub
openerena: false
mqttoppath:
maxkpad:
opmaxkpad:
mqttca:
mqttcrt:
mqttkey:
hassdiscovery:
dhcpena: true
ipaddr:
ipsub:
ipgtw:
dnssrv:
nwhw: 2
rssipb: -1
hostname: nukihub
nettmout: -1
restdisc: true
rstbcn: -1
lockStInterval: 1800
configInterval: 3600
batInterval: 1800
kpInterval: 1800
kpEnabled: false
accLvl:
regAsApp: false
nrRetry: 5
rtryDelay: 100
crdusr:
crdpass:
pubauth: false
pubdbg: false
prdtimeout: 60
hasmac: true
macb0: -16
macb1: 97
macb2: -35
MQTT connected: Yes
Lock firmware version: 2.15.3
Lock hardware version: 11.1
Lock paired: Yes
Lock PIN set: No
Lock has door sensor: Yes
Lock has keypad: No
Network device: Built-in Wifi
Uptime: 1576 minutes
Heap: 90760
Stack watermarks: nw: 6104, nuki: 488, pd: 232
Restart reason FW: NetworkDeviceCriticalFailure
Restart reason ESP: ESP_RST_SW: Software reset via esp_restart.

I would try pairing again, something likely went wrong there.

@binderth Were you able to fix your issues?

Oh, yes. I did a reset of my ESP and now it works as intended. Something was off I guess.

I'll close the issue

Perfect

@binderth @technyon: Close?

Yes. Wirry


Fix account host

Added account_host parameter to AzureBlobFileSystem, which is needed for package to work in other Azure clouds.

Thanks!


Buttons at bottom of extension aren't working

Name of Defective Feature

Open Settings button on extension

System Settings (Required)

ToolKit Version: 2.6.0
Operating System (Mac/Windows/Etc): Windows
Browser (Chrome/Firefox/Safari): Chrome
Browser Version: 66.0.3359.181
Other Extensions:

Export of Toolkit Settings (Bottom Left of Settings Page)

Settings Export
PASTE SETTING EXPORT HERE

Steps to Reproduce the Behavior

Open the extension.
Select the Open Settings button.

Expected Behavior

I expected to open settings and make updates.
Actual Behavior

Open the extension.
Select the Open Settings button.

Links to buttons aren't active (I can only click in the text area Welcome to Toolkit which disables the extension).

Hey @fourlittlebirds, thank you for reporting this issue. A couple of questions:

Do you have any other Chrome extensions installed?
Is this on a computer managed by an IT department, like at a company or school?

@fourlittlebirds can you try the following to see if the Options page will display?

go to chrome://extensions/
find the Toolkit extension
Click the Details button for the extension
Scroll down to the Extension options link and click it. You should see a page with a URL that looks like this chrome-extension://mkgdgjnaaejddflnldinkilabeglghlo/options/index.html

The options page should open. If it doesn't, do you see any type of message explaining why it isn't? The point in trying this method is to use "native" Chrome to open the page and not what's in the Toolkit extension.

Interesting, I've never opened the extensions that way, only done it via the way I documented above. In any case, both options, Open settings button as in screen captures above and via the path provided by @dbaldon. Yes, this is a school computer so not sure if that played a role yesterday. Thanks for your help. I'll go ahead and close this. Love that this tool is here and appreciate how responsive you are. Cheers for community!

@fourlittlebirds thanks for checking and reporting back. Strange behavior but glad it's now working as intended. Feel free to reopen this Issue if it happens again.


client: added context for http client

Bypass context to the http request.

Chiefly due to violating the Semver requirements (changing Send's signature). I'm going to deny this PR.


Fix using < or > in a reference name

These characters are valid and only the last <...> must be treated as the embedded URL.
This PR removes using the inline lexer inside references, reSt doesn't allow nested inline nodes and the only thing required is getting the embedded URL (if there is any). This reduces a lot of complexity

You are correct, this was too fragile. I've now rewritten the code to delegate parsing the embedded URI to the rules/roles that need it and removing it from the global inline lexer. This is the safest and shouldn't break anything, given embedded URIs do not exist on their own.

I like it, it will also make things easier for us in the php-domain text-roles


More descriptive failure messages in HttpMessageParser

I'm currently investigating an issue where we're suddenly running into the following error: HTTP chunk size exceeds the configured limit of 1048576 bytes.
This is possibly due to an infrastructure change we made which results in a change of behavior when chunked transfer encoding is used. I did not find a way to determine the actual size of the incoming chunks and it would be really helpful if it was part of that error message.
My proposal here would be to extend all relevant errors in HttpMessageParser with extra parameter values. e.g. for the max chunk size, the full hex size could be read before validating against maxChunkSize and then including the size parameter in the error message:
https://github.com/apache/incubator-pekko-http/blob/4f80b1ef7cad2833382a659e7dd0b1a54078d126/http-core/src/main/scala/org/apache/pekko/http/impl/engine/parsing/HttpMessageParser.scala#L309-L320
Given that on line 318 the actual encountered illegal char is part of the error message, I think having the size in the other error is warranted. Apart from max chunk size there are a few other messages that could be improved such as max chunk extensions and amount of chunk trailer headers.
Any thoughts on this proposal?

probably should have been closed after https://github.com/apache/pekko-http/pull/528 was merged


About pg: About PyMC, introduce videos

the du_email substitution doesn't work yet because it is introduced by #88, but once it is merged it will show just like it does on the preview of that PR


Installing through NuGet on VS2013 For Web

Hello
Installing Fluent Validation in Visual Studio Express 2013 for Web is failing. Not sure what exactly is wrong. Below is the output from the NuGet Console.
I've attempted to install older versions of the FV package but the same error occurs. I've installed on blank ASP .NET MVC projects to ensure that my other packages are not conflicting and the same error occurs.
Any ideas whats wrong?
NuGet Command: Install-Package FluentValidation

PM> Install-Package FluentValidation
Installing 'FluentValidation <IP_ADDRESS>'.
Successfully installed 'FluentValidation <IP_ADDRESS>'.
Adding 'FluentValidation <IP_ADDRESS>' to Server.
Uninstalling 'FluentValidation <IP_ADDRESS>'.
Successfully uninstalled 'FluentValidation <IP_ADDRESS>'.
Install failed. Rolling back...
Install-Package : Specified argument was out of the range of valid values.
Parameter name: supportedFrameworks
At line:1 char:1

Install-Package FluentValidation

  + CategoryInfo          : NotSpecified: (:) [Install-Package], ArgumentOutOfRangeException
  + FullyQualifiedErrorId : NuGetCmdletUnhandledException,NuGet.PowerShell.Commands.InstallPackageCommand

NuGet Command: Install-Package FluentValidation.MVC5

PM> Install-Package FluentValidation.MVC5
Attempting to resolve dependency 'FluentValidation (≥ <IP_ADDRESS>)'.
Attempting to resolve dependency 'Microsoft.AspNet.Mvc (≥ 5.1 && < 5.3)'.
Attempting to resolve dependency 'Microsoft.AspNet.WebPages (≥ 3.1.0 && < 3.2.0)'.
Attempting to resolve dependency 'Microsoft.Web.Infrastructure (≥ <IP_ADDRESS>)'.
Attempting to resolve dependency 'Microsoft.AspNet.Razor (≥ 3.1.0 && < 3.2.0)'.
Installing 'FluentValidation <IP_ADDRESS>'.
Successfully installed 'FluentValidation <IP_ADDRESS>'.
Installing 'Microsoft.Web.Infrastructure <IP_ADDRESS>'.
Successfully installed 'Microsoft.Web.Infrastructure <IP_ADDRESS>'.
Installing 'Microsoft.AspNet.Razor 3.1.0'.
You are downloading Microsoft.AspNet.Razor from Microsoft, the license agreement to which is available at http://aspnetwebstack.codeplex.com/license. Check the package for additional dependencies, which may come with their own license agreement(s). Your use of the package and dependencies constitutes your acceptance of their license agreements. If you do not accept the license agreement(s), then delete the relevant components from your device.
Successfully installed 'Microsoft.AspNet.Razor 3.1.0'.
Installing 'Microsoft.AspNet.WebPages 3.1.0'.
You are downloading Microsoft.AspNet.WebPages from Microsoft, the license agreement to which is available at http://aspnetwebstack.codeplex.com/license. Check the package for additional dependencies, which may come with their own license agreement(s). Your use of the package and dependencies constitutes your acceptance of their license agreements. If you do not accept the license agreement(s), then delete the relevant components from your device.
Successfully installed 'Microsoft.AspNet.WebPages 3.1.0'.
Installing 'Microsoft.AspNet.Mvc 5.1.0'.
You are downloading Microsoft.AspNet.Mvc from Microsoft, the license agreement to which is available at http://aspnetwebstack.codeplex.com/license. Check the package for additional dependencies, which may come with their own license agreement(s). Your use of the package and dependencies constitutes your acceptance of their license agreements. If you do not accept the license agreement(s), then delete the relevant components from your device.
Successfully installed 'Microsoft.AspNet.Mvc 5.1.0'.
Installing 'FluentValidation.MVC5 <IP_ADDRESS>'.
Successfully installed 'FluentValidation.MVC5 <IP_ADDRESS>'.
Install failed. Rolling back...
Install-Package : Already referencing a newer version of 'Microsoft.AspNet.Razor'.
At line:1 char:1

Install-Package FluentValidation.MVC5

  + CategoryInfo          : NotSpecified: (:) [Install-Package], InvalidOperationException
  + FullyQualifiedErrorId : NuGetCmdletUnhandledException,NuGet.PowerShell.Commands.InstallPackageCommand

Which .net version are you targeting?
On Monday, July 27, 2015, sqzr1<EMAIL_ADDRESS>wrote:

Hello
Installing Fluent Validation in Visual Studio Express 2013 for Web is
failing. Not sure what exactly is wrong. Below is the output from the NuGet
Console.
I've attempted to install older versions of the FV package but the same
error occurs. I've installed on blank ASP .NET MVC projects to ensure that
my other packages are not conflicting and the same error occurs.
Any ideas whats wrong?
NuGet Command: Install-Package FluentValidation
PM> Install-Package FluentValidation
Installing 'FluentValidation <IP_ADDRESS>'.
Successfully installed 'FluentValidation <IP_ADDRESS>'.
Adding 'FluentValidation <IP_ADDRESS>' to Server.
Uninstalling 'FluentValidation <IP_ADDRESS>'.
Successfully uninstalled 'FluentValidation <IP_ADDRESS>'.
Install failed. Rolling back...
Install-Package : Specified argument was out of the range of valid values.
Parameter name: supportedFrameworks
At line:1 char:1

Install-Package FluentValidation
- ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
- CategoryInfo : NotSpecified: (:) [Install-Package],
ArgumentOutOfRangeException

FullyQualifiedErrorId :
NuGetCmdletUnhandledException,NuGet.PowerShell.Commands.InstallPackageCommand

NuGet Command: Install-Package FluentValidation.MVC5
PM> Install-Package FluentValidation.MVC5
Attempting to resolve dependency 'FluentValidation (≥ <IP_ADDRESS>)'.
Attempting to resolve dependency 'Microsoft.AspNet.Mvc (≥ 5.1 && < 5.3)'.
Attempting to resolve dependency 'Microsoft.AspNet.WebPages (≥ 3.1.0 && <
3.2.0)'.
Attempting to resolve dependency 'Microsoft.Web.Infrastructure (≥
<IP_ADDRESS>)'.
Attempting to resolve dependency 'Microsoft.AspNet.Razor (≥ 3.1.0 && <
3.2.0)'.
Installing 'FluentValidation <IP_ADDRESS>'.
Successfully installed 'FluentValidation <IP_ADDRESS>'.
Installing 'Microsoft.Web.Infrastructure <IP_ADDRESS>'.
Successfully installed 'Microsoft.Web.Infrastructure <IP_ADDRESS>'.
Installing 'Microsoft.AspNet.Razor 3.1.0'.
You are downloading Microsoft.AspNet.Razor from Microsoft, the license
agreement to which is available at
http://aspnetwebstack.codeplex.com/license. Check the package for
additional dependencies, which may come with their own license
agreement(s). Your use of the package and dependencies constitutes your
acceptance of their license agreements. If you do not accept the license
agreement(s), then delete the relevant components from your device.
Successfully installed 'Microsoft.AspNet.Razor 3.1.0'.
Installing 'Microsoft.AspNet.WebPages 3.1.0'.
You are downloading Microsoft.AspNet.WebPages from Microsoft, the license
agreement to which is available at
http://aspnetwebstack.codeplex.com/license. Check the package for
additional dependencies, which may come with their own license
agreement(s). Your use of the package and dependencies constitutes your
acceptance of their license agreements. If you do not accept the license
agreement(s), then delete the relevant components from your device.
Successfully installed 'Microsoft.AspNet.WebPages 3.1.0'.
Installing 'Microsoft.AspNet.Mvc 5.1.0'.
You are downloading Microsoft.AspNet.Mvc from Microsoft, the license
agreement to which is available at
http://aspnetwebstack.codeplex.com/license. Check the package for
additional dependencies, which may come with their own license
agreement(s). Your use of the package and dependencies constitutes your
acceptance of their license agreements. If you do not accept the license
agreement(s), then delete the relevant components from your device.
Successfully installed 'Microsoft.AspNet.Mvc 5.1.0'.
Installing 'FluentValidation.MVC5 <IP_ADDRESS>'.
Successfully installed 'FluentValidation.MVC5 <IP_ADDRESS>'.
Install failed. Rolling back...
Install-Package : Already referencing a newer version of
'Microsoft.AspNet.Razor'.
At line:1 char:1

Install-Package FluentValidation.MVC5
- ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
- CategoryInfo : NotSpecified: (:) [Install-Package],
InvalidOperationException

FullyQualifiedErrorId :
NuGetCmdletUnhandledException,NuGet.PowerShell.Commands.InstallPackageCommand

—
Reply to this email directly or view it on GitHub
https://github.com/JeremySkinner/FluentValidation/issues/97.

Hi @JeremySkinner, I am targeting version 4.5

I just tried it on a blank 4.5 project (in vs2013) and it worked fine. Are you running the updates of VS2013 and nuget?

I'm using VS2013 12.0.31101.0 REL. I'll update VS2013 and NuGet and hopefully that will stop the error.

Ok, I'm running VS2013 Update 4 and Nuget 2.8

yay that fixed the error. Thanks

Great, I'm glad it's sorted.


Revert to orginal nnictl log trial and nnictl trial kill

Revert to original nnictl log trial and nnictl trial kill.
Change the name trialid totrial_id in argparse.

@SparkSnail @chicm-ms please help review


Update README.md

VMware Tanzu by Broadcom updates for 2 CVE issues and integration with Bitnami commercial Tanzu Application Catalog including blog posts and details of SBOM usage.

VMware Tanzu by Broadcom updates for 2 CVE issues and integration with Bitnami commercial Tanzu Application Catalog including blog posts and details of SBOM usage.

Thank you for this submission!


Add Set field to HTTPRequestHeaderFilter

This patch adds Set field to HTTPRequestHeaderFilter.
Currently Add(append) and Remove are available in
HTTPRequestHeaderFilter. But it is not possible to overwrite a
header.

/ok-to-test

Thanks for this addition @nak3! We're still trying to figure out what we'd like to include in updates to the v1alpha1 API vs new work on the v1alpha2 API. We should be able to discuss this more at our community meeting next week. Until then, I'd like to hold this change to ensure we're not adding more than we want to to v1alpha1.
/hold

Envoy can do this easily, see append flag:
https://www.envoyproxy.io/docs/envoy/latest/api-v3/config/core/v3/base.proto.html?highlight=headervalueoption#config-core-v3-headervalueoption
FWIW its part of Istio API as well.
On Tue, Nov 24, 2020 at 10:02 AM Rob Scott<EMAIL_ADDRESS>wrote:

Thanks for this addition @nak3 https://github.com/nak3! We're still
trying to figure out what we'd like to include in updates to the v1alpha1
API vs new work on the v1alpha2 API. We should be able to discuss this more
at our community meeting next week. Until then, I'd like to hold this
change to ensure we're not adding more than we want to to v1alpha1.
/hold
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/kubernetes-sigs/service-apis/pull/475#issuecomment-733144256,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEYGXJFOCFHFLPOT7WSGFTSRPYLNANCNFSM4UA4AZTQ
.

We discussed this a bit further yesterday and this seemed like a reasonable change. We also got https://github.com/kubernetes-sigs/service-apis/pull/492 merged in yesterday which clarifies that we can make additive changes to an existing API. With that I think this is mostly good to go.
/hold cancel
/retest

I'm going to approve this and wait for a final LGTM from @hbagdi or @jpeach.
/approve

/lgtm
@jpeach How does https://github.com/kubernetes-sigs/service-apis/pull/475#discussion_r542945413 sound to you? We had discussed in the office hours to track the ordering bug separately.
/hold
Please cancel the hold if that is resolved.
There is another minor thing where if the PR was based on the latest master then the PR description could add release notes. This is minor enough that we can ignore.

@jpeach Friendly ping. Could you please check above comment?

/hold cancel


Set Complex Value

The hint that comes up in the hover says:
"Each parameter should be seperated by colan (,)" but it's a comma in the brackets and in the template example where it is "1,0;"
Which is it, a comma (,) or a colan (:).
Using Art-Net v1.2.0

I think you need to read it as <channel>,<value>;<next_channel>,<next_value>;.

I agree, and that's what I did.
But my point really is that the documentation is not clear - in fact it's contradictory and therefore confusing for somebody with a) a non english background and b) a non technical person who is looking to use a more advanced product that the straightforward Stream Deck native application.


TAP 1.4.0 New Release QS Code Changes

Issue #, if available:

TAP 1.4.0 New Release Code Changes

Description of changes:

tap-setup-scripts:  update all scripts to support both single and multi-cluster TAP deployments. All tap-values profiles and resources were added.
Added a new parameter TAPClusterArch in CFT
aws-tap-entrypoint-existing-vpc.template.yaml: update all AWS resources to support both single and multi-cluster TAP deployments based on the condition of the input parameter TAPClusterArch.
Several AWS resources were created to support all four EKS clusters.
Update Kubernetes to version 1.23 & added the required EBS CSI Driver Addon to all EKS clusters to install TAP
Updated TAP to version 1.4.0
Complete End-to-End validation of TAP 1.4.0 on single & multi cluster deployments

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Please take care of a few files during Merge.

submodules/quickstart-amazon-eks is not deleted.
scripts/init-submodules.sh is not modified.

PS:  I have some difficulty excluding a few files in PR. Hence, I am not sure how to restore the above to the correct files.

Need to re-add the EKS submodule as it was deleted.

Not sure of how e2e tests worked without it. ?

Need to fix all of the ARNs to sub the AWS::Partition pseudo-parameter

Need to fix the parameter & output descriptions.

Other suggestions below.

Done. Please check.

/do-e2e-tests

/do-e2e-tests

/do-e2e-tests

/do-e2e-tests

/do-e2e-tests

/do-e2e-tests

/do-e2e-tests

@satya-dillikar Almost there. Need you to either update the outputs with insufficient description values or remove them. Removing the excess quotes is optional.
After that, all that's left on this PR should be for @annaone to review the descriptions for the new parameter & outputs for both templates when she's back. Once approved, we should be all set for this one for launch.

@tlindsay42, done. please start e2e.

/do-e2e-tests

Replaced with #131


Seccomp

High-level description
This PR introduces two new flags mesos_seccomp_enabled and mesos_seccomp_profile_name. The first flag is used to enable Seccomp isolator. The second flag specifies a name of a default Seccomp profile, e.g. default.json. Also, this PR includes a default Seccomp profile which is installed into /etc/dcos/mesos/seccomp/default.json. For more details, see design doc.
Corresponding DC/OS tickets (obligatory)

DCOS_OSS-3830 Enable linux/seccomp isolator by default for mesos in gen/calc.py
DCOS_OSS-3829 Add --enable-seccomp-isolator flags to mesos build script
DCOS_OSS-3828 Add default seccomp config in DC/OS

@mesosphere-mergebot label Ready For Review

Hey @abudnik  / @Gilbert88  - What kind of integration test will be suitable for this in DC/OS?
I can imagine DC/OS being very high level abstraction for this feature, but still interested in knowing thoughts on having any integration test addition that will be suitable for this feature.
Tests that live at one of these places:

https://github.com/dcos/dcos/blob/master/packages/dcos-integration-test/extra/test_mesos.py
https://github.com/dcos/dcos/tree/master/test-e2e

There is one test that can exercise that the default profile is placed at the correct location.

If this feature is enabled on dc/os, any UCR based integration tests would exercise the code change.
@orsenthil are you saying that we should verify that the default.json is at the correct location?

Hi @Gilbert88 , (cc: @abudnik )
Think about these scenarios.

When seccomp isolator is enabled in a DC/OS cluster, and an job that is run on UCR tries to invoke a restricted system call, feature should give desired error response or failure vs the differing response when isolator is disabled in the DC/OS cluster.

Also, a test that makes sure that seccomp isolation feature is available only on DC/OS 1.13 and above and not in earlier versions of DC/OS.

I was suggesting that these kind of tests will be valuable addition along with this feature.

@orsenthil @Gilbert88 @abudnik
I agree that we should add an integration test for this feature. However, to add an integration test for a feature, I think that feature should be enabled in DC/OS by default, right? In this PR, I see this seccomp feature is disabled by default.

@qianzhangxa probably we could enable it by default in the future. But today only a small group of customers request seccomp and leveraging libseccomp may have some perf impact which we need more testing against. I would suggest do not turn it on for 1.13.0

Hi @Gilbert88  - Yeah, I did notice that the feature is disabled by default, but it definitely gives an option to enable the feature, that's the reason I requested for a test.
You could make a note and JIRA, and please plan to bring in the test separately.

@Gilbert88 I agree to disable this feature for now. And another comment is, will we document the two new parameters (mesos_seccomp_enabled & mesos_seccomp_profile_name) and also the default seccomp profile included in DC/OS installer package?

@Gilbert88 I agree to disable this feature for now. And another comment is, will we document the two new parameters (mesos_seccomp_enabled & mesos_seccomp_profile_name) and also the default seccomp profile included in DC/OS installer package?

@qianzhangxa I think you were expecting all documentation would be included in this PR. Actually we have a JIRA to follow up docs https://jira.mesosphere.com/browse/DCOS_OSS-3870

To follow up with the integration test:
https://jira.mesosphere.com/browse/DCOS_OSS-4808


Terrain clipping not working

Describe the bug
I'm trying to implement the terrain clipping feature but I can't make it work. I'm using a baked Ocean Depth Cache which is placed at the same height as the ocean, clip surface and clip below terrain are both enabled in the material. Draw instanced is disabled on the terrain. I can see that the waves are flattened under the terrain but there's no clipping. (In case you're wondering, it's the same in edit and play mode)
Screenshots / video

Log
No errors/warnings
Versions
2019.4.0f1 Built-in
Platform
Windows
Hardware
GTX 1070

It seems like I was using an older baked depth cache (from a previous version). If anyone else comes across this issue just delete it and bake a new one.


capsh doesn't work with the container image which has no /bin/bash

Capsh is a great tool to drop capabilities for preventing jailbreak from chroot.
However, it hardcodes /bin/bash to run(in chrooted rootfs), thus we can not run
containers which don't have /bin/bash.
Maybe we have 3 options;

If there is no /bin/bash but /bin/sh, add a wrapper shell script as /bin/bash. (which just exec /bin/sh with given parameters)
If there is no /bin/sh, we just fail to run, and warn the reason.
Fix capsh (to use /bin/sh or just directly run given command) or make another command.

This commit just avoid using capsh. I would like to try contribute capsh to support -x (exec) mode or make capexec to fix this issue.

See my libcap repository. I committed 2 patches which allow minc to execute given command directly from capsh.


how to invite bot into server?

how to invite bot into server

This bot isn't public anymore. If you'd like to use it, you must the host the bot.

How do I do that
Get Outlook for iOShttps://aka.ms/o0ukef

From: Kody<EMAIL_ADDRESS>Sent: Tuesday, April 17, 2018 7:03:28 PM
To: kodycode/PokeBot-Python
Cc: hi1234567891234567890; Author
Subject: Re: [kodycode/PokeBot-Python] how to invite bot into server? (#1)
This bot isn't public anymore. If you'd like to use it, you must the host the bot.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/kodycode/PokeBot-Python/issues/1#issuecomment-382184494, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AkshPUYmIFapeq1wKQ4xogE-_DeVg32Lks5tpnS_gaJpZM4TZHe7.

Oh nevermind
Get Outlook for iOShttps://aka.ms/o0ukef

From: Srinivas Sirimarapu<EMAIL_ADDRESS>Sent: Thursday, April 26, 2018 8:13:34 PM
To: kodycode/PokeBot-Python; kodycode/PokeBot-Python
Cc: Author
Subject: Re: [kodycode/PokeBot-Python] how to invite bot into server? (#1)
How do I do that
Get Outlook for iOShttps://aka.ms/o0ukef

From: Kody<EMAIL_ADDRESS>Sent: Tuesday, April 17, 2018 7:03:28 PM
To: kodycode/PokeBot-Python
Cc: hi1234567891234567890; Author
Subject: Re: [kodycode/PokeBot-Python] how to invite bot into server? (#1)
This bot isn't public anymore. If you'd like to use it, you must the host the bot.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/kodycode/PokeBot-Python/issues/1#issuecomment-382184494, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AkshPUYmIFapeq1wKQ4xogE-_DeVg32Lks5tpnS_gaJpZM4TZHe7.


Multiple build for 11 devices

This pr have 11 devces only have wo work on 10

can be merged @LordShenron


feat: created get all job applications endpoint

I added the get job applications endpoint to the route of the project at /api/v1/jobs/{job_id}/applications. The endpoint accepts the get method.
Description
This endpoint fetches all job applications of a given job ID. The endpoint is only accessible by a superadmin.
Related Issue (Link to issue ticket)
The issue for this pull request can be found here.
Motivation and Context
This endpoint lets the superadmin view all the applications to a job opening. The request job_id must be of a job which exists in the database. Any other id get a 404 error.
How Has This Been Tested?
The endpoint has been tested. The unit tests are in the file tests/v1/job_application/test_fetch_all_applications.py.

[x] Data is added appropriately to the database.
[x] Appropriate response and status code is returned.
[x] Users must be authenticated and a superadmin.
[x] Non existing job_id will throw a 404 error.

The test can be run by:  PYTHONPATH=. pytest tests/v1/job_application/test_fetch_all_applications.py.
Affected areas of the code

I added a job_application.py file to the route, service and schema folders. The files outline the model for my response data and logic for the endpoint respectively.
I added a test file at tests/v1/job_application/test_fetch_all_applications.py.

Screenshots (if appropriate - Postman, etc):
Success Response

Non superadmin response

Non existing job ID

Types of changes

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the readme accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Pull latest changes and update your code accordingly because everything you need is already there. SO no need to create new schemas and all of that

It's done now.


[Enchance] internlm message to prompt

Thanks for your contribution and we appreciate it a lot. The following instructions would make your pull request more healthy and more easily receiving feedbacks. If you do not understand some items, don't worry, just make the pull request and seek help from maintainers.
Motivation
Please describe the motivation of this PR and the goal you want to achieve through this PR.
Modification
Please briefly describe what modification is made in this PR.
BC-breaking (Optional)
Does the modification introduce changes that break the backward-compatibility of the downstream repositories?
If so, please describe how it breaks the compatibility and how the downstream projects should modify their code to keep compatibility with this PR.
Use cases (Optional)
If this PR introduces a new feature, it is better to list some use cases here, and update the documentation.
Checklist

Pre-commit or other linting tools are used to fix the potential lint issues.
The modification is covered by complete unit tests. If not, please add more unit tests to ensure the correctness.
If the modification has a dependency on downstream projects of a newer version, this PR should be tested with all supported versions of downstream projects.
The documentation has been modified accordingly, like docstring or example tutorials.

I found we both implemented the chat template differently from the huggingface codes.
https://huggingface.co/internlm/internlm-chat-7b-v1_1/blob/main/modeling_internlm.py#L769

I found we both implemented the chat template differently from the huggingface codes. https://huggingface.co/internlm/internlm-chat-7b-v1_1/blob/main/modeling_internlm.py#L769

It's wrong, and will be fixed

@Harold-lkk Please provide some example messages containing two or more system roles. Tested OK with single system role.


Cou should handle KeyInterrupt

When a user uses KeyInterrupt, COU should give a hint where it stopped to the user

Duplicated by #103


Add project user settings to HS project

Adding new resource type for new JobQ API calls.
Ticket: https://intranet.scrapinghub.com/issues/32562.
Related with https://github.com/scrapinghub/dash/pull/951 and https://github.com/scrapinghub/jobq/pull/73.

Not needed because of revised approach to Dash/JobQ/Kumo settings, closing.


Deprecated method error.

When starting up the server, multiverse core gives a deprecated method error:
https://pastebin.com/SymhuS9V
Using latest release.
Running Paper 1.14.3 #117 build

I have noticed this same error at start up.

Duplicate of https://github.com/Multiverse/Multiverse-Core/issues/2000


givenName and familyName not being updated on PATCH requests

I'm having this issue while testing the provisioning from Azure where the givenName and the familyName are not being stored in the database correctly. This is what my mapping looks like:
  def self.scim_attributes_map
    return {
      id: :uuid,
      externalId: :external_id,
      userName: :email_address,
      active: :active,
      name: {
        givenName:  :first_name,
        familyName: :last_name,
      },
      title: :job_title,
    }
  end

This is what the Azure request looks like:
        patch "/scim_v2/Users/#{user_id}",
              headers: {
                "Authorization" => "Bearer #{authorization_token}",
                "Content-Type" => "application/scim+json; charset=utf-8"
              },
              params: {
                "schemas" => [
                  "urn:ietf:params:scim:api:messages:2.0:PatchOp"
                ],
                "Operations" => [
                  {
                    "op" => "add",
                    "value" => {
                      "displayName" => "Foo Bar",
                      "title" => "Bar Foo",
                      "name.givenName" => "Foo",
                      "name.familyName" => "Bar",
                      "name.formatted"=>"Foo Bar",
                    },
                  },
                ]
              }.to_json

Expected output:
Store the givenName in the column first_name and the familyName in the last_name column.
Actual output:
Those are attributes are not being stored at all on PATCH requests.
Additional info:
On create requests, those fields are properly populated though, so the mapping is correct. It must be something on the from_scim_patch! method.

Hi @pond, do you have any idea on what is going on here? Are you able to replicate the issue?

Sorry for the delay @eduardoborba - super busy in the day job and this is now a one-man-band for maintenance and development sadly so it doesn't get the attention it should. I'll look into this as soon as I can, you're not being ignored and you've not been forgotten; sorry again for the length of time it's taken to even send this response, though.

One question - are you able to modify that payload? The specific issue is the dotted notation inside the attributes within the value object, which I can't see documented just about anywhere as valid for PATCH "add", but it kinda makes sense in context; however, really you ought to have e.g. name.givenName in path and value is then just "Foo"; or path is name, then your value contains e.g. "givenName" => "Foo".
For the syntax of path, see:

https://www.rfc-editor.org/rfc/rfc7644#page-34

The path is optional for add operations, but:

https://www.rfc-editor.org/rfc/rfc7644#section-<IP_ADDRESS>

...says:
   The operation MUST contain a "value" member whose content specifies
   the value to be added.  The value MAY be a quoted value, or it may be
   a JSON object containing the sub-attributes of the complex attribute
   specified in the operation's "path"."

...which does not appear to permit a JSON object where relative attribute paths are specified; just sub-attributes of a given path.

@pond That's the payload we are getting from Azure, we don't really control that. This is what our attributes mapping looks like on the Azure provisioning settings:

Let me know if you have any suggestions to fix that by changing the mapping.

I've spent an unhealthy amount of time ploughing through mile after mile over blustering, exceptionally over-complex documentation from Microsoft, all of which seems to totally miss the point and none of which includes any screenshots matching what you have above LOL. Unusually it's not a lack of documentation this time, but a totally overwhelming amount of documentation - but all of it seems very similar and there is no clear steer on which bit of it is relevant at any given time.
The closest thing to useful that I could find is, of course, not from Microsoft, but this document:

https://docs.ifs.com/techdocs/23r2/030_administration/010_security/025_scim/

...which does show things like name.familyName as a user attribute mapping. It would seem that Microsoft's implementation just treats that as a blunt string with no understanding of what it means, throwing it directly into PATCH requests which are technically malformed (as far as I can tell - SCIM itself is an inconsistent and over-complex protocol that reeks of design-by-committee with nobody involved being responsible for producing a reference implementation and test suite).
This basically seems to mean that I somehow have to implement the nightmare of arbitrary paths inside already-extremely-awkward PATCH requests.
What I don't understand is why you're seeing this and nobody else is. We've had bug reports from the MS SCIM validator, and once fixed people have seemed happy. Perhaps they just gave up on Scimitar for Azure AD, or perhaps they got it to work - but in that case, why is your situation different? There's just no way I can devote the time to trying to understand the overwhelming mess of Microsoft's Azure-now-Entra to try and work that out.

Well, I think https://github.com/RIPAGlobal/scimitar/pull/125 should do it - under internal review now.

@eduardoborba Please could you give https://rubygems.org/gems/scimitar/versions/2.7.3 a try? I suspect we might need more than one iteration on this, but it'd be good to see if it at least moves things forward. Thanks!

Hey @pond,
Really appreciate the effort. I'm going to test that later today and will let you know.

@pond it's working perfectly now, thank you very much!
Feel free to close this issue :)

Fantastic news! Thanks for the detailed bug report and I'm glad that it's working.


clojure: <IP_ADDRESS>6 -> <IP_ADDRESS>3, refactor, closes #35136

I've also tried bumping to <IP_ADDRESS>9, but that didn't resolve the issue.

Failure on aarch64-linux (full log)
Partial log (click to expand)
error: while evaluating the attribute 'installPhase' of the derivation 'clojure-<IP_ADDRESS>3' at /var/lib/gc-of-borg/nix-test-rs-2/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-aarch64-community-2/pkgs/stdenv/generic/make-derivation.nix:148:11:
while evaluating the attribute 'buildInputs' of the derivation 'openjdk-8u172b02' at /var/lib/gc-of-borg/nix-test-rs-2/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-aarch64-community-2/pkgs/stdenv/generic/make-derivation.nix:148:11:
while evaluating the attribute 'buildCommand' of the derivation 'openjdk-bootstrap' at /var/lib/gc-of-borg/nix-test-rs-2/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-aarch64-community-2/pkgs/stdenv/generic/make-derivation.nix:148:11:
No bootstrap for system

Success on x86_64-darwin (full log)
Partial log (click to expand)
post-installation fixup
strip is /nix/store/v8ifq0hapblpr2z0hx4wvwg02w4aczqy-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/xl0x501lc61d8k50dpkgzrbxmnf7qp5i-clojure-<IP_ADDRESS>3/bin
patching script interpreter paths in /nix/store/xl0x501lc61d8k50dpkgzrbxmnf7qp5i-clojure-<IP_ADDRESS>3
/nix/store/xl0x501lc61d8k50dpkgzrbxmnf7qp5i-clojure-<IP_ADDRESS>3/bin/.clj-wrapped: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/19cc89zq5n68ibr53izy4zgcr5qg2rza-bash-4.4-p12/bin/bash"
/nix/store/xl0x501lc61d8k50dpkgzrbxmnf7qp5i-clojure-<IP_ADDRESS>3/bin/.clojure-wrapped: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/19cc89zq5n68ibr53izy4zgcr5qg2rza-bash-4.4-p12/bin/bash"
strip is /nix/store/v8ifq0hapblpr2z0hx4wvwg02w4aczqy-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/vaynqm35q8nxrqi0sb50lj6dbmygcg4m-clojure-<IP_ADDRESS>3-prefix/libexec
patching script interpreter paths in /nix/store/vaynqm35q8nxrqi0sb50lj6dbmygcg4m-clojure-<IP_ADDRESS>3-prefix
/nix/store/xl0x501lc61d8k50dpkgzrbxmnf7qp5i-clojure-<IP_ADDRESS>3

Success on x86_64-linux (full log)
Partial log (click to expand)
patching script interpreter paths in /nix/store/na6f97rz7j5skk49hjhhhr4p58gpsxly-clojure-<IP_ADDRESS>3
/nix/store/na6f97rz7j5skk49hjhhhr4p58gpsxly-clojure-<IP_ADDRESS>3/bin/.clj-wrapped: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/fcxliihmhm2ak9z4890gk0qw63zsbrcx-bash-4.4-p12/bin/bash"
/nix/store/na6f97rz7j5skk49hjhhhr4p58gpsxly-clojure-<IP_ADDRESS>3/bin/.clojure-wrapped: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/fcxliihmhm2ak9z4890gk0qw63zsbrcx-bash-4.4-p12/bin/bash"
checking for references to /build in /nix/store/na6f97rz7j5skk49hjhhhr4p58gpsxly-clojure-<IP_ADDRESS>3...
shrinking RPATHs of ELF executables and libraries in /nix/store/dyxh35f0xqaapy0skkpb717nbkj7ksy9-clojure-<IP_ADDRESS>3-prefix
strip is /nix/store/adidfx4pa7vmvby0gjqqmiwg2x49yr27-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/dyxh35f0xqaapy0skkpb717nbkj7ksy9-clojure-<IP_ADDRESS>3-prefix/libexec
patching script interpreter paths in /nix/store/dyxh35f0xqaapy0skkpb717nbkj7ksy9-clojure-<IP_ADDRESS>3-prefix
checking for references to /build in /nix/store/dyxh35f0xqaapy0skkpb717nbkj7ksy9-clojure-<IP_ADDRESS>3-prefix...
/nix/store/na6f97rz7j5skk49hjhhhr4p58gpsxly-clojure-<IP_ADDRESS>3


merge from arlac77/template-arlac77-github,arlac77/template-esm-only,arlac77/template-node-app

package.json

chore(deps): add ^7.3.4 remove ^7.3.3 (devDependencies.c8)

:tada: This PR is included in version 10.0.6 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Control Plane Gateway status

What
The control plane Gateway status should provide a rolled up view of the status of the per gateways it represents. It should also represent the condition of policies attached to those gateways.

[x] Define what our CP gateway status will contain and communicate
[ ] Implement the rolled up status

Note for the policies at the CP level each of these should add in a status condition to the control plane gateway status which is not part of this work

Proposal for aggregation https://github.com/Kuadrant/multicluster-gateway-controller/pull/150


SNOW-1063081 Bump opentelemetry-python dependencies to 1.23.0

This PR:

bumps opentelemetry-python dependencies to 1.23.0.
adjusts the SnowflakeLoggingHandler implementation to behave the same even with all the changes to opentelemetry-python's between 1.12.0 and 1.23.0

LGTM for the instrumentation scope patch, but please address Bogdan's comments.


Unmount /data volume using ansible before destroying resources

This is to solve the intermittent issue with volumes not being able to be destroyed. When I manually sudo umount /dev/xvdh and the run ./databox.sh down again, it instantly solves the issue, so maybe when we run ./databox.sh down it should first run some teardown tasks on ansible like unmounting any volumes.

Merged in #26


Test Data - 1000 Objects

Write mock data generation code.

1000 user, follower, service and event creation is provided


Python Authentication for a TM1 instance with OIDC

Hi, is it possible to launch authentication with Python for a tm1 instance with Security 5 (OIDC)?

I'm assuming you are using OIDC through CAM.
Then you can just follow this sample
https://code.cubewise.com/tm1py-help-content/how-to-authenticate-to-tm1-with-tm1py

Thank you Marius for the feedback. Unfortunately, we are not using OIDC through CAM.
Do we have steps for the TM1Py authentication via OIDC without CAM?

Just curious, how are you using OIDC without CAM? TM1s support for native OIDC basically in a beta state and not well tested.
From: Darshan Manjunath @.>
Reply-To: cubewise-code/tm1py @.>
Date: Tuesday, April 25, 2023 at 5:10 AM
To: cubewise-code/tm1py @.>
Cc: Subscribed @.>
Subject: Re: [cubewise-code/tm1py] Python Authentication for a TM1 instance with OIDC (Issue #905)
Thank you Marius for the feedback. Unfortunately, we are not using OIDC through CAM.
Do we have steps for the TM1Py authentication via OIDC without CAM?
—
Reply to this email directly, view it on GitHubhttps://github.com/cubewise-code/tm1py/issues/905#issuecomment-1521442476, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AEK7GZQ62WMLGP7B53K64QLXC6IFLANCNFSM6AAAAAAXD6VGE4.
You are receiving this because you are subscribed to this thread.Message ID: @.***>


Refactor typescript model types

change Runnable interfaces to types
create a base type called Runnable (RunnableType is still needed)
move all common properties to Runnable
change outside code to use Runnable instead of Test | TestGroup | TestSuite

Codecov Report

Merging #156 (4f1cb99) into main (623319f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #156   +/-   ##
=======================================
  Coverage   93.87%   93.87%
=======================================
  Files         109      109
  Lines        2742     2742
=======================================
  Hits         2574     2574
  Misses        168      168

Continue to review full report at Codecov.

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


KYC/KYB fails to get status

Affected Portal:
DevHub
Description:
Recipient Verification Status is failing to get the status.
User Type:
Regular Users
Reproduction Steps

Open any proposal on neardevhub.org
See Recipient Verification Status section. Shows error "Failed to get status"

Expected behavior
Should see user's verification status.
Visual Reference

Priority
High (P1) – Unexpected error or behavior
Environment Details

Device: Desktop
OS: Mac
Browser: Chrome
Version: Latest

It's working again!


Why not use the new semantic elements from HTML5?

It would be interesting to update the bootstrap to use the new sectioning elements (header, footer, section, article, aside and nav)?

assuming you mean the bootstrap examples ? (as bootstrap itself does not come with any actual prescribed/ready-made HTML)

I think the docs page (which uses <header>, <main> for example) does a pretty good job.
I do believe that there could be better use of <section> throughout the actual docs pages to group each section within the CSS, Components and Javascript page but I'm not sure if there is a reason that this is currently not done.

Use any elements you like. We use a few here and there, but they're not required. Nothing changes for the components though—they're just as customizable and functional.