Fix compilation errors with deprecated sprintf function

This commit modifies sprintf calls to fix the following compilation
errors using XCode 14.0 beta on MacOS:

'sprintf' is deprecated: This function is provided for compatibility
reasons only.  Due to security concerns inherent in the design of
sprintf(3), it is highly recommended that you use snprintf(3) instead.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

This is deliberate.

It is statically known that these cannot overflow.
snprintf does not exist or is buggy on older MSVC versions.

@HansKristian-Work Thanks, so what would be the recommended way to fix the build failure on newer XCode version? disable the warning? use #ifdef?
Incidentally, not sure if there is some kind of consistency rules between KhronosGroup repos, but I noticed a similar commit was done for SPIRV-Tools: https://github.com/KhronosGroup/SPIRV-Tools/pull/4811

build failure on newer XCode version

Is it a straight up build failure? I guess you can ifndef _MSC_VER and comment about compatibility with old MSVC.

OK did the change, but i suspect we'd like to switch to snprintf at some point for MinGW too, if we can't disable the warning.
On macOS, the build failure is due to the deprecation warning.

Thanks for the review!

Thanks for the review!


Scrollify diseble on mobile.

Hi, I try to diseble Scrollify on mobile. Is it possible?

Use the disable method.

can you write code for disable on mobile.

This isn't a Scrollify issue. You should post your question on stackoverflow.com


⚠ Minecraft has degraded performance

In 8346581, Minecraft (https://www.minecraft.net) experienced degraded performance:

HTTP code: 200
Response time: 1038 ms

Resolved: Minecraft performance has improved in ac95b6d.


Helpers for calling FFI functions with "compatible types"

It would be awesome if the std::ffi module contained utilities which serves as common conversions from Rust types to C types in FFI functions. For example in git2-rs I have a macro which will call a function but convert each argument to it's "C equivalent" before doing so.
The conversion trait is pretty ad-hoc at the moment, but it encompasses a number of useful conversions such as:

&T => *const T
&mut T => *mut T
CString => *const c_char
Option<CString> => *const c_char (nullable)
Option<&T> => *const T (nullable)
bool => c_int

These sorts of conversions may not be appropriate for Rust at large, but the general idea is that types like Option<CString> are pretty painful to convert to a nullable pointer today:
let foo: Option<CString> = ...;
let ptr = foo.as_ref().map(|s| s.as_ptr()).unwrap_or(0 as *const _);

This issue is also motivated by rust-lang/rust#16035 because it would serve two purposes:

We could remove/deprecate as_ptr in favor of as_ref. This would break code such as the line above (because as_ptr would return a reference that would not be coerced to a raw pointer) but the old behavior could be opted into with this conversion trait.
The need to call as_ptr in the first place actually largely goes away as you can in theory pass the &CString to the C call directly (when using a macro like shown above).

More broadly this sort of helper or trait plays into the story of writing unsafe code in Rust (and improving the ergonomics thereof) and may have other considerations. I'm sure there are also quite a few variants of the trait I've got already floating around in other crates as well :). Regardless, it would be nice to have a standard method of converting to a "C compatible" representation and perhaps even a nice convenient macro.

cc @aturon

Yeah, I also wrote macros in my bindings to do these conversions and it was a pain. I didn't think to create a trait like you did, so my macros got ugly. I definitely think that would be useful.

This, or rather https://github.com/rust-lang/rust/issues/16035 must be fixed by beta, which suggests this RFC has to be approved and implemented in less than 3 days that’re left.


listing.map isn't a function

r.getHot(subreddit).then(listing => { subredditPost = (listing.map(submission => submission.url))                  message.channel.send(subredditPost[Math.floor(Math.random() * subredditPost.length)]);         });
This code was working fine earlier today. I was trying to get a random selection from an array of the hot posts from a subreddit. The error is "listing.map is not a function". I am super confused about this.
If it matters, the message.channel.send is from discord.js. I am using this for a discord bot.

Sorry, I'm not sure what's going on here. I can't reproduce the issue using the code that you've provided. I'd recommend double-checking your code to make sure everything else is doing what you'd expect (e.g. printing listing to make sure it's actually a listing).
If you're still not sure what's going on I'd recommend posting the full code on /r/redditdev.


Encounters until yachtzee choice

RE: Forum post here:
This is just what I have so far, I'm not totally sure if it's everything, first time I've done something like this.
I've heard I need to make a test, but am unsure of how to even start that. I mostly just fumbled my way through this copying how the NEP choice is done. Trying to work my way through that now.

Saw your forum post. On my phone,so can't look at code, but there is an array of String preference name in Preferences.java of preferences to reset on Ascension.

@jaadams5 it would have been good to hold off merging until Veracity or Chris had the chance to review, as I said?

@jaadams5 it would have been good to hold off merging until Veracity or Chris had the chance to review, as I said?

I reviewed their comments, did not see anything that I interpreted as meaning they wanted changes or that they intended to review again and noted that they had had 24 hours to respond to your observation.
It is quite possible that I missed some comment somewhere but since there were no workflow defined Requested Changes nor any unresolved conversations I figured people were done.  If you had retagged them or somehow indicated your review was contingent upon them I would not have pushed it.   Perhaps next time you will do more to make it obvious to others that  you don't want something pushed (don't approve and leave a change request would be things that jump out at me in GitHubDesktop) and I will wait longer before I push a non-Dev PR.  OK?

I couldn't have been clearer in my approval comment
On Sat, 9 Sept 2023, 14:26 Jamie Adams, @.***> wrote:

@jaadams5 https://github.com/jaadams5 it would have been good to hold
off merging until Veracity or Chris had the chance to review, as I said?
I reviewed their comments, did not see anything that I interpreted as
meaning they wanted changes or that they intended to review again and noted
that they had had 24 hours to respond to your observation.
It is quite possible that I missed some comment somewhere but since there
were no workflow defined Requested Changes nor any unresolved conversations
I figured people were done. If you had retagged them or somehow indicated
your review was contingent upon them I would not have pushed it. Perhaps
next time you will do more to make it obvious to others that you don't want
something pushed (don't approve and leave a change request would be things
that jump out at me in GitHubDesktop) and I will wait longer before I push
a non-Dev PR. OK?
—
Reply to this email directly, view it on GitHub
https://github.com/kolmafia/kolmafia/pull/1941#issuecomment-1712511874,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABL4LWAZBALVGKV3MTSMR3XZRU75ANCNFSM6AAAAAA4MPQNN4
.
You are receiving this because your review was requested.Message ID:
@.***>

FWIW @gausie I interpreted your comment exactly the same way jaadams5 did: "this is okay to merge, but check that the existing review comments have been fixed first".
If you wanted me or V to review it separately, I would have expected you to not approve it -- make your approval + comment merely a comment.


add static analysis tests

Pull Request Test Coverage Report for Build 738424391

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

Totals

Change from base Build 734128482:
0.0%

Covered Lines:
2982

Relevant Lines:
2982

💛  - Coveralls


hide/disable entries in the web interface

Hello,
I love this utility but I was wondering if it would be possible to hide/disable entries in the web interface so they can be kept as template or for future use without being deleted.
For instance, there could be a checkbox somewhere to disable it which would grey out the text and remove them from the export but leave them in the list?

Came here for the same thing. I like having the default actions present in the config UI to refer to, but I dont actually need them in the Nautilus context menu. Being able to disable them would be a great addition.

Hey there - you mean the delivered actions on first start? I think I could
a) put them in a submenu and
b) add a "disable" flag to submenus and items so that they stay in the
config but don't get parsed or displayed.
then you could disable that submenu when you start adding your own - and
disable any other that you might want to keep around but not use
Would that fit your requirement?
Thanks for using A4N
Martin
On Mon, Oct 7, 2024 at 2:45 PM Kieron @.***> wrote:

Came here for the same thing. I like having the default actions present in
the config UI to refer to, but I dont actually need them in the Nautilus
context menu. Being able to disable them would be a great addition.
—
Reply to this email directly, view it on GitHub
https://github.com/bassmanitram/actions-for-nautilus/issues/57#issuecomment-2396829102,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADAKLWVNUF5VLCJR6GW344TZ2J65TAVCNFSM6AAAAABPP3DHWWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOJWHAZDSMJQGI
.
You are receiving this because you were assigned.Message ID:
@.***>

Hi @bassmanitram, thanks for the quick response.

you mean the delivered actions on first start

Yeah, I think that's what I'm referring to (the actions that seem to be included by default) - screenshot included below to clarify.
I dont think it's necessary to hide them all within a sub-menu. In fact, I think doing so would make it more difficult to get familiar with this app. It's useful to have top-level examples of a command (e.g. the "Open in Gedit" command).
I think all we need is a bool/switch on each action in the web UI so we can enable/disable them from the Nautilus context menu. If an action is disabled, it's still visible in the web UI but will not appear in the Nautilus context menu. If it's enabled, it's visible in the UI and Nautilus context menu.
Looking back over the requirement @Batwam described:

hide/disable entries in the web interface

I think they mean something along the lines of what I'm describing, as opposed preventing the action from appearing in the web UI.

Screenshot for reference
These are the actions that seem to be created by default. They're useful for reference, so I'd like to include them in the UI but not show them in the Nautilus context menu.

Yeah, what I meant would be to have entries which are listed in the UI so they can stay visible (eg templates) but do not appear in Nautilus.
I haven't looked at how the underlying config file looks like but essentially something similar to commenting out a line of code where it's visible to the person setting it up but not actually actioned.


✨ Proposing new lint options

This PR proposes new lint options in preparation for upcoming CI checks. See issue #91.
Breaking changes
n/a
Fixes / Enhancements

Updated lint options in preparation for upcoming CI checks.

Docs
n/a

:tada: This PR is included in version 1.0.0-beta.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Interface Configuration missing property devServer

Bug report

What is the current behavior?
Using<EMAIL_ADDRESS>the property devServer is missing from the TypeScript interface Configuration. But it is a documented property and it works in v4.
If the current behavior is a bug, please provide the steps to reproduce.

What is the expected behavior?
The property should be present in the interface.

Other relevant information:
webpack version: 5.0.0-beta.17
Node.js version: v14.3.0
Operating System: Ubuntu
Additional tools: TypeScript

It's in the webpack-dev-server typings. Install and add them.

@mightyiam You can open an issue about improve typings in webpack-dev-server

Thank you. It is indeed an issue in @types/webpack-dev-server but only for webpack v5.


Multiple Tours on Page

If the page has multiple tours there isn't a way to clear the old ones, such as the tours get implemented based on the button/option a user presses. At the end of the first tour run through, the core code still has the array referencing the previous tour, and runs through all those options. How can I empty out the internal array on the TourTip?

I added the following to the line 219
publicMethod.reset = function () {
hideTip();
tourTipObjects = [];
$("[id^='TourTip']").remove();
clearTourTipElements();
}


Machine-identifiable placeholder values

From #12:
Minor quibble about using "REPLACEME": this is an arbitrary value that will be swapped out as Pipeformer runs and puts in the real user inputs, so couldn't it be more specific to 1) the data being populated (e.g. "SECRETS_MANAGER_SECRET_PLACEHOLDER") and/or 2) structured to be more clearly computer-readable placeholder input (e.g. "@@PLACEHOLDER@@")?

This could let you do a sort of type checking/validation that the input value is being swapped for the placeholder token expected and add some error checking bounds around token substitution.

We need to investigate this a bit more before committing to a path, but I think this is a good idea.
This will provide a mechanism to allow pipeformer to determine where in a deployment we have reached, as well as providing an option for determining whether we need to re-prompt users for input values on pipeline updates.

Closing down project: #52


Busybox automatically uses builtin commands, ignoring ones in $PATH

I've compiled GNU grep and sed and replaced the Busybox ones in my path, but calling them from the shell will always call the Busybox version unless I specify an absolute path.
Would it be possible to make Busybox give priority to executables in the path rather than its own builtins?

The shell in a default build of BusyBox for Windows always looks for a built-in applet before searching PATH. This preference for applets can be overridden by setting the shell variable BB_OVERRIDE_APPLETS (as described in the release notes for FRP-5181).
In the particular case above, setting BB_OVERRIDE_APPLET="grep sed" will unconditionally override the two applets and always search for a suitable alternative on PATH.
Alternatively, the setting BB_OVERRIDE_APPLET=";grep sed" will first look on PATH but if no matching executable is found it'll fall back to using the built-in applet.
It's also possible to configure this sort of setting at compile time, as mentioned in the FRP-5301 release notes.
For example, setting the build-time configuration OVERRIDE_APPLETS to the string + will result in a BusyBox shell which (by default) always searches PATH first and only uses a built-in applet if that search fails.

Thanks for the info!
Small typo, you wrote "BB_OVERRIDE_APPLET" (singular) in your usage examples. I'm specifying just in case somebody else comes across this issue and tries to copy-paste the examples.

Yes, I noticed the error myself and updated the comment.


Some(?) scripts don't read from stdin?

textkit text2words text.txt  ok
textkit text2words < text.txt  NOK
cat text.txt | textkit text2words  NOK
It's not always obvious how to use commands, as some expect file names, other accept streams, etc.
The documentation is not explicit on this point: "With no file passed in, many textkit commands default to standard in. This can be indicated explicitly by using a dash (-) to indicate standard in."
(I haven't found how to use text2words on STDIN.)

another example is that tokens2json does not work like tokens2counts


Spell out that running gitea with your own user is unsupported

I just wanted to run gitea in the easiest way possible, so I ran it with my own user only to find
that I could no longer log in with SSH anymore.

For added discoverability for people having the same issue (but not knowing that it is because of Gitea), maybe the error message should be included also, namely Server refused to allocate pty (Putty gave me this error, not mentioning Gitea anywhere)?

LGTM

It's a pity to not support running from own user, if we can document all the issues it's better, or is it only the ssh keys ? (that "regenerate keys" command could be made safer)

@sztanpet Can you also update the other language documents?

No sorry, I do not know the other languages

@sztanpet OK.

LGTM

Could you send another PR to back port to 1.0?

there it is

There is an open issue to resolve this issue. Until than we will have this phrase which is warning the users.


Handle disabled state gracefully

Describe the feature
nuxt-auth is working great in my project. However I'm using also histoire to build and test my components in isolation. In that case I set isEnabled: false for nuxt-auth in nuxt.config.ts
The problem is that in that case I get a bunch of errors everywhere where on the server side I have #auth import:
Could not resolve import "#auth"
Question: would it be possible to handle such cases more gracefully and avoid those errors, when the module is disabled?
Additional information
No response

I see similar issue has raised #454

Hi @peterbud!
So we do a bunch of testing in our projects as well. The way we approached it was to override the useAuth composable with our custom custom one made for testing.
This composable then also mocks a user account and returns demo functions that our testing suite can use.
You can add this into a testing file:
const pathToMocks = resolve('./nuxt-auth.ts')

nuxt.hook('imports:extend', (_imports) => {
  _imports.push({ name: 'useAuth', from: pathToMocks })
})

This will then overwrite the useAuth composable with the mocked one, allowing you to use it in testing.

Thanks for the reply and the help. Can you point me to an example where this is used somewhere?
When I have tried to add this hook to nuxt.config.ts, I got the following error:
Duplicated imports "useAuth", the one from "[...]mock-useAuth.ts" has been ignored and "[....]/node_modules/.pnpm/@sidebase+nuxt-auth@0.6.0-rc.1_next-auth@4.22.5_vue@3.3.7/node_modules/@sidebase/nuxt-auth/dist/runtime/composables/authjs/useAuth" is used

Can you point me to an example where this is used somewhere?

This is very dependent on your testing setup. I do not have much experience with historie, but I assume there is a setup file, similarly to how storybook does this. Inside this setup file you can then inject this new hook. You have to ensure it is only being used for your historie setup and does not override the actual composable in the app.


docstring standardization

Hello,
I've noticed the docstrings weren't following any convention so I made some changes and made them consistent.
if you want to follow a specific convention let me know and I'll go through the code again.
There was some text on the docstrings that was broken in a way that made it difficult to read.
I've added some newlines to help with that.
I've also added a tox env so you can build the docs using
$ tox -e docs -- html

I added some :param on some docstrings and fixed a few complaints from sphinx so the HTML documentation should also look better.
hope it will be useful.
cheers!

LGTM!

Can you rebase so we can merge?

Will this trigger build failures if the documentation is broken? That would be pretty cool!

@Tarrasch I've rebased my changes.

I think something went wrong when you rebased – mind trying again?

Also "fixed URL" might not be the best commit msg :)

other than that, LGTM

I'v tweaked the command to generate the docs since it was causing tox to throw a warning.
I've fixed a typo.
I've changed the first line of the commit message.

@erikbern it may be possible to have the build break if the docs generation fails.
I've tried to introduce some errors and make it fail but sphinx is only giving me warnings.
if I find a way I'll send another PR.
will that work?

@erikbern I thought you wanted me to rebase to squash the number of commits.
what do you mean by rebase, exactly?

I think I got it.
pull changes from luigi/master and merge the conflicts?

Yeah "git pull --rebase" or "git rebase master"
On Tuesday, February 3, 2015, Pedro<EMAIL_ADDRESS>wrote:

I think I got it.
pull changes from luigi/master and merge the conflicts?
—
Reply to this email directly or view it on GitHub
https://github.com/spotify/luigi/pull/693#issuecomment-72694822.

got it.

travis failed build due to 'pep8' not being present.
let me have a look...

I figured out that you guys run autopep8 to fix PEP8 warnings.
I've tweaked it a bit so it doesn't add a new line after class declaration.
I think it doesn't makes sense to have the blank line if the next line is the start of the docstring so I kept in on the pep8 check (which understands what is going on) but removed it from the autopep8.
I figure out how to make build break due to sphinx errors.
I'm preparing another PR for that.
I think the rebase is good.
let me know.

LGTM let's merge

Thank you very much! :)


Prowler versions later than 2.3.0RC3 skips scanning multi-region

When using Prowler versions later than 2.3.0RC3, the swtich -f 'us-east-1 eu-west-1...' no longer scan multiple regions correctly. Only one region is randomly selected from the specified regions to scan.

Hi @ssteo, option -f region is used to query just one region. I have tested it in latest code in master with this command ./prowler -f 'us-east-1 eu-west-1' -c check42 and it shows results from both regions correctly. To be honest I never tried regions separated with space and between single quotes (thanks!) but it works. Remember that for those checks that are global you won't see any difference.

Thanks for the response. Actually I wasn't sure if -f region supports multiregion initially but looking at https://github.com/toniblyx/prowler/blob/master/prowler#L236-L240 it appears to be able to handle space separated multiple regions.
Did you try with region specific resources? I was scanning EBS volume which exist in some regions but the scan results from the current master branch code is not showing some of the regions. So I reverted back to this commit 43d95ac18c2618ac5d0cb6e973934886bba29a40 and so far it works fine.

can you please share the exact command you are running to test it in my side? I don't see the link between the version label and regions...

Sure, here's the command used.
./prowler -b -q -f us-east-1 us-west-2 ap-south-1 ap-southeast-1 eu-central-1 eu-west-1 sa-east-1 -g pma -r eu-west-1 -A [aws_accound_id] -R [internal_iam_role_name] -M html,json

I have tested this one ./prowler -b -q -f 'us-east-1 us-west-2 ap-south-1 ap-southeast-1 eu-central-1 eu-west-1 sa-east-1' -c check41 -r eu-west-1  -M html,json with latest code in master and works fine for those regions with resources, in this case security groups. Have you used single quotes?

@toniblyx yeah, there's single quote in my command too. Sorry, I've accidentally left out the single quote when pasting in GitHub comment earlier.

I'll try again later with the latest master branch code whenever possible. Thanks


[error] [gradle-server] Gradle version is required

Extension Name: vscode-gradle
Extension Version: Private build based on PR #919
OS Version: Windows 10
VSCode version: 1.59.0
Describe the bug
Nothing is working, except a message "Gradle: Build Error" on status bar.
To Reproduce

use spring initializr to create a simple gradle project.
open it, and you see the screenshot

Expected behavior

I expect it's working out-of-box, but I don't see much insightful message from the log after I click the status bar item.
the "Gradle: Build Error" is not obvious, so I spend time wondering what's happening. Maybe a warning/error color is better?
the "Gradle: Build Error" stays there forever. How do I get it resolved?

Screenshots

Output from "Gradle Tasks"
[info] [gradle-server] Server started, listening on 59365
[info] Gradle client connected to server
[error] [gradle-server] Gradle version is required
[error] Error getting build for c:\Users\admin\projects\demo-gradle: Gradle version is required
[info] Found 0 tasks

FYI, my settings:

after changing to use gradle wrapper, tasks are correctly listed, but dependencies are still not there, and the status bar error message is the same.
logs from output:
log.txt

after changing to use gradle wrapper, tasks are correctly listed, but dependencies are still not there, and the status bar error message is the same.
logs from output:
log.txt

This comes from circular dependencies and has been fixed in #923. For the Gradle local problem, I'll continue to investigate and fix

Here are some findings about this issue:
related discusstion: https://github.com/microsoft/vscode-gradle/issues/243#issuecomment-621174072
In vscode-java, there are a lot of Gradle related settings, they are:

java.import.gradle.home: The local gradle distribution path. See https://docs.gradle.org/current/javadoc/org/gradle/api/invocation/Gradle.html#getGradleHomeDir--
java.import.gradle.user.home: This directory is used to cache downloaded resources, compiled build scripts and so on. See https://docs.gradle.org/current/javadoc/org/gradle/api/invocation/Gradle.html#getGradleUserHomeDir--
java.import.gradle.version:  The gradle version will be used. If the wrapper is disabled or missing
java.import.gradle.wrapper.enabled: Use wrapper or not

In a short word, java.import.gradle.user.home is used to cache resources, and the other three settings are used to specific gradle distribution (wrapper, specific version or local distribution). In the reference: https://github.com/redhat-developer/vscode-java/blob/master/README.md#supported-vs-code-settings, the priority of the three settings is java.import.gradle.wrapper.enabled, java.import.gradle.version, java.import.gradle.home. If a more front setting is valid (has been set and with a valid value), the rest settings will be ignored.
Currently, vscode-gradle consumes the first two settings, java.import.gradle.wrapper.enabled, and java.import.gradle.version, with the correct priority. It doesn't support java.import.gradle.home currently, that the root cause of this issue.
So I plan to support java.import.gradle.home with a lowest priority among these three settings to support the local distribution scenario. Could you please provide any comments? Or is there any misunderstanding from me in the content? @Eskibear @testforstephen @jdneo

fixed in #926


Fix generated Update endpoints

This PR fixes #154 in which non-unique edges are cleared in PATCH requests if not specified. The expected behavior is that PATCH with an array replaces the existing members with the PATCH members, but does not change the members of the array if the field is not specified in the PATCH body.

Any idea if this will be merged?

Hey @agola11, thank you for the contribution and sorry for the relayed review.
Can you please run go generate and check in the missing files to git?


Use UNL to retrieve manifest

High Level Overview of Change

The xrpl web socket (http://xrpl.ws/) is only for mainnet and nodes from other networks are hit because the nodes are interconnecting. Pull manifests from individual unl using http request also (see https://xrpl.org/validator-list.html#request-format) to yield more comprehensive result.
Type of Change

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Refactor (non-breaking change that only restructures code)
[ ] Tests (You added tests for code that already exists, or your new feature included in this PR)
[ ] Documentation Updates
[ ] Release

Before / After

Manifests for validators on the list of networks would be updated. API calls to retrieve validators of chains such as amm that are not interconnect with mainnet would yield a non-empty list of validators.

Closed in favor of #57


Incorrect behavior keyboard_suggestions

Software Versions

Python: 3.8
OS: Android 9
Kivy: master
Kivy installation method: python-for-android develop with kivy==master

Describe the bug
Keyboard suggestions for TextInput do not work correctly on android. Words are inputted twice when you use a suggestion or swipe as you can notice when you delete the words or press enter. Setting keyboard_suggestions to false does not deactivate suggestions.
Expected behavior
Keyboard suggestions or swiping should input words correctly and it should be possible to deactivate the suggestions feature with keyboard_suggestions set to false.
To Reproduce
from kivy.app import App
from kivy.lang import Builder

kv = """
Screen:
    TextInput:
        size_hint_y: None
        height: dp(45)
        hint_text: "keyboard_suggestions: False"
        write_tab: False
        keyboard_suggestions: False
        pos_hint: {"top": 0.9}
    TextInput:
        size_hint_y: None
        height: dp(45)
        hint_text: "keyboard_suggestions: True"
        write_tab: False
        keyboard_suggestions: True
        pos_hint: {"top": 0.7}
"""

class Test(App):
    def build(self):
        return Builder.load_string(kv)

Test().run()

Gif

I'm wondering if it's due to https://github.com/kivy/kivy/pull/7231. Can you try a commit before that? Or maybe 2.0.0?
@dwmoffatt could you take a look?

I'm wondering if it's due to https://github.com/kivy/kivy/pull/7231. Can you try a commit before that? Or maybe 2.0.0?
@dwmoffatt could you take a look?

I'll take a look

I'll take a look

GBoard:

Issue confirmed when using swipe or suggestions.

Samsung Keyboard:

The first inputted character, keeps into TextInput.

Reverting #7231 fixed the issue.
It could potentially disrupt the UX for multiple users. @dwmoffatt any news?

I think I know what the problem is. There are some other variations/flags that need to get set for some of the autocomplete and suggestions to work properly based on how you are entering the data.
https://developer.android.com/reference/android/text/InputType
I haven't had a chance to try anything out yet. Hopefully next week I will have some time.

Update:
Looks like some device vendors and keyboards are de-facto ignoring the InputType.TYPE_TEXT_FLAG_NO_SUGGESTIONS flag, so we can't really rely on this one to disable suggestions.

I'd also add that for Gboard and other keyboards its built-in "Copy" and "Paste" buttons don't work: "Copy" is disabled even if text is selected, "Paste" just does nothing.
Probably "Copy" just can't detect that something is selected, but seems that "Paste" should work fine.

Hi,
I would like to help you in solving the poblem, since I need it to find a solution quickly.
May you tell me which files (and/or part of codes) I have to check and modify (including the event on keyboard) to try to find a solution?
Thank you

i have same problem
kivy 2.1.0
on Android [2 differend phones, 4y difference + gboard] if i go to next word it keep repeating last suggested on spacebar
iam too short to help with keyboard or even files where to look :(
i owe you beer if you fix it somehow :)

Kivy master
Python-for-android develop
No suggestion or swipe.

Fixed via https://github.com/kivy/python-for-android/pull/2692


fix: switch to pagefind's node api

With this change, pagefind will only print the total indexed pages, since it doesn't seem to print anything else to the console if not running as a binary.
Closes #76

Thanks for the PR. I have implemented this myself and will close it now.


Using parentNode.removeChild when element.remove is not defined

This fix popper.destroy method on browsers that doesn't support Element.remove method

@FezVrasta it's done :)

Also, it would be nice if you could remove the notice from the README about Internet Explorer.

@FezVrasta done again :)

merged


ARM release

Would it be possible to have a release for linux arm? Thanks

I'm not as familiar with the ARM side of go.. I notice there's a bunch of versions of it:
https://github.com/golang/go/wiki/GoArm#supported-architectures
Are they backwards compatible or would this add 4 new binary releases?

AFAIK they're pretty much backward compatible. I'm not that much familiar with the whole ARM ecosystem but ARMv5 seems very old, and was even considered for removal (https://github.com/golang/go/issues/17082).
I think a release with GOARM=6	GOARCH=arm should be fine for most ARM users.

@aelsabbahy I think it will work fine just to support the default ARM version, as suggested by Dave Cheney. Or at least until there is a huge demand for other ARM versions


Re running Server profile creation from template playbook is redploying the OS and updating the server profile each time.

Hi,
Server profile creation with OSDeployment setting from server profile template is updating the server profile and re deploying the OS from Image streamer each time i run the playbbook.
Below is the Server profile playbook.
This playbook is only compatible with HPE Synergy Enclosures

- name: Gather facts about the Server Profile Template to retrieve connection information
  oneview_server_profile_template_facts:
    config: "{{ config }}"
    name: "{{ server_template_name }}"
  delegate_to: localhost

- name: Find the connection by management network name
  set_fact: connection={{ server_profile_templates.0.connections | selectattr('name', 'match', management_network_name) | first }}

- name : "Create server profile with deployment plan {{ deployment_plan_name }}"
  delegate_to: localhost
  oneview_server_profile:
    config: "{{ config }}"
    data:
        name: "{{ inventory_hostname }}"
        server_template: "{{ server_template_name }}"
        osDeploymentSettings:
          osDeploymentPlanName: "{{ deployment_plan_name }}"
          osCustomAttributes:
            -  name: MGMT.connectionid
               value: '{{ connection.id }}'
            -  name: MGMT.dhcp
               value: true
            -  name: MGMT.ipv4disable
               value: false

-  name: MGMT.dns1
value: "<IP_ADDRESS>"
-  name: MGMT.domain
value: "etss.lab"
-  name: MGMT.dns2
value: '<IP_ADDRESS>'
-  name: MGMT.gateway
value: '<IP_ADDRESS>'
            -  name: interface_name
               value: ens3f4

-  name: MGMT.netmask
value: '<IP_ADDRESS>'
            -  name: fqdn
               value: "{{ inventory_hostname }}.etss.lab"
            -  name: dhcphost
               value: "{{ inventory_hostname }}"
            -  name: MGMT.networkuri
               value: '{{ connection.networkUri }}'
            -  name: MGMT.constraint
               value: 'dhcp'
            -  name: SSH_CERT1
               value: "{{ lookup('file', '~/.ssh/root_ansible1.pub') }}"
            -  name: SSH_CERT2
               value: "{{ lookup('file', '~/.ssh/root_ansible2.pub') }}"

-  name: ProxyExclusionList
value: bootstrap
-  name: ProxyHost
value: proxy.houston.hpecorp.net
-  name: ProxyPort
value: 8088
- name: Power on the server hardware
  oneview_server_hardware:
    config: "{{ config }}"
    state: power_state_set
    data:
      hostname : "{{ server_hardware.name }}"
      name: "{{  server_hardware.name }}"
      powerStateData:
        powerState: "On"
        powerControl: "MomentaryPress"
  when: server_hardware.powerState == "Off"
  delegate_to: localhost

- name: Wait for SSH connection to server
  wait_for: host="{{ ansible_host | default(inventory_hostname) }}" port=22 state=started timeout=6000
  delegate_to: localhost

After replicating the issue, I've noticed two causes for the updates:
1- After the Server Profile is created with a management connection, a MAC is assigned to that connection and it is also returned by GET operations. This makes it so that the following scenario occurs:

We are not passing in a something.mac in our playbook
We are receiving a something.mac when performing a GET in Oneview
Comparing both, the module thinks it should perform an update to remove the something.mac that was retrieved but is not a part of the playbook, which does not change the situation and starts a loop

2- The playbook has a couple of True/False values declared as follows:
                -  name: MGMT.dhcp
                   value: true
                -  name: MGMT.ipv4disable
                   value: false

This means that for Python, these are respectively True and False boolean values. However, in the OneView documentation it is specified that all values for osCustomAttributes are strings.
Snippet from OneView documentation:

value Identifies the value of the custom attribute - string

This means that we are passing in Booleans, retrieving Strings and the comparison between both is returning False as expected.
The correct way to solve this is to use the String version 'true' and 'false' for values inside osCustomAttributes.
However, since it is a more usual scenario for true and false values to be Boolean instead of String, this case could be exceptionally treated before the comparison, to make it so the user does not have to worry about this.


.NET 7.0 Preview 7

Release Notes
Please report any issues you find with .NET 7 Preview 7, either responding to this issue, creating a new issue or creating a new issue in one of the following repos:

dotnet/aspnetcore - for ASP.NET Core questions
dotnet/efcore - for Entity Framework Core questions
dotnet/sdk - for CLI tools and questions
dotnet/runtime - for runtime, API, or installer issues and questions
dotnet/winforms - for WinForms issues
dotnet/wpf - for WPF issues
nuget/home - for NuGet questions and issues

Known Issues
If there are any issues with this release we will track them here and check issues off as they're resolved. See the linked issues for details on progress and resolution details.

Still cannot build WPF projects using source generators coming from Nuget packages.
Although there is a workaround in known issues of release notes 6.0, none of dotnet/wpf, dotnet/sdk, dotnet/msbuild fix this.

In ITrigonometricFunctions<TSelf> Atan2 and Atan2Pi methods are no longer included.
We can implement it manually with some helper basing on Atan or AtanPi method, but it will not support hardware acceleration (unless the JIT does something). And for example, architectures such as x86 and AMD64 have the FPATAN instruction, which is a hardware implementation of Atan2.

I was assuming .NET 7 would be faster.  This code runs significantly slower in .NET 7 preview 7 compared to .NET 6.
I get .NET 6 - 650ms, .NET 7 - 850ms.
using MathNet.Numerics.IntegralTransforms;
using MathNet.Numerics;
using System.Diagnostics;
using System.Numerics;

Control.UseManaged();

var buffer = Generate
    .Square(2048, 32, 32, -20.0, 20.0)
    .Select(r => new Complex(r, 0))
    .ToArray();

var sw = new Stopwatch();
sw.Start();

for (int i = 0; i < 10000; i++)
{
    Fourier.Forward(buffer, FourierOptions.AsymmetricScaling);
}

sw.Stop();

Console.Write(sw.Elapsed.TotalMilliseconds);

@PhilPJL Please use Benchmark.dot.net for benchmarking C# code instead of a stopwatch. If you still see perf regression, please consider opening the issue at the runtime repo. https://github.com/dotnet/runtime

@ShreyasJejurkar
Interesting, with BenchmarkDotNet.  I get:
.NET 5: 630ms
.NET 6: 652ms
.NET 7: 554ms
So .NET 7 is faster. Good :)

Closing in favor of https://github.com/dotnet/core/issues/7800


Remove build dependencies from final image (multistage build)

This multistage build takes the resulting image size down from 346MB to
90MB.

Thank you, good idea!

Thanks!


Cannot get rules (HTTP 401)

The bug occurs when using the code example for Filtered-Stream. I have set the bearer token through the console as it mentions but I still receive a 401 which would hint towards me not being authorized. I don't know if I have done something wrong regarding the setting of the token, but I did this through the console in VSCode.
The actual error message I received was the one below. I'm not fabulous with Python or the API so I'm a little stuck.
Exception has occurred: Exception Cannot get rules (HTTP 401): {"title":"Unauthorized","detail":"Unauthorized","type":"about:blank","status":401}   File "C:\Users\Rhys\OneDrive\Documents\TwitterCrawl.py", line 25, in get_rules     raise Exception(   File "C:\Users\Rhys\OneDrive\Documents\TwitterCrawl.py", line 89, in main     rules = get_rules()   File "C:\Users\Rhys\OneDrive\Documents\TwitterCrawl.py", line 96, in <module>     main()

@RPBTwisted Did you find the solution to this?


[PINOT-4662] Remove deleted segments after N day retention

Add a job to the retention manager that removes old deleted segments from deleted directory.
Add unit test.

Codecov Report

Merging #1342 into master will increase coverage by 0.08%.
The diff coverage is 72.72%.

@@            Coverage Diff             @@
##           master    #1342      +/-   ##
==========================================
+ Coverage   66.84%   66.93%   +0.08%
==========================================
  Files         781      781
  Lines       38126    38165      +39
  Branches     5005     5010       +5
==========================================
+ Hits        25487    25544      +57
+ Misses      10976    10965      -11
+ Partials     1663     1656       -7

Impacted Files
Coverage Δ

...troller/helix/core/retention/RetentionManager.java
75% <100%> (+0.62%)
:arrow_up:

.../com/linkedin/pinot/controller/ControllerConf.java
46% <33.33%> (-0.4%)
:arrow_down:

...m/linkedin/pinot/controller/ControllerStarter.java
56.33% <50%> (-0.4%)
:arrow_down:

...ntroller/helix/core/PinotHelixResourceManager.java
55.79% <61.53%> (+0.04%)
:arrow_up:

.../controller/helix/core/SegmentDeletionManager.java
75.45% <80.95%> (+1.29%)
:arrow_up:

.../impl/dictionary/FloatOnHeapMutableDictionary.java
88.88% <0%> (-6.67%)
:arrow_down:

.../query/aggregation/groupby/utils/FloatToIdMap.java
68.75% <0%> (-6.25%)
:arrow_down:

...pby/NoDictionarySingleColumnGroupKeyGenerator.java
90.9% <0%> (-1.02%)
:arrow_down:

...com/linkedin/pinot/core/common/DataBlockCache.java
76.51% <0%> (+0.67%)
:arrow_up:

...lter/predicate/NotInPredicateEvaluatorFactory.java
82.69% <0%> (+0.96%)
:arrow_up:

... and 10 more

Continue to review full report at Codecov.

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


CampaignAdmin only shows preview (no item list) in ~800px window width

It makes the whole section unuseable in small resolutions. Should be the other way around: Hide preview, only show list.
/cc @tractorcow @clarkepaul

I'm working on a fix for this.

Fixed and merged by https://github.com/silverstripe/silverstripe-framework/pull/5405


ChatGPT like doc assistant on home page using ChatGPT plugin

Type
Suggestion to Improve
Feedback
Now there is a ChatGPT plugin, it may be interesting to have this accessable for anyone right in the doc site, perhaps below the quickstart?
This can act as a documentation helper or even code writing assistant.
Screenshots
Paste relevant images or screenshots here

developer.tbd.website URL
No response
Browser
No response
Relevant log output
No response

this would be soooo dope! I'm envisioning it right up top as a search bar in the header. so from anywhere on the site, people can query stuff

@angiejones so a search bar box - and then the response pops down below it? (trying to visualise it - do you have any sketching skills?)

response could appear in a Drawer component

nice. Adding https://github.com/openai/openai-cookbook/blob/main/examples/How_to_call_functions_for_knowledge_retrieval.ipynb as some ideas on how I may do it.


Getting dark shimmer animation for big views

I have even tested this on the sample app as well. It shows the dark animation like in the gif. How to fix this issue?

it worked after I adjust color of views and shimmer color

@chathura2020 yes, to achieve the final effect, you have to modify both the shimmer color and the background of the views.


To make Assets tradable on the P2P marketplace - Enhancement

The Market API now is limited to tokens and it would be really cool to have the ability to trade assets for NXS or tokens on the marketplace

Yeah that will be key for the geo-spacial markets. Since I'll be going into the market code to enhance it with other requested features I'll tackle this guy at the same time.

When is geo-special markets meant to be on the road map? IT seems like 5.1 was meant for NFT marketplace, but now it not?
is this 5.2? 6.0?

Closing this issue as there is no development happening and Nexus hitting rock bottom as Colin has takien over Nexus treasury keys and email infrastructure to control Nexus.


fix fill-rule attribute name

React warns about incorrect attribute name and discards the "fill-rule" attribute.

@aaronshaf is it possible merge this PR?

This is great, but React also complains about the xmlns attribute on the svg tags. There's a newer pull request that addresses both issues at once.

Done


Add button to download delegate list

I've manage to add a button to download the list of delegate for the master neuron
The button doesn't have css, but it does the job for now
(I've recreated the pull request, I f*cked up the last one lol)

I don't have approval rights but I would pass off on this change at a code level. Its how I would have done it. It probably should have the same CSS treatment as everything else but I'm not a frontend guy so...

Oh you mean in the satori.py file ?
I didn’t see it from that angle, it’s a good idea too !

@VictorCyprien This is great, I love it.
I'm going to merge this in now, but I have a couple suggestions that would improve it, I hope you're willing:

take a look at the dashboard-oracles.html or dashboard-predictors.html and you'll see a little download button found at the top right of the card, might look better that way and the user doesn't have to scroll all the way to the bottom.
(never mind, I did this one): lets just call the server once to get the data - we do that on page load so lets throw it in a variable and just use that when we hit the button.

I just did the second one, but the first one would be nice I think! thanks


update for #1048 - safe delete

#1048 provides  a confirmation on the delete operation when we have a safeDelete attribute

Thanks!


The dataset links for oxford_pets_image_segmentation are broken

The links https://www.robots.ox.ac.uk/~vgg/data/pets/data/images.tar.gz & https://www.robots.ox.ac.uk/~vgg/data/pets/data/annotations.tar.gz should be https://thor.robots.ox.ac.uk/~vgg/data/pets/images.tar.gz & https://thor.robots.ox.ac.uk/~vgg/data/pets/annotations.tar.gz.
I submitted a pull request to address the issue: https://github.com/keras-team/keras-io/pull/1198

Pull request was merged.


What does one need to do to convert a standard VSCode extension to a code-server one?

Description
There are more and more Github issues related to outdated and/or not working extensions on the code-server's marketplace. In this regards, what does it take to convert a standard VSCode extension to a code-server compatible one? Are there any documentation and examples on that?
I am asking as the community could probably help with porting and maintaining some of the extensions if using the Microsoft marketplace is really a no go from legal perspective.
For example, as part of https://github.com/cdr/code-server/issues/1169 I tried hard to bring the newest version of the Kotlin language server for VSCode to code-server. I also had in mind sharing the outcome with the community. However, I was not successful with running the newest version of https://github.com/fwcd/vscode-kotlin-ide with code-server while I managed to build the Kotlin language server and patch the VSCode extension to contain it.
Related Issues
https://github.com/cdr/code-server/issues/1169 and also other Github issues reporting outdated and/or not working extensions.

I think this is something to deal with broken extensions, not really a compatibility issue since Microsoft has done a lot of stuff to make sure your extensions work without any code modifications to the extension. So this is more of our registry not building stuff properly.


Render image from arbitrary view in Python

Hi, thank you for this awesome project.
I have read the documentations on using kire to render image from a 3d file. However, the view can only be controlled via num_azimuth and elevation.
How can I render image from a 3d file, given any arbitrary viewing angle with Python?
Also, does kiuikit allow control over the distance from the camera to the object, so that I can zoom in and out to the object?

@KhanhNgoDuy Hi, thanks for your interest!
Sure, you'll need to modify the code a little for arbitrary view angles here. You can use --radius (in CMD) or the mouse wheel (in GUI) for zooming.


docs: Update demo app to include more examples

This PR is a work-in-progress to update the demo app to include more examples.
It now includes a new main activity with buttons for each example - the previous example is now "Basic Map Activity":

It also adds an example for using the map in a column to troubleshoot and serve as documentation for https://github.com/googlemaps/android-maps-compose/issues/78, and this "Scrolling Map Activity" currently looks like this:

Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[ ] Make sure to open a GitHub issue as a bug/feature request before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[ ] Ensure the tests and linter pass
[ ] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

Nice! #24 is also somewhat related


A way to disable "setting non-standard global variable" ?

Hello Peter,
There is a way via command line to just disable this setting? On our (kinda old) code I need to expose some globals in a module. There is a way to disable this check via command line args?
I still want to check for accessing global variables (I don't want --no-global)
Example:
-- test.lua
function bar() end
foo()

Checking test.lua                        Failure

    test.lua:1:10: setting non-standard global variable bar
    test.lua:2:1: accessing undefined variable foo

Total: 2 warnings / 0 errors in 1 file

I just want one warning on this (accessing undefined variable foo)

Hi ! How can I activate this setting ?

@X-Raym hi, you can pass -d to luacheck or add allow_defined = true to luacheck config called .luacheckrc. Alternatively, pass the globals you define to --globals.

Thank you again for your quick support !
I tried to modify the file located in /usr/local/lib/luarocks/rocks-5.2/luacheck/0.15.0-1/spec/configs/project but without success. The same errors message are still visible.
If may be more flexible if we had buttons (or at least config settings) to toggle these behaviors right from Atom. 🎅
Maybe it could filter luacheck warnings list output rather than modifying its config file.
Just an idea, I don't know if it is possible.

@X-Raym try modifying file called .luacheckrc in the root of your project.


Figure representations

Hello,
Thank you for your interesting project!
I would like to know if you could add the code part that generates the animations of the trajectories that can be seen in your video and in the paper?
Thank you!

I'm looking forward to the code , too! Thank you!


Restore top-level error key in query response

I'm not sure which PR introduced this behavior, but given an invalid query such as SELECT bottom(f) FROM m, we used to (i.e. 1.3.4 and earlier) return the error in JSON as:
{"error":"error parsing query: invalid number of arguments for bottom, expected at least 2, got 1"}

and now on master the error JSON is:
{"results":[{"statement_id":0,"error":"invalid number of arguments for bottom, expected at least 2, got 1"}]}

The newer format is more specific, which is more helpful when using multiple statements in a single query, but this change is breaking for any clients that depended on the behavior of error being a key at the top level.
Assuming we want to keep the more specific error, my recommendation is to still return the first error as a top-level error for 1.4 but deprecate it, so that clients have the entire 1.4 lifecycle to switch to handling errors nested inside results. In 1.5 we can fully remove the top-level error key.
If we decide not to restore the top-level error key, then we need to call it out as a breaking change in the changelog.

I have a few questions that come to my mind that may greatly influence how we think about this or what we do.
First, if there is an error preparing a query to execute, should it prevent any query from executing? So if I do:
CREATE DATABASE db0;
SELECT count(value), host FROM cpu;

The above is considered invalid because the second statement is not valid. Should the create database be prevented or should execution only be aborted after the second command is attempted to be run? I think SQL might do the former, but I'm not actually sure if that's the case.
Another question is how much we care about chunked responses. Chunked responses have been a continuous mess throughout the entire time. It would be possible for me to keep the current architecture and propagate the error to the top level, but only for non-chunked responses. Maybe we could do that and list the breaking change in chunked responses and then do better with the result output refactor?
The rationale for why this change was made is I wanted to separate the parser from the logic of the query engine. So the parser handles the grammar alone, but the validations come from attempting to execute the actual query. That way, it would be possible to use the influxql package to parse the following: SELECT mean(value) FROM cpu GROUP BY time(1m) and then rewrite the AST condition to include the time range without receiving a parsing error. If possible, I would like to keep that since combining the validation with a query to the execution of the query also makes it less likely that the validation will have an error.

For context, I tried this in Postgres and came out with something similar to the new method:
# update users set username = 'foo' where id = 1; select count(*), id from users;
UPDATE 1
ERROR:  column "users.id" must appear in the GROUP BY clause or be used in an aggregate function

This one had what could likely be considered a compilation error. You can statically determine that users.id was going to be wrong here, but it's syntactically valid SQL. But, the update command still ran.
To be fair, I can do syntactically invalid SQL and Postgres appears to be fine with that. It still runs the first command and then fails on the second one with a syntax error. So we don't necessarily have to emulate everything Postgres does.
I do think it might be good to propagate the error to the top level for non-chunked responses, leave chunked responses in the dust, and finally get to making a new output encoding that doesn't have these same problems and is properly defined.


Helper could not be found

Hi,
I am encountering something really weird.
After installing this addon (as I have always done), I met in the console and input was not show :
Uncaught Error: Assertion Failed: A helper named 'ember-flatpickr' could not be found
I tried to relaunch server, reinstall, and generate addon, but nothing works.
This is the first time I meet this. How to solve it ?
Regards,

I think, this is due to the pod structure of the component (?).
I am using ember-cli 2.5.
So I solved this issue with a little trick.
/* components/ember-flatpickr/component.js */

import emberFlatpickr from 'ember-flatpickr/components/ember-flatpickr/component';

export default emberFlatpickr;

@Swizz could you confirm the version you have installed? I recently switched from using ember-browserify to just using flatpickr as a bower dependency, so doing a fresh ember install ember-flatpickr should run the blueprint and pull in flatpickr from bower. Not sure if that is contributing to this issue or not.

@rwwagner90  Sure, I am dealing with
/* from npm */
"ember-cli": "2.5.0",
"ember-flatpickr": "0.1.11"

/* from bower */
"ember": "~2.5.0",
"flatpickr": "flatpickr-calendar#1.8.8"

But ... the bower dependencie seem to be pulled, because, the proxy component works well.
The weird thing is around the component/helper registering.
It is really uncommon, I installed a new addon, and It works well at the first time.

@Swizz seems like all the versions should be fine then. If you start from scratch and install, everything works though, right?

I have this issue as well. But, I am using pods.

@NullVoxPopuli can you please do the following?

Remove ember-flatpickr from your package.json
Remove flatpickr from your bower.json
rm -rf node_modules bower_components
npm i
bower i
ember install ember-flatpickr

Let me know if clearing everything out works or not, and then please confirm the versions of ember-flatpickr and flatpickr you have.

I'll give that a go tonight.
I wasn't blocked or anything, thanks to @Swizz's technique of convincing the resolver ember-flatpickr really does exist.

@NullVoxPopuli if it still doesn't find it, it is probably a bug with pods themselves, as I'm just using the standard addon exports, which should register it to the app namespace.

@NullVoxPopuli I wonder if there is an issue with me using pods in the addon and you then using the component inside another pod? Perhaps I should put the addon back to normal structure, not pods?

Yeah.
My bet would be on that if the error persists, it has to do with my app still being in transition, and having traditional structure and pods structure (under app/pods/)

I'd keep your component as pods and then maybe provide the 'hack' that I have? https://github.com/NullVoxPopuli/aeonvera-ui/blob/master/app/components/ember-flatpickr.js

cause, my hunch is this is a pods issue

(get hyped for ember's module unification!!)

@NullVoxPopuli yeah, it feels like a pods issue, but I don't know if it would be an issue, if I didn't use pods here too. Would you like to submit a PR to update the README with something like "if you get the error (whatever the error is), use this hack"? Would be an easy contribution 😃
So ready for module rfc to be implemented!

@Swizz @NullVoxPopuli I switched this addon to use traditional structure, instead of pods, and it appears to fix the issue. Please grab 0.7.0 and delete your workaround files 😄


Add hardknott and honister compatibility, update some packages

Hi @zboszor, thank you for creating this pull request.  The only issue I see is that I don't believe the branch will be backwards compatible with Gatesgarth with the new append/prepend syntax so the COMPATIBLE_LAYER should only be hardknott, and we should branch off to a new hardknott git branch.
Let me know if this sounds right to you and I'll create a new branch to pull into. If you wish to backport the openblas/scikit-learn updates to the Gatesgarth branch I would also take those.
Thanks,
Jack.

Ok, so we now have a hardknott branch, and a new master branch which will become the honister.

Hi @zboszor, thank you for creating this pull request.  The only issue I see is that I don't believe the branch will be backwards compatible with Gatesgarth with the new append/prepend syntax so the COMPATIBLE_LAYER should only be honister, and we should branch off to a new honister git branch.
Let me know if this sounds right to you and I'll create a new branch to pull into. If you wish to backport the openblas/scikit-learn updates to the Gatesgarth/Hardknott branches I would also take those.
Thanks,
Jack.

See<EMAIL_ADDRESS>for the change.
The bitbake changes to accept : along with _ for the override syntax was patched back to dunfell, i.e. bitbake 1.46.x.
So the compatibility with gatesgarth is ensured.

Thanks!


Remove --f16 \ from run_unsup_example.sh

The --f16 is used only for Cuda devices and since the example is for CPU devices, removing --f16 \ corrects the problem.

Hi,
Thanks for your contribution. Though this script also works for GPU and thus --fp16 is useful.

Hi,
Thanks for your contribution. Though this script also works for GPU and thus --fp16 is useful.

It was showing an error in my MacBook. Thought it would be helpful since the example was for CPU. Just wanted to let you know!

Thanks for letting me know! Though this repo is mainly designed for Linux+GPU environment. I'll close the issue for now.


CI Cluster is not working

Following #1069 CI cluster is not processing token properly

we should document this kind of errors
I have seen multiple users received this, but then at that time I don't remember how to fix it.

did you? how did you see this? 422 is a catch-all for a lot of things in github api, in this case it's a "permission denied" but may be a invalid argument or whatever,


Match the django admin-autocomplete container for select2

The select2tree widget uses the default container, e.g. select2-container--default, which causes it to differ in styling compared to the other select2 elements django admin uses, which use select2-container--admin-autocomplete.
Is it possible/easy to make the app match the other autocomplete containers to give it a more uniform appearance, especially in dark mode?

I'll think


Update to pycryptodome

Hi!
Pycrypto is deprecated. Switching to pycryptodome is straightforward.
I also updated the script documentation in python.
Cheers!

Thank you, Daniel! I didn't realize Pycrypto was deprecated.

Thank you, Daniel! I didn't realize Pycrypto was deprecated.


(Waiting for Toolchain fix): BLE stack don't compile after the last me-no.dev commit

Hello Neil, after the last commit of arduino-esp32 there are some troubles with your library
Best wishes and thanks for your great work
Editor: Moved to Pastebin: https://pastebin.com/GT41096V

Ouch!!!   This is not good.  Tracking as #82 for the implications across both Arduino and ESP-IDF.

Hello again!
I'm not sure about what is happening in my software, I think I'm mixing apples with oranges and this doesn't work
After the #82, I try to compile with arduino-esp32 without BLE support to check other parts of my system and it works fine
When I back to use BLE I found a strange error
Any ideas?
Rebooting...
ets Jun  8 2016 00:22:57
rst:0xc (SW_CPU_RESET),boot:0x1b (SPI_FAST_FLASH_BOOT)
configsip: 0, SPIWP:0xee
clk_drv:0x00,q_drv:0x00,d_drv:0x00,cs0_drv:0x00,hd_drv:0x00,wp_drv:0x00
mode:DIO, clock div:1
load:0x3fff0010,len:4
load:0x3fff0014,len:716
load:0x40078000,len:0
load:0x40078000,len:11572
entry 0x40078a14
assertion "res == pdTRUE" failed: file "/Users/ficeto/Espressif/ESP32/esp-idf/components/esp32/./cpu_start.c", line 337, function: start_cpu0_default
abort() was called at PC 0x4016dc97 on core 0
Decoding 11 results
0x40089dcc: invoke_abort at /Users/ficeto/Espressif/ESP32/esp-idf/components/esp32/./panic.c line 553
0x40089ecb: abort at /Users/ficeto/Espressif/ESP32/esp-idf/components/esp32/./panic.c line 553
0x4016dc97: __assert_func at /Users/ivan/e/newlib_xtensa-2.2.0-bin/newlib_xtensa-2.2.0/xtensa-esp32-elf/newlib/libc/stdlib/../../../.././newlib/libc/stdlib/assert.c line 63 (discriminator 8)
0x40083e41: start_cpu0_default at /Users/ficeto/Espressif/ESP32/esp-idf/components/esp32/./cpu_start.c line 337 (discriminator 1)
0x40083fe4: call_start_cpu0 at /Users/ficeto/Espressif/ESP32/esp-idf/components/esp32/./cpu_start.c line 2070x40078b33:0x3ffe3e70 0x40007c31:0x3ffe3eb0 0x4000073d:0x3ffe3f20

Howdy, unfortunately we are just plain "broke" on Arduino at the moment.   The story in detail can be found in #82 but in summary ...

ESP-IDF Git master changed BLE Client APIs
ESP-IDF Git master changed toolchain (compilers, linkers etc)
BLE C++ libraries against ESP-IDF stopped working
Arduino-ESP32 re-based itself against ESP-IDF Git master
BLE C++ libraries against Arduin-ESP32 stopped working
BLE C++ libraries re-worked to accommodate ESP-IDF BLE Client API changes
Toolchain stopped working when building C++ code that includes some standard libraries (see #1032)
ESP-IDF works with BLE C++ when we drop back to older tool chain
Arduino-ESP32 still does not work with BLE C++ as we have no obvious recipe short of surgery to tell Arduino-ESP32 to use an alternative tool chain.

Ok, Neil thanks for the support

The great news is that the Arduino ESP32 project has been updated.  Please re-build your base Arduino ESP32 environment.  The even better news is that the owner of that project has decided to include the BLE C++ classes as part of the distribution.  So it is now baked in.  Please re-test and let us now see where we stand.

Hello Neil, I try with my software and doesn't work again, the error is similar to the previous failure
rst:0xc (SW_CPU_RESET),boot:0x1b (SPI_FAST_FLASH_BOOT)
configsip: 0, SPIWP:0xee
clk_drv:0x00,q_drv:0x00,d_drv:0x00,cs0_drv:0x00,hd_drv:0x00,wp_drv:0x00
mode:DIO, clock div:1
load:0x3fff0010,len:4
load:0x3fff0014,len:956
load:0x40078000,len:0
load:0x40078000,len:11856
entry 0x40078a34
assertion "res == pdTRUE" failed: file "/Users/ficeto/Desktop/ESP32/ESP32/esp-idf-public/components/esp32/./dport_access.c", line 187, function: esp_dport_access_int_init
abort() was called at PC 0x4014ea37 on core 0
Decoding 12 results
0x40089f20: invoke_abort at /Users/ficeto/Desktop/ESP32/ESP32/esp-idf-public/components/esp32/./panic.c line 553
0x4008a01f: abort at /Users/ficeto/Desktop/ESP32/ESP32/esp-idf-public/components/esp32/./panic.c line 553
0x4014ea37: __assert_func at /Users/ivan/e/newlib_xtensa-2.2.0-bin/newlib_xtensa-2.2.0/xtensa-esp32-elf/newlib/libc/stdlib/../../../.././newlib/libc/stdlib/assert.c line 63 (discriminator 8)
0x40164d8a: esp_dport_access_int_init at /Users/ficeto/Desktop/ESP32/ESP32/esp-idf-public/components/esp32/./dport_access.c line 187 (discriminator 1)
0x40083ed4: start_cpu0_default at /Users/ficeto/Desktop/ESP32/ESP32/esp-idf-public/components/esp32/./cpu_start.c line 334
0x400840ac: call_start_cpu0 at /Users/ficeto/Desktop/ESP32/ESP32/esp-idf-public/components/esp32/./cpu_start.c line 207

The symptom you are seeing seems close to #110 ... in your script/sketch ... do you have WiFi enabled as well?  Do the samples that come with the BLE C++ classes work?

Next, I try your examples
BLE_uart works fine sending and receiving!
BLE_client needs a "return true" in line 65
BLE_server works fine
BLE_notify works fine
BLE_scan scan
BLE_write works fine, writing and receiving in serial 0 port
That wasn't a deep test but they compile and using rRF connect software I can see the esp32 device and interact with it.

Hello Neil, yes my software uses wifi and BLE simultaneously, I'm going to disconect wifi while BLE is working
Thanks for the help

Ok wifi disconnected, but another issue raises. I think your last comment in #110 is the key,  and stack size is creating troubles when the code is a bit more complicated than a simple test.
rst:0xc (SW_CPU_RESET),boot:0x1b (SPI_FAST_FLASH_BOOT)
configsip: 0, SPIWP:0xee
clk_drv:0x00,q_drv:0x00,d_drv:0x00,cs0_drv:0x00,hd_drv:0x00,wp_drv:0x00
mode:DIO, clock div:1
load:0x3fff0010,len:4
load:0x3fff0014,len:956
load:0x40078000,len:0
load:0x40078000,len:11856
entry 0x40078a34
E (30481) esp_pthread: pthread_getspecific: not supported!
E (30482) esp_pthread: pthread_setspecific: not supported!
abort() was called at PC 0x400f5963 on core 0
Decoding 17 results
0x4008953c: invoke_abort at /Users/ficeto/Desktop/ESP32/ESP32/esp-idf-public/components/esp32/./panic.c line 553
0x4008963b: abort at /Users/ficeto/Desktop/ESP32/ESP32/esp-idf-public/components/esp32/./panic.c line 553
0x400f5963: __cxxabiv1::__terminate(void ()()) at /builds/idf/crosstool-NG/.build/src/gcc-5.2.0/libstdc++-v3/libsupc++/eh_terminate.cc line 112
0x400f59aa: std::terminate() at /builds/idf/crosstool-NG/.build/src/gcc-5.2.0/libstdc++-v3/libsupc++/eh_terminate.cc line 112
0x400f60d6: __cxa_get_globals at /builds/idf/crosstool-NG/.build/src/gcc-5.2.0/libstdc++-v3/libsupc++/eh_globals.cc line 133
0x400f6163: __cxa_throw at /builds/idf/crosstool-NG/.build/src/gcc-5.2.0/libstdc++-v3/libsupc++/eh_throw.cc line 65
0x400e4fcb: std::__throw_length_error(char const) at /builds/idf/crosstool-NG/.build/src/gcc-5.2.0/libstdc++-v3/src/c++11/functexcept.cc line 122
0x400e6b9e: std::__cxx11::basic_string  , std::allocator  >::_M_create(unsigned int&, unsigned int) at /builds/idf/crosstool-NG/.build/xtensa-esp32-elf/build/build-cc-gcc-final/xtensa-esp32-elf/libstdc++-v3/include/bits/basic_string.h line 2200
0x400e6d2a: std::__cxx11::basic_string  , std::allocator  >::_M_mutate(unsigned int, unsigned int, char const*, unsigned int) at /builds/idf/crosstool-NG/.build/xtensa-esp32-elf/build/build-cc-gcc-final/xtensa-esp32-elf/libstdc++-v3/include/bits/basic_string.h line 2200
0x400e708d: std::__cxx11::basic_string  , std::allocator  >::_M_replace(unsigned int, unsigned int, char const*, unsigned int) at /builds/idf/crosstool-NG/.build/xtensa-esp32-elf/build/build-cc-gcc-final/xtensa-esp32-elf/libstdc++-v3/include/bits/basic_string.h line 2200
0x400e70cd: std::__cxx11::basic_string  , std::allocator  >::assign(char const*) at /builds/idf/crosstool-NG/.build/xtensa-esp32-elf/build/build-cc-gcc-final/xtensa-esp32-elf/libstdc++-v3/include/bits/basic_string.h line 2200
0x400e70dd: std::__cxx11::basic_string  , std::allocator  >::operator=(char const*) at /builds/idf/crosstool-NG/.build/xtensa-esp32-elf/build/build-cc-gcc-final/xtensa-esp32-elf/libstdc++-v3/include/bits/basic_string.h line 2200
0x400e0a92: FreeRTOS::Semaphore::give() at C:\Users\javier\Documents\Arduino\hardware\espressif\esp32\libraries\BLE\src/FreeRTOS.cpp line 126
0x400dda31: BLEServer::handleGATTServerEvent(esp_gatts_cb_event_t, unsigned char, esp_ble_gatts_cb_param_t*) at C:\Users\javier\Documents\Arduino\hardware\espressif\esp32\libraries\BLE\src/BLEServer.cpp line 60
0x400dcca5: BLEDevice::gattServerEventHandler(esp_gatts_cb_event_t, unsigned char, esp_ble_gatts_cb_param_t*) at C:\Users\javier\Documents\Arduino\hardware\espressif\esp32\libraries\BLE\src/BLEDevice.cpp line 47
0x4010339d: btc_gatts_cb_to_app at /Users/ficeto/Desktop/ESP32/ESP32/esp-idf-public/components/bt/bluedroid/btc/profile/std/gatt/btc_gatts.c line 54
:  (inlined by) btc_gatts_cb_handler at /Users/ficeto/Desktop/ESP32/ESP32/esp-idf-public/components/bt/bluedroid/btc/profile/std/gatt/btc_gatts.c line 731
0x400ff83a: btc_task at /Users/ficeto/Desktop/ESP32/ESP32/esp-idf-public/components/bt/bluedroid/btc/core/btc_task.c line 85

Aha ... that's actually good news.   But let's be sure that we now realize that we have two stories here.  One is the problem running BLE and WiFi together which appears to be stack size that is hard-coded in Arduino BLE.
The second issue, the one that I am going to address here, is the last problem you posted.  The good news is that we've been working this one as #121 and just this morning made a major breakthrough ... to the point where good users like yourself can use it.
What I'd like to suggest is that we don't talk about this current problem in this issue any more but instead let me encourage you to piggyback on #121 which was the first to report that issue.  Looking forward to hearing what you think after reading #121.

Hello Neil, after following your comments in #121 I've got my BLE working fine 👍
Few moments ago @me-no-dev has made a commit to increase the stack size
#define CONFIG_MAIN_TASK_STACK_SIZE 8192
Apparently, this wasn't the final solution when I'm using BLE and WIFI...
Best regards
Rebooting...
ets Jun  8 2016 00:22:57
rst:0xc (SW_CPU_RESET),boot:0x1b (SPI_FAST_FLASH_BOOT)
configsip: 0, SPIWP:0xee
clk_drv:0x00,q_drv:0x00,d_drv:0x00,cs0_drv:0x00,hd_drv:0x00,wp_drv:0x00
mode:DIO, clock div:1
load:0x3fff0010,len:4
load:0x3fff0014,len:956
load:0x40078000,len:0
load:0x40078000,len:11856
entry 0x40078a34
assertion "res == pdTRUE" failed: file "/Users/ficeto/Desktop/ESP32/ESP32/esp-idf-public/components/esp32/./ipc.c", line 85, function: esp_ipc_init
abort() was called at PC 0x401502b3 on core 0
Decoding 12 results
0x4008a3a8: invoke_abort at /Users/ficeto/Desktop/ESP32/ESP32/esp-idf-public/components/esp32/./panic.c line 553
0x4008a4a7: abort at /Users/ficeto/Desktop/ESP32/ESP32/esp-idf-public/components/esp32/./panic.c line 553
0x401502b3: __assert_func at /Users/ivan/e/newlib_xtensa-2.2.0-bin/newlib_xtensa-2.2.0/xtensa-esp32-elf/newlib/libc/stdlib/../../../.././newlib/libc/stdlib/assert.c line 63 (discriminator 8)
0x40166989: esp_ipc_init at /Users/ficeto/Desktop/ESP32/ESP32/esp-idf-public/components/esp32/./ipc.c line 85 (discriminator 1)
0x400841de: start_cpu0_default at /Users/ficeto/Desktop/ESP32/ESP32/esp-idf-public/components/esp32/./cpu_start.c line 332
0x400843bc: call_start_cpu0 at /Users/ficeto/Desktop/ESP32/ESP32/esp-idf-public/components/esp32/./cpu_start.c line 207

I'm at work just now ... but first opportunity I get I'll be re-testing here with the increased stack size.  What I have been trying to avoid doing is learning how to build the Arduino-ESP32 libraries from source.  Ideally, I just wanted to be a "user" of those libraries and the BLE code becomes just a layer on top of them.  However I'm re-thinking that notion.  Last I looked, I had convinced myself that the error we had was as described in #110 which is the primary tracker for WiFi/BLE memory issues.  What I believe is that the amount of stack space associated with the main task is hard-coded by the Arduino-ESP32 libraries before distribution of the Arduino-ESP32 environment.  I can't remember off the top of my head what the previous value was but even 8192 may be too low.
Since I am ignorant on how to build the Arduino-ESP32 libraries for testing, I hadn't attempt to try any tests to see what values would work.  What I believe I must do now is take the hit and learn (and document) how to build the Arduino-ESP32 libraries here/locally ... so that I can make changes to the build myself so that I can then be specific about what values may be needed.  This may take some time.

Hello Neil, the previous size was 4096
Why don't just report that 8192 isn't enough and try with the next magic number 12k
In my experience stack isn't an exact science, it depends on how you declare the variables, how you make the calls, etc.
Maybe the real trouble isn't the main stack, but the CONFIG_IPC_TASK_STACK_SIZE that is currently 1024
Thanks for your great work here.

What we have is a game of parts.  Espressif makes the physical ESP32 chip.  Espressif also has a software team that owns the ESP-IDF.  Within that team there are individuals for toolchain, individuals for FreeRTOS, individuals for BLE ... and the list goes on.  It is also my understand that the Arduino-ESP32 is also built by Espressif but by a distinct individual/team there.  What that means to you and me is that when we want to write "BLE and WiFi" apps, these are layered upon layers upon layers ... for example BLE and WiFi live on top of Arduino-ESP32 which lives on top of ESP-IDF which lives (loosely) on top of FreeRTOS ... and at the bottom there is the hardware.  What that implies is that a "problem" could be introduced (or need resolution) somewhere other than the layer in which we are working.  This means that we need to collaborate between the layers.   I'm pretty sure that the Arduino-ESP32 layer team would be happy to change the stack size ... but they are asking "To What?".   And that becomes a question for thee and me.  The correct answer is for us to tell them what stack size we need.  However, we don't have a science for saying "we need 8K/12K/16K/20K etc etc" ... or do we ...?
Within an ESP-IDF/Arduino/FreeRTOS application we can ask the runtime "tell me about the high water mark of stack used".  What this means is that if we set the stack to be ... oh ... lets say 50K ... we could run up WiFi and BLE together ... let them cook for a while ... and then ASK the runtime, how much stack are you using or maximally used?  This would give us some evidence for what we need.
And in there is the puzzle ... in order to get that information, we need to set the stack to be ... say 50K.  In order to achieve that ... we need to have the skills to be able to "tweak" the lower level layer (in this case Arduino-ESP32) to change the stack size.  This requires that we re-built the Arduino-ESP32 libraries with the new stack.  And that is something Ive never done before.   It may be very easy or it may be rocket science ... I haven't had time to go look yet.  And that's the next piece of territory that needs explored.  Once I have the size and shape of what it looks like to build the Arduino-ESP32 libraries from source and make tweaked changes ... I'll be able to run experiments to determine if stack size is what is needed ... and if not ... what else may be at fault.

@nkolban just add a new project, import Arduino as component and call the WiFi and BLE API. Tweak the "magic" number and tell me the result (you can even use the sdkconfig bundled in arduino/tools/sdk in order to have the same config as what is used by Arduino). There is nothing else fancy :) Arduino is just a component in IDF and does not require any extra care ;)

Following mr @me-no-dev instructions, I created a new ESP-IDF project called arduino.  I then created a sub-folder in there called components and in components cloned the Arduino-ESP32 project using:
$ git clone https://github.com/espressif/arduino-esp32.git

I then entered the Arduino-esp32 project and performed a:
$ git submodule update --init

This created an environment I could use.  I then copied the "sdkconfig" from the Arduino project to my own sdkconfig so we would be the same.  I upped the stack size to 20K and built and ran.
Unfortunately, it continued to crash.  I enabled gdb debugging and got the following back trace:
#0  0x4008a060 in invoke_abort () at /home/kolban/esp32/esptest/esp-idf/components/esp32/./panic.c:139
#1  0x4008a12e in abort () at /home/kolban/esp32/esptest/esp-idf/components/esp32/./panic.c:148
#2  0x4010145e in __assert_func (file=0x3f4018a0 "/home/kolban/esp32/esptest/esp-idf/components/esp32/./cpu_start.c", line=347,
    func=<optimized out>, failedexpr=0x3f4018ec "res == pdTRUE") at ../../../.././newlib/libc/stdlib/assert.c:63
#3  0x40080ee8 in start_cpu0_default () at /home/kolban/esp32/esptest/esp-idf/components/esp32/./cpu_start.c:347
#4  0x4008108b in call_start_cpu0 () at /home/kolban/esp32/esptest/esp-idf/components/esp32/./cpu_start.c:207

At first I thought this is the same error we had been seeing ... but now that I look much closer ... even though we are still failing in cpu_start.c we aren't failing at the line I expected ... instead we are failing here:
    portBASE_TYPE res = xTaskCreatePinnedToCore(&main_task, "main",
                                                ESP_TASK_MAIN_STACK, NULL,
                                                ESP_TASK_MAIN_PRIO, NULL, 0);
    assert(res == pdTRUE);   /// <---- This is line 347
    ESP_LOGI(TAG, "Starting scheduler on PRO CPU.");
    vTaskStartScheduler();
    abort(); /* Only get to here if not enough free heap to start scheduler */

Now I'm going to hit the books to see if we can get more diagnostics from this error.

Its teh thing, ive got same or simmilar error on esp-idf stack (no arduino).
At first i didnt got it, then i did some changes in menuconfig and then it started to crash like here described.

Here is what I'm starting to find ... 3 permutations:

No BLE, Yes WiFi
Yes WiFi, No BLE
Yes WiFi, Yes BLE

My digging seems to be showing that we are running out of memory when trying to create tasks.  So I switched on verbose boot logging.  This has shown me the following information:

No BLE, Yes WiFi

I (660) heap_init: Initializing. RAM available for dynamic allocation:
I (672) heap_init: At 3FFAFF10 len 000000F0 (0 KiB): DRAM
I (683) heap_init: At 3FFC8B38 len 000174C8 (93 KiB): DRAM
I (689) heap_init: At 3FFE0440 len 00003BC0 (14 KiB): D/IRAM
I (696) heap_init: At 3FFE4350 len 0001BCB0 (111 KiB): D/IRAM
I (707) heap_init: At 4008F29C len 00010D64 (67 KiB): IRAM

Yes BLE, No WiFi

I (753) heap_init: Initializing. RAM available for dynamic allocation:
I (765) heap_init: At 3FFAFF10 len 000000F0 (0 KiB): DRAM
I (777) heap_init: At 3FFCF600 len 00010A00 (66 KiB): DRAM
I (783) heap_init: At 3FFE0440 len 00003BC0 (14 KiB): D/IRAM
I (789) heap_init: At 3FFE4350 len 0001BCB0 (111 KiB): D/IRAM
I (801) heap_init: At 4009154C len 0000EAB4 (58 KiB): IRAM

Yes BLE, Yes Wifi ... failure

I (824) heap_init: At 3FFAFF10 len 000000F0 (0 KiB): DRAM
I (835) heap_init: At 3FFD4E90 len 0000B170 (44 KiB): DRAM
I (842) heap_init: At 3FFE0440 len 00003BC0 (14 KiB): D/IRAM
I (848) heap_init: At 3FFE4350 len 0001BCB0 (111 KiB): D/IRAM
I (860) heap_init: At 40092348 len 0000DCB8 (55 KiB): IRAM

Summarizing the interesting lines:

No BLE, Yes WiFi - (93 KiB): DRAM
Yes BLE, No WiFi - (66 KiB): DRAM
Yes BLE, Yes Wifi ... failure - (44 KiB): DRAM

What I might now be sensing is that we might be needing to increase the stack size for both BLE and WiFi to run ... but the mere presence of BLE and WiFi together linked into the environment results in there being not enough RAM for them to be together.
My gut is telling me that DRAM is where task stacks may be being allocated from.
I'm also remembering (thought can't find it) a forum or chat post which talked about the latest ESP-IDF and toolchain requiring an extra 20K of RAM to support C++ exception processing.

Non-constant static data and zero-initialized data is placed by the linker into the 256 kB 0x3FFB0000 — 0x3FFF0000 region. Note that this region is reduced by 64kB (by shifting start address to 0x3FFC0000) if Bluetooth stack is used. Length of this region is also reduced by 16 kB or 32kB if trace memory is used. All space which is left in this region after placing static data there is used for the runtime heap.

Have a look at this guys ... https://github.com/espressif/esp-idf/issues/1072 ... this may be part of the story ... I'm not yet sure if the "patch/fix" mentioned in 1072 is available yet ... but it does sniff close.

You can try, for debug purpose, to switch in menuconfig Reserve memory for two cores --> OFF and Run FreeRTOS only on first core --> ON. It lets me pass thru issues with low memory

Now my test program crash at https://github.com/nkolban/esp32-snippets/blob/master/cpp_utils/tests/BLETests/SampleClientWithWiFi.cpp#L110

Guys, the problem is fixed.
Take a look into this thread: esp-idf/issues/1072

The problem is fixed for me too.
Best regards

Closing for now.  Re-open as needed.

Apparently the stack size was changed in the Arduino environment 6 months ago ... see here:
https://github.com/espressif/arduino-esp32/pull/767/files

Yes, I updated my post after looking at the history. Sorry about that. Still weird stuff going on. My buddy in the UK has same code base as me but he builds under IDF and gets past the bootloader loop I don’t. I’ve compared sdkconfigs and don’t see any real differences yet, still looking as he uses a different esp32 board so digging in there at the moment.
Read your post about it may be related to creating tasks I think you may be right. I have other code with wifi/ble both enabled and it runs fine but I don’t use any tasks. In my current code I use tasks. However, I commented out both calls to setup the tasks and it still fails at boot going into a loop even after a clean build. Weird.
From: Neil Kolban<EMAIL_ADDRESS>Sent: Monday, April 23, 2018 10:31 AM
To: nkolban/esp32-snippets<EMAIL_ADDRESS>Cc: ifrew<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [nkolban/esp32-snippets] (Waiting for Toolchain fix): BLE stack don't compile after the last me-no.dev commit (#81)
Apparently the stack size was changed in the Arduino environment 6 months ago ... see here:
https://github.com/espressif/arduino-esp32/pull/767/fileshttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fespressif%2Farduino-esp32%2Fpull%2F767%2Ffiles&data=02|01||a180d689d1c14b20982908d5a9400ad6|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636601014875131413&sdata=hxKudTxMG%2BhTE9IPUExREaqJ4XXbPY5IrvQ63L%2FWovA%3D&reserved=0
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnkolban%2Fesp32-snippets%2Fissues%2F81%23issuecomment-383657450&data=02|01||a180d689d1c14b20982908d5a9400ad6|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636601014875131413&sdata=SCNS9RG3qOjb9uP12LILAzLVjNE5VuV64fCCoCl8BYE%3D&reserved=0, or mute the threadhttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAL-x4C35tbrpj17VKGMiJ10-iPk8sVY4ks5trg_tgaJpZM4Phh0l&data=02|01||a180d689d1c14b20982908d5a9400ad6|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636601014875131413&sdata=KhqXzN2dB%2FmvEtAnftr2%2BfupiErTOGEvW2pfirMiqFs%3D&reserved=0.


Flux CD Executor Plugin

Overview
Botkube has existing support for kubectl and helm. We need to add support for the Flux CD CLI to run Flux commands through Botkube.
Acceptance Criteria

[ ] Flux executor plugin should have the same experience (commands, flags) as Flux CD CLI
[ ] Implement Flux CD plugin for Botkube

interactivity is out of scope
Must-have commands: reconcile, suspend, resume

[ ] Design a configuration structure for the Flux executor plugin
[ ] Document it in plugins folder under the main Botkube repository.

Stretch:

[ ]  - Additional commands are nice-to-have but not part of this scope, we should make sure they can be added lateradd helm history command
[ ] Document under docs.botkube.io

Reason
Flux CD support has been requested by the community and is useful for quickly managing Flux deployments from chat tools. See: https://github.com/fluxcd/flux2/discussions/3561

@mszostok and I met with the Flux Devrel and there's community interest. How should we proceed to put fit this into the engineering calendar


Update UI/UX for XCM transfer

Is your feature request related to a problem? Please describe.

XCM transfers require user to input a limit on how much of the amount being sent will be used as transaction fee.
Not enough transaction fee can lead to the loss of fund
Setting the limit too high can eat up a large portion of the fund

Describe the solution you'd like

Allow users to fill the information by themselves
Have a suggested amount by default for user
Warn users about risks when using XCM transfers

To be resolved in #528


Sorting a date column with invalid date causes unexpected ordering of results

Description
If you have a date column with invalid data row and you sort by selecting the header, the sort order looks unpredictable, invalid data row ends up in the middle of the sort order.
Version
Data-grid: V1.2.4
Browser: Chrome
Screenshot

Code
https://gist.github.com/ppeyro/4631a0a51b7030c0bcc91bc8fa4d7159

Part of Q4 so this is no longer covered by the warranty.


Build a custom haiku initializer

A custom haiku initilizer should be built to either init with a sample drawn from default prior or be able to pass in the numpyro distributions for the inference process

The simplest trick to do this would be an helper function coupled with a built-in dictionary or whatever, this might help in #67 also

Closed in favor of transition to equinox


Enable ap_delius_context_api in Dev and Preprod

This will allow us to confirm the new endpoints work as expected before retiring the Community API endpoints for getting offender details

See also #1354


Feature: Option to load comments automatically?

Give the user an option to configure whether comments are loaded automatically or not shown until "View Comments" is clicked (the default).

@Ibrahim-Islam
the feature can be done, just trigger the button be clicked when scroll bottom
and we can also use disqus instead of issue comment system
btw, about the option key name, do you have any idea ?

On second though, I think its best to just load the comments asynchronously when an issue/article loads up and have something like comment_limit that will fetch comments in batches when clicked View More if github comments. Otherwise, user can opt for services like Disqus or something else.

i see, and i will add the feature a bit later.
thanks for the advice, and the pr are welcomed :)

Sure. I will try to work on it then.


don't show kubeconfig in the deployment flow for the vdc deployer

let's keep the option from the vdc dashboard only

Verified


[Sema][NFC] Add regression test for SR-14533

This just add a regression test for SR-14533, was about to start looking at it, but I believe #38235 fixed it :)
Or maybe #38199?
So just adding the test regression test @xedin

@swift-ci Please smoke test

I think it's #38235 what fixed this.

I think it's #38235 what fixed this.

Yeah, that was my first guess as well
I just resolved the JIRA ticket and reference the PR there :)

Great, thank you!


Update README.md

Add a link to a help video created by me

I think the HA forums would be a better place for this video. Adding it to the README opens a slippery slope of others wanting to add links to videos or tutorials.

Hi, its okay for me, that you don’t want to have it in your Github, but for me personly as a user of this addon, I would really love to have more support how to use it. And even when there are dozends of videos on the readme – the more – the better.
Von: Charles Garwood @.>
Gesendet: Montag, 27. Dezember 2021 22:28
An: cgarwood/homeassistant-fullykiosk @.>
Cc: Senbei123 @.>; Author @.>
Betreff: Re: [cgarwood/homeassistant-fullykiosk] Update README.md (#57)
I think the HA forums would be a better place for this video. Adding it to the README opens a slippery slope of others wanting to add links to videos or tutorials.
—
Reply to this email directly, view it on GitHub https://github.com/cgarwood/homeassistant-fullykiosk/pull/57#issuecomment-1001767465 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AMRSEVFJBCVRRA3GPHWK5Z3UTDK4PANCNFSM5FW42UJQ .
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675  or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub .
You are receiving this because you authored the thread.  https://github.com/notifications/beacon/AMRSEVEGME4CT7AVQVPE5KTUTDK4PA5CNFSM5FW42UJ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOHO24EKI.gif Message ID: @.*** @.***> >


Time-of-flight charge transport test demo and tutorial

Write a simple tutorial and demo that shows how to run ToF simulations and provides examples of the types of data and information produced by the simulation.

Neat film with different internal potential values using uncorrelated Gaussian DOS
Neat film with different magnitudes of disorder using uncorrelated Gaussian DOS
BHJ films with different magnitudes of tortuosity due to anisotropy (from Ising_OPV) using uncorrelated Gaussian DOS

Explanation of the ToF test and example data will be shown in the upcoming user manual PDF


Chain configs (telos-mainnet, telos-testnet, etc..)

Describe the feature
Re-implement the Telos chain config changes from telos-main-old to telos-main in a single commit.
Incorporate changes from these commits:

[x] a6b9d65e704dd1772888298e3c079c9cdbc1bed3
[ ] c189ef57d5095ce7d6f8006244dd2b038199bdee
[ ] 05a4c238e26c05f2f25f44edd2a0b8a149a1c0b9

Created draft PR #34 with changes from first commit. There are 17 failing tests at the moment, probably due to chain name which fallbacks to id, since name implementaion is in alloy.


PHP: syntax error on update

Hello,
I'm getting this error in logs when updating:
updater | ERROR <job_25976837> Error processing vimeo/psalm (Dependabot::SharedHelpers::HelperSubprocessFailed)
updater | ERROR <job_25976837> syntax error, unexpected 'string' (T_STRING), expecting function (T_FUNCTION) or const (T_CONST)
updater | ERROR <job_25976837> /home/dependabot/dependabot-updater/vendor/ruby/2.6.0/gems/dependabot-common-0.117.2/lib/dependabot/shared_helpers.rb:111:in `run_helper_subprocess'

Config: https://github.com/kubawerlos/php-cs-fixer-custom-fixers/blob/master/.dependabot/config.yml

I have the same issue. I think this is related to PHP 7.4.
It also seems that Dependabot does not respect the "php": "^7.4", settings in composer.json.

I just found out, that PHP 7.3 is hardcoded. So Dependabot is unusable with PHP 7.4 right now. 😢
https://github.com/dependabot/dependabot-core/blob/9d61e6e5ffb25a1df09f7a8a06497b6ac9b2791e/Dockerfile#L99-L107

@pehbehbeh I've missed it somehow, let's try to fix this ☝ .


c_api: some experiences

Hi all,
I've been hacking c_api this weekend in order to run some experiments from R. Doing so, I ran into some issues, which I would like to share here:
Exposing C++ Objects
Why not directly exposing some C++ Objects like Booster? This would make creation of R (#19) and Python(#11)-Bindings quite straight forward, since Rcpp and python-boost provide excellent support for wrapping C++ Objects into these languages.
As a plus, Python and R garbage collector will automatically call the destructor of those objects, so no need to register any finalizer calling some explicit destruction method, etc.
Moreover, exceptions can be simply propagated into R or Python, without having to worry about unrolling the stack correctly.
There is really no reason to go back to C (with all the nasty void *-casts...) in order to provide simple and clean bindings for those languages. If you still find there is good reason to stick to plain C, one could also think of an api.hpp, which exposes Booster and some Dataset objects and c_api.h is only providing a thin C-layer on top of this (reusing the objects from api.hpp; that's actually what I did for my experiments).
Error handling
Support for error messages is not yet available (cf. LGBM_GetLastError) and in my eyes this could be easily solved by propagating exceptions. Unfortunately, there exists in the moment one big show stopper, when it comes to fatal logging:
log.h terminates the complete process in case of a fatal error with exit(-1), which is a no go for any library use case. No one wants in such a situation to lose his whole R/Python session, just because he gave some bad parameters to LightGBM.

@bwilbertz

Some languages don't support directly using c++ object,  like julia.
However, your suggestion is considerable, I can try to expose c++ object as well.
I will quick fix the exit(-1) this while using c_api.
For the error message and exception, we need to use RAII feature, so it may take some time to enable this.

@guolinke thanks for picking this up (and pushing RAII forward).
For the different headers like boost/python.hppand Rcpp.h:
No, you would not need them on the API side (i.e. inside the src-folder). The idea would be to include them only inside the respective package dirs.
Since your design of c_api is closely following the lines of xgboost, let's pick this as an example:
Rcpp.h would be included in xgboost_r.cc and functions like
SEXP XGBoosterCreate_R(SEXP dmats) {
  SEXP ret;
  R_API_BEGIN();
  // transform input args
  [...]
  BoosterHandle handle;
  CHECK_CALL(XGBoosterCreate(BeginPtr(dvec), dvec.size(), &handle));
  ret = PROTECT(R_MakeExternalPtr(handle, R_NilValue, R_NilValue));
  R_RegisterCFinalizerEx(ret, _BoosterFinalizer, TRUE);
  R_API_END();
  UNPROTECT(1);
  return ret;
}

could be just expressed as:
using namespace LightGBM;
SEXP XGBoosterCreate_R(SEXP dmats) {
  // transform input args
  [...]
  Rcpp::XPtr < Booster > handle(new Booster(_input_args));

  return handle;
}

Similar with smart pointers from Python Boost (or even directly map the complete Booster object into a python object using boost::python::class).

@bwilbertz
Your point is :

put "Booster" and its relative functions to same class, and place into one header.
For python and R binding, using different code the expose "Booster" and "Dataset" in their own package dirs?

Yes, having certain C++ objects exposed in a stable API and wrapping them for R & Python into their corresponding smart pointers + exception handling.
Note, that for providing a R package, there is no way to do so without writing a certain amount of C or C++ code specific for R. It's just the question if you want to handle all the R PROTECT/R_RegisterCFinalizerEx stuff by yourself or use a more modern Rcpp style (since we already have nice C++ objects around).
And I guess that you want to use some kind of garbage collection in R/Python, since relying on the end user to explicitly calling a delete function in order to free memory could be a bit problematic.

@bwilbertz OK, I see.
So we still call the c style function in Python/R, right? These c++ objects are mainly use for GC?
My original idea is wrapping a basic class in Python by using these c_api, and user use this basic class or more high level interfaces without caring about c_api.
I am not familiar with R, but it will be better if we can delete it in c++.

@bwilbertz  one other problem is it needs additional library to compile api for R. Can we avoid this?

Which additional library do you mean? Rcpp?
The only dependency would be that the user needs Rcpp package installed, when installing a potential RLightGBM package (because compilation of R C/C++ code takes places during installation of R packages). But Rcpp is anyway required for a huge number of recent R packages, so not really a restriction.

OK, it seems reasonable.
However, for boost/python.h, it needs to install boost. So i think it is better to use c_api directly by ctypes in python.

Fine. I will close this issue for now, since it was only meant to share some experiences.


Support lt,gt,le,ge,ne,eq operators for blob

It utilizes bytes.Compare from the standard library. The operators lt, gt, le and ge are only useful when comparing blobs of the same size, as they will return false otherwise. A use case for this is f.i. when storing IP addresses as blobs, to compare them and/or select ranges.

@metafex Before proceeding I would like to ask if the existing support for this via string(blob1) relop string(blob2) is usable in your use case? Note: The code fragment is QL, not Go.

Theoretically it is usable, but it would need a lot of extra logic as the queries are generated and would need extra type-checking if a parameter is a blob or if the corresponding field is one.

Added a test for various blob sizes and added that strings and blobs can be compared.