Update gitignore to ignore generated files in src/

Closes #185

bump, @ritz078 ?

I am just figuring out a better way to do this. removing build files totally from that folder and putting them inside specific folders that can be excluded from git.

It's probably also worth setting up an npm prepublish command so similar files are ignored until published to npm

Few things:

dist folder is needed by cdnjs
even if we try to remove it we need task in prepublish

This will take some time as will need to refactor the folder structure.

just an fyi, this PR doesn't change the dist folder

i know that. I will just change the folder structure and write prepublish scripts so the dists get created before publish.

fixed in master


Added back MonitoringExporter, fixed db issues in Jenkins run

clean run: http://wls-jenkins.us.oracle.com/job/weblogic-kubernetes-operator-javatest/3339/

If we are not building the MonitoringExporter Project in our test infra, can we remove the git clone command in buildMonitoringExporter.sh script

We still need to clone monitoring project, we  build coordinator image and use endtoend  and deployments scripts to run our integration tests

If someone wants to run the tests with lower version of monitoring exporter , we should provide the option to give mon exporter branch value as well. You can work on a new PR for this..merging this one..


reducer: add keep dims option

@JohanMabille can we merge this one for the 0.20 branch!?

Superseded by #1474


Proposal: Allow spreading List of templ.Components, similar to { children ... }

Idea: We want to pass in a list of components as props and spread it inside a templ, similar to how we spread { children ...}
Reason: Sometimes children is not enough when we need to define multiple children rendering point, especially on complex wrapper components. At this point, we will define something similar to a render props from React.
Example:
templ Modal(actions []templ.Component) {

             {...children}

             {...actions}

}

templ ActionButton1() {
    <button>Submit</button>
}

templ ActionButton2() {
    <button>Cancel</button>
}

templ Main() {
   @Modal([]templ.Component{
          ActionButton1(),
          ActionButton2(),
   }) {

Content

   }
}

This currently doesn't work. Current workaround is to iterate through actions directly.
templ Modal(actions []templ.Component) {

             {...children}

             for _, action := range actions {
                   @action
             }

}

This proposal will improve the succinctness of the template.
Suggested implementation:
We can define a CombinedComponents type that implement templ.Component and represent a slice of templ.Component. Rendering will proceed by iterating over each children component.
Then at those spread point we instead render a wrapped CombinedComponents.

This strikes me as a map function in Go, so you could implement it as a function like this:
func Chain(components ...templ.Component) templ.Component {
  return templ.ComponentFunc(func(ctx context.Context, w io.Writer) (err error) {
    for _, c := range components {
      if err = c.Render(ctx, w); err != nil {
        return err
      }
    }
    return nil
  })
}

And use it in templ like:
templ Modal(actions []templ.Component) {

             {...children}

             @mypackage.Chain(actions...)

}

As an aside, if was designing it today, I wouldn't have added { ...children } as a special case, but would have created a @templ.Children() function instead - to get the children from the context, and rendered them out.
I was initially enthusiastic about adding something new to templ's namespace, but I'm not sure it's a good thing to hide the fact it's doing a loop just to save two lines of code.
I suspect something would pop up in the Go stdlib if it was something Go-ish. What do other folks think?

@joerdav basically proposed the same API I did, but for a different situation in https://github.com/a-h/templ/issues/786#issuecomment-2298507263
Maybe we should do something here?

Closed in #929


Add ability to use voice with navigation

Would be a nice to have where you can activate voice with a phrase like
Hey Comma, take me home which will route you to the preset home address

Something like this?
https://github.com/commaai/openpilot/wiki/iOS-OpenPilot-Navigation-Shortcuts

Closing as this is not an issue. Features requests can be discussed in Discord.


”Coroiu”, ”Crivești”, ”Vizureni” are listed in the drop-down menu

Description:
When user writes ”Coroiu”, ”Crivești”, ”Vizureni” the locations appear in the drop-down menu next to the comune and the county which is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Coroiu"

Expected results
The location appears in the drop-down menu as "Sat Coroiu, VASLUI (Tutova)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Crivești"

Expected results
The location appears in the drop-down menu as "Sat Crivești, VASLUI (Tutova)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Write in the search bar "Vizureni"

Expected results
The location appears in the drop-down menu as "Sat Vizureni, VASLUI (Tutova)".


Add SessionDetailScreenTest

Idea Description
Add a test like this.
https://github.com/DroidKaigi/conference-app-2022/blob/9a5093652a62da0c96e7914be6702cee5c56f1b0/feature-sessions/src/test/java/io/github/droidkaigi/confsched2022/feature/sessions/SessionsScreenTest.kt#L13
Reference images and links
You can check the README
https://github.com/DroidKaigi/conference-app-2022#make-test-scalable-by-using-robot-testing-pattern

Can I take this issue?

Thanks! Assigned 👍

@l2hyunwoo May I ask if you are working on this? I'm really looking forward to seeing the test 👍


Initial commit of SmartGateway Operator

Thanks submitting your Operator. Please check below list before you create your Pull Request.
New Submissions

[X] Have you selected the Project Community Operator Submissions in your PR on the right-hand menu bar?
[X] Are you familiar with our contribution guidelines?
[ X] Have you packaged and deployed your Operator for Operator Framework?

Updates to existing Operators

[ ] Is your new CSV pointing to the previous version with the replaces property?
[X ] Is your new CSV referenced in the appropriate channel defined in the package.yaml ?
[X ] Have you tested an update to your Operator when deployed via OLM?

Your submission should not

[ ] Modify more than one operator
[ ] Submit operators to both upstream-community-operators and community-operators at once
[ ] Modify any files outside the above mentioned folders
[ ] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[x] Description about the managed Application and where to find more information
[x] Features and capabilities of your Operator and how to use it
[ ] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[ X] Human readable name and 1-liner description about your Operator
[X ] Valid category name1
[ ] One of the pre-defined capability levels2
[X ] Links to the maintainer, source code and documentation
[ X] Example templates for all Custom Resource Definitions intended to be used
[ ] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file a PR against this repo and explain your need
2 For more information see here

using operator-sdk version: v0.10.0, commit: ff80b17737a6a0aade663e4827e8af3ab5a21170
I got the test pass for the spec and status block. Why is the CVP scorecard test failing?
operator-sdk scorecard --cr-manifest deploy/crds/smartgateway_v1alpha1_smartgateway_cr.yaml --csv-path deploy/olm-catalog/smartgateway-operator/0.1.0/smartgateway-operator.v0.1.0.clusterserviceversion.yaml
WARN[0000] Could not load config file; using flags
WARN[0000] Plugin directory not found; skipping external plugins: stat scorecard: no such file or directory
Basic Tests:
Spec Block Exists: 1/1
Status Block Exists: 1/1
Writing into CRs has an effect: 1/1
OLM Tests:
Spec fields with descriptors: 0/4
Status fields with descriptors: 0/1
Provided APIs have validation: 0/1
Owned CRDs have resources listed: 0/1
CRs have at least 1 example: 1/1
Total Score: 60%

/retest CVP

/test scorecard

/retest

@aneeshkp Did you run the scorecard in (predeployed) OLM? This uses just the CSV and no files from the deploy/ subdirectory. This could be an issue with the CSV that's causing the scorecard failure, though the pipeline doesn't give us much detail other than pass/fail status.
Try deploying the operator to OLM by pushing the metadata to quay and deploying using an OperatorSource. Then run the same scorecard command as before, but also passing the --olm-deployed flag.

/retest CVP


Rebooting using Telecommand

Implemented System Reboot using Telecommand. I achieved this by using NVIC_SystemReset().
I tried to print to UART a statement to notify the user the board is rebooting but the system reboots before it can get the response to the terminal.

I moved the reboot command outside of the flash telecommand section and I added the scan_12c_bus. I have ensured the debug_tools/debug_uart.h header is in the file. Is there anything else that I am missing and is there supposed to be code for the scan_12c_bus code within this branch?

Looking at the diff, this seems like it's getting worse with each commit.
I reverted this branch back to commit 1abc2b46. I also rebased it to main to resolve the merge conflicts.
You should be good to just Squash and Merge now. If you wanted to make any more changes, make sure to use GitHub Desktop to delete your local branch, and then pull down the latest version of this branch.

Sure thing, Thanks for the help in fixing this.

No worries! Always happy to help! :)


Usage of Session entities in user entity resolution

As System entity for GivenName and LastName is locale specific , so I am populating session entity to supplement my user entity PlayerFullName.  PlayerFullName is a composite entity consisting of a combination of System.GivenName and System.LastName. Even though I see that session entity gets populated with the name that i provide at run time but dialog flow is not able to resolve the PlayerFullName entity with the data supplied.  Am i missing something or  my understaing of dialogflow session entity is not correct. Please help
Thanks in advance.

Hey, trying to understand your setup.
You're creating a SessionEntity , is that correct or are you creating a EntityType and putting Entities inside it?

Yes I am creating sessionentity through API
var response = await sessionEntityTypesClient.createSessionEntityType(sessionEntityTypeRequest)
and I can see in the newly created entity in response.

@nnegrey as my dialogflow agent is in en-Au locale and I need to populate FirstName or LastName in my composite entity . It works for English name but the user base consists of non-English name. So i am trying to leverage session entity where during launch of the agent I supplement all the name that exist in the database.

Gotcha, one thing about SessionEntities is that they only exist for that Session, are you creating a new one for each created Session?
If you use it often, it may be worth creating a permanent one.

I have a dialogflow agent with webhook fulfilment  as AWS lambda serverless. The flow is when user launches the agent through google assistant, webhook is called to fulfil the welcome intent, during that fulfilment we load  all the employee reporting to that user in the SessionEntities  using dialogflow api. The fulfilment response is served by agent  and now user asks "what is the status of PersonName". So any non-English name is not getting picked up.
Is this not the same session that user has created through google assistant. If not then what purpose SessionEntities  serves in context of webhook fulfilment. I cannot populate the entity with entire list of person name in the database, as there are close to million record in the database. Please correct my understanding.

yes  I do, Is session Id parameter in createSessionEntityType is any arbitrary string or do we need to supply session id with the session property that we recieve in webhook fulfilment request. which looks like
"projects/agent-id/agent/sessions/ABwppHFLGocXk5tqY715LG61kv11SQTEAT4SYkG3Ci4AHD1YubHZtWRrPVGAy8mAne7aYRDgDpdOapstGs3ty5KC"
Because right now  i am supplying a new Guid every time.
The entity in my dialog flow looks like

What kind of training phrases do you have to help populate the entity?


New InvokeEndpoint JSON deserialization error

Hello all,
I've been actively working on a RAG project using Kendra and Llama 2. Everything has been functioning well until very recently, where I encountered the following error: ValueError: Error raised by the inference endpoint: An error occurred (ModelError) when calling the InvokeEndpoint operation: Received client error (422) from primary with message "Failed to deserialize the JSON body into the target type: inputs: invalid type: sequence, expected a string at line 1 column 11".
I also attempted to run the most recent kendra_chat_llama_2.py file, and it encountered the same error. My question is whether this issue is stemming from AWS SageMaker or if it's related to the code in the git repository? Your insights would be greatly appreciated.
Thank you!

fixed


AG Styling

[ ] need multiple AGs to see output
[ ] bottom seems to be affected by css selector (idk why every other selector aint workin)
[ ] Maybe repurpose code from vertretungsplan-ersatz to create tables?

regarding 2nd point: you probably use .field__item or similar to select the items in the main section. But there are actually many elements (also some in the footer) that have this generic CSS class. You can limit it to the main section by writing css #main .field__item { }  or ```css
.field--name-field-ag .field__item { }


Poppup is showing in the wrong coordinates

Using chrome on Monterey (21F79) on the recent MacBook Pro.
Popup is showing in the wrong coordinates and behind the Chrome window.

I have the same issue regarding the position of the pop with the same build.


talk: incoming DM says "0 members"

Describe the bug, optionally with screenshots

I have this notification for an incoming dm, then there are a few issues.

This DM is not possible to find outside of this notification (it's not in the DM list)
When I click the notification it goes to a DM that says it has zero members
I'm not sure if DMs I'm sending them are working

closing until we see more


ENH Added SURE support for MxNE and irMxNE

The goal of this PR is to implement SURE (Stein's unbiased risk estimate) to MxNE and irMxNE solvers.
WIP: Writing unitary tests.

@PABannier can you merge main branch in this branch and see if you can make CIs happy?

@agramfort It didn't...
For the record, I did:
git remote add upstream `https://github.com/mne-tools/mne-python.git
git fetch upstream
git merge upstream/master

Looks good https://29547-1301584-gh.circle-artifacts.com/0/dev/auto_examples/inverse/mixed_norm_inverse.html#sphx-glr-auto-examples-inverse-mixed-norm-inverse-py

@agramfort Awesome! What was the issue?

Thanks @PABannier !


Correction for outdated references

Replaced outdated references of "Azure Data Lake Store" with the current "Azure Data Lake Storage".

@AzureMentor : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


Option to force option in duplicate album dialog

This issue was automatically migrated from Google Code.
Original author<EMAIL_ADDRESS>(January 26, 2013 22:12:20)
Original issue: https://github.com/google-code-export/beets/issues/509

+1 (I would like to Keep Both since I keep multiple copies of the same song in different formats - usually FLAC and AAC)

+1

+1
I usually have both mp3 and flac files for the same release and would like to keep both.

If you want to keep both files, check out beets-alternatives
This will do the job very well

@pkess I'm sorry, but I don't see how this will help me not having to press 'keep' with every import of release of which I already have a version in a different format.
What I'm looking for is when I import...

an identical release but different format (e.g. mp3 and flac): automatically keep the release
an identical release with same format (mp3 V0 and mp3 320): automatically select the highest bitrate or if that is not possible, present me with a choice.

Could you please elaborate how I can use beets-alternatives to achieve that?
thx!

+1
Apparently Beets does not allow duplicates in quite mode, it outputs: "This album is already in the library! Skipping."
But I'm interested in a library having the same album but with different codecs. Is it possible to set Beets in quiet mode but allowing duplicates (storing those in the library)?
If currently it is not possible, I think that an easy solution for that is to enable a parameter in the config.yaml file that would allow Beets to change the default option importing duplicates. For example:
import:
         duplicates: Skip new/Keep both/Remove old/Ask

Setting as default the "Ask" option when importing normally and setting as default the "Skip new" or "Remove old" in quiet mode. Being able to change such default option would allow people to set "Keep both" option in quite mode, if necessary.

Thanks for the thoughts -- that's exactly how I imagine this working. It shouldn't be too big a project if anyone's interested in a way to get familiar with the beets source code.

Great! This issue is here since January 2013 and there is quite a bit of people interested on that. So, let's move!
Anyone familiar with the code? I'm not familiar with the code and I need some help :)

Awesome! The wiki page called Hacking has an intro to the source, and the developer section of the docs has more detail on the main concepts involved. You'll want to look in importer.py, which contains th call to resolve_duplicate that asks the user. Jump in and let us know if you get stuck!

Hi, I was just wondering if there had been any movement on this? Like some other folks I'd love to be able to import my collection which includes many albums in multiple formats (FLAC and MP3) without having to manually input K(eep both) each time. With a music collection that is over 1.5tb this is way too time consuming to be feasible for me at the moment.

As a general rule, there's no news that's not posted in the issue tracker. If you're interested in this, please consider helping out!


fix(errors): adjust message of QuotasExceededError

Codecov Report

Merging #404 (6f0fada) into main (723e322) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main      #404   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           44        44
  Lines          636       636
  Branches       129       129
=========================================
  Hits           636       636

Impacted Files
Coverage Δ

...s/src/scw/errors/standard/quotas-exceeded-error.ts
100.00% <100.00%> (ø)

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


Build error caused by Lint rule ktlintMainSourceSetCheck

Describe the bug
Android build is affected by lint during RN build.
Ideally check tasks do not make sense to run on a shipped library? esp like RN.
We see build error on check tasks and that too of test files.
Reproduction steps
Run android build via npx react-native run-android
SDK logs
> Task :datadog_mobile-react-native:ktlintMainSourceSetCheck FAILED
/Users/jshah/workstuff/my-sample-app/node_modules/@datadog/mobile-react-native/android/src/main/kotlin/com/datadog/reactnative/DatadogSDKWrapper.kt:19:1 Unexpected indentation (4) (should be 2)
/Users/jshah/workstuff/my-sample-app/node_modules/@datadog/mobile-react-native/android/src/main/kotlin/com/datadog/reactnative/DatadogSDKWrapper.kt:20:1 Unexpected indentation (8) (should be 4)

...

> Task :app:bundleReleaseJsAndAssets
Execution optimizations have been disabled for task ':app:bundleReleaseJsAndAssets' to ensure correctness due to the following reasons:
  - Gradle detected a problem with the following location: '/Users/jshah/workstuff/my-sample-app'. Reason: Task ':app:bundleReleaseJsAndAssets' uses this output of task ':datadog_mobile-react-native:bundleDebugAar' without declaring an explicit or implicit dependency. This can lead to incorrect results being produced, depending on what order the tasks are executed. Please refer to https://docs.gradle.org/7.3.3/userguide/validation_problems.html#implicit_dependency for more details about this problem.
  - Gradle detected a problem with the following location: '/Users/jshah/workstuff/my-sample-app'. Reason: Task ':app:bundleReleaseJsAndAssets' uses this output of task ':datadog_mobile-react-native:bundleLibCompileToJarDebug' without declaring an explicit or implicit dependency. This can lead to incorrect results being produced, depending on what order the tasks are executed. Please refer to https://docs.gradle.org/7.3.3/userguide/validation_problems.html#implicit_dependency for more details about this problem.

...

> Task :datadog_mobile-react-native:ktlintTestSourceSetCheck FAILED
/Users/jshah/workstuff/my-sample-app/node_modules/@datadog/mobile-react-native/android/src/test/kotlin/com/datadog/reactnative/DdLogsTest.kt:38:1 Unexpected indentation (4) (should be 2)
/Users/jshah/workstuff/my-sample-app/node_modules/@datadog/mobile-react-native/android/src/test/kotlin/com/datadog/reactnative/DdLogsTest.kt:39:1 Unexpected indentation (4) (should be 2)

...
1: Task failed with an exception.
-----------
* What went wrong:
Execution failed for task ':datadog_mobile-react-native:ktlintMainSourceSetCheck'.
> A failure occurred while executing org.jlleitschuh.gradle.ktlint.worker.ConsoleReportWorkAction
   > KtLint found code style violations. Please see the following reports:
     -<EMAIL_ADDRESS>
* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.
==============================================================================

Expected behavior
No build error should be present
Affected SDK versions
"<EMAIL_ADDRESS>Latest working SDK version
"<EMAIL_ADDRESS>Did you confirm if the latest SDK version fixes the bug?
No, cannot easily upgrade
Integration Methods
Yarn
React Native Version
0.68.2
Package.json Contents
No response
iOS Setup
No response
Android Setup
No response
Device Information
Android emulator
Other relevant information
No response

Hi @jayshah123 👋
Thank you for reaching out! I see that you are using<EMAIL_ADDRESS>which was released on Jun 8, 2023.
I would suggest updating to the latest version, where the problem should not persist. We will double check the issue in the meantime.
Is there anything blocking you from updating the SDK?

I believe an RN upgrade might be needed. Is there a compatibility table available? Or all versions of your lib compatible with older versions of RN?

Our latest release supports v0.68.2, you can try updating and reach out if you encounter any issues.
If you have other Datadog packages installed, please make sure they have matching versions.


Issue with Date and Price mapping

from #40
Hi @alvarobartt .. updated investiny version however ran into some issues --

The output is missing 10/14 (Friday) and displaying 10/15, prices off by a day. This is for DXY.

Thanks for reporting this, I may create a label for all the date-related issues... Let me check in detail! Anyway, why there is information for 10/16/2022 which is actually a Sunday?

Soooo @KapsTrvl this is what I can see:
>>> from investiny import search_assets
>>> res = search_assets(query="DXY")
>>> print(res)
[{'symbol': 'DXY', 'full_name': 'NYSE:DXY', 'description': 'US Dollar Index', 'type': 'Index', 'ticker': '942611', 'exchange': 'NYSE'}, {'symbol': 'DXYN', 'full_name': 'NASDAQ:DXYN', 'description': 'The Dixie Group Inc', 'type': 'Stock', 'ticker': '15953', 'exchange': 'NASDAQ'}]
>>> from investiny import historical_data
>>> d = historical_data(investing_id=942611)
>>> d["date"]
['09/19/2022', '09/20/2022', '09/21/2022', '09/22/2022', '09/23/2022', '09/26/2022', '09/27/2022', '09/28/2022', '09/29/2022', '09/30/2022', '10/03/2022', '10/04/2022', '10/05/2022', '10/06/2022', '10/07/2022', '10/10/2022', '10/11/2022', '10/12/2022', '10/13/2022', '10/14/2022', '10/16/2022', '10/17/2022']

so everything looks good to me, at least according to the information provided by Investing.com at https://www.investing.com/indices/usdollar-historical-data

FYI same code as above, but using also the from_date and to_date args, so make sure that you're using the latest investiny version which is currently 0.7.1 as you can see at https://github.com/alvarobartt/investiny/releases/tag/v0.7.1
>>> from investiny import historical_data
>>> d = historical_data(investing_id=942611, from_date="10/02/2022", to_date="10/17/2022")
>>> d["date"]
['10/03/2022', '10/04/2022', '10/05/2022', '10/06/2022', '10/07/2022', '10/10/2022', '10/11/2022', '10/12/2022', '10/13/2022', '10/14/2022', '10/16/2022', '10/17/2022']

DXY starts ticking Sunday evening (NY Time- EST), so investing.com seems to be displaying accordingly. I am using the latest version, ran the command you gave in the previous ticket. Did it once more, however same issue again ..

Likely time zone from where user initiating the call causing the issue (I am running it from new york time zone)

DXY starts ticking Sunday evening (NY Time- EST), so investing.com seems to be displaying accordingly. I am using the latest version, ran the command you gave in the previous ticket. Did it once more, however same issue again ..

Likely time zone from where user initiating the call causing the issue (I am running it from new york time zone)

So timezones again... let me check tomorrow morning to see whether I can fix the issue. A simple conversion to UTC should work, but not sure, let's check tomorrow!


Allow repository administrators to create or edit submission forms directly in the user interface

We are utilizing a DSpace 5 instance and are exploring the possibility of migrating to DSpace 7.
Over the past year, our federated repository structure has undergone significant changes. The administrator function has been decentralized to different community levels, resulting in some difficulties in implementing change controls at the central library.
In DSpace 7, it would be beneficial to have a feature that enables an overall repository administrator to easily modify submission forms from the user interface. This feature should be user-friendly and accessible, such as a form-like interface, a UI data-frame, or an exported spreadsheet. This should be achievable without needing to access an XML file in the Configuration directory on the server or contracting a third-party to make the changes.
A colleague has informed me that other DSpace repositories in our region, such as https://repository.escholarship.umassmed.edu/, have customized features built into their XMLUI interface. Their administrators can update or create submission forms via an uploaded spreadsheet. We believe it would be useful if this custom feature could be integrated as a core function within DSpace Angular.

Agreed that this would be a great feature.  If someone already has a way to achieve this, we'd love for the code or concepts to be shared back... DSpace is very much a collaboration.  Most of the code / new features are contributed via community members or service providers.
In the meantime, this needs a volunteer to investigate or contribute code back.  I'll leave this ticket open for others to comment on or provide suggestions.


Write prerendered files after each page render, don't keep the html in memory

Benchmark:
prerender: { parallel: 1 }
55mb large pages/hello/index.page.ts
in writeHtmlFile : gc(); console.log(process.memoryUsage().heapUsed)
Before: multiple instances of the page are kept in memory, for the whole run

After: the page gets cleared from the memory after each page render

Writing files early is definitely a big improvement 💯
Let me know if you're on twitter and I'll mention you on the next release announcement tweet.


Commands for plugin not working/also group sync not working.

I am running the latest version of spigot, and the latest version of the plugin and can't seem to get the commands or the group sync to work.
While trying to do the commands it just doesn't recognize that the commands even exist, below I've linked the config for the commands
https://imgur.com/a/wngrs
The rank sync doesn't work even though I've connected my server to the site. I'm not sure what I'm doing wrong here any help would be lovely!
https://imgur.com/a/LHcwt

In the v1 plugin you can't have both one command with subcommands and individual commands. Disable one.
Your group sync should look like this:
6: nameless.donator

Thanks so much, so this is how the group sync should look?
https://imgur.com/a/8DZjH

Yes that should work, give your donators the nameless.donator permission

Thanks, great plugin, thanks for the quick response!


HDInsight on AKS icons to be added in the inventory list

Hi,
There is a new HDInsight on AKS service which will be going public preview in next 10 days. Can you please add the icons for it in the repository?
You can contact me and I will share it with you.

@snehalsonwane
It would be great if you could add a link to the documentation you are following for these steps? This would help us redirect the issue to the appropriate team. Thanks!!

On this link https://learn.microsoft.com/en-us/azure/architecture/icons/ , there is a downloadable file for azure services icons and my question is how to add the icons for new services which are not in the list.

@snehalsonwane
I've delegated this to @Zimmergren, a content author, to review and share their valuable insights.

Thank you for submitting this request - we're discussing offline now. #please-close


Abstraction for morphology collection.

Currently, MorphIO requires the file, filename or HDF5 group which contains the morphology. This PR adds an interface for loading morphologies by name from a Collection object.
The first commits improve style, turn off problematic flags, add missing ctors and fix a bug in the HDF5 reader. Furthermore, they add gulrak's implementation of std::filestystem. The last commit adds the proposed feature and its tests.

Please let me know if you want to split certain commits into their own PRs.

This would implement part of #443.

Thank you for the review. I'll change the unaddressed comments as suggested.
Currently, I'm working on integrating this into TouchDetector. We can try this "new" API (it's mostly copied from an internal library) there before merging it. This would check the C++ API and currently missing thread-safety (WIP).
After that we'll also need to prepare SpatialIndex. Which would check that the Python API is fine.

I'm basically happy with this.  Is there anyone else you'd like to review it?

I've run it through TD and that works. I'm checking SI.

I've tried merging this several times, but w/ the github outage, I think it's best I wait until tomorrow.


[Helm] Disable Postgres DB flag doesn't work

Description:
When setting only DP components, we are setting flags as --set wso2.apk.cp.enabled=false --set wso2.apk.cp.postgresql.enabled=false, but the DB pod is getting spin up.

admin@sampath ~ % kubectl get pods -n apk
NAME                                                           READY   STATUS    RESTARTS      AGE
apk-test-cert-manager-5489f4cc85-l9r8d                         1/1     Running   0             2m28s
apk-test-cert-manager-cainjector-778cf8949d-p598x              1/1     Running   0             2m28s
apk-test-cert-manager-webhook-cbf84648b-8q6mh                  1/1     Running   0             2m28s
apk-test-wso2-apk-adapter-deployment-7564c57f66-vtm7f          1/1     Running   4 (53s ago)   2m28s
apk-test-wso2-apk-gateway-runtime-deployment-f759fdbb4-9j8qs   2/2     Running   0             2m28s
apk-test-wso2-apk-idpds-deployment-5495544b9c-zwt8k            1/1     Running   0             2m28s
apk-test-wso2-apk-idpui-deployment-74f45c8cc6-4cqmg            1/1     Running   0             2m28s
apk-test-wso2-apk-runtime-ds-deployment-589c4d4d54-ltwlb       1/1     Running   0             2m28s
wso2apk-db-service-0                                           1/1     Running   0             2m28s
admin@sampath ~ %

to remove database from deployment need to comment postgresql: section in values.yaml


Add Cloutier et al 2017 toggleJS AJ examples

https://doi.org/10.3847/1538-3881/aaa54e

Things that are unclear:

[ ] show one or many examples
[ ] ask Cloutier for tips on how he created transparent versions


Create a Business Service with Node.js Using Visual Studio Code

Tutorial URL: https://developers.sap.com/tutorials/cp-apm-nodejs-create-service.html
Please specify the step you are referring to
Step 9.
It says ' @insertonly'. It should have been @readonly.

Hi,
The description for step 9 is correct saying "Any GET or DELETE to an order fails since we’ve specified the Orders entity to be @insertonly in srv/cat-service.cds." cause we have modified in step 6.3. the cat-service.cds for the Order: "entity Orders @insertonly as projection on my.Orders;"
Best regards,
Iwona


Skips many songs.

First before playing anything I saw thus error, realised something is wrong cuz its a pytube error.
Traceback (most recent call last):
  File "/usr/lib/python3.11/threading.py", line 1038, in _bootstrap_inner
    self.run()
  File "/home/anamitra/.local/lib/python3.11/site-packages/mopidy_ytmusic/repeating_timer.py", line 18, in run
    self._method()
  File "/home/anamitra/.local/lib/python3.11/site-packages/mopidy_ytmusic/backend.py", line 99, in _refresh_youtube_player
    self.playback.update_cipher(playerurl=url)
  File "/home/anamitra/.local/lib/python3.11/site-packages/mopidy_ytmusic/playback.py", line 25, in update_cipher
    self.PyTubeCipher = Cipher(js=response.text)
                        ^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/anamitra/.local/lib/python3.11/site-packages/pytube/cipher.py", line 43, in __init__
    self.throttling_plan = get_throttling_plan(js)
                           ^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/anamitra/.local/lib/python3.11/site-packages/pytube/cipher.py", line 405, in get_throttling_plan
    raw_code = get_throttling_function_code(js)
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/anamitra/.local/lib/python3.11/site-packages/pytube/cipher.py", line 311, in get_throttling_function_code
    name = re.escape(get_throttling_function_name(js))
                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/anamitra/.local/lib/python3.11/site-packages/pytube/cipher.py", line 296, in get_throttling_function_name
    raise RegexMatchError(
pytube.exceptions.RegexMatchError: get_throttling_function_name: could not find match for multiple

then, when I try to play a song, it skips many songs.
  translate_uri error "'NoneType' object has no attribute 'get_signature'"
WARNING  2023-07-05 23:50:02,024 [12921:Core-10 (_actor_loop)] mopidy.core.tracklist
  Track is not playable: ytmusic:track:tKaTJhgBFMM
ERROR    2023-07-05 23:50:02,464 [12921:YTMusicBackend-7 (_actor_loop)] mopidy_ytmusic
  translate_uri error "'NoneType' object has no attribute 'get_signature'"
WARNING  2023-07-05 23:50:02,465 [12921:Core-10 (_actor_loop)] mopidy.core.tracklist
  Track is not playable: ytmusic:track:u9hHetyLtbQ
ERROR    2023-07-05 23:50:02,736 [12921:YTMusicBackend-7 (_actor_loop)] mopidy_ytmusic
  translate_uri error "'NoneType' object has no attribute 'get_signature'"
WARNING  2023-07-05 23:50:02,737 [12921:Core-10 (_actor_loop)] mopidy.core.tracklist
  Track is not playable: ytmusic:track:u1LBHKQjhLU
ERROR    2023-07-05 23:50:03,014 [12921:YTMusicBackend-7 (_actor_loop)] mopidy_ytmusic
  translate_uri error "'NoneType' object has no attribute 'get_signature'"
WARNING  2023-07-05 23:50:03,015 [12921:Core-10 (_actor_loop)] mopidy.core.tracklist
  Track is not playable: ytmusic:track:-00bkI1871M

same here, songs are just being skipped instead of being played.

Sorry, I forgot to update the status of the problem, my bad 😅 but this from #71  can be a temporary fix.
Don't know if this breaks anything else yet.

@bendschs if you are still here can you check if mopidy is throwing errors when trying to search ytmusic through webclients like iris or musicbox?

Sorry, I forgot to update the status of the problem, my bad 😅 but this from #71 can be a temporary fix.
Don't know if this breaks anything else yet, it shouldn't.

this doesn't fix, it suppresses exception. tracks still skipped

Sorry, I forgot to update the status of the problem, my bad 😅 but this from #71 can be a temporary fix.
Don't know if this breaks anything else yet, it shouldn't.

this doesn't fix, it suppresses exception. tracks still skipped

This fixed for me before now there seems to be a new exception

Sorry, I forgot to update the status of the problem, my bad 😅 but this from #71 can be a temporary fix.
Don't know if this breaks anything else yet, it shouldn't.

this doesn't fix, it suppresses exception. tracks still skipped

This fixed for me before now there seems to be a new exception

#71 fixed it for me as well, i had to follow this link to get cypher.py though: https://github.com/yogeshgirmal/pytube/blob/master/pytube/cipher.py
for me it is still working.
hope to have helped
bendsch

Well, it magically started working again. After replacing cipher.py, that is.


[SP] Add augmented reality sample with Polymer UI

BUG=XWALK-6362

@huningxin what is this Appveyor error?

Went green! @leonhsl @halton please help to review. Thanks!

@leonhsl could you spend some time to review?

Other than above comments, lgtm

@leonhsl thanks for reviewing. Update.

lgtm, merging..

Merged


Where is the color set for the codebox language text ?

I have edited all my codebox color settings to make the colors be more readable. However, I cannot find where the code box language color is set. This is also the color for the code brackets as well.
Could someone please point me to where that super dark grey color is coming from?
See the screencap - the red arrows point toward the items affected by the dark color....

Not a direct answer, but you can try the Settings Search plugin. It allows you to search within the settings menu, which may shed some light on your issue.

Hmm I also don't like the color because of its low contrast, so I did some experiments. Here's what I found:

The color codes for that gray is #666666
#666666 remains the same across different themes, light/dark mode.
#666666 does not appear anywhere in settings.

TLDR:  I guess that cannot be customized.

the code box language color:

Code bracket color:
Find the following customization item to set

Thank you! it feels like I had checked the color options about 30x and have never seen "Code Block punctuation Color" and "Code Block language Color". Then you mention and there they are right in front of me...
Also thanks for the Search settings plugin tip. Such a helpful plugin.

Custom Codebox Syntax Highlighting  Colors


[BUG] Cannot read properties of null (reading 'map')

What happened?
Tried adding a plot directory with the add plot directory btn in the gui.
Version
1.3.5
What platform are you using?
Windows
What ui mode are you using?
GUI
Relevant log output or stacktrace
Cannot read properties of null (reading 'map')

URL
#/dashboard/plot

Stacktrace
at webpack://@chia/gui/src/components/plot/PlotAddDirectoryDialog.tsx:95:25 map
at webpack://@chia/gui/node_modules/react-dom/cjs/react-dom.production.min.js:157:136 Ch
at webpack://@chia/gui/node_modules/react-dom/cjs/react-dom.production.min.js:180:153 oi
at webpack://@chia/gui/node_modules/react-dom/cjs/react-dom.production.min.js:269:342 a=b.pendingProps;f=e._init;e=f(e._payload);b.type=e;f=b.tag=hk(e);a=lg(e,a);switch
at webpack://@chia/gui/node_modules/react-dom/cjs/react-dom.production.min.js:250:346 Pj
at webpack://@chia/gui/node_modules/react-dom/cjs/react-dom.production.min.js:250:277 Pj
at webpack://@chia/gui/node_modules/react-dom/cjs/react-dom.production.min.js:250:137 Pj
at webpack://@chia/gui/node_modules/react-dom/cjs/react-dom.production.min.js:243:162 Lj
at webpack://@chia/gui/node_modules/react-dom/cjs/react-dom.production.min.js:123:114 jg
at webpack://@chia/gui/node_modules/scheduler/cjs/scheduler.production.min.js:18:342 unstable_next

I assume that this occurred due to the 'plot directories' entry in config.yaml had a null value after deleting plot

This issue was fixed in the latest 1.4 version. Please upgrade your version


feat(html): html template automatic compress

Introduces the core code of Google html compressor
Custom Beetl engine to compress html
After Beetl reads the template, the HTML is automatically compressed before parameters are injected
Formatting html files

Closes #991

looks good


docs: Add top-level Go applications page

Also move middleware.md into docs/go/ and add some minimal content to
the page.
Also add /docs/go/build.md with a description of the Go build process.
Also add /docs/go/tools.md doc with a minimal description of bbbfancontrol
and bbbsupervisor.
Also add link to Go docs from docs/os/build-details.md.

Thanks for the review, @Stadicus! PTAL.

Just to make sure I understand the goal of this pull request:

Please see the updated commit description for d4f62e1f77d670f6a6e0195bf1ef11686ef9a495, and let me know if anything remains unclear about the changes made in this PR.

* it adds a new `/docs/go/` directory

* according to the `permalink`s you add three additional subdirectories under `/docs/go/`

As described in a comment thread, the permalinks were there as a workaround to try to create predictable URLs, but I've dropped them at this point and I am now just referencing the .md files in the links, letting Jekyll sort out where the <a href="..."> should point.

* it describes the generig Go build process in `/docs/go/build.md` and adds additional sub-chapters in `/docs/go/build/...`

* it therefore replaces current chapter 5 (Middleware) and will add all these Middleware detail chapters under `/docs/go/middleware/...`

* it also moves current "Custom Tools" (chapter 7.4) with `/docs/go/tools.md` and creates a subdirectory to add detailed documentation in `/docs/go/tools/...`

I think that's a good approach, as long as
* the sub-subdirectories are really necessary (meaning having 3+ additional `.md` files in them), which I'm not sure regarding the Go build documentation

* the overall build process is additionally described early on (as currently proposed as chapter 2 "Creating the Base image")

* all relevant detail documentation of the Middleware and custom Go tools will become part of this structure.

I'm not 100% clear if you are suggesting the three bulletpointed changes should be made in this PR, or for future work?

Please see the updated commit description for d4f62e1, and let me know if anything remains unclear about the changes made in this PR.

Apologies, I had overlooked some references to Go Services, which I've now updated to Go applications consistently and force-pushed as 8f7c81daacd67afb4f3af72bad7213c327c95027.

the sub-subdirectories are really necessary (meaning having 3+ additional .md files in them), which I'm not sure regarding the Go build documentation

Ok, I have misunderstood your intent to create an additional level of sub-directories. ✅

the overall build process is additionally described early on (as currently proposed as chapter 2 "Creating the Base image")

Not meant for this pull-request, I just wanted to point out that there is an "entry point" missing for people approaching this project to understand how to get started with building the whole thing. Can be a single page overview "3) Creating the Base image" without all the sub-chapters.

all relevant detail documentation of the Middleware and custom Go tools will become part of this structure.

I just want to make sure that we know where the additional detail chapters for the Middleware are supposed to be later on (as you deleted the middleware.md chapter). How would you structure it to include "API spec", "Noise Framework", "HSM integration", etc.... ?

Where would you now put the supervisor.md documentation in that structure?

Not meant for this pull-request, I just wanted to point out that there is an "entry point" missing for people approaching this project to understand how to get started with building the whole thing. Can be a single page overview "3) Creating the Base image" without all the sub-chapters.

Sounds good.

all relevant detail documentation of the Middleware and custom Go tools will become part of this structure.

I just want to make sure that we know where the additional detail chapters for the Middleware are supposed to be later on (as you deleted the middleware.md chapter). How would you structure it to include "API spec", "Noise Framework", "HSM integration", etc.... ?

I would personally add that as subsections to middleware.md, or if that grew too large, to more specific documents like middleware-api.md etc.

Where would you now put the supervisor.md documentation in that structure?

The docs/go/tools.md doc mentions the existence of the bbbsupervisor, and links to the source code for it. I'd suggest either adding supervisor-related content directly to tools.md or making it into a subpage under docs/go/.
No strong feelings here (or in general) on how to structure the documentation, I'd be happy to go with whatever you prefer. :)

utACK https://github.com/digitalbitbox/bitbox-base/commit/ca7208176ddffcaba5e8fc0e48bf961d831f4b08


Temporary dataset_path for remote fs URIs not built properly in arrow_dataset.py::load_from_disk

Describe the bug
When trying to build a temporary dataset path from a remote URI in this block of code:
https://github.com/huggingface/datasets/blob/42f6b1d18a4a1b6009b6e62d115491be16dfca22/src/datasets/arrow_dataset.py#L1038-L1042
the result is not the expected when passing an absolute path in an URI like hdfs:///absolute/path.
Steps to reproduce the bug
dataset_path = "hdfs:///absolute/path"
src_dataset_path = extract_path_from_uri(dataset_path)
tmp_dir = get_temporary_cache_files_directory()
dataset_path = Path(tmp_dir, src_dataset_path)
print(dataset_path)

Expected results
With the code above, we would expect a value in dataset_path similar to:
/tmp/tmpnwxyvao5/absolute/path
Actual results
However, we get a dataset_path value like:
/absolute/path
This is because this line here: https://github.com/huggingface/datasets/blob/42f6b1d18a4a1b6009b6e62d115491be16dfca22/src/datasets/arrow_dataset.py#L1041
returns the last absolute path when two absolute paths (the one in tmp_dir and the one extracted from the URI in src_dataset_path) are passed as arguments.
Environment info

datasets version: 1.13.3
Platform: Linux-3.10.0-1160.15.2.el7.x86_64-x86_64-with-glibc2.33
Python version: 3.9.7
PyArrow version: 5.0.0

Hi ! Good catch and thanks for opening a PR :)
I just responded in your PR


Reduce number of metrics sent to influxdb

Sam talked the changes through with me, all sounded good.


Bad command line error on mistyped compiler flags

❯ roc fmt ~/testing/cli_test.roc
error: Found argument '/home/dankey/testing/cli_test.roc' which wasn't expected, or isn't valid in this context

USAGE:
    roc [OPTIONS] [ROC_FILE] [-- <ARGS_FOR_APP>...] [SUBCOMMAND]

For more information try --help

Instead it should say the second argument is malformed
Or to go the extra mile, do a flag suggestion by similar words bank
I had an idea to not even exit on bad keyword, rather
error: Found unexpected flag `fmt`. Perhaps you meant `format`.

Proceed with command `roc format cli_test.roc` [Y/n]? _
    ...

:100: I've been wanting that functionality for a long time


Add benchmark for valideer

Added a new benchmark for Valideer, yet another validation library (Disclaimer: I am the main author).
Despite the fact that high performance was not among valideer's goals and that it is a pure python package written ~7 years ago, it appears to be as fast as pydantic when the latter is compiled with Cython and 1.6x faster when not. Here are some benchmark results on python 3.6.7:
pydantic (cython)        best=15.165μs/iter  avg=15.842μs/iter  stdev=0.685μs/iter
valideer                 best=15.686μs/iter  avg=15.885μs/iter  stdev=0.219μs/iter
pydantic (pure python)   best=24.488μs/iter  avg=25.639μs/iter  stdev=0.815μs/iter
toasted-marshmallow      best=34.630μs/iter  avg=35.930μs/iter  stdev=1.627μs/iter
marshmallow              best=35.155μs/iter  avg=36.345μs/iter  stdev=1.453μs/iter
trafaret                 best=37.089μs/iter  avg=37.947μs/iter  stdev=1.109μs/iter
django-restful-framework best=157.965μs/iter avg=162.176μs/iter stdev=3.537μs/iter

Package versions:

pydantic==0.30.1
valideer==0.4.2
toastedmarshmallow==2.15.2.post1
marshmallow==2.19.5
trafaret==1.2.0
djangorestframework==3.10.0

Thanks so much for this, sorry I haven't dealt with it. I'm going to defer further changes to benchmarks until after v1 (see #576), but after that I promise to deal with this.
Sorry again for the delay.

Sure no rush, thanks for the update.

I've merged master, and updated benchmarks. I'll merge this once tests pass.
valideer is still the fasted other than pydantic :-)

Great, thank you!


Fix for deploy error with deprecated cli

Why are these changes needed?
CLI is being deprecated and this removes the file that was causing the deploy ci step to fail. Also removed residual cli/ mentions in the repo.
Related issue number

Checks

[ ] I've made sure the tests are passing.
Testing Strategy

[ ] Unit tests
[ ] Manual tests
[ ] This PR is not tested :(

@kevin85421


Nvidia Deepstream SDK module runtime backoff with Exit Code 255

I was trying to run Nvidia Deepstream SDK example by referencing this documentation. But when I was trying to run Deepstream to output the inferred videos to an RTSP video stream and visualize this RTSP stream with VLC I am running into Exit Code 255 error and the runtime status shows backoff. I was trying to run same steps that was mentioned in the ReadMe file but don't understand why the Deepstream SDK module doesn't work this way.
Here's the snapshot of my deployed Azure IoT Edge resources,

@leela-uppuluri Please do note that this repository is primarily for addressing feedback on Azure Docs. I believe it would be best to raise an issue on the repository that you've linked instead and/or post a query on MSDN or StackOverflow for engaging the broader community.

@leela-uppuluri Since there is no specific azure doc issue to address here, we will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


feat(skeleton): add box primitive

Summary
Introduce a new skeleton component <KSkeletonBox /> which allows users to compose skeletons to form their own placeholder content. This was born from needing to create a custom placeholder that looks more like the content it is place-holding. For example, Kong Manager there are multiple cards with different layouts, so instead of making the cards look generic everywhere, allow them to be composable and specialized to the content. e.g. plugin select cards, workspace dashboard cards. Check new docs in this PR for more information on usage.
Changes made:

export kskeleton as a plugin, auto-importing components
use KSkeletonBox in all the skeleton variants
slot content in skeleton card
allow KSkeletonBox to be used as a primitive outside of KSkeleton

PR Checklist

[x] Does not introduce dependencies
[x] Functional: all changes do not break existing APIs and if so, bump major version.
[x] Tests pass: check the output of yarn test packages/
[x] Naming: the files and the method and prop variables use the same naming conventions as other Kongponents
[x] Framework style: abides by the essential rules in Vue's style guide
[x] Cleanliness: does not have formatting issues, unused code (e.g., console.logs), or leftover comments
[x] Docs: includes a technically accurate README, uses JSDOC where appropriate

Can you take a look at the overflow of cards while you're here? If you shrink to real small it breaks out of the card

Can you take a look at the overflow of cards

should be responsive now


add commenting to codebase

I'd like to use this project as my boilerplate since it clearly incorporates all the best practices around a universal react and redux application but I'm finding it very challenging navigating the codebase due to it's lack of commenting.

Coming from a non-JavaScript background, I agree that we should have more comments in the codebase.   The complete lack of commenting is a bit of the 'maverick spirit' embraced by some developers.   I would hope we could move to a level of comments that include:

One line at the top of each file explaining the real purpose of the file.  That is, a line for ./src/containers/Widgets/Widgets.js like Demonstrates fetching and reloading strategies using WidgetForm components.  Not long; not boilerplate.

One line per non-boilerplate function describing the purpose.  For example, render() never requires an explanation, but asyncValidate() should have one line.

There will be a space for guides and walkthroughs, lest the code get riddled with comments explaining ES6 idioms or descriptions of packages.  Still, the project could do with some minimal documentation standard.

+1


New items dont display on scroll, have to click 'Load More Items'

My listview doesn't display new items when scrolling to the last element. I have to click 'Load More Items' in order to load the next page.
This is my ListView:
echo ListView::widget([
                           'dataProvider' => $newitems,
                           'itemOptions' => ['class' => 'col-sm-4 featured-item'],
                           'itemView' => '_item',
                           'pager' => ['class' => ScrollPager::className(), 'item'=>'.featured-item', 'enabledExtensions'=> Array( ScrollPager::EXTENSION_TRIGGER, ScrollPager::EXTENSION_SPINNER, ScrollPager::EXTENSION_NONE_LEFT, ScrollPager::EXTENSION_PAGING ) ],
                           'summary' => false
                      ]);

And this is my _item.php:
  echo "

      $title

       $model->text

";

The 'Load More Items' button does seem to feed in when scrolled to the bottom, but shouldn't the plugin automatically load the new items when scrolled to the bottom?

triggerOffset' sets the amount of times for the scrolling to work. The default is 0` and must be set to a higher value for new items to be loaded on scroll.


[BUG] Logger System Unused

Observed Behavior
The debugger system is unused.
Expected Behavior
Making mistakes such as trying to move the chassis prior to calibrating should be logged appropriately.
How to Reproduce

Configure chassis
Move chassis without calibrating it

Project Info
Pros Kernel: 3.8.0
LemLib version: 0.4.5
Additional Libraries: None
Errors / Screenshots / Code
None
Additional Notes
Some examples of when the logger should be used:

Turning to face a point very close to the robot [Warning]
V5 device command failed to run successfully [Error]
Chassis sensor calibration failed [Error]
Odom configuration error (e.g 2 tracking wheels in same position) [Error]
Failed to read SD card [Error]
Improperly formatted file [Error]
Same motors on left and right side drivetrain [Error]

and many more situations that should be addressed in a PR

Merge the PR, it's done iirc

Implemented in master branch


Add an uber-department

To avoid name and department clash issues, RuboCop decided to grant each extension its own department.
More info https://github.com/rubocop-hq/rubocop/pull/8490

Before submitting the PR make sure the following are checked:

[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[ ] CHECK ONCE AGAIN, e.g. Rails/PersistenceCalledOutsideExample
[-] Added tests.
[-] Updated documentation.
[-] Added an entry to the CHANGELOG.md if the new code introduces user-observable changes.
[ ] The build (bundle exec rake) passes (be sure to run this locally, since it may produce updated documentation that you will need to commit).

@pirj I force-pushed to this branch, but the build is still failing. I’ll have a look at it later today.

the build is still failing. I’ll have a look at it later today.

I think I fixed it :-)

Thanks! LGTM.

Squashed into 2 commits.
Added Changelog entries.
Changed PR target branch.
Specs are passing. 🎉

I think we’re ready to merge.

@jesperronn We have a comprehensive document describing all the breaking changes in the 2.0 release and the migration procedure, you may check https://github.com/rubocop-hq/rubocop-rspec/pull/1013 if you're interested.


feat(node): print peer ID and addr on startup

This just prints the peer ID and local address on startup, so I can use it to test connections with geth

Codecov Report

Merging #655 (b5fba92) into main (bec1937) will increase coverage by 0.08%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #655      +/-   ##
==========================================
+ Coverage   72.11%   72.19%   +0.08%
==========================================
  Files         260      260
  Lines       26315    26315
==========================================
+ Hits        18976    18999      +23
+ Misses       7339     7316      -23

Flag
Coverage Δ

unit-tests
72.19% <0.00%> (+0.08%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

bin/reth/src/node/mod.rs
0.00% <0.00%> (ø)

crates/net/network/src/manager.rs
47.25% <0.00%> (+0.23%)
:arrow_up:

crates/transaction-pool/src/pool/txpool.rs
58.21% <0.00%> (+0.51%)
:arrow_up:

crates/transaction-pool/src/test_util/mock.rs
60.37% <0.00%> (+6.66%)
:arrow_up:

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


Remove 'using namespace std;' from header, (re)move some unused includes

This PR removes using namespace std; from simplomon.hh so that .cc files that include that header can still decide for themselves what parts of the std namespace they want.
It also moves some #includes that can be in the .cc rather than the .hh, making it a bit faster to compile simplomon.

This is more pain than it is worth by now, sorry. Will remove the using namespace std from simplomon.hh.


Create a logo for the website

This issue depends on issue #1
Create a logo on the basis of the suggested name and update it in the project.

@ashutoshkrris please assign me this issue.

Hi @AnushkaSingh91
Thanks for your interest.
This issue depends on the issue #1. So, you might need to wait for that to be resolved. If you are okay with it, I'll assign the issue to you.

@ashutoshkrris I'll surely wait for #1 to get resolved, please assign this issue, it will give a boost to my Hacktoberfest journey.

Sure @AnushkaSingh91

PLEASE ASSIGJN ME THE TASK

PLEASE ASSIGN ME THE TASK

Hi @rishabhknowss
@AnushkaSingh91 is already assigned and working on this issue.
You can look for other issues in the repository to work on.

@ashutoshkrris I have created the logo according to the given name provided, please select from the following.
And can you please brief me about where to add the final logo.


Why permission denied on ..?

~/D/cross-class-list ❯❯❯ ..
zsh: permission denied: ..

you deleted the above folder, and it no longer exists, which causes this error.

this can't be. is there no alias for .. = cd ..?

There is.

In which module? When I list alias, I do not see it there.

Never mind, I stand corrected. run setopt and paste your output.

~ ❯❯❯ setopt
alwaystoend
autonamedirs
autopushd
autoresume
nobgnice
braceccl
nocaseglob
cdablevars
nocheckjobs
noclobber
combiningchars
completeinword
extendedglob
extendedhistory
noflowcontrol
histexpiredupsfirst
histfindnodups
histignorealldups
histignoredups
histignorespace
histsavenodups
histverify
nohup
incappendhistory
interactive
interactivecomments
longlistjobs
monitor
nonomatch
pathdirs
promptsubst
pushdignoredups
pushdsilent
pushdtohome
rcquotes
rmstarwait
sharehistory
shinstdin
zle

because you don't have setopt AUTO_CD (you mentioned disabling it earlier) you need to alias '..' to 'cd ..'

Again, this isn't a prezto issue, this is a general zsh thing. This issue should be closed.

ok, thanks - all good now


The  key words of mx.symbol.Variable()

In the document http://mxnet.io/api/python/symbol.html#mxnet.symbol.Variable, It says that Variable() has a key word dtype. But in the Python implement code, there isn't.

Maybe you have the wrong MXNet version, but in the latest version, it is just here.

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!


get_elements fails on empty SpatialData

When SpatialData has no elements and I select no elements from it, I expected it to be a valid operation. However, an error is raised:
from spatialdata import SpatialData
import spatialdata_plot
SpatialData().pp.get_elements(elements=[])
# ValueError: SpatialData object must have at least one coordinate system to generate a mapping.

This is because the implementation tries to get coordinate system names, which are bound on the existence of elements.
While it is unlikely users will want this rare use case, when I test my own code systematically with a matrix of edge cases, this issue makes my tests fail and I have to add workarounds.

An alternative solution is to remove the exception in _get_coordinate_system_mapping. I didn't want to touch it first, but when I think about it, an empty mapping should be a valid return value.

I would have preferred to add the test case as an additional parametrization of test_can_subset_to_one_or_more_images (then we would also need to rename it to …zero_or_more…). Unfortunately, it cannot use the more flexible sdata fixture with param="empty", only get_sdata_with_multiple_images. It is also confusing that the fixture value (called "sdata") is a function that needs to be called to get "sdata".
It's an interesting workaround to the problem of selecting a fixture by name and parametrizing it, since the sdata fixture only takes a string as request param. My approach to multiple parameters in requests is to use a dictionary. That way we could rewrite the first test function to:
# conftest.py
@pytest.fixture
…
def sdata(request) -> SpatialData:
    request.param: dict
    name = request.param.pop("name") # TODO: here we would need to provide a default
    if name == "full":
        …
    else:
        s = request.getfixturevalue(name)
        if callable(s):
            kwargs = request.param
            s = s(**kwargs)
    # print(f"request.param = {request.param}")
    return s

# test_pp.py
@pytest.mark.parametrize(
    "sdata, keys",
    [
        (dict(name="empty"), []),
        (dict(name="get_sdata_with_multiple_images", share_coordinate_system="all"), "data1"),
        (dict(name="get_sdata_with_multiple_images", share_coordinate_system="all"), ["data1"]),
        (dict(name="get_sdata_with_multiple_images", share_coordinate_system="all"), ["data1", "data2"]),
    ],
    indirect=["sdata"]
)
def test_can_subset_to_zero_or_more_images(sdata, keys):
    """Tests whether a subset of images can be selected from the sdata object."""
    clipped_sdata = sdata.pp.get_elements(keys)

    assert list(clipped_sdata.images.keys()) == ([keys] if isinstance(keys, str) else keys)

Note that the "sdata" is now not the name for the get_sdata_with_multiple_images but it is the actual sdatafixture, which give us more flexibility choosing and parametrizing other fixtures.
But maybe there is even another approach.

Hi @aeisenbarth, I leave the conftest.py question to the others and I comment the first two posts.
I have just added an API that is similar to get_elements() (ideally a refactored version of get_elements() would be upstream, see here https://github.com/scverse/spatialdata-plot/issues/138), please check it out, maybe it's what you need: https://github.com/scverse/spatialdata/pull/426.

Agreeing with Luca here, moving get_elements() upstream (and with it the entire pp accessor) was on our roadmap for quite a bit now, just with low priority. I think I wouldn't really be worth fixing it here for now.

I believe now get_elements() is not used anymore, is this correct? In that case the bug reported maybe be gone.

Yes, for myself, this is obsolete now and can be closed.

Yes, for myself, this is obsolete now and can be closed.


Rename PrometheusExporterHttpService to PrometheusExpositionService

.. because exporter and exposition are different things, as documented
in:

https://prometheus.io/docs/instrumenting/writing_exporters/
https://prometheus.io/docs/instrumenting/exposition_formats/

Thanks for reviewing!


Migrate naming rules atests to new framework

Codecov Report
Base: 97.08% // Head: 97.08% // No change to project coverage :thumbsup:

Coverage data is based on head (d7f3126) compared to base (8977d64).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #719   +/-   ##
=======================================
  Coverage   97.08%   97.08%
=======================================
  Files          24       24
  Lines        3057     3057
=======================================
  Hits         2968     2968
  Misses         89       89

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Specifically ignore network address/port in git

We currently have a production and development server running. The development branch should always hit the development server and the master branch should always hit the production server. Unfortunately we don't currently have a way of allowing these specific addresses in the Network Manager to remain constant to their branch. We can't gitignore the file because we will want to make other changes to it.
Find out how to ignore Network Manager network address and port changes in the in git.

Ended up creating a new file with the network address and port that I would add to the gitignore separately. The issue with this is you can't add a file to gitignore that is currently being tracked, and to untrack a file will commit a deletion of it from the remote repo.
Going to need to go a different route with this, maybe some sort of environment variable. This starts to get into the build process which is the next thing I'm planning on addressing so I will add this address/port switching to the CI issue that I'm going to open.
Closing this issue unresolved since it's getting added to a new ticket.

Renaming this issue to be more generic. I don't think this will end up being a workaround with git.


How to set configs for multi classes?

Hello, wangguojun2018!
Thanks for your great work of CenterNet3d. The default config is for car class. But it also has potential for pedestrian and cylicst detection, can you provide a multi class config as in MMdetection3d.

I try to change some settings in default config, like num_classes=3 and change the base from 'kitti_3d_car.py' to 'kitti_3d_3class.py'. But it turns out that the network still only do detections on 1 class.


Add Font Awesome Icons to our Buttons - HTML not added in the editor

CatPhotoApp

Like

Info

Delete

Things cats love:

cat nip
laser pointers
lasagna

Top 3 things cats hate:

flea treatment
thunder
other cats

 Indoor
 Outdoor
 Loving
 Lazy
 Crazy

<button type="submit">Submit</button>

Challenge Name

Add Font Awesome Icons to our Buttons
Issue Description

The HTML which is rendered in the view is not present in the editor.
Browser Information

Browser Name, Version:
Operating System:
Mobile, Desktop, or Tablet:

Your Code

Screenshot

I assume that it is because you didn't add this:

on the top of your work

@mikah13: The font-awesome library has already been included behind the scenes so there is no need to include it here. Thanks for helping!
@bja09: From your screenshot it looks like you haven't added the the icon elements at all, much less the font-awesome classes. You can readily receive help/guidance on these challenges in the Help room. Happy coding!

@cdrainxv  Yes, indeed, but look at the html and check if I have a Like, Info or Delete button as HTML tags  + text , if you see  in the right of the screenshot what is rendered is a different thing than what's in the HTML. I don't have a place where to add he i tag for the icon element.

  <button class="btn btn-block btn-primary fa fa-thumbs-up">Like</button>


feature: playlist support

Currently, only favorite (playlist) can be played.
It would be really useful to have multiple playlists to add items and play from.

Yes, this could be a great feature, but it could become very complex to make because:

favorites saved will need to have some kind of menu to add them to a playlist
saving (exporting/importing) favorites can become more complex and "space consuming" because every single music can have playlists where its saved (and we should save playlists, too).

There is a lot of work required to make this working and I don't think I'll implement very soon, but this is surely something to add...

i successfully imported playlists from Youtube Music by exporting them as JSON and then slightly editing the file, for playlists from Spotify and other Music Platforms i used a Service that syncs playlists from-to different platforms and then exporting the Synced Youtube playlist as JSON.
Musicale successfully imported the playlists to my favorites although the posters/artworks aren't present.


Update Storybook to v5

Note: builds on top of #225 — I'll leave this as draft until that PR is merged.

What:
Updates Storybook to v5.

Why:
Storybook v3 is very old now, and I'm attempting to modernise the project ahead of contributing changes to the component itself. Storybook v6 required more substantial changes, so I've left that for another PR.

How:
Fairly straightforward upgrade to Storybook v5, which also required an update to vue-loader v15. There's currently a checked-in version of the generated docs, so I've updated those files too.
I also updated css-loader, though that wasn't strictly necessary.

Checklist:

[ ] Documentation N/A
[ ] Tests N/A
[x] Ready to be merged
[x] Added myself to contributors table

hey thanks @djw for the work! I have not been maintaining this repo so appreciate the help. Would you be interested in helping me as a co-maintainer? I can try and get to your PR for review but could be good to have some extra help triaging tickets and working on new releases.
For this PR specifically, I am worried about the react-syntax-highlighter_languages_highlight_ deps, there's so many of them and I imagine we don't need all the languages (especially all of them checked into source control). Looks like v6 tree shakes them https://github.com/storybookjs/storybook/issues/9282. I was actually thinking of removing storybook because I didn't always find it super helpful and was more of a chore to maintain. Seems like you get some value out of it so happy to keep it. If we can't remove all of the built output files then I can approve and we can check in for now and hopefully remove when we migrate to v6.

Sorry, I decided to switch to another package. Will close this!


sum of a column of all the pages of a filter

I am looking for the sum of a column of all the pages of a filter, I managed to obtain the sum of the current page:
I manage to pass the sum of the total of the records of the table in JSON parameter without worries, but if I add a filter, for example by date, I cannot find how to recover the sum of the lines concerned by the filter ...
Datatable load page by page in AJAX therefore impossible to calculate in JS and on the server side it is the datatable which takes care of the filter ...
I can't figure out how to tell him to give me that filter sum ... has anyone ever done that?
I followed :
https://datatables.net/examples/advanced_init/footer_callback.html
But with :
// Total over all pages
total = api
.column (2, {filter: 'applied'})
.data ()
.reduce (function (a, b) {
return intVal (a) + intVal (b);
}, 0);
I don't have the sum of the total with the filter.
Server side:
-> with ('total', $ datas-> sum ('amount')) returns the total of all the rows in the table but not the current filter
return Datatables::of($datas)
            ->editColumn('created_at', function ($datas) {
                return $datas->created_at->format('d-m-Y');
            })
            ->filterColumn('created_at', function ($query, $keyword) {
                $query->whereRaw("DATE_FORMAT(created_at,'%d-%m-%Y') like ?", ["%$keyword%"]);
            })
            ->with('total', $datas->sum('montant'))
            ->make(true);

An idea ?

I think what you need is the withQuery api. See https://yajrabox.com/docs/laravel-datatables/master/response-with#query-closure for ref.

This code block may help you to did the trick
->with('total', function() use ($datas) {
return $datas->sum('montant');
})

I have the same problem, I can show the value of the sum but if I do a search it does not update the value, I look for something like this in laravel: https://www.webslesson.info/2019/07/how-to-calculate-total-of-column-in-datatable-using-php-with-ajax.html but I have not been able to adapt it, any idea?


Docs inconsistent on whether a resource tag is required

Compare:
Note: Beginning in Windows 10, version 1803, the Resource element can be omitted.
(from the docs for Resources)
with
Every package must declare at least one language for resources
(from the docs for Resource)
Both statements can't be true.

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

ID: 3f798e27-5c18-7611-cb89-e2d7bc580b77
Version Independent ID: 72314c01-61a2-7405-1a0c-9c9f21495f09
Content: Resource (Windows 10 package schema) - Windows UWP applications
Content Source: winrt-related-src/schemas/appxpackage/uapmanifestschema/element-resource.md
Product: uwp
Technology: winrt-reference
GitHub Login: @drewbatgit
Microsoft Alias: drewbat

Thanks for your feedback. This page has been updated to remove the contradictory note.


add the -l flag to find_changelist

Currently it truncates the changelist description and wont find changelists with long descriptions

fixed in 0.3.10


#862 Provide component and version

This is just a suggestion to solver the issue with different versions of components.
This is not the full task.
If it's decided that we should do this, we can go on and update the other json files.

Closing in favor of AMP Start 2.0


Remove the dependency of Guava

Since Chimera is a library, we should not make a dependency on Guava.

Closed in #62


Split dcos-adminrouter-reload.service between masters and agents

Previously it was failing every run because the nginx reload pointed to a
non-existent configuration file. This makes it point to the master configuration
on masters, and agent configuration on agents.
Should unblock #551
cc: @branden @jgehrcke @darkonie

failed integration tests are either flaky tests or AWS having issues currently :/

:ship: . If you click through the status checks they're all green.

:+1:


[Bug]: Note review pane not showing notes for future dates

Describe the bug
This issue seems inconsistent, and only appeared since the last update (1.12.6)
I'm not sure if this one is only a problem on my end.
By default, all the notes I need to review for 'today' are shown correctly.
But any notes for 'tomorrow', other dates, and 'new' are not shown.

Note that 'New' and 'Thursday' are expanded, but empty.
However, (randomly) opening different notes causes some of the empty notes to show. Sometimes.

Note that 'New' is now visible, but 'Thursday' is still empty.
Both the requirement of number of notes/specific notes to open before the empty list starts showing seems to be different each time.
Which of the expanded segments of the review pane shows up also seems to be semi random. The first few I keep expanded seems to get priority, but it seems to not work.
To Reproduce

Tag a note to mark it for note review
Review it as hard, good, or easy (doesn't matter what)
Check the date for when it will come up again
Close the note/switch to another note
Restart Obsidian
Go to review pane
Try expanding the date from (3)

Expected behavior
All the notes for future dates being visible from the start, or being consistently regenerated by resetting the review pane or something.
Screenshots
No response
OS of your device
Win10
Obsidian version
1.6.7
Plugin version
1.12.6
Installer version
1.6.5
Additional context
No response
Config file
data.json
This issue isn't file specific, but I'm uploading the file I used to reproduce this on a new note
The tag #BT is used here to identify this note for note review
The content is irrelevant
Untitled.md

Hello, I am having the same issue. Only notes scheduled for the current day are shown correctly. I am experiencing roughly the same behavior as OP. Whenever I explicitly open a note that should be shown, it will then appear in the dropdown.
I have noticed that whenever the SR Sync operation is done (whatever function creates this console message in debug mode "SR: Sync took 12ms"), the note review pane will refresh. Meaning, that if I find the files to fill out the dropdown by clicking on the file, they will disappear again (the bug will occur) on that SR: Sync operation.
In the console, I noticed this error message, but I'm not sure if it is related or not.
 TypeError: this.noteReviewQueue.reviewDecks is not iterable     at ReviewQueueListView.redraw (plugin:obsidian-spaced-repetition:15426:56)     at eval (plugin:obsidian-spaced-repetition:15403:70)     at e.tryTrigger (app.js:1:723190)     at e.trigger (app.js:1:723123)     at t.trigger (app.js:1:2283688)     at t.activeLeafEvents (app.js:1:2269215)     at l (app.js:1:522936)     at c (app.js:1:523052)
I was poking around in the console, but I'm not very familiar with JS/TS. I was just looking at the error though and changed line 15426 to (adding the underscore to reviewDecks)
    for (const [deckKey, deck] of this.noteReviewQueue._reviewDecks) {
that fixed the console error message, but not the issue. could be 100% unrelated, but figured I'd mention

I have the same bug and did a bit of debugging, just on why the 'New' cards aren't showing.
This line: https://github.com/st3v3nmw/obsidian-spaced-repetition/blob/master/src/gui/review-queue-list-view.tsx#L93 seems to evaluate to false for the cards I am expecting to render but are missing.
Because it evaluates to false and is passed into the parameter 'hidden', the card is added to the list but has display: none set to it.
t("NEW") evaluates to 'New' with my current locale settings and deck.activeFolders contains ['#review', 'Today']. This is despite in my GUI I can see the 'New' and 'Yesterday' folders are expanded (with the arrow to the left of the title pointed down, instead of right). Maybe 'activeFolders' isn't meant to imply they are expanded, but I'm unsure.
So a theory could be that 'deck.activeFolders' isn't being populated properly -- but if files are re-appearing when we navigate to them, this wouldn't make sense.
As a horrific workaround, when I use the SR plugin I am opening the dev tools and deleting this snippet of code:
https://github.com/st3v3nmw/obsidian-spaced-repetition/blob/master/src/gui/review-queue-list-view.tsx#L208C1-L210C10 so that all cards appear all of the time. It breaks expanding the items, but I'd prefer that to this bug.

Updating to version 1.30.0 broke the plugin (It completely disappeared despite restarting or re-enabling it).
However, it was fixed when I reinstalled the plugin. And now the note review pane is displaying all the dates as well! The oldest technique in technology lol.
@wpasc @jtaylor100 If you still have the issue try reinstalling the plugin.


initialize commonLbConfig in default cluster

[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[X] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

@ramaraochavali: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests_istio
19f4dc46811453c6a2911a1139bfdbe072645013
link
true
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test unit-tests_istio

@ramaraochavali: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

gencheck_istio
19f4dc46811453c6a2911a1139bfdbe072645013
link
unknown
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test gencheck_istio

@ramaraochavali: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

gencheck_istio
19f4dc46811453c6a2911a1139bfdbe072645013
link
unknown
/test gencheck_istio

integ-pilot_istio
19f4dc46811453c6a2911a1139bfdbe072645013
link
unknown
/test integ-pilot_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test integ-pilot_istio

@ramaraochavali: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-multicluster_istio
19f4dc46811453c6a2911a1139bfdbe072645013
link
unknown
/test integ-security-multicluster_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test integ-security-multicluster_istio

@ramaraochavali: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-multicluster_istio
19f4dc46811453c6a2911a1139bfdbe072645013
link
true
/test integ-security-multicluster_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test integ-security-multicluster_istio


com.squareup.SocketRocket.NetworkThread(18): EXC_BAD_ACCESS

[x] Review the documentation: https://facebook.github.io/react-native
[x] Search for existing issues: https://github.com/facebook/react-native/issues
[x] Use the latest React Native release: https://github.com/facebook/react-native/releases

Environment
React Native Environment Info:
System:
OS: macOS High Sierra 10.13.6
CPU: x64 Intel(R) Core(TM) i5-5257U CPU @ 2.70GHz
Memory: 611.18 MB / 8.00 GB
Shell: 5.3 - /bin/zsh
Binaries:
Node: 9.5.0 - /usr/local/bin/node
Yarn: 1.6.0 - /usr/local/bin/yarn
npm: 6.2.0 - /usr/local/bin/npm
Watchman: 4.9.0 - /usr/local/bin/watchman
SDKs:
iOS SDK:
Platforms: iOS 11.4, macOS 10.13, tvOS 11.4, watchOS 4.3
IDEs:
Xcode: 9.4.1/9F2000 - /usr/bin/xcodebuild
npmPackages:
react: 16.4.1 => 16.4.1
react-native: 0.56.0 => 0.56.0
npmGlobalPackages:
create-react-native-app: 1.0.0
react-native-cli: 2.0.1
Description
Crash randomly when run on iOS, the cause is when WebSocket's delegate callback, the delegate already released, leads to EXC_BAD_ACCESS.
More detail can see #6117.
Reproducible Demo
Crash randomly.
potential fix PR
#19489

I had updated the project from 0.56 to 0.57 today! God bless me    O(∩_∩)O

It looks like you are using an older version of React Native. Please update to the latest release, v0.57 and verify if the issue still exists.
The "⏪Old Version" label will be removed automatically once you edit your original post with the results of running react-native info on a project using the latest release.

Still crashed on V0.57!!!!

@hramos Apologize if disturb, I see you already imported #19489 , any progress about this? 🤔

Left some comments in #19489.

no update on this issue, can we safely add this PR to our code ? if not what can we do to avoid the many crash caused by it ? 👍  Thanks a lot.

FYI: latest proposed fix is at #22439 (which is referenced above).


Properly override setWebOptions method

Web options were properly implemented inside, but since the interface method was not overriden, it could not be called properly.

thanks for the contribution. can you update the package-version also and update the changeling. I will merge this after that and publish this.

@balvinderz version bumped and changelog updated

Thanks for contributing :)

https://pub.dev/packages/video_player_web_hls
Publish 1.2.1


Discord Bot

Tasks

[x] List
[x] Chat
[x] Find
[x] Servers
[ ] Reload
[x] Info

Removed reload

Will be with version 4.0.0


[Bug] Minor - Email "Добре дошъл в Подкрепи.бг" - the link for Discord server in unreadable

Describe the bug
In the email "Добре дошъл в Подкрепи.бг" the link for Discord server in unreadable because it is blue on blue background
To Reproduce
Steps to reproduce the behavior:

Cоmplete the flow and register as  "Доброволец"
Open the email send in the end of the registration
Observe that the link for Discord server in unreadable because it is blue on blue background

Expected behavior
he link for Discord server in readable
Screenshots

https://github.com/podkrepi-bg/api/pull/252


message_row: Extract selectors from .message_row

This PR makes a significant set of changes to the structure of CSS in message_row.css. It flattens a number of selectors, and reduces the .message_row selector only to selectors that represent additional classes.
A follow-up PR to this will further flatten the selectors inside of .include-sender and .messagebox-content, but that's too much churn to add into this already quite massive refactor.
Screenshots and screen captures:

Before
After

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[x] Calls out remaining decisions and concerns.
[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[x] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

Merged, as the only feedback I had was a small readability detail that can be fixed forward, and I'd like to integrate code moves as fast as possible to manage merge conflict risk.


Add script to handle dropdown menus

Closes #27, which is one of the issues we needed to solve in #57.

I hope you don't mind me merging into submodule_refactor instead of version-2-0-0. This depends on stuff in there.

@JonasUJ I've already merged the submodule_refactor branch into master, so there is no need to target it with this PR.

Oh, then what is the version-2-0-0 branch for?

It was the branch used by #58

It was the branch used by #58

I've gone through and removed some of the merged branches now, to prevent further confusion.

I wish there was a more Pythonic way to write this

I feel you and I actually didn't mean for it to be like that. I saw that I redefined handle_event on every iteration, and without even  thinking more about it moved it above the loop. If i had noticed the hide() and show() calls in the function, I wouldn't have expected this to work, but since it did I actually didn't notice this horror before you pointed it out.

I wish there was a more Pythonic way to write this

I feel you and I actually didn't mean for it to be like that. I saw that I redefined handle_event on every iteration, and without even thinking more about it moved it above the loop. If i had noticed the hide() and show() calls in the function, I wouldn't have expected this to work, but since it did I actually didn't notice this horror before you pointed it out.

So, callbacks will have access to this, right, but would element be available then inside this? In that case, we could just define show and hide at the top, they can take element as a parameter, and handle_event can grab element from this and pass it into the function calls?
I'm not sure that will work but.. maybe?

The sad thing is that if they just included this one CSS rule in Bulma, it would have almost the same behaviour and there would be no need for 40 lines of js
.dropdown:not(.is-hoverable) .dropdown-trigger:focus-within + .dropdown-menu {
    display: block;
}

The sad thing is that if they just included this one CSS rule in Bulma, it would have almost the same behaviour and there would be no need for 40 lines of js

Bulma is quite receptive to pull requests - you could always open one to them and see what jgthms says?


Cursor get stuck after CC number

If you type your CC, the cursor goes to the expiration date. Then if you delete the expiration date, your cursor jump back to the end of CC. Now you are stuck, when you try to type anything, nothing is happening. Same for Expiration date too, after you comeback from CVC.

@pashan I haven't been able to reproduce this behavior. What options do you use to display the view controller? Also any context about the device (what kind of iPhone/iPad, locale settings)? When I hit the delete/backspace key on the numeric pad, it stops at the field I'm in (e.g. when deleting the expiration date, it stays in the expiration date and doesn't jump to the CC text field).


Open source the SqlServer module

Please consider open sourcing SqlServer module. It would be a great way to build a strong community around it and accelerate its growth and development.
Benefits (but not limited to):

A wider pool of potential contributors and users, which can lead to more bug reports, feature requests, and contributions.

The ability to leverage the expertise of the open-source community to improve and maintain the solution.

The opportunity to give back to the community and contribute to the advancement of technology.

Let's discuss the potential benefits and any concerns you may have to evaluate this further.

hey ronas! I saw a blog post recently that stated they are indeed planning to open source this module and it's coming soon 🥳

yes, this is planned for v23, as per roadmap...


Add Logstash to the Marketplace UI and ARM template

This PR adds Logstash to the Marketplace UI and ARM template. This is an initial implementation, with possibility to expand in future based on feedback and usage.

Provide the option to deploy a single instance of Logstash, with configurable VM SKU. The default will remain the same and not deploy Logstash.

Allow configurable JVM heap size.

Allow a Base-64 encoded string form of a Logstash conf file be provided, to start Logstash with. Expose variables that can be referenced within the conf file

${ELASTICSEARCH_URL}: the Elasticsearch endpoint
${LOGSTASH_SYSTEM_PASSWORD}: password of the built-in logstash_system user
${ELASTICSEARCH_CACERT}: the path to the CA cert used to secure the Elasticsearch

The value is Base-64 encoded to match how the value is encoded when received from an uploaded file within the UI.

Allow Logstash keystore password to be specified, for use in Logstash versions that have keystore functionality.

Allow additional Logstash plugins to be installed.

Allow additional YAML configuration to append to logstash.yml configured by the template.

For Logstash 6.3.0+  and for Logstash < 6.3.0 where X-Pack plugin is installed, configure X-Pack monitoring. Document whether monitoring is enabled based on if and how TLS is configured for the Elasticsearch HTTP layer.

Closes #157

Before this PR is merged, a 6.3 branch must be created and the elastic.co docs set to pick up documentation from that branch. This then makes the way to create a 6.4 branch which will include documentation about Logstash (in addition to subsequent PRs that will add support for 6.4.0 and 6.4.1).

@jsvd I've requested a review 😃


Two types of asynchronous validation

Hi! Thanks a lot for your form. I would like to make two different async validations: 1/ for submitting the form 2/ blur some fields. For async fields validation it's required another url and another response processing. How could I identify that validation was triggered due to field blurring? Thanks in advance!

I would recommend doing the submit validation when you actually submit the form. If the promise returned from the function passed to handleSubmit is rejected, the rejection reason can be your validation errors. Does that make sense?
render() {
  const submit = (values) => {
    return new Promise((resolve, reject) => {
      // the ajax call would go here
      reject({ name: 'That username is taken' });
    });
  };
  const {fields, handleSubmit} = this.props;
  return (<div>
      // ... field rendering here ...
      <button onClick={handleSubmit(submit)}>Submit</button>
    </div>);
}

Yep! Thanks for idea!

Could you please give more details about dispatching submit validation.
I made as you advised, but replace const submit with dispatch(validateForm(values)). validateForm invokes action, that looks:
case SET_VALIDATION_ERROR:
    const errors = {};
    const errorsData = action.payload.errors;
    if (errorsData.length > 0) {
      for (let el of errorsData) {
        errors[el.source.parameter] = el.title;
        errors.valid = false;
      };
    }
    return {
      ...state,
      errors
    };

But it doesn't work, the form doesn't show the errors. Could you plz advice how could I set form errors with dispatch(action). Thanks in advance!

You need to be returning a promise. Do you have a redux middleware that returns promises?

I use redux-effects-fetch.
The code of action:
import { fetch } from 'redux-effects-fetch';

function validateForm(data) {
  console.log(data);
  return bind(fetch(opt('formValidationUrl'), {
    method: 'POST',
    headers: {
      'content-type': 'application/json;charset=UTF-8'
    },
    body: JSON.stringify({user: data }),
  }), registerUser, setValidationError)
}

Should setValidationError also return the promise?

I had not seen redux-effects before. Interesting.
Here's how the redux-form submit validation works:

The handleSubmit() function that redux-form provides expects a (values) => Promise parameter. For you to get submit errors into redux-form that promise needs to be rejected with a {field1: 'error1', field2: 'error2'} shaped object.

What actions you dispatch inside that function is up to you.


Unify openshift cronjob deployment approach.

Current state:

delete-pvcs and import-images use own make file
validation is deployed via separate target in root make file

The separate target in the root make file is preferred.
Consider incorporating into the global deployment process (make deploy).

Another thing to improve:
@lbarcziova was having a problem that the validation job was at some point not able to download the image from Docker Hub.
The reason (correct me if I'm wrong) was that she'd built & pushed the image with podman.
So if the cluster has problems with podman built&pushed images, then we should enforce using docker in those makefile targets.

and/or report the podman problem is it gets resolved in future

and/or report the podman problem is it gets resolved in future

yes, it might actually be the same problem as https://github.com/packit/packit-service/issues/680

@jpopelka Looks like you want this opened.

Whoever will do this, please also change rebuild-base-image to run once a week and not every day.
So that my images are not rebuilding over and over again when I'm working on something for more days.

and/or report the podman problem is it gets resolved in future

https://github.com/docker/hub-feedback/issues/1871 looks like the same problem

latest comment in the hub tracker seems to indicate the images can now be pulled

latest comment in the hub tracker seems to indicate the images can now be pulled

validation is deployed via separate target in root make file

I don't see any such target in the main Makefile, only in cron-jobs/validation/Makefile.
So all the cron-jobs are being deployed by their own Makefiles. Closing
All the other comments are being resolved by building images via Github Actions and using Quay.io as image registry.