Allow unspecified (null) HDFS host in Path URI if it is defined by fs.defaultFS

The idea is that you shouldn't be compelled to provide the HDFS host if it's already set in the configuration (which it usually is on a cluster). So you should be able to say hdfs:///path/to/file
I've written a negative unit test for this. Writing a positive one is hard, as the Hadoop configuration files would need to be changed to write the fs.defaultFS URI for the mini cluster, and this would interfere with other tests. I've tested it manually though and it works fine.

Coverage decreased (-0.06%) to 76.409% when pulling ef0f5b723da66788523fb5e12c1c1561dbbe61d3 on tomwhite:allow-null-host into 2a39e08e8d440270298ddc48b39605b95a5ef9f9 on damiencarol:master.

Coverage decreased (-0.06%) to 76.409% when pulling 87f6c3471a3fdc5ea1d0b7d7ba10cbcf50abdca9 on tomwhite:allow-null-host into 2a39e08e8d440270298ddc48b39605b95a5ef9f9 on damiencarol:master.

@damiencarol thanks for merging this. It looks like the release 1.0.3 (with this in it) never made it to Maven Central. Would you be able to publish it please?

@tomwhite sorry for the delay. I forget the 'deploy' stage.
Done.
It is in OSS. May take few minutes. Sorry for the delay
SHA1 a9a958535a9965cd00fb1ecadacf9d06cd874e5e
MD5 a5456a1d542211fca4a5812db484b095

@damiencarol I can retrieve 1.0.3 now. Thanks!


When in CSS document  letter l is always on CAPS (L)

When in CSS document,  letter "l" is always on CAPS (L).  When pasted in a clipboard or other external document the letter l is back to normal (not in CAPS).  The other way around is also true.  When copy from the clipboard in normal letter "l", when in CSS then becomes "L"
I have restarted my computer by the way......several times.....

Can you attached a screen shot and sample code?

Hi Johannes;
I am sending you the website files and the videos (one showing the bug and
the other showing the extensions).
Hope you can tell me what to do next.
Regards
main.css - website1 - Visual Studio Code 2019-0...
https://drive.google.com/file/d/1T03YOz-9YiQLcztjEJMPi5Vu0Ul5Z6Os/view?usp=drive_web
● main.css - website1 - Visual Studio Code 2019...
https://drive.google.com/file/d/1zrxmn4PWPyKtrfnjGujAV9m269tDir1J/view?usp=drive_web
On Tue, May 28, 2019 at 10:32 AM Johannes Rieken<EMAIL_ADDRESS>wrote:

Can you attached a screen shot and sample code?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/74424?email_source=notifications&email_token=AIHYS4CXKJPLHBV7FLNVA5DPXTUYXA5CNFSM4HP5VJ7KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWLL2RA#issuecomment-496418116,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIHYS4DDN7XHZHC2RA36NWDPXTUYXANCNFSM4HP5VJ7A
.

The font you are using isn't the default font but seems to the root of your problem. Not sure how that happened, but open the settings editor (F1 > Settings), search for "font", and change it back to another font

You were right!  Changing fonts had the desired effect.  Thanks a lot for
your help.
By the way I am now using: Monaco, Menlo, "Droid Sans Mono" (in that order)
The trio before was: Consolas,'Courier New',monospace'
On Wed, May 29, 2019 at 9:35 AM Johannes Rieken<EMAIL_ADDRESS>wrote:

The font you are using isn't the default font but seems to the root of
your problem. Not sure how that happened, but open the settings editor (F1

Settings), search for "font", and change it back to another font

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/74424?email_source=notifications&email_token=AIHYS4GUIRPELKF7DINDKW3PXYW2JA5CNFSM4HP5VJ7KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWOODDI#issuecomment-496820621,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIHYS4FK4AVSEWUQEYJPVJTPXYW2JANCNFSM4HP5VJ7A
.


access-om3 tests

I wrote a very simple access-om3 model for these tests when I was playing about with the repro CI workflows a month or so ago. Should I move this here, or is there still some dust that needs to settle?

Make a PR and we can discuss details. @jo-basevi is best placed to advise if there are any structural requirements.


Add Srcery style

Migrate Srcery (https://srcery-colors.github.io/) to highlight.js.
Screenshot,

Please add a changelog entry.

changelog entry added.

all done. Please review.

You didn't add a link to yourself, line 65 or so.

Sorry. I'm not familiar with markdown. The link is added.


Rewrite the golden tests

Closes #2130.
Rewrite the golden tests to be file based (yet again, after #1855). This time,
we use hard-coded paths relative to the root of the project. This makes
updating the results much easier than before. Previously, you had to run the
test with an empty expected list and then copy-paste the new output. Now, you
can just delete the file and run the test again, no more copy-pasting needed.
We follow a template-based approach like we did for the roundtrip tests: for
each ledger, we now only have to construct a value of the Examples record
and feed that to the test runner instead of having to redefine tons of tests
for each ledger.
Moreover, we now exhaustively test all serialisation format, not just the
default encoders, but all versions (NodeToNode and NodeToClient) as
well.

bors merge


Rename mapping_{2,3}d::SparsePoseGraph.

RFC=0001

@wally-the-cartographer merge

Merge requested by authorized user wohe. Merge queue now has a length of 1.


ANTs interfaces in nipype outdated?

@mattcieslak -- any idea why this command is failing with registering the original dwi's to the predicted?
antsRegistration --collapse-output-transforms 1 --dimensionality 3 --masks /Users/derekpisner/Downloads/test_subs/SWU4/sub-0025629/ses-1/dwi/emc_correction/emc_wf/enhance_and_skullstrip_template_mask_wf/fill_holes/average_trans_TruncateImageIntensity_mask_FillHoles.nii.gz --float 1 --initialize-transforms-per-stage 0 --interpolation BSpline --output [ transform, transform_Warped.nii.gz ] --transform Rigid[ 0.2 ] --metric Mattes[ /Users/derekpisner/Downloads/test_subs/SWU4/sub-0025629/ses-1/dwi/emc_correction/emc_wf/dwi_model_emc_wf/initial_model_iteration/predict_dwis/mapflow/_predict_dwis89/predicted_ten_b1000_-0.98_0.18_0.10.nii.gz, /Users/derekpisner/Downloads/test_subs/SWU4/sub-0025629/ses-1/dwi/sub-0025629_ses-1_dwi_tmp_92.nii.gz, 1, 48, Random, 0.15 ] --convergence [ 100x100, 1e-06, 20 ] --smoothing-sigmas 8.0x2.0mm --shrink-factors 2x1 --use-estimate-learning-rate-once 1 --use-histogram-matching 0 --transform Affine[ 0.15 ] --metric Mattes[ /Users/derekpisner/Downloads/test_subs/SWU4/sub-0025629/ses-1/dwi/emc_correction/emc_wf/dwi_model_emc_wf/initial_model_iteration/predict_dwis/mapflow/_predict_dwis89/predicted_ten_b1000_-0.98_0.18_0.10.nii.gz, /Users/derekpisner/Downloads/test_subs/SWU4/sub-0025629/ses-1/dwi/sub-0025629_ses-1_dwi_tmp_92.nii.gz, 1, 48, Random, 0.2 ] --convergence [ 100, 1e-06, 20 ] --smoothing-sigmas 2.0mm --shrink-factors 1 --use-estimate-learning-rate-once 1 --use-histogram-matching 0 --winsorize-image-intensities [ 0.002, 0.998 ]  --write-composite-transform 0
Segmentation fault: 11

(which appears to not throw an error if you add an -o to the end manually?)
but this is all coming from:
# Register original images to the predicted images
register_to_predicted = pe.MapNode(ants.Registration(from_file=ants_settings),
                                       iterfield=['fixed_image', 'moving_image'],
                                       name='register_to_predicted')
register_to_predicted.synchronize = True

where ants_settings=
{
    "dimension": 3,
    "float": true,
    "winsorize_lower_quantile": 0.002,
    "winsorize_upper_quantile": 0.998,
    "collapse_output_transforms": true,
    "write_composite_transform": false,
    "use_histogram_matching": [ false, false ],
    "use_estimate_learning_rate_once": [ true, true ],
    "transforms": [ "Rigid", "Affine" ],
    "number_of_iterations": [ [ 100, 100 ], [ 100 ] ],
    "output_warped_image": true,
    "transform_parameters": [ [ 0.2 ], [ 0.15 ] ],
    "convergence_threshold": [ 1e-06, 1e-06 ],
    "convergence_window_size": [ 20, 20 ],
    "metric": [ "Mattes", "Mattes" ],
    "sampling_percentage": [ 0.15, 0.2 ],
    "sampling_strategy": [ "Random", "Random" ],
    "smoothing_sigmas": [ [ 8.0, 2.0 ], [ 2.0 ] ],
    "sigma_units": [ "mm", "mm" ],
    "metric_weight": [ 1.0, 1.0 ],
    "shrink_factors": [ [ 2, 1 ], [ 1 ] ],
    "radius_or_number_of_bins": [ 48, 48 ],
    "interpolation": "BSpline"
}

I might be missing something obvious here, so please let me know if that's just the case.
and @oesteban - are all of the ANTs interfaces in nipype up-to-date?

Yup, that looks good. Closing this issue for now since the ANTs calls that were version-sensitive were mostly for reg, which we're replacing with dipy calls.


While changing the readiness probe for cert-controller in values.yaml the cert-controller deployment readiness doesn't get changed

Change the value of readiness port for cert-manger in values.yaml, it will not changed in the deployment.yaml of cert-controller since the readiness port is hardcoded in the cer-controller-deployment.yaml file. (line no 74-78)
      readinessProbe:
        httpGet:
          port: 8081
          path: /readyz
        initialDelaySeconds: 20

It should be
      readinessProbe:
        httpGet:
          port: {{ .Values.certController.readinessProbe.port }}
          path: /readyz

Thank you, @rnadeem-amzn! You are totally right, and I fixed it in https://github.com/external-secrets/external-secrets/pull/2732


Support pull_request_target events

What this PR does / Why we need it
In #10, one of this action's workflows was modified to be triggered by the pull_request_target event, and similar changes have been made in other actions-ecosystem repositories as well, but this action does not actually support this event. Since pull_request_target is considered safer than pull_request and also allows execution from forks, I believe it would be more appropriate to update the action's code rather than the workflows.
Which issue(s) this PR fixes
Fixes no issues

Similar to #12, but seems like this repo is unmaintained now maybe???
ping @micnncim


Upgrading to 0.5.6 from 0.5.4 causes tests to fail with Can't find variable: heimdall

It appears as though var _heimdall$registerMon = heimdall.registerMonitor(...) is being called in ember-data/-private/system/relationships/state/relationship.js, and causing the issue.

I have not had a chance to dig in, but here are some random thoughts that might help someone trying to look at this:

ember-data uses a custom babel plugin to strip the internal heimdall usage when the --instrument flag is not passed
we run "modules only" babel transpilation for addons inside the engine
ember-cli's behavior for transpilation changes between<EMAIL_ADDRESS>and<EMAIL_ADDRESS>(in 2.13 we use babel@6 by default)

@fushi - A few questions:

Can you share your ember-cli version?
Was ember-data a dep of the app or the engine?
What version of ember-cli-babel does both the app and engine depend on?
Can you create an repo that demos the issue?

<EMAIL_ADDRESS><EMAIL_ADDRESS>(app dep)
<EMAIL_ADDRESS>(though several addons seem to be using 5.2.4).
As for the repoduction repo, I'm trying to find a reproducible combination/setup


net: LookupAddr force use of cgo no matter what

While investigating #12189, found that the new Go 1.5 implementation of LookupAddr always calls cgo first, even when the DNS configuration on the machine does not merit it, and even when GODEBUG=netdns=go is set in the environment. (If the tree was built with CGO_ENABLED=0 ./make.bash then the cgo call fails quickly and the code falls back to the pure Go version. But it is still called.)
Previous releases of Go had no cgo implementation of LookupAddr, so they always used the pure Go version.
Using cgo unconditionally means that a thread is held by a blocked call (except when CGO_ENABLED=0 of course), where in previous releases it was only a goroutine being held. This is a significant resource increase.
It's even worse that setting GODEBUG, which is supposed to keep cgo from being used, does nothing in this case.
LookupAddr should respect GODEBUG and even without GODEBUG it should not use cgo if the resolver config can be handled by pure Go.

Dupe of #11238? There is golang.org/cl/11233 and it's marked as R=go1.6.

Thanks for the investigation and sorry to trouble you. The change looks good to me even though it appends a trailing dot unconditionally when GODEBUG=netdns=cgo and the name is stored in local database such as /etc/hosts. I now think it's fine because /etc/hosts could be considered as a substitute for some DNS RRs; A, AAAA, PTR (and CNAME on some platform), and actually mDNS speaker implementations use it as a source of RRs.


Do not support websocket 13?

It seems that do not support websocket 13.

Hello @zutuanwang,
I have no clue what you mean with websocket 13 and why you think this is not supported.
Could you elaborate a bit more please?
Thank you!
Greetings
marci4

No answer for over a month and no clear question/issue.
Closing!
Greetings
marci4


Date Invalid, on creation of Daily Note

Hi there, thank you Mathis for this beautiful workflow.
My issue here is When creating new daily notes their title is : "Invalid Date".
In the template file I got "Type mismatch, expected date and time"

Hey there, thanks for your input.
It is surely linked to a plugin update, but as I don't have time to spend on this template anymore, I won't be able to find out why it happened.
If you find a solution, feel free to make a pull request to help others around 💫
Sorry, and have a great day !

Hi, fair enough, I will try again tomorrow.
thank you


Update homebrew formulas to 0.15.0

CC @nicolasnoble  @stanley-cheung  @murgatroid99

@jcanizales

@y-zeng , once you have the PR, please add @stanley-cheung  as reviewer.
https://github.com/grpc/homebrew-grpc/pulls?q=is%3Apr+is%3Aclosed

Updated in https://github.com/grpc/homebrew-grpc/pull/82


How to store states in dynamic collections

Consider this code, the error is
`error[E0499]: cannot borrow self.state_hashmap as mutable more than once at a time
fn new() -> Counter {
let mut res = Counter::default();
	res.states = HashMap::new();

	res.state_hashmap.insert(1, button::State::new());
	res.state_hashmap.insert(2, button::State::new());
	res.state_hashmap.insert(3, button::State::new());

	res
}

fn view(&mut self) -> Element {
Row::new()
.padding(20)
.align_items(Align::Center)
	.push(
		Button::new(self.state_hashmap.get_mut(&1).unwrap(), Text::new("Increment"))
			.on_press(Message::IncrementPressed),
	)

	.push(
		Button::new(self.state_hashmap.get_mut(&2).unwrap(), Text::new("Increment"))
			.on_press(Message::IncrementPressed),
	)

	.push(Text::new(self.value.to_string()).size(50))

    .into()
}`

Each call to state_hashmap.get_mut(...) will borrow mut state_hahmap for as long as the Button lives. This means that you cannot call it twice.
A possibility would be to use BTreeMap instead of a hashmap (to guarantee that state_map.iter_mut() always gives you the states in the same order, and to use an iterator to collect your states.
fn view<'a>(&'a mut self) -> Element<'a> { // There is no need to make lifetimes explicit but this is that your code will desugar to
  let mut row = Row::new().padding(20).align_items(Align::Center);
  let keys = [1, 2];
  for (_, state) in self.state_map.iter_mut().filter(|(k, _) keys.contains(k)) {
    row = row.push(Button::new(state, Text::new("Increment")));
  }
  row.into()
}

Keep in mind though that self will be borrowed mut for as long as your Element lives

An other possibility is to store your states in RefCell,
res.states_map.insert(1, RefCell::new(button::State::new()));
res.states_map.insert(2, RefCell::new(button::State::new()));
res.states_map.insert(3, RefCell::new(button::State::new()));

fn view(&mut self) -> Element {
Row::new()
.padding(20)
.align_items(Align::Center)

	.push(
		Button::new(self.state_hashmap.get(1).unwrap().borrow_mut(), Text::new("Increment"))
			.on_press(Message::IncrementPressed),
	)

	.push(
		Button::new(self.state_hashmap.get(&2).unwrap().borrow_mut(), Text::new("Increment"))
			.on_press(Message::IncrementPressed),
	)

	.push(Text::new(self.value.to_string()).size(50))

    .into()
}

Thx, looks like the question was more about rust itself rather than iced.

Why are you storing your states in a Map-like structure? Consider using a fixed-size array if you know how many element there are going to be (or maximum number of elements) or a Vec<S>. Then you can mutably iterate through this. It will be much simpler, and it will be ordered correctly.
let states = [State::default(); 3];

states
    .iter_mut()
    .fold(Row::new(), |mut row, state| {
        row.push(Button::new(state, content))
    })

Good idea also, I'm thinking of using Vec.


Twitch -  urlopen error _ssl.c:835: The handshake operation timed out

Checklist

[x] I'm reporting a broken site support
[x] I've verified that I'm running youtube-dl version 2021.06.06
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar issues including closed ones

Verbose log

[twitch:vod] 460762727: Downloading stream metadata GraphQL
[twitch:vod] 460762727: Downloading video access token GraphQL
[twitch:vod] 460762727: Downloading m3u8 information
**ERROR: Failed to download m3u8 information: <urlopen error _ssl.c:835: The handshake operation timed out> (caused by URLError(timeout('_ssl.c:835: The handshake operation timed out',),))**
[download] Downloading video 58 of 481
[twitch:vod] 460759684: Downloading stream metadata GraphQL
[twitch:vod] 460759684: Downloading video access token GraphQL
[twitch:vod] 460759684: Downloading m3u8 information
[download] July 9, 2019 # 352 has already been recorded in archive
[download] Downloading video 59 of 481
[twitch:vod] 460757138: Downloading stream metadata GraphQL
[twitch:vod] 460757138: Downloading video access token GraphQL
[twitch:vod] 460757138: Downloading m3u8 information
**ERROR: Failed to download m3u8 information: <urlopen error _ssl.c:835: The handshake operation timed out> (caused by URLError(timeout('_ssl.c:835: The handshake operation timed out',),))**

Description

Twitch seems to fail on public vods. I put in the channel in the list to download (https://www.twitch.tv/maryjleeee/videos/all) but while youtube downloads work, these Twitch downloads do not.

It seems to work for me. Some network problems at your end? Cat sat on the router, say?

I don't think so, other videos (from youtube) are downloading just fine.


deleteOutdatedTCString: wrong timestamp parsing

Hi,
Starting from May 3, 2024, the consent window began to be shown every time our game starts. According to the logs, the deleteOutdatedTCString() method is triggered. It seems that now the timestamp is parsed incorrectly for some reason. Relevant only for new installs or reinstalls.
Perhaps the TCF String format has changed somehow?
2024/05/07 16:34:38.982 8842 8842 Info GoogleUMP GoogleUMP::Initialize
2024/05/07 16:34:38.983 8842 8842 Info GoogleUserMessagingPlatform::GDRPHelper GoogleUserMessagingPlatform::GDRPHelper:: deleteOutdatedTCString now =<PHONE_NUMBER>983 - timestamp =<PHONE_NUMBER>200 - daysAgo = 1223
2024/05/07 16:34:38.984 8842 8842 Info GoogleUMP GoogleUMP::deleted outdated TCF String

Thanks for the report, I'll try to see if there are changes in the TCF documentation about it.
I'll try to find if Google UMP take into account the outdated string now, as last time I had the problem was in the version 1.0, maybe they now handle outdated TCF datas correctly now.

After some reading of the Google documentation, I found:

If a user has already made a consent decision, Google's consent management solution won't request to gather new consent until the TC string is expired or otherwise becomes invalid.

So, it seems that google UMP now takes care of the expired TCF strings.
I will publish an update without this old fix.


[REQUEST] PerChannel Setting

Problem
Current implementation has not supported Perchannel Quantization. Would consider add it into config?
Solution
Implements per-channel quantization parameters in conversion/qparams.py and optimizes GEMM computation through deferred scaling, where column-wise scaling factors are applied to the final accumulation results to minimize arithmetic operations and improve computational efficiency.
Alternatives
No response
Explanation
For higher than 4 bit, the accuracy is nearly lossless with per-channel quantization. It would be efficient for inference if the per-channel setting got support.
Examples
No response
Additional context
No response
Acknowledgements

[X] I have looked for similar requests before submitting this one.
[X] I understand that the developers have lives and my issue will be answered when possible.
[X] I understand the developers of this program are human, and I will make my requests politely.

Could you elaborate? EXL2 already has one FP16 scale per output channel, as well as a 4-bit scale per group of weights.


Example of loading a github frictionless dataset and rendering it

e.g. https://github.com/datasets/finance-vix

@rufuspollock @anuveyatsu does PR #624 fix resolve this as well?

@virgoaugustine no, this is quite different (and this issue is under specified).

@luccasmmg can you close this - either a duplicate or as done with link to example repo.

FIXED In this example https://github.com/datopian/portaljs/tree/main/examples/github-backed-catalog


PyPI name

Reserve name https://pypi.org/project/pipstrap/ to prevent namejaking?

Good idea!

Nobody should be using pipstrap anymore anyway.


Create Blog “why-you-should-avoid-using-the-switch-statement-in-javascript-and-what-to-use-instead”

Automatically generated by Netlify CMS

👷 Deploy Preview for astounding-pika-bfabf8 processing.

Name
Link

🔨 Latest commit
191cbc73bfcbb3ec879114d9692433e029b514c6

🔍 Latest deploy log
https://app.netlify.com/sites/astounding-pika-bfabf8/deploys/66c9185f36feef000807fe70


Set up dependency scanning

Related to #2

Set up security scanning of dependencies (at least the Docker image and npm). Ideally these are scanned for all changes, as well as periodically for the HEAD of development as well as the latest released version.

[...], as well as periodically for the HEAD of development as well as the latest released version.

This was resolved in 0c34ccaeb444c8c6e4bdb6faefaee9a0db725bfa


Fix use of scaling in vispy points

References and relevant issues
In #6025, the vispy update broke the efforts from #5312. This is actually because the changes in vispy apparently do not handle the scaling kwarg properly at initialization. We can fix it here by not passing it.
I can't believe this has been wrong for so long, and I somehow never connected this issue that kept bugging me with that PR... but here we are.
Description
Try the following on main and here and move the camera around:
import napari
import numpy as np
v = napari.Viewer(ndisplay=3)
pl = v.add_points(np.array([[1, 1, 1], [10, 10, 10]]), scale=[1, 5, 5])
pv = v.window._qt_viewer.canvas.layer_to_visual[pl]

This fixes two things:

point sizes being all messed up by scaling
the anisotropic scale rotation wobblyness you see above

This returns to the correct, pre-#6025 situation.

This is what I see on main with the code above (fresh env, ensure vispy's up to date):
https://github.com/napari/napari/assets/23482191/80917a02-2e28-4fee-8b42-2fce84f3444b

This is what I see on main with the code above (fresh env, ensure vispy's up to date):

My env was a fresh napari dev install. But I was able to reproduce now, but only when I'm zoomed out.

Should be good to go then :)


Fix test build when VS is not installed on the root drive

BuildTestTools.cmd currently fails if Visual Studio is installed in an odd place. The environment variable  VS120COMNTOOLS should have the correct path.

Hi @braden, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

Looks good. Can you add the corresponding entry for VS140COMNTOOLS?

Updated


Missing user.conf after installation/run

Hello @bluppfisk!
Could you please clarify where I can find user.conf, app work as expected but can't figure out how to save my preferences/assets after reboot needs to set up all again.
Thanks in advance!

Hi. It should be in the project root folder (where run.sh is). I suspect you can just create one. If not, add something like below and store it there as user.conf.
max_decimals: 4
plugins:
- binance: true
- bitfinex: true
- bitstamp: true
- bittrex: true
- cexio: true
- gdax: true
- gemini: true
- hitbtc: true
- kraken: true
- okcoin: true
- poloniex: true
- unocoin: true
recent:
significant_digits: 2
tickers:
- asset_pair: XXBTZEUR
  default_label: cur
  exchange: kraken
  refresh: 3

Hi. It should be in the project root folder (where run.sh is). I suspect you can just create one. If not, add something like below and store it there as user.conf.
max_decimals: 4
plugins:
- binance: true
- bitfinex: true
- bitstamp: true
- bittrex: true
- cexio: true
- gdax: true
- gemini: true
- hitbtc: true
- kraken: true
- okcoin: true
- poloniex: true
- unocoin: true
recent:
significant_digits: 2
tickers:
- asset_pair: XXBTZEUR
  default_label: cur
  exchange: kraken
  refresh: 3

Made me hardcode preferences, and this works! But found that:
(coinprice-indicator) vt@wlp:~/coinprice-indicator$ pygame 1.9.6 Hello from the pygame community. https://www.pygame.org/contribute.html Coin Price Indicator v1.9.9 running! WARNING:root:Asset Discovery: API server returned an error: 301 WARNING:root:Asset Discovery: Expecting value: line 1 column 1 (char 0)
Appear when trying to Discover Assets click button. Just FYI, that supposed to be an issue for why user.conf didn't create.
Thank you!

Made me hardcode preferences, and this works! But found that:
(coinprice-indicator) vt@wlp:~/coinprice-indicator$ pygame 1.9.6 Hello from the pygame community. https://www.pygame.org/contribute.html Coin Price Indicator v1.9.9 running! WARNING:root:Asset Discovery: API server returned an error: 301 WARNING:root:Asset Discovery: Expecting value: line 1 column 1 (char 0)
Appear when trying to Discover Assets click button. Just FYI, that supposed to be an issue for why user.conf didn't create.
Thank you!

Try running the asset discovery and see if this happens still afterwards :)

Try running the asset discovery and see if this happens still afterwards :)

closing for now

closing for now


Better contrast for yellow

Thanks for waiting so long for a response. I was on holiday in the meantime and didn't think someone would actually submit a PR :)


[$500]  iOS - Video - User needs to tap on play button twice for the video to restart

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.44-0
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4345661
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:
Pre-requisite: the user must be logged in

Go to any chat
Send a video
Tap on the video preview
Tap on the full screen button
Wait for the video to end
Tap on the play button

Expected Result:
The video should be restarted after tapping on the play button one time
Actual Result:
The user needs to tap on the play button twice for the video to restart
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[x] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/d934eab1-364a-457d-b2d9-2c02ae274439
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01b0749f5a1ec15cd0
Upwork Job ID:<PHONE_NUMBER>726202368
Last Price Increase: 2024-02-27

We think that this bug might be related to #vip-vsp
CC @quinthar

cc @francoisl

Looking at PRs today, will check this tomorrow.

Proposal
Please re-state the problem that we are trying to solve in this issue.
User needs to tap on play button twice for the video to restart on iOS.
What is the root cause of that problem?
iOS logic for checking shouldReplayVideo has an issue.
https://github.com/Expensify/App/blob/dcf2e3e1ceebc8e598e6b51b61ebf76e297a2696/src/components/VideoPlayer/shouldReplayVideo.ios.ts#L6-L11
What changes do you think we should make in order to solve the problem?
Update shouldReplayVideo method for iOS and remove the e.isPlaying check.

yeah I think e.isPlaying check is unnecessary. Good to use @ShridharGoel 's proposal.
But before we formally accept, I want to check with @tienifr @jjcoffee It was added in this PR. Do you think it would be break anything if we remove e.isPlaying check.
🎀 👀 🎀 C+ reviewed.

@mananjadhav I would like to update my proposal. We should change the implementation to this (tested the below code):
export default function shouldReplayVideo(e: AVPlaybackStatusSuccess, isPlaying: boolean, duration: number, position: number): boolean {
    if (!isPlaying && e.isPlaying && duration === position && !e.didJustFinish) {
        return true;
    }
    return false;
}

@ShridharGoel With the updated version. I am not confident if we've covered the exact root cause. Could you please share some more information related to the documentation/some reference that shows this is the correct solution.

Thanks for the bump, I'll check tomorrow.

@ShridharGoel I saw your proposal. Few questions:

This is only an issue on iOS, then why do need to update the BasePlayer.  How will it impact Android?
I am not sure about false here. Because even if you don't pass fallback, e.isPlaying can be false/undefined, which still ends up being !e.isPlaying && e.isPlaying.
I still don't understand the root cause from the proposal on why is it a problem on iOS only and not Android.

Thanks for the clarification. That makes sense. The updated proposal here looks good to me. Works well as well.
🎀 👀 🎀 C+ reviewed.

I am not sure if there exists any test for video replay. If it doesn't then we don't need to add a new one

Can you ask if we have one in expensify-open-source? If we don't, that's a good reason to add one imo

Invited you for Upwork payment here https://www.upwork.com/jobs/~0184c7f8e7c24fbbc1 @ShridharGoel

$500 approved for @mananjadhav.

On hold for payment, not overdue

Paid and contract ended


Group tags with name and description

This PR

Enforces that tags are displayed after the display name and description but before all other properties for code snippets and runtime images. See next comment for motivation why a new category source was introduced.
Closes https://github.com/elyra-ai/elyra/issues/1334
Adds a placeholder for the runtime image image_name property. For the placeholder to be rendered, https://github.com/elyra-ai/elyra/pull/1293 is required.

New arrangements:

Developer's Certificate of Origin 1.1
   By making a contribution to this project, I certify that:

   (a) The contribution was created in whole or in part by me and I
       have the right to submit it under the Apache License 2.0; or

   (b) The contribution is based upon previous work that, to the best
       of my knowledge, is covered under an appropriate open source
       license and I have the right under that license to submit that
       work with modifications, whether created in whole or in part
       by me, under the same open source license (unless I am
       permitted to submit under a different license), as indicated
       in the file; or

   (c) The contribution was provided directly to me by some other
       person who certified (a), (b) or (c) and I have not modified
       it.

   (d) I understand and agree that this project and the contribution
       are public and that a record of the contribution (including all
       personal information I submit with it, including my sign-off) is
       maintained indefinitely and may be redistributed consistent with
       this project or the open source license(s) involved.

This is what the metadata would be rendered like without the introduction of the source category. Note the placement, which doesn't cleanly separate metadata metadata (display name, description, tags) and metadata:

Interesting.  In thinking about ordinals for this, it strikes me that if we do add a uihint to address this, it should probably be two-dimensional, so that you can express that something should be on a row by itself.  More like a coordinate (e.g., (1,1) = first row, first item; (3,2) = third row, second item; etc.)
Also, this implies a category named "metadata" that perhaps could be treated specially in that it doesn't trigger a group label and consists entirely of name, description, and tags.  That way, those three items would always precede the instance's application properties.


Refactoring owlapi mapper

from owlapy.owl_ontology_manager import SyncOntologyManager
from owlapy.render import owl_expression_to_dl
ontology_path = "KGs/Family/father.owl"
onto = SyncOntologyManager().load_ontology(ontology_path)
for owl_subclass_of_axiom in onto.get_tbox_axioms():
    print(owl_subclass_of_axiom)
    print(owl_expression_to_dl(owl_subclass_of_axiom))

from owlapy.class_expression import OWLClass, OWLObjectIntersectionOf, OWLObjectSomeValuesFrom
from owlapy.owl_property import OWLObjectProperty
from owlapy import owl_expression_to_sparql, owl_expression_to_dl
from owlapy.owl_ontology_manager import OntologyManager
from owlapy.owl_axiom import OWLDeclarationAxiom, OWLClassAssertionAxiom
from owlapy.owl_individual import OWLNamedIndividual, IRI
from owlapy.static_funcs import create_ontology
# Using owl classes to create a complex class expression
male = OWLClass("http://example.com/society#male")
hasChild = OWLObjectProperty("http://example.com/society#hasChild")
hasChild_male = OWLObjectSomeValuesFrom(hasChild, male)
teacher = OWLClass("http://example.com/society#teacher")
teacher_that_hasChild_male = OWLObjectIntersectionOf([hasChild_male, teacher])

# Create an ontology via ontology manager directly
ontology = create_ontology("file:/example_ontology.owl",with_owlapi=True)
john = OWLNamedIndividual("http://example.com/society#john")
male_declaration_axiom = OWLDeclarationAxiom(male)
hasChild_declaration_axiom = OWLDeclarationAxiom(hasChild)
john_declaration_axiom = OWLDeclarationAxiom(john)
john_a_male_assertion_axiom = OWLClassAssertionAxiom(john, male)
axioms=[male_declaration_axiom, hasChild_declaration_axiom, john_declaration_axiom, john_a_male_assertion_axiom]
ontology.add_axiom(axioms)

leads to the following error:
Traceback (most recent call last):
  File "/home/cdemir/Desktop/Softwares/owlapy/asd.py", line 23, in <module>
    ontology.add_axiom(axioms)
  File "/home/cdemir/Desktop/Softwares/owlapy/owlapy/owl_ontology.py", line 1103, in add_axiom
    self.owlapi_ontology.addAxioms(self.mapper.map_(axiom))
  File "/home/cdemir/anaconda3/envs/temp_owlapy/lib/python3.10/functools.py", line 926, in _method
    return method.__get__(obj, cls)(*args, **kwargs)
  File "/home/cdemir/Desktop/Softwares/owlapy/owlapy/owlapi_mapper.py", line 509, in _
    java_list.add(self.map_(item))
  File "/home/cdemir/anaconda3/envs/temp_owlapy/lib/python3.10/functools.py", line 926, in _method
    return method.__get__(obj, cls)(*args, **kwargs)
  File "/home/cdemir/Desktop/Softwares/owlapy/owlapy/owlapi_mapper.py", line 291, in _
    return OWLDeclarationAxiomImpl(self.map_(e.get_entity()), self.map_(e.annotations()))
  File "/home/cdemir/anaconda3/envs/temp_owlapy/lib/python3.10/functools.py", line 926, in _method
    return method.__get__(obj, cls)(*args, **kwargs)
  File "/home/cdemir/Desktop/Softwares/owlapy/owlapy/owlapi_mapper.py", line 156, in _
    return self.parser.parse(owl_expression_to_manchester(e))
org.semanticweb.owlapi.manchestersyntax.renderer.org.semanticweb.owlapi.manchestersyntax.renderer.ParserException: org.semanticweb.owlapi.manchestersyntax.renderer.ParserException: Encountered male at line 1 column 1. Expected one of:
	Class name
	Object property name
	Data property name
	inverse
	not
	(
	{

The error cause by owl_expression_to_manchester(e)
for i in axioms:
    print(owl_expression_to_manchester(i))

Traceback (most recent call last):
  File "/home/cdemir/Desktop/Softwares/owlapy/asd.py", line 26, in <module>
    print(owl_expression_to_manchester(i))
  File "/home/cdemir/Desktop/Softwares/owlapy/owlapy/render.py", line 538, in owl_expression_to_manchester
    return ManchesterRenderer.render(o)
  File "/home/cdemir/anaconda3/envs/temp_owlapy/lib/python3.10/functools.py", line 926, in _method
    return method.__get__(obj, cls)(*args, **kwargs)
  File "/home/cdemir/Desktop/Softwares/owlapy/owlapy/render.py", line 379, in render
    raise NotImplementedError
NotImplementedError

Suggestions: We need to add tests in our code.

Summary:
from owlapy.owl_ontology_manager import SyncOntologyManager
ontology=SyncOntologyManager().create_ontology("file:/example_ontology.owl")
for owl_subclass_of_axiom in SyncOntologyManager().load_ontology("KGs/Family/father.owl").get_tbox_axioms():
    ontology.add_axiom(owl_subclass_of_axiom)

Do you think that you can fix it @alkidbaci ?

So it seems like ManchesterOWLSyntaxClassExpressionParser of owlapi cannot parse classes that don't exist in the ontology. So when you want to add a new class through OWLDeclarationAxiom it tries to parse it to owlapi format before adding it. Its like a chicken-egg problem. A solution for this would be that the mapper per should map atomic classes without using the parser.

The problem that still prevails is when adding an axiom which contain some kind of complex class expression, normally the ontology add_axiom method will automatically add the missing classes but in our case that is restricted by the fact that the Manchester parser needs the class to exist in the ontology before parsing.
Now the solutions for this would be:

To implement mapping function for each distinct type of class expression (and stop using the Manchester parser for it)
Implement a dynamic entity checker (an owlapi class which is used as an argument to a Manchester parser) that does not check for existing entities. I'm working on this second point and so far i have implemented the class, seems to work with complex class expressions but there are some unexpected hidden "issues" (a class is added twice in the ontology) which I noticed and I'm trying to fix.

Thank you for the analysis of the issue❤


Support for WWW-Authenticate headers which does not strictly follow RFC 2617

OkHttp expects WWW-Authenticate header to have realm parameter going right after Digest token in HttpHeaders.parseChallenges(Headers responseHeaders, String challengeHeader) method, e.g.
WWW-Authenticate: Digest realm="DS-2CD2432F-IW", nonce="4d6a6c4551305a43515449364e6d497a4d6a63314e546b3d", qop="auth", stale="FALSE"
However some web servers (e.g. widely used Hikvision IP cameras) can send parameters in different order, e.g.
WWW-Authenticate: Digest qop="auth", realm="DS-2CD2432F-IW", nonce="4d6a6c4551305a43515449364e6d497a4d6a63314e546b3d", stale="FALSE"
This situation is not handled properly in OkHttp.
UNIT TEST:
`
private static void test() {
    // Strict RFC 2617 header
    Headers headers = new Headers.Builder()
            .add("WWW-Authenticate", "Digest realm=\"DS-2CD2432F-IW\", nonce=\"4d6a6c4551305a43515449364e6d497a4d6a63314e546b3d\", qop=\"auth\", stale=\"FALSE\"").build();

    Assert.assertTrue(HttpHeaders.parseChallenges(headers, "WWW-Authenticate").size() == 1);

    // Not strict RFC 2617 header. Generates error in OkHttp 3.4.1.
    headers = new Headers.Builder()
            .add("WWW-Authenticate", "Digest qop=\"auth\", realm=\"DS-2CD2432F-IW\", nonce=\"4d6a6c4551305a43515449364e6d497a4d6a63314e546b3d\", stale=\"FALSE\"").build();
    Assert.assertTrue(HttpHeaders.parseChallenges(headers, "WWW-Authenticate").size() == 1);
}

`

Got the same issue with widely used IP Webcam app version 1.12.4r
https://play.google.com/store/apps/details?id=com.pas.webcam
GET /shot.jpg HTTP/1.1
User-Agent: Android/6.0 (6.0 - 5X) com.alexvas.dvr.pro
Host: <IP_ADDRESS>:8080
Connection: Keep-Alive
Accept-Encoding: gzip
HTTP/1.1 401 Unauthorized
Content-Length: 0
WWW-Authenticate: Digest qop="auth", realm="IP Webcam", nonce="1470841389"

Please see also #2772, where I have linked working code which parses WWW-Authenticate parameters (named as well as anonymous ones).

HttpHeaders.parseChallenges is basically broken, I come across a @TODO stub in it, saying:

This currently only handles schemes with a 'realm' parameter;
It needs to be fixed to handle any scheme and any parameters
http://code.google.com/p/android/issues/detail?id=11140

https://github.com/rburgst/okhttp-digest uses response.challenges() which uses HttpHeaders.parseChallenges() to get scheme via Challenge instance. If realm=value is not the first parameter, Challenge is not returned and okhttp-digest does not work.

https://github.com/rburgst/okhttp-digest uses response.challenges() which uses HttpHeaders.parseChallenges() to get scheme via Challenge instance.

Are you sure it's okhttp-digest bothering you?

If realm=value is not the first parameter, Challenge is not returned and okhttp-digest does not work.

The order should be the very least of your worries , since Realm is the only parameter being parsed,

Check this line
https://github.com/rburgst/okhttp-digest/blob/master/src/main/java/com/burgstaller/okhttp/DispatchingAuthenticator.java#L35
What I'm trying to say is that if realm cannot be parsed in HttpHeaders.parseChallenges(), scheme is not returned.

Okay, I see, seems someone is working on it. Good luck!

No, nobody is working. That is an issue in OkHttp.
Check unit tests submitted in first post.

That's sad, there're two ways to solve your problems, either to

implement #2772 proposal(closed, at this point) to fix HttpHeaders.parseChallenges, but the downside is unnecessary overhead as  WWW-Authenticate header get parsed two times.
or create a new issue under okhttp-digest, for Response.challenges is not usable for parsing Digest scheme.

Why it should be okhttp-digest issue? The problem is that HttpHeaders.parseChallenges() method in OkHttp does not properly parses realm value in WWW-Authenticate header.
But yes, implementing #2772 will fix the issue.

Fixed with https://github.com/square/okhttp/pull/4252


Upload UX Improvements

Task checklist (roughly):

[x] Add status bar message for "already uploaded" case
[x] If local and gist is different and lastLocalDownload < lastGistUpload, display popup dialog to ask for one-time force update

[x] Remove "Don't show again" UX and code
[x] Change dialog box to "The gist settings changed since you last downloaded them.  Do you want to upload your current local settings to the gist anyway?"  Single yes/no button.

[x] If local and gist is different and lastLocalDownload >= lastGistUpload, upload with only a status bar message
[x] NLS for new dialogs and messages
[x] Update README doc (reverted to original)
[x] Update wiki

Short description of what this resolves:

Improve the upload user experience

Detect if gist has changes that are not on the user's machine and prompt for overwrite
Remove popup option to turn on forceUpload and "Don't show again"

Changes proposed in this pull request:

Streamline Upload UX

Remove "Don't show this dialog again" feature and setting
Change "Gist is newer than what you downloaded" dialog to a simple one-time force upload button

Fixes: #1035
How Has This Been Tested?

Since only the Upload logic was changed, it was necessary to test only upload features

Test the simple case of the user starting with synced settings and uploading without a change.

[x] Should see status bar message saying settings already uploaded

Test the simple case of the user starting with synced settings and uploading a change.

[x] No dialog popup and status bar indicates the successful upload

Modify the gist with another client and try to upload changes from the client under test.  Expect to see a prompt asking if the user wants a forced upload.

[x] Verify that selecting Yes proceeds with the upload
[x] Verify that selecting No or closing the dialog does nothing and show canceled status msg

Verify that automatic upload on setting change still works.

[x] Start with synced settings and make a change - should complete normally
[x] Push a change from another host and make a change - should get prompt to overwrite

Upload after reset

[x] Reset, repeat test 1
[x] Reset, repeat test 2
[x] Reset, repeat test 3

Upload with forceUpload=true, All should do an upload

[x] Reset, repeat test 1
[x] Reset, repeat test 2
[x] Reset, repeat test 3 (expect no dialog)

New Gist: Delete gist, reset settings, and perform an upload

[x] New gist should be created and upload completes

Screenshots (if appropriate):
n/a
Checklist:

[x] I have read the contribution guidelines.
[x] My change requires a change to the documentation and GitHub Wiki.
[x] I have updated the documentation and Wiki accordingly.

@karl-lunarg
When are you expecting to complete this PR.

When are you expecting to complete this PR.

I think I am finished.  It would be good to get a few people to test.

cmd.updateSettings.info.gotLatestVersion in package.nls.zh-tw.json is simplified chinese. It should be added in package.nls.zh-cn.json

cmd.updateSettings.info.gotLatestVersion in package.nls.zh-tw.json is simplified chinese. It should be added in package.nls.zh-cn.json

Sorry, but it looks like Google Translate gave me the same translation for simplified and traditional.  Could you please supply the correct strings for both of these?

@karl-lunarg
Before running manual tests.
I have pulled PR, run the following scenario:

Reset The extension settings
Reload Code
Upload and Get Token and Gist from GitHub
Hit upload

Upload isnt working, instead its showing the dialog "Sync: The settings in the gist have changed since ..."

Upload isnt working, instead its showing the dialog "Sync: The settings in the gist have changed since ..."

This was a bug in the call to the IsGistNewer github service function.
After a reset, customSettings.lastDownload is null.  The call to IsGistNewer converts the customSettings.lastDownload null value to a Date, which results in Jan 1 1970.   Later, inside of IsGistNewer, the code checks for a null localLastDownload by using !localLastDownload which  evaluates to false in this case because Jan 1 1970 is a valid date.   Then the gistLastUpload > localLastDownload check always is true when the localLastDownload  is Jan 1 1970.
This caused the upload code to think that the gist is newer and so it showed the dialog.
customSettings.lastDownload is already a Date, so there is no need to convert it to a Date when calling IsGistNewer.  Further, converting it is harmful because it breaks the null check in IsGistNewer.
Fix is to not do the conversion.  I'll push a fix for this.

@karl-lunarg
Exactly! Thank you for a detailed explanation. Fantastic Work

How Has This Been Tested?

I hope you have tested all the possible scenarios you have mentioned.  (Ref : https://github.com/shanalikhan/code-settings-sync/issues/1035#issuecomment-528342595 , https://github.com/shanalikhan/code-settings-sync/issues/1035#issuecomment-528388545 and above scenarios)
I have tested it and it looks fine but as Settings Sync dont have proper test cases defined since start and i always miss that. I just want to know are you available to fix if somethings comes up after release regarding this feature ? So i can release hitfix for it if somethings happens.

In these recent commits, I refactored the two commits that contained the related changes for moving the download date conversion to inside ofisGistNewer() into one commit since it makes better sense and is more bisectable.
I also added a commit to remove trailing whitespace so that tslint runs clean.

I hope you have tested all the possible scenarios you have mentioned

I did pretty thorough testing after the main changes, but I want to do them all again and more after these more recent minor changes.  I'll do that in the next day or so.  This project is difficult to test because of all the possible cases and the difficulty in setting up some of them.

I just want to know are you available to fix if somethings comes up after release regarding this feature ?

I use vscode and this extension a lot (thanks for it, btw), so I have some interest in keeping it in good shape, so I'll try to pay attention to the repo.  I have improved the code a bit by making it simpler, fixing a few bugs, and adding some comments to help make the logic clearer.  So I'm hoping that you and/or the other contributors to this project who know a lot more than me about all this can take the first crack at any problems.  I'll help out as needed.

I use vscode and this extension on several dev machines.   On one of them, I can't seem to test the extension using the Extension Development Host, the second instance of vscode that pops up when you debug the extension from the primary instance.  The extension under test fails to load in the EDH and I get messages in the Debug Console saying "ReferenceError: navigator is not defined".  I've searched all over and can't find a solution.  One of my other dev machines is set up with the same OS, etc and does not have this problem.  It feels like I'm missing an npm component or something.   I do have everything working on the other machine, so I don't think it is a problem with the extension.  But it would be nice to solve the problem on the broken machine so I can work on this more easily.
Any ideas?

I did pretty thorough testing after the main changes, but I want to do them all again and more after these more recent minor changes. I'll do that in the next day or so.

Perfect. Just let me know once your testing is completed and yes testing this extension is hard specially when you are already have dev. env. and using this extension for your env. for this i would suggest you to use vscode portable and setup that code for this extension only so you dont have to switch to extension debugging env. and your personal env. for work.
Yes there are actually alot of use cases that needs to be tested. I often sometimes miss but eventually complete the testing before releasing.

The extension under test fails to load in the EDH and I get messages in the Debug Console

Have you tried running vscode in Admin Mode, just make sure you are installing the npm components as an admin mode. If you want to install and validate the components you can try running these commands.
npm i --no-optional
npm dedupe
npm up

The extension under test fails to load in the EDH and I get messages in the Debug Console

I got my eyeballs on the two machines at the same time and found that I had just installed npm/nodejs differently and incorrectly on one of them.  I've reinstalled and I think I have it sorted.
I should have mentioned that this was on Linux.  I found this guide helpful.

Testing:
I've finished testing and updated the test plan I used up in the top comment.  Things look pretty good.
But there is one corner-case problem:

Start with local machine with settings synced to the gist
A different machine uploads a settings change to the gist
On local machine, reset settings
On local machine, upload settings

The settings will upload to the gist, overwriting the change made by the other machine, without prompting.
This is caused by the lack of a localLastDownload date after the reset.  Without this, I can't tell if the gist has changed since the last download.
One way out of this is to throw a new dialog message if the files are different and there is no localLastDownload date.  Something like "Settings Sync is unable to reconcile the differences between your local settings and your gist.  Consider downloading the settings or doing a forced upload now.  Do you want to upload your current local settings to the gist now?"  And there would be the same Yes/No buttons as the other dialog.
Another option would be to try to merge the above idea in the existing dialog, saying something like "The settings in the Gist have changed since you last downloaded them or you have reset your Sync Settings.  Do you want to upload your current local settings to the gist now?".
I sort of prefer the new dialog, since giving the user two reasons for the popup error is not very useful.
@shanalikhan Should we fix this for this release?   The exposure to these conditions might be pretty small, but there is unannounced data loss.  (Although, it can be recovered from the git gist).

one corner-case problem

Yes you are right, @karl-lunarg Settings Sync doesnt check for that case.

I sort of prefer the new dialog, since giving the user two reasons for the popup error is not very useful.

If you can fix that dialog, i would recommend to add that new dialog too in this PR so UX will be perfect handling all of the situations.
Instead of
"Settings Sync is unable to reconcile the differences between your local settings and your gist. Consider downloading the settings or doing a forced upload now. Do you want to upload your current local settings to the gist now?"
I would suggest:
"Upload will replace the older version of Settings in GitHub Gist. Consider downloading the settings or doing a force upload. Do you still want to upload forcefully ?"
Phrase can be improve furthermore.

@shanalikhan : I agree that it is good to fix this now.
I wasn't completely sure if you wanted the new dialog or not.  We have a lot more information when there is a valid last download date, so I felt a new dialog was better since the existing gistNewer message can be more definite about what changed.  So I added a new dialog with a message pretty close to your suggestion.
This adds more NLS entries.  @YunChaoTsai, I likely got the Chinese translations wrong again, so if you could please look at those in the last commit and use the GitHub suggestion mechanism to fix them, that would be great, just like you did last time.
I did the testing for this latest change and things look good.
I think I am done.  This seems to be a reasonable improvement over the current version.

Thanks @karl-lunarg for all the effort you put into this. Really appreciated!

@karl-lunarg
Thank you for you detailed effort and testing all of the possible scenarios. I will release the next version in few days.


Exporters linked to players do not work with whitelisted items

"I stashed a whitelisted entity into my inventory after linking myself to an exporter and still nothing happened."

Cannot reproduce.


Update free-programming-books-cs.md

a new link to a book added
it's free, license CC BY-ND 3.0 CZ
What does this PR do?
Add Resource(s) | Remove Resource(s) | Add info | Improve Repo
For resources
Description
Why is this valuable (or not)
How do we know it's really free?
For book lists, is it a book?
Checklist:

[x] Not a duplicate
[x] Included author(s) if appropriate
[x] Lists are in alphabetical order
[x] Needed indications added (PDF, access notes, under construction)

Excellent! Welcome to free-programming-books!


[improve][meta] Improve fault tolerance of blocking calls by supporting timeout

Motivation
We've got a lot of blocking calls without a timeout in the BookKeeper metadata plugin. In a bad situation, it will trigger some deadlocks.
Modifications

Improve the fault tolerance of blocking calls by supporting timeout.

Verifying this change

[x] Make sure that the change passes the CI checks.

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

@horizonzy Please help take a look, thanks.

Codecov Report

Merging #21028 (c0df801) into master (0cb1c78) will increase coverage by 39.58%.
Report is 6 commits behind head on master.
The diff coverage is 62.42%.

@@              Coverage Diff              @@
##             master   #21028       +/-   ##
=============================================
+ Coverage     33.56%   73.14%   +39.58%
- Complexity    12198    32267    +20069
=============================================
  Files          1621     1875      +254
  Lines        126970   139577    +12607
  Branches      13857    15344     +1487
=============================================
+ Hits          42618   102095    +59477
+ Misses        78748    29403    -49345
- Partials       5604     8079     +2475

Flag
Coverage Δ

inttests
24.23% <1.73%> (-0.01%)
:arrow_down:

systests
25.25% <0.57%> (?)

unittests
72.41% <62.42%> (+40.31%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed
Coverage Δ

...ookkeeper/PulsarLedgerUnderreplicationManager.java
51.09% <52.85%> (+51.09%)
:arrow_up:

...etadata/bookkeeper/PulsarLedgerManagerFactory.java
61.53% <57.14%> (+30.98%)
:arrow_up:

...ookkeeper/LongHierarchicalLedgerRangeIterator.java
81.66% <66.66%> (+81.66%)
:arrow_up:

...metadata/bookkeeper/PulsarRegistrationManager.java
62.56% <70.96%> (+53.16%)
:arrow_up:

...kkeeper/LegacyHierarchicalLedgerRangeIterator.java
72.15% <71.42%> (+72.15%)
:arrow_up:

...ulsar/metadata/bookkeeper/PulsarLayoutManager.java
66.66% <77.77%> (+46.66%)
:arrow_up:

...ar/metadata/bookkeeper/AbstractMetadataDriver.java
75.92% <100.00%> (+26.86%)
:arrow_up:

... and 1514 files with indirect coverage changes


[ServiceBus] prepare for February 2024

update release date in CHANGELOG

API change check
API changes are not detected in this pull request.


Get-DbaDatabaseFreespace - Divide by zero error encountered

Have  failure with Get-DbaDatabaseFreespace -SqlServer SQL777 | Format-Table
 WARNING: See error log *************\Documents\dbatools-exceptions.txt for more details.
 Unable to query SQL777 - REPORT_DEV
 At ************\Documents\WindowsPowerShell\Modules\dbatools\functions\Get-DbaDatabaseFreespace.ps1:195
 char:6
 +                     throw "Unable to query $($s) - $($db.name)"
 +                     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     + CategoryInfo          : OperationStopped: (Unable to query...REPORT_DEV:String) [], RuntimeException
     + FullyQualifiedErrorId : Unable to query SQL777 - REPORT_DEV

It looks like problem with
,CAST((FILEPROPERTY(f.name, 'SpaceUsed')/(f.size/1.0)) * 100 as DECIMAL(15,2)) as [PercentUsed]
in underline query.

in underline query and be fixed by
,ISNULL(CAST((FILEPROPERTY(f.name, 'SpaceUsed')/NULLIF((f.size/1.0),0)) * 100 as DECIMAL(15,2)),0) as [PercentUsed]
failure occurs on full-text index file.

IMHO the proposed fix isn't the best solution. Could you please run the following query on the offending database ?
from sys.database_files AS f```

Here you go
spaceused   size
----------- -----------
574448      684752
2232        395504
19048       20048
574448      0

(4 row(s) affected)

And according to BOL 0 should be expected

uh, gotcha, you're on 2005 ?
But it still needs a more careful review 'cause even for snapshots and FileStream the reported data is wrong.
I'll change the title a bit to reflect this and the other bugs

I am on
Microsoft SQL Server 2012 - 11.0.5058.0 (X64)  	May 14 2014 18:34:29  	Copyright (c) Microsoft Corporation 	Developer Edition (64-bit) on Windows NT 6.2 <X64> (Build 9200: )
but compatibility level for the database is SQL Server 2005

This was fixed by @scinobe in an earlier PR. @white-bee plesase let us know if you continue to encounter the issue in the newest release.


pubsub subscription filtering support

Signed-off-by: Iker Rejano<EMAIL_ADDRESS>

:rocket: merging this would avoid me having to maintain a fork.
CC/ @n3wscott

Thank you!!!


ipptool reports values from previous response when no response returned

If ipptool is used to run two tests, and the printer never responds to the IPP operation in the second test, ipptool will report the status and other operation response values from the most recent successful response. The values should be zeroed out so that a "successful-ok" isn't presented to the user when there isn't a response returned by the printer. This is very misleading to the uninitiated.
First discovered in IPP Everywhere Self-Certification v1.1 Update 4 and reported as istopwg/ippeveselfcert#87

Fix is in CUPS upstream, will get merged to ippsample when we do the next merge.


available containers for my system

I'm trying to create the following image:
FROM mcr.microsoft.com/windows/server:XXX

RUN powershell -Command \
    Install-WindowsFeature BranchCache -IncludeManagementTools ; \
    Remove-Item -Recurse C:\Windows\Logs\CBS

CMD [ "powershell" ]

But regardless what tag I'm trying to use (from the list available at https://hub.docker.com/_/microsoft-windows-server) , after
docker build -t image:v1

I always get either
a Windows version XXX-based image is incompatible with a 10.0.19044 host

or
no matching manifest for windows/amd64 10.0.19044 in the manifest list entries

If I use the mcr.microsoft.com/windows/servercore:ltsc2019 image, I get the following error
Install-WindowsFeature : The request to add or remove features on the
specified server failed.
Installation of one or more roles, role services, or features failed.
The source files could not be found.

Is this not meant to work or am I doing it wrong?

Can you please confirm you are using a Windows 10 host? That would explain the issue. Also, how did you get the other imagento run (docker pull and run command)?
I believe the binaries ot this feature were removed in Server Core as it's a Server role. You would have to use the Server image instead.

I can confirm that I'm using a windows 10 host (Version 21H2 (Build 19044..2728).
As you expected I got the other image running with docker pull mcr.microsoft.com/windows/servercore:ltsc2019. From then on I used docker compose with (docker-compose.yaml)
services:
  server:
    image: mcr.microsoft.com/windows/servercore:ltsc2019
    command: powershell ./app/script.ps1
    ports:
      - "8080:80"
    volumes:
      - .\serverfiles:c:\app
  client1:
    image: mcr.microsoft.com/windows/servercore:ltsc2019
    # Add any necessary environment variables or ports for the client
    volumes:
      - .\clientfiles:c:\scripts
    command: cmd /c "c:\\scripts\\script.bat"```

and then
docker compose up

On a different windows 10 machine I didn't need the pull, but was just including the part in the docker-compose.yaml file. What I did need to do, was switching to windows containers in docker desktop.
Is there any workaround to get the BranchCache Feature running for docker containers running on a windows 10 host?
I would like to test out how the peer 2 peer install of BITS performs. We have a very large client that needs to be installed on a large amount of machines that are often sitting behind networks that can access the internet only with small data rates.

Ok, so the problem here is that the Server image is based on Windows Server 2022, which is a higher version than Windows 10, hence the pull process fails.
Alternatively, can you please try the Windows image? docker pull mcr.microsoft.com/windows:ltsc2019
Please note this image might not have the server features available, I honestly don't know and haven't tested, but at least this one is compatible with Windows 10.

Ok great, the windows image is available and also allows to do "Enable-BCDistributed".
I don't know if this is a little too much out of scope, but it seems that "Enable-BCDistributed" needs a running firewall. But the firewall seems to be inactive and any attempt at changing that results in access denied error messages.
Is there a way to get the firewall running?

For Windows containers, you don't change the Firewall on the container itself, but instead you leverage the Firewall config from the host. Is the feature not working because of the firewall state inside the container?

When I try to execute
Enable-BCDistributed

inside the windows docker container via powershell, I get the following error:
Enable-BCDistributed : Could not access the Windows Firewall configuration.
2023-04-14 12:05:22 At line:1 char:1
2023-04-14 12:05:22 + Enable-BCDistributed
2023-04-14 12:05:22 + ~~~~~~~~~~~~~~~~~~~~
2023-04-14 12:05:22     + CategoryInfo          : NotSpecified: (MSFT_NetBranchCacheOrchestrator:r
2023-04-14 12:05:22    oot/StandardCi...cheOrchestrator) [Enable-BCDistributed], CimException
2023-04-14 12:05:22     + FullyQualifiedErrorId : HRESULT 0x800706d9,Enable-BCDistributed

I found some instructions that indicated that I should try to check whether the firewall is running by:
Get-Service -Name "MpsSvc"

But as a result I get:
Status   Name               DisplayName
------   ----               -----------
Stopped  MpsSvc             Windows Defender Firewall

When I try to start the firewall by
Start-Service -Name "MpsSvc"

I get the following error:
Start-Service : Service 'Windows Defender Firewall (MpsSvc)' cannot be started due to the following error: Cannot start service MpsSvc on computer '.'.
At line:1 char:1
+ Start-Service -Name "MpsSvc"
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : OpenError: (System.ServiceProcess.ServiceController:ServiceController) [Start-Service], ServiceCommandException
    + FullyQualifiedErrorId : CouldNotStartService,Microsoft.PowerShell.Commands.StartServiceCommand

any update on this?

I see you opened another issue which seems related to this one. Is that the case? I'll loop someone from the product team, but ant to reduce duplicates.

Sorry for the duplicate, I had the impression that the conversation went far away from the initial question (the heading of this issue is no more related to the late part of this discussion). Meaning: I had an issue with finding the right container (so that issue is actually already solved). Now I have an issue with the container itself (or how windows container in general work or how Microsoft tools on these containers may not work as intended).
Shall I mark this as solved and continue on the other issue?

Thanks, I'll go ahead and close this one so we can focus on the problem at hand.


Remove unnecessary migrations stuff

closes #360

But it completely breaks tests, I will fix them later.


aurbuild: default root/pool values

repose uses $PWD when -r or -p are not specified, however -r/-p without arguments fails.

I don't like the idea of having these things explicit, so I've made the -d, -r and -p options mandatory with bfa4cc2464e8d39e2061a2d19ce91cebad78a354.


Is there a repo with the tutorial.html?

I want to submit a fix to one of the examples. It's OK if there's no repo, the example is this one:
var does_end = String.ends_with("needle", "e")
does_end = "needle".ends_with(e)
does_end = "e" |> "needle".ends_with

I think the second version needs quotes around the e:
does_end = "needle".ends_with("e")

@radiofreejohn
Here you go https://github.com/jesserayadkins/lily/blob/gh-pages/tutorial.html :)

I didn't think to check the branches, thanks!


UnversionedPatch model is missing.

What's wrong
The UnversionedPatch model is missing.
How to reproduce
If you take a look at CoreV1Api.md#patch_namespace, you will see a reference to UnversionedPatch. Click it.
The docs don't exist for it. A quick look at kubernetes/client/models shows that the model itself is missing too.

Ack. The problem is missing such model in k8s spec itself. #98 fixed (hacked) patch operation for now.

hi, 'UnversionedPatch' model is not function now. I want to modify replica number using 'patch_namespaced_deployments_scale',but the 'UnversionedPatch' model is missing.
Please help me

Hi,
It is missing from the docs themselves. The way I'm using it in my current
code base is that I build the model using the usual constructors, and then
have a function turning it recursively into a dict representation. And I
use that to patch.
On Thu, 30 Mar 2017 05:59 TracyBin<EMAIL_ADDRESS>wrote:

hi, 'UnversionedPatch' model is not function now. I want to modify replica
number using 'patch_namespaced_deployments_scale',but the
'UnversionedPatch' model is missing.
Please help me
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kubernetes-incubator/client-python/issues/117#issuecomment-290295910,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAdUOrSy3RHiaeBXwZYx_40W8CpYxv6Fks5rqyiUgaJpZM4LvrQ9
.

--
Mikolaj Pawlikowski

UnversionedPatch is now object (fixed in preprocessing step). Thus it is not referenced anymore.


Background color due to ErrorMsg

I have the following entry in vimrc:
match ErrorMsg '\%>80v.\+'. Runing :Calendar shows, as expected, all columns after character #80 with another background color (in my case it is red).  How do I solve this without disabling the above instruction?

autocmd BufEnter,FileType *
      \   if &l:filetype ==# 'calendar'
      \ |   match
      \ | else
      \ |   match ErrorMsg '\%>80v.\+'
      \ | endif

Awesome!


Dynamic trees dont spawn when using Custom Cubic world generation setting

Having dynamic trees installed with this mod and selecting the "custom cubic" world type prevents dynamic trees from spawning it's trees instead of the vanilla ones. Dynamic trees do spawn when using normal world generation. These can also be grown from acorns etc. Only world generation does not agree

https://github.com/OpenCubicChunks/CubicDynamicTreesAddon is intended to solve it but it needs an update to be compatible with latest DT versions.

Updated and available on curse now.


Install and use cmake 3.10 on the agent machines

This might be best done with containers.
The agent images have 3.17 and 3.18. We want to support Ubuntu 18.04 (which has cmake 3.10.x installed by default). We need to land cmake 3.10.x on the machine and use it to build.
It won't help to make 3.17 or 3.18 behave as 3.10 because new features not in 3.10 will still work and we want the build to fail.

@danieljurek Please investigate the cost to fix this

Costing estimate: 5

On Linux/Windows it may be preferable to use container infrastructure to bypass all of the customization done to the DevOps hosted images.
On MacOS work will have to be done to install the desired version of CMake on the agent either side-by-side or instead of the installed version of CMake.

@danieljurek  has provided swag of 5 days for this work.    My understanding is that this will not be release blocking, so it can wait until he returns.  If you need it before then, we should discuss further.


MagicOnMob VerificationHandler

Addresses Issue https://github.com/apollo-rsps/apollo/issues/335 where a VerificationHandler for MagicOnMobMessage is requested. Verifies the following conditions:

The Mob exists (i.e. index is valid)
Type of the Mob matches the Mob at the index
The Mob is within distance of the player

The verification for combat related conditions are left for plugins to handle.

Codecov Report

Merging #426 into kotlin-experiments will decrease coverage by 0.03%.
The diff coverage is 0%.

@@                   Coverage Diff                    @@
##             kotlin-experiments     #426      +/-   ##
========================================================
- Coverage                 23.23%   23.19%   -0.04%
  Complexity                  811      811
========================================================
  Files                       643      644       +1
  Lines                     11123    11139      +16
  Branches                   1632     1636       +4
========================================================
  Hits                       2584     2584
- Misses                     8251     8267      +16
  Partials                    288      288

Impacted Files
Coverage Δ
Complexity Δ

...message/handler/MagicOnMobVerificationHandler.java
0% <0%> (ø)
0 <0> (?)

...rg/apollo/game/message/impl/MagicOnMobMessage.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

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 cd06a69...5d83a8f. Read the comment docs.

Merged in 1b3f1ab.


SVG stack overflow

I had been trying to use sprites to reduce the number of requests on a web page. My sprite consisted of SVGs of different sizes, using stack is gives white spaces due to the square view-box of parent SVG
stack.zip

Hey @AgrawalAmey,
so what exactly is the issue you'd like to file? All I see is expected behaviour ... As far as I understood SVG stacks, it's recommended that all shapes have the same size, so they are particularly useful for icon systems (if at all).
In any case, if you think you've spotted erroneous behaviour, please describe in detail what you'd expect and post your complete configuration.
Thanks & cheers, Joschi

Hi @jkphl,
Thanks for your response. I had been experimenting with says to substitute the symbol sprite so that the good old img element could be used. I tried running the process to create a stack for graphics of an  entire website, obviously the size of images are different. Running on default configuration with example set to true (Check out the attachment) I realized that though the view-box for every child SVG element
is defined independently, as the parent SVG element has a square defined only square images are rendered perfectly. Others receive unexpected white space. This might not be refereed to as a bug for the module. But any suggestion in order to fix this problem will be really helpful.
Thanks, Amey
svg-sprite.zip

@AgrawalAmey In your case (and probably any similar one) I'd definitely recommend using the symbol sprite instead of stacks. Use it just as you'd do with a stack (<img> without width and height attributes and with fragment identifiers, e.g. #10) and throw in a polyfill for older Internet Explorers if necessary. That should just work out of the box in most of the browsers. IMO there's almost no real use case for stacks.

@jkphl , Thanks for your suggestion. I had been just trying to explore options to substitute the use of <use> tag.

@AgrawalAmey Ah, sorry, I just realize that my suggestion was wrong: I wanted to recommend the view sprite, not the symbol sprite.


Allow direct call of helpers: shortcut to execute

Pull Request Test Coverage Report for Build 345

4 of 4 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.5%) to 99.271%

Totals

Change from base Build 336:
0.5%

Covered Lines:
1226

Relevant Lines:
1235

💛  - Coveralls


Local development including extensions using emulator suite

Bug report

Extension name: firestore-stripe-subscriptions

Describe the bug
For the development of new features we use the emulator suite to ensure that there is no impact on the production environment. As far as I know the function emulator does not support Firebase extension. It only emulates functions that are included in the local functions\index.js.
Now we would like to add the new supported one-time-payments to the application.
In the package.json there are commands to start the Firebase emulator but I am not sure how this should work:
https://github.com/stripe/stripe-firebase-extensions/blob/next/package.json
However, when executing the command "cd firestore-stripe-subscriptions/ && firebase ext:dev:emulators:start --test-params=../.env --project=stripe-subs-ext", several errors occur:

error: unknown option '--test-params'
-error: ext:dev:emulators:start is not a Firebase command

We need a way to develop and test the purchase flow locally in combination with the Stripe test environment (api keys, endpoints, webhooks).
Can you show me a way to achieve this? Is there a best practice here?
To Reproduce
Steps to reproduce the behavior, please provide code snippets or a repository:

Checkout https://github.com/stripe/stripe-firebase-extensions
Execute npm run dev:subs in project

Expected behavior
The emulator will be started and the functions will be initialized within the function emulator.
System information

OS: Windows 10
Browser Chrome

@Spitzbua those commands are currently only documented and supported for developers in the extension publishers’ early access program. However, you can probably figure out how to unlock them by poking around in the firebase/extensions repo 👀
I'd recommend:

Filing your feature request on the firebase-tools repo, which is where emulator commands are implemented, and
Joining the early access program to learn more about emulators + extensions


Improvement of OneNote plugin

Description of the new feature / enhancement
I would like to improve the current OneNote plugin.
The main goal would be to port many of the features from this OneNote plugin[^1] for Flow Launcher (an application that also uses Wox).
It should help with some of the points mentioned here #22153.
Outline of potential new features

View notes hierarchically (support for notebooks, section groups, sections and pages).
Quickly create new notebooks, section groups, sections and pages.
Search only in a specific hierarchy item.
View recent items.

[^1]: Full disclosure this is made by me! 😄.
Scenario when this would be used?
Ideally whenever you want to use OneNote!
Supporting information
Evidence would be the already created plugin.
For instance here is a gif of the "Notebook Explorer" feature:

The readme for repo also contains other examples of what the plugin can do.
This would also change the plugin dependency from Scipbe.Common.Office.OneNote to Linq2OneNote[^1].

Hi @Odotocodot. What would be your solution? 🙂

Hi @Jay-o-Way.
I would like to add as many of the features from the flow plugin as possible. Mainly these 5:

Notebook Explorer
Create New Items
Recent Pages
Scoped Search
Title Search

Current progress can be see here: https://github.com/microsoft/PowerToys/compare/main...Odotocodot:PowerToys:feature/improve-onenote-plugin
Still fixing bugs and I havent started on any localization yet, but most of the features have been implemented.

Hi, currently extremely busy so haven't been able to work on the branch.


Encapsulate EquivalenceClass into a struct

Which issue does this PR close?
related to https://github.com/apache/arrow-datafusion/pull/8006
Rationale for this change
As I proposed in https://github.com/apache/arrow-datafusion/pull/8006#discussion_r1380454715 I think making stucts rather than typedefs leads to easier to maintain code as the functionality is easier to find and it allows / encourages more documentation and encapsulation
This also was a good excuse for me to try and work with the code some
What changes are included in this PR?

Make EquivalenceClass a real structure rather than a typedef
Add docs

I actually think maybe the right thing to do is to call this thing PhysicalExprSet or something as it is more general than just equivalence classes
I plan to move all the functions into a new module rather than calling functions in utils as a follow on PR
Are these changes tested?
Existing tests

Are there any user-facing changes?
yes, since the APIs in question are pub this is technically a breaking API change. However, given they are all quite new (within this week) and not yet released, I don't think there will be as much direct impact.

I believe this PR is ready for reivew @ozankabak and @mustafasrepo  -- however, I believe it may cause non trivial merge conflicts with https://github.com/apache/arrow-datafusion/pull/8107. However, I think the style in this PR would make stuff like https://github.com/apache/arrow-datafusion/pull/8107 easier to implement 🤔

I implemented @ozankabak 's suggestion to treat EquivalenceClass actually like a class (rather than a vec with order). I think it is quite good now.

Any concerns with merging this @mustafasrepo ?

Any concerns with merging this @mustafasrepo ?

Not at all. Thanks @alamb as always for this PR.


Text is unreadable in "Light Mode"

Describe the bug
Text, especially hyperlink and placeholder text, is very hard to read in "light mode" in the demo page.
To Reproduce
Steps to reproduce the behavior:

Go to https://emilk.github.io/egui/index.html
Click on "Switch to light mode" button in the top left corner
Some problem happen to edge handling between text and background, rendering normal text extremely thin while text in other colour totally unreadable

Expected behavior
Proper text rendering. If alpha-blending handling of text is too hard to maintain well, don't use anti-alias in text rendering, as the monospace font is unaffected in the demo.
Screenshots

Desktop:

OS: Linux Mint 20.1
Browser: Firefox 87

Smartphone:

OS: Android 8.0
Browser: Chrome 89

Additional context
Because smartphone is running a hidpi screen (200%dpi in my phone browser), the bug is less prominent there. But the italic-grey text is still horrible and practically unreadable as shown in screenshot 3.

@emilk This is fixed by effd3c7440d9f70c0eff677871a60be539cf1113


Rubbish values from Node.source after editing tree

After editing a certain range of the tree using Tree.edit / Tree.edit_s, some calls to Node.source will return rubbish values.
This seems to only apply to nodes that were not affected by the edit. Nodes that did experience changes will return their source text correctly, while unaffected nodes return what I can only assume is junk.

Fixed via 9660f206


Feature Request: Linting rules apply only to new changes

Use case:
We need to onboard existing schemas that do not conform to our desired set of linting rules, but want to prevent any new non-compliant schema changes from ending up in our supergraph.
Proposed Solution:
We'd like to be able to optionally only run linting against any new changes to a target's schema.
The initial push may need to be a force push or as part of this setting, it may ignore the first push of a schema.
I'm not too familiar with Hive's codebase yet so I'm not sure if linting runs with compatibility or on push.
It may be nice to somehow show the existing linting errors as well so teams are aware of these issues, but they should not block compatibility/publishing.

Thank you @jdolle !

We need to onboard existing schemas that do not conform to our desired set of linting rules, but want to prevent any new non-compliant schema changes from ending up in our supergraph.

This is a very interesting idea, and we do want to explore the option to support this kind of linting process for the schema policy checks.
As a (very) hacky workaround, I can propose to use a warning in the meantime, and use the #eslint-ignore directives on the parts of the schema that you might want to ignore.

I'm not too familiar with Hive's codebase yet so I'm not sure if linting runs with compatibility or on push.

We are using ESLint under the hood, and running https://github.com/B2o5T/graphql-eslint . Theoretically, we can use any ESLint plugin to achieve this kind of functionality. As it's not supported as part of ESLint's core, @B2o5T suggested that maybe we can integrate https://www.npmjs.com/package/eslint-plugin-diff into the process (or a similar plugin), and then allow Hive users to run the linting process only for the changed files.


aplicando mvp

ejemplo de implementacion de clean architecture y mvp sobre arquitectura offline

Of course, leave this branch apart, everyone who sees the video wants to see the original code, this branch is going to be a guide on how to implement it with clean.

Implementation was improved to run a new merge.


Deployed and funded chequebook ERROR

Hi,
I installed bee-dashboard. In status tab, in node setup division only one unchecked. Deployed and funded chequebook. In details it says "Your chequebook is either not deployed or funded." What should i do?

HI @ahmeria , thank you for creating the issue. Which version of bee-dashboard and bee are you running? Do you have address of the chequebook or at least of your bee node eth address? With this information we could look at the blockchain if it is really deployed and funded.

Hello,
Versions:
Bee-dashboard : 0.7.0
Bee: 1.1.0
Addresses:
Chequebook: 0xF5be1283C534E137Ef97D6b5277f8C5B8AA9c0F1
Eth: 0xfb7948713f3083a090b3ce6f6ea80f2b44006ebb
Can you check it? Thanks
From: Vojtech Simetka @.***
Sent: Tuesday, August 31, 2021 6:10 PM
To: ethersphere/bee-dashboard @.>
Cc: ahmeria @.>; Mention @.***>
Subject: Re: [ethersphere/bee-dashboard] Deployed and funded chequebook ERROR (#193)
HI @ahmeria https://github.com/ahmeria  , thank you for creating the issue. Which version of bee-dashboard and bee are you running? Do you have address of the chequebook or at least of your bee node eth address? With this information we could look at the blockchain if it is really deployed and funded.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/ethersphere/bee-dashboard/issues/193#issuecomment-909326588 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AKME2CY7E5NMIKAZCEIQMD3T7TWCZANCNFSM5C3EGKHA .
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675  or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub .   https://github.com/notifications/beacon/AKME2CZN4XTY63ZTKCBRRKLT7TWCZA5CNFSM5C3EGKHKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOGYZTR7A.gif

So neither of these addresses has any BZZ balance = the error you are seeing is correct.
This is a chequebook with some token balance: https://blockscout.com/xdai/mainnet/address/0xb30347c3A847102fcbDb43B3F15D07a8aa51Ad49/tokens
This is yours: https://blockscout.com/xdai/mainnet/address/0xF5be1283C534E137Ef97D6b5277f8C5B8AA9c0F1/tokens
That being said, we currently don't support running bee in lightnode (which does not require chequebook nor xBZZ balance). Are you running bee node in lightnode? You can find out in the bee config file.


RemoveAll & MkdirAll to Filesystem interface

For some reason the original author chose to implement the equivalent of the underlying OS os.RemoveAll() and os.MkdirAll() as global vfs functions and left them out of the vfs.Filesystem interface.

[x] Move RemoveAll & MkdirAll to Filesystem interface

As a transition, the memfs & bucketfs MkdirAll and RemoveAll method implementations would actually be an alias to their respective vfs functions. For now that seems pretty sensible without code duplication.


Unable to use those plugins on assigned mailboxes with Full Access permissions.

This page does not provide the explanation that users assigned Full Access to other mailboxes may not be able to report messages within those mailboxes unless they are directly logged into them. Consider scenario where a PA is working CEOs emails and is unable to report the phishing ones... The expected functionality here would be that if user has full access (or read and manage) mailbox permissions, they should be able to report emails within that mailbox.

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 445892a6-5948-33e4-ce60-72670829d8ef
Version Independent ID: a7407a6f-6563-9a4b-4a95-7ed908ab322b
Content: Enable the Report Message or the Report Phishing add-ins
Content Source: microsoft-365/security/office-365-security/submissions-users-report-message-add-in-configure.md
Service: microsoft-365-security
Sub-service: mdo
GitHub Login: @Dansimp
Microsoft Alias: dansimp

Looking into this.

@WalterMccan could you please explain what the exact error is you are seeing here and how can I reproduce it?

@dhairyya There is no error. Outlook simply doesn't allow the user with full permissions to use the report add-in. The add-in is not showing at all and there is no ability to add it.
To reproduce:

Create 2 mailboxes. Assign full permissions of the first mailbox to another mailbox user.
Wait for Outlook to show the first mailbox for the second user and navigate to it.
Attempt to report a message within the second mailbox.

@WalterMccan if you are trying to report a message from shared and delegate mailbox please read the following MC post https://admin.microsoft.com/AdminPortal/Home?ref=MessageCenter/:/messages/MC553274
Let me know if you have any questions or doubt

@dhairyya Thank you for this informative post. It seems like the feature is only available for the OWA which is unfortunate as majority of people use the desktop version. Additionally this feature is not supported by Report Message add-in. I don't think it will work for us so we may look at some third party solutions or invest time in developing own add-in. Thank you for the information :)


We no longer need to install Bundler 1 globally

Since https://github.com/dependabot/dependabot-core/pull/5513 I think it's no longer needed.
For running updater, Bundler 1 is not used anyways, and for running native helpers, the Bundler 1 native helper installer installs Bundler 1 to an isolated location already.
Just opening a PR so that CI can prove me wrong.

Actually we should be okay to merge this now that https://github.com/dependabot/dependabot-core/pull/6492/ is merged, as this simply retains parity...


onChange not called by enter method

Are you reporting a bug?

Please create a Codepen that demonstrates your issue. You can start from the Basic example.

Provide the steps to reproduce the issue, e.g.:

Focus on the input field
Type c, and wait for suggestions to appear
Press Down
Neither Pressing Enter no clicking the item does trigger onChange with the related method of enter or click

Observed behaviour: onChange not triggered if value selected by enter method
Expected behaviour: onChange should be triggered if new value is selected by pressing enter key

Reason:
The actual value is already considered to have changed after Down was pressed so that the update check within maybeCallOnChange always fails:
https://github.com/moroshko/react-autosuggest/blob/d626cef720df0ed5a4c7df4d539836b86bda6afa/src/Autosuggest.js#L316-L322
Also the code that deals with Up, Down arrow keys should not updated the value, before being committed by Enter key press 🤔
https://github.com/moroshko/react-autosuggest/blob/d626cef720df0ed5a4c7df4d539836b86bda6afa/src/Autosuggest.js#L602-L653

This bug report seems to be wrong.
correct API to use should be onSuggestionSelected in favour of onChange


Add a policy for PDBM: invalidate build cache

Enable PDBMs to invalidate their build projects' cache
Build may fail if the cache is problematic (e.g. wrong / troublesome dependencies was used)
This feature is inspired from @justinusokky's feedback

HI @salvianreynaldi , thanks for the PR. Also thanks for the feedback @justinusokky .
I will get back to you soon..

From the discussion on #17
Let's postpone this change


H-Heaps

Challanges

Min Heap with insertion deletion
Max Heap with inserion deletion
Heap Sort
Heapify

You are allowed to make separate files if you want to make a concise contribution.

Hey, I am satyam and I want to work on this. Can u please allot me this?

please follow Contributing and Algorithms Guidelines , your submission must match these


Exact Permission for Connecting Log Analytics workspace

As per this doc it mentioned a permission called
Power BI workspace owner, or Log Analytics workspace owner . but there isn't any such role exist on Azure. What is the correct role required ? So far only the Owner role worked. But the owner is super admin kind of role and from the securit side we should not give and provide a lest permission as possible. So what is the exact permission is required in this case

Thanks for your comment, @jayendranarumugam. This issue has been assigned to the doc author for review.

Hi @jayendranarumugam, you might try checking out this article. Does this give you information you need? https://learn.microsoft.com/en-us/azure/azure-monitor/logs/log-powerbi

@paulinbar I dont think so, the link you are referring is still the same wordings Power BI workspace owner, or Log Analytics workspace owner  but there isn't any such role exist on Azure. What is the correct role required ? So far only the Owner role worked. But the owner is super admin kind of role and from the securit side we should not give and provide a lest permission as possible. So what is the exact permission is required in this case

Hi @jayendranarumugam, are you sure that the information you need isn't in either the Prerequisites section here: https://learn.microsoft.com/en-us/azure/azure-monitor/logs/log-powerbi#prerequisites or the Required permissions section that follows it? Or in the docs that those sections link to?


Fixed broken getting of body as a string

->getContents() only reads the stream from wherever the current position is - it does not rewind it. This can be hugely problematic if, for example, a logging middleware is attached to the client, which leaves the pointer not at the start of the stream/

@GrahamCampbell Any chance of having a test on this?


Annotate connectives in xrefs

Crossreferences autogenerate with internationalisation connectives and nesting of cross-reference text:
<xref target="note1"><location target="note1" connective="and"/><location target="note2" connective="and"/></xref>
can end up rendered as:
 <xref target="note1">Clause 2.1, Note 1</xref> and <xref target="note2">Clause 2.2, Note 1</xref>
      Clause 2.2, <xref target="note2">Note 1</xref> and <xref target="note3">Note 2</xref>

As we are semantically annotating cross-reference text out of https://github.com/metanorma/isodoc/issues/617, it would make sense to annotation the inserted "and" and "," in the foregoing, as spans of class fmt-xref-connective. Doing so would involve updating isodoc-i18n, since it is internationalised, so I will delay it.

Bringing this forward, since it's just going to make rewriting test cases quicker to do it now.


Prompt not updating after dir change.

[x] I have read through the manual page (man fzf)
[x] I have the latest version of fzf
[x] I have searched through the existing issues

Info

OS

[ ] Linux
[x] Mac OS X
[ ] Windows
[ ] Etc.

Shell

[ ] bash
[x] zsh
[ ] fish

Problem / Steps to reproduce
running this cd $(fd --type directory --min-depth 1 --prune | fzf --tac)
This brings up dir listing and I can choose whichever dir I want to goto, it works but..it does not update the prompt until I hit enter again.
note: I am using Starship for my prompt in iTerm.

found this on some obscure site, seems to have worked..at least for now
zle reset-prompt


unsupported CSI driver: rook-ceph.rbd.csi.ceph.com

What steps did you take and what happened:
[A clear and concise description of what the bug is.]
backup logs:
level=error msg="Error attempting to get volume ID for persistent volume" backup=velero/production-backup error="rpc error: code = Unknown desc = unsupported CSI driver: rook-ceph.rbd.csi.ceph.com" error.file="/home/regressionTest/go/src/github.com/AliyunContainerService/velero-plugin/velero-plugin-for-alibabacloud/volume_snapshotter.go:264" error.function=main.checkCSIVolumeDriver group=v1 logSource="pkg/backup/item_backupper.go:438" name=pvc-8b09731c-a57f-47d2-98bf-cf98d90ba6bd namespace= persistentVolume=pvc-8b09731c-a57f-47d2-98bf-cf98d90ba6bd resource=persistentvolumes volumeSnapshotLocation=default
What did you expect to happen:
create backup
Anything else you would like to add:
[Miscellaneous information that will assist in solving the issue.]
Environment:

Velero version: (use ark version):
Client:
Version: v1.5.2
Server:
Version: v1.2.0
Kubernetes version: (use kubectl version):
Client Version: version.Info{Major:"1", Minor:"18", GitVersion:"v1.18.6",
Server Version: version.Info{Major:"1", Minor:"18", GitVersion:"v1.18.6",
Kubernetes installer & version:
Cloud provider or hardware configuration:
OS (e.g. from /etc/os-release): Ubuntu 20.04.1 LTS

Hi, i have the same issue when I use velero 1.5.2 with ali cloud.
Client:
Version: v1.5.2
Server:
Version: v1.5.2
I am trying to create a backup and I get a log that is similar to what @Hcreak posted.
Can you please advise how to fix it?
Regards,
Nikita.

Hi,
This issue can be solved by adding the value snapshotVolumes: false to your schedule template in your values.yaml in velero chart.
For example:
schedules:
  mybackup:
    labels:
      myenv: foo
    annotations:
      myenv: foo
    schedule: "0 0 * * *"
    template:
      ttl: "240h"
      includedNamespaces:
      - foo
      **snapshotVolumes: false**


tweaking junit dependencies to avoid warnings at runtime in Intellij

tweaking junit dependencies to avoid warnings at runtime in Intellij


There's double enablement

It is reported that set the hosting startup environment at the same time call services.EnableServiceProfiler() leads to double enablement.
Needs to reproduce the issue and fix.

Fixed in 1.1.2-beta1: https://www.nuget.org/packages/Microsoft.ApplicationInsights.Profiler.AspNetCore/1.1.2-beta1


New replicator lost data between 2023-11-02 and 2023-11-06

My new replicator instance has been running 2023-10-23. This week I noticed that number of reactions/casts/messages considerably dropped between 2023-11-02 and 2023-11-06:
| Reactions | Messages | Casts
-- | -- | -- | --
2023-10-28 | 24690 | 75937 | 8371
2023-10-29 | 17822 | 59950 | 5755
2023-10-30 | 21989 | 63951 | 7003
2023-10-31 | 20454 | 87297 | 7417
2023-11-01 | 34044 | 92775 | 8465
2023-11-02 | 5 | 20345 | 804
2023-11-03 | 84 | 17711 | 1573
2023-11-04 | 19 | 17620 | 1024
2023-11-05 | 3 | 16266 | 1037
2023-11-06 | 2 | 21329 | 1366
2023-11-07 | 6159 | 37805 | 4139
2023-11-08 | 21773 | 62269 | 7974

Replicator uses nemes.farcaster.xyz:2283 hub.
After restarting the replicator, the data for that period was not backfilled.
There are no unexpected errors logged.
The replicator did not crash during that period.
Neynar database didn't experience such a drop during the period.

I'd be more than happy to provide more information and help to solve this issue. Just tell me what you need!

Sorry for the delay in response. At this time, the replicator has a number of issues and we don't recommend using it. Check out the Shuttle project if you're interested in this functionality.


Feature: Add a digestabot workflow and pin digests

:gift: This change adds a digestabot workflow to update digests daily, and pins several image names to digests in support of this.
This is WIP until https://github.com/chainguard-images/actions/pull/102 merges and I update the workflows to point to their source of truth
/kind feature

Ok, this is RFAL and gets us part of the way there, let's follow-up with EXTRA_ARGS stuff.