PowerShell 7 is unable to create a junction on relative paths while PS 5 can

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
Try to create a junction with powershell 7:
PS E:\> mkdir 'Program Files (x86)\Steam'
PS E:\> New-Item -ItemType Junction -Path 'C:\Program Files (x86)\Steam' -Target 'Program Files (x86)\Steam\'

Expected behavior
Create the symlink in PS7 like in PS 5.

Actual behavior
Gives error:

New-Item: Could not find item Program Files (x86)\Steam\.

Error details
No response
Environment data
This version does not create the symlink:
$PSVersionTable

Name                           Value
----                           -----
PSVersion                      7.1.3
PSEdition                      Core
GitCommitId                    7.1.3
OS                             Microsoft Windows 10.0.19043
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

This version DOES work:
$PSVersionTable

Name                           Value
----                           -----
PSVersion                      5.1.19041.1023
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.19041.1023
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

### Visuals

_No response_

I think you confused the Path with the Target

@237dmitry no I didn't, because the folder C:\Program Files (x86)\Steam was moved to another drive.
Also, using Value instead of Target doesn't work:
PS E:\> mkdir 'Program Files (x86)\Steam'
PS E:\> New-Item -ItemType Junction -Path 'C:\Program Files (x86)\Steam' -Value 'Program Files (x86)\Steam\'

Which is recommended here: https://github.com/PowerShell/PowerShell/issues/6232

Finally found the problem. PowerShell 7 can't work with relative paths when creating junctions on directories. However PS5 can. So to fix it, you must provide always a full path. So with a full path works with PS 5 and PS 7, and you can use Target or Value both work.
Either way it works now. Remember to run powershell as admin which can be required:
New-Item -ItemType Junction -Path 'C:\Program Files (x86)\Steam' -Value  (Get-Item 'Program Files (x86)\Steam\').FullName
New-Item -ItemType Junction -Path 'C:\Program Files (x86)\Steam' -Target (Get-Item 'Program Files (x86)\Steam\').FullName

I think you are running into an issue where relative symlink paths are relative to the process's (not the PS file system provider's) current working dir.  See https://github.com/PowerShell/PowerShell/issues/15235#issuecomment-859286180

Yeah but something changed too much if works with PS5 and not with PS7. This is kind of incompatibility thing which shouldn't happen I guess...

This is kind of incompatibility thing which shouldn't happen

Agreed but unfortunately regressions happen, and often when someone has the good intention of fixing some bug or implementing an enhancement request.

Yeah the idea if that's a regression or done on purpose.


Does it work with vanilla Spring 4/5

I did not see Jasypt have official support for Spring 4/5, but when I tried to include @EnableEncryptableProperties and @EncryptablePropertySource, it does not work as expected.

It should work, @EnableEncryptableProperties imports some configuration that registers a BeanFactoryPostProcessor to decorate property sources present in the Environment.
@EncryptableProperty source is simple, it just enables one specific property source with jasypt capabilities.
Do you have (or can create) a test app in Github where the problem manifests?

@ulisesbocchio Thanks for the reply.
I will upload a sample later.
But I saw the spring-boot package get used in the EnableEncryptablePropertySourcesPostProcessor.java, could it be a problem?

Here is ref repo:
https://github.com/chenrui333/jyspert-spring-sample
The command line to run is ./gradlew appRun

jasypt-spring-boot does not work with vanilla Spring as properties source loading is different,
Spring Boot used PropertySourceLoader, while vanilla Spring  used PropertySourcesPlaceholderConfigurer

Alright, let me take a look at your project...sorry for the delay

Thanks.

Any update?

I'll get back to you today... sorry for the delay

You're missing this in your TestConfig:
@Bean
    PropertySourcesPlaceholderConfigurer propertySourcesPlaceholderConfigurer() {
        return new PropertySourcesPlaceholderConfigurer();
    }

That's mandatory if you want to enable PropertySources in general. Basically, without that bean, not even  Spring's @PropertySource annotation works. In Spring Boot, there's an auto configuration for it in org.springframework.boot.autoconfigure.PropertyPlaceholderAutoConfiguration

Let me re-try it.

Another quick question is that whether I can use gradle to setup system property that it can be picked up by Spring configuration.

you gotta pass the system property to tomcat or whatever servlet container you're using when you run your application if you use it out of the box. Otherwise you can define your own SpringEncryptor bean and retrieve the password from wherever you want.


#101 - Completed.

@rultor merge

@rultor merge

@driver733 OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@driver733 Done! FYI, the full log is here (took me 2min)

Coverage increased (+0.4%) to 69.47% when pulling fc0e662779610868afcacb8cdde29e37fd27c511 on #101 into cd0a420ad610fc97b2832064f97a798c34a324f9 on master.


Indexation of user profiles by search engines

From user feedback:

When you google for "user name" one of the top results is his list of payments. This is a really weird thing for people looking him up in advance

It's understandable that some users may not want to have their Open Collective profiles popping in their Google results, especially because of the list of transactions where you can see the amounts that a person received through expenses.
I see two ways to resolve this situation:

Either we decide that all user profiles should be private and we add NOINDEX on all user's pages to ask Google to forget them
Or we make that opt-in/opt-out by adding a section in the settings to change the "Profile visibility", with two options: Public (current behavior) or Unlisted (won't appear in search engines)

Making the user profiles is not on the table at the moment, as doing so would go against transparency.
Context
https://opencollective.slack.com/archives/G01E5EWDH40/p1616079967001000

I think it's ok to have users profiles with noindex -- many users are surprised their info shows up in google.
unless there's an argument for keeping them public, I would err on the side of protecting them

We can start with that, and implement (2) (opt-in) if people ask to have their profiles public

I agree - be default noindex for personal profiles. But what about corporate sponsors like Airbnb or something, who are using us because they want that recognition and exposure?

I agree - be default noindex for personal profiles. But what about corporate sponsors like Airbnb or something, who are using us because they want that recognition and exposure?

If we apply that only to user profiles, sponsors like Airbnb won't be impacted (as long as they're registered as an organization).

I don't see any Individual complaining that their profile is not indexed. We could always have a setting for this if needed.
However, I see it as yet another step for less openness/transparency. Next time, people will ask us to have their profile or their transactions private by default, be prepared.


Update functions.php

replace duplicate code

Q
A

Is bugfix?
❌

New feature?
❌

Breaks BC?
❌

Fixed issues

👍


Duplicate definitions of React headers when compiling for iOS.

When installing from react-native link, the file AIRMaps.xcodeproj is included in XCode and the installation is the same as Option 3 (https://github.com/airbnb/react-native-maps/blob/master/docs/installation.md)
While it is working for Android, it was impossible to build on iOS because React (0.42) was already set in XCode and would use different headers, leading to multiple duplicate entries for properties and classes included from React.
I had to change the configuration of 'Header Search Paths' (Option 3, Step 3) from $(SRCROOT)/../../react-native/React to $(BUILT_PRODUCTS_DIR)/include/React in order to connect to the correct headers files and make the map work on iOS.

I have the same issue

In Libraries/AirMaps.xcodeproj > Build Settings > Search Paths,
setting 'Always Search User Paths' to "Yes"
and adding "$(SRCROOT)/../../react-native/React" (recursive) to 'User Header Search Paths'
seems fixes this issue even after reusing the command "react-native link" (that would normally overwrite 'Header Search Paths').

Actually it doesn't fix the issue when adding a new module with yarn. So it's still not the preferred way. Anyone else encountered this issue ?

I recently made a fresh example build with pods and GoogleMaps with no issues.
https://github.com/alvelig/react-native-maps-pods-example.
Try to check it out or build through the README.MD. If it does not help, let me know.


[BP-1.19][FLINK-35699][k8s] Fix shading Jackson through fabric8

What is the purpose of the change
The flink-kubernetes artifact shades Jackson classes coming through fabric8, but since Jackson 2.15, Jackson is a multi-release JAR, which requires some additional relocations for correct shading.
What Jackson does itself can be checked here.
Brief change log

Added exclusions for Jackson related meta files, which otherwise overwrites the Flink ones.
Added relocations for JDK-specific code.

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

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

Documentation

Does this pull request introduce a new feature? no

@flinkbot run azure


Add .yarn to .gitignore

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
The .yarn folder got created at the time of running yarn, giving the message "too many changes in Git"
Steps To Reproduce

Open the project in your IDE.
Run yarn
Notice the number of files changed. It is 10K+.

Expected Behavior
.yarn directory should be ignore by Git.
Screenshots

Device Information [optional]
- OS: 22.04
- Browser: Firefox
- version: 110

Additional context
No response
Are you working on this issue?
Yes

@danger-ahead i belive issue #3647 is also for fixing the same issue take a look

I believe this is different. But yeah, similar.

can i get  the opportunity to solve this issue

@danger-ahead @Arnabdaz I want to work on this issue

@danger-ahead Hii , I am new to open-source , I want to work on this issue


Fails to deploy contract with soroban-client to 0.2.1 and sorban 0.3.2

What version are you using?
So, I was testing the new commit https://github.com/stellar/soroban-example-dapp/commit/da61364736f7f5e0e6be8e3bb50891d92173c527 that uses soroban-sdk 0.3.2
I had different errors with different versions of soroban-cli
With soroban 0.2.1, while running ./initialize, I managed to deploy the contract, but not to initialized it.
Initialize the crowdfund contract
error: parsing contract spec: parsing contract spec

Then, I updated to soroban 0.3.3, and I did not even manged to deploy the contract. Got:
Deploy the token contract
error: transaction submission failed

I was looking at the recent changes, and found that in the Docs it is used
docker run --rm -it \
  --platform linux/amd64 \
  -p 8000:8000 \
  --name stellar \
  stellar/quickstart:soroban-dev@sha256:8046391718f8e58b2b88b9c379abda3587bb874689fa09b2ed4871a764ebda27 \
  --standalone \
  --enable-soroban-rpc

Did the change to ./quickstart.sh and now works.
Will open PR

I realized that in fact the project is using soroban 0.3.2, so I moved to that version and I had to fix initialize.sh
Everything in the https://github.com/stellar/soroban-example-dapp/pull/74 PR

So I am still very confused about this:
With the lastest commit https://github.com/stellar/soroban-example-dapp/commit/da61364736f7f5e0e6be8e3bb50891d92173c527 the following packages are used:

rs-soroban-env v0.0.11
rs-soroban-sdk v0.3.2
rs-stellar-xdr v0.0.11
js-soroban-client (npm package: soroban-client) v0.2.1

Following leigh recommendation the recommended setups say that: for this setup

soroban-cli should be 0.3.3
js-soroban-client should be in v0.2.0
stellar quickstart docker should be: stellar/quickstart:soroban-dev@sha256:8046391718f8e58b2b88b9c379abda3587bb874689fa09b2ed4871a764ebda27

So, after installing soroban-cli 0.3.3 I got the following error while ./initialize.sh
When using standalone:
Using standalone network
Fund token admin account from friendbot
{
  "type": "https://stellar.org/horizon-errors/server_error",
  "title": "Internal Server Error",
  "status": 500,
  "detail": "An error occurred while processing this request.  This is usually due to a bug within the server software.  Trying this request again may succeed if the bug is transient. Otherwise, please contact the system administrator."
}Deploy the token contract
error: transaction submission failed
Build the crowdfund contract
cargo fmt --all
cargo build --target wasm32-unknown-unknown --release
    Finished release [optimized] target(s) in 0.06s
cd target/wasm32-unknown-unknown/release/ && \
	for i in *.wasm ; do \
		ls -l "$i"; \
	done
-rwxrwxr-x 2 wachuma wachuma 8371 dic 13 08:15 soroban_crowdfund_contract.wasm
Deploy the crowdfund contract
error: transaction submission failed
Contract deployed succesfully with ID:
Initialize the crowdfund contract
error: transaction simulation failed: Could not unmarshal transaction

What I did is to use that docker version, in this commit  and it did solve the deploy problem...
Deploy the token contract
success
success
Token contract was deployed with TOKEN_ID: 83ef099c3dd37c454ff87b16c20b2f7db6be2467832161b40d7c8fee72850a16
Build the crowdfund contract
cargo fmt --all
cargo build --target wasm32-unknown-unknown --release
    Finished release [optimized] target(s) in 0.06s
cd target/wasm32-unknown-unknown/release/ && \
	for i in *.wasm ; do \
		ls -l "$i"; \
	done
-rwxrwxr-x 2 wachuma wachuma 8371 dic 13 08:15 soroban_crowdfund_contract.wasm
Deploy the crowdfund contract
success
success
Crowdunfund contract was deployed with ID: 6d8a27e42a2f124e2dfc1575ddc52327c5c5d96a9ebb59c90db713c24252917d
Initialize the crowdfund contract
success
null

However, when running the front end:

Apparently I read correctly the information from the blockchain (wallet/hooks/useContractValue.tsx is working correctly.
But there are errors when trying to sign messages (wallet/hooks/useSendTransaction.tsx failing)
I am now debugging, and will write the errors in a next Comment

First error, when Minting:
timeout it not defined in useSendTransaction- fixed
After fixing it, I can say that Minting is working, however, apparently it does only mints  42 EXT??
When "approve and back up this project"
error while handleSubmit:  Unable to send message to extension

Error: XDR Read Error: Unknown ScHashType member for value 32
    at x

I guess that the error comes from trying to send the transaction to the wallet, because when minting it does gives a private jey

My guess is that there is something wronw while building the transaction
Because even if I hardcode the secret key, it's bugging in SorobanClient.Server.simulateTransaction(txn);

There are too many things here.  This issue will onlye refer to "Failing to deploy contract with soroban-client 0.2.1"

What I did is to use that docker version, in this commit  and it did solve the deploy problem...

So I'm opening a pull request for this:
https://github.com/stellar/soroban-example-dapp/pull/77


Make source parameter to StartTraining methods a URI instead of a string

This should take a SAS URI to storage container holding training files.
Also: find a more descriptive name than source
Consider an overload that handles local filepaths for Docker containers.

This should take a SAS URI to storage container holding training files.
Also: find a more descriptive name than source

Addressed in PR: https://github.com/Azure/azure-sdk-for-net/pull/11159

Consider an overload that handles local filepaths for Docker containers.

Created separate issue: https://github.com/Azure/azure-sdk-for-net/issues/11229


Screen markers widgets

This feature allows you to attach screen markers to widgets by pressing shift when creating the screen marker. Also works on items.

Maybe show a translucent hover effect on the widget? this would help the player see what they're selecting

You may have already seen the little checkstyle errors that Travis is reporting, but just in case here they are :)
[INFO] There are 3 errors reported by Checkstyle 8.3 with checkstyle.xml ruleset.
[ERROR] src/main/java/net/runelite/client/plugins/screenmarkers/ScreenMarkerKeyListener.java:[27,8] (imports) UnusedImports: Unused import - java.awt.event.KeyAdapter.
[ERROR] src/main/java/net/runelite/client/plugins/screenmarkers/ScreenMarkerKeyListener.java:[30,8] (imports) UnusedImports: Unused import - net.runelite.api.Client.
[ERROR] src/main/java/net/runelite/client/plugins/screenmarkers/ScreenMarkerKeyListener.java:[31,8] (imports) UnusedImports: Unused import - net.runelite.client.callback.ClientThread.

looks like it's just three unused imports, so in ScreenMarkerKeyListener.java remove these lines:
(Line 27) import java.awt.KeyAdapter;
(Line 30 ) import net.runelite.api.Client;
(Line 31) import net.runelite.client.callback.ClientThread;

Alright fixed those imports and added an indicator to see what you are selecting.


fix: replace include with include_tasks

What this PR does / why we need it:
fix: replace include with include_tasks
ansible.builtin.include has been deprecated and removed from ansible-core. Because of that, release scripts are not working - https://github.com/kubevirt/kubevirt-tekton-tasks/actions/runs/7183095492/job/19561070924
Release note:
NONE

/lgtm

/cherrypick release-v0.15

/retest

/retest

/retest

/retest

/retest

/check-dco


No releases - how to properly set up?

Hi all,
first of all I want to say thanks for theses scripts. I've just recently found borg-cron-helper and am planning to build it into my borg container Therefore I want to automate the installation of borg-cron-helper. The Readme states to use releases and compare them against a PGP key. But there are no releases (yet?).
Are future releases planned or shall I just use git to get the repo?
Either way this is a bug report either for the missing of releases described in the README or for a wrong set up description in the README.

Are future releases planned or shall I just use git to get the repo?

Yes, they are. However, i want to test the thing a bit more and maybe change the one thing or so.
For now you can use the git repo. BTW: You can verify this too as I sign my git commits. Just use git verify-commit:
$ git verify-commit HEAD
gpg: Signature made Thu Sep 15 15:23:54 2016 CEST
gpg:                using RSA key 0x05D40A636AFAB34D
gpg: Good signature from "rugk git (software signing only<EMAIL_ADDRESS>[<trust level here>]
Primary key fingerprint: ABA9 B8F6 F448 B07F D7EA  4A1A 05D4 0A63 6AFA B34D

I'll close this issue, if we have a working release.

done


Add 2023 Numerical Homework with a new RK4 solver

Add 2023 Numerical Homework with a new RK4 solver,  this solver can solve equations just like Modelica.

Add a new solver to solve any type of equations.


Sidelobe subtraction before deconvolution

Sidelobe subtraction before deconvolution did not produce errors, but did produce weird results. The directory where it is located is /nfs/mwa-09/r1/djc/EoR2013/Aug23/fhd_nb_decon_Nov2015_presidelobesubtract/. Below is the 2d power spectra for that test, and below that is the fhd_nb_decon_Nov2015 (normal decon) for comparison purposes.

I believe this is now fixed. The bug was that sources near the edge of the image were aliasing in to the center of the FoV.


DRILL-8316: Convert Druid Storage Plugin to EVF & V2 JSON Reader

DRILL-8316: Convert Druid Storage Plugin to EVF & V2 JSON Reader
Description
Converts the Druid storage plugin to use the V2 JSON reader.
Documentation
No user facing changes.
Testing
Ran existing unit tests.

@Z0ltrix Would you mind reviewing this?   The unit tests pass, but I'm not 100% certain this is handling large data sets correctly.

@jnturton
Could you do a review of this.  I realized that this has been languishing and we might as well merge it if it can be.
The one area which I'm a little hesitant about is the ScanBatchCreator.  Basically, since I didn't write this storage plugin and it was a bit more complicated than some of the ones I've written, I'd like another set of eyes on it.


update to new pybrickschema API

Saw in the new docs that you can spec multiple profiles, so this fixes #209 - not sure that we need all 3, but the output of Brick.ttl is the same as the the Brick.ttl we were getting a few days ago in the nightly build.

Well, so I'm busted for not running the tests before pushing. (generate_brick does the right thing, though!)
I can fix the tests with the new profile or we can set a new default in py-brickschema, let me know

Well, so I'm busted for not running the tests before pushing. (generate_brick does the right thing, though!)
I can fix the tests with the new profile or we can set a new default in py-brickschema, let me know

I'll fix the tests and update the profile, mainly because I feel bad that I broke the Brick generation, but also because there may be a few small warts with the 0.2 release that I need to fix in conjunction with the tests. brickschema==0.2.3a1 should have a "brick" profile that does what we want.

I'll fix the tests and update the profile, mainly because I feel bad that I broke the Brick generation, but also because there may be a few small warts with the 0.2 release that I need to fix in conjunction with the tests. brickschema==0.2.3a1 should have a "brick" profile that does what we want.

I tried to get my changes to build on your branch here, but I messed it up and this PR didn't get merged. Your changes have been reflected (along with a few more fixes), so I'm closing this PR.

I tried to get my changes to build on your branch here, but I messed it up and this PR didn't get merged. Your changes have been reflected (along with a few more fixes), so I'm closing this PR.


fix for stage_nldas bug

@jread-usgs i assume you have changed or will shortly change the sw units to "W m^-2". i'm merging this anyway so i can add my own changes.


Updating Prepare to Dye Plus to 1.3.9

Changelog: ### 1.3.9 (2024-01-15)
@Yarden-zamir these changes contain engineer's code updates only, tested myself and seems to be working

waiting for curseforge to approve 1.3.9

ok @Yarden-zamir this is ready now, contains engineers code, and has been tested by myself


Chore/logto scripts

Adds new lifecycle scripts for logto
Type

[ ] Dependency upgrade
[ ] Bug fix
[ ] New feature
[x] Dev change

@edge33 this doesn't look that much related to logto :D I guess you have some unintended commits

@edge33 this doesn't look that much related to logto :D I guess you have some unintended commits

indeed, I branched off the wrong branch, I fixed the pr


Add fallback ConfigReferenceResolver

Here is a suggestion for how to support fallback reference resolution. I'm interested in whether you would suggest any major changes -- like, is the options the right place to hold the resolver and does it make sense to create the path interface.

One thing I forgot to ask: I've added some "@since x.x.x" but didn't know which version. Is there a version I can use or should I just drop them?

using 1.3.2 probably has a good chance to be correct, thanks

Done.
Related question: is cutting 1.3.2 on the horizon? I ask because I have to decide if I should switch to building the latest from source to get this change or if I can wait a while and then just use the one from maven.

@2m I saw you were triaging some bugs today; do you have time to make a release? the instructions are in CONTRIBUTING.md, though it may be that I'm the only one with the maven central  permissions, we should probably fix that anyway...
It's nice before a release to go through all the issues (as it appears you are) and also see if any PRs are unmerged that should have been merged, but probably a bad idea to block on work that we don't have time for, can always make another release.


python: doc mistake fixed

Brief Description

In the doc for python client there is a broken url here
https://github.com/apache/rocketmq-clients/blob/92c627d55af415a5597ee9ea0efff27c313582b5/python/README.md?plain=1#L9
The problem is that the url is repeated twice.
How Did You Test This Change?
Not needed

Thanks! @Diwakar-Gupta


Rename of node / agent

As per mailing list discussion:
https://keylime.groups.io/g/main/topic/node_agent/30833147

@jetwhiz  / @nabilschear - might look like a very heavy review, but github makes it very easy to speed read through and should only take 2-3 mins

what should we do with the binary content (e.g., presentations, papers, etc.) in the repo?

one other potential issue: this will break if you have an old keylime.conf in /etc because of the rename of the cloud_node section

hmm, not to sure on the binary objects..I could change the .doc stuff.

one other potential issue: this will break if you have an old keylime.conf in /etc because of the rename of the cloud_node section

What a code check be acceptable for this (I could put something into config parser)? the other option is we release this as a non backwards compatible release?

I have been playing around with this and I don't like adding checks in, its starting to get messy.
We could do something like this:
if config.has_section('cloud_node'):
    cert_path=config.get('cloud_node','revocation_cert')
else:
    cert_path=config.get('cloud_agent','revocation_cert')

But it means collecting some technical debt, as every time someone needs to use that config call, they need to make sure they check for both values.
The other option is to run something to update the cloud_node section to cloud_agent, but configparser deals with _sections as a dict, so the code becomes over burdened - also there are many entry points (keylime_verifier, keylime_registrar etc).
I honestly think we are best making the change and highlighting it in the release notes. I don't believe we (keylime) are used widely just yet, so we should not cause breakage for any users?

it's not like there's a lot of users yet.  so no backwards compatibility is not a big deal.

i was looking at the rest api doc.  it seems like we should also get rid of the word instance from our keylime vocabulary.  and replace it with agent too.

@lukehinds and you thought this PR was big. now it's even bigger!  i've banished instance from keylime too.  One question for @jetwhiz and @lukehinds , i looked at https://github.com/keylime/python-keylime/blob/master/doc/keylime-elevator-slides.pptx?raw=true and i wasn't sure if we should take the word node out of there too?

@nabilschear , the more, the merrier as far as I am concerned.
for the elevator slides, I am not of any strong opinion, you guys can do as you please there. Once this is decided, we should be good to merge then?

i went and actually looked at all the instance changes.  Hopefully tests pass now.  i think i caught all the perl -pie errors.  One more person to look over the whole PR would be good.  @jetwhiz

woohoo it passed the build!

sorry guys, messed up here fixing a merge request, working on getting this open again.


Make token data available to session

This PR changes the jwt authenticator to include all the fields in the tokenData in the sessionData it provides, so that the contents of the token will be available on the ember session.
For example, if your token payload is the following:
{
  "sub": "1234567890",
  "name": "John Doe",
  "admin": true
}

then you'll be able to do the following:
session.get('data.authenticated.name') //=> 'John Doe'

This fixes issues https://github.com/jpadilla/ember-simple-auth-token/issues/50 and https://github.com/jpadilla/ember-simple-auth-token/issues/120.
This PR includes https://github.com/jpadilla/ember-simple-auth-token/pull/207.

@jpadilla does this seem like a reasonable thing to do? Alternatively, is there another way you'd recommend to get at the contents of the token?

@tomclose, Why you need to overload your jwt payload with these information? You can't use another API for "current_user"? I think the payload should be as light as possible because it is sent back and forth every time. Or am I wrong?

@johnunclesam as far as I know, this is one of the advantages to using a jwt. From jwt.io:

JSON Web Token (JWT) is an open standard (RFC 7519) that defines a compact and self-contained way for securely transmitting information between parties as a JSON object.

and

Self-contained: The payload contains all the required information about the user, avoiding the need to query the database more than once.

They then give the following example payload:
{
  "sub": "1234567890",
  "name": "John Doe",
  "admin": true
}

You definitely could use another api call to fetch the current user, but it seems like pulling it out of the jwt payload is also a valid thing to do.

Also worth saying: in my actual use-case the piece of information I want to extract from the token is just the user_id, not the entire user.

+1

Having my user ID in the JWT payload I'd love to be able to access token data directly from the session service. Hope this change moves forward in future versions.

I'd actually like to chime in with a bit of a counter point on this topic. I don't think this addon should be adding data into the session without some sort of extra work by application.
I totally see the use case, and I actually do quite a bit of it through a custom authenticator. There's some data we send in our JWT's that we don't want persisted in the session (ie. last login time, total site visits).
So a better approach to this PR, I think would be having at least some sort of configuration flag that either include/exclude certain keys.

@donaldwasserman "There's some data we send in our JWT's that we don't want persisted in the session" - is there a reason it shouldn't be persisted? If the token contains the data, the data will be persisted in the token. It is just a matter of persisting it decoded in addition to persisting it encoded.

@fenichelar - I just save some random properties I don't necessarily want automatically overridden by refresh tokens and whatnot. If I'm the only person who does this, I'm not going to pitch a fit about it and will update my app.

@donaldwasserman Does it make sense to put the decoded properties in a object like tokenData so it won't override any other things set by the application?

@tomclose Great idea! Based on your idea, I made the token data available in the session but I made it available through tokenData instead of being available directly in the session:
session.get('data.authenticated.tokenData.name') //=> 'John Doe'

I think this is a good approach based on @donaldwasserm concerns.

I think the new solution solves this so unless there are any other concerns, I'll close :)


Online documentation

AutoFixture has rather decent documentation in the form of formal XML documentation comments in the source code, but unfortunately, few people discover it.
We should set up a site with on-line documentation, built from those XML comments, but we'd need an automated process for it. We should also consider how our rapid release cycle impacts the documentation; ideally, there should be a complete set of documentation files for each release - at least for every major and minor release.
Perhaps it would also be a good idea to ship the documentation with each NuGet package?
It's been quite a while since I last looked into the state of documentation generators for .NET, so I don't know what's good these days. Last time I looked, Sandcastle was the only viable alternative, and I found it quite difficult to work with.
We should keep in mind that we'd need to generate documentation from both C# and F# code bases.

Unless we can easily put this in our current build automation pipeline (e.g. using F# Formatting, or similar), should we close this?


Dynamic step size for sliding windows

Looks good! As an aside, we should start thinking about more flexible/less intrusive ways to attach state to queries without having to add fields to primitive nodes.

Maybe I should add a JSON field to pipeline_query?

Maybe we could go even more generic with a pipeline_config table with a schema like: (namespace oid, key text, value bytea)?

We have pipeline_tstate, maybe we should utilize that?

That would actually work because distinct is a varlena but that seems like a pretty awful hack. Definitely go ahead and merge this and lets talk about the generic state storage at some point.

We can probably create a new jsonb field in pipeline_tstate and rename the catalog to something a little more sane.


Pagination Summary is buggy

Hi. I'd like to point out there's a serious bug with the pagination. I followed what's said in Silverstripe's website tutorial and I remarked that if I change the PaginationSummary's value, I sometimes get a number of pages different from what I assigned and sometimes bizarre behaviours, too.
I will give you two examples. To put you in context, in my tests, I have a total of 6 pages. The only thing that will change is the summary of those pages.
1st example with PaginationSummary(3):
When on the 1st page, I get pages 1, 2, 3, ..., 6.
On the 2nd page, I get pages 1, 2, 3, ..., 6.
On the 3rd page, pages 1, 2, 3, 4, ..., 6. (You can notice a difference in the number of pages, it's not a summary of 3 pages, but 4.)
On the 4th page, pages 1, ..., 3, 4, 5, 6. (Same behaviour.)
On the 5th page, pages 1, ..., 4, 5, 6. (Back to normal.)
On the 6th page, pages 1, ..., 4, 5, 6.
In summary, we see an inconsistent number of pages shown which can affect the layout of the page, but still isn't that bad as the pagination still makes sense which isn't the case in the next example.
2nd example with PaginationSummary(4):
On the 1st page, I get pages 1, 2, 3, 4, 5, 6. (Quite long for what I asked for.)
On the 2nd page, pages 1, 2, 3, 4, 6. (Surprise! No ellipsis, but page 5 is missing, too!)
On the 3rd page, pages 1, 2, 3, 4, 5, 6.
On the 4th page, pages 1, 2, 3, 4, 5, 6.
On the 5th page, pages 1, 3, 4, 5, 6. (No ellipsis and page 2 is missing.)
On the 6th page, pages 1, 2, 3, 4, 5, 6.
The loop and the PaginationSummary function are obviously failing somewhere. It seems like PaginationSummary reaches it's end at some point, but doesn't return the value false, so the condition can't get into the ELSE which returns the ellipsis.What do we get then? No ellipsis and a missing page. That seems to occur only when the ellipsis is supposed to be there to replace only 1 single missing page as if the function confuses that page for the first or the last page. The number of pages presented by the pagination summary is inconsistent, too.

The first example seems fine to me. Always show the first and last page and then 3 pages around the current page (one either side of and including the current page).
The second one is clearly doing something wrong for 2nd and 4th instances but all others are as I'd expect.
The number passed to the summary will be how many pages to show (not including the first and last pages) before truncating.
For 6 pages showing a summary with length 4, there are only 4 pages between 1 and 6 (pages 2, 3, 4, 5) so you'd expect all of them to always show.
The bug is not the lack of ellipsis, but the missing number.
Would you be prepared to patch this? Also, please confirm which version of SS you're using.

Thank you for your answer!
If the function works as you said, I still see a problem with the first example in instances 2 and 5.
For example, let's take the second instance. If you have a pagination summary of length 3, you would expect this result: page 1 (the first page, not counted), pages 2, 3, 4 (the ones considered as summary, including the current one), then an ellipsis... (skipping page 5) and finally page 6. That's unfortunately not the case here. I get 1, 2, 3, ..., 6. It looks erratic to me. What do you think?
Also, just to be sure I'm getting the function's supposed behaviour, when a person is on the first page the page gets included in the pagination summary length as the current page always gets included (despite it being the first or the last one), right?
My version is 3.5.0.

Giving a look at the source code shows me pagination doesn't work that way.
Passed down length should be an even number. When the current page is not close to the first or last one, the current page doesn't count in the length, the length is divided by 2 and is added before and after the current page to give a visually balanced pagination. I don't know how the function behaves when the current page is the first or last one, or the half of the length can't be evenly added to both sides of the current page, because the previous one is the first page or close to it. I will check it up more.

I just made a fix and now the page that isn't included in the pagination returns a null value making it possible to detect as skipped and show an ellipsis instead.
I forked PaginatedList.php and pulled a request.

OK - thanks for looking into this. I've merged the fix into 3.4 which should then be merged up into 3.5 and master (4.0) eventually.


No redirects, please

This PR entered Yelp's fork in https://github.com/Yelp/hacheck/pull/4
If a service being healthchecked happens to return a redirect, I want to treat that as healthy. If it needs to know about some upstream service's health, that's its job to do, not mine. Don't follow redirects.

+1 this is a good change
Note that HAproxy treats 3xx as "okay" for healthchecks, which might be unexpected behavior to some people. shrug.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Josh Snyder seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Add Azure Storage Explorer Documentation

DX TED built an open source, cross-platform blob storage explorer
(storageexplorer.com). This pull requests adds a short article document
basic usage.
cc @tamram

Hi @felixrieseberg, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (feriese). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, or work for Microsoft Open Technologies, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

@felixrieseberg Thanks for the contribution!  We're reviewing your request.

Hey Felix -
We can't publish your article to azure.com because we can't promote tools on our official storage documentation site that the storage team does not support themselves.
For this reason, we need to close your PR.
Tamra


How to send midi messages to port opened by other software?

Hello!
I'm trying to send midi messages to port opened by other software but not succeed: I have to use mido.open_input or mido.open_output to get an opened Input/Output instance first.
Is there any solution for sending midi messages to port opened by other software?
Thanks a lot!

Unless your driver is multiclient which, in my experience is rare (Only RME comes to mind), you can’t open an already opened port.
You may be able to workaround that by creating a virtual port for the other software and merging its data with yours.


Remove ipc_cs folder

src/ipc_cs/client.rs and src/ipc/client.rs are very similar. Those files and the surrounding folders should be merged into a single folder src/ipc.

This will be fixed with #44.


Using Apollo in the Store Context alongside of Shopify-buy

Hi i'm using a fork of this starter to build an application.
I'd like to build a login/customer auth processes but this starter uses the shopify-buy sdk which doesn't support generating the access token needed to perform other mutations.
Could i use Apollo alongside the shopify-buy sdk in the  store-context or will I need to refactor instances of the client?

I implemented Apollo Client alongside Shopify buy. it seems to be work correctly.
I would have gone with Apollo and custom queries for this starter.


[Swift 3.1] Fix for the intermittent URLSession Test Failures on async wait.

The corresponding commit on master:
https://github.com/apple/swift-corelibs-foundation/pull/982/commits/a4497372448ed388d507028b7d7a68fc333a491c
PR: https://github.com/apple/swift-corelibs-foundation/pull/982

This is enabling URLSession tests on 3.1.

@swift-ci test and merge


Add support for ADR on US_902_928 band

This pull request adds support for ADR on the US 902-928 band.  The power limits are capped at 20dBm, although the 500kHz channels appear to allow 26dBm.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@htdvisser This latest change adds support for the 2 dB TX power steps that are used in the US band after I identified an issue with some additional tests.  I believe I have now addressed all of the previous comments.

@mgranberry: I couldn't push to your branch, so we'll continue with this in feature/us-adr


Add a requirements file

Add a requirements file to allow automatically installing dependencies. Note: this list was derived from only the postgres example.

We're now generating up requirements.txt for all projects using pipreqs.
This makes the requirements.txt for PostgreSQL just
Flask==0.12.2
psycopg2==2.7.3

We'll merge that change in ASAP with all requirements.txt files included. Thanks for the reminder.


Is there a authentication service for Oauth 2.0 with parameters in post body?

Many networks(Youtube, Soundcloud...) with Oauth 2.0 are required to put parameter in post body when getting a access token, and the default service is to put parameter in query. Thus could we add a new service for this functionality ?

Also JIRA Oauth (1.0a) authentication requires parameters in the body.

Seems a nice improvement. There's no need for a new service we only need to
add a new SignatureType type:
https://github.com/fernandezpablo85/scribe-java/blob/master/src/main/java/org/scribe/model/SignatureType.java
On Fri Feb 20 2015 at 7:43:49 AM Marcello Steiner<EMAIL_ADDRESS>wrote:

Also JIRA Oauth (1.0a) authentication requires parameters in the body.
—
Reply to this email directly or view it on GitHub
https://github.com/fernandezpablo85/scribe-java/issues/525#issuecomment-75219269
.

Also required for Azure.

Closing (but waiting for the PR)


Jetbrains Add files via upload

Jetbrains

@yubrajc First stage of verification passed, official domain for Manmohan Memorial Polytechnic is mmp.edu.np as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@yubrajc Pull request merged. Please start requesting the licenses in about an hour, to let the changes to propagate through our system. Thank you.


The text in the nodes of the svg image is not readable

This is a...

[x ] Bug Report

Problem:
On this page: https://kubernetes.io/docs/tutorials/kubernetes-basics/cluster-intro/
The text (white font on blue background) in the nodes of the svg image is not readable
Here is the source: https://kubernetes.io/docs/tutorials/kubernetes-basics/public/images/module_01_cluster.svg
Proposed Solution:
Enlarge the text in the svg or remove it, if it is not important.
Page to Update:
https://kubernetes.io/docs/tutorials/kubernetes-basics/cluster-intro/
My browser: Build-Identifikator: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:52.0) Gecko/20100101 Firefox/52.0

@steveperry-53 do you know who maintains the bootcamp tutorial?

Ping?  I thought Katacoda owned it.

@jaredbhatti who can make this SVG image more legible for us


Update KeyCloak

Update KeyCloak application deployment  to the latest version.
Update the Keycloak examples to the latest version.
Add properly self signed certificate for KeyCloak TLS communication inside the cluster (Ask Luuk).
Data migration from old version to a new one

Just to confirm that the update will be to the latest 8.0.2 release, which contains the fix for the Chrome 80 Same Site cookie update:
https://www.keycloak.org/2020/02/keycloak-802-released.html
Thanks

Keycloak 9.0.0 was released on Monday February 17 2020 https://www.keycloak.org/2020/02/keycloak-900-released.html

@toszo Do you have a date when will Epiphany with the upgraded Keycloak be released? Thanks.

@gj25 It is planned for 0.6.0 release (end of March'20).


Beautiful tweets

Describe the project you want to add with tech stack
This project involves the improving the tweeting site
Expected behavior
it would take new tweets and display all the tweets
Screenshots (optional)

Additional context (optional )

Hi,I would like to contribute to this issue.Please assign this to me.


How to use cronet network stack with video

Hi, we want to use cronet network stack in videos. i can see exo-player is also available inside the library but problem is we are using  and we are dependant on its events.  so now my query is

Is it possible to embed cronet into existing implementation of video? if yes then please provide support for that.
If not ,then how can i  use exoplayer instead of , without impacting existing functionality.

Thanks

I confirm this is not support yet:
Doc to integrate it is available here:
https://exoplayer.dev/network-stacks.html
This is a new feature to be implemented...
Let us know if you plan to integrate it


Don't include windows.h from platform.h

This partially reverts commit c150b9832d7d07761ec7869a625e6c46ba62206a.
Now params.cpp includes host.h which also gets the definition for MAX_PATH.
Signed-off-by: Stefan Weil<EMAIL_ADDRESS>

This addresses issue #2390.

Do you think it there reason to keep host.h and platform.h separated?
host.h is used only in few places:
src/ccmain/paragraphs.cpp:#include "host.h"                 // for NearlyEqual
src/ccmain/recogtraining.cpp:#include "host.h"  // for NearlyEqual
src/ccstruct/blobbox.cpp:#include "host.h"        // for NearlyEqual
src/ccstruct/blobbox.h:#include "host.h"              // for PRId32
src/ccstruct/boxword.cpp:#include "host.h"       // for NearlyEqual
src/ccstruct/imagedata.cpp:#include "host.h"        // for PRId64
src/ccstruct/ocrpara.cpp:#include "host.h"  // For NearlyEqual()
src/ccstruct/pdblock.cpp:#include "host.h"  // for PRId32
src/ccstruct/polyblk.cpp:#include "host.h"  // for PRId32
src/dict/dawg.h:#include "host.h"               // for PRId64

Do you think it there reason to keep host.h and platform.h separated?

Yes, see my comment.
The more difficult question is whether we may remove host.h from the 4.1 API.

Do not worry about that - I can ran abi-compliance-checker locally so I will backport all possible changes from master.
Also I found nice extension for Visual studio that eliminate not needed  includes (which is based on include-what-you-use), so maybe we can better optimize includes.


Terra.should.beAccessible, Terra-search-field

Summary
Addresses the Terra-search-field part of https://github.com/cerner/terra-core/issues/1370
Also removes the limiting accessibility tests for terra-collapsible-menu-view
Thanks for contributing to Terra.
@cerner/terra

Currently waiting on translations for "Submit Search" so we have default aria-labels.
https://jira.cerner.com/browse/TRANS-5504


build(all): bump dependencies for http-proxy-agent and handlebars

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/graycoreio/daffodil/blob/master/CONTRIBUTING.md#commit
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

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

What is the current behavior?
Some of our dev dependencies have sec advisories.

https://www.npmjs.com/advisories/1325
https://www.npmjs.com/advisories/1324
https://www.npmjs.com/advisories/1316
https://www.npmjs.com/advisories/1300
https://www.npmjs.com/advisories/1184

What is the new behavior?
This bumps the packages to remove the afflicted packages.
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information

This Lgtm


Dart VM Observer cause app crashed -> error: unreachable code

Steps to Reproduce

Execute flutter run on the code sample
I make a page which List.builder 200 pic in NestedScrollView with ResizedImageHandler and the memory is fine;
...
when debug my app on VM Observer, and find out the reason of  the unreleased extenal memory,and I saw there lot of image quotation and when I clicked the "imageInfo" and will crash every time.
...
Tap on Dart VM Observer -> persistent handles -> Inbound references -> ImageInfo  -> and then crash (everytime)

Expected results:
want to find out the _image References to the chain；
Actual results:
the app is crash and seem to crash by the VM Observer,
../../third_party/dart/runtime/vm/object.cc: 19225: error: unreachable code version=2.15.0 (stable) (Fri Dec 3 14:23:23 2021 +0100) on "ios_arm64" pid=85762, thread=42755, isolate_group=main(0x103037e00), isolate=main(0x101052e00) isolate_instructions=1078f34c0, vm_instructions=1078ee590

Not available at present the code.sorry...

the crash log

flutter doctor -v

the crash log (1/2)

the crash log (2/2)

click the imageinfo of Inbound references  in Observer everytime will crash

@ManInTheWind
Can you provide the code sample along with the crash log in text format instead of in screenshots ?
Thanks.


restore: add importing progress and optimize the accuracy of restore progress

What problem does this PR solve?
Better progress calculation and log
resolves partially #440
What is changed and how it works?

Remove the progress of file and add progress of engine(count for import). After all files restore finished, the state will be 'importing' instead of 'post-progress'. And after all engines import finished, the state will be 'post-progress'.
Add engine count metrics at estimating chunks count
Use the total pending chunks count instead of estimated chunks if pending chunk count > estimated chunk count to gain better accuracy. This can avoid the chunks progress exceeding 100%
Add total progress percentage. The value of total progress is estimatedWriteProgress * 0.8 + estimatedImportProgress * 0.2
Calculate chunk count based on chunk start offset to be consistent with the chunk count estimation logic.
Flush data and index engine and save all chunk checkpoints so that if restore engine meets context cancel all chunk checkpoints are saved.

New log output style:
[2020/12/07 00:59:44.725 +08:00] [INFO] [restore.go:694] [progress] [total=57.2%] [tables="7/9 (77.8%)"] [chunks="2642/4247 (62.2%)"] [engines="17/27 (63.0%)"] [speed(MiB/s)=131.06370869464234] [state=writing] [remaining=52m21s]
[2020/12/07 01:09:44.725 +08:00] [INFO] [restore.go:694] [progress] [total=65.0%] [tables="7/9 (77.8%)"] [chunks="3020/4247 (71.1%)"] [engines="18/27 (66.7%)"] [speed(MiB/s)=134.3286817015894] [state=writing] [remaining=43m10s]
[2020/12/07 01:59:44.725 +08:00] [INFO] [restore.go:694] [progress] [total=97.4%] [tables="7/9 (77.8%)"] [chunks="4601/4601 (100.0%)"] [engines="25/27 (92.6%)"] [speed(MiB/s)=132.1332473121803] [state=importing] [remaining=3m30s]

Check List
Tests

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

Side effects
Related changes

count you also add the estimated progress in /progress/task and /progress/table

count you also add the estimated progress in /progress/task and /progress/table

Lightning doesn't support return the progress for any individual table. Please submit a feature request issue.

LGTM
CI fail since TiDB keep the format d:<PHONE_NUMBER>3709551616.0 back

LGTM
CI fail since TiDB keep the format d:<PHONE_NUMBER>3709551616.0 back

/lgtm

/lgtm

/merge

/merge

/merge

/merge

/lgtm

/lgtm

/merge

/merge


prometheus -  Unify the testing across all packages.

/cc @Gouthamve @caniszczyk @idvoretskyi

This looks more like a small 1-2 days of work. While this can be picked up people looking to contribute, I'm apprehensive to adding it as full GSoC idea.

Yes this is a rather short assignment.
@Gouthamve  What is the general  time frame we need to define something as a task?
I am thinking maybe I can stack some other refactoring to make it a bigger task.
https://github.com/prometheus/prometheus/issues/3647 for example

So GSoC would last 3 months and while it could have several tasks, I'd ideally have tasks that are going to take more than 2weeks.

ok go it.


Correct static method call generation in CSharpHelper

Fixes #13985
/cc @bricelam

PS Possible candidate for patch

Description
EF Core contains some code for generating simple C# literals which is used for things like scaffolding migrations. The Npgsql provider needs to generate some different literals than our other providers and this exposed a couple of bugs.
Customer Impact
Prevents people using the Npgsql provider from using NodaTime types in migrations.
Regression?
No.
Risk
Very low--only impacts code generation and only in specific cases. Very isolated fix.

Approved for 2.2.1

@roji Can you re-base this patch on release/2.2? That will get it into the patch, and then we can merge release/2.2 back into master like we normally do.

FYI I'm going to make a few changes to release/2.2 first to ensure this repo is ready to patch.

Thanks Nate.

https://github.com/aspnet/EntityFrameworkCore/pull/14054

Rebased this on latest release/2.2, let me know if anything else is needed.
Do we also need a separate PR to get this into master or does some bot do that?

Do we also need a separate PR to get this into master or does some bot do that?

A bot will catch it if we forget to do it.


undefined method `supported_currencies' for #Spree::AppConfiguration:0x00007fc93c9152f0)

How do we define `supported_currencies`, I have tried all possible options like
Spree::Config.supported_currencies = ['USD', 'EUD']
Spree::Config[:supported_currencies] = ['USD', 'EUD']
Spree::Config['supported_currencies'] = ['USD', 'EUD']

None of the above configuration work for me. Can some one help me please.

I am trying to figure this out too. This looks like a very basic step to be missing from the setup.

Hi @tashi-179
I use this copy for Spree 4 https://github.com/spree-four/spree_multi_currency
Preferences are set in this file:
https://github.com/spree-four/spree_multi_currency/blob/master/lib/spree_multi_currency/engine.rb

@Tashows Can you elaborate your solution a little more. Whereabout in the admin section?

Hi @Voyotel
I don't think you will find a fix for this from something in the admin, its happening because the preferences have not been initiated properly when the app loads.
What version of Spree are you using and what version of rails are you using?

@MatthewKennedy I'm in Spree 4.1 and rails 6.0.3

@Voyotel If you replace the spree-contrib version with this:
gem 'spree_multi_currency', github: 'spree-four/spree_multi_currency'
That should work fine.
The multi currency functionality is being built into Spree 4.2 so at some point that extension will be a legacy extension.

@MatthewKennedy Thanks! That worked :)

Hi all, adding this to the file application.rb helped me to resolve this problem
config.after_initialize do
      Spree::Config.allow_currency_change  = true
      Spree::Config.show_currency_selector = true
      Spree::Config.supported_currencies   = 'GBP,USD,EUR'
end

@Voyotel If you replace the spree-contrib version with this:
gem 'spree_multi_currency', github: 'spree-four/spree_multi_currency'
That should work fine.
The multi currency functionality is being built into Spree 4.2 so at some point that extension will be a legacy extension.

This is not the solution. We should figure it out how to fix the current problem.

Hi all, adding this to the file application.rb helped me to resolve this problem
  config.after_initialize do
    Spree::Config.allow_currency_change  = true
    Spree::Config.show_currency_selector = true
    Spree::Config.supported_currencies   = 'GBP,USD,EUR'
  end

This looks like promising solution. Let me check and see if it is working.

Hi all, adding this to the file application.rb helped me to resolve this problem

  config.after_initialize do
    Spree::Config.allow_currency_change  = true
    Spree::Config.show_currency_selector = true
    Spree::Config.supported_currencies   = 'GBP,USD,EUR'
  end
This looks like promising solution. Let me check and see if it is working.

this did not work for me .... @tashi-179 and the repo seems to have moved @MatthewKennedy so I can't try your idea.
when I add the above I get app/models/spree/preferences/configuration.rb:64:in method_missing': undefined method allow_currency_change=' for #<Spree::AppConfiguration:0x00007fb40d0e5f10> (NoMethodError)
so I remove the first line and so on...but no good
#
gem 'rails', '~> 6.0.3', '>= <IP_ADDRESS>'
gem 'spree', '~> 4.1'

Hi all, adding this to the file application.rb helped me to resolve this problem

  config.after_initialize do
    Spree::Config.allow_currency_change  = true
    Spree::Config.show_currency_selector = true
    Spree::Config.supported_currencies   = 'GBP,USD,EUR'
  end
This looks like promising solution. Let me check and see if it is working.

this did not work for me .... @tashi-179 and the repo seems to have moved @MatthewKennedy so I can't try your idea.
when I add the above I get app/models/spree/preferences/configuration.rb:64:in method_missing': undefined method allow_currency_change=' for #<Spree::AppConfiguration:0x00007fb40d0e5f10> (NoMethodError)
so I remove the first line and so on...but no good
#
gem 'rails', '~> 6.0.3', '>= <IP_ADDRESS>'
gem 'spree', '~> 4.1'


[CAMEL-21199] Camel-jackson not properly marshalling 4-byte characters

Issue: https://issues.apache.org/jira/browse/CAMEL-21199
Jackson doesn't handle 4-byte characters well. Marshalling a 4-byte Japanese kanji character results in two UTF-16 escapes to be written instead of the character itself. While this is ok for emoji an such, it's not for natural languages.
Jackson issue: https://github.com/FasterXML/jackson-core/issues/223
Reproducer:
from("file:data?file-name=input.txt&noop=true")
    .log("${body}")
    .unmarshal().json(JsonLibrary.Jackson)
    .log("${body[0]['name']}")
    .marshal().json(JsonLibrary.Jackson, true)
    .log("${body}");

with the file input.txt containing:
[{"name": "システム𩸽"}]
Solution:
While it's an error in Jackson-core, it could be avoided by forcing Jackson to use WriterBasedJsonGenerator instead of UTF8JsonGenerator (see the original Jackson issue reproducer in the topmost comment)

This should be fixed in jackson instead of workaround in camel that can affect others.

Yes and if its dangerous for jackson, then why is it not dangerous for camel ?
What you can/should do is to create a JIRA ticket, and add a new option to the dataformat, where you can configure this with a boolean (default false) and then only use this new code if enabled. Then existing is not affected at all.

Yes and if its dangerous for jackson, then why is it not dangerous for camel ?
It was claimed dangerous for Jackson to dabble with UTF8JsonGenerator which is erronous. This solution forces Jackson to use WriterBasedJsonGenerator inside, which seems to work fine.

What you can/should do is to create a JIRA ticket, and add a new option to the dataformat, where you can configure this with a boolean (default false) and then only use this new code if enabled. Then existing is not affected at all.
Seems like an overkill to me, but if you like, I can surely add the option

Yes and if its dangerous for jackson, then why is it not dangerous for camel ?

It was claimed dangerous for Jackson to dabble with UTF8JsonGenerator which is erronous. This solution forces Jackson to use WriterBasedJsonGenerator inside, which seems to work fine.

What you can/should do is to create a JIRA ticket, and add a new option to the dataformat, where you can configure this with a boolean (default false) and then only use this new code if enabled. Then existing is not affected at all.

Seems like an overkill to me, but if you like, I can surely add the option

If its so non dangerous why has jackson not fixed it already. Please respect the large existing user-base of Camel that dont want hacks in commonly used components.
Also we need unit tests to go with that thanks.

If its so non dangerous why has jackson not fixed it already

Don't ask me. Ask them :)

@davsclaus I agree that it should be fixed in Jackson. Sadly, it hasn't been fixed for a long time and they don't consider it much of an issue. This hack as you call it just tries to overcome the error by invoking different string-writing mechanism (on their side) and should be same as soon as calling Jackson marshal method ends.
I see no point in providing an option. It's hardly an option for Camel users. It's either fixed, or not.
If you don't want this fix (which I can understand), feel free to close it. I'll than try to create a fix for Jackson itself. But I doubt they will accept it...

@davsclaus I agree that it should be fixed in Jackson. Sadly, it hasn't been fixed for a long time and they don't consider it much of an issue. This hack as you call it just tries to overcome the error by invoking different string-writing mechanism (on their side) and things should be same as soon as calling Jackson marshal method ends.
I see no point in providing an option. It's hardly an option for Camel users. It's either fixed, or not.

I am sorry but we cannot risk affecting everyone else with some code change that Jackson itself does have out of the box, and its been outstanding since 2015. Would such a change cause unforseen changes to existing users, performance degration or what else.
Are there other reported tickets at Jackson than this single instance?

Also we need unit tests to go with that thanks.

It cannot be unit-tested. It's integration issue.

What we ask is that there is a junit test class in the PR that tests this code change.

If you don't want this fix (which I can understand), feel free to close it. I'll then try to create a fix for Jackson itself. But I doubt they will accept it...

Maybe take more time to investigate if there are other reports on Jackson. But in the mean time there is only a solution you can do and that is on the Camel side. But that requires more work which you then must step up and do.
And it would require a JIRA ticket in the Apache Camel JIRA at
https://issues.apache.org/jira/projects/CAMEL/

I am sorry but we cannot risk affecting everyone else with some code change that Jackson itself does have out of the box, and its been outstanding since 2015. Would such a change cause unforseen changes to existing users, performance degration or what else.

I'm not sure if we talk about the same thing here. Jackson offers two string generators - UTF8JsonGenerator (which we are using at the moment and which contains the error) and WriterBasedJsonGenerator (which seems to handle 4-byte characters fine). I don't see switching from one to another risky for Camel users, since they should behave exactly the same (not speaking about the error itself).

@davsclaus I've added a test for the fix. Also, the Camel JIRA issue has been created.

BTW, I am only reviewing the technicalities here. @davsclaus points are a lot more important than my notes.

@orpiske That's fine. Thak you very much for your review!

We use unicode escape to have foregin symbols in the camel source code.
See for example
https://github.com/apache/camel/blob/main/core/camel-core/src/test/java/org/apache/camel/component/file/FileConvertBodyToUTF8Test.java
Can you see if you can replace those to escaped values in this PR unit test.

Added option useWriter in this PR
https://github.com/apache/camel/pull/15538

On main branch we have the new option useWriter that can be enabled for this use-cases.

@rnetuka what is the status of this ?

PR for Jackson issue 223 has been merged: https://github.com/FasterXML/jackson-core/pull/1335. There will be a new Jackson option to combine utf-16 surrogate characters into a one unicode char in their UTF8JsonGenerator.
I'm closing this PR and will create a replacement with the new option being supported.


⚠ Fortnite has degraded performance

In 88571d5, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1727 ms

Resolved: Fortnite performance has improved in 4c9d2d0.


[Bug]: MyReviews.vue page has null content

Expected Behavior
No response
Actual Behavior
When I initiate a storefront project with vendure, the MyReviews.vue page has empty content.
Here is the generated project: https://github.com/ekwoue/storefront-vendure
The page that's empty: https://github.com/ekwoue/storefront-vendure/blob/master/pages/MyAccount/MyReviews.vue
I tried on another workstation under Windows 11 and Node 13.14.0 and I had the same problem.
Does MyReviews.vue page need to be generated?
Possible Solution
No response
Steps to reproduce
No response
What version of vendure integration are you using?
1.5.0
What version of Node.js are you using?
14.19.0
What browser (and version) are you using?
Chrome
What operating system (and version) are you using?
Windows 11
Relevant log output
No response
Code of Conduct

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

Hey @ekwoue
Thanks for reporting that. This is kind of a strange issue TBH. I will investigate it in the upcoming days.

After second thought, Vendure does not have a reviews plugin implemented yet, so maybe this is the issue. We might have at some point removed this page as there is no support for reviews yet

Yeah, Vendure has no built-in reviews support.

@ekwoue
I know what is the problem. For some strange reason, the MyReviews.vue is a binary file in the template-vendure. I will remove it as Vendure does not have Reviews so we wont be able to cover it in VSF.

Will be released with a new 1.2.0 version -> https://github.com/vuestorefront-community/template-vendure/pull/11

Closing the issue as this bugfix was merged and released with 1.2.0


Add code mirror on code snippet display

Ideally, we want code snippets to display syntax highlighting as below:

Resolved by #589


Use TravisCI postgres service instead of docker for unit tests.

Summary
Use the built-in postgres service instead of gnomock/docker for unit tests. This should be significantly faster and more reliable.
Test Plan
Make sure the tests run faster.

Codecov Report

Merging #726 (7973741) into develop (25058e5) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 7973741 differs from pull request most recent head 97a137c. Consider uploading reports for the commit 97a137c to get more accurate results

@@           Coverage Diff            @@
##           develop     #726   +/-   ##
========================================
  Coverage    53.77%   53.77%
========================================
  Files           23       23
  Lines         3669     3669
========================================
  Hits          1973     1973
  Misses        1434     1434
  Partials       262      262

Continue to review full report at Codecov.

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


Fix option ordering with Ballerina cmds

Purpose
With PR https://github.com/ballerina-platform/ballerina-lang/pull/9991 option vs param ordering will be enforced for Ballerina commands.
This PR fixes incorrect usage, where options have been specified after the source to run.

These changes are required due to https://github.com/ballerina-platform/ballerina-lang/pull/9991.
https://github.com/ballerina-platform/ballerina-lang/pull/10173 needs to be merged to fix the test command ordering.


Property rdf:rest not found while printing Ask

I have the following sample scala code:
val constructModel = ModelFactory.createDefaultModel()
val arqQuery = ARQFactory.get().createQuery(constructModel, "CONSTRUCT { ?s ?p ?o } WHERE { ?s ?p ?o. }")
val construct = new ARQ2SPIN(constructModel).createQuery(arqQuery, null).asInstanceOf[Construct]
val ask = SPINFactory.createAsk(ModelFactory.createDefaultModel(), construct.getWhere)
val askString = ARQFactory.get().createCommandString(ask)

Assuming it is a correct usage of the API (I found it hard to find any docs other than autogenerated), then I always end up by the last line triggering the following exception:
Caused by: org.apache.jena.shared.PropertyNotFoundException: http://www.w3.org/1999/02/22-rdf-syntax-ns#rest
	at org.apache.jena.rdf.model.impl.ModelCom.getRequiredProperty(ModelCom.java:1231)
	at org.apache.jena.rdf.model.impl.ResourceImpl.getRequiredProperty(ResourceImpl.java:172)
	at org.apache.jena.rdf.model.impl.RDFListImpl.getTail(RDFListImpl.java:249)
	at org.apache.jena.rdf.model.impl.RDFListImpl$RDFListIterator.next(RDFListImpl.java:1220)
	at org.apache.jena.rdf.model.impl.RDFListImpl$RDFListIterator.next(RDFListImpl.java:1183)
	at org.topbraid.spin.model.impl.ElementListImpl.getElements(ElementListImpl.java:75)
	at org.topbraid.spin.model.impl.ElementListImpl.print(ElementListImpl.java:115)
	at org.topbraid.spin.model.impl.AbstractSPINResourceImpl.printNestedElementList(AbstractSPINResourceImpl.java:132)
	at org.topbraid.spin.model.impl.QueryImpl.printWhere(QueryImpl.java:224)
	at org.topbraid.spin.model.impl.AskImpl.printSPINRDF(AskImpl.java:32)

Therefore, I assume that the getWhere method somehow produces an ElementList that is not valid for Jena.
Am I doing something wrong or is this really a bug? Is there any workaround? Using spin-rdf-2.0.0.

I assume the bug would come from QueryImpl.getWhere
Element element = SPINFactory.asElement(whereS.getResource());
return (ElementList)element;

I believe this is related to how the WHERE clause is being serialized. You are creating the ASK query in a different model and when the ARQFactory attempts to iterate the ElementList it does not attempt to resolve the nodes in the old model. So, with the current SPIN implementation I think you need to stick with a single model or "copy" the relevant WHERE parts of the model (not sure how that will work with respect to the blank nodes in the list though).


Can no longer run locally

activator run yields
--- (Running the application, auto-reloading is enabled) ---

[info] p.c.s.NettyServer - Listening for HTTP on /0:0:0:0:0:0:0:0:9000

(Server started, use Ctrl+D to stop and go back to the console...)

[warn] Credentials file /Users/martijnv/.ivy2/.credentials does not exist
[info] JavaScript linting on 2 source(s)
[error] undefined:23
[error]     // Libraries
[error]     ^
[error]
[error] SyntaxError: Unexpected token / in JSON at position 373
[error]     at Object.parse (native)
[error]     at /Users/martijnv/dev/maproulette2-myfork/target/jshint/jshint-shell.js:29:24
[error]     at Object.<anonymous> (/Users/martijnv/dev/maproulette2-myfork/target/jshint/jshint-shell.js:66:2)
[error]     at Module._compile (module.js:541:32)
[error]     at Object.Module._extensions..js (module.js:550:10)
[error]     at Module.load (module.js:458:32)
[error]     at tryModuleLoad (module.js:417:12)
[error]     at Function.Module._load (module.js:409:3)
[error]     at Function.Module.runMain (module.js:575:10)
[error]     at startup (node.js:160:18)
[error] undefined:23
[error]     // Libraries
[error]     ^
[error]
...

delete your target directory under "maproulette2-myfork", and then try again. I have a suspicion that something is hiding there. You might be able to do sbt clean first as well and then run.

should have mentioned, tried that (removing /target as well as project/project and project/target but error persists. I sense that it has to do with sbt-less or sbt-jshint. I thought it might be related to this bug in an older version of sbt-jshint so next I tried to update plugins.sbt to current version 1.0.4 of sbt-jshint, which produced a whole different class of errors to do with the the JS hinting itself:
[info] JavaScript linting on 2 source(s)
[error] /Users/martijnv/dev/maproulette2-myfork/app/assets/javascripts/main.js:15: Strings must use singlequote.
[error]     "positionClass": "toast-top-center",
[error]                    ^
[error] /Users/martijnv/dev/maproulette2-myfork/app/assets/javascripts/main.js:15: Strings must use singlequote.
[error]     "positionClass": "toast-top-center",
[error]                                        ^
[error] /Users/martijnv/dev/maproulette2-myfork/app/assets/javascripts/main.js:16: Strings must use singlequote.
[error]     "toastClass": "notification",
[error]                 ^
[error] /Users/martijnv/dev/maproulette2-myfork/app/assets/javascripts/main.js:16: Strings must use singlequote.
[error]     "toastClass": "notification",
[error]                                 ^
[error] /Users/martijnv/dev/maproulette2-myfork/app/assets/javascripts/main.js:17: Strings must use singlequote.
[error]     "closeButton": false,
[error]                  ^
[error] 51 errors found

My guess is the default behavior for the jshint plugin was changed between 1.0.1 and 1.0.4 and it's doing much more strict checking than before. That's as far as I got for now.


Add mixing events to the library method

Add mixing event to 2D library methods.

@pjgrandinetti @deepanshs This issue relates to the fact that library 2D methods use the fictitious total mixing between events (pathways all have amplitude of 1) and that total mixing behavior might lead to discrepancies in spectral amplitudes.
Should we work towards adding in explicit MixingEvent objects between events for named methods (e.g. angle of 0 and phase of 0)?


Ensure Tab keyboard functionality after #37146 merge

Description
After
https://github.com/twbs/bootstrap/commit/1510c25e9be4d8f7f2ba23622bfdfd9afebb6598.
https://twbs-bootstrap.netlify.app/docs/5.2/components/navs-tabs/#javascript-behavior,
@julien-deramond pointed out:

The user for navigating between the tabs has to do the following:

Press "Tab" -> "Home" is focused
Press "Right" -> "Profile" is selected (not really focused with the blue outline)
Press "Tab" -> "Profile" is focused
Press "Right" -> "Contact"  is selected (not really focused with the blue outline)
Press "Tab" -> "Contact" is focused

/cc @patrickhlauke
Type of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Refactoring (non-breaking change)
[ ] Breaking change (fix or feature that would change existing functionality)

Checklist

[x] I have read the contributing guidelines
[ ] My code follows the code style of the project (using npm run lint)
[ ] My change introduces changes to the documentation
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] All new and existing tests passed

Live previews

https://deploy-preview-37200--twbs-bootstrap.netlify.app/docs/5.2/components/navs-tabs/#javascript-behavior

I haven't followed the around with https://github.com/twbs/bootstrap/commit/1510c25e9be4d8f7f2ba23622bfdfd9afebb6598 but I can confirm that this PR fixes the behaviour mentioned in the description and is back to normal as it was in the v5.2.

confirming that, based on the live preview, this works as it should. assuming that this does not reintroduce the problem noted in https://github.com/twbs/bootstrap/issues/37107, and that the JS tests all still make sense, this looks good to me


prepend/append for commands

Hello,
I found a bug: if the command called by the button inserts text, the text will be inserted inside the code of the button and the button will not be removed if remove=true.
To reproduce:

Install the calendar ICS plugin : https://github.com/muness/obsidian-ics
create a button that calls the command ICS
observe the text being inserted in the code

name Add Daily Agenda
type command
action ICS: import events
remove true

Perhaps one way to solve this would be to first remove the button then launch the command.
Another way would be to let users append/prepend/etc with commands in case they return text.
Note: the bug only happens if one stays in the "preview" mode. If I click the button, then quickly switch to edit mode, the text is entered correctly.

Ah interesting bug. It's a bit of a race between the button removal and insert the text. It definitely tries to do the removal first.
I wonder if I can make it insert below the button...
In the meantime, a workaround would be to execute the command inside a templater template and then use an append template button.

I guess making it insert just below the button could work!

In the meantime, a workaround would be to execute the command inside a templater template and then use an append template button.
I did not know we could call commands inside templater templates. I will check the docs (but if you have a code example at hand that would great)

I did not know we could call commands inside templater templates. I will check the docs (but if you have a code example at hand that would great)

Yeah! So I hook into obsidian's command execution method (this is how buttons works too):
<%* app.commands.executeCommandById("ics-plugin:import_events") %>

I did not know we could call commands inside templater templates. I will check the docs (but if you have a code example at hand that would great)

Yeah! So I hook into obsidian's command execution method (this is how buttons works too):
<%* app.commands.executeCommandById("ics-plugin:import_events") %>

Great, I did not know we could do this. Thanks a lot for this!


Allow users to view all posts with the same tag

I want to be able to click on an article tag and see all other posts that share that same tag.

You are now able to click on an article tag and see all other posts that share that same tag.

Alright @billymeinke you can now click on an article tag and see all other posts that share the same tag on the live site: https://flow.coe.hawaii.edu/

Looks good, thanks Myles.
On Wed, Jan 20, 2016 at 9:46 AM, Myles Enriquez<EMAIL_ADDRESS>wrote:

Alright @billymeinke https://github.com/billymeinke you can now click
on an article tag and see all other posts that share the same tag on the
live site: https://flow.coe.hawaii.edu/
—
Reply to this email directly or view it on GitHub
https://github.com/uhm-coe/flow/issues/4#issuecomment-173337952.

--
Billy Meinke, MEd
@billymeinke https://twitter.com/billymeinke / Web
http://billymeinke.github.io


Sign in – Mapping Queues Non-Pilot Analyst (NPU)

As a Non Pilot Ministry, I want to authenticate and login with IDIR, so that I can securely process fee related requests.
Assumptions & Scope
What are the assumptions for this story?
What is IN scope?
What is NOT in scope?
Acceptance Criteria
**Scenario 1: Requests that will Appear in the Business Queue – (Non-Pilot Ministry) **
•	GIVEN The Request queue for the Business queue is opened
•	WHEN there are requests to: Transportation, Transportation Investment Corp
•	THEN those requests shall appear in the Business queue as NPM
•	AND the requests will be actionable for processing analysts assigned to this group
Scenario 2: Requests that will Appear in the Justice Health Queue – (Non-Pilot Ministry
•	GIVEN The Request queue for the Justice Health queue is opened
•	WHEN there are requests to: Attorney General (MAG), Public Safety and Solicitor General, Coroners Service of BC (OCC), Emergency Management BC (EMB), Health (HTH), Mental Health and Addictions (MHA), Independent Investigation Office (IIO)
•	THEN those requests shall appear in the Justice health queue as NPM
•	AND the requests will be actionable for processing analysts assigned to this group
Scenario 3: Requests that will Appear in the Social Education Queue – (Non-Pilot Ministry)
•	GIVEN The Request queue for the Social Education queue is opened
•	WHEN there are requests to: BC Public Service Agency, Social Development and Poverty Reduction (MSD), Advanced Education and Skills Training (AED), Government Communications and Public Engagement (GCP), Education and Child Care (EDU), Children and Family Development (CFD)
•	THEN those requests shall appear in the Social Education queue as NPM
•	AND the requests will be actionable for processing analysts assigned to this group
Scenario 4: Requests that will Appear in the Resource Queue – (Non-Pilot Ministry)
•	GIVEN The Request queue for the Resource queue is opened
•	WHEN there are requests to: Agriculture, Food and Fisheries (AGR), Energy, Mines and Low Carbon Innovation (EML), Environment and Climate Change Strategy (MOE), Environmental Assessment Office (EAO), Indigenous Relations and Reconciliation (IRR), Declaration Act Secretariat (DAS), Ministry of Forests (FOR), Lands, Water and Resource Stewardship (LWR)
•	THEN those requests shall appear in the Resource queue as NPM
•	AND the requests will be actionable for processing analysts assigned to this group
Scenario 5: Requests that will Appear in the Central Queue – (Non-Pilot Ministry)
•	GIVEN The Request queue for the Central queue is opened
•	WHEN there are requests to: Finance (FIN), (Crown Agencies Secretariat (CAS), Liquor Distribution Branch (LDB), Office of the Premier (OOP) , Citizens’ Services (CTZ), Order of British Columbia Advisory Committee (OBC), Medal of Good Citizenship Selection Committee (MGC)
•	THEN those requests shall appear in the Central queue as NPM
•	AND the requests will be actionable for processing analysts assigned to this group
Dependencies? What is the impact of this dependency? (If so, link dependency in the ticket, make it visible in a team´s backlog)
Validation Rules? (If yes, list here)
Design
@xxx - please link the Design here
Definition of Ready

[ ] Is there a well articulated User Story?
[ ] Is there Acceptance Criteria that covers all scenarios (happy/sad paths)?
[ ] If there is a user interface, is there a design?
[ ] Does the user story need user research/validation?
[ ] Does this User Story needs stakeholder approval?
[ ] Design / Solution accepted by Product Owner
[ ] Is this user story small enough to be completed in a Sprint? Should it be split?
[ ] Are the dependencies known/ understood? (technical, business, regulatory/policy)
[ ] Has the story been estimated?

Definition of Done

[ ] Passes developer unit tests
[ ] Passes peer code review
[ ] If there's a user interface, passes UX assurance
[ ] Passes QA of Acceptance Criteria with verification in Dev and Test
[ ] Confirm Test cases built and succeeding
[ ] No regression test failures
[ ] Test coverage acceptable by Product Owner
[ ] Ticket ready to be merged to master or story branch
[ ] Developer to list Config changes/ Update documents and designs
[ ] Can be demoed in Sprint Review
[ ] Tagged as part of a Release
[ ] Feature flagged if required
[ ] Change Management activities done?

@m-prodan - for your review

Closing request, as this is a stale ticket and this had been handled through other tickets
@m-prodan @lmullane


Spark 2.0.0 support/documentation

Using this library in Spark 1.4 worked fine.
The below csv will work for test purposes

1,2,3
3,4,5

My dependency is the following

// https://mvnrepository.com/artifact/com.databricks/spark-csv_2.10
compile group: 'com.databricks', name: 'spark-csv_2.10', version: '1.4.0'

Spark 1.4 code that works fine
public static void main(String[] args) {
        SparkConf conf = new SparkConf().setAppName("my app");

        JavaSparkContext sc = new JavaSparkContext(conf);

        SQLContext sqlContext = new org.apache.spark.sql.SQLContext(sc);

        DataFrame df = sqlContext.read()
                .format("com.databricks.spark.csv")
                .option("header", "true")
                .option("nullValue", "")
                .load("/home/aseem/data.csv");
        df.show();
    }

But in Spark 2.0 when I try to use the equivalent
   public static void main(String[] args) {
        SparkSession spark = SparkSession
                .builder()
                .appName("my app")
                .getOrCreate();

        Dataset<Row> df = spark.read()
                .format("com.databricks.spark.csv")
                .option("header", "true")
                .option("nullValue", "")
                .csv("/home/aseem/data.csv")
                ;

        df.show();
    }

I get the following error

Exception in thread "main" java.lang.RuntimeException: Multiple sources found for csv (org.apache.spark.sql.execution.datasources.csv.CSVFileFormat, com.databricks.spark.csv.DefaultSource15), please specify the fully qualified class name.

CSV datasource was ported into Spark 2.0. Try .format("csv") instead :).

@HyukjinKwon
Do I need to include the dependency or is it now part of Spark 2.0? Also it would be good to document that.

@HyukjinKwon
I ran with format('csv'). Seems to be no.

Have you taken out the existing CSV datasource? It seems Spark got confused which one should be used.

@HyukjinKwon
I was able to run it on the sample csv that I have provided when I removed the dependency from my build.gradle

// https://mvnrepository.com/artifact/com.databricks/spark-csv_2.10
compile group: 'com.databricks', name: 'spark-csv_2.10', version: '1.4.0'

What I guessed from your comment

CSV datasource was ported into Spark 2.0. Try .format("csv") instead :).

was that I don't need to use this library, already integrated in Spark 2.0. What do you mean by take out existing CSV dataSource?

Oh, yes. It is part of Spark 2.0 and yes, I think this should be documented.

https://issues.apache.org/jira/browse/SPARK-16893

Oh, maybe this issue is within this library maybe.. cc @falaki

@HyukjinKwon
I would expect that the need to use .format("csv") when I am already using the function .csv() to be documented in spark. A simple POC cannot be done without facing the error that I faced unless Spark guide mentions that or better still handle that. But yes, makes sense to document it in both places.

@HyukjinKwon I'm having this same issue in Databricks release 2.29, Spark 2.0. I ended up needing to specify the entire csv package as
sqlContext
  .read
  .format("org.apache.spark.sql.execution.datasources.csv.CSVFileFormat")
  .option("header","true")
  .schema(schema)
  .load(csvPath)

We're currently mixing Spark 2.0 and 1.6, so I have the csv library set to automatically attach to all clusters. Is there any chance it can just prefer the Spark 2.0 built in library?
Also all of the following threw that same error @anshbansal originally reported:
.format("csv")
.format("com.databricks.spark.csv")
.format("csv")
.csv(csvPath)

Hi @chrishfish , I recall it is discouraged to mix Spark versions. Would it be possible not do to this but just use one of both this and Spark's internal one?

@HyukjinKwon I mean we're mixing as in we have multiple clusters up at a time, one 1.6 and one 2.0. I'm partway through migrating all our jobs onto 2.0, but until I finish we'll have both up. I can try setting the com.databricks.spark.csv library to only load on the 1.6 clusters?

Having issues with Spark 2.0 ,
If i may ask, how is the new appropiate way of loading a csv as a DataFrame in java?

Nevermind, found it. It would be good to add it to the Java Docs.

All these issues are arising because while starting spark shell or program we are also trying to import separately the jar file for spark csv as a practise that is carried over from earlier 1.6 version. Please remove that an everything looks fine then onwards.

Get rid of this parameter "--packages com.databricks:spark-csv_2.11:1.4.0" from your Spark-Submit command if you are passing one. Spark 2.0 is confused with it's inbuilt csv library and the one you are passing.

@chrishfish This worked perfectly to resolve the issue.

@HyukjinKwon I'm having this same issue in Databricks release 2.29, Spark 2.0. I ended up needing to specify the entire csv package as
sqlContext
  .read
  .format("org.apache.spark.sql.execution.datasources.csv.CSVFileFormat")
  .option("header","true")
  .schema(schema)
  .load(csvPath)

I think we can close this cc @falaki


Trying to uninstall rbenv

Hi.
I have been trying to uninstall rbenv, however it seems to be lingering on my system! I used sudo apt-get remove rbenv, and it said it had removed it. However, when I ran this command:
user@user-pc-SATELLITE-C55-A:/opt/metasploit-framework$ bundle exec gem list, I got this:
rbenv: version ``2.3.1' ``is not installed (set by /opt/metasploit-framework/.ruby-version) benjamin@benjamin-pc-SATELLITE-C55-A:/opt/metasploit-framework$.
I had checked my ~/.bash_profile file, however it had no references to rbenv in it.
Thanks in advance!

Intuitively, this doesn't sound like a rbenv issue, but I would be happy to help.
What exactly did you do to attempt to uninstall rbenv and how was it installed in the first place?
Additionally, more information would be off great use to diagnose. Would you kindly provide the output of:
echo '$PATH:' $PATH
echo uname -a: $(uname -a)
echo lsb_release -a: $(lsb_release -a)
echo gem: $(which gem)
echo ruby: $(which ruby)
echo rbenv: $(which rbenv)

If you installed rbenv using apt-get, then apt-get remove rbenv is a good first step. Next steps should be:  nuke the ~/.rbenv directory and all references to "rbenv" in ~/.bashrc, .bash_profile, or similar shell initialization files.


A robust time-stepping scheme

It is very important that the time-step provided to SPIDER is reasonable, since it directly determines how easily it can feedback with other components of the wider model.
There are currently three schemes for setting the time-step, but none of them are very good. It would certainly be worthwhile thinking about more robust methods for setting this value, and also having it adjust to various conditions in an appropriate way (e.g. decreasing to resolve sudden changes in a variable).
This is connected to Issue #62.

Completed.


Nuxt support for named views for extendRoutes config

Version
v2.1.0
Reproduction link
https://codesandbox.io/s/jvjm62ll5v
Steps to reproduce
extend route in nuxt.config.js
router: {
    extendRoutes (routes, resolve) {
      let index = routes.findIndex(route => route.path == '/')
      const children = [{
        path: 'sidebar',
        components: {
          default: resolve(__dirname, 'components/Sidebar.vue'),
          two: resolve(__dirname, 'components/SidebarTwo.vue')
        }
      }]

      routes[index]['children'] = children
      console.log(util.inspect(routes, false, null))
    }
  },

add two components:
/components/Sidebar.vue
<template>

Sidebar Default

</template>


Broken data: Support for spread in destructuring

The page Support for spread in destructuring is broken. No data is shown as the HTTP request GET https://caniuse.com/process/get_feat_data.php?type=support-data&feat=mdn-javascript_builtins_regexp_lookbehind_assertion returns [].

Thanks, turns out that data was removed from BCD. I have updated the link to it accordingly (now points at Rest in arrays).


Add GlobalSign Root R6 to cacert.pem

Describe the feature
Can we add new GlobalSign Root R6 certificate (https://support.globalsign.com/ca-certificates/root-certificates/globalsign-root-certificates) to cacert.pem file as it is used by default in aws-cli shipped by amazon? Yes we can override this behavior by env variables, but it's an extra step that can be avoided for publicly acknowleged root certificate.
Use Case
Some custom third-party s3 providers already use new GlobalSign certificates, but we have some inconsistencies with aws-cli v1 and v2, as v1 uses updated certifi with updated certificate list, but v2 still depends on outdated cacert.pem.
Proposed Solution
add GlobalSign Root R6 certificate to cacert.pem
Other Information
No response
Acknowledgements

[ ] I may be able to implement this feature request
[ ] This feature might incur a breaking change

SDK version used
ALL
Environment details (OS name and version, etc.)
aws-cli/2.15.59 Python/3.11.8 Linux/6.8.0-31-generic exe/x86_64.ubuntu.24

Thanks for the feature request. Upon discussing with the team, the recommendation here is to install certifi to get access to newer certs. That should be used by default if it is installed. Further investigation by the team is required involving changes to cacert.pem, but this feature request is not planned.


Add ref to git config

Give the ability  to pass  the githash to be used during build or whatever...
What I am not sure if ref will do another call to github.

@ahelal, please write a test for this feature. As per the contribution guidelines.
Then I'll accept, thanks.

Also, ahelal, since it is a git repo, you can get the SHA via git commands.
git rev-parse HEAD will do what you want.
I think we can stick with that method as it is standard.


Avoid creating obsolete gateway-api namespace

Small issue in #1051 - both manifests create the required namespace and we only renamed gateway-api to gateway-system in one of them.
Before:
namespace/gateway-system created
validatingwebhookconfiguration.admissionregistration.k8s.io/gateway-api-admission created
service/gateway-api-admission-server created
deployment.apps/gateway-api-admission-server created
namespace/gateway-api created

after:
namespace/gateway-system created
validatingwebhookconfiguration.admissionregistration.k8s.io/gateway-api-admission created
service/gateway-api-admission-server created
deployment.apps/gateway-api-admission-server created
namespace/gateway-system unchanged

/kind cleanup

Thanks @markmc!
/lgtm
/approve
/ok-to-test

Looks like a flake in kind setup?
W0323 18:06:05.931931    7279 loader.go:221] Config not found: /tmp/tmp.Tek9Dsu9mb/.kubeconfig
The connection to the server localhost:8080 was refused - did you specify the right host or port?

/retest


add: Saksham,Nandita,Maimuna,Krisha

Description
Added the enhancement to search with a list of skills
Changes Proposed
Added a new folder helper that consist of the two files

@sakshamgupta912 Where is profile?