KeyError: 'stop_time_update'

I've tried a few github projects using the mta api and am running into the same issue in multiple.
When running in normal configuration, nothing changed, I get this error:
Traceback (most recent call last):
  File "mta_notification.py", line 46, in <module>
    station_time_lookup(realtime_data, 'R36N')
  File "mta_notification.py", line 37, in station_time_lookup
    unique_arrival_times = unique_train_schedule['stop_time_update'] # arrival_times is a list of arrivals
KeyError: 'stop_time_update'

This is some sort of dictionary error, but I don't understand why it doesn't see this key when I can see it fine when printing that data.
Is it something to do with processing power? I'm running this on a Raspberry pi zero, quite low powered.

I've found one potential cause/band-aid, but I'm not experienced enough to fix the bug. Increasing the time interval in the following line helped me:
Change the following line
if time_until_train > 3: to if time_until_train > 10:
However, increasing it too much, say to 20 mins, results in the same error.


Module 1 - Lab 1 - Exercise 2 - downloaded Word doc password protected

Module: 01
Lab/Demo: 01 (exercise 2)
Task: 01
Step: 03
Description of issue
The file 'RS4_WinATP-Intro-Invoice.docm' is password protected.
Through checking on other MSFT sites, (reference - https://winatpmanagement.securitycenter.windows.com/client/management/static/AttackSimulationDIYv2.pdf ) the pwd appears to be:
WDATP!diy#
Although pwd is included in the 'learn more' portion of the simulation,  Instructions here should be updated to include the pwd, as many students will immediately try and open the doc, not know the pwd, and call support.

The password is already listed in the simulation instructions as you've noted.  That said I have updated the lab instructions to be more clear about exactly which instructions to look at first so the learners see that password.


tensorflow install for go

bojuzideMacBook-Pro:~ manman$ go get github.com/tensorflow/tensorflow/tensorflow/go
github.com/tensorflow/tensorflow/tensorflow/go
ld: library not found for -ltensorflow
clang: error: linker command failed with exit code 1 (use -v to see invocation)
bojuzideMacBook-Pro:~ manman$

bojuzideMacBook-Pro:~ manman$ "go get github.com/tensorflow/tensorflow/tensorflow/go"
github.com/tensorflow/tensorflow/tensorflow/go
ld: library not found for -ltensorflow
clang: error: linker command failed with exit code 1 (use -v to see invocation)
bojuzideMacBook-Pro:~ manman$

What is the fix for this - I've tried multiple approaches to installing libtensorflow including brew but no luck

@angadn: you most likely forgot to:
export LIBRARY_PATH=$LIBRARY_PATH:/usr/local

After Installing TensorFlow on Mac OS X go through steps 2 and 3 of Installing TensorFlow for Go.

i still get this error, and have tried everything here and else where

@oziee Thank you this worked for me after a lot of other things didn't


Semi-colons stripped from multi-line default exports

I am running prettier against master for all the great jsx fixes. I'm still seeing it strip semi-colons from some default exports.
My original code:
export default ({ children }) => (
  <Provider store={store} key="provider">
    <IntlProvider locale="en">
      {children}
    </IntlProvider>
  </Provider>
);

Prettier-formatted code:
export default ({ children }) => (
  <Provider store={store} key="provider">
    <IntlProvider locale="en">
      {children}
    </IntlProvider>
  </Provider>
)

I know there have been other issues around semi-colons in default exports but it seems this is an edge case that hasn't been fixed yet.

Thanks for reporting this! It indeed hasn't yet been fixed.
@existentialism do you want to take a shot at this one since you did #287?

Gah, not sure why I didn't think about this. Fix otw!

@existentialism btw, we are on gitter if you want to chat with us :) https://gitter.im/jlongster/prettier

@existentialism @vjeux thanks for the quick feedback! you all rock!

Thanks @existentialism! When I saved the file with the above example in it, I can confirm that it put the semi-colon back!


IAM for cluster-group - List and create not working

Bug description
It is not possible to list or create an IAM policy

List: output is null
Create: Output: Error: Scope needs to be specified

Scope is specified

Expected behavior
list
Steps to reproduce the bug / Relevant debug output
command: tanzu mission-control iam list --scope cluster-group -n
Output: null
Command: tanzu mission-control iam add-binding --scope cluster-group --role  --users  --cluster
Command: tanzu mission-control iam add-binding --scope cluster-group --role  --users  -n
Output:
Error: Scope needs to be specified
Output of tanzu version
version: v1.1.0
buildDate: 2023-11-01
sha: d0679f5a
arch: arm64
Environment where the bug was observed (cloud, OS, etc)
TMC SaaS

Thanks for the report @bwaldecker we're investigating but to help can you provide the output of tanzu plugin list?

maybe to highlight: listing and creation on cluster level works as expected
output from "tanzu plugin list"
Standalone Plugins
  NAME           DESCRIPTION                                                        TARGET  VERSION  STATUS
  pinniped-auth  Pinniped authentication operations (usually not directly invoked)  global  v0.31.1  installed
  telemetry      configure cluster-wide settings for vmware tanzu telemetry         global  v1.1.0   installed

Plugins from Context:  TMC-SaaS
  NAME                  DESCRIPTION                                                     TARGET           VERSION  STATUS
  account               Account for tmc resources                                       mission-control  v0.2.0   installed
  agentartifacts        helm for tmc resources                                          mission-control  v0.1.17  installed
  aks-cluster           Manage and unmanage tmc aks clusters                            mission-control  v0.3.2   installed
  apply                 Create/Update a resource with a resource file                   mission-control  v0.3.14  installed
  audit                 Run an audit request on an org                                  mission-control  v0.1.17  installed
  cluster                                                                               mission-control  v0.2.12  installed
  clustergroup          A group of Kubernetes clusters                                  mission-control  v0.1.17  installed
  context                                                                               mission-control  v0.1.12  installed
  continuousdelivery    Continuousdelivery for tmc resources                            mission-control  v0.1.17  installed
  data-protection       Data protection for tmc resources                               mission-control  v0.1.17  installed
  ekscluster                                                                            mission-control  v0.2.2   installed
  events                Events for any meaningful user activity or system state change  mission-control  v0.1.17  installed
  helm                  helm for tmc resources                                          mission-control  v0.1.18  installed
  iam                   IAM Policies for tmc resources                                  mission-control  v0.1.17  installed
  inspection            Inspection for tmc resources                                    mission-control  v0.1.17  installed
  integration           Get available integrations and their information from registry  mission-control  v0.1.17  installed
  management-cluster    A TMC registered Management cluster                             mission-control  v0.2.18  installed
  policy                Policy for tmc resources                                        mission-control  v0.1.17  installed
  provider-aks-cluster  Manage and unmanage tmc provider aks clusters                   mission-control  v0.1.18  installed
  provider-eks-cluster                                                                  mission-control  v0.1.17  installed
  secret                secret for tmc resources                                        mission-control  v0.1.19  installed
  setting               Setting plugin for tmc resources                                mission-control  v0.2.15  installed
  tanzupackage          Tanzupackage for TMC resources                                  mission-control  v0.4.3   installed
  workspace             A group of Kubernetes namespaces                                mission-control  v0.1.19  installed


Inconsistent behavior between API and CLI when processing the start_url

The CLI may fail to report an invalid start_url member if it's specified as a relative URL. Internally, the tool converts any relative values to absolute using the target site as the base URL before performing the validation. This transformation may cause certain invalid URLs to become valid. Calling the API directly uses the original value obtained from the manifest.

Added validation of the start_url before it is transformed (see https://github.com/manifoldjs/ManifoldJS/commit/b629bcbf3edcda7d8a193d84f9b05dda753877b5).


Test med gridLayer og image overlay

Coverage decreased (-0.3%) to 43.208% when pulling 63a40e60c261a806b35e4cb1769243f044c418c5 on feature/bugfix into 91d17b855bcba836d32d96e65cf15b5ba61a797e on master.


Manwadkaraksh/disabled update button bug

This PR fixes the issue where update iModel button would remain disabled when previously set map extent coordinates were removed.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Akshata Manwadkar seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


fix(client): Remove horizontal scrollbar from donation page

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] All the files I changed are in the same world language (for example: only English changes, or only Chinese changes, etc.)
[x] My changes do not use shortened URLs or affiliate links.

Removes the horizontal scrollbar by wrapping the row in a container.  This made the form a little small. To fix this I adjusted the column classes.  At the same time, I tried to smooth out the transition from small to large screens
Closes #36127

@moT01 So it does.  There may not even be a conflict - it looks like they don't touch that bit of the code.

Hi @ojeytonwilliams looks like we got into a merge conflict. Can you rebase this?

@raisedadead That'll teach me to make predictions.
I made three commits because they cover slightly different things. The first commit just merges, the second is a tweak to the style and the final one deals with a warning from the donate-other component. Apparently, even though the buttons aren't <li>s, React wants them to have keys.


refactor(levm): memory module

Motivation

The Memory implementation was similar to type aliasing a Vec<u8>. The current implementation unnecessarily adds an extra layer of indirection.
The current Memory implementation has some bugs and not all its methods are panic safe.

Description

Memory is now a type alias for Vec<u8>.
All the Memory "methods" are implemented as functions in the memory module and called taking advantage of the namespacing (e.g. memory::load_word(...)).
All the Memory functions are panic safe.
Memory access and expansion cost functions were refactored and moved to the memory module. As a side note, in a future refactor we will be updating the gas cost type from U256 to usize, this PR kind of introduces this notion, but it turns the usizes into U256 wherever is necessary for backward compatibility the implementation.

I think the problem with the EF Tests run will be fixed after https://github.com/lambdaclass/ethrex/pull/1394 is merged.


Two tests fail during build on Ubuntu.

Confirm by changing [ ] to [x] below to ensure that it's a bug:

[x] I've gone though Developer Guide and API reference
[x] I've searched for previous similar issues and didn't find any solution

Describe the bug
Building as follows:
mkdir build
cd build
cmake .. -DCMAKE_BUILD_TYPE=Release -DBUILD_ONLY="ecr"
make
I get two build failures:
[----------] Global test environment tear-down
[==========] 387 tests from 68 test cases ran. (4722 ms total)
[  PASSED  ] 385 tests.
[  FAILED  ] 2 tests, listed below:
[  FAILED  ] HttpClientTest.TestRandomURLWithProxyAndOtherDeclaredAsNonProxyHost
[  FAILED  ] HttpClientTest.TestRandomURLWithProxy
Scrolling up, they seem to both be the same problem:
[ RUN      ] HttpClientTest.TestRandomURLWithProxyAndOtherDeclaredAsNonProxyHost
/home/jpl/aws-sdk-cpp/aws-cpp-sdk-core-tests/http/HttpClientTest.cpp:40: Failure
Failed
The curl error code has to be 28 in [curlCode: 7, Couldn't connect to server]
[  FAILED  ] HttpClientTest.TestRandomURLWithProxyAndOtherDeclaredAsNonProxyHost (5 ms)
...
[ RUN      ] HttpClientTest.TestRandomURLWithProxy
/home/jpl/aws-sdk-cpp/aws-cpp-sdk-core-tests/http/HttpClientTest.cpp:40: Failure
Failed
The curl error code has to be 28 in [curlCode: 7, Couldn't connect to server]
[  FAILED  ] HttpClientTest.TestRandomURLWithProxy (4 ms)
SDK version number
Platform/OS/Hardware/Device
Ubuntu 18.04 LTS instance inside Docker.
To Reproduce (observed behavior)
See build steps above.
Expected behavior
Expect a clean build.
Additional context
At this point I'm trying to figure out how to turn off tests.

This fails on the master branch or if I "git checkout 1.8.167" but is fine if I "git checkout 1.8.145", which is from mid-February.

@jplflyer I've found the flags below for turning off the tests in the CMakeLists.txt file
-DENABLE_TESTING=OFF -DAUTORUN_UNIT_TESTS=OFF -DENABLE_HTTP_CLIENT_TESTING=OFF

Hi,
Thanks for reporting.
I created a clean 18.4 Ubuntu instance in EC2 and build the SDK following your instructions, but I couldn't reproduce.
For the build I installed libssl, libcurl and zlib that were available in apt-get.
The errors seem related to interactions with Curl, can you check what version of Curl you using? I got 4.5.0, I want to see if there is a specific version I need for it to reproduce?

I'll see if I can create a Dockerfile with just enough to reproduce it.

Okay, I have a repo on GitHub:
https://github.com/jplflyer/FailingAWS
There's a README, a Dockerfile, and a Makefile. Directions in the README, but it's a matter of building the docker image (takes about 10 minutes on my Mac), attaching to the container, then CD-ing into the waiting directory and doing the build process. Everything is in the README.md (hopefully).

And as a reminder, building used to work. If I check out a previous version from mid-February, that builds properly. So it's something new.

The reason those test didn't fail in previous versions is because they are new, we added support for filtering proxy hosts between those versions. Thanks for sharing the Docker construction. Super useful, I will try it out today and reply with my findings.

Thank you for the explanation. My workaround is working -- I just check out an older version. It'll be interesting to know what I might be doing in my instance that's problematic.

I was able to reproduce following your steps

There are so many reasons it could be failing for me. It might be a setting in the phusion base image. Maybe I need to use a newer version than the default GCC. Maybe there's a network configuration I'm not doing. Maybe there's just something weird about proxies from Docker containers. But that's a pretty minimal Dockerfile.
Maybe I shouldn't be building from the master branch, and there's a release branch I should use, but I didn't find one.
In any case, I look forward to knowing what the ultimate answer is.

I tested building like this
cmake .. -DCMAKE_BUILD_TYPE=Release -DBUILD_ONLY="ecr"
cmake --build .

and never failed. But when building using make like this
cmake .. -DCMAKE_BUILD_TYPE=Release -DBUILD_ONLY="ecr"
make -j9

It fails the test you mention in the first run, but giving it a retry succeeds the build.
Not sure what it is yet. But it explains how it passed CI at least. It consistently fails make build when triggered by make directly in the first time, but not in the retry, neither when called directly from cmake --build.

I've never been fond of cmake and don't know the implications of what you've discovered so far, although that seems both interesting and weird. Gotta love the weird ones, huh?

The bug seems very build related, I coudn't reproduce in other platforms, neither using different parameters to call cmake in that particular one, but I got a consistent way to reproduce in the docker you shared.
So, I gave it a try to see what happens if cmake is upgraded if it is still reproduceable.
Moving a minor version up (3.10 -> 3.11) makes is unreproducable keeping all things the same.
This is how I did the upgrade for cmake (couldn't use apt-get since 3.10 is the last released version to 16.04)
sudo apt purge cmake
wget https://github.com/Kitware/CMake/releases/download/v3.11.0/cmake-3.11.0-Linux-x86_64.tar.gz
tar -xvf cmake-3.11.0-Linux-x86_64.tar.gz
cd cmake-3.11.0-Linux-x86_64
sudo cp -r bin /usr/
sudo cp -r share /usr/
sudo cp -r doc /usr/share/
sudo cp -r man /usr/share/

I hope this helps unblock you.
I will keep looking into it in case there is more in it to find, but triage seems to indicate its an old sharp edge of cmake we are hitting that was addressed in cmake itself already almost 3 years ago (3.11 is a 2018 version, current one is 3.20).

I have a similar issue, where I'm building the SDK in a Docker image based off of Ubuntu 20.04.
I bisected the unit-test failing regression to happen somewhere between 1.8.166 and 1.8.167.
  The current setup uses CMake 3.16.0.
I tried invoking cmake --build . instead of ninja, and it still fails on the same tests.

Thanks for the added input. I'm looking into improving the CMakeFile definition to better reproduce and control the building.
I will update when I have more info.

Hi,
Update on the broken tests. The test line checking what requests where captured by the httpmocker is trying to read the "back" of an empty vector.
I'm adding assertion to check that never happens in the mocker and reviewing all the test suite to check we don't hit the same issue in any other place.
I will keep updating.

The fix was released yesterday for the Raspeberry fail in main branch.
I'm now testing it in the docker provided by @jplflyer  to confirm the fix works in both cases.

Ubuntu 16 build has another blocking issue when running the tests.
It now fails on the HttpClientTest.TestRandomURLWithProxy test.
I will update after looking into it in more detail.

Todays release includes the fix the other error. Tested in the provided container.  Build success.
Thanks @jplflyer for the container reproduction, it was really useful for root-causing both issues.

Thank you, @sdavtaker. I have verified it now builds cleanly. Happy to have helped with the container, and ecstatic to have a working build.


Add algorithm for computing intersection rect.

Issue #62
Not sure if this level of detail is too agonizing or not, please advise.

LGTM


Expand TaskOutput::Return values in the turbo-tasks macros

Description

Testing Instructions

Closes PACK-3041

#8096  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @bgw and the rest of your teammates on  Graphite


Make bedrock2 target depend on coqutil

See related discussion here: https://github.com/project-oak/silveroak/issues/268#issuecomment-800498955
After I ran into the third_party bug we'd been seeing on CI locally, I made a quick bash script to run the make repeatedly so I could see if the transient bug still existed. This change seems to fix the problem; I ran the bug-finding script on the current main and found the transient bug 7 times in 100 runs, and found it 0 times with the fix.

Self-merging this one, let me know if there are objections and I'll revert.


from origin 'null' has been blocked by CORS policy:

var writer = HanziWriter.create('target', '我', {
  charDataLoader: function(char, onComplete) {
    $.getJSON("/data/" + char + ".json", function(charData) {
      onComplete(charData);
    });
  }
});

I want use it in Android but  load local data error:
index.html:1 Access to XMLHttpRequest at 'file:///xxxx/hanzi/data/%E4%B8%96.json' from origin 'null' has been blocked by CORS policy: Cross origin requests are only supported for protocol schemes: http, data, chrome, chrome-extension, chrome-untrusted, https.

Sorry for the slow reply. This seems like more of a question about how to load a local json file in an Android webview. I don't have a lot of experience with Android webviews, but you can try googling about it. Maybe something like this? https://stackoverflow.com/questions/20425481/android-webview-ajax-local-files

Sorry for the slow reply. This seems like more of a question about how to load a local json file in an Android webview. I don't have a lot of experience with Android webviews, but you can try googling about it. Maybe something like this? https://stackoverflow.com/questions/20425481/android-webview-ajax-local-files

I have make it ，thank you


Why we have 2 http client dependency?

Hi, I am super new to java, So I am wondering why we have 2 http client dependencies?
https://github.com/meilisearch/meilisearch-java/blob/2713a446c3198b2aab22bf3f9df64b1769197ec7/build.gradle#L41-L43

Hi @ppshobi,
It comes from this PR.
I'm not sure of the why, I think the goal was to allow the user to create a custom HTTP client.
Maybe @eskombro could answer better.

I think I got why multiple dependencies are listed, but whenever I try to build the project I am getting errors like following

/Project/meilisearch-java/src/test/java/com/meilisearch/sdk/http/CustomOkHttpClientTest.java:19: error: package okhttp3 does not exist
import okhttp3.*;

or

/Project/meilisearch-java/src/test/java/com/meilisearch/integration/DocumentsTest.java:14: error: package org.jetbrains.annotations does not exist

is there a command where I can Install all the dependencies?

Normally ./gradlew build should install all the dependencies.
But you can try:
./gradlew build --refresh-dependencies

@alallema Thanks closing this issue


Add database tables for storing data generated by tarmo admins

There are some pieces of information we cannot fetch directly from any datasource. Rather, admin of the tarmo application should make decisions regarding:

[ ] visibility field value for each object in tamperewfs_luonnonmuistomerkit, tamperewfs_luontorastit, tamperewfs_luontopolkureitit, museovirastoarcrest_rkykohteet and museovirastoarcrest_muinaisjaannokset tables + OSM tables (request from client)
[ ] season field value for each object in lipas_pisteet and lipas_viivat tables.

After the decisions are made, the generated data needs to be stored somewhere. In other words, adding more database tables is needed.

In later phases adding more this kind of data (data created for this application which cannot be fetched directly from any datasource) might become topical.

A field storing image urls from each object in kooste schema is one such possibility
Another possibility is adding a field into which tarmo admin could add some additional information regarding to an object in kooste schema (inner notes, not visible to the user)

@pavetsu14 visibility field propably to OSM-parking is needed also - this was requested by the client (some of the parking places are not publicly available)

@pavetsu14 season is not necessary information at all. It was requested that the skiing routes etc. are always visible (there are people who love to check the routes before hand and design their winter activities). But winter sports could be arranged with the filter-tool to be  "at the bottom" of the list when it is summer. I'll add another issue on this.

We have not intended to show season value for the users. It is just stored in the database in order to make it possible t show only summer POIs in summer but if this is not desired, we can surely alter it show that the order changes. Either way, the knowledge about season each point is related to, is needed.

Or just to have it done in the UI/frontend: if category is skiing or skating ==> icon goes to last when date is 1.5.

I think this issue has become obsolete, closing this.


Named dashboard regions / launch widget to region

As described in the Google Group (at https://groups.google.com/forum/#!topic/ozone-developers/gH7qd19eLyc as well as previous threads from other folks), we're looking for an option to name dashboard regions, and then launch a new widget into a specific region.

At this point we don't plan to add new features to OWF.  Please see OZP for the next-gen of Ozone platforms.
As additional dashboard capabilities are added to OZP, we will keep the idea of named panes in mind.


Move gnome-vrr to a multistage build

Hmmm, what do I copy from gnome-vrr?

https://ostree.readthedocs.io/en/stable/manual/buildsystem-and-repos/ feels relevant.

Ok so here's the best solution I have so far (thanks to jstone for this one):
In ublue-os/gnome-vrr we add the copr, but then download the RPMs and put them in a payload directory
Then here in ubuntu we:
COPY --from=ublue-os/gnome-vrr /rpms/* /rpms
rpm-ostree install /rpms/blah
rpm-ostree override /rpms/blah

It would be nice if the consuming image had the least amount of steps possible so that the package management is done in the builder.
This was a poor choice as an example of reusing layers, I should have done udev rules because that's just copying text files around lol.


Update Set-CsOnlineVoiceApplicationInstance.md

https://github.com/MicrosoftDocs/office-docs-powershell/issues/3414

Docs Build status updates of commit ecc6927:
:clock10: Full build: the average full build time is 14 min(s) 10 sec(s), this is based on the last 339 build(s) for this repository.

Docs Build status updates of commit ecc6927:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/Set-CsOnlineVoiceApplicationInstance.md
:white_check_mark:Succeeded
View (skype-ps)

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


Chart: Visually represent % of program period completed to date

We want to represent the percentage of program period completed to date. By "Program period", we mean the date rate from the reporting start date to the reporting end date. This graphic appears on both the home page and program pages.
Early visual exploration
People struggled to interpret my original donut chart design that looked like this:

They were puzzled by the use of the term "reporting period" -- wondering if it had something to do with donor reporting timelines, or if it was the same thing as a target period. And the circular graphic didn't seem to evoke a clock as I hoped it would.
I propose we ditch the chart in favor of an icon that evokes time, and use plain text to communicate the percentage, something like this:

The language -- Program period is XX% complete -- removes ambiguity, and the icon strongly evokes time. The icon is also way less effort than generating a donut chart.
Placement of the graphic
Home page
Every "program blob" displays the "program period completed" graphic near the program name.

Program page
Again, we position the graphic in line with the program name at the top of the page.

Link to Reporting period modal > Rename "Program period"
In both places, the graphic should link to the reporting period modal.
While we're at it, I propose we rename this page "Program period" -- to be consistent with the link wording and to remove the term "reporting period" that is confusing people.
Housekeeping

[ ] Unit test/s written
[ ] i18n strings included
[ ] Information contained in the graphic is accessible to people using screen readers

This is complete and wired up in branch 561 - will update to staging when that branch completes testing/merging

Validated in both country and program homepages, completed programs only show 100%, programs in progress are reflected correctly for programs checked in Guatemala, Jordan, and Iraq.


Fela js backend for the library

🚀 Feature Proposal
Is it possible to use a Fela js backend for this library?
Motivation
If Fela js can be used for this library, it would reduce the amount of CSS used because it is an Atomic CSS library
Example
Most of the code would not change, people would use the library as regularly
Pitch
To be honest, I don't know if it would be a good idea because the Fela js uses atomic which comes with drawbacks, the drawback is the CSS selector must be a class selector which means utilities like space between (spaceX, spaceY) and divide would not work as expected.
If you think it is a bad idea, then I would try to write my own utility on top of Fela js and take inspiration from this library.

Hi @joseDaKing, Fela is not popular enough to be integrated as a backend in xstyled.
But @xstyled/system is very low level, I think it could be used with Fela without any problem.


cheqd artifact does not work

The cheqd binary seems to be have been built incorrectly.
Inside the cheqd:v1.4.4 docker
~ $ which cheqd-noded
/bin/cheqd-noded
~ $ /bin/cheqd-noded
sh: /bin/cheqd-noded: not found

I looked at a similar issue but the Makefile already consumes env LDFLAGS
Help would be appreciated! thanks!

The makefile is consuming LD_FLAGS, but not LDFLAGS, which I'm guessing is the root cause here.

Thanks @agouin!


Registered Microsoft.QnAMakerDialog

Fixes #2234
Description

Registered Microsoft.QnAMakerDialog in component registration and load it in declarative way.
Specific Changes

Registered Microsoft.QnAMakerDialog and make it configurable
Moved TemplateInterface to botbuilder-dialogs

Testing

We already have tests for QnAMakerDialog

Pull Request Test Coverage Report for Build 130693

36 of 41 (87.8%)  changed or added relevant lines in 8 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.05%) to 80.019%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

libraries/botbuilder-ai/src/qnamaker-utils/bindToActivity.ts
2
3
66.67%

libraries/botbuilder-ai/src/qnaMakerDialog.ts
26
30
86.67%

Files with Coverage Reduction
New Missed Lines
%

libraries/botbuilder-ai/src/qnaMakerDialog.ts
1
32.06%

Totals

Change from base Build 129555:
-0.05%

Covered Lines:
12669

Relevant Lines:
15100

💛  - Coveralls

@luhan2017 and @vishwacsena FYI.
TemplateInterface should have its name changed to just Template as well.

@luhan2017 and @vishwacsena FYI.
TemplateInterface should have its name changed to just Template as well.

We already have a class named Template in botbuilder-lg, is there any suggestions for the name? Thanks.

Needs more discussion, there are some parity issues to clean up here and we can't take this change to botbuilder-ai/package.json until we're sure that adaptive-expressions is browser compatible.

As far as I know, adaptive-expressions has been used in Composer so it should be browser compatible, but we still need to prove that.

I'll hold this PR and extract the declarative refactoring part out of this PR and merge with some changes what I want to make to fix #2314 .


[All] Update to beta07

Updated the PR to include all of the changes, rather than bouncing into another PR.


XML mapper

coverage: 58.266% (+0.1%) from 58.12%
when pulling 725126b62054765c8305f89cf296da612cccf8c3 on dev/xml-mapper
into b64cc65fb86879f750f8a8a20dbb0a9126d37f38 on main.


use matplotlib sliders and widgets if not in a notebook

Maybe do this if not using the ipympl/notebook backend. Although this would prevent using %matplotlib qt, but does anyone actually do that?

See https://mpl-interactions.readthedocs.io/en/latest/Installation.html for the same example with both ipywidgets and matplotlib sliders

maybe have a prefer_ipywidgets argument that defaults to true.
Should also rename the jupyter subpackage to something else. maybe pyplot? that seems a bit silly, but is also pretty representatvie

and rename general to something like niche? or maybe something more inviting but that captures that these are significant additions on top of stock matplotlib whereas the sliders are basically just automating the process of calling plot over and over

the argument approach would allow the wonky case of wants to use qt in the notebook but also wants ipywidgets sliders

Layout is properly annoying to figure out. It seems like shrinking the main plot vertically is not a great option. One possibility may be to make use of https://matplotlib.org/tutorials/intermediate/constrainedlayout_guide.html but as a temporary measure I think I will just put the sliders and radio buttons to the right of the plot.


Pane/Tab context menu

The Chore
We need a method of allowing users to execute actions from within a pane, where those actions are specific to the content of the pane itself.
Some examples of these 'actions' could be:

Changing the view to display a settings area for that data grid or chart
Executing transactions such as cancelling all orders or depositing
Changing chart overlays and studies
Changing deal ticket types
Changing number precisions

Tasks

[ ] Design
[ ] Review
[ ] Dev (ui-toolkit)

Additional details / background info

Open to any ideas here but my initial thoughts here are that this is one or more contextual (the context being the current selected tab) menus (possibly similar to our Dropdown component) from the top right of each tab bar. Alternatively this menu could exist within the tab itself.
Some examples gathered on Miro

Started looking at this as part of Console v3. See Cancel all, Settings icon + Expand Icon in the Positions tab (is incorrectly functionality but just to show what a button + icons could look like.


Only perform regression tests when AWS secrets are available

Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)

Checklist:

[x] Pull request title reflects its content
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings
[x] New and existing unit tests pass locally with my changes

Description:
Regression tests should be skipped for this PR, as I am trying to merge from a fork, which doesn't have the secrets available. I also intend to test if the tests are run when the secrets are available.

Appears to work as intended. See:

when running from fork (skips)
when running from openDAQ/openDAQ (runs)


When is a connection closed or can we explicitly close a connection?

I haven't seem an example for closing a connection.
There are some logic related creating connection pool and closing connection pool shown as below.
new sql.ConnectionPool(config).connect().then(pool => {...
pool.close();
So when is a connection released?
Or can we explicitly close a connection (if necessary)?

It depends on your configuration. Connections are created automatically when you execute a request. Maximum number of concurrent connections is configurable via pool.max option. The pool. idleTimeoutMillis option tells the pool how long should the connection stay idle before it is removed from the pool. You can also change the pool.min option that specifies the minimum number of connections there can be in the pool.
So, the connections are created/released automatically based on those three options. There is no way to close the connection explicitly. Once you call pool.close(), all active connections in the pool will be closed.

@patriksimek
Thanks for explanation.
Yes, I agree, basically the pool settings are enough.


Make SwiftUI previews work without preview mode

The current KeyboardKit preview mode is a way to make KeyboardKit previews work in other bundles, since SwiftUI previews don't support fetching assets from other packages by default.
There is however another way to solve this, that would let us remove the preview mode altogether and that would make external previews use the correct assets instead of replacements.

This will be available in 6.1.


Broken link in read_midc documentation

The link to Variable Style Rules (https://pvlib-python.readthedocs.io/en/latest/variables_style_rules.html) appears to be broken:
https://github.com/pvlib/pvlib-python/blob/b3c1b5e9fba606c3a89c052d07c9e500518bba5d/pvlib/iotools/midc.py#L155-L207

Thanks @williamhobbs, I didn't think to check for existing links when I moved those pages under user_guide in #1173.  Looking at the results of $ git grep "pvlib-python\..*/en/", there are several links to update:

README.md:Please see the [Installation page](http://pvlib-python.readthedocs.io/en/stable/installation.html) of the documentation for complete instructions.
docs/tutorials/irradiance.ipynb:    "http://pvlib-python.readthedocs.io/en/latest/clearsky.html\n",
pvlib/iotools/midc.py:    <https://pvlib-python.readthedocs.io/en/latest/variables_style_rules.html>`_.
pvlib/modelchain.py:    See https://pvlib-python.readthedocs.io/en/stable/modelchain.html

As an aside, is it desirable for any link like this to point to latest?  Maybe we should standardize to stable everywhere.

we should standardize to stable everywhere.

+1


open_manipulator_pro_teleop error!

I experimented with gazebo and real hardware.
However, a manipulator malfunction occurred.
The control gui, keyboard control, and joystick control all have errors when the x value drops below (ex. 0.25).
My guess seems to be an error in the inverse kinematics of the open manipulator msgs.
I want a solution for this.
Thank you, Please reply.
https://youtu.be/ytdMD7-c3MI

@YDK18
Thank you for reporting the bug. It should work now. Test it again and let us know the result
Regards,
Ryan

@YDK18
Thank you for reporting the bug. It should work now. Test it again and let us know the result
Regards,
Ryan

Hello  I am the same team with @YDK18 . I just tested the latest code that uploaded 1 hour ago. However, there is still something wrong. Please refers to below link.
https://www.youtube.com/watch?v=p-SLIsoBsak&feature=youtu.be
With the modified code, it seems like manipulator does not hit the ground suddenly at x position at 0.240.
Actually, I wonder why the manipulator moves like that ?
But there is no feedback or error message. If arm can't reach the goal position because of the physical limitation, then I think maybe there should be a error report and stop movement.
Looking forward to your reply.
Thank you.

@rjshim
Hello  I am the same team with @YDK18 . I just tested the latest code that uploaded 1 hour ago. However, there is still something wrong. Please refers to below link.
https://www.youtube.com/watch?v=p-SLIsoBsak&feature=youtu.be
With the modified code, it seems like manipulator does not hit the ground suddenly at x position at 0.240.
Actually, I wonder why the manipulator moves like that ?
But there is no feedback or error message. If arm can't reach the goal position because of the physical limitation, then I think maybe there should be a error report and stop movement.
Looking forward to your reply.
Thank you.

@Fred159
Please download the latest version and test it again.

@Fred159
Please download the latest version and test it again.

@rjshim
Hello. I was trying to test the lastest version.
The y and z axes ran correctly but axis-x malfunction occured again.
As shown the picture, a strange posture came out.

@rjshim
I download the latest version in master at https://github.com/ROBOTIS-GIT/open_manipulator_pro.git.
We use ubuntu 16.04. LTS. ROS: kinetic
Do you mean the version with kinetic-devel branch? But I also test the kinetic version, it doesn't work either.
Arm works well in y direction and z direction. When arm reached the limitation of the physical size, terminator gave error messages and stop. However the arm still works strange in x direction. Arm moved with a not understandable behavior.

@YDK18
Sorry for the late reply. I just fixed fk and ik bugs and updated the package. Please test it and let us know the result.
Regards,
Ryan

As there have been no updates, I will close this issue.
Ryan


Priority price estimator

Implements a priority price estimator that tries several inner estimators. I made this smarter than a naive implementation by only retrying unsuccessful queries and continuing to request bulk estimates.
Test Plan
new unit test and manual test

This fixes #1139 right?

yes


Cannot install cli 3.1.0

I'm new to ocaml and reason. I have tried numerous times over the past two days to install the cli, but to no avail. I have tried the installation instructions on VM's with Linux Mint 17 and 18 and finally with Ubuntu 16.04.4. In every case, the error is the same: https://gist.github.com/bsommardahl/97664bd601f8b344174522647ca4130d
I'm hoping someone can see something that I'm missing. Here's some more information about my system config...
glibc
I noticed the following note in the readme.md: The installation requires glibc >=3.4.21. If you're on macOS, you should be good to go. If you're on Ubuntu, reason-cli requires Ubuntu 16.04. Otherwise the postinstall might fail.
After a fresh install of Ubuntu 16.04.4, the version of glibc seems to be 2.23-0ubuntu10 according to this:
Package: libc6
Architecture: amd64
Version: 2.23-0ubuntu10
Multi-Arch: same
Priority: required
Section: libs
Source: glibc
Origin: Ubuntu
Maintainer: Ubuntu Developers<EMAIL_ADDRESS>Original-Maintainer: GNU Libc Maintainers<EMAIL_ADDRESS>Bugs: https://bugs.launchpad.net/ubuntu/+filebug
Installed-Size: 10953
Depends: libgcc1
Suggests: glibc-doc, debconf | debconf-2.0, locales
Breaks: hurd (<< 1:0.5.git20140203-1), libtirpc1 (<< 0.2.3), locales (<< 2.23), locales-all (<< 2.23), lsb-core (<= 3.2-27), nscd (<< 2.23)
Replaces: libc6-amd64
Filename: pool/main/g/glibc/libc6_2.23-0ubuntu10_amd64.deb
Size: 2580230
MD5sum: fa06d9e551121f74f6ac5dbbab5f4043
SHA1: a7a25a7cca8aa8420b82b2e8a2d46dcee93f1aca
SHA256: bd05c3487325a4386dee6abb02ad904e1f2d8d3d0adc0df8e8f29168fbe2b5bb
Homepage: http://www.gnu.org/software/libc/libc.html
Description-en: GNU C Library: Shared libraries
 Contains the standard libraries that are used by nearly all programs on
 the system. This package includes shared versions of the standard C library
 and the standard math library, as well as many others.
Description-md5: fc3001b0b90a1c8e6690b283a619d57f
Task: minimal
Supported: 5y

Package: libc6
Priority: required
Section: libs
Installed-Size: 10947
Maintainer: Ubuntu Developers<EMAIL_ADDRESS>Original-Maintainer: GNU Libc Maintainers<EMAIL_ADDRESS>Architecture: amd64
Source: glibc
Version: 2.23-0ubuntu3
Replaces: libc6-amd64
Depends: libgcc1
Suggests: glibc-doc, debconf | debconf-2.0, locales
Breaks: hurd (<< 1:0.5.git20140203-1), libtirpc1 (<< 0.2.3), locales (<< 2.23), locales-all (<< 2.23), lsb-core (<= 3.2-27), nscd (<< 2.23)
Filename: pool/main/g/glibc/libc6_2.23-0ubuntu3_amd64.deb
Size: 2583544
MD5sum: eeb2f237384d67accc1998a7c791ead7
SHA1: 2a05d2a1466c7ce169deb4d72096e37874dff70b
SHA256: 68d69a0fb2dae7578a2a7054bc0cd5f0994d523f60d540bc6708ff3f2fb33e18
Description-en: GNU C Library: Shared libraries
 Contains the standard libraries that are used by nearly all programs on
 the system. This package includes shared versions of the standard C library
 and the standard math library, as well as many others.
Description-md5: fc3001b0b90a1c8e6690b283a619d57f
Multi-Arch: same
Homepage: http://www.gnu.org/software/libc/libc.html
Bugs: https://bugs.launchpad.net/ubuntu/+filebug
Origin: Ubuntu
Supported: 5y
Task: minimal

node
Node and npm versions are as follows:
byron@ubuntu:~$ node -v
v8.10.0
byron@ubuntu:~$ npm -v
5.6.0

I found a work-around by @cirpo at https://github.com/reasonml/reason-cli/issues/43 and managed to get it installed after all.  The reason website offered help right away with this issue: https://github.com/reasonml/reasonml.github.io/pull/157
Lesson learned. Read the documentation.


ValueError: text input must of type str (single example), List[str] (batch or single pretokenized example) or List[List[str]] (batch of pretokenized examples).

Hello,
I was trying to train the mt5 model on the paws-x dataset but it gives me the following error (although my dataframe looks exactly like the dataframe in the example colab notebook):
ValueError: text input must of type str (single example), List[str] (batch or single pretokenized example) or List[List[str]] (batch of pretokenized examples).
My dataset looks like this:

source_text
target_text

paraphrase: Im Oktober 1560 traf er sich heiml...
Im Oktober 1560 traf er sich in Paris heimlich...

paraphrase: Die Saison 1975 - 76 der National ...
 Saison de.

What is wrong?

Would you mind sharing a colab notebook. i will take a look.

It has problems with other chars, e.g. "ó". "ó".encode('utf-8') also doesn't work.

T5 models are text to text models & they expect source_text and target_text to be a string. I would suggest to make sure your dataframe has values in string format.
You can convert your dataframe in the expected format:
df = df.applymap(str)

ByT5 crashes with nonlatin chars like "ąśćłó".

Did you put the raw data into torch.data.Dataset and then pass it to the model.call() function directly??  If you did this in low version transformers like 4.9.0, you may get this error. But huggingface had already fixed this.

I think the issue can be closed. Shivanandroy's answer above works.

T5 models are text to text models & they expect source_text and target_text to be a string. I would suggest to make sure your dataframe has values in string format.
You can convert your dataframe in the expected format:
df = df.applymap(str)


VIH-6594 update pipeline endpoints

JIRA link (if applicable)
VIH-6594
Change description
update pipeline endpoints
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[x] No

/azp run


Supporting POST requests and request headers

This PR adds support for making POST requests and setting request headers. The added changes include:

The setRequestMethod (part of the HttpUrlConnection class) to specify the request method (currently only POST and GET supported).
The setDoOutput, getDoOutput and getOutputStream methods (part of the UrlConnection class), and the OutputStream and OutputStreamWriter classes to write the body of the POST request.
The SetRequestProperty method to add request headers.

Title could be updated :-)

@bghgary ready to merge.


python312Packages.oci: 2.122.0 -> 2.124.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python312Packages.oci is: Oracle Cloud Infrastructure Python SDK
meta.homepage for python312Packages.oci is: https://github.com/oracle/oci-python-sdk
meta.changelog for python312Packages.oci is: https://github.com/oracle/oci-python-sdk/blob/v2.121.0/CHANGELOG.rst
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.121.0 with grep in /nix/store/ckb55j30w14bix7k3cwz26fqqxwgwikd-python3.12-oci-2.121.0
found 2.121.0 in filename of file in /nix/store/ckb55j30w14bix7k3cwz26fqqxwgwikd-python3.12-oci-2.121.0

Rebuild report (if merged into master) (click to expand)

8 total rebuild path(s)

8 package rebuild(s)

First fifty rebuilds by attrpath
certbot-full
lexicon
oci-cli
python311Packages.certbot-dns-ovh
python311Packages.dns-lexicon
python311Packages.oci
python311Packages.ocifs
python312Packages.oci

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/ckb55j30w14bix7k3cwz26fqqxwgwikd-python3.12-oci-2.121.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python312Packages.oci https://github.com/r-ryantm/nixpkgs/archive/b2e975722ad1f00a95a07e4598d3a17c8b05a8fc.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/b2e975722ad1f00a95a07e4598d3a17c8b05a8fc#python312Packages.oci

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/ckb55j30w14bix7k3cwz26fqqxwgwikd-python3.12-oci-2.121.0
ls -la /nix/store/ckb55j30w14bix7k3cwz26fqqxwgwikd-python3.12-oci-2.121.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package marked as broken and skipped:

python312Packages.oci

2 packages failed to build:

certbot-full
python311Packages.certbot-dns-ovh

4 packages built:

lexicon (python311Packages.dns-lexicon)
oci-cli
python311Packages.oci
python311Packages.ocifs

Maintainer pings
cc @ilian for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python312Packages.oci is: Oracle Cloud Infrastructure Python SDK
meta.homepage for python312Packages.oci is: https://github.com/oracle/oci-python-sdk
meta.changelog for python312Packages.oci is: https://github.com/oracle/oci-python-sdk/blob/v2.124.0/CHANGELOG.rst
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.124.0 with grep in /nix/store/s85v80cl43w5xaq9srmibdbkn4vv2r1k-python3.12-oci-2.124.0
found 2.124.0 in filename of file in /nix/store/s85v80cl43w5xaq9srmibdbkn4vv2r1k-python3.12-oci-2.124.0

Rebuild report (if merged into master) (click to expand)

9 total rebuild path(s)

9 package rebuild(s)

First fifty rebuilds by attrpath
certbot-full
lexicon
oci-cli
python311Packages.certbot-dns-ovh
python311Packages.dns-lexicon
python311Packages.oci
python311Packages.ocifs
python312Packages.oci
python312Packages.ocifs

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/s85v80cl43w5xaq9srmibdbkn4vv2r1k-python3.12-oci-2.124.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python312Packages.oci https://github.com/r-ryantm/nixpkgs/archive/603c12bff2edfb2b3d107361e95eddf75fcbfd16.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/603c12bff2edfb2b3d107361e95eddf75fcbfd16#python312Packages.oci

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/s85v80cl43w5xaq9srmibdbkn4vv2r1k-python3.12-oci-2.124.0
ls -la /nix/store/s85v80cl43w5xaq9srmibdbkn4vv2r1k-python3.12-oci-2.124.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

15 packages built:

certbot-full
lexicon (python311Packages.dns-lexicon)
lexicon.dist (python311Packages.dns-lexicon.dist)
oci-cli
oci-cli.dist
python311Packages.certbot-dns-ovh
python311Packages.certbot-dns-ovh.dist
python311Packages.oci
python311Packages.oci.dist
python311Packages.ocifs
python311Packages.ocifs.dist
python312Packages.oci
python312Packages.oci.dist
python312Packages.ocifs
python312Packages.ocifs.dist

Maintainer pings
cc @ilian for testing.


Scala Common Enrich: change maxColumns to 140 in .scalafmt.conf

The codebase is barely readable when everything is squashed vertically

Hi benjben,
It's a real  "good first commit" one!
Can I work on this?

@benjben I've sent pull request for this. Feel free to comment or ask for any changes.

Closed by 696fcd35d62d4d82f84c24b8ab087a5588b63fea


SweetAlertOptions.progressSteps inconsistent type definition

The type definition of the progressSteps field inside of the SweetAlertOptions class specifies an array of SweetAlertOptions, however the example shows it being used as an array of strings.
The setProgressSteps method also takes an array of strings, therefore I think the progressSteps field inside of the SweetAlertOptions class should be updated to expect an array of strings.

The example you linked uses the array of SweetAlertOptions:
var steps = [
  {
    title: 'Question 1',
    text: 'Chaining swal2 modals is easy'
  },
  'Question 2',
  'Question 3'
]

Array of strings is the edge case.

No it doesn't?
The example sets:
swal.setDefaults({
  input: 'text',
  confirmButtonText: 'Next →',
  showCancelButton: true,
  animation: false,
  progressSteps: ['1', '2', '3']
})

Note: progressSteps: ['1', '2', '3']

Oops, my bad sorry.

If I pass the array of SweetAlertOptions to the progressSteps field when calling swal.setDefaults (example below), then this happens (note the step numbers are replaced with [object Object]).
        let steps: SweetAlertOptions[] = [
            {
                title: 'Question 1',
                text: 'Chaining swal2 modals is easy'
            },
            {
                title: 'Question 2',
                text: 'q2'
            },
            {
                title: 'Question 3',
                text: 'q3'
            }
        ];

        swal.setDefaults({
            input: 'text',
            confirmButtonText: 'Next →',
            showCancelButton: true,
            animation: false,
            progressSteps: steps
        });

Yeah, it should definitely be array of strings, could you please submit a PR with the fix?

Thanks a lot @michaelbull for reporting and fixing this! :beers:

Thanks again!


Remote server black screen

https://mimic.js.org/guides/remote.html
Tried following the instructions but I just get a white screen  with no errors on localhost:5000 (chrome, incognito doesn't help)

If you have a white screen it's probably loaded. However because mimic is by default closed it's very hard to see currently.
Look at the bottom left corner of the page. I'll work on making it expanded by default when running in remote mode.

Your right, it's ok. missed that :)
Great feature. Thanks!


Windows webusb instruction

https://www.npmjs.com/package/@ledgerhq/hw-transport-webusb

"How to Install" is a null link for intercom launcher trigger, which will be set up in Intercom later.

@guyuxiang @williamleha status of  design?

Remove "I DID IT"


Feat: dark & light mode functionality + Fix rtl mode

@MaTeMaTuK Hello, how are you? Could you approve this request?

Hi guys ! how are you ? I have problems with your library.  Dark-Light mode and Tooltip position are not working! also rtl mode. When you can fix this problems?

Hi guys ! how are you ? I have problems with your library. Dark-Light mode and Tooltip position are not working! also rtl mode. When you can fix this problems?

Hi, i think this project is no longer being maintained. you can get a clone from my fork:
https://github.com/fmoosavi/gantt-task-react
and if there was any problem, ill be glad to help


Update Spyder to handle newer python-lsp-server (v1.4.0)

Problem Description
Hi!  https://github.com/python-lsp/python-lsp-server has been upgraded to 1.4.0, but Spyder itself says it can only handle versions less than 1.4.0.  I'm presuming this is on some checklist for resolving at some point, but I couldn't see it.

Hi @juliangilbey ! I think we will be bumping up the constraint when we release Spyder 5.3.0 (hopefuly next week). Maybe @ccordoba12 has more info about that

So exciting! Thanks! (Meanwhile, I'm trying to build pydevd and debugpy for Debian, which is taking a lot of effort!)

This will come in 5.3.0, as @dalthviz said.
@juliangilbey, please don't push for things like this next time. We maintain python-lsp-server, qtconsole, python-lsp-black and several projects. Sometimes they need to be released before the next Spyder version and the only way to bump their constraints here is to wait for the next release.

OK @ccordoba12, I'll wait patiently next time! Thanks for your ongoing amazing work!

Thanks to you too for understanding!

Just in case, maybe for future releases it could be useful to have an issue to describe the plan that we have regarding a release in terms of for example dependencies version support (as the question here), difficulties being faced, in general things that are implicit to the release but that don't have an issues for the corresponding GitHub milestone. With that, I think, we will let users and contributors know in a more clear way the state of release and we will give them more info about what is going on in terms of the release planning

Sure, I think that's a good idea and I was thinking about that too, so that our release manager (which has been you @dalthviz for the last releases) also knows what package versions need to bumped before the release.
I'll open a new issue about it for this release.


Configurations not deleted on uninstall

Configurations not deleted on uninstall on W10.

Thanks for the info. This is expected behaviour from the MSI install, and as the files are only a few KB in size, I don't think it's really too much of an issue.

But I uninstalled and installed hoping to have the original config, and it reloaded the last config. There should at least be a way to know where it is (wasn't in the folder where RWC was installed).


Fixed typo

OPS Build status updates of commit c25d67f:
:clock10: Preparing: average preparing time is 44 sec(s)

OPS Build status updates of commit c25d67f:
:clock10: Incremental building: average incremental building time is 29 sec(s)

OPS Build status updates of commit c25d67f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/quickstarts/excel-quickstart-angular.md
:white_check_mark:Succeeded

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


MinAlertLevel = suggestion causes flycheck-vale to break

This package doesn't work with this in my .vale.ini:
MinAlertLevel = suggestion # suggestion, warning or error

error in process sentinel: flycheck-add-overlay: Undefined error level: nil
error in process sentinel: Undefined error level: nil

It works again once I set the minimum level to warning.

I'm not actively maintaining this any more (I've switched away from Emacs), but I'd be happy to help you (or anyone) take over if you're so inclined.

I'm not actively maintaining this any more (I've switched away from Emacs), but I'd be happy to help you (or anyone) take over if you're so inclined.

#14 works for me nice! :+1:

I'm not actively maintaining this any more (I've switched away from Emacs)

Thank you for inviting me as a collaborator. I suspect that we lost you to VSCode. But I hope/suspect you return to Emacs sooner than later :smile:


Use default build script for reportsDashboards and remove customized build script

Signed-off-by: Rupal Mahajan<EMAIL_ADDRESS>Description
chromium was removed from dashboards-reports as part of https://github.com/opensearch-project/reporting/pull/586.
chromium binary is not required in artifact anymore.
Issues Resolved
List any issues this PR will resolve, e.g. Closes [...].
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Hi @rupal-bq What do you think about moving this build.sh in the respective repository? Its will be easier to manage too.

Hi @rupal-bq What do you think about moving this build.sh in the respective repository? Its will be easier to manage too. cc: @peterzhuamazon @dblock

Seems good idea to me if many plugins have special use cases to handle. But I am not sure what is required to do so.

Just move the script under scripts folder. Example https://github.com/opensearch-project/alerting/tree/main/scripts

Codecov Report

Merging #3055 (0cb8109) into main (af1be01) will decrease coverage by 0.30%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #3055      +/-   ##
==========================================
- Coverage   93.17%   92.87%   -0.31%
==========================================
  Files         167      167
  Lines        4602     4602
==========================================
- Hits         4288     4274      -14
- Misses        314      328      +14

Impacted Files
Coverage Δ

src/system/temporary_directory.py
64.44% <0.00%> (-28.89%)
:arrow_down:

src/system/os.py
94.73% <0.00%> (-5.27%)
:arrow_down:

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

Just move the script under scripts folder. Example https://github.com/opensearch-project/alerting/tree/main/scripts and delete from here

Adding script to dashboards-reporting https://github.com/opensearch-project/dashboards-reporting/pull/21. deleted from here.

@zelinh You okay with this change?

@rupal-bq  @zelinh  Does removing the script will automatically trigger default build script. We want to remove for observability Dashboards as well?

@rupal-bq @zelinh Does removing the script will automatically trigger default build script. We want to remove for observability Dashboards as well?

@vamsi-amazon Our current workflow actually have a list for place to look for the build script. https://github.com/opensearch-project/opensearch-build/blob/af1be01fb3a1275214b4a964d1737b46de69a9da/src/paths/script_finder.py#L61-L76
If it doesn't find the build script from all of those location above, the last choice is to use the default build script.

For dashboards-observability, I don't see a build script at https://github.com/opensearch-project/dashboards-observability.
I will update this PR to remove https://github.com/opensearch-project/opensearch-build/blob/main/scripts/components/observabilityDashboards/build.sh as well so default can be used


fix: move react and react-dom under peerDependencies

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)

This PR moves react and react-dom from the dependencies list to peerDependencies. This change is to be explicit in the framework we support.

Needs rebase


Translate key.zoom

It would be nice to look into potentially hooking into the translator and throwing in our own translations so that key.zoom can be renamed to Zoom.

I’m probably going to do this through means of a texture pack of some kind if amethyst ever supports loading them at runtime.


[BUG] Error with prod build and MUI

What happened?
I am encountering an error in PRODUCTION while utilizing MUI Theme Provider. The error is not present when I comment out Theme Provider, but appears when I use MUI Theme Provider. It is important to note that this error does not occur in DEVELOPMENT environment.

Version
Latest
What OS are you seeing the problem on?
MacOSX
What browsers are you seeing the problem on?
Chrome
Relevant log output
ntent.1d473d26.js:32 Error: Minified React error #130; visit https://reactjs.org/docs/error-decoder.html?invariant=130&args[]=undefined&args[]= for the full message or use the non-minified dev environment for full errors and additional helpful warnings.
    at us (content.1d473d26.js:32:49735)
    at jt (content.1d473d26.js:30:18086)
    at Qt (content.1d473d26.js:32:1475)
    at Rp (content.1d473d26.js:32:44752)
    at Pp (content.1d473d26.js:32:40057)
    at jm (content.1d473d26.js:32:39985)
    at cs (content.1d473d26.js:32:39848)
    at Wc (content.1d473d26.js:32:36162)
    at wp (content.1d473d26.js:32:35113)
    at _e (content.1d473d26.js:40:1686)
C

(OPTIONAL) Contact Details
<EMAIL_ADDRESS>Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate problems.

@RehanShakir is this replicable with the MUI example?

The error is on production. And the solution is that we have to import the MUI Theme Provider and useTheme from "@mui/material/styles"  instead of "@mui/material
What I am doing previously that is
import { ThemeProvider } from "@mui/material"
The above import is working fine on dev but its casuing issues on production.
After importing the Theme Provider from ("@mui/material/styles") the error got fixed.


New version: NLopt v1.0.2

Registering package: NLopt
Repository: https://github.com/JuliaOpt/NLopt.jl
Created by: @stevengj
Version: v1.0.2
Commit: 4b363d93d5b6eb47654a22e79e24539e071cb3fb
Reviewed by: @stevengj
Reference: https://github.com/JuliaOpt/NLopt.jl/commit/4b363d93d5b6eb47654a22e79e24539e071cb3fb#commitcomment-136653645
Description: Package to call the NLopt nonlinear-optimization library from the Julia language

Closed by #98655.


:information_source:  #161 About page

Description
Updated code per request and made it mobile,tablet and desktop responsive.
Fixes # (issue) - Example: Fixes #78 , #76
Type of change
Please delete options that are not relevant.

[x] Enhanced clear response depending on screen size

Screenshot (optional)

##mobile responsive

##tablet res

#updated laptop layout

#updated tablets-page

#updated phones page

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

check the pr title


Example won't compile on windows after latest updates

What OS are you using (uname -a, or Windows version)?
Windows 11 21H2 Build 22000.739

What programming language are you using (C/C++/Go/Rust)?
go version go1.18.3 windows/amd64
gcc (tdm64-1) 10.3.0
vscode 1.68.1

What did you expect to see and what you saw instead?
I expected the example basic.go to pop up a windows.
Instead I have compile errors that say:
cannot use webview.HintNone (constant unknown with invalid type) as webview.Hint value in argument to w.SetSize
could not import C (cgo preprocessing failed) (compile)
I cannot figure out if this is a Go bug, vscode bug, or gcc. But I swear it all used to work fine until the latest updates.
package main

import "github.com/webview/webview"

func main() {
	w := webview.New(false)
	defer w.Destroy()
	w.SetTitle("Basic Example")
	w.SetSize(480, 320, webview.HintNone)
	w.SetHtml("Thanks for using webview!")
	w.Run()
}

If I compile on the command line I get...
C:\Users\username\Development\Golang\example>go build
# github.com/webview/webview
In file included from webview.cc:1:
webview.h:839:10: fatal error: WebView2.h: No such file or directory
  839 | #include "WebView2.h"
      |          ^~~~~~~~~~~~
compilation terminated.

I'm pretty sure it has more to do with Go and cgo. But I'm hoping someone here would have figured it out first.

Since you're trying the Go example, it looks like you might simply have skipped a few steps such as not downloading WebView2 or maybe your haven't set the cgo-specific environment variables according to the readme so that it can find WebView2. Also check the MinGW-w64 requirements in the readme if you have issues with TDM-GCC.
If you've made sure of these things and still have an issue then please share the exact steps to reproduce the issue.

Same error.
I reproduce all steps from readme, use WinLibs recomended in readme, WebView2 downloaded and extracted to right place, environment variables set as readme say.

I'm unable to reproduce this issue as described on Windows 11 21H2.
I installed MinGW-w64 from WinLibs and latest version of Go.
Are you by any chance running the commands using PowerShell? If that's the case then please try the good old Command Prompt.

I've copied the exact commands from the readme, pasted them into a new batch script named build.bat, ran the script and the Go example program compiled and ran just fine on my end.
Please see if you missed anything here and if so then I would love to know how you came to miss those steps and how the readme can be improved so that those steps are not missed again by anyone else.
mkdir my-project && cd my-project
mkdir build libs "libs/webview"
curl -sSLo "libs/webview/webview.h" "https://raw.githubusercontent.com/webview/webview/master/webview.h"
curl -sSLo "libs/webview/webview.cc" "https://raw.githubusercontent.com/webview/webview/master/webview.cc"

mkdir libs\webview2
curl -sSL "https://www.nuget.org/api/v2/package/Microsoft.Web.WebView2" | tar -xf - -C libs\webview2
copy /Y libs\webview2\build\native\x64\WebView2Loader.dll build

go mod init example.com/m
go get github.com/webview/webview

set CGO_CPPFLAGS="-I%cd%\libs\webview2\build\native\include"
set CGO_LDFLAGS="-L%cd%\libs\webview2\build\native\x64"

curl -sSLo basic.go "https://raw.githubusercontent.com/webview/webview/master/examples/basic.go"

go build -ldflags="-H windowsgui" -o build/basic.exe basic.go && "build/basic.exe"

Are you by any chance running the commands using PowerShell? If that's the case then please try the good old Command Prompt.

Yes, in Command Prompt example compiled fine. Thanks.

TDM-GCC does not seem to have that header so I would like to suggest one of the currently recommended distributions of MinGW-w64.

The reason why you didn't have this issue before is that the library had to be compiled separately from the Go build using Visual C++. which had the header, and therefore Go/cgo did not build the core library and therefore never needed to include that header. The library can now be built entirely with MinGW-w64 and therefore Go/cgo, without the intermediary step with Visual C++. Because of that your MinGW-w64 distribution needs a more up-to-date Windows SDK.

The newer compiler did it.
Thanks!
C:\Users\peter\Development\Golang\my-project>gcc --version
gcc (MinGW-W64 x86_64-ucrt-posix-seh, built by Brecht Sanders) 12.1.0
Copyright (C) 2022 Free Software Foundation, Inc.

I hope you don't mind one more Q...
Updating the compiler and using the new env variables solved it for the command line. But I'm still having trouble getting VScode to accept it. I never needed the env variables before so never needed to solve this.
My interpretation of the CGO docs says the following should work. But I still get the errora:
cannot use webview.HintNone (constant unknown with invalid type) as webview.Hint value in argument to w.SetSize
could not import C (cgo preprocessing failed) (compile)
I tried a bunch of variations but still no luck. Would you mind giving me a hint?
Thanks

Did you try to remove the CGO_ prefix? I'll give vscode a try in a few hours.

@precisionpete I got the same issue and I decided to use Command Prompt instead of PowerShell. It seems PowerShell does not take the effect of these commands
set CGO_CPPFLAGS="-I%cd%\libs\webview2\build\native\include"
set CGO_LDFLAGS="-L%cd%\libs\webview2\build\native\x64"

then go env still shows:
set CGO_CFLAGS=-g -O2
set CGO_CPPFLAGS=
set CGO_CXXFLAGS=-g -O2
set CGO_FFLAGS=-g -O2
set CGO_LDFLAGS=-g -O2
set PKG_CONFIG=pkg-config

Change terminal in VSCode to old Command Prompt and it will work.


Support gentle terminate for the target process

In addition to the option to simply kill the debug process, please consider using a kill signal for processes that are not paused.
This will allow them to execute cleanup code and dispose things properly. With any luck this will also resolve the mystery of accumulating node processes on Windows for dotnet new react projects.
You can always force terminate after a timeout for things that don't honour the signal.

To make sure I understand your suggestion with regard to react projects - does the target process need to be gentle termination for this? Or is it just a matter of terminating the child processes?

To be honest I don't know. What I have observed is that

When I repeatedly kill a react project the debug process using the "stop" button in the compile-debug-edit cycle, I get a lot of orphan node processes. When there's enough to make my CPU fan start I clean them up with taskkill -im node.exe -f
When I remember to unpause the thread and use Ctrl+C to send a kill signal, this doesn't seem to happen.

@PeterWone wrong the following additional comments that I am moving to this issue:

I wrote the "Gentle termination" issue. While it certainly is closely related, I want to remind whoever deals with this that part of the problem is that forcibly terminated processes don't run their exit handlers, dispose their resources or flush their buffers. The idea underpinning the gentle termination issue was that you don't get these problems when you send the process a kill signal and let it terminate itself.
It's not always that simple, so you might need a timeout falling back to forced termination, and if code were paused in the debugger you'd have to get user consent to resume execution because sometimes that can be a bad idea with persistent consequences.
I guess my proposition is "If you can use a kill signal then do so."
@gregg-miskelly yes thorny issues.
For debugger paused code, you really need to consult the user with a dialog like this.

Debug stop
Code is paused in the debugger.
You could use radio buttons and emphasis like this for a simple way to remember the last choice for the session and also provide the opportunity to change your mind.  Yes I know these are checkboxes not radio buttons but it's markdown.

Debug stop
Code is paused in the debugger.  This started with a discussion on how you can't put up UI to ask users what to do in VS Code.
From @justinmchase
You probably don't need to ask the user anything, I think you have enough information already; at least in VSCode you would. The debugger is either attaching to a running process or launching a process to be debugged.
When attached it offers the button "disconnect" and when that is clicked you would do nothing, the process was launched and is being managed elsewhere.
When "launched" the UI offers you the button "Stop", which when pressed you would hard kill all the processes. When you hit stop you expect all of the processes to immediately stop and not keep running and do more work, the whole process tree should just abruptly end.
I would not try to ask the user what behavior they wanted in this case. You never want to "resume and send a kill signal" for example. If the user wants that they can just hit F5 to resume and then press Ctrl+C in the terminal to send the kill signal. You don't need to be clever here.
When I press Stop, it means stop. If the Stop button does anything other than stopping it will be highly annoying.
From @PeterWone
@justinmchase then leave your setting on the default of legacy behaviour, and not be inconvenienced in the slightest.
VS Code is used by people at all levels of expertise. Many would not understand that there is a huge difference in behaviour between a process responding to a kill signal and a process being terminated. I do know, and even so there have been many occasions when mysterious erratic bugs have been rooted in things not being disposed as a result of force termination. On more than one occasion I have killed the process out of habit and then spent the next two minutes rebooting to fix it.

From @PeterWone:

I have cloned the code-base to look into the issues of implementing this. If you have any start-here pointers they would be welcome. For example, where to find the button handler code and some notes on how it's put together.

This extension doesn't provide the button. The button is owned by VS Code.
It would be possible for you to fully implement this outside of the debug adapter, which is a closed-source project. Also, if you wanted to handle the command working from break mode, your proposal here is really at least a week of fairly challenging work from an expert on the debugger. So even if the debugger was open source I wouldn't say this is "up for grabs".
If you wanted a stop gap, which you might be able to do is make a new VS Code extension that sent the signal to the target process itself. It could do this as a separate VS Code command in the command well, and you could assign it a keyboard shortcut so it is easy to invoke. This wouldn't work if the target process was stopped (in break mode), but it would from run mode. The one hard part would be knowing the process id of the target process. You might be able to do a reasonable job of guessing though by following the process tree and looking for the children of vsdbg/vsdbg-ui processes. There is also a process event that has this info, but I don't think a VS Code extension has access to that. See here for more information on VS Code extensions.

@gregg-miskelly Sorry you're quite right. I was talking about the vscode repo. I think I maybe we ought to close this and I'll start a new thread there, since that's where it will have to be done.

I haven't seen it mentioned yet in this issue, but the vscode team seems to have a good way to tie into this behavior here: microsoft/vscode#54384 so I figured I would bring it up.

That mechanism sends INT, services expect TERM.


Fixed configured detection

Currently the "not configured" screen is shown, even if a proper configuration is provided. This PR fixes this issue and only shows the screen if there is no loadpoint configured

Fix #244


Navigation restructure

Restructure of site navigation into sections - Brand, Marketing, Product, Website - to more easily divide content between these streams. Pages and page content remains as before, but the tooling and formatting of the front matter has been updated. Sections driven by implementing Jekyll Collections. This has made it easier to generate the sidebar nav and to embed a breadcrumb navigation into each page. We've also simplified the page layout and front matter to reduce the number of variables in play.

Ive merged in an update to add in a general Team section and to update some of the page content. This catches up some changes that had been missed in the last merge.


[Bug] int_iterable__list_user_unnest inconsistency between warehouses

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the issue
Depending on the warehouse, different results are produced for model int_iterable__list_user_unnest when using seed data. For example, in Bigquery and Snowflake, email_list_ids with an empty string are not included, while they are in Postgres and Databricks. This results in Postgres and Databricks having more rows in this intermediate model (see screenshots). This is most likely due to the different sets of rules required for each warehouse.

Relevant error log or model output
No response
Expected behavior

Decide whether or not to include rows with empty email_list_ids.
Implement decision across all warehouses so they to produce the same outputs.

dbt Project configurations
using seed data
Package versions
8.0
What database are you using dbt with?
postgres, redshift, snowflake, bigquery, databricks
dbt Version
1.5+
Additional Context
No response
Are you willing to open a PR to help address this issue?

[X] Yes.
[ ] Yes, but I will need assistance and will schedule time during our office hours for guidance
[ ] No.

@fivetran-catfritz I wonder if it is this line that is causing the issue you are seeing.
https://github.com/fivetran/dbt_iterable/blob/7863ee136350119cdc725875aa3becc2762f3ecb/models/intermediate/int_iterable__list_user_unnest.sql#L73
It may not be the incremental strategy but instead how the different databases handle empty lists. Do you see this same error across the databases when doing full refreshes? Or is this only seen on incremental runs?


web.css issue on Safari in macOS

In Safari, there seems to be display issue with the web.css for the "_port_active_led" button:

While in Chrome, it works fine:

Also, by default, the _port_active_led is not lit for the first login until I click on a port in KVM-Switch.

Thanks for reporting! Is this with the most recent version? The version before was okay?
I don't own any apple devices, but will see how far I get with wine. If you already have a solution, feel free to let me know ;)


Error opening maps for the first time

I'm submitting a ...  (check one with "x")

[ ] question
[x ] any problem or bug report

OS:  (check one with "x")

[ x] Android
[ x] iOS
[ ] Browser

cordova information:
call-number 0.0.2 "Cordova Call Number Plugin"
card.io.cordova.mobilesdk 2.1.0 "CardIO"
com-badrit-base64 0.2.0 "Base64"
cordova-base64-to-gallery 4.1.3 "base64ToGallery"
cordova-plugin-android-permissions 1.0.0 "Permissions"
cordova-plugin-androidx 1.0.2 "cordova-plugin-androidx"
cordova-plugin-androidx-adapter 1.1.0 "cordova-plugin-androidx-adapter"
cordova-plugin-app-event 1.2.1 "Application Events"
cordova-plugin-app-version 0.1.9 "AppVersion"
cordova-plugin-appminimize 1.0.1 "AppMinimize"
cordova-plugin-camera 2.4.1 "Camera"
cordova-plugin-compat 1.2.0 "Compat"
cordova-plugin-console 1.1.0 "Console"
cordova-plugin-daniel-readcalls 0.0.1 "ReadCalls"
cordova-plugin-datepicker 0.9.3 "DatePicker"
cordova-plugin-device 1.1.7 "Device"
cordova-plugin-email-composer 0.8.15 "EmailComposer"
cordova-plugin-file 6.0.2 "File"
cordova-plugin-file-opener2 2.2.1 "File Opener2"
cordova-plugin-file-transfer 1.7.1 "File Transfer"
cordova-plugin-filechooser 1.2.0 "File Chooser"
cordova-plugin-filepath 1.5.6 "cordova-plugin-filepath"
cordova-plugin-fingerprint-aio 1.7.0 "FingerprintAllInOne"
cordova-plugin-firebasex 5.0.0 "Google Firebase Plugin"
cordova-plugin-google-analytics 1.8.6 "Google Universal Analytics Plugin"
cordova-plugin-googlemaps 2.6.2 "cordova-plugin-googlemaps"
cordova-plugin-ionic-webview 4.1.1 "cordova-plugin-ionic-webview"
cordova-plugin-is-debug 1.0.0 "IsDebug"
cordova-plugin-local-notification 0.8.4 "LocalNotification"
cordova-plugin-mauron85-background-geolocation 3.0.1 "CDVBackgroundGeolocation"
cordova-plugin-nativestorage 2.3.2 "NativeStorage"
cordova-plugin-network-information 2.0.2 "Network Information"
cordova-plugin-request-location-accuracy 2.3.0 "Request Location Accuracy"
cordova-plugin-screen-orientation 2.0.2 "Screen Orientation"
cordova-plugin-splashscreen 4.1.0 "Splashscreen"
cordova-plugin-statusbar 2.4.3 "StatusBar"
cordova-plugin-whitelist 1.3.4 "Whitelist"
cordova-sms-plugin 1.0.0 "Cordova SMS Plugin"
cordova-sqlite-storage 2.6.0 "Cordova sqlite storage plugin"
cordova.plugins.diagnostic 3.9.2 "Diagnostic"
es6-promise-plugin 4.2.2 "Promise"
ionic-plugin-keyboard 2.2.1 "Keyboard"
onesignal-cordova-plugin 2.5.0 "OneSignal Push Notifications"

@ionic-native/core : 4.3.3
@ionic-native/google-maps : 4.20.0

Hello,
When installing the application, and opening maps for the first time, the application crashes, doing the process by android studio, it brings the following log:
java.lang.NullPointerException: Attempt to invoke virtual method 'boolean org.json.JSONObject.has(java.lang.String)' on a null object reference
     at plugin.google.maps.PluginMarker._create(PluginMarker.java:250)
     at plugin.google.maps.PluginMarkerCluster$3.run(PluginMarkerCluster.java:395)
     at android.os.Handler.handleCallback(Handler.java:873)
     at android.os.Handler.dispatchMessage(Handler.java:99)
     at android.os.Looper.loop(Looper.java:193)
     at android.app.ActivityThread.main(ActivityThread.java:6923)
     at java.lang.reflect.Method.invoke(Native Method)
     at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:493)
     at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:870)

However, when you open the application a second time, the error does not occur.
I would like to know the cause of the problem.

Hi, follow the repository https://github.com/EglisonHenrique/erro-maps
The error occurs when you install the app and open the map for the first time, from the moment you close the app and reopen, the map works normally.

Thank you for preparing.
I wonder why you specify "-24.728839", "-53.737562" as String instead of number.

I put to do the test, now changed to number, but got the same result. Would it have any impact on using String or Number?

lat/lng should be number not string.
And try to fix the height of map div for test.
#map {
        height: 500px;
}

lat / lng changed to number.
Map height to 500px, but error persists

Okay, I got the reasons.
There are two reasons.
1) your JSON data is too large. This causes memory overflow.
This plugin keeps the data in both side, JS and native.
Since you pass whole JSON data to the native side, this causes memory overflow.
I increase the number of cache size, but you also should slim down your data.
For example, keeps only the minimum necessary data
markers.push({
  position: { lat: element.geoLatitude, lng: element.geoLongitude },
  codProspect: element.codProspect   // <--keep only marker id
});

instead of keeping whole data.
markers.push({
  position: { lat: element.geoLatitude, lng: element.geoLongitude },
  prospect: JSON.stringify(element)
});

2) map visible region
The reason of the error occurs only the first time is that map tile is loaded after second time.
Even MAP_READY is fired, the camera information is not synchronized with JS and native side when the first time, because the native map is still loading.
That's why the marker cluster try to pass the whole location data to the native side.
Then memory overflow is occurred.

I increased cache memory size up to 10MB. Please reinstall the plugin from the multiple_maps branch.


Clicking on a username in a query window outputs userinfo again as new messages.

Steps

Click on a user in a regular window for which no query window is open.
After the query window opens click on the username.

Expected
Nothing happens as we already have a query window open so the conversation is already ongoing.
Actual
The user information is shown again.

Impact
I'd say the impact is rather low, it mostly confused me on mobile as it is easier to accidentally click on a user in chat there than on desktop.
When this happens in a channel and it opens up a window with the user it is clear what happend.
In this case when that window is already open it just is confusing, I personally thought at first that I found a bug with how the backscroll is loaded.

Are you using #1631?

No, I use the npm installed version. Also I am not entirely sure if they are related as it isn't technically a duplicate message, from what I can see what happens is that lounge simply sends another query and pastes the response back into already existing window.
I guess what I am specifically saying is that when a username is clicked for which a query window is already open lounge should simply switch to that window instead of sending out query again.

Ah yes, this kinda works as expected.
#533 is kinda related here.

Right, I see. As I said it is a rather low impact thing but since it works slightly different from how other client do this it confused me for a moment.
I totally understand that there are higher priority things to work on.

Ah yes, this kinda works as expected.

Sure, it's what we know happens. But that doesn't mean it should do it. It clearly shouldn't, it's such a weird UX thing.

Actually, now that I had a look at it a bit more (and saw ot is whois and not query command)  I think it isn't even correct. It means that every time I click on someone it sends out a whois even though I do not intend to do so.
On most networks this isn't really an issue except that soms IRCds notify users if a whois is done on them. (I think that unrealIRCd does this with certain user modes). Meaning that every time I accidentally click on someone they'll get notified.
Wouldn't it be better to offer users a choice of what they want to do when they click another user's name? Similar to how irccloud does this for example. Out the top of my head when clicking a user there you get a pop-up with

Username
Hostmask
Whois
Send message
Invite
Ignore

And if applicable also OP related things like op, voice, kick, ban, deop.
I realize I just expanded the scope hugely and I wouldn't be surprised if this already has an issue but figured I'd mention it anyway.

So I just found out that right clicking on a channel or user shows a context menu.
https://github.com/thelounge/lounge/blob/15e9ea5001420fa08466335f4abc1fe9558939ec/client/js/lounge.js#L77
Which from what I can gather is actually intended for what I propose. To me it makes sense to do maybe one of the following things:

Change this this bit of code to open the context menu on click and include whois as one of the options.
At the very least add the proposed actions to the context menu so people have a choice. This works sort of fine for desktop users but leaves the menu out of reach for mobile users.

Personally I am thinking 1. makes most sense, the context menu is already accessible in other areas through left click so.

I did some work on this earlier today to see how easy it would be to implement this. Resulting in this

Where whois and conversation work. The OP related actions do not yet work.
@YaManicKill & @xPaw would this be something worthwhile for me to develop a bit further and put in a pull request?

I had a PR for kick / ban - https://github.com/thelounge/lounge/pull/348
One major issue is banning should be by hostname, which the client can't get. Additionally, banning does not kick, so at the time we added a Kickban option.
However, I think we can make ban kick as that's what most users will expect.

Interesting, how did you handle context for showing those options?

What do you mean by context?

Well not everyone has the permissions to kick or ban people, so it doesn't make sense to show the options where they cannot be used.

I didn't get that far -- the reason I closed the PR was due to lack of information being available client side, like hostnames and op status.


import style from './styles' not found

Hello Team,
I am using your component for InlineEdit but i found an issue when used your given example and didin't found where you located import style from './styles'.
I am using with create-react-app its not working properly.
Please help me to fix it.
Thanks !

Hi @irontonnystark,
Thanks for reaching out.
To leverage the React:ions library in create-react-app, you will need to add an scss loader to your webpack config. This may require you to "eject" the app, so you can implement your own configuration.
Take a look here: https://github.com/GetAmbassador/react-ions/issues/630#issuecomment-391815441 for details on how to edit your webpack config.
Let me know how it goes!
Best,
Sam

Closing, as there seems to be no activity here.