CSS class is not included in the dev mode

Description

those items should have a class called .repository .repository-summary .sub-menu .item in repo.css, but this class is not included.

Need a suggestion about how to find out why 😭
Gitea Version
latest codebase
Can you reproduce the bug on the Gitea demo site?
Yes
Log Gist
No response
Screenshots
No response
Git Version
No response
Operating System
No response
How are you running Gitea?
run in dev mode
Database
None

Never mind. It's the IDE debug configuration. After building the frontend manually, everything works well.


Feature: CoreLightningRest partial payouts

Implements changes requested in Issue #536 + small improvements of the checks on the same functionality in lndrest.

I merged @rbensberg branch which puts corelightningrest.py onto the newer plugin clnrest which comes integrated with CLN and should just work without too many hoops.
Although NOW for some reason it hangs when verifying the payment of the quote (when minting tokens). If you stop it and then check the invoice manually cashu invoice <amount> --id <quote_id> doesn't have any problem finding it.

paid_invoice_stream was the problem. Fixed it.

Thank you for the fixes in lndrest.py. Could you put those changes into a separate PR?

Yea sure.


docs: update contributing guide

Checklist

[x] Fix linting errors
[x] Tests have been added / updated (or snapshots)

This change is

:tada: This PR is included in version 1.0.0-alpha.10 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


rmdir_syscall() updates

Description
Fixes # (issue)
The following changes include new comments and unit tests for rmdir_syscall(). Moreover, a fix of the bug related to write permission of a parent directory inside rmdir_syscall() is proposed. In addition to that, a fix for lind_deltree() bug related to not setting a parent directory's write permission before iterating through child directories is included. Finally, the description for the bug related to the absence of checking search permission while walking the file tree is included in the ut_lind_fs_search_permission_bug_with_rmdir() unit test. The issue (#283) for that bug has been opened as well.
Type of change

[ ] More detailed comments for rmdir_syscall()
[ ] New unit tests for rmdir_syscall()
[ ] Bug fixes for rmdir_syscall() and lind_deltree()
[ ] Bug description inside ut_lind_fs_search_permission_bug_with_rmdir() unit test

How Has This Been Tested?
To run the tests, we need to run cargo test --lib command inside the safeposix-rust directory.
All the tests are present under this directory: lind_project/src/safeposix-rust/src/tests/fs_tests.rs

Test A - ut_lind_fs_rmdir_normal()
Test B - ut_lind_fs_rmdir_empty_path()
Test C - ut_lind_fs_rmdir_nonexist_dir()
Test D - ut_lind_fs_rmdir_root()
Test E - ut_lind_fs_rmdir_nonempty_dir()
Test F - ut_lind_fs_rmdir_nowriteperm_child_dir()
Test G - ut_lind_fs_rmdir_nowriteperm_parent_dir()
Test H - ut_lind_fs_search_permission_bug_with_rmdir()

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] Any dependent changes have been added to a pull request and/or merged in other modules (native-client, lind-glibc, lind-project)

@rupeshkoushik07 can you take a look at this one?


async-graphql v3 graphql_subscription_upgrade_with_data replacement.

Greetings!
graphql_subscription_upgrade_with_data used generic for websocket and I was able to use wrapper (https://github.com/async-graphql/async-graphql/pull/534#issuecomment-855689750). But now, GraphQLWebSocket::new doesn't use generic, and I can't use wrapper.
What should I do?

Sorry, I forgot this, please upgrade to 3.0.3, I also added GraphQLWebsocket::new_with_pair. 😁

Thanks!


Fatal error - Missing "snsr.h"

I'm installing avs-device-sdk on to Raspberry Pi (OS - Stretch). During compilation I get an error stating snsr.h file is missing. I did look into the folders present in the avs-device-sdk project wasn't able to locate the header file. Kindly, help by providing some inputs.

This header file is from Sensory wake-word detector. You need to clone the following library:
https://github.com/Sensory/alexa-rpi
Ans double check your make files that correct include path is configured.
Alternatively, you can build SDK without wake-word detection (only hold to speak and tap to speak features will work, thou).

Cloned the repo from https://github.com/Sensory/alexa-rpi and copied the "snsr.h" from Sensory to sdk-source/avs-device-sdk/KWD?Sensory/include/Sensory
After this, I don't see the "snsr.h" missing error but run into the following error
make[2]: * No rule to make target '/home/pi/sdk-folder/third-party/alexa-rpi/lib/libsnsr.a', needed by 'KWD/Sensory/src/libSENSORY.so'.  Stop.
CMakeFiles/Makefile2:4233: recipe for target 'KWD/Sensory/src/CMakeFiles/SENSORY.dir/all' failed
make[1]: * [KWD/Sensory/src/CMakeFiles/SENSORY.dir/all] Error 2
Makefile:138: recipe for target 'all' failed
make: * [all] Error 2

Cloned the latest repo again and followed the instructions to run SampleApp on Raspberry Pi 3 Model B. Things seem to be working. Closing this issue.

Hi, I came across the same issue, I did the same steps but I still can’t get over the resulting error. Could you clarify your steps after copying snsr.h?


Ruby script to convert FASTA to CSV - FAQ edits

Looks good

Some of the FAQ stuff seems to be missing punctuation. Consider adding that for consistency?

Some of the FAQ stuff seems to be missing punctuation. Consider adding that for consistency?

Will do!


Menu item text "Supprimer Definitivement" too wide on MacOS (line break)

Related issue (same cause?): https://github.com/readium/readium-desktop/issues/339

Another instance of clipped text in menu items:

As a general comment, I noticed excessive verbosity in French and English UI labels. I suggest we do a complete check-over at the next opportunity (lesser priority right now)

Supprimer définitivement -> Supprimer
Ajouter l'extrait à la bibliothèque -> Importer

+1 to Laurent's suggestion!
When time allows I will do a complete review pass on the English locale, I think we can use more concise terms. I am not so sure how "compact" other locales can be (especially French tends to be quite verbose), but shorter is usually better :)

If I may – been there, done that –, keep in mind German but also Dutch or Danish actually may occasionally be a lot more verbose (and you end up with a ratio of 3:1 from English), so do not hesitate to protect against very lengthy labels (e.g. ellipsis, etc.) in any case.

+1 for ellipsis.
For example https://github.com/readium/readium-desktop/issues/427

fixed in 56fb184

PR: https://github.com/readium/readium-desktop/pull/443


fix(portal): fix title

📥 Proposed changes
affects: @fremtind/portal
probably a regression from when we changed to use the old style of layout
☑ Submission checklist

[x] I have read the CONTRIBUTING doc
[x] yarn build works locally with my changes
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have added necessary documentation (if appropriate)

💬 Further comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did, what alternatives you considered, etc...

@vegardsoerlie @eduardodir rask nok response på feilrapport?

@lfbergee Fantastisk 🥇


The Img_encoder pre-trained model might be corrupted

Hi,
Thanks for open-sourcing the excellent work!
However when I run the code with Img_encoder.pth.tar, it shows RuntimeError: unexpected EOF, expected 757379 more bytes. The file might be corrupted. Maybe there's something wrong with the model? I'd be appreciated if you might re-upload it.
Thank you!

Hmmm I don't know why but after calling the model in the code, the file size become smaller and the pytorch package say that it is corrupted.
I uploaded another model, please use it, although i am not sure that it would work :(
I will check it on weekend.
Thank you :)

Thank you so much!

I updated files and codes to fix errors :)
Please check it out 😉


about multi dialogues llama2 prompt templates?

llama2 chat format:
<s>[INST] <<SYS>>
{{ system_prompt }}
<</SYS>>

{{ user_msg_1 }} [/INST] {{ model_answer_1 }} </s><s>[INST] {{ user_msg_2 }} [/INST] {{ model_answer_2 }} </s><s>[INST] {{ user_msg_3 }} [/INST]

I follow the below code to train assistant style conversation model using SFTTrainer.But in llama2,the conversation data is format like above?how to set instruction_template,and response_template?if I set instruction_template = "[INST]" and response_template='[/INST]' .is it right?
from transformers import AutoModelForCausalLM, AutoTokenizer
from datasets import load_dataset
from trl import SFTTrainer, DataCollatorForCompletionOnlyLM

dataset = load_dataset("timdettmers/openassistant-guanaco", split="train")

model = AutoModelForCausalLM.from_pretrained("facebook/opt-350m")
tokenizer = AutoTokenizer.from_pretrained("facebook/opt-350m")

instruction_template = "### Human:"
response_template = "### Assistant:"
collator = DataCollatorForCompletionOnlyLM(instruction_template=instruction_template, response_template=response_template, tokenizer=tokenizer, mlm=False)

trainer = SFTTrainer(
    model,
    train_dataset=dataset,
    dataset_text_field="text",
    data_collator=collator,
)

trainer.train()

You can preprocess your data with by passing a formatting_func operation to be in the desired format that you pass the dataset to the SFTTrainer. See https://huggingface.co/docs/trl/sft_trainer#format-your-input-prompts. As an input it takes one row of your dataset. Does that work?

You can preprocess your data with by passing a formatting_func operation to be in the desired format that you pass the dataset to the SFTTrainer. See https://huggingface.co/docs/trl/sft_trainer#format-your-input-prompts. As an input it takes one row of your dataset. Does that work?

thanks. it does work


Add possibility to set a fallback url

Short description of what this resolves:
This commit allows to set a fallback url in case the provided url is not accessible.
Changes proposed in this pull request:
That is triggered by setting the key "fallback-url" for the element like this:
"tool" : {
    "url" : "https://not-accessible-url/tool.phar",
    "fallback-url" : "https://fallback-url/tool.phar"
}

In case the provided url is not accessible, the code will check for the fallback-url key and if set, the fallback url will be checked for accessibility and used to download the asset. Otherwise it displays the error message At least one given URL are not accessible!.
Fixes: #

Thanks for the contribution!


Test Experimental Class Structure using EEG

New Experimental class Structure created through #183 has not been tested using an EEG stream. Needs to be rigirously tested using an EEG to make sure that nothing breaks and all other functions work in the same way.

Have you tested the updated code now? Regarding the visualN170 experiment, the documentation does not appear to have updated the case code. n170.present(duration=record_duration, eeg=eeg_device, save_fn=save_fn) module 'eegnb.experiments.visual_n170.n170' has no attribute 'present'.

Thanks for noting this @LIN-DN . The .present method is now replaced with thenrun method. However it sees the example notebook in the master branch is still showing the old syntax. In the develop branch it is correct.
@oreHGA @tmorshed @Parvfect I think seeing as this is a key documentation file it should be fixed with a direct commit to master rather than waiting for the next merge of develop to master. What do you think? Pls check and confirm I got this correct.


getDeviceLocale returning invalid language code for France

Summary

Version
latest

Affected OS
ios

OS Version
11.4.1

Current behavior

Set the device language to be French(France) and DeviceInfo.getDeviceLocale() returns fr-US
Expected behavior
Should get fr-FR instead of fr-US

We will be working with react-native-localize to make sure locale support is 1st class - you should investigate migrating to their APIs for locale information. That said, I think we get this info raw from the device, so it's possible that there is just a very unexpected locale code, but it's coming from the device and we're just serving it up...

https://github.com/react-native-community/react-native-device-info/issues/627


Support comma separated list of deployment hosts

I'm using Deployer version 6 and would like to be able to use a comma separated list of deployment hosts. Currently only ranges are supported which could be quite dangerous for some setups, I'd like to provide a list of values such as:
[10,11,12]
example:
host('ipv6:[36:37:38]')
    ->set('branch', 'dev')
    ->stage('brands')
    ->set('repository', 'git@fudge:company/application-fudge')
    ->user('root')
    ->set('deploy_path', '/var/www/include-forms/fudge');

You can list a few hosts as args.


Sattellite Builder and Rocket Assembling Machine cause blank screens

When using either the scan function or the build function for both machines the entire screen goes blank.  The same color as the horizon (black and night and blue during the day).  I can still access the game there's just no screen to interact with as long as I'm looking at the pad in action.
If i close the builder's ui and look away, the screen returns.  If i wait until it finishes, the screen returns.
AR version 1.20.2-1.2.2
when screenshotted: this is what shows up:

Not sure what happened, patching one of my mods seems to have resolved the issue.
Closing issue.

Having this issue on our Custom Modpack, I got a blank screen while a friend of mine standing at the same location did not get a blank screen.
@SplitInfinitive Could you share a modlist perhaps so we also could fix this?

AdvancedRocketry-1.10.2-1.2.2
LibVulpes-1.10.2-0.2.5-universal

List of mods we use: https://gist.github.com/Elvarion/b745b82dcd2687f83ffd7610ae0dae9d


(IAC-449) Fix location to drop space else deployment fails

There is an error in the variables.tf with the default location (=Azure region). When not specifying a location file and using the default your terraform apply will fail like this:
│ Error: "resource_group_name" may only contain alphanumeric characters, dash, underscores, parentheses and periods
│
│   with module.aks.data.azurerm_public_ip.cluster_public_ip[0],
│   on modules/azure_aks/main.tf line 111, in data "azurerm_public_ip" "cluster_public_ip":
│  111:  data "azurerm_public_ip" "cluster_public_ip" {

The cause of that is on line https://github.com/sassoftware/viya4-iac-azure/blob/main/modules/azure_aks/main.tf#L116 where it says resource_group_name = "MC_${var.aks_cluster_rg}_${var.aks_cluster_name}_${var.aks_cluster_location}"
The var.aks_cluster_location is built from the location in the variables.tf, what happens is that the resource group name gets a space in it, MC_name-rg_name_East US, which is not allowed. Correct region would be "eastus", which this PR fixes.

Signed-off-by: Roeland<EMAIL_ADDRESS>

Should be reviewed and tested. @ranieuwe thx for the PR.

@ranieuwe, for the error you mentioned above, did you not set the required argument "location" in the terraform.tfvars file?
From what I saw the default location "East US" does get converted to "eastus" correctly. My guess is something else might be causing this issue. To make sure could you let us know what steps let you to this error?

No, I have not set any vars whatsoever. This is a vanilla git clone, tf init tf plan, tf apply. I am not using any terraform.tfvars on purpose. The logic asks for tenant_id, subscription_id and prefix, that's all. That's what I provided and then it runs into the problem above.
Error can be replicated by cloning the git repo, tf init and then: terraform apply -destroy -var="tenant_id=72c9d080-bc8c-4c9d-b922-8faa6f3975a8" -var="subscription_id=1d8655d1-6f0f-442a-94b1-1de8b77d2310" -var="prefix=sv4"
The problem exists because you try to construct the resource group from the location and there is a mismatch. I didn't see any logic that would convert East US to "eastus" in your TF templates anywhere FWIW.
Noticed that I did miss the default configuration setting in the docs, added that to the PR just now.

So the usage of location in creating the resource group name is the issue and not the fact it's listed as East US. For terraform that is a valid value; however as shown in the error above the generation of the name is where the failure is taking place.
The code in question is here : https://github.com/sassoftware/viya4-iac-azure/blob/main/modules/azure_aks/main.tf#L116
@riragh, it may be better to either setup the values without spaces or have a local variable that would do the conversion if we allow this.
@ranieuwe if you could please review the docs here : https://github.com/sassoftware/viya4-iac-azure/blob/main/docs/CONFIG-VARS.md#required-variables that outlines which variables are needed and wihch can be optional. Thanks.

So the usage of location in creating the resource group name is the issue and not the fact it's listed as East US. For terraform that is a valid value; however as shown in the error above the generation of the name is where the failure is taking place.

Yes. The PR is to correct the default for the required variable, as it isn't working.

Yes the PR is correct, I am trying to make sure the IAC team understands what is happening and giving them some guidance here. The documentation does say the location variable is required which you failed to provide. Someone will be looking at this in the near future. Thx.

Thanks for clarification @ranieuwe. I made another small update to location field present under modules/azure_aks/variables.tf.
I have tested the changes in this PR, good to be merged after approval.


Refactor names and delimiters of complex values in pretty-format

Summary
There are some inconsistencies in names of complex types for various options:

specific case for empty list is Arguments [] but general case with min: true is []
specific case for empty array is Array [] and so on, but general case with min: true is []
when hitMaxDepth for typed array is [Array] instead of [Uint32Array] and so on
when hitMaxDepth for constructed object is [Object] instead of '[' + (val.constructor ? val.constructor.name : 'Object') + ']' and I’m not sure tests cover this case

If y’all are willing to approve breaking changes to edge cases, I can submit an additional PR.
Whether or not, this PR applies advice from Kent Beck: move similar elements closer together.

Move empty arguments and array from printBasicValue to printComplexValue
Move complex value name and delimiters from helpers to printComplexValue
Therefore delete redundant helpers printArguments and printArray

Effect of refactoring on performance:

Time to print empty arguments or array increases similar to empty object. I think the reason for the original logic is to avoid computing indent when it isn’t used for those values. It has been confusing to me that the special cases in printBasicValue (where I don’t expect to find arguments or arrays) are inconsistent with the general cases in printCompexValue :(
Time saving from evaluating isToStringedArrayType only 1 time instead of 2 and calling printList directly for arguments and array is subtle, within the precision of measurement.

Test plan
Jest

Seems like circle is unhappy.

In case it helps, I will push another commit to fix this unrelated problem:
$ eslint . --cache --ext js,md

/Users/mark/gitforks/jest/packages/jest-haste-map/src/index.js
  831:1  error  Delete `······`  prettier/prettier

✖ 1 problem (1 error, 0 warnings)
  1 error, 0 warnings potentially fixable with the `--fix` option.

error Command failed with exit code 1.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@e6704bf). Click here to learn what that means.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master    #3986   +/-   ##
=========================================
  Coverage          ?   59.97%
=========================================
  Files             ?      196
  Lines             ?     6814
  Branches          ?        6
=========================================
  Hits              ?     4087
  Misses            ?     2724
  Partials          ?        3

Impacted Files
Coverage Δ

packages/jest-haste-map/src/index.js
92.96% <ø> (ø)

packages/pretty-format/src/index.js
97.98% <100%> (ø)

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 e6704bf...43a4664. Read the comment docs.

Happy to accept a PR for breaking changes when you are making edge cases more consistent.


Feature Request : Mark files as sensibles (like files with passwords)

I'd love a feature where we can right click on a file and mark it as "Sensitive", so everytime we open it there is a confirmation button to click which unlock the contents.
It's pretty useful when we are streaming our screen or people are next to the computer, and we click inadvertently on a file containing credentials.

Really useful, looking forward to see it implemented in near future

Hello, I would like to contribute to this. Is it still open/needed?


Store instance of Destination in DestinationConfig

Fixes https://github.com/microsoft/reverse-proxy/issues/788

This is consistent with ClusterConfig. The open question is if DestinationConfig serves any purpose or if it can be eliminated? It does not track any additional state. I'll bring this up in our Tuesday meeting and get back to you.

This is consistent with ClusterConfig. The open question is if DestinationConfig serves any purpose or if it can be eliminated? It does not track any additional state. I'll bring this up in our Tuesday meeting and get back to you.

Maybe it could be used for custom load balancing policies that need state per destination.

This is consistent with ClusterConfig. The open question is if DestinationConfig serves any purpose or if it can be eliminated? It does not track any additional state. I'll bring this up in our Tuesday meeting and get back to you.

Maybe it could be used for custom load balancing policies that need state per destination.

When we looked at doing something similar for passive health checks we found that using ConditionalWeakTable to store the data locally worked better.
https://github.com/microsoft/reverse-proxy/blob/1fb29bbe27ff316f06996bc08eabc17cb0e79282/src/ReverseProxy/Service/HealthChecks/TransportFailureRateHealthPolicy.cs#L34

We discussed this briefly but didn't come to any conclusions. We'll revisit it when we do a fine grained API review.


[People Picker] fix #234

Q
A

Bug fix?
[x]

New feature?
[ ]

New sample?
[ ]

Related issues?
fixes #234

What's in this Pull Request?
This patch attempts to fix #234 by

update the state with nextProps, if there is an update in the component
don't skip updating selectedPersons even if defaultSelectedUsers has no length

Codecov Report

Merging #236 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #236   +/-   ##
=======================================
  Coverage   72.71%   72.71%
=======================================
  Files          16       16
  Lines         744      744
  Branches      138      148   +10
=======================================
  Hits          541      541
  Misses        159      159
  Partials       44       44

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 3bb83cc...ee93225. Read the comment docs.

Thanks @tsekityam, I manually merged this to the DEV branch. You created the PR to the master branch.


Fill in the tags for empty quotes or add extra tags to completed quotes

Hi! Can I work on this issue? I need some description.

@aayushi221 You can work on this. Take a look at the updated description and let me know if you are clear.

I want to work on this one please.

This is an open issue! Go ahead. All PRs will be merged after october 1st. Happy Hacktoberfest.

I want to work on this issue.

Hey @devcer I have updated the tags, kindly review the pull request

@aayushi221 Your PR has merge conflicts. please resolve them to merge it.

hey! can i contribute

@shivarajloni Sure you can.


Fix mem cache TTL

This fixes an issue where the memory cache was updating the last fetched date even when the cache is using TTL. The disk cache works this way and the memory cache should function in a comparable way.

How do we feel about this bug fix?

@jerelevine sorry for the slow response! It's a good bug fix for the current behavior!
However I'm not sure that behavior is actually correct, there's a lot of discussion here if you're interested in participating: https://github.com/pinterest/PINCache/pull/209

@jerelevine now that #209 has landed, is this PR necessary?

I'm going to close this out, lemme know if you want to re-open!


Problem with hci_uart and L2CAP CoC connections

Hi, Zephyr community!
I am experiencing connection issues between an Android 10 device and the hci_uart example running on an nrf PCA10040 evaluation kit. The problem arises when I use L2CAP channel connection from the android app side, as Bluez states that the PSM value used to create the connection is not supported.
Btmon log:
> ACL Data RX: Handle 0 flags 0x02 dlen 18

LE L2CAP: LE Connection Request (0x14) ident 2 len 10

PSM: 1 (0x0001)
Source CID: 83
MTU: 65535
MPS: 1021
Credits: 65535

< ACL Data TX: Handle 0 flags 0x00 dlen 18

LE L2CAP: LE Connection Response (0x15) ident 2 len 10

Destination CID: 0
MTU: 0
MPS: 0
Credits: 0
Result: Connection refused - PSM not supported (0x0002)

I have tried several PSM values to solve this, but without luck. I'm starting to wonder if Zephyr's hci_uart supports L2CAP CoC connections, or is my Zephyr configuration wrong, or is this a Bluez issue?
With regards,
James

hci_uart only provides an HCI transport between a host (e.g. BlueZ) and the Zephyr controller. It's completely agnostic to what traffic flows over this transport on higher layers like L2CAP (i.e. hci_uart knows absolutely nothing about L2CAP). So I don't think the issue can be with the hci_uart application itself, rather most likely something in BlueZ or Android.

Thanks @jhedberg,
I saw no documentation on Bluez about assigning my own dynamic PSMs, so I will try rebuilding my own driver in an embedded device and let you know if this can be bypassed or not.
Will keep you updated.
WR,

Hi, Zephyr community and @jhedberg ,
After digging around for information about Bluez, I was unable to assign dynamic PSMs myself, so I wrote a custom HCI driver on an embedded system. Thus I was able to generate custom PSMs as I liked and I got it to work, so this is likely a Bluez limitation.
Thanks for thinking along. This issue can now be closed.
WR,

@jjamesson IIRC BlueZ gives you dynamic PSMs when you set the PSM to 0 in the struct sockaddr_l2
Anyway, I'll close this then since it's not a Zephyr issue.


Add test for magnetics module

Please provide a complete description.
It doesn't just add a test but also functionality


chore(deps): Updated deps and keeping maximum backwards compatibility

This is an alternative PR to #126 to try and keep more backwards compatibility

This also fails with Elixir < 1.7. Closing.


Live stream subtitles of Das Erste does not work

Which channel does not work correctly?
Das Erste (ARD)
What is the problem?
Live stream subtitles of channel Das Erste no longer work at all. Tatort, Tagesschau and so on - no subtitles. CC button offers only "Unknown"/"Unbekannt".
Mediathek versions of shows work as expected, with subtitles.
No problems with subtitles with ARD Mediathek app.

Thank you very much for your bug report and sorry for the late feedback.
Sadly ARD decided to burn their subtitles into a special video stream instead of providing a subtitle layer in addition to the video stream. There is no easy fix for Zapp, so it will remain this way until subtitles are privided the "proper" way again.

Thank you for your explanation. Unfortunately, I have not noticed this (badly) change.

Sadly ARD decided to burn their subtitles into a special video stream instead of providing a subtitle layer in addition to the video stream.

It does not seem to be a general problem of the ARD. Most (all?) "third programs" of the ARD provides subtitles which are usable with Zapp.
phoenix (ARD/ZDF), one (ARD), and alpha (ARD/BR) has subtitles – also in Zapp.

@cemrich The subtitles of Das Erste are back. I only noticed it today because I had mostly used the ARD Mediathek app. Did you solve the problem with one of the 7s releases?


Added option to specify certificate format

When issuing certificates or signing CSRs, the format was hard-coded to "der".
This pull request introduces a way to specify a different format, e.g. "pem".

We use DER-encoded certificates to be able to return the certificate as X509Certificate or KeyStore including the appropriate return type. Changing the format will cause our utility methods to no longer work. That should also work with pem-encoded certificates.
Requesting pem_bundle when issuing a certificate should return a CertificateBundle instead of just Certificate type.
That being said, the PR doesn't fit conceptually the Spring Vault API. If you need to request different certificate types, it would be easier to make createIssueRequest(…) public instead.

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.


DefaultDapsDriver vs TokenManagerService

The de.fhg.aisec.ids.idscp2.drivers.default_driver_impl.daps.DefaultDapsDriver seems to duplicate functionality of de.fhg.aisec.ids.tokenmanager.TokenManagerService - except that the DefaultDapsDriver seems to use the "old" DAPS token endpoint - which seems to fail.

Correct. We need to change this to a binding between those modules. It doesn't make sense that this functionality is duplicated.


Needs reporter support

Currently the test result is directly dumped into process.stdout.
However, in order to integrate with CI, reporters like junit XML formats is required.

Yeah we did not gave it a much thought. And at the moment we are more focused on extending the support for complex webpack setups. So adding an api that support reporters is not a priority.
I think this will not prevent it from using in common CI's like circle ci or travis though? I am not familiar with junit XML formats.
But it would be cool to have reporters working with this. So pull requests are always welcome :)

Seems that storyshots 3.0.0 is integrated with jest, using jest's reporter, which connects with CI painlessly. Hooray!


Remove Demo Info for V1

Fixes #402
Removed demo info from:

Home Page
Chapter Page

Sizing of the buttons is off.

I think the button size is defined by the text. Is there anyway for them to be the same size regardless of text?

This is a tACK for me.

@sabakhilji I think will be resolved in #407


Nice work!

Hi there,
Stumbled across your library while I was trying to debug a pipeline problem I was having, and it was a huge help.
Just wanted to say thanks :)

Quick update: I've started extending your project. I started with an envelope follower, and I'm moving on to a vocoder.
This is my first experience with React, believe it or not...it's been fun, and I like how you have thing set up.

@shyndman cool! Feel free to open pull requests here with new modules, I am open to collaboration on this project.

Oh, great.
Do you have a lot of experience in this area? I'm new to it, but eager to learn more. I'm basically reading everything I can get my hands on.

I have a lot of experience with React. However, when it comes to Web Audio API and Digital Signal Processing this project was my way of learning them.

Well, I like your React. Quick demo video of the vocoder. It's not perfect yet, but getting there.
Ignore the big visualization. It's there to help me debug the thing (and is actually likely to get a lot more complex).
https://user-images.githubusercontent.com/42326/142569195-228e871a-d8c4-4d5f-aa92-f4e0d84b7fa7.mov

Nice! 🔥
I would not ignore the visualisation as my primary need for this project was to visualise the signal chain and understand what effect various components have on the signal – that's why I added Analyser and Meter.

Thanks! I've kept going all day on it.
I've had the same desire already. I was thinking fancy tooltips on the
connection ports. Frequency spectrums for signal ports, scalars for
a-rates....and maybe a scrolling line graph for the k-rates? Just thinking
about loud here. I wonder if we could even animate the patch cables in some
interesting way...
Man...that standardized-audio-context library really threw me off. I
was seeing way more worklet processors being created than what I expected.
I had no idea what it was — debugged the worklet-loader, and the related
framework code, without success.
So I started digging into that library, and after a lot of reading, I found
that it creates a "backup" audio context in a situation that I was
regularly encountering, and my processors were being kicked off in two
places at roughly the same time. Soooo, basically, it wasn't a problem at
all...but was a lot of work to figure it out. :)
Also, do you find the build times kind of slow? My laptop is terrible, but
I thought you might have tips in any case.
Scott
On Fri, Nov 19, 2021 at 1:36 PM Michał Ochman @.***>
wrote:

Nice! 🔥
I would not ignore the visualisation as my primary need for this project
was to visualise the signal chain and understand what effect various
components have on the signal – that's why I added Analyser and Meter.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/michalochman/react-web-audio-graph/issues/8#issuecomment-974312737,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAKKVWK3XWU5RCGYUZUMJTUM2KLTANCNFSM5HQMNU5Q
.

I was thinking fancy tooltips on the connection ports. Frequency spectrums for signal ports, scalars for a-rates....and maybe a scrolling line graph for the k-rates? Just thinking about loud here. I wonder if we could even animate the patch cables in some interesting way...

I'm up for building more tools.

Man...that standardized-audio-context library really threw me off. I
was seeing way more worklet processors being created than what I expected.
I had no idea what it was — debugged the worklet-loader, and the related
framework code, without success.
So I started digging into that library, and after a lot of reading, I found
that it creates a "backup" audio context in a situation that I was
regularly encountering, and my processors were being kicked off in two
places at roughly the same time. Soooo, basically, it wasn't a problem at
all...but was a lot of work to figure it out. :)

Good to know you figured it out. I have added standardized-audio-context, so the app can work on browsers other than Chrome. It was not easy from TypeScript perspective, but I think was worth doing.

Also, do you find the build times kind of slow? My laptop is terrible, but
I thought you might have tips in any case.

Build times are not bad for me, the app is usually available within 10-15 seconds of starting. However, my laptop is trying to fly away if I have a lot of analysers and meters in the graph.

Build times are not bad for me, the app is usually available within 10-15 seconds of starting. However, my laptop is trying to fly away if I have a lot of analysers and meters in the graph.

So I discovered something that explains that a bit. Every time an AudioWorkletNode is created, you'd expect to see a single AudioWorkletProcessor, but we're actually getting 2. And they don't get cleaned up properly. It impacts every processor.
I have confirmed that hot module reloading is to blame. I got a lot of headroom for adding more nodes.
Release builds were never affected, because no HMR.

Something firing twice in development mode sounds a lot like StrictMode
component. Can you see if you can reproduce this behaviour even with this
component removed from main index.tsx file?
On Sat, 20 Nov 2021 at 20:32, Scott Hyndman @.***>
wrote:

Build times are not bad for me, the app is usually available within 10-15
seconds of starting. However, my laptop is trying to fly away if I have a
lot of analysers and meters in the graph.
So I discovered something that explains that a bit. Every time an
AudioWorkletNode is created, you'd expect to see a single
AudioWorkletProcessor, but we're actually getting 2. And they don't get
cleaned up properly. It impacts every processor.
I have confirmed that hot module reloading is to blame. I got a lot of
headroom for adding more nodes.
Release builds were never affected, because no HMR.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/michalochman/react-web-audio-graph/issues/8#issuecomment-974700543,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAMNSIUGEFC7BXSYHDQVPNLUM7ZV7ANCNFSM5HQMNU5Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

It isn't firing twice. The WorkletAudioNode is only being constructed once on the react side. Two worklets appear.

Side thought: I'd like to see tweakable gain on every input and output. I expect the operation is basically free, especially in the gain 0 and 1 cases.

I would suggest creating new issues for feature requests – it will be easier to keep track of them. For instance, the idea of black-boxed nodes (#3) is to be able to define "complex" nodes made out of "primitive" nodes, e. g. an oscillator node connected to gain input with the gain output being the only output of the complex node. In my opinion, it would cover the scenario you are describing. If you look at the DelayEffect component, it's how it works already. However, I would like this to be defined during runtime rather than in code.
I am not particularly attached to the "black-boxed" name, it mostly comes from black-box testing, where we describe the system by its inputs and outputs, not its internal components. However, I think the name would serve well.

The vocoder is 87 nodes internally. 😮  That number will grow.
I hear you on defining in the UI, but it's still too clunky...I have some ideas on that too.
OK, I will add to your feature requests.
Also, want a reproduction on the bug I'm talking about? You can see for yourself.

Also, want a reproduction on the bug I'm talking about? You can see for yourself.

Sure, why not.


Debounce search input

This should fix #29. I also upgraded hiredis to be able to run on El Capitan with Node 5.

Thanks! Will try to get this released later today or tomorrow.


test: define more tests

artifact checkout
health
object push
single tag
basic builds

Revert t odraft, pending tg::directory! from #302


Allow configuring auth settings in local development

Improve documentation
Link
Local development
Describe the problem
This page does not describe how to change auth settings when I'm running supabase in local dev. I'd like to change settings like requiring the email address to be confirmed before logging in and enable additional auth providers.
Describe the improvement
Adding instructions on how to change the auth settings (if this is possible).
Additional context
Issue field per this request on Discord.

Thanks for opening this @emckay! I was waiting for the OSS dashboard integration to see if I can offload the settings there, but looks like you'd have to use container env vars both for the CLI and self-host.
This is currently not possible, so I'm turning this issue into a feature request instead.

hi, any update on this or a temporary work around?

@soedirgo any update here?


VCP fails to dynamically provision storage with error NoPermission

Is this a BUG REPORT or FEATURE REQUEST?:
/kind bug
What happened:
VCP is unable to provision dynamic storage, static works fine
What you expected to happen:
VCP to successfully provision storage dynamically
How to reproduce it (as minimally and precisely as possible):
Set up VCP according to documentation here, using the vSphere permissions here. Create storage class, create a claim using the storage class, claim fails to provision storage with error NoPermission
Anything else we need to know?:
relevent logs(?):
I0924 13:13:12.911381       1 vsphere_util.go:123] error "NoPermission" getting accessible datastores for node &{NodeName:infranode1 vm:0xc42720f2b0 VMUUID:}
E0924 13:13:12.911391       1 vsphere.go:1077] Failed to get shared datastore: NoPermission
I0924 13:13:12.911400       1 vsphere.go:1129] The canonical volume path for the newly created vSphere volume is ""
I0924 13:13:12.911413       1 pv_controller.go:1433] failed to provision volume for claim "test-project/sdfawefw" with StorageClass "vsphere-nprd": NoPermission
I0924 13:13:12.911445       1 event.go:218] Event(v1.ObjectReference{Kind:"PersistentVolumeClaim", Namespace:"test-project", Name:"sdfawefw", UID:"26a10844-bffb-11e8-beda-005056964b02", APIVersion:"v1", ResourceVersion:"1579982", FieldPath:""}): type: 'Warning' reason: 'ProvisioningFailed' Failed to provision volume with StorageClass "vsphere-nprd": NoPermission

Static provisioning works, (create disk, create pv, create pvc) but something about the dynamic provisioning portion is missing privileges and I don't know what or how to find out what it is.
Possibly related to https://github.com/kubernetes/kubernetes/issues/67206?
Environment:

Kubernetes version (use kubectl version):
Openshift 3.10
Client Version: version.Info{Major:"1", Minor:"10+", GitVersion:"v1.10.0+b81c8f8", GitCommit:"b81c8f8", GitTreeState:"clean", BuildDate:"2018-09-07T19:14:28Z", GoVersion:"go1.9.4", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"10+", GitVersion:"v1.10.0+b81c8f8", GitCommit:"b81c8f8", GitTreeState:"clean", BuildDate:"2018-09-07T19:14:28Z", GoVersion:"go1.9.4", Compiler:"gc", Platform:"linux/amd64"}
Cloud provider or hardware configuration:
vSphere ESXi 6.5 (VM version 13)
OS (e.g. from /etc/os-release):
Red Hat 7.5
Kernel (e.g. uname -a):
3.10.0-862.14.4.el7.x86_64 #1 SMP Fri Sep 21 09:07:21 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux
Install tools:
Openshift-ansible playbook
Others:
vsphere.conf
vsphere-nprd.yaml

Any news here? We  have also the issue with dynamically provision storage not working.

No sadly, I've pretty much given up on the dynamic portion, our VMware admins think the account is set up correctly according to the permissions required for dynamic provisioning without storage policy based placement, and they don't want to give the account the permissions required for that so I'm stuck.

@benhwebster
Do you have System.View privilege on the user. This is required to call RetrieveProperties
This is failing at https://github.com/kubernetes/kubernetes/blob/master/pkg/cloudprovider/providers/vsphere/vclib/virtualmachine.go#L214

Finally found the missing permission! we were missing read-only permission on one of the ESXi clusters running the hosts running the infrastructure nodes (or one of the children of the cluster, we let it propagate, you could probably just add ESXi cluster as one of entities for the read-only role in those tables.) Thank you Divyen for pointing me in the right direction!


Catch IOException parsing manifest

Description of Change
Catch and log IOException when parsing manifest for two reasons:

To prevent an ExceptionCollection exception being thrown by engine.analyzeDependencies() when analyzing unreadable jars/zips. This is useful because these errors could be common (x) and unnecessarily bloat any ExceptionCollection. Also it would make it easier to determine whether to continue or not especially for external uses of engine.analyzeDependencies().

(x) when compressing files on macOS it creates mirror ._ files (same filename except with the ._ prefix), these files aren't actual copies they just contain file information, but if Dependency Check  sees ._some_file.jar it will fail to parse its manifest throwing an IOException because it's unreadable.

To be consistent with other usages of try (JarFile jar = new JarFile(dependency.getActualFilePath()))

https://github.com/jeremylong/DependencyCheck/blob/fb2b3159e8dedaad06cb3b6f667975616fbd848f/dependency-check-core/src/main/java/org/owasp/dependencycheck/analyzer/JarAnalyzer.java#L313
https://github.com/jeremylong/DependencyCheck/blob/fb2b3159e8dedaad06cb3b6f667975616fbd848f/dependency-check-core/src/main/java/org/owasp/dependencycheck/analyzer/JarAnalyzer.java#L914
Have test cases been added to cover the new functionality?
yes

Are you sure you opened this PR against the right repository? For DepCheck to actually consume it, it should target jeremylong/DependencyCheck (jeremylong:master).

Are you sure you opened this PR against the right repository? For DepCheck to actually consume it, it should target jeremylong/DependencyCheck (jeremylong:master).

Yeah it was intentional I didn't want to open this just yet in case we preferred the other PR

Closing as I've created a new PR directed towards jeremylong:master https://github.com/jeremylong/DependencyCheck/pull/848


solve the issue of image loading

in the humidity image was not loading when we open on the live server

Thanks for the PR. It has been merged.
I will be glad if you can leave a star on this repository.


feat: install scarb. first curl, second asdf

https://docs.starknet.io/documentation/quick_start/environment_setup/#installing_scarb
https://github.com/cairo-book/cairo-book/blob/1a6fda11d197e20d318531ce861d6c943951b842/src/ch01-01-installation.md?plain=1#L48
asdf global scarb 2.5.3
$ scarb --version
scarb 2.5.4 (28dee92c8 2024-02-14)
cairo: 2.5.4 (https://crates.io/crates/cairo-lang-compiler/2.5.4)
sierra: 1.4.0

asdf install scarb  not working
asdf install
$ which scarb
$HOME/.asdf/shims/scarb
curl install
$ which scarb
$HOME/.local/bin/scarb

I don't understand the issue

As a beginner, follow the starkio documentation to install scarb, and then use asdf to install scarb. scarb was still installed in the starkio documentation, and using asdf to install did not work.

I still don't understand what the issue is. The instructions in the book are correct.

Mainly tell beginners to know how to view the installation directory of A.
Just close it if you don't need it.


#127: added hints for reason submission

Added hints as discussed in #127 .
One thing I'm not certain is whether we should use 'yellow' or not. Yellow (#FFFF00) is a bit too bright for me, but gold (#FFD700) is not strong enough as a warning.

Pull Request Test Coverage Report for Build 489

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 87.55%

Totals

Change from base Build 481:
0.0%

Covered Lines:
151

Relevant Lines:
168

💛  - Coveralls

這個 PR 居然開 22 天了！
感謝 @nonumpa 提醒 QQ

Updated. I decide to keep the button yellow.


fmt not allowing repeated use of placeholders

So I just tried to use fmt for defining a snippet, and I came across the issue that it's not possible to reuse placeholders.
Lets say I have this snippet.

local ls = require('luasnip')
local s = ls.snippet
local fmt = require('luasnip.extras.fmt').fmt
local i = ls.insert_node

local snippets = {}

snippets.typescriptreact = {
    s(
        'component',
        fmt(
            [[
            import React from 'react';
            const {filename} = () => {{
              {insert}
            }}

            export default {filename};
        ]],
            {
                insert = i(0),
                filename = f(function()
                    return vim.fn.expand('%:t:r')
                end, {}),
            }
        )
    ),
}

ls.snippets = snippets

I then create a file named Test.tsx, and invoke the snippet.
The snippet then expands to:
import React from 'react';
const  = () => {

}

export default Test;

If I remove the last use of {filename} in the snippet, it will correctly place the filename in const {filename} = () => {

Hah, of course:
The nodes aren't copied anywhere, meaning that the same object ends up two times in the nodes-list of the snippet (which can wreak all sorts of havoc).
Definitely a bug, thanks for opening an issue about it👍


SocketChangeTracker not allowing the server to have a self-signed cert

I found and fixed some Obj-C replicator bugs today. A few of them might also appear in the Java (and C#) code; it’d be worth taking a look.
couchbase/couchbase-lite-ios@d222ba1 — The SocketChangeTracker wasn’t allowing the server to have a self-signed cert even if the replicator had been configured to allow that cert. This may or may not be a cross-platform issue but it’s a case that should be tested for.
couchbase/couchbase-lite-ios@5e8b5c7, couchbase/couchbase-lite-ios@500fa80 — Added some unit tests, to test auth failures and SSL.

@tleyden We would actually like to use a java client with a self-signed cert. Do you know if this is a difficult thing to get working (i.e. could we reasonably patch this ourselves)? For parity with iOS I imagine you'd want some static field on Replicator that could allow additional certs.

Just to be clear, we're getting a pretty vanilla exception:
Jul 06, 2015 2:16:04 AM com.couchbase.lite.util.SystemLogger e
SEVERE: RemoteRequest: io exception.  url: https://<IP_ADDRESS>:4982/db_3_19_14_2/_session
javax.net.ssl.SSLPeerUnverifiedException: peer not authenticated
	at sun.security.ssl.SSLSessionImpl.getPeerCertificates(SSLSessionImpl.java:421)
	at org.apache.http.conn.ssl.AbstractVerifier.verify(AbstractVerifier.java:129)
	at org.apache.http.conn.ssl.SSLSocketFactory.connectSocket(SSLSocketFactory.java:322)
	at org.apache.http.impl.conn.DefaultClientConnectionOperator.openConnection(DefaultClientConnectionOperator.java:129)
	at org.apache.http.impl.conn.AbstractPoolEntry.open(AbstractPoolEntry.java:164)
	at org.apache.http.impl.conn.AbstractPooledConnAdapter.open(AbstractPooledConnAdapter.java:119)
	at org.apache.http.impl.client.DefaultRequestDirector.execute(DefaultRequestDirector.java:347)
	at org.apache.http.impl.client.AbstractHttpClient.execute(AbstractHttpClient.java:555)
	at org.apache.http.impl.client.AbstractHttpClient.execute(AbstractHttpClient.java:487)
	at org.apache.http.impl.client.AbstractHttpClient.execute(AbstractHttpClient.java:465)
	at com.couchbase.lite.support.RemoteRequest.executeRequest(RemoteRequest.java:184)
	at com.couchbase.lite.support.RemoteRequest.run(RemoteRequest.java:103)
	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:471)
	at java.util.concurrent.FutureTask.run(FutureTask.java:262)
	at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.access$201(ScheduledThreadPoolExecutor.java:178)
	at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:292)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
	at java.lang.Thread.run(Thread.java:745)

Hi @sapieneptus,
Following links might help you.
https://github.com/couchbase/couchbase-lite-java-core/pull/9
https://forums.couchbase.com/t/android-401-error-when-trying-to-connect-to-sync-gateway-custom-key-store/4046

@hideki On doing
AssetManager am = PocApplication.applicationContext.getAssets();
    InputStream is = am.open("my_cert.pem");

    // Load CAs from an InputStream
    CertificateFactory cf = CertificateFactory.getInstance("X.509");
    InputStream caInput = new BufferedInputStream(is);
    Certificate certificate;
    try {
        certificate = cf.generateCertificate(caInput);
    } finally {
        caInput.close();
    }

I get java.security.cert.CertificateException :
com.android.org.conscrypt.OpenSSLX509CertificateFactory$ParsingException: com.android.org.conscrypt.OpenSSLX509CertificateFactory$ParsingException: java.lang.RuntimeException: error:0D0680A8:asn1 encoding routines:ASN1_CHECK_TLEN:wrong tag
Anyone encountered this? using own bundled certificate in android environment..

Hi @NandishAndDev,
it seems certificate file formatting issue. see following link.
http://stackoverflow.com/questions/21138485/certificateexception-opensslx509certificatefactoryparsingexception

@hideki yeah the cert content was not liked by the api.. now its resloved and cblite is able to do https connections..

Todo: Need to review current codes.


form.on("part") is called repeatedly?

Hi I've just started looking at this library, and wasn't sure if the form.on("part", listener) is called repeatedly or called once when the form data is first found. I need a clarification since i'm making small module to abstract all multipart parsing. I need it to basically accept req and callback function and call callback function with readable stream when is found.
It would kinda break my api if callback was called repeatedly with 'partial' form stream, which I presume is the case since you can access its bytecount through passed part.
I just need a nice one readable stream that I can pass along to other parts of my application.
Is this true? and if so is there a way that I can receive one readable stream instead of partials?
Thanks you in advance!

Hopefully the docs make it clear, and if not, let me know what we need to change to clarify: https://github.com/andrewrk/node-multiparty#part-part
The part event is emitted for every single part on the form. A "part" is either a field or a file. If a form contains 4 files and 1 field, then the part event will be emitted 5 times.

If you want to just get one callback when everything is done, please see the example in the readme: https://github.com/andrewrk/node-multiparty#usage

Thanks for quick response! It probably is more due to the fact that I never had to deal with multi-part previously and thus unfamiliar. So according to your first comment, does that mean that if I upload one image file from client, part event will be called twice? once for field and once for raw data?

Or is it arbitrarily partitioned by what client decides to send per request by its file size or whatnot?
Also as to your second comment, does that temporarily save to disk or save in memory?

So according to your first comment, does that mean that if I upload one image file from client, part event will be called twice? once for field and once for raw data?

No, if there is truely only one image and nothing else in the form then it'll only be called once. If there is something else in the form, like a field, then you'll get a second call for that other item.

Or is it arbitrarily partitioned by what client decides to send per request by its file size or whatnot?

It's not arbitrarily, it's literally once per each "part" of the form. A "part" is either a field or a file. If a form contains 4 files and 1 field, then the part event will be emitted 5 times.

Also as to your second comment, does that temporarily save to disk or save in memory?

To disk, so a client that sends you a 4GB file won't crash your app.

Aha! Thanks for your clarification. Just one last question. Is there a way that I can enforce / check if the part is called once only? that is, client sends one thing only per each request. This is just to make module compliant with what other parts of my application expect it to do.
I guess I can set up a simple flag variable and check it but anything internally implemented?

So remember, you can always use the callback style used in the example I linked to instead of using the part event. If you really want to still use the part event and you only ever want a single part (I assume you want that one and only one to be a file?), just use the restricting options we provide: https://github.com/andrewrk/node-multiparty#multipartyform
Set maxFields to 1.

Yeah I guess the callback style makes more sense as to my case. Thanks for your help.
Great work btw!


compileSdkVersion is not specified. Please add it to build.gradle

Describe the bug
I am having build error on the latest ver 12.0.0
* What went wrong:
A problem occurred configuring project ':react-native-camera-kit'.
> compileSdkVersion is not specified. Please add it to build.gradle

But I already have it on my build.gradle which already works on the previous ver 12.0.0-beta5 that I am using which is no longer here.
It's the same app I just upgraded the ver and rebuild the app and this error is now telling me too add the compileSdkVersion which I already have.
build.gradle
buildToolsVersion = "31.0.0-rc1"
minSdkVersion = 21
compileSdkVersion = 31
targetSdkVersion = 31
ndkVersion = "21.4.7075529"

app/build.gradle
android {
    compileSdkVersion rootProject.ext.compileSdkVersion

    compileOptions {
        sourceCompatibility JavaVersion.VERSION_1_8
        targetCompatibility JavaVersion.VERSION_1_8
    }

    defaultConfig {
        applicationId "com.xxxxx"
        minSdkVersion rootProject.ext.minSdkVersion
        targetSdkVersion rootProject.ext.targetSdkVersion
        versionCode 1
        versionName "0.0.1"
    }

How do I fix this?
Smartphone (please complete the following information):

Device: Honor 10, 8 mobile and Google pixel 2 emulator

OS: Android 10

RN ver: 0.63.4

same issue...any solution?

I added this steps then it works:
https://github.com/teslamotors/react-native-camera-kit/blob/master/docs/kotlin.md

I have the samme issue, compileSdkVersion not specified, and when I add those kotlin steps, I just get different errors.. I think this has to do with gradle versions, can anyone provide @unit-002 can you pls provide gradle version used?

I have this versions on my build gradle
buildToolsVersion = "31.0.0-rc1"
minSdkVersion = 23
compileSdkVersion = 31
targetSdkVersion = 31
ndkVersion = "21.4.7075529"

Same issue, any solution yet?

Same issue. Any one found a solution yet ?

same issue anyone have resolve it .

Same issue any one has resolved it

Hello Everyone,
I am new to react native development. when i am building and running my code, it is working nicely in ios but i don't know what is the problem, i am not able to fix the problem of android app. After running "yarn android" it's shows me the error mentioned below in an image. I tried many solution from git and stack overflow but nothing helps. I hope i will get some help from here so i will find out where i am doing wrong. I have attached other configuration details for further details.
Please help me out i am struggle from last two days.
Any kind of help or suggestion will be appreciable

Same issue.

Here is the solution worked for me
Add compileSdkVersion which is located in android/build.gradle to android/app/build.gradle
For my case it was like this in android/build.gradle:
buildscript {     ext {            ...            compileSdkVersion = 33          } }
And I changed my android/app/build.gradle to:
defaultConfig {  applicationId "com.mybulapp"  ...  compileSdkVersion 33 }
Also ignore the deprecated error

had the same problem. solved it by adding the following line to default config in app/build.gradle
compileSdkVersion rootProject.ext.compileSdkVersion
for those who might struggle to find defaultConfig, you'll find it inside android { }

For me, just loading project in Android Studio and running gradle sync to download everything it needs solved the issue


Insult with correct spelling

Nobody is looking to insult anyone. A better word would be klåpare.

test


Enhancing lightbox directive for generic directive calls

Removed hardcoded instances of picture.image_url and picture.medium_url from the ofLightBox directive.
Added thumbnailUrl and displayUrl which serve as final urls to be added into the templates for the Light Box directive
This is a generic change in code so that calls to this directive can be made without changing js code.

Coverage remained the same at 99.297% when pulling ef095f4353006e1dde6963fa4f1c38b5917650a3 on TanSA05:tansa05_enhancelightbox into 9b6ff7498fb083200764289378d6369aef7ccc9b on openfarmcc:master.

Awesome, good job @TanSA05!


Write more variables into zlib.pc

Added version, that is parsed from zlib.h and includedir and libdir

Some C/C++ programs have configure scripts that check zlib versions. This PR just fills some of those placeholder so zlib.pc can be used by configure script.

I thought about this and have to consider it a backwards-incompatible change as this will crash hard if a header file provided by any system does not contain the desired information, information that previously wasn't provided.

Making zlib.pc is only done when compiling from source, which is part of this repo, so zlib.h will always contain that information, that's how zlib's build system is doing it. But better safe than sorry, so there will be no panic on version fail.

No worries, git stores previous commits too.

assuming that you are sure that the generated includedir and libdir variables will be correct on any system.

If prefix is valid than includedir and libdir should be valid too.

Thanks again!


In the storyboard, when I set a label to TTTAttributedLabel, all the interfaces in the controller become white

In the storyboard, when I set a label to TTTAttributedLabel, all the interfaces in the controller become white, a pure white interface but the elements inside still exist.

I had the same problem. Since the project seems discontinued, here is a very simple replacement which subclasses UITextView: https://stackoverflow.com/questions/50505334/how-can-i-accurately-detect-if-a-link-is-clicked-inside-uilabels-in-swift-4/50873577#50873577

@JJNev  Thank you, the solution you provided is also good, it is equivalent to self-realization of this function, I hope TTTAttributedLabel can solve this problem


Add example for bzlmod and fix usage

Add an example for how to consume via bzlmod and fixes https://github.com/mvukov/rules_ros/issues/32 by removing the dev_dependency = True from rules_python.
Also bumps rules_python to the latest version.

Not sure why adding a new example breaks existing examples

@mvukov I cannot reproduce the issue locally. Any idea?

Addressed in #43 , I believe.


Implement BoxesController routes, views, and methods

There is a lot of work to do on this one. See below:

[ ] Write specs
[x] Write controller methods
[x] Write views
[x] Add helper methods for athletes (will make the views easier to write)

Specs could still use some polish (and the actual test code), but I am going to move that to a new PR. Merging this.


Rename Clear and Save As buttons

Current behavior
Distance and Direction has Clear and Save As buttons, The same functionality on the Coordinate Conversion pane are named Clear All and Export. Requesting a change to match Coordinate Conversion.
Expected behavior
Rename Clear and Save As buttons to Clear All and Export

Research to see if standard way/terminology is used across Pro

looks good

[ ] Defense What's new page PR8015 must be merged and built first
[ ] Distance and Direction October Resources
[ ] \DefenseSolutions\DistanceAndDirection\October2019\Dev\108\
[ ] Military Tools for ArcGIS October Resources
[ ] \DefenseSolutions\MilitaryToolsforArcGIS\October2019\Dev\333\


Firefox - scrolling placeholders don't work

The Ctrl and Alt + wheel placeholders don't work in Firefox

Should be fixed now!


UnboundLocalError local variable 'holdings_id'

I'm getting the following:
File "ae_gui", line 30, in update_it
update(settings.mms_id, settings.input_file, settings.error_file, settings.api_key, settings.base_url)
File "C:\AlmaEnumerator\alma_enumerator-master\AE.py", line 524, in update
items_to_update = get_info_from_csv(input_file)
File "C:\AlmaEnumerator\alma_enumerator-master\AE.py", line 477, in get_info_from_csv
item_info[holdings_id].append(dict(zip(keys, info)))
UnboundLocalError: local variable 'holdings_id' referenced before assignment
I've tried a couple different things, but just get various errors.

Thanks for reaching out about this bug! Would you be able to provide the input file you're using to update your records?
My initial guess is that there's a mismatch between the input CSV's format and what the get_info_from_csv function is expecting.

Sure. Here's the output file created after fetching the data
output.txt
Thanks for taking a look at this.

OK. The problem was the code was overly precise about identifying the holdings ID in the CSV. When I initially developed Alma Enumerator, all the holdings MMS IDs were 16 digits in my collection, so I matched for that. I've updated it so it should recognize any MMS ID of a valid length (8-19 digits.)
If you replace your AE.py file with the new one, that should take care of your problem. Let me know if it doesn't or if you encounter other issues.

Works perfectly! Thank you.


Spellcheck no longer working in other languages

Expected Behavior
When writing a full sentence in a language other than english, spellcheck should kick in and recognize the written language, then provide spell suggestions for that language. (this worked in all prior versions of Franz and was the main reason me and many in our office used Franz for instead of slack)
Current Behavior
When writing a full line, franz sticks to English spellcheck and does not switch. apart from engilsh spelling suggestions it suggests a list of gibberish words that don't really make sense.
Screenshots (if appropriate):

Possible Solution
Use the standard system libraries or the appropriate webkit library for spellcheck.
Steps to Reproduce (for bugs)

Write non-english sentence, Swedish example: "Hej där hur läget?"
All words get red marking
Right click with cursor over word
See list of english suggestions and random gibberish

Context
This (for me) is the unique selling point for Franz. Slack has had horrible spell check so far that only supports American English. As someone that switches between Icelandic, Norwegian, Swedish, Dutch and English, Spelling and grammar check is an awesome thing to have. Franz now seems moving into the same direction as slack.
Spellcheck worked like it does in Google Chrome in all older versions of Franz which was perfect
Your Environment

Franz Version used: 5.0.0.beta.10
Operating System and version: Mac OSX Sierra 10.12.6

I encountered the exact same thing when switching to version 5. This used to work like a charm in previous versions.

In my case (Beta 13), there's no spell check at all, not even in the language Franz is, nor English, nothing. No SpellCheck. Do I need to do anything to activate the Spell Check? And to select the multiple languages to check?
Thanks.

I have same problem here. I was on beta 13 and sticks with beta 14. Spellcheck only work in english.

@AGonLen, @kecinzer Please always add your OS to your comment. Otherwise it's impossible for me to test the issues.
Also, what is the language you want to write?

Sure!!!
When I wrote the message:
Franz Version used when wrote message: 5.0.0.beta.13
Operating System and version: Windows 10 Pro Fall Creators Update
Now:
Franz Version used now: 5.0.0.beta.14
And spell checker is working in my language (Spanish), and although I have Franz setup with English language, it does not correct.
Is it possible to select the "active" languages?
Can the same message/text have multiple languages?
Does it detect and change automatically from one language to another?

Right now the spellchecker selects the system language, not the Franz language. While Franz has support for a few languages. People might want to write in their native language even though Franz might not be available in that language.
I have started a test to do manual and automatic language selection. Unfortunately this is working so incredibly unreliable that I had to disable it for beta 14.

That explains, and it's quite some progress, many thanks BTW!!!! At least it will correct when I speak in my language, but would be good to be able to select 2-3 languages for people that might speak different languages on their chats/conversations.
The manual selection or languages (or even automatic if it could be done, similar to what Chrome does) would be awesome!!! :)

Also, would be good and great to create your know dictionary and add words that you use (specially names and things like that...)

I'm sorry. I'm on macOS 10.12.6 and Franz 5.0.0-beta.14. I'm using czech language. And even if I set czech language as system default for spell check, Franz keep checking in English.

Now the spellcheck menu is not showing when right clicking...
Sometimes it happened, but after a Franz close and restart was working, but now not working at all. For a fraction of a second I see the menu but then it's gone.
Franz Version used: 5.0.0.beta.14
Operating System and version: Windows 10 Pro Fall Creators Update

Same here.
Ubuntu 16.04
Version: 5.0.0-beta.14
Release: 1.7.9 / linux / x64
Brazilian Portuguese.

MacOS 10.13.6
Franz 5.0.0-beta.18.903
+1 for multi-language spellcheck. I live in Spain, but write daily in both English and Spanish. My system language and Franz language are set to English.
Very few Electron apps support multi-language spellcheck, I'm assuming it's difficult to implement. A few do however. Wavebox provides spellcheck for up to two languages. Not perfect, but better than being limited to just one.
Here's how they handle spellcheck configuration:

And here it is in use:

Glad to see Franz back in action! 👍

Any news about this "bug"?
I really enjoy Franz, and using only as a Gmail "client"; however, the very fact that it does not support any other language than English is really annoying. I speak Portuguese and most of my emails are in my native language.. I tried to change gmail language.. Franz language...gmail "check spelling".... and nothing appears to work...the worst is the red underline....  That's a shame because I would really like to keep using the app..
Btw,
Ubuntu 18.04
Version 5.0.0-beta.18

@pinheirof You can set your Ubuntu main language to Portuguese so you don't have your native language underlined.
In my case, I use 50/50 two different languages, so Franz is anoying to use, underlining 50% of my writings.
We need to have this "disable spellcheck" working, it sound like an easy thing to fix.
Also Ubuntu 18.04
Version 5.0.0-beta.18

MacOS 10.14, Slack Desktop app 3.3.2: Spell checker works only for English, doesn't work for Russian language, even if I manually select it.

It'd be great to be able to add languages like browsers can, or to use something native on certain platforms so you could extend it that way.  My issue isn't as bad as users of other languages, I just want en_AU, but it is annoying.

I have the same issue. Spell checker is not working. I could manage to get it working but doing this annoying procedure:

Start slack -> preferences -> Advanced
Change from automatic to a specific language.
Disable Spell checker and enable it again.

Doing this I get it working only when the desktop app is running. If I close the app I have to do the same things again.
Ps. I tried to reinstall the app from the apple store and it is the same


Blog post

Inaugural blog post. There will be lots of these. May need guidance on blog post header.

Cleaned up and applied text styling. Minimized/cleaned-up image.

I loved it at first sight, will give it a read ASAP.


Bump to node-sass 2.0.1 stable

I'll look into this. Either way multiple dependencies are in need of an update, so I'll look into that. I will also consider this an answer to your comment on #17.

@davidguttman I've prepared a branch (including this pull request) with updated dependencies. However, I'm not sure what version to use as the "new version". I think (assuming we're using semver 2.0) it should be 0.7.0 (since the updated dependencies could be considered an added feature), but maybe you prefer another number?
I've pushed the branch (using git flow) to a develop branch. I'll prepare a new release when I have confirmation on the version number.

Yep, I agree with 0.7.0. Unless it's a bugfix, usually a minor bump is in order.

Great! Then all that needs to be done is publish the new v0.7.0 tag to npm :) (Incidently this can also be done automatically through travis, if you're interested in that ;) It's pretty easy, just adjust the .travis.yml file. Here's an example for another project of mine: browserify-notify Obviously you would need to adjust te credentials to match your own account, but that's all there is to it)

done!
On Thu, Mar 12, 2015 at 12:28 PM Adriaan Callaerts<EMAIL_ADDRESS>wrote:

Great! Then all that needs to be done is publish the new v0.7.0 tag to npm
:) (Incidently this can also be done automatically through travis, if
you're interested in that ;) It's pretty easy, just adjust the .travis.yml
file. Here's an example for another project of mine: browserify-notify
https://github.com/call-a3/browserify-notify/blob/master/.travis.yml
Obviously you would need to adjust te credentials to match your own
account, but that's all there is to it)
—
Reply to this email directly or view it on GitHub
https://github.com/davidguttman/sassify/pull/21#issuecomment-78582398.

Thanks both of you :-)


[Bug]:

Describe the bug
Hi,
i've tried to load the workflow.json in pyhton exploiting the procedure mentioned in the documentation, in fact it does not work. The workflow json do not include the agents section and even by adding it manually exporting the respective agent from the UI, but even agents miss the link with skills and secrets.
Steps to reproduce

create a workflow in autogenstudio UI

download worflow.json

run the script:
_from autogenstudio import WorkflowManager
workflow_manager = WorkflowManager(workflow="path/to/your/workflow_.json")_

run the workflow on a task
task_query = "What is the height of the Eiffel Tower?. Dont write code, just respond to the question."
workflow_manager.run(message=task_query)
Model Used
No response
Expected Behavior
No response
Screenshots and logs
Additional Information
autogen_studio

Hi,
The entire work needs to be exported (instead of the download button which will likely be removed in future updates).

Click export on workflow card
Click download in the pop up. This will export the entire workflow with agents, skills etc. in a json file
Import the json file in workflow manager.


Enable translate - "all" and Enhancements for Language Code Handling

Enable translate - "all" and Enhancements for Language Code Handling
Changes:

Version Bump:

Updated the version in pyproject.toml from 0.1.2 to 0.1.3.

Main CLI Improvements:

Added support for the all keyword in the --language-codes option:

When all is passed, the CLI now dynamically loads all available language codes from the font_language_mappings.yml file.
This makes it easier to apply translation for all supported languages without manually listing each language code.

Included logic to extract top-level language codes (e.g., ar, en, fr, etc.) from the font mappings file.
Debug messages are logged to provide insight into the loaded language codes when debug mode is enabled.

Fix in Font Language Mappings:

Fixed an issue with the Norwegian language code, correcting "no" to no in font_language_mappings.yml.

I have reviewed the changes and everything looks good.


SlackHandler ignores includeStacktraces

I'm using MonologBundle to include stacktraces in logs. It works for StreamHandler, but not for Slack.
monolog:
    handlers:
        main:
            type:  stream
            path:  %kernel.logs_dir%/%kernel.environment%.log
            level: error
            include_stacktraces: true
        slack:
            type: slack
            token: xxx
            channel: "#errors"
            bot_name: ErrorBot
            icon_emoji: :ghost:
            level: critical
            include_stacktraces: true

Yeah, the problem seems to be SlackHandler uses LineFormatter::stringify which doesn't handle exceptions, and it also doesn't use this->getFormatter() like the include_stacktrace does.. lots of small problems which make it not so easy to fix, but it sure isn't impossible :)

I think you can do it with a formater.
In my case I just used a global handler that does $e->__toString() before sending it to slack. works fine

Hi @websirnik does the problem still occur?
Can you check this #1127 and tell me if this happens also for you?

Closing as no activity here in a while.


Fix yotta dependencies

Recent changes in various repositories broke the build of the yotta
module again :( This change fixes the build. Build tested with
frdm-k64f-gcc. I didn't update the yotta version number because I
don't know what is your policy with regards to version changes.

Build finished. No test results found.

Sorry, I just realized something: this problem only appears on the development branch. The build still works on master, although yotta says error: mbed-drivers does not meet specification ~0.7.0 required by minar.

Please disregard the master branch. The current version is in development, and master only reflects the latest public release - except that I forgot to push to it for the last release.
Anyway, with current development, after a yotta update I'm seeing a lot of "does not meet specification" errors but the build still completes. I'm still going to merge your PR and push to the registry, as I need to push a version here anyway (security fixes).


Fail if field lengths are not same in INTERSECT and EXPECT

Which issue does this PR close?
Closes #3632
Are there any user-facing changes?
Users will see failure if they are using INTERSECT and EXPECT on tables with different field count.

CI failure unrelated to this PR.


Fix regex for CD and add update_all script as static file

Not sure if the changes work, please double-check

The shorthand character class did probably not work in an ERE, now it shoul work.


More than 2 slices with stop annotation in 3d annotator

This currently leads to undefined behavior and should be handled better.

Implemented in 16e1954


[SPARK-47152][SQL][BUILD] Provide CodeHaus Jackson dependencies via a new optional directory

What changes were proposed in this pull request?
This PR aims to provide Apache Hive's CodeHaus Jackson dependencies via a new optional directory, hive-jackson, instead of the standard jars directory of Apache Spark binary distribution. Additionally, two internal configurations are added whose default values are hive-jackson/*.

spark.driver.defaultExtraClassPath
spark.executor.defaultExtraClassPath

For example, Apache Spark distributions have been providing spark-*-yarn-shuffle.jar file under yarn directory instead of jars.
YARN SHUFFLE EXAMPLE
$ ls -al yarn/*jar
-rw-r--r--  1 dongjoon  staff  77352048 Sep  8 19:08 yarn/spark-3.5.0-yarn-shuffle.jar

This PR changes Apache Hive's CodeHaus Jackson dependencies in a similar way.
BEFORE
$ ls -al jars/*asl*
-rw-r--r--  1 dongjoon  staff  232248 Sep  8 19:08 jars/jackson-core-asl-1.9.13.jar
-rw-r--r--  1 dongjoon  staff  780664 Sep  8 19:08 jars/jackson-mapper-asl-1.9.13.jar

AFTER
$ ls -al jars/*asl*
zsh: no matches found: jars/*asl*

$ ls -al hive-jackson
total 1984
drwxr-xr-x   4 dongjoon  staff     128 Feb 23 15:37 .
drwxr-xr-x  16 dongjoon  staff     512 Feb 23 16:34 ..
-rw-r--r--   1 dongjoon  staff  232248 Feb 23 15:37 jackson-core-asl-1.9.13.jar
-rw-r--r--   1 dongjoon  staff  780664 Feb 23 15:37 jackson-mapper-asl-1.9.13.jar

Why are the changes needed?
Since Apache Hadoop 3.3.5, only Apache Hive requires old CodeHaus Jackson dependencies.
Apache Spark 3.5.0 tried to eliminate them completely but it's reverted due to Hive UDF support.

https://github.com/apache/spark/pull/40893
https://github.com/apache/spark/pull/42446

SPARK-47119 added a way to exclude Apache Hive Jackson dependencies at the distribution building stage for Apache Spark 4.0.0.

#45201

This PR provides a way to exclude Apache Hive Jackson dependencies at runtime for Apache Spark 4.0.0.

Spark Shell without Apache Hive Jackson dependencies.

$ bin/spark-shell --driver-default-class-path ""

Spark SQL Shell without Apache Hive Jackson dependencies.

$ bin/spark-sql --driver-default-class-path ""

Spark Thrift Server without Apache Hive Jackson dependencies.

$ sbin/start-thriftserver.sh --driver-default-class-path ""

In addition, last but not least, this PR eliminates CodeHaus Jackson dependencies from the following Apache Spark deamons (using spark-daemon.sh start) because they don't require Hive CodeHaus Jackson dependencies

Spark Master
Spark Worker
Spark History Server

$ grep 'spark-daemon.sh start' *
start-history-server.sh:exec "${SPARK_HOME}/sbin"/spark-daemon.sh start $CLASS 1 "$@"
start-master.sh:"${SPARK_HOME}/sbin"/spark-daemon.sh start $CLASS 1 \
start-worker.sh:  "${SPARK_HOME}/sbin"/spark-daemon.sh start $CLASS $WORKER_NUM \

Does this PR introduce any user-facing change?
No. There is no user-facing change by default.

For the distributions with hive-jackson-provided profile, the scope of Apache Hive Jackson dependencies is provided and hive-jackson directory is not created at all.
For the distributions with default setting, the scope of Apache Hive Jackson dependencies is still compile. In addition, they are in the Apache Spark's built-in class path like the following.

The following Spark Deamon don't use CodeHaus Jackson dependencies.

Spark Master
Spark Worker
Spark History Server

How was this patch tested?
Pass the CIs and manually build a distribution and check the class paths in the Environment Tab.
$ dev/make-distribution.sh -Phive,hive-thriftserver

Was this patch authored or co-authored using generative AI tooling?
No.

FYI, I noticed that https://github.com/apache/hive/pull/4564 already cut the Jackson 1.x deps out, pending 2.3.10 release ...
cc @sunchao @wangyum

Actually, we are the one who asked that, @pan3793 . 😄

To @pan3793 , for the record, we've been waiting for that. There is no ETA for now. In addition, every dependency update has a risk of reverting as you see in the history. No matter what happens with 2.3.10 in the Hive community and in Spark community, we will delete this dependency in Apache Spark 4.0.0.
[SPARK-44197][BUILD] Upgrade Hadoop to 3.3.6
[SPARK-44678][BUILD][3.5] Downgrade Hadoop to 3.3.4

Thank you so much, @viirya !
Merged to master for Apache Spark 4.0.0.

Should we change mv $f "$DISTDIR"/hive-jackson/ to mv "$f" "$DISTDIR"/hive-jackson/ ?

Should we change mv $f "$DISTDIR"/hive-jackson/ to mv "$f" "$DISTDIR"/hive-jackson/ ?

Feel free to make a followup, @bjornjorgensen .


Update OWNERS for 1.8

🚨 THIS PR MUST BE MERGED MANUALLY 🚨
This PR updates the OWNERS file for previous, pre-Prow releases of Kubernetes.
Right now, OWNERS files in branches release-1.7 through release-1.9 only list reviewers. Because there are no approvers in the OWNERS file, this makes it impossible to approve changes to those branches using Prow.
While technically we don't accept changes to deprecated branches, this PR solves a problem that would prevent merging fixes for critical/security issues.
Related PRs:

Branch
PR

release-1.7
#9490

release-1.8

release-1.9
#9491

/lgtm


"with/from" field compacting and hyperlinking

at present we display:

[x] cdc13 for (PomBase:SPBC582.03)

For these
UniProtKB-KW:KW-0067
UniPathway:UPA00031
UniProtKB-SubCell:SL-0022
InterPro:IPR000058
PANTHER:PTHR1092/Panther:PTHR31492 (CHECK CORRECT CASE)
Pfam:PF00005
SMART:SM00317
Prosite:PS01164
I don't think our users pay much attention to these sources anyway. We could just hyperlink the object ID (KW-0067, UPA00031, IPR000058 etc), and have the full  prefix:ID visible only via a mouse over). This would save quite a bit of space on some pages where these are often the only with field entries.
These are OK
EC:<IP_ADDRESS>
PDB:1TA9
MODS
These are all fine (they are very rarely used anyway, most only a couple of times)
AGI_LocusCode:AT4G20910
ECOGENE:EG10451
FB:FBgn0004837
MGI:101917
WB:WBGene00000988
dictyBase:DDB_G0269150
RGD:621834
This one is special, because we have  6669 of them.
SGD:S000000002
I nicer display for users/ and a space saver would be SGD:VPS8 so they immediately know which gene (would this be bad practice if   you could see  SGD:S000000002 by mouse over?)
Kim do you know where to access the information to make the hyperlinks for the above?
The display text/mouse over  part of this could be Yaml  file for curators?

Kim do you know where to access the information to make the hyperlinks for the above?

Yep, the web code needs to use GO.xrf_abbs
I think this overlaps with: #65

It overlaps with #65 in that that answers the question that you know where  to get the hyperlinks from.
The other part I would like input from @mah11  and @Antonialock  about whether sensible...
If no consensus we can iron out on Tuesday...

particularly:
(would this be bad practice if you could see SGD:S000000002 by mouse over?)

OK, Hope this is a good plan.
For all "with" column (except the PomBase ones which are fine as the are)
i) Display only the ID as the link.
ii) Display the prefix by mouse over
(this means SGD will use S000000002)
We can reassess  whether we show the  prefix too once we get rid of (suppress) the remaining 4000 IEA annotations ;)
It may seem a little unconventional, but
i) it matches what SGD do
ii) Users pay little attention to the sources of inferred information. Also the source is often arbitrary (in fact we flip-flop between which source we used for many annotations of this type dependent on the parsing.
However, this will change once this ticket is implemented:
https://github.com/pombase/pombase-chado/issues/594
(this ticket will also substantially reduce the number of  with fields from IEAs)

i) Display only the ID as the link.
ii) Display the prefix by mouse over

I've done that.  I got a bit carried away and did for external IDs in extensions too.  Does that make sense?

I think for now we  should keep them in extensions. Probably we will handle some(most) of these differently by having  "natural language" in most cases.
We have discussion tickets about how to display PRO ID labels.
For most things (like" GO terms we already show descriptions rather than IDs...)

I've put the prefixes back on the external IDs in extensions.
I think this is done now.


ion-back-button defaultHref does not support relative URLs

Bug Report
Ionic version:

[x] 4.0.1
Current behavior:

Ionic navigates to a 404 when using ion-back-button with a [defaultHref] of a relative URL such as ../.
Expected behavior:

The Angular Navigation Docs state:

A note on navigation with relative URLs: Currently, to support multiple navigation stacks, relative URLs are something not supported

Ideally until we figure out how to support multiple stacks, which may not be possible, we should maybe think about adding support via a config option for apps with a single navigation stack.
At a minimum the ion-back-button Docs should be updated to make this more clear. I thought I had read though the Angular Navigation docs beforehand but was only made aware of the limitation by searching through the Ionic Slack history.
Steps to reproduce:

Related code:

N/A

Other information:

Ionic info:

Ionic:

   ionic (Ionic CLI)             : 4.10.1 (/Users/adamduren/.nvm/versions/node/v10.14.1/lib/node_modules/ionic)
   Ionic Framework               : @ionic/angular 4.0.1
   @angular-devkit/build-angular : 0.12.2
   @angular-devkit/schematics    : 7.2.2
   @angular/cli                  : 7.2.2
   @ionic/angular-toolkit        : 1.2.2

Cordova:

   cordova (Cordova CLI) : 8.1.2<EMAIL_ADDRESS>   Cordova Platforms     : android 7.1.1, browser 5.0.4, ios 4.5.5
   Cordova Plugins       : cordova-plugin-ionic-keyboard 2.1.3, cordova-plugin-ionic-webview 2.2.0, (and 18 other plugins)

System:

   Android SDK Tools : 26.1.1 (/Users/adamduren/Library/Android/sdk)
   NodeJS            : v10.14.1 (/Users/adamduren/.nvm/versions/node/v10.14.1/bin/node)
   npm               : 6.6.0
   OS                : macOS Mojave
   Xcode             : Xcode 10.1 Build version 10B61

@adamduren I am also having a similar issue , when i use ion-back-button with [defaultHref] i get 404 error .  did you find a path forward yet ?

I came up with a solution that works but it's not ideal. Here's a gist. Hope this helps.
Note: It doesn't have to be used as a decorator. I was just wanting a reason to play around with them. It could just be a function that's made available to the template via other means. One idea is a pipe, it would need to be impure since it would rely on window.location global state.

You could also tie into the router to get the latest url and avoid checking window.location.

Thanks for the issue. I am going to close this as this issue was resolved in Ionic Framework v4.2.0. If you are still running into issues after updating to the latest version of Ionic Framework, please open a new issue. Thanks!


Issue in icon: PlusIcon is not quite visible

Describe the issue:
PlusIcon is not quite visible when it is used, hence suggested to change the icon, to one which is more visible.

closing this as this issue is not valid


Tutorial on how to port your EVM app from Ethereum to Ethermint

Step-by-step video demo of doing this and showing performance differences across the platforms.

replaced by https://github.com/tendermint/ethermint/issues/310


PPL support json_delete, append functions

Description
Add PPL support for the following json functions

json_delete
json_append

Related Issues
#970
Check List

[ ] Updated documentation (docs/ppl-lang/README.md)
[ ] Implemented unit tests
[ ] Implemented tests for combination with other commands
[ ] New added source code should include a copyright header
[ ] Commits are signed per the DCO using --signoff

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

@LantaoJin can u plz review again ?
I've removed the json_expend from this PR only to address:

`json_delete'
json_append


Allow custom cluster types to read from and utilize eds_cluster_config

Commit Message: Allow custom cluster types to read from and utilize eds_cluster_config
Additional Description:
Usage of eds_cluster_config is currently restricted to clusters that have type explicitly set to EDS. Envoy also uses this type for determining whether clusters are initialized during the primary or secondary phase, meaning it is not possible to write a custom cluster type that performs endpoint discovery via EDS.
This CL introduces uses_eds_config to the CustomClusterType proto and changes the requirement for having eds_cluster_config set to include the case in which a custom cluster type has this flag set. Custom clusters that utilize the EDS config are omitted from the primary initialization phase, similar to clusters of type EDS.
Risk Level: low (small optional feature)
Testing: unit test
Docs Changes: inline on new field in cluster.proto
Release Notes: Adds a boolean field to CustomClusterType indicating that the custom cluster type uses EDS for endpoint discovery, and permits the custom cluster type to specify eds_cluster_config if that field is set.
Fixes #15663
API Considerations: the default value of uses_eds_config (false) results in no behaviour change (the cluster will be initialized during the primary phase; the cluster cannot specify eds_cluster_config).
Signed-off-by: Eugene Chan<EMAIL_ADDRESS>

Is the purpose of this just to not be treated as a primary cluster? It definitely seems cleaner to have the cluster extensions themselves somehow announce that they should be treated as secondary clusters instead of requiring setting some additional config field.

Is the purpose of this just to not be treated as a primary cluster?

And to relax the restriction that only clusters that have the explicit type of EDS be allowed to set eds_cluster_config; otherwise, yes.
It does seem that at the time we begin the primary initialization phase we don't have the cluster implementations available to us, only the config. Thoughts would be appreciated on the review thread.

As an alternative, why not have the config message for the custom cluster type include an EdsClusterConfig field of its own, rather than using the existing top-level eds_cluster_config field in the Cluster proto?  This would be consistent with the long-term goal of changing even the built-in cluster types to be extensions.

@pianiststickman I would like to use EDS to fetch endpoints in custom cluster extension. I am interested to know how you used EDS in custom cluster.


chore: python-flask-docker to 0.0.3

chore: Promote python-flask-docker to version 0.0.3

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


pull request for e2e test at 12/22/2016 10:26:17

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Update Ceph.md

Alluxio configure S3 interface,we should add the following code to the conf/alluxio-site.properties file,when Ceph as the underlying storage.

Alluxio configure S3 interface,we should add the following code to the conf/alluxio-site.properties file,when Ceph as the underlying storage.


Latest version breaks compatibility with Python 3.6

The latest version breaks compatibility with Python 3.6. There is now a dependency on nbconvert >= 6.1 and this requires python >= 3.7.
Collecting nbconvert>=6.1 (from nbdev)
  Could not find a version that satisfies the requirement nbconvert>=6.1 (from nbdev) (from versions: 4.0.0, 4.1.0, 4.2.0, 4.3.0, 5.0.0b1, 5.0.0, 5.1.0, 5.1.1, 5.2.1, 5.3.0, 5.3.1, 5.4.0, 5.4.1.dev0, 5.4.1, 5.5.0, 5.6.0, 5.6.1, 6.0.0a0, 6.0.0a1, 6.0.0a2, 6.0.0a3, 6.0.0a4, 6.0.0a5, 6.0.0a6, 6.0.0b7, 6.0.0rc0, 6.0.0, 6.0.1, 6.0.2, 6.0.3, 6.0.4, 6.0.5, 6.0.6, 6.0.7)
No matching distribution found for nbconvert>=6.1 (from nbdev)

If you check the settings.ini, the minimum python version is 3.7 now, 3.6 support was dropped.
CC @jph00, can probably close


Support gh repo commands

Issue Description
Type: feature request
Describe what happened (or what feature you want)
Essentially, :Octo repo <blah> support. In particular, I'd really like the list, clone, and view commands to be executed by Octo (especially list, which I imagine would need the most work to fit into Octo's current model, the others are just executing the relevant gh commands). Along the same lines, it'd be nice if you could just provide :Octo org/repo or :Octo <repo_url> to open the given repo in the browser.

Thanks for the suggestion. I will work on the Octo repo list for now but Im not sure what should be done if one of the repos is selected, show the README.md?
The list command will be implemented as a telescope menu and then I think the best approach would be to implement clone or open in a browser as actions that can be triggered from the menu.
as for:

Along the same lines, it'd be nice if you could just provide :Octo org/repo or :Octo <repo_url> to open the given repo in the browser.

Currently Octo <issue_pr_url opens the issue in neovim and from there you can do Octo issue|pr browser. I think that treating repo URLs differently (directly go to the browser) my be confusing. So maybe the best approach would be to create a new buffer type that represents the repo (basically the README + some repo details at the top) and the enable Octo repo browser and Octo repo clone.

just provide :Octo org/repo

For consistency this should be implemtend as Octo repo view org/repo

Sounds good to me!

I implemented a first version on https://github.com/pwntester/octo.nvim/pull/155 Can you please take a look and let me know if you would add any additional to the repo buffer or if you have any other feedback?


Non correct size of taken picture

When image taken or chosen somehow it is not squared but later you apply Scale to Fit... As result image is not rightly scaled.
i.e. you create image as 100x200 and later scale it to 200x200.  (example numbers to give you idea)
Happening only on real device (tried on iPhone 6). If you run it on Simulator it works correctly.

@iSpartak Excelent! I going to fix that

Thanks! Good code!

Just wonder if it was confirmed as an issue? :)

Hi, I changed some things in the project. Can you test now?


Cleanup

Fixes #107

Codecov Report

Merging #110 into master will increase coverage by 0.36%.
The diff coverage is 2.17%.

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

Looks good to merge


HTML Elements being stripped when pasted to markdown docs.

Need to encode html entities so they can be pasted into issues without being converted to html elements in the MD document.

Patch pushed to v0.0.3-alpha


conflict style and link tags position

it seems that react-helmat placed style tags before link tag.

so custom style can't work!
Ref:

https://github.com/cssinjs/react-jss/issues/127
http://cssinjs.org/setup/?v=v8.1.0#specify-dom-insertion-point
https://stackoverflow.com/questions/46916402/how-to-add-visible-html-comment-in-jsx

see  public/index.html