feat(remote_wal): impl kafka log store

I hereby agree to the terms of the GreptimeDB CLA
What's changed and what's your intention?

[x] Implemented a ClientManager to manage construction and accesses of rskafka client. Through the manager, the kafka log store is able to contact kafka cluster.
[x] Implemented a KafkaLogStore to provide read/write interfaces for interacting with Kafka remote wal.
[x] Built and initialized KafkaLogStore upon starting the datanode.

TODO

[ ] Integration tests.

Checklist

[x]  I have written the necessary rustdoc comments.
[ ]  I have added the necessary unit tests and integration tests.

Refer to a related PR or issue link (optional)

Had an offline discussion with @WenyXu . We all agree that the split/merge of WAL entries should happen inside logstore. It's the logstore that know the exact log entry size limit and how to handle the split/merge. AFAIK rskafka employs some batch mechanism so that we can simply forwrd wal entries to Kafka records in a 1:1 manner and let rskafka to handle the batching stuff. Only one thing to notice: be careful to handle large single wal entries, split those large entries inside logstore and ensure the splited parts get produced sequentially and atomically.

Splitting a way-too large log entry is in planning. However, it may be implemented in a subsequent PR since we need to investigate how rskafka client handles a way-too large log entry and then design an appropriate splitting strategy.


SBOM via packit v2

SBOM implementation for mod vendor
Resolves https://github.com/paketo-buildpacks/go-mod-vendor/issues/13

Closing in favor of #311


Update README to remove broken instructions

The README currently has instructions for locally linking the package, but doesn't actually work.
This is just an initial step in revamping the documentation for pipeline-editor.  Before continuing we need to consolidate commands into make or npm scripts.
Developer's Certificate of Origin 1.1
   By making a contribution to this project, I certify that:

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

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

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

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

Oh nevermind found the issue: https://github.com/elyra-ai/pipeline-editor/issues/112. Sounds good, we can add the correct info when that's updated.

This is being updated with #116


onBeforeAll and onAfterAll hooks/events are not working on HandlerTypes.ALL

*All hooks and events are not working as expected.
Here it my model;
class Post extends Model {
  get handlers() {
    return [HandlerTypes.ALL];
  }
}

Here is my hooks:
// app/Hooks/Post/onBeforeAll.ts
import { IBeforeAllContext } from "axe-api";

export default async ({}: IBeforeAllContext) => {
};

// app/Hooks/Post/onAfterAll.ts
import { IAfterAllContext } from "axe-api";

export default async ({}: IAfterAllContext) => {

};

Axe API should also have onBeforeSearch and onAfterSearch hooks/events. It is also another bug.


show code lens for properties

It might show wrong signatures until https://github.com/fsharp/FSharp.Compiler.Service/pull/662 is merged.

Released in 2.8.3. Thanks!


🐛 | Wrong configuration path in gpio_control.py

Hi everyone,
First of all, thanks a lot, this is a great project and my kids are already loving it!
I wanted to report a bug I found while enabling GPIO.
I tried installing push buttons to control Play/Pause and VolumeControl following instructions here:
https://github.com/MiczFlor/RPi-Jukebox-RFID/wiki/Using-GPIO-hardware-buttons
and here:
https://github.com/MiczFlor/RPi-Jukebox-RFID/tree/master/components/gpio_control
But when pressing the button, the play/pause wasn't working even though correctly configured in:
~/RPi-Jukebox-RFID/settings/gpio_settings.ini
Then I realized that the script "gpio_control.py" is actually looking for the config file somewhere else:
pi@raspberrypi:~/RPi-Jukebox-RFID $ cat components/gpio_control/gpio_control.py | grep "gpio_settings.ini"
    config_path = os.path.expanduser('~/.config/phoniebox/gpio_settings.ini')

I could fixed the issue by copying the gpio_settings.ini into the .config/phoniebox directory and then gpio controls were working. I looked around and there are indeed two config files (third one being an example):
pi@raspberrypi:~/RPi-Jukebox-RFID $ sudo find / -name "gpio_settings.ini"
/home/pi/.config/phoniebox/gpio_settings.ini
/home/pi/RPi-Jukebox-RFID/settings/gpio_settings.ini
/home/pi/RPi-Jukebox-RFID/components/gpio_control/example_configs/gpio_settings.ini

We could fix the bug by correcting the path in gpio_control.py (but I don't know if there are other dependencies in the code) or by updating the documentation (cf two links above).
Thanks and cheers
Romain
PS: I'm using the 2.1.1 Master version

Hi,
sorry for getting confused by not up-to-date docs.
This is already adressed in #1096 and fixed in develop branch.
We will release 2.2 soon, which includes this fix. In the meantime you could manually add the changes made in develop branch to your local installation.
I close this issue as duplicate.

Thanks for the info, I had missed #1096  while looking for existing issues about this bug.


Cache DNS Lookup

Some of our components require hundreds of secrets to be collected from vault at once.   (We have hundreds of services requesting hundreds of secrets in bursts sometimes)  This was actually hitting limits of our DNS resolver.   Simply doing the lookup once, and then use it for consequent http requests would solve this.

I think we should solve this at the OS level.    There is probably a caching parameter that we're missing.
However,  I think we could reduce the amount of requests that we send for a secrets file by only requesting each path once.    I'll file a new issue for that.

However, I think we could reduce the amount of requests that we send for a secrets file by only requesting each path once.

Do you mean for each "store" (like e.g. secret/sentry)? That's a good idea.

@rkrzr yes!  exactly.

In our production setup, we currently cache DNS lookups on the host level. This is IMO the better place for this: vaultenv shouldn't attempt to cache this. Otherwise it would explicitly override the TTLs for the vault.service.consul DNS record, which I'd call unexpected behavior.
For any other people running into this: you can configure consul to set a TTL on the vault service definition. The rest should be handled automatically. Config snippet:
{
  "dns_config": {
    "service_ttl": {
      "vault": "5s"
    }
  }
}

More info here: https://www.consul.io/docs/guides/dns-cache.html

In our production setup, we currently cache DNS lookups on the host level.

The title of this issue is a bit misleading. We still don't cache any DNS lookups, what we instead do, is get every secret (which can contain multiple key-value pairs) only once, instead of for every key.
The associated PR is clearer: https://github.com/channable/vaultenv/pull/27

We still don't cache any DNS lookups, what we instead do, is get every secret (which can contain multiple key-value pairs) only once, instead of for every key.

Both statements are true. We do the DNS caching, but dnsmasq takes care of it, not vaultenv :smile:


Add go.mod

@sbinq, do you have the rights to merge this PR?

@sbinq, do you have the rights to merge this PR?

seems no :)

@mkorolyov , could you merge this PR?


add hostname in upload notifications

For issue #502

Codecov Report

Merging #596 into master will increase coverage by 0.29%.
The diff coverage is 75.00%.

@@            Coverage Diff             @@
##           master     #596      +/-   ##
==========================================
+ Coverage   51.35%   51.65%   +0.29%
==========================================
  Files          93       94       +1
  Lines        3139     3177      +38
==========================================
+ Hits         1612     1641      +29
- Misses       1181     1185       +4
- Partials      346      351       +5

Impacted Files
Coverage Δ

pkg/config/pipeline.go
57.69% <ø> (ø)

pkg/transfers/pipeline/notify/mock_sender.go
0.00% <0.00%> (ø)

pkg/upload/agent.go
0.00% <ø> (ø)

pkg/upload/mock_agent.go
0.00% <0.00%> (ø)

pkg/transfers/pipeline/aggregate.go
13.01% <100.00%> (+4.73%)
:arrow_up:

pkg/transfers/pipeline/notify/email.go
64.06% <100.00%> (+1.15%)
:arrow_up:

pkg/transfers/pipeline/notify/slack.go
75.00% <100.00%> (+7.00%)
:arrow_up:

pkg/upload/ftp.go
54.54% <100.00%> (+3.05%)
:arrow_up:

pkg/upload/sftp.go
64.64% <100.00%> (-1.06%)
:arrow_down:

... and 2 more

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 613e2cd...7db4703. Read the comment docs.

New Slack message:
New email template:
A file has been uploaded to ftp.bank.com:3294 - 20200529-131400.ach
Name: Moov
Debits:  $105.00
Credits: $0.00

Batches: 1
Total Entries: 1


The URLs in published Maven Central is wrong

https://search.maven.org/artifact/com.google.cloud/google-cloud-websecurityscanner/2.4.0/jar

We still have the <url> elements in the individual modules. That's the reaosn.


The build is failing

The last successful build was 11 months ago with 64ef021e7cf66f1639c5e44b52aedbbfcf48d587.
I'd like to submit a pull request to update the lodash dependency, but I'm new to the project and I don't know how to test my changes against a broken build.

Tests are passing again.


fix(api): v4 only

@anfibiacreativa Video showing Codespaces APIv4 working for this PR. What else would you like to see in terms of verifying the PR?
https://youtu.be/0V3eyH_5mpA

Manually testing in dev container

Comment out Stripe in azure.yaml and docker-compose.yml.

Start local services in dev container

npm run start:services

Rename env file for Azure Functions

mv ./packages/api/local.settings.sample.json ./packages/api/local.settings.json

In new dev container terminal, Build API

npm run start:host --workspace=api

Output should include APIs such as:
Azure Functions Core Tools
Core Tools Version:       4.0.5455 Commit hash: N/A  (64-bit)
Function Runtime Version: <IP_ADDRESS>54

[2024-01-04T16:49:20.258Z] Worker process started and initialized.

Functions:

        delete-favorites: [DELETE] http://localhost:7071/api/favorites

        get-favorites: [GET] http://localhost:7071/api/favorites

        get-listings: [GET] http://localhost:7071/api/listings

        get-listings-by-id: [GET] http://localhost:7071/api/listings/{id}

        get-openapi: [GET] http://localhost:7071/api/{filename?}

        get-payments: [GET] http://localhost:7071/api/payments

        get-payments-by-id: [GET] http://localhost:7071/api/payments/{id}

        get-reservations: [GET] http://localhost:7071/api/reservations

        get-reservations-by-id: [GET] http://localhost:7071/api/reservations/{id}

        get-users: [GET] http://localhost:7071/api/users

        get-users-by-id: [GET] http://localhost:7071/api/users/{id}

        patch-reservations-by-id: [PATCH] http://localhost:7071/api/reservations/{id}

        post-checkout: [POST] http://localhost:7071/api/checkout

        post-favorites: [POST] http://localhost:7071/api/favorites

        post-payment: [POST] http://localhost:7071/api/payments

        post-users: [POST] http://localhost:7071/api/users

In new dev container terminal, use cURL to get all listings from PostgreSQL.

curl "http://localhost:7071/api/listings" --verbose

This proves that the API can successfully talk to PostgreSQL.

Use cURL to favorite a listing for a fake user, adding favorite into MongoDB.

curl -X POST http://localhost:7071/api/favorites -H "Content-Type: application/json" -d '{"listing": {"id": "1"}, "user": {"id": "123"}}' --verbose

Use cURL to get all favorited items for fake user, querying into MongoDB.

curl "http://localhost:7071/api/favorites?userId=123" --verbose

Closing in favor of #424

#please-close


Typographical Error.

There is a typo on one of the documentation page.
Please take a comment to try out the mutations and queries specified in the GraphQL explorer, or if you're using Rest, make API calls with a Rest client such as Postman or Insomnia.
Congratulations, you've successfully run your first Tensei app 🎉.

Thanks @chuxmykel for sharing this. Will push a fix soon.

Lol. Welcome. I initially used my private account and I had to switch. That's why I opened the issue twice. Apologies.


istio 1.6.3 imagePullSecrets prometheus Not in force

istio 1.6.3 imagePullSecrets prometheus Not in force
[ ] Configuration Infrastructure
[ ] Docs
[X] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
set global.imagePullSecrets,promethrus deployment use imagePullSecrets
Steps to reproduce the bug

create istioOperator, reference demo,update  global.imagePullSecrets、 global.hub、prometheus.hub
set global.imagePullSecrets
implement istioctl install -f ***.yaml
prometheus can not install

Version (include the output of istioctl version --remote and kubectl version and helm version if you used Helm)
1.6.3
How was Istio installed?
istioctl  use custom configuration
istioctl install -f ***.yaml
Environment where bug was observed (cloud vendor, OS, etc)
centos

Do you mean the global.imagePullSecret doesn't work for prometheus deployment?

您是说global.imagePullSecrets不适用于Prometheus部署吗？

yes
prometheus charts  deployment.yaml not use global.imagePullSecrets

Please append your istioOperator yaml that is used to deploy istio.


could not determine kind of name for C.Mix_OpenAudioDevice

Ubuntu 16.04 try to use this example:
https://github.com/veandco/go-sdl2-examples/tree/4298a5894b3ec15532eb78de70c71a528095865e/examples/audio_wav_memory
and get an error:
# github.com/veandco/go-sdl2/mix
../extgo/src/github.com/veandco/go-sdl2/mix/sdl_mixer.go:162:5: could not determine kind of name for C.Mix_OpenAudioDevice

How can I fix it?

I read in the source that: Mix_OpenAudioDevice is not supported before SDL 2.0.9
I am installed this, my sdl2-config --version shows: 2.0.9 but this error still exists.
Any ideas?

I read in the source that: Mix_OpenAudioDevice is not supported before SDL 2.0.9
I am installed this, my sdl2-config --version shows: 2.0.9 but this error still exists.
Any ideas?

Hi @EugeneGlevsky, what does pkg-config --cflags sdl2 print? Could you use the printed include directory to find SDL_version.h and verify that the version is at least 2.0.9?

Hi @EugeneGlevsky, what does pkg-config --cflags sdl2 print? Could you use the printed include directory to find SDL_version.h and verify that the version is at least 2.0.9?

pkg-config --cflags sdl2  return: -D_REENTRANT -I/usr/include/SDL2  I fount SDL_version.h and open, in the source I can see:
SDL_MAJOR_VERSION   2  SDL_PATCHLEVEL          9
But if I try to run: sudo apt install libsdl2-mixer-dev  it says:
is already the newest version (2.0.1+dfsg1-1)
In the current moment I comment this function in source code and it is compiling without errors and working. But I want to fix it without this way.

pkg-config --cflags sdl2  return: -D_REENTRANT -I/usr/include/SDL2  I fount SDL_version.h and open, in the source I can see:
SDL_MAJOR_VERSION   2  SDL_PATCHLEVEL          9
But if I try to run: sudo apt install libsdl2-mixer-dev  it says:
is already the newest version (2.0.1+dfsg1-1)
In the current moment I comment this function in source code and it is compiling without errors and working. But I want to fix it without this way.

Ahh sorry, it appears the message was wrong. It needs SDL2_mixer 2.0.2. I will correct it right away.

Ahh sorry, it appears the message was wrong. It needs SDL2_mixer 2.0.2. I will correct it right away.

@veeableful Thank you so much. I make pull in my version from git and not I can see this:
# github.com/veandco/go-sdl2/mix ../extgo/src/github.com/veandco/go-sdl2/mix/sdl_mixer.go:28:32: error: missing binary operator before token "("  //#if !(SDL_MIXER_VERSION_ATLEAST(2,0,2))

@veeableful Thank you so much. I make pull in my version from git and not I can see this:
# github.com/veandco/go-sdl2/mix ../extgo/src/github.com/veandco/go-sdl2/mix/sdl_mixer.go:28:32: error: missing binary operator before token "("  //#if !(SDL_MIXER_VERSION_ATLEAST(2,0,2))

Hmm perhaps the Go version is out of date. Could you tell me which version is your Go toolchain?

Hmm perhaps the Go version is out of date. Could you tell me which version is your Go toolchain?

go version go1.15.6 linux/amd64

go version go1.15.6 linux/amd64

Ahh I think it must be the gcc version then. Could you try installing newer GCC? I'm using GCC 10 and the problem doesn't appear to me.

Ahh I think it must be the gcc version then. Could you try installing newer GCC? I'm using GCC 10 and the problem doesn't appear to me.

Ok, thank you so much!

Ok, thank you so much!


Contributions from the Open Source R&D team in Bristol and London

Code from the PoC in 2023 of a Financial Health Check bot that uses a Graph DB to frame and manage the interaction flow with an LLM. Relevant to the InferGPT project goals so contributing this in case it is of use in the development of InferGPT or derivatives. Code is submitted with a permissive MIT License - parts are based on original Code by Chris Booth hence naming CB and OC on the License file. Whilst others were involved have just named OC and CB for simplicity. If others involved in the original PoC would like to be named they can contact one of us - happy to add them to the documentation of the repository. Further background on the Graph approach used can be found in this blog: https://blog.scottlogic.com/2023/07/26/how-we-de-risked-a-genai-chatbot.html Some of the Architectural challenges and risks the PoC set out to mitigate are discussed here: https://blog.scottlogic.com/2023/05/04/generative-ai-solution-architecture.html

Apologies if this will mess up the simplicity of the upstream InferGPT repo! If you want to reject this and create a fork I can put it there instead?

Also spotted a typo with your license file.


Stream settings hidden menu overflow

Minimize window to mobile width (365px)
Press the hamburger menu icon to open list of streams
Press the down chevron to the right of any stream's name

The stream settings that extend after pressing the down chevron should overflow and be hidden outside of the window's view as seen in the picture.

Thanks for the bug report @joyhchen!  I updated the title to not have "Quality assurance" in it and added labels.  It's not totally clear to me what the right solution is here.
@brockwhittaker FYI.

@joyhchen can you elaborate on what you mean by "[the stream settings] should overflow and be hidden outside of the window's view as seen in the picture."
I'm not fully sure what that entails, as to me it sounds like cutting off the settings to a place where it is completely inaccessible/off screen on mobile.
I think this is a good bug though, and perhaps the direction to go with this is some kind of overlay.

@brockwhittaker Ah, sorry my wording was a bit confusing. Yup, I meant to say that the rest of the settings menu is inaccessible because the overflow of the menu isn't visible.

Thanks, @joyhchen. I see what you mean. I think we'll just have to fix this when we fix popover design in general and come up with a mobile solution that perhaps uses overlays.

This feels like a bite-size issue for someone who knows CSS well.

It seems that the user interface now changed and this problem in no longer present in and updated UI. Should this bug be closed?

Yeah, agreed, closing.


IGNITE-14221: hide SQL Constants when IGNITE_TO_STRING_INCLUDE_SENSITIVE=false

Currently, if some user passes values in query body (e.g SELECT * FROM CARDS WHERE CARD_NUM = '123') then values available via query history view and events (EVT_SQL_QUERY_EXECUTION) event if IGNITE_TO_STRING_INCLUDE_SENSITIVE=false.
This PR fixes the above scenario and replaces values with a question mark when logging a query.
The Contribution Checklist

[ ] There is a single JIRA ticket related to the pull request.
[ ] The web-link to the pull request is attached to the JIRA ticket.
[ ] The JIRA ticket has the Patch Available state.
[ ] The pull request body describes changes that have been made.
The description explains WHAT and WHY was made instead of HOW.
[ ] The pull request title is treated as the final commit message.
The following pattern must be used: IGNITE-XXXX Change summary where XXXX - number of JIRA issue.
[ ] A reviewer has been mentioned through the JIRA comments
(see the Maintainers list)
[ ] The pull request has been checked by the Teamcity Bot and
the green visa attached to the JIRA ticket (see TC.Bot: Check PR)

Notes

How to Contribute
Coding abbreviation rules
Coding Guidelines
Apache Ignite Teamcity Bot

If you need any help, please email<EMAIL_ADDRESS>or ask anу advice on http://asf.slack.com #ignite channel.

Hello, @alex-plekhanov
Can you, please, take a look at my changes?

Hello, @zstan
Can you, please, take a look at my changes?


Hide keyboard when opening drawer?

I have a text input in my center view. I'd like to hide the keyboard when the drawer is opened. Is there a delegate method that is called when the drawer is opened to resign first responder or another way to achieve this?

@kcharwood the drawerVisualStateBlock is called in UIGestureRecognizerStateChanged, after self.view.userInteractionEnabled = NO so it is late. It does not work
Here is my code
[self setDrawerVisualStateBlock:^(MMDrawerController *drawerController, MMDrawerSide drawerSide, CGFloat percentVisible) {
        [[UIApplication sharedApplication] sendAction:@selector(resignFirstResponder) to:nil from:nil forEvent:nil];
    }];


Ability to disable audio CAPTCHA

I understand the audio CAPTCHA is used for accessibility for the visually impaired. However, it renders a CAPTCHA 100% ineffective because of how good speech recognition software has gotten.
Can there be a way to turn this off? If not, there is no point to using the CAPTCHA anymore.

+1 but considering Google's culture, I wouldn't expect them to want to associate their brand with such a move that could easily be twisted into making them look bad with the SJW's

I'm updating the v1.2 branch to support the v3 API for reCAPTCHA which doesn't show challenges, just returns a confidence score for the user. This might help you. You can see a demo at: https://recaptcha-demo.appspot.com/recaptcha-v3-request-scores.php


Release 2.0.0

No, thanks for the reminder. I'll get it done this week

Thank you! 🎉

2.0.0 is available in maven central. @erikghonyan thanks again for all the contributions!


provide warning message when kapp delete didnt find given app

Currently it just says "Succeeded" when deletion is a noop (tries to be idempotent by default). @DTTerastar thinks warning would be a nice indicator.

fixed in https://github.com/k14s/kapp/commit/673156bbd9692e1e00b05f25b259f2ae70127f2f


Incorrect height when browser zoom is more than 100% and textarea width is 100%

How to reproduce:

open google chrome
increase your browser zoom
open http://jsfiddle.net/2ph33L0t/12/
change browser size so that last line of textarea has only one word - "dapibus_"
refresh the page

Result: word "dapibus_" is not visible.
This is a very dangerous bug as not visible text can mislead users, what can result in very serious consequences.

Archiving repository.


Extracting metadata for images on ChangeData event?

Question
Is there a way I can get an images size/format (eg width x height/ png or jpeg from first buffer segment ) when ChangeData event occurs from web_sys or wasm_bindgen structs?
What I've tried
Currently I am thinking of trying ~from_serde~ into_serde from wasm-bindgen as it may be available but need to research how the struct should look like.  What I am nervous about though is the warning given by this function on large content.
Currently I convert to base64 and assign to image element and have js script alter onload but this is an issue when re-rendering js content.
pub fn canvas_area(&self) -> Html {
    let mut data_url = "data:image/png;base64,".to_string();
    data_url.push_str(&base64::encode(self.props.image.raw.as_ref().unwrap()));

    html! {
        <>
        <img class="hide" id="original_content" src={ data_url } />
        <canvas id="edit_content" >
        </canvas>
        <script>
          {"
            let img = document.getElementById('original_content');

            // we want to wait for our image to load
            img.onload = () => {
                let canvas = document.getElementById('edit_content');
                // let's store the width and height of our image
                const inputWidth = img.naturalWidth;
                const inputHeight = img.naturalHeight;
                // blah blah do stuff here
            };

            // start loading our image
            img.src = img.src;
          "}
        </script>
        </>
    }
}

Hey @webbrandon! I don't ChangeData is what you want, it's only used for input, select, and textarea elements.
What we need here is support for listening to load and error events for img elements. Then, you could add a callback in Yew along with a Ref to check the image dimensions. Would you be interested in adding this?

Yes.  I will begin to dig into the this.  Thank you for the direction.

@webbrandon would you be interested in writing up an issue for this? I think we could discuss more there. I have some uncertainty about browser support for this feature since it seems that onload is not in the spec anymore? https://developer.mozilla.org/en-US/docs/Archive/Mozilla/XUL/Attribute/image.onload

Interesting. Yes let me grab some context around the specs and will post a topic.

@webbrandon someone just wrote up a feature request for onerror: https://github.com/yewstack/yew/issues/1232. I think we will still need one for onload though.

I was wrong about onload. I think mdn just removed the old img-specific page in favour of the GlobalEventHandlers mixin one: https://developer.mozilla.org/en-US/docs/Web/API/GlobalEventHandlers/onload

I don't think we need a separate issue for onload. Let's move forward with #1232

Awesome just had time to start looking and seems you got what you need! Reviewing commits now.

https://html.spec.whatwg.org/multipage/media.html#event-media-loadedmetadata
Thank you @jstarry for guiding this ship.


Update Clinical Significance values for Predisposing evidence to use ACMG values

Currently we use Positive and Negative for Clinical Significance values but it would make more sense to use the nomenclature from the ACMG guidelines for Predictive evidence:
https://www.acmg.net/docs/Standards_Guidelines_for_the_Interpretation_of_Sequence_Variants.pdf
From Table 5:

Pathogenic
Likely pathogenic
Benign
Likely benign
Uncertain significance

@malachig This is straightforward to do on our end! Is there a conversion we can use for our existing values? Or should we introduce these new values alongside our existing ones?

Lets just set the few we have to NULL and we will manually select the right answer for each and have these reviewed by editors.

(notes for myself)
This will be a little tricky as its stored as an enum, so we can't just remove these values right off the bat. We'll need to add the new ones at the end of the enum and overwrite the existing values to null. Then in a subsequent migration we'll want to left shift everything by two and delete the old values in the middle of the enum.

This sounds like a good approach to me.


[Bug] There are cases where storage cannot be canceled normally.

[REQUIRED] Please fill in the following fields:

Unity editor version: 2021.3.14f1
Firebase Unity SDK version: 10.1.1
Source you installed the SDK: .unitypackage (.unitypackage or Unity Package Manager)
Problematic Firebase Component: Storage (Auth, Database, etc.)
Other Firebase Components in use: Auth, Analytics, FireStore, Installation, CloudFunction (Auth, Database, etc.)
Additional SDKs you are using: Facebook, AdMob (Facebook, AdMob, etc.)
Platform you are using the Unity editor on: Mac, Windows (Mac, Windows, or Linux)
Platform you are targeting: Android, iOS (iOS, Android, and/or desktop)
Scripting Runtime: IL2CPP (Mono, and/or IL2CPP)
Pre-built SDK from the website or open-source from this repo: Pre-built

[REQUIRED] Please describe the issue here:
(Please list the full steps to reproduce the issue. Include device logs, Unity logs, and stack traces if available.)
We are using Admob plugin together.
Rarely, there was a case where a crash occurred while exposing Admob ads while transferring to Firebase storage. (I think the crash shouldn't happen)
Anyway, to solve this problem, I canceled the upload task right before displaying AdMob ads using CancellationToken.
But the following error occurs.

I suspect that there is no controller, but I am not sure, so I ask a question.

When I canceled, I also found the following log.
[Firebase]InnerException:System.AggregateException: One or more errors occurred. ("_p_firebase__storage__MonitorController" has been disposed)

It's on the Firestore side, but I wonder if it's not related to pull request
Steps to reproduce:
Have you been able to reproduce this issue with just the Firebase Unity quickstarts (this GitHub project)? No
What's the issue repro rate? Rarely reproduced.
It happens when canceling while transferring to storage, and it seems to happen when canceling a little late.
Relevant Code:
        storageCts_ = new CancellationTokenSource();
        CancellationToken cToken = storageCts_.Token;

        var uploadRef = reference.Child(remotePath_);
        uploadRef.PutFileAsync(localPath_, metaData, progressMonitor, cToken).ContinueWithOnMainThread(HandleTask);

It is called as below just before viewing the ad.
        if (storageCts_ == null || storageCts_.IsCancellationRequested)
        {
            return;
        }

        storageCts_.Cancel();

Hi @henry-dreamo,
Thanks for reporting this. I'd like to clarify a few things:

Version 10.3.0 of the SDK has been released recently. Could you try it out and see if the issue persists?
Could you also share the list of devices that also experience this behavior?
Does the crash persist when you remove Admob from your implementation?

Thanks for the update, @henry-dreamo. While waiting for the results of your update, would it be possible for you to provide a minimal, reproducible example of your implementation so that we can identify what's causing this behavior?

I will make a reproducible sample project when I have time.
First of all, please note that I will attach the stack where this crash appears.

Thanks for sharing this information, @henry-dreamo. I look forward to the results of your update.

Yes. It will be updated to a new version tomorrow.
I'll check again after the update.
I'm leaving a comment so this issue doesn't close.

@paulinon
We updated the app version by updating the Firebase version.
Exception handling was also performed to check if UploadTask was terminated.
The issue has been resolved.


[Negative progress] <as proxy for conditional formatting - databars like excel>

Description
Excel has this super cool data bars from conditional formatting..

which also supports negative vlaues..

Your progress bar is a wonderful proxy to achieve same affect (give a plain markdown file, a python snippet could update the numbers column with  your proxy as conditional formatting databars).. if you could also support negative numbers, that need would be much more complete.. Kindly consider..
Tech notes
No response
Screenshots or Diagrams
No response
Additional information
No response

@guibranco Sorry for the delay. Kindly find an example below. For below numbers in excel, I get below conditional formatting.

I would like to get this effect for markdown table

@guibranco excel achieves this, based on entire value of data selected (for ex, if I selected just 100 to 0 in above cells, the bar would have been different)..
However, each proxy bar's is independent, unaware of any data outside, so to achieve similar effect, yes, we should be able to set the center (or origin or start) of progress from middle or from left.
A. If origin set as middle, and negative value given, the progress emanates from middle to left..
B. If origin set as middle, and positive value given, the progress emanates from middle to right..
C. If origin set as left, and negative value given, it can be NA or absolute (negative negated)
D. If origin set as left, and positive value given, current behavior
E. If origin set as right, and negative or positive value given, NA or can also be absolute, negating sign, but progress from right to left
I request A and B. Others, can be, as you see fit.


add ARGB4444 texture format support

certain texturelist.xml contain texture files that use their format as ARGB4444 (uncompressed?) can't be extracted, resulting in this error when reaching the 'tex' folder for that ifs (geo and afp folders extract fine, along with cversion and magic files):
  File "ifstools_bin.py", line 3, in <module>
  File "ifstools\ifstools.py", line 79, in main
  File "ifstools\ifstools.py", line 30, in extract
  File "ifstools\ifs.py", line 156, in extract
  File "ifstools\handlers\ImageFile.py", line 46, in extract
  File "ifstools\handlers\GenericFile.py", line 19, in extract
  File "ifstools\handlers\GenericFile.py", line 25, in load
  File "ifstools\handlers\ImageFile.py", line 85, in _load_from_ifs
NotImplementedError: Unknown format argb4444
[82496] Failed to execute script ifstools_bin

unsure if this can be fixed or implemented into the program for unpacking & repacking, but if it can that'd be fantastic

Email the broken IFS to<EMAIL_ADDRESS>and I'll fix it within 3 weeks (travelling rn). The new format should be trivial to implement, thanks for letting me know it exists.

Fixed with v1.5. Repacking isn't supported properly, so repacking will convert to argb8888.


Add support for specifying app version externally

Ideas here:

https://gist.github.com/TheHippo/7e4d9ec4b7ed4c0d7a39839e6800cc16
https://www.reddit.com/r/golang/comments/4cpi2y/question_where_to_keep_the_version_number_of_a_go/
https://stackoverflow.com/a/15217754/903870

Loose notes:

Multiple tags in the repo
Need to be able to programatically generate a "version" string that represents the current state of the code that a binary is being built from

This means that if we're 1:1 with the most recent tag in the Git repo, we should use that tag (e.g., v0.3.2) as our version string. Since Semantic Versioning (and go modules) dictate having a v prefix, we should probably include that also.
If we're not 1:1 with the most recent tag, we should preferably use some modified version of the string that represents the closest tag and the current commit hash, potentially even the current branch name.

Thankfully, git describe already gives us most of what we're after.
From current master branch:
$ git describe --always --long --tags
v0.3.2-3-gcc6f0e5

From the current (unsquashed, messy) feature branch for #3:
$ git describe --always --long --tags
v0.3.2-22-g0d49111

This tells us:

The latest tag is v0.3.2
We are 22 commits since that tag
The latest commit is 0d49111

I don't (yet) recall where the g prefix for the hash comes from.

More examples.

$ git checkout v0.3.2
Note: switching to 'v0.3.2'.

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by switching back to a branch.

If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -c with the switch command. Example:

  git switch -c <new-branch-name>

Or undo this operation with:

  git switch -

Turn off this advice by setting config variable advice.detachedHead to false

HEAD is now at 64d5533 Merge pull request #51 from atc0005/i50-prep-for-v0.3.2-release

From there, we can see that we're 1:1 with the chosen v0.3.2 tag:
((v0.3.2))$ git describe --always --tags
v0.3.2

((v0.3.2))$ git describe --always --long --tags
v0.3.2-0-g64d5533

As the official docs indicate, this tells us that we're 0 commits away from the v0.3.2 tag.

Just finished with #55. Our results are a little different now; we now get the same output regardless of whether we opt to display lightweight tags also via the --tags flag (since we only have annotated tags now):
$ git describe --always --long
v0.3.2-3-gcc6f0e5

$ git describe --always --long --tags
v0.3.2-3-gcc6f0e5

I don't (yet) recall where the g prefix for the hash comes from.

From https://git-scm.com/docs/git-describe:

The hash suffix is "-g" + unambiguous abbreviation for the tip commit of parent (which was 2414721b194453f058079d897d13c4e377f92dc6). The "g" prefix stands for "git" and is used to allow describing the version of a software depending on the SCM the software is managed with. This is useful in an environment where people may use different SCMs.

Looks like the --dirty flag could come in handy:
$ touch hello.txt

$ git describe --always --long --dirty
v0.3.2-22-g0d49111

$ git add hello.txt
$ git describe --always --long --dirty
v0.3.2-22-g0d49111-dirty

Would be a useful way to tell that the resulting binary was not cleanly built from 100% committed code.

Now that I've read over the details, I'm going to end up going with the output from git describe --always --long --dirty since i think it gives enough useful information without cluttering up the version string too much.

I have this stubbed out and placed into a separate feature branch. Once #3 is completed and lands in master I'll rebase the feature branch and then get this done and in for the v0.4.0 milestone.


Cannot set initial month

I want the input blank on page load but for the initial calendar to begin six months from today, but it defaults to the current month and it seems there is no method to alter this.
I can get it to have October as the initial view rather than this month (April) but to achieve this I have to setDate to October, but I don't want an initial date set, just the initial view to be October
Thanks

Just use the setDate to change the default date.

I tried that but it sets the date in the input which I did not want. I've got past that now by clearing the input after calling setDate(). Thx for the reply


Multitable Joins in Spark returns no data

When building a workspace in Spark, it was noticed that a four table join query returned no results, although the same query in MongoDB with similar data works just fine.
To use the following query, you need to first open the /loc/customers.data file and then add a dropdown from the Setup Form card and display it. Mirror this deck and add a Query card with the following SQL Query.
SELECT
  o.order_key AS OrderKey,
  o.purchase_date AS PurchaseDate,
  c.last_name || ", " || c.first_name AS Name,
  c.city AS City,
  c.state AS State,
  i.clothing_size AS Size,
  i.clothing_type AS Type,
  i.clothing_color AS Color,
  i.price AS UnitPrice,
  oi.qty AS Quantity,
  i.price * oi.qty AS PricePaid,
  SUM(i.price * oi.qty) AS TotalPaid,
  i.clothing_brand AS Brand
FROM
  :results AS c
JOIN `/loc/orders.data` AS o
ON c.customer_key = o.customer_key
JOIN `/loc/ordered_items.data` AS oi
ON oi.order_key = o.order_key
JOIN `/loc/inventory_items.data` AS i
ON i.item_key = oi.item_key
GROUP BY
  i.price * oi.qty

If you limit this query to join on just the :results of /loc/customer.data and /loc/orders.data you will get the expected results.

A 2-way join between Orders and Customers did return data - so hopefully this turns out to be low effort. Hence scheduled for May, since it would be good to be able to demo this functionality at Spark Summit 2017.

This may be a qscript issue. There is a know qscript bug (@wemrysi is working on it now for #1929) where we fail to align join provenance with other related provenance. Given how we use the results of joins in this query, we may be hitting that bug.

This issue is solved on backend side. There are separate issues on the frontend that effect this functionality (and some other demo workspaces), but issue for it is already opened on slamdata repo https://github.com/slamdata/slamdata/issues/2142


Pass route params to async data?

Is this possible (like in Nuxt), or am I doing it wrong? Trying to accomplish something like this:
// pages/[id].vue

<template>

Post {{ id }}

{{ post }}

</template>

<saber>
import axios from 'axios'

export default {
  params() {
    return {
      id: this.params.id // or this.$route.params.id ??
    }
  },

  async data() {
    const { data: post } = await axios
      .get(`https://jsonplaceholder.typicode.com/posts/${this.$route.params.id}`)
    return {
      post
    }
  }
}
</saber>

Getting the following error

It looks like in the Docs you're providing a hard coded array of params.

The params method in <saber> block is used to get actual URLs. Just like https://nuxtjs.org/api/configuration-generate#routes but here we only need the params part.

A potential solution is to allow adding routes programmatically so you won't need dynamic routes:
saber.node.js:
export const routes = async () => {
  const posts = await getPosts()
  return posts.map(post => ({
    path: `/post/${post.id}`,
    component: 'components/post.vue'
  }))
}

@fb-erik currently the data generated by <saber> block is directly injected to the component's data() method at compile time, so we won't be able to know the exact params that's been used at that moment. We can just inject the data for all params to component's data() method but that sounds a bit silly 😂

I see, so the only solution to pre-fetching data for dynamic routes is by using router.addRoutes in saber.app.js and passing in data using props?

Trying to do a similar thing.. saber.app.js doesn't appear to like async OR the below just doesn't appear to work. should it?
saber.app.js
import axios from "axios";
export default async ({ router }) => {
  const { data } = await axios.get("/sitemap-api");
  const routes = data.map(post => ({
    path: post.url,
    component: `templates/${post.template}`,
    props: { post },
  }));
  router.addRoutes(routes);
};

@samburgers it does not sound like a good idea to bootstrap the router after a runtime API call, maybe we can pre-evaluate that piece of code at compile time too, like that in <saber> block:
import posts from 'saber-preload-loader!./get-posts'

// get-posts.js
// runs by a webpack loader
module.exports = () => axios.get('./...').then(res => res.data)

what you explained in the below comment is exactly what i need to do... ie hit an API and grab all the routes/urls, and then inside the component have the ability to query the API again for further API calls (post content, related API calls etc)
https://github.com/egoist/saber.js/issues/6#issuecomment-391779007
but adding that to saber.node.js appears to have no affect on the build/pages.

@samburgers yep that's still a proposal

the <saber> tag also seems to introduce some poor developer-experience (breaks editor things like prettier, linters, syntax highlights etc). I would be a big fan of moving everything to saber.node.js like your comment above, which would open up further control over multiple content sources too.

@samburgers Here's also an alternative experimental, undocumented API to bring <saber> block as a .js file: https://github.com/egoist/saber.js/blob/master/lib/webpack/saber-js-loader.js
So you can do this in a vue component:

Basically you can write everything that <saber> block supported in .saber.js too


Not generating the language files anymore

What is your Quests version / build number (do not say "latest")?
3.9.1-b86
Describe the problem and how to replicate it. Any console errors to post using Pastebin.com?
Make a new server, install the plugin Quests, and check if the language files were created.

https://github.com/PikaMug/Quests/wiki/Casual-‐-Translations#troubleshooting

I did the troubleshooting, still it doesn't appear, despite the data and modules files being generated

@Katalijst In that case, try creating a the lang folder manually and downloading your language from https://github.com/PikaMug/Quests/tree/master/main/src/main/resources/lang
For French, path should look like /Quests/lang/fr-FR/strings.yml and in your config.yml, set language: fr-FR


Why does the linux-musl platform cross compile have a dependency on rubygems 3.3.22?

Hi,
In my Github actions workflow when I cross compile with x86_64-linux-musl platform specified in my matrix using  oxidize-rb/cross-gem-action@main it produces a gem that has a dependency on ruby gems 3.3.22
None of the other platform gems that are built have this dependency. In my app it uses ruby 2.7.8 with rubygems 3.1.6 - so the bundle install fails on the remote server with:
dedup_csv-0.1.8-x86_64-linux-musl requires rubygems version >= 3.3.22, which is
incompatible with the current version, 3.1.6

Is there a way to not have this dependency on rubygems 3.3.22?
my gihtub actions workflow:
https://github.com/kingsleyh/dedup_csv/blob/main/.github/workflows/gem.yml

You should use modify your gemspec to use ext/mygem/extconf.rb instead of Cargo.toml (and add rb_sys ~> 0.9 as a gemspec dep. Then you can remove any required_rubygems_version constraint.
https://github.com/kingsleyh/dedup_csv/blob/7dd6bd321af62a0e00e4cc7e6526a05c8a59aaaf/dedup_csv.gemspec#L28C2-L28C49

Ah thanks - I will try that

hmm even though I have no required_rubygems_version in my gemspec even after changing to use extconf.rb and adding rb_sys as a gemspec dependency - the musl gem produced now was a runtime dependency on rb_sys but it still has a rubygems dependency of 3.3.22 - if I don't build the musl gem at all then the remote server picks the linux one instead of the musl and it works.  It's only the musl gem that has this rubygems dependency - maybe its related to the way it's built in docker?
https://rubygems.org/gems/dedup_csv/versions/0.1.12-x86_64-linux-musl

Use RbSys::ExtensionTask instead of Rake::ExtensionTask, that should do it


[FEATURE]: Dashboard for Contributors for adding Hackathons on client side

Detailed description
dashboard to create hackathons on the client side
Context
It will help the contributors to add hackathons
Possible implementation
No response
Additional information
I would like to work on this under gssoc23
Code of Conduct

[X] I agree to follow this project's Code of Conduct

assigned it @suzy-g38


Solve some clippy warnings

To ease review I separated commits by clippy lint rules so it is easier to revert each, but we can squash before merge if necessary.
Special attention to 3729a474719d76caa7977a31ed7b34b5777d034d as that required manual intervention. The others were automatic.
About #4169 - I suggest adding to CI the lints that everyone agrees that should be an error (which can be an empty set right now). Then we can fix a category and enable on CI incrementally.

Something that caught my attention on this commit:
Dropping the &GasCounter does nothing, so clippy seems right and I changed it, but then I was curious about the original intention. It is used as a function parameter down some lines, so it looks like gas is counted here. Should I remove the comment as well?

Thanks a lot for this, it's very appreciated

Right after we merge this PR, I suggest we start enforcing Clippy in the CI (minus some annoying lints, possibly). Something like cargo clippy -- -A clippy::len_zero -A clippy::match_like_matches_macro.

By the way @shyba, how were you able to isolate fixes for specific lints so cleanly? Does Clippy have some feature like --fix --but-only-for-this-lint? Or did you apply fixes manually?

By the way @shyba, how were you able to isolate fixes for specific lints so cleanly? Does Clippy have some feature like --fix --but-only-for-this-lint? Or did you apply fixes manually?

Most were automatic. I used some bash commands.
cargo clippy -- -W clippy::all 2>&1 | grep github | sed -e ' s/ //g' | sort | uniq -c | sort -h shows the lint rules sorted by frequency.
cargo clippy --fix --allow-dirty -- -A clippy::all -W  clippy::manual_str_repeat will try to fix only the named lint.
I can imagine a bash script for that, but I was reviewing the changes while messing with mold and sccache so I went ahead to run those manually and compare times. Using codegen-units=1024,  mold and sccache building is way faster and takes between 600-2000mb of RAM instead of what it was taking before. I found that might be useful for development.

Rebased, fixed lint as well.

LGTM, will leave approval to @neysofu
@shyba if you do decide to add anything else please do so in a separate PR. Once again, thanks for submitting this PR 🙏

@mangas Thanks!
Rebased, but had to change two commits during it. clippy::map_clone  and clippy::redundant_closure

@neysofu @shyba we should probably enable the clippy checks on CI after this

Agreed. I'll review today, and after merging let's figure out a strategy for CI linting.


Download Log Docs Are Broken

Going to https://docs.microsoft.com/en-us/rest/api/azure/devops/audit/download log?view=azure-devops-rest-5.1 it appears the docs are broken/non-existent.

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

ID: 3dff67ec-67ca-b24c-ad47-0c8dd622773f
Version Independent ID: 0c37350f-3e0c-f710-7974-19a3762903b6
Content: Download Log (Azure DevOps Audit)
Content Source: docs-ref-autogen/5.1/audit/Download%20Log.yml
Product: devops
Technology: devops-ref
GitHub Login: @wnjenkin
Microsoft Alias: whjenkin

Any updates on this?  Looking for API to download logs.

Alan, I found this which might be useful to you:  https://docs.microsoft.com/en-us/rest/api/azure/devops/audit/download log/download log?view=azure-devops-rest-6.0

Thanks @mapoitras. At the time of reporting this issue, download log was not live on 6.0-preview, even though it was in the docs. I haven't double checked to see if its live now though.
Ultimately I ended up using query log (https://docs.microsoft.com/en-us/rest/api/azure/devops/audit/audit log/query?view=azure-devops-rest-5.1) and having a simple script to loop whenever I had a continuation token. This results in me making 4 or 5 api calls for my amount of logs, but gets a similar result. That may be a good direction for you to take if the 6.0-preview doesn't work.


"The broadcaster has indicated that this channel is intended for mature audiences."

The bot doesn't click the "Start Watching" button and I don't know if this affects drops and needs to be address.
Screenshot: https://i.imgur.com/J2cByz6.jpg

Hmm.... Looks strange.
Please share the actual streamer’s link.

how is that strange? all streamers that are deemed as "18+" have that

Please attach the actual streamer’s link.

Please attach the actual streamer’s link.

I fixed this by emulates mouse click on this button element whenever element exist on first stream load. Test case https://www.twitch.tv/m0e_tv
#root > div > div.tw-flex.tw-flex-column.tw-flex-nowrap.tw-full-height > div > main > div.root-scrollable.scrollable-area.scrollable-area--suppress-scroll-x > div.simplebar-scroll-content > div > div > div.persistent-player > div.player-aspect > div.video-player > div > div > div > div.content-overlay-gate.player-overlay-background.player-overlay-background--darkness-0.tw-absolute.tw-align-items-center.tw-bottom-0.tw-c-text-overlay.tw-flex.tw-flex-column.tw-justify-content-center.tw-left-0.tw-right-0.tw-top-0 > div > div.content-overlay-gate__allow-pointers.tw-mg-t-3 > button

Thanks for help @heyimrendy ! I will write a jquery to detect this button!

New version release!
Please try it.


Modify Rules to match licenses

Does not update the YAML tags, but the descriptions were changed, and some names were changed. The tags are left where they are.

I'm not inclined to accept this. Apart from the editorialization/warning added, which I'd rather not have, unclear that the tweaks are improvements.


X3D Touch Sensor controlling animation in FBX file

Enables x3d files with objects containing Touch Sensors, to start fbx files that have key frame animations using the <ROUTE> node in X3D from the Touch Sensor to the inlined fbx file.
To invoke the entire animation, set the 'toField' to 'set_enabled':
<ROUTE fromField="isOver" fromNode="myTouchObj"
toField="set_enabled"  toNode="someObject"/>
for an animation from one frame to another, use:
<ROUTE fromField="isOver" fromNode="myTouchObj"
toField="2, 5"  toNode="someObject"/>
which will run the animation from key frame 2 to key frame 5.
myTouchObj is the DEFined name of the X3D <TouchSensor> node.
someObject is the DEFined name of the X3D <Inline> node with the url of the fbx file.
There is a pull request of test cases and documentation in the GearVRf-Tests repo under:
x3d->control_fbx_key_frame_animations
GearVRf-DCO-1.0-Signed-off-by: Mitch Williams
<EMAIL_ADDRESS>

Sooner rather than later we should move x3d to the org.gearvrf package. It seems you need too many public methods for no other reason but because x3d is in a separate package. Looks like x3d needs to be tightly integrated with the rest so it belongs to the same package then.

Most of the files in the X3D folder support X3D activities and don't interact with GearVR framework.  Perhaps only 3 files that interface with GearVR framework are: X3Dobject.java (the parser), AnimationInteractivityManager.java, which acts on sensors, and AnchorImplementation.java, the X3D version of HTML's Anchor tags going to other web files including x3d pages, or cameras.
So, perhaps just these 3 files..

Why does it need more than one public method. Isn't it only referenced from the asset loader?
Sent from my Verizon, Samsung Galaxy smartphone
-------- Original message --------
From: Mihail<EMAIL_ADDRESS>Date: 6/27/17 8:58 AM (GMT-08:00)
To: Samsung/GearVRf<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: Re: [Samsung/GearVRf] X3D Touch Sensor controlling animation in FBX file (#1255)
Sooner rather than later we should move x3d to the org.gearvrf package. It seems you need too many public methods for no other reason but because x3d is in a separate package. Looks like x3d needs to be tightly integrated with the rest so it belongs to the same package then.

You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/Samsung/GearVRf/pull/1255#issuecomment-311403587, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AQw1S3zjvxApNtNdSIqHhc77L7HYWiuOks5sISalgaJpZM4OF540.

I think the reference was to having to return some protected values in one of the GVRAnimations that had a reference to GVRKeyFrameAnimation.  I had to access the saved key frame animations in order to build new key frame animations from the specified key frames.

The X3D parser as-is is unusable without GVRf. It should be all be in the same package as GVRf since it requires, and will probably continue to, access to non-public APIs and state.

X3dObject.java is almost 4000 lines long and indents so deeply the code is not readable. Please make some internal classes or functions or something other than a with 20 levels of indentation.

I know Nola has changes to X3DObject parser relating to GVRMesh, and Tom and I scheduled some issues that I need to improve X3D parser.  I agree the X3D parser needs some clean-up and this will happen after SIGGRAPH.
Since this X3D feature calling an FBX animation is an undocumented, non-spec, work-around, what about a re-work of GVRAnimation to enable selecting animations from specific key frames?
It really seems like invoking an animation for specific key frames would be a nice feature for animators working in FBX.
Is there anything I should check into regarding moving X3DObject into GVRf same package?

We are planning an overhaul of the entire animation system later this year. If you want to rework GVRAnimation to supply the features you need that is a good first step. I won't be able to work on animation until after Siggraph.

Upon Tom's approval (or Mihail or Nola), my recommendation is to withdraw this Pull Request, and modify GVRAnimation to enable selecting animations by a range of key frames (from key frame 4 to 9 for example), that would be useful to FBX or X3D animations.  My modifications were based on a request from the art team, which can still use this code in the mean time for their own development and testing.

Modifying GVRAnimation sounds good. It seems like a useful capability in general.

Go for it!

@mwitchwilliams So we close this one?

Unless you think we should wait for Tom, then go ahead and close it.

We can always reopen unless you delete the branch.


3d support roadmap

A work-in-progress list of things required to support 3d models:

dune-copasi should already support 3d simulation

presumably membrane reactions go from a flux across a 1d edge between elements -> flux across the surface of a 2d face between 3-d elements?

extend Pixel to support 3d

should be straightforward, but simulating might become very slow

extend GUI to visualize 3d images, geometry, concentration

this could just be 2d slices
or maybe some kind of 3d rendering: might be easier to use e.g. for selecting compartments

extend GUI to visualize 3d meshes
automatic generation of 3d triangular mesh from stack of 2d pixel images / voxel image
extend SBML support to include 3d models

need to ensure consistency between 2d & 3d models

(optional) add support for SBML import/export of mesh geometry as well as pixel-based geometries

dune-copasi should already support 3d simulation

presumably membrane reactions go from a flux across a 1d edge between elements -> flux across the surface of a 2d face between 3-d elements?

I assume you mean the transmission conditions. If so, yes. It just happens at the intersection between two entities of the world dimension.


Module does not report error to keymetrics

When the module cannot connect to redis, not any issues is thrown to keymetrics

I get Message : Redis connection to <IP_ADDRESS>:6379 failed - connect ECONNREFUSED, is this not enough?


Add dependency on once_cell to Cargo.toml

The gtk-rs examples fail to compile because multiple examples make use of once_cell, but there is no dependency on once_cell in Cargo.toml.
error[E0658]: use of unstable library feature 'rustc_private': this crate is being loaded from the sysroot, an unstable location; did you mean to load this crate from crates.io via `Cargo.toml` instead?
  --> src/bin/basic_subclass.rs:13:1
   |
13 | extern crate once_cell;
   | ^^^^^^^^^^^^^^^^^^^^^^^
   |
   = note: for more information, see https://github.com/rust-lang/rust/issues/27812

Thanks!


Implement axis parameter on cupy.unique

Fixes #6691
This is different from the numpy implementation, since they use structured arrays to keep track of the index changes. This implementation sorts the axis elements manually (since there's no a clear way to do it more "generic" via argsort)

I've just extended the NaN comparison (lexicographical sorting element-wise on pairs of rows of the axis, looking if left_row < right_row) to consider the following cases:
equal_nan=False and non-complex dtype

If both elements are NaN, then the left row is marked as "lesser" and moved to the left of the result.
If left_elem = NaN and right_elem != NaN, then the left row is marked as "lesser" and moved to the left of the result.
If  left_elem != NaN and right_elem = NaN, then the left row is marked as "greater" and moved to the right of the result.

equal_nan=True and non-complex dtype

If both elements are NaN, then the comparison is skipped to the next element.
If left_elem = NaN and right_elem != NaN: same as equal_nan=False.
If  left_elem != NaN and right_elem = NaN: same as equal_nan=False.

equal_nan=False and complex dtype

If left_elem = (NaN, NaNj) and right_elem = (NaN, NaNj), then the left row is marked as "lesser" and moved to the left of the result.
If left_elem = (NaN, NaNj) and right_elem = (Y, NaNj), then the left row is marked as "lesser" and moved to the left of the result.
If left_elem = (NaN, NaNj) and right_elem = (NaN, Wj), then the left row is marked as "lesser" and moved to the left of the result.
If left_elem = (NaN, NaNj) and right_elem = (Y, Wj), then the left row is marked as "lesser" and moved to the left of the result.
If left_elem = (X, NaNj) and right_elem = (NaN, NaNj), then the left row is marked as "greater" and moved to the right of the result.
If left_elem = (X, NaNj) and right_elem = (Y, NaNj), then the comparison is made between the real parts.
If left_elem = (X, NaNj) and right_elem = (NaN, Wj), then the left row is marked as "greater" and moved to the right of the result.
If left_elem = (X, NaNj) and right_elem = (Y, Wj), then the comparison is made between the real parts.
If left_elem = (NaN, Kj) and right_elem = (NaN, NaNj), then the left row is marked as "greater" and moved to the right of the result.
If left_elem = (NaN, Kj) and right_elem = (Y, NaNj), then the left row is marked as "lesser" and moved to the left of the result.
If left_elem = (NaN, Kj) and right_elem = (NaN, Wj), then the comparison is made between the imaginary parts.
If left_elem = (NaN, Kj) and right_elem = (Y, Wj),  then the left row is marked as "lesser" and moved to the left of the result.
If left_elem = (X, Kj) and right_elem = (NaN, NaNj), then the left row is marked as "greater" and moved to the right of the result.
If left_elem = (X, Kj) and right_elem = (Y, NaNj), then the comparison is made between the real parts.
If left_elem = (X, Kj) and right_elem = (NaN, Wj), then the left row is marked as "greater" and moved to the right of the result.
If left_elem = (X, Kj) and right_elem = (Y, Wj), then the comparison is performed normally.

equal_nan=True and complex dtype

If left_elem = (NaN, NaNj) and right_elem = (NaN, NaNj), then the comparison is skipped.
The other cases remain the same as equal_nan=False

@toslunar Could you start a review?

@toslunar, thanks for your review! I've just updated the implementation to use trim_zeros instead of elementwise scan, I'm just missing a performance comparison between calling unique on CPU vs the proposed implementation

Here are the profiling results where the current implementation (using trim_zeros + subtraction)  is compared against pre/post-processing on GPU and then calling NumPy's unique function as proposed by @toslunar on https://github.com/cupy/cupy/pull/6886#pullrequestreview-1140313305.
The comparison script corresponds to this one: https://gist.github.com/andfoy/7244295739c7636ed0e16ac87ca9d458. All the times reported consist of the maximum between the GPU and the CPU elapsed times.
As it is possible to observe, the current implementation based on a lexicographical sort of the rows of the dimension being sorted is much faster than a potential implementation based on calling unique on CPU and then indexing

Size
dtype
GPU-only (ms)
GPU for pre/post-processing + CPU (ms)

3x3
uint8
0.314971
0.236437

5x5
uint8
0.288600
0.240530

10x10
uint8
0.291211
0.250625

100x100
uint8
0.369488
0.664650

500x500
uint8
0.406018
5.873621

1000x1000
uint8
0.800287
20.733934

5000x5000
uint8
9.005074
460.965287

10000x10000
uint8
34.741940
1857.037542

3x3
int32
0.336972
0.209263

5x5
int32
0.304245
0.234205

10x10
int32
0.304105
0.263415

100x100
int32
0.419373
0.677363

500x500
int32
0.572477
6.331609

1000x1000
int32
1.202842
22.220180

5000x5000
int32
16.069151
554.746273

10000x10000
int32
63.799783
2073.969380

3x3
int64
0.417643
0.267951

5x5
int64
0.401194
0.231783

10x10
int64
0.294274
0.253717

100x100
int64
0.414672
0.696820

500x500
int64
0.854722
6.263096

1000x1000
int64
2.015497
23.052823

5000x5000
int64
39.345610
585.807965

10000x10000
int64
156.096988
2264.345225

3x3
float32
0.309342
0.217062

5x5
float32
0.321131
0.243317

10x10
float32
0.313809
0.269831

100x100
float32
0.415692
0.674457

500x500
float32
0.643197
6.248911

1000x1000
float32
1.586711
22.506326

5000x5000
float32
21.515723
556.062853

10000x10000
float32
85.151700
2076.375974

3x3
float64
0.374232
0.216780

5x5
float64
0.329921
0.250573

10x10
float64
0.326812
0.266707

100x100
float64
0.465202
0.678642

500x500
float64
1.021641
6.314075

1000x1000
float64
2.320133
23.217900

5000x5000
float64
48.405550
588.974063

3x3
complex64
0.344953
0.221207

5x5
complex64
0.307549
0.240872

10x10
complex64
0.311879
0.257227

100x100
complex64
0.411276
0.685826

500x500
complex64
0.970940
6.654078

1000x1000
complex64
2.772588
23.736537

5000x5000
complex64
78.739919
619.654977

3x3
complex128
0.368012
0.211796

5x5
complex128
0.331925
0.245640

10x10
complex128
0.339596
0.266747

100x100
complex128
0.616204
0.713759

500x500
complex128
3.348005
6.816852

1000x1000
complex128
12.008427
34.436096

/test mini

/test mini

CI failure seems unrelated to this PR.

@andfoy Sorry for our late review. LGTM!


Add terminateIfRunning WorkflowIDReusePolicy

It common scenario when a workflow should be started with new parameters. Terminating and then starting workflow has race conditions. Adding terminateIfRunning WorkflowIDReusePolicy would allow to eliminate race conditions.

termination & start a new can be done by similar to continue as new
this should be easy to do since after the refactor of DB layer & business logic layer, terminate the currently running workflow & start a new workflow can be done within one transaction.

Yes wenquan, I will pick it up as promised long long time ago ..

https://github.com/uber/cadence/issues/2653

This feature will be ready in server 0.13.0 and related client releases.


Remove all usage of core Date

We can't use it in 0.19, so we're going to prepare by replacing it with a custom DateTime type that represents a day, month, year, hour, and minute - all of the components that matter to this package - and leave it up to the caller to get what it wants from that data.
Perhaps most importantly, time zones don't matter to this package because it does not let the user select a time zone. It just allows the user to select a day, month, year, hour, and minute.

sorry @abadi199, this was meant to be a pr on our NRI fork


Error: CPI error 'Bosh::Clouds::CloudError' with message 'Creating disk of size '5120': Waiting for disk: Expected PVC to become ready (ClaimBound phase)' in 'create_disk' CPI method

Hi,
I ran into a problem today while trying to deploy the latest CF with the latest BOSH deployed into k8s. I've gotten quite far so far, and am pretty impressed with where this is going.
However, I ran into this problem which I can't seem to understand why the disk is never created.
Task 100 | 22:49:21 | Updating instance consul: consul/e59309c4-38cf-4819-a073-3e3abbd6d10c (0) (canary) (00:20:03)
                   L Error: CPI error 'Bosh::Clouds::CloudError' with message 'Creating disk of size '5120': Waiting for disk: Expected PVC to become ready (ClaimBound phase)' in 'create_disk' CPI method
Task 100 | 23:09:24 | Error: CPI error 'Bosh::Clouds::CloudError' with message 'Creating disk of size '5120': Waiting for disk: Expected PVC to become ready (ClaimBound phase)' in 'create_disk' CPI method

Looking at the code here:
https://github.com/bosh-cpis/bosh-kubernetes-cpi-release/blob/241f60a6c303ce35612f9c17b44f98fbc7804756/src/github.com/cppforlife/bosh-kubernetes-cpi/disk/factory.go#L50
It called NewDiskCID, I assumed to create the disk and return its id.
But then hopping over to https://github.com/bosh-cpis/bosh-kubernetes-cpi-release/blob/master/src/github.com/cppforlife/bosh-cpi-go/apiv1/disks.go#L22-L26 shows it doesn't seem to really do anything (just return a nested struct with the cid set in it).
Am I missing something? (or am I losing my mind? lol) Either is possible :)
Thanks, Ryan

I should also point out I'm running this in an acs-engine deployment in Azure. Looking at the storage class/provisioner props - it looks like these are hardcoded to 'standard' and 'default'.
I wonder if that is actually what is messing with this?

Yeah creating a new storage class called 'standard' using managed Azure disks (cos I'm using vmss) did the job. Excuse the brainfart up there, didn't quite understand how PVC worked :)


depends: upgrade qt to 5.15.3 and associated patchfiles

     adapted from bitcoin/bitcoin@640eb772

Deleted in favour of newer version


Detect duplicate slots

This doesn't raise an error when declaring:
class Foo:
    __slots__ = ('a', 'a', 'b')

This probably doesn't occur often though

implemented in v0.10.0


Why another component?

Hi,
why did you build own components instead of working with the guys from https://github.com/yuche/vue-strap ?

@psi-4ward hello and thanks for your interest :)  yuche/vue-strap was soo great and we where already using it. but we need a completly different road-map as there are many breaking changes in both Bootstrap 4 and Vue.Js 2 (just look at that repo's requirements)


Feature Request: List if folder is marked as "viewed"

Currently, individual files can be marked as "viewed" in GitHub Pull Requests, and the extension properly shows a "check mark" icon next to those files.
It would be nice, for when reviewing larger PRs, to show if all files within a folder have been marked as viewed. The example below show that I've viewed the 2 files, but when the folder is collapsed, it's not possible to tell that I've already marked everything in that folder as viewed.
I don't think the scope of this would necessarily need to expand to being able to mark/unmark entire folders, but at least having the indicator on the folder would be nice!

If I can add something on top of this, it'd be nice if also one can mark all files in a folder as viewed. For example, when reviewing a PR and a whole inlined dependencies folder has changed (Go's vendor for example).
If you don't mind, I'd love to take a stab at this. Always wanted to dive into vscode-exntesions 😀

@thepalbi I would accept a PR for this. This change will touch a lot of files, but here are some code pointers:
You will need to set a contextValue on DirectoryTreeNode for it have the "mark as viewed" action:
https://github.com/microsoft/vscode-pull-request-github/blob/771ea78afac82966a9de4268bd2770688141dcd3/src/view/treeNodes/directoryTreeNode.ts#L10-L11
Example of what the contextValue might be. You probably only need the "viewed"/"unviewed" part:
https://github.com/microsoft/vscode-pull-request-github/blob/771ea78afac82966a9de4268bd2770688141dcd3/src/view/treeNodes/fileChangeNode.ts#L85-L86
Example of how to use when to add the "mark as viewed" action with the context value:
https://github.com/microsoft/vscode-pull-request-github/blob/771ea78afac82966a9de4268bd2770688141dcd3/package.json#L1372-L1381
Then the "mark as viewed" command would need to handle the incoming tree node (DirectoryTreeNode):
https://github.com/microsoft/vscode-pull-request-github/blob/771ea78afac82966a9de4268bd2770688141dcd3/src/commands.ts#L875-L885
Finally, for the little "✓" decoration, you would need to update the decoration provider:
https://github.com/microsoft/vscode-pull-request-github/blob/771ea78afac82966a9de4268bd2770688141dcd3/src/view/fileTypeDecorationProvider.ts#L15-L16

Also I would love a feature which let me toggle (hide) the files that has been marked as viewed.

@alexr00 Thank you for the update! Just saw it this morning after a VSCode (and plugin) update, and this looks fantastic!


Release Google.Cloud.ServiceUsage.V1 version 1.0.0

Changes in this release:
No API surface changes; just dependency updates and promotion to GA.

Created release for Google.Cloud.ServiceUsage.V1-1.0.0


IdentityServer 4 using IIS

Hi All
I'm trying to run IdentityServer4.Samples using IIS and it is not working.
On localhost it works fine but using IIS not.
I already added question with info here
http://stackoverflow.com/questions/37257043/identityserver-4-using-iis
Can you please help me to find problem?

Try again tomorrow or friday. We are currently updating everything to RC2

Also for me all work fine with localhost (IdServer and MVC App) but when I try to use IIS there are some problems...At this time, for test only, I'm using only one server where are configured IdServer that response on 1941 port and MVC app that response on port 3308... (I don't know if it is the right way?)

MVC App  at x.x.x.x:3308: seem to work correctly on IIS;
IdServer at x.x.x.x:1941: when I try to access Secure Page from MVC app (http://x.x.x.x:3308/Home/Secure) or when I try to go to UI login page of the IdServ (http://x.x.x.x:1941/ui/login) this error occour:

System.InvalidOperationException: The view 'Index' was not found. The following locations were searched:

~/UI/Home/Views/Index.cshtml
~/UI/SharedViews/Index.cshtml
   at Microsoft.AspNetCore.Mvc.ViewEngines.ViewEngineResult.EnsureSuccessful(IEnumerable`1 originalLocations)
   at Microsoft.AspNetCore.Mvc.ViewResult.<ExecuteResultAsync>d__26.MoveNext()
...

Instead, working locally, on localhost all works fine...but...
Is there a "better way" to deploy on IIS? I think that a good way should be to have IdSrv4 and MVC App on different server with different IP Address (so I have to configure correctly Startup.cs with different Authority and redirect Url ...I think)...
Thanks

I deployed IdSrv4 on IIS on localhost...port 1941
But seem that Server start listening on port  26379 (random)
...
Content root path: C:\www\IdSrv
Now listening on: http://localhost:26379/
...

but I send request on 1941:
info: Microsoft.AspNetCore.Hosting.Internal.WebHost[1]
      Request starting HTTP/1.1 GET http://localhost:1941/

What is wrong?
This is the complete log:
info: Microsoft.Extensions.DependencyInjection.DataProtectionServices[0]
      User profile not available. Using 'HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\ASP.NET\4.0.30319.0\AutoGenKeys\S-1-5-82-3006700770-424185619-1745488364-*********-4004696415\DataProtection' as key repository and Windows DPAPI to encrypt keys at rest.
dbug: Microsoft.AspNetCore.Hosting.Internal.WebHost[3]
      Hosting starting
dbug: Microsoft.AspNetCore.Hosting.Internal.WebHost[4]
      Hosting started
Hosting environment: Production
Content root path: C:\www\IdSrv
Now listening on: http://localhost:26379/
Application started. Press Ctrl+C to shut down.
dbug: Microsoft.AspNetCore.Server.Kestrel, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60[1]
      Connection id "0HKS9VUNQ13I3" started.
info: Microsoft.AspNetCore.Hosting.Internal.WebHost[1]
      Request starting HTTP/1.1 GET http://localhost:1941/
dbug: Microsoft.AspNetCore.StaticFiles.StaticFileMiddleware[4]
      The request path / does not match a supported file type
dbug: Microsoft.AspNetCore.Routing.Tree.TreeRouter[1]
      Request successfully matched the route with name '' and template ''.
dbug: Microsoft.AspNetCore.Mvc.Internal.MvcRouteHandler[1]
      Executing action Host.UI.Home.HomeController.Index (IdentityServer)
info: Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker[1]
      Executing action method Host.UI.Home.HomeController.Index (IdentityServer) with arguments () - ModelState is Valid'
dbug: Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker[2]
      Executed action method Host.UI.Home.HomeController.Index (IdentityServer), returned result Microsoft.AspNetCore.Mvc.ViewResult.'
dbug: Microsoft.AspNetCore.Mvc.Razor.RazorViewEngine[1]
      View lookup cache miss for view 'Index' in controller 'Home'.
fail: Microsoft.AspNetCore.Mvc.ViewFeatures.Internal.ViewResultExecutor[3]
      The view 'Index' was not found. Searched locations: ~/UI/Home/Views/Index.cshtml, ~/UI/SharedViews/Index.cshtml
fail: Microsoft.AspNetCore.Diagnostics.DeveloperExceptionPageMiddleware[0]
      An unhandled exception has occurred while executing the request
System.InvalidOperationException: The view 'Index' was not found. The following locations were searched:
~/UI/Home/Views/Index.cshtml
~/UI/SharedViews/Index.cshtml
   at Microsoft.AspNetCore.Mvc.ViewEngines.ViewEngineResult.EnsureSuccessful(IEnumerable`1 originalLocations)
   at Microsoft.AspNetCore.Mvc.ViewResult.<ExecuteResultAsync>d__26.MoveNext()

I solved the problem simply deploying "UI" folder.... very easy

You are right, I also publish my solution on stackoverflow
http://stackoverflow.com/questions/37257043/identityserver-4-using-iis

Yes, this is the best way:
"publishOptions": {
  "include": [
    "wwwroot",
    "UI",
    "damienbodserver.pfx",
    "web.config"
  ]}


JWT Token

Nowhere in the documentation you tell exactly how to send the JWT for authentication.
Digging the code, it looks like an "Authorization" header is expected in the GET request. Is that correct?
Also, just to clarify, you don't perform any kind of payload validation on the JWT, there is a simple hash validation, right?

Hi @alesinicio ,
Thanks for opening this issue!
Your concern is fair, it is not properly described how you should use the system with JWT.
Of course now you know, that it is the standard Authorization: Bearer {token} but this should be described.
On your second question yes, it is a simple hash validation. I was thinking in implementing some kind of a payload validation but did not come up with a solution because at that point the service is trading flexibility for security while also introducing more complexity in integration. However, this is a topic I would like to take a deeper look once I have more time.

Thanks for the feedback!
As a suggestion for the hash validation, maybe you could check the existence of a fixed-name bash script (or python, or whatever seems the most widely adopted) on a specific directory. If the script exists, the token is passed to it for validation. If not, you fallback to the simple hash validation.
The script would obviously be provided by the user, thus retaining full flexibility (works like an user-provided extension).


CAMEL-20958: camel-aws - Kinesis consumer loops on closed shards

Description

Target

[ ] I checked that the commit is targeting the correct branch (note that Camel 3 uses camel-3.x, whereas Camel 4 uses the main branch)

Tracking

[ ] If this is a large change, bug fix, or code improvement, I checked there is a JIRA issue filed for the change (usually before you start working on it).

Apache Camel coding standards and style

[ ] I checked that each commit in the pull request has a meaningful subject line and body.

[ ] I have run mvn clean install -DskipTests locally from root folder and I have committed all auto-generated changes.

/component-tests aws


merge package from arlac77/npm-package-template

.travis.yml

chore(travis): remove node versions 10.8.0
chore(travis): add node versions 10.9.0

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


nixos/home-assistant: fix bug causing config file not to be written (including http.server_port)

Motivation for this change
This fixes  #82076 by ensuring that the configuration.yaml file is always (over)written. I think this approach simplifies things a bit, and to me aligns more with what I would assume "default config" means. I'm not sure it's the right way though, since it changes the behaviour of  applyDefaultConfig.
It would be great to get some feedback on a few points in particular:

Would is be better to set defaultConfig to a subset of the contents of the default configuration.yaml file plus the port and timezone? For now I just have frontend in there. This might be less of a breaking change but it would mean keeping up to date when home-assistant changes that file. For reference, this is what that file looks like:# Configure a default setup of Home Assistant (frontend, api, etc)
default_config:

# Uncomment this if you are using SSL/TLS, running in Docker container, etc.
# http:
#   base_url: example.duckdns.org:8123

# Text to speech
tts:
  - platform: google_translate

# These files all start off empty
group: !include groups.yaml
automation: !include automations.yaml
script: !include scripts.yaml
scene: !include scenes.yaml

Tests: I've never written nixos tests before, so I don't know if this is the right approach, or whether this kind of test is necessary.

NOTE: some of the home-assistant tests seem to be failing currently, but this appears to be caused by #82441
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@colemickens or @dotlambda does this change make sense to either of you?

I'm not sure I'm familiar enough, or having enough time, to review this well. cc: @mic92 even though I think they've seen it.

ping @colemickens @dotlambda @Mic92 @peterhoeg
I'm sure @matt-snider would appreciate a review very much so. Is anyone able to sacrifice some time and help out with this? :heart:

Thank you @Mic92!

@matt-snider what do you need to push this through to the finish line?

@aanderse Thanks for following up! I think the main suggestion from @Mic92 was to remove the tests. Otherwise, from his comment "So the actual bug fix is this line?", I interpret that a more minimal change would be preferable. That's understandable, since it probably would have allowed this to be more easily approved.
Perhaps we should just close this PR, since it's over a year old, and I haven't run it with newer versions of home assistant. It doesn't seem to affect many people, judging by activity on this PR and the original issue.

Sorry things didn't go super smooth.

@aanderse Not at all. I should have been more proactive with this one.
Thank you for all the work you and many others do on Nix. It really is an incredible project! I'd like to help out more but you know... time management :-/


ttf file not found

I want to generate picture by this depictor, But I find that circled_large.ttf and circled.ttf not find.  How could I get these ttf file?
hope for you reply.

thank you !!!
feel free to close this issue


Make cancelTask usable

Sorry for the already pulled commits.

Okay, I will attempt to fix this mess and try again with the PR.


Render Markdown in HTML

Hello, is there any chance to get Markdown syntax working inside HTML elements?
Now this

# Hello

renders as
# Hello

I know that some md compilers does it and some not, but can we do that with passing some option to compiler?

Hmm, you mean using the compiler in this library for all createElement if the sole child is a string?

Ah nvm I understand. This appears to be a bug.

Looking into this further, it's a tough one. We're currently parsing the contents of html blocks as inline unless another HTML tag is detected. If we change that behavior, the whitespace of the HTML starts to matter.
For instance, if a nested HTML tag has 4 spaces in front of it, that technically makes it an indented code block. Things get progressively more complex from there with the various edge cases and now having to start being aware of what whitespace is worth paying attention to and what should be discarded...

my inclination is to leave things as-is

you meant leave it to render 4 space indents as code blocks?

Yup. You can see the PR I made which fixes this use case, but it just introduces a new edge case of intentional indented code blocks not working inside HTML since we have to choose.
That being said, I’m open to merging the PR since HTML in markdown is a hack as-is and just add a note somewhere describing caveats.

yes that's what I would do

Is there a way to disable this behavior? I need the MD syntax to stay untouched inside my component:
<MyOwnSyntax>
  * Foo
  * Bar
  * Baz

  **test**
</MyOwnSyntax>

Use a fenced code block
On Wed, Jun 12, 2019 at 7:19 AM Ivan Kleshnin<EMAIL_ADDRESS>wrote:

Is there a way to disable this behavior? I need the MD syntax to stay
untouched inside my component:

  * Foo
  * Bar
  * Baz
test

—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/probablyup/markdown-to-jsx/issues/153?email_source=notifications&email_token=AAELFVR3RGG7KVL533FCTL3P2DSWVA5CNFSM4ERGXX2KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXQHBTI#issuecomment-501248205,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAELFVWU4HUPBCOSTDLLOJLP2DSWVANCNFSM4ERGXX2A
.

Each of them will add two lines to MD content. And I'll have multiple cases like this per file:
<Quiz>
* Option-1
* Option-2
* Option-3
</Quiz>

<Quiz>
* Option-1
* Option-2
* Option-3
</Quiz>

vs this:
<Quiz>
'''
* Option-1
* Option-2
* Option-3
'''
</Quiz>

<Quiz>
'''
* Option-1
* Option-2
* Option-3
'''
</Quiz>

– quite a burden.

It’s markdown in a markdown processor, not sure what to tell you. Putting
it in a code block is the only way to tell the compiler it’s not meant to
be processed.
On Wed, Jun 12, 2019 at 8:08 AM Ivan Kleshnin<EMAIL_ADDRESS>wrote:

Each of them will add two lines to MD content. And I'll have multiple
cases like this per file:

Option-1

Option-2

Option-3

Option-1

Option-2

Option-3

vs this:

'''

Option-1

Option-2

Option-3

'''

'''

Option-1

Option-2

Option-3

'''

– quite a burden.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/probablyup/markdown-to-jsx/issues/153?email_source=notifications&email_token=AAELFVRYAROI55UFF3RXWVLP2DYLDA5CNFSM4ERGXX2KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXQLDYI#issuecomment-501264865,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAELFVQKUO26I6MYJS2MIALP2DYLDANCNFSM4ERGXX2A
.


Make input label layout as a parameter

Currently the layout is two-row, first is the label text, second is an input.
It needs to be so that one (in the markup, or in the custom code) can set which layout is chosen.
Two-line layout:
Parameter_1 label
[Parameter_1 text field]
Parameter_2 label
[Parameter_2 dropdown]
...

One-line layout:
Parameter_1 label [Parameter_1 text field]
Parameter_2 label [Parameter_2 dropdown]

This issue has been mirrored in Jira: https://reddata.atlassian.net/browse/GROK-14928

This issue has been mirrored in Jira: https://reddata.atlassian.net/browse/GROK-16053