Extra x-axis appearing upon resize

Hi,
Here is my graph. It is required to be resizable:

When I resize it, however, another x-axis appears:

Here is my code. Thanks!

Turns out that the second x axis was always there, it was just hidden initially. I fixed the problem by removing the duplicate axis:
$('.c3-axis-x').eq(1).remove();
Not a great solution, but it works.

I'm experiencing this too after updating to 0.4.11-rc1 in Axis, and I don't even need to resize the browser window. Investigating a bit further, it's actually the subchart x-axis, and changing the style property from visibility: hidden to display: none hides it. I'm not sure what the issue is; if I just rip that bit of code out and put it in a jsFiddle, it's still visible, even without any CSS to override it or anything. Rather confused, behaviour persists in both Firefox and Chrome.

I've figured it out, pretty edge-case-y — if somehow subchart.visible is false and subchart.x.axis.show is true, and the SVG area gets extended, the subchart axis is visible. PR in the works...

Actually, this is way less edge-case than I thought — subchart.x.axis.show is truthy by default, all you need to do is extend the SVG area and it's visible.

For now you can get around the issue by configuring the chart with subchart: { axis: { x: { show: false } } }


[firebase_admob] Upgrade SDK version from 18.1.1 to 19.1.0

The firebase_admob package version 0.9.3+2 uses SDK 18.1.1 on Android, which was released almost a year ago. I can see the following line in the build.gradle file:
dependencies {
api 'com.google.firebase:firebase-ads:18.1.1'
}
But the Google Mobile Ads SDK is already in version 19.1.0:
https://developers.google.com/admob/android/rel-notes
Can you estimate when this version will become available? Is it safe to manually edit the line from "flutter.pub-cache\hosted\pub.dartlang.org\firebase_admob-0.9.3+2\android\build.gradle" to use the latest version of the SDK?
We're using Open Bidding Beta, and we need to use the latest version to avoid adding extra parameters to the AndroidManifest.xml.
Thanks.

Adding on top of this, when we try to add the adapters of third party networks like AppLovin or AdColony via gradle, which is required to add them to the bidding process, we get an error due to incompatible dependencies:
`Could not determine the dependencies of task ':app:preReleaseBuild'.

In project 'app' a resolved Google Play services library dependency depends on another at an exact version (e.g. "[17.0.0]", but isn't being resolved to that version. Behavior exhibited by the library will be unknown.

Dependency failing: com.google.android.gms:play-services-measurement-api:17.0.0 -><EMAIL_ADDRESS>but play-services-measurement-sdk-api version was 17.1.0.
The following dependencies are project dependencies that are direct or have transitive dependencies that lead to the artifact with the issue.
-- Project 'app' depends on project 'firebase_admob' which depends onto<EMAIL_ADDRESS>-- Project 'app' depends onto<EMAIL_ADDRESS>-- Project 'app' depends onto<EMAIL_ADDRESS>-- Project 'app' depends onto<EMAIL_ADDRESS>-- Project 'app' depends onto<EMAIL_ADDRESS>-- Project 'app' depends onto<EMAIL_ADDRESS>-- Project 'app' depends onto<EMAIL_ADDRESS>-- Project 'app' depends on project 'firebase_analytics' which depends onto<EMAIL_ADDRESS>-- Project 'app' depends onto<EMAIL_ADDRESS>-- Project 'app' depends onto<EMAIL_ADDRESS>-- Project 'app' depends onto<EMAIL_ADDRESS>-- Project 'app' depends onto<EMAIL_ADDRESS>Their adapters require the lastest version of the Google Mobile Ads SDK. Do you have any recommendation for this case?

I can attest to what is written above, trying to add the latest mediation adapters(FAN etc.) to an Android project is not pretty. If a newer version of the AdMob SDK was used it would make everything go a lot smoother.

I'm  https://stackoverflow.com/questions/62325206/flutter-app-integrating-adcolony-with-mediation this issue now, can someone help me which version I should add to work adcolony?

@spremathilaka There's probably a way you could hack this by using resolutionStrategy.force in your build.gradle but I would suggest against it. I did it in a React Native project a couple of years ago and it became really annoying to maintain - not to mention this kind of solution won't help with the iOS side.
I recently added mediation to a Flutter project this week and this is what I did:

Removed the firebase_admob library from my pubspec.yaml
Added the native iOS dependency to my Podfile
Added native(Swift) calls to load/show ads and the relevant MethodChannel calls in Dart
Added the mediation adapters to my Podfile

You could do the same on Android above you'd just add the dependencies to your build.gradle file instead. This obviously won't help with banners either only interstitials.
I would rather just use firebase_admob than do the above but the version of AdMob it uses is too outdated and it causes all sorts of conflict issues with the latest mediation adapters.

any updates here?

The firebase_admob plugin will be deprecated in April 2021 in favor of Google Mobile Ads SDK for Flutter.
Google Mobile Ads SDK for Flutter is a new Flutter plugin that supports more Ads formats than firebase_admob does. Google Mobile Ads SDK for Flutter currently supports loading and displaying banner, interstitial (full-screen), native ads, and rewarded video ads across AdMob and AdManager. It also supports displaying banner and native ads as Widgets as opposed to being overlayed over all app content.
We're now recommending projects currently using firebase_admob migrate to Google Mobile Ads SDK for Flutter following the instructions outlined here.
Issues with migrating or with using the new package should now be raised over on the Google Mobile Ads SDK for Flutter repository.


Add slack:notify:failure task

Q
A

Bug fix?
No

New feature?
Yes

BC breaks?
No

Deprecations?
No

Fixed tickets
N/A

Do not forget to add notes about your changes to CHANGELOG.md

#158
Please update docs.


Create hns-re2sd.txt

http://www.hns-re2sd.dz

@mehdaoui-A First stage of verification passed, official domain is hns-re2sd.dz as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@mehdaoui-A I am closing this request as a duplicate to #18923


Picture capturing with RTSP stream

To capture pic, another solution would be to get the RTSP stream and extract a JPG from there.
But for that we couldn’t find out how to install a package like FFMPeg…
from: Hans Stanglmayr

Closing issue, please re-open if issue still exists.


At least 1 report directory should be mandatory for parse command

Do not accept 0 positional arguments for parse command as it can lead misleading behavior as it will not find reports as either CodeChecker analyze or  CodeChekcer  check has default output report directory.
So at least 1 positional arguments must be mandatory.
usage: CodeChecker parse [-h] [-t {plist}] [-e {html}] [-o OUTPUT_PATH] [-c]
                         [--suppress SUPPRESS] [--export-source-suppress]
                         [--print-steps]
                         [--verbose {info,debug,debug_analyzer}]
                         [file/folder [file/folder ...]]

Parse and pretty-print the summary and results from one or more 'codechecker-
analyze' result files.

positional arguments:
  file/folder           The analysis result files and/or folders containing
                        analysis results which should be parsed and printed.
                        (default: /home/ednikru/.codechecker/reports)

This heavily contradicts the default behaviour of CodeChecker analyze which will use the folder under the "workspace" for output.
Why are we making it so hard for people who just want to use CodeChecker with default settings?


Support OpenAPI Specification

Support OpenAPI Specification
We need to implement OpenAPI support in the ebrick framework to standardize API documentation and provide integration for client-side tools and services.
Requirements:

Provide OpenAPI documentation for all exposed endpoints.
Use the OpenAPI 3.0 (or higher) specification.
Automatically generate OpenAPI documentation from route definitions and models.
Ensure real-time sync between API changes and documentation.
Support interactive API documentation via tools such as Swagger UI or ReDoc.

Some aspects need consideration:

Whether to enable API docs generation for any specific module or all modules.
Whether to generate API docs on the application booting-up phase or run a specific command line to trigger.
Whether to automatically generate API docs by source code analysis or using Gin Swagger and Swaggo with user manually inputted General API Info Comment


Binding network bridge to externally accesible interface

Hi,
I know this is a usage question but I'm stuck on this. I see that theres some new features in Docker that now enables this:
# create a new bridge network with your subnet and gateway for your ip block
$ docker network create --subnet <IP_ADDRESS>/24 --gateway <IP_ADDRESS> iptastic

This created a new bridge network interface on my Docker host that looks like this:
br-b255cf4703fc Link encap:Ethernet  HWaddr 02:42:66:B6:77:D3
inet addr:<IP_ADDRESS>  Bcast:<IP_ADDRESS>  Mask:<IP_ADDRESS>
UP BROADCAST MULTICAST  MTU:1500  Metric:1
RX packets:0 errors:0 dropped:0 overruns:0 frame:0
TX packets:0 errors:0 dropped:0 overruns:0 carrier:0
collisions:0 txqueuelen:0
RX bytes:0 (0.0 B)  TX bytes:0 (0.0 B)

I then created a new container with a static IP address like this:
$ docker run --rm -it --net iptastic --ip <IP_ADDRESS> nginx

Since I'm running my Docker host machine (RancherOS) in Virtualbox, is there a next step to bind the newly created network and associated container to the existing externally accessible interface, in this case eth1 ?
So far so good.

If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
For more information about reporting issues, see CONTRIBUTING.md.
You don't have to include this information if this is a feature request
(This is an automated, informational response)

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

OK - for those who hit this issue. This post worked perfectly for me.
http://ispyker.blogspot.co.uk/2014/04/accessing-docker-container-private.html?showComment=1458315067913#c1119300992631997718

looks like you found your answer, so I'll close this. For these kind of questions, forums.docker.com may be a better location

Thanks, actually it was even simpler than the blog. I just did this from my Mac's OS.
sudo route -n add <IP_ADDRESS>/16  boot2docker ip``


how to delete an expiration date from a permission with GraphAPI?

The expirationDate is optional, but cannot be null

should be fixed by #145


PR #79 open threads

Closes: https://github.com/tasitlabs/tasit/issues/84
Note: I'm assuming that "integration test boostrap" should be done after v0.1.0. Am I right?

Note: I'm assuming that "integration test boostrap" should be done after v0.1.0. Am I right?

Yep, I think so. So should we split that out into a separate issue?

Yep, I think so. So should we split that out into a separate issue?
Yes, I'll create an issue.

Note: Last CI failed, I've just committed the fix but seems that CircleCI is facing instability right now.

Integration test issue: https://github.com/tasitlabs/tasit/issues/86


Download CSV Functionality is Broken

Bug report
Download CSV Functionality is no longer working correctly
What is the actual(incorrect) behavior?
Clicking on Download CSV takes the user to a blank screen with no file being downloaded.
What is the expected(correct) behavior?
The screen should not change and a CSV file should start to download.

Its not broken anymore I fixed it.

This is already prod so I am going to close it.


v1.11 backports 2021-11-26

[x] docs: update Helm reference after updates for latest -rc release (cc @aanm)
[x] #17983 (@borkmann)
[x] #17975 (@errordeveloper)
[x] #17973 (@borkmann)
[x] #17968 (@aanm)
[x] #17969 (@aditighag)
[x] #17995 (@aanm)
[x] #17993 (@joestringer)
[x] #17929 (@brb)
[x] #17813 (@jibi)
[x] #17856 (@gandro)
[x] #18008 (@aanm)

Once this PR is merged, you can update the PR labels via:
$ for pr in 17983 17975 17973 17968 17969 17995 17993 17929 17813 17856 18008; do contrib/backporting/set-labels.py $pr done 1.11; done

/test-backport-1.11

On hold, needs a rebase on top of https://github.com/cilium/cilium/pull/17986. I'll mark as draft for now.

/test

/test-backport-1.11

/test-backport-1.11

k8s-1.21-kernel-4.19 hit https://github.com/cilium/cilium/issues/18014

The gke-stable failed because there was multiple IP addresses in cilium-host interface. I assume it will be fixed by https://github.com/cilium/cilium/pull/17762

Thanks André! Summary:

gke-stable is https://github.com/cilium/cilium/issues/17307 and should be fixed by https://github.com/cilium/cilium/pull/17762
k8s-1.17-kernel-4.9 looks like a hiccup (Recording test results\nNo test report files were found. Configuration error?)
k8s-1.18-kernel-4.9 is due to a failure to download kubectl in a BeforeAll block
k8s-1.19-kernel-4.9 is https://github.com/cilium/cilium/issues/13071
k8s-1.21-kernel-4.19 is https://github.com/cilium/cilium/issues/18014
k8s-1.22-kernel-4.9 is https://github.com/cilium/cilium/issues/17942

I'm still struggling to triage the ConformanceGKE failure. @aanm can you also have a look at the CodeQL failure please? I'm not sure how I should handle it?

/test-gke

/test-1.18-4.9

/test-1.19-4.9

/test-1.21-4.19

/test-1.22-4.9

/ci-gke-1.11

The four Jenkins workflows that failed again all hit https://github.com/cilium/cilium/issues/18060 :thinking:
I've seen this on another PR so it's unlikely related to the current backport, though.


Update README.md

Updates to example documentation, added section for windows cmd equivalents.

@googlebot I signed it!


Support exit code in shjs.

This is similar to #168, but with more targeted changes.

LGTM

LGTM


404 in link to New-AzSubscription

The link to New-AzSubscription ("To see a full list of all parameters, see New-AzSubscription") results in a 404.

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

ID: 371189c2-154f-2716-bbc0-8496a76c5ac9
Version Independent ID: bb5cd103-ff51-1753-f269-e2c1aa49a8c8
Content: Programmatically create Azure Enterprise subscriptions
Content Source: articles/azure-resource-manager/programmatically-create-subscription.md
Service: azure-resource-manager
GitHub Login: @jureid
Microsoft Alias: jureid

@yobyot Thanks for the comment. We are actively investigating and will get back to you shortly with an update.

@yobyot Verified and updated the document as required. Thanks.


Reduce resources for some high-mem tools

Reduce the cores/mem allocations for some high-mem tools to reduce congestion in the high-mem queues.  @Slugger70, @jlqfab what do you think?  We have sacct data but I'm not sure how to tell how efficient the allocations have been for the different tools.
halved:
flye: 120 -> 60
hifiasm: 120 -> 60
racon: 120 -> 60
medaka_consensus_pipeline: 120 -> 60
seqtk_seq: 120 -> 60
raven: 120 -> 60
hisat2: 60 -> 30
busco: 60 -> 30
spades tools: 60 -> 30

quartered:
minimap2: 120 -> 30

taken off high-mem altogether:
quast: 60 -> 16

HI Catherine. I've looked at memory use data in sacct for the first 4 of these. There was no MaxRSS (max memory used) for any of these except hifiasm which showed (on pulsar-qld-high-mem0) usage of ~107.
We could drop memory assigned for all of some of these, and see how they go - sacct has a '--state OUT_OF_MEMORY' query that we could use to pick up which jobs are failing due to being out of memory.

We should look at all of the tools we send to high mem and investigate their scaling with resources and make sensible decisions based upon that rather than across the board reduction.


feat: add required sections

Expected
Add the required sections in the issue #84
Goals

[X] Contact form
[X] Questions
[X] Get in touch

Hi @esau-morais, could you be able to resolve the conflicts? :relaxed: Otherwise I'm not able to merge.

we can't merge in to dev branch and can't run your PR
Failed to compile.
./src/components/contact/questionsList.js


[microTVM] Add Cortex-M DSP schedules for optimal conv2d layouts

This pull request adds fast microTVM DSP schedules for the optimal (for Cortex-M) conv2d and depthwise_conv2d layouts - NHWC/OHWI and NCHW/OIHW, respectively. By letting us use the special SMLAD DSP instruction without rearranging data, 25% of the instructions in the inner loop can be removed (for the int16 case).
Additionally, this change allows both the conv2d and depthwise_conv2d fast DSP schedules to use the same underlying intrinsic, a variation of a tensordot operator. This makes the code for these schedules much more compact. This PR also:

Adds unit tests for the new schedules
Does not affect the old schedules, which are still used when they apply

The cases in which these new fast schedules apply are strictly different

Adds support for int8, int16, and int32 input data types in the new schedules
Adds a change_constant_shape utility to tvm.topi

I've also written a comment below delving into why these layouts are optimal.

In #12856, we discussed how NHWC was a bad format to use for depthwise_conv2d in microTVM (and likewise NCHW a bad format for regular conv2d). From this, one might ask the question:

Given a conv2d on Cortex-M4 with n groups, what are the optimal data and kernel layouts?

When choosing these layouts, we really want data that will be multiply-accumulated to be next to each other in memory. The primary reason is that doing this lets us use the __SMLAD instruction with minimal overhead, which does two multiply-accumulates with one instruction. The secondary reason, however, is to let us use *ptr++ when reading both the input data and kernel as much as possible, as *ptr++ is one instruction on Cortex-M.
For depthwise convolutions, channels do not interact with each other at all, so they have no reason to be near each other in memory. This applies to both the input data and the kernel. Hence, NCHW is the optimal data layout and OIHW the optimal kernel layout. By similar reasoning, NHWC and OHWI are optimal for regular Conv2D operators. But we can generalize further - for a generalized Conv2D with n groups and c channels, the optimal layouts are NCHWxc/OIHWxi where x = c / n.
Now, assume we are performing a generalized Conv2D with n groups and c channels, using data layout NCHWxc and kernel layout OIHWxi. For the int16 case, to convolve one entire row (width * channels / groups individual parameters), all we need to do is copy/paste this code width * channels / (2 * groups) times (the 2 coming from the fact two int16 values fit into an int32).
uint32_t tensor_batch = *tensor++;
uint32_t kernel_batch = *kernel++;
sum = __SMLAD(tensor_batch, kernel_batch, sum);

This code does not depend on the number of groups, which allows us to use the same tensorize function for both regular and depthwise convolutions!

Additionally, while these schedules are finished and ready to use when appropriate, there are still three changes I must make before they contribute to performance improvements on common tiny models (e.g. MobileNetV1 or ResNet).

Add support for non-word-aligned kernels. For int32 input dtype, the current schedules have no limitations. However, for int16 and int8 input dtypes, we require that the kernels must fit evenly into words. For example, this means that our depthwise schedule would not currently work with a 3x3 kernel when used with the int16 input dtype, as it would take up 4.5 words in memory.

Arm Cortex-M does not support fast unaligned memory accesses, so the fix here is kinda gnarly. The int16 case here is important enough that I really ought to add this feature, but the fix for int8 is less useful and requires way more casework. Hence, I'll probably only fix this for int16.

Use the out_layout attribute to allow changing layouts. Many common models (like MobileNetV1) alternate between regular and depthwise convolutions to improve performance. Since we prefer the NHWC data layout for regular convolutions and the NCHW data layout for depthwise convolutions, it would be really nice for our regular convolution schedule to be able to take in data in NHWC and output NCHW when appropriate. This should actually be pretty straightfoward.
Add legalization code for Cortex-M to change the layouts when appropriate.

I plan to address 1 + 2 in one follow-up PR, and 3 in another.

@areusch @AndrewZhaoLuo @ekalda would you mind re-reviewing?

hey @ekalda this is definitely something we should look at. we haven't done that yet though.

Thanks @ekalda! As Andrew said, I've only looked closely at the M4/M7, and these are what I had in mind when writing this schedule.
MVE is super cool though, and it would be straightforward to extend the tensordot kernel to work with it. Definitely worth doing once we're seeing performance improvements from these schedules.


Do we need CI in this repo?

I believe the only reason we had CI in this repo was to test the python scripts for building the bootstrap runtime.

@richlander @mmitche @sunandabalu

It has samples.  Maybe we should be building them?  There is CI available for this repo already.

I think we could build the samples but it hasn't been a problem to date. Are there any other things that CI covers here?

I'm happy to have this issue turn into. CI should build the samples if thats the only purpose of CI left :)

@Petermarcu I'm not that familiar with what is in this repo.  If no CI is needed, then just empty out the netci.groovy file for now, or augment it to build the samples.

Sounds good. I think emptying it out for now is probably the right thing to do.

https://github.com/dotnet/core/pull/1069

This can be closed now, right?


Fix list generation of token_ids

range(start,stop) is inclusive, exclusive

In the initial commit there was a mistake for inferring the upper_id
https://github.com/Convex-Labs/honestnft-shenanigans/blob/62f9bd8c421b369a68ae9d1b0d402bb515e1ed35/metadata/pulling.py#L318-L319
Generating the list of ids was actually programmatically correct
https://github.com/Convex-Labs/honestnft-shenanigans/blob/62f9bd8c421b369a68ae9d1b0d402bb515e1ed35/metadata/pulling.py#L343
But this PR modified the first mistake and introduced another mistake: https://github.com/Convex-Labs/honestnft-shenanigans/pull/12
https://github.com/bryanhpchiang/honestnft-shenanigans/blob/d2c8a9b2e0efcbb23e57a2044df447b62904de8f/metadata/pulling.py#L330-L331
https://github.com/bryanhpchiang/honestnft-shenanigans/blob/d2c8a9b2e0efcbb23e57a2044df447b62904de8f/metadata/pulling.py#L356-L358


[DRAFT] TLS 1.3 Handshake and Key Integration

Issue # (if available):
Integrate deriving secrets into the handshake as well as handshake record encryption/decryption for TLS 1.3.
Opening this for an initial draft, self talk tests to follow.
Description of changes:
Insert entry points secret generation, as well as getting the hashshake hashes at the correct time. eg (after ServerHello, ServerFinished).
This change also accounts for some

optional CCS handling
handshake fragmentation

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

Codecov Report

Merging #1272 into master will decrease coverage by 0.28%.
The diff coverage is 13.33%.

@@            Coverage Diff             @@
##           master    #1272      +/-   ##
==========================================
- Coverage   88.56%   88.27%   -0.29%
==========================================
  Files         230      230
  Lines       19213    19288      +75
==========================================
+ Hits        17016    17027      +11
- Misses       2197     2261      +64


PROJECT DOCUMENTATION TRANSLATION IN PORTUGUESE (BRAZIL)

I took the liberty of translating the documentation of this project into the Portuguese language to facilitate the use of this tool by Brazilian students.

I think the readme in your case should be renamed to README.pt-br.md. You could then add a shield to the main readme off to your translated version. You can see an example at https://github.com/jonatasemidio/multilanguage-readme-pattern.

I think the readme in your case should be renamed to README.pt-br.md. You could then add a shield to the main readme off to your translated version. You can see an example at https://github.com/jonatasemidio/multilanguage-readme-pattern.

Hello. Thank you very much for the warning. Correction made.

I think the readme in your case should be renamed to README.pt-br.md. You could then add a shield to the main readme off to your translated version. You can see an example at https://github.com/jonatasemidio/multilanguage-readme-pattern.

Hello. Looking at the documentation provided more closely, I believe your suggestion applies when there are multiple READMEs in different languages in the same repository. In this case, the fork I made is an independent repository for Brazilian Portuguese. The suggested and formalized approach is beyond the scope since the entire repository is in Portuguese.


Correction for raw affiliation Sorbonne Université, CNRS, LIP6, Sorbonne Université, Paris, France

Correction needed for raw affiliation Sorbonne Université, CNRS, LIP6, Sorbonne Université, Paris, France
raw_affiliation_name: Sorbonne Université, CNRS, LIP6, Sorbonne Université, Paris, France
new_rors: 02feahw73;02en5vm52;05krcen59
previous_rors: 02feahw73;02en5vm52
works_examples: W4387952456
contact: 097ed785b4f27bd47c9de8867bf4a53b:f67b2f328d68beab267458b073 @ sorbonne-universite.fr

This issue was accepted and ingested by the OpenAlex team on 2024-11-01. The new affiliations should be visible within the next 7 days.


Industry the company associated with

Hi JerBouma,
Thanks for this wonderful package. Very useful resource indeed. I was wondering is it possible to provide symbol of a company ans fish out the industry/sector that its associated with ?  Some companies have mix of products that it woule be listed in two different Industry. It would be useful to get the associated industry given the symbol. Thanks

Hi Rajan,
The database will always be based on one classification which they are the most closely related to. You could say Amazon is in a lot of industries as they provide so many different products. However, their core business is being an online marketplace therefore the industry is titled "Internet Retail".
Your request can be achieved with the following code:
import financedatabase as fd

equities = fd.select_equities()

symbol_industry = {}

for symbol, values in equities.items():
    symbol_industry[symbol] = values['industry']

In a neat format (DataFrame):
df pd.DataFrame.from_dict(symbol_industry, orient='index', columns=['Industry'])

Out[11]:
                               Industry
                                   None
A                Diagnostics & Research
AA                             Aluminum
AAALF
AAALY                              None
                                 ...
ZYXI                    Medical Devices
ZZHGF   Insurance - Property & Casualty
ZZHGY                              None
ZZLL     Internet Content & Information
ZZZOF  Other Industrial Metals & Mining
[20063 rows x 1 columns]

Thank you so much JerBouma. Appreciate the answer.


Signup Modal

User Story
AS AN avid reader
I WANT to search for new books to read,
SO THAT I can keep a list of books to purchase.

Acceptance Criteria
GIVEN a book search engine,
WHEN the toggle is set to Signup,
THEN I am presented with three inputs for a username, an email address, and a password, and a signup button.

See PR https://github.com/nestibry/book-search-engine/pull/21


Notify user the doc index has completed

When the doc index build was blocking it was obvious when it was complete.
Now that it's in the background (much improved) you're not really sure when the full docs are available.
The toolbar might be a good place to show some info.

Created https://github.com/awslabs/aws-shell/issues/67: Add progress status for long running commands.


[Question]: Set the frequency_penalty when using openailike

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
How to set the frequency_penalty and other model parameters when using openailike?
I am currently setting as below:
Settings.llm = OpenAILike(model="meta-llama/Meta-Llama-3-8B-Instruct", api_base=openai_api_base, api_key="",  max_tokens=2000,   frequency_penalty=0.8,  presence_penalty=0.5 ,  top_p=0.9,  stop=stop_phrases,  model_kwargs={     "frequency_penalty": 1.0,   })
But when checking on my vllm server (On a different instance hence using OpenAILike) it shows 0 as frequency_penalty.

Set it under additional kwargs
additional_kwargs={...}

@logan-markewich Many thanks, It works perfectly


Edge - Second Tab

Description
add a second tab in the browser (edge) and some styling fixes
Type of change

[x] New feature (non-breaking change which adds functionality)

I have read the CLA Document and I hereby sign the CLA


How to export trained model to be used by javascript

Is there  a way to reuse a trained model form other environments? or how much work would it take to be able to reuse a trained model from javascript, as is done by tensorfire?

Looks like there are ways https://github.com/nok/sklearn-porter to port pieces of it to JS.
Though I am not certain I see an advantage of that vs running the Rasa NLU server and interacting with it via HTTP from javascript. Keep in mind that the full Rasa solution is a pipeline so each individual piece would have to be ported or similar solutions implemented from the featurizer, entity model, intent model, etc.

Well i would like the ability to run a trained model offline and on the browser, i would like to start working on an example of this but maybe you can point me to some of te steps required to run a Rasa trained model once i port the estimator to a JS compatible format?

If you can successfully port it you wont need help running it. What is the use case for needing to run serverless? Would it not be possible to run run the server locally?


Fixed AttributeError for owner when starting repair by id

The owner attribute is not available when running spreaper start <id> which will result in an AttributeError.

Thanks for the pointer about owner and cause arguments being wrong! I fixed it in a bit more thorough way in PR #59.


basket: Do None check of unit_price_excl_tax before multiplication

If price of a product, which is already in a basket becomes unavailable, Line.line_price_excl_tax property throws a TypeError while multiplying quantity with unit_price_excl_tax, since unit_price_excl_tax is None. Do a None check before multiplication to avoid this.

This is an issue which happended to me also a number of times. I remember discussing this with @maikhoepfel and that we should fix this at the proper location.
@maikhoepfel ? :-)

@lijoantony @mvantellingen we could try to re-create this case with test. What do you think?

Finally added test to re-create this case - #2283.

Instead of falling back to zero price, I think it's better to skip price calculation and not to show price in the basket template, since it's unavailable. Closing in favor of #2283. Thanks.


Wildcards Broken With Batch Sizes >1 (SD.Next)

Hi,
When I am using wildcards to create some variety in my images I have recently experienced a problem. When generating in batches, the prompt text saved in the PNG file seems to different from the image if the batch size is greater than 1. Let me explain with a workflow:

Created a wildcard called Test Github.txt which contains the following text:

top-down view, high camera angle of a 20 year old Jessica Alba wearing a red cocktail dress standing amidst a vast desert, cinematic photo
top-down view, high camera angle of a 20 year old Jessica Alba wearing a blue cocktail dress standing amidst a volcanic mountain, cinematic photo
top-down view, high camera angle of a 20 year old Jessica Alba wearing a black cocktail dress standing amidst a frozen ice world, cinematic photo
The 3 wildcard enteries are the same except for the colour of the dress and the environment.

Entered in the posiitive prompt text box Test Github and set the batch size to 3.

Here are the result:

As you can see, 3 images have generated but all 3 are in the same location and the same coloured dress. The prompt texts are often different for each image. The first image matches the prompt as shown here:

The next two images however have a different prompt text that is not reflected in the image as shown here:

NOTE: This does not happen with batch count, only batch size.

Well no response from anyone. Is this an issue unique only to me?

Hey, I can't offer a solution I'm afraid but I just wanted to reassure you and say you're not the only one, I experience this too with batch size and not with batch count.


tea --magic is slow 🐢 when run in shell startup

tea --magic adds about 100ms of extra latency to my shell startup, which is a lot to me c:
❯ hyperfine --warmup=3 "~/.tea/tea.xyz/v0.21.0/bin/tea --magic"
Benchmark 1: ~/.tea/tea.xyz/v0.21.0/bin/tea --magic
  Time (mean ± σ):     157.7 ms ±  60.6 ms    [User: 97.5 ms, System: 22.9 ms]
  Range (min … max):   103.6 ms … 345.7 ms    25 runs

If I may give my 2¢, while I'm as big a Deno lover as the next guy it's just not suitable for the sub-100ms startup times maniacs like myself crave for our terminal.
As a best-case scenario I measured the fastest possible time the magic could run
#!/usr/bin/env deno run
// magic.ts
console.log(`add-zsh-hook -Uz chpwd() {
  if [ "\${TEA_MAGIC:-}" != 0 -a -x "/Users/m0c0j7y/.tea/tea.xyz/v0.21.0/bin"/tea ]; then
    source <("/Users/m0c0j7y/.tea/tea.xyz/v0.21.0/bin"/tea +tea.xyz/magic -Esk --chaste env)
  fi
}

# if the user put tea in eg. /usr/local/bin then don’t pollute their PATH
if ! tea --prefix 2>&1 >/dev/null; then
  export PATH="/Users/m0c0j7y/.tea/tea.xyz/v0.21.0/bin:$PATH"
fi

function command_not_found_handler {
  if [ "\${TEA_MAGIC:-}" != 0 -a -x "/Users/m0c0j7y/.tea/tea.xyz/v0.21.0/bin"/tea ]; then
    "/Users/m0c0j7y/.tea/tea.xyz/v0.21.0/bin"/tea -- $*
  fi
}
`)

❯ hyperfine --warmup=3 ./magic.ts
Benchmark 1: ./magic.ts
  Time (mean ± σ):      63.6 ms ±   7.6 ms    [User: 37.2 ms, System: 17.8 ms]
  Range (min … max):    57.3 ms …  88.6 ms    40 runs

compared to something like
package main
// magic.go

import "fmt"

func main() {
	fmt.Println(
		`add-zsh-hook -Uz chpwd() {
  if [ "${TEA_MAGIC:-}" != 0 -a -x "/Users/m0c0j7y/.tea/tea.xyz/v0.21.0/bin"/tea ]; then
    source <("/Users/m0c0j7y/.tea/tea.xyz/v0.21.0/bin"/tea +tea.xyz/magic -Esk --chaste env)
  fi
}

# if the user put tea in eg. /usr/local/bin then don’t pollute their PATH
if ! tea --prefix 2>&1 >/dev/null; then
  export PATH="/Users/m0c0j7y/.tea/tea.xyz/v0.21.0/bin:$PATH" fi

function command_not_found_handler {
  if [ "${TEA_MAGIC:-}" != 0 -a -x "/Users/m0c0j7y/.tea/tea.xyz/v0.21.0/bin"/tea ]; then
    "/Users/m0c0j7y/.tea/tea.xyz/v0.21.0/bin"/tea -- $*
  fi
}`)
}

❯ hyperfine --warmup=3 ./magic
Benchmark 1: ./magic
  Time (mean ± σ):      10.5 ms ±   9.2 ms    [User: 1.5 ms, System: 3.0 ms]
  Range (min … max):     0.0 ms …  36.9 ms    160 runs

  Warning: Command took less than 5 ms to complete. Note that the results might be inaccurate because hyperfine can not calibrate the shell startup time much more precise than this limit. You can try to use the `-N`/`--shell=none` option to disable the shell completely.

The difference is even starker when run "e2e"

possible solutions:

Cache the output of tea --magic in some file and update the shell one-liner into a two-liner (🤢 amirite?) to use that file if it exists

Yeah 100ms is not acceptable for sure. Once we are 1.0.0 I plan to rewrite in Rust. But until then deno is much faster for dev.
Having said this, I'm surprised it is this much since --magic is a pretty hot path.
I can see a few quick fixes, make tea a shell script that handles some of the options fast and then delegates the rest. Unfortunately shell magic involves calling tea so that is every cd. Possibly we could cache that though.
I'll give it some thought. Thanks for the (very) detailed report!

Thank you!  btw when including zsh start time the difference is even more dramatic

Deno recently merged a PR to allow for snapshotting of ES modules (https://github.com/denoland/deno/pull/17460). When/if this gets extended for use in deno compile, we should see a pretty hefty perf improvement. In my (rather limited) understanding, it essentially converts typescript from a JIT language to a compiled language by doing the compile step ahead of time.
While this obviously will not "solve" the problem of typescript being slower than rust, it should alleviate a lot of the pain for users in the mean-time.

That's good news. If we can get startup down 50% - 80% then a rewrite in rust may never be necessary.

there's always the option of shipping a second binary. tea --magic --silent is, as you say, a hot path. writing it in Rust and shipping {{prefix}}/bin/tea-magic is probably quite easy, if it's a big win for responsiveness.

pure rust is a slight improvement:
$ hyperfine --warmup=3 "tea --magic"
Benchmark 1: tea --magic
  Time (mean ± σ):      47.5 ms ±   1.3 ms    [User: 43.3 ms, System: 3.4 ms]
  Range (min … max):    46.2 ms …  53.8 ms    60 runs

$ hyperfine --warmup=3 "./tea-magic zsh"
Benchmark 1: ./tea-magic zsh
  Time (mean ± σ):       0.6 ms ±   0.5 ms    [User: 0.3 ms, System: 0.2 ms]
  Range (min … max):     0.2 ms …   9.4 ms    1084 runs

  Warning: Command took less than 5 ms to complete. Note that the results might be inaccurate because hyperfine can not calibrate the shell startup time much more precise than this limit. You can try to use the `-N`/`--shell=none` option to disable the shell completely.
  Warning: Statistical outliers were detected. Consider re-running this benchmark on a quiet PC without any interferences from other programs. It might help to use the '--warmup' or '--prepare' options.

zsh startup adds a fair bit. I believe it's sourceing the functions:
$ hyperfine --warmup=3 'zsh -c "source <(./tea-magic zsh)"'
Benchmark 1: zsh -c "source <(./tea-magic zsh)"
  Time (mean ± σ):      51.1 ms ±   1.2 ms    [User: 44.1 ms, System: 5.2 ms]
  Range (min … max):    50.0 ms …  59.1 ms    56 runs

$ hyperfine --warmup=3 'zsh -c "echo hi"'
Benchmark 1: zsh -c "echo hi"
  Time (mean ± σ):       1.6 ms ±   0.8 ms    [User: 0.4 ms, System: 0.6 ms]
  Range (min … max):     1.0 ms …  10.5 ms    680 runs

$ hyperfine --warmup=3 'zsh -c "source <(env)"'
Benchmark 1: zsh -c "source <(env)"
  Time (mean ± σ):       5.6 ms ±   0.9 ms    [User: 1.0 ms, System: 3.7 ms]
  Range (min … max):     5.0 ms …  19.9 ms    336 runs

seems like maybe deno gave us a speed up?
$ time tea --magic
_tea_chpwd_hook() {
  if [ "${TEA_MAGIC:-}" != 0 -a -x "/usr/local/bin"/tea ]; then
    source <("/usr/local/bin"/tea +tea.xyz/magic -Esk --chaste env)
  fi
}

typeset -ag chpwd_functions

if [[ -z "${chpwd_functions[(r)_tea_hook]+1}" ]]; then
  chpwd_functions=( _tea_chpwd_hook ${chpwd_functions[@]} )
fi

if [[ "$PATH" != *"$HOME/.local/bin"* ]]; then
  export PATH="$HOME/.local/bin:$PATH"
fi

if ! command -v tea 2>&1 >/dev/null || ! tea --prefix 2>&1 >/dev/null; then
  export PATH="/usr/local/bin:$PATH"
fi

function command_not_found_handler {
  if [ "${TEA_MAGIC:-}" != 0 -a -x "/usr/local/bin"/tea ]; then
    "/usr/local/bin"/tea -- $*
  fi
}
tea --magic  0.06s user 0.01s system 98% cpu 0.074 total

$ hyperfine --warmup=3 'zsh -c "source <(tea --magic=zsh)"'
installed: ~/.tea/crates.io/hyperfine/v1.16.1
Benchmark 1: zsh -c "source <(tea --magic=zsh)"
  Time (mean ± σ):     109.2 ms ±   2.0 ms    [User: 95.7 ms, System: 12.6 ms]
  Range (min … max):   107.3 ms … 114.9 ms    27 runs

That looks like we might be twice as fast as we used to.

Not on my machine.  No statistical difference
❯ hyperfine --warmup=3 "zsh -c 'source <(~/.tea/tea.xyz/v0.21.0/bin/tea --magic=zsh)'"  "zsh -c 'source <(~/.tea/tea.xyz/v0.25.3/bin/tea --magic=zsh)'"
Benchmark 1: zsh -c 'source <(~/.tea/tea.xyz/v0.21.0/bin/tea --magic=zsh)'
  Time (mean ± σ):     241.5 ms ±  10.5 ms    [User: 178.6 ms, System: 43.3 ms]
  Range (min … max):   232.2 ms … 264.5 ms    12 runs

Benchmark 2: zsh -c 'source <(~/.tea/tea.xyz/v0.25.3/bin/tea --magic=zsh)'
  Time (mean ± σ):     252.9 ms ±  15.6 ms    [User: 182.5 ms, System: 44.9 ms]
  Range (min … max):   235.7 ms … 287.6 ms    12 runs

Summary
  'zsh -c 'source <(~/.tea/tea.xyz/v0.21.0/bin/tea --magic=zsh)'' ran
    1.05 ± 0.08 times faster than 'zsh -c 'source <(~/.tea/tea.xyz/v0.25.3/bin/tea --magic=zsh)''

k so time tea --prefix is ~10ms. Thus our position that this is deno’s fault is wrong. Somehow --magic adds 90ms.

What are you running on?  It's about ten times slower on my machine
❯ hyperfine --warmup=2 "tea --prefix"

Benchmark 1: tea --prefix
  Time (mean ± σ):     164.8 ms ±  13.4 ms    [User: 118.1 ms, System: 34.0 ms]
  Range (min … max):   142.0 ms … 199.5 ms    17 runs

I'm running on a 2019 MacBook Pro, 2.4 GHz 8-core i9 with 32 GB of RAM

Deno is responsible for almost half that time
❯ hyperfine --warmup=1 "deno run <(echo \"console.log('I love tea!')\")"
Benchmark 1: deno run <(echo "console.log('I love tea!')")
  Time (mean ± σ):      44.3 ms ±  10.9 ms    [User: 24.8 ms, System: 17.9 ms]
  Range (min … max):    34.0 ms …  62.4 ms    39 runs

Deno is responsible for almost half that time
❯ hyperfine --warmup=1 "deno run <(echo \"console.log('I love tea!')\")"
Benchmark 1: deno run <(echo "console.log('I love tea!')")
  Time (mean ± σ):      44.3 ms ±  10.9 ms    [User: 24.8 ms, System: 17.9 ms]
  Range (min … max):    34.0 ms …  62.4 ms    39 runs

I'm not so sure. I think a fairer comparison would be to first deno compile your console.log and then benchmark it instead. If I'm not mistaken, deno compiled binaries has optimizations on startup time.

hyperfine ./tea
Benchmark 1: ./tea
  Time (mean ± σ):      23.6 ms ±   0.6 ms    [User: 19.4 ms, System: 3.4 ms]
  Range (min … max):    22.8 ms …  26.5 ms    111 runs

hyperfine --warmup 3 "tea --magic"
Benchmark 1: tea --magic
  Time (mean ± σ):      36.6 ms ±   0.3 ms    [User: 32.8 ms, System: 4.1 ms]
  Range (min … max):    36.1 ms …  38.0 ms    76 runs

hyperfine --warmup 3 "tea -E"
Benchmark 1: tea -E
  Time (mean ± σ):      43.9 ms ±   0.5 ms    [User: 41.9 ms, System: 5.3 ms]
  Range (min … max):    43.1 ms …  46.8 ms    64 runs

A general surprise I have here is that deno compile doesn't seem to do anything? Does it really just generate a binary blob that is the js engine and dump all the code as a data section?

Based on the behavior with patchelf/rpath_fix, I had assumed this was what it was doing for the last few months.

deno compile should transpile and vendor the code (internally) by itself. I'm a little surprised that such a simple import adds so much delay? Could you share a benchmark or a reproc for this?

sure thing. I'll set something up.

I'm closing this since --magic has been removed and since there hasn't been activity here for 2 months


Please block "ctrl/cmd + tab" in break view

A bit odd feature request, but still. I'm using combination of options:

don't let me skip breaks (mini breaks)
breaks are fullscreen

So I have fullscreen break without skip button, but I still can just do cmd+tab (on macosx) and therefore switch tabs. Stretchly break view left working in background.
Not sure that's important, but I keep doing that automatically... 😆

closing in favor of #437 as it's pretty much the same thing


Fix for flaky test where rate limiter failed to fire when expected.

Increases the limit duration so no chance of crossing the duration boundary mid-test.
Closes #500
PR Checklist

[x] Have you followed the guidelines in CONTRIBUTING.md?
[x] Have you got 100% test coverage on new code?
[x] Have you updated the prose documentation?
[x] Have you updated the reference documentation?

@Goldziher you reckon I have the reasoning right here? Detailed here

Hit this again: https://github.com/starlite-api/starlite/actions/runs/3103164409/jobs/5026281794


save icon with red dot

Icon Request

Icon name:

Save icon with red dot that indicates content has been modified

Tags of this icon:

Save icon red dot

Use case:

Indicate content has been modified.

Screenshots of similar icons:

emmmm...
I think this may not be the best practice to indicate the status changes.
For example, in HTML you could draw an <div> element with red bg and white border in the corner of the save icon to describe the changes. Red dot(badge) is the variable rather than the icon, just like our website
In addition, RemixIcon is a monochrome icon library, for now, so we could not add a red element in the black and white icon.

Ok. Thanks for the advice...

The common interaction design method is:
disable the save icon when no content modified.


Some small typos

1. Preface (ii)

2. Acknowledgements

3. A Journey Towards CQRS (page 7)

Thanks for reporting your issue! It's fixed now and a new book version is ready. You are now part of the contributors!

@carlosbuenosvinos It's my pleasure, glad to help out!


Split instances

Split the instances into their respective buckets so that they can be imported as needed.

@phadej Sorry to keep bugging you, anything I can do to help you test?

Thanks for the remindier. Looks good.


AT-20: Highlight indoor points of interest on a campus building's floor map

Test Case ID: AT-20
User Story: CC-20 (#20)
As a user, I want to be able to select the visibility of access points (printers, washrooms, fountains, study rooms, fire escapes, elevators, stairs), so that my access points of choice can be highlighted on the indoor maps.
Acceptance Criteria
Given the user has opened the side-menu panel by selecting the hamburger menu button and clicking on the Settings option,
The user should be able to change the visibility of different access points and the indoor maps should get updated accordingly.

Accepted


paginateScan does not support Dax Client

So the v3 SDK method, paginateScan, does not support Dax Clients.
So to do a full scan on a table with 25,000 items, I need to do my own paginated Scan, which I am implementing, but its super slow, despite using Dax. With 1000 segments, its still timing out after 20 seconds.
Is there a Dax equivalent of paginateScan?

So the v3 SDK method, paginateScan, does not support Dax Clients.

Not at my computer atm so I can't tell why this is the case? Is it a typing error or calling https://github.com/aws/aws-sdk-js-v3/blob/main/lib/lib-dynamodb/src/pagination/ScanPaginator.ts#L30 with a DAX client fails?
I do believe DAX's backend cluster supports scan so I would think lib-ddbs paginateScan could work

So the v3 SDK method, paginateScan, does not support Dax Clients.

Not at my computer atm so I can't tell why this is the case? Is it a typing error or calling https://github.com/aws/aws-sdk-js-v3/blob/main/lib/lib-dynamodb/src/pagination/ScanPaginator.ts#L30 with a DAX client fails?
I do believe DAX's backend cluster supports scan so I would think lib-ddbs paginateScan could work

Check lines 42-45. Dax client simply isn't supported.
My own paginated scan is super slow even for measly 1500 records. I've tried with and without using Segments. No improvement when using them.

I added an example of how to fake shim the document client so that it works with dax
https://github.com/kwojcicki/amazon-dax-client-v3/blob/main/test/dynamoDBDocumentClient.js#L58
its probably something that could be moved into the core library but that will mess with everyone's instanceof/prototype checks which I would rather not do unless I absolutely have to

Thank you for the example!
I would argue it should be in core as paginateScan etc are crucial to being able to use dynamodb in any meaningful way.

ya the implementation looks pretty simple to copy over and stable for the long term, I can add that

I tried using your example in my "parallel paginated scan" code, where I run multiple paginateScans, using segments.
I hit this error
DaxClientError: ConnectionException: write EPIPE
    at Socket.<anonymous> (/var/task/node_modules/amazon-dax-client-sdkv3/src/BaseOperations.js:110:23)
    at Socket.emit (node:events:519:28)
    at emitErrorNT (node:internal/streams/destroy:169:8)
    at emitErrorCloseNT (node:internal/streams/destroy:128:3)
    at process.processTicksAndRejections (node:internal/process/task_queues:82:21) {
  time:<PHONE_NUMBER>000,
  code: 'ConnectionException',
  retryable: true,
  requestId: null,
  statusCode: -1,
  _tubeInvalid: false,
  waitForRecoveryBeforeRetrying: false,
  '$metadata': { attempts: 1, totalRetryDelay: 0 }
}

function timeout is 1min, code errored after 24s

Ok, so after tweaking some numbers, ie the number of parallel paginateScans to run, its working. But, again, its not faster than DynamoDB itself, and I'm using 3x dax.r5.2xlarge nodes.
What am I missing here? I thought DAX was supposed to be faster?

DAX is only faster (to my knowledge) if you are actually getting cache hits, otherwise its an extra server your packets have to hop through with 0 benefit.
DAX does track scan hits https://docs.aws.amazon.com/amazondynamodb/latest/developerguide/dax-metrics-dimensions-dax.html if the scan hit rate is high and its still slower then its possible there's performance improvements to be made on the server/client code side, if the scan hit rate is low then the performance being slower is expected.
The performance will also depend on your data size, given you are going through 1-2 more hops (with DAX) if you are transferring 1GB records then its possible even with a high hit rate that DAX won't be super useful

The table I'm scanning is only 90mb.
Here is another error from today
DaxClientError: ConnectionException: Connection is closed by server
    at Socket.endListener (/var/task/node_modules/amazon-dax-client-sdkv3/src/BaseOperations.js:59:23)
    at Socket.emit (node:events:531:35)
    at endReadableNT (node:internal/streams/readable:1696:12)
    at process.processTicksAndRejections (node:internal/process/task_queues:82:21) {
  time:<PHONE_NUMBER>879,
  code: 'ConnectionException',
  retryable: true,
  requestId: null,
  statusCode: -1,
  _tubeInvalid: false,
  waitForRecoveryBeforeRetrying: false,
  '$metadata': { attempts: 1, totalRetryDelay: 0 }
}

This is with about 90 parallel (segments) paginateScans running.

Added pagination to the core client as part of https://github.com/kwojcicki/amazon-dax-client-v3/commit/8769f4ad4725b7f8bc64d454753c13153aae12e7
also added the ability to modify the under http client params https://github.com/kwojcicki/amazon-dax-client-v3/commit/b917dbbf4a89fb3db6effa12abc5e41e02c24d10
i'll start stress testing my dax cluster more to reproduce the errors but I assume there is not much that can be done on the client side besides increasing timeouts/waiting between queries

The table I'm scanning is only 90mb.

Approximately how many items is that spread over?

The table I'm scanning is only 90mb.

@paul-uz  Approximately how many items is that spread over?

Approx 25,000 records.
I'll try the latest version to try paginated scan via dax

@kwojcicki i think some fo the new typings are a bit off.
The paginateScan config object wants a startingToken but this should be optional?
And I get this error on my client initialisation

Type 'DynamoDBDocumentClient' is missing the following properties from type 'DynamoDBClient': getDefaultHttpAuthSchemeParametersProvider, getIdentityProviderConfigProvider

I'll update those typings and modify my test scripts later to use typescript. Kind of blindly making the types for now and hoping they work.
Regrading the performance issues you are facing I can't seem to really replicate them.
I've got a 100mb DDB table (no GSI's). The data was setup as follows:
const randomString = (length) => {
    let result = '';
    const characters = 'ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789';
    const charactersLength = characters.length;
    let counter = 0;
    while (counter < length) {
        result += characters.charAt(Math.floor(Math.random() * charactersLength));
        counter += 1;
    }
    return result;
}

for (var i = 0; i < 25_000; i++) {
    const putItem = new PutCommand({
        TableName: 'test',
        Item: {
            CommonName: `${i}`,
            Data1: randomString(800),
            Data2: randomString(800),
            Data3: randomString(800),
            Data4: randomString(800),
            Data5: randomString(800)
        }
    });

    await documentDaxClient.send(putItem);
}

Then I've got a 1769mb lambda and a 3 node (dax.r4.large) DAX cluster, doing a 90 segment scan against that.
let i = 0;
const totalSegments = 90;

const parallelScan = async (segmentId) => {
    const paginator = documentDaxClient.paginateScan({
        pageSize: 1000
    }, {
        TableName: 'test',
        Segment: segmentId,
        TotalSegments: totalSegments
    });

    for await (const val of paginator) {
        i += val.Items.length;
        if (i % 500 === 0) {
            console.log(val.Items[0]);
        }
    };
}

console.time("scan");
const promises = [];
for (let j = 0; j < totalSegments; j++) {
    promises.push(parallelScan(j));
}

await Promise.all(promises);
console.log(`scanned ${i} in`);
console.timeEnd("scan");

This takes between 3-4 seconds depending on the # of cache hits. Which is slower than the regular DDB version of this, but its also not really expected to be faster given
https://docs.aws.amazon.com/amazondynamodb/latest/developerguide/dax-prescriptive-guidance.html

In addition, for bulk reads, you should run full table scans directly against a DynamoDB table.

and some inefficiencies the DAX client has with sdkv3 that I'm trying to resolve.
But I don't see any timeouts nor issues like that, could you provide more info about your setup so I can reproduce your behavior

and some inefficiencies the DAX client has with sdkv3 that I'm trying to resolve.

I removed the unnecessary round trip of json parsing that I initially introduced when upgrading this to sdkv3. May help slightly with the DAX scan times.


update packages

:white_check_mark: Build TheIdServer 4.2.1-fix-dependencies1-0225 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/0f04427a35 by @)


Compatability with Chrome v80+ with SameSite cookie changes

Chrome is showing a warning about a breaking change coming 4th Feb with Chrome v80:
A cookie associated with a cross-site resource at https://gist.github.com/ was set without the `SameSite` attribute. A future release of Chrome will only deliver cookies with cross-site requests if they are set with `SameSite=None` and `Secure`. You can review cookies in developer tools under Application>Storage>Cookies and see more details at https://www.chromestatus.com/feature/5088147346030592 and https://www.chromestatus.com/feature/5633521622188032.

thx for your alert but i think its a cookie for grafana, not the plugin


Update your code

New apps that use UIWebView are no longer accepted. Instead, use WKWebView for improved security and reliability.

@roughike could you please update the package with this fix?
https://github.com/roughike/flutter_twitter_login/issues/32#issuecomment-624033596


is possible to add multiple view into one day cell.

Thanks for this fantastic library. if I'd like to add some textview/image view into one day cell. is it possible to achieve this requirement?

You can accomplish this using decorators. Documentation is here https://github.com/prolificinteractive/material-calendarview/blob/master/docs/DECORATORS.md

how to add Multiple decorators in one day

By adding the day to multiple decorators.


Microsoft Teams App Installation (Option 2: Configure via MDM such as Intune)

Is it necessary to assign the teams app in Microsoft Intune to the Surface Hub or will the teams app automatically install when the "./Vendor/MSFT/SurfaceHub/Properties/VtcAppPackageId" is configured for the surface hub?

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

ID: f256cd5b-3ec8-2d55-9ae4-ff9849d52eae
Version Independent ID: 52c2271b-df38-5dc9-c2a1-553cdf3b5496
Content: Deploy Microsoft Teams for Surface Hub
Content Source: Teams/teams-surface-hub.md
Service: msteams
GitHub Login: @chuckedmonson
Microsoft Alias: chucked

@nicolonsky Thank you for this feedback
I am honestly not sure. I will get this over to someone who can answer this for you. If you need an expedited answer though contact Support since they can also handle this to resolution.
I was able to get this into the backlog for the Teams/Skype for Business team. The team will get this prioritized into a sprint plan. Thank you for reporting and making the docs better. Much appreciated.
I made a note to request the team to update this when the work is complete.

Hi @nicolonsky thanks for your feedback and comments, we really appreciate them.
As @danieldhudson says, you do need to install the Teams app as the document says before setting it to default calling and meeting application. You could also use intune to assign it and push it. Hope this information helped you so we can safely close this issue.

@get-itips  Thank you very much for the contribution and sharing this explanation. @ nicolonsky
Hope this comment is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Question: how to import certificate in browser

hi,
Is there any way to import certificate in browser?
Thanks

posting in images


Subscription Update

Implements #1859 and fixes #1859.

bors merge

Implements #1859 and fixes #1859.

Meant to say Implements #1859 and fixes #1858. Closing #1858...


change all logs to winston

Patch

change all logs to winston
make all logs consistent in how they report their filename
report when LOG=verbose what reference is missing for 404s.

:+1:


Separating route securities from other dependencies.

Related Issue(s):

#765

Description:
Currently if you have more than one security dependency on a single route, if either fail then the route will be unauthorised. This is a known issue in FastAPI and the suggested solution is to use a single dependency to merge the sub dependencies. This makes configuration more difficult and removes the ability to use default security dependencies.
Suggested solution is to separate route securities from other dependencies. This then allows them to be merged when dynamically.
There are two options for merging in this pull request.
merge_dependencies1 uses the traditional merging method but this requires dependencies return a value or None rather than raise an error. This means it would need to be used for all routes even those without multiple dependencies.
merge_dependencies2 calls each dependency itself which allows them to raise their own errors so can be used only on routes that have multiple dependencies.
PR Checklist:

[x] pre-commit hooks pass locally
[ ] Tests pass (run make test)
[ ] Documentation has been updated to reflect changes, if applicable, and docs build successfully (run make docs)
[ ] Changes are added to the CHANGELOG.

Moving to stac-fastapi-elasticsearch-opensearch see issue #765 for details.


Run setup.py test against entire test folder

At the moment, only the content of test/integration is discovered in the test
phase of the Travis CI build for plugins/apps. Discovering other types of
tests, typically unit tests, should be harmless.
In terms of testing time, migrating testing logic to unit tests rather than
integration tests should allow a more thorough coverage while limiting
execution time.

No objections but perhaps try this PR against https://github.com/ome/omero-plugins to see if anything breaks?

See https://travis-ci.org/ome/omero-plugins/builds/499155148. Two failures:

omero-parade which is known to fail
scripts which seems to fail the latest version of flake8. I can open a PR to fix that if needed

Neither seems related to adding more test files though, eh? If that's the case, :+1:

https://github.com/ome/omero-metadata/pull/18 might be a good example of location where additional tests should being executed in Travis once this is merged.


Third argument to add_snapshot_fun should be optional

Currently an error gets thrown if it is not provided

Fixed in #47


CI

I am planning on integrating Travis CI for testing & linting of this project whenever PRs are opened

As well, look into integrating https://github.com/typicode/husky so you can't commit unless the linting / tests pass

Husky is now integrated into the project

and CI is now set up on Travis-CI


[improvement](build index)Make build index and clone mutually exclusi…

…ve and add timeout for index change job (#35724)
Currently the index change job and clone task can be executed at the same time. If the clone task gets stuck at this point, it will cause the index change job to get stuck as well and keep retrying. To solve this problem, we can refer to alter job and make index change job exclusive with clone task, and introduce the timeout to prevent infinite retries of build index.
Add the following checks and status in FE.

Check if table is stable (build index is not allowed when clone is in progress)
1.1. Tablet is HEALTHY.
1.2. Whether the tablet is included in the Tablet scheduler, if so, it means the current tablet is doing clone.
When creating the index change job, set the timeout at the same time.

pick from master #35724

run buildall

run buildall


VTOL rate control improvements

Replaces #9190

Not merging just yet!

@LorenzMeier yes, please don't! It breaks Flight Review :)

@PX4/testflights Any chance you can test fly this. It touches multirotor, vtol and fixed wing.

@PX4/testflights please try this on all VTOLs, as well as regression testing on MC and FW platforms.

test on pixhawk 1 and crash on stabilize mode
https://review.px4.io/plot_app?log=8f5cbf75-cb1f-4fd3-88ed-d18d425c7cfb
on pixhawk v4 pro crash on stabilize mode
https://review.px4.io/plot_app?log=014173eb-4156-4828-a97d-492c417f3577

@dannyfpv Hi! Thanks for testing and sorry for the crash. I checked the log and I already found the bug. Unfortunately, there was sign error in throttle in stabilized mode that was introduced during the lasted rebase on master. SITL did not catch that bug because it does not use test stabilized mode.
However, in future I suggest to bench test PRs which introduce massive changes like this one does before actually flying the vehicle. In this case you would have had no throttle in stabilized mode.
Anyway, I hope the convergence is not broken too badly...

Thanks for the feedback @RomanBapst, don't worry about the convergence it fulfilled its duty! Now let's make sure we have tests for stabilized mode, should I open an issue for tracking @dagar or is this covered somewhere already?

@mrpollo I was already thinking about this. I think it would not be too hard to use the dronecode SDK in the CI and actually 'simulate' a human flying with a radio controller. That would just be a sluggish controller outputting RC commands.

tested on pixhawk v4 pro
https://review.px4.io/plot_app?log=482daec6-d8b5-416f-bb3c-e57782fec3bd
https://review.px4.io/plot_app?log=4fc131d3-e215-4188-895c-d59a7161747d

I took another pass and this looks good. Let's merge and keep going.

Great team work everyone 🎉


Added roles in the application register

Revision done, minor comments pending.
Good job improving the tests.

Done


PR Reviewers changes apply when tapping on an empty space

Describe the bug
PR Reviewers changes apply when tapping on an empty space.
To Reproduce

Go to any PR
Click on the floating button
Click on the "Reviewers" button
Change reviewers list
Tap on the empty space outside of the dialog
See changes in Reviewers of a PR

Expected behavior
When I tap on the empty space I wanna discard all changes and close dialog. Or maybe do nothing.
Screenshots

Smartphone:

Device: iPhone 6s
OS: iOS 12
Browser Safari
Version 1.25.0

Happy to change the design here so there is an explicit “are you sure?” dialog when cancelling after changes.
Sent with GitHawk

August123!
Sent with GitHawk

I feel that this is a bit trickier, as eg. labels or assignees should then also mimic this behavior. Not sure if it makes sense for labels, however; but having different behavior for different "entities" feels like an even worse user experience. Thoughts?


Photo Like Votes

The "Photo Like Votes" feature allows users to like or unlike photos by clicking a like button, with a visual indication of whether a photo is liked or not. The number of likes is displayed next to the button and updates immediately when a user likes or unlikes a photo. Photos are sorted by the number of likes and timestamp, with the most liked photos appearing first. The system ensures that a user can only like a photo once and prevents race conditions in high-traffic scenarios.

"The 'Photo Like Votes' feature has been successfully implemented with real-time updates and robust handling of concurrency to prevent race conditions. All identified use cases and requirements have been addressed and tested. Please review and confirm if everything aligns with expectations, or let me know if any updates or modifications are needed!"

Great job on implementing the 'Photo Like Votes' feature with real-time updates and concurrency handling. It might be helpful to add metrics or logs to monitor the performance under high load and ensure scalability. Let me know if further testing assistance is needed!

Great Work Team, The feature looks well-implemented and thoroughly tested. If no additional changes are required, this can be considered complete. Great work!


ISSUE: Plugin broken after latest RL update

Title self explanitory, the latest update broke Emote Clue Items
RuneLite
BOT
— Today at 12:16 PM
Some hub plugins failed to build against the latest RuneLite version, this means that they're not available until the owner updates them.

Thank you for reporting this issue! I will start investigating

Currently, a hotfix is under review. You can follow the status at https://github.com/runelite/plugin-hub/pull/3007


Error occurring in prompt_for_user_token() (Python 2.7.12)

Hello,
The following error occurs in the prompt_for_user_token() function when using Spotipy 2.4.3 and 2.4.4.
I am currently running Python version 2.7.12. I've set the Spotify auth environment variables in my bashrc.
$python my_playlists.py [USERNAME]
Traceback (most recent call last):
  File "my_playlists.py", line 18, in <module>
    token = util.prompt_for_user_token(username, scope)
  File "/usr/local/lib/python2.7/dist-packages/spotipy/util.py", line 56, in prompt_for_user_token
    token_info = sp_oauth.get_cached_token()
  File "/usr/local/lib/python2.7/dist-packages/spotipy/oauth2.py", line 135, in get_cached_token
    if 'scope' not in token_info or not self._is_scope_subset(self.scope, token_info['scope']):
  File "/usr/local/lib/python2.7/dist-packages/spotipy/oauth2.py", line 156, in _is_scope_subset
    needle_scope = set(needle_scope.split())
AttributeError: 'NoneType' object has no attribute 'split'

If I revert back to Spotipy-2.3.8 this error does not occur. (but 2.3.8 is missing some functionality which is available in 2.4.4 that I would like to use)
It would be greatly appreciated if any suggestions/fixes could be provided.

I solved it with this #issuecomment-317192954

Same solution worked for me.

If I revert back to Spotipy-2.3.8 this error does not occur. (but 2.3.8 is missing some functionality which is available in 2.4.4 that I would like to use)

There was a bug with 2.4.4 where all the code was actually from a much older version. Please upgrade:
pip install spotipy --upgrade


Allow testing with bundler 2

Otherwise I get the error:
$ bundle exec thor spec
Bundler could not find compatible versions for gem "bundler":
  In Gemfile:
    bundler (~> 1.0)

  Current Bundler version:
    bundler (2.0.1)
This Gemfile requires a different version of Bundler.
Perhaps you need to update Bundler by running `gem install bundler`?

Could not find gem 'bundler (~> 1.0)' in any of the relevant sources:
  the local ruby installation

CI needs #649 to get to green.

Closing in favor of #649. I added this change to that PR to make things simpler for myself in order to pick up both patches locally.


Zoom in and out of canvas

There were some early tests using escobar5/raphael-pan-zoom which:

Works in a desktop context but, as written, don't update the position and offset of the raphael-sketchpad pen so paths don't appear where they should.
Seems to work correctly in a mobile context except that the entire HTML canvas, rather than just the SVG element, is zoomed in and out.

https://dmitrybaranovskiy.github.io/raphael/reference.html#Paper.setViewBox
There are two things to figure out:

UI/UX for taking raphael.sketchpad out of editing mode and listening for pan/zoom events
Adjusting the offset (_offset) for the pen in raphael.sketchpad


I have a slide presentation which allows users to zoom in, on an iPad they can pinch-out to zoom in. I've got additional slides that I manually add in at the end of the presentation

### Expected Behavior
When the user gets to the new, added slides they should still be able to pinch-out to zoom into the image
Actual Behavior
Pinching-out does nothing, double clicking does zoom in, but the users can't scroll around the image, it just scrolls the image as through it was the normal size.
Any help would be greatly appreciated.

Quick update, I managed to fix it by calling
swiper.zoom.init();
on the line after the appendSlide action. As it seems the problem was the zoom touch gestures weren't being added to the new div. This fixes it and it seems to work perfectly.

Issue is closed because of outdated/irrelevant/not actual/needed more information/inactivity.
If this issue is still actual and reproducible for latest version of Swiper, please create new issue and fill the issue template correctly:

Clearly describe the issue including steps to reproduce when it is a bug.
Make sure you fill in the earliest version that you know has the issue.
Provide live link or JSFiddle/Codepen or website with issue


personalise the template app

This PR:

adds/changes/removes etc

Testing
Description on how {APP-NAME} can be tested.

[ ] fresh install works
[ ] upgrade from previous version works

Checklist

[ ] Update changelog in CHANGELOG.md.
[ ] Make sure values.yaml and values.schema.json are valid.

merging this in


fix auto deplay paimon oss/s3 jar file

Proposed changes
No need to deploy paimon oss/s3 jar files manually.
Include them in preload-extensions-jar-with-dependencies.jar
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Fix Keycloak session leak

The sessions opened by Login seem to expire after an hour or so, but we still always have a lot of open sessions if we don't close them.

Checklist

[x] Keep pull requests small so they can be easily reviewed.
[x] Categorize the PR by setting a good title and adding one of the labels:
bug, enhancement, documentation, change, breaking, dependency
as they show up in the changelog

Finally got around to testing this. Works on the beta cluster, after realizing what we are doing in the keycloak adapter is actually wrong..


Remove EoS warning in the "About Thorium" page

I think "To get future Thorium updates, you'll need Windows 10 or later. This computer is using [Insert Windows Version]." warning in the "About Thorium" page should be removed, it is false and ruins the about page aesthetically.

@Daniel7689 I will remove this.

@Daniel7689 There you go > https://github.com/Alex313031/thorium-win7/commit/d8c060e96113fdc9227f00e57a86e0b6838ed0b1
I will be making new builds based on 109.0.5414.141, with more backported patches from thorium M111

@Daniel7689 Hi, just also wanted to make a note that I just now made a patch to (on top of not showing the annoying EOS stuff), make thorium think it is still "servicable" on win7/8. This will be useful because I eventually would like to make an update server for Windows, that way thorium will auto update (on linux it already has auto update), and this will let me also extend that capability to the win7 builds.


error: cannot insert multiple commands into a prepared statement

I installed a local version of SqlPad, loaded it up in a browser, connected to a remote database, and ran a query with no problem.
I then installed a version of SqlPad on an EC2 instance, loaded it up in a browser, connected to the same remote database, tried the same query, and got the error "error: cannot insert multiple commands into a prepared statement".  I found this page (https://github.com/brianc/node-postgres/issues/33) which seems possibly related, but I couldn't tell for sure.
Is it possible that there is some dependency that is not listed in the package.json that I have locally but am missing on the server?
The query looks like:
/* create temporary table */
create temporary table users_from_acme_company (
    user_id uuid
);

/* select data into the temporary table */
INSERT INTO users_from_acme_company (user_id)
select user_id from users
join institutions_users iu on iu.user_id = users.id
join institutions on institutions.id = iu.institution_id
where institutions.name = 'ACME';

select * from users_from_acme_company;

That's interesting - I'm not sure what it would be but I can dig into it a bit.
May I ask what version of node.js and postgres you are using?
I don't think it would be a missing dependency. If anything it might be slightly different module versions that could cause it. I'll see if there is anything in the node-postgres usage I should be doing differently too. The queries sent to postgres should be sent as text statements, not prepared statements, so its a very confusing error message to be receiving: https://github.com/brianc/node-postgres/wiki/Prepared-Statements

Thanks for the reply Rick, and things for sharing SqlPad.
Our node version is v0.10.33
Our Postgres version is 9.3.3
Not sure it's helpful, but the AMI we're using on the EC2 instance
is amzn-ami-hvm-2014.09.2.x86_64-ebs (ami-146e2a7c)
Let me know if you figure anything out.  Thanks again.

Nate

On Tue, Feb 24, 2015 at 11:27 AM, Rick Bergfalk<EMAIL_ADDRESS>wrote:

That's interesting - I'm not sure what it would be but I can dig into it a
bit.
May I ask what version of node.js and postgres you are using?
I don't think it would be a missing dependency. If anything it might be
slightly different module versions that could cause it. I'll see if there
is anything in the node-postgres usage I should be doing differently too.
The queries sent to postgres should be sent as text statements, not
prepared statements, so its a very confusing error message to be receiving:
https://github.com/brianc/node-postgres/wiki/Prepared-Statements
—
Reply to this email directly or view it on GitHub
https://github.com/rickbergfalk/sqlpad/issues/12#issuecomment-75789653.

Thanks, and you're welcome!
The talk about versions got me thinking and I've pinpointed the issue (although its unrelated to node and postgres versions).
In version 1.2.1 of SqlPad I changed the postgres module used to run queries to https://github.com/brianc/node-pg-cursor. I think it is accidentally handling plain text queries as a prepared statements, because the same queries work in version 1.2.0.
Until there's a fix ready, you can work around this by explicitly install version 1.2.0 on your server. That will allow you to run a query with multiple statements.

Awesome.  Did you change this in the last few days?  Is that why it works
on the first install I did but not on the second one?
On Wed, Feb 25, 2015 at 2:19 PM, Rick Bergfalk<EMAIL_ADDRESS>wrote:

Thanks, and you're welcome!
The talk about versions got me thinking and I've pinpointed the issue
(although its unrelated to node and postgres versions).
In version 1.2.1 of SqlPad I changed the postgres module used to run
queries to https://github.com/brianc/node-pg-cursor. I think it is
accidentally handling plain text queries as a prepared statements, because
the same queries work in version 1.2.0.
Until there's a fix ready, you can work around this by explicitly install
version 1.2.0 on your server. That will allow you to run a query with
multiple statements.
—
Reply to this email directly or view it on GitHub
https://github.com/rickbergfalk/sqlpad/issues/12#issuecomment-76034698.

Yep - I did some work on it over the weekend. I didn't think about the timing of your issue, but it makes sense. You likely have 1.2.0 on the first install and 1.2.1 on the second one.

I downgraded to 1.2.0 and it works.  Thanks again, and good work!
On Wed, Feb 25, 2015 at 2:45 PM, Rick Bergfalk<EMAIL_ADDRESS>wrote:

Yep - I did some work on it over the weekend. I didn't think about the
timing of your issue, but it makes sense. You likely have 1.2.0 on the
first install and 1.2.1 on the second one.
—
Reply to this email directly or view it on GitHub
https://github.com/rickbergfalk/sqlpad/issues/12#issuecomment-76040378.

@nbrustein Just a heads up - version 1.3.0 has been published to npm. It has a work-around built in for this issue, so feel free to upgrade.

awesome.  thanks.
On Sun, Mar 1, 2015 at 6:05 PM, Rick Bergfalk<EMAIL_ADDRESS>wrote:

@nbrustein https://github.com/nbrustein Just a heads up - version 1.3.0
has been published to npm. It has a work-around built in for this issue, so
feel free to upgrade.
—
Reply to this email directly or view it on GitHub
https://github.com/rickbergfalk/sqlpad/issues/12#issuecomment-76637089.

it worked
On Sun, Mar 1, 2015 at 8:48 PM, Nathan Brustein<EMAIL_ADDRESS>wrote:

awesome.  thanks.
On Sun, Mar 1, 2015 at 6:05 PM, Rick Bergfalk<EMAIL_ADDRESS>wrote:

@nbrustein https://github.com/nbrustein Just a heads up - version
1.3.0 has been published to npm. It has a work-around built in for this
issue, so feel free to upgrade.
—
Reply to this email directly or view it on GitHub
https://github.com/rickbergfalk/sqlpad/issues/12#issuecomment-76637089.


[WIP] Add support for a few missing options from buildah run

 --add-host
 --cgroup-parent
 --cpu-period
 --cpu-quota
 --cpu-shares
 --cpuset-cpus
 --cpuset-mems
 --memory
 --memory-swap
 --security-opt
 --ulimit

Signed-off-by: Daniel J Walsh<EMAIL_ADDRESS>

On the plane flight back from Europe, I was adding a bunch of options needed for buildah run and buildah build.  This is how far I got,  If someone would like to take this patch over or break the patch into several smaller patches, that would be welcome.

I currently commented out support for seccomp, since this will be much more complicated.

Taking this over.

@umohnani8 there are a number of trello cards set up for these.  Please grab any that you're going to work on.  I'm on most at the moment, but am not actively working them, I probably should remove myself from them.
Also I'd suggest doing smaller patches and we'll need to also update "podman build".

@TomSweeneyRedHat yeah, I plan to break this into smaller patches. And will add myself to the trello cards. Thanks for the info :)

Closing since @umohnani8 has taken it over.


acos 0.99+ values are out by several degrees

Hi,
Acos input values between ~0.99 - 1.0 lead to some fairly imprecise output values, off by up to 5 degrees.
eg.
input=0.9941521, expected=0.1082002,  real=0.06253052, diff=0.04566966, :: degrees diff:2.616679
input=0.9961014, expected=0.0883307,  real=0,          diff=0.0883307,  :: degrees diff:5.060977
input=0.9980507, expected=0.06244908, real=0,          diff=0.06244908, :: degrees diff:3.578069

I found this whilst implementing a quaternion angle diff function which had some nasty snapping in these values leading to it being 2-5 degrees off floating point math acos.
I'm not sure what to suggest to improve it. I was considering generating a larger LUT table, however, I'd probably need to change some of the precision bit stuff.
I thought I'd raise it incase you also see this as an issue?
Thanks for putting this project up on github.

Yeah, that is expected for such a small LUT
In the past of this library, I had LUT entry for each single FP value, but that resulted in a ~30-60kb LUT file (for each function)
I thought to get it back, as I also saw some small issues when I was implementing transformation logic
I will think about whenever it's possible to have both approaches

There is one thing you can do to reduce the deviation:

idk why last 3 entries were 0, but with that change, issue is only on the last case

Thanks for the suggestion! Adding those extra values appears to reduce the problem a fair bit, say by ~50%. It's less noticeable in my application now.
I slowed down what I was doing and saw that a 2-3(?) deg snap still happens as shown in the video I've included. The red arrow is fixed point and the magenta arrow is floating point. They are both trying to rotate towards the green line.
https://user-images.githubusercontent.com/51151665/114925169-af2f7180-9e26-11eb-8bca-86bb9334cccc.mp4
It's tricky. You have ample precision in parts of the table where the curve delta is small, but then at the extremes of the acos graph you have a larger value delta. Ideally you'd have some sort of 'flexi' LUT which accommodated that (but still had only 512 values). It might be that the easiest thing is to simply increase the LUT size slightly though. I'm not sure, what do you think? Have you used it much in a real application yet?

Regarding the last 3 entries, I did spot this in Data.cs:
//Special handling for value of 1, as graph is not symmetric
            AcosLut.Add(AcosLut[ACOS_VALUE_COUNT - 1]);
            AcosLut.Add(AcosLut[ACOS_VALUE_COUNT - 1]);

I guess that accounts for the zero values? I confess I don't understand what this is for yet.

The problem with LUTs which have non-uniform deltas is querying becomes a fair bit more expensive
I've done 2D physics & transformations, but it doesn't use Acos/Asin (which I assume is also affected by that issue)
Soon I will also dive into 3D, so will face this issue too
I think I will make a branch which will use full LUT (so no lerping will be needed) on a weekend

Cool, maybe this is the best approach. I wonder if it's worth trying a compromise size between the two extremes later, but probably best to have correctness first and then if it is a performance issue you could always come back to it.
I see that FixedMath.net has ~2mb lut for some of these functions so it's probably not terrible, even though that lib is less performance orientated.
I'll be using it for lots of 3d rotations so I'll probably do some profiling at some point.

Ideally your whole LUT fits in CPU memory, at least L3, otherwise any physics will be dead slow. 2MB won't really fit realistically next to all the other LUTS.

Yes @Vincenz099 it's a good point. Are any difficulties thrown up by a compromise solution between the current ~2kb(?) and the full 2Mb. Maybe 8-16kb is sufficient to avoid these sort of snapping issues I've observed in visualisations.

I remember we had similar problems with other Luts, where the near to zero and near to 1 are very inacurate, one could introduce an extra lut for the lower range, it's what the solution was before.. you could even add like 16 values just for the sub 0.99 or whatever. experiment with that.

The problem is simply this, for trigonometry  you are sampling on a constant range, but it is not a linear scale. Forgive my paint skills but lets say you have a value like this, and you sample constantly, because you linear interpolate between two samples you loose a lot in the beginning as you see.
https://i.imgur.com/8w47XfY.png

Ye, that was the most apparent on SQRT lut

Pushed the LUT for the positive side of Asin, which is used for the negative side, and for Acos
Chech HugeAsinAcos branch
It has an accuracy of 0.0001

I also tried to increase the LUT size from 512 elements to 4k elements, but the distance between the last entry and pre-last was still too big
So I have LUT entry for each FP value right now in the 0-1 range

Hi,
I've been running tests to try and understand the original problem (shown in the video above) more. Although acos was inaccurate, I don't think it was the sole cause of my rotation jitter.
So far I think the thing that was causing the most precision loss was conversions from 4x4 matrices to quaternions and back.
I was getting a quaternion rotation from a transform 4x4 matrix, modifying the quaternion, then applying it back to the transform matrix, and repeating that each frame. Instead of this I switched to storing a separate quaternion rotation and only manipulating that. The 4x4 matrix is effectively only a by-product of the quaternion rotation now (quaternion is applied to the matrix but I don't read from the matrix when changing rotation).
This isn't a complete answer but I am guessing that going between matrices & quaternions repeatedly involved a fair few trig functions which may have contributed to the precision loss, but I'm not sure.
The jitter is eliminated by avoid these extra conversions. My only lesson learned so far is to bare in mind these sort of things happen in fixed point!


LaTeXTransform: Escape math delimiters

MathJax renders all text in math mode anyway.
$, $$, \(...\), and \[...]\ will be escaped in order to hide them in the output.

I'm confused... isn't this the latex transform where you hand straight maths over?

Yes, but if you use Math from IPython.display or sympy it will display $$ before and after the equation.

Ah. Good to know!
Thanks @lgeiger.

If it's OK, I'll make a patch release later today.

This does change the functionality in the frontend without being backwards compatible. For strict semver this needs to be a new transform (or incorporate options in a way that let this be enabled/disabled). Otherwise, it's a major release.

Willing to wager that if we've been broken this whole time and it wasn't compatible with the current Jupyter notebook implementation then a patch release is appropriate.

@rgbkrk I checked beforehand. The notebook doesn't display the dollars surrounding an equation.
I can take a closer look at the current behavior of the notebook and check for compatibility.


Livereload for slim/haml/rb

Breakfast is doing live reloads only on saving files with .html inside file name (besides css/js assets). Is it possible to reload on slim/haml saving without having .html inside template name? Also, is it possible to reload on ruby files changes?

Hey @devilcoders sorry for the delay. Yep both should be possible. I'm pushing out a pretty big release in the next few days. I'll probably make ruby changes default to off

@devilcoders if you follow the upgrade instructions there should now be support for slim, haml and ruby.
Let me know if something doesn't work quite how you expect it.

@PatKoperwas Cool, thanks. We switched to webpack-rails actually, but I will check new updates and see if we can go back. Thanks!

Fixed with #4


set-override.py creates new version of alarm-overrides-value schema

I've noticed the AlarmOverrideUnion.avsc is bumped to version 2 the first time set-override.py is executed. However, executing set-activation.py does not bump AlarmActivationUnion.avsc. No Java Kafka Streams apps are even running. Using describe-version.py to dump both version 1 and 2 then running diff reveals they're identical.  More investigation needed.

This is likely due to custom AvroSerializerWithReferences code as overrides use it, activations do not.

Fixed in release 4.2.0 by using config use.latest.version


[BUG] Should experts that aren't currently staff be included if they have an external affiliation?

The user with iamID =<PHONE_NUMBER> has na ppsAssocitions.  I notice they have isStaff=true, and isExternal=true.  Can you describe the association of users like this to Davis ?
  {
    "iamId": "1000140132",
    "ppsId": null,
    "employeeId": null,
    "isStudent": false,
    "isStaff": true,
    "isExternal": true,
    "ppsAssociations": null,
    "sisAssociations": null,
  }

From IET:
Hi Quinn,
Thank you for contacting IT Express. It looks like this user is currently just a temporary affiliate sponsored by the Earth and Planetary Sciences department but has had on and off staff/faculty status with UC Davis over the past few years.
Please let me know if you need any additional information.

@Vensberg whatever you decide, we need to modify the code a bit, so assign me and send me a note in the issue.

@qjhart, it looks like this change in IAM triggers deletion of Elements data, so there is nothing for us to work with.


feat: Update doc and code for Maven plugin

@AdamKorcz friendly ping. Can you resolve the comments if my answers clarify things?

@AdamKorcz friendly ping. Can you resolve the comments if my answers clarify things?

Sorry for the delay. Your changes LGTM!

@kpk47 can you take a look?

@kpk47 ptal


OS X build errors on master [phab:T102]

Created by benjyw at 2016-02-10 16:33:45:
Following these instructions: https://www.kythe.io/2015/06/05/osx-support/ and building at a recent master HEAD.
Got this error during bazel build //....
INFO: Found 749 targets...
INFO: From Linking kythe/cxx/indexer/cxx/test:
ld: warning: option -noall_load is obsolete and being ignored
error: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/install_name_tool: changing install names or rpaths can't be redone for: bazel-out/local_darwin-fastbuild/bin/kythe/cxx/indexer/cxx/test (for architecture x86_64) because larger updated load commands do not fit (the program must be relinked, and you may need to use -headerpad or -headerpad_max_install_names)
ERROR: /Users/benjyw/src/kythe/kythe/cxx/indexer/cxx/BUILD:95:1: Linking of rule '//kythe/cxx/indexer/cxx:test' failed: clang failed: error executing command /Users/benjyw/src/kythe/tools/cpp/clang -o bazel-out/local_darwin-fastbuild/bin/kythe/cxx/indexer/cxx/test '-Wl,-rpath,$ORIGIN/../../../../_solib_darwin/' ... (remaining 86 argument(s) skipped): com.google.devtools.build.lib.shell.BadExitStatusException: Process exited with status 1.

Note that building at the v0.0.17 tag works fine, so I'm not stuck, but thought you might find this information useful.

My versions:
Kernel:
[~/src/kythe][master]$ uname -a
Darwin xxxx.local 15.3.0 Darwin Kernel Version 15.3.0: Thu Dec 10 18:40:58 PST 2015; root:xnu-3248.30.4~1/RELEASE_X86_64 x86_64

OS X:
[~/src/kythe][master]$ sw_vers
ProductName:	Mac OS X
ProductVersion:	10.11.3
BuildVersion:	15#212

gcc/clang:
[~/src/kythe][master]$ gcc --version
Configured with: --prefix=/Applications/Xcode.app/Contents/Developer/usr --with-gxx-include-dir=/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.11.sdk/usr/include/c++/4.2.1
Apple LLVM version 7.0.2 (clang-700.1.81)
Target: x86_64-apple-darwin15.3.0
Thread model: posix

bazel:
[~/src/kythe][master]$ bazel version
Build label: 0.1.4-homebrew
Build target: bazel-out/local_darwin-fastbuild/bin/src/main/java/com/google/devtools/build/lib/bazel/BazelServer_deploy.jar
Build time: Sat Jan 23 12:15:20 2016<PHONE_NUMBER>)
Build timestamp:<PHONE_NUMBER>
Build timestamp as int:<PHONE_NUMBER>

kythe:
[~/src/kythe][master]$ git rev-parse HEAD
a10b8b927bceef37204a6fb5dac34d2f45c5950e

tool paths:
[~/src/kythe][master]$ ./setup_bazel.sh
Using GOROOT found at /usr/local/Cellar/go/1.5.3/libexec
Using node.js found at /usr/local/Cellar/node/0.10.33_1/bin/node
Using clang found at /usr/bin/clang

zarko wrote at 2016-02-10 19:13:38:
This is working for me.
bazel version
Build label: 0.1.5
Build target: bazel-out/local_darwin-fastbuild/bin/src/main/java/com/google/devtools/build/lib/bazel/BazelServer_deploy.jar
Build time: Tue Feb 9 19:11:16 2016<PHONE_NUMBER>)
Build timestamp:<PHONE_NUMBER>
Build timestamp as int:<PHONE_NUMBER>

uname -a
Darwin XXX 15.3.0 Darwin Kernel Version 15.3.0: Thu Dec 10 18:40:58 PST 2015; root:xnu-3248.30.4~1/RELEASE_X86_64 x86_64

sw_vers
ProductName:	Mac OS X
ProductVersion:	10.11.3
BuildVersion:	15#212

gcc --version
Configured with: --prefix=/Applications/Xcode.app/Contents/Developer/usr --with-gxx-include-dir=/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.11.sdk/usr/include/c++/4.2.1
Apple LLVM version 7.0.0 (clang-700.0.72)
Target: x86_64-apple-darwin15.3.0
Thread model: posix

git rev-parse HEAD
187331c92a4764b736657163428195730993ff71

./setup_bazel.sh
Using GOROOT found at /usr/local/go
No node.js installation found.
Using clang found at /usr/bin/clang

zarko wrote at 2016-02-10 19:14:36:
Note that both build //... and test //... are working at that commit.

benjyw wrote at 2016-02-17 01:46:35:
This still happens at current master ():

$:[~/src/kythe][master]$ bazel build //...
INFO: Waiting for response from Bazel server (pid 37121)...
INFO: Found 756 targets...
INFO: From Linking kythe/cxx/indexer/cxx/test:
ld: warning: option -noall_load is obsolete and being ignored
error: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/install_name_tool: changing install names or rpaths can't be redone for: bazel-out/local_darwin-fastbuild/bin/kythe/cxx/indexer/cxx/test (for architecture x86_64) because larger updated load commands do not fit (the program must be relinked, and you may need to use -headerpad or -headerpad_max_install_names)
ERROR: /Users/benjyw/src/kythe/kythe/cxx/indexer/cxx/BUILD:95:1: Linking of rule '//kythe/cxx/indexer/cxx:test' failed: clang failed: error executing command /Users/benjyw/src/kythe/tools/cpp/clang -o bazel-out/local_darwin-fastbuild/bin/kythe/cxx/indexer/cxx/test '-Wl,-rpath,$ORIGIN/../../../../_solib_darwin/' ... (remaining 86 argument(s) skipped): com.google.devtools.build.lib.shell.BadExitStatusException: Process exited with status 1.
INFO: Elapsed time: 9.189s, Critical Path: 4.75s

Will try with bazel 0.1.5 and see if that helps any.

benjyw wrote at 2016-02-17 01:59:42:
That didn't help. Same error with bazel 0.1.5.

benjyw wrote at 2016-02-18 00:09:18:
I've submitted a code review that fixes this for me:
#1007

zarko wrote at 2016-03-21 23:55:52:
Fixed in #1008


Adding support to lessOptions

Thank you so much for the PR, I really appreciate!
I like the example and test code, it makes the JS code clean.
If it's not too much work, could you reduce the size of the sample and test by importing only the necessary functions, I don't think that we need the entire tinycolor.less file to support the test code.
Thanks,

It's not a problem, I will do when got back to my computer.
Gullit Miranda
Em dom, 25 de nov de 2018 20:59, Jean-Philippe Bergeron <
<EMAIL_ADDRESS>escreveu:

Thank you so much for the PR, I really appreciate!
I like the example and test code, it makes the JS code clean.
If it's not too much work, could you reduce the size of the sample and
test by importing only the necessary functions, I don't think that we need
the entire tinycolor.less file to support the test code.
Thanks,
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jean343/styless/pull/31#issuecomment-441481734, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AB8L9jGd-QT1WDrI0lEGe_fkoHvYl7Mvks5uyyDagaJpZM4Yxm-6
.

so the settings I made will only work after the package is published, which
isn't a problem for me
Gullit Miranda
Em dom, 25 de nov de 2018 21:00, Jean-Philippe Bergeron <
<EMAIL_ADDRESS>escreveu:

@jean343 commented on this pull request.
In examples/.babelrc
https://github.com/jean343/styless/pull/31#discussion_r236094861:

@@ -1,7 +1,7 @@
{
"plugins": [

"styless",

["../dist", { "lessOptions": { "javascriptEnabled": true } }],

I understand that it is easier to test this way, but I prefer importing
styless to test the npm package.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jean343/styless/pull/31#pullrequestreview-178102037,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB8L9jiZLrUD_TQYx2C45HNsZ_N0e9ufks5uyyEGgaJpZM4Yxm-6
.

@jean343, updated

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket: