AAPT: error: unknown element  found

Did you verify this is a real problem by searching the NativeScript Forum and the other open issues in this repo?
Searched for it but didnt find anything.
Tell us about the problem
Im getting this error when building the app
\platforms\android\app\build\intermediates\manifests\full\debug\AndroidManifest.xml:24: AAPT: error: unknown element <provider> found.

Which is what im using in the AndroidManifest to be able to open files like pdfs
Please provide the following version numbers that your issue occurs with:

Did the error happen while the app was being constructed? (buildtime error) Yes
tnsrunandroidlogtrace.txt
Did the error happen while the app was executing? (runtime error)

Please tell us how to recreate the issue in as much detail as possible.

AndroidManifest.xml
<?xml version="1.0" encoding="utf-8"?>
<manifest
    xmlns:android="http://schemas.android.com/apk/res/android" package="__PACKAGE__" android:versionCode="10000" android:versionName="1.0">
    <supports-screens android:smallScreens="true" android:normalScreens="true" android:largeScreens="true" android:xlargeScreens="true" />
    <uses-sdk android:minSdkVersion="17" android:targetSdkVersion="__APILEVEL__" />
    <uses-permission android:name="android.permission.CALL_PHONE" />
    <uses-permission android:name="android.permission.CAMERA" />
    <uses-permission android:name="android.permission.INTERNET" />
    <uses-permission android:name="android.permission.READ_EXTERNAL_STORAGE" />
    <uses-permission android:name="android.permission.RECORD_AUDIO" />
    <uses-permission android:name="android.permission.WRITE_EXTERNAL_STORAGE" />
    <application android:name="com.tns.NativeScriptApplication" android:allowBackup="true" android:icon="@drawable/icon" android:label="@string/app_name" android:theme="@style/AppTheme">
        <activity android:name="com.tns.NativeScriptActivity" android:label="@string/title_activity_kimera" android:configChanges="keyboardHidden|orientation|screenSize" android:theme="@style/LaunchScreenTheme">
            <meta-data android:name="SET_THEME_ON_LAUNCH" android:resource="@style/AppTheme" />
            <intent-filter>
                <action android:name="android.intent.action.MAIN" />
                <category android:name="android.intent.category.LAUNCHER" />
            </intent-filter>
        </activity>
        <activity android:name="com.tns.ErrorReportActivity" />
    </application>
    <provider
        android:name="android.support.v4.content.FileProvider"
        android:authorities="org.nativescript.MedicationManagement.provider"
        android:exported="false"
        android:grantUriPermissions="true">
        <meta-data
            android:name="android.support.FILE_PROVIDER_PATHS"
            android:resource="@xml/provider_paths"/>
    </provider>
</manifest>

According to the documentation the <provider> tag should be nested inside the <application> element:

You are right, thank you.

hello, can someone help me to sort out what's wrong.
After I have installed video-plus-plugin i'm getting the issue "unexpected element  found in "
I tried placing the provider in application tag manually but it's not working ,
I even tried removing android and add it again.........but nothing worked

I'm also facing the same problem
How to solve it.
I have just did :---
npm install cordova-plugin-video-capture-plus
npm install @ionic-native/video-capture-plus
ionic cap sync
Then tried building the application it is giving
android\capacitor-cordova-android-plugins\build\intermediates\library_manifest\debug\AndroidManifest.xml:16:5-24:16: AAPT: error: unexpected element  found in .
If I edit file to keep  tag inside tag, It is rebuilding and giving same error with outside  tag.
Is there any way to fix it???????????

hello, can someone help me to sort out what's wrong.
After I have installed video-plus-plugin i'm getting the issue "unexpected element
found in "
I tried placing the provider in application tag manually but it's not working ,
I even tried removing android and add it again.........but nothing worked

You are right, thank you.

how u fixed it tell us

hello, can someone help me to sort out what's wrong.
After I have installed video-plus-plugin i'm getting the issue "unexpected element
found in "
I tried placing the provider in application tag manually but it's not working ,
I even tried removing android and add it again.........but nothing worked

You are right, thank you.

how u fixed it tell us
please reply

Try to remove android from the project and add again ,that solved my issue
On Thu, 30 Sep, 2021, 9:11 PM prasadnarayana1996, @.***>
wrote:

hello, can someone help me to sort out what's wrong.
After I have installed video-plus-plugin i'm getting the issue "unexpected
element
found in "
I tried placing the provider in application tag manually but it's not
working ,
I even tried removing android and add it again.........but nothing worked
You are right, thank you.
how u fixed it tell us
please reply
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/NativeScript/android-runtime/issues/1093#issuecomment-931438875,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AODAPF5VEG52W23FYBRT6KTUESAIXANCNFSM4FG4CQCA
.
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.

@joey0xx, according to the documentation the <provider> tag should be nested inside the <application> element:

This was my issue, thanks a lot for saving my day.


Update tcgsearch with new traditional games room.

More needs to be changed that just that. The file has to be renamed as well because plugins are named after the room. I am doing it.


Firefox headless support

According to the documentation there is no support "yet" for firefox headless mode.
Currently our UI regression test set runs headless, so it can also run inside a headless Linux VM.
I would very much like to switch from our own framework to just Karate for these tests, but the headless feature is currently missing.
You can enable this mode via the command line parameter -headless to firefox.exe. However, you cannot set these arguments directly in the configure driver step; only arguments to geckodriver.exe.
The argument has to be specified via the capabilities object as follows (including the "alwaysMatch" segment):
{"capabilities":{"alwaysMatch":{"browserName":"firefox","moz:firefoxOptions":{"args":["-headless"]}}}}
See also https://developer.mozilla.org/en-US/docs/Web/WebDriver/Capabilities/firefoxOptions and
https://w3c.github.io/webdriver/#example-5
Are you willing to accept a PR for this feature?

@paaco yes, do submit a PR. please use the develop branch. and refer to the Developer Guide wiki. if you use slack, please see this: https://twitter.com/KarateDSL/status/937271659785961477

Updated DriverOptions::getCapabilities to make into pull request.
DriverOptions.zip

@paaco do you think you can submit these changes using a standard "Pull Request" - it is hard for me to review / merge changes if you just send me source code. does that make sense ?
https://help.github.com/en/github/collaborating-with-issues-and-pull-requests/about-pull-requests

@ptrthomas That makes perfect sense. I'm actually working on getting the PR there, I just uploaded the zip in case I need to use another PC to get it done.

Ok, the PR is there. I refrained from making a bigger change, but I think the switch/case statement inside getCapabilities() smells like the functionality could be better off inside the underlying *WebDriver. I cannot test locally if Edge or Safari understand the "alwaysMatch" option, so I implemented this only for Firefox. And maybe I should also update the documentation.

@paaco you made the change exactly where I would have gone with it for now - I lean towards putting the "conditional stuff" in one place :) but you are absolutely right, it could belong in the *WebDriver classes ! thanks.

oops I didn't mean to close this, @paaco also feel free to suggest a readme edit

You are way too fast (smile), but here's an updated one-liner.

released 0.9.5

@paaco , @ptrthomas : Can we confirm if this was released? I was trying to use configure driver { type: 'geckodriver', headless: true } but it still opens the firefox browser. I checked the readme edit that @paaco provided above and that is not present in the current release too. I see it was merged in the develop branch back in January but not sure if it made to the release.  The getCapablities method for firefox in the DriverOptions.java file attached above is also not present in the current release and in 0.9.5 . I checked in https://github.com/intuit/karate/tree/v0.9.5/karate-core/src/main/java/com/intuit/karate/driver/DriverOptions.java.

@mihirmehta The initial code has been reworked to more generic code and now you have to use the webDriverSession field as described here: https://github.com/intuit/karate/tree/master/karate-core#webdriversession
It's a bit complicated with all the brackets, but basically you should do something like:
* configure driver = { type: 'geckodriver', webDriverSession: { "capabilities": { "alwaysMatch": { "moz:firefoxOptions": { args: ["-headless"] } } } } }
See also https://developer.mozilla.org/en-US/docs/Web/WebDriver/Capabilities/firefoxOptions#Example

@paaco : Thank you very much for this. This was really helpful and it works for me now with the webDriverSession field. Thank you once again.


Clean up

Changes

Changed the discover icon from the "eye" to the OpenBazaar tent
Cleaned up some copy
Cleaned up styling in chat

I didn't end up making changes to variants. I tried, but couldn't get them working. I will log a few issues that we can clean up later (after 2.0 release)

This looks good. @rmisio if you sign off as well we can merge it in.


[draft] [poc] Use fixed storage slots

This fixes storage slot locations to their current locations for zora-1155-contracts.
Pattern is simply to use storage.slot = SLOT to match the current storage slots statically across the contracts.
Unfortunately solidity does not have a native way of setting a storage slot but this seems to work.
Primitive and sequential values are wrapped in a struct in order to get the necessary storage ref to them.
This is a proof of concept and should be investigated further.

Closing as this approach is just a draft.


Update tests run in CI for Token

The Chore
Now that we have the ability to run tests using tags we no longer need to only run view tests. We should ideally have smoke tests which run on PR that cover all the critical components of the app.
Tasks

[ ] Reduce number of view tests which cover the same components
[ ] Add a small number of flow tests to increase cover

Additional details / background info
Currently Governance flows are failing due to a bug: #1471

Found a bug causing Chrome to crash in CI. The nightly runs are unlikely to pass until this is fixed. #1546


Create card management page (Druckerei) on Self Service Portal

Is your feature request related to a problem? Please describe.
The card management system (“Digital Print Shop”) should allow users to generate PDF files containing an activation code for the KoblenzPass app and a static QR code for identification (hybrid option).

Only PDF files/passes authorized/provided through the user data interface can be created
As with the self-service portal, user must enter the correct data to create the PDF file
The card management system does not allow to invalidate cards

When a card is created from the print shop (Druckerei), an entry must also be created in the assignment table (Zuweisungstabelle) with the UserHash so that these cards can also be deleted via the authorization table (Berechtigungstabelle).
Additional context
TBD?
blocked by #1418

@f1sh1918 is it a duplicate of https://github.com/digitalfabrik/entitlementcard/issues/1419?

@f1sh1918 is it a duplicate of https://github.com/digitalfabrik/entitlementcard/issues/1419?

@seluianova
Yes it looks like this issue can be closed.
In the other one the assignment table between cards and user data is missing but there is also a separate issue for that


Automatic patch generation learned from human-written patches

http://dl.acm.org/citation.cfm?id=2486893

The authors investigated 62,656 human-written patches were collected from Eclipse JDT4. Their new patch generation approach, PAR, was evaluated on 119 real bugs from open source projects.
All materials used in this paper and detailed patch generation results are available: https://sites.google.com/site/autofixhkust/


Group messages. - adding recipients

Create message. Add person on. Add another single person on. Add existing grouped people -- resets to just that last group of people.
Have to add people 1 at a time. Cannot add multiples without it resetting.

This was implemented this way on purpose, specifically in the beta at the moment.
If you select a group chat, it will clear the selected participants and replace it with just the group members.
How often are you trying to create chats with a group + individuals? I found/thought that I was more likely to select a user, then realize I already had a group chat with them, so selecting the group chat overrode the previous selection.

I have a team of 26 I manage and I create various groups several times a day. Often adding other teams or members and then combining with others.
I personally used the ability to add the other groups often. Adding 20+ individually is time consuming and can lead to missing people. Which is enough to add later.

I have a team of 26 I manage and I create various groups several times a day. Often adding other teams or members and then combining with others.
I personally used the ability to add the other groups often. Adding 20+ individually is time consuming and can lead to missing people. Which is enough to add later.

ok, i think it's something im ok with reverting back to the original case

I have a team of 26 I manage and I create various groups several times a day. Often adding other teams or members and

If you really need it reverted now, you can go back onto the stable release.


what is with encrypting data protection keys at rest

From my understanding setting the key persistence location turns off default key encryption at rest.
what does "rest" mean?
why should we do it? Is it because the persisted key is not encrypted form? what are the vulnerabilities?

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

ID: 99307104-c0d7-3e82-f0b2-af61dcfe072a
Version Independent ID: 315c02cb-54c4-e386-da85-6e4311ca10c2
Content: Share cookies among apps with ASP.NET and ASP.NET Core
Content Source: aspnetcore/security/cookie-sharing.md
Product: aspnet-core
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Hello and Happy Holidays! ⛄ @hoksource ...

setting the key persistence location turns off default key encryption at rest.

Yes ... the info in the Warning messages at the tops of Key encryption at rest in ASP.NET Core and Key storage providers in ASP.NET Core are valid according to engineering. The default they reference is the in-memory provider. Actually, that provider isn't actually listed (perhaps, it should be).
You are correct about "at rest" ... it means that the keys aren't encrypted where they're stored.
The vulnerability for not encrypting at rest is that anyone with access to the key storage location can get the keys and then decrypt anything encrypted by Data Protection.
The current advice is always to encrypt at rest when possible. In addition to those Warning messages ...

Keys should be protected at rest when possible. The system should figure out an appropriate default protection mechanism and apply it automatically.

From: https://docs.microsoft.com/aspnet/core/security/data-protection/introduction#design-philosophy
@hoksource Leave this open. I'd like to talk to the team about this after the holidays (after Jan. 2).

Looking over the topics, I think it's ok as we have it.
I'm going to close as answered, but we can re-open to discuss further if needed.


can't get it working with 4.4.12

I'm still getting default mongoose messages;
{ [MongoError: E11000 duplicate key error index: buharmetre-development.users.$username_1 dup key: { : "Bonesoul" }]
  name: 'MongoError',
  message: 'E11000 duplicate key error index: buharmetre-development.users.$username_1 dup key: { : "Bonesoul" }',
  driver: true,
  code: 11000,
  index: 0,
  errmsg: 'E11000 duplicate key error index: buharmetre-development.users.$username_1 dup key: { : "Bonesoul" }',
  getOperation: [Function],
  toJSON: [Function],
  toString: [Function] }

and yes i've plugged it as a plugin already;
var beautifyUnique = require('mongoose-beautiful-unique-validation');
var UserSchema = new mongoose.Schema({
	username: { type: String, unique: true, trim: true, required: true },
	email: { type: String, unique: true, trim: true, required: true, lowercase: true }
});

UserSchema.plugin(beautifyUnique); // beatuify unique errors.

Hello @raistlinthewiz, which version are you using?
We have made some unpublished changes that could solve this kind of bugs, could you try using the version from GitHub directly?

Hi @raistlinthewiz, sorry to bug you with that, but can you check that this still happens with release 3.0.0 that I just published? Thank you. :)

Hi @raistlinthewiz, I'm closing this issue as I can't reproduce and I don't have any feedback. Please feel free to leave a comment if you want it re-opened. :)


Added UI/UX Design challenges under README.md

Added UI/UX Design challenges under README.md

Hello @jfmartinz , I've made the changes you've asked me to.


Installation stuck on Downloading mono framework installer (OSX)

Hi,
I'm trying to install on OSX 10.11.6. I have Mono installed via Homebrew. The installer stucks when it tries to download Mono. Any idea?
Thanks!

Nevermind. It just took forever to download Mono... It worked when I forgott about the installer in the background while making coffee. Sorry about that.


\bigg with trailing whitespace

\bigg {\uparrow } throws an error (Missing or unrecognized delimiter) while real TeX seems fine with the trailing whitespace.
Note that  \bigg {\uparrow} works fine in MathJax..

This is due to line 2041 in the TeX input jax.  It should probably be
         else if (c === "{" && braceOK) {
           this.i--;
           c = this.GetArgument(name).replace(/^\s+/,'').replace(/\s+$/,'');
         }

in order to remove surrounding white space.

Thanks, Davide!


enhancement of social icons

What is the change?
enhancement of social icons
Related issue?
close: #820
How was it tested?
I have tested it in my local system .
Checklist:
Before you create this PR, confirm all the requirements listed below by checking the checkboxes [x]:

[x] Have you followed the Contribution Guidelines while contributing.
[x] Have you checked there aren't other open Pull Requests for the same update/change?
[x] Did you lint your code before making the Pull Request?
[x] Have you made corresponding changes to the documentation?
[x] Your submission doesn't break any existing feature.
[x] Have you tested the code before submission?

Screenshots or Video:
Add a screenshot or demo video if appropriate.
https://user-images.githubusercontent.com/67833077/117249251-e5c53e80-ae5e-11eb-88a2-9e42333cdfe8.mp4

This doesn't look like an enhancement, they were already colored, you jest removed the color added them on hover..
I liked that they had color by default instead of on-hover.

One thing I do notice that the icons don't have the original color that they should have like the instagram icon has a yellowish color that is not the default color it should have. You can work on the icons by adding the correct colors to them.
Closing this PR for now, comment to reopen if required


MODIFIED BOT_CORE.PY

Hi, i have modified some commands code in Bot_Core.py for Coop support unit purpose (Shaman), how to apply into the bot as i did press saved but the bot still not follow my code...
I modified the code to priority copy shaman unit with harleyquin instead of copy itself.
Thanks for your reply in advanced....

You would have to add the shaman to the series list somewhere else in the code. Changing the unit there will only select it if it's already in the special merge series.

You would have to add the shaman to the series list somewhere else in the code. Changing the unit there will only select it if it's already in the special merge series.

Hi Jim, can you advise which line have to add shaman in the code please... i tried to modified the code in special merge but harlequin still not copying shaman

What units are you using in the deck?

What units are you using in the deck?

Support decks...
(Shaman, cauldron, harlequin, witch, demonologist)
Possible to let the deck only copy shaman? i run bot more on PVE only

If you have shaman as the DPS unit it should work. It will try to keep the
cauldrons at 4 only.
If I'm honest the whole script needs replacing and keeping more simple unit
merging.
On Fri, 2 Feb 2024, 00:47 hatehackppl, @.***> wrote:

What units are you using in the deck?
Support decks...
(Shaman, cauldron, harlequin, witch, demonologist)
Possible to let the deck only copy shaman? i run bot more on PVE only
—
Reply to this email directly, view it on GitHub
https://github.com/Jimm1432/Rush-Royale-Bot/issues/15#issuecomment-1922584400,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AI2CHWOM7TFBEMMY7JCASODYRQZQXAVCNFSM6AAAAABBD47IYWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRSGU4DINBQGA
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

If you have shaman as the DPS unit it should work. It will try to keep the cauldrons at 4 only. If I'm honest the whole script needs replacing and keeping more simple unit merging.
…
On Fri, 2 Feb 2024, 00:47 hatehackppl, @.> wrote: What units are you using in the deck? Support decks... (Shaman, cauldron, harlequin, witch, demonologist) Possible to let the deck only copy shaman? i run bot more on PVE only — Reply to this email directly, view it on GitHub <#15 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AI2CHWOM7TFBEMMY7JCASODYRQZQXAVCNFSM6AAAAABBD47IYWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRSGU4DINBQGA . You are receiving this because you modified the open/close state.Message ID: @.>

I tried to use back clean script and set shaman as DPS. Unfortunately Harleyquin still not copying shaman, it only merge itself. I think have to modify some script in core.py to let harleyquin copy only shaman. But i have no idea which section which line i have to modify honestly...

You need to add it to the special merge list, I'll have a look and see if I
can make you a core.py for your deck but it won't work for any other decks
On Fri, 2 Feb 2024, 23:53 hatehackppl, @.***> wrote:

If you have shaman as the DPS unit it should work. It will try to keep the
cauldrons at 4 only. If I'm honest the whole script needs replacing and
keeping more simple unit merging.
… <#m_1734796800078743494_>
On Fri, 2 Feb 2024, 00:47 hatehackppl, @.> wrote: What units are you
using in the deck? Support decks... (Shaman, cauldron, harlequin, witch,
demonologist) Possible to let the deck only copy shaman? i run bot more on
PVE only — Reply to this email directly, view it on GitHub <#15 (comment)
https://github.com/Jimm1432/Rush-Royale-Bot/issues/15#issuecomment-1922584400>,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AI2CHWOM7TFBEMMY7JCASODYRQZQXAVCNFSM6AAAAABBD47IYWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRSGU4DINBQGA
https://github.com/notifications/unsubscribe-auth/AI2CHWOM7TFBEMMY7JCASODYRQZQXAVCNFSM6AAAAABBD47IYWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRSGU4DINBQGA
. You are receiving this because you modified the open/close state.Message
ID: @.>
I tried to use back clean script and set shaman as DPS. Unfortunately
Harleyquin still not copying shaman, it only merge itself. I think have to
modify some script in core.py to let harleyquin copy only shaman. But i
have no idea which section which line i have to modify honestly...
—
Reply to this email directly, view it on GitHub
https://github.com/Jimm1432/Rush-Royale-Bot/issues/15#issuecomment-1924915864,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AI2CHWJXGCP4ERLUWHQ37N3YRV36VAVCNFSM6AAAAABBD47IYWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRUHEYTKOBWGQ
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

You need to add it to the special merge list, I'll have a look and see if I can make you a core.py for your deck but it won't work for any other decks
…
On Fri, 2 Feb 2024, 23:53 hatehackppl, @.> wrote: If you have shaman as the DPS unit it should work. It will try to keep the cauldrons at 4 only. If I'm honest the whole script needs replacing and keeping more simple unit merging. … <#m_1734796800078743494_> On Fri, 2 Feb 2024, 00:47 hatehackppl, @.> wrote: What units are you using in the deck? Support decks... (Shaman, cauldron, harlequin, witch, demonologist) Possible to let the deck only copy shaman? i run bot more on PVE only — Reply to this email directly, view it on GitHub <#15 (comment) <#15 (comment)>>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AI2CHWOM7TFBEMMY7JCASODYRQZQXAVCNFSM6AAAAABBD47IYWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRSGU4DINBQGA https://github.com/notifications/unsubscribe-auth/AI2CHWOM7TFBEMMY7JCASODYRQZQXAVCNFSM6AAAAABBD47IYWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRSGU4DINBQGA . You are receiving this because you modified the open/close state.Message ID: @.> I tried to use back clean script and set shaman as DPS. Unfortunately Harleyquin still not copying shaman, it only merge itself. I think have to modify some script in core.py to let harleyquin copy only shaman. But i have no idea which section which line i have to modify honestly... — Reply to this email directly, view it on GitHub <#15 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AI2CHWJXGCP4ERLUWHQ37N3YRV36VAVCNFSM6AAAAABBD47IYWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRUHEYTKOBWGQ . You are receiving this because you modified the open/close state.Message ID: @.**>

Thank you Jim, much appreciated

@Jimm1432 sorry for bothering, im still not able to find the option to let harlyquin copy shaman code. could you help please

Sorry for the very late reply I just haven't had the time to be doing things with the computer, a simple solution would be to just rename the shaman icon to demon_hunter in all units. Then add it as DPS unit in your profile. You might need to edit so it will merge at rank 4+ and not start saving them.
One day soon I want to totally scrap the current unit merge lists and streamline it somehow, it's so messy and slow. I'll be updating for the premium upgrade, it will claim the ads and buy the  cards. If I can get it working correctly I'll add using sands of time with a lose in clan tournament.

You are the best!


ChattableManager should utilize CollisionShape to find nearby nodes

Some parts of the overworld code iterate over all nodes in a group, such as 'Chattables', to find whether a nearby node can be chatted with. This checks many unnecessary nodes which are very far away. The code could be improved by using CollisionShape nodes to find nodes near enough to chat with.
This is mostly for performance reasons, but also for code cleanliness. The idea of "find a nearby node in a certain radius" is exactly what CollisionShape is for, there's no reason to reinvent the wheel.

ChattableManager no longer exists, as the "free roam mode" has been removed.


ChatDown [Typing] not recognized when compiling .transcript

When attempting to compile .transcript file from BookTable-happyPath.chat in the Chatdown/Examples folder, CLI outputs this error:
skipping unknown tag [Typing]
Similarly produces the same error when manually trying to use the [Typing] tag elsewhere.

Fixed in latest update


Add output path argument

iamfplayer always writes the output .wav file to the same directory as the input .iamf file.
It is useful for users to be able to control this with a flag. I think it is OK to default to the current behavior if the flag is not present.

@jwcullen
Thanks for your suggestion, you idea makes it be more flexible for users.
I will improve this.
Thanks.

fixed by #104


Final pass over GPflow notebooks

Various minor edits/typo fixes, plus revert UK spelling to US as previously discussed.
See https://prowlerio.atlassian.net/browse/PTKB-7779

Note: would be good if we could apply the same changes to the develop-2.0 branch (which will be The Future(tm), once we've managed to squash all remaining bugs...), not sure if git cherry-pick would work though ?

Hi @akhayes, I'm afraid this PR got caught by the move to gpflow2 -- any chance you could fix up this PR to cleanly apply on top of now-current develop (previously develop-2.0)?

Hi @akhayes, I'm afraid this PR got caught by the move to gpflow2 -- any chance you could fix up this PR to cleanly apply on top of now-current develop (previously develop-2.0)?

I thought I'd sorted this out with Artem the other day :-(
But yes I can fix - I currently have a fork of GPflow/GPflow at akhayes/GPflow, what do I need to do to make sure that I am working from the right version?

are these changes the same as in #1146 ? (in which case, can this PR be closed in favour of #1146?)

Closed in favour of #1165


Duplicate Array node outputs

I just updated the plugin and it seems quite a few things have changed.  Anyways,  am trying to add a "MakeArray" node,  but it returns duplicate outputs
graph = blueprint.UberGraphPages[0]
arrayNode = graph. graph_add_node(K2Node_MakeArray)
#set positions here
arrayNode. NumInputs = 5
arrayNode. node_allocate_default_pins()
It returns duplicate [0] elements as well as 2  outputs

Just did a compile,  even though the nodes are blue (staticmeshcomponent),  blueprint error says the array is undetermined. So How do I set it to a specific type,  in this case a static mesh component

Okay!  Am answering my own question.  I ended up using this inside the loop to get the meshes.
node = K2Node_VariableGet()
node. VariableReference = MemberReference(MemberName = "objName", bSelfContext = True)
And it works. Am bummed, Variable_get? How to use it

Okay!  Am answering my own question.  I ended up using this inside the loop to get the meshes.
node = K2Node_VariableGet()
node. VariableReference = MemberReference(MemberName = "objName", bSelfContext = True)
And it works. Am bummed, Variable_get? How to use it
And how to set the name of a component and have it update on the blueprint. It updates, but "internally",

Okay!  Am answering my own question.  I ended up using this inside the loop to get the meshes.
node = K2Node_VariableGet()
node. VariableReference = MemberReference(MemberName = "objName", bSelfContext = True)
And it works. Am bummed, Variable_get? How to use it
And how to set the name of a component and have it update on the blueprint. It updates, but "internally",

Sorry, can you close this issue and open a new one with the specific question ? Sorry but i completely lost your objective :) Thanks a lot


Use React as peer dependency

Hi, thanks for building this package! It's exactly what I'm looking for.
When I use the Wrapper component in my application, I get the following error when it is rendered:
Uncaught Error: Invalid hook call. Hooks can only be called inside of the body of a function component. This could happen for one of the following reasons:
1. You might have mismatching versions of React and the renderer (such as React DOM)
2. You might be breaking the Rules of Hooks
3. You might have more than one copy of React in the same app
See https://reactjs.org/link/invalid-hook-call for tips about how to debug and fix this problem.

I did some searching and found that a common cause for this problem is when there are two separate versions of React in use in your application. I checked in my project's yarn.lock and after installing this package, there's a new entry there for React, because my application depends on an older version of React than this package does:
<EMAIL_ADDRESS><EMAIL_ADDRESS>  version "16.8.6"
  resolved "https://registry.yarnpkg.com/react/-/react-16.8.6.tgz#ad6c3a9614fd3a4e9ef51117f54d888da01f2bbe"
  integrity sha512-pC0uMkhLaHm11ZSJULfOBqV4tIZkx87ZLvbbQYunNixAAvjnC+snJCg0XQXn9VIsttVsbZP/H/ewzgsd5fxKXw==
  dependencies:
    loose-envify "^1.1.0"
    object-assign "^4.1.1"
    prop-types "^15.6.2"
    scheduler "^0.13.6"

-- New entry
<EMAIL_ADDRESS>  version "17.0.2"
  resolved "https://registry.yarnpkg.com/react/-/react-17.0.2.tgz#d0b5cc516d29eb3eee383f75b62864cfb6800037"
  integrity sha512-gnhPt75i/dq/z3/6q/0asP78D0u592D5L1pd7M8P+dck6Fu/jJeL6iVVK23fptSUZj8Vjf++7wXA8UNclGQcbA==
  dependencies:
    loose-envify "^1.1.0"
    object-assign "^4.1.1"

After taking a look at the package.json here, I see that React is in dependencies. I did some more reading and I found some recommendations that React should be set as a peer dependency.
I think this change alone wouldn't make it possible to use this package in my application as it stands, because we aren't on React 17 yet. If, through peer dependencies, this package also supported react 16.8 and above, I could use it! But that's another topic, and not a bug per se.
I haven't taken the time to create a sample app to reproduce this issue, but it should be reproducible by using this library in any application that has a different version of React as a regular dependency.
Let me know if any other details would be helpful! Thank you.

@jpoehnelt Thank you so much! I moved forward temporarily by mimicking this library in a component of my own, but as soon as I finish my current work I'll give this a shot again and follow up.

Hi @jpoehnelt following up as requested -  1.1.1 works perfectly! Thanks again for addressing this and changing the minimum version.

@LucasBrandt Can you share why you chose this library instead of one of the react component libraries? Thanks!

@jpoehnelt Sure - My project was previously using fullstackreact/google-maps-react, which had been giving us no problems.
The project uses react-geosuggest to allow selecting a location from the Google Places API throughout the application. There was a lot of code duplication doing that, so I wanted to create a reusable component to centralize some Geosuggest-related handling and styling. The library I was using previously didn't work for this use case because it has an unresolved issue that when used to wrap a component, it would wrap it in an unstyleable div. That became an issue for my use case because the reusable component would need to be styleable, so I had to find something new.
I was looking for a simple library with the ability to delay rendering a component until the Google Maps JS library has been loaded. I was hoping to find something that makes it easy to render a placeholder in loading and error states, had good docs, and which seemed active in case there were issues to resolve later down the line. I didn't find many options, but this one seemed to fit the bill and you knocked it out of the park in terms of support for the issue!

Thanks for the response!


trackScreenView doesn't track on iOS

"react-native-google-analytics-bridge": "^5.3.3",
"react-native": "0.48.3",
platform: iOS
tracker.trackScreenView('Home') doesn't track anything in Google Analytics.
tracker.trackEvent('Actions', 'Successful action') works just fine, but I cannot track screen views.

I'm seeing this too.
@slickdev have you set GAILogLevel to "4" (Verbose) in your Info.plist?

@AlexMouton I haven't tried that. I can look into it.

I am seeing the hit sent. So perhaps its a google issue?
"VERBOSE: GoogleAnalytics 3.17 -[GAIBatchingDispatcher dispatchWithCompletionHandler:] (GAIBatchingDispatcher.m:633): Sending hit(s) GET: https://ssl.google-analytics.com/collect...
2017-09-21 14:23:36.814739-0700 RealTalk[3232:826365] INFO: GoogleAnalytics 3.17 -[GAIBatchingDispatcher didSendHits:response:data:error:] (GAIBatchingDispatcher.m:226): Hit(s) dispatched: HTTP status -1
2017-09-21 14:23:36.817889-0700 RealTalk[3232:826442] INFO: GoogleAnalytics 3.17 -[GAIBatchingDispatcher deleteHits:] (GAIBatchingDispatcher.m:530): hit(s) Successfully deleted
2017-09-21 14:23:36.834387-0700 RealTalk[3232:826442] INFO: GoogleAnalytics 3.17 -[GAIBatchingDispatcher didSendHits:] (GAIBatchingDispatcher.m:237): 1 hit(s) sent

HTTP status -1 yields some stack overflow, but no resolutions.

It's probably something to do with your Google Analytics views (although I cannot know what). Have you made an own app view in Google Analytics?
If trackEvent is received, there is no reason why trackScreen should not.

What do you mean by app view? I created a "web property" in Google Analytics and copied the UA code. It's tracking events properly. Is there another step I should be doing that I missed?

Well, I'm unsure if it is important or not. But I usually create an App-view under the Web-property:
https://support.google.com/analytics/answer/1009714
Select App in step 4.

We are experiencing the same issue, we ended up creating a new property just for Mobile. Creating the view will end up showing events only and no views we currently don't know why. If you create a new view for mobile on an already existing web property that was only configured for web this issue happens at least for us.

@indieOcean recommendation works for me.

@indieocean Could you explain what do you mean by: "we ended up creating a new property just for Mobile"?
We encounter the same issue (no page/screen tracking) with a newly created GA property of type Website.
If I choose the Mobile App type, GA ask me for my Firebase Apps and don't give me a tracking number :-(

Same issue. trackScreenView() and trackScreenViewWithCustomDimensionsValues hits don't show up anywhere in GA from what we can see.
trackEvent() works but for some reason shows under a (not set) label.

Any Progress here?

Do not select Mobile app type property, that is Firebase default as you say. Create a new Web property, but add an App view: https://support.google.com/analytics/answer/1009714 (That is to say select App in step 4). You can then set that as the default view on the new property.

hrm... I'm also running into this issue, but on Android.
GoogleAnalytics.trackScreenView will break the app and error out. The error thrown on the screen seems to be
No static method zzalv()Lcom/google/android/gms/common/util/zzd; in class Lcom/google/android/gms/common/util/zzh; or its super classes (declaration of 'com.google.android.gms.common.util.zzh' appears in /data/app/com.my.app.debug-1/base.apk

I was troubleshooting a similar issue on Android this morning.  In looking at the adb logcat, I noticed that it told me that it wasn't sending my stuff off because dry run was enabled.  I'd never set it.  Once I explicitly told the service to disable dry run, I started seeing page logs in Analytics Realtime.
Since the way the analytics service is instantiated on the Android side involves grabbing the react context, I thought perhaps that may have had something to do with the erroneous initial setting.
Anyway, try explicitly setting dry run to false.

I am having the same issue and with the same RN version. What I found is that Android works fine, and that if I create a new View for App, I do get less results which I am guessing is the view filtering the Android hits and showing me the iOS ones, however now I cannot see Android data and the "Screen" show empty.
Isn't this a bug? Or the thing is that we shouldn't be using trackView at all? Having to check the view on the events page is counter intuitive.

I should probably add something in the README about waiting a bit I guess.
I haven't yet encountered a single case where this is either;

Not defined an App View in GA (sudden limitation Google has imposed, not this library)
or, just have to wait a bit, everything outside of Real Time takes a bit of time.


Unable to load AWS credentials from any provider in the chain

So I am working on setting up TitanDB in AWS. I am following the set up instructions as closely as I can (this tutorial and accompanying set up resources need improvement). I deployed my cloud resources using the CloudFormation stack creation template provided in this repository.
When I connect to the gremlin server via my local gremlin console, it appears that Java can't find the AWS credentials. The way I dug this up was to run:
 :> g = TitanFactory.open("/usr/local/packages/dynamodb-titan100-storage-backend-1.0.0-hadoop1/conf/gremlin-server/dynamodb.properties")
The exception comes back as:
org.apache.tinkerpop.gremlin.groovy.plugin.RemoteException: Unable to load AWS credentials from any provider in the chain
If I am not mistaken, this can only happen if it is not found in Environment variables, Java system properties, The default credential profiles file, Amazon ECS container credentials, and Instance profile credentials fail to yield information on the credentials. (http://docs.aws.amazon.com/sdk-for-java/v1/developer-guide/credentials.html)
This seems like unexpected behavior, where the CloudFormation template should have made sure that credentials would be available to Gremlin when started.
What is the expected behavior here? Do I need to include credentials in the dynamodb.properties file? Did I do something wrong to screw up the instance profile? How would I verify this kind of mistake?

The issue that I had was that the EC2 instance did not have permission to assume the IAM role that I specified in the CloudFormation template.
Others who have this issue can verify that this is the case by running the following commands on their EC2 instance:
curl http://<IP_ADDRESS>/latest/meta-data/iam/security-credentials/<NAME OF IAM ROLE>
Where  is the name of the role you created for the CloudFormation template. The response should look something like this if there is an issue:
{
  "Code" : "AssumeRoleUnauthorizedAccess",
  "Message" : "EC2 cannot assume the role Titan-Cloudformation-Builder.  Please see documentation at http://docs.amazonwebservices.com/IAM/latest/UserGuide/RolesTroubleshooting.html.",
  "LastUpdated" : "2017-05-27T00:54:26Z"
}

The way that I resolved this was to go to the IAM Mgmt Console, find the role, and adjust the Trust Policy such that the Statement array now included the following:
{
      "Effect": "Allow",
      "Principal": {
        "Service": "ec2.amazonaws.com"
      },
      "Action": "sts:AssumeRole"
}

Had to wait 5 minutes before the effect took place. Verified that the change was successful by running the following once again:
curl http://<IP_ADDRESS>/latest/meta-data/iam/security-credentials/<NAME OF IAM ROLE>
Which gave me some json that indicated that the role had now been assumed.
I then had to restart my gremlin server:
sudo service gremlin-server stop && sudo service gremlin-server start
Titan then began doing it's thing, I saw the tables show up in my DynamoDB console, and on to bigger and better things.
I'm not sure if this was the "AWS way", so I would love some feedback if any care to give it.

The documentation needs to improve on how the IAM role the instance profile assumes needs to be created. Regarding your specific issue, the reason the lack of STS AssumeRole permission is not caught is twofold:

AWS CloudFormation parameters do not support IAM roles as an AWS specific parameter type, as is the EC2 key pair name.
The user-data script does not check for the presence of an assumable role. Perhaps the user-data script should fail if STS cannot assume the specified role. I will investigate this check in a soon to come PR.

It appears that CloudFormation does not check that STS trust provider is ec2.amazonaws.com when creating the AWS::IAM::InstanceProfile resource. AFAIK InstanceProfiles are only useful in an EC2 context, so perhaps this check is useful. In my work so far, I found that creating a stack with a role with anything but a ec2.amazonaws.com identity provider causes the EC2 instance to be unreachable after startup. I think the best thing to be done in this case is to improve the documentation about the IAM roles you need to create.
@aaronlangford31 I suggest you take the JanusGraph compatibility available on the master branch out for a spin. CloudFormation process is smoother, and incorporates support for recent versions of TinkerPop.


stop parameter values don't always stop generation

Stop words don't always stop generation

@jmorganca Here is an example of where the stop param was given but Ollama failed to stop generation:
% curl -d '{"prompt":"const primes=[1,2,3,","model":"codellama:7b","options":{"num_ctx":100,"stop":["\n"]}}' http://localhost:11434/api/generate
{"model":"codellama:7b","created_at":"2023-08-30T04:14:43.730248Z","response":" I","done":false}
{"model":"codellama:7b","created_at":"2023-08-30T04:14:43.780393Z","response":" believe","done":false}
{"model":"codellama:7b","created_at":"2023-08-30T04:14:43.829942Z","response":" you","done":false}
{"model":"codellama:7b","created_at":"2023-08-30T04:14:43.879305Z","response":" meant","done":false}
{"model":"codellama:7b","created_at":"2023-08-30T04:14:43.928631Z","response":" to","done":false}
{"model":"codellama:7b","created_at":"2023-08-30T04:14:43.978099Z","response":" write","done":false}
{"model":"codellama:7b","created_at":"2023-08-30T04:14:44.028311Z","response":":","done":false}
{"model":"codellama:7b","created_at":"2023-08-30T04:14:44.12742Z","response":"\n import","done":false}
{"model":"codellama:7b","created_at":"2023-08-30T04:14:44.177132Z","response":" math","done":false}
{"model":"codellama:7b","created_at":"2023-08-30T04:14:44.276432Z","response":"\n pr","done":false}
{"model":"codellama:7b","created_at":"2023-08-30T04:14:44.326199Z","response":"imes","done":false}
{"model":"codellama:7b","created_at":"2023-08-30T04:14:44.376749Z","response":" =","done":false}
{"model":"codellama:7b","created_at":"2023-08-30T04:14:44.427742Z","response":" []","done":false}
{"model":"codellama:7b","created_at":"2023-08-30T04:14:44.527658Z","response":"\n for","done":false}
{"model":"codellama:7b","created_at":"2023-08-30T04:14:44.577572Z","response":" i","done":false}
{"model":"codellama:7b","created_at":"2023-08-30T04:14:44.628179Z","response":" in","done":false}
{"model":"codellama:7b","created_at":"2023-08-30T04:14:44.679126Z","response":" range","done":false}
{"model":"codellama:7b","created_at":"2023-08-30T04:14:44.73Z","response":"(","done":false}
{"model":"codellama:7b","created_at":"2023-08-30T04:14:44.781045Z","response":"1","done":false}
{"model":"codellama:7b","created_at":"2023-08-30T04:14:44.831974Z","response":"0","done":false}
^C

Running on the latest version of ollama (just updated) on macOS.


[11.x] Add BackedEnum support to Gate methods

Hi,
Using Enums enhances code readability and reduces typos; it also improves IDE support by providing better autocomplete
This change is backward compatible and shouldn't bring any breaking changes.
enum Abilities: string {
    case VIEW_DASHBOARD = 'view-dashboard';
    case EDIT = 'edit';
    case UPDATE = 'update';
}

// Before
Gate::define('view-dashboard', function (User $user) {
    return $user->isAdmin;
});

Gate::authorize('view-dashboard');
Gate::inspect('view-dashboard');
Gate::check('view-dashboard');
Gate::any(['edit', 'update], $post);
Gate::none(['edit', 'update]], $post);
Gate::allows('update', $post)
Gate::denies('update', $post)

// After
Gate::define(Abilities::VIEW_DASHBOARD, function (User $user) {
    return $user->isAdmin;
});

Gate::authorize(Abilities::VIEW_DASHBOARD);
Gate::inspect(Abilities::VIEW_DASHBOARD);
Gate::check(Abilities::VIEW_DASHBOARD);
Gate::any([Abilities::EDIT, Abilities::UPDATE], $post);
Gate::none([Abilities::EDIT, Abilities::UPDATE], $post);
Gate::allows(Abilities::UPDATE, $post)
Gate::denies(Abilities::UPDATE, $post)

@diaafares - once these 2 (https://github.com/laravel/framework/pull/52677, https://github.com/laravel/framework/pull/52679) got merged together they led to build failures due to the enum collide of
enum AbilitiesEnum: string {
    case VIEW_DASHBOARD

@iBotPeaches I added a new PR to fix the issue
https://github.com/laravel/framework/pull/52683
Can you please check it to see if it solves the problem?


Export-Excel changes SQL date datatype to datetime in excel

I am creating a series of spreadsheets for users based on a Code assigned to each person, i.e. all the rows from a table where MRPC = x.
I am using an array and a foreach and it works perfectly except the date columns from excel are datetime in the resulting spreadsheets. Is there a way to force excel to conform to the datatype sent to it?

Have you tried using the Send-SQLDataToExcel instead of Export-Excel - this drops a table straight into the spreadsheet
Export-Excel converts dates / times / datetime objects to a floating point number and formats them correctly. The easiest thing may be to format the columns as date after doing the import.

@jhoneill does Send-SQLDataToExcel do any data type translation. I'm guessing the "drops it straight in" does the data as is?

Send-SQLDataToExcel hands export-excel a dataTable object, and a stack of parameters.
When Export-Excel sees one of those it just calls EpPlus's  LoadFromDataTable method - it's as light touch as possible ....
[checks ... ]
After tossing the data over the wall , Export-Excel then looks for columns in the table whose dataType is dateTime or timespan and sets the format on those in Excel.
But a data table looks like it uses .NET types so it's possible that you can't get a pure date back you only ever get DateTime. Meaning format the columns. is the way forward.

Cool. Figured I'd ask "Joe Walsh" how you played that riff :)


OSSM-2376: Don't start federation controllers until informers have synced (#717)

This is a manual cherry-pick of #717, because an automated one has failed.
This PR slightly differs from the original, because in maistra-2.2 federation controller uses Endpoints instead of EndpointSlices.

/retest


Implement an XML Wrapper Class to hide TinyXML Implementation

TinyXML is a great library and I see no need to change it, however TinyXML's interface doesn't need to be exposed in the public NASD interface.
Justification
I tend to not really care much for the pImpl idiom in open-source projects as it needlessly complicates maintenance (with the benefit of improved compile times and cleaner interfaces with less implementation details that end users don't care about), but in this case it makes perfect sense. It would allow us to switch to a different XML library or even implement our own without the user ever having to recompile all of their code, simply relink.
Requirements
The XML used in NAS2D is fairly simplistic. All definitions almost exclusively use tag attributes instead of tag content and on occasion uses comments.
Any interface will need to provide the user with the ability to add tags, attach attributes to those tags and also provide content for those tags as well as quickly and easily find tags and interpret attributes.

With the latest commit, I've been pouring over the TinyXML interface and I'm thinking I may not need to build a wrapper for it. Instead, I can simplify the interface as much as possible, pull out the C-influenced guts and replace it all with a container implementation.
Can eliminate the need for all of the raw pointers and new/deletes that the library requires.
Once cleaned up and the interface minimized, implementation details can start being hidden. This may be a good place to utilize a pImpl design pattern to hide the details.
Finally, once the interface is fully customized, it can be rolled into the NAS2D namespace.

So definitely a larger project than I expected. I've pulled TinyXML's source files apart and separated everything out into individual source and header files. I removed a lot of unnecessary functionality and pulled support for UTF encoding (for the scope of NAS2D and its uses of XML, UTF encoding is unnecessary and needlessly complicates the code).
Anyway, the intention here is to provide a minimal XML implementation based on well tested and mature code... which means that a lot of functions that aren't needed or used are going to get pulled. Sorry, no need to include functionality that isn't used.

Another benefit of having pulled TinyXML apart is that I discovered the TiXmlHandle class (now XmlHandle) which is far more effective at looking for and navigating elements/nodes than XmlAttributeParser so that code can be effectively pulled with some of its interfaces possibly merged in.

With the latest commit 6738da544acf78c22ba84b549bcf0982a15aa3a3 it's safe to say that the TinyXML integration is now complete.
That stated, some of the documentation still needs work but I don't see a reason to do much changing of the rest of the library. If anything having the library ignore any unknown elements and just simply not process them is an option as well and will remove code that will likely never be executed. This would eliminate the XmlUnknown class, all code to create/parse an XmlUnknown but would require messages be output to indicate that these elements/tags/whatever are explicitly being ignored.


[skmultiflow->creme] Merge k-NN algorithms

This is the beginning of the efforts to merge the lazy module of skmultiflow to creme.neighbors.
This PR depends on #319 and #321, so I am making it a draft. The idea is to get some feedback and plan forward our next steps.
Here is the checklist:

[x] Update basic documentation
[x] Use creme's base classes in the estimators
[x] Make algorithms work in a single-instance basis
[x] Handle data conversion
[ ] Test the algorithms
[ ] Update the examples in the docstring

The kNN algorithms rely on some utility functions from skmultiflow.utils.utils.py. That is why I brought them to creme.utils.skmultiflow_utils.py

Hi @MaxHalford and @jacobmontiel. I changed the usage of collections.deque to utils.Window.
Besides that, I created a circular buffer logic that eliminates the need to perform (the costly) np.delete and np.vstack operations. I still have to do some testing with it. But the basic idea is to manually circle in the matrix an index that indicates where new observations should be inserted and also keep a binary instance mask to filter/select the filled slots.
I believe we should get some nice speedups since numpy does not need to keep reallocating a matrix (window_size, n_features)  every time an insertion takes place.

Good progress @smastelini. When you're done you can remove KNeighborsClassifier and KNeighborsRegressor. We'll have to benchmark what's the fastest way to proceed when dicts are used natively. Note that we could also use utils.VectorDict to speed things up.

Hi @MaxHalford and @jacobmontiel.
I recently updated the KNNRegressor. My next step is going to check the requirements to bring the ADWIN-based k-NN, which is implemented in skmultiflow. It might be worth checking if we want to make more cKDTree's parameters accessible for the user too.

My next step is going to check the requirements to bring the ADWIN-based k-NN, which is implemented in skmultiflow.

This should be done in another PR as I am in the middle of porting ADWIN and the other drift detection methods. As long as we have the basic classes we can move forward.

It might be worth checking if we want to make more cKDTree's parameters accessible for the user too.

+1 on the idea of giving the user this flexibility.

This should be done in another PR as I am in the middle of porting ADWIN and the other drift detection methods. As long as we have the basic classes we can move forward.

Nice! I added the pop and popleft methods in KNeighborsBuffer just to be ready for the future. After the drift detection algorithms are merged, we can easily implement the ADWIN-kNN variants.

+1 on the idea of giving the user this flexibility.

Same here!


How to use globalResources now?

Hello,
I have migrated my old skeleton to the new one and several things are not working anymore.
How shouldI use that now?
import { PLATFORM } from 'aurelia-pal';
export function configure(config) {     config.globalResources(         PLATFORM.moduleName('./bar/nav-bar')    ) }
I am getting the following error:
Unable to find module with ID: components/resources/bar/nav-bar
Can someone help on that?

did you update your router map?
it also needs that PLATFORM.moduleName('jsfilename')
On Tue, Apr 25, 2017 at 9:18 AM, Hugo Marcelo Del Negro <
<EMAIL_ADDRESS>wrote:

Hello,
I have migrated my old skeleton to the new one and several things are not
working anymore.
How you I use that now?
import { PLATFORM } from 'aurelia-pal';
export function configure(config) {
config.globalResources(
PLATFORM.moduleName('./bar/nav-bar')
)
}
I am getting the following error:
Unable to find module with ID: components/resources/bar/nav-bar
Can someone help on that?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/aurelia/skeleton-navigation/issues/813, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AMuN2OnHYp-XP-_HSjTd8MkFtpqnn-iHks5rzfK2gaJpZM4NHeEz
.

--
Josh Sherman
www.joshandmariamusic.com

Yes,
I am just trying what it was in the old way:
  aurelia.use
    .standardConfiguration()
    .developmentLogging()
    .plugin(PLATFORM.moduleName('components/resources'));

Do I need to place them into the ModuleDependenciesPlugin ?

@hmarcelodn That actually looks more like a .feature call in which case including index might work.
See also: https://github.com/aurelia/webpack-plugin/wiki/Managing-dependencies#platformmodulename
  aurelia.use
    .standardConfiguration()
    .developmentLogging()
    .feature(PLATFORM.moduleName('components/resources/index'));

@hmarcelodn seems like @Thanood answered your question.

@Thanood  Yes it worked thanks.
I am having another issue with setRoot:
await aurelia.start();
await aurelia.setRoot(PLATFORM.moduleName('./app/guest/app'));

I'm not sure if setRoot works with relative paths.
Maybe try this:
await aurelia.setRoot(PLATFORM.moduleName('app/guest/app'));

Thanks @Thanood all this worked.


Improve tracking statistics

Since GitHub Insights/Traffic won't allow us to move further than 14 days, we can use our ReadTheDocs webpage to get a better sense of users over time. João had a good idea to do this by making the documentation through the repo not a good place to read/redirect to our RTD page so we can have a better understanding of our user base.

Still, the documentation in the repository is placed where it should. We need to accommodate our practices and everytime we refer to IDPConfGen, always give links related to the RTD pages. In the README of the main repository update the references to the Installation and Usage to the links of RTD.


About the speed

@cydonia999
Amazing working! This is the most accurate Face detecting project I have used  .
But speed is a little slow.
is there any params I can use for increase the speed of detecting?
my mac pro spend 200 seconds to handle a 2880*1680 pic(5M byte).
Is there any params settings or method of modify source code to  ignore  too small and too big scale to scan? I think   ignore  too small and too big scale can increase speed for me
Thank you !!

Hi~Have you solved this problem?

@cydonia999
Amazing working! This is the most accurate Face detecting project I have used .
But speed is a little slow.
is there any params I can use for increase the speed of detecting?
my mac pro spend 200 seconds to handle a 2880*1680 pic(5M byte).
Is there any params settings or method of modify source code to ignore too small and too big scale to scan? I think ignore too small and too big scale can increase speed for me
Thank you !!

Hi, Have you solved this problem?


Remove lastIP from user schema and script to update existing users

Coverage decreased (-0.01%) to 63.757% when pulling 02fb781d634c45d35012ace55fc0a963d02336d4 on remove-lastip-from-schema into 10da779e13067f5eb2637c64c96331b30ec8ba85 on master.


replace all misspelt to_integral() with to_integer() but remain to_integral() marked deprecated

replace all misspelt to_integral() with to_integer() but remain to_integral() marked deprecated

It's to_integral as in integral_type. int as in integer is just one of the possible integral types along with long, long long, short, char, bool and so on. I think to_integral is a better fit that to_integer. What's the reason for changing the name exactly?

Just for note:

in·te·gral
/ˈin(t)əɡrəl,inˈteɡrəl/
adjective
MATHEMATICS
of or denoted by an integer.

So integral is correct.  It's not an integer, but it's denoting an integer.

I didn't notice integral_type, but I think integral_type is OK. As an adjective, integral refers to both integers and integration, which is a bit confusing. However, in to_integral(), integral is obviously a noun, meaning the result of an integration
in·te·gral
I. \ˈin-ti-grəl (usually so in mathematics); in-ˈte-grəl also -ˈtē- also ÷ˈin-trə-gəl\ adjective
 Date: 1551
 1.
   a. : essential to completeness : constituent
     <an integral part of the curriculum>
   b.
     (1) : being, containing, or relating to one or more mathematical integers
     (2) : relating to or concerned with mathematical integrals or integration
   c. : formed as a unit with another part
     <a seat with integral headrest>
 2. : composed of integral parts
 3. : lacking nothing essential : entire
 • in·te·gral·i·ty \ˌin-tə-ˈgra-lə-tē\ noun
 • in·te·gral·ly \ˈin-ti-grə-lē; in-ˈte-grə- also -ˈtē-\ adverb
II. noun
 Date: circa 1741
  : the result of a mathematical integration — compare definite integral, indefinite integral

Moreover, there's a std::to_integer(), so changing to_integral() to to_integer() makes more sense of continuity

I'm not a native speaker. You two are confusing me. :smile:
I'll leave this kind of discussion to you and @OvermindDL1 I can only learn something this time, I don't think I can add much instead.

Btw it's not to_integer, it's to_integer<T> where you specify the desired type.
In this case, it's more similar to an hypothetical to_underlying rather than to_integer<T> since the type is fixed.

Neither am I. I just pay some more attention to these trivial names. I was looking for a method to inspect the inner structure of an entity, and someone let me use to_integer().  However,  what I only found was to_integral(), instead of to_integer(). I thought it was a misspelling in the first place.

Btw it's not to_integer, it's to_integer<T> where you specify the desired type.
In this case, it's more similar to an hypothetical to_underlying rather than to_integer<T> since the type is fixed.

Really? What I found was this function without templates. Users should use a macro ENTT_OPAQUE_TYPE(clazz, type) to specify in the inner type of an entity.
constexpr auto to_integral(const clazz id) ENTT_NOEXCEPT
{
...
}

I googled more just to find that many languages and libraries use to_integer() while far more languages and libraries use to_integral() based on the numbers of results. 46,300(to_integer()) vs  2,040,000(to_integral()).
Should have I closed this PR.

Honestly I'd just leave it as it is, I see integral used in similar contexts elsewhere often enough that it seems clear either way, and not changing it means not surprising users of this library (and I'm not a fan of keeping deprecated functions, even just trampolines, as it adds in more vectors for bugs).  ^.^

Thanks for reporting and for discussing it @qiujiangkun really appreciated. :heart:


Current Session Enabled Change

If EnabledUserNames is empty consider the current user enabled.
Likewise, if the DisabledUserNames is empty do NOT cosider the current user disabled.

@seangwright I have made a minor change to the current session enabled logic so you don't have to explicitly configure users as enabled. Actually, now that I look at it again - there is no need to check if the DisabledUserNames array is empty.


sap_hana_install: Improve the hdblcm configfile processing

We can get the output of hdblcm --dump_configfile_template=... and convert the configfile template into a Jinja2 template so that we get a customized Jinja2 template for each HANA version. This addresses the fact that different HANA versions have a different set of parameters.
So we can run the role with tag sap_hana_install_tag_preinstall or sap_hana_install_tag_create_configfile, then check and if necessary modify the role variables according to the requirements of hdblcm, and finally run the HANA installation with a configfile created from the Jinja2 template.

I implemented the following flow:

Check the existence of file configfile.cfg in the directory configfiles below sap_hana_install_software_extract_directory.
If this file exists, use it for installing SAP HANA by the hdblcm command.
If this file is not present, perform the following three steps:
Get the hdblcm configfile template directly from the hdblcm command.
Convert the configfile template into a Jinja2 template and download it to the control node.
Process the Jinja2 template, using the configured role variables or default settings, to create a customized hdblm configfile.


[Bug]: crash with itemphysics (ClassNotFoundException: EntityItem)

Forge Version
<IP_ADDRESS>60
Galacticraft Version
4.1.3
Log or Crash Report
https://mclo.gs/CH0ImsS
Reproduction steps
download itemphysics and Galacticraft-Continuation, try to launch - crash

for me i don't get crash

compat added in V4.1.4


boards: dk6_qn9090: fixed DTS errors

I removed the lines as commented code should not be in the code-base


saving .npz weights and modulations from experiment_meta_learning.py

Hi, it is not clear to me how to save .npz weights and modulations in this repository.

Hi, are there any issues when you follow the instructions here: https://github.com/deepmind/functa#create-or-download-modulations-for-celeba-hq-64 ? Let me know what the issue is and I'll try to help out.

The code modulation_dataset_writer.py works with your pretrained models. If I want to train a model myself how can I save the weights, modulations and config in such a way that it can be used as input of modulation_dataset_writer.py?

For this you would need to use the checkpoint stored in the directory specified by config.checkpoint_dir in the meta-learning experiment, load the checkpoint, write the config and params as a dict to an npz file, then use this npz file instead of the pretrained npz files. I think the one thing that's not clear is how to load the checkpoint to obtain the config of the experiment and params of the checkpoint. Let me look into this and get back to you.

Thank you very much. Yes the part that you have highlighted is not clear to me

Looking into this, it seems like the checkpointing functionality for the opensourced jaxline is not functioning as-is: https://github.com/deepmind/jaxline/issues/4
However if you look at the latest comment to this issue, there is a link to an example of some deepmind code where you can manually save the config and params: https://github.com/deepmind/deepmind-research/blob/master/adversarial_robustness/jax/experiment.py#L220-L225
You should be able to add similar lines of code inside the step method of experiment_meta_learning.py to store the self.config (this only needs to be stored once) and self._params for the meta_learning experiment, and then use the saved npz files to modulation_dataset_writer.py. Hope that makes sense!

Ok, thank you very much. The lines of code that you mentioned are related to params and state, but the config file cannot be read by means of the code:  https://github.com/deepmind/deepmind-research/blob/master/adversarial_robustness/jax/experiment.py#L220-L225

Yes, but the config dict is stored in self.config of the experiment, so it can be saved in the same way you save the params/state. This is what I meant when I was just saying that the config only needs to be stored once. Also there is no state that is used in the meta learning experiment, so you'd only need the params (saved at every few training iterations) and the config (saved once at the beginning of training). I hope that's clear.

yes, thank you very much. I will try

I have managed to create a similar code compatible with the pretrained models that you have uploaded.
I have not tested yet with the same config that you have uploaded, but the npz file produced is similar.
This is the code I have added in data_utils.py:

from jaxline import utils
from ml_collections import config_dict
import numpy as np
import os
import jax
import pickle
import copy
class NumpyFileCheckpointer(utils.Checkpointer):
"""A Jaxline checkpointer which saves to numpy files on disk."""

def init(self, config: config_dict.ConfigDict, mode: str):
self._checkpoint_file = os.path.join(config.checkpoint_dir,
'checkpoint.npz')
self._celeba_file = os.path.join(config.checkpoint_dir,
'celeba_params_latents.npz')
self._checkpoint_state = config_dict.ConfigDict()
del mode
def get_experiment_state(self, ckpt_series: str) -> config_dict.ConfigDict:
"""Returns the experiment state for a given checkpoint series."""
if ckpt_series != 'latest':
raise ValueError('multiple checkpoint series are not supported')
return self._checkpoint_state
def save(self, ckpt_series: str) -> None:
"""Saves the checkpoint."""
if ckpt_series != 'latest':
raise ValueError('multiple checkpoint series are not supported')
exp_mod = self._checkpoint_state.experiment_module
#print(exp_mod.getattribute)
global_step = self._checkpoint_state.global_step
f_np = lambda x: np.array(jax.device_get(utils.get_first(x)))
f_np2 = lambda x : dict(x)
to_save = {}
to_save2 ={}
#print(self._checkpoint_state)
for attr, name in exp_mod.CHECKPOINT_ATTRS.items():
if name == 'global_step':
raise ValueError(
'global_step attribute would overwrite jaxline global step')
if name =='config':
#print(exp_mod.getitem[attr])
np_params = jax.tree_map(f_np2, getattr(exp_mod, attr))
#np_params = jax.tree_map(f_np, getattr(exp_mod, attr, dict))
else:
np_params = jax.tree_map(f_np, getattr(exp_mod, attr))
to_save[name] = np_params
#print(np_params)
if name == 'params' or name =='config':
to_save2[name] = np_params
#to_save[name] = np_params
to_save['global_step'] = global_step
#print(to_save2)
#with tf.io.gfile.GFile(self._checkpoint_file + '_tmp', 'wb') as fp:
#  print(fp)
#  np.save(fp, to_save)
#with tf.io.gfile.GFile(self._celeba_file + '_tmp', 'wb') as fp:
#  print(fp)
#  np.save(fp, to_save2)
#np.save(self._checkpoint_file, to_save)
a_file = open(self._checkpoint_file, "wb")
b_file =open(self._celeba_file, "wb")
pickle.dump(to_save, a_file)
pickle.dump(to_save2, b_file)
#tf.io.gfile.rename(
#    self._checkpoint_file + '_tmp',
#    self._checkpoint_file,
#    overwrite=True,
#)
#tf.io.gfile.rename(
#    self._celeba_file + '_tmp',
#    self._celeba_file,
#    overwrite=True,
#)
def can_be_restored(self, ckpt_series: str) -> bool:
"""Returns whether or not a given checkpoint series can be restored."""
if ckpt_series != 'latest':
raise ValueError('multiple checkpoint series are not supported')
return tf.io.gfile.exists(self._checkpoint_file)
def restore(self, ckpt_series: str) -> None:
"""Restores the checkpoint."""
experiment_state = self.get_experiment_state(ckpt_series)
#with tf.io.gfile.GFile(self._checkpoint_file, 'rb') as fp:
#ckpt_state = np.load(fp, allow_pickle=True)
#.item()
a_file = open(self._checkpoint_file, "rb")
ckpt_state=pickle.load(a_file)
experiment_state.global_step = int(ckpt_state['global_step'])
exp_mod = experiment_state.experiment_module
print(ckpt_state)
for attr, name in exp_mod.CHECKPOINT_ATTRS.items():
if name != 'config':
#  exp_mod.CHECKPOINT_ATTRS.setitem(attr,ckpt_state[name]
setattr(exp_mod, attr, utils.bcast_local_devices(ckpt_state[name]))
def restore_path(self, ckpt_series: str) -> Optional[str]:
"""Returns the restore path for the checkpoint, or None."""
if not self.can_be_restored(ckpt_series):
return None
return self._checkpoint_file
def wait_for_checkpointing_to_finish(self) -> None:
"""Waits for any async checkpointing to complete."""
@classmethod
def create(
cls,
config: config_dict.ConfigDict,
mode: str,
) -> utils.Checkpointer:
return cls(config, mode)_
Then I have changed the last part of experiment_meta_learning.py:
if name == 'main':
flags.mark_flag_as_required('config')
platform.main(Experiment, sys.argv[1:])
app.run(functools.partial(platform.main, Experiment))
with:
def main(_):
flags.mark_flag_as_required('config')
platform.main(
Experiment,
sys.argv[1:],
checkpointer_factory=data_utils.NumpyFileCheckpointer.create,
)
if name == 'main':
app.run(main)

Nice. Would you be happy for me to close the issue now?

Yes, sure. Thanks


Build fails

Hi,
I unfortunatelly can't get the docker up and running. After I build the Image and gonna start docker-compose it fails:
Creating TonieboxSync ... done
Attaching to TonieboxSync
TonieboxSync    | Traceback (most recent call last):
TonieboxSync    |   File "/tonie/tonie_sync/start.py", line 11, in
TonieboxSync    |     from tonie_sync import TonieSpotifySync
TonieboxSync    |   File "/tonie/tonie_sync/tonie_sync/init.py", line 1, in
TonieboxSync    |     from .sync import *
TonieboxSync    |   File "/tonie/tonie_sync/tonie_sync/sync.py", line 7, in
TonieboxSync    |     from spotdl.command_line.core import Spotdl
TonieboxSync    | ModuleNotFoundError: No module named 'spotdl.command_line'
TonieboxSync exited with code 1
How Do I get this running?
Thanks!

I get the same errors. Any updates availables?
I am searching for a way to sync tonies with spotify...

Hi, thanks for your interest in this project! Actually, I have never really used this code, since I found that the content I was interested in, was not reliably downloaded by Spotdl (mostly audiobooks for children, which are simply not available on youtube...). Spotdl went through a lot of rebranding and recoding, and also some legal issues I guess, so the Spotdl related code for sure needs some updates.
Have you checked @rado0x54 fork of the project? It looks like he did some adjustments to keep spotify working: https://github.com/rado0x54/tonie_sync

and of course the original project which inspired me:
https://github.com/maximilianvoss/musicsync
it is actively maintained and uses a different (more robust) approach for downloading tracks. It appears to be available as docker now as well!
I think I will archive this repo for now... feel free to fork!


Publish actions running in forked repositories

The three publish-related actions contained in the repository should run only on the main repository, not on the forked ones.
The owner of a forked repository should disable all actions, but in this way the Linter and CodeQL actions will be disabled too.
I propose to add an owner check in the publishing actions, so they will be skipped on forked repositories.
E.g.
jobs:
  publish:
    name: Publish on NuGet
    runs-on: ubuntu-latest
    if: github.repository_owner == 'marcominerva'


[Bug]: WithCleanup(true) does not properly work

Testcontainers version
2.4.0
Using the latest Testcontainers version?
Yes
Host OS
Linux
Host arch
x64
.NET version
7.0.100
Docker version
Client:
 Cloud integration: v1.0.29
 Version:           20.10.22
 API version:       1.41
 Go version:        go1.18.9
 Git commit:        3a2c30b
 Built:             Thu Dec 15 22:36:18 2022
 OS/Arch:           windows/amd64
 Context:           default
 Experimental:      true

Server: Docker Desktop 4.16.3 (96739)
 Engine:
  Version:          20.10.22
  API version:      1.41 (minimum version 1.12)
  Go version:       go1.18.9
  Git commit:       42c8b31
  Built:            Thu Dec 15 22:26:14 2022
  OS/Arch:          linux/amd64
  Experimental:     false
 containerd:
  Version:          1.6.14
  GitCommit:        9ba4b250366a5ddde94bb7c9d1def331423aa323
 runc:
  Version:          1.1.4
  GitCommit:        v1.1.4-0-g5fd4c4d
 docker-init:
  Version:          0.19.0
  GitCommit:        de40ad0

Docker info
Client:
 Context:    default
 Debug Mode: false
 Plugins:
  buildx: Docker Buildx (Docker Inc., v0.10.0)
  compose: Docker Compose (Docker Inc., v2.15.1)
  dev: Docker Dev Environments (Docker Inc., v0.0.5)
  extension: Manages Docker extensions (Docker Inc., v0.2.17)
  sbom: View the packaged-based Software Bill Of Materials (SBOM) for an image (Anchore Inc., 0.6.0)
  scan: Docker Scan (Docker Inc., v0.23.0)

Server:
 Containers: 0
  Running: 0
  Paused: 0
  Stopped: 0
 Images: 29
 Server Version: 20.10.22
 Storage Driver: overlay2
  Backing Filesystem: extfs
  Supports d_type: true
  Native Overlay Diff: true
  userxattr: false
 Logging Driver: json-file
 Cgroup Driver: cgroupfs
 Cgroup Version: 1
 Plugins:
  Volume: local
  Network: bridge host ipvlan macvlan null overlay
  Log: awslogs fluentd gcplogs gelf journald json-file local logentries splunk syslog
 Swarm: inactive
 Runtimes: io.containerd.runc.v2 io.containerd.runtime.v1.linux runc
 Default Runtime: runc
 Init Binary: docker-init
 containerd version: 9ba4b250366a5ddde94bb7c9d1def331423aa323
 runc version: v1.1.4-0-g5fd4c4d
 init version: de40ad0
 Security Options:
  seccomp
   Profile: default
 Kernel Version: <IP_ADDRESS>-microsoft-standard-WSL2
 Operating System: Docker Desktop
 OSType: linux
 Architecture: x86_64
 CPUs: 8
 Total Memory: 23.34GiB
 Name: docker-desktop
 ID: FYG7:2I2H:LWWB:Y4VE:53I3:WKTA:64ON:JMU6:IBXY:Y6EG:UFDN:3KBU
 Docker Root Dir: /var/lib/docker
 Debug Mode: false
 HTTP Proxy: http.docker.internal:3128
 HTTPS Proxy: http.docker.internal:3128
 No Proxy: hubproxy.docker.internal
 Registry: https://index.docker.io/v1/
 Labels:
 Experimental: false
 Insecure Registries:
  hubproxy.docker.internal:5000
  <IP_ADDRESS>/8
 Live Restore Enabled: false

WARNING: No blkio throttle.read_bps_device support
WARNING: No blkio throttle.write_bps_device support
WARNING: No blkio throttle.read_iops_device support
WARNING: No blkio throttle.write_iops_device support

What happened?
Configuring a builder for a container with "WithCleanup(true)", does not fully delete the resource.
The error happens when running a test twice. At the first run, the container is started and attached to a specific IP in an associated network (attached with "WithNetwork(network)").
After the test finishes, it starts again with the same setup, only this time starting the container fails, with the following error:
"Docker.DotNet.DockerApiException : Docker API responded with status code=Forbidden, response={"message":"Pool overlaps with other one on this address space"}".
The only way I found to fix this, is manually calling "container.DeleteAsync()" after the test finish, that way the second iteration works as expected.
Relevant log output
No response
Additional information
No response

After the test finishes, it starts again with the same setup

What does that mean? According to the error message it sounds you create a resource with the same (static) configuration again:
Pool overlaps with other one on this address space

Can you please share your test configuration?

What part of the configuration do you want me to share?
And yes, I am indeed running the same resources again with the same IPs, but I would expect it to work since I expect it to clean up on test finish. Only after adding "DeleteAsync" it really deletes everything

What part of the configuration do you want me to share?

Your test code.

The Resource Reaper cleans up the resources after the test processes have been finished. If you would like to clean up during the tests, it is necessary to call DisposeAsync() or DeleteAsync().

I guess that explains the behavior then. By the time the resource reaper really cleans up the resources, the new test starts and tries to bind the container to the same IP as in the previous test run, although the network resources from previous run have not fully cleaned up. Does it make sense that this is what's happening?
Is there a way to block the test until the resource reaper finishes its action?

Does it make sense that this is what's happening?

It is possible - yes, but since I have not seen the test code, I only can guess.

Is there a way to block the test until the resource reaper finishes its action?

Do you use the same network name? I would assume different network names will work.

Is there a way to wait for the Resource Reaper to finish before the test finishes?

No.
We consider it a bad practice to use fixed names (values) for resources. This usually indicates poor design. Depending on fixed names, port bindings, etc. can result in unreliable tests.

Can we consider this as a feature request?

The Resource Reaper (moby-ryuk) container has a graceful period to reconnect, and if the connection is dropped for more than 60 seconds (which indicates that the tests are finished), it starts to clean up the resources. The behavior you are experiencing is by design.
That said, you may want to look into using the network alias feature, which allows you to access containers via their names rather than their IP addresses.
If you still choose to rely on static names, configurations, etc. and do not follow the best practices, you will need to take care of the cleanup by yourself.

Due to internal business logic, I am unable to use network alias where the problem is caused in.

Can you please refer me to a doc that explains on those best practices?
Would it not be reasonable to consider an "on-demand" cleanup by the RR?

Can you please refer me to a doc that explains on those best practices?

The link above shows how to set up the network configuration.

Would it not be reasonable to consider an "on-demand" cleanup by the RR?

As I said, you can disable Ryuk and implement your own logic as you suggested. However, I would like to note that disabling Ryuk may result in leaving resources behind and could potentially mess up the CI and development environment. Ryuk is the only way to cleanup the environment reliable.

I wouldn't want to disable Ryun. As you said it might mess the environment and I would like to make use of it to clean up my resources. Only thing I would expect is some way make it clean up on-demand, even with a static method "await Ryuk.CleanupAsync()" or something similar, instead of manually implementing this cleanup logic.
I would try helping implement this, but currently I'm trying to understand if it's a legitimate feature request. Is there a reason why it couldn't be supported?

Ryuk runs as an independent container ("out of process"). We start it once and never touch it again.
If you'd like, I can take a look at your test and see if there's anything that can be improved in your setup.
I spoke with the other TC maintainers too, and we don't believe that this is a missing feature. From what we can see, it's working as intended and expected, so we don't have any plans to change its behavior.

Thanks for the good intentions, I believe there would be ways to make this work with some workarounds. I'm just trying to avoid them with feature that to me sounds trivial. I'm not trying to argue here, but I think it worths a second thought merely because adding such feature would not change current behavior, only enable a non-conflicting option to invoke the cleanup. I'm not a maintainer of this repository, but if you seek the perspective of a user - this is a desired feature.

I would expect is some way make it clean up on-demand, even with a static method "await Ryuk.CleanupAsync()"

You can clean up individual resources by calling DisposeAsync() or DeleteAsync() on your test's teardown.
Calling a cleanup method from the test process will never work reliable. If the test process dies or the test throws an exception (while cleaning up), your resources will remain. Even a member like Ryuk.CleanupAsync() will run into this issue.
We truly appreciate when users take the time to share their perspectives and suggestions with us. We always strive to find solutions and help our users as much as possible. We take user feedback seriously and discuss feature requests like this with other maintainers to get different perspectives from other languages. We acknowledge that your feature request is desired by you as a user, but after careful consideration, we've concluded that it doesn't fit into how TC (moby-ryuk) currently works.
I hope that you'll continue to use TC and that we can continue to work together to improve it. If you have any further suggestions or feedback, please feel free to share them with us.


Angular-1.3.2- not supported

this theme not support in Angular-v1.3.2. or v1.6.2
i have put code in index.html --- its working fine
but i have create route and load view through  or  </ ng-view>
----not working

Hi,
It is not about the theme, you need work on AngularJS Routing Mech


original_dst_cluster: add override by filter state

Commit Message: Add an override by filter state to ORIGINAL_DST cluster (which takes priority over header override and restored local address).
Additional Description: ORIGINAL_DST cluster is used as a generic tunnel upstream for an internal listener (e.g. encoding TCP streams into HTTP/2). When used this way, there are two actual destination addresses. The first address is the logical address for which, for example, an RBAC policy must be applied on the internal listener. The second address is the physical address of the tunnel host. This change allows us to set the destination address for ORIGINAL_DST at the very last step in processing while doing the host selection, so that everything before that can use the logical address. Prior art couples the same behavior with HTTP/1.1 CONNECT transport, which would be confusing to re-use for ORIGINAL_DST.
One internal limitation is that the filter state is only read from the connection filter state. That works fine for tcp_proxy. There is no reason why it can't be done per-request, but that requires changing LoadBalancerContext throughout.
Risk Level: low (opt-in)
Testing: unit
Docs Changes: none (filter states are not documented in general, maybe they should be).
Release Notes: none

/retest

/assign-from @envoyproxy/maintainers

/assign-from @envoyproxy/envoy-maintainers


"allowed_mentions" properties are not set to false by default

The message_content allowed_mentions property does not have it's value set to false by default, as mentioned on this wiki page:
https://replaceitem.github.io/carpet-discarpet/parsables/allowed-mentions

If the allowed_mentions are specified in the message content, all the options are false, meaning all mentions are disabled (See here).
What you are probably refering to is not specifying allowed_mentions at all (meaning it is not an entry in the message content map), in which case mentions are not disabled.
Hope this clears it up.

Reading back on it my issue was phrased poorly. What I was actually referring to was message_content defined only by a string, which obviously does not define the allowed_mentions properties, like this:
dc_send_message(channel, 'This is just a string');

Your comment does clear that up, thanks!


put code into modules

i.e. one module for each tab
this will make much easier to maintain, improve and troubleshoot bugs

next year


Fix default setting

push validate/defaults into datastore
we weren't setting a timestamp in route insert when we needed to create an app
there. that whole thing isn't atomic, but this fixes the timestamp issue.
closes #738
seems like we should do similar with the FireBeforeX stuff too.
also:

app name validation was buggy, an upper cased letter failed. now it doesn't.
uses unicode now.
removes duplicate errors for datastore and models validation that were used
interchangably but weren't.

i checked and we'd have to do an api call to get the app after doing a post route. it kind of sucks (our tests suck, really, 2/3 of them)


Release 0.4.0

I think now's a good time to create another release.
I'd like to target a cutoff point within the next week or so, and hopefully https://github.com/js-temporal/temporal-polyfill/pull/105 can land first? It would be great to have that included.

I can drive a release starting sometime late next week.

hopefully #105 can land first? It would be great to have that included.

Agree. But I wouldn't hold the release for it. If it's ready, that's great. If not, we can pick it up in the next one.

I started working on this, but then hit a roadblock and forgot to post an update here.
I was trying to ensure that we had test262 coverage parity with the spec polyfill, and found that when I sync up the test262 repo to 2f592de0 that 62 tests are failing, all of which seem to be year-zero.js tests.
I think it's because we are missing e60ef9eac7a35b67b065f25d6e560a9bf4ba8bd5, and that made me wonder what else we might be missing.

My intent is to draft a release cut at 670cda6bd269db66c8ce97eb73f941b20abb92ec.
I skimmed over changes to tc39/proposal-temporal and I don't think anything else relevant is missing - most of the other commits seems to been test removals (as they've been ported to test262 - yay!).

Great!  Heads up I'm watching https://github.com/tc39/proposal-temporal/issues/2075, and if it's a bug then it may make sense to fix it before this release. Let's decide tomorrow.

False alarm. According to https://github.com/tc39/proposal-temporal/issues/2075#issuecomment-1058862771 it's not a bug. Release should be good to go!

Thanks so much @12wrigja for getting the release out the door! Closing.


x/tools/gopls/internal/regtest/diagnostics: failures in TestEditGoDirectiveWorkspace

TestEditGoDirectiveWorkspace was added 2022-03-17 in CL 393534 (attn @suzmue; CC @findleyr). Unfortunately, it has failed several times since then:
--- FAIL: TestEditGoDirectiveWorkspace (534.58s)
    --- FAIL: TestEditGoDirectiveWorkspace/experimental (534.41s)
        diagnostics_test.go:2247: waiting on:
            	Unmet: empty diagnostics for "main.go"

            err:context deadline exceeded

            state:
            #### log messages (see RPC logs for full text):
            	Info: "2022/03/18 21:47:13 go env for /tmp/workdir-host-l...
            	Info: "2022/03/18 21:47:14 go/packages.Load\n\tsnapshot=0...
            	Error: "2022/03/18 21:47:14 building workspace mod file: ...
            	Error: "2022/03/18 21:47:14 errors loading workspace: no ...
            	Info: "2022/03/18 21:47:44 background imports cache refre...
            	Info: "2022/03/18 21:47:44 background refresh finished af...

            #### diagnostics:
            	main.go (version 0):
            		(2, 7): type parameters require go1.18 or later
            		(2, 9): undeclared name: any (requires version go1.18 or later)
            	go.mod (version 0):
            		(0, 0): no module path for file:///tmp/workdir-host-linux-amd64-wsl/tmp/gopls-regtest-4097331373/TestEditGoDirectiveWorkspace/experimental/work/go.mod

            #### outstanding work:
            	Error loading workspace: 0.00
            #### completed work:
            	Load: 1
            	Setting up workspace: 1
            	diagnosing initial workspace load: 1
            	diagnosing files changed on disk: 1
        runner.go:324: closing editor: Shutdown: context deadline exceeded
FAIL
FAIL	golang.org/x/tools/gopls/internal/regtest/diagnostics	577.903s

Given the variety of platforms on which it has failed, I suspect that either the test or the implementation under test is racy.
greplogs --dashboard -md -l -e 'FAIL: TestEditGoDirectiveWorkspace' --since=2022-01-01
2022-03-25T20:23:27-e309672-3dac914/linux-amd64-wsl
2022-03-22T21:30:29-87a8611-ac3efc8/linux-amd64-wsl
2022-03-18T23:20:51-5ea13d0-7ca6902/aix-ppc64
2022-03-18T19:32:40-5ea13d0-a682a5c/plan9-amd64-0intro
2022-03-18T13:39:41-5ea13d0-4aa1efe/linux-amd64-wsl

Thanks, I believe this is a gopls bug: I have noticed that this behavior is flaky in practice.

This is a duplicate of #51930

Ah, I think that issue didn't show up in my issue search because it refers to TestEditGoDirective rather than TestEditGoDirectiveWorkspace. (But if we believe the underlying bug is the same for both, I'm happy with consolidating the issues.)


issue in calculator

the calculator's result is error . for  example 1/2.0 should be 0.5 but the result is 0.0

Following this checklist to help us incorporate your contribution quickly and easily:

[ ] Make sure there is a JIRA issue filed
for the change (usually before you start working on it).  Trivial changes like typos do not
require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Format the pull request title like [HOP-XXX] - Fixes bug in SessionManager,
where you replace HOP-XXX with the appropriate JIRA issue. Best practice
is to use the JIRA issue title in the pull request title and in the first line of the commit message.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Run mvn clean install apache-rat:check to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[ ] If you have a group of commits related to the same change, please squash your commits into one and force push your branch using git rebase -i.

Trivial changes like typos do not require a JIRA issue (javadoc, comments...).
In this case, just format the pull request title like (DOC) - Add javadoc in SessionManager.
If this is your first contribution, you have to read the Contribution Guidelines
If your pull request is about ~20 lines of code you don't need to sign an Individual Contributor License Agreement
if you are unsure please ask on the developers list.
To make clear that you license your contribution under the Apache License Version 2.0, January 2004
you have to acknowledge this by using the following check-box.

[ ] I hereby declare this contribution to be licenced under the Apache License Version 2.0, January 2004
[ ] In any other case, please file an Apache Individual Contributor License Agreement.

No this works as expected and is explained in the documentation.
https://hop.apache.org/manual/latest/pipeline/transforms/calculator.html#_faq_on_length_and_precision_and_data_types_affecting_the_results


added a blog post on role based authentication with react

This is an article that illustrates user authentication and role assignment using LoginRadius and React.Js

Checklist

Please note that in case of below checklist not updated accordingly, the maintainers have the right to immediately close your pull request.

[x] You created a branch from dev branch.
[x] You PR is raised on dev branch and not onmaster.
[x] Blog follows Blog Guidelines.
[x] You have read the Contribution Guidlines before creating this PR.
[x] Checked for any duplicate blog entry.
[x] Checked for all grammatical errors in the blog.
[x] You have not published the blog content anywhere else on the internet on platforms like medium, dev.to etc.
[x] The content is not copy-pasted from anywhere else on the Internet.

Description of change

Author Details for Blog Post Entry

id: Victory Tuduo
bio: I am a Software Developer versatile with React.Js and front-end web developer.
github: https://github.com/Victory-ET
linkedin: https://www.linkedin.com/in/tuduo-victory-7ba4641a4/

@abhir9 @pareek-narendra @puneetsingh24 . Please when will the review process take place?

Hi @Victory-ET
We'll review this in the next 72 hours and get back to you if any changes are necessary.

Hi @Victory-ET
I have reviewed the article, and it seems to be missing "Handling User Authentication" and related content, as per the finalized outline. While you may have covered this in "Using the LoginRadius API," it is important to cover "Handling User Authentication," and it's relevant subtopics to ensure readers can be clear and get value from the content.
Apart from this, please try editing the article for consistency and conciseness. For example, ensure that where applicable:

Please ensure "LoginRadius" is written consistently with uppercase L and R. Similarly, "ReactJS" with an uppercase S.
Please edit to remove where sentences are needlessly wordy and ensure grammar is solid, among others.

All of these changes can make the article better and provide more value to readers. Would you mind making these changes and letting us know?
After receiving these changes, we'll do an in-depth technical/content review to move forward.
Please feel free to ask any questions you may have.
Thank you,
@raghunath-r-a

Hi @raghunath-r-a,
I have made the requested changes and added a new section for handling user authentication.
Regards,
@Victory-ET

older PR get merged without content review. This has been reverted , @raghunath-r-a   please refer new PR
https://github.com/LoginRadius/engineering-portal/pull/726


[MRG] Rewrite tests in test_semver.py #615

Rewrite tests using Pytest's parametrizing functionality.

The coverage test fails because the coverage apparently went down. This sometimes happens when you delete a lot of code. I'll take a look if I can see something suspicious or if this is is "noise".

Looks good to me. I think the coverage computation was ab it confused. Merged :)

It was a good learning experience! Thanks :)


when value is null, label does not take into account its title

Thanks Abir, I confirm it is solved


Add a Krane CLI with one command

What are you trying to accomplish with this PR?
Start down the road of adding a new CLI Krane. Our first step is adding krane version
See https://github.com/Shopify/kubernetes-deploy/issues/256 for lots of details
How is this accomplished?
We've picked Thor to help make the cli as easy as possible. We've picked version to be the first command because it's just about trivial.
What could go wrong?
Its new code, so the only real issue is if the dependency causes people headaches.
CLI
➜  kubernetes-deploy git:(cli) ✗ be krane version
[INFO][2019-08-14 11:06:56 -0700]	Krane Version: 0.26.7

➜  kubernetes-deploy git:(cli) ✗ be exe/krane
Krane commands:
  krane help [COMMAND]  # Describe available commands or one specific command
  krane version         # Prints the version

➜  kubernetes-deploy git:(cli) ✗ be exe/krane version
[INFO][2019-08-13 16:43:39 -0700]	Krane Version: 0.26.7

➜  kubernetes-deploy git:(cli) ✗ be exe/krane version -v
ERROR: "krane version" was called with arguments ["-v"]
Usage: "krane version"

Oops, didn't notice the failed CI before I :+1: but it's just a simple rubocop fix

This is ready for 👀 though I don't want to merge it till after the -f flag PR. I played with what it might look like to add the --check flag in https://github.com/Shopify/kubernetes-deploy/commit/e6306af47deee1776bce9d5414c36b2843b3e5f5 and feel good that there is a reasonable way forward for more than just trivial commands. @KnVerey took a stab at deploy in https://github.com/Shopify/kubernetes-deploy/compare/cli...knverey_cli and this pr took some but not all of the ideas.