Add GET Images

Based on JSP Image API calls. Gets all the annotation information, and image information and sets it as a single JSON for the UI (which can then expect the same response as the JH Spawner UI expects from JSP API)
Work in progress:
Optional parameters not all set
Build status not checked
Part of: #148 and opendatahub-io/odh-dashboard#214

Ran formatting and fixed integration test issue in the force pushes.

@maroroman Can you make the changes above from @DaoDaoNoCode

Removed unused imports. My VS Code has a nice habit of automatically adding imports, but not removing them.

Added check for null/undefined in the tagAnnotations since getting images failed if tag annotations were null (which is the default for the imagestream yaml)

@maroroman @dlabaj what are we doing with this PR?  looks like it was destined for master.  Is this to be discarded, reworked?

@maroroman Can you update this PR to go into main.

Replaced by #254


What is the license for the code in the repository?

Your readme has an MIT license specified but the LICENSE file is for LGPL3.
Which license reflects the one for the code? it is not clear the way the information is displayed.
Thank you.

Oh I'm sorry for the wrong the license file, I will update the correct license as an MIT one.

I updated the license file ^^


Translations missing in app

Many of the strings on the Rewards page have not been translated including:

Referral section
Channel subscriber rewards
View rewards
Daily Watch Reward

There may be others, but those are the ones I'm seeing.

Specifically for rewards, since those strings are set on internal-apis, we would need a way to pass the current language to the reward/list call. And have the api return translated strings.

Closing in favor of https://github.com/lbryio/internal-apis/issues/1186


Better use Securely or implement self natives manager

reference https://github.com/LavaMoat/securely/issues/4
Securely has design problems and Snow is a victim to these problems, this needs to be handled in order to have better control on how natives are consumed and also to patch vulnerabilities that Snow has due to Securely

solved #36


1.0.1 removed from pypi?

Hello,
I just wanted to point out that it seems that v 1.0.1 has been removed from pypi - I see that 1.1.1 was released earlier today, but there appear to be no previous versions on pypy: https://pypi.org/project/csv2md/#history
Were earlier artifacts intentionally removed?

Apologies - this is the wrong repo for that project, please ignore this!


adap.tv VPAID tag

I was trying to use the player to play the following tag:
http://ads.adaptv.advertising.com/a/h/XUUFc52Hk6Fg01bMBS16Tqn_qngdDNKLCE+L94vK1JQ=?cb=[CACHE_BREAKER]&pageUrl=http%3A%2F%2Ftest.com&description=VIDEO_DESCRIPTION&duration=VIDEO_DURATION&id=VIDEO_ID&keywords=VIDEO_KEYWORDS&title=VIDEO_TITLE&url=VIDEO_URL&eov=eov
this is a demo XML tag that should play on each try.
I took the demo files and changed the adURL var and made sure all the dependencies work properly.
Do you have any suggestions what I might be doing wrong?

I've just tried the tag on the demo page and everything seems to work : http://mailonline.github.io/videojs-vast-vpaid/

@massa84 Not helpful, I have the same problem as @yairneumann11 with a Liverail tag, it works fine on the demo page, on my own demo page, but if I update the adUrl in the demo file it does not work at all, I cannot see any outgoing request to it.
As a side note it's extremely annoying to have to use gulp to run the demo.

@yairneumann11
Are you using videojs-vast-vpaid plugin? or the VPAIDFLASHClient?
If the VPAIDFlashClient, is a bit low level to use directly, the demo used in the VPAIDFlashClient is only a fast prototype to test the VPAIDFlashClient without depending on the plugin, and to allow other projects to use it

closing the issue, when more details are provided please reopen the issue

Hi!, If I am using  videojs-vast-vpaid plugin.How i get it work


Cherry-pick: Added a specific version for protobuf (#2201)

Added a specific version for protobuf in requirements.txt
Signed-off-by: Andrews Arokiam<EMAIL_ADDRESS>
What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Release note:

publish kserve sdk <IP_ADDRESS> version

/lgtm


Expand ReadsSparkSink to write to a sharded-bam

Currently, it only writes to a single-file BAM.

is this done?

ReadsSparkSink.writeReadsSharded does it right now. The only thing missing
is renaming all of the part files as .bam. I don't know how much we care to
do that.
On Wed, Sep 30, 2015 at 9:34 PM, Adam Kiezun<EMAIL_ADDRESS>wrote:

is this done?
—
Reply to this email directly or view it on GitHub
https://github.com/broadinstitute/hellbender/issues/854#issuecomment-144591017
.

we don't care as long as the reading code can recognize them and read them. Does it?

I haven't chained these together, but I can use samtools view with no problem. So I assume our tools would work too. (I think we can close this).


WIP: norm implementation

Initial proof-of-concept for https://github.com/QuantStack/xtensor/issues/417, please comment!
Remark: I had to remove auto-deduction of the xreducer's result_type, because it didn't work in all cases. Instead, the result_type must be specified explicitly in the call to make_xreducer_functor.

Sorry, forgot to upload the most important files.

Looks good to me.

Remark: I had to remove auto-deduction of the xreducer's result_type, because it didn't work in all cases.

Too bad, this was really elegant; just out of curiosity, which kind of situation was problematic ?

which kind of situation was problematic ?

The functor's first argument is exactly the result_type that we don't know yet. So far, the code relied on the fact that value_type was convertible to result_type, but this is no longer true when the norm of a complex array is computed.
Meanwhile, I noticed that I can solve this problem by passing the first argument through the init functor. So, I was able to restore the elegant solution.

Closing this one since #440 has been merged.


VkRipper unable to download images in their original size

Ripme version:
Java version: 1.8.0_111
Operating system: Microsoft Windows [Version 10.0.16299.309]
Exact URL you were trying to rip when the problem occurred: https://vk.com/album-156891750_249435170
Please include any additional information about how to reproduce the problem:

Expected Behavior
Download all images in their original size.
Actual Behavior
Downloaded images not in their original size.

I get the size from RipMe what I would get by downloading by hand.
Are you just dealing with VK compressing the photos?

This is the comparison of the first image in the album that I posted in my original description.
What I get from Ripper: https://i.imgur.com/PBnwhNC.jpg
Dimensions: 715 x 1080
What I get from downloading by hand: https://i.imgur.com/N4vZi0h.jpg
Dimensions: 1431 x 2160
If this is due to some settings about compression, please tell me where I can change it so that it will download the original size.


ModulNotFound Error

I wanted to install the module on my Raspberry Zero in a virtual environment.
It works fine with pip, the module is found in python. Pip installs it in the correct environment.
But as soon as I run the strandtest.py, I get the error ModulNotFound.
Without virtual environment with compiling the c-files by myself it works...

The little cogs in my brain just unstuck; are you running with sudo?

ok my fault,
when i start the programm with sudo i leave the virtual environment, so he can't find it, if the modul is only install in the environment, therefore i have to install it on the global python-path.
And we need sudo to get access to the pins, right? I can't start it, without sudo.

Use sudo --preserve-env PATH python example.py and it should work. I've published a new version of the package, too, which fixes some bugs and documents this caveat.

Yeah thank you, that works for me.


FilePicker doesn't open panel on latest SPFX 1.16.1

Category
[* ] Bug
Version
Please specify what version of the library you are using: [3.12.0]
Expected / Desired Behavior / Question
Expected that when I click on FilePicker control it should show panel.
Observed Behavior
When click on FilePicker it just doesn't open panel, nothing happens.
No message in console or Sharepoint log.
Steps to Reproduce
Create new solution and WebPart using latest microsoft toolchain.
Add FilePicker to the WebPart
        <FilePicker
                  buttonIcon="FileImage"
                  buttonLabel='Select Employee Documents'
                  onSave={(filePickerResult: IFilePickerResult[]) => {}}
                  // onChange={(filePickerResult: IFilePickerResult[]) => _onFilePickerSave(filePickerResult)}
                  context={props.context as any}
                  hideLinkUploadTab={true}
                  hideLocalMultipleUploadTab={false}
                  hideOneDriveTab={true}
                  hideRecentTab={true}
                  hideWebSearchTab={true}
                  hideLocalUploadTab={true}
                  hideOrganisationalAssetTab={true}
                />

run the web part.
The shows FilePicker but it doesn't react on click.
I have checked version it on SPFX 1.15.2 and it works fine there.
Also I have noticed that WebPartContext is not compatible anymore with FilePicker
Type 'WebPartContext' is not assignable to type 'BaseComponentContext'.
Types have separate declarations of a private property '_serviceScope'.ts(2322)
IFilePickerProps.d.ts(33, 5): The expected type comes from property 'context' which is declared here on type 'IntrinsicAttributes & IntrinsicClassAttributes & Readonly & Readonly<...>'
but it still works with other controls.

I have found the reason why FilePicker didn't work for me,
it turned out that  FocusTrapZone placed above  FilePicker caused this.
It even prevent opening file input.
After removing FocusTrapZone all works fine.


Getting error when trying to install a docset

I get this error when M-x counsel-dash-install-docset:
JSON-read: JSON readtable error
I tried accessing: https://raw.githubusercontent.com/Kapeli/feeds/master/
But I get:
400: Invalid request

Seems like a firewall issue.


Missing AFKPlus:jar:3.2.4 needed for building

When building the plugin with mvn I get this error:
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  31.727 s
[INFO] Finished at: 2021-11-05T12:05:27+01:00
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal on project afkpluspapi: Could not resolve dependencies for project net.lapismc:afkpluspapi:jar:0.3.0: Could not find artifact net.lapismc:AFKPlus:jar:3.2.4 in spigotmc-repo (https://hub.spigotmc.org/nexus/content/repositories/snapshots/) -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/DependencyResolutionException

How can I build the plugin?

I'm sorry, but notifications for this repo seem to be off.
Which means I have missed years of issues posted here. I've rectified the issue so that I should be assigned to issues from now on and am currently handling the backlog.
This issue has been fixed in the most recent update (about to go live) by adding a maven repo that has my projects in it.


Mapillary Date Filter

I'm working on adding a date filter to the Mapillary layer. The plan is to use a calendar date picker similar to the Mapillary web app. I'm thinking it will live inside the current Mapillary box of the Photo Overlays list.
I should have an initial version next week to show off and make sure we're all on the same page of how this will look.

Hi @zlavergne this sounds like https://github.com/openstreetmap/iD/issues/4518, right?
Personally, I will be very happy with any kind of filtering!
Just as a suggestion, I wonder if a calendar date picker is even the easiest (first) step.
My personal use cases are all about reducing the clutter of too many images, like …

only show recent images – last 30 days, last 12 month, last 3 years
only show images from specific users (only me, only someone; this is https://github.com/openstreetmap/iD/issues/5307)
ideally there where a "only show images that are taken from sidewalks, not streets – but AFAIK this is not a filter the API provides

But back to the date picker: In my mind the pre defined selections would be faster to choose than the date picker and less complexity. – Depending on the UseCase you have in mind of course.

JOSM uses this:

only show recent images – last 30 days, last 12 month, last 3 years

This we've found is not feasible for many users, who need a specific date ranger.

only show images from specific users

The username filter will be great, but lower demand.

ideally there where a "only show images that are taken from sidewalks, not streets

A lot of backend on the Mapillary side to first classify the images this way, based on content and GPS, not reliable to do for now. :) But would be really good.

#4518 is indeed photo date filtering, but I'll leave this open since Zach seems to be working on something specific to Mapillary.

only show recent images – last 30 days, last 12 month, last 3 years

This we've found is not feasible for many users, who need a specific date ranger.

I could see how mapping teams might need to see a specific date range, but most iD users just need the photo overlay filtered down to a manageable amount of data. Date pickers are finicky and aren't useful if you don't know beforehand what dates to enter. Filtering by the previous 30/90/365 days is simpler but is blind to the actual data clustering.
I think my preferred solution is a dropdown for filtering to the most recent 1/2/3/5/10 capture sequences in the visible map area. This could be set to something low by default so mappers don't get sequence spaghetti right away. Then we could consider date filtering (or even just filtering to specific sequences) as an advanced option.

We will test some prototypes and share results!

Hi folks - any movement on this added functionality yet? I would love to be able to reduce the spaghetti effect in popular areas.

Hi, you can try the date filter in this PR: https://github.com/openstreetmap/iD/pull/7847 let me know if you have any feedback or suggestions.

https://github.com/openstreetmap/iD/issues/4518#issuecomment-712434240


chore: modify jest config, ts-jest => 26.xx.xx, tsconfig'include add …

I found some big problems:

ts-jest does not check the converted type file, the previous configuration was problematic

I have now fixed it, but due to the strict mode of tsconfig, there are some type errors that need to be reworked  :sob:

ref: #1737

Tests are passing, changes look good. Anything else you want to do in this PR, or is it good to merge?

Tests are passing, changes look good. Anything else you want to do in this PR, or is it good to merge?

I think it is OK to merge, and I will initiate PR in future updates

Great, thanks again!


keyvault check-name always returns true

Describe the bug
Calling az keyvault check-name --name {name of an existing vault} --subscription {subscription name}
always returns { "message": null, "nameAvailable": true, "reason": null }
Related command
az keyvault check-name
Errors
No errors, just an invalid response.
Issue script & Debug output
No error, so nothing to debug
Expected behavior
Similar response to when I call az storage account check-name --name {existing account}:
{ "message": "The storage account named {existing account} is already taken.",  "nameAvailable": false, "reason": "AlreadyExists" }
but with the message referring to a key vault name.
Environment Summary
azure-cli                         2.52.0
core                              2.52.0
telemetry                          1.1.0
Extensions:
azure-devops                      0.21.0
redisenterprise                    0.1.2
Dependencies:
msal                            1.24.0b1
azure-mgmt-resource             23.1.0b2
Python location 'C:\Program Files\Microsoft SDKs\Azure\CLI2\python.exe'
Extensions directory 'C:\Users\Alberto.Gonzalez.azure\cliextensions'
Python (Windows) 3.10.10 (tags/v3.10.10:aad5f6a, Feb  7 2023, 17:20:36) [MSC v.1929 64 bit (AMD64)]
Legal docs and information: aka.ms/AzureCliLegal
Your CLI is up-to-date.
Additional context
No response

Thank you for opening this issue, we will look into it.

@algonzalez-wf Currently az keyvault check-name only checks name availability of mhsm, vault is not covered:

Got it. Thanks. I'll use $result = az keyvault list --resource-group $rgName --resource-type vault --query "contains([].name, '$kvName')" instead


[Bug] Cannot mount cross region open data

Describe the bug
Suppose my swb is in us-east-1 region, I create a linux workspace and try to mount https://registry.opendata.aws/hecatomb/ open data which is hosted on S3 us-west-2 region.
The workspace can be created successfully, but the hecatomb data cannot be mounted to the EC2 instance.
I think it should be a common bug when swb try to mount cross region data.
I did some debug as below:
Below is S3Mount parameter in cloudformation
[{"id":"hecatomb","readable":true,"writeable":false,"bucket":"hecatombdatabases","prefix":"/"}]
in the main/solution/post-deployment/config/environment-files/bin/mount_s3.sh file, there is code as below:
if [ "$s3_role_arn" == "null" ]
then
printf 'Mounting internal study "%s" at "%s"\n' "$study_id" "$study_dir"
goofys --region $region --acl "bucket-owner-full-control" "${s3_bucket}:${s3_prefix}" "$study_dir"
The region is the workspace region, not s3 bucket region.
I think this cause the error of mounting data.
Could you try to fix this bug?
Thanks
Mingtong

Hi @dengmingtong, thanks for bringing up this issue! I've added this to our team's backlog.

No, I did not enable TRE feature. You can repeat this issue when you mount another regions data.

any update?

This still appears to be a problem in the latest version of SWB; it has already spawned two possible duplicate issues too, in #1013 and #1045.
Appears to be quite an oversight that mounting of open data is only supported in the same region...

I have also encountered this bug. There are 119 open data on SWB but none of them appears to be available to us-east-2.  Please

If this is a limitation of SWB, can we list the region of the datasets so that we know what datasets are available?
@SanketD92 mentioned TRE (AppStream) could solve this issue. Could you elaborate how AppStream helps and how to enable AppStream?

@BoPeng You can work around this by adding a script to your AMIs to mount the open data buckets either without specifying the region, or alternatively by looking up and specifying the correct region. (I think it should work just without specifying the region - the problem is that the script that ships with SWB hardcodes --region us-east-1).
You can create a script based off the existing mount_s3.sh that's added by the SWB bootstrap script.

@tdmalone Why do we even have option --region for command goofys in mount_s3.sh if the region can be auto detected?
https://github.com/awslabs/service-workbench-on-aws/blob/1d2fe14e04218070aa7034a9868002643da88bc6/main/solution/post-deployment/config/environment-files/bin/mount_s3.sh#L80
I just tried this command with a dataset in us-east-1, the first two commands work, and the third does not, so it is better not to specify --region at all.
goofys gdc-target-phs000218-2-open /mnt/fah
goofys --region us-east-1 gdc-target-phs000218-2-open /mnt/fah
goofys --region us-east-2 gdc-target-phs000218-2-open /mnt/fah

@BoPeng I'm not sure! There may be a reason, but I haven't done enough testing with it. First thing I'd want to check with your first command is whether it still works for non us-east-1 buckets. If it does, you're probably right.

Yes, I actually tried to mount another dataset on us-east-2
goofys fah-public-data-covid19-cryptic-pockets /mnt/fah

and it worked.

I created a PR to demonstrate the idea, although I do not have a test environment to test the patch.


Compatibility with neverlink

In rare cases, a native-image target needs various JARs available at native compile time only, so classes can be referenced for compile-time directive mechanisms like GraalVM Features.
Most of the time these dependencies are inert on the compile or runtime classpath; because they export no usable functionality, directive classes are typically compiled out early and do not end up in the final binary. Keeping them on the classpath, then, is normally fine.
The problem arises when directive classes such as Features depend on classes within the SVM API. Such dependencies are embedded within GraalVM in addition to being made available via Maven. When native-image is invoked, these classes are present automatically, but before that step, when Bazel is building the regular java with javac, it won't be able to find those symbols.
The user's first impulse is probably to add @maven//:org_graalvm_nativeimage_svm to their deps, but this yields a nasty warning from the native-image compiler (and rightfully so, since these dependencies must always align perfectly if they are provided on the classpath or modulepath):
Error: Class-path entry file:///.../processed_objectfile-23.1.1.jar contains class com.oracle.objectfile.pecoff.cv.CVUtil. This class is part of the image builder itself (in file:///.../objectfile.jar) and must not be passed via -cp. This can be caused by a fat-jar that illegally includes svm.jar (or graal-sdk.jar) due to its build-time dependency on it. As a workaround, -H:+AllowDeprecatedBuilderClassesOnImageClasspath allows turning this error into a warning. Note that this option is deprecated and will be removed in a future version.

The user may sensibly try neverlink next, but, because of this issue, the following surfaces:
Error: Feature <feature> class not found on the classpath. Ensure that the name is correct and that the class is on the classpath.
com.oracle.svm.core.util.UserError$UserException: Feature gvm.workerd.feature.WorkerDFeature class not found on the classpath. Ensure that the name is correct and that the class is on the classpath.

Now Bazel can find the dependency and build it with javac, but Bazel considers native-image a run-phase task (I think?), and so omits the entire dependency from the classpath.

This probably involves switching to transitive_compile_time_jars or combining it with our current use of transitive_runtime_jars.


Add find command to DG

Codecov Report
Base: 74.28% // Head: 74.28% // No change to project coverage :thumbsup:

Coverage data is based on head (78b69de) compared to base (76275f5).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #157   +/-   ##
=========================================
  Coverage     74.28%   74.28%
  Complexity      648      648
=========================================
  Files           102      102
  Lines          1894     1894
  Branches        224      224
=========================================
  Hits           1407     1407
  Misses          422      422
  Partials         65       65

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Cannot Inject on iOS 16.5 Rootless Palera1n

Heyya
Currently trying to Inject a dylib into a Process on ios 16.5, an iPhone 8.
Right now, my target process is SpringBoard, and the dylib is just this code:
__attribute__((constructor))
static void customConstructor() {
    NSLog(@"Test loaded");
}

however, when i execute opainject on my phone, the process gets killed. Thats the output before the kill:
OPAINJECT HERE WE ARE
RUNNING AS 0
Got task port 7427 for pid 35!
[prepareForMagic] done, ropLoop: 0x19594B5F4
[createRemotePthread] Created bootstrap thread... now waiting on finish
[createRemotePthread] Bootstrap done!
[createRemotePthread] WARNING: failed to get thread state of thread 0 when trying to find pthread, error: (ipc/send) invalid destination port (268435459)
[createRemotePthread] Found pthread: 4355
[sandboxFixup] not needed, bailing out.
[injectDylibViaRop] Preparation done, now injecting!
[injectDylibViaRop] dlopen: 0x1BAD28C04, dlerror: 0x1BAD2A0F4
Killed

I tried to run opainject as root and as mobile user, to see if it makes any difference. but it does not.
after adding some logs, i found the crashing line is this one here:
https://github.com/opa334/opainject/blob/main/rop_inject.m#L321
but i dont have any clue whats the problem or how to fix it.
Would be nice if you can fix it. If you need any more logs or infos from me, im happy to help

16.4 broke several thread related APIs, I looked into this before but it seems very hard to fix although there might be an easy way out as debuggers also need these APIs, maybe it now checks whether the target process is traced via ptrace... Not sure.

Okay as it turns out only the
<key>com.apple.private.thread-set-state</key>
<true/>

entitlement is enough to make opainject work on iOS 16.4+

Just tried it out, injection works fine on my 16.5 phone.
Thanks, didnt though it would be such a small thing


Pinentry "dumb" React component

@keybase/react-hackers Ready for a look, just a stub awaiting the integration.  Screenshot attached.

I love the screenshot/gif, we should do that more often!

:+1: % C


fix(script): price extraction logic

What this PR does
Fixes issue #12, that's causing the scheduled workflow to fail.
Where should the reviewer start

Take a look at Actions (workflows) ➡  Extract data from DRTT (main.yml)
The workflow fails due to the issue in line 58 of the script.py file.

Related issues

Closes #12

@Dntfreitas
Can you review the pipeline on GitHub actions?
The fix looks good to me.
Thanks for the contribution @HarryVasanth.

@carlosrsabreu looks good to me. You can merge :)


artifactCleanup scheduled job defined in source code

The CRON scheduled job is defined in the source code, but would be better as configurable via properties file.
Further all policies are scheduled on a single CRON.
The pull request used on commit solves these two issues, making the plugin more easily configurable and not requiring the script to be changed.

It's been two months, I'm going to close this.

Please explain the cron settings used in the properties file.
It's not 'standard' linux cron layout!

It uses the Quartz specification, searching I found this useful: http://www.quartz-scheduler.org/documentation/quartz-2.x/tutorials/crontrigger.html


Check router before navigating in singleton collections

Fixes this issue found in discord
See video for explanation: https://www.loom.com/share/8a7329a9dfcb47429c8652648329c7dc

@kldavis4 Pushed up that fix.


Update docs structure Reference/GitHub

Purpose

GitHub is mostly used as part of our infrastructure, so we should move the content to a more appropriate location.
Changes

Move the GitHub page, both in the filesystem and in the website page structure. Remember to check for broken links that may result in moving this folder to a new location (mostly images in the GitHub folder and links to any workflow content).
Current Location: reference/github
New Location: reference/infrastructure/github
Updated Page Tree
nav:
  ...
- Reference:
    - reference/index.md
    ...
    - Infrastructure:
      - GitHub:
        - Development Workflow:
          - Overview:
            - reference/infrastructure/github/workflow/index.md
          - Creating Issues: reference/infrastructure/github/workflow/issues.md
          - Developing on Branches: reference/infrastructure/github/workflow/branches.md
          - Pull Requests: reference/infrastructure/github/workflow/pr.md
    ...

Let's keep the page structure flat


Doughnut chart section showing with no color

Expected behavior
Hi I am trying to create a vue component with some props so that when I import it and I pass the props, a doughnut chart is displayed with as many colors as the number of label in my array that i pass as a prop like so :

if my component is like so : <DoughnutChart labels=["bleue","red", ...] (someOtherProps) />
Current behavior
here is what I have :

As you can see, I have each section but no color on it. I also have the labels and I can click on it to make a section disapear
Reproducible sample
https://www.chartjs.org/dist/master/chart.umd.js
Optional extra steps/info to reproduce

<template>

My Budget

      <doughnut-chart
        :ValuesLabel="labels"
        :topLabel="title"
        :dataChart="dataChart"
      />

  </template>

<template>

      <canvas id="myChart" width="100" height="100"></canvas>

{{ topLabel }}

  </template>

Possible solution
I suppose I might not have imported all  the necessary stuff but I dont know what I missed
Context
No response
chart.js version
v4.3.0
Browser name and version
i am on the last version of Brave
Link to your project
No response

@Karljoy973 maybe I'm wrong but in your chart options you disabled the colors plugin:
  plugins: {
    colors: {
      enabled: false
    }
  }

Have you tried to enable it?

@Karljoy973 furthermore I think you need to register the colors plugin:
import { Chart, ArcElement, DoughnutController, Legend, Tooltip, Colors } from 'chart.js';
Chart.register(DoughnutController, ArcElement,Legend, Tooltip);

Chart.register(Colors);

I LOVE YOU !

Thank you @Karljoy973 maybe you can add the answer to SO ;) https://stackoverflow.com/questions/76352364/how-to-fix-chart-js-doughnut-chart-with-missing-background-colors-and-missing-la

you gave it


Use with React Native

Hello @kalabro @supersnager !
Can you please help me with how can I use use-chat with React native?

@iammeetpatel Please describe your problem

I tried to set up the same as the example here https://github.com/chatscope/use-chat#basic-usage in React Native and run the app it didn't work.

@iammeetpatel This example uses the web components @chatscope/chat-ui-kit-react (file Chat.js). If you want to use the use-chat library with React Native you need to use some RN components within the <ChatProvider/>.


WIP: Purge Managed HSMs after deletion

I'm unable to test this currently since Remove-AzKeyVaultManagedHsm lacks -InRemovedState, nor does the service currently support purging.

@heaths I assume this is outdated now based on your recent work.

No longer needed after Azure/azure-sdk-tools#1912


Hover Effect

Describe the Feature you want to end
I want to add Hover effect on the Header links.

Expected behavior
Whenever the cursor will be hovering over those links, it will make a hover effect & it will definitely improve the user experience.
Screenshots (optional)

Additional context (optional )

Hey, I want to contribute on this. Can you please assign me for this?

Please assign me this issue

assign me this issue please

I can see that the hover effect is not visible till now. If so, please assign me this.

I can see that the hover effect is not visible till now. If so, please assign me this.

It is visible, you can work for that in dark mode by opening new issue


[DOC-131] Remainder of pages from root ETKB space

This is the last batch of pages from the main Tech Docs ETKB space that need to be imported. A few more pages exist that do not need to be recreated here.

I missed the link warnings when running npm start and now the build fails due to broken links. I created this PR to address the problems - one was a file naming issue, the other was a path.
https://github.com/Ed-Fi-Alliance-OSS/ed-fi-alliance-oss.github.io/pull/73


Ruby Capstone Kanban project

Hello we are a group of two members:

Team member 1: @sarazaToussaint.
Team member 2: @ibr5500.

And here's the link to the project: link

Hi Team,
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
To Highlight 💯

Descriptive issue info✔
Tasks distributed equally✔

Optional suggestions
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

Thank you @gemmen29 for the review.


Support data attributes on MR custom elements

I was hoping data-attributes would be supported natively.
I wanted to store a default color so I can switch the color on hover and switch back to the default color on hover-end. It seems data attributes are not supported on <mr-….> elements.
Sample code:
<body>
  <mr-app camera="camera: orthographic">
    <mr-surface>
      <mr-container>
        <mr-row>
          <mr-panel margin="0 0.03" default-color="black"></mr-panel>
          <mr-panel margin="0 0.03" default-color="red"></mr-panel>
          <mr-panel margin="0 0.03" default-color="blue"></mr-panel>
        </mr-row>
      </mr-container>
    </mr-surface>
  </mr-app>
</body>

<script>
  let panels = document.querySelectorAll('mr-panel')

  for (const panel of panels) {
    panel.color = panel.dataset.defaultColor;

    panel.onHover = (event) => {
      switch (event.type) {
        case 'hover-start':
          panel.color = "yellow";
          break;

        case 'hover-end':
          panel.color = panel.dataset.defaultColor;
          break;

        default:
          break;
      }
    }
  }
</script>

</html>

@lobau I'm pretty sure you need to prepend the data attribute with data-
https://developer.mozilla.org/en-US/docs/Learn/HTML/Howto/Use_data_attributes

oh lol you're correct. Not the case for customelements, but it is for standard data attributes.


Let's go on an adventure.

In general, the move from just text to text plus related things seems like a good move - certainly titles, bossbar, etc.

:tada:

I think that Sound might be a step too far for inclusion in the main library - and could potentially be better suited in an add-on, kinda like how we have the serializers separated out of the main project.

The only issue there is that Audience would not contain these methods, which is the goal.

Not entirely sold on the introduction of ExaminableProperty - but I don't feel super strongly about it either way.
I just wonder whether the same data could be exposed using Map<String, Object> in the same, if not nicer way - without the need to pollute the library with another dependency.

Definitely not in a nicer way.


Blazor potentially no longer client-side technology?

Hi,
I watched today's ASP.NET Community Standup stream, and although the Electron demo was impressive, I'm somewhat surprised that the team is now seriously considering changing direction and running Blazor on the server. While there are some valid scenarios (and I think this should definitely be an option in the future), what got me (and others) so excited about this technology was the fact that it is purely client-side framework (webserver serving only static files).
To be honest, I was hoping that by this time (has it been a year since the first Blazor demo?) I would already be writing production apps in what I think is the best UI framework that Microsoft has been working on for quite some time. Wasn't this project supposed to be given a go/no go in May?
Thanks and keep up the great work! Very impressive what such a small team has been able to accomplish so far!

@daniel007 @danroth27 said in gitter channel

I think of server-side blazor as a stepping stone to the client-side solution
We can make progress on a production runtime and then move to the client as .NET on WebAssemlby matures

I watched the standup too and I do not like the server side direction/exploration.  It reminds me too much of web forms. I think developers need to know that they are writing client-side code. When the lines start to blur between what is client and what is server, then that is when the problems come.  Blazor needs to stay client side. We already have viable server-side frameworks with asp.net and asp.net core.
$0.02

Please keep it client side and make it better 😢

Awesome work! Really excited to use this in production.
To answer Steve’s question.
Client Side would work for our large scale business app. Mainly because of heavy UI components that would be too slow due to latency. Saving time on Web API would be good, but we need these regardless due to mobile apps and business API’s.

Blazor is and will continue to be a client-side framework. There are no plans to change that. However, Blazor's architecture is flexible such that it can be used in a variety of scenarios including out of process scenarios like Web Workers, Electron apps, and server-side apps. These scenarios are being investigated in addition to running Blazor client-side in the browser.

I was just watching the presentation and Steve missed out on a big feature of having Blazor running on the server and just rendering on the browser.
That is security. By having everything running on the server you are not exposing and of the business logic or code in the browser.
I use a product called Visual WebGUI and I am currently using Wisej and we had a security audit that we passed with flying colors in part to the fact that browser is dumb and is essentially rendering what the server told it to.
https://news.ycombinator.com/item?id=484514

For more info
https://wisej.com/docs/html/Security.htm

Server-side, we have that today already - ASP.Net Mvc. I think it will confuse developers if BLAZOR also does server-side stuff...
Let BLAZOR be the client-side option for us that are tired of the big messy javascript frameworks.

I'm not sure to what extent security concern is a valid argument. Client applications (whether hosted in the browser or desktop) should not keep any confidential information (besides what is needed for authentication/authorization), service layer (Web API) takes care of that.
I briefly scrolled through the Gitter feed and it appears that @danroth27 is trying to sell the server option as a higher priority project. Can you share the main reasons?
In my experience, server applications are more difficult (and costly) to maintain. Modern computers have plenty of processing power to distribute the workload evenly, if I remember one of the early Blazor videos starring @SteveSandersonMS , he mentioned something similar.
Blazor will be a tough sell to the management when there are other viable server solutions. Modern (web) development is clearly shifting towards exclusive client-side development (where I think it should be), if Blazor can run in the browser and maybe as a PWA, it is a winner.
Someone mentioned startup time, I guess that depends on how long it is. If on a typical setup, an application can load the bare minimum within one or two seconds, that would be perfectly fine in my case (mostly enterprise applications). Then the program can start loading other modules/areas/assemblies on demand as needed (I believe this is being addressed under Lazy Loading elsewhere).
Thanks for taking my feedback into considerations.

I briefly scrolled through the Gitter feed and it appears that @danroth27 is trying to sell the server option as a higher priority project. Can you share the main reasons?

That was certainly not my intention! Blazor as a client-side web UI framework is still our top priority. Pragmatically, Blazor on the server may be easier to deliver initially as it has no dependency on delivering .NET on WebAssembly. But the long-term goal is still to deliver Blazor on the client.

I do believe that the Server side option is currently more approachable since it's backed by .NET Core and not bound to the mono compilation to WASM which is currently 'holding us back' (perf. issues etc), let me be clear though the mono team is doing an amazing job! Maybe consider the server side option for now and later on when mono to wasm has more feature move to the client side?

Yes, I have to take back what I said about server-side. I was thinking about server-side in the "old way", you know do action on each web request and all that. But this is an entirely "new" way using server-side, no need to wait for each request. Do everything via websockets, this is wonderful. I think you are on to something here. Keep up with the good work Blazor people!!


request: more settings in 2.31 Hidden Games Manager

Only Remove Followed Games
Only Remove Games Tagged [select list - 8.3 game tags] [+ Tag]
        [+ Tag] adds another Only Remove Games Tagged [select list - 8.3 game tags] line
        personally only need one tag, but reckon others may prefer more
        suggest running multiple tags sequentially not concurrently
Thanks again for all your hard work 🥇

what?

main settings option 2.31 Hidden Games Manager
[eye + x] button on https://www.steamgifts.com/account/settings/giveaways/filters
popup "+ Add / remove hidden games:" currently has the following settings...
Add all owned games.
Add all ignored games.
Add all banned games (requires syncing delisted games in the settings menu).
Only remove games from text area.
Only remove owned games.
Only remove wishlisted games.
Only remove banned games (requires syncing delisted games in the settings menu).
what do you need me to clarify?
Followed games on Steam, similar but separate to Wishlist
Tags for dynamic selectlist from main settings option 8.3 Game Tags, edited via last button on top row of main settings [Manage Game Tags]

rotfl I just realised this is not the ESGST project, sorry about that mate, I clicked on the wrong link


[FEATURE] [Workflow] Customized workflow mapping could be converted to conductor workflow

a/c:
data plane could be able to able to convert user configured workflow into conductor executable workflow

There will be three stages for configuration:

validationStage (optional)
preparationStage (optional)
executionStage (required)

Workflow template:
{      "id": "workflow_01",      "validationStage": [],      "preparationStage": [],     "executionStage": [         {             "taskName": "delete_order_task",             "task_type": "http",             "endpoints": [                 {                     "path": "",                     "method": "",                     "serverKey": "",                     "mappers": {                         "request": [                             {                                 "source": "",                                 "sourceLocation": "BODY",                                 "target": "",                                 "targetLocation": ""                             }                         ]                     }                 }             ],             "notificationUrl": "http://slack.notice/api"         }     ] }
implement of delete eline order:
{      "id": "workflow_01",      "validationStage": [      	{             "taskName": "status_checking_task",             "task_type": "http",             "endpoints": [                 {                     "path": "/api/v2/company/consolecore-poping-company/connections/{id}",                     "method": "GET",                     "serverKey": "",                     "mappers": {                         "request": [                             {                                 "source": "",                                 "sourceLocation": "BODY",                                 "target": "",                                 "targetLocation": ""                             }                         ]                     }                 }             ],             "rejectCondition": "${status_checking_task.output.response.body.state == 'DELETED'}",             "skipCondition": ""         }      ],      "preparationStage": [         {             "taskName": "disable_order_task",             "task_type": "http",             "endpoints": [                 {                     "path": "/api/company/consolecore-poping-company/connections/{id}/disable",                     "method": "PUT",                     "serverKey": "",                     "mappers": {                         "request": [                             {                                 "source": "",                                 "sourceLocation": "BODY",                                 "target": "",                                 "targetLocation": ""                             }                         ]                     }                 }             ],             "rejectCondition": "",             "skipCondition": "${status_checking_task.output.response.body.state == 'DISABLED'}"         }     ],     "executionStage": [         {             "taskName": "delete_order_task",             "task_type": "http",             "endpoints": [                 {                     "path": "/api/v2/company/consolecore-poping-company/connections/{id}",                     "method": "",                     "serverKey": "",                     "mappers": {                         "request": [                             {                                 "source": "",                                 "sourceLocation": "BODY",                                 "target": "",                                 "targetLocation": ""                             }                         ]                     }                 }             ],             "notificationUrl": "http://slack.notice/api"         }     ] }

Conductor workflow schema can be generated from customized mapping schema:
`---
kind: kraken.component.api-workflow
apiVersion: v1
metadata:
key: mef.sonata.api-workflow.order.eline.delete
name: Geographic Address Management
version: 1
spec:
validationStage:
- taskName: status_checking_task
taskType: http
endpoint:
path: "/api/v2/company/consolecore-poping-company/connections/{id}"
method: GET
serverKey: ''
mappers:
request:
- source: ''
sourceLocation: BODY
target: ''
targetLocation: ''
conditionCheck:
caseExpression: "${status_checking_task.output.response.body.state}==DELETED"
buildInTask: reject_order_task
preparationStage:
- taskName: disable_order_task
taskType: http
endpoint:
path: "/api/company/consolecore-poping-company/connections/{id}/disable"
method: PUT
serverKey: ''
mappers:
request:
- source: ''
sourceLocation: BODY
target: ''
targetLocation: ''
executionStage:
- taskName: delete_order_task
taskType: http
endpoint:
path: "/api/v2/company/consolecore-poping-company/connections/{id}"
method: ''
serverKey: ''
mappers:
request:
- source: ''
sourceLocation: BODY
target: ''
targetLocation: ''
notificationUrl: http://slack.notice/api
`


Should VRG deal with PVCs from different storage providers?

A VRG CR, based on the PVC selector, can end up with PVCs from different StorageClasses and hence different storage providers.
An initial question is if such a selector is supported? and if not, we need to ensure to check PVCs and their StorageClasses for sameness and error out otherwise.
NOTE: Initial development would assume a single ReplicationClass and enforce sameness of the driver
If supported, it may be difficult to match a PVC to a ReplicationClass even just matching their StorageClass and ReplicationClass drivers, as a PVC may require a different ReplicationClass based on the RPO (or policy) objectives it may have.
An alternative would be to create different VRG per driver/policy, but use a common AVR to group these for application orchestration across clusters. Which in turn needs a manner of expression of such relations in the AVR CR.
Initial discussion link: https://github.com/RamenDR/ramen/pull/13#discussion_r595308821

With the schedule and VolRepClass selector, VRG is agnostic to storage provider and mix of drivers.


Add include_nil property option

Implements a :include_nil option for properties as mentioned in apotonick/roar#7.

Should we call that option include_nil or include_if_nil? Thanks, Bernd!

@cowboyd Do you mean a global switch to decide if nil properties should be included or not? Cause the problem here is how we define per property whether or not to include it in the doc if it's nil.

I like @cowboyd's version. I guess it would look like this.
property :beer, :represent_nil => true
represent_nil :as => false

Ok, one global configuration "how to convert nil values" and a :represent_nil option. How to set the conversion per property, then?
I have to refactor Definition to make it easier to add custom options - behavior like this can then be added as module and doesn't have to be hard-wired in the core. Cool?

@apotonick Yes, sounds good!


This browser or app may not be secure.

hi, developer. I try to use this python code, but when I login youtube, it notice me that "browser is not secure", how can I do?
my firefox version is 89.0 and had installed and configure the geckodriver

I have the same issue

You don't log in, just load the cookies, get them with a extension like editThisCookie
async function setCookies(channelName) {        let p = require("electron").remote.app.getAppPath();        p = p.substring(0, p.lastIndexOf("\\"));        let cookieJar = JSON.parse(          fs.readFileSync(            p + "/cookieJar/" + (channelName || "Só idéias legais!") + ".json",            "utf8"          )        );        cookieJar.forEach(async (cookie) => {          cookie.sameSite = "None";          cookie.storeId = "None";          cookie.secure = true;          await driver.manage().addCookie(cookie);        });      }

You don't log in, just load the cookies, get them with a extension like editThisCookie
async function setCookies(channelName) { let p = require("electron").remote.app.getAppPath(); p = p.substring(0, p.lastIndexOf("\\")); let cookieJar = JSON.parse( fs.readFileSync( p + "/cookieJar/" + (channelName || "Só idéias legais!") + ".json", "utf8" ) ); cookieJar.forEach(async (cookie) => { cookie.sameSite = "None"; cookie.storeId = "None"; cookie.secure = true; await driver.manage().addCookie(cookie); }); }

I use editthiscookie to get the same cookies for different channels

@pokima yes, even the same account as manager for multiple channels/brand accounts, you can get different cookies and use them

You don't log in, just load the cookies, get them with a extension like editThisCookie
async function setCookies(channelName) { let p = require("electron").remote.app.getAppPath(); p = p.substring(0, p.lastIndexOf("\\")); let cookieJar = JSON.parse( fs.readFileSync( p + "/cookieJar/" + (channelName || "Só idéias legais!") + ".json", "utf8" ) ); cookieJar.forEach(async (cookie) => { cookie.sameSite = "None"; cookie.storeId = "None"; cookie.secure = true; await driver.manage().addCookie(cookie); }); }

How do I get Login Cookies?

How do I get Login Cookies?
Did you try the extension? just click the icon > export > paste them

I don't quite understand.. So I run firefox I use that  cookie editor extension, go to youtube while logged in, and click the export button in cookie editor, to copy the cookies.
So now I have cookie data in my clipboard.
I could put that in a file c:\thispythonproject\cookie.json
But how do I get this program to load the cookies from that file?
A comment mentions "async function setCookies(channelName) {.........}"
Well, having done a bit of javascript before, I suppose that's javascript rather than python.
Where am  I meant to put that javascript line?
And I suppose straight after that line there should be a line that calls it?
I guess Selenium will run a browser or headless browser, and it's doing that so that's fine, but I suppose there must be a way to inject javascript into a page, and if injecting it into the right page then hey presto it could work.
But could anybody please explain how to do that?
(So I can go from having that cookie data, in a json file, to getting the cookies auto loaded into the browser that the program starts?)
Thanks


Bug in expanding cluster?

Hi, in GDBScan, the last line of the expand method says "if new point is a cluster"... but in the "if clause" the point is evaluated with the current neighbourhood... not with the new neighbours.
    neighbours.foldLeft(neighbours) {
      case (neighbourhood, neighbour @ Point(row)) =>
        // if not visited yet, create a new neighbourhood
        val newNeighbours = if (!(visited contains neighbour)) {
          visited add neighbour
          getNeighbours(neighbour, points.filterNot(_.row == neighbour.row))
        } else {
          Seq.empty
        }
        // Add to cluster if neighbour point isn't assigned to a  cluster yet
        if (!(clustered contains neighbour)) {
          cluster add neighbour
          clustered add neighbour
        }
        // if the neighbour point is a cluster, join the neighbourhood
        if (isCorePoint(neighbour, neighbourhood)) neighbourhood ++ newNeighbours else neighbourhood
    }

I think the last line should be:
 // if the neighbour point is a cluster, join the neighbourhood
   if (isCorePoint(neighbour, newNeighbours)) neighbourhood ++ newNeighbours else neighbourhood

If you agree I can do a PR.

@jasonbaldridge @gabeos please could you take a look into this?
Any doubt don't hesitate to ask please.

It's been a very since I wrote this. If you could provide a failing test for this to verify your assumption then accepting a PR would be a lot easier.

hey @carlos-verdes ! I know this is quite old, but... did you manage to verify that issue?
I'm running into another issue now and it might be related.
At some point, the append neighbourhood ++ newNeighbours throws
Java.lang.IllegalArgumentException
            at scala.collection.immutable.VectorPointer$class.gotoFreshPosWritable0(Vector.scala:1195)
            at scala.collection.immutable.Vector.gotoFreshPosWritable0(Vector.scala:62)
            at scala.collection.immutable.VectorPointer$class.gotoFreshPosWritable1(Vector.scala:1204)
......

Scala: 2.11
For what I could tell at looking at the Vector code it is running out of 'levels'
Any clues?? @muuki88
Thanks!

Thanks for reporting this @btafel :smile:  . Unfortunately I can't help in any regards as I haven't used this code for 7 years now. Also I mostly work on Scala 2.13 projects.

lol it makes sense! I know it is old - very old - but I have to say that it works :) at least in most cases. I was inspired by this article https://www.oreilly.com/content/clustering-geolocated-data-using-spark-and-dbscan/ and got it running smoothly.
I'm also trying another approach using UserDefinedAggregateFunction and Smile

Wow! I'm really happy that this is still in use and useful to others :heart_eyes:
Thanks for the hints to other libraries :-1:


Add docs

Use Sphnix for auto-documenting. Need some work specially in dependencies.
Also add some documentation as an example

I tried "make html" and got
Theme error:
no theme named 'alabaster' found (missing theme.conf?)

This is strange since the theme is a built-in theme.
http://sphinx-doc.org/theming.html
Anyway I changed it to default and tested it in another clone, worked okay.

Solid.  {{merged}}


[tests-only] Sometimes the API test can run for hours without timeout

Sometimes the API test step in CI can run for hours until the drone terminates it.
Example builds:
https://drone.owncloud.com/owncloud/ocis/29416/14/6
These might be the causes:

test steps don't timeout if the server doesn't respond

I've seen this problem
@aduffeck and @butonic has this been fixed yet? can we close ticket?


Flaky E2E test_upgrading test

https://github.com/software-mansion/protostar/actions/runs/3251227971/jobs/5335969088

Probably not relevant anymore


Autosave as an option

It would be nice if there was an option to autosave the current file before compelling when either compile (check) or compile (upload) is pressed.

I think it's duplicate of #31.


Change the load dialogue to make the load messages more visible

as it says

THese are Chuck's comments
Selected 270 LY csv file for loading/adding dataset.
Process was under way.
It indicated that 20000 of 20000 stars had loaded.
I clicked on it.
The table disappeared.
I looked for the dataset. It was not there.
I figured the load had failed and tried to add it again.
TRIPS told me it was still loading. However, there was no box or other processing advice/warning on the display to indicate this at all.
It finally loaded and plotted when I went away.
I think if a process is still underway, there needs to be something on the display which indicates that. Maybe blinking if it’s actually pausing/impeding other activity.

load dialogue is now more responsive.
I update the feedback more by dropping the report rate from 20,000 stars per update to 2,000 stars per update
I put the loading message in a larger bold red font.
I think its fixed and if needs be, we can reopen

The only way it was misleading is that it had loaded
"20000 of 20000 stars"
...which made no sense.
I would normally expect the second value to be actual total records to load, as in:
"20000 of 140000"
Sent from my Verizon, Samsung Galaxy smartphone
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: L Mitchell @.>
Sent: Saturday, January 15, 2022 5:45:39 PM
To: ljramones/trips @.>
Cc: Subscribed @.***>
Subject: Re: [ljramones/trips] Change the load dialogue to make the load messages more visible (Issue #342)
load dialogue is now more responsive.
I update the feedback more by dropping the report rate from 20,000 stars per update to 2,000 stars per update
I put the loading message in a larger bold red font.
I think its fixed and if needs be, we can reopen
—
Reply to this email directly, view it on GitHubhttps://github.com/ljramones/trips/issues/342#issuecomment-1013766101, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQPXLNRWMMESQBPCZBIRHETUWH2JHANCNFSM5MAIJRYQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

There is no way that I can apriori know how many records are going to be loaded.
Originally, I would have the first value be the number of records per update and the second number be the cumulative total. So 2000 of 2000 then 2000 of 4000.
 I can't show how far along it will be.

It actually will show you 20000 of 140000 if you wait long enough.
Early on, it loaded very quickly, but since we increased the complexity of each line and number of data items, its really slowed down.
On a less powerful computer, the time between updates is going to look like it's hung and not doing anything.
I changed the load factor form 20,000 to 2,000 to give a more responsive update message.

If I understand what you’re saying (and I might not be), then how does it help a user understand “how far along” the process is?
From: L Mitchell @.>
Sent: Saturday, January 15, 2022 6:03 PM
To: ljramones/trips @.>
Cc: cegannon1 @.>; Comment @.>
Subject: Re: [ljramones/trips] Change the load dialogue to make the load messages more visible (Issue #342)
There is no way that I can apriori know how many records are going to be loaded.
Originally, I would have the first value be the number of records per update and the second number be the cumulative total. So 2000 of 2000 then 2000 of 4000.
 I can't show how far along it will be.
—
Reply to this email directly, view it on GitHubhttps://github.com/ljramones/trips/issues/342#issuecomment-1013767980, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQPXLNRHKYIRDSJ5JLR65FLUWH4JFANCNFSM5MAIJRYQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you commented.Message ID<EMAIL_ADDRESS>


Apple Music: New PR

Description
I finally found a way to show music artwork while showing all results and on top of that, I added a menu bar and other awesome features. I am using the Last FM api, which just requires a free api key. You can find the artwork for most of the tracks and there are only some tracks that do not have artwork.
I am also using the Cache functionality with background refresh on the menu bar command. So it is like very fast. The background refresh only takes like 10s for me. So I think I could even increase the frequency to hourly instead of daily...
Check out the grid view for tracks, albums, and playlist :
https://user-images.githubusercontent.com/77864399/189293404-4b53ad06-2b6e-4085-8ef4-7292a057b85a.mov
There are specific preferences for customizing the layout of tracks within albums and playlists.
Checkout out the menu bar :
https://user-images.githubusercontent.com/77864399/189293430-a97d8882-bbca-4437-b9d5-34afbd896c5f.mov
After implementing these features with the menu bar, I removed the individual commands for them.
Screencast
Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

I took a very fast overview of the code and left some comments. I'll have a deeper look in the weekend

I took a very fast overview of the code and left some comments.

Where did you leave the comments?

I'll have a deeper look in the weekend

Sounds good, here's an api key if you want to try it out:
ec594775361acd41d9a2663da621de0c

@yug2005 perhaps remove that key from 'public' eyes?

Where did you leave the comments?

Inside the review

Inside the review

I do not see a review on the PR

Error handling is now broken. I have Cloud Library turned off, and  I tried Add To Library anyway. The error is logged, but I get the "Added to Library" success message.

@Rawnly How do I fix the ordering for the import statements? Is there a tool I need to install? Otherwise, it should be ready for another review...

Fine I'll try to review it later this week

@yug2005 can you check slack?

We're working on a PR refactor on https://github.com/yug2005/extensions/pull/3


TableOfContents links do not work when report is rendered via .show()

The links do not redirect. HTML instead of markdown could possibly help.

The bug only seems to occur in Visual Studio Code. Works in JupyterLab.


Remove unnecessary creation of BUILD dir for certain commands.

mbed CLI was creating the BUILD directory even in cases where it wasn't needed. This PR defers the creation of the BUILD dir until before compile steps only.
This should fix #413.

Thanks for the feedback @screamerbg, could you take a look again?

Thanks :)


Spilz87 patch 1 (IDFGH-10997)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 21 committers have signed the CLA.:white_check_mark: StevenMacias:x: dobairoland:x: IshaESP:x: rahult-github:x: wmy-espressif:x: BetterJincheng:x: jack0c:x: nachiketkukade:x: laukik-hase:x: hmalpani:x: jgujarathi:x: AdityaHPatwardhan:x: Dazza0:x: mahavirj:x: wang tao:x: MaxwellAlan:x: Espressif-liuuuu:x: ESP-Marius:x: ESP-YTGerd:x: spilz87:x: zhangyanjiaoespwang tao seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Seems this PR was created by mistake, thanks for contribution, feel free to reopen.

Actually it’s a real proposal but I don’t know how to do it, I never do it before.
The explanation of the issue and this modification are here  #12183
If you can help me to propose it properly, it could be very nice
thanks in advance


Fix linter findings

This PR is stacked on  top of #53
This PR has changes:

Move linter configuration to pyproject.toml out of Makefile. Also disabled the linter rule format in favor of black
Changes to address linter findings. They are split in 3 commits:

Miscellaneous linter finding fixes.
Disable the linter rule invalid-overridden-method in async variant. See the commit message for rationale and possible further development.
Changed logging to use the recommended lazy logging message formatting instead of f-string formatting. (note: f-string formatting could be allowed (by disabling a linter rule))

Also formatted files with make black.
Now both make lint and make black should produce clean results.

Merged. Thanks @sofkaski!
Thanks @githubDante for the review.


Allow operation modes for climate to not have a temperature

eg, dry may not have a temperature depending on your AC model.

Why do you have a bot that is marking pull requests as stale?

Please disable stale bot from closing PRs.


reduce batch size by 10x

We need to discuss this before letting it integrate into mobile.

Codecov Report
Base: 57.05% // Head: 57.05% // No change to project coverage :thumbsup:

Coverage data is based on head (108be6f) compared to base (d99dc65).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##              dev     #223   +/-   ##
=======================================
  Coverage   57.05%   57.05%
=======================================
  Files          35       35
  Lines        5074     5074
=======================================
  Hits         2895     2895
  Misses       2179     2179

Impacted Files
Coverage Δ

zingolib/src/lightclient.rs
62.60% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

How can we safely test this?

How can we safely test this?

Right now, we have no tests that involve large numbers of blocks... Generating blocks is computationally expensive, so I don't think we want to tell regtest to generate thousands of blocks during a test run. I think we want to generate a large blockchain (ideally several thousand blocks), with some sends back and forth, and save it as a test vector somewhere. That way, we can spend a long time generating it once, and then we can have tests that rescan it.

Great answer, but in the short term I was thinking of a manual test!


Added AttenuationTest to render-fidelity-tools tests

Added AttenuationTest model and its golden files to render-fidelity-tools tests

I just disabled the broken test runner. Please pull master and let's see if we can get these tests to pass.

@googlebot I signed it!

Here are the attenuationtest screenshots. How do they look?

Based on the Don McCurdy viewer https://gltf-viewer.donmccurdy.com/ this looks reasonable


Sometimes, the analysis for a function can exceed the display area

Describe the bug
Sometimes, when you view the analysis for a function in Graphing Mode, some items may be too long to show in the analysis window, and are unable to wrap because even a single expression exceeds the width. Therefore, the expression is directly chopped, let the user unable to read the entire expression.
Note that you're still able to copy the entire expression though.

Steps To Reproduce

Go to Graphing Mode
Enter 3x^3+5x^2-7x+1 (for example)
Click on 'Analyze Function'
See error

Expected behavior
The expression for the 3 x-axis interceptions, while long, should be shown completely,

Screenshots

Device and Application Information

OS Build: 10.0.19041.0
Architecture: X64
Application Version: 10.2007.1.0
Region: zh-CN
Dev Version Installed: False

Requested Assignment

I'm just reporting this problem.  I don't want to fix it.

This is your friendly Microsoft Issue Bot. I've seen this issue come in and have gone to tell a human about it.

Here's a workaround:
Adjust Calculator's width to let it show either graphs or equations only. A toggle will show up and then use it to switch to Equation Mode

In the Equation Mode, you can have chance to adjust the size of analysis results:


change int to float

Closes issue #353 - change inaccurate variable name

Hey there 👋 ,
Thanks for your help,
Indeed there's a mistake, but can you change it random_num instead? It's what we use for the explanation.

Change in ipynb
Also in hands-on.mdx

Thanks again 🤗

Thanks @simoninithomas - updated!

Awesome thanks, I merge it 🤗


Consider including old variant of U+0031

I believe I'm not the only one who prefers the old version of U+0031 (digit 1) with a less angled top stroke, which was replaced by the current steep angle in version 3.001.
I've therefore been using version 3.000 but I'd love to see alt-hack provide this config option so I can enjoy other updates of latest versions, while still keeping my digit 1 with a flat angled top stroke.
Old digit 1 (my preferred style)

New digit 1 after v3.001 (steeper angle: more legible, but less aesthetically pleasing IMHO)

Please see #49 instead.


Body.Header.search icon alignment doesn’t work

As reported via the forum by "crazy developer" (Ayoub EL Aboussi).
The forum thread also nicely contains a suggested fix from Ayoub.

Actually as noted the code on the forum does not quite work, but it pointed me in the right direction.


Allow heartbeat tool to not publish to collection if users want to only utilize the custom callbacks.

Some customers may want to customize the tool so that it doesn't publish the heartbeat data as a Ditto collection, but instead pipe that data via a custom callback to somewhere else.
This adds a new configuration flag publishToDittoCollection which defaults to true.
Closes #113

Is this saying that users can either publish to the heartbeat collection OR do something else with a custom callback? But not both?

Left a comment.
Is the callback implemented in another ticket/story?

The callback is already implemented as both a Combine publisher and a closure that are triggered if setup, but there is currently no way to opt out of also sending the data via a Ditto collection.


Add RT Bus Vehicle Position Schema

This adds capability to convert RT Bus Vehicle Position files to parquet.

adds RT Bus Vehicle config details config_rt_bus_vehicle.py
adds test file with operator PII information removed to tests/test_files
adds testing coverage for config_rt_bus_vehicle.py

LGTM.


IAP Promoted Subscription Flow?

Version of react-native-iap
4.5.4
Version of react-native:
0.62.0
Platforms you faced the error (IOS or Android or both?):
iOS
Expected behavior
Not sure
Actual behavior
Invalid product id
Tested environment (Emulator? Real Device?)
Real Device iPhone 6s iOS 13.6.1
This is not an issue but rather a request for help on understanding how the IAP promoted product/subscription flow works.
I am able to test successfully IAP auto-renewing subscription on a real device via a sandbox user.
I was hoping after a successful purchase I could allow users to enter a promo code that unlocks the app for a period of time.
I keep getting confused between a "promoted product" and a "promotional code".
I am currently trying to allow the user to "buy" the promoted product via the following code.
My response is always "Invalid product id".  I have created the product promo in app store connect, waited 48 hours, it is in "Ready for Submit" mode and I am using the correct product id but no luck.
`
if(Platform.OS === "ios") {
let signingKey = await getSigningKey(promoCode);
        console.log('signing key');
        console.log(signingKey);

        const offer = {
            identifier: promoCode.toString().toUpperCase(),
            keyIdentifier: signingKey.keyID,
            nonce: signingKey.nonce,
            signature: signingKey.signature,
            timestamp: signingKey.timestamp
        };

        const sku = 'tsr-social';
        const uid = '';

        try {
            const resp = await RNIap.buyPromotedProductIOS(sku, uid, offer);

            console.log('buy offer resp');
            console.log(resp);
        }
        catch(err) {
            console.log('buying promo offer err');
            console.log(err);
        }

    }

`
Thanks!

I think you are looking for requestPurchaseWithOfferIOS
The method buyPromotedProductIOS should be used for promoted products. These are the in-app products that are displayed directly in the App Store. You need to use a listener for that.

  I was using "." intellisense and that method wasn't appearing.
Then with your suggestion I put on my helmet and noticed I was able use it via "Import {}".
It's all working!!
Been struggling on this for a month or more!
Cheers!!


[provenance] Change "Bridge" terminology to "Transfer"

See discord thread
Overall, let's start using 'transfer' as the term to describe the movement of credits to/from registry accounts to our platform.

[ ] Request to change Bridge label on provenance pages to Transfer

A few changes were necessary here:

changed "Bridge" to "transfer"
changed "transfer" to "send"
added info icon w/ tooltip explaining that a Send is when an "Asset moved from one account to another, either via sale or inter-account transfer."
changed "view/hide XX transfers" to "view/hide XX transactions"


feat: make rest api return error details on edc exceptions and validation error by default

What this PR changes/adds
Return the "api error detail" response payload by default
Why it does that
The clients should be aware of the failure reason.
Further notes

At the end having the possibility to include the stacktrace as it was hinted in the issue didn't seem a good idea, clients should only be aware of errors resolvable on their side, and the stacktrace won't help them at all, neither in a development environment
Removed the "verbose" config settings as it has lost its value.

Linked Issue(s)
Closes #1492
Checklist

[x] added appropriate tests?
[x] performed checkstyle check locally?
[x] added/updated copyright headers?
[x] documented public classes/methods?
[x] added/updated relevant documentation?
[x] added relevant details to the changelog? (skip with label no-changelog)
[x] formatted title correctly? (take a look at the CONTRIBUTING and styleguide for details)

Codecov Report

Merging #1729 (d0de197) into main (56a8d08) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1729      +/-   ##
==========================================
- Coverage   67.39%   67.39%   -0.01%
==========================================
  Files         802      802
  Lines       17185    17182       -3
  Branches     1101     1100       -1
==========================================
- Hits        11582    11579       -3
  Misses       5124     5124
  Partials      479      479

Impacted Files
Coverage Δ

...onnector/extension/jersey/JerseyConfiguration.java
100.00% <ø> (ø)

...econnector/extension/jersey/JerseyRestService.java
94.28% <100.00%> (-0.31%)
:arrow_down:

...extension/jersey/mapper/EdcApiExceptionMapper.java
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Issue with zooming in iOS

I've only been able to try this on iOS9.1 and WP8.1 and the problem only seems to happen on the iOS9.1 phone (iphone 5) so I don't know how many platforms this will affect.
1.) Make sure there is a marker or polygon in view
2.) Start to pinch to zoom, but keep both fingers on the screen and rotate them around being sure to keep the marker/polygon in view
Eventually the marker/polygon will break loose from the map...the map will stop following your fingers, but the marker/polygon will continue to move with your fingers
The map will only update when you remove your fingers from the screen.

@bmeine Which leaflet version are you using? We've had similar issues (e.g. #3798) with pinch interactions, and they are version-specific.
Please make sure the bug is reproducible in either 0.7.7 (stable branch) or in the master branch.

Sorry, didn't mention that I actually upgraded to 0.7.7 to try to fix the problem before I posted the issue. I was also able to reproduce it by visiting Leaflet's sample pages using Safari on my iPhone.
It can also be reproduced by rapidly zooming in and out (using pinch gesture)...keep repeating and the issue will eventually show up.
The other thing is that if you wait ~10 seconds or so the issue will vanish and the map will behave as expected again...almost like some queue got backed up or something.

I observe the same effect with both a marker and a canvas layer using Leaflet 0.7.7.

Spent a lot debugging this issue and looks like it's some really weird iOS Safari bug related to CSS transforms. Switching markers to use top/left properties for positioning fixes the issue. I have absolutely no idea what exactly triggers it and why it's not reproducible on 1.0.0.
The workaround is to include this patch once before adding any markers:
L.Marker.prototype._setPos = function (pos) {
    L.DomUtil.setPosition(this._icon, pos, true);
    if (this._shadow) L.DomUtil.setPosition(this._shadow, pos, true);
    this._zIndex = pos.y + this.options.zIndexOffset;
    this._resetZIndex();
};

Closing the issue for now because:

This seems to be an obscure browser bug
It's not critical to map usability
There's a workaround above
It's not reproducible in 1.0.0-rc.2


Fix update-profile to exit properly on success

The subcommand was working correctly, but it always printed an error message (cannot update profile: %!w(<nil>)) and exited with a non-zero code. This patch fixes it.

Thank you


[🐞] Login Issues on PayPal SDK Version in Live Environment 0.112.0

@tdchow @ChrisGuzman @mahmoud-turki On PayPal SDK v0.112.0 on Live Environment unable to login when we try to login SDK launches with webpage ask to enter username and password but in older SDK v [0.8.8]. I see there is a option to login with OTP. But in latest SDk 0.112.0 I won’t see that option .so when I try to login with valid credentials on web page its stay on the same page shows  the spinner icon infinitely loads. This is not the case on Sandbox environment .

If you are still running into this issue please provide us with the following information.

Steps to reproduce the issue.

Thank you!


Support allow_other/allow_root Fuse mount options (#7900)

cherry-pick of Alluxio/alluxio#7900
cherry-pick commit is Alluxio/alluxio@4c22e33b8a61

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4619/
Test PASSed.

I think this PR is a subset of #9487 ? @gpang

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4643/
Test FAILed.

Jenkins, test this please.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4647/
Test PASSed.

alluxio-bot, merge this please.


Remove possibility to use other bindings than PatrolBinding during integration tests

This logic should be simplified:
https://github.com/leancodepl/patrol/blob/d89c3869e0e42d8207ac711922dac7a2f3f7a174/packages/patrol/lib/src/common.dart#L61-L75
In particular:

BindingType.integrationTest should be removed (#1368)
actually, the whole BindingType enum should be removed
New logic:

if (nativeAutomation) then initialize PatrolBinding
else, don't do anything (use the default binding that flutter test initializes)

I doubt specifying using the default integration_test binding works now, and we don't even have tests for them. Also, I see no benefit in allowing this.
See also:

#1368

See also #1273

This should be together with #1273, preferably in a single PR.


When press download open a new tab

When press download open a new tab on the main Jupyterlab window if this is not possible display the notebook in a new tab on the "MORE" window

Timeout


Add support for tracking checklist progress

Adds a new setting for a reference note to compare.
Marks any checked task as complete for the day.

Firstly, well done for understanding the quite complex code well enough to make any addition. It has evolved significantly over the years, and so the design may rightly feel a bit odd. Secondly, I hope this means my commenting was in pretty good shape.
Thirdly, this feels sufficiently different from the other ways that summaries work that it will definitely need a new subsection in the README before it's released. I think a before and after example would be good.

@jgclark updated the documentation, any other changes?

Well done again, @aaronpoweruser.
Good news:  I've asked Eduard to give you write permission for this repo, and he says he's now done so.
So please enjoy being able to merge and then release this yourself.
To release use npc plugin:release jgclark.Dashboard
and then add a comment in the Discord channel that's similar to the CHANGELOG entry.


Choose open WTR ports better

This took a while to figure out, but I think it's race-free now.

@rictic Safari is definitely flaky around importing modules. I don't think it's related to this PR since the effect we would see from using a bad port for Sauce is that the browser can't load the tests at all. We've seen the module flakiness before, usually on lit-element-styles_test.js though.
After replacing all calls to wtr with the new script, we just passed 3/4 of the Sauce jobs, including both IE jobs. I can rerun this a few times. It might already be an improvement.


Update identity.framework version

$subject
Related PR: https://github.com/wso2/carbon-identity-framework/pull/4266

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/3042672181

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/3042672181
Status: success


Summarize the change in the pull request title
Describe your change, specifically why you think it's needed.
Fixes #Issue_Number (if necessary)

@MichielOverweel : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Verified.
@MichielOverweel Thanks for submitting this fix.


Cut a new version?

Hi there,
Any plans to bump the minor version? I've been hit by the bug at PR #116. I can point my Gemfile to master in the meanwhile.
Thanks very much :)

Hi @bernardeli, I'm on it!

You can try v3.0.0.beta1


fix for issue #1138

Camel-K client install fails cluster setup for azure kubernetes service (aks) without the azure RBAC client support.
Error: No Auth Provider found for name "azure"`
By adding client-go import for azure pkg will allows AKS users to use camel-k client CLI to install camel operators to AKS cluster.
Fixes :#1138
Release Note
fix issue #1138

I guess goimports complains about the two imports not being in alphabetical order ;)


convert pages format to github markdown

This is an automated conversion from Pages to github flavored markdown.
Steps:

In pages, export as epub.
pandoc -f epub -t markdown_github -o osquery.md osquery.epub

This document can be cleaned up/edited/contributed to from here.

Thanks! Will drop notice of this in osquery Slack as well


lib: src: Bubble sort dark mode gif inverted

Bubble sort gif works perfectly fine in light mode but camouflages with the background in dark mode.
To Reproduce
Steps to reproduce the behavior:

Turn on dark mode
Bubble Sort bars camouflage with black background.

Expected behavior
The gif should be inverted in dark mode.
Screenshots

Device: Asus max pro m1
OS: Android
Screen Width: 440px

Additional context
I would like to work on this issue.

Ah, the thing is because sort implementation is yet not finalized yet, I haven't updated this sort gifs and they'll probably be remove once the are finalized. Check #63.


gyb on Windows

Closes #14280

@swift-ci please test

CC: @AndrewSB

Build failed
Swift Test OS X Platform
Git Sha - e0067aef02c1ecb44f4a0821ba0701b9e115173c

Build failed
Swift Test Linux Platform
Git Sha - e0067aef02c1ecb44f4a0821ba0701b9e115173c

Closes #14280

@compnerd this looks good, edit your first comment to include "Closes #14280"

@swift-ci please test

Build failed
Swift Test Linux Platform
Git Sha - e0067aef02c1ecb44f4a0821ba0701b9e115173c

Build failed
Swift Test OS X Platform
Git Sha - e0067aef02c1ecb44f4a0821ba0701b9e115173c


FIX - ios image rotation returning blank image (#9)

I can't take the credit :-) This fix is included in this issue and resolves the issue.  Can we merge it into the plugin proper?
https://github.com/mkloubert/nativescript-bitmap-factory/issues/9

+1 for a merge please.

+1


[ Discussion ] Go through the currency conversion flow chat and provide feedback.

i put it on dework


Improve errors from middleware

Errors from middleware have always been a bit lacking in information. To combat this, I have introduced a MiddlewareError which should wrap all the relevant bits of information you would want to allow you debug your middleware. This new error should be logged during a failed middleware execution during an in-flight request through Hoverfly. This error is also output via the API when setting middleware, resulting in in appearing in hoverctl.
Please find an example below of the new style errors.
Testing middleware against Hoverfly...
Could not set middleware, it may have failed the test

Middleware failed
Command: python /tmp/hoverfly/hoverfly_575209900
exit status 1

STDIN:
{"response":{"status":200,"body":"ok","encodedBody":false,"headers":{"test_header":["true"]}},"request":{"path":"/","method":"GET","destination":"www.test.com","scheme":"","query":"","body":"","headers":{"test_header":["true"]}}}

STDERR:
Traceback (most recent call last):
  File "/tmp/hoverfly/hoverfly_575209900", line 31, in <module>
    main()
  File "/tmp/hoverfly/hoverfly_575209900", line 23, in main
    t = payload_dict['data']['pairs'][0]['request']['headers']['Authorization'][0]
KeyError: 'data'

Thanks. Looks good to me. It should be a big assistant to users.


firefox on ipad not supported, documentation unclear

Today a teacher went in to pxt via firefox on a recent model ipad. We were then forwarded to a documentation page which stated that this browser was not supported. On that page it listed which browsers are support on which platform, but ipad (/ios) is not in that list. And under mac it shows firefox as supported, which is confusing as a user.
Please add "iOS (iPad & iPhone)" as a platform and list which browsers are supported on them.

Related issue: https://github.com/Microsoft/pxt/issues/342

This if fixed. Validated in the ipad


[WIP] Add flex_max

WIP, please don't merge.
This PR adds --flex_maxm a soft limit for the number of flexible residues; only the --flex_max closest residues to the ligand are retained.

From a first visual inspection it seems to work but more tests are needed.
One thing I realised is that a flexible residues can be selected because of backbone atom near the ligand; it would be more beneficial to consider only side-chain atoms in the search of flexible residues, since they are the ones that actually move? I don't know if there is an easy way to do this...

Assume standard pdb atom names to identify backbone?

A test case that makes sure the right residues are selected that that flexible docking actually changes the side chain conformations would be appreciated.

Sure! Can I add such tests to test_flexmin.py (to be renamed test_flex.py) or do you want a separate test?

Whatever makes the most sense to you.

The flexible docking test takes ~30s to complete with one flexible residue and ~90s with two flexible residues. Is the one with two flexible residues still acceptable?

Try setting --num_mc_steps to something low like 50.  That will do less sampling, speeding things up, while still testing multiple residues.

Great suggestion, thanks!

Tests are now in a decent shape (testing both docking with and without --flex_max, and CNN-minimisation). gninaflex takes ~60s to run (5x docking and 2x minimisation). I guess it's fine, compared to the ~700s for gninacheck?
The PR is almost ready to review. The only thing I have left to do is to understand why the NULL pointer in FlexInfo::extractFlex affects tests that have nothing to do with flexible residues.

Thank you for pointing out the problem; it was a dirty solution. I changed the check on file extension to a check on NumResidues(), which seems to solve the problem more generally. I left the if (a->GetResidue() != NULL) guard, but I added an exception in case the information is missing (in order not to silently ignore the problem).


chore: release master

:robot: I have created a release beep boop
allaround-components: 1.2.1
1.2.1 (2023-06-03)
Bug Fixes

add missing sass type (9ab4cfe)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/wholesome-ghoul/allaround-components/releases/tag/allaround-components-v1.2.1 :sunflower:


Using Ransack with Formtastic

Hi,
How can I use formtastic to render the search form? It doesn't work if I just swap out search_form_for with semantic_form_for.
Thanks,

Dan

That's actually one of the items on my todo before 1.0 -- I may have to go back to the alias_method_chaining form_for, as I did in MetaSearch, but I'd like to avoid it if possible. Just need to find a reasonable way to work with Formtastic and SimpleForm without doing so.

+1

Inspired by lucapette's approach to using SimpleForm, I came up with a semantic_search_form_for helper.  Use it as you'd normally use semantic_form_for, just passing in your search (@q) instead of a record.
def semantic_search_form_for(*args, &block)
  opts = args.extract_options!
  opts[:builder] = Formtastic::FormBuilder

  # add the default form class
  # (works whether existing class is a String like
  # "foo bar" or an Array like ["foo", "bar"])
  opts[:html] ||= {}
  opts[:html][:class] ||= []
  opts[:html][:class] << ' ' if opts[:html][:class].is_a? String
  opts[:html][:class] << Formtastic::Helpers::FormHelper.default_form_class

  search_form_for(*args, opts, &block)
end

Disclaimer:  I wrote this five minutes ago and it has neither been used in production nor significantly tested, but it does appear to be working perfectly in the one place I'm currently using it.

That looks like just the ticket. I'll give it a try, thanks Jonathan.


Is RK -> RCT prefix normalization still in use?

In the OSS repo I believe all the core libraries have been using the RCT prefix for a good while now. If there aren't any / too many RK prefixes internally then it could make for a good bootcamp task to remove the RK -> RCT remapping code.

We still use RK for all our internal modules. It's useful to have a different namespace to differentiate them from the OSS versions (although we could just manually export RKText as RCTText instead of having that be automated by the infra).


Replace siphash by hash/maphash

What does this PR change? What problem does it solve?
In Go 1.17.1, maphash has become quite a bit faster than siphash, so we can drop one third-party dependency. maphash is just an interface to the standard Go map's hash function, which we already trust for other use cases.
Benchmark results on linux/amd64, -benchtime=3s:
name                                             old time/op    new time/op    delta
IndexHasUnknown-8                                  50.6ns ±10%    41.0ns ±19%  -18.92%  (p=0.000 n=9+10)
IndexHasKnown-8                                    52.6ns ±12%    41.5ns ±12%  -21.13%  (p=0.000 n=9+10)
IndexMapHash-8                                     3.64µs ± 1%    2.00µs ± 0%  -45.09%  (p=0.000 n=10+9)
IndexAlloc-8                                        700ms ± 1%     601ms ± 6%  -14.18%  (p=0.000 n=8+10)
IndexAllocParallel-8                                205ms ± 5%     192ms ± 8%   -6.18%  (p=0.043 n=10+10)
MasterIndexAlloc-8                                  319ms ± 1%     279ms ± 5%  -12.58%  (p=0.000 n=10+10)
MasterIndexLookupSingleIndex-8                      156ns ± 8%     147ns ± 6%   -5.46%  (p=0.023 n=10+10)
MasterIndexLookupMultipleIndex-8                    150ns ± 7%     142ns ± 8%   -5.69%  (p=0.007 n=10+10)
MasterIndexLookupSingleIndexUnknown-8              74.4ns ± 6%    72.0ns ± 9%     ~     (p=0.175 n=10+9)
MasterIndexLookupMultipleIndexUnknown-8            67.4ns ± 9%    65.5ns ± 7%     ~     (p=0.340 n=9+9)
MasterIndexLookupParallel/known,indices=25-8        461ns ± 2%     445ns ± 2%   -3.49%  (p=0.000 n=10+10)
MasterIndexLookupParallel/unknown,indices=25-8      408ns ±11%     378ns ± 5%   -7.22%  (p=0.035 n=10+9)
MasterIndexLookupParallel/known,indices=50-8        479ns ± 1%     437ns ± 4%   -8.82%  (p=0.000 n=10+10)
MasterIndexLookupParallel/unknown,indices=50-8      406ns ± 8%     343ns ±15%  -15.44%  (p=0.001 n=10+10)
MasterIndexLookupParallel/known,indices=100-8       480ns ± 1%     455ns ± 5%   -5.15%  (p=0.000 n=8+10)
MasterIndexLookupParallel/unknown,indices=100-8     391ns ±18%     382ns ± 8%     ~     (p=0.315 n=10+10)
MasterIndexLookupBlobSize-8                        71.0ns ± 8%    57.2ns ±11%  -19.36%  (p=0.000 n=9+10)
PackerManager-8                                     254ms ± 1%     254ms ± 1%     ~     (p=0.285 n=15+15)

name                                             old speed      new speed      delta
IndexMapHash-8                                   1.12GB/s ± 1%  2.05GB/s ± 0%  +82.13%  (p=0.000 n=10+9)
PackerManager-8                                   208MB/s ± 1%   207MB/s ± 1%     ~     (p=0.281 n=15+15)

name                                             old alloc/op   new alloc/op   delta
IndexMapHash-8                                      0.00B          0.00B          ~     (all equal)
IndexAlloc-8                                        400MB ± 0%     400MB ± 0%     ~     (p=1.000 n=9+10)
IndexAllocParallel-8                                401MB ± 0%     401MB ± 0%   +0.00%  (p=0.000 n=10+10)
MasterIndexAlloc-8                                  258MB ± 0%     262MB ± 0%   +1.42%  (p=0.000 n=9+10)
PackerManager-8                                    73.1kB ± 0%    73.1kB ± 0%     ~     (p=0.382 n=13+13)

name                                             old allocs/op  new allocs/op  delta
IndexMapHash-8                                       0.00           0.00          ~     (all equal)
IndexAlloc-8                                         907k ± 0%      907k ± 0%   -0.00%  (p=0.000 n=10+10)
IndexAllocParallel-8                                 907k ± 0%      907k ± 0%   +0.00%  (p=0.009 n=10+10)
MasterIndexAlloc-8                                   327k ± 0%      317k ± 0%   -3.06%  (p=0.000 n=10+10)
PackerManager-8                                       744 ± 0%       744 ± 0%     ~     (all equal)

I've also benchmarked SipHash-1-3. That's slightly slower than maphash and its performance is less portable, since it only has assembler for amd64.
Was the change previously discussed in an issue or on the forum?
When SipHash was introduced, in #2812, @MichaelEischer already noticed that it was one of the bottlenecks in index lookup performance.
Checklist

[x] I have read the contribution guidelines.
[x] I have enabled maintainer edits.
[ ] I have added tests for all code changes.
[ ] I have added documentation for relevant changes (in the manual).
[ ] There's a new file in changelog/unreleased/ that describes the changes for our users (see template).
[x] I have run gofmt on the code in all commits.
[x] All commit messages are formatted in the same style as the other commits in the repo.
[x] I'm done! This pull request is ready for review.

Oh snap. I thought Go 1.14 was the minimum required version, but it's 1.13. That doesn't have maphash yet.

Bumping the minimum required go version to 1.14 should be fine. We'll probably soon see libraries requiring 1.14 anyways (as in #3407).

Added a Go version bump. I resisted the temptation to refactor internal/test to use the new testing.T.Cleanup function :)