[sveltekit-pod] Add content for Blog

Add blogs content for framework
Background
As a visitor of the framework specific Framework.dev site I would like to have the ability to see enough Blog content to keep me informed and find this site to be resourceful and useful.
Acceptance

[ ] Research a list of Blogs (at least 8-10 blogs)
[ ] Add the blog information in the data files
[ ] Test each individual look and external link

NOTE:

This task is timeboxed to a maximum of 4 hours.

https://joyofcode.xyz/categories/svelte
https://developer.mozilla.org/en-US/docs/Learn/Tools_and_testing/Client-side_JavaScript_frameworks/Svelte_getting_started
https://www.freecodecamp.org/news/search?query=svelte
https://dev.to/t/svelte
https://dev.to/t/sveltekit


macOS version doesn't use jemalloc

Although dmg package for macOS bundles jemalloc as same as packages for linux platforms, it doesn't use jemalloc.
To enable it, LD_PRELOAD environment variable is required. But probably no one sets it on macOS.

Probably we need to add EnvironmentVariables to td-agent.plist.

will you fix for 4.2.0?

No. I'm in final testing for 4.2.0.
It should be fixed in future version.

FYI; td-agent 4.5.0 (or under version?) has dead lock issue on macOS.
If we use jemalloc on macOS, we might have to revert to downgrade jemalloc version.
https://github.com/jemalloc/jemalloc/issues/895

And also, we should use DYLD_LIBRARY_PATH on macOS instead of LD_LIBRARY_PATH to replace malloc implementation with jemalloc.

FYI; td-agent 4.5.0 (or under version?) has dead lock issue on macOS.

Thanks for the information.
Do you mean jemalloc?
- td-agent 4.5.0
+ jemalloc 4.5.0

It seems that it's affect to jemalloc 4.5.0 - 5.0.0.
<= 4.4.0 or >= 5.1.0 aren't affected.

https://github.com/jemalloc/jemalloc/releases/tag/5.1.0
https://github.com/rust-lang/rust/pull/41861

https://github.com/rust-lang/jemalloc/compare/11bfb0dcf85f7aa92abd30524bb1e42e18d108c6...3288e0659c08fb5006f6d6dd4b5675ed0c2c432a

If we use jemalloc on macOS, we might have to revert to downgrade jemalloc version.
https://github.com/jemalloc/jemalloc/issues/895

td-agent 4.2.0 includes an amazing challenge for jemalloc.
It downgrades jemalloc to 3.6.0:  #305

And also, we should use DYLD_LIBRARY_PATH on macOS instead of LD_LIBRARY_PATH to replace malloc implementation with jemalloc.

You are right, I forgot it.
Thanks for notifying it.

FYI; td-agent 4.5.0 (or under version?) has dead lock issue on macOS.

BTW it might not be so critical for Ruby, because it uses Giant VM Lock.

Thanks for the information.
Do you mean jemalloc?

Yes, I mean jemalloc.

It seems that it's affect to jemalloc 4.5.0 - 5.0.0.
<= 4.4.0 or >= 5.1.0 aren't affected.

  * [rust-lang/jemalloc@11bfb0d...3288e06](https://github.com/rust-lang/jemalloc/compare/11bfb0dcf85f7aa92abd30524bb1e42e18d108c6...3288e0659c08fb5006f6d6dd4b5675ed0c2c432a)

No, it bumped from 4.1.0 to 4.5.0.
So 4.1.1 - 5.0.0 are possible to be affected.

Oh, it's widely spreaded issue...


Skip any inaccessible file:// source links

This allows continuing to next source quicker and avoids outputting a
lot of debug lines.

Personally I'd word the info message as "Failed to" instead of "Could not" (due to a general desire to avoid the word "not"), but that's a pretty minor nit; I'd consider the merge conflicts here to be the only real blockers...


Make --location not required

Description
--location is required in CodeGen generated command with get location from resource group. In this way, it should not be required.

expected:

Solution
add required=False in location in manual
    with self.argument_context('disk-pool') as c:
        c.argument('location', arg_type=get_location_type(self.cli_ctx), required=False,
                   validator=get_default_location_from_resource_group)

close since PR merged


[enhancement] Update covers when manga info is swiped

when manga info is swipe refreshed.  Delete the cached cover and repull

This was done a while back


fix fileName undefined iOS 8

Motivation (required)
What existing problem does the pull request solve?
in iOS the app crashes https://github.com/react-community/react-native-image-picker/issues/640
Test Plan (required)
open imagePicker , try to select an image (crashes on iOS 8).

Get the same bug on IOS 11.2.5. Should this pull request be merged on a new version as soon as possible?

@gregogalante These changes seems to target iOS 8, I'm not sure how this will help on iOS 11.


amplitude normalization in create_fb_matrix

🐛 Not necessarily a Bug
Not exactly a bug, but I think it makes more sense if the triangular Mel filterbanks are area-normalized, as opposed to the current behavior where they are height-normalized.
To Reproduce / Expected behavior
Steps to reproduce the behavior:
import matplotlib.pyplot as plt
import torchaudio
import librosa

plt.figure(figsize=(16, 8))
plt.subplot(2, 1, 1)
plt.plot(torchaudio.transforms.MelScale(n_stft=1025, sample_rate=16000).fb.detach().cpu().numpy())
plt.title('torchaudio Mel filterbanks')

plt.subplot(2, 1, 2)
plt.plot(librosa.filters.mel(sr=16000, n_fft=2048).transpose())
plt.title('librosa Mel filterbanks')
plt.show()

The former has an advantage of preserving the energy similar to the STFT, while the latter keeps the overall spectral envelope unchanged.
If this is an intended behavior, feel free to close this issue. Thanks!
Environment

What commands did you used to install torchaudio (conda/pip/build from source)?

I did pip install torchaudio -f https://download.pytorch.org/whl/torch_stable.html, following README.md

What does torchaudio.__version__ print? (If applicable)

'0.3.0'
Please copy and paste the output from our environment collection script.
Collecting environment information...
PyTorch version: 1.2.0
Is debug build: No
CUDA used to build PyTorch: 10.0.130

OS: Ubuntu 16.04.5 LTS
GCC version: (Ubuntu 5.4.0-6ubuntu1~16.04.11) 5.4.0 20160609
CMake version: version 3.11.1

Python version: 3.7
Is CUDA available: Yes
CUDA runtime version: 10.0.130
GPU models and configuration:
GPU 0: Tesla V100-SXM2-16GB
GPU 1: Tesla V100-SXM2-16GB
GPU 2: Tesla V100-SXM2-16GB
GPU 3: Tesla V100-SXM2-16GB
GPU 4: Tesla V100-SXM2-16GB
GPU 5: Tesla V100-SXM2-16GB
GPU 6: Tesla V100-SXM2-16GB
GPU 7: Tesla V100-SXM2-16GB

Nvidia driver version: 410.79
cuDNN version: /usr/lib/x86_64-linux-gnu/libcudnn.so.7.4.2

Versions of relevant libraries:
[pip3] numpy==1.16.5
[pip3] pytorch-memlab==0.0.3
[pip3] torch==1.2.0
[pip3] torch-bst==0.0.0
[pip3] torch-data==0.0.1
[pip3] torchaudio==0.3.0
[pip3] torchvision==0.4.0
[conda] blas                      1.0                         mkl
[conda] faiss-gpu                 1.5.3            py37h1a5d453_0    pytorch
[conda] mkl                       2019.4                      243
[conda] mkl-service               2.3.0            py37he904b0f_0
[conda] mkl_fft                   1.0.14           py37ha843d7b_0
[conda] mkl_random                1.0.2            py37hd81dba3_0
[conda] pytorch-memlab            0.0.3                    pypi_0    pypi
[conda] torch                     1.2.0                    pypi_0    pypi
[conda] torch-bst                 0.0.0                     dev_0    <develop>
[conda] torch-data                0.0.1                     dev_0    <develop>
[conda] torchaudio                0.3.0                    pypi_0    pypi
[conda] torchvision               0.4.0                    pypi_0    pypi

Thanks for pointing this out :) Do we have a comparison with kaldi for this graph?

I can still reproduce this PR after #292.

This would be breaking backward compatibility of course, so we'd need strong demand to change this.

Is there a simple way to convert from one to the other as a post-processing?
I'd be open to having an option to switch too. Please feel free to open a pull request for this :)

To answer the first, I was able to convert from one to the other by adding a linear function of Mel frequency, with minimal numerical differences.
I haven't used Kaldi, but I agree that breaking backward compatibility is a bad idea. I think it'd still be helpful to the community if we can make the output exactly match librosa's outputs, as it is very popular for Music processing in Python, and after NVIDIA's Tacotron 2 implementation adopted the same filterbank many TTS systems started to use Librosa's Mel spectrograms as a common mid-level representation for different vocoder implementations, for instance.

To answer the first, I was able to convert from one to the other by adding a linear function of Mel frequency, with minimal numerical differences.

If this is a general transformation that can be used elsewhere, it could be a solution. Otherwise, offering a toggle/switch is more appropriate.

I haven't used Kaldi, but I agree that breaking backward compatibility is a bad idea. I think it'd still be helpful to the community if we can make the output exactly match librosa's outputs, as it is very popular for Music processing in Python, and after NVIDIA's Tacotron 2 implementation adopted the same filterbank many TTS systems started to use Librosa's Mel spectrograms as a common mid-level representation for different vocoder implementations, for instance.
I'll be interested in working on a PR for this, in the following ~4 weeks timeline. If you feel that's okay please feel free to assign this to me!

Thanks! That would be great :)

+1 for this!

To answer the first, I was able to convert from one to the other by adding a linear function of Mel frequency, with minimal numerical differences.

@jongwook do you have a gist for this?

@delip I did, and unfortunately I no longer have access to it.
I'm feeling bad about not getting my hands on this over the break, but I should be able to soon. I will keep you posted!

https://gist.github.com/jongwook/d6fdf57a6fd06d56a2bfd6b505e7804c
The notebook above demonstrates that Torchaudio mel spectrogram and librosa mel spectrogram differ by a linear function of Mel frequency, when htk=True is specified in librosa.
I'm thinking of adding a keyword argument normalized=False to MelScale and MelSpectrogram, and use librosa-style filterbank when normalized=True is given.
Regarding htk=True, since the librosa default Slaney filterbank is also quite widely used (ultimately from MATLAB auditory toolbox), I'd be also keen to add another optional kwargs filterbank="htk" or "slaney".
Ultimately, it'd be nice to have module torchaudio.compliance.librosa, but first things first!

Hi,
I am using the MelSpectrogram module of torchaudio v0.4 to define a spectral loss function for training my model. Although it converges properly and the output signal gets improves towards the target one, there are subtle tonal artifacts that cannot be removed with further optimization. Could this normalization technique be a solution for that? (i.e. to use area normalization as in librosa instead of height normalization).

https://gist.github.com/jongwook/d6fdf57a6fd06d56a2bfd6b505e7804c

@jongwook  Could you please share this notebook again? As the current shared one doesn't work. Or instead provide the linear fucnction to change the normalization style of the torchaudio mel filterbank.
Many thanks in advance

This is implemented in #589.


Error message overlays overhaul

Fixes MA-2283

New error message added for 404 and 500 status codes.
Error message overlays don't persist on screen persist, take
extra space and look weird anymore.

Here's how the error message overlays looked before before (intentional duplication 😉 ) and now:

Before
Before
Now

@1zaman @BenjiLee @aleffert @mdinino plz review

@@             master       #709   diff @@
==========================================
  Files           477        477
  Lines         20909      20900     -9
  Methods           0          0
  Messages          0          0
  Branches       2942       2942
==========================================
  Hits           6232       6232
+ Misses        14677      14668     -9
  Partials          0          0

Review [#709][cc-pull] on Codecov

Powered by Codecov. Last updated by [dfd8828...65d2cb8][cc-compare]
[cc-base-branch]: https://codecov.io/gh/edx/edx-app-android/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/edx/edx-app-android/compare/dfd8828f8605e787e997356224076a3894aca959...65d2cb88fd18171eb85465a5f5d87c9f3d51de55
[cc-pull]: https://codecov.io/gh/edx/edx-app-android/pull/709?src=pr

YES!

Looks great so far. Just the one pending deprecation comment.

Other than a few points I commented on 👍

👍

I think we should be showing persistent error messages when GET requests fail, since this results in not having something shown to the user that he expects. However, should we have POST requests displaying temporary error messages? Currently these could be missed if the used switched out the app while they were being displayed (although we could try to have some logic to have it only shown when the Activity is resumed I suppose). On the other hand, having a persistent error message would also be annoying, specially if the cause is not apparent. cc @bguertin, @BenjiLee, @mdinino

I think we should be showing persistent error messages when GET requests fai

Yes, I agree all load errors should typically be persistent errors.

should we have POST requests displaying temporary error messages.

If the POST is due to an explicit user action, then yes it should definitely be persistent until the user acknowledges it. Typically, you'd use a dialog for this.
Some examples from Google:
https://www.google.com/design/spec/patterns/errors.html#errors-user-input-errors

How should we go about this issue @aleffert, since we already have a ticket open (with not much done on it) on the various error states and their respective UI -> UX-2868.
cc @chris-mike

@miankhalid can you sum up the specific question? Generally speaking we should follow the android guidelines. Probably we should get rid of the custom black overlay messages and replace them with standard UI like the snackbar?

@aleffert sure.
The overlay message with the red background, is what we are currently using in app whenever we face an error/exception whether it be a network error or http error (which is persistent). Also, this view isn't actually an overlay i.e. it takes up space from the top plus its appearing/disappearing animation is really bad too.
The overlay message with the grey background, is what we use for displaying interim or info messages e.g. "Your comment has been posted.". This view isn't persistent.
The ticket in question requires us to show a message when we try to perform something on a deleted post, response or comment, which iOS shows as a grey overlay message. But @1zaman and @bguertin have a different opinion on it that the error message should be persistent and reside in a dialog.

If the POST is due to an explicit user action, then yes it should definitely be persistent until the user acknowledges it. Typically, you'd use a dialog for this.

Some examples from Google:
https://www.google.com/design/spec/patterns/errors.html#errors-app-errors

^ excerpt from Brian's [comment above] (https://github.com/edx/edx-app-android/pull/709#issuecomment-220632510).

@miankhalid yeah, I think it makes sense to follow the platform convention in those cases.

@aleffert I am not sure how our error and info messages should translate and/or divide into a combination of SnackBar and Dialogs. Can we have the UX team give a pass on it soon ? (so that we don't have to redo or overdo things)

It looks like the guide recommends only using dialogs for errors that block normal operation, or allow more than one user action is response. I don't think our errors meet those requirements, so we might want to go with Snackbars. The guide says they're transient, but it looks like they have an option for persistence as well. If we use them in conjunction with CoordinatorLayout, then they can also be dismissed by swiping them out. They're also automatically queued. Overall they're looking like a reasonable option for general purpose errors. But yes, we should definitely get a final decision on this from the UX team.

I think there are two questions here:

Should we replace "flying messages" with snackbars?

I don't think we need to tackle that problem just yet. But in my opinion, the snackbars look and behave much nicer than our flying messages. If we don't mind differing from iOS here (iOS does not have "snack bars") then I think we should switch. Also, getting the UX just right may require significant changes (e.g. using CoordinatorLayout).

What type of errors should use what UX (dialog vs. snackbar)? I've created a document to track this here: https://openedx.atlassian.net/wiki/display/MA/Displaying+Errors

I think snackbars is the way to go. When we do this? I don't know. I would prefer sooner than later since UX is trying to overhaul some of the error handlings issues right now.

Document looks fine to me.

Is there any overlap with https://github.com/edx/edx-app-android/pull/718?

I like snackers too. It fits with the Android look and feel
On Tue, May 31, 2016 at 3:36 PM, Christopher Lee<EMAIL_ADDRESS>wrote:

Is there any overlap with #718
https://github.com/edx/edx-app-android/pull/718?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/edx/edx-app-android/pull/709#issuecomment-222796415,
or mute the thread
https://github.com/notifications/unsubscribe/AR10TzUHCrxeuuLzG2Yy9dUVyUwj25oDks5qHI3KgaJpZM4IgWZf
.

--
Regards,
Marc DiNino
Android Engineer
EDX

*snack bar
On Tue, May 31, 2016 at 3:37 PM, Marc Dinino<EMAIL_ADDRESS>wrote:

I like snackers too. It fits with the Android look and feel
On Tue, May 31, 2016 at 3:36 PM, Christopher Lee<EMAIL_ADDRESS>
wrote:

Is there any overlap with #718
https://github.com/edx/edx-app-android/pull/718?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/edx/edx-app-android/pull/709#issuecomment-222796415,
or mute the thread
https://github.com/notifications/unsubscribe/AR10TzUHCrxeuuLzG2Yy9dUVyUwj25oDks5qHI3KgaJpZM4IgWZf
.

--
Regards,
Marc DiNino
Android Engineer
EDX

--
Regards,
Marc DiNino
Android Engineer
EDX

Is there any overlap with #718?

There shouldn't be much overlap for now, since I have only removed the current offline bar from the various layouts at the moment. I was also thinking of refactoring the fly-in/warning messages in the BaseFragmentActivity decoration layout while I was at it, but didn't get the chance to do so yet.

For Android, go with snackbars, and moving forward we should always utilize native functionality for alert messages. Do you need any other UX input for this PR?

@lizcohen: According to the Error Handling wiki, the POST requests would qualify as blocking errors, and we'll use a dialog to handle those. We can use Snackbars in the place of the existing fly-in messages though, there should be separate tickets created for making those changes.

So, I've implemented a mechanism for showing error dialogs on POST requests, but its listener/callback system might need improvement/feedback.
Care to give it a look ? @1zaman @BenjiLee @bguertin @mdinino

@miankhalid Were you ready for another pass?

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/edx/edx-app-android/pull/709#issuecomment-224598018,
or mute the thread
https://github.com/notifications/unsubscribe/ABoa1PsUTXXQlVIPWslsrQVq67E2XDnHks5qJssIgaJpZM4IgWZf
.

I've added the Reviewers section to PR's description, need another pass team: @BenjiLee @bguertin @mdinino @1zaman

👍

When making a comment in a deleted response, we get a 404, and the dialog pops up. When we navigate back, that deleted comment is still there. Should we do a refresh on that page when a 404 occurs?

When making a comment in a deleted response, we get a 404, and the dialog pops up. When we navigate back, that deleted comment is still there. Should we do a refresh on that page when a 404 occurs? (same thing for making a response on a deleted thread)

That might not be a good idea cuz it'll introduce much more complexity than it looks.
For starters, if a response is deleted the dialog should say "The response has been deleted" and:

if the user is on Add Comment activity, its should be finished and View Comments activity should also be finished.
if the user is on Responses activity, the response row should be removed upon voting/reporting.

It wouldn't be a good UX if we just remove the row without notifying the exact reason for it.
p.s. the case would get even more complex if a post is deleted and a user is trying to add a comment to one of its responses 😉

I don't think we need to automatically finish any Activities; that's a bad UX, that would cause the user to lose the current context. Instead, we can use a much simpler flow by just displaying a customized error message on getting a 404 error to the effect that the entity has been deleted, and posting an associated event that would cause it to be removed from whatever Activity that's displaying it in the back stack (only from lists not if it's the totality of the entity that's being displayed).

👍 👍 👍

There has already been quite a lengthy discussion here (MA-2284) as to why we are showing a generic message.
We'll have to run with the UX team and the Docs team again if we want something different.
p.s. Deleting something in the previous activity based on results from a following activity isn't a super cool UX either.

There has already been quite a lengthy discussion here (MA-2284) as to why we are showing a generic message.

We'll have to run with the UX team and the Docs team again if we want something different.

OK, I suppose the current implementation fixes the main issue, and can be merged as it is for now, but we could take another look to refine this design later. cc @lizcohen

p.s. Deleting something in the previous activity based on results from a following activity isn't a super cool UX either.

I think it's fine if the user has already been told (through the dialog) that the entity has been deleted. After that, keeping it there in the list would probably present more confusion than removing it.

OK, I suppose the current implementation fixes the main issue, and can be merged as it is for now, but we could take another look to refine this design later.

agreed.

I think it's fine if the user has already been told (through the dialog) that the entity has been deleted. After that, keeping it there in the list would probably present more confusion than removing it.

The best UX that me as dev can come up with is, dim the whole row with a lighter color and display a little label over it saying: DELETED

The best UX that me as dev can come up with is, dim the whole row with a lighter color and display a little label over it saying: DELETED

That would be clutter that doesn't present any real benefit and just obstructs the UI, in my opinion.

meh .. thats for the UX people to decide 😉

After the user sees the generic message that tells them the action cannot be completed, we can remove the post, however if we can at some point alert the user specifically that the post was deleted instead of the generic alert, that would be best.

OK, I have created tickets on this for Android and iOS at MA-2525 and MA-2526.


Add a Gitter chat badge to README.md

phase2/grunt-drupal-tasks now has a Chat Room on Gitter
@arithmetric has just created a chat room. You can visit it here: https://gitter.im/phase2/grunt-drupal-tasks.
This pull-request adds this badge to your README.md:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

Can this be closed?


The predicted image is in almost null (all pixels are the same in grey)

Hi, everyone. I am tring to train the model on my own datasets and wanna know if everything is going well.
So, after few steps, I use the model like this:
"model = PConvUnet(vgg_weights=None, inference_only=True)
model.load(r"D:\projects_test\PConv-Keras-master\data\logs\myDataset_phase1\weights.01-7.00.h5", train_bn=False)"
and weights.01-7.00.h5 the midresult of the training.
However, it seems the prediction does not work well as the title depicted. Does anyone know what is going wrong or not? Thank you.

solved.

@bytemani how did you solve it ?!


Fix Typo - wether -> whether

Fixed Typo
Description
Fixed Typo wether -> whether
Status
Done, Ready for Review

Thanks.
As far as I know this file is automatically generated by pylint (I could be wrong, it might be example included in pylint docs) so if that's indeed the case it might be a good idea to also fix it upstream :)

Merged, thanks.

Could you please add the tag Hacktoberfest?


Merge main into 1.x

Description
Merging branch main into 1.x since it is behind.
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Closing this PR to create a separate one where merge conflicts are resolved locally since the GitHub UI cannot resolve them within the PR itself.


Docs: Move the field content into a 'fields' directory/path

This PR moves our field content into a folder (and url path) called /fields

@BarnabyBishop I just realised /docs/fields will be a 404, is that something we want to redirect as well or do we consider that fine? Unsure of common practices in this area, when not having landing pages

@BarnabyBishop I just realised /docs/fields will be a 404, is that something we want to redirect as well or do we consider that fine? Unsure of common practices in this area, when not having landing pages

I think its fine. There isn't anywhere sensible to redirect it to anyways.


All results audio tabbing, single result link, and play/pause fixes

Fixes

Fixes #662 by @zackkrida
Description

Spacebar or play button click to play/pause on boxed audio
Click or enter to visit single result

We probably still need to announce to screen reader users that space will play/pause and click will visit the single result. There may be a way to do what I'm doing and wrap in a NuxtLink for improved usability.
Testing Instructions

Checklist

[x] My pull request has a descriptive title (not a vague title like Update index.md).
[x] My pull request targets the default branch of the repository (main) or a parent feature branch.
[x] My commit messages follow best practices.
[ ] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no visible errors.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

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 open source license
    indicated in the file; 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.

Merging, even if this has problems it's still vastly preferable to the previous approach.


Fixed TracingKafkaClientSupplier to properly override getAdmin

getAdminClient has been deprecated in favor of getAdmin.
Calls to getAdminClient are throwing an UnsupportedOperationException in kafka-streams version < 3.0.0, while calls to getAdmin are also throwing an UnsupportedOperationException in kafka-streams version >= 3.0.0

@TYsewyn could you update the license headers and push again?

@TYsewyn I think jeqo meant to do so on the modified file only. Could you please drop the last too commits and update only the header of the modified file?

Sure, I can do that. Can any of you fix the other license headers in the main branch?
None of the PRs is actually going to build right now, see https://github.com/openzipkin/brave/runs/4510741287?check_suite_focus=true#step:4:110

@TYsewyn could you please open a PR with header updates for the POMs?

@TYsewyn this PR address the license issue https://github.com/openzipkin/brave/pull/1315

@TYsewyn do you mind rebasing?

Could we exhibit a failing test case?

Just following up. Do we know when this is going to be merged (and eventually released)?

Somehow this comment got through:
Quick question: will this work with both <3.0.0 and >=3.0.0?

Yes, it should. getAdminClient was deprecated in <3.0.0, and getAdmin was added for >=3.0.0.
<3.0.0 calls getAdminClient, and >=3.0.0 calls getAdmin.
TracingKafkaClientSupplier still implements getAdminClient. This implementation calls getAdmin which returns an instance of AdminClient.

Ideally I'd like to exhibit a test case where we use <3.0.0 and >3.0.0 (just to make sure we don't break the implementation in further changes) but I guess that can backfilled later. I will merge as it is. If you fill so, a unit test as I described would be good.

I believe that would be quite difficult to achieve since you would need to have both version on your classpath at the same time. Unless you create separate submodules for Kafka integration tests.

New release is out: https://github.com/openzipkin/brave/releases/tag/5.13.6

Unless you create separate submodules for Kafka integration tests.

That sounds reasonable. Any input @jeqo?


Mention "native associations" also in the main docu

@chgeo : Please remove the sentence "For new projects, cds add hana automatically adds this configuration." if it won't make it into the release.

Fine for me. Just take over the suggestions from the release note PR.

done

@smahati can you review these docs so that we can close the topic?  Also see the release note PRs for August (which have been merge already).


Cold/Flu:  'Registered Clinics for this Worksite'

User Story
“As a user registering a worksite, I want see all the clinics already registered at my worksite so I can contact and coordinate with colleagues who have already registered.”
Tasks

[ ] Create grid to contain data fields to be pulled from the DB
Columns:

Worksite contact
Ministry
Type of Clinic
Number of employees

[ ] Logic: When a user selects a city and worksite from the drop down, check DB to see if the worksite already has a clinic registered

YES display grid populated with existing registration info (Display OPEN clinics ONLY)
NO  continue to "clinic registration'

Acceptance Criteria

[ ] When a user selects a city and worksite, if a clinic has already be registered at that site, registration information is displayed.
[ ]  the information display should be tabular allowing for one or many registration entries

Meeting with Mike needed. Caleb/Stella.
Stella will setup.

waiting for API to Fill data

@mikeshi-bcgov
We need one API endpoint. which returns clinics/registrations filter by WorksiteID. Once schema fix please prepare this API endpoint on priority.
Please see ticket details for required keys in response for each clinics/registrations.
(fields are Worksite Contact, Ministry, Type Of Clinic/Registration, Number of Employees)

@bhumin-fw
Sure Bhumin, I will bring this to the team in this afternoon's meeting, to see how to present these fields out of the new schema

Is #1000 blocking this?

@Stella-Archer
yes #1000 blocking this ticket.

@mikeshi-bcgov
https://analytics-testapi.psa.gov.bc.ca/apiserver/api.rsc/ods_datamart_influenza_worksites/
https://analytics-testapi.psa.gov.bc.ca/apiserver/api.rsc/ods_datamart_influenza_worksite_ministries/
Above API endpoints only give worksites list or mapping.
We need below API endpoints:

Worksite list filter by provided ministry_id.
(When we pass ministry_id in API endpoint it will only list worksites link to that ministry. Right now I have to fetch worksite ids and then have to trigger worksites API)

Registrations based on provided worksiteID.

This API endpoint should accept worksite_id as filter in request and in response it should send registrations under selected worksites.
response should contain at-least below value
worksite-contact: String (name of primary contact)
registration-type: String (value of is_closed key from registration table which contain "open" & "closed" as value)
ministry: String (Name of registration's ministry)
number-of-employee: Int (number of employee value from registration table)

FYI: This is priority along side add/update registration API.

@mikeshi-bcgov
It did revert blank, do I need to pass filter here? or you are cleaning and reloading tables?
<EMAIL_ADDRESS>'https://analytics-testapi.psa.gov.bc.ca/apiserver/api.rsc/$metadata#ods_datamart_influenza_vw_worksites_ministries_joined', 'value': []}

@mikeshi-bcgov
It did revert blank, do I need to pass filter here? or you are cleaning and reloading tables?
<EMAIL_ADDRESS>'https://analytics-testapi.psa.gov.bc.ca/apiserver/api.rsc/$metadata#ods_datamart_influenza_vw_worksites_ministries_joined', 'value': []}

Mike: the data was reloaded.

@mikeshi-bcgov to add two entries for registration table.

@EduardoRamirezBC
Two dummy entries have been added to registrations table

@bhumin-fw
API EndPoint for the item requested #2 ( 2. Registrations based on provided worksiteID.) is available now as well, please test
https://analytics-testapi.psa.gov.bc.ca/apiserver/api.rsc/ods_datamart_influenza_vw_registrations_worksites_joined/

I'll test the endpoint today and update the ticket.

Dummy data in for and Iman to test the endpoint today

@mikeshi-bcgov
The https://analytics-testapi.psa.gov.bc.ca/apiserver/api.rsc/ods_datamart_influenza_vw_registrations_worksites_joined works great and I can see the registration items including the ones I sent last night. filter by application_id also works.
Thanks!

@bhumin-fw  - Hi Bhumin, yes, let me know when we can connect today. Thanks C

@chriscaldwell-psa @bhumin-fw I'm looking for this in both DEV and TEST and don't see 'clinics at this worksite' populating is there a particular ministry/worksite combo I need to query to see the test data?

@Stella-Archer
try below combination.

Thanks @bhumin-fw oddly enough that did not work for me :(
TEST

DEV

Hey @bhumin-fw ,
I tested this in Test and Dev and it didn't work out for me either.
Test

Dev

this work great, PO approved and closing this ticket


[image_picker]: How to select previously selected images in gallery by default to avoid repeated selections

e.g.
selected images: [1.png, 2.png]
images in gallery: [1.png, 2.png, 3.png, 4.png]
when _picker.pickMultiImage(): [1.png(selected), 2.png(selected), 3.png, 4.png]
1.png and 2.png have been selected.

Hi @NRMrYang
image_picker invokes native picker and selections returned to image_picker, so image_picker doesn't have control over what was selected
Closing, as this isn't an issue with image_picker itself. If you disagree, please write in the comments and I will reopen it.
Thank you

Thanks for your reply


Allow specification of ECR registry region

We have a situation where our Buildkite agents run in and access other AWS resources from a different region to the container registry they need to pull from. This is resulting in the ECR plugin logging in to the wrong region, and hence failing to authenticate.
This change creates an optional BUILDKITE_PLUGIN_ECR_REGISTRY_REGION (name open to negotiation 😄 ) environment variable to explicitly specify which region the registry you are trying to access is in. i.e., setting BUILDKITE_PLUGIN_ECR_REGISTRY_REGION=us-east-1 lets us pull from the Virginia region even if AWS_(DEFAULT_)REGION=ap-southeast-2.

Sorry I missed this! Thanks, this looks great!


[ADD] <languages/Rust>

Category Name
Languages
Subcategory Name
Rust
Additional Context
Rust is a systems programming language that focuses on safety, speed, and concurrency. It is designed to be memory-safe, thread-safe, and free of data races.
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I have checked that this category does not exist
[X] I have checked that this subcategory does not exist
[X] I am willing to work on this issue (optional)

@Mouh1b assigning to you, please feel free to tag any of the maintainers if you need any help :)


Media messages Randomly Sent in Messages?

Hello,
I was chatting with a friend today via IG:dm on my Windows 10 machine when suddenly, images that appeared to be ads started populating, attached to my messages to my friend. I have no idea where the ads came from. It happened twice, both ads were for some "Living Church of God's Provision", with much of the ad being in Spanish. What is odd is that the specific friend I was chatting with does speak Spanish and lives in Miami. Again though, this was with the ads appearing to come from me.
I was using v2.6.1 and now see the latest is v2.6.5. I'll install that newer version to see if it recurs, but the whole issue seemed very strange indeed.

Hi where did you install your IGdm from? Also can you share screenshots of the Ad?

On 8 Aug 2019, at 7:15 PM, duplicate-issues[bot]<EMAIL_ADDRESS>wrote:
Hey @jhlee2222,
We did a quick check and this issue looks very darn similar to
#1037 - I have sign with facebook in instagram and i can't log in on the app
#1036 - it's something missing
#1035 - Notifications are not showing
This could be a coincidence, but if any of these issues solves your problem then I did a good job 😄
If not, the maintainers will get to this issue shortly.
Cheers,
Your Friendly Neighborhood ProBot
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I downloaded from this URL: https://igdm.me/
Or so I thought. I have the last installer, if there's any way to check.
Also, attached is the strange ad image:
[cid:em05fab256-3228-47b7-974d-6321a0777a8a@skuld]
I'm now on the "pro" version, downloaded from that site, specifically Pro 1.3.6. Let me know if that's not the right source for official releases please. I'll let you know if I see any similar activity. Also, let me know if you get any hints that it could be something else simply playing with my account, and not IG:dm specifically.
------ Original Message ------
From: "Ifedapo .A. Olarewaju" <notifications<EMAIL_ADDRESS>To: "ifedapoolarewaju/igdm" <igdm<EMAIL_ADDRESS>Cc: "jhlee2222" <jhlee2222<EMAIL_ADDRESS>"Mention" <mention<EMAIL_ADDRESS>Sent: 8/8/2019 2:22:48 PM
Subject: Re: [ifedapoolarewaju/igdm] Advertisements Randomly Sent in Messages? (#1038)
Hi where did you install your IGdm from? Also can you share screenshots of the Ad?

On 8 Aug 2019, at 7:15 PM, duplicate-issues[bot] <notifications<EMAIL_ADDRESS>wrote:
Hey @jhlee2222,
We did a quick check and this issue looks very darn similar to
#1037 - I have sign with facebook in instagram and i can't log in on the app
#1036 - it's something missing
#1035 - Notifications are not showing
This could be a coincidence, but if any of these issues solves your problem then I did a good job 😄
If not, the maintainers will get to this issue shortly.
Cheers,
Your Friendly Neighborhood ProBot
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/ifedapoolarewaju/igdm/issues/1038?email_source=notifications&email_token=ALG4LBQGJJBXN4Q6P4T5OOTQDRP7RA5CNFSM4IKNGI6KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD34PUII#issuecomment-519633441, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ALG4LBSPBFPH5REP2PAYGJ3QDRP7RANCNFSM4IKNGI6A.

I'll try attaching the image here....

Ugh...it continues, though not ads specifically. This happened on the pro version. I'm just casually typing and random images get sent.

Hmm these messages look like they are coming from your account. Can you check your chat on the mobile App to see if these messages are present there?

On 8 Aug 2019, at 7:36 PM, jhlee2222<EMAIL_ADDRESS>wrote:
Ugh...it continues, though not ads specifically. This happened on the pro version. I'm just casually typing and random images get sent.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

I can confirm the messages are present in the full IG App. However, they only get inserted into the conversation when I'm using IGdm.
I wouldn't rule out something on my account, but checking  my settings on Instagram I have "not authorized any applications to access your Instagram account." I don't expect you to be an IG account support guide, but can you think of anywhere else in my account I should check before ruling out the account as a source of the issue?

it seems to me like someone might be meddling with your account, can't tell where from, but I'll suggest you change your password

I have same issue, and it only happens when I use 3 dots. I tend to use the periods( ... ), rather than look for real punctuation. Lately, I have been getting pics of that Epstien guy.

@greenhalgh checked it on my machine, same issue; it only happens when enclosed by other characters (not whitespace). Also, when opening a message with three periods followed by three or more characters it gets stuck on sending.

I will add I'm one to use a lot of ellipses in conversations, as well as "non-standard" emotiocons that don't always translate to emoji.
I've changed my password just to rule out any third-party interference though.
------ Original Message ------
From: "Lucas" <notifications<EMAIL_ADDRESS>To: "ifedapoolarewaju/igdm" <igdm<EMAIL_ADDRESS>Cc: "jhlee2222" <jhlee2222<EMAIL_ADDRESS>"State change" <state_change<EMAIL_ADDRESS>Sent: 8/13/2019 4:57:01 PM
Subject: Re: [ifedapoolarewaju/igdm] Media messages Randomly Sent in Messages? (#1038)
@greenhalghhttps://github.com/greenhalgh checked it on my machine, same issue; it only happens when enclosed by other characters (not whitespace). Also, when opening a message with three periods followed by three or more characters it gets stuck on sending.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://github.com/ifedapoolarewaju/igdm/issues/1038?email_source=notifications&email_token=ALG4LBTAZQNJEOPSD6MU5E3QEMNZ3A5CNFSM4IKNGI6KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4G6WWQ#issuecomment-521005914, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ALG4LBUWZARQJLNTQ7FZMCLQEMNZ3ANCNFSM4IKNGI6A.

Same thing for me. I get pictures of Jeffrey Epstein with about every other message!

I see now what the issue is. The messages with ... are wrongly parsed as URLs, and so IGdm thinks you are sharing a link and it tries to fetch the media content of that link (just as it is done on IG mobile), and when this happens, it gets the pictures of these wrongly parsed URLs and adds it to your message just as you've complained.
I'll work on a fix for it this week

Much appreciated!
On Sat, 17 Aug 2019 at 10:27, Ifedapo .A. Olarewaju <
<EMAIL_ADDRESS>wrote:

I see now what the issue is. The messages with ... are wrongly parsed as
URLs, and so IGdm thinks you are sharing a link and it tries to fetch the
media content of that link (just as it is done on IG mobile), and when this
happens, it gets the pictures of these wrongly parsed URLs and adds it to
your message just as you've complained.
I'll work on a fix for it this week
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ifedapoolarewaju/igdm/issues/1038?email_source=notifications&email_token=AALBPKMNF6U6U2TJGBUUFJTQE6Y57A5CNFSM4IKNGI6KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4QGKOY#issuecomment-522216763,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AALBPKNVCAN44HRULZEQD2DQE6Y57ANCNFSM4IKNGI6A
.

--
www.mathemafrica.org
http://www.flickr.com/photos/jonstraveladventures

this issue is now fixed in the latest version


[BUG] invalid character 'A' looking for beginning of value

What is the bug?
terraform apply returns
│ Error: invalid character 'A' looking for beginning of value
│
│   with opensearch_index.index1,
│   on opensearch.tf line 1, in resource "opensearch_index" "index1":
│    1: resource "opensearch_index" "index1" {

How can one reproduce the bug?
resource "opensearch_index" "index1" {
  name             = "index1"
  index_knn        = true
  default_pipeline = "my-pipeline-id"
  mappings         = <<EOF
{
    "properties": {
      "entity_name_v": {
        "type": "knn_vector",
        "dimension": 384,
        "method": {
          "name": "hnsw",
          "engine": "lucene",
          "space_type": "cosinesimil"
        }
      },
      "entity_name": {
        "type": "text"
      }
    }
  }
EOF
}

Fails with this error when the provider is using the custom domain URL.
provider "opensearch" {
  url         = "https://search.mydomain.com"
  healthcheck = false
}

This custom domain is a CNAME record in Route53 with value of the AWS provided domain URL.
It does not fail when using the AWS provided domain URL.
provider "opensearch" {
  url         = "https://my-domain.us-east-1.es.amazonaws.com"
  healthcheck = false
}

terraform apply
What is the expected behavior?
index1 created successfully
What is your host/environment?
hashicorp/terraform:1.6.6 Docker container OR MacOS 14.1 (23B2073)
opensearch-project/opensearch provider 2.2.0
AWS OpenSearch version OpenSearch_2_11_R20231113-P4 (latest)

@nikki603 Is custom endpoint configured? https://docs.aws.amazon.com/opensearch-service/latest/developerguide/customendpoint.html

@rblcoder Yes it's configured. I confirmed the custom URL works various ways like in browser to access the Dashboard, Postman requests, and Logstash output plugin.

@nikki603 Are you able to do a curl from the command line using the custom url?

@rblcoder I still get the same error

@nikki603 What is the authentication mode you are using? Is it basic authentication or something else?

@rblcoder fine grain access control is enabled so I'm using IAM auth. I'm using AWS SSO locally. When running from a CI/CD pipeline I will be using the aws_access_key, aws_secret_key, and aws_token params.

@nikki603 I am able apply the following terraform code, I have defined a custom endpoint for my AWS OpenSearch Service domain.
  required_providers {
    opensearch = {
      source = "opensearch-project/opensearch"
      version = "2.2.0"
    }
  }
}

provider "opensearch" {
  url = "https://customsubdomain.domain.tld"
  healthcheck        = "false"
  aws_region          = "region"

  version_ping_timeout = "10"

}

resource "opensearch_index" "index" {
  name = "sample"
  number_of_replicas = "1"
  number_of_shards = "1"
}```

@nikki603 I also tested modifying an existing AWS OpenSearch domain and specifying a custom endpoint, it is working.

I see that I was missing  aws_region  = "region" when using a custom domain. It works now. Thanks for the attention!

Great, good to hear it works now!


Docs

I wish the docs were just a little more complete on this and would show how to simulate the various forms of colour blindness on an image.

I wrote a much more complete tutorial and reference manual this weekend -- see if it does what you need?
https://colorspacious.readthedocs.org/en/latest/

(Closing this for now b/c there's no todo item for me in it ATM, but feedback is very much welcome)


Change btc address

I have new btc address. Can I change it?

Yes, that’s not an issue and the correct way to amend the contract.  Thanks for the update and creating the PR for it.

@Noc2
Thank you!


Can't set min value

There is an error Error in render: "TypeError: this.min.isBefore is not a function" when I set min value, however I can set max value and it works.

There is an error Error in render: "TypeError: this.min.isBefore is not a function" when I set min value, however I can set max value and it works.

Seems like you are passing a string value when you need to pass a moment value. You just need to import moment and pass the value in min like this min: moment("2010-10-21").  We also encountered this error, and this is our solution. Hope this helps you too.
`

`

@iamshengsheng
yeah, thanks a lot!


Add latest_version

This should return the latest version of the Delta table.  Delta tables have version 0, version 1, version 2, etc.

I would like to work on this one. Want to get familiar with delta-rs's python bindings.


HttpProvider send jsonrpc error

I think should change http provider  send function like below:

date['id'] return as a String  with Vara network, so parse it incase of  failed.

If you can make a PR, it is always welcome :)

This has been fixed at #456 , published to pub.dev under polkadart: 0.4.4


Default fee reduction in Qt

Change default fee per byte from 100 sat/byte to 1 sat/byte in Qt.

We should edit the PR description to automatically close the issue when it merge. This will also make it immediately see the source issue too.

I think it already does this @ultimaweapon, After I merged your PR related to TOR build yesterday it closed related issue automatically.

Nope. You need to put a special keyword in the PR description. See https://help.github.com/en/articles/closing-issues-using-keywords for how to do it.

Ok, well it definitely did it in that case at least.

Any other outstanding issues on this?

@reubenyap right now Sigma team is quite busy for beta release so we need somebody to review this.

Updated and squashed as one commit


feat(XbyK Update): update XbyK libraries to version 30.0.0. Update dancing goat to the latest version.

Motivation

Updates integration XbyK references to version 30.0.0
Updates .NET reference to version 8
Updates Dancing Goat example to use the latest api changes
Returns back search view example

Do not forget to update version of XbK npm packages as well https://github.com/Kentico/xperience-by-kentico-lucene/blob/333c048188dbe95e4bfbd2ed17cd336eacfc3da7/src/Kentico.Xperience.Lucene.Admin/Client/package.json#L16

Also when looking at GH actions there are warnings about the use of obsolete API, fix this:

Do not forget to update version of XbK npm packages as well.
https://github.com/Kentico/xperience-by-kentico-lucene/blob/333c048188dbe95e4bfbd2ed17cd336eacfc3da7/src/Kentico.Xperience.Lucene.Admin/Client/package.json#L16
@Rumec This is constantly forgotten during updates. We should consider implementing some kind of automatic check.

@bkapustik fix this


Fix argument parsing

[ ] - arguments do not all need flags
[ ] - can you pipe things into ciphey?

https://realpython.com/python-command-line-arguments/

This is now fixed, Ciphey accecpts first argument without a flag as the text, or you can use the -t argument


Cmake error on raspberry pi 3b+ environment

when I execute " ./install-script " in /kinesis-video-native-build/, there was a problem,
following error code is problem.
how to Cmake successfully?
`Setting up for KVS Producer SDK installation
======= make sure the following libraries have been installed =======

pkg-config
libssl-dev
cmake
libgstreamer1.0-dev
libgstreamer-plugins-base1.0-dev
gstreamer1.0-plugins-base-apps
gstreamer1.0-plugins-bad
gstreamer1.0-plugins-good
gstreamer1.0-plugins-ugly
gstreamer1.0-omx (if on raspberry pi)
gstreamer1.0-tools
libcurl4-openssl-dev
liblog4cplus-1.1-9
liblog4cplus-dev
Checking jsmn at /home/pi/amazon-kinesis-video-streams-producer-sdk-cpp/kinesis-video-native-build/downloads/local/lib/libjsmn.a
Clean up artifacts
done Clean up artifacts
-- The C compiler identification is GNU 6.3.0
-- The CXX compiler identification is GNU 6.3.0
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- System name is Linux
-- Kinesis Video PIC path is ../kinesis-video-pic
-- Kinesis Video Producer path is ../kinesis-video-producer
-- Kinesis Video JNI path is ../kinesis-video-producer-jni
-- Kinesis Video Gstreamer demp app path is ../kinesis-video-gst-demo
-- Kinesis Video Open Source path is /home/pi/amazon-kinesis-video-streams-producer-sdk-cpp/kinesis-video-native-build/downloads
-- Configuration Unix Makefiles
-- Bitness 32 bits
-- Looking for pthread.h
-- Looking for pthread.h - found
-- Looking for pthread_create
-- Looking for pthread_create - not found
-- Check if compiler accepts -pthread
-- Check if compiler accepts -pthread - yes
-- Found Threads: TRUE
-- exe libraries Threads::Threads;dl;rt
-- Found PkgConfig: /usr/bin/pkg-config (found version "0.29")
-- Could NOT find Log4cplus (missing:  LOG4CPLUS_LIBRARIES LOG4CPLUS_INCLUDE_DIR)
Log4cplus is not found. Only c producer will be built.
CMake Error at /usr/share/cmake-3.7/Modules/FindPackageHandleStandardArgs.cmake:138 (message):
Could NOT find OpenSSL, try to set the path to OpenSSL root folder in the
system variable OPENSSL_ROOT_DIR (missing: OPENSSL_LIBRARIES
OPENSSL_INCLUDE_DIR)
Call Stack (most recent call first):
/usr/share/cmake-3.7/Modules/FindPackageHandleStandardArgs.cmake:378 (_FPHSA_FAILURE_MESSAGE)
/usr/share/cmake-3.7/Modules/FindOpenSSL.cmake:385 (find_package_handle_standard_args)
CMakeLists.txt:155 (find_package)

— Configuring incomplete, errors occurred!
See also "/home/pi/amazon-kinesis-video-streams-producer-sdk-cpp/kinesis-video-native-build/CMakeFiles/CMakeOutput.log".
See also "/home/pi/amazon-kinesis-video-streams-producer-sdk-cpp/kinesis-video-native-build/CMakeFiles/CMakeError.log".
`

can you follow the instructions here: https://github.com/awslabs/amazon-kinesis-video-streams-producer-sdk-cpp/blob/master/install-instructions-linux.md#install-steps-for-ubuntu-17x-and-raspbian-stretch-using-apt-get to get all the dependencies.

Closing the issue for now as it has been answered. Please reopen if needed.


keyof ignores constraint

Bug Report

🔎 Search Terms

keyof, generic constraint
🕗 Version & Regression Information

This is a crash ❌ No
This changed between versions _______ and _______
This is the behavior in every version I tried, and I reviewed the FAQ for entries about keyof
I was unable to test this on prior versions because it seems to be consistent behavior in all tested versions

⏯ Playground Link

Playground link with relevant code
💻 Code

// Implementation of event emitter - sample without all methods
type Listener<Event> = (event: Event) => void;

// Here the "key: K" complains "An index signature parameter type must be either 'string' or 'number'"
type Listeners<Events, K extends string & keyof Events> = {[key: K]: Listener<Events[K]>[]}

class EventEmitter<Events extends {[key: string]: unknown}> {
  #listeners: Listeners<Events, keyof Events>

  constructor() {
    this.#listeners = {}
  }
}

// Usage
type MyEvents = {
    connect: void
    reconnect: number
    error: Error
    disconnect: void
}
const emitter = new EventEmitter<MyEvents>()

🙁 Actual behavior
I am getting ts errors even though the types work as expected. See // usage section
🙂 Expected behavior
The keyof returns the same type as the constraint generic and thus will be valid as index signature.

Having keyof T as an indexer type would require #26797.

So, does that mean it is currently not possible to do?
btw. why is K extends string & keyof Events not enough to make it an indexable type since it requires 'string' or 'number' and the generic K must be a string and also keyof Events?

The type string & keyof Events is the same as keyof Events. And a union of keys can not be an index signature. You can use a mapped type instead.

This works without issue
type Listeners<Events, K extends keyof Events> = Record<K, Listener<Events[K]>>;

Thanks guys,
the error message wasnt clear to me, but know I understand that you cant use a union of types which you get from the keyof - e.g. type union = 'a' | 'b' | 'c' to index but instead use it in either Record which handles the union case or use mapped type like {[key in K]: ... } instead of {[key: K]:
Lastly, do you know why there is the last error at the .push(listener) line in .on() method ts playground
Argument of type 'Listener<Events[Type]>' is not assignable to parameter of type 'Listener<Events[keyof Events]>'.
  Type 'Events[keyof Events]' is not assignable to type 'Events[Type]'

it seems to me that those types should be the same, no? One is Events[keyof Events] (so basically any values of the Events) and the second one is the concrete event Events[Type] where the type is defined as constraint of keyof Events
  on<Type extends keyof Events>(type: Type, listener: Listener<Events[Type]>): Unsubscribe  {
    if (!type) throw new Error('Cannot call <EventEmitter>.on without a type');
    if (!listener) throw new Error('Cannot call <EventEmitter>.on without a listener');
    this.#listeners[type] = this.#listeners[type] || [];
    this.#listeners[type].push(listener);
    return () => {
      this.off(type, listener);
    }
  }

Looks like I had a mistake in the mapped type, working version for anyone interested
ts playground
Thanks @MartinJohns and @RyanCavanaugh


Which users are connected through the web client

Is there a possibility to see who is working with the web client? One possible use case could be updates of web client or statistics or...
Thank you.

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

ID: 38213294-2784-2341-ac3e-77d2b017e2a2
Version Independent ID: 8d30b4c5-68df-5fa2-d7e9-ede9b406527f
Content: Set up the Remote Desktop web client for your users
Content Source: WindowsServerDocs/remote/remote-desktop-services/clients/remote-desktop-web-client-admin.md
Product: windows-server-threshold
GitHub Login: @Heidilohr
Microsoft Alias: helohr

Thanks for your dedication to our documentation. If we have been unable to review your issue in a timely manner, we sincerely apologize for the delayed response. We have captured the informtion you have provided for review. We are closing this issue for now, but if you feel that it's still a concern, please respond and let us know. If you determine another possible update to our documentation, please don't hesitate to reach out again. #please-close


[FIXED JENKINS-33823] Enhance mavenJob siteArchivingDisabled/fingerpr…

…intingDisabled/resolveDependencies
o Added siteArchivingDisabled
o Added fingerprintingDisabled
o Added resolveDependencies

Thank you for this pull request! Please check this document for how the Jenkins project handles pull requests.


Create more tests

See https://github.com/fallen/Pytition/pull/70

I've done some work on this recently:

I wrote tests for all functions in views.py
I enabled test coverage computation and storage in jenkins ( http://jenkins.sionneau.net:8080/job/Pytition/job/master/lastBuild/cobertura/ ) using Cobertura Jenkins plugin
There is now a "Coverage" %tage badge in the README.md

Global coverage is now 87% of all lines (not taking templates into account)
views.py coverage is 60% : http://jenkins.sionneau.net:8080/job/Pytition/job/master/lastBuild/cobertura/pytition_petition/views_py/

Closing this ticket when reaching 90%!

90% test coverage reached!
Closing :)


Force datetime guesser input to be string

Pull Request Description (replace this text with your description)
Closes #xxxx

After creating the pull request: in order to pass the release_notes_updated check you will need to update the "Future Release" section of docs/source/release_notes.rst to include this pull request.

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

No linked issues found. Please add the corresponding issues in the pull request description.  Use GitHub automation to close the issue when a PR is merged

@christopherbunn a note that Woodwork's core requirements restrict it from installing pandas 2.0 or higher.


support for hexadecimal?

Hi,
I'm using python Bokeh and it's using Numeral.js to do number formatting. I'm looking for a way to display my unsigned integer as hexadecimal string but I can't find it. Could you please add support for hexadecimal output?
Thank you.
Quan.

Hi, this repository is no longer maintained. You can get an improved version here: https://github.com/foretagsplatsen/numbro. That one is still maintained and you can report your issues there.

Thanks, I'll close this then.


update method not typified correctly

When using update the return state of the callback function could have any type of value as key.
The only keys that can be used must be in the state type.

Updated method nows throws error when more keys are added to the return state of the update callback.
Added test to check this case.


Can Update but can't Select rows

Hi,
I'm using react-native-mssql on react-native 0.59.9 on Android
I'm able to update the rows in my MSSQL database, but when trying to select any rows.. it throws multiple errors.. couldn't find why
  let sqlConfig = {
      server: '<IP_ADDRESS>', //ip address of the mssql database
      username: 'sa', //username to login to the database
      password: 'aaaaa', //password to login to the database
      database: 'new', //the name of the database to connect to
      port: 1433, //OPTIONAL, port of the database on the server
      // timeout: 10  //OPTIONAL, login timeout for the server
  };

  MSSQL.connect(sqlConfig)
  .then(() => {
      console.log('Did connect');
      //  let query = 'UPDATE USERS SET Status=2;  // works perfectly
      let query = 'SELECT TOP 1 * FROM USERS';  // not working
      MSSQL.executeQuery(query)
      .then(rows => {
          console.log(rows);
          MSSQL.close();
      })
      .catch(error => {
          console.log('Query error!', 'Querry: ' + query + '\r\nError: ' + JSON.stringify(error));
          console.warn(error);
          MSSQL.close();
      });
  })
  .catch(error => {
      Alert.alert('Connect error!', JSON.stringify(error));
  });

I'm getting these errors

Did connect
-------------------------------------------
App.js:82 Query error! Querry: SELECT TOP 1 * FROM USERS
Error: {"framesToPop":1,"nativeStackAndroid":[],"userInfo":null,"code":"REACT-NATIVE-MSSQL"}
-------------------------------------------
Error: byte[] cannot be cast to java.lang.Object[]

Could you please provide any help about this issue ?

Hi,
That error looks to be a problem converting one of the fields in your users table. Specially it looks like you have a byte array field and the plugin isn't able to convert that to something react native can understand. Can you confirm this?
You could bypass this issue by not selecting that particular column, but if you can provide me with the columns data type and I update the plugin to hopefully handle it.

What a rapid response and solution @davidstoneham  .. thanks
it was solved by not selecting a particular column as you mentioned
I found that the cause was the column (Sid)
many thanks again

I just happened to be repling to emails and the github notification came in at the right time.
Thanks for the info and great to hear the work around works. I'll take a look at parsing varbinary data types for the plugin.

I've just tested many types and found that (timestamp) columns causes the same issue


Emscripten build (demo, quick and dirty)

This is a very ugly hack to demonstrate build with Emscripten
Not intended to merge as it will just ruin the beauty of the implementation and most likely there is a much better way to do this. Mainly for educational purposes.
emcc -O3 run.c \
  -o web/llama2.js \
  -s EXPORTED_FUNCTIONS='["_main", "_main_loop", "_malloc", "_free"]' \
  -s EXPORTED_RUNTIME_METHODS='["ccall"]' \
  -s ALLOW_MEMORY_GROWTH=1 \
  --preload-file model.bin \
  --preload-file vocab.bin

The vocab.bin is generated in a similar way explained in #9
The produced artifacts will be generated in the web subfolder.
Example: https://ggerganov.com/llama2.c

Very cool ty! I'm not as familiar with emscripten, I'll take some time to look at this and whether there is a way to support it gracefully.

@ggerganov one thing I really like is the tokenizer inside C here, removing the need to use the run_wrap.py and sentencepiece. But it's not clear how you produced vocab.bin, presumably it's an export script?

@ggerganov one thing I really like is the tokenizer inside C here, removing the need to use the run_wrap.py and sentencepiece. But it's not clear how you produced vocab.bin, presumably it's an export script?

nvm got it working in https://github.com/karpathy/llama2.c/commit/3bfa5665d1bd278041cf272bd46db19129d6c3fd

Yes, for this PR I had exported the vocab from llama.cpp, which has been generated with a similar script as yours.
I'm not sure about the cause of the leading space - we have it in llama.cpp output as well and haven't dug to understand how to fix it yet.

The tokenizer stores a leading space on some tokens.
In python we use the Decode function, that removes the leading space on the first token
But this repo is using the id_to_piece() function to export the tokenizer
python3
>>> import sentencepiece as spm
>>> sp = spm.SentencePieceProcessor(model_file='tokenizer.model')
>>> sp.id_to_piece([9038])
['▁Once']

Then it is simply printed, without considering if it is the first token:
https://github.com/karpathy/llama2.c/blob/f5650891d53e15605bb066a978c0182b12e91cd8/run.c#L471

I recreated @ggerganov's changes against the current tree (with support for prompt): https://github.com/gohai/llama2.c/commits/emscripten

I've built a JS framework around this idea to run Language Models on Web.
This also leverages Emscripten to compile llama2.c into WASM JS.
llama2.c on Web Demo: https://rahuldshetty.github.io/ggml.js-examples/llama2_tinystories.html
ggml.js Framework: https://github.com/rahuldshetty/ggml.js


Reset LookAndFeel in ImageJ1Helper

When using the @Parameter notation for a File in a fiji plugin - the look and feel is changed on Windows (e.g. see this issue: https://github.com/mobie/mobie-viewer-fiji/issues/525). This is due to the LegacyUI chooseFile calling IJ1Helper functions here.
E.g. for ij1Helper().getDirectory() this calls the imagej1 DirectoryChooser here. This changes the look and feel here to the system one - which is not the same as the one fiji opens with for Windows.
Would it be possible to record and reset the swing look and feel for all the IJ1Helper functions called in chooseFile? i.e.  ij1Helper().getDirectory(), ij1Helper().saveDialog() and ij1Helper().openDialog()? Thanks!

Great suggestion, @K-Meech!
In my opinion, the general issue is that ImageJ (1.x) modifies the look and feel at all, instead of respecting the current one. But there are also plugins out there that change the LaF globally. But modifying IJ1Helper to avoid as much as possible any changes to the global state is certainly desirable.
Do you think that remembering the current LaF and resetting it after this line:
https://github.com/imagej/imagej-legacy/blob/0944272eb82ecc39ac9ceb8b7b63f6d6ce711155/src/main/java/net/imagej/legacy/IJ1Helper.java#L1353
... would do it?
Or otherwise (and maybe better?), suggest a change to DirectoryChooser in the original ImageJ to be less intrusive?

I think that would solve it yes - thanks @imagejan! Although, I imagine this would also have to be done for parts that use the ImageJ1 SaveDialog / OpenDialog as well - I think they do the same setting to system LAF.
Modifying it directly in the original ImageJ is probably the cleanest solution though! Where would I suggest this change? I'm happy to put a PR together, but I wasn't sure which repository to do it against?

Let's ping @rasband here:
Wayne, do you think the DirectoryChooser can be modified in a way to reset the original look-and-feel after temporarily using the system look-and-feel for the folder choice dialog on Windows?

The DirectoryChooser in the latest ImageJ daily build (1.53o15) restores the original look and feel after temporarily using the system look and feel on Windows.
The commit is at
https://github.com/imagej/imagej1/commit/834d41b87163c879dc623ab2ee4d7121d6b2ea07

Great - thanks @rasband!


fix: entry width calculation with separator

Fixes a small error in the entry width calculation related to the separator width.
This was leading to ever worsening trailing spaces at the end of entries as we add more display_stat items.
Before:

After (with fix):

(btw experimenting with adding owner/group fs_stat info but the width can't be static due to non-deterministic username lengths so not super simple - maybe not worth it)
I've tested this using a mix both options for git_status and disable_devicons and looks good.

btw experimenting with adding owner/group fs_stat info but the width can't be static due to non-deterministic username lengths so not super simple - maybe not worth it

It's not super simple, but shouldn't be too hard either. It's mostly dependent on the largest file name. In other words, in your screenshots of the top off my head it would just fit without issues.
The problem is more that we'd need to add a ranking and omit fs_stat in some order if the space is too narrow.


[Event Change] Codeunit::"Item Jnl.-Post Line" - Function InitValueEntry - OnInitValueEntryOnAfterAssignFields

We like to request the following Event:
Object: Codeunit::"Item Jnl.-Post Line"
Function: InitValueEntry
Publisher Event: OnInitValueEntryOnAfterAssignFields
Reason: add an extra Parameter
Where is the event placed:
Event Code:
[IntegrationEvent(false, false)]
local procedure OnInitValueEntryOnAfterAssignFields(var ValueEntry: Record "Value Entry"; ItemLedgEntry: Record "Item Ledger Entry"; ItemJournalLine: Record "Item Journal Line")
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


[BUG] Not authorized users can authenticate in client

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
When an user will be log in at a client that isn't authorized, the Keycloak deny the access, but if this user log in a client that are been authorized, this same user can log in everyone client while the session is valid.
Expected Behavior
The role validation must be run in everyone authentication maked by user
Steps To Reproduce
No response
Version
- Keycloak: 19.0.1
- This extension: 19.0.0

Anything else?
No response

Can you please post you flow configuration? It may be a misconfiguration in combination with the cookie authenticator.

Yes, follows.

The behaviour that you experience is not an issue of this authenticator. It's due to how your flow is configured.
If a user is not authenticated, the Cookie authenticator will be skipped, because no SSO cookie exists.
Kerberos authenticator will be skipped as well, because it is disabled.
Identity provider redirector will most likely also be skipped.
Last alternative is 2FA forms subflow.
If the user logs in to a client that is not restricted, the Restrict user authentication on clients authenticator will return successfully. The user will be logged in and a SSO cookie will be set.
Now, if afterwards the user tries to login to a restricted client, a cookie is present and the first alternative (the Cookie authenticator) will return successfully. Since it is an alternative, no further authenticators or subflows will be executed.
Please make sure to configure the authenticator as described in the README.md (see this picture).
Alternatively, this issue could serve as a good example for your use case.


Fix performance degradation for apriori, knn_train, pca_transform

make test

Performance was checked, it's ok. ID: e9ebfd48-03c3-f127-8d16-8cdcd4b723a1


Is it possible to show cell separator?

I trying to add separator between field so would like to know if there is any option show the line?

you'll need to hook into the tableview separatorColor property.


fIXED MAIN PAGE LINKS

FIXED MAIN PAGE

👷 Deploy Preview for micks-first-blog processing.

Name
Link

🔨 Latest commit
dcbb58180ddb65ead6a9420246591cb4b432429d

🔍 Latest deploy log
https://app.netlify.com/sites/micks-first-blog/deploys/65e3682d1e07ab0008b38943


[DRAFT] OpenXR: Use the XR_FB_foveation_vulkan extension to generate the density map for VRS

This builds on top of PR https://github.com/godotengine/godot/pull/99551
But instead of manually building the density map, it uses the XR_FB_foveation_vulkan extension to get the density map from the OpenXR runtime.
So, you can use the same foveation settings that are currently used for OpenGL (setting the "Foveation Level" or "Foveation Dynamic"), and that should get used by the OpenXR runtime in order to build the density map that we're using.
I'm not crazy about the interdependencies this PR makes between OpenXRFBFoveationExtension, OpenXRVulkanExtension and OpenXRSwapChainInfo, but I'm not sure there's a way around it? I'll spend some more time thinking about if there's a more decoupled way to do this.
This marked as a DRAFT both because it depends on PR https://github.com/godotengine/godot/pull/99551, which isn't merged, but also it's just kind of hacking itself into the renderer. We need some way to use the manually built density map when the XR_FB_foveation_vulkan extension isn't available.

I'm really on the fence about this one. Was just talking to @DarioSamo about density maps in general, what my ideas are for improving what we have right now, and if we do this right, our build in solution will be as good as, if not better than what is on offer here. And it works on any platform (well currently any Vulkan based platform).
So far the feedback I've gotten talking to people in the know, most game engines have already solved this the same way we have, with their own logic for generating density maps, because they ran into the same problems we did with what OpenXR offers. Most vendors this don't have a need to supporting something in the runtime. Especially when looking at PCVR vendors who are dealing with a wide hardware spectrum. That is not to say that in some point in the future we will see a vendor neutral extension for this but thats a big unknown.
So doing structural changes to our rendering engine to support an edge case currently only supported by Meta, and potentially not something that will be introduced by others, while we already have a viable solution that we can improve open is.. uhm.. Meh..
The other thing that is a potential problem here, is that the density map has to be adjusted to the resolution at which we're rendering, and since we're allowing that to be overridden, we might not be getting a density map at the right size.

I think there's a few advantages to using the density map from OpenXR:

The "dynamic" feature of foveated rendering works! I tested it by doing two RenderDoc captures: one where I was looking at a part of the scene where it tanked the FPS (down to like 36fps), one where I was looking at a single color quad (and fps went back up to 72fps). I could see that density map changed, so that it was really scaling down resolution in the first capture, but was a pure (1.0, 1.0, 0.0) texture in the second (so not scaling resolution down at all)
The density map can be geared to the hardware. When looking at RenderDoc with this PR, Dario said that the binning stage was much shorter with this PR than with the custom density map from his PR. He said something like he thought it had an easier with the density map from OpenXR because it was "more binary" (ie less gradient). But this is something that would be hardware specific - maybe different hardware would be able to work better with differently generated maps?
Because "dynamic" works, I suspect that XR_META_foveation_eye_tracked would work as well, allowing the OpenXR runtime to update the density map based on eye position, and it'd be able to use the right "filters" and latency for foveated rendering. Our manual density map generation can use XR_EXT_eye_gaze_interaction, but that really isn't the right extension for foveated rendering: it uses "filters" and latency that are designed for interacting with UIs, which means it will prioritize things like stability (ie not bouncing around too quickly), rather than prioritizing the things that matter for foveated rendering (like latency)

So, I'd really like to be able to have the option to let the OpenXR runtime generate the density map!
I'm not crazy about my implementation so far, because it couples a whole bunch of things that I don't want coupled, but that's hopefully something I can work out before taking this out of draft.

@dsnopek the problem is, the solution Dario and I were discussing tonight kind of changes things a lot. What we're thinking of is to leave providing a texture for non XR usage only, and instead provide eye center and radius data if XR_VRS is specified, and generating the density map in the shader that is currently in the rendering engine (and eventually move that into the rendering driver so that we're independent of hardware/GPU architecture).  It's not a big step to make the density map adjust based on frame timing.
I do hear what you're saying, but many of those issues are because we need to improve our implementation, and so far its just not gotten the attention it deserves. Solving that properly means that we have a good solution for all platforms, and seeing this extension does not seem to be getting wide adoption, we're going to have to do that anyway.
There is a difficulty that right now those with the knowhow, aren't sharing, so its going to take more experimentation to get it right. But its worth doing.
My only concern is that right now, there is no proper extension in OpenXR that allows us to get usable eye tracking data, so supporting eye tracked foveated rendering alongside fixed foveated rendering is a problem. We're using the eye gaze extension which is not really suitable for this.
That is definitely a place where using this extension is a plus.


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/69781

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/70017

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/70724

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/71112

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/71140

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/74375

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/78640

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/100988


Fixed #26381 -- Made UserCreationForm reusable with custom user models that define USERNAME_FIELD.

https://code.djangoproject.com/ticket/26381

As USERNAME_FIELD is documented as mandatory I'm not sure it's worth raising a more descriptive exception.

merged in efa9539787dbdd06cd2169023edcf7db3e2ff0c4, thanks!


Fix broken slide when slide content is wrapped in a tag. issue #1316

Related to the issue from @selrond https://github.com/akiran/react-slick/issues/1316#issue-341951239
As with IMG, we should be able to swipe when there is an A tag in the slide content. A tag became draggable when href is setted.

Why closed this? @Xb0X

@Xb0X @tranlehaiquan I am still seeing an issue with this behavior in production, this fix was never merged?

Why wasn't this merged?


[Bug Report] Play with .png file attachment

Bug Description
Executing play command with URL [link]
displays

and bot immediately disconnects from a voice channel.
Steps to Reproduce

!play https://cdn.discordapp.com/attachments/1029873804924694561/1064237733339480074/donkeh.png

Expected Result
Bot prints out
Error loading: Unknown file format
and doesn't leave a voice channel.
Debug Output
  java.version = <IP_ADDRESS>
  java.vm.name = OpenJDK 64-Bit Server VM
  java.vm.specification.version = 17
  java.runtime.name = OpenJDK Runtime Environment
  java.runtime.version = <IP_ADDRESS>+1
  java.specification.version = 17
  os.arch = amd64
  os.name = Windows 10

JMusicBot Information:
  Version = 0.3.8
  Owner =<PHONE_NUMBER>08066911
  Prefix = !
  AltPrefix = null
  MaxSeconds = 0
  NPImages = false
  SongInStatus = false
  StayInChannel = false
  UseEval = false
  UpdateAlerts = true

Dependency Information:
  JDA Version = 4.4.0_352
  JDA-Utilities Version = 3.0.5
  Lavaplayer Version = 1.3.78

Runtime Information:
  Total Memory = 37
  Used Memory = 16

Discord Information:
  ID =<PHONE_NUMBER>045207602
  Guilds = 2
  Users = 4

Additional Info
Bot behaves as expected for some other URLs containing .png files, eg. link
Checklist

[X] I have looked for information about this within the documentation
[X] I have searched for similar issues on the issues page
[X] I am running the latest version of the bot:

I would like to work on a pr for this bug if nobody else minds.


[BUG] gmsh discards labels equal to 0

From @Laar:
Gmsh doesn't really like having 0 as a label. So using 0 as a label for a region means that everything about the regions gets discarded if I load it in gmsh itself.

Gmsh documentation states "All the node, element and entity tags (their global identification numbers) should be strictly positive. (Tag 0 is reserved for internal use.)"


Ellipse receives unwanted shadow when created with height=0

When creating an ellipse entity with height=0 the ellipse receives what appears to be a shadow:

When not defining a height or setting it to undefined the shadow is gone but performance drops significantly:

Ive created a simple example in sandcastle that demonstrates the issue:
http://cesiumjs.org/Cesium/Apps/Sandcastle/?src=Hello World.html&label=Showcases&gist=a82de61d54d42a87288835acd7f76c56

Thanks for submitting this @tomermoshe! I believe this may be a duplicate of #1825. Please comment in that issue if you have any additional thoughts. Thanks again!


Resolve build target automatically like Gradle

I'm not sure this is possible, but I didn't realize that if you are using a multiplatform capable library, you have to only depend on the jvm version of it. I was banging my head against the wall until i looked in my external dependencies folder and saw only jvm versions of libraries that gradle auto-resolved. it would be nice if kscript could do the same.
example:
works
@file:DependsOn("com.github.ajalt.clikt:clikt-jvm:3.4.0")
fails during kscript package step
@file:DependsOn("com.github.ajalt.clikt:clikt:3.4.0")

I have not used mulitplatform yet, and won't be of much help here. I guess the support boils down to the kotlin-scripting dependency resolver, which is used under the hood.


README: Change info to information everywhere

Trivial change used to test cla checker

Still no good, sorry everyone. I think we're close but will have to try this at least one more time


Don't block UI when on a wrong chain

What it solves
Resolves #2924
How this PR fixes it
Resolves the 3-click problem by not showing the backdrop and a network switch popup when connected to a "wrong" chain.
However, the Safe will be read-only.
A note: a lot of transaction-related logic depends on Web3 being initialized before a tx-initiating component has been rendered. We're not using hooks enough in this area.
How to test it

Connect to Polygon
Go to a Rinkeby Safe
Safe should be read-only
Switch to Rinkeby
Safe should become active
Switch to xDai
Safe should become readonly

Screenshot

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 3 (33.33%)  changed or added relevant lines in 3 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.06%) to 31.641%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/components/AppLayout/Header/components/Layout.tsx
0
1
0.0%

src/components/AppLayout/Header/index.tsx
0
1
0.0%

Files with Coverage Reduction
New Missed Lines
%

src/components/AppLayout/Header/components/Layout.tsx
1
0%

Totals

Change from base Build<PHONE_NUMBER>:
0.06%

Covered Lines:
2976

Relevant Lines:
8365

💛  - Coveralls

I'm thinking maybe also to add a warning on top asking to switch the network

Yep, it seems to add value after this change. And the goal is achieved: do not block the UI

to me it feels much better than the status quo! might lead to some users forgetting to switch and wondering why they are in read-only mode or not able to interact with the safe but I feel like prominently displaying the "Switch network" button as u do now should be enough.

might lead to some users forgetting to switch and wondering why they are in read-only mode

I agree 💯. Ideally we should allow users to interact with the app and only ask to switch when we interact with the wallet. Unfortunately, in the current architecture, there's no single point at which we can hook this in.
It will be possible after a refactoring that we've discussed with the devs recently.

Cool, I think this will be an improvement. I wonder if we could add the switch button next to the "Read Only" label? Cause that's where the user might be looking when confused about why they can't fully control a Safe they own.

@KristinaMayman this was my thought too and I actually tried it but it looked cramped in there.
Maybe it's a visual design matter.

@TanyaEfremova should we try to iterate on this?

@katspaugh @KristinaMayman The alert in the sidepanel does look cramped, as there are a lot of components shown (read-only label, address actions, balance, etc.). It is difficult for a user to focus on the right thing. A button in the navigation is sufficient, but the full screen alert bar would be a more elegant and straightforward solution.

I think this is the clearest of all the options. I think it would justify the "READ ONLY" test as well.

@TanyaEfremova @iamacook thanks! I agree, will make it a full-width banner then.

to me the full-width red banner feels like there is an immediate action required, which is not the case and makes using the read only mode uncomfortable. I would like to discuss this is in a short meeting before we change it.

It could be network colour-specific instead of red.

This was my previous attempt before I opted for the header.

Issue:
You can go through the whole process of safe creation while being in the wrong network. You can actually pay the fee and see the safe "being created" when actually it will never be (since you paid in the wrong network)
Keeping the spirit of this ticket to make things read only, we have to make the safe creation form unavailable while being in the wrong network (a soft read only status kinda thing)

@francovenica fixed ✅

Works like a charm.
The continue button is grayed out
The "switch to network" button works fine
I've checked that even resets the process if you are in the last steps of the safe creation and the user decides to change network


Analyst - Expand all & collapse all application sections

As an analyst
I want to expand and collapse all the application sections with a single click
so that I don't have to individually open/close them one by one.
Acceptance Criteria:
Screen

Wireframes

[ ] KEEP THIS: by default the tabs are all closed except for the tabs where there is missing info

Page Behaviours

[ ] Expand all and Collapse all options are both visible on the page at all times
[ ] The expanded and collapsed state resets to default state for the accordion when applicant clicks away from the Application subpage

Actions

[ ] Clicking "Expand all" opens all sections of application
[ ] Clicking "Collapse all" closes all sections of application including the ones that are missing info

[ ] Make a PR with the update to the Button Toolkit

Not included:

.
.

Ultra Refinement

We're using the toolkit accordion. Has the functionality for default expanded/collapsed, look into what's preventing this to be done to have it be dynamically controlled.

DB:

.
.

Middleware:

.
.

Frontend:

.
.

Tests:

.
.

Documentation:

.
.

Hey @AntBush I noticed that this is already in prod?

Yeah I think this was forgotten to be added in the review column

I don't think we explicitly said this in the ticket but should the plus signs change to minus signs when we click 'Expland all' and vice versa to be consistent with how the folders behave?

@AntBush can we get 4px margin below so it's not touching?

I've created a PR for this: https://github.com/bcgov/CONN-CCBC-portal/pull/1376


Bump version to v8.4.0

Jenkins, test this please

Jenkins, test this please

Jenkins CI test (for commit 870a32a0f6b3447f9b005903c1915f35ac39db24, target branch v8) failed with status FAILURE.

Jenkins CI test (for commit 870a32a0f6b3447f9b005903c1915f35ac39db24, target branch v8) failed with status FAILURE.


Added unit tests

In this pull request the following was done:

Unit tests were added to the calculate.js and operate.js functions.

Hi @AbelG101
My partner has not made any contact with me this week. I am currently reaching out to student success to ask them for guidance.

Hello @dvcorreia1984 In the future please attach the screenshot in your PR description so we don't cause any inconvenience by Invalidating your PR.


google prototype

:white_check_mark: Build TimeHACK AppVeyor-152 completed (commit https://github.com/TimeHACKDevs/TimeHACK/commit/fc17fea7fd by @lempamo)

:white_check_mark: Build TimeHACK AppVeyor-153 completed (commit https://github.com/TimeHACKDevs/TimeHACK/commit/bc9db2deda by @lempamo)


AC Fixed sample readme update

Restructed to match the new template, with differences for FPGA templates. Made minor changes to some formatting. Updated some branding. Added category to sample. Corrected some grammar and spelling issues.
Existing Sample Changes
Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes Issue#
External Dependencies
N/A
Type of change
Please delete options that are not relevant. Add a 'X' to the one that is applicable.

[x] Documentation quality

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[x] Command Line
[x] VSCode

Adding @jimmytwei for notices.


How payment works

Hello,
I configured all the plugin to work with a production envinroment.
I tried to make a payment using a TopUp credit card without any money availability, however the payment went fine... and I received this result
Object
applePayCard: null
card: Object
lastTwo: "XX"
network: "BTCardNetworkVisa"
Object Prototype
localizedDescription: "ending in XX"
nonce: "f9XXXXXX-7d93-XXXa-XXXX-825db0bXXXXX"
payPalAccount: null
threeDSecureCard: null
type: "Visa"
userCancelled: false
venmoAccount: null
Am i not understanding how it works? Have I to call other methods to validate the payment?
N.B. there is no tracks of the transaction inside the braintree portal.
Thanks
Michele

Little update,
I tried also with a wrong CCV and it worked...
Another think is that it sounds strange to me it that it does not require the card credit holder details...
Thanks
Michele

Please read Braintree docs. The plugin only prepare the transaction, you have to settle it thru your server.


Area of a rhombus

Title of article
Write a C# program to compute area of Rhombus

Takes the length of two diagonals
Uses the formula to calculate the area
Return the value

Area of a rhombus = ( diagonal 1  *  diagonal 2 ) / 2
Additional information
Example:
Diagonal 1 = 6, Diagonal 2 = 8
Area of the rhombus = (6*8)/2 = 24
Programme
Diagonal 1 : 6
Diagonal 2 : 8

Area : 24

Code of Conduct

[X] I have read and agree to the Codinasion project's Code of Conduct

i'm doing this.. @harshraj8843


takeUntil operator cannot complete a StateObservable

Description
The takeUntil operator doesn't work when applied on a StateObservable.
Playground
To demo the bug, I have created the following playground: https://codesandbox.io/s/react-rxjs-bug-takeuntil-1-d69w72
You should expect the counter to stop incrementing when pressing the STOP button.
Environment
rxjs 7.6.0
@react-rxjs/core 0.10.3
@react-rxjs/utils 0.9.5
react 18.2.0

Fixed at<EMAIL_ADDRESS>


"Unknown location" OpenStreetMap explanation

I saw the tweets because it sometimes says "Unknown location, do you know it? Tell @openstreetmap" e.g. https://twitter.com/Sentinel2Bot/status/785543415547191296
what does this mean? and what are we supposed to be telling OpenStreetMap?
...is my question. I guess the answer might be added to the README here

Good point. My idea was that someone might recognise the place in the picture and then take the time to update what ever it is that needs updating in @openstreetmap so that next time the reverse geocoding would work.
From watching images for which it doesn't work it seems most are above the sea, so maybe not that useful.

Yeah. It sort of sounds like a call to action, but then the action isn't entirely clear. Is there's a higher level in the geocoding result which you could look at. If not, then maybe that's an interesting use-case which could be a issue to raise on geocoder (are you using nominatim?)
What I mean is...  I guess ideally for this use case, reverse geocoding should be able to say something about every location on the planet, even if it's pretty vague: "atlantic ocean", "arctic circle", "siberia"

Yes using nominatim (ugly code). Using the display name property and shortening it when it doesn't fit in a tweet.
Here some ideas on a better tweet text:

Unknown location, do you know it? Tell @openstreetmap
@openstreetmap doesn't know the name of this place, do you? Tell them
Do you know the name of this place? Tell @openstreetmap
Do you know the name of this place? Tell @openstreetmap
Teach @openstreetmap what this place is called:
Where is this? Make @openstreetmap smarter:


Release mixed up

https://github.com/jimradford/superputty/releases/tag/1.4.10 somehow mixes up with <IP_ADDRESS>

Yup, I tagged it wrong. The assembly version is correct.

... not it keeps asking if one wants to update!

You can turn that off in preferences for now. If you are using that tagged release it shouldn't be asking to update?

@jimradford I think next version should be 1.4.11 or <IP_ADDRESS>, to avoid confusion with version sorting.


Version 3.22 LIVE updates

Update status:

[x]  English
[x]  French - France
[x]  German - Germany
[ ]  Italian - Italy
[x]  Portuguese - Brazil
[ ]  Spanish - Spain

English, German and Brazilian in #203

French done in #192


Update actions

Run go mod tidy in check-generate target.
Update some actions.

Currently the following actions are used.
$ find .github/* -type f | xargs grep uses: | cut -d':' -f 3 | sort -u
 actions/checkout@v2
 actions/download-artifact@v3
 actions/setup-go@v3
 actions/setup-python@v3
 actions/upload-artifact@v3
 azure/setup-helm@v1
 goreleaser/goreleaser-action@v2
<EMAIL_ADDRESS><EMAIL_ADDRESS> rajatjindal/krew-release-bot@ba5e167fbd1526240f5f5d2eaf1ec573fc5942de

Signed-off-by: Masayuki Ishii<EMAIL_ADDRESS>

@yokaze
Good review! I will update the python version in the next PR.
https://github.com/cybozu-go/moco/pull/399


i18n with Twig 1.0

Fatal error: Declaration of Twig_Extensions_Node_Trans::compile() must be compatible with that of Twig_NodeInterface::compile() in /media/DATA/Main/Dropbox/Chas/Perrong/Billboo/prototype/lib/Twig/Extensions/Node/Trans.php on line 124
I get that error running Twig 1.0 on PHP 5.3.3. It appears to be fixed by adding the parameter type Twig_Compiler to the Twig_Extensions_Node_Trans::compile() method. This might be to do with how my PHP is configured, just though you should know.

@fabpot this issue should be closed


output chi map as FITS extension 6

This PR outputs a map of the chi-values chi = sqrt(chi2) = |image - model|/sigma for each pixel as FITS extension 6 (name CHI).
This is useful as it is a statistically meaningful "relative error" map. Because each pixel follows a chi distribution with one degree of freedom, plotting the chi map with a scale [0, 2.57585] for example shows where residual pixels are excluded at the 99% confidence level.

Looks cool.