Descriptions with embedded  */ lead to invalid typescript code

This is a great tool!
I was trying it out with google config connector. I found that for some files, the comments are not being generated correctly which leads to invalid typescript code.
Here's an example:
https://github.com/GoogleCloudPlatform/k8s-config-connector//blob/586304868023033c2ccfb029e379c09d05dc93b1/crds/cloudfunctions_v1beta1_cloudfunctionsfunction.yaml#L107
Leads to this:
/**Event types match pattern `providers/*/eventTypes/*.*`.
Event types match pattern
The pattern contains:*/

hi, thank you for the issue:
can you write a pull request fixing this issue ?


Question...

I can't test my mod with the "lime test windows" command in version 1.4.1 of the kade engine, how do I test and transform it into a real mod like whitty?

this isn't an issue. make a discussion https://github.com/KadeDev/Kade-Engine/discussions

@KadeDev close pls


_make_crp

Hello,
First，thanks for your contributions. But I have a question about the function of _make_crp in your depth_deocoder.
In the code, you use the sequence of four 1*1 convolutions, so I want to know why?
Thanks for your reply.

The architecture is inspired by a lightweight structure used in segmentation named "Light-weight RefineNet".
11 convs is used to save memory,  you can extend it to 33 convs will obtain better performance.


Fix issue 15864 - chmgen triggers exception in std.regex

This was marked as blocked for release, no idea why it was shipped anyway.
EDIT: This simply a bugfix for current master

Auto-merge toggled on

So this was introduced by https://github.com/dlang/phobos/pull/4103?

Yes


Fix saving of bool and null values to the database

The Database::val method did not work correctly with bool values. They were interpreted as strings "1" and "" which is not valid for database. It is now converted to integer instead of string.
The Database::changesetMapToList method did not work properly with null values as they were converted to an empty string when used as key in an associative array. It now uses the Database::val and Database::raw methods to avoid this problem.

The use of Database::val and Database::raw methods is still necessary for both bool and null values because without it the value would be converted to string ("" in case of null and "1" or "" in case of bool) which would be later interpreted as string in subsequent Database::val call.


[BUG] Scrollbar on Mobile shows twice

🐞 Bug report
Description
The Scrollbar Thumb is shown twice in mobile view. It can also be seen with chrom dev tools:

Reproduction
https://taiga-ui.dev/tui-scrollbar
Expected behavior
Only show normal mobile scroll thumb and not custom thumb
Versions

OS: iOS / Windows
Browser Safari / Chrome Device Emulator

Global scrollbar should not be visible on mobile devices. I don't see this bug on my Samsung phone. Will try an iPhone when I can soon.


Mediaelement 2.13.2: RTMP video issues

Hi,
We are using mediaelement player version 2.13.2 in our application to play the videos and audios. We observed some issues while playing videos with RTMP using flashmediaelement.swf. For short videos (say duration 5sec)
problem 1: The seek bar/progress bar is not showing the actual progress of the video. (For eg: progress is stopped at 3sec for 5sec video.)
problem 2: Once video is completely played, the start time is showing the end time instead of 00:00. (even for longer videos).
Regards
Krishnapriya

Unfortunately that's an old version and I'm not familiarized with. If you can provide the URLs for some of your RTMP sources I can take a look and see if there's anything I can do (for your side and for the latest version)

Hi,
Currently we don't have any video URLs that can be shared. I tried with the RTMP video that is present in http://www.mediaelementjs.com/. ie., rtmp://firehose.cul.columbia.edu:1935/vod/mp4:sample.mp4.
Even with this, I am facing the same problems.
And also observed other issue like after video is completely played, there is no play button and we need to click on pause button which will show the play button.
Thanks,
Krishnapriya

Have you tried this with the latest version of the player?

I tried playing video in http://www.mediaelementjs.com/ by selecting source as RTMP. There also I am facing the Problem 1 and the play button is not converted to pause button while playing video.

I checked the issue with the progress bar but I cannot reproduce it. I see the progress bar showing the correct time. Which browser did you use? I'll definitely check the issue with the play/pause button

This is what I see on Chrome and FF

OK I've fixed the issue of the play/pause (and some others that I found). So if you can test http://mediaelementjs.com and confirm please I'd appreciate it. Thanks for reporting this

Yes, the play/pause issue issue got fixed on latest. But the issue with the progress bar is still present.
The issue present in all browsers chrome/ FF. The progress bar stops at 9:52 even though the video is still playing and the end time is 9:56.

@krishnapriyaMandadi I had another round of fixes for the RTMP shim; it should be working fine now with the progress bar/current time elements. Let me know if it worked for you

Closing this ticket since no more responses have been posted on more than a week. If you need this to be reopened, let us know. Thanks


Include preface and appendix

The preface (To the Judicious and Impartial Reader) and appendix (An Appendix) are as much a part of the document as the chapters.

I agree, thanks for the PRs!


How I can host API v3?

Hi,
I've installed this repo on my IIS, but this is API V2.
How I can host V3?
Cheers,

Hi @grzegorz-ogrodowski ,
V3 protocol is based on client/server communication using json. On NuGet.org, this protocol is implemented using static blobs, which allows us to provide a scalable and reliable service.
The Gallery web service is not accessed at any point in the communication between client and server using V3 protocol, hence it's not enough to deploy the Gallery to support V3.
V3 protocol has not been official documented at this point in time, and doesn't provide any additional functionality on top of V2 protocol.
That said, if you are still interested in hosting V3, we have a pipeline of jobs that reads from the Gallery and creates a V3 index. The code is open source, and I can point you to it.


Missing content

Expected behavior
Enter in a mail and all the information contained in it show up properly (adjunct archive, image, etc)
Actual behavior
Enter in a mail and just the tittle appear a the rest is disappeared
Environment
K-9 Mail version: 5.708
Android version: 7.0
Account type (IMAP, POP3, WebDAV/Exchange): IMAP
Is weird because the contain of the mails appear on the previsualization lines
Oukitel c5 stock rom, almost aosp android nougat
The problem persist in all the mails

Sounds like an issue with the WebView component on your device. Do other email apps display the message content?

I didn't know that this worked with webview, so I updated android webview and it fix. Now there's no problem. But, It was working the Samsung internet webview before the problem, idk why it stop to working


Use the which package in dart_binaries.dart

https://pub.dartlang.org/packages/which

Looks good! It seems that will simplify the code a lot :)


example tools

It would be nice to have a suite of example tools to further show off the syntax and utility of hts-nim.
Please leave a comment here with a description.
The strengths of hts-nim are speed, so things that are too slow in python would be good candidates. It has very good syntax for VCF and BAM.

a VCF QC tool that takes a multi-sample VCF and finds big contiguous chunks where a sample has all missing genotypes could indicate that a processing chunk (e.g. from a single-sample GVCF) was lost. This could output a plotly plot showing the rate of missingness across each chrom for each sample.

VCF-VCF comparison, to determine concordance (given two orthogonal VCFs) or sens/spec (given a truth set).

A pileup engine! I'll admit that doesn't really count as a tool but would enable the implementation of many.

These are good ideas--though maybe more than bite-sized. :)
I'll be writing a pileup engine soonish.
I like the idea of VCF-to-VCF comparison, but there are a lot of considerations for that -- does it do normalization, does it do each sample? etc? I'll have to thnk more about a common use-case

Here's one that I ported over from a bcftools plugin I wrote a while ago. I'm sure there are a few things that I should re-optimize for nim.
tabulate.nim bins and tabulates integer data from the genotype column in a VCF, like DP, MQ, etc.
See also tabulate.c, the bcftools plugin.

A sambamba slice, pretty please? :D

@brainstorm what would it do that sambamba slice does not?
I guess there could be an API-ish way to access uncompressed blocks.

@brentp Sambamba is in a delicate spot when it comes to release engineering/CI/CD and overall reliability right now. slice is on 0.6.7, unreleasable for several platforms automatically in bioconda at the moment of writing this because it has several underlying issues (TL;DR: rabbit hole of fixes/updates):
https://twitter.com/braincode/status/1017216491265986560
I just wanted to have something functional/deployable to drop into our pipes ;)

I'll have a look. Not sure if htslib even exposes this--I think sambamba could do it because it has its own wrappers.

@brainstorm I was crawling through source for bcftools concat --naive when I remembered this thread.
bcftools/vcfconcat.c::naive_concat  checks each BGZF block header ( at least to get the block size) and copies each block w/out decompressing. EOF blocks for each file are omitted, a single EOF block is placed at close.
Bcftools' naive concatenation trusts that all input VCFs have compatible headers and writes out the first file's VCF header without checks. Naturally all input must be gzipped, but all input files must either be BCF or VCF.
@brentp I don't think that htslib provides an API for this beyond bgzf_raw_read, bgzf_raw_write, and other functions in htslib/bgzf.h and htslib/hfile.h, at least for the block IO. Do you thinks those two functions would be useful enough to be worth adding to hts_concat.h and bgzf.nim?
I'm not certain if htslib exposes block positions in the result from a hts_itr_query call, but that function calls a lot of helpers that may do most of the legwork for helping get a list of blocks for a queried region.

yes, if those are in the exposed api (in the header) files, then they can be added to hts_concat.h, if you or someone else wants to write an example or an hts-nim api on how to use the raw_read/write to get uncompressed blocks from a given region then it would be a great addtion to bgzf.nim.

Some other bigger ideas but I may try to tackle a few:

Replacement for picard MarkDuplicates. This is a huge bottleneck for our pipeline. I may try to tackle.
Bit size - window - generates regions from a sam / vcf for piping parallelizing in programs (e.g. parallelizing variant calling by chrom or region). I'm planning on adding this to seq-collection. Very useful with GNU-parallel.
Extracting reads that map to different contigs.
Extract read pairs that overlapr; Could you use this to estimate errors/artifacts?


We are uploading dataset to the cloud disk, I will reply you as soon as the upload is completed

can you give a url?

http://transcriptorium.eu/~htrcontest/MathsICDAR2021/MFD2021/
You can download the dataset from this URL, before that you need to register an account follow the chapter Registration and access to data in http://transcriptorium.eu/~htrcontest/MathsICDAR2021/
It is a private dataset provided by sponsor, so I have no right to upload it to the cloud.

the e-mail need these message ,but I don't  join in this competition，can you provide some fellow information for me,thanks.
Group name and acronym
Institution
Participants and e-mail
Contact person

@myh12138
Pls send email to<EMAIL_ADDRESS>to ask for data access.

the e-mail need these message ,but I don't join in this competition，can you provide some fellow information for me,thanks.
Group name and acronym
Institution
Participants and e-mail
Contact person

you can register now by sending email to<EMAIL_ADDRESS>The following information is your real information, for example,
Group name and acronym: Just a nickname
Institution: Your company or school
Participants and e-mail: Yourself and your email
Contact person: Yourself


Single strelka

Add new runSingleStrelka process.
cf Strelka user Guide

Hej, I guess the actual containers in your  docker repo are missing.
Szilva

I'm just missing VEP and snpEff containers that are uploading right now ;-)


NDK

Description
Migrate nostr code to NDK.
I've kept it OOP like the original code.

[x] nwc sender
[x] nwc receiver
[x] crosspost
[x] zap note

closes https://github.com/stackernews/stacker.news/issues/1549
Screenshots
Additional Context
Was anything unclear during your work on this PR? Anything we should definitely take a closer look at?
Checklist
Are your changes backwards compatible? Please answer below:
yes
On a scale of 1-10 how well and how have you QA'd this change and any features it might affect? Please answer below:
zap notes are not tested by they should work™
For frontend changes: Tested on mobile? Please answer below:
n/a
Did you introduce any new environment variables? If so, call them out explicitly here:
no

Logging for nwc is a bit tricky because NDK abstracts away everything and uses  npm:debug internally. I've got it to work by adding a custom debug logger that proxies to the wallet logger, but i am not sure if the verbosity is too high.

Very strange...
It works on firefoxdev and chrome  🤔
https://github.com/user-attachments/assets/64163cd3-3b93-49d5-806b-9d4fa6c256f1
and brave 173.89
but not on the regular firefox(???)

But i remember this being an issue in the past., maybe it is an update they are rolling out or a security rule some setups don't have. I just saw it working and trusted the library to do the right thing.
Luckily NDK exposes the nwc class, so , to avoid surprises, i've reapplied the alby workaround modified for multiple relays.

I've enabled debug and there is this exception being thrown by the worker
2024-12-01T13:39:19.448Z ndk:pool:main Failed to connect to relay wss://relay.primal.net/ ReferenceError: WebSocket is not defined

turns out ndk expects WebSocket to be available in nodejs, the doc says to use a polyfill, but we don't have it on the worker.
So i've added this to worker/index.js
import ws from 'isomorphic-ws'
if (typeof WebSocket === 'undefined') {
  global.WebSocket = ws
}

In case someone is wondering: I am keeping this branch up-to-date with master because I am building my code for #1558 on this.


[AfreecaTV] Ability to switch between servers with options

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm requesting a site-specific feature
[X] I've verified that I'm running yt-dlp version 2023.03.04 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
Most parts of the world
Example URLs
https://play.afreecatv.com/m0m099/live
Provide a description that is worded well enough to be understood
AfreecaTV has more than one server/CDN and I would like to have the ability to use options in the command line to switch between them.
To what I know, yt-dlp uses "gcp_cdn" and streamlink uses "gs_cdn_pc_web".
The reason why I want this feature is that gcp is giving me some issues where it would be very unstable at a certain day/time and streamlink doesn't have the --no-hls-use-mpegts command that I use with yt-dlp.
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
[debug] Command-line config: ['-vU', 'https://play.afreecatv.com/m0m099/live']
[debug] Encodings: locale cp950, fs utf-8, pref cp950, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>[392389b7d] (win_exe)
[debug] Python 3.8.10 (CPython AMD64 64bit) - Windows-10-10.0.22621-SP0 (OpenSSL 1.1.1k  25 Mar 2021)
[debug] exe versions: ffmpeg N-107944-g4fce3bab64-20220830 (setts), ffprobe N-107944-g4fce3bab64-20220830
[debug] Optional libraries: Cryptodome-3.17, brotli-1.0.9, certifi-2022.12.07, mutagen-1.46.0, sqlite3-2.6.0, websockets-10.4
[debug] Proxy map: {}
[debug] Loaded 1786 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Available version<EMAIL_ADDRESS>Current version<EMAIL_ADDRESS>Current Build Hash: 5590c57bd0433ed239a2deaaf92e2ad6f37fe50f53664c821575cafe106a9421
yt-dlp is up to date<EMAIL_ADDRESS>[afreecatv:live] Extracting URL: https://play.afreecatv.com/m0m099/live
[afreecatv:live] m0m099: Downloading JSON metadata
[afreecatv:live] 245401012: Downloading access token for sd stream
[afreecatv:live] 245401012: Downloading metadata for sd stream
[afreecatv:live] 245401012: Downloading access token for hd stream
[afreecatv:live] 245401012: Downloading metadata for hd stream
[afreecatv:live] 245401012: Downloading access token for hd2k stream
[afreecatv:live] 245401012: Downloading metadata for hd2k stream
[afreecatv:live] 245401012: Downloading access token for original stream
[afreecatv:live] 245401012: Downloading metadata for original stream
[afreecatv:live] 245401012: Downloading channel metadata
[debug] Formats sorted by: hasvid, ie_pref, lang, quality, res, fps, hdr:12(7), vcodec:vp9.2(10), channels, acodec, filesize, fs_approx, tbr, vbr, abr, asr, proto, vext, aext, hasaud, source, id
[debug] Default format spec: best/bestvideo+bestaudio
[info] 245401012: Downloading 1 format(s): original
[debug] Invoking ffmpeg downloader on "https://live-global-cdn-v02.afreecatv.com/live-stm-01/auth_playlist.m3u8?aid=.A32.7bbT56vyHM9fKZk.Jd0P35RgmksLW32apUsJiTT30jbEaCSkVevKEMukEgl4XtdqRQ2jltIS4uxOnlUy-k3UtdZAj3AnSYmvOp8VNzG9hTskKMuLvgSww_QBvWE"
[download] Destination: 항공캠 궁금하지않앙？..ㅎ♥ 2023-03-09 20_08 [245401012].mp4
[debug] ffmpeg command line: ffmpeg -y -loglevel verbose -headers "User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/95.0.4638.17 Safari/537.36
Accept: text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8
Accept-Language: en-us,en;q=0.5
Sec-Fetch-Mode: navigate

As of 9073ae6458f4c6a832aa832c67174c61852869be, yt-dlp is using gs_cdn_pc_app
This could easily be added as an extractor-arg though


virtio-win binary drivers are only legal for use on RHEL hosts?

Describe the bug
I stopped to read some of the license agreements in the virtio-win drivers before installing them, and discovered the following rather surprising terms in the installer, specified as applying to the virtio-win drivers themselves:
2. Program License Grant.

Red Hat grants User a non-exclusive, non-transferable license to use the Program during the term of and on systems with fully-paid applicable Red Hat subscription(s) provided User may not:
...

I don't have any problems with the rest of the license (including the terms against modification of the software), but the bit about "fully-paid applicable Red Had subscription(s)" is throwing me off. These drivers are available for public download for free. Proxmox and Fedora both recommend using them to enhance Windows guests. I personally would like to use them to enhance a Windows guest running on an Ubuntu device. I don't own any RHEL subscriptions, and don't use RHEL at all.
Does this mean that I don't have a license to use the virtio-win drivers? If not (i.e., I can use these), it would be ideal if this could be made more clear. If so (i.e., I can't use these), then some explicit notice along the lines of "You may not use these drivers on a guest running on a host that is not licensed with an applicable Red Had subscription" would be helpful, ideally if it were on the first page of the installer.
Screenshots

Host:

Disto: Kubuntu 22.04 LTS

VM:

Windows version: Windows 11 Enterprise Evaluation

More confusing, the driver ISO file ships a file titled "virtio-win_license" which appears significantly newer due to the copyright dates, and just contains a generic BSD-3-Clause license. Which license is the right one?

@ArrayBolt3
Thank you for reporting this issue.
There is definitely a problem with the license agreement in the public virtio-win package. This agreement is  just a copy of the license agreement which comes with internal package, available for the RH customers.
We will try to fix this issue in the next public release. Meanwhile nothing stops you from using the public virtio-win package on non-RH products.
All the best,
Vadim.


[Issue #264] feat: dont sync prices with transactions

Description
Removes the syncing of prices together with transactions. The reasoning for this is that the syncing was causing significant overload when syncing the transactions, and it was unnecessary, since our code is already supposed to have all prices synced upon build.
Test plan
Rebuild the codebase, it should function just as before (tho now a bit faster)
Remarks

Now we only sync the codebase transactions after ALL prices have been syncing,
Left console.time on the code to time how long is it taking to upsert the transacitions into the DB. I've noticed it takes significantly longer in the BullMQ jobs then when using the API. When we get reasonable upsert prices, I'll take it off.

Depends on

[x] #471

Checks failing.


[Feature Request] Vertical select

E.g. alt/option - command + up or down arrow to vertically insert cursors across multiple lines. Apologies if I overlooked something and this is already possible!

Can do 🙂 follow along here: https://github.com/timhor/obsidian-editor-shortcuts/projects/1#card-86401295

Was just looking for this. Subscribed. Yay!

This is now available in v1.11.0 😀


Fix column split race condition.

Close https://github.com/dmlc/xgboost/issues/10314 .

cc @rongou .


Providing a minor change to where the from_http and to_binary methods/functions come from.

Without this you will see messages like the following:
deprecation.DeprecatedWarning: to_binary is deprecated as of 1.6.0. Use cloudevents.conversion.to_binary function instead
caught to_binary is deprecated as of 1.6.0. Use cloudevents.conversion.to_binary function instead
Function raised to_binary is deprecated as of 1.6.0. Use cloudevents.conversion.to_binary function instead

@sferich888 tests are failing?

@matejvasek yea I'm investing why, I haven't quite figured out out what is going on on. This shouldn't have broken anything but that's what I get for making assumptions.

/approve
/lgtm


What about docs?

Hey there, I wonder what we should do for documentation on this project. Any thoughts @esccoding ?

So, what you're saying is that we should

[ ] Create a docs folder
[ ] Create a new index.md document
[ ] Turn on GitHub Pages
[ ] Decide if we want to go with something more complex


Plugin doesn't work

Hi,
I'm trying to figure out what is happening, my configuration looks well, I tested with the commands on the readme, but it doesn't run or generate the certificate. I don't know why.
And I'm using the plugin in database mode.
{
    "api_uri": "https://acme-v02.api.letsencrypt.org",
    "domains": [
        "subdomain.domain.site"
    ],
    "storage": "shm",
    "cert_type": "rsa",
    "tos_accepted": true,
    "account_email"<EMAIL_ADDRESS>    "storage_config": {
        "shm": {
            "shm_name": "kong"
        },
        "kong": {},
        "redis": {
            "auth": null,
            "host": null,
            "port": null,
            "database": null
        },
        "vault": {
            "host": null,
            "port": null,
            "https": true,
            "token": null,
            "kv_path": null,
            "timeout": null
        },
        "consul": {
            "host": null,
            "port": null,
            "https": true,
            "token": null,
            "kv_path": null,
            "timeout": null
        }
    },
    "renew_threshold_days": 14
}

I just forgot to run the generate command.
curl https://mydomain.com -k

curl https://mydomain.com -k
Created successfully but still can't work


Configuring IPAM Pools for IPv4 and IPv6 in the Same Module

We are currently facing an issue while trying to configure IPAM (IP Address Management) pools for both IPv4 and IPv6 within a single module. We have encountered errors that suggest mixing IPv4 and IPv6 scopes may not be supported. To help us resolve this issue, we kindly request your assistance in providing a basic example demonstrating how to successfully configure IPAM pools for both IPv4 and IPv6 within the same IPAM module.
Our objective is to create IPAM pools for both IPv4 and IPv6 addresses within a single IPAM module because, as per our understanding, AWS allows only one IPAM per Region to be used.
To clarify our request further, we are looking for a basic module usage example that illustrates how to configure the IPAM module to provide two separate scopes for IPv4 and IPv6 addresses.
Currently in examples we can see how to use IPv4 or IPv6. Maybe you could update the multiple scopes example to add IPv6 as well?
Thank you for your assistance.

Hi, thanks for opening this issue. IPAM uses several concepts that you can review here
The way this module is designed you would have 2 invocations. 1st invocation creates the ipam, default scopes, then pools for either ipv4 or 6. the 2nd invocation would not create ipam but would reference the scope from the 1st instantiation. You'd then create pool structures for the other type (ipv6 if your 1st was ipv4, etc). You can see an example of referencing an ipam created in another instantiation here: https://github.com/aws-ia/terraform-aws-ipam/blob/main/examples/multiple_scopes/main.tf#L58-L59
Hope this helps!

Hi @robodorm did this answer your question?

Hello, I'm a colleague of @robodorm and just wanted to let you know that we've abandoned the idea of using IPAM v6 pools because we realized that https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/vpc_ipam_pool_cidr_allocation only supports IPv4.

I gave it another try but it looks like this TF module supports only BYOIP IPv6 pools: top_cidr is a required argument, but if I provide it as an empty list, I get the following error:

ipv6 can only be specified at the top level iirc

Yes, we specified the address family:
module "this6" {
  source  = "registry.terraform.io/aws-ia/ipam/aws"
  version = "~> 2.0.0"

  top_netmask_length = 52

  ipam_scope_id   = module.this.ipam_info.public_default_scope_id
  ipam_scope_type = "public"

  create_ipam = false

  top_name       = "global ipam 6"
  address_family = "ipv6"

  pool_configurations = {
    corporate-eu6 = {
      locale = "eu-central-1"
      aws_service = "ec2"

      sub_pools = {
        sandbox = {
          name                 = "sandbox-dev-6"
          netmask_length       = 56
          ram_share_principals = [local.ou_sandbox]
          locale               = data.aws_region.current.name

          sub_pools = {
            eks-sandbox-6 = {
              netmask_length       = 64
              ram_share_principals = [local.ou_sandbox]
              locale               = data.aws_region.current.name
            }
          }
        }
      }
    }
  }
}

only BYOIP IPv6 pools

failed to read this. yes initially this was written for byoip... i would have to think about how to use aws provided... might not be too hard.
can you please open a feature request and include your HCL example

Many thanks for support!


Microsoft.HanaOnAzure: Remove readOnly attribute from customer-provided properties in CreateHanaInstance

After completing https://github.com/Azure/azure-rest-api-specs/pull/5945, I discovered I could not update our CLI extension to call the Create HanaInstance API as expected as some properties were marked as read-only in the Swagger. This PR resolves this.
Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[X] I have reviewed the documentation for the workflow.
[X] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-python

Automation for azure-sdk-for-js

Automation for azure-sdk-for-java

Automation for azure-sdk-for-go

Can one of the admins verify this patch?

@sergey-shandar Why are so many of the automatic checks failing? After clicking through and looking at the logs, it seems to be an issue with Git pulling from the remote. What can I do to resolve this?

@lagalbra is the a problem in the OpenAPI spec that it doesn't reflect service API?

What are you referring to as the "OpenAPI spec" and the "service API"?
All the PR checks seem to be passing now.


Such grid

What does this change?
Adds the first pass at a grid system. Column numbers, column and gutter widths based on our design guide
So far functionality
Allows you to define a grid layout like this:
<Grid>
  <Row>
    <Col wide={4} leftCol={4} desktop={4} tablet={3}>
      <Component.a/>
    </Col>
    <Col wide={12} leftCol={10} desktop={8} tablet={9}>
      <Component.b/>
    </Col>
  </Row>
</Grid>

Only supports the following breakpoints:

wide
leftCol
desktop
tablet

If any of these breakpoints is omitted, it is assumed the contents will stretch to 100% of the width of the container at that breakpoint.
Mobile breakpoints and tweaks are intended to be fluid and hence not supported here.
Todo

[ ] Support starting component at a specific column
[ ] For cols that start at a specific column, support an auto width
[ ] Prevent styling of Grid helper components
[ ] Enforce acceptable number of columns at compile time
[x] Types
[x] Clearfix <Row> components

Why?
For consistent design

I think  might merge this as-is and capture the TODOs on the Trello board.
@GHaberis @sndrs sounds good?

why not!

Thanks. I might back out the Flow types because I'm not confident in them. I'll add them again when we get Flow working properly


Added support for QuickLook target folders

Just a simple addition to the option. For my needs (QLSVGSprite), I'd rather install for each user, giving instructions to create the ~/Library/QuickLook folder if needed.
But, from what I gather from other discussions, maybe the common QuickLook install location, /Library/QuickLook, is the better option for common usage.

maybe the common QuickLook install location, /Library/QuickLook, is the better option for common usage

I'm with that, just because ~/Library/QuickLook does not exist by default on any Mac OS X or macOS installation. For most users the global folder will be better, if we are looking for installation easiness.

Thanks @nunopedrosa!


Is there any way to upload file from memory?

I haven't found. It would be great to not create temporary files for data from memory.

No, there's not.
Shouldn't be hard to implement new file type based on io.Reader or bytes slice (I'd go for first one), and maybe refactoring sendFiles a bit so it accepts io.Reader slice for files (or even better -- interface{} slice, and add handling for io.Reader conforming types and for string).
I'm sure PRs are welcome, would you be able to work on this?

Hi, unfortunately, I don't have much time now

@ilyaglow I want to debug thumbnails before submitting PR

Oh, is it possible to split io.Reader type support and thumbnails feature in two separated PRs?

If someone wants to play with io.Reader as a file attachment - I've cherry-picked a commit https://github.com/stek29/telebot/commit/56519659734f4afe5183eb6a8cb8db0e13677c67 to my fork, so you can use the following import: gopkg.in/ilyaglow/telebot.v3.

@ilyaglow of course it is, and that's what I am going to do, I just wanted to make sure it works, and I wanted to test that with thumbnails.
I'll submit a PR once I'm sure that it works fine, and doesn't leave unclosed files.


Popover max-width

Hi, great project. Very easy to use and effective!
I'm trying to make some step popovers wider using CSS but can't get it to work.
It seems that Bootstrap defines max-width for popovers:
.popover {
  ...
  max-width: 276px;
  ...
}

I've added a class to my own custom stylesheet (included after all other stylesheets):
.popover-double-width {
  max-width: 552px !important;
}

I am then setting the HTML template option for the step to be:
"

    <button class='btn btn-default' data-role='prev'>« Prev</button>
    |
    <button class='btn btn-default' data-role='next'>Next »</button>
    <button class='btn btn-default' data-role='end'>End tour</button>

"

I only changed to the top line of the default string to include my double width class.
However when I start the tour and go the step that should be wider, it's the same width as the others and when I inspect the popover element in Chrome I do not see a reference to the double width class anywhere.
Am I doing something wrong? How should I make a popover wider?
Thanks in advance, Adrian.

Do you want it to be max-width: 552px, or do you actually want an exact width of 552px? In my case, I knew I wanted a popover with a width of 300px, so I did:
{
    ...
    template: "
\

\
        ...\

",
    ...
}

.popover.tour.project-tour {
  max-width: none;
  width: 300px;
}

Using the slightly more specific CSS target of .popover.tour.project-tour, rather than just my custom class, gave me more specificity than that bootstrap CSS so I didn't need to use important.
That being said, even if I replicate your conditions in mine I'm able to get it to work (project-tour{ max-width: 552px !important;} gives me a popover that's 552px wide).

Thanks @apennell !
I must've had a typo somewhere as I seem to have got it working now...
Here is a JSFiddle just in case anyone in future is interested :)


Split UART into UART0 + UART1

I've dusted off my bl602 and am planning to do a project with it.
The main change in this PR is making both UART peripherals available,
but it also updates the dependencies and rebuilds the code with the latest svd2rust.
This will also force some updates to the HAL. I'll create a PR if this gets approved.

Hi! Usually it's best to split these things into separate PRs, so that issues with one PR don't impact the other.
Would you mind updating CI to use a newer version of the compiler as required by the newer svd2rust?

MSRV on riscv/riscv-rt is Rust 1.59, this seems like a good version to bump our MSRV to.

Alright, will do. I'll create a new PR to:

update CI
update dependencies
And then I'll change this PR to only include the UART change.

I'll rebase this once the other PR is merged + do add a commit that regenerates the rust code to reflect the UART updates.

This should be good to merge now.


Update README with loki> examples

I was able to start the lokijs-server and run the lokijs-client. But, I am not sure how to use the prompt interface.
I tried:
loki> OP_INSERT "name" "foo"

and the lokijs-server crashed with:
{"collection":"\"name\"","obj":"foo"}
Result is undefined
net.js:625
    throw new TypeError('invalid data');
    ^

TypeError: invalid data
    at Socket.write (net.js:625:11)
    at Socket.<anonymous> (/tmp/lokijs-server/server/server.js:40:12)
    at emitOne (events.js:77:13)
    at Socket.emit (events.js:169:7)
    at readableAddChunk (_stream_readable.js:153:18)
    at Socket.Readable.push (_stream_readable.js:111:10)
    at TCP.onread (net.js:531:20)

If you can update the README.md with few examples, it will helpful. Thanks!

it's been ages since i was working on this but i think the last parameter should be a JSON object (instead of "foo"), e.g. loki> OP_INSERT "name" {"foo":"bar"}

That exits the server with a different error ("Result is undefined"):
$ node index -p 47134 -f /tmp/lokijs.log
{ port: '47134', filename: '/tmp/lokijs.log' }
ENV NODEJS
Listening to 47134
{"collection":"\"name\"","obj":{"foo":"bar"}}
Result is undefined
net.js:625
    throw new TypeError('invalid data');
    ^

TypeError: invalid data
    at Socket.write (net.js:625:11)
    at Socket.<anonymous> (/tmp/lokijs-server/server/server.js:40:12)
    at emitOne (events.js:77:13)
    at Socket.emit (events.js:169:7)
    at readableAddChunk (_stream_readable.js:153:18)
    at Socket.Readable.push (_stream_readable.js:111:10)
    at TCP.onread (net.js:531:20)

Any other suggestions on this?

@shakthimaan I will try and look at this today - will update later.

Thanks! I can help in testing the same.

@techfort Have you had time to debug this?


Switches for Regional Settings

Is there any plan to add additional switches to the New-PnPSite and/or Set-PnPSite commands? I could really use an option for setting the region. I realize there are site templates and PnP provisioning, but I don't have the time to invest in that at the moment.
https://COMPANY.sharepoint.com/sites/SITENAME/_layouts/15/regionalsetng.aspx

@robertcaretta Thank you for this feedback
I want to make sure this question gets addressed properly.
We have a separate repo for the PnP PowerShell issues. We want your issue to get the proper attention it deserves.
Please open your issue in this repo: https://github.com/SharePoint/PnP-PowerShell
I am going to close this issue so that we can track to correct one in the PnP PowerShell repo.


Update scalafmt-core to 3.5.8

Updates org.scalameta:scalafmt-core from 3.5.3 to 3.5.8.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta", artifactId = "scalafmt-core" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "org.scalameta", artifactId = "scalafmt-core" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Codecov Report

Merging #1952 (d486632) into unstable (956c4d8) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##           unstable    #1952      +/-   ##
============================================
- Coverage     77.09%   77.08%   -0.01%
============================================
  Files           415      415
  Lines         12676    12676
  Branches        568      568
============================================
- Hits           9772     9771       -1
- Misses         2904     2905       +1

Impacted Files
Coverage Δ

.../forsyte/apalache/tla/pp/ConstSimplifierBase.scala
96.47% <0.00%> (-1.18%)
:arrow_down:

Continue to review full report at Codecov.

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


Incosistent end-effector position from PinocchioEndEffectorKinematicsCppAd

Hi,
I am trying to add a soft constraint (EndEffectorConstraint similar to mobile manipulator example) for an arm EE tracking of a quadruped manipulator. This is done by adding an additional cost term on top of the default costs on state and input which are defined in the ocs2_legged_robot example. For tracking an EE pose a new buffer is created at the ReferenceManager and a new ros topic at the RosReferenceManager for communicating TargetTrajectories of the EE pose (although not sure if this is the best way to go, see issue 49).
I am not achieving good tracking and it does not seem to be a tuning issue. I have noticed the below unexpected behavior:
Within the EndEffectorConstraint::getValue() I print the position of the EE to be tracked through endEffectorKinematicsPtr_->getPosition(state).front() and I am comparing it with the published frames in the /tf topic (by simply rosrun tf tf_echo odom ee_frame). When the frame to be tracked belongs to the lower body (frames up to torso_2 link for centauro) it seems to work well and the position is consistent. In contrast, for any of the frames of the arm links the endEffectorKinematicsPtr_ gives position that is way different from the tf.
Note that:

endEffectorKinematicsPtr_ points to a PinocchioEndEffectorKinematicsCppAd object constructed similarly with the feet EE kinematics in your ocs2_legged_robot ddp example. In particular with the following piece of code:

// define EE kinematics object
std::unique_ptr<EndEffectorKinematics<scalar_t>> eeKinematicsPtr;
const auto infoCppAd = centroidalModelInfo_.toCppAd();
const CentroidalModelPinocchioMappingCppAd pinocchioMappingCppAd(infoCppAd);
auto velocityUpdateCallback = [&infoCppAd](const ad_vector_t& state, PinocchioInterfaceCppAd& pinocchioInterfaceAd) {
  const ad_vector_t q = centroidal_model::getGeneralizedCoordinates(state, infoCppAd);
  updateCentroidalDynamics(pinocchioInterfaceAd, infoCppAd, q);
};
eeKinematicsPtr.reset(new PinocchioEndEffectorKinematicsCppAd(*pinocchioInterfacePtr_, pinocchioMappingCppAd, {name},
                                                              centroidalModelInfo_.stateDim, centroidalModelInfo_.inputDim,
                                                              velocityUpdateCallback, name, modelSettings_.modelFolderCppAd,
                                                              modelSettings_.recompileLibrariesCppAd, modelSettings_.verboseCppAd));

Since EndEffectorConstraint::getValue() is called multiple times along the horizon of the optimization I get the EE position at those times. In my comparisons with the /tf I actually use the first EE position which corresponds to the time closest to the current one. (In any case the EE position does not change much along the horizon since I am comparing after the robot has entered the "steady state response", i.e. it does not really move.)

The state vector passed as argument to the endEffectorKinematicsPtr_->getPosition(state) is consistent with the state observation (from /legged_robot_mpc_observation topic).

I was wondering if you have any clues about what can be causing this strange behavior or how to debug this?
Thank you

Solved. Joint ordering within the centroidal model was not consistent with the one within the pinocchio model.


Update cloud-flow-run-metadata.md

Narrowing the roll out note to sovereign clouds:
"This feature is in the process of rolling out and is not available in the sovereign clouds region yet."

Please delete this PR. I'll make it in the private repo.


docker-compose up fails

Make sure these boxes are checked before submitting your issue - thank you!

[X] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[X] I have reproduced the issue with at least the latest released version of superset.
[X] I have checked the issue tracker for the same issue and I haven't found one similar.

Superset version
latest  commit on master
Expected results
successfully start docker container
Actual results
Command "/usr/local/bin/python -u -c "import setuptools, tokenize;__file__='/tmp/pip-install-fa_crv71/cryptography/setup.py';f=getattr(tokenize, 'open', open)(__file__);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, __file__, 'exec'))" install --record /tmp/pip-record-chrjo_4e/install-record.txt --single-version-externally-managed --compile" failed with error code 1 in /tmp/pip-install-fa_crv71/cryptography/ ERROR: Service 'superset' failed to build: The command '/bin/sh -c pip install --upgrade setuptools pip     && pip install -r requirements.txt -r requirements-dev.txt     && rm -rf /root/.cache/pip' returned a non-zero code: 1
Steps to reproduce
I followed these steps: https://github.com/apache/incubator-superset/blob/master/docs/installation.rst
based on that commit https://github.com/apache/incubator-superset/commit/02aa3c6395a381e59dced848df55cb1daf733e95

I was able to solve by editing the cryptography package version in requirements.txt, found the solution here: https://github.com/apache/incubator-superset/issues/4870#issuecomment-444189653

I was able to solve by editing the cryptography package version in requirements.txt, found the solution here: #4870 (comment)

Nice. Could you make a PR?


program

[Enter feedback here]

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

ID: 11bbaa44-52ed-33fd-d710-af73d3665ec9
Version Independent ID: b11deb17-311c-3aa3-78a7-1f8cdc7e9e5e
Content: Azure Container Registry authentication with service principals
Content Source: articles/container-registry/container-registry-auth-service-principal.md
Service: container-registry
GitHub Login: @mmacy
Microsoft Alias: marsma

@uriux-andrewd thanks for reaching out. If you have feedback please open a new issue with a description of the problem.


Change Readme to Yii3 extension

Q
A

Is bugfix?
❌

New feature?
❌

Breaks BC?
❌

Fixed issues
-

This extension give confusion to Yii2 user (especially new Yii2 user) since it readme state this extension as Yii2 extension.

👍


[Bug]: Performance Issue

Is there an existing issue for this?

[X] I have searched the existing issues

Description
Data population is taking more than 5 mins for customers, even though the data fetching is completed within first 5 secs.
Steps To Reproduce
https://theappsmith.slack.com/archives/C0341RERY4R/p1692013417747689
Public Sample App
No response
Environment
Production
Issue video log
No response
Version
latest

Also having this issue as described.

Please see profiling metrics

This issue has been fixed in v1.9.33
Duplicate of https://github.com/appsmithorg/appsmith/issues/26400


feat(hedera): added support for hedera network by krypc team

Pls fix the issues.

Pls fix the issues.

Fixed

Still fail.

Still fail.

subject added

plz close this as soon as possible

plz close this as soon as possible

You still have linter issues. Pls fix all your issues in one shot if you would like to close this sooner. It’s going to take multiple rounds like now if you are only fixing some but not the rest of the issues.

I have changed the commit msg, guide me if i am wrong

Your PR is still failing. Pls fix the issues.

commit message changed

i have corrected commit msg.

Please correct all commit msg.

I revised my commit message in accordance with the guidelines provided at https://commitlint.io/

checklist passed please merge.

when it will merge?


Convert DCTap data model to ShEx

Convert the DCTap data model to ShEx.

First version done at #69, from now on, we can create more specific issues.


Add a wiki page on usage with React context

+1 Would love to know how to use useStateMachine with createContext and TypeScript.


Divison Function need to be modified

It works only for integer value, need to add support for float also.

Hi @justudin  - I just opened up a PR (#5) to address this issue.


Broadcast schemaForm.error for a specific form

Similar to #589 broadcast an schemaForm.error.name could not be enough when we have multiple forms. One solution could be to create multiple controllers (one for form) but IMO is not the best solution.
Any plan to pass the form name on broadcast in order to Inject the right errors in the right form?
If not, any hint about a possible solution?

I personally so have a different controller per form. With Angular 2 and the shift to components this is likely to be even more common I would expect. However I will keep it in mind when we start refactoring the validations into their own component/repo.

@Anthropic thanks. Could you give me a hint what/where to modify so I will start doing it by myself?
About the discussion on Angular 2, isn't it the way around? They are trying to get rid of controllers, and adding a controller for each form doesn't seem to be ideal.
Also, I guess one of the reasons to use schema-form is to avoid code repetition in html templates and  DOM elements. Let's say I have a list of forms that is coming from the server so I don't know in advance how many forms I have to create. I would use an ng-repeat:
<form ng-repeat="form in formList"
  sf-schema="form.schema"
  sf-form="form.formDef"
  sf-model="model"></form>

attaching the right controller for the right form could be kind of tricky.
Also if I have 100 form definitions it means I should create 100 controllers just to capture the right broadcasted schemaForm.error.name for the right form. Doesn't seems an option to me

I provided a PR if interested

@LeonardoGentile thanks for that, I will take a look this week, I am not doing the release this weekend as I need some questions answered first from the lead developer who is a notoriously busy man in demand, so plenty of time for this to make it in :)

Released as part of 0.8.13


“FishSpeech.available False”, what is it?

Launching comfyui displays this: FishSpeech.available False
But I don't know what it is.

Launching comfyui displays this: FishSpeech.available False But I don't know what it is.

A TTS, specific installation can refer to FishSpeech.


How to set user agent in the auth request ?

our domain is protected with IAP authentication, and its blocking our e2e test.
so we have added API authentication using the code snippet below
import { GoogleAuth } from 'google-auth-library';
const auth = new GoogleAuth();
const url = `${configList.baseURL}/dashboard`;
const targetAudience = "51297171365dbt6s3oc4d09cf.apps.googleusercontent.com";
async function IapAuth() {
    try {
            console.info(`request IAP ${url} with target audience ${targetAudience}`);
            const client = await auth.getIdTokenClient(targetAudience);
            const res = await client.request({
                url, headers: {
                    user-Agent: 'e2e.test'
                }
            });
            IAP_token = res.config.headers.Authorization
            process.env.IAP_token = IAP_token
            console.log(`√ Iap token has been obtained ----> ${JSON.stringify(IAP_token)}`)
    } catch (err: any) {
        console.error(JSON.stringify(`X ---> ${JSON.stringify(err.message)}`));
        process.exitCode = 1;
    }
}

But since our domain is protected by DataDome, we need to pass a certain user agent in the request in order to not be blocked by Recaptcha, but the solution above didn't work out  since my user agent is not overnight the existing one but it appends with it

and I'm getting

the response below
response: {
    config: {
      url: 'https://test.net/ch/test/en/accounts/mpre/dashboard',
      headers: [Object],
      paramsSerializer: [Function: paramsSerializer],
      validateStatus: [Function: validateStatus],
      responseType: 'json',
      method: 'GET'
    },
    data: {
      url: 'https://geo.captcha-delivery.com/captcha/?initialCid=AHrlqAAAAAMAcG9_Htx_-xsAJcnjBA==&cid=y4Yi1XOmjjBKzFYD5yHexjTu9D8m4fr7R8VJ0qXItUW6~ml8Qy29O9I4.35XmXgvk4p9x8GKyoGcSJ2a8xWgXhlzp1qBP1wcuZPGoD8XXA-70sd-l5qy1iIIxQ753Gx&referer=http%3A%2F%2Fim-search-demo-predev.pricehubble.net%2Fch%2Fimmoworlddashboard%2Fen%2Faccounts%2Fmpre%2Fdashboard&hash=9F1FB446984167FE5DE89E73FFEE6E&t=fe&s=32675&e=71208da2bdf44b4b6ce35cd599f70dfd86f153f9cde560aab23d80d53832f711'
    },
    headers: {
      'accept-ch': 'Sec-CH-UA,Sec-CH-UA-Mobile,Sec-CH-UA-Platform,Sec-CH-UA-Arch,Sec-CH-UA-Full-Version-List,Sec-CH-UA-Model,Sec-CH-Device-Memory',
      'alt-svc': 'h3=":443"; ma=2592000,h3-29=":443"; ma=2592000',
      'cache-control': 'max-age=0, private, no-cache, no-store, must-revalidate',
      charset: 'utf-8',
      connection: 'close',
      'content-length': '463',
      'content-type': 'application/json;charset=utf-8',
      date: 'Wed, 16 Nov 2022 13:31:58 GMT',
      pragma: 'no-cache',
      server: 'istio-envoy',
      'set-cookie': 'datadome=y4Yi1XOmjjBKzFYD5yHexjTu9D8m4fr7R8VJ0qXItUW6~ml8Qy29O9I4.35XmXgvk4p9x8GKyoGcSJ2a8xWgXhlzp1qBP1wcuZPGoD8XXA-70sd-l5qy1iIIxQ753Gx; Max-Age=31536000; Domain=.pricehubble.net; Path=/; Secure; SameSite=Lax',
      via: '1.1 google',
      'x-datadome': 'protected',
      'x-datadome-cid': 'AHrlqAAAAAMAcG9_Htx_-xsAJcnjBA==',
      'x-dd-b': '537',
      'x-envoy-upstream-service-time': '39'
    },
    status: 401,
    statusText: 'Unauthorized',
    request: {
      responseURL: 'https://test.net/ch/test/en/accounts/mpre/dashboard'
    }
  },
  config: {
    url: 'https://itest/ch/test/dashboard/en/accounts/mpre/dashboard',
    headers: {
      Authorization: 'Bearer eyJhbGciOiJSUzI1NaWF0IjoxNjY4NjA1NTE3LCJpc3MiOiJodHRwczovL2FjY291bnRzLmdvb2dsZS5jb20iLCJzdWIiOiIxMTc5NDk1NzQzODI4NTIwOTg3NTEifQ.HtGYf8R1NCE-MEgPDKvLz77gsdDpLPTbHpK26XYm1gAHhelOlObszZBMBbg3OhWc9VLStsg1rIHdv7VTEm3TB-v_orrpeTyQnqI11MvWvNWevJ4mr6KkDjO5AX8TKM9lOs-VBTVmHqWUcS3IADcpgJbExqwN42_0-VOUc0M9Wk_UC8lH0BaSXBwh5YNRkh5iNUnaTEmHa1bsqvV3Dcmhd1Lso3LtLJ80zcYz0nPVpdQdAESXp3nSOxETEdQ-ySFwF4NNZOy5CTp-AuJjmnpqHaGt7wDZGtf3Gux7f6Ps8e0rmZVnSHrK1T-LVtk6Z32yhb2ctM0ZsULIFFCaA2hOsw',
      'User-Agent': ' e2e_test' google-api-nodejs-client/8.7.0',   -----> need to change this
      'x-goog-api-client': 'gl-node/16.15.1 auth/8.7.0',
      Accept: 'application/json'
    },
    paramsSerializer: [Function: paramsSerializer],
    validateStatus: [Function: validateStatus],
    responseType: 'json',
    method: 'GET'
  },
  code: '401'
}

PLEASE READ: If you have a support contract with Google, please create an issue in the support console instead of filing on GitHub. This will ensure a timely response.

Hey @amrsa1,
You can extend the DefaultTransporter class and use the configure method to set the User-Agent for now:
class MyTransporter extends DefaultTransporter {
  configure(opts?: GaxiosOptions): GaxiosOptions {
    const options = super.configure(opts);
    if (!options.headers) options.headers = {};
    // Can also use `opts?.headers?` as well if you'd like to set per-request.
    options.headers['User-Agent'] = 'e2e_test';
    return options;
  }
}
client.transporter = new MyTransporter();

const res = await client.request({...});

In the next major release, this functionality will be made easier by this PR: https://github.com/googleapis/google-auth-library-nodejs/pull/1406

Closing as the question has been addressed, even though the linked PR isn't merged.

I wanted a generic way to do this so I took the example from @danielbankhead and made https://gist.github.com/josephdpurcell/b434efc4870e4e7ba8387f9f7b98fd47#file-workaround-ts which uses the default User-Agent if its set.


Update gpt_cust_kb.py

I rewrote quite a bit of the code to make this more functional. It now asks for your API key (though I haven't tested this, since I already have my API key set as an environmental variable) and has a field to ask for a PDF, which then indexes the PDF. If you reload the page, the radio buttons update with every json in the folder. I'm not sure how to make them update without the reload.
I have almost no python experience, so this is probably kind of a mess, but it should give you something to work from and seems pretty functional from my quick testing. Thanks for putting this together so that I had a chance to even get this far.

Hi @pacmanincarnate thanks for you contribution and sorry for the late reply, I have been on vacation. However I did use some ideas from your commit as to which  direction to take the final update so I appreciate that! Check out the new functionality and let me know what you think!


Ripple button border

I was noticing some weird gaps between the border and the actual fill of the ripple buttons and tried fixing it but couldn't seem to figure out what was causing it. When I was testing, it seemed like what part of the fill that was getting cut off was based on where the button was on the page, but that didn't help me much with actually trying to fix it.
So what I did instead was got rid of the border altogether. It's not a perfect fix and I get it if you don't choose to merge this commit, but I thought it at least serves the purpose of bringing awareness to this weird bug. I commented out a section of the code instead of deleting it because I thought it would be good to see what the code was like before I edited it, but I forgot that GitHub shows you the changes I made anyways.

In the commits section it shows I had made 3 commits where 2 of them are called "Merge pull request...". Those were mistakes I meant to pull from this main sbuttons repo to my forked repo to update it but ended up trying to merge mine with the main. I'm not sure how to get rid of the commits and have no idea why they ended up here, but I don't think they cause any problems or anything so just wanted to let you know.

Hi @drewren25 thank you for your contribution. I'm looking at the current Ripple button and honestly I'm not seeing where the problem is. If you have more details on the bug (maybe screenshots) you can create a new issue with it and maybe if it's valid someone can work on it. I'm closing this PR but I'll add the "hacktoberfest-accepted" label because I appreciate your work.

Ok no problem! I'll create a new issue about it. Weird that you aren't seeing it maybe it just a problem on my end?


GET /socket.io/?EIO=4&transport=websocket HTTP/1.1" 200 0 45.359629

I am having same issue on my local drive virtual machine and I am not able to resolve it. I tried updating the python-socket it didn't work. I would appreciate if someone could help. It was working on the Udacity work-space but when I tried one day later it gave me same error. I didn't update anything on the Udacity work-space. I have python-socketio==1.6.1 on the work-space and it was working fine until yesterday.
I am getting following issue on the local virtual machine as well as on Udacity's work-space and I am not able to sort it out, help will be really appreciated.
"GET /socket.io/?EIO=4&transport=websocket HTTP/1.1" 200 0 45.359629"

Have you figured out? I have same issue

@liupaladin I think yes I have it happens when your memory is full. To solve this issue I would suggest using a high end computer with high GPU and RAM and or the provided workspace.


Support cron scheduling via GCP Cloud Scheduler

Currently cron jobs are managed using Cloud Tasks and Redis. Investigate a way to also support Cloud Scheduler to eliminate the dependency on Redis.

Is it already worked one? If not, I can take a look at this one. I guess the starting point would be here https://github.com/keypup-io/cloudtasker/blob/master/lib/cloudtasker/cron/schedule.rb.

Is it already worked one? If not, I can take a look at this one. I guess the starting point would be here https://github.com/keypup-io/cloudtasker/blob/master/lib/cloudtasker/cron/schedule.rb.

@Fodoj not it's not worked on yet. Feel free to give it a try! 😄
You can create a new module called CloudScheduler (instead of cron). The implementation should be relatively straightforward but there isn't much doc about the shape of the Cloud Scheduler webhooks. So you'll have to experiment via ngrok.
Cloud Scheduler does support unique names for jobs. It should therefore be possible to lookup / create if missing, same as the cron extension (see: https://github.com/keypup-io/cloudtasker/blob/master/lib/cloudtasker/cron/schedule.rb#L55)
Also, considering names must be truly unique in Cloud Scheduler, I recommend prefixing the schedule names with the gcp_queue_prefix (see https://github.com/keypup-io/cloudtasker/blob/master/lib/cloudtasker/config.rb#L188)
Let me know if you get stuck anywhere, I'm happy to help.

@Fodoj not it's not worked on yet. Feel free to give it a try! 😄
You can create a new module called CloudScheduler (instead of cron). The implementation should be relatively straightforward but there isn't much doc about the shape of the Cloud Scheduler webhooks. So you'll have to experiment via ngrok.
Cloud Scheduler does support unique names for jobs. It should therefore be possible to lookup / create if missing, same as the cron extension (see: https://github.com/keypup-io/cloudtasker/blob/master/lib/cloudtasker/cron/schedule.rb#L55)
Also, considering names must be truly unique in Cloud Scheduler, I recommend prefixing the schedule names with the gcp_queue_prefix (see https://github.com/keypup-io/cloudtasker/blob/master/lib/cloudtasker/config.rb#L188)
Let me know if you get stuck anywhere, I'm happy to help.

I'm not promising anything... but here I go :)

Hello all. First off, thank you for developing this gem - it's absolutely fantastic! I'm using this with ActiveJob in my current project.
Second, I'm wondering if there's any updates on this? I can definitely see a need for this in the future. I understand there is no way currently to implement cron + ActiveJob + cloudtasker? Similarly for job locks/uniqueness? Again, thank you for all the effort!

One way I have integrated GCP cloud scheduler is to create an endpoint to kick of the workers based on the post body parameters. This is an example in rails:
`class WorkersController < ApplicationController
def create
Workers.worker_class(worker_params[:worker_name].to_sym).perform_async(worker_params[:args])
render json: {}, status: :no_content

end
private
def worker_params
params.permit(%i[worker_name args]).require(:worker_name)
params[:worker]
end
end
class Workers
def self.worker_class(job_name)
workers[job_name]
end
def self.workers
{
fake_worker1: FakeWorker1,
fake_worker2: FakeWorker2,
fake_worker3: FakeWorker3
}
end
end
`

Is this still going to be added? I see @ceritium worked on it but there hasn't been any updates on that PR in the last 6 months.

Is this still going to be added? I see @ceritium worked on it but there hasn't been any updates on that PR in the last 6 months.

Hi @MohamedBeydoun, please take a look at my last comment on the PR.

Is this still going to be added? I see @ceritium worked on it but there hasn't been any updates on that PR in the last 6 months.

Hi @MohamedBeydoun, please take a look at my last comment on the PR.

Thanks!


COM-156: Prevent overlapping of clear-button with select-arrow

Since the end-adornment for the clear-button needs absolute positioning, it is no longer possible to use the clearable prop in combination with an  endAdornment. However, this is not a common use-case, and an endAdornment on a select is generally not supported by MUI, see: https://github.com/mui/material-ui/issues/17799.
An alternative would have been to position the select-arrow relative. Unfortunately, this would cause the clickable element that opens the select to shrink and prevent opening the select by clicking the select-arrow. Since this would confuse users, the endAdornment is now also positioned absolutely.

Previously
Now

However, this is not a common use-case, and an endAdornment on a select is generally not supported by MUI, see: mui/material-ui#17799.

Why don't we remove support for endAdornment for selects as well? I can't think of a likely use case anyway

I can't really think of a good use-case either...
However, there might be some older projects, that manually implement the Clear-Button as an endAdornment, from before the clearable Prop existed. That should be considered a breaking change imo and wait for v5 or v6.

However, this is not a common use-case, and an endAdornment on a select is generally not supported by MUI, see: mui/material-ui#17799.

Why don't we remove support for endAdornment for selects as well? I can't think of a likely use case anyway

I can't really think of a good use-case either... However, there might be some older projects, that manually implement the Clear-Button as an endAdornment, from before the clearable Prop existed. That should be considered a breaking change imo and wait for v5 or v6.

Yes, of course, removing it completely would be a breaking change. @thomasdax98 could you please add a task for this future improvement? Thanks!


Verify model confidences manually before 3.0 release

Slack Thread
We have run model regression tests in the integration phase of architecture revamp. However, those tests do not check the distribution of confidences that are output by the models. This distribution can be generated by training and testing on a dataset and examining the plot generated in intent_histogram.png. The examination should check if the confidence distribution of correct and wrong predictions looks "approximately" the same when trained with 2.8.x and 3.0.0 (They won't be exactly the same because of some changes that come with 3.0).
The datasets and configs (can be found in training-data repo) on which this should be tested at at the least (covers english and german dataset + configs that are frequently used by customers):
Dataset: public/Sara
Configs:

en/cvf_bert_diet_responset2t.yml
en/cvf_diet_responset2t.yml
en/cvf_embedding_responseb2b.yml
en/cvf_bert_embedding_responseb2b.yml

Dataset: private/service_faq
Configs:

en/cvf_spacy_diet_responset2t.yml
en/cvf_diet_responset2t.yml
en/cvf_embedding_responseb2b.yml
en/cvf_spacy_embedding_responseb2b.yml

Definition of Done:

Training and evaluation run for the above dataset and config combo using 2.8.x and a release candidate of 3.0.0 / main branch of Rasa OSS
Verified that intent_histogram.png look "approximately similar" in every unique instance of the dataset and config combo.

@dakshvar22 so for the first one we would run:
rasa train nlu --nlu public/Sara/train --config configs/en/nlu/cvf_bert_diet_responset2t.yml
rasa test nlu --nlu public/Sara/test --config configs/en/nlu/cvf_bert_diet_responset2t.yml --model the_model.tar.gz

for 2.8.x and 3.0.0rc and then compare the intent_histogram.png for each?

Yep

en/nlu/cvf_spacy_diet_responset2t.yml and en/nlu/cvf_spacy_embedding_responseb2b.yml don't seem to exist. Did you mean de/... for both?

Ahh, yes sorry my bad.

In fact I guess all the private/service_faq should be german configs right?

Theoretically yes but you wouldn't need to for others because practically the language key does not matter for the other configs I mentioned. But specifically for *_spacy_* configs you'll have to necessarily use german configs since its only available for that.

rasa_test_2_vs_3_results.zip
@dakshvar22 This zip contains all the results for all the configs in the description for 2.8.14 and 3.0.0rc3. The histograms look comparable to me - although would be good to get another (more histogram experienced) pair of eyes on them.

Looking at them now 👀

They don't look abnormally different which is great!
However, there is one interesting observation I could make: The histograms for configurations which use no transformer layers (*_embedding_*) look almost (and exactly in some cases) identical which does not seem to be the case for configurations which use transformer layers (*_diet_*). The difference is much more in the latter.
Do we know why this would be the case? I would like for us to at least know what we have changed that would result in such an observation?
Tagging @ka-bu since she might have more context here.

Does not sound that surprising that those runs with transformer layers yield larger differences in general, no?
To put the differences into perspective it might help to re-run the same experiment (e.g. just re-run training on 2.8.14 with one of the diet configs and re-evaluate a few times). Or are those results on 2.8.14 so stable / much more similar to what you saw on 2.8.x before (in comparison to 3.0) now @dakshvar22 ?

They won't be exactly the same because of some changes that come with 3.0

Which changes would that be here? Cause we're testing only the NLU component here (and the same tf updates have been done in both branches), no?

@m-vdb @joejuzl There are a few open questions here (apologies for not replying to Kathrin's question earlier) and I don't think the issue should be closed.
Do you want the conversation to happen anywhere else?

sounds good @dakshvar22 -- I closed it because it was in the "Done" column, surely a mistake.
It looks like @ka-bu had some questions for you to answer in this comment. Could you provide more input?

Does not sound that surprising that those runs with transformer layers yield larger differences in general, no?

@ka-bu Why should they yield large differences if we haven't touched the TF part of the codebase during the refactor? AFAIK, the featurization also hasn't changed or has it?

Does not sound that surprising that those runs with transformer layers yield larger differences in general, no?

@ka-bu Why should they yield large differences if we haven't touched the TF part of the codebase during the refactor? AFAIK, the featurization also hasn't changed or has it?

My thinking there was that more layers / computations (i.e. including transformer layers) imply more chances for differences due to numerical issues / non-deterministic operations. And no, the featurization should not have changed (the biggest changes related to featurization was the lookup table but that only affects the policies not the nlu part discussed here).

If the featurization didn't change + random seed was set in the config (which it is if the configs were taken from the dataset repo) + experiment was run on CPU, the results should actually be completely reproducible.
Some operations are not deterministic only on GPU. They are still deterministic on CPU. @joejuzl Can you please confirm that you ran these experiments locally on a CPU machine?
My gut feeling is that the feature vectors produced by CountVectorsFeaturizer has changed. 🤔

Which changes would that be here?

Now that I think of it, I think I was really referring to just the lookup table bit but that shouldn't affect NLU components.
Another bit that can cause a difference: Is it possible that the order in which data points are read in 3.0 is different from the order in which they were read in 2.8.x?

@joejuzl Can you please confirm that you ran these experiments locally on a CPU machine?
Yeah I ran them locally on my macbook.

Are you saying that the difference between 2.8.x and 3.0.x is more than the difference expected from e.g. re-running on 2.8.x?

Are you saying that the difference between 2.8.x and 3.0.x is more than the difference expected from e.g. re-running on 2.8.x?

Re-running on 2.8.x multiple times on CPU will yield exactly same model confidences (if run on CPU), and hence the confidence histograms will be exactly the same. So, we should really find out what is causing the difference in model confidences when the same config + dataset + CPU machine is used but with 2.8.x and 3.0 installations of rasa.
I should emphasize that the differences don't appear to be large (for e.g. - in private/service_faq dataset, the distribution seems to be shifted for 10-11 training examples) so it isn't a high-priority investigation, but nevertheless it should be done at some point in time to prevent an unknown regression causing larger regressions in the future.

Makes sense - and yes I agree this needs to be investigated further. I will move this back to the inbox.


[Running PR] Enable building on JitPack

This is a helper PR to keep an easier-to-follow history of the branch that enables building on JitPack.
No plans to merge.

Commits https://github.com/wordpress-mobile/react-native-masked-view/pull/1/commits/959000abf16d7a024c9c084af7a9fc7d86293573 and https://github.com/wordpress-mobile/react-native-masked-view/pull/1/commits/f65a51a3320e58404d7f38d967bfd1f42b439ca9 were originally pushed on branch wp/bump-version but since we had actually been using them in production for a while, I merged that branch to feat/jitpack moments ago, just to keep things slightly cleaner.

Closing in favor of https://github.com/wordpress-mobile/react-native-masked-view/pull/3


A question about search.nixos.org

I was wondering if I could acess some sort of api for search.nixos.org. I am writting a wrapper over the nix cli tools in rust, so it would help my project alot. When I use nix search nixpkgs <query> I get good json results, but I do not search through haskellPackages, nodePackages, and other package groups. Can I enable this in nix search. Is there an api for search.nixpkgs.org. If there is no solution, it is not a big deal, I was just wondering. I am quite new to NixOS so scream at me :-) if I am stupid!

The search has its own repo at https://github.com/NixOS/nixos-search, and we found an issue discussing the API for the search: https://github.com/NixOS/nixos-search/issues/296
Apparently, it's mostly just base ElasticSearch, which has an existing API doc. Hopefully the comments there help you out!

Ok, I will try to integrate this into my cli!
Thanks!

I am still a bit confused, so is it ok if I make an issue on their repo?
Thanks for responding so quickly!

Pretty sure it would be! We're not involved in NixOS or the search things very much, but that's definitely the better place to ask questions, and you could probably close this one and open something there. Good luck!


 U: .

 U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3835/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 26s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+1 :green_heart:
mvninstall
5m 20s
master passed

+0 :ok:
refguide
3m 45s
branch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 59s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+0 :ok:
refguide
3m 41s
patch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.

_ Other Tests _

+1 :green_heart:
asflicense
0m 19s
The patch does not generate ASF License warnings.

19m 10s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3835/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3835

Optional Tests
dupname asflicense refguide

uname
Linux 3ef6b64e279b 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 76ceda3b58

refguide
https://nightlies.apache.org/hbase/HBase/HBase-PreCommit-GitHub-PR/PR-3835/1/yetus-general-check/output/branch-site/book.html

refguide
https://nightlies.apache.org/hbase/HBase/HBase-PreCommit-GitHub-PR/PR-3835/1/yetus-general-check/output/patch-site/book.html

Max. process+thread count
78 (vs. ulimit of 30000)

modules
C: . U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3835/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


FinalizePurchaseAsync required?

Hi,
I've recently upgraded to version 5.6.1 of the InAppBillingPlugin.
Previously, after making a purchase:
   billing.PurchaseAsync(productId, ItemType.Subscription)

I had the following code to AcknowledgePurchase, for Android only:
   if (Device.RuntimePlatform == Device.Android)
                {
                     //Must call AcknowledgePurchaseAsync else the purchase will be refunded
                     await billing.AcknowledgePurchaseAsync(purchase.PurchaseToken);

                }

When I upgraded the package, the readme says that this has changed to FinalizePurchaseAsync, and it also needs to be called on iOS after purchase. However the documents online indicates that this is only required, and only on android, if you turn off auto-finalize.
Pls could someone clarify?
For completness, what I have now is this; i've not set any auto-finalize flags. Would this work or should i remove the FinalzePurchaseAsync part?
    var billing = CrossInAppBilling.Current;
               try
               {
                   var connected = await billing.ConnectAsync();
                   if (!connected)
                   {
                       //we are offline or can't connect, don't try to purchase
                       return new BaseResponse() { IsSuccess = false, Message = "Unable to connect" };
                   }

                   var purchase = await billing.PurchaseAsync(productId, ItemType.Subscription);

                   //possibility that a null came through.
                   if (purchase == null)
                   {
                       return new BaseResponse() { IsSuccess = false, Message = "Purchase failed" };
                   }

                   else if (purchase.State == PurchaseState.Purchased)
                   {

                       // need this for both iOS and Android now, apparently.
                       var ack = await billing.FinalizePurchaseAsync(purchase.TransactionIdentifier);
                   }

                   // app specific code..

Yes, I reverted it back so iOS auto finalizes like it did previously.
You need to do it on Android just like you did before.
I will update docs and such.


Merge the redundant readme

I noticed there was some company and developer info in the other README so I just changed it to some  tags at the top of the README markdown file so there's nothing lost. Old readme deleted.

See #3


refactor(admin): update yarn.lock file of admin.

🍰 Pullrequest

Issues

None

Todo

[X] None

Is this the browser list?

@Mogge No it only updates 'caniuse-lite'.
I did update backend, frontend, database and the other folders but their was nothing to update for me!


'npm run build' fails

Any guesses?
$ npm run build

<EMAIL_ADDRESS>build C:\Users\cstepnitz\ozp2\ozp-center-release-1.1.35
NODE_ENV="production" gulp build

'NODE_ENV' is not recognized as an internal or external command,
operable program or batch file.
npm ERR! Windows_NT 6.1.7601
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Program Files\nodejs\node_modules\npm\bin\npm-cli.js" "run" "build"
npm ERR! node v4.4.7
npm ERR! npm  v2.15.8
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>build: NODE_ENV="production" gulp build
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script 'NODE_ENV="production" gulp build'.
npm ERR! This is most likely a problem with the center-ui package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     NODE_ENV="production" gulp build
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs center-ui
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!
npm ERR!     npm owner ls center-ui
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     C:\Users\cstepnitz\ozp2\ozp-center-release-1.1.35\npm-debug.log

I don't think they support windows.

@cstepnitz If you haven't addressed this already, you can replace NODE_ENV="production" gulp build in package.json with set NODE_ENV=production&& gulp build on Windows.

has this issue been resolved?

@akonsowski It appears that package.json hasn't been updated to address Windows/Linux compatibility, so I'd say in that sense the issue is not resolved.  But if the official answer is "ozp-center development doesn't support Windows" then I think that should be documented in order for this to be considered resolved.  For what it's worth, I changed package.json locally to address this particular issue but ran into other issues when trying to run ozp client projects on Windows.

I documented this in Developer Setup for future reference.

thanks


Global leak variables

I was using mocha along with nock and it warned me about two global variable leaks, thisInterceptor and `body.
This small pull request fixes those leaks.

Merged, thanks!


feature/drf-social-auth

Based on django-rest-social-auth. Refactored some parts and added new features to be compatible with the given task. Created base classes and observer classes to make code DRY. Used Abstraction principle in subclass views.
TODO:

no need to allauth it should be removed from reqs and settings
write test for developed parts

Sorry for the date of my commits, I forgot to commit my changes to the forked repo and I was forced to recommit them in the forked repo at the end of work. I have to inform you that it took about 3-4 days to research and implement.

Hi @mehran-rahmanzadeh are you able to go through the todo items you listed (namely remove allauth) and resolve the conflicts?

Hello @ziyapx, modifications made.

This is how we wanted this done:
https://github.com/djangoflow/django-df-auth/pull/6


Software Documentation - MoCap

We need to add a "MoCap" section under the "Hardware" section of our documentation. See this link for full layout https://stanford-msl.slack.com/archives/CBLRMNEQ7/p1566425187004900
This will have subsections for each of the following:

[ ] Calibration
[ ] Configuration

Moved to Jira: https://asjira.stanford.edu/jira/projects/MSLQ/summary


getFileData method doesn't load at all,

var Flash = {
/* Flash.getFileData() is called after the file has been read /
getFileData: function(base64) {
showResult(base64)
},
/ getButtonLabel() permits to define another label for the "Load a file" button in the Flash version */
getButtonLabel: function() {
return "Load a file";
}
};
function showResult(b) {
$('#file-result').text(b)
}
unable to get binary data on showresult function
below reference added to code:
swfobject.embedSWF("../Content/FileToDataURI.swf", "file-object", "200px", "90px", "10", "../Content/expressInstall.swf", {}, {}, {});//}
and below 3 files are refereed;
Content/FileToDataURI.swf
../Content/expressInstall.swf
../Scripts/swfobject.js" refered as javascript link in page

You'll have to provide more information....
Is the demo working for you : https://aymkdn.github.io/FileToDataURI/ ?
Which browser do you use ?
Do you have a link where I can test it ?
Any other info ?

No answer, so I'm closing it.

IE8


Long Running Checkers

We'd love to have checkers that do things for more than a check.
This should be implemented as async subtasks.

The flow could be as follows:
startAsync(fun)
startTimeout(fun, time)
startInterval(fun, time, rounds)
returning a (string)-handle on which stop(handle) can be called.
Under the hood, the functions should call a asyncio task, or even perform a request (for this we could migrate from flask to sanic)


fix tab text truncation

Should fix an issue reported there https://github.com/dashpay/dash/pull/778#issuecomment-220550343 by simply making toolbar tabs/buttons wider.

This is not only light theme fault, its also in the blue theme.
Transactions tab first left column is now too wide and therefore columns are misaligned.

@akijuh that is not the same problem as this PR. Please report new issues in https://github.com/dashpay/dash/issues so that evreyone was aware instead of reporting in already merged PRs. Thanks!

Ok, i am new with github...

@akijuh np :)


Added multithreading support for Serverspec

New Serverspec multithreading mode gives us up to 4.5x performance boost when running tests :
$ time rake -m -j 8
{
  "succeed": true,
  "example_count": 490,
  "failure_count": 0
}
rake -m -j 8  39% cpu 41.076 total

$ time rake
{
  "succeed": true,
  "example_count": 490,
  "failure_count": 0
}
rake  6% cpu 3:10.76 total

Plus JSON-outputs from different specs are now aggregated into short JSON-summary with proper exit codes and detailed JSON-file (serverspec_results.json).

LGTM


Secrets logged in clear text after parsing issue.

While reviewing our application logs, I have noticed a line that looked like this (redacted for obvious reasons).
{"level":50,"time":1605105915359,"pid":17,"hostname":"platform-kubernetes-external-secrets-578c5b47f4-gh4db","stack":"TypeError: Cannot use 'in' operator to search for 'DATABASE_PORT' in DATABASE_HOST=<redacted> DATABASE_PORT=3306 DATABASE_USER=<redacted> DATABASE_PASSWORD=<redacted> DATABASE_NAME=<redacted> AWS_ACCESS_KEY_ID=<redacted> AWS_SECRET_ACCESS_KEY=<redacted> AWS_DEFAULT_REGION=<redacted> BUCKET_NAME=<redacted> TARGET_FOLDER=int\n    at /app/lib/backends/kv-backend.js:44:24\n    at runMicrotasks (<anonymous>)\n    at processTicksAndRejections (internal/process/task_queues.js:97:5)\n    at async Promise.all (index 1)\n    at async Promise.all (index 1)\n    at async SecretsManagerBackend.getSecretManifestData (/app/lib/backends/kv-backend.js:116:42)\n    at async Poller._createSecretManifest (/app/lib/poller.js:93:1

It looks as if when external-secrets has a problem parsing a JSON secret it exposes the whole secret as clear text in the logs, which depending on the situation can create a big security risk.
This issue is in the same area as https://github.com/godaddy/kubernetes-external-secrets/issues/97, so both could be dealt with at the same time:
# File lib/backends/kv-backend.js

      if (shouldParseValue) {
        let parsedValue
        try {
          parsedValue = JSON.parse(value)
        } catch (err) {
          this._logger.warn(`Failed to JSON.parse value for '${key}',` +
           ' please verify that your secret value is correctly formatted as JSON.' +
           ` To use plain text secret remove the 'property: ${property}'`)
          return
        }

        if (!(property in parsedValue)) {   // <-- the problem is on how the error is handled in this line
          throw new Error(`Could not find property ${property} in ${key}`)
        }

Comment to remove "stale" label.


Add support for language modeling (GPT2)

What does this PR do?

Language modeling is now supported:

a README dedicated to this task is added
the script run_clm.py enables to run an example

Besides, in order to use big enough batch sizes with GPT2, gradient checkpointing and sharded DDP are now supported.
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

The following command runs fine for a few steps:
python run_clm.py \
    --model_name_or_path gpt2 \
    --dataset_name wikitext \
    --dataset_config_name wikitext-2-raw-v1 \
    --per_device_train_batch_size 8 \
    --per_device_eval_batch_size 8 \
    --do_train \
    --do_eval \
    --output_dir /tmp/test-clm \
    --gaudi_config_name Habana/gpt2 \
    --use_habana \
    --use_lazy_mode \
    --gradient_checkpointing

but at the 7th step I get an error:
RuntimeError: FATAL ERROR :: MODULE:DEVMEM Allocation failed for size::1646821376
The memory error should have happened before in the warmup steps no? @libinta

The following command runs fine on 1 HPU:
python run_clm.py \
    --model_name_or_path gpt2 \
    --dataset_name wikitext \
    --dataset_config_name wikitext-2-raw-v1 \
    --per_device_train_batch_size 4 \
    --per_device_eval_batch_size 8 \
    --do_train \
    --do_eval \
    --output_dir /tmp/test-clm \
    --gaudi_config_name Habana/gpt2 \
    --use_habana \
    --use_lazy_mode

However, with the same command in distributed mode on 8 HPUs, the warmup steps are veeeeery slow. Any idea what is going on here? @libinta

The following command runs fine for a few steps:
python run_clm.py \
    --model_name_or_path gpt2 \
    --dataset_name wikitext \
    --dataset_config_name wikitext-2-raw-v1 \
    --per_device_train_batch_size 8 \
    --per_device_eval_batch_size 8 \
    --do_train \
    --do_eval \
    --output_dir /tmp/test-clm \
    --gaudi_config_name Habana/gpt2 \
    --use_habana \
    --use_lazy_mode \
    --gradient_checkpointing

but at the 7th step I get an error: RuntimeError: FATAL ERROR :: MODULE:DEVMEM Allocation failed for size::1646821376
The memory error should have happened before in the warmup steps no? @libinta

@regisss  Can you reduce the batch size to get ride of the memory issue?


Possible unneeded file nrt/NoveltyClient.hpp

Even on /dev this exists:
https://github.com/flucoma/flucoma-core/blob/dev/include/clients/nrt/NoveltyClient.hpp
I might be wrong, but it looks out of date (in terms of style and necessity).

Yes, I keep meaning to kill it


Hub responds with 200 HTTP success but has an error json in the body

https://github.com/boltcard/boltcard/blob/main/lnurlw/lnurlw_callback.go#L120
On this line the hub can respond with a 200 HTTP success code, but the response can contain an error. Here is an example
{"level":"info","msg":"{\"error\":true,\"code\":6,\"message\":\"Something went wrong. Please try again later\"}","time":"2023-03-24T02:25:21Z"}

We need to check for existance of error:true and then report this failure back to the requester.

So the issue still stands, in the case of not enough funds:
Payment request made from here: https://github.com/boltcard/boltcard/blob/8f83e04564f93010d33589de9d1ca3600c981e25/lnurlw/lnurlw_callback.go#L129
This method logs an error.
{"level":"info","msg":"{"error":true,"code":2,"message":"not enough balance. Make sure you have at least 1% reserved for potential fees"}","time":"2023-08-03T03:03:51Z"}
The POS has no idea what the issue is and is left in a "this might happen" / loading state.
What is the benefit of making this response asynchronous?
Can we ignore the spec and do this syncronously so that we can send the error back?
Would be a better UX


Add RSS Feed

RSS feed requested in https://twitter.com/freekmurze/status/628161553792126976 I'll add this asap.

Done https://vinkla.com/feed.xml

Nice! Don't forget to add a link tag to all your pages, so rss readers can find the feed automatically.

How do I do that?

Yep, that's the way to do it. The href-attribute should point to https://vinkla.com/feed.xml.

Okay done fb6993612d4363e3c32b2a335a27d4560804f519


Enhance config handling

create classes for fritzbox config and influx config
support reading env vars for fritzbox and influx sections

implemented


Fix phpstorm link

Hi @Butochnikov,
Thanks for the PR. What is your OS and PhpStorm version?
@luanfreitasdev I can confirm the current handler is broken, and this PR fixes it.
I am on MacOS and PhpStorm 2021.3.3.
Thanks

👍 LGTM
Working on windows with the aid of PhpStormProtocol.
NOTE: This is the handler used in Laravel's Flare.


"before all hook Timeout" (RN 0.45.1, XCode 9.0.1, Detox 5.9.2)

Description
When running detox build/test the build works but the test seems to open the app, quickly close the app, then eventually times out during the before all and after all hooks.
Steps to Reproduce
I don't know how to reproduce the issue, other than following the getting started commands with a similar version of React Native, XCode and Detox. I've seen some similar issues tied to packages like firebase, but this app doesn't use firebase as a dependency. I'm unclear as to what version of Detox that would work with Xcode 9 and with React Native 0.45, but I'm struggling to find a version that works for me.
 The stdout and stderr logs were recreated, you can watch them with:
tail -F $HOME/Library/Developer/CoreSimulator/Devices/F9732E28-49F9-4199-84EB-7EFCC0F6096C/data/tmp/detox.last_launch_app_log.{out,err}

"before all" hook

0 passing (3m)
1 failing

"before all" hook:
Error: Timeout of 180000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.

The error from the stdout logs send out this ...
2017-10-24 14:05:01.266 mobileTech[21768:262542] Crash handler setup started.
2017-10-24 14:05:01.266 mobileTech[21768:262542] Crash handler setup completed.
2017-10-24 14:05:01.266 mobileTech[21768:262542] Enabling accessibility for automation on Simulator.
2017-10-24 14:05:01.570 mobileTech[21768:262542] ☣ DETOX:: Detox Action Sent: login
2017-10-24 14:05:01.691 mobileTech[21768:262542] ☣ DETOX:: Detox Action Received: loginSuccess
2017-10-24 14:05:01.740 [info][tid:main][RCTCxxBridge.mm:184] Initializing <RCTCxxBridge: 0x6000001d1d00> (parent: <RCTBridge: 0x6000000bcb60>, executor: (null))
2017-10-24 14:05:01.756 [warn][tid:main][RCTBridge.m:114] Class RCTCxxModule was not exported. Did you forget to use RCT_EXPORT_MODULE()?
libc++abi.dylib: terminating with unexpected exception of type std::__1::bad_function_call: std::exception
Signal caught: Abort trap: 6
0   EarlGrey                            0x00000001259b6979 grey_signalHandler + 249
1   libsystem_platform.dylib            0x0000000116858b3a _sigtramp + 26
2   ???                                 0x000000010f1e66d8 0x0 +<PHONE_NUMBER>
3   libsystem_c.dylib                   0x00000001164f6107 abort + 127
4   libc++abi.dylib                     0x000000011629091f __cxa_bad_cast + 0
5   libc++abi.dylib                     0x0000000116290aa3 _ZL25default_terminate_handlerv + 241
Detox in package.json
"detox": {
	"configurations": {
		"ios.sim.debug": {
			"binaryPath": "ios/build/Build/Products/Release-iphonesimulator/mobileTech.app",
			"build": "xcodebuild -project ios/mobileTech.xcodeproj -scheme mobileTech -configuration Release -sdk iphonesimulator -derivedDataPath ios/build",
			"type": "ios.simulator",
			"name": "iPhone 8"
		}
	}
}

Upgrade you RN to latest version (0.49+) or use an older version of Detox.


[WIP] Add support to use file PV's as VM disks

Now virt-launcher pod will use VM PV claims and use disk.img file from PV to start a VM.

logic is done
need to implement tests

Have problems with usage of our ISCSI storage.
Signed-off-by: Lukianov Artyom<EMAIL_ADDRESS>

@cynepco3hahue is it possible to give it a dns name instead of an ip?

@rmohr do you mean to give service DNS name instead of service IP?

I found some W/A for the ISCSI problem

add kube-dns service IP to vagrant machine /etc/resolve.conf
use headless service for ISCSI

I checked a start of VM with an ISCIS W/A, and now it can pass login stage but stuck again on devices path discovery.
Under k8s code
if iscsiTransport == "tcp" {
	devicePath = strings.Join([]string{"/dev/disk/by-path/ip", tp, "iscsi", b.Iqn, "lun", b.Lun}, "-")
}

when in our case tp equals to iscsi-demo-target.kube-system.svc.cluster.local, but under node I can see directories with IP instead of DNS name
[root@master by-path]# pwd
/dev/disk/by-path
[root@master by-path]# ll
total 0
lrwxrwxrwx. 1 root root  9 Feb  6 08:30 ip-<IP_ADDRESS>:3260-iscsi-iqn.2017-01.io.kubevirt:sn.42-lun-1 -> ../../sda
lrwxrwxrwx. 1 root root  9 Feb  6 08:30 ip-<IP_ADDRESS>:3260-iscsi-iqn.2017-01.io.kubevirt:sn.42-lun-2 -> ../../sdb
lrwxrwxrwx. 1 root root  9 Feb  6 08:30 ip-<IP_ADDRESS>:3260-iscsi-iqn.2017-01.io.kubevirt:sn.42-lun-3 -> ../../sdc

I think it because DNS can not resolve IP to name
[root@master ~]# host iscsi-demo-target.kube-system.svc.cluster.local
iscsi-demo-target.kube-system.svc.cluster.local has address <IP_ADDRESS>
[root@master ~]# host <IP_ADDRESS>
Host <IP_ADDRESS>.in-addr.arpa. not found: 3(NXDOMAIN)

For testing purpose I can create VM's when I will set TP to ISCSI pod IP, but we can not run VM's with disks(except ephemeral) from cluster/ directory until we edit ISCSI TP.
@rmohr @fabiand Does it good enough?

@cynepco3hahue looks like we need to properly configure the isns server. Then the kubelet should do the discovery via the isns server which can sit behind the service IP. The isns server should then contain the entry on where to connect to the target ...

@rmohr I will check it, I do not sure I want to introduce another container with isns server.
I finished writing NFS server and it works fine so we can consider using it.

@rmohr I will check it, I do not sure I want to introduce another container with isns server.
I finished writing NFS server and it works fine so we can consider using it.

sure, works for me.

@rmohr @davidvossel Do we have some nice place to put string constants like
"/var/run/kubevirt-private" and "vm-disks"?

@aglitke I think it only related to our ISCSI container, in the case when you have normally configured ISCSI that resolvable by DNS, all must work.

@rmohr @davidvossel Hi guys I think it is done, I re-enabled storage tests and it passed fine.
We still have problems to run VM's from cluster/ directory.
Three problematic VM's to run:

vm.yaml in this case user will need to change ISCSI target portal to current ISCSI pod IP
vm-iscsi-auth.yaml in this case user will need to change ISCSI target portal to current ISCSI with auth pod IP
vm-pvc.yaml in this case user will need to add kube-dns to resolv.conf

The single usage of NFS container under vm-pvc.yaml so we can consider that we do not need it :smile:

test ci

I pulled the work in this branch into #734.
The PVC logic in #734 was left unchanged. My changes just involved simplifying the demo PVCs and stripping out the direct ISCSI volume support in favor of the new pvc logic.

Close in favor of https://github.com/kubevirt/kubevirt/pull/734