Some Radio Buttons seems to deselect automatically after selection.

Describe the bug

Some radio buttons deselect automatically when selected.
Attached are the questions with such behaviors.
new_pregnancy_registration.json.zip
This happens with the latest SDK code on main

Component
SDC library
To Reproduce
Steps to reproduce the behavior:
Load the attached form to the SDK then test using the steps shown on the attached video
https://user-images.githubusercontent.com/4817531/212714001-5b944d84-d6ed-4a6a-9927-f892ac73a9f8.mov
Expected behavior

[ ] The selections on a radio button should only change if the radio button is clicked on and not automatically.

Smartphone (please complete the following information):

Device: Xiaomi 10T pro
Android version: Android 11

I also see below issue

When single choice question get enabled and user try to provide input by clicking one of the answer option, it does not accept the input.
nested question with type string of enabled question does not accept the input, keyboard get closed when user start typing the input

https://user-images.githubusercontent.com/86107848/212870289-ff4cbc38-68f6-4f21-9596-deeab8d61fe3.mov
https://user-images.githubusercontent.com/86107848/212871341-a7dd9b94-bbe9-4fb4-9fe7-8b0b1cc3b044.mov

@dubdabasoduba  are you by any chance using a QuestionnaireResponse as well? Just want to see if this Issue is similar to Issue #1784

I ran git bisect, and found this issue first started appearing with the introduction of repeated groups:
c888d5c75e7c996406d4af0f1d10893692388fc6 is the first bad commit
commit c888d5c75e7c996406d4af0f1d10893692388fc6
Author: Kevin Most<EMAIL_ADDRESS>Date:   Thu Nov 10 09:19:56 2022 -0500

    Repeated groups (#1559)

    Support repeated groups

 .../main/assets/repeated_group_questionnaire.json  |  92 +++++
 .../android/fhir/catalog/ComponentListViewModel.kt |   5 +
 catalog/src/main/res/menu/component_menu.xml       |   8 +-
 catalog/src/main/res/values/strings.xml            |   3 +-
 .../datacapture/MoreQuestionnaireItemComponents.kt |   2 +-
 .../fhir/datacapture/QuestionnaireViewModel.kt     | 131 +++++--
 .../QuestionnaireItemGroupViewHolderFactory.kt     |  15 +
 .../questionnaire_item_group_header_view.xml       |  36 +-
 datacapture/src/main/res/values/strings.xml        |   1 +
 .../fhir/datacapture/QuestionnaireViewModelTest.kt | 406 ++++++++++++---------
 .../QuestionnaireItemGroupViewHolderFactoryTest.kt |  30 ++
 11 files changed, 498 insertions(+), 231 deletions(-)
 create mode 100644 catalog/src/main/assets/repeated_group_questionnaire.json

looking further into the change that caused this

I found a simple solution, which is changing the statement:
    } else if<EMAIL_ADDRESS>==
        Questionnaire.QuestionnaireItemType.GROUP
    ) {

to just an else:
    } else  {
<EMAIL_ADDRESS>{
        this.addItem(it.createQuestionnaireResponseItem())
      }

however, I am not sure if this diverts us from the FHIR standard (I am still a novice when it comes to the standard). If a Questionnaire Item type is "choice" or "text", can it have child items? The link below states that the type "group" should have at least one child item, but is unclear whether other types can:
https://www.hl7.org/fhir/valueset-item-type.html

Answer to my own question:
Anything but display items can have child items. https://www.hl7.org/fhir/questionnaire-definitions.html#Questionnaire.item.item

@dubdabasoduba is there a reason for the questionnaire attached to have nested items? While we find a fix to support nested items, is it possible to flatten out the items?

@omarismail94 yes, the nested items make it easier to write the enableWhen expressions. Instead of replicating the same enableWhen expression you just write it for the parent.

The flattened forms work well FYI. I tested them on our imlementation

Thanks, good to know the flattened forms work! I know it isn't as nice as nested, but at least it unblocks you while we work on this issue. I had an initial solution working, but it created new problems when validating the questionnaire :(

Final finding for today, it seems like this is a known issue in our codebase. Looking at the test files, I see a  unit test that has the same setup as this issue, but is ignored. Seems like Kashyap wrote this test, so will chat with him if he shows up to the developer call tomorrow

@joiskash

Updated the questionnaire to remove noise:
new_pregnancy_registration_min.json.txt
@dubdabasoduba please try to include minimum examples while creating issuse in the future (removing unnecessary information in your example and just the bare minimum required to reproduce the problem) - that makes it much easier to debug and to get to the bottom of the issue more quickly.

Updated the questionnaire to remove noise:
new_pregnancy_registration_min.json.txt
@dubdabasoduba please try to include minimum examples while creating issuse in the future (removing unnecessary information in your example and just the bare minimum required to reproduce the problem) - that makes it much easier to debug and to get to the bottom of the issue more quickly.

Makes sense.

@dubdabasoduba will you please test if PR #1881 fixes your issue?

@dubdabasoduba will you please test if PR #1881 fixes your issue?

I will test and raise a ticket for bugs if any

This works well.


Keep spiked ConnectionHandler, set pool in thread

The current functionality of AutoReplica is to set ActiveRecord::Base.connection_handler in the replica context, and reset it to the original when done. This, however, turns out to be risky.
ActiveRecord::Base.connection_handler is implemented using activesupport's class_attribute: the implementation of that is to define a method every time the variable is set, and, if the class variable is already existing, remove the existing method and then define a new one. This opens up to a race condition (as we observed in production): if the thread is killed in the middle of this operation (for example, by rack_timeout), ActiveRecord::Base can end up without a connection_handler method.
To avoid this, we propose to wrap ConnectionHandler once and keep it wrapped, and signal the switch to a replica pool via a thread-local variable.

TL;DR - OK, but we need to look into this in a bit more detail.
I'm a bit hesitant on this right now because we wanted to remove the mutex (and thread locals) around ConnectionHandler, as it seems that this protection is only required for AR3. Which we no longer use (plus - supporting only 1 Rails version back sort of makes sense). So in essence I would rollback the thread locals first, before we apply this PR. @roysab what is your plan with tackling this?
Additionally I wonder - should this override (the hijack of .connection_handler) be applied in a Railtie then? Because otherwise one-time application might be just as unsafe as doing it on every call to replica context, if Rails materializes this method at call time?

Cool! I believe the implementation here is thread-safe, though, without the need for the mutex. In fact, I have the feeling that the current master will fail in a scenario where two threads are using different connection pools / connection params: I think the second thread will overwrite the custom_handler, and the first thread will be left using the new handler :/
You are very right to say that it should be applied in a Railtie. I didn't do that for simplicity, since (at least in our use case) the error is most likely to manifest itself within the replica context. But it's definitely still a gamble. Happy to work on moving the wrapping to a Railtie.

@me what Rails version are you folks using in production? We would like to sunset AR3 support in addition to addressing the issue in this PR, in one spike.

@julik we still have AR3 in production for some apps; but we don't use threading with it, so I don't think that would affect us.

@me will you be ok jumping on as a co-maintainer for the current major version? Then we can bump the major version, ditch AR3 support (which removes a lot of code), and you can massage the current one to do the thing you need.

@julik sure, would be happy to!

Can you spin off a branch for 1.x support and do these changes there then? :-)

Ok! But let me understand, what's the plan? Would you like to bring these changes into master as well?

No, the plan is that it stays on the branch for 1.x, which will still support Rails 3. Exactly because the connection_handler assignment in Rails 3 and Rails 4 differs.

I'll close this PR so that you can work on a branch directly.

Ok!


Question

Hello, I ran predict.py, and the output is always bad frame. The comment says that because the frame cannot be recorded, an exception is reported, but why can’t the frame be recorded? Can you help me?

Duplicate of #8


Docker image version tagging

Instead of only using latest tag, can you also tag the image with the release version?
Example: ghcr.io/alexjustesen/speedtest-tracker:v0.11.0

it's already like this: https://github.com/alexjustesen/speedtest-tracker/tags

Thanks for quick reply.
I actually based my issue on tags not showing up in docker hub: https://hub.docker.com/r/ajustesen/speedtest-tracker/tags
Just validated that tags are ok in ghcr.
great project btw

it's already like this: https://github.com/alexjustesen/speedtest-tracker/tags

Yeah they're right, noticed this last night while on DH. This should def be addressed. @UnAfraid in the past you've helped with getting the Docker workflow polished, any chance you'd be able to help here too?

it's already like this: https://github.com/alexjustesen/speedtest-tracker/tags

Yeah they're right, noticed this last night while on DH. This should def be addressed. @UnAfraid in the past you've helped with getting the Docker workflow polished, any chance you'd be able to help here too?

Sure thing, i'll open up a PR soon.

Just validated that tags are ok in ghcr.

sorry, I tend not to use docker hub when possible, ghcr.io is much better, so I never tested it on DH. :)

Sure thing, i'll open up a PR soon.

The power of open-source and the spirit of collaboration never ceases to surprise me, even after all these years. Thank you guys. :)

Sure thing, i'll open up a PR soon.

Really appreciate it, thank you!


Multiple Panels?

Clicking on the "Open JSON tree viewer" menu option keeps opening new JSON Viewer panels - is that supposed to happen?  I would expect only a single instance and repeated clicking on the menu item to toggle it open / closed (shown / hidden) as most other Notepad++ docking panels operate.
Is this by design?  Is there a way to configure it to behave more like a toggle?
Cheers.

Hi @vinsworldcom , thanks for reminding me to fix this! It was definitely a bug, and it's now fixed.
Hitting Ctrl+alt+shift+j or the menu command should close the tree view if it's already open as of release 3.3.0.


Suggest to optimize the DefaultKeyGenerator.generateKey()

Feature Request

when lastTime <= currentMillis, throw new IllegalStateException is too violent.  suggest that, when moving backwards less than 5ms，sleep and wait。greater than 5ms throw new IllegalStateException。
when lastTime is a new time, the sequence always be 0. if the QPS is low，Most distributed id will be end with 0，if the distributed id is the sharding column，This will lead to the data to lean。suggest that ，when lastTime is a new time ，the sequence is a random number between in [0, 64).

The second point is really good, I just create new issue to process it dependent.

for 1st suggestion, I just add a static method setMaxTolerateTimeDifferenceMilliseconds() at
DefaultKeyGenerator, user can adjust their expected threshold if they like, default value is zero.


Proposal: An input field for crontab syntax - CronEditorField, CrontabField, or CronExpressionField

Related to jowilf/starlette-admin#439
Why do you need this field? What does it solve? Please describe.
To be able to edit input fields in crontab format.
Describe the solution you'd like
JossyDevers/cron-expression-input: Cron UI: Input component to generate cron expressions easily and intuitively
Additional context

:white_check_mark: CDN Support
:white_check_mark: JavaScript Initialization
:white_check_mark: Tag Initialization
:x: Data Attribute Initialization

One other library: Marak/cron-editor: Visual Cron Editor - Edit and Create Cron Jobs With Ease


Refactor + implement abstraction for Java arrays

This issue has three parts:

Remove the JArray# primitive.
In hindsight, it appears to make things too complex. For example, declaring an int[] array now is:
data IntArray = IntArray (Object# (JArray# Int#))

The reason for doing this was to encode the array status in the primitive type, but it seems unnecessary now. If anyone has a use case for having such complex types, let me know.

Support array declarations in the CLASSannotation.
Example:
data {-# CLASS "int[]" #-} IntArray = IntArray (Object# IntArray)
type instance Super IntArray = Object

Java arrays can be thought of as special classes that are distinct with no relationship between them in terms of inheritance. We need a way to abstract over them in a nice way so that we can deal with Java arrays easily when working with FFI.
The interface will look something like below:
{-# LANGUAGE FunctionalDependencies #-}
class (Class a) => JArray elem a | a -> elem where
   newJArray :: Int -> Java c a
   indexJArray :: Int -> Java a elm
   writeJArray :: Int-> elem -> Java a ()

arrLength :: JArray a elem => Java a Int
jarrayToList :: JArray a elem => Java a [elem]
jarrayFromList :: JArray a elem => [elem] -> Java c a

Looking for feedback on improving naming and the interface itself.

Done


[Modal] can't close modal

when i start to fetch data, i set loading state to true(Modal' s visible set as true),
as the fetch finish, i set loading state to false(Modal's visible set as false),
but the loading modal sometime can close, sometime can't close 。
i open the debug logger, the loading state(true/false) logger of the whole fetch process  is absolute the same in this two kind of result。。

I encountered the same problem. What's your solution？

one page have mutil modal or modal+alert, the second modal(or alert) should delay  xxx millisecond to change state before the first modal close
for more detail u can see this link: http://blog.csdn.net/mqy1023/article/details/53447082

谢谢

#10471


Upgrade to Beam 2.32.0

This version of Beam does not have an explicit dependency on log4j.
There are a couple of other things that need to change due to the
upgrade.

The new version pulls in a dependency that is not on Maven Central
but on packages.confluent.io, so we need to explicitly add this repo.

The new version has a dependency on flogger 0.6, which removed the
LoggerConfig class (see https://github.com/google/flogger/issues/142).
We therefore backported the class. In the long term we should do what
was suggested in the issue and use the normal JDK Logger config
directly.

This change is

Please note that currently the presubmits do not pass because of the added maven URL, which is not recognized by the gradleproxy used in update_dependency.sh. This will be fixed in a separate CL, but this PR itself is ready for review.


Cannot change certain Editor.Options when the TestApp is compiled with .NET Native toolchain

Steps to reproduce:

Add a toggle button to the ItemsControl in MainPage.xaml in the TestApp:

<Button Content="Line Numbers" Click="Numbers_Click" />

Add the click event handler to MainPage.xaml.cs:

private void Numbers_Click(object sender, RoutedEventArgs e)
{
    options.LineNumbers = options.LineNumbers == LineNumbersType.On ? LineNumbersType.Off : LineNumbersType.On;
}

Run the TestApp without compiling with the .NET Native toolchain. Clicking the button a few times will toggle the line numbers on and off as expected.
Run the TestApp with compiling with the .NET Native toolchain. Now you see that reenabling the line numbers will not work, so they disappeared permanently.

This issue also occurs e.g. with Editor.Options.WordWrap and maybe even for more Editor.Options. I have not tested them all.
And turning off the .NET Native Compiler is of course not an option since all UWP apps you want to submit to the Microsoft Store need to be compiled with this toolchain.

There is some rd.xml madness going on and I could not find any clue in the documentation on how to solve this. <Namespace Name="Monaco.Editor" Serialize="All" Dynamic="All"/> does not seem to work with JsonConverters (like LineNumbersTypeConverter). A quick workaround for me was to reset the LineNumbers Property back to the old string type in IEditorOptions:
 [JsonProperty("lineNumbers", NullValueHandling = NullValueHandling.Ignore)]
 string LineNumbers { get; set; }

Thanks for digging in and finding the cause @WelterDevelopment. There may be some rd.xml stuff we can do in the component, I'll have to take a look.


GHA MacOS: bgzip executable could be identified.

Missing a system dependency? Might be able to circumvent this with one of the echotabix alternatives:
https://github.com/RajLabMSSM/echolocatoR/actions/runs/3357812148/jobs/5563951744#step:21:1
Run options(crayon.enabled = TRUE)
Loading required package: sessioninfo
── R CMD build ─────────────────────────────────────────────────────────────────
* checking for file ‘.../DESCRIPTION’ ... OK
* preparing ‘echolocatoR’:
* checking DESCRIPTION meta-information ... OK
* installing the package to build vignettes
* creating vignettes ... ERROR
Error: --- re-building ‘BD_GWAS.Rmd’ using rmarkdown
--- finished re-building ‘BD_GWAS.Rmd’
--- re-building ‘echolocatoR.Rmd’ using rmarkdown
Quitting from lines 85-95 (echolocatoR.Rmd)
Error: Error: processing vignette 'echolocatoR.Rmd' failed with diagnostics:
bgzip executable could be identified.
--- failed re-building ‘echolocatoR.Rmd’
--- re-building ‘finemapping_portal.Rmd’ using rmarkdown
Downloading: https://github.com/RajLabMSSM/Fine_Mapping_Shiny/raw/master/www/data/GWAS/Nalls23andMe_2019/ASXL3/multi_finemap/ASXL3.UKB.multi_finemap.csv.gz
trying URL 'https://github.com/RajLabMSSM/Fine_Mapping_Shiny/raw/master/www/data/GWAS/Nalls23andMe_2019/ASXL3/multi_finemap/ASXL3.UKB.multi_finemap.csv.gz'
Content type 'application/octet-stream' length 157348 bytes (153 KB)
==================================================
downloaded 153 KB
Downloading: https://github.com/RajLabMSSM/Fine_Mapping_Shiny/raw/master/www/data/GWAS/Nalls23andMe_2019/BIN3/multi_finemap/BIN3.UKB.multi_finemap.csv.gz
trying URL 'https://github.com/RajLabMSSM/Fine_Mapping_Shiny/raw/master/www/data/GWAS/Nalls23andMe_2019/BIN3/multi_finemap/BIN3.UKB.multi_finemap.csv.gz'
Content type 'application/octet-stream' length 230[37](https://github.com/RajLabMSSM/echolocatoR/actions/runs/3357812148/jobs/5563951744#step:21:38)0 bytes (224 KB)
==================================================
downloaded 224 KB
Downloading: https://github.com/RajLabMSSM/Fine_Mapping_Shiny/raw/master/www/data/GWAS/Nalls23andMe_2019/ASXL3/LD/ASXL3.UKB.LD.csv.gz
trying URL 'https://github.com/RajLabMSSM/Fine_Mapping_Shiny/raw/master/www/data/GWAS/Nalls23andMe_2019/ASXL3/LD/ASXL3.UKB.LD.csv.gz'
Content type 'application/octet-stream' length 66098 bytes (64 KB)
==================================================
downloaded 64 KB
Downloading: https://github.com/RajLabMSSM/Fine_Mapping_Shiny/raw/master/www/data/GWAS/Nalls23andMe_2019/BIN3/LD/BIN3.UKB.LD.csv.gz
trying URL 'https://github.com/RajLabMSSM/Fine_Mapping_Shiny/raw/master/www/data/GWAS/Nalls23andMe_2019/BIN3/LD/BIN3.UKB.LD.csv.gz'
Content type 'application/octet-stream' length 100237 bytes (97 KB)
==================================================
downloaded 97 KB
--- finished re-building ‘finemapping_portal.Rmd’
--- re-building ‘plot_locus.Rmd’ using rmarkdown
Failed with error:  'there is no package called 'pals''
Failed with error:  'there is no package called 'pals''
The magick package is required to crop "/private/var/folders/24/8k48jl6d249_n_qfxwsl6xvm0000gn/T/RtmpBvAYS5/Rbuildabab4d92368a/echolocatoR/vignettes/plot_locus_files/figure-html/trk_plot-1.png" but not available.
Failed with error:  'there is no package called 'pals''
Failed with error:  'there is no package called 'pals''
The magick package is required to crop "/private/var/folders/24/8k48jl6d249_n_qfxwsl6xvm0000gn/T/RtmpBvAYS5/Rbuildabab4d92368a/echolocatoR/vignettes/plot_locus_files/figure-html/modify track-1.png" but not available.
Failed with error:  'there is no package called 'pals''
Failed with error:  'there is no package called 'pals''
The magick package is required to crop "/private/var/folders/24/8k48jl6d249_n_qfxwsl6xvm0000gn/T/RtmpBvAYS5/Rbuildabab4d92368a/echolocatoR/vignettes/plot_locus_files/figure-html/trk_plot.xgr-1.png" but not available.
Failed with error:  'there is no package called 'pals''
Failed with error:  'there is no package called 'pals''
The magick package is required to crop "/private/var/folders/24/8k48jl6d249_n_qfxwsl6xvm0000gn/T/RtmpBvAYS5/Rbuildabab4d92368a/echolocatoR/vignettes/plot_locus_files/figure-html/trk_plot.QTL-1.png" but not available.
--- finished re-building ‘plot_locus.Rmd’
--- re-building ‘QTLs.Rmd’ using rmarkdown
[tabix] the index file exists. Please use '-f' to overwrite.
Failed with error:  'there is no package called 'seqminer''
Quitting from lines 74-83 (QTLs.Rmd)
Error: Error: processing vignette 'QTLs.Rmd' failed with diagnostics:
there is no package called 'seqminer'
--- failed re-building ‘QTLs.Rmd’
--- re-building ‘summarise.Rmd’ using rmarkdown
The magick package is required to crop "/private/var/folders/24/8k[48](https://github.com/RajLabMSSM/echolocatoR/actions/runs/3357812148/jobs/5563951744#step:21:49)jl6d249_n_qfxwsl6xvm0000gn/T/RtmpBvAYS5/Rbuildabab4d92368a/echolocatoR/vignettes/summarise_files/figure-html/super_summary_plot()-1.png" but not available.
--- finished re-building ‘summarise.Rmd’
SUMMARY: processing the following files failed:
  ‘echolocatoR.Rmd’ ‘QTLs.Rmd’
Error: Error: Vignette re-building failed.
Execution halted
Error: Error in proc$get_built_file() : Build process failed
Calls: <Anonymous> ... build_package -> with_envvar -> force -> <Anonymous>
Execution halted
Error: Process completed with exit code 1.

Actually, this is coming directly from echotabix:
https://rdrr.io/github/RajLabMSSM/echotabix/src/R/get_bgzip.R
I'll implement some alternative bgzip options.
Also, the error should read: "gzip executable could NOT be identified"


Publish Extension at Eclipse Open VSX

Hi there,
your extension is quite useful to Fortran devs and I appreciate it much in my daily work.
I found a VSCODE fork https://vscodium.com that is completely published under a MIT license and I would like to use your extension in VSCODIUM. However, it is under discussion if extensions of the MS Marketplace may be legally installed in VSCODIUM, which is technically easily possible.
The legal discussion could be effectively circumvented if your extension could also be published at https://open-vsx.org (https://github.com/eclipse/openvsx/wiki/Publishing-Extensions) .
What are your thoughts about it?
Thanks,
Maarten

Thanks for the suggestion and sorry for the delay. I am certainly not opposed to this, so I'll take a look and update here.


Trader fails on BTC-e

Get this error when trying to trade
ERROR: placeOrder Exchange API returned the following error:
ERROR: You incorrectly entered one of fields.
I was trading LTC-USD if that helps
Thanks

Just happens on BUY orders, I Think there is an issue with number of decimals.
"ERROR","message":"You incorrectly entered one of fields."


How to Define specific DataSets sizes such as 64GB, 128GB

Am trying to test MonogDB, CassandraDB using YCSB. Where I have to test on different datasets sizes 32GB, 64GB, 128GB. But I can't figure out how to calculate it from available parameters. Please respond with any clue on that. Any help is highly appreciated.
Thanks and Regards
Anugnu Veeravathni

As per the xml file of workloads, I came to know one recordcount is 1kb. That means (1kb10241024) is 1GB. But I wanted to try 1TB load, where Recordcount will be more than the limit suggested. I even read that changing field length will increase load, Is this accurate? If yes, can anyone help me in the equation please.
And one more thing, I need some idea on usage of element name 'target'.
Please help me out  guys. @busbey @twblamer @levaphenyl
Thanks in Advance

If the total size is all what matters, then yes, changing fieldlength and filedcount will help.
Basically, the formula are:
row_size = fieldcount * fieldlength
load_size = recordcount * row_size

Please see the workload template and the parallel workload documentation for more info.
Please notice that YCSB is a tool and as such, you are responsible for designing the test case :wink:. I suggest closing the issue.

Thanks a lot @levaphenyl
Sorry for pinging my question in Issues.


feat:show confirmation modal on click of delete api key button

Summary
Fixes issue #342
Description
Added confirmation alert on click of delete action button and -

if user confirms, then API key is deleted
if user declined confirmation, it does not delete API key

How to Test the Changes

Clone the repository and switch to the branch(feat-add-confirmation-on-delete-apikey) where this PR is made-

git clone https://github.com/TeamShiksha/logoexecutive.git
cd logoexecutive
git checkout <branch-name>

Go to the project directory and ensure Yarn is installed and dependencies are up-to-date & run below command

yarn install

Configure env and required stuffs to run project according to Development guide

Start the application

yarn start

Login to account and navigate to dashboard. Now create API keys and click on delete action button and verify if it shows confirmation modal or not.

Context (Optional)
Clicking on delete action button does not display any confirmation alert to the user.
Screenshots or Recordings (Optional)

Checklist

[x] I have tested the changes locally and they work as expected.
[x] I have added/updated tests that cover the changes.
[ ] I have updated the documentation to reflect the changes.
[x] This pull request follows the project's coding guidelines.

@aps08 Kindly review it and once if everything works merge it or suggest me if something needs to improve. I'll do it asap. TY.

Where is the coverage report ?

Hi @aps08 ,
here is the coverage report:-

server coverage report

client coverage report

two tests(Delete API Key fails on wrong keyId, Delete API key and removes it from the list) for dashboard and one test suites are failing.

Cannot merge due to incomplete PR. The screenshot provided is not a coverage report. 2 Test cases are failing due to your changes. The pre-commit should pass in order to merge the PR.

@aps08  Fixed the test for dashboard and committed, and here are the coverage report after running yarn coverage:server and yarn coverage:client.

client coverage

server coverage


[api-docs] Move locale list from dispatch to reference and link to it (#4335)

A new commit was made into the api-docs repo.
https://github.com/discord/discord-api-docs/commit/c39627c76f049c9f3c26fe40edde75017b76ba7e
This is a bot created issue.

this is discord moving its docs around
We can close this


Talon side spoken form tests

Fixes #449
Fixes #1675

Potential manual tests to add
For these, just bake payload and spoken form into consts in the test itself

[x] "slice past"
[x] "two tokens forward"
[x] "one token"
[x] "take inside pair"

Checklist

[x] Handle if Talon was in sleep mode initially
[x] Add deprecation warning if user says "to after"?  I believe we support this today, and some users are still saying that, as indicated by slack questions from the past month.  We could probably have this as an alternate spoken form for "after" and show a warning if they use it
[x] Stop using generateSpokenForms to check whether to run a test; instead read spokenFormError field on recorded test
[x] How do we actually want to launch this? Launch config in vscode and then?
[x] Add notifications when test start and stop
[x] I have added tests
[/] I have updated the cheatsheet
[/] I have updated the docs
[x] I have not broken the cheatsheet

Worth running through the checkboxes in these two issues to see if you've addressed everything.  If not, I'm not saying you should do it in this PR, but we should prob not close those issues:

https://github.com/cursorless-dev/cursorless/issues/449
https://github.com/cursorless-dev/cursorless/issues/1440

@pokey Updated and ready for review

Ok this looks good to me, but for some reason is still failing for me locally, so we'll need to debug. Also worth having a look at the changes I made

The changes just looks fine. I just discover that after Talon has restarted some of the tests are failing for me as well. Touching a single Talon user file fixes this for some reason.

Its the Cursorless simple actions list that don't get updated to default unless you touch a file. I will investigate further


4.5 beta - module nodenext appears to break resolution when combining package.json exports and @types

Bug Report
🔎 Search Terms
4.5 beta
module resolution nodenext
@types
package.json exports
Could not find a declaration file for module
🕗 Version & Regression Information
Typescript: typescript@next (4.5.0-dev.20211006)
Node: v14 AND v16
Npm: v6 AND v7

This is an unexpected failure introduced by 4.5 beta using module=nodenext

💻 Recreation Link
(Was easier to make a mock repo than manage dependencies in ts playground)
https://github.com/JacobLey/typescript-issue-recreation
See readme, but effectively run npm i + npm run recreate-issue.
🙁 Actual behavior
When using module=nodenext, typescript fails to find type definitions for yargs/helpers.
Emitted error is:
broken/yargs/index.ts:2:25 - error TS7016: Could not find a declaration file for module 'yargs/helpers'. '/Users/jacobley/typescript-issue-recreation/node_modules/yargs/helpers/helpers.mjs' implicitly has an 'any' type.
  If the 'yargs' package actually exposes this module, consider sending a pull request to amend 'https://github.com/DefinitelyTyped/DefinitelyTyped/tree/master/types/yargs'

2 import { hideBin } from 'yargs/helpers';
                          ~~~~~~~~~~~~~~~

Found 1 error.

🙂 Expected behavior
When the proper @types/yargs is installed, typescript should be able to find (or fall back on) type definitions for yargs/helpers
Current Understanding (Guess  ¯\(ツ)/¯)
yargs correctly uses the package.json exports to declare what files are outputted. https://github.com/yargs/yargs/blob/main/package.json#L15
However they do not emit typings with the package itself, and definitely do not provide a custom types condition in exports (and it is unreasonable to expect that to exist, given the feature has existed for less than a week).
Instead typescript users should install the @types/yargs package, which is usually handled gracefully.
However it appears that because typescript discovers a declaration for /helpers in yarg's package.json exports, it simply tries to find the types there, fails, and gives up.
Pre-4.5 (or module != nodenext) typescript would have ignored package.json entirely and fallen back on @types/yargs.
Expected/ideal behavior would be for typescript to continue falling back on @types when package.json exports are declared but do not point to a typescript declaration. Although if yargs were to declare types in their package.json exports, that should be selected first.
My current workaround is to explicitly declare this mapping using the paths attribute of tsconfig (see working example from recreation).
It is also worth noting that importing the root yargs package appears to work just fine. Despite that being declared in package.json exports as well. I assume there must be some special handling for root/index imports that correctly falls back on @types.
Also worth noting that I have been unable to trigger vscode (which is configured to use locally installed version of typescript) to recreate this issue. It appears to correctly find the typings for yargs/helper everytime (provided @types/yargs is installed)

I made a similar comment in the recreation README but worth mentioning here.
My usage of yargs is simply because it is the first package I ran into with this issue while testing out 4.5 beta. It is a very popular package so I suspect this issue could/would pop up for many users when 4.5 is officially released.
If my diagnosis is correct, this is by no means an issue with yargs itself, and could be recreated with any other package that:

Uses package.json exports field
Exports a file under some path (not just root/index)
Do not include native typings and rely on something like @types

https://github.com/DefinitelyTyped/DefinitelyTyped/pull/56495 should update the current version of the @types/yargs package to provide proper esm entrypoints to mirror yargs itself. It might take a bit to merge, since we have to check that the DT automation can handle the changes.

It seems that ESM resolution is inconsistent across packages though.
Another (cherrypicked) example is @types/config which exposes a submodule defer
import { deferConfig } from 'config' works just fine for me with nodenext.
Neither that type declaration nor the original package use the exports field.
So it seems that by given logic, if the root package uses exports, then the @types/package-name must also use package.json exports?
While I am generally supportive of updating packages/types to properly use the exports field, this seems like an unnecessary restriction that might break for more packages?
Why not use the same resolution methods for @types/yargs as for @types/config?

From esm you'd need to import config/defer.js - esm resolution does not add extensions for you.

I can confirm that post-https://github.com/DefinitelyTyped/DefinitelyTyped/pull/56495, the version of @types/yargs we have published now has esm entrypoints and proper exports map entrypoints looking at them, and everything seems to be in working order. There's probably a pile of other DT packages which need updating to better mirror their source packages in this way (I think we generated a list a few weeks ago), but I don't think this is the issue to track that effort in, so I'm going to chalk this one up as done.


Is it possible to write rows into Logfile in Hierarchical way?

Hello,
Does NLog support any kind of Hierarchical way on logging into file?
I mean something like this kind of two level logfile.
Init done.
Read done.
------Error: Wrong password.
Read done.
Start copying
------Error: Access Denied.
Start copying
End Copying
So, can this be done with "levels" or templates or some other way.
Best Regards,
Lamik

That's possible with When-Layout-Renderer and Pad-Layout-Renderer, but it will get messy soon.

Thanks for reply. Are there any samples how to use When-Layout-Renderer and Pad-Layout-renderer?
Best Regards,
Lamik

Currently no. It will be great if you post your config when you got it working.

Closed as this question is answer, please reopen as needed


Add files via upload, clean up and fixes

Fixing errors for character sheets and beautify content
Changes / Comments
Beautify code
Fix buttons to move content outside of buttons.
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[ ] Does the pull request title have the sheet name(s)? Include each sheet name.
[x] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.


Completed 8 surveys, however the "Surveys menu item" only says 7 completed

Not sure how the menu is counting "Surveys", but it seems to be one off.

Nevermind, this is actually related to issue #341.

Picture shows the "Incomplete" About My Family survey.


tomcat error after deploy war file

i have got the error after deploy war file.
This application has no explicit mapping for /error, so you are seeing this as a fallback.
Thu May 02 11:11:11 IST 2019
There was an unexpected error (type=Internal Server Error, status=500).
I/O error on POST request for "https://qah1-appstore.e-pragati.in/AppStore/store/v1/users": Connection reset; nested exception is java.net.SocketException: Connection reset

It looks like a java error, please refer this https://stackoverflow.com/questions/31134333/this-application-has-no-explicit-mapping-for-error OR try with some other war file.

it may help you
https://www.logicbig.com/tutorials/spring-framework/spring-boot/disable-default-error-page.html

it may help you
https://www.logicbig.com/tutorials/spring-framework/spring-boot/disable-default-error-page.html

thanks for the reply ...their is some domain issues..now resolved.


Error while running the example program

Description:
When run the example program described in the README.md file it gives an error called (wso2/github)GithubError {"message":"Organization should have a name"}
Suggested Labels:
Suggested Assignees:
Affected Product Version:
OS, DB, other environment details and versions:
OS - Ubuntu 18.04
Steps to reproduce:
Related Issues:

Closing as we no longer maintain this repo.

Closing as we no longer maintain this repo.


Added landing page from dmsp_aws_prototype to app and dockerized the app

I wanted to see if I could incorporate the landing page from dmsp_aws_prototype to the app, and use Apollo GraphQL client to make calls to the Apollo Server.
Additionally, I dockerized the app for dev and prod.

Looks great @jupiter007! I requested on small change to the image you're pulling for your Dockerfiles.
What is the difference between .ts and .tsx? Is it just that .tsx includes HTML?

Thank you for looking at my PR @briri. The ".tsx" is equivalent to using ".jsx", which is used to indicate that we are using JSX in the component. ".ts" would be used like we would use ".js", which would be just pure javascript, but with type annotations.

Hi @briri, I made the updates you requested. Please take another look when you get the chance and let me know if it looks ok. Thanks.


D2DCustomEffects: instructions when using built-in Visual Studio HLSL Compiler?

I am trying to get custom effects in Direct2D working in my c++ project which is included in an c#/XAML solution. I have been trying to adapt the CustomPixelShader project from the D2DCustomEffects solution.
My problem (I think) is that you are configuring a Custom build step in the project solution to get fxc to compile the shader. I have tried to mimic the Custom build step config in the property pages for my project. I have noticed that in my project property pages I have an additional property - HLSL compiler - that is not present in the CustomPixelShader example project (see image).

When I try to build the project without trying to configure HLSL Compiler, I get the error
Error	X1507	failed to open source file: 'd2d1effecthelpers.hlsli'
The path to d2d1effecthelpers.hlsli is included in the Custom build step config and I have checked that the file is there. However, if I add the path to d2d1effecthelpers.hlsli to Additional include directories in the HLSL Compiler config, that error message disappears but I get other problems instead. I’m interpreting this as if the HLSL compiler is hijacking the compilation of the shader, even though I really want to use a Custom build step to do that.
My questions are: have you had any experience with using the HLSL Compiler at all? Is there some reason that I absolutely can not use it to compile the shader when implementing a custom effect, or do I just have to configure it properly? Is there anything obvious that I am missing?
Also, is creating a Custom effect the only way of creating a custom pixel shader in Direct2D? I don’t mind it not being linkable, I just want to get a simple version of a pixel shader into my Direct2D project, with no extra features for now. Do you now of any simpler examples out there?
I’m using Visual Studio Community 2017, v 15.3.5.

An addition: if you have any idea on where to find proper documentation on how to configure the HLSL compiler I would appreciate it, I haven't been able to find anything on that. I tried to configure the HLSL Compiler to build according to the rules in your Custom build step, but failed.

I got an answer in another forum (link to answer).
Summary for others with the same problem: right click on the .hlsl-file in the solution explorer in Visual Studio. That will result in showing the properties for that file. Under Configuration Properties -> General you can change Item Type to something other than "HLSL Compiler". Changing it to "Text" worked for me, "Does not participate in build" was suggested in the answer in the other forum. If that does not do the trick, changing the value for Exclude from build to "No" could work. For me, just changing the Item type of the .hlsl-file was enough.
This should result in the HLSL Compiler disappearing from the project property pages (see the image in the question), and the HLSL Compiler will not try to compile the .hlsl-file.


Correct "html hover" on chargeback tables

This PR adds some js code to the view to permit a correct hover on html tables. The current tables with tiers on the metrics, do that the table-hover class of patternfly does not works correctly. With this change the user can understand better the table makink a hover.

hi @lpichler, @gtanzillo, do you know the reason about the travis fails?

hi @amaurygonzalez, I don't know what is the reason but it is not related to this PR

@himdel can you please review this PR ?

@amaurygonzalez The overall approach is fine, I think, but:

there's no reason to copy that JS to 4 different places, as it is, you're adding a new hover handler to every row, every time you add a row :) .. probably want something like $(document).on('mouseenter', 'tr.rdetails', that_first_function); $(document).on('mouseleave', 'tr.rdetails', that_second_one); - and do it only once, on the top-level (registering the event handler like this should make it work even for elements added after).

[id^='"+this.id.slice(0,-2)+"'] - this assumes there's only up to 10 lines (assuming 0-based index), otherwise it won't work.. you may want something like _.lastIndexOf to find the last underscore, instead of hardcoding that -2

(minor) tbody tr.rdetail seems hardly descriptive enough, and too specific at the same time .. may want to think of a better name (in general, prefer descriptive class names for selectors, instead of combinations of element type and generic sounding class names)

why is cb_rate_show matching the end of id instead of the beginning? I mean, why is that one different? (But should at least add a _ to the match, because matching [id$="1"] will match both rate_detail_row_1 and rate_detail_row_11 ;).)

Thanks @himdel  all your suggestions are very valid. I am working in it!

@amaurygonzalez Agree with @lpichler that the failure is not related to your PR. I saw the same failure yesterday on other PRs. Not sure if it got fixed but you can try rebasing and pushing again to see if it goes green.

Closed/reopened to restart Travis


DMR Implementation

@rick51231 I am trying to get good source code references for the data link and control layer for DMR. I am planning to inject the code on my hardware and test the same. Does this code help? If you have any good reference links please share across

Hi, @alwinhb
Everything related to DMR is in this folder: https://github.com/rick51231/node-dmr-lib/tree/master/src/DMR
The best source of information about DMR is the ETSI standard: https://www.dmrassociation.org/dmr-standards.html

@rick51231 Thanks for the quick help, Does this https://github.com/rick51231/node-dmr-lib/tree/master/src/DMR contain all the DMR ETSI Standard protocols?

@alwinhb , definitely not. It contains only what is needed for basic work with sending data. In the near future I plan to launch a beta of voice decoding.

@rick51231 Thanks, That means as of now it can only send short messages and gps information. But not the audio right?

@alwinhb , short data and gps - yes. Audio - partially, just released beta

@rick51231 Thanks for the update, Do you have voice samples used for decoding?

I am working with XNL/XCMP protocols and need help obtaining the encryption keys referenced
Any guidance or documentation would be appreciated.
Best regards

@mojtabamazhari you can obtain this information from your local Motorola parner


Automated cherry pick of #2017: fix list order by

Cherry pick of #2017 on release/2.10.0.
#2017: fix list order by

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.10.0@9ffc238). Click here to learn what that means.
The diff coverage is 0%.

@@               Coverage Diff                @@
##             release/2.10.0   #2019   +/-   ##
================================================
  Coverage                  ?   7.62%
================================================
  Files                     ?     410
  Lines                     ?   63188
  Branches                  ?       0
================================================
  Hits                      ?    4815
  Misses                    ?   57963
  Partials                  ?     410

Flag
Coverage Δ

#aFlag
7.62% <0%> (?)

Impacted Files
Coverage Δ

pkg/cloudcommon/db/db_dispatcher.go
0% <0%> (ø)

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 9ffc238...68ba504. Read the comment docs.

/lgtm

/approve


fix: flatmap is not supported by some browsers

Fixes #2358

Description of what you did:

My PR is a:

[ ] 💥 Breaking change
[x] 🐛 Bug fix
[ ] 💅 Enhancement
[ ] 🚀 New feature

Please confirm that your PR fulfills these requirements

[x] Multiple commits are squashed into one commit.
[x] The commit message follows Conventional Commits, which allows us to autogenerate release notes; e.g. fix(plugins/plugin-k8s): fixed annoying bugs
[x] All npm dependencies are pinned.

i'm pretty sure that flatMap and flat both came in the same generation of support, and so this change won't help those older browsers.
we have a flatten impl elsewhere in the code already

solved, thanks!


Enhanced Readme to increase clarity and readability

Description
This revised Markdown offers improved readability, clarity, and presents the readme for better user-experience
Type of change

This pull request fixes a bug in Radius and has an approved issue (issue link required).
This pull request adds or changes features of Radius and has an approved issue (issue link required).
This pull request is a minor refactor, code cleanup, test improvement, or other maintenance task and doesn't change the functionality of Radius (issue link optional).

Fixes: #issue_number
Auto-generated summary

🤖 Generated by Copilot at a9ac657
Summary
:memo::sparkles::pencil2:

This pull request enhances the README.md file of the radius project. It improves the clarity and style of the documentation and fixes some minor errors.

The README.md file was quite bland
So the author gave it a new hand
They fixed punctuation
And rewrote the introduction
To make it more clear and more grand

Walkthrough

Rewrite introduction paragraph to improve readability and appeal (link)
Add periods at the end of sentences in Security and Code of Conduct sections (link)

@shalabhms @willdavsmith
Kindly review the PR and, please merge it.

@shalabhms @willdavsmith Kindly review the PR and, please merge it.

Thanks @Arcturus22 for submitting the change. We are reviewing the PR now.

Thanks so much for your very close review of the Radius README, Arcturus22, and this PR.  I've assigned this to myself and will review and keep you in the loop.

Thanks so much for your very close review of the Radius README, Arcturus22, and this PR. I've assigned this to myself and will review and keep you in the loop.

Thanks for the review. I will be awaiting your response.

Arcturus22, I reviewed this PR in detail and I’m leaving comments on which parts we'd like to accept and the others where we'd like to keep the original version. Please provide an updated PR and I can re-review and merge.  Here’s my rationale which I hope is helpful.  As you observed in your very close review of the README, we deliberately adopted a somewhat unadorned and journalistic style of writing about Radius: We focused on using language that is as objective and concise as possible, with an emphasis on educating the community, while avoiding language that might read as “selling” the project to anyone.   As you observed, this style can admittedly result in prose that is somewhat “bland” vs “grand.”  Our approach stems partly from our recognition that, while we believe Radius has significant potential, we also recognize we have a lot of work to do, jointly with the community, before we all fully realize that potential.  So, we work to ensure our language and tone aligns with that reality.  We also expect Radius community members to be predominantly engineers who often are distracted by more marketing-oriented language.  So, we avoid most adjectives and adverbs that are strongly positive.  With that context in mind, here is how I applied our communication style in reviewing your PR.
Where I interpreted your edits to:
Help increase message and clarity, I ask you to resubmit in a new PR
Result in nearly identical message and clarity, I indicated we'd like to keep the original language
Add positive adjectives or adverbs (such as “sophisticated” or “meticulously”), I indicated we'd like to keep the original language.
Thanks again for this PR. It required me think much more thoroughly about documentation contribution guidance.  I will update the doc contributor guide so others also have this context up front.

Arcturus22, I reviewed this PR in detail and I’m leaving comments on which parts we'd like to accept and the others where we'd like to keep the original version. Please provide an updated PR and I can re-review and merge. Here’s my rationale which I hope is helpful. As you observed in your very close review of the README, we deliberately adopted a somewhat unadorned and journalistic style of writing about Radius: We focused on using language that is as objective and concise as possible, with an emphasis on educating the community, while avoiding language that might read as “selling” the project to anyone. As you observed, this style can admittedly result in prose that is somewhat “bland” vs “grand.” Our approach stems partly from our recognition that, while we believe Radius has significant potential, we also recognize we have a lot of work to do, jointly with the community, before we all fully realize that potential. So, we work to ensure our language and tone aligns with that reality. We also expect Radius community members to be predominantly engineers who often are distracted by more marketing-oriented language. So, we avoid most adjectives and adverbs that are strongly positive. With that context in mind, here is how I applied our communication style in reviewing your PR. Where I interpreted your edits to: Help increase message and clarity, I ask you to resubmit in a new PR Result in nearly identical message and clarity, I indicated we'd like to keep the original language Add positive adjectives or adverbs (such as “sophisticated” or “meticulously”), I indicated we'd like to keep the original language. Thanks again for this PR. It required me think much more thoroughly about documentation contribution guidance. I will update the doc contributor guide so others also have this context up front.

Thanks for the thorough and detailed review of my Pull Request. Now, I understand your state of mind when writing the Readme file.
I will make the suggested changes. Please clarify this.
In some comments, you mentioned updated PR and in some you said new PR, so do you want a single updated PR or would you like multiple PRs (wherever new PR is mentioned)?

@jonvsm
Should I close this Pull Request and create multiple separate for each of the reviewed changes?

@jonvsm
I have made the suggested changes. Please review them.

@jonvsm
Please review the changes I have made.


Why there is no jquery version?

kind of (+autoUnmask option for numeric):
$('#price,#sum').cleave({ numeral: true, numeralThousandsGroupStyle: 'thousand', autoUnmask: true});

(function($, window, undefined){ 'use strict'

  $.fn.cleave = function(opts) {

    var defaults = {autoUnmask: false},
        options = $.extend(defaults, opts || {});

    return this.each(function(){

      var cleave = new Cleave('#'+this.id, options), $this = $(this);

      $this.data('cleave-auto-unmask', options['autoUnmask']);;
      $this.data('cleave',cleave)
    });
  }

  var origGetHook, origSetHook;

  if ($.valHooks.input) {

    origGetHook = $.valHooks.input.get;
    origSetHook = $.valHooks.input.set;

  } else {

    $.valHooks.input = {};
  }

  $.valHooks.input.get = function (el) {

    var $el = $(el), cleave = $el.data('cleave');

    if( cleave ) {

      return $el.data('cleave-auto-unmask') ? cleave.getRawValue() : el.value;

    } else if( origGetHook ) {

      return origGetHook(el);

    } else {

      return undefined;
    }
  }

  $.valHooks.input.set = function (el,val) {

    var $el = $(el), cleave = $el.data('cleave');

    if( cleave ) {

      cleave.setRawValue(val);
      return $el;

    } else if( origSetHook ) {

      return origSetHook(el);

    } else {
      return undefined;
    }
  }
})(jQuery, window);

And how can i get getRawValue()
$(".my-input").getRawValue()

just use $(".my-input").val()

Nice, so I added a link in README, jQuery deserves more love than any other libs.
https://github.com/nosir/cleave.js#jquery-fn-usage

The line "var cleave = new Cleave('#'+this.id, options), $this = $(this);" should be "var cleave = new Cleave(this, options), $this = $(this);" to be fully compatible, you are otherwise assuming that whatever was passed to the JQuery function has an ID which may or may not be true

@Swizzy Thanks, you are right. I just don't remember why I needed 'id')

Any ideas on how to get .destroy() to work with this jQuery version?
Here's a fiddle of my attempt to get this to work.
My basic requirement is to be able to remove Cleave dynamically.

	$('#btnRemoveCleave').on('click', function() {
  	console.log('destroy');
    abc.data('cleave').destroy();
  });

Any ideas on how to get .destroy() to work with this jQuery version?
Here's a fiddle of my attempt to get this to work.
My basic requirement is to be able to remove Cleave dynamically.

$('#btnRemoveCleave').on('click', function() {
    console.log('destroy');
    abc.data('cleave').destroy();
});

@armenio Thank you! That worked

Just a minor change for others.
var cleave = new Cleave('#'+this.id, options), $this = $(this);
// should be
var $this = $(this), cleave = new Cleave($this.first(), options);

Can anyone please tell me how to import the addon for phone numbers?


'ModelConfig' object has no attribute 'model_name'. Did you mean: 'model_path'?

When i run examples/benchmark/txt2img/mii-sd.py, i ran into the following error:
'ModelConfig' object has no attribute 'model_name'. Did you mean: 'model_path'?

Any update on this?


appveyor: store build artifacts

Changes proposed in this pull request:

store build artifacts in app veyor

Your great patch is much appreciated. We are considering to apply your patch into the SoftEther VPN main tree.
SoftEther VPN Patch Acceptance Policy:
http://www.softether.org/5-download/src/9.patch
You have two options which are described on the above policy.
Could you please choose either option 1 or 2, and specify it clearly on the reply?

one

Nice. Thank you very much. I just squshed your PR into master


refactor(connlib): use events to handle ICE candidates

Instead of using the ControlSignal trait for ICE candidates, we introduce a poll_next_event function on Tunnel that reads ICE candidates from the various receivers connected to the callbacks of the PeerConnection.
There are still a few TODOs in the code and it relies on unpublished code in futures-bounded but I am putting it up for early review and to see whether CI passes.

@conectado Would appreciate an early review of this.

@conectado I had an attempt at making it more type-safe and it actually turned out quite well I think. Due to the existing split of functions on the Tunnel into different modules, I was able to "specialisesome of these impls to not be generic over theIceState` but specify a particular one.
This allows us to directly call functions on the two different ICE states and only be generic over the final poll function which is now type-safe towards the kind of ID we are returning.
Curious to hear what you think!

@conectado I had an attempt at making it more type-safe and it actually turned out quite well I think. Due to the existing split of functions on the Tunnel into different modules, I was able to "specialise" some of these impls to not be generic over the IceState but specify a particular one.
This allows us to directly call functions on the two different ICE states and only be generic over the final poll function which is now type-safe towards the kind of ID we are returning.
Curious to hear what you think!

Looks great but while we move to the new architecture I'd be wary of adding too many generic parameters on Tunnel, one extra is okay but I've had project where it grows to 10+ parameters and it starts being tedious to write the type and quite confusing to read!

@conectado I had an attempt at making it more type-safe and it actually turned out quite well I think. Due to the existing split of functions on the Tunnel into different modules, I was able to "specialise" some of these impls to not be generic over the IceState but specify a particular one.
This allows us to directly call functions on the two different ICE states and only be generic over the final poll function which is now type-safe towards the kind of ID we are returning.
Curious to hear what you think!

Looks great but while we move to the new architecture I'd be wary of adding too many generic parameters on Tunnel, one extra is okay but I've had project where it grows to 10+ parameters and it starts being tedious to write the type and quite confusing to read!

Yeah I fully agree. We can always re-purpose the current IceState to a role_state and store all things in there that are specific to a particular role. This actually gives me an idea that on what I want to change here to make this clearer.


Colourisation: async / await not coloured

method-id DoWorkAsync() yielding rt as string occurs any async.
    await Task::Delay(1000)
end method.

method-id Button_Click() async-void.
    await Task::Delay(1000)
end method.

method-id Button_Click() async-value.
    await Task::Delay(1000)
end method.

delivered in 4.4.3


support openEuler

SE-O DEK doesn't support openEuler officially, here is my try best patch to support it
it is not a total solution though, use at your own risk, DO NOT MERGE

upgrade ansible to support ansible_os_family for openEuler
openEuler has python3 only
openEuler has no firewalld installed
openEuler doesn't have /etc/redhat-release
adjust additional repos for openEuler
openEuler doesn't support to disable selinux
openEuler has no yum-utils, moreutils and inxi available
openEuler has its own way to install Docker

bump docker-py version from 6.0.1 to 6.1.0 to fix the urllib3 v2 incompatibility issue


Fix links to Why Aren't PipelineResources in Beta? v0.31.x

Changes

Backport https://github.com/tektoncd/pipeline/pull/4572
Links to the "Why Aren't PipelineResources in Beta?" section in the docs
should have aren-t in the fragment instead of arent. This can be
confirmed by clicking the link icon beside the heading and checking the
browser address bar.
/kind documentation
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[x] Docs included if any changes are user facing
[ ] Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Release notes block below has been filled in or deleted (only if no user facing changes)

Release Notes

/release-note-none

/retest


[mssql] Cannot set VARBINARY column to NULL in data editor

Repro steps:
create table vbtest (
  pk int primary key,
  vb varbinary(200) null
)

insert into vbtest
  (pk, vb)
values
  (1, hashbytes('SHA1', 'somepassword'))

Open the table in the dataset editor, right click the vb column and choose Edit -> Set to NULL. The column looks blank (not NULL) in the UI, and after refreshing the original values return.
The Script buttons shows this command:
-- Actual parameter values may differ, what you see is a default string representation of values
UPDATE foo.dbo.vbtest
SET vb=0x
WHERE pk=1 AND vb=0x go

which is clearly incorrect.

thanks for the bug report


Redesign CD pipeline page structure

Currently the page with CD pipelines is quite overloaded with information, let's make an approach so that there is one combined page to understand the nesting structure of CD pipelines and also a page for each CD stage with a detailed display of data.
Acceptance Criteria:

EDP user can open a separate page to display detailed information for each CD stages;
CD pipelines page contain information about main pipeline structure, without detailed information about CD stages;

Implemented.


SMSZ

Key
Value

View
https://database.shabados.com/sources/3/page/5/line/52

Source
Vaaran Bhai Gurdas Ji

Page
5

ID
SMSZ

Line
pUrk kuMBk ryckw³; invlI⁴ krm BuieAMgm⁵ Gyry [

@Sarabveer the 3 superscript numbers should be removed, correct?

RE ABOVE: If so, can we remove them programmatically throughout the source?


Add HubSpot specific Dispatcher changes

This restores several features to the Dispatcher that HubSpot is using:

Status information for PendingMove objects.
Better thread labeling.
Makes the Dispatcher Closeable.
Adds a public API for manipulating the move thread allocator lot size for invoking executePendingMove instead of dispatchAndCheckContinue.

This does not include the modification of adding maxIterationTime to Source objects because it's possible to set this at the Dispatcher level using a new constructor that was introduced since our internal fork version. That new constructor covers the use case we had.

@bbeaudreault I've tested this internally and all metrics indicate it is working as expected. Per our discussion, I'd like to get your approval here to merge.


[FEATURE] - Implement conventional changelog config spec

- [ ] bitbucket
- [ ] gitlab
Shall we support other platforms ? We will support them all

Could be better to implement the conventional changelog config specification

It would be way simpler to implement some pest parser crate like conventional_commits_parser_rs

Work in progress  here conventional_commits_parser_rs/pull/1


Minor release for version 2.0.1

Check List

[x] Merge #93 for fix on default configuration and compilation errors
[x] Update changelog
[x] Update Package.json

Changelog is now right i guess.


feat: Introduce name attribute to MenuItem component

Fixes #4168.

Codecov Report

Merging #4169 (ab0c62d) into master (bb2db59) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #4169   +/-   ##
=======================================
  Coverage   99.75%   99.75%
=======================================
  Files         180      180
  Lines        3241     3241
=======================================
  Hits         3233     3233
  Misses          8        8

Impacted Files
Coverage Δ

src/collections/Menu/MenuItem.js
100.00% <ø> (ø)

Continue to review full report at Codecov.

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

Unfortunately, we cannot merge this change as it is, please check https://github.com/Semantic-Org/Semantic-UI-React/issues/4168#issuecomment-1018401512.


test(e2e): increase wait time for fiat connect transfer

Description
There have been several timeouts on e2e tests while waiting for the Fiat Connect transfer in the wallet. This PR doubles that timeout to 4 minutes.
Test plan

Tested in CI

Related issues
N/A
Backwards compatibility
Yes
Network scalability
N/A

Closing in favor of #5557.


Zero Matrix LCCI

Write an algorithm such that if an element in an MxN matrix is 0, its entire row and column are set to 0.
Example 1:
Input:
[
[1,1,1],
[1,0,1],
[1,1,1]
]
Output:
[
[1,0,1],
[0,0,0],
[1,0,1]
]
Example 2:
Input:
[
[0,1,2,0],
[3,4,5,2],
[1,3,1,5]
]
Output:
[
[0,0,0,0],
[0,4,5,0],
[0,3,1,0]
]

this the comment when pull

this is the comment when pull


docs: add logo

Changes
Added a logo to logos page.
Testing
Not related, just updating documentation.
Docs
Added a logo of Blessing Skin. Here is a project which uses snowpack: https://github.com/bs-community/customskinloader-gui .

I've never heard of this before but I love it! Thanks for adding!


#951: Force formula re-calculation on file open from Excel

Added Calculcation Properties to Workbook and implemented generation of  'fullCalcOnLoad' attribute in order to force Excel to recalculate all formulas when workbook is open.

Whats the state here? Any work left?
Im also very interested in this feature.

@zymon Thanks for submitting this great PR - docs, tests, everything! Only comment is please don't modify the package version - this is managed during releases and is also likely to hit merge collisions.
Travis is green, will merge


Issue #217 : Added Implementation of Air-Cargo-problem Fig 10.1 for AIMA3e branch

Please check if the solution is correct. Also I wasnt sure if i have to write comments to explain the code. Please update me on that and also tell any changed in the code if they were to be made. Isssue #217

This is a neat little example. Unfortunately, it takes an imperative programming approach to what is intended to be a declarative approach (i.e. PDDL). The ideal way to tackle this problem is to have a specialized parser based on the FOL one, that knows how to extract out the Action, PRECOND, and Effect sections - validate they are of the correct form and make them available to the caller. The Air Cargo example itself them would just be a text file mirroring Figure 10.1 exactly.

ok i will try to do it with that approach..........


Compass helper image-url() returns broken URLs when relative_assets and debug_assets are activated

With realtive_assets and debug_assets disabled, this line of Sass:
background-image: url('foo.png')

results in this string:
background-image: url("/__assets/images/foo.png");

This URL works fine, the image gets displayed.
With realtive_assets and debug_assets activated, the same line of Sass results in the following CSS:
background-image: url("__assets/images/foo.png");

According to CSS spec, relative URLs are defined relative to the location of the CSS file. With the CSS file being located at /__assets/stylesheets, browsers would look for the image at the following URL:
http://localhost:4567/__assets/stylesheets/__assets/images/foo.png

The described behavior is identical for both Compass 0.12 and 1.0.

I'm doing some issue-gardening 🌿🌷🌾 and came across this issue.
Sorry that this has lingered for quite a while! With Middleman 4 the external pipeline is preferred method for assets. Does that help with this?


Patterns Table X-Axis[BUG]

What is the bug?
Clicking in the patterns table changes the X-axis of the chart to Jan 1 2022, regardless of the selected interval
How can one reproduce the bug?
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

What is the expected behavior?
A clear and concise description of what you expected to happen.
What is your host/environment?

OS: [e.g. iOS]
Version [e.g. 22]
Plugins

Do you have any screenshots?
If applicable, add screenshots to help explain your problem.
Do you have any additional context?
Add any other context about the problem.

duplicate of #1249 , the x-axis is beginning of the year because the internal interval value is year instead of what is displayed on UI


Add HMH Sistemas to index.html

We want to help as we can. Our team has experience with full stack and DevOps. This time will be our first time in a community project.

Added in https://github.com/opentffoundation/manifesto/pull/533


Fetch built docs and extract

Signed-off-by: Sutrannu<EMAIL_ADDRESS>

Can we get the complete website or webroot copied out of the docker container and into the src directory? maybe some place  like sawtooth-website/build or similar.

Can we get the complete website or webroot copied out of the docker container and into the src directory? maybe some place like sawtooth-website/build or similar.
@rberg2
The web root will be in generator/archive/htdocs after build.

Move docs/index.html into the site builder


set validator locale to C [CPP-886]

https://doc.qt.io/qt-6/qml-qtquick-doublevalidator.html#locale-prop
https://doc.qt.io/qt-6/qml-qtqml-qt.html#locale-method
enforce C locale to prevent typing '1,234' vs '1.234' on different keyboard layouts

https://swift-nav.atlassian.net/browse/CPP-886?focusedCommentId=72900
not sure if this is an issue required to fix but this should standardize locale as '1.234'

Frontend and Release Workflow Started here


Any workaround with ILMerge?

Anyone knows any workaround how can i embed tesseract.dll into my application?
I am using the Nuget package, and since version <IP_ADDRESS> when the Tesseract.Interop was made internal I am not able to embed it in my app as it throws an exception "the interface ILeptonicaAPISignatures should be public"
Any help will be highly appreciated!

I see,ok, thanks for your help!


Saving to assets does not work

Which IMT are you using?  Stable, BETA, BETA with Early Access
stable
What computer OS are you using? (ex:  Linux (version), MACOS, Windows (version))
windows
Describe the bug
I don't know how to save and publish imt's assets.Nothing happens when I click the Save to Asset button.
To Reproduce
Steps to reproduce the behavior:

Go to --Preset
Click on --Save to asset

Expected behavior
A clear and concise description of what you expected to happen.
I can see and publish my assets
Screenshots
Add screenshots to help explain your problem.

Additional information
Add any other relevant information pertaining to the issue goes here.

Error was fixed in version <IP_ADDRESS>


Incorrect variable type when sending photos in the method SendPhotoAsync

I would like to request/propose a feature

When using SendPhoto as a parameter, photo is of type InputOnlineFile.
In the TelegramBot documentation (https://core.telegram.org/bots/api#sendphoto), you can use the string type with file_id,
Will there be a solution to this problem?

Sorry, solution found, thanks @vova_lantsov
You can close it.


Excessive updating of screen

I haven't been able to pinpoint the exact point in the source code, but when running tippercanoe over an SSH connection, excessive printing to the screen really chews up bandwidth and uses up a lot of CPU within the screen session (running in between).
Perhaps tippercannoe could update the screen slightly less frequently?
This is just after printing when updating the %.
Sorting part 14 of 14
28965430 features, 466114348 bytes of geometry, 331596645 bytes of metadata, 635138385 bytes of string pool

Sorry to take so long to see this @andrewharvey. Is it the "Reordering geometry" percentage, the "Merging" percentage, or the tiling percentage that is the problem? Or all of them?

Would it make sense to simply have a quiet flag that would suppress all updates?

@morganherlocker OK, added -q in 138699d243483b2907ae480fd1d063938fb0e2a6

Sorry to take so long to see this @andrewharvey. Is it the "Reordering geometry" percentage, the "Merging" percentage, or the tiling percentage that is the problem? Or all of them?
@ericfischer It's okay, it's not urgent. Thanks for looking into this. I'm not actually sure which percentage. I'll try to find out exactly which one is causing me the issue.

I'm not convinced a quiet flag is the best fix here, since I can suppress it already by redirecting stderr to /dev/null. As a user I still want progress updates, just not so frequently that providing those updates is causing a bottleneck itself.

Sorry to take so long to see this @andrewharvey. Is it the "Reordering geometry" percentage, the "Merging" percentage, or the tiling percentage that is the problem? Or all of them?

@ericfischer It's only the tiling percentage which is updating too frequently, could that perhaps update less frequently rather than just quieting everything? I'll do some testing to see if I can find a good value.

@ericfischer I've done some testing, it's only https://github.com/mapbox/tippecanoe/blob/master/tile.cc#L530 which is the problem. However I'm noticing some strange behaviour.
For some reason it's repeating 0.1%  0/0/0 about 15,000 times before moving on to 0.2%  0/0/0, however when I added in fprintf(stderr, "%f / %f\n", progress, oprogress) just before the if (progress != oprogress), it suddently only prints one 0.1%  0/0/0 and about 15,000 0.000000 / 0.000000.
Are you able to confirm if it's printing repeated 0.1%'s for you like I'm seeing or if it's just me running into this issue?

@andrewharvey Thanks for investigating. When I tested just now I got:
  0.1%  0/0/0
  0.2%  0/0/0
  0.3%  0/0/0
  0.4%  0/0/0

etc with no repeats. Maybe there is some failure of floating point equality? What kind of computer and what kind of data set are you working with?

@ericfischer Hmm. When I compile without -O3 I get the same output as you now. Maybe a compiler bug then?
I'm using gcc version 4.9.2 (Debian 4.9.2-10) part of Debian jessie.

Or I'm corrupting memory in some way that the optimizer exposes, although I don't see any obvious cause. My Linux system has gcc 4.6.3 so it may be doing something different than your 4.9.2.

I've done some more debugging, I would say it's actually the former. I've noticed oprogress is getting set to 0, and it's not by the oprogress = progress; line.
I noticed in my dataset, it always printed 6.5% twice, so I ran the program up until https://github.com/mapbox/tippecanoe/blob/master/tile.cc#L533

(gdb) cont
Continuing.
  6.5%  0/0/0

All good up to this point, oprogress is 6.4, progress is 6.5 and so oprogress gets updated to 6.5...

(gdb) print oprogress
$1 = 6.4000000000000004
(gdb) print &oprogress
$2 = (double *) 0x7fffffffd530
(gdb) watch *0x7fffffffd530
Hardware watchpoint 2: *0x7fffffffd530
(gdb) next
Hardware watchpoint 2: *0x7fffffffd530

Old value = -1717986918
New value = 0
(gdb) print oprogress
$3 = 6.5

...continuing progress is 6.6 so another line gets printed and oprogress get's updated to 6.6...

(gdb) cont
Continuing.
  6.6%  0/0/0

Hardware watchpoint 2: *0x7fffffffd530

Old value = 0
New value =<PHONE_NUMBER>

... but now oprogress seems to be getting overwritten...

(gdb) cont
Continuing.
Hardware watchpoint 2: *0x7fffffffd530

Old value =<PHONE_NUMBER>
New value = 6585600
0x00007ffff6b48ea8 in __sprintf (s=0x7fffffffd5d0 "", format=0x425683 "%s/geom%d.XXXXXXXX") at sprintf.c:27
(gdb) bt
#0  0x00007ffff6b48ea8 in __sprintf (s=0x7fffffffd5d0 "", format=0x425683 "%s/geom%d.XXXXXXXX") at sprintf.c:27
#1  0x0000000000415c37 in traverse_zooms (geomfd=0x7fffffffdb50, geom_size=0x7fffffffdb30, metabase=0x7ffff5fe7000 "\b", stringpool=0x7ffff61d7000 "\001depth", file_bbox=0x7fffffffdc90, file_keys=0x7fffffffd830, midx=0x7fffffffdc8c, midy=0x7fffffffdc88, layernames=0x7fffffffd820, maxzoom=14, minzoom=0, outdb=0x631478, droprate=2.5, buffer=5, fname=0x7fffffffe585 "out.mbtiles", tmpdir=0x4242f1 "/tmp", gamma=0, nlayers=1, prevent=0x7fffffffe020 "", full_detail=12, low_detail=10, min_detail=7) at tile.cc:759
#2  0x00000000004079dc in read_json (argc=1, argv=0x7fffffffe290, fname=0x7fffffffe585 "out.mbtiles", layername=0x648f10 "top100k", maxzoom=14, minzoom=0, outdb=0x631478, exclude=0x7fffffffe000, include=0x7fffffffdfe0, exclude_all=0, droprate=2.5, buffer=5, tmpdir=0x4242f1 "/tmp", gamma=0, prevent=0x7fffffffe020 "") at geojson.c:971
#3  0x000000000040825d in main (argc=4, argv=0x7fffffffe278) at geojson.c:1164
(gdb)

I've looked through the code but I can't see it writing beyond the allocated size of the array, so I'm stumped.
It's repeatable every time I run the program.
What else can I try? If you wanted to look yourself the data I'm using is at http://tianjara.net/misc/tippecanoe_points.json.xz

Thanks. I'll download your data set and see if I can reproduce the problem.

Oh, it is a buffer overflow in that sprintf(), I think. The size of the array previously assumed that there were only 4 temporary files, but there can be up to 64 now, so it is sometimes going a character too long.

I wasn't able to reproduce it locally but I think I fixed this in 9d25afa. Could you please check if it works for you now? Thanks!

Thanks Eric. I now realise I've been noticing different issues on different systems, so when I was debugging locally I was detecting the buffer overflow, but when running on another system it was the float comparison.
So the progress != oprogress double comparison is causing an issue, apparently 0.300000 != 0.300000 because (int) (progress * 100) is 30 and (int) (oprogress * 100) is 29. Perhaps you should use fabs(progress - oprogress) > EPSILON rather than progress != oprogress. This resolves the issue for me.

Fixed in 56910fd016dfa5d6484c


Upgrade cudatoolkit to 11.0

This PR updates the cudatoolkit version in the daint build environment to 11.0 (the current default).

Maybe we can try to move up to cudatoolkit/11.2.0_3.36-<IP_ADDRESS>_2.2__g3ff9ab1?


Accessibility: Luminosity contrast ratio of keyboard focus indicator fails to meet the required ratio of 3:1

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: VS Code 1.89
OS Version: Windows 11 Enterprise 23H2 (OS build 22631.3296)

Copied from: https://github.com/microsoft/vscode-azureapicenter/issues/134
GitHub Tags
#A11yTCS; #A11ySev2; #Win32; #WCAG1.4.11; #GH_VSCodeextensionforAPICenter_Win32_Apr2024; #DesktopApp; #VS Code Extension for API Center; #A11yMAS; #Non Text Contrast; #A11YWCAG2.1;
Environment Details:
Application Name: VS Code Extension for API Center
OS: Windows 11 Enterprise 23H2 (OS build 22631.3296)
Repro Steps:

Create an API center in azure portal.
Open Visual studio Code.
Tab till API center extension in the left side panel and press enter.
Tab till definition under the Api name which you have created, and press enter.
The name which you have provided to the definition will appear for ex-"openapi4", right click on it.
Go to "Generate HTTP file" using the down arrow.
Verify whether the Luminosity contrast ratio of keyboard focus indicator on "Generate API client", "Generate HTTP file" and "Open API documentation" buttons fails to meet the required ratio of 3:1

Actual Result:
Luminosity contrast ratio of keyboard focus indicator on "Generate API client", "Generate HTTP file" and "Open API documentation" buttons fails to meet the required ratio of 3:1.
Expected Result:
Luminosity contrast ratio of keyboard focus indicator on "Generate API client", "Generate HTTP file" and "Open API documentation" buttons should be greater than or equal to required ratio of 3:1.
User Impact:
People with low vision often have difficulty perceiving tab focus that have insufficient contrast if Luminosity contrast ratio of visual focus indicator fails to meet the required ratio of 3:1. This can be exacerbated if the person has a color vision deficiency that lowers the contrast even further. Providing a lightness difference of 3:1 or greater can make these items more distinguishable when the person does not see a full range of colors.
Attachment:

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.89.0. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

Tested on latest 1.89.0, still have this issue.

Primary Bug:https://github.com/microsoft/vscode-azureapicenter/issues/134

Could you confirm the theme used here? I believe this was fixed for our default light/dark themes. See https://github.com/microsoft/vscode/issues/203400

Could you confirm the theme used here? I believe this was fixed for our default light/dark themes. See #203400

@AnKushSingh05 Please confirm,

Could you confirm the theme used here? I believe this was fixed for our default light/dark themes. See #203400

@AnKushSingh05 Please confirm,

I am using the dark visual studio theme

David is out on paternity leave.

Will be fixed via #224642


🔧 wallet name on first load

PR Type

[x] Bugfix

Context

[x] Continue #6021
[x] Fix wallet name on first load

Before submitting pull request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality

Screenshot 📸

Copilot Summary

🤖 Generated by Copilot at d8fc24e
The pull request cleans up the WalletAsset component and adds local storage functionality for the wallet name. This enhances the user experience and the maintainability of the code.

🤖 Generated by Copilot at d8fc24e

We've cleaned up the WalletAsset code, me hearties
No more console logs or unused vars to see
We've used a getter for the walletName, so tidy
And stored it in the localStorage, heave ho, heave ho

and on-chain identity? 👀

and on-chain identity? 👀

can you be more specific?


Dripsy's Button does not work with variants

I made a codesandbox to reproduce:
https://codesandbox.io/s/relaxed-spence-vtprq?file=/src/App.js
Despite defining buttons in my theme, passing variants to the <Button /> component from dripsy does not change the styles. Any ideas?
Dripsy version: ^2.1.0

Hey @harrisrobin, thanks for the issue.
Yeah, it's confusing that this is involved in the package at all. I should remove it.
You can see my temporary solution here: https://github.com/nandorojo/dripsy/issues/59#issuecomment-731614281

If you want to use a custom button, like from react-native-paper, you can use the useSx prop that will be in the next version. See #23

Just released v2.2.0 with support for useSx: https://github.com/nandorojo/dripsy/releases/tag/v2.2.0


Support print

[ ] String (print("str"))
[ ] Int expression (print(1+2))

@tdv42  both should work now, but there is no automatic test to check this.


Does the attention used in codes the same with the one in paper?

I find in function attn_head() (in utils/layers.py)
'''
simplest self-attention possible
f_1 = tf.layers.conv1d(seq_fts, 1, 1)
f_2 = tf.layers.conv1d(seq_fts, 1, 1)
logits = f_1 + tf.transpose(f_2, [0, 2, 1])
coefs = tf.nn.softmax(tf.nn.leaky_relu(logits) + bias_mat)
'''
In my understanding,the codes equals to $$f_1 W_1 + f_2 W_2$$
but in the paper, the chose attention mechanism use concatenation, and
$$W_1 = W_2 = W$$
Did I get something wrong?

Hello, I have encountered the same problem as you.
Also, in ./utils/layers.py, I didn't understand how the code calculates the correlation between f_1 and f_2.
I have seen the code for the pytorch version, and I think the implementation of the two is different in this place.
The pytorch version of the code is sensitive to the choice of random seeds, and switching to a different random seed can make the results very different.

Hello,
Thank you for your issue and interest in GAT!
The way in which attention heads are implemented here is exactly equivalent to the one in the paper, and it uses TensorFlow broadcasting semantics heavily.
For more details, see my response in this issue:
https://github.com/PetarV-/GAT/issues/15
Thanks,
Petar

Hi, Peter.
I have read the issue. Thank you very much for your reply, he has helped me a great deal.
Thanks，
Ice

Thanks a lot, Petar, I understood!


GPT-4V - suggesting follow-up question doesn't work

Please provide us with the following information:

This issue is for a: (mark with an x)
- [ ] bug report -> please search issues before submitting
- [ ] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

Any log messages given by the failure

Expected/desired behavior

OS and Version?

Windows 7, 8 or 10. Linux (which distribution). macOS (Yosemite? El Capitan? Sierra?)

Versions

Mention any other details that might be useful

Thanks! We'll be in touch soon.

resolved by #260


Fix for Linux make shim

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[ ] Have you successfully run brew tests with your changes locally?

The make shim for Linux had a bug in that it was removing Library/Homebrew/shims/linux/super directory from PATH for the duration of execution of make. This caused some undesired results.
Here, I moved the code that removes $HOMEBREW_LIBRARY/Homebrew/shims/linux/super directory from PATH to within the if/else clause to emphasize the fact that we have to do it only to detect the "real" make executable.
Props for identifying the issue go to @xu-cheng!

Has this already been merged to Linuxbrew? I think it would be good on Linux-specific PRs to identify which of them have been already merged and tagged in Linuxbrew and for which this is the first time the code will be used. I was a bit disappointed that the previous make shim PR had not been sufficiently tested on Linux.

It never got a chance to get tested on the CI because we had almost 2 weeks without bottling, due to various broken things, which took me forever to fix. I am currently building some bottles with this change right now, I'll come back to you once I am confident enough this works correctly.

Just one small comment, should we not use command -v instead of which?

Agreed 👍

I was a bit disappointed that the previous make shim PR had not been sufficiently tested on Linux.

My fault, sorry.

should we not use command -v instead of which

Agreed 👍

command -v catches aliases. type -pP?

With zsh: type -pP xxx gives: type: bad option: -P.

:) You’re either joking or missed the shebang on line 1
On Sun, Aug 5, 2018 at 6:49 AM Michka Popoff<EMAIL_ADDRESS>wrote:

With zsh: type -pP xxx gives: type: bad option: -P.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Homebrew/brew/pull/4570#issuecomment-410514911, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AMhAT8sxNMFIfEY81KRM8li446g7NFvIks5uNtvEgaJpZM4VmhJT
.

Woups sorry, didn't see the shebang. I was playing around with the type options in my terminal, outside of the brew context :)
I am OK with type -pP then.

I think this is probably good to be re-reviewed? Bottles have been building on Linux for a while with this change.

I restarted travis, seems like a network error

Leaving this as which for now until the change has been made/tested in Linuxbrew/brew. I'd like for anything like this to go through testing in Linuxbrew/brew until (hopefully relatively soon!) Homebrew/brew becomes the default for Linuxbrew users.


Update Composer dependencies (2024-02-06-00-15)

Visual regression test passed!
View the visual regression test report


UserInterfaceSprint_LeeFormento-FSW14

Answered all the questions
Finished home page
Finished about page
Finished CSS file.
Mid-way through the project, I realized that I should have been doing "git commit -m" whenever I've finished writing out a feature. I'll do that next time.

Good Job Lee! You're doing amazing! Keep up the good work ❤


breaking: Remove RawNode.TickQuiesced()

Summary
This PR removes the TickQuiesced() method from the RawNode in the Raft library. This method was deprecated in v3.6.0-alpha.0 and is scheduled for removal in the roadmap for the v4.0.0 release.
To advance the logical clock, please use the Tick() method instead.
Related ticket: #66
Related PR: #62
Changes

Removed the TickQuiesced() method from RawNode.

Breaking Change
This PR introduces a breaking change. Any code using TickQuiesced() will no longer compile.
Migration Guide
Replace all occurrences of TickQuiesced() with Tick():
// Before
rawNode.TickQuiesced()

// After
rawNode.Tick()

@ahrtr
Hello. This is my first time contributing to etcd-io/raft.
I handled a ticket in the v4.0.0 milestone. Please let me know if you have any updates on the release status of v3.6.0 (If there are any critical blockers, I would like to help).
Thank you.

Thanks for raising this PR. We haven't cut release-3.6 branch yet. Will approve & merge this PR once it's done.

Thank you for checking and have a great weekend.


Don't use floats for currency representation.

Please use a decimal type or store it in two ints. Inevitable representation errors will lead to money being added or lost.

I changed the code to use Decimal values now using the Shopspring library
Thank you for your suggestion.


BuildKite integration not working with latest BuildKite version when not building for pull request.

I have just been trying to setup Danger on an iOS project that builds on BuildKite. I am not sure if this has changed recently but BuildKite seems to be setting an environment variable with BUILDKITE_PULL_REQUEST=false when the build is not the result of a pull request.
This causes Danger to try the url GET https://api.github.com/repos/.../pulls/false which results in a 404.
I am not sure if BuildKite changed recently but this occurs with the latest version of the BuildKite agent running on OSX.

It looks as if when the build is not the result of a pull request the environment variables are as follows.
BUILDKITE_PULL_REQUEST_REPO=
BUILDKITE_PULL_REQUEST=false
When it is a pull request they are
BUILDKITE_PULL_REQUEST_REPO=git://github.com/....git
BUILDKITE_PULL_REQUEST=1
Can probably check if BUILDKITE_PULL_REQUEST_REPO is nil before proceeding.

Can probably check if BUILDKITE_PULL_REQUEST_REPO is nil before proceeding.

Sounds like a good idea, wanna give it a shot?

I have given it a go!


[FLINK-19939][table-planner-blink] Remove redundant union from multiple input node

What is the purpose of the change
Consider the following SQL and the execution plan.
WITH
  T1 AS (SELECT COUNT(*) AS cnt FROM x GROUP BY a),
  T2 AS (SELECT COUNT(*) AS cnt FROM y GROUP BY d),
  T3 AS (SELECT b AS cnt FROM x INNER JOIN y ON x.b = y.e)
SELECT cnt FROM
  (SELECT cnt FROM T1)
  UNION ALL
  (SELECT cnt FROM T2)
  UNION ALL
  (SELECT cnt FROM T3)

MultipleInputNode(readOrder=[1,0,0,0], members=[\nUnion(all=[true], union=[cnt])\n:- Union(all=[true], union=[cnt])\n:  :- Calc(select=[CAST(cnt) AS cnt])\n:  :  +- HashAggregate(isMerge=[true], groupBy=[a], select=[a, Final_COUNT(count1$0) AS cnt])\n:  :     +- [#3] Exchange(distribution=[hash[a]])\n:  +- Calc(select=[CAST(cnt) AS cnt])\n:     +- HashAggregate(isMerge=[true], groupBy=[d], select=[d, Final_COUNT(count1$0) AS cnt])\n:        +- [#4] Exchange(distribution=[hash[d]])\n+- Calc(select=[b AS cnt])\n   +- HashJoin(joinType=[InnerJoin], where=[=(b, e)], select=[b, e], build=[right])\n      :- [#1] Exchange(distribution=[hash[b]])\n      +- [#2] Exchange(distribution=[hash[e]])\n])
:- Exchange(distribution=[hash[b]])
:  +- Calc(select=[b])
:     +- LegacyTableSourceScan(table=[[default_catalog, default_database, x, source: [TestTableSource(a, b, c, nx)]]], fields=[a, b, c, nx])
:- Exchange(distribution=[hash[e]])
:  +- Calc(select=[e])
:     +- LegacyTableSourceScan(table=[[default_catalog, default_database, y, source: [TestTableSource(d, e, f, ny)]]], fields=[d, e, f, ny])
:- Exchange(distribution=[hash[a]])
:  +- LocalHashAggregate(groupBy=[a], select=[a, Partial_COUNT(*) AS count1$0])
:     +- Calc(select=[a])
:        +- LegacyTableSourceScan(table=[[default_catalog, default_database, x, source: [TestTableSource(a, b, c, nx)]]], fields=[a, b, c, nx])
+- Exchange(distribution=[hash[d]])
   +- LocalHashAggregate(groupBy=[d], select=[d, Partial_COUNT(*) AS count1$0])
      +- Calc(select=[d])
         +- LegacyTableSourceScan(table=[[default_catalog, default_database, y, source: [TestTableSource(d, e, f, ny)]]], fields=[d, e, f, ny])

The two unions here in multiple input here is actually redundant, as the amount of data shuffled will not change even if they're moved out of the multiple input node. We should remove such redundant union from multiple input nodes.
Brief change log

Remove redundant union from multiple input node

Verifying this change
This change added tests and can be verified by running the added tests.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

The failure is caused by https://issues.apache.org/jira/browse/FLINK-20006


In-line comments recognition

Dear Sir,
I just started using the Modern Fortran extension and I like it so far. The source files I am working with are written in different Fortran versions: starting Fortran77 and up to Fortran95. The OS used is OpenVMS. As of now I have one problem: the editor does not recognize the in-line comments, which start with the exclamation mark. The editor marks those as problems and except for that it is a bit annoying, it also affects the following code analysis.
Is there a way to adjust some settings in order to avoid the problem or it can only be fixed in the extension source code?
Thank you for your help in advance!
Respectfully,
Mike Faynberg

Could you post a minimal working example of the code that is giving you grief?

Interesting. I cannot replicate your problems. Specifically the errors I get are that at line 98 the loop ends unexpectedly, which is correct, since the if-conditional you have added is missing its parenthesis and hence the compilation will abort at line 95.

As for the other 2 errors it's due the PROGRAM begin and ending.
BTW you have a non-standard character at line 49.
I would suggest you provide some additional information about your system, vscode extension versions etc.m see help>about as well as your gfortran --version output and any additional settings you pass to the extension.
Also have you tried opening a simple fortran file/project? Did that work? If not you might have to uninstall and reinstall your extension.

I think my help can be very limited since I don't work with fixed form Fortran.

do you know any way to force VSCode recognizing the .txt files as source files

Open a .TXT file, Press Ctrl + P, type Change Language Mode, Configure File Association for .TXT and select  Fortran.

With regards to the second file that you sent, does it compile?
You seem to also have run into #87 which marks some of your long lines as comments.
However, that should only affect the highlighting and not the linting.
I noticed that you use d or D characters for comments. This is not considered a standard comment, so for linting to work (with gfortran) you pass -fd-lines-as-comments. If you don't the compiler will throw errors.
In general, if you are using certain arguments/flags, custom include paths, etc. for your compilation, you will have to also include them as arguments for the extension with linterExtraArgs in your settings.json. You don't have to include any optimisation arguments.
What I would suggest is you create a Minimal Working Example and paste the code here.
The files that you have provided are too big, full of project specific variables so we need a considerable amount of time to try and understand the code.
An example with a single erroneous linting result would be a good place to start.

Hi Gnikit and thank you so much!
Here is what I would like to say:

By pressing Ctrl + P I can navigate over the list of currently opened files. When I typed Change Language Mode in the text box, I receive a "No matching results" reply. which seems logical under the circumstances... What do I miss there?
I am not interested in compilation - at least not now. I am only using the environment for editing/linting the code written in VSI Fortran (for OpenVMS);
Regarding your recommendation for correct processing of the lines that start with D/d - where should I type the
-fd-lines-as-comments in order it to be processed? Sorry, I have no experience with VSCode so far...
Best,
Mike

Re:

https://stackoverflow.com/questions/29973619/how-to-make-vs-code-treat-a-file-extensions-as-a-certain-language
The linting cannot produce reasonable results if the code cannot compile. That's what linting is after all, it is a mock compilation to identify mistakes
In your settings file (settings.json) add the option "fortran.linterExtraArgs": ["your", "args"],. Please have a look at the README.md: https://github.com/krvajal/vscode-fortran-support/blob/master/README.md of the extension. Other arguments I tend to use are "-ffree-line-length-none", ffixed-line-length-none", to prevent the 72 character limit. See gfortranflags and arguments for more. Also, I assume you have installed gfortran in your Windows machine otherwise the linter will not work at all.

Gnikit, thank you! I will certainly follow your advice!

I am closing this issue due to inactivity, feel free to reopen if the problem persists


HTTP Client examples

Add sample code for HTTP Client, as supplemental material for the docs. This code is intended to help users get started and play with the client. It includes an express-based server for easy testing.

Used a workaround for running the tests in actions because running it as part of the matrix caused issues because both instances used the same port. For the sake of simplicity I chose not to make the port configurable but to have two separate jobs which run in isolation.
Addressed the other comments, hope that's ok @FrankEssenberger