stream validation fails

Hi,
the master-code gives me a "double callback!" error when validating my file-response. i could nail it down to 17e77be805d277b18177eaddfa65aaa561f42bfe, but i am fairly overwhelmed with what happens here. tag  v0.10.1 works well.
this is my path-definition:
  produces:
    - 'image/png'
    - 'image/jpeg'
  responses:
    200:
      description: Successful request.
      schema:
        type: file

the controller responds with res.sendFile(filePath). i try to provide a test for it soon but maybe somebody has an idea already.

I'll look into it.

update: it seems to be a problem with express.sendFile(). i tried to set up a test and that seems to work (although I'm not sure if i did it right). but since the tests not making use of express

@whitlockjc, @SandyChapman, i made a test for this issue. i hope you can come up with a solution.

I'll take a peek.

To backpack onto this, it appears in my project that validations don't properly validate when using pipe() from the request package. I believe that is a "stream" correct?
For example  request.get(url).pipe(res) will "bypass" validations. I have set required values for the response in my swagger.yaml file but they have no effect.

@smoebody @stv8 : I've had this PR open for some time now to address this issue I think. #367
Can either of you verify? If so, then maybe @whitlockjc would be kind enough to merge it.

@SandyChapman I can confirm that your change fixes my problem

@SandyChapman would you mind explaining what exactly your change does? I am a bit confused because it looks like it has to do with streaming files but I am just streaming a json response. Or is that basically the same thing as streaming a file?

@stv8 : I believe what's fixing your problem is the portion of the code:
if(writtenData.length === 0) {
      // Some code expects val to be undefined and not an empty string
      // when no data is passed to 'end' or written with 'write'.
      val = undefined;
}

Based on my memory of 8 months ago and this comment, I think the validation occurs on any defined value. If val is undefined, the validation is skipped. Hence, if we write nothing to the output, we force the val to be undefined and bypass the validator.
The fix has to do with there being different code paths for pipe vs json or data.
The rest of the code is just ensuring that if you are using a file type for your response, we don't parse to a string and mangle the encoding. Validation isn't run on file types, but the conversion from buffer to string will mess of the encoding of binary files as its interpreted as UTF8 in the toString method, so in this case we know validation isn't happening and we don't need the string conversion.
I'm going off my memory here as I'm not a JS dev and it's been a while so this might not be correct, but hopefully it's helpful.

Thanks, I also think it might be this code you added as well. It appears to handle my JSON response because it is not a file but is an instance of buffer, which then sets val to be my JSON string. Now that val is defined it triggers the validations correctly.
var isFile = checkIsFile(schema, req.swagger.apiDeclaration ? '1.2' : '2.0');
     if (val instanceof Buffer && !isFile) {
     val = val.toString(encoding);
}

i can confirm that it fixes this issue. conflicts need to be resolved manually though. @SandyChapman can you please have a look whether my test in PR #464 is obsolete by your PR's test or not.

Once we get @smoebody's question answer, I'll take the appropriate course of action.  Thanks for all the help everyone.

@whitlockjc i merged @SandyChapman s fix in my PR #464 and resolved all conflicts. can you merge into master please? i need this :) http://giphy.com/gifs/tN1lvnT4M6nte

@smoebody @stv8 : I've had this PR open for some time now to address this issue I think. #367
Can either of you verify? If so, then maybe @whitlockjc would be kind enough to merge it.

this pull request resolve my problem


Increase timeout for TestFirstStep in order to avoid it failing sometimes on CI

rename few vars on testutil.go to use same convention as in NewHelper func

Codecov Report

Merging #632 into rusk-ipc will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##           rusk-ipc     #632      +/-   ##
============================================
- Coverage     55.46%   55.45%   -0.02%
============================================
  Files           159      159
  Lines         10473    10474       +1
============================================
- Hits           5809     5808       -1
- Misses         3954     3956       +2
  Partials        710      710

Impacted Files
Coverage Δ

pkg/core/consensus/reduction/firststep/testutil.go
97.43% <100.00%> (ø)

pkg/gql/notifications/broker.go
75.67% <0.00%> (-2.71%)
:arrow_down:

pkg/core/data/base58/base58.go
72.82% <0.00%> (-1.09%)
:arrow_down:

pkg/core/data/transactions/provider.go
0.00% <0.00%> (ø)

pkg/core/consensus/round.go
70.70% <0.00%> (+0.63%)
:arrow_up:

Continue to review full report at Codecov.

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


Using playground I have problem

Missing required module 'secp256k1' when I'm beginning to code the package on SPI playground, can you help me resolve this

import K1
let alice = K1.ECDSA.PrivateKey()

@dannpr hey! I dont think K1 will work in a playground.
Instead you can create a tiny Swift Package and "play" with K1 in a unit test target of your package.

oh ok thanks ! I've found a way to make it work but I'd like to generate a standard public key like 0x... is there a truc because only let swift alicePublicKey = alice.publicKey give me this bash PublicKey(impl: K1.K1._PublicKeyImplementation(wrapped: K1.FFI.PublicKey.Wrapped(raw: __C.secp256k1_pubkey(data: (35, 77, 102, 27, 209, 204, 77, 182, 30, 12, 119, 5, 48, 97, 255, 242, 78, 88, 53, 176, 225, 67, 117, 200, 16, 44, 115, 143, 109, 146, 236, 13, 161, 176, 28, 46, 132, 241, 48, 162, 91, 207, 230, 113, 204, 73, 235, 117, 192, 116, 187, 63, 213, 175, 137, 101, 67, 66, 167, 247, 110, 180, 102, 102)))))

Ho and another question are you using secure enclave ? Because my goal is to use secure enclave. I know that cryptokit is using it

You can't. No third party library author can make use of SecureEnclave as you intend to. CryptoKit ONLY supports signing with SecureEnclave for the curve P256 (aka secp256r1 (notice r instead of k)) it does not even support SecureEnclave usage for the other curves in CryptoKit, e.g. Curve25519.
What "using Secure enclave" really means is that the signing takes place on the secure chip, and that it is impossible to read out the private key from the chip. You can only do that with P256.

Thanks a lot ! I'm using K1 and everything seems to work well, thanks !
Lasts questions, I hope I can make sense :

I see that even if I'm doing let privateKey = K1.ECDSA.PrivateKey()  I cannot have access to the private key of the key pair, right ?
About secure enclave, I can first authenticate the user with faceID by using secure enclave and then use your package to create a keypair on the K1 curve , verify and sign data right ? And store on Keychain the public key.

thanks again for your responce


Optimize editor icon SVGs using svgcleaner

This makes editor binaries about 230 KB smaller. I looked all icons in the editor help and node creation dialog, they still seem to display fine.
The command used was:
parallel svgcleaner --multipass {} {} ::: editor/icons/*.svg

PS: Is there any use to committing optimized SVGs to the godot-design repository? Committing only the source SVGs seems more logical to me.
Also, if this PR is merged, we should update instructions in godot-design to refer to svgcleaner instead of scour.

Thanks!


Remove-Item -Recurse against folders should use a .NET method on Windows instead of recursive list-and-delete

Problem
Possibly a non-issue, but this became apparent when trying to excise a poorly-name directory on Windows ('..). The folder was non-empty, and could not be entered with PowerShell, CMD, or Explorer to empty.
$files = Get-ChildItem;
$dir = $files[0]; # This is the badly-named directory

Remove-Item -Recursive -Force $dir; # Fails
# I assume because it tries to walk the contents and remove them individually,
# but the oddly-named directory "'.." cannot have its contents enumerated

$dir.Delete($true); # Succeeds, is System.IO.DirectoryInfo.Delete(bool recurse)

Solution
It would possibly speed up and increase the resilience of Remove-Item -Recursive operations against filesystem providers if PowerShell shortcutted directory removals to call the .NET System.IO.DirectoryInfo.Delete(bool recurse) method instead of recursively listing-and-deleting contents.
Testing
For testing purposes, the folder '.. was generated in an Angular project with the typo'd command ng generate component "'..\foo".

AFAIK Delete(true) does the same thing, it calls Dir.EnumerateFileSystemEntries and it itself will traverse down and delete the files before it deletes the actual parent dir The difference is that PowerShell adds a few more checks in there to handle things like like OneDrive reparse points, or broken symlinks, that I believe .NET struggled with in the past. Maybe that's no longer an issue and it could be simplified as you are suggesting. Even if it can't be simplified and there is another reason for PowerShell to enumerate and delete the contents itself then the fact that $dir.Delete($true) works means there's a bug in the logic here which should be fixed.


Standardize Documentation

standardize manpage markdown formatting  #2815
@rhatdan @rpjday PTAL

Tests are failing because you have white space.

@rhatdan Cleaned the whitespace up!

/approve
I only care about consistency.

Fine, Fix the "," to not be bold.

@ashley-cui Can you give us an update?

@rhatdan got caught up in other stuff on Friday, just need to change the "," to not be bold, and then I should be able to re-commit

@rhatdan  updated, PTAL

/lgtm


fix(HLS): Fix bad init segment request with byterange when using media playlist

@joeyparrish I plan to make an improved testsuite for media playlist, but it will have to be when I return (in 4 weeks).


Foundations section content audit removal

Based on content audit, some existing information on Foundations pages is outdated.
Macchiato goal:

Remove/merge certain Foundations pages (Scale, Division, Opacity)
Remove irrelevant/inaccurate information from the existing Foundations pages

14 Nov update: On track, made good progress

Squashed and merged


Clear the affinity on Noobaa if it was set previously to handle upgrades

We are adding affinity to Noobaa if not in noobaa-standalone mode or if placement is specified. But we are not clearing the affinity if the affinity was set previously. This is breaking upgrade.
https://bugzilla.redhat.com/show_bug.cgi?id=2314636

/retest

/cherry-pick release-4.17

/cherry-pick release-4.17


Tokio rt stack overflow with poem

Expected Behavior
Normal working
Actual Behavior
program crush after any mutation with
thread 'tokio-runtime-worker' has overflowed its stack
Steps to Reproduce the Problem
I can't reproduce this, but I know I have a lot of recursive types.
Specifications

Version: 4.0
Platform: Windows 10

I just wrote a lot of Boxes. It works
But I'm very sad that I can't see the border

Stack overflow occurs even if the large type is in another resolver

I recommend that large types be allocated on the heap as much as possible.


Fix ocis admin creation for idm

The admin user was created as part of the demo user set. But we need the
admin to be created always.

Hooking myself in to get notified when merged, @EParzefall fyi

@rhafer pls keep in mind that this will need a change here too (when the LibreIDM part gets finalized):
https://owncloud.dev/ocis/getting-started/demo-users/ (Following users are available in the demo set)

Current text in the docs
To create the demo users, run the inital setup step with an additional environment variable. ACCOUNTS_DEMO_USERS_AND_GROUPS=true ./bin/ocis server will generate the demo users listed in the table below. By default, it only generates the admin and one user for IDP and Reva respectively.

So this is aligned with the described behavior in the dev docs.

I doubt according the statement in dev docs Following users are available in the demo set: that  the admin user is part of the demo table. An admin user is now created OUTSIDE the demo env variable.
My point was that one must take care that the admin user will not longer be part of the demo users table - I may be wrong...


Cookies might not be available, fallback to ephemeral store is required

Hello
It is possible to block entirety of cookies in safari, and our customers sometimes do. Since we want to use fastboot, that would make our application just not working at all for these guys.
It should be possible to implement a fallback in cookie store similiar to that in adaptive store.
Would that be approved?

This should be possible to implement in a custom session store, but it's not something we want to have in ember-simple-auth directly.


No recommendation for external references for semantic identifier

closes #376
closes #396
semantic identifier for semanticId, valueId, isCaseOf no recommendation any longer to use external references

@J-Blume: may you please have a look? Thank you


Exit on errors

Ability to specify error codes that immediately stop the test. It might be handy if an authentication token is used and expired during the test.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

8 of 10 (80.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.2%) to 88.65%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/bench_run.rs
3
4
75.0%

src/http_bench_session.rs
5
6
83.33%

Totals

Change from base Build<PHONE_NUMBER>:
-0.2%

Covered Lines:
289

Relevant Lines:
326

💛  - Coveralls


Add more Euphoria and NmMessage features

I have to admit this, we didn't offer concrete ways to make peds do multiple NM behaviors without starting behaviors from the beginning again before. But sending multiple NM behaviors can be a game changer when we let scripts update NM behaviors without starting behaviors from the beginning.
I had to made a quite big demo script for a example scripts to show you how powerful sending multiple NM behaviors can be though. I also decided to create message helper classes (by porting CustomHelper) because you'll be tired of making NM messages with the base class when pushing NM messages to a ped.

Looks like you can pass NaturalMotion messages to peds when an arbitrary NM task is running. We can make peds to a electrocute NM behavior while they are running a CTaskNMHighFall task for high falling.

I didn't resolve conflicts properly but now I do.

I think custom message classes would be useful when you want to dispatch NM messages to various peds since you might not want to write boilerplate NM setting statements, but I agree with adding GTA.NaturalMotion.CustomHelper.Update() at least.

It's just a bit superflous to me having two separate ways of filling out NM messages in the API. If we didn't already have those existing helpers in the current API, I'd look at it differently.
You could write your example like this and do it with the current API too I think:
static private void SendShotBaseNmMessages(Euphoria messages)
{
  // some NM configs

  messages.BalancerCollisionsReaction.Start();

  messages.FallOverWall.MoveLegs = true;
  ...
}
static private void SendShotElectrocuteMessages(Euphoria messages)
{
  // some NM configs when stun gun bullet hits a ped
}

static private void OnShotProcess()
{
  SendShotBaseNmMessages(ped.Euphoria);

  var lastDamageRecord = ped.GetAllDamageRecords().OrderBy(x => x.GameTime).FirstOrDefault();

  // apply electrocute messages only when stun gun bullet hits a ped
  if (lastDamageRecord != default && lastDamageRecord.WeaponHash == WeaponHash.StunGun)
  {
    SendShotElectrocuteMessages(ped.Euphoria);
  }
}

I do think the helpers you created are still pretty useful, just maybe as a separate library, rather than in core SHVDN. Because even with the existing API it should be possible to use them like this (provided something is done about the start argument set by Message.SendTo):
static private List<Message> GetShotBaseNmMessages()
{
  // some NM configs
}
static private List<Message> GetShotElectrocuteMessages()
{
  // some NM configs when stun gun bullet hits a ped
}

static private void OnShotProcess()
{
  foreach (var message in GetShotBaseNmMessages())
    message.SendTo(ped);

  var lastDamageRecord = ped.GetAllDamageRecords().OrderBy(x => x.GameTime).FirstOrDefault();

  // apply electrocute messages only when stun gun bullet hits a ped
  if (lastDamageRecord != default && lastDamageRecord.WeaponHash == WeaponHash.StunGun)
  {
    foreach (var message in GetShotElectrocuteMessages())
      message.SendTo(ped);
  }
}

Affirmative. I admit we don't need to bother to duplicate similar classes just to apply predefined NM messages to various peds in the same API version.
When we create the v4 API, we may as well consider adding custom message classes (instead of CustomHelper), the properties for the explicit start parameter and CNmParameterResetMessage, and receive multiple NM messages like ReceiveNmMessages(params Message[] messages). Hope we will remember these proposals before releasing SHVDN v4.x.

So other than some implementation changes in NativeMemory, this commit has the following changes:

Add Message.RemoveArgument, CustomHelper.Update, Ped.IsRunningRagdollTask, a new overload for Ped.Ragdoll, and a demo scripts to show new NM features
Don't implicitly set start parameter to true on Message.SendTo call
Don't clear parameters after CustomHelper.Start is called


Use NTLM hash in place of password

Hi,
Would it be possible to implement the ability to use a NTLM hash in place of the password?
Regards,
Z

Hi,
Definitely not in the short term but it's in my to-do list !


Support QUERY operation (http method)

It would be great if the openapi spec will support the http QUERY method (draft) in v3.1
The QUERY method is very useful and we use it in couple of projects. Is it possible to extend the spec v3.1 or v3.2?

Related: #1747

oh, thank you!


Presentation: Demo Flatcar CAPZ deployment from Community Gallery at Flatcar Office Hours

Preliminary target date is the Office Hours call on Tuesday, January 10, at 5:30 PM CET.

Demo is here: https://youtu.be/e8QVlOyfa5w?t=543


[CI] Target node 10 and 11

Since node 10 go to LTS, should we target new node since they usually have better performance ?

Already contain both.


Add missing Delay instructions to Target

Summary
This commit adds a missing Delay operation to the target for backends. All the IBM backends have a delay instruction in the basis gates, but in the case of backends with a properties payload defined the basis gates is not looked at for the more rich data set available in the properties payload. This means the delay instruction which is not present in the properties is not included in the output target. This causes compilation with any circuit containing a delay to fail because it's not in the target instruction set for the backend. This commit fixes this by ensuring all backends have delay defined for all backends regardless of how instructions are parsed.
This commit is effectively a port of Qiskit/qiskit-ibm-provider#351 and Qiskit/qiskit-ibm-provider#346
Details and comments
Co-authored-by: Kevin Tian<EMAIL_ADDRESS>Fixes #528

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

-2 of 3 (33.33%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.03%) to 64.449%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit_ibm_runtime/utils/backend_converter.py
1
3
33.33%

Files with Coverage Reduction
New Missed Lines
%

qiskit_ibm_runtime/utils/backend_converter.py
1
13.86%

Totals

Change from base Build<PHONE_NUMBER>:
-0.03%

Covered Lines:
3216

Relevant Lines:
4990

💛  - Coveralls


Main Button label shows with no text

Setting main menu item title to "" shows a bit of a label background with no text next to the main button.
Could be because the extra padding recently added?

My bad! Thought that was the correct configuration.
Thanks!!!

Hi, fixed the bug, you can now update to 1.1.1
Thanks!


Delete timestamp

Deletes old (-49 hours) granule from image mosaic as part of the dataset-rotation worker.

Tests are failing, with the message
/home/runner/work/rabbitmq-worker/rabbitmq-worker/src/dataset-rotation/index.js
Error:   78:8  error  Unnecessary semicolon  no-extra-semi

see https://github.com/klips-project/rabbitmq-worker/actions/runs/5507236560/jobs/10036931263?pr=158
Please make the linter happy before merging ;-)

Thanks for the review.


Can't open hyperlinks in the built-in markdown preview

When I click the hyperlink of [[link]] in the built-in VSCode Markdown preview, it can't be opened because the path of linked file is wrong, as the working directory will be added to the front of the path.

For example:

After I click the hyperlink "Tips" in markdown preview, an error will pop up.

You can find the additional working directory as the underlined part.

OS: Windows 10 21H1 (The same problem happened in Windows 7, either)
VSCode version: 1.62.0

I have disabled all extensions for markdown but the problem still remains.

However, I found it works fine on my Linux system.

Moreover, I found if I use ![[link]] and click the hyperlink icon on the right, it will open the browser with the address: https://file+.vscode-resource.vscode-webview.net/(file path)

In the system above(Windows 10), nothing happens when click this icon.

Hi, thanks for reporting.
Seems like a Windows specific problem. Last time I tested this extension it worked fine under Windows, so it could be related to some recent updates in VSCode. Likely logic here and here should be checked.
Do you btw use latest version of this extension?

I'm experiencing same issue
Mac Monterey 12.0.1
VS Code Version: 1.62.3
Memo Version: v0.3.9

The same issue in Arch linux : 无法读取文件'/home/proitheus/Documents/vscode-memo/home/proitheus/Documents/vscode-memo/help/Features/Features.md'

Hi, it seems that internal logic for resolving links changed once again in VSCode inside markdown preview.
This problem should be fixed in the latest version of Memo (tested under macOS and Windows). Please update from Marketplace!


Remove kfp_tekton from the testing notebooks

Related to: https://issues.redhat.com/browse/RHOAIENG-4784
Description
The end-to-end tests in the notebooks repository are failing because the kfp_tekton library has been removed and replaced with odh-elyra.
Ref on CI test: https://prow.ci.openshift.org/view/gs/test-platform-results/pr-logs/pull/opendatahub-io_notebooks/476/pull-ci-opendatahub-io-notebooks-main-notebooks-e2e-tests/1771201766046568448#1:build-log.txt%3A518
Merge criteria:

[x] The commits are squashed in a cohesive manner and have meaningful messages.
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has manually tested the changes and verified that the changes work

For reference - kfp-tekton removal in #467

/test notebooks-e2e-tests


GoogleAnalyticsBridge.nativeTrackerId is not an object

I'm getting the following error on the listed devices :
iPad 2, iPad Retina, iPhone (5 and below)

however everything is fine on iPad Pro , iPad Air 2, iPad Air, iPhone (5s and above)
dependencies:

"react": "15.2.1",
"react-native": "0.31.0",
"react-native-google-analytics": "^1.3.2",
"react-native-google-analytics-bridge": "^3.0.0",

Just got same error on Android

Are both of you using RN 0.31?

Maybe you missed to add the package in your MainApplication:
Add this import:
import com.idehub.GoogleAnalyticsBridge.GoogleAnalyticsBridgePackage;

And update your getPackages implementation:
    @Override
    protected List<ReactPackage> getPackages() {
      return Arrays.<ReactPackage>asList(
          new MainReactPackage(),
          new GoogleAnalyticsBridgePackage() // add this line
      );
    }

@jerolimov Just double checked it, It's loaded for me and as I mentioned everything works as expected on : iPad Pro , iPad Air 2, iPad Air, iPhone (5s and above)

@el-lsan How are you testing it on those devices, through emulator?
Are you able to check which version of iOS those devices are running? Just to be sure; RN supports iOS 7.0 and above.
@joselitojunior1 Can you try @jerolimov's suggestion? For now, I'll assume your issue (on Android) is a different one than the iOS one.

@cbrevik I'm testing it on emulator and the IOS version on all devices that I listed is 9.3 (deployment target is IOS 8.0 and above)

@cbrevik it was a NPM/RNPM problem, afterall. Working fine on 0.31
thx

I'm still getting the error with RN 0.31.
I tried rnpm unlink and rnpm link as well.
the error only shows when run through command line i.e. react-native run-ios.
runs fine when I run it through Xcode, though.

Any clue? Works fine in iPhone, but having this problem in android

I was able to solve it on my end. It was indeed npm or rnpm problem.
On the iOS end, our package.json was set up incorrectly so that we were missing peer dependencies for a third party package.
On the Android side, it turns out rnpm didn't fully install the module. After following through the manual installation process, we found places where the needed changes weren't made -- namely, the changes necessary inMainApplication.java.
Hope this is helpful for someone and sorry for the false alarm!

@krapspark - How did you fix the missing dependencies on iOS?
I am on 0.31.0 and am experiencing the same issues.  Similar to you, it runs fine when compiling through xcode.

@robputer I noticed that there were warnings when i did a fresh npm install. i.e. rm -rf node_modules then npm install.
The warnings said something about missing peer dependencies.
The fix was setting the package versions in package.json to be pinned to the minor version (e.g. "react": "~15.2.1") rather than major version (e.g. "react": "^15.2.1")

Is this still a problem @el-lsan? I have been unable to replicate this.
Could you possibly provide a small sample project in which this happens? Does it happen with the example project in this repository?

Unable to reproduce, and issue seems to have gone stale. Closing for now.

got the same error

yarn pacage.json:
    "react": "^15.3.0",  // 15.3.2
    "react-native": "^0.35.0",  // 0.35.0
    "react-native-google-analytics-bridge": "^3.1.0",  // 3.1.0

OK, I found the problem.
The problem is I have multiple targets in my .xcodeproj and react-native link react-native-google-analytics-bridge will only do the job on one of targets. It made the native modules not be included.
To test whether your NativeModules is loaded, you may set a breakpoint in the -init method and launch the iOS app. It should stop at the breakpoint line when it initialized.
If not, please check your .xcodeproj -> Build Phase -> Link Binary with Libraries section. If there is no libRCTGoogleAnalyticsBridge.a on the list, follow the instruction Manual installation iOS for setup.

Do we have to detach from Expo if we want to do a manual installation?


Typo "Azure Open AI"→"Azure OpenAI"

https://learn.microsoft.com/en-us/azure/ai-studio/how-to/quota
https://github.com/MicrosoftDocs/azure-docs/blob/main/articles/ai-studio/how-to/quota.md
#PingMSFTDocs

Thanks, @hyoshioka0128!


Rework kind setup by removing Ingress and embedded etcd

This PR is a pretty big refresh to the kind setup because I was discussing it with an interested party and we just couldn't get the setup to work properly. I assume it has been broken in many ways for a while.
The changes in this PR are:

Eliminate usage of an Ingress. If we are going to have a node port exposure via nginx-ingress-controller, might as well expose kcp directly. This switches the kcp-front-proxy Service to be of type NodePort.
Make the kcp-front-proxy Service more configurable to pre-allocate a service IP and a specific node port, so we can hack in DNS entries (on the kcp Pod side) and node port forwards (on the kind side) pointing to the front proxy.

Subsequently, this PR sets up a host alias in the kcp pod to make sure that traffic to the external name gets routed to the kcp-front-proxy Service.

Get rid of using the latest tag. This is a setup that new users will end up with to try out kcp, there is no reason to expose them to development versions.
Stop using embedded etcd. It's broken (since #81).

Please give this a spin on your system before approving it, but for my kind setup it works without issues as far as I can say.

/approve
/lgtm


modal backDrop failed to resize on screen rotation

On iPad, if you rotate the device when a modal is open, the backdrop doesn't resize even if you close and reopen the modal.
It resize if you scroll element "behind" the backdrop.
test with v1.0.0-beta.13 and ionic-bower#master
Can be reproduce with the login box in the "sidemenu" stater app
thanks,

Greetings!
My sensors indicate a reply was requested, however, since we have not received one yet, this will be closed. If you are still experiencing this issue, please feel free to reopen it and include any information that may have been requested.
Thank you!


Add more clarity around bold 600 decision

In GitLab by @tauriedavis on Apr 2, 2020, 22:29
Its not clear from looking at pajamas what font weight we use for bold text. You have to inspect the text or find the variable in GitLab UI.
We should include the variable name and it's value on the typography page. We can also include explanatory text that explains why we use 600 over 700

It’s because some system fonts have a semi-bold, which would be mapped to 600, and that weight was preferred. For system fonts without a semi-bold it would be mapped to the 700 weight, which is why it could look extra bold in other OSs.

In GitLab by @uhlexsis on Apr 3, 2020, 03:30
Thanks for starting this!

In GitLab by @gitlab-bot on Apr 3, 2020, 05:56
Setting ~"devops::create" ~"group::ecosystem" based on ~"Category:FE/UX Foundations".

In GitLab by @tauriedavis on Dec 16, 2021, 23:53
Bold value is now listed on the type fundamentals page


⚠ ClamAV has degraded performance

In 93cc04e, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 441 ms

Resolved: ClamAV performance has improved in 92defa4.


[#131] Delete old folder hibernate-rx-core

It probabaly hasn't been deleted when we rebase the related PR.
The files are already in the right place in: hibernate-reactive-core

hi @DavideD, sorry about that. I accidentally checked these in after rebasing the DB2 PR. I removed these files in PR #140 before I noticed this one.

No problem


Sorting attrs?

In my output, similar attrs in different log lines are not printed in a consist order. For example:
Oct  6 12:05:22.593 INF Query sql="-- name: Ping :one\nSELECT $1\n" args="[1]" time=2.085667ms rowCount=1 pid=6185 module=grpc-public traceID=e0be34409ac17877e5f664456f044058
Oct  6 12:05:22.823 INF Query args="[1]" sql="-- name: Ping :one\nSELECT $1\n" rowCount=1 time=1.956472ms pid=6185 traceID=e0be34409ac17877e5f664456f044058 module=grpc-public

This makes human reading of such log lines difficult, especially if many of them are printed continuously and you need to search for a particular attr every time.
Would it be possible to sort attrs as they are printed, so that they're always printed in the same order?

Interesting! Will write a code snippet, thank you

Thanks for your quick response @lmittmann :-) I thought this might be a general issue with slog attrs, but it was not. :-)
The root issue turned out to be that the pgx.Logger API uses attrs map[string]any. This was translated into slog attrs using a simple for k, v := range attrs loop, which of course does not guarantee a stable order. By rewriting this adapter to first sort the keys, and then pass the attrs in stable order to slog, the issue is resolved.
The approach taken by tint to print in order of appearance indeed seems best to me, and if someone really desires their attrs to be sorted before printing them, it is probably easy to write an slog.Handler which does this.


WHY?!

No need to get angry. If you don't like it, don't use it. Seem simple
enough.
On Tue, Mar 7, 2017, 6:35 PM Dan Wiegand<EMAIL_ADDRESS>wrote:

Why all the extra crap? Do you think people who want to use this as a
boilerplate want to have to delete all your "MyAmazingComponents"?
FAR from a boilerplate, so annoying
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/RyanCCollins/react-redux-simple-starter/issues/4, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AEaBIOaWpWah7Ifyv5lH66xtmKELM9BVks5rjgX3gaJpZM4MWQa7
.


What about replace Hackney with Tesla?

Describe the feature or improvement you're requesting
Tesla is easier to control than Hackney.
(ex. http2, retrying, ...)
What about replace Hackney with Tesla, and set default Tesla adapter as Hackney adapter?
Additional context
No response

It can be a breaking change, but I think it's worth it.

hey, thanks for the suggestion. It would be really cool, especially the use of middleware would simplify part of the client and make it more flexible.
I will evaluate this, but I certainly won't be able to do anything anytime soon. If so, I will try to see if it is possible to integrate it into a first major version (maybe with a compatibility layer to be retrocompatible with the old apis)

I would rather Req: https://github.com/wojtekmach/req

This would be helpful.
https://elixirforum.com/t/mint-vs-finch-vs-gun-vs-tesla-vs-httpoison-etc/38588/11


Add performance tests

We should have performance tests so that we know when a change to a library results in a performance regression. These tests should be run in our nightly CI jobs, or otherwise aligned with engsys team recommendations.

We do not currently have any perf tests for iOS client libraries.  When we do (or to help get started on them) please reach out to @mikeharder so we can get them integrated into release perf runs.


Fix( unit ): using the. exact modifier error when all system modifiers

What kind of change does this PR introduce? (check at least one)

[x] Bugfix
[ ] Feature
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

If yes, please describe the impact and migration path for existing applications:
The PR fulfills these requirements:

[x] It's submitted to the dev branch for v2.x (or to a previous version branch), not the master branch
[x] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[x] All tests are passing: https://github.com/vuejs/vue/blob/dev/.github/CONTRIBUTING.md#development-setup
[x] New/updated tests are included

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:

Can you provide a rationale for this change please?

new Vue({
    el: document.createElement('div'),
    template: "<div @click.ctrl.alt.shift.meta.exact></div>"
});

vue.js:634 [Vue warn]: Failed to generate render function:

SyntaxError: Unexpected token ) in

with(this){return _c('div',{on:{"click":function($event){if(!$event.ctrlKey)return null;if(!$event.altKey)return null;if(!$event.shiftKey)return null;if(!$event.metaKey)return null;if()return null;}}})}

(found in )

Never mind, we can use this repro:
https://template-explorer.vuejs.org/#<div %40click.ctrl.alt.shift.meta.exact><%2Fdiv>


Renaming image loader

New Pull Request Checklist

[x] I have read and understood the CONTRIBUTING guide

[x] I have read the Documentation

[x] I have searched for a similar pull request in the project and found none

[x] I have updated this branch with the latest master to avoid conflicts (via merge from master or rebase)

[x] I have added the required tests to prove the fix/feature I am adding

[x] I have updated the documentation (if necessary)

[x] I have run the tests and they pass

[x] I have run the lint and it passes (pod lib lint)

This merge request fixes / reffers to the following issues: #2291
Pull Request Description
Maybe the final renaming. 😅
Renaming all SDWebImageLoader protocol and related class naming, to SDImageLoader.
This also remove the SDWebImageDownloader.h header in SDWebImageManager. Since we can custom loader/cache, we'd better only expose the protocol, but not the acutal class, which cause more dependency. (The implementation does not get effected)
This SDWebImageLoader is introduced in 5.x, so it's not a API break change to 4.x user.

Codecov Report

:exclamation: No coverage uploaded for pull request base (5.x@0a0bdc9). Click here to learn what that means.
The diff coverage is 68.42%.

@@          Coverage Diff           @@
##             5.x    #2326   +/-   ##
======================================
  Coverage       ?   69.24%
======================================
  Files          ?       48
  Lines          ?     6639
  Branches       ?        0
======================================
  Hits           ?     4597
  Misses         ?     2042
  Partials       ?        0

Flag
Coverage Δ

#iOS
70.36% <72.22%> (?)

#macOS
67.92% <64.7%> (?)

Impacted Files
Coverage Δ

SDWebImage/NSButton+WebCache.m
0% <0%> (ø)

SDWebImage/UIButton+WebCache.m
62.26% <100%> (ø)

SDWebImage/UIImageView+WebCache.m
21.42% <100%> (ø)

...age/FLAnimatedImage/FLAnimatedImageView+WebCache.m
76.81% <100%> (ø)

SDWebImage/SDWebImageDownloader.m
81.09% <100%> (ø)

SDWebImage/MapKit/MKAnnotationView+WebCache.m
70.37% <100%> (ø)

SDWebImage/SDWebImageDownloaderOperation.m
87.71% <100%> (ø)

SDWebImage/UIImageView+HighlightedWebCache.m
75.86% <100%> (ø)

SDWebImage/UIView+WebCache.m
61.44% <100%> (ø)

SDWebImage/SDAnimatedImageView+WebCache.m
65.62% <100%> (ø)

... and 3 more

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 0a0bdc9...cec9cbe. Read the comment docs.

@bpoplauschi Well...Are you busy recently ? I guess we have taken too long time for 5.0 release...Maybe we can get it done before WWDC 2018 ? 😅

👍


Build Restic on DragonFlyBSD

Output of restic version

root@:~/restic # ./restic version
restic 0.17.2-dev (v0.17.2-257-g408ec41a1) compiled with go1.23.2 on dragonfly/amd64

uname -a reports:
root@:~/restic # uname -a
DragonFly  6.4-RELEASE DragonFly v6.4.0-RELEASE #54: Fri Dec 30 09:06:50 PST 2022     root@pkgbox64.dragonflybsd.org:/usr/obj/usr/src/sys/X86_64_GENERIC  x86_64

What should restic do differently? Which functionality do you think we should add?

Whilst I realise that DragonFlyBSD is an incredibly niche platform, it was fairly easy, to add very generic support for it, as a derivative of FreeBSD, with official GoLang support, so it could help Restic become accessible, to an additional user base, if we also supported it.
What are you trying to do? What problem would this solve?

Out of curiosity, I decided to try building the latest checkout of Restic, on DragonFlyBSD, after noticing that the GoLang toolchain supports it, and seeing that no-one else has mentioned it, in the issue tracker.
On the first attempt, I received the following error:
root@:~/restic # ./buildr.sh ^M^M
go run build.go^M
warning: GOPATH set to GOROOT (/root/go) has no effect^M
warning: GOPATH set to GOROOT (/root/go) has no effect^M
# github.com/restic/restic/internal/fs^M
internal/fs/node.go:25:24: undefined: nodeFillGenericAttributes^M
internal/fs/node.go:28:26: undefined: nodeFillExtendedAttributes^M
internal/fs/node.go:259:12: undefined: nodeRestoreExtendedAttributes^M
internal/fs/node.go:266:12: undefined: nodeRestoreGenericAttributes^M
build failed: exit status 1^M
exit status 1^M
*** Error code 1^M
^M
Stop.^M
make: stopped in /root/restic^M

After copying internal/fs/node_aix.go, to internal/fs/node_dragonfly.go, and tweaking it, I was able to produce a working binary, that runs on DragonFlyBSD/x86-64, albeit without support for extended attributes.
If there's any interest, I can supply this change, as a pull request?
Did restic help you today? Did it make you happy in any way?
I'm surprised by how easy to work with, the Restic codebase is, despite having very limited knowledge of GoLang.

After copying internal/fs/node_aix.go, to internal/fs/node_dragonfly.go, and tweaking it, I was able to produce a working binary, that runs on DragonFlyBSD/x86-64, albeit without support for extended attributes.

That sounds simple enough that we can risk adding yet another platform. Support for the platform will be limited though, if something platform-specific is broken then it will likely stay that way until someone contributes a fix. In particular, our CI tests only cover Linux/macOS/Windows, that is the other platforms may break without us noticing. All other platforms are just built but not tested by the CI: https://github.com/restic/restic/blob/408ec41a1d1d9f94e9eca78f34a9f86aed84e18f/.github/workflows/tests.yml#L210-L240 .

After setting up a new DragonFlyBSD VM, installing the latest release of Go (involves curl https://dl.google.com/go/go1.23.3.dragonfly-amd64.tar.gz --output - | tar xvf -, moving the "go" directory to /usr/local, and adding "/usr/local/go/bin", to the PATH, in the "set path()" stanza of ~/.cshrc, on DragonFlyBSD), it looks like I can build the latest master branch version of Restic:

Creating a new repository, with an empty password seems to work properly:

I'm also able to backup /root, without any surprises:

Running a backup of / seems mostly uneventful:

It looks like /proc/PID/dbregs, and /proc/PID/fpregs files are larger than their advertised 0-byte, or 512-byte sizes, being system-generated, but I'm able to stop a backup, and start a new backup, excluding /proc, and everything finishes, without errors.
I also needed to exclude /root/TestRepository, since Restic tries to archive its own active repository, and I haven't tried to do anything, involving BSD flags (DragonFlyBSD has "chflags", but there doesn't seem to be many objects, other than /tmp (has the "noscache" flag, according to ls -lo /), that use them).
I'm unsure of what'd be necessary, to plumb in FUSE mounting support, if it's possible, and DragonFlyBSD 6.4.0 doesn't seem to ship any obvious way of manipulating extended attributes, out-of-the-box (but, the manual pages, for TAR, and CURL at least mention flags exercising them), so I guess we can call this the basics of support, for the time being...

Thanks a lot for testing!

involving BSD flags (DragonFlyBSD has "chflags", but there doesn't seem to be many objects, other than /tmp (has the "noscache" flag, according to ls -lo /), that use them).

There's no BSD flags support so far. But that's already sort of tracked via https://github.com/restic/restic/issues/4076 .

I'm unsure of what'd be necessary, to plumb in FUSE mounting support, if it's possible

There is already fuse support for freebsd, so maybe dragonflybsd could just work.

DragonFlyBSD 6.4.0 doesn't seem to ship any obvious way of manipulating extended attributes, out-of-the-box (but, the manual pages, for TAR, and CURL at least mention flags exercising them)

pkg/xattr doesn't have DragonFlyBSD support, so maybe the xattr syscall just don't exist there?

so I guess we can call this the basics of support, for the time being...

Yes that is definitely good enough for now. I'll close this issue for now. We can figure out additional details later on if someone (or you) takes care of it.


Return vs Signal distribution plot

While the "mean daily return by quantile" plot tells us how well the factor differentiates forward returns across the signal/factor values, the shown results depend strongly on the number of quantiles used. So it would be nice to add a distribution plot of returns vs signal/factor to properly see the relationship between returns and signal
Just to give an idea of what I mean, here are some pictures.

I give up on this, too many plots already


kabstraction in approximate unification

I have hit the following issue many times now in different contexts.
constants (f : ℕ → ℕ) (E : (ℕ → ℕ) → ℕ)
          (H : ∀ (g : ℕ → ℕ), E (λ x, g (f x)) = 0)

example (g h : ℕ → ℕ) : E (λ x, h (g (f x))) = 0 := by simp [H]

yields the following trace
[type_context.is_def_eq_detail] [1]: E (λ (x : ℕ), h (g (f x))) =?= E (λ (x : ℕ), ?x_0 (f x))
[type_context.is_def_eq_detail] process_assignment ?x_0 (f x) := h (g (f x))
[type_context.is_def_eq_detail] [2]: f x =?= g (f x)
[type_context.is_def_eq_detail] [3]: f =?= g
[type_context.is_def_eq_detail] on failure: f =?= g
[type_context.is_def_eq_detail] on failure: f x =?= g (f x)
[type_context.is_def_eq_detail] on failure: E (λ (x : ℕ), h (g (f x))) =?= E (λ (x : ℕ), ?x_0 (f x))
[type_context.is_def_eq] E (λ (x : ℕ), h (g (f x))) =?= E (λ (x : ℕ), ?x_0 (f x)) ... failed  (approximate mode)
0. [debug.simplify.try_rewrite] fail to unify 'H':
------------------------------------------------
E (λ (x : ℕ), h (g (f x)))
=?=
E (λ (x : ℕ), ?x_0 (f x))
------------------------------------------------

This is as expected according to A5: https://github.com/leanprover/lean/blob/master/src/library/type_context.cpp#L1536-L1539
However, what I want in this situation is to kabstract all occurrences of f x, to yield the following assignment:
?x_0 := (λ (fx : ℕ), h (g fx))

Even if we do not add any features to the type-context, it would be good to have a canonical solution to this problem.
A few possibilities:

Try this in approximate mode when FO fails.
Add a new kapproximate flag.
Have an annotation that tells the type-context explicitly to use this approach, e.g.

(H : ∀ (g : ℕ → ℕ), E (λ x, (:kapprox g) (f x)) = 0)

Expose kabstract and probably many other things in Lean, so that users can write special converters for each lemma that requires this kind of unification.

Any thoughts? There may also be satisfactory workarounds that I have overlooked. At first glance I like the precision of (3), but I haven't thought through all the details yet.

I have hit the following issue many times now in different contexts.

This is yet another gadget for the unifier/matcher. The example you have described is very abstract.
Could you elaborate it?
Did you try to workaround it?
Do you think this will happen in other application domains?
 Your approach is clever, I had not thought of that. I spoke to Jeremy about this issue though, and we have a different workaround (actually, a more elegant solution) for the specific case I was stuck on when I posted. Jeremy has yet to hit an issue like this at all, and we agreed it was not worth adding another gadget to the unifier unless more annoyances accumulated in other application domains. I am happy to close this issue for now if you agree.

No problem. We can reopen if we find it again.


leaflet marker cluster

can i directly add a geojson file to marker cluster ?if so how can i do it

Hi,
I am not sure I understand exactly what you are trying to achieve?
In case you have GeoJSON compliant data in a separate file, you could:

Load the file content
Parse the content into a JS object (typically with JSON.parse)
Convert the JS object containing the GeoJSON data into Leaflet layers with L.geoJSON
Add the Leaflet GeoJSON Layer Group into a MarkerClusterGroup, so that point features are clustered

If you need further help on how to use Leaflet and/or Leaflet.markercluster, please consider asking on StackOverflow, because you will reach a wider audience there.
The issues in this GitHub repo are for bugs.

Hi,
I am not sure I understand exactly what you are trying to achieve?
In case you have GeoJSON compliant data in a separate file, you could:

Load the file content
Parse the content into a JS object (typically with JSON.parse)
Convert the JS object containing the GeoJSON data into Leaflet layers with L.geoJSON
Add the Leaflet GeoJSON Layer Group into a MarkerClusterGroup, so that point features are clustered

If you need further help on how to use Leaflet and/or Leaflet.markercluster, please consider asking on StackOverflow, because you will reach a wider audience there.
The issues in this GitHub repo are for bugs.

ok thanks sorry for raising this as an issue.


⚠ Bofrost.at has degraded performance

In 2d378ea, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1721 ms

Resolved: Bofrost.at performance has improved in 184353b after 6 minutes.


Consider what CLI options "treemaker" should allow to override the config file

Should the -d, -n, -t, etc. flags be allowed here? I don't think they should be used, but they were there before so they should probably be allowed.
Should the dataset be allowed to be specified like this?
In other words, we need to clean up what command line options are allowed where.

Resolved, see 2e17735afe6506239227e12146151fba96b1a585.


[WIP] Spruce up API and add examples

TODO

[ ] runFresh' -> runFresh
[ ] Flipped run* args for Reader and State*
[ ] Moar examples

Motivation
This library has the most going for it among the extensible-effects family, so I'm putting some effort in to make it more attractive.

Where should a extensible-effects -> freer-effects migration guide go?

Either new section README.md or separate file like MIGRATION.md.


Write better Analyzers tutorial

Currently only an advanced example, but should be a simple example with a function like there is for Covasim: https://docs.idmod.org/projects/covasim/en/latest/tutorials/tut_analyzers.html#Custom-analyzers
In general, we should probably check what "light-lift" things we can port from Covasim/HPVsim/etc documentation.

Closed by #673


Rename ILMTypes to Image as well

ILMTypes are currently called Vision... instead of Image... even tho everything else is called Image...
They should be renamed

ILMType is updated but docu still contains LitVisionEncoder - should be updated as well


fix(rangeSlider): left handle default position

fixes #2449

That doesn't really fix the issue. it's both the left and right handles and the labels that should be fixed. Let me know if you want to check together.

Hard to take a decision on this one, let's wait for instantsearch.css 👍


Add experimental Windows10 platform pipeline

So that we can evaluate Windows10 builder and update the tests as needed.

I've assigned the new pipeline to the Experimental agent pool and launched a build: https://builds.gradle.org/viewQueued.html?itemId=29202866


Update project for Swift 3

Update project for Swift 3 (Issue #8), corresponding to Swinject PR #113.

@jakubvano, would you merge this PR if it does not break your changes in PR #15?
I merged master into this PR branch (in 33a6360) to update your PR #15 code to support Swift 3 (in c2601e1).

👍looks good. Merged and assigned to milestone

Thanks @jakubvano👍


Remove ?project_folder=/proj from the requests

This is no longer necessary with the multi-user support

done and updated on the live demo website t2wml.isi.edu


Fix github actions

Amends #23674
See https://github.com/microsoft/vcpkg/actions/runs/2022438982

This does seem to me to be the correct thing to do, but I also can't seem to test it because PR testing isn't running for me... https://github.com/microsoft/vcpkg/pull/23723


visualizationResultToBytes converts anything that looks like a string

Pull Request Description
Use InteropLibrary.isString and asString to convert any string value to byte[]
Important Notes
Also contains a support for Metadata.assertInCode to help locating the right place in the code snippets.
Checklist
Please include the following checklist in your PR:

[x] The documentation has been updated if necessary.
[x] All code conforms to the
Scala,
Java,
style guides.
All code has been tested:

[x] Unit tests have been written where possible.

I tried various ways to write a test. Modify RuntimeVisualizationsTest to work with JavaScript object - it does, but it doesn't encode the value to byte[]. At the end I separated the conversion into own (Java) class and wrote a special TruffleObject that handles the "to string" conversion.


Visual Studio 2017 Build Failure and LNK4217 Warnings

What platform/OS are you using?
Windows 10
What compiler are you using? what version?
Visual Studio 2017 Community Edition
What's your CMake arguments?
C:>
mkdir aws-sdk-cpp-build
cd aws-sdk-cpp-build

cmake -G "Visual Studio 15 2017 Win64" -DBUILD_ONLY="s3;s3-encryption" -DBUILD_SHARED_LIBS=OFF -DFORCE_SHARED_CRT=OFF -DCMAKE_INSTALL_PREFIX="C:\aws-sdk-cpp-build\install" C:\aws-sdk-cpp

msbuild /p:Configuration=Release /p:Platform=x64 INSTALL.vcxproj

Can you provide a TRACE level log? (sanitize any sensitive information)
The build initially progresses, but ends with the following messages:
[...]
C:\aws-sdk-cpp-build\INSTALL.vcxproj" (default target) (1) ->
"C:\aws-sdk-cpp-build\ALL_BUILD.vcxproj" (default target) (3) ->
"C:\aws-sdk-cpp-build\aws-cpp-sdk-s3-integration-tests\aws-cpp-sdk-s3-integration-tests.vcxp
roj" (default target) (12) ->
aws-c-event-stream.lib(event_stream.obj) : error LNK2019: unresolved external symbol __imp
__wassert referenced in function aws_array_list_length [C:\aws-sdk-cpp-build\aws-cpp-sdk-s3-
integration-tests\aws-cpp-sdk-s3-integration-tests.vcxproj]
aws-c-common.lib(error.obj) : error LNK2001: unresolved external symbol __imp__wassert [C:
\aws-sdk-cpp-build\aws-cpp-sdk-s3-integration-tests\aws-cpp-sdk-s3-integration-tests.vcxproj
]
aws-c-common.lib(array_list.obj) : error LNK2001: unresolved external symbol __imp__wasser
t [C:\aws-sdk-cpp-build\aws-cpp-sdk-s3-integration-tests\aws-cpp-sdk-s3-integration-tests.vc
xproj]
aws-c-common.lib(encoding.obj) : error LNK2001: unresolved external symbol __imp__wassert
[C:\aws-sdk-cpp-build\aws-cpp-sdk-s3-integration-tests\aws-cpp-sdk-s3-integration-tests.vcxp
roj]
C:\aws-sdk-cpp-build\bin\Release\aws-cpp-sdk-s3-integration-tests.exe : fatal error LNK112
0: 1 unresolved externals [C:\aws-sdk-cpp-build\aws-cpp-sdk-s3-integration-tests\aws-cpp-sdk
-s3-integration-tests.vcxproj]
48 Warning(s)
20 Error(s)
The issue appears to be that the INSTALL.vcxproj does not build the depedencies first. I have managed to work around this by doing the following after running cmake:
msbuild /p:Configuration=Release /p:Platform=x64 .deps\ALL_BUILD.vcxproj
msbuild /p:Configuration=Release /p:Platform=x64 INSTALL.vcxproj
This seems to solve the build situation, but I have noticed the following warnings when the INSTALL.vcxproj msbuild (step 2 above) completes:
[...]
"C:\aws-sdk-cpp-build\INSTALL.vcxproj" (default target) (1) ->
"C:\aws-sdk-cpp-build\ALL_BUILD.vcxproj" (default target) (3) ->
"C:\aws-sdk-cpp-build\aws-cpp-sdk-s3-integration-tests\aws-cpp-sdk-s3-integration-tests.vcxp
roj" (default target) (12) ->
LINK : warning LNK4098: defaultlib 'MSVCRT' conflicts with use of other libs; use /NODEFAU
LTLIB:library [C:\aws-sdk-cpp-build\aws-cpp-sdk-s3-integration-tests\aws-cpp-sdk-s3-integrat
ion-tests.vcxproj]
aws-c-event-stream.lib(event_stream.obj) : warning LNK4217: locally defined symbol fwrite
imported in function aws_event_stream_message_to_debug_str [C:\aws-sdk-cpp-build\aws-cpp-sdk
-s3-integration-tests\aws-cpp-sdk-s3-integration-tests.vcxproj]
aws-c-event-stream.lib(event_stream.obj) : warning LNK4217: locally defined symbol __stdio
_common_vfprintf imported in function fprintf [C:\aws-sdk-cpp-build\aws-cpp-sdk-s3-integrati
on-tests\aws-cpp-sdk-s3-integration-tests.vcxproj]
aws-c-common.lib(error.obj) : warning LNK4049: locally defined symbol __stdio_common_vfpri
ntf imported [C:\aws-sdk-cpp-build\aws-cpp-sdk-s3-integration-tests\aws-cpp-sdk-s3-integrati
on-tests.vcxproj]
aws-c-event-stream.lib(event_stream.obj) : warning LNK4217: locally defined symbol __stdio
_common_vsprintf imported in function snprintf [C:\aws-sdk-cpp-build\aws-cpp-sdk-s3-integrat
ion-tests\aws-cpp-sdk-s3-integration-tests.vcxproj]
aws-c-common.lib(common.obj) : warning LNK4217: locally defined symbol free imported in fu
nction s_default_free [C:\aws-sdk-cpp-build\aws-cpp-sdk-s3-integration-tests\aws-cpp-sdk-s3-
integration-tests.vcxproj]
aws-c-common.lib(common.obj) : warning LNK4217: locally defined symbol malloc imported in
function s_default_malloc [C:\aws-sdk-cpp-build\aws-cpp-sdk-s3-integration-tests\aws-cpp-sdk
-s3-integration-tests.vcxproj]
aws-c-common.lib(common.obj) : warning LNK4217: locally defined symbol realloc imported in
function s_default_realloc [C:\aws-sdk-cpp-build\aws-cpp-sdk-s3-integration-tests\aws-cpp-s
dk-s3-integration-tests.vcxproj]
aws-c-common.lib(error.obj) : warning LNK4217: locally defined symbol __acrt_iob_func impo
rted in function aws_register_error_info [C:\aws-sdk-cpp-build\aws-cpp-sdk-s3-integration-te
sts\aws-cpp-sdk-s3-integration-tests.vcxproj]
aws-c-common.lib(error.obj) : warning LNK4217: locally defined symbol abort imported in fu
nction aws_register_error_info [C:\aws-sdk-cpp-build\aws-cpp-sdk-s3-integration-tests\aws-cp
p-sdk-s3-integration-tests.vcxproj]
aws-c-common.lib(cpuid.obj) : warning LNK4217: locally defined symbol atoi imported in fun
ction aws_common_private_has_avx2 [C:\aws-sdk-cpp-build\aws-cpp-sdk-s3-integration-tests\aws
-cpp-sdk-s3-integration-tests.vcxproj]
aws-c-common.lib(cpuid.obj) : warning LNK4217: locally defined symbol getenv imported in f
unction aws_common_private_has_avx2 [C:\aws-sdk-cpp-build\aws-cpp-sdk-s3-integration-tests\a
ws-cpp-sdk-s3-integration-tests.vcxproj]
48 Warning(s)
0 Error(s)
When I compile my project, I have the same warnings above which leads me to believe that the dependency builds are ignoring the -DBUILD_SHARED_LIBS=OFF which was specified. I might be wrong, but this is what my research on the LNK4217 has pointed me towards. Has anyone else had this problem?

Following further investigation, the warnings only appear if aws-sdk-cpp is built with the following cmake flag:
-DFORCE_SHARED_CRT=OFF
My project is building static linked, and therefore requires these flags (/MT instead of /MD).

Thanks @singku
I see the fix in aws-c-common. Any chance aws-sdk-cpp can be updated to checkout a newer commit/tag which contains the fix?

Will update the commit ASAP. Thanks for the reminder.

Does it mean that's "normal" until now to get the linker errors when doing a release build?


GUI interface needs a "deselect all" option

when checking lots of checkboxes, it is easier to have an option to deselect all semiologies rather than individually ensuring all are deselected before querying a new patient

Makes sense. Why a new SemiologyVisualisation label? We already had slicer.


Update Link in Tutorials Tab

The Three Essential Habits for the New Player link goes to the old wiki this is the new location on the new wiki

This is not the issue of the client. The client gets the tutorials data from the API and displays it.


version bump

Coverage remained the same at 98.101% when pulling 1941294f5a4043ea1138feb99b7ec6a688f0f955 on 3.4.0 into 516260e0bbbcf42cfe53778be61a5acb77deb8fa on master.


unmanaged-cluster kind provider should consider not using kind- prefix for name and context

Feature Request
As a followup to https://github.com/vmware-tanzu/community-edition/issues/3382, we should consider renaming clusters created via the kind provider to not have the kind- prefix.
For example, currently, when creating a unmanaged-cluster:
❯ tanzu unmanaged-cluster create my-cluster

the name and context are named as kind-my-cluster and kind-my-cluster:
❯ k config view --minify
apiVersion: v1
clusters:
- cluster:
    certificate-authority-data: DATA+OMITTED
    server: https://<IP_ADDRESS>:41221
  name: kind-my-cluster
contexts:
- context:
    cluster: kind-my-cluster
    user: kind-my-cluster
  name: kind-my-cluster
current-context: kind-my-cluster

In order to change this behavior, it would involve mutating the kubeconfig to correctly rename the cluster and it's context. This might not be as simple as
%s/kind-my-cluster/my-cluster/g

since it's possible there are other clusters named my-cluster.
From a conversation in sig-kind (https://github.com/kubernetes-sigs/kind/issues/2668) the standard is usually to name clusters / contexts with the name of the provider first. So for kind it's kind-. For gke, it's gke-. Etc.
One thing we may want to consider is prefixing with a different discoverable name like unmanaged- or tanzu-unmanaged-.
Describe alternatives you've considered
We could also document this well enough so it's discoverable by users. Having the kind- prefix lets the user know which provider had bootstrapped their cluster and enables users to have clusters of varying providers with the same "name".

Adding things to a user specified name without telling them is a bad user experience. If I supply a name then I expect that name - not some derivative name. While documentation would be an improvement to the current situation, this feels more like a bug to me.


Add build testing CI workflow

I think that CI testing helps your development.
CI tells you build failing if code was broken.
https://github.com/jiro4989/nodesnim/actions
And calculator example could not be compiled.

Thank you for reporting a problem with the calculator.


unable to generate certs for second proxy

I'm trying to run the jwilder/nginx-proxy twice, each with it's own dedicated IP.
For 'reasons' I cannot have the domains of proxy1 in the certificate of proxy2.
I modified the jwilder/nginx-proxy to look for VIRTUAL_HOSTS instead of VIRTUAL_HOST and run it as proxy2.
This set-up seems to work without issue.
The docker run for proxy1 looks as follows:

docker run -d -p ip.proxy.1:80:80 -p ip.proxy.1:443:443 --network isolated --name proxy
-v /home/user/conf/htpasswd:/etc/nginx/htpasswd
-v /home/user/certs:/etc/nginx/certs:ro -v /etc/nginx/vhost.d
-v /home/user/conf/proxy1/cust.conf:/etc/nginx/conf.d/my_proxy.conf:ro
-v /usr/share/nginx/html -v /var/run/docker.sock:/tmp/docker.sock:ro
-v /home/user/logs/proxy/:/var/log/nginx/:rw jwilder/nginx-proxy

the docker run for proxy2 looks as follows:

docker run -d -p proxy.ip.2:80:80 -p proxy.ip.2:443:443 --network isolated --name proxy2
-v /home/user/certs:/etc/nginx/certs:ro -v /etc/nginx/vhost.d
-v /usr/share/nginx/html -v /var/run/docker.sock:/tmp/docker.sock:ro
-v /home/user/logs/proxy2/:/var/log/nginx/:rw proxy2

The docker run for the letsencrypt companion looks as follows:

docker run -d
--name ssl
-v /home/user/certs:/etc/nginx/certs:rw
--network isolated
--volumes-from proxy
--volumes-from proxy2
-v /var/run/docker.sock:/var/run/docker.sock:ro
jrcs/letsencrypt-nginx-proxy-companion

I notice the 2nd proxy passes the .well-known request to the upstream.
In the upstream logs I get:

<IP_ADDRESS> - - [15/Feb/2017:10:12:30 +0000] "GET /.well-known/acme-challenge/Umzbul_challenge_Mzb5v8E0Uk-ydRc HTTP/1.1" 404 169 "-" "Mozilla/5.0 (compatible; Let's Encrypt validation server; +https://www.letsencrypt.org)" "letsencrypt.ip"

in the proxy2 logs I get:

example.com letsencrypt.ip - - [15/Feb/2017:10:12:30 +0000] "GET /.well-known/acme-challenge/Umzbul_challenge_Mzb5v8E0Uk-ydRc HTTP/1.1" 404 169 "-" "Mozilla/5.0 (compatible; Let's Encrypt validation server; +https://www.letsencrypt.org)"

in the letsencrypt companion I get:

2017-02-15 10:12:30,075:ERROR:acme.challenges:256: Unable to reach http://example.com/.well-known/acme-challenge/Umzbul_challenge_Mzb5v8E0Uk-ydRc: HTTPConnectionPool(host='example.com', port=80): Max retries exceeded with url: /.well-known/acme-challenge/Umzbul_challenge_Mzb5v8E0Uk-ydRc (Caused by NewConnectionError('<requests.packages.urllib3.connection.HTTPConnection object at 0x7f1d056d9250>: Failed to establish a new connection: [Errno 113] Host is unreachable',))
2017-02-15 10:12:30,075:WARNING:simp_le:1303: example.com was not successfully self-verified. CA is likely to fail as well!

Is the letsencrypt companion able to serve 2 nginx-proxies or should I run it twice with some modifications?

Do you have any solution yet?

Duplicate of #242 so closing to have only one issue tracking this.


String escape doesn't work

Issue Description
If I run print(arrow.utcnow().format('YYYY-MM-DDTHH:mm:ss[h]')), I expect 2019-12-10T00:56:51h to be returned, but I'm getting 2019-12-10T00:56:51[12]
System Info

🖥 macOS 10.13.6
🐍 Python 3.7.4
🏹 0.14.4

Hi @torioLuz, support for this feature was added in Arrow v0.15.3:
env ❯ python3
Python 3.7.3 (v3.7.3:ef4ec6ed12, Mar 25 2019, 16:52:21)
[Clang 6.0 (clang-600.0.57)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import arrow
>>> arrow.__version__
'0.15.4'
>>> arrow.utcnow().format('YYYY-MM-DDTHH:mm:ss[h]')
2019-12-10T01:25:56h

Please upgrade Arrow to access these features :).


Route with controller not found for url 'http://site.com/api/products/1' for Nunit/Resharper test runner

Hi there,
I have a test (nunit) that does the following
        var expectation = new { controller = "Products", id = "1" };
        RouteAssert.HasApiRoute(config, "/api/products/1", HttpMethod.Get, expectation);

Now the crazy thing is that I have a test runner called Ncrunch (helps with TDD) that runs tests in the backgroudn and the tests are passing! But teamcity and resharper both throw errors
Here is what it throws. I know the routes are valid. Is anyone getting the same issue ?
MvcRouteTester.Assertions.AssertionException : Route with controller not found for url 'http://site.com/api/products/1'
at MvcRouteTester.Assertions.ExceptionAssertEngine.Fail(String message) in c:\Code\MvcRouteTester\src\MvcRouteTester\Assertions\ExceptionAssertEngine.cs: line 9
at MvcRouteTester.Assertions.Asserts.Fail(String message) in c:\Code\MvcRouteTester\src\MvcRouteTester\Assertions\Asserts.cs: line 14
at MvcRouteTester.ApiRoute.Generator.CheckValid(String url, HttpMethod httpMethod) in c:\Code\MvcRouteTester\src\MvcRouteTester\ApiRoute\Generator.cs: line 91
at MvcRouteTester.ApiRoute.Generator.ReadRequestProperties(String url, HttpMethod httpMethod) in c:\Code\MvcRouteTester\src\MvcRouteTester\ApiRoute\Generator.cs: line 43
at MvcRouteTester.ApiRoute.ApiRouteAssert.ReadRequestProperties(HttpConfiguration config, String url, HttpMethod httpMethod, String body) in c:\Code\MvcRouteTester\src\MvcRouteTester\ApiRoute\ApiRouteAssert.cs: line 98
at MvcRouteTester.ApiRoute.ApiRouteAssert.HasRoute(HttpConfiguration config, String url, HttpMethod httpMethod, String body, IDictionary2 expectedProps) in c:\Code\MvcRouteTester\src\MvcRouteTester\ApiRoute\ApiRouteAssert.cs: line 29    at MvcRouteTester.RouteAssert.HasApiRoute(HttpConfiguration config, String url, HttpMethod httpMethod, IDictionary2 expectedProps) in c:\Code\MvcRouteTester\src\MvcRouteTester\RouteAssert.cs: line 186
at MvcRouteTester.RouteAssert.HasApiRoute(HttpConfiguration config, String url, HttpMethod httpMethod, Object expectations) in c:\Code\MvcRouteTester\src\MvcRouteTester\RouteAssert.cs: line 162
at MyApi.Api.Search.Tests.App_Start.WebApiConfigTests.TestMe() in WebApiConfigTests.cs: line 39

The same problem on version 2.0.0


node 9.11.0 support

when does it support the newer versions for node 9,10 ?

+1

One of the challenges with this package is that a modified version if the nodejs is embedded within it.
What we really need is very clean, easy to follow, instructions for applying a different (newer, older, whatever) version of nodejs. Maybe even a script. There is no way this project can keep up with the rate nodejs increments any other way.

Hi all, I have instead packaged NodeJS 8.11.3 as well as 10.6.0 based on this master branch (1.6.0). You may download it via:
https://github.com/slee047/node-packer/releases
To use NodeJS 8, you may download via the master branch on the link above.
To use NodeJS10, you may download via the Series10 branch on the link above too.


Disposing EventPipeEventSource does not exit Process

I have a simple application where I listen to GC events:
var provider = new EventPipeProvider("Microsoft-Windows-DotNETRuntime", EventLevel.Informational, (long)ClrTraceEventParser.Keywords.GC);

var client = new DiagnosticsClient(pid);
var session = client.StartEventPipeSession(provider);
var source = new EventPipeEventSource(session.EventStream);

source.NeedLoadedDotNetRuntimes();
source.AddCallbackOnProcessStart(process =>
{
    process.AddCallbackOnDotNetRuntimeLoad(runtime =>
    {
        runtime.GCEnd += (p, gc) => GCEnd(p, gc);
    });
});

source.Process();

When disposing the source, the thread remains stuck in source.Process:
>	System.IO.Pipes.dll!Interop.Kernel32.ReadFile(System.Runtime.InteropServices.SafeHandle handle, byte* bytes, int numBytesToRead, out int numBytesRead, nint mustBeZero) Line 581	C#
 	System.IO.Pipes.dll!System.IO.Pipes.PipeStream.ReadCore(System.Span<byte> buffer) Line 288	C#
 	System.IO.Pipes.dll!System.IO.Pipes.PipeStream.Read(byte[] buffer, int offset, int count) Line 35	C#
 	Microsoft.Diagnostics.FastSerialization.dll!FastSerialization.IOStreamStreamReader.Fill(int minimum) Line 171	C#
 	Microsoft.Diagnostics.FastSerialization.dll!FastSerialization.MemoryStreamReader.ReadByte() Line 67	C#
 	Microsoft.Diagnostics.FastSerialization.dll!FastSerialization.Deserializer.ReadTag() Line 734	C#
 	Microsoft.Diagnostics.FastSerialization.dll!FastSerialization.Deserializer.ReadObject() Line 215	C#
 	Microsoft.Diagnostics.Tracing.TraceEvent.dll!Microsoft.Diagnostics.Tracing.EventPipeEventSource.Process() Line 153	C#

I assume it will be unblocked after the next event is pushed, but because I'm only listening to GC it can take an arbitrarily long amount of time.
I think the FastSerializer should take a cancellation token or something?

Never mind, I misread the documentation of source.StopProcessing, which suggests to dispose the session first. That fixed it.


outdated weight loading for ResNet50

maybe that is the main cause, since with hyper-parameter tunning, we could not get better results than 70%, maybe the weight will solve the problem! @agisga

44b8cd317ad511b38dbf94046129329ee9405317


Performance producer [help needed]

Hi,
I have a performance issue - maybe I'm using the wrong settings.
Hope, you have any suggestions for me.
The results of the producer are very low - compared to the kafka benchmark.
Both have been executed via a separate docker container directly on the docker host using the same kafka broker and should be therefore comparable.
The results of the producer (with kafkajs):
kafka-performance    | Topic created: performance-3bddd504-9a03-40ad-934b-b56662c3f3b2 Partitions: 10
kafka-performance    | Producer connected to kafka brokers
kafka-performance    | { produced: 10000, 'time (ms)': 7909 }
kafka-performance    | { produced: 10000, 'time (ms)': 4800 }
kafka-performance    | { produced: 10000, 'time (ms)': 4693 }
kafka-performance    | { produced: 10000, 'time (ms)': 4599 }
kafka-performance    | { produced: 10000, 'time (ms)': 5873 }
kafka-performance    | { produced: 10000, 'time (ms)': 5810 }
kafka-performance    | { produced: 10000, 'time (ms)': 4620 }
kafka-performance    | { produced: 10000, 'time (ms)': 4575 }
kafka-performance    | { produced: 10000, 'time (ms)': 4610 }
kafka-performance    | { produced: 10000, 'time (ms)': 4546 }
kafka-performance    | { 'totaly produced': 100000, 'time (ms)': 52037 }
kafka-performance    | Producer disconnected
kafka-performance exited with code 0

Compared to the benchmark with /bin/kafka-producer-perf-test.sh --num-records 3000000 --record-size 1024 --topic test_performance --throughput 100000 --producer-props bootstrap.servers=<IP_ADDRESS>:9092:
kafkaPerformance | 123271 records sent, 24654.2 records/sec (24.08 MB/sec), 966.4 ms avg latency, 1248.0 ms max latency.
kafkaPerformance | 158460 records sent, 31685.7 records/sec (30.94 MB/sec), 1004.5 ms avg latency, 1254.0 ms max latency.
kafkaPerformance | 255840 records sent, 51168.0 records/sec (49.97 MB/sec), 607.5 ms avg latency, 837.0 ms max latency.
kafkaPerformance | 255285 records sent, 51057.0 records/sec (49.86 MB/sec), 605.9 ms avg latency, 779.0 ms max latency.
kafkaPerformance | 257160 records sent, 51432.0 records/sec (50.23 MB/sec), 597.7 ms avg latency, 704.0 ms max latency.
kafkaPerformance | 249780 records sent, 49956.0 records/sec (48.79 MB/sec), 615.0 ms avg latency, 687.0 ms max latency.
kafkaPerformance | 250725 records sent, 50145.0 records/sec (48.97 MB/sec), 610.8 ms avg latency, 782.0 ms max latency.
kafkaPerformance | 249435 records sent, 49887.0 records/sec (48.72 MB/sec), 616.6 ms avg latency, 737.0 ms max latency.
kafkaPerformance | 260820 records sent, 52164.0 records/sec (50.94 MB/sec), 590.5 ms avg latency, 679.0 ms max latency.
kafkaPerformance | 257595 records sent, 51519.0 records/sec (50.31 MB/sec), 596.9 ms avg latency, 715.0 ms max latency.
kafkaPerformance | 253875 records sent, 50775.0 records/sec (49.58 MB/sec), 604.2 ms avg latency, 765.0 ms max latency.
kafkaPerformance | 255450 records sent, 51090.0 records/sec (49.89 MB/sec), 599.6 ms avg latency, 683.0 ms max latency.
kafkaPerformance | 3000000 records sent, 46625.843151 records/sec (45.53 MB/sec), 649.70 ms avg latency, 1525.00 ms max latency, 597 ms 50th, 1012 ms 95th, 1495 ms 99th, 1522 ms 99.9th.
kafkaPerformance exited with code 0

Here is my code:
const { Kafka } = require("kafkajs");
const { v4: uuid } = require("uuid");

// Create the client with the broker list
const kafka = new Kafka({
    clientId: uuid(),
    brokers: ["<IP_ADDRESS>:9092"],
    ssl: null,      // refer to kafkajs documentation
    sasl: null,     // refer to kafkajs documentation
    connectionTimeout: 1000,
    retry: {
        initialRetryTime: 100,
        retries: 8
    }
});

const topic = "performance-" + uuid();
const numPartitions = 10;
let n = 100000;
let p = 10000;

const createTopic = async function (topic, numPartitions) {
    let admin = await kafka.admin();
    await admin.connect();
    let topics = [{
        topic: topic,
        numPartitions: numPartitions,
        replicationFactor: 1
    }];
    try {
        await admin.createTopics({
            // validateOnly: true,  // default false
            waitForLeaders: false,  // default true
            timeout: 1000,          // default: 1000 (ms)
            topics: topics,
        });
        await admin.disconnect();
        console.log("Topic created: " + topic + " Partitions: " + numPartitions);
    } catch (err) {
        if (err.error === "Topic with this name already exists") return { topic: topic };
        console.err("Failed to create topic " + topic);
        return { topic: topic, error: err };
    }
    return { topic: topic };
};

const send = async function (producer, index) {

    // set message content
    let content = {
        event: "index.created",
        payload: { index: index },
        meta: { test: true },
        version: "1.0.0",
        uid: uuid(),
        timestamp: Date.now()
    };

    // send message
    try {
        await producer.send({
            topic: topic,
            messages: [
                    { value: JSON.stringify(content) }
            ]
        });
        // console.log("Event emitted", { topic: topic, event: content.event, uid: content.uid, timestamp: content.timestamp, version: content.version });
        return { topic: topic, event: content.event, uid: content.uid, timestamp: content.timestamp, version: content.version };
    } catch (err) {
        console.error(`Failed to emit event ${content.event} to topic ${topic}`, { content: content, error: err });
        throw err;
    }

};

const run = async function () {

    await createTopic(topic, numPartitions);

    const producer = await kafka.producer({ allowAutoTopicCreation: true });
    await producer.connect();
    console.log("Producer connected to kafka brokers");

    let ts = Date.now();
    let count = 0;
    for (let i = 0; i < n; i += p) {
        let ts = Date.now();
        let jobs = Array.from(Array(p),(x,index) => index + 1);
        await Promise.all(jobs.map(j => send(producer, j)));
        let te = Date.now();
        console.log({
            "produced": p,
            "time (ms)": te-ts
        });
        count += p;
    }
    let te = Date.now();
    console.log({
        "totaly produced": count,
        "time (ms)": te-ts
    });

    await producer.disconnect();
    console.log("Producer disconnected");

};
run();

KafkaJS doesn't do any batching on its own, and you're awaiting each of the produce calls -- so essentially you send messages one-by-one. The performance test most certainly batches.
You can provide multiple messages for send, see https://kafka.js.org/docs/producing for details.
This should at least be a good start, there's probably some more tunables here.

Ok, thanks!
Yes, this is the answer - the throughput is now comparable.
I packed now 100 messages together:
kafka-performance    | Topic created: performance-af5e2b31-9ea2-4400-8638-520bb98fc520 Partitions: 10
kafka-performance    | Producer connected to kafka brokers
kafka-performance    | { produced: 100000, 'time (ms)': 5018 }
kafka-performance    | { produced: 100000, 'time (ms)': 4218 }
kafka-performance    | { produced: 100000, 'time (ms)': 4061 }
kafka-performance    | { produced: 100000, 'time (ms)': 4069 }
kafka-performance    | { produced: 100000, 'time (ms)': 4129 }
kafka-performance    | { produced: 100000, 'time (ms)': 4134 }
kafka-performance    | { produced: 100000, 'time (ms)': 4071 }
kafka-performance    | { produced: 100000, 'time (ms)': 4169 }
kafka-performance    | { produced: 100000, 'time (ms)': 4073 }
kafka-performance    | { produced: 100000, 'time (ms)': 4072 }
kafka-performance    | { produced: 100000, 'time (ms)': 4148 }
kafka-performance    | { produced: 100000, 'time (ms)': 4063 }
kafka-performance    | { produced: 100000, 'time (ms)': 4176 }
kafka-performance    | { produced: 100000, 'time (ms)': 4083 }
kafka-performance    | { produced: 100000, 'time (ms)': 4089 }
kafka-performance    | { produced: 100000, 'time (ms)': 4128 }
kafka-performance    | { produced: 100000, 'time (ms)': 4108 }
kafka-performance    | { produced: 100000, 'time (ms)': 4113 }
kafka-performance    | { produced: 100000, 'time (ms)': 4072 }
kafka-performance    | { produced: 100000, 'time (ms)': 4083 }
kafka-performance    | { produced: 100000, 'time (ms)': 4153 }
kafka-performance    | { produced: 100000, 'time (ms)': 4095 }
kafka-performance    | { produced: 100000, 'time (ms)': 4137 }
kafka-performance    | { produced: 100000, 'time (ms)': 4112 }
kafka-performance    | { produced: 100000, 'time (ms)': 4053 }
kafka-performance    | { produced: 100000, 'time (ms)': 4129 }
kafka-performance    | { produced: 100000, 'time (ms)': 4094 }
kafka-performance    | { produced: 100000, 'time (ms)': 4173 }
kafka-performance    | { produced: 100000, 'time (ms)': 4066 }
kafka-performance    | { produced: 100000, 'time (ms)': 4109 }
kafka-performance    | { 'totaly produced': 3000000, 'time (ms)': 124204 }
kafka-performance    | Producer disconnected
kafka-performance exited with code 0


JWT Authentication Backend

Create a robust authentication flow using jwt.
Need to create User Model with name(required), email(normal email validation),mobile no (10 digit number),address(required), password(more than 6 characters). Controller methods for signing in user, signing up user. Models should be placed in model folder and controller methods in authenticationController.js file in controller folder.

I would to work on this, I am an swoc participant

@07souravkunda please unassign this issue as i already got another issue assigned to me in another project

ok

I would like to work on this.

Assigned to @sanjay270899


Feature/issue 651- Remote Control - Seat

Fixes #651
This PR is ready for review.
Risk
This PR makes minor API changes.
Testing Plan
Added new unit tests and expanded existing ones
Summary
Added RPC's, structs, enums as defined in the Remote Control - Seat proposal.
CLA

[x] I have signed the CLA

Codecov Report

Merging #711 into develop will increase coverage by 0.58%.
The diff coverage is 100%.

@@              Coverage Diff              @@
##             develop     #711      +/-   ##
=============================================
+ Coverage      41.38%   41.97%   +0.58%
- Complexity      2656     2758     +102
=============================================
  Files            362      372      +10
  Lines          15835    16009     +174
  Branches        1685     1685
=============================================
+ Hits            6554     6720     +166
- Misses          8973     8983      +10
+ Partials         308      306       -2

Impacted Files
Coverage Δ
Complexity Δ

...om/smartdevicelink/proxy/rpc/SeatMemoryAction.java
100% <100%> (ø)
8 <8> (?)

...martdevicelink/proxy/rpc/enums/MassageCushion.java
100% <100%> (ø)
2 <2> (?)

...om/smartdevicelink/proxy/rpc/enums/ModuleType.java
100% <100%> (ø)
2 <0> (ø)
:arrow_down:

...com/smartdevicelink/proxy/rpc/SeatControlData.java
100% <100%> (ø)
34 <34> (?)

...evicelink/proxy/rpc/RemoteControlCapabilities.java
100% <100%> (ø)
10 <2> (+2)
:arrow_up:

...java/com/smartdevicelink/proxy/rpc/ModuleData.java
100% <100%> (ø)
10 <2> (+2)
:arrow_up:

...com/smartdevicelink/proxy/rpc/MassageModeData.java
100% <100%> (ø)
6 <6> (?)

...vicelink/proxy/rpc/enums/SeatMemoryActionType.java
100% <100%> (ø)
2 <2> (?)

...tdevicelink/proxy/rpc/SeatControlCapabilities.java
100% <100%> (ø)
34 <34> (?)

...smartdevicelink/proxy/rpc/enums/SupportedSeat.java
100% <100%> (ø)
2 <2> (?)

... and 14 more

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 0bf0067...0662190. Read the comment docs.

@mrapitis PR looks good. Though, there are some best practices that we started to follow recently for RPCs:

We started adding a constructor that has all mandatory params as described in issue 154.
We started adding @Nonnull annotation for all mandatory params as described in proposal 0166

Also there are some minor spacing issues here and there.

@mrapitis looks very good. Thanks.

@mrapitis There are also still some extra spaces/tabs and some extra lines in different files in the PR.

@bilal-alsharifi @joeygrover we have updated spacing and added more documentation.  Please re-review when time permits.  Thanks.


Update member API not working

crowd.dev edition
Cloud (hosted)
Version
No response
What happened?
When trying to update member's organization with API (PUT method)- https://docs.crowd.dev/reference/put_tenant-tenantid-member-id it is not working.
It is giving error as Invalid value { name: XX}

Command / Request
No response
Relevant log output
No response

Thanks for letting us know @RaginiAngarka!
We will take a look at this.

Hey @RaginiAngarka
Thanks for raising this!
Our apologies; there was an error in the docs. I just fixed it.
The update method only takes the IDs of the relations. You have to send the organization's ID there. Otherwise, you can use the upsert method for sending the full organization.


Publishing package to hex.pm

I made some minor changes so I could publish this package on hex.pm.  I'm using it as a dependency in my jose project (JSON Object Signing and Encryption).
Edit: Forgot to mention: thank you for the excellent library! :smile:

Merged. Thank you


Modern slavery statement

https://trello.com/c/Yda37Ncb/2570-add-modern-slavery-statement-to-list-of-corporate-information-for-organisations-whitehall-publisher-3

It looks like the changes to the locale files will need rebasing against main.
Also some of the translations appear to have been removed in the second commit (e.g. speaking_notes --> other: Çıxış üçün qeydləri in Azerbaijani).

So I'm running the task specified here and that seems to mess up all the translations

@brucebolt I've manually fixed all translations so it should be ready for review again


How to get project id?

This could be a silly question, and I apologise in advance if this is covered elsewhere, but I can't immediately find a way to get what I need...
I want to be able to get all the Milestones for a project that I have on gitlab.com.  The project in question is https://gitlab.com/gep13/fakeproject.  In order to get the milestones, I have to provide the project id, and I don't see a way to get this information easily.  Can you please provide an example to explain what I am missing?
Thanks!

simply use gep13/fakeproject or look here:

@casz perfect, thank you very much!


Better curl error messages for ^6.0

@Spomky Can you make a ^6.0 release with the changes from https://github.com/Spomky-Labs/jose/pull/314? I am currently running php 5.6.
Thanks.

Hi @fkupper,
Sorry for the late answer, I am very busy this month.
I will take care of it.

Hi @Spomky,
Never mind, this is not a killing issue. Do it when you can.
Thanks for the awesome library.

You are welcome.
It is now done. Tagged v6.1.7.
Please note that this library will be marked as deprecated by the end of the year.
It will only receive security fixes up to end of 2019.
When your application will be able to run on PHP 7.x, you should really consider to use the new JWT Framework Project instead of this library.

Thanks for the warning, I am aware of the lifespan.


Add Kotlin/JS installation instructions to README

Resolves #7

As per #9, this PR is now incorrect until a new host is established.

Please see #10

PR is now outdated by subsequent updates.


docs: improve docs with ice cream machine instead of ice machines

Resolve very important distinction of "ice machines" vs "ice cream" machines.
Ensure users are aware of the key infrastructure difference between ice-cream and ice machines.
Repository owner note: Please update repo description in GitHub to align since it also incorrectly says "ice machines".

Thank you for this provider and the critical part it will play in many enterprise deployments. 😀

Can't believe I missed this 🤦 .
Thanks @sheldonhull!


Updated downloads poser provider

Hey,
I provided a fix for the "Downloads" Poser provider. IntelliJ API now provides a general JSON endpoint, which contains an integer of the "downloadsCount".
Best, Christian

👍


Int to number node doesn't work

See: https://github.com/comfyanonymous/ComfyUI/issues/368#issuecomment-1498003825
Maybe voice your opinion on how this is considered a bug to randomly not allow certain types as input only.

These are removed until the issue is resolved.

These are removed until the issue is resolved.

This is a pity, because I was hoping to build a workflow that could read the size of incoming images and match them with corresponding empty latents for image generation and other operations.

These are removed until the issue is resolved.

This is a pity, because I was hoping to build a workflow that could read the size of incoming images and match them with corresponding empty latents for image generation and other operations.

Well there is no node to get the width/height of a image and pass it along, but the rest of the functionality is already there if it were, such as Number to Int to then plug into a EmptyLatent.

I added a Image Size to Number node in 2ff0c04a656d1d69f8eb24d76159dfd8cbf59a04

I added a Image Size to Number node in 2ff0c04
I'll add a latent one later.

That's really helpful for my workflow! I appreciate your efforts and I'm looking forward to seeing the latent one as well.


Use CDN alternative instead of GitHub API for docs

Palindrom docs are a static site now 🎉 https://palindrom.github.io/docs/


Handling Nulls (and possibly invalid values, too)

Is your feature request related to a problem? Please describe.
This is a issue that's dear to me, probably because I work with Access & SQL Server and am always annoyed that I don't have a proper nullable type system to work with and find it clumsy to work with nulls. Variant is the only data type capable of storing a Null value but then it goes on to muddy the water with storing any other data types, which is contrary to how database engines expect their data to be. Therefore, this code is dangerous:
Dim SomeDate As Date

SomeDate = Me.SomeBoundDateControl.Value

The SomeBoundControl is surely bound to a column that can only contain a date. Yet that same column could potentially contain a Null. The Value property of an Access control is a Variant data type. Thus I could do:
Me.SomeBoundDateControl.Value = "Feb 2 1980"
Me.SomeBoundDateControl.Value = 1 'Oh, hi, 1899 called!
Me.SomeBoundDateControl.Value = ""
Me.SomeBoundDateControl.Value = vbEmpty 'That is not the null you're looking for

What utter nonsense! And this also can be a potential runtime error or worse will just coerce to some valid value which is almost never what one wants.
Within Access, we can guard against nulls using Nz() function or IsNull() function:
Dim SomeDate As Date

SomeDate = Nz(Me.SomeBoundDateControl.Value, 0)

But because it's Variant, this becomes woefully inadequate:
Dim SomeDate As Date

SomeDate = Nz(Me.SomeUNBoundControlContainingUserInput.Value, 0

Nothing's stopping the user from typing in an empty string or nonsense like "Feb 31 2021". Therefore, it's common to guard against invalid inputs by something like the following:
For Date data:
If IsDate(...) Then

For String data:
If Len(... & vbNullString) Then

For  any integer data (may be too strict):
If Not ... Like "*[!0-9]*" Then

Or (may be too loose)
If IsNumeric(...) Then

All those are horrid. Why must we go through the mental gymnastic of checking that a Variant is a proper data type and otherwise convert it into the desired data type? The default behavior of raising an error is also horrid because now we have error-as-the-control which is also a bad coding pattern to have.
Describe the solution you'd like
We need a way of properly constraining the allowable values. As an example, I should be able to do this:
Dim SomeDate As Date?

SomeDate = Me.SomeBoundControl.Value

If IsNull(SomeDate) Then
  'No valid date was entered
Else
  'We have a valid date
End if

The Date? would indicate that it can contain any valid Date value OR Null. Not strings. Not integers. Not floats. Not some hairball from your crazy aunt Bea's even crazier cat.
I'm not sure how this could be implemented. If we want a full compatibility with VB* clients, the tB compiler could emit additional instructions to do a safe coercion of the invalid value into Null so that the Variant will only be of either VT_NULL or VT_DATE in this situation. That results in less cluttered code.
However, that may end up doing too much and possibly violating the principle of least astonishment. Alternatively the tB compiler could emit instruction to raise an error if it cannot fit either VT_NULL or VT_DATE so there's no surprises when garbage input is given and Null is gotten back.
Yet another approach is to treat this as a compile-time inspection only. There would be nothing special  for the VB* consumers but for tB code, it would be easy to find suspicious casts and then mark them to indicate that they should only contain a certain data type or Null.
Also, there is a complementary approach to borrow from .NET (mainly C#). We can do this instead:
SomeDate = Forms("SomeForm")?.Controls("SomeControl)")?.Value

The ?. operator indicates that if the object is null, the Value should return null as well, which helps to prevent runtime error when the form SomeForm or the control SomeControl are not open or otherwise available for consumption, thus preventing a runtime error. That eliminates the need for an On Error Resume Next + On Error GoTo 0 block around the dangerous access, which again helps with the code readability.
And there's also this operator:
SomeDate = Me.SomeBoundControl.Value ?? Date()

which indicates that if it's Null (or maybe invalid value, too?), Date() is provided as the default. This is like Access' Nz() function but that wouldn't require everyone to reference the Access object library just to have that functionality.
Those 2 operators can be used in conjunction with the nullable types which would then immensely simplify the process in handling values without having to get into ugly mechanics of handling each data type differently.

I've been thinking some more about this. There are many ways of representing an absence of something in VBx, many of them relate to Variants:

Empty: A Variant that hasn't been assigned yet
Missing: An optional parameter that hasn't been supplied. In VBx, optional parameters can only be Variants (although see #15).
Null: A Variant holding a null value from a database query
Nothing: a null reference. Only applies to objects, or Variants holding an object.
Blank: (Listed for completeness): An empty Excel cell.

One massive headache here is when it comes to method invocation. If you accept a Variant that you're expecting (but not sure) contains an object, you have two choices:

Try and call the method you're expecting to be there, and handle any error if the Variant contains something else (or nothing at all)
Test for every possible nothingness condition before making the call

Neither are optimal from a boilerplate or performance point of view.
To be clear, this is a problem in many languages. Tony Hoare, the inventor of the null reference, called it his billion dollar mistake (I would argue that, by now and adjusted for inflation, it's many trillions of dollars in 2023).
Describe the solution you'd like
Everything @bclothier noted in the OP. Particularly, a set of robust null operators, of which I'd argue the most important is ?.. This would mean: "Do I have an object reference? If so, continue to evaluate this line of code. Otherwise, bail out".
As he stated, that would mean that it's safe to write this:
SomeDate = Forms("SomeForm")?.Controls("SomeControl)")?.Value
without checking two different points for failures, or accepting and handling the failure as an error.

Note, this doesn't protect from a situation where the wrong type of object is supplied. But in practice, that's going to be far less likely, and thus fine to treat as a runtime error. As a C# developer, I see Variant as roughly equivalent to dynamic in C#.


Exclude hidden fields for collection_check_boxes in developer_query_component.html

Fixes #289

Pull request checklist

[x] Your code contains tests relevant for code you modified - No tests required
[x] You have linted and tested the project with bin/check - Please see the attached screenshot.

Looks great! Thanks for the contribution.


Does it support 'query' command?

Thanks for the DynamoDb-GUI-Client, I can work on dynamodb in GUI tool. I have some questions:

I saw 'Scan' in the tab 'Records', does it equal to 'scan' API in DDB? Does the DynamoDb-GUI-Client support 'query'?
Only one filter condition in the DynamoDb-GUI-Client? Can I have more filter conditions?
Scan or Query only on Table? Can I  Scan or Query on GSI?
All the questions refer to AWS console.
Thanks.

For now, it only supports 'Scan' and 'Yes' it is done with DynamoDb 'scan' method. I have already planned to add GSI and 'query' support in the future. Thank you.