Android native navigator to flutterView, then the MaterialApp routes run twice times

Steps to Reproduce

android code:

public class AnotherActivity extends FlutterActivity {
...
    @Override
    public FlutterView createFlutterView(Context context) {

        WindowManager.LayoutParams matchParent = new WindowManager.LayoutParams(-1, -1);
        FlutterNativeView nativeView = this.createFlutterNativeView();
        FlutterView flutterView = new FlutterView(this, (AttributeSet) null, nativeView);
        flutterView.setInitialRoute("/route1");
        flutterView.setLayoutParams(matchParent);
        this.setContentView(flutterView);
        return flutterView;
    }
...
}

flutter code:

void main() => runApp(InitPage());

class InitPage extends StatelessWidget {
  var temp = window.defaultRouteName;
  @override
  Widget build(BuildContext context) {
    print('Flutter routes $temp');
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      initialRoute: '/',
      routes: {
        "/": (context) {
          print('show times');
          DefaultPage();
        },
        "/route1": (context) {
            print('show route1');
            MyApp();
         },
        "/route2": (context) => DefaultPage(),
      },
    );
  }
}

when i navigator to flutter view from android .The Logcat panel output twice 'show times' and 'show route1'

I use release aar package in app/libs. But I have no this problem if I use debug aar package.

Logs

2019-04-08 18:38:27.590 12248-12448/com.example.android_flutter I/flutter: Flutter routes /route1
2019-04-08 18:38:27.603 12248-12448/com.example.android_flutter I/flutter: show times
2019-04-08 18:38:27.604 12248-12448/com.example.android_flutter I/flutter: show route1
2019-04-08 18:38:27.612 12248-12448/com.example.android_flutter I/flutter: show times
2019-04-08 18:38:27.612 12248-12448/com.example.android_flutter I/flutter: show route1

Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel master, v1.3.15-pre.26, on Microsoft Windows [Version 10.0.17134.648], locale zh-CN)
[√] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
[√] Android Studio (version 3.3)
[!] IntelliJ IDEA Ultimate Edition (version 2018.1)
    X Flutter plugin not installed; this adds Flutter specific functionality.
    X Dart plugin not installed; this adds Dart specific functionality.
[√] Connected device (2 available)

! Doctor found issues in 1 category.

asdf

Hi, sorry for the delay, I just want to check if you could solve this problem.

yep, i got it

How did you solve the issue?
I'm having a (I think) related problem where if I create a custom FlutterActivity very similar to your AnotherActivity above Flutter starts the app with the default route of '/' and then switches to the route specified in flutterView.setInitialRoute.

This seems to be the routing rules. I can't solve it

Could everyone who still has this problem please file a new issue with the exact descriptions what happens, logs and the output of ' flutter doctor -v'  please.
All system setups can be slightly different so its always better to open new issues and reference related issues.


Support embassy-usb on ESP32-S2 and ESP32-S3

Whilst this crate supports usb-device (which is not async), it'd be really cool if it also supported the async embassy-usb.
Some efforts have begun here.

These registers should be accessible by index. The 0 at the end can vary from 0 to 6.
pub type DOEPCTL0 = crate::Reg<doepctl0::DOEPCTL0_SPEC>;
pub type DOEPINT0 = crate::Reg<doepint0::DOEPINT0_SPEC>;
pub type DOEPTSIZ0 = crate::Reg<doeptsiz0::DOEPTSIZ0_SPEC>;
pub type DOEPDMA0 = crate::Reg<doepdma0::DOEPDMA0_SPEC>;
pub type DOEPDMAB0 = crate::Reg<doepdmab0::DOEPDMAB0_SPEC>;

Probably the PACs should get patched to collect these registers into arrays. We use https://github.com/rust-embedded/svdtools for patching SVDs in https://github.com/esp-rs/esp-pacs


Update App.js

Used class properties for initializing state, making the constructor simpler.
Utilized arrow functions for class methods to avoid the need for manual binding.
Improved the readability and formatting of the code. Encapsulated code related to custom trigger buttons and event handling into separate functions.
Modularized the setup of the searchIndex for better separation of concerns.
Got some code for us? Awesome 🎊!
Please include a description of your change & check your PR against this list, thanks!

[x] There's a clear use-case for this code change, explained below
[x] Commit message has a short title & references relevant issues
[x] The build will pass (run yarn test:all and yarn lint)

We appreciate your contribution!

🤖 Generated by Copilot at 4a5c3c5
Refactored and formatted the main App component of the sodo-search app to enhance its functionality and readability. Used React hooks, memoization, and custom hooks to optimize rendering and state management. Applied consistent code style with Prettier.

Hey @PratyushJoshi, sorry we didn't review this PR in a timely fashion. Looks like there's probably some great changes here. If you're still interested in working on this please re-raise your PR and we'll work through it.


affected/package: runtime

What version of Go are you using (go version)?

$ 1.20.2

Does this issue reproduce with the latest release? yes
What operating system and processor architecture are you using (go env)? debian - amd64
go env Output
$
GO111MODULE="on"
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOOS="linux"
GOPATH="/home/mohammadreza/go"
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GOVCS=""
GOVERSION="go1.20.2"
GCCGO="gccgo"
GOAMD64="v1"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOWORK=""
CGO_CFLAGS="-O2 -g"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-O2 -g"
CGO_FFLAGS="-O2 -g"
CGO_LDFLAGS="-O2 -g"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -Wl,--no-gc-sections -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build3308778073=/tmp/go-build -gno-record-gcc-switches"

What did you do?
I have written a rest api service that panics every few minutes.
This panic is related to the golang libraries themselves
load of this service is high, but it provided enough resources to the service (100 GIG RAM , 40 CPU CORE , ... )
fatal error: missing stack in shrinkstack
runtime stack:
runtime.throw({0x1a8bd5f?, 0x0?})
/usr/local/go/src/runtime/panic.go:1047 +0x5d fp=0x7fbbf9ffaaa8 sp=0x7fbbf9ffaa78 pc=0x43a9bd
runtime.shrinkstack(0x0?)
/usr/local/go/src/runtime/stack.go:1164 +0x198 fp=0x7fbbf9ffaac8 sp=0x7fbbf9ffaaa8 pc=0x454978
runtime.scanstack(0xc000d0a000, 0xc0000c2638)
/usr/local/go/src/runtime/mgcmark.go:775 +0xf2 fp=0x7fbbf9ffacd0 sp=0x7fbbf9ffaac8 pc=0x421b92
runtime.markroot.func1()
/usr/local/go/src/runtime/mgcmark.go:239 +0xb5 fp=0x7fbbf9ffad20 sp=0x7fbbf9ffacd0 pc=0x420a95
runtime.markroot(0xc0000c2638, 0xc74, 0x1)
/usr/local/go/src/runtime/mgcmark.go:213 +0x1a5 fp=0x7fbbf9ffadc0 sp=0x7fbbf9ffad20 pc=0x420745
runtime.gcDrain(0xc0000c2638, 0x7)
/usr/local/go/src/runtime/mgcmark.go:1069 +0x39f fp=0x7fbbf9ffae20 sp=0x7fbbf9ffadc0 pc=0x42283f
runtime.gcBgMarkWorker.func2()
/usr/local/go/src/runtime/mgc.go:1367 +0x7a fp=0x7fbbf9ffae70 sp=0x7fbbf9ffae20 pc=0x41ed1a
runtime.systemstack()
/usr/local/go/src/runtime/asm_amd64.s:496 +0x49 fp=0x7fbbf9ffae78 sp=0x7fbbf9ffae70 pc=0x46c189

What did you expect to see?
What did you see instead?

please fill out the bug report template

please fill out the bug report template

edited

@seankhliao plz help me

no solution ???

Is there a way that we can reproduce the problem ourselves?  Right now this looks like a form of memory corruption.  Do you call C functions using cgo?  Have you tried running your program using the race detector?

Is there a way that we can reproduce the problem ourselves? Right now this looks like a form of memory corruption. Do you call C functions using cgo? Have you tried running your program using the race detector?

no , i dont call cgo function .
yes i run with race  detector and it is ok


Thread support2

Per https://github.com/42wim/matterircd/pull/320, split out support for replying to older threads using parent/post ID.
10:59  @@epjhhxi3fpd33kb7usmyfuza6r testing replying to old thread
11:00  @@epjhhxi3fpd33kb7usmyfuza6c testing replying to invalid thread
11:00  @@epjhhxi3fpd33kb7usmyfuza6r
11:00  @@epjhhxi3fpd33kb7usmyfuza6rabc

@42wim, so something like this?

Yes, thank you :)

Would this mean I wouldn't need to enable PrefixContext, or SuffixContext to be able to reply to threads?

Would this mean I wouldn't need to enable PrefixContext, or SuffixContext to be able to reply to threads?

With this change, you can reply to threads with the thread / parent ID (@@.{26}). There is no option to show the thread/parent ID though, as that was rejected in https://github.com/42wim/matterircd/pull/320 in favor of PrefixContext/SuffixContext.

How would you know what the thread/parent ID would be without the PrefixContext/SuffixContext?

How would you know what the thread/parent ID would be without the PrefixContext/SuffixContext?

Web UI.
@42wim would you accept a patch enabling showing of thread/parent ID? (e.g. ShowThreadID) So would be similar to https://github.com/42wim/matterircd/pull/320 just renamed config option.

@hloeung what about integrating it in SuffixContext and PrefixContext and adding another option called ThreadContext which only works when one of the former options are enabled.
An empty ThreadContext uses the default context settings, but setting ThreadContext="parentid" will show the parentID instead of the matterircd generated id.

@hloeung what about integrating it in SuffixContext and PrefixContext and adding another option called ThreadContext which only works when one of the former options are enabled.
An empty ThreadContext uses the default context settings, but setting ThreadContext="parentid" will show the parentID instead of the matterircd generated id.

Works for me, https://github.com/42wim/matterircd/pull/348


Skip showback models for full-refresh tests

I'm not entirely sure what is going on, but this spec is failing on jansa and not master on a model with no data in it with the following error:
  1) ManageIQ::Providers::Openshift::ContainerManager::Refresher Targeted refresh doesn't impact unassociated records
     Failure/Error: after_full_refresh = serialize_inventory

     ActiveRecord::AttributeMethods::Serialization::ColumnNotSerializableError:
       Column `data` of type ActiveRecord::ConnectionAdapters::PostgreSQL::OID::Jsonb does not support `serialize` feature.
       Usually it means that you are trying to use `serialize`
       on a column that already implements serialization natively.

Just calling ManageIQ::Showback::DataView.all will raise this error
https://travis-ci.com/github/ManageIQ/manageiq-providers-openshift/jobs/330997162#L1933-L1951

https://github.com/ManageIQ/manageiq-consumption/pull/172 has to be backported


bug(forge coverage): missing coverage when return variables are named

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (818eeb9 2024-08-31T00:23:20.447780000Z)
What command(s) is the bug in?
forge coverage --report lcov
Operating System
macOS (Apple Silicon)
Describe the bug
When return variables are named in the function definition, the generated lcov file fails to cover the return line:

Removing named variables resolves the issue:

This only appears in some of the functions.

@peitalin I don't think that's something we can fix as there's no hit for such, rather you could just remove the return statement in functions with named vars as it is a noop anyways and will keep your code cleaner

Ah yep makes sense, that approach works!


Fix some bugs in date time processing

Signed-off-by: Breezewish<EMAIL_ADDRESS>
What have you changed? (mandatory)
This PR fixes some bugs preventing us to treat time error as warning in some cases.
Also changed to accept reference when formatting date time.
What are the type of the changes? (mandatory)

Improvement (change which is an improvement to an existing feature)

How has this PR been tested? (mandatory)
No need.

/run-integration-tests

/run-integration-tests

@lonng @sticnarf PTAL, thanks

/run-integration-tests

/run-integration-ddl-test


Template import consider as string in _render

Here's an example
// my-tempate.js
export default `<button on-click="${() => this.click()}"

//index.js
import { LitElement, html } from '@polymer/lit-element';
import template from './my-template.js

export default class Login extends connect(store)(LitElement) {
    constructor() {
        super();
    }

    _render() {
        return html`${template}`;
    }

    _stateChanged(state) {
        console.log(state);
    }

    click() {
        console.log('i m here');
        this.dispatchEvent(new CustomEvent('click'));
    }

In this scenario it render the html as string
and if I change render function to
 _render() {
        return html([`${template}`]);
    }

Then in this case it render the html page but on click function doesn't work.
And I don't to put html in same js file, it clutter more in terms of codewise.I want to make it as small module like the way we do it in react,angular.
How can we achieve import of html with all function working that are define in html?

I believe you should export TemplateResult from the start:
// my-tempate.js
export default html`<button on-click="${() => this.click()}">Click me</button>`;

Some details.

Then in this case it render the html page but on click function doesn't work.

It won't work anyway. It is because simple template literal just converts your () => this.click() to a string. It will work properly if and only if you use html tag on template literal.

() => this.click()

This part won't work as well if you move declaration to a separate file. this will be undefined because it consumes the root scope which is undefined for ES2015 modules.
Things I mentioned above means that lit-html is designed to be a replacement for JSX. You cannot use lit-html outside of your component as well as you cannot use JSX outside of React (and derivatives). The only one approach I can imagine is to export a function that receives arguments and applies them to a html template but I don't think it is a good approach since it creates unnecessary complexity.
export default (onClick) => html`<button on-click="${onClick}">Click me</button>`;

@Lodin this context will change, it will not take Login click function
Throws error click is not defined

You might try something like this: https://glitch.com/edit/#!/zinc-needle?path=template.js:6:51
Here I've update the template.js to export a standard function that returns a TemplateResult and looks like:
import {html} from '@polymer/lit-element';

export function template() {
  return html`
          Web Components are ${this.mood}!
          <button on-click="${() => this._click()}">Change mood</button>`;
}

And I'm matched the _render function with a template getter that binds the remote template to the local scope like so:
get template() {
  return template.bind(this)();
}

_render() {
  return html`${this.template}`;
}

This is of course is complexity that you could take or leave, but the getter gives you a clean accessor to cache the bound template locally if you're interested in that sort of thing.

Thanks for the suggestion @Westbrook , this approach is more clean.Hopefully they will introduce a feature to modularize html as well

That conversation is definitely ongoing and the powers of it matched with the Template Instantiation Spec are pretty exciting.

Can this be closed now?


Update DB dump acc. to recent dropapp schema migrations

A couple of schema migrations have happened in dropapp. This PR ports the changes to the peewee models, and updates the database dump.
Check db/migrations/ for corresponding phinx migrations in dropapp.
@spaudanjo @flisowna please have a look at the changes in the GraphQL API caused by the migrations. Is there any action necessary to generate the TypeScript GraphQL types?

 EPIC: Database Cleanup

 Database cleanup Product MYSQL table

 BUG/ISSUE: Shipment and transfers tables do not currently exist in the live staging / production databases


I wish /docs/user-guide/ had a PDF of the documentation to download for offline use

I wish /docs/user-guide/ had a PDF of the documentation to download for offline use

Sorry.  You're welcome to clone the repo locally, but we don't plan to make PDFs available.


Quota exceeded raise now an exception (fixes #64)

Codecov Report

Merging #74 (cc53d52) into master (593a4be) will increase coverage by 0.10%.
The diff coverage is 0.00%.

:exclamation: Current head cc53d52 differs from pull request most recent head 8d368d9. Consider uploading reports for the commit 8d368d9 to get more accurate results

@@            Coverage Diff             @@
##           master      #74      +/-   ##
==========================================
+ Coverage   55.93%   56.03%   +0.10%
==========================================
  Files          12       12
  Lines         438      439       +1
  Branches       59       59
==========================================
+ Hits          245      246       +1
  Misses        186      186
  Partials        7        7

Impacted Files
Coverage Δ

src/gardena/smart_system.py
35.67% <0.00%> (ø)

src/gardena/devices/device_factory.py
42.85% <0.00%> (+2.85%)
:arrow_up:

Continue to review full report at Codecov.

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


selfhost: Add support for bindings with is operator

This implements variable bindings for enum variants for both
typed and struct-like variants. It de-sugars variable bindings
into typed optionals + pattern mathing with rebinding of inner
values followed by a forced unwrap.
Example:
enum Foo {
    Bar(i64)
}

let foo = Foo::Bar(5)

// This:
if foo is Foo::Bar(n) {
    println("{}", n)
}

// De-sugars to this:
mut tmp_n: i64? = None
if foo is Foo::Bar {
    match foo {
        Bar(n) => {
            tmp_n = n
        }
    }
    let n = tmp_n!
    println("{}", n)
}

Oh sweet!
Does if foo is Foo::Bar(n) also let you do things like: if foo is Foo::Bar(n) and n > 100 ?

Converting it into a Draft while I work no the various comments.

Did a complete refactor of this and will open a new PR. Closing this one.


can't download the released model

I couldn't download the released mode.
How can I get them?
SEEM Focal-L and X-Decoder Focal-L checkpoints.
bellow is error message.
This XML file does not appear to have any style information associated with it. The document tree is shown below.
<Error>
PublicAccessNotPermitted
<Message>Public access is not permitted on this storage account. RequestId:5c2d36f8-601e-0012-2ca8-93d64b000000 Time:2023-05-31T10:11:10.4213753Z</Message>
</Error>

This happens to me too. when I try to wget them.

Thanks for raising the issue. We had updated the links.

Thanks for raising the issue. We had updated the links.

I also still get the same error. What was updated?


failed to select all occurrences by word

If use short-cut  command + shift + F  or  command + ctrl + G,
it alerts "command 'extension.findBySelection' not found".
I need this:
https://stackoverflow.com/questions/46539714/select-all-occurrences-of-selected-word-in-vscode

{
  "key": "shift+cmd+l",
  "command": "editor.action.selectHighlights",
  "when": "editorFocus"
}
{
  "key": "shift+cmd+l",
  "command": "extension.findBySelection"
}
{
  "key": "shift+cmd+l",
  "command": "addCursorsAtSearchResults",
  "when": "fileMatchOrMatchFocus && searchViewletVisible"
}
{
  "key": "shift+cmd+l",
  "command": "selectAllSearchEditorMatches",
  "when": "inSearchEditor"
}

@sandy081  I install the keymap extension:  k--kato.intellij-idea-keybindings recommended by vscode

Looks like this keymap extension is outdated, please file an issue against it.


open files as in a web-browser

if you type in the address bar in the web browser: "file:///path to the file", the file will open. I tried to do this with the network extension, but there was no result

Browsers have security restrictions that prevent websites from just accessing any file from your computer (the only way they can do so is through a file prompt).

Browser security features working as intended, sorry


Close worker if jobchan is closed

Close worker if jobchan is closed
Also prevent nil pointer if sign method is not defined

In current implementation, controller, pool and worker share the same context. When controller and pool is cancelled, worker is cancelled too, so what is the difference in this PR?


Can't Compile

app.configure is not a function

I'm no longer supporting this library as I don't use it any longer so I'm going to close this issue.


Users seeking support

The readme says all development discussion will take place on gitter. The docs say there is a slack channel. I couldn't sign in to the slack channel because it says I need to "Contact the workspace administrator for an invitation".
Is all communication going to occur via gitter? If so can the read me be changed to make this clearer?

I have now updated the webpages to direct interested contributors to the https://github.com/arm61/pylj/blob/master/CONTRIBUTING.md only.
Further, I have updated both the https://github.com/arm61/pylj/blob/master/CONTRIBUTING.md and https://github.com/arm61/pylj/blob/master/README.md to reflect the use of gitter as the source of all discussion about development but also linked the https://gitter.im/pylj/HELP room in the https://github.com/arm61/pylj/blob/master/README.md
This is all in https://github.com/arm61/pylj/commit/b17af7dcd913b25857f95cc6f067b0549a51fdc1


Kirill1 - basic car methods, no unit test

add car, view car bu user, htmx, fragments

PLS check conflicting


New Feature: alphasort the Dependencies in dependency-check-report.html

If you have even 10 or more dependencies, its hard to review the dependency-check-report.html given that the Dependencies are reported in some order other than Alphabetically.
Can it by default, or at least optionally, alphabetically sort the dependencies in the table at the top, and the details below? That make things easier to find and also lump together similar libraries (e.g., spring-, commons-).

There is sorting available within the report by clicking on the column. There are a few enhancements that can be made:

List should be alpha sorted when the report is first loaded
The sorting should be made case insensitive

I believe this has been resolved.


Npm publish

Missing package.json contents added. Install docs updated. Relates to #28.

@mattlewis92 It's now available in npm. Here: https://www.npmjs.com/package/angular-aside

Great, thank you so much! :smile:


Upgrade devDependencies - babel

Before submitting, please confirm you've

[x] read and understood our Contributing Guidelines
[x] completed Mattermost Contributor Agreement
[x] executed npm run lint:js for proper code formatting

Please provide the following information:
Summary
Upgrade babel stack and introduce babel-preset-env. It generates more optimized codes for Electron and brings latest JavaScript features to us.
Issue link
#702
Test Cases
General use cases.
Additional Notes
https://circleci.com/gh/yuya-oc/desktop/596#artifacts

@yuya-oc I approved, let me know if there's anything specific I should test.

@jasonblais Basically none. But generated codes should be changed, so we'd be happy to use it for a moment.

@yuya-oc Sounds good, I'll test it out this week 👍

@yuya-oc Haven't seen any issues so this is okay to merge.
I'll keep using it for the next day or two and raise issues if I find any.

@jasonblais I see. Thanks!


Feature/raw contains string or data

Previous MessagePack versions use type raw for sending string or binary data. The user itself has to decide if the data should be handled as a string or NSData. Your current MessagePack.swift implementation always assumes the new MessagePack Specification: String and Binary have their own types. For this reason I introduced a compatibility mode (MSGP_COMPATIBILITY_MODE).
Additional I fixed the length detection for unsigned integers and the new binary type.

What was wrong with the length detection for unsigned integers?

Actually there´s nothing wrong with your implementation, my bad. I thought we had the same situation like in the previous pull request / wrong length detection for array 16 and array 32. I just tried it and it work´s perfectly.
But there´s still an issue with int64: The length should be 8 (bytes) and not 2 (bytes). And there´s the question regarding the MessagePack Compatibility mode.
Should we close this pull request and I'll make a new one?

Let's definitely tackle one problem per PR if that's okay.

OK, I´ll close this one and made a new one.


static av1_intra_prediction_edges: Make non-mut by replacing #[derive(BitfieldStruct)] with bools_bitfield_struct! for const fns

#[derive(c2rust_bitfields::BitfieldStruct)] doesn't generate const fns (https://github.com/immunant/c2rust/issues/1027), so it can't be used in a const block to initialize a static.  Thus, the static has to made mut and unsafe and initialized at runtime.
I opened https://github.com/immunant/c2rust/issues/1027 and looked into making the generated fns const, but they used a lot of traits, which can't contain const fns, so it seemed like non-trivial work.  I also tried looking for other off-the-shelf bitfield crates, but they all either didn't support const fns or still relied on traits within them, and thus only could create const fns with nightly features.
Thus, I rolled my own macro_rules! macro, bools_bitfield_struct! that generates const fns.  The macro is specific for bitfields that are always bools, which makes things a bit simpler and have a cleaner API.  And it also doesn't have to worry about being #[repr(C)] at all, which might makes things a bit simpler, too.  Overall, though, it's quite a simple macro, so it seemed much easier to do it this.  Since rav1d only used bitfields here, I kept the macro private in this file, but if we ever need bitfields that are all bools elsewhere, we can extract the macro.  The c2rust-bitfields dependency is also now removed as it's unused.

Mostly superceded by #501, but I'm leaving this open (and intending to merge in order) so the history doesn't get overly complicated.

I wouldn't normally want to add a custom macro for this, but since you're ultimately replacing it with the bitflags crate I'll approve this one as-is so that you can preserve merge order 👍


FEA: add model freezing for KBRD recommender module

Fix #47

Add model freezing for KBRD
Freeze KBRD recommendation module while training the conversation module
Able to freeze in DP mode and standard mode

Thanks for your contribution!


Runtime error while running onxx model on arm nn android

Hi i am getting runtime error while running onnx model on armnn android  error is:
(base) parag.j@CPU-349U:~/Downloads/armnn-eshwar/3dlm$ adb shell 'LD_LIBRARY_PATH=/data/local/tmp /data/local/tmp/3DLM'
CANNOT LINK EXECUTABLE "/data/local/tmp/3DLM": cannot locate symbol "_ZN15armnnOnnxParser11IOnnxParser6CreateEv" referenced by "/data/local/tmp/3DLM"...

Please help with that

Did you build armnnOnnxParser? It seems the dynamic linker is not able to find it.

yes i have built it

Hi @paragikjain,
Is this a problem you still need help with?
One little thing I noticed in your original error is that you are overwriting the LD_LIBRARY_PATH variable rather than adding to it.
Colm.

HI Thanks for response . Now i am not getting this issue.


--enable-all with other options doesn't run most of the clang-tidy checkers

I am running the command:
CodeChecker check -j 13 -b "hg pull -u && hg update -r default && ./mach build -j 23" -o codechecker-results --enable-all --enable alpha --disable google-readability-casting --disable llvm-include-order --disable google-readability-braces-around-statements
Unfortunately, with this serie of enable/disable, most of the clang-tidy checkers are not executed.

Could you upload the contents of ./codechecker-results/metadata.json, particularly the value of the checkers dict? IIRC that dict contains every checker that was executed in the particular analysis.

I am getting this list:
"clang-tidy": ["bugprone-integer-division", "bugprone-suspicious-memset-usage", "bugprone-undefined-memory-manipulation", "cert-dcl58-cpp", "cppcoreguidelines-special-member-functions", "google-build-namespaces", "google-global-names-in-headers", "misc-assert-side-effect", "misc-bool-pointer-implicit-conversion", "misc-dangling-handle", "misc-definitions-in-headers", "misc-fold-init-type", "misc-forward-declaration-namespace", "misc-forwarding-reference-overload", "misc-inaccurate-erase", "misc-incorrect-roundings", "misc-inefficient-algorithm", "misc-lambda-function-name", "misc-macro-repeated-side-effects", "misc-misplaced-const", "misc-misplaced-widening-cast", "misc-move-const-arg", "misc-move-constructor-init", "misc-move-forwarding-reference", "misc-new-delete-overloads", "misc-noexcept-move-constructor", "misc-non-copyable-objects", "misc-redundant-expression", "misc-sizeof-container", "misc-sizeof-expression", "misc-string-compare", "misc-string-constructor", "misc-string-literal-with-embedded-nul", "misc-suspicious-enum-usage", "misc-suspicious-missing-comma", "misc-suspicious-semicolon", "misc-swapped-arguments", "misc-throw-by-value-catch-by-reference", "misc-unconventional-assign-operator", "misc-undelegated-constructor", "misc-uniqueptr-reset-release", "misc-unused-raii", "misc-use-after-move", "misc-virtual-near-miss"]},
As you can see, for example, there is no modernize-* clang tidy checker.


Error: Running Homebrew as root is extremely dangerous and no longer supported

Fresh install of Sierra 10.12.2 and this ansible role installs Homebrew 1.1.0 but it errors and fails at the following point
TASK [geerlingguy.homebrew : Force update brew after installation.] ************
fatal: [<IP_ADDRESS>]: FAILED! => {"changed": true, "cmd": ["/usr/local/bin/brew", "update", "--force"],
"delta": "0:00:00.147456", "end": "2016-12-27 16:02:43.209803", "failed": true, "rc": 1, "start": "2016-12-27 16:02:43.062347",
"stderr": "Error: Running Homebrew as root is extremely dangerous and no longer supported.\n
As Homebrew does not drop privileges on installation you would be giving all\n
build scripts full access to your system.",
"stdout": "", "stdout_lines": [], "warnings": []}

PLAY RECAP *********************************************************************
<IP_ADDRESS>                  : ok=9    changed=5    unreachable=0    failed=1

To unblock progressing the role to allow completion I'm thinking I should just disable the specific "Force update" task which could be done via this approach http://allwrightythen.com/using-a-forked-ansible-galaxy-role/
Could someone please clarify the best way forwards to workaround and/or fix this?
Thanks,
Matthew
p.s. for reference the previous commands run were
Matthews-MacBook-Air:macos-bootstrap matthewcosgrove$ sudo ansible-galaxy install -r pre-req-roles.yml
- downloading role 'dotfiles', owned by geerlingguy
- downloading role from https://github.com/geerlingguy/ansible-role-dotfiles/archive/1.1.0.tar.gz
- extracting geerlingguy.dotfiles to /etc/ansible/roles/geerlingguy.dotfiles
- geerlingguy.dotfiles was installed successfully
- downloading role 'homebrew', owned by geerlingguy
- downloading role from https://github.com/geerlingguy/ansible-role-homebrew/archive/2.1.0.tar.gz
- extracting geerlingguy.homebrew to /etc/ansible/roles/geerlingguy.homebrew
- geerlingguy.homebrew was installed successfully
Matthews-MacBook-Air:macos-bootstrap matthewcosgrove$ ansible-playbook main.yml -i inventory -K
Traceback (most recent call last):
  File "/usr/local/bin/ansible-playbook", line 43, in <module>
    import ansible.constants as C
  File "/Library/Python/2.7/site-packages/ansible/constants.py", line 161, in <module>
    DEFAULT_LOCAL_TMP         = get_config(p, DEFAULTS, 'local_tmp',        'ANSIBLE_LOCAL_TEMP',      '$HOME/.ansible/tmp', istmppath=True)
  File "/Library/Python/2.7/site-packages/ansible/constants.py", line 81, in get_config
    makedirs_safe(value, 0o700)
  File "/Library/Python/2.7/site-packages/ansible/utils/path.py", line 67, in makedirs_safe
    raise AnsibleError("Unable to create local directories(%s): %s" % (to_native(rpath), to_native(e)))
ansible.errors.AnsibleError: Unable to create local directories(/Users/matthewcosgrove/.ansible/tmp): [Errno 13] Permission denied: '/Users/matthewcosgrove/.ansible/tmp'
Matthews-MacBook-Air:macos-bootstrap matthewcosgrove$ sudo !!
sudo ansible-playbook main.yml -i inventory -K

Got further by running
sudo chown -R $USER:admin /usr/local
brew update --force

but more suggestions would be very welcome

@matthewcosgrove - Are you using a downstream project that uses this role? The sudo ansible-galaxy install -r pre-req-roles.yml command isn't in any of this role's documentation, nor does my mac-dev-playbook use it.
Homebrew shouldn't be installed with sudo, and the role would/should fail if being run as your current user without the /usr/local directory permissions set correctly...

@geerlingguy ah yes, I meant put the link to https://github.com/matthewcosgrove/macos-bootstrap which is effectively a stripped down version of your playbook but which I started from scratch.
And I appreciate that sudo should not be used for brew however as seen in the dump in my p.s. above I only added sudo to get over another issue. Excerpt is
Matthews-MacBook-Air:macos-bootstrap matthewcosgrove$ ansible-playbook main.yml -i inventory -K
Traceback (most recent call last):
  File "/usr/local/bin/ansible-playbook", line 43, in <module>
    import ansible.constants as C
  File "/Library/Python/2.7/site-packages/ansible/constants.py", line 161, in <module>
    DEFAULT_LOCAL_TMP         = get_config(p, DEFAULTS, 'local_tmp',        'ANSIBLE_LOCAL_TEMP',      '$HOME/.ansible/tmp', istmppath=True)
  File "/Library/Python/2.7/site-packages/ansible/constants.py", line 81, in get_config
    makedirs_safe(value, 0o700)
  File "/Library/Python/2.7/site-packages/ansible/utils/path.py", line 67, in makedirs_safe
    raise AnsibleError("Unable to create local directories(%s): %s" % (to_native(rpath), to_native(e)))
ansible.errors.AnsibleError: Unable to create local directories(/Users/matthewcosgrove/.ansible/tmp): [Errno 13] Permission denied: '/Users/matthewcosgrove/.ansible/tmp'
Matthews-MacBook-Air:macos-bootstrap matthewcosgrove$ sudo !!

To clarify my position, the steps on https://github.com/matthewcosgrove/macos-bootstrap worked fine on a VirtualBox VM using El Capitan but when applied to Sierra on a clean Macbook Air install I came across the issues above which prevented me from progressing. I am currently creating a Sierra Virtual Box to reproduce the issue on another machine, but if you have any thoughts it would be appreciated.

@geerlingguy Works fine on the VirtualBox VM and comparing the two I see that after running my ./install-ansible.sh script it created the .ansible directory owned by root on the Macbook Air but owned by my user on the VirtualBox VM.
It's not clear to me how that can be the case as I have run through the same set of steps and scripts for both, but anyway I will close this issue.


chore(package): update mocha to version 3.3.0

https://greenkeeper.io/

Codecov Report

Merging #86 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #86   +/-   ##
=======================================
  Coverage   58.33%   58.33%
=======================================
  Files           1        1
  Lines          12       12
=======================================
  Hits            7        7
  Misses          5        5

Continue to review full report at Codecov.

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


No error message if tails server is down

When creating a credential definition for a revocable credential, AcaPy does not throw any error even if the tails server is down and the tails file can not be stored on the tails server.
Only the holder raises an error as soon as it tries to generate a presentation.
I would have expected an error to be returned by POST /credential-definitions or at least via the webhook /topic/revocation_registry/. Also in the logs of AcaPy I did not see any message indicating that there is an issue writing a file to the tails server.

Discussion in AcaPug Call 3rd of May 2022:
Idea 1: It would be nice if the tails file is uploaded first and then written to the ledger

tails server is currently verifying that the tails file is already mentioned in a transaction on the ledger before it accepts tails files -> this check is in place to ensure that only tails files are being uploaded
still, it would be possible to rethink the tails server to make it possible to first upload tails files and then write them to the ledger

Idea 2: Should consider an alternative revocation mechanism that is not using tail files at all
Idea 3: Only set the RevReg status to active once the tails file has been uploaded successfully


Registering private IPs thru mesos-consul

I am provisioning containers on mesos framework using Calico (multi host networking solution). Calico assigns a separate IP for each of the container that I provision thru Calico from my defined IP pool.
Issue is when my container got registered thru mesos-consul, in my Consul I see the HOST IP of the container. Instead is there a way I can register container's IP thru mesos-consul and have Consul report Container's IP instead of Host IP?
Thanks
Lax

You can try changing the IP order via the --mesos-ip-order option. The default is netinfo,mesos,host.

Ok. I was using mesos,host. Would preceding the order with docker would help?

You can try docker. netinfo might also work as well.

Ok. Some how when I gave in like "--mesos-ip-order=netinfo,mesos,host" didnt work. Will try just having either of docker or netinfo. Thanks

seeing weird behavior. When I change mesos-ip-order and re-deploy mesos-consul, I keep seeing the error 'Timed out waiting for initial ZK detection'. This causes the mesos-consul to keep re-deploying every 2 mins.
I tried pinging & telneting the ZK IP and port, that seems to work fine. Also I do see 'json.info_0000000002' file when 'ls /mesos' on ZKCli. Is there anything I am missing?

Can you post your mesos-consul config?

{
"args": [
"--zk=zk://zookeeper.service.local:2181/mesos",
"--mesos-ip-order=docker,mesos,host"
],
"container": {
"type": "DOCKER",
"docker": {
"network": "HOST",
"image": "ciscocloud/mesos-consul"
}
},
"id": "mesos-consul",
"instances": 1,
"cpus": 0.1,
"mem": 128,
"labels": {
"task_type": "infra"
}
}

That looks good. Hmm. Does zookeeper.service.local resolve properly? The IP order processing occurs after the initial zookeeper connection so that shouldn't be an issue...

Ok, It does resolve properly. I am re-spinning my cluster again just in case if it helps issue go away

On re-spun system with the above app definition, it came up fine this time. But when I deploy a container in consul I still see Host IP though

That's not good. Can you post the task from Mesos's state.json? I'm wondering if the container IP is even in the state file...

you mean content of json.info_0000000002 file?

No. curl http://mesos-master:5050/master/state.json. That's where mesos-consul gets its data from. I want to see if the container IP is in the mesos state data.

Here is my task data seen in the state.json file
            {
                "id": "redis2.92a2aeca-194d-11e6-8438-467bb8096b30",
                "name": "redis2",
                "framework_id": "fa6b2f1e-0483-4dad-9867-d0794f6daf40-0000",
                "executor_id": "",
                "slave_id": "48c73e8e-81a4-423f-ae84-9a6265e6264c-S2",
                "state": "TASK_RUNNING",
                "resources": {
                    "cpus": 0.2,
                    "disk": 0,
                    "mem": 64,
                    "ports": "[4740-4740]"
                },
                "statuses": [
                    {
                        "state": "TASK_RUNNING",
                        "timestamp":<PHONE_NUMBER>.03674,
                        "labels": [
                            {
                                "key": "Docker.NetworkSettings.IPAddress",
                                "value": ""
                            }
                        ],
                        "container_status": {
                            "network_infos": [
                                {
                                    "ip_address": "",
                                    "ip_addresses": [
                                        {
                                            "ip_address": ""
                                        }
                                    ]
                                }
                            ]
                        }
                    }
                ],
                "discovery": {
                    "visibility": "FRAMEWORK",
                    "name": "redis2",
                    "ports": {
                        "ports": [
                            {
                                "number": 4740,
                                "protocol": "tcp"
                            }
                        ]
                    }
                },
                "container": {
                    "type": "DOCKER",
                    "docker": {
                        "image": "redis",
                        "network": "HOST",
                        "privileged": false,
                        "parameters": [
                            {
                                "key": "net",
                                "value": "infrastructure"
                            }
                        ],
                        "force_pull_image": false
                    }
                }
            },

Inside container I do see it has assigned IP such as  <IP_ADDRESS>, <IP_ADDRESS>

In the case of calico provisioned container, as you seen above container ip is not available in mesos state data. But the container internally does have an IP, is there a way we can ask mesos-consul to pull IP from inside the container and have it registered?

Not really. mesos-consul runs on one node and would have to communicate with docker on each follower node.

Finally able to get Calico's IP registered thru mesos-consul. Apparently version 0.28 of Mesos has a fix which makes calico's Ip available. Earlier I was running version 0.27.
Now one final question, since my current mesos-ip-order is docker,mesos,host. I end up seeing consul list internal IP for both Calico provisioned and Bridge provisioned. Is there any way can I have mesos-consul to register internal Ip for a specific set of containers and host ips for other set of containers?


Point entity position not saved

On visEditor when saving a scene, the position of Point entities is always saved as (0,0).

Fixed, thanks!


Loan option page & social media link copy updates

Copy updates
Changes

Intro paragraph on /loan-options & title/metadescription for /loan-options and /loan-options/conventional-loans
Social media content on various pages

Review

@sonnakim
@mthibos

Screenshots
Loan options copy & metadescriptions:

Social share copy:
/explore-rates:

/loan-estimate

/closing-disclosure

/mortgage-estimate

/mortgage-closing

/loan-options

/loan-options/FHA-loans

/loan-options/conventional-loans

/loan-options/special-loan-programs

Notes

I updated the links in Twitter to use www.consumerfinance.gov instead of localhost to get more accurate character counts. The /loan-options count was a little over so I tweaked its content slightly -- removing the initial The and subsituting an ampersand for and.

Checklist

[ ] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[ ] Code follows the standards laid out in the front end playbook
[ ] Passes all existing automated tests
[ ] New functions include new tests
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged
[ ] Visually tested in supported browsers and devices
[ ] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)

@virginiacc   Looks great!
Just one edit (not your fault, it was wrong in the copy deck, apologies):
/loan-options:
Say "CFPB" not "@CFPB" in email, so we don't look like idiots:

I'll leave it to @mthibos to approve this, since I'm traveling today!


Command module:make not working as expected

Hello there!
The second time you run php artisan module:make it fails!
What are the steps I did to reproduce the bug?

I created a new laravel version
I added nWidart/laravel-modules package
I create 2 modules php artisan module:make Post and then php artisan module:make Forum
This is the error I got:

Also I tried to change the composer.json for the created package adding:
"extra": {
        "laravel": {
            "providers": [
                "Modules\\Post\\Providers\\PostServiceProvider"
            ],
            "aliases": {

            }
        }
    },

I forgot to autoload the modules:
        "psr-4": {
            "App\\": "app/",
            "Modules\\": "Modules/",```


RFC: rename "structure-like" to "non-rec structure"

This RFC is tracking a suggestion by Leo in this comment to rename the "structure-like" concept to "non-rec structure" to prevent confusion in the future.
This would involve:

renaming isStructureLike to isNonRecStructure (and is_structure_like to is_non_rec_structure in the kernel)
renaming matchConstStructureLike to matchConstNonRecStructure
and tracking down all other places "structure-like" appear.

RFC accepted


Line number for check failure is wrong.

After fixing #43 I realized that every failed test gets annotated with  Check failure on line 1 in XXX.
I expected that the line number will be computed correctly. Unfortunately only line 1 is shown.
Example Report: https://github.com/mustaphazorgati/taskana/runs/1840214460?check_suite_focus=true

Fixed in v1.

Unfortunately this is still not fixed.
I've tried v1 and v1.0.8.

Build: https://github.com/mustaphazorgati/taskana/runs/1916926316?check_suite_focus=true

Hey @jmisur!
This issue is currently not fixed. I've tried again: https://github.com/mustaphazorgati/taskana/runs/2130607083?check_suite_focus=true
Please reopen this. :)

Thanks for reopening! :)
If it helps: You can download the surefire xml files from the workflow: https://github.com/mustaphazorgati/taskana/actions/runs/661020161
Artifact-id: surefire-reports

I checked your surefire results and found out an interesting piece (which I noticed previously but forgot about). Your testcase classname is a bit weird.
<testcase name="should_SplitListIntoChunks_When_CallingPartitionBasedOnSize" classname="pro.taskana.common.api.CollectionUtilTest(H2)" time="0.049">
I know you're running some test matrix setup which makes tests work on different dbs. How did you set this up? Is this something manual or out of the box? Is it always in this format classname(strategy) or it's a custom format? I need to know if I can take this example as the generic pattern so I can extract a raw classname from it. That's why line resolution doesn't work.

Hey @jmisur,

How did you set this up?
we use the maven property surefire.reportNameSuffix to add the information on which database the test ist executed.
See here: https://github.com/Taskana/taskana/pull/1536/files#diff-9c5fb3d1b7e3b0f54bc5c4182965c4fe1f9023d449017cece3005d3f90e8e4d8

The documentation of the property can be found here: http://maven.apache.org/surefire/maven-surefire-plugin/test-mojo.html#reportNameSuffix

Thank you for fixing this :)

Fixed in v1.


Time issue again

hi sir 😅 it is me again. the fix you did already fix the issue but another issue again. for example:
(2022-03-10 12:00 AM) due: 1hr 3min 21s but after the due of 1hr 3min 21s the due: should state that it is OVERDUE however it reset and show due: for the next day of 12:00 am
Like this on my  notes:
Departure (2022-03-10 12:00 AM) due: 16h 38min 26s

Expected result: and it should stay OVERDUE
Departure (2022-03-10 12:00 AM) due: OVERDUE

This am/pm system is frustrating...
I hope it finally works now, thanks again

yea sorry man 😅 after the fix the time for the current date doesnt work now 😅 for example is set the due today with:
(2022-03-10 11:00 pm) it says OVERDUE already eventhough it has 2hours before the due, this is working before the last fix 😅

one day I'll just start writing unit tests on all of my projects..
This is fixed now. Again. Probably...

all is fixed now, sorry if it got you frustrated 😅 thank you very much for fixing this great plugin bro. 😊


[FEATURE] Planning for Web Attributes to Metadata switch-over

Planning for the web metadata switch.
Right now on desktop we have metadata.json files which include the metadata for attributes built into them.  We need to convert the web version to stop using the global viame/attributes endpoint and start using either the metadata in the file or an individual metadata.json file that can be used to be portable between the desktop and web versions.
First thing is to swap over the current attributes to utilizing the metadata for the folder:

Would help to be able to do incremental updates instead of wholesale metadata updates, similar to how the track.json file upserts and delete works.
This will require a useAttributes tool as a sibling to use Styling and others to load and manage the attributes data locally
Edits to attributes through the additional system will now cause metadata updates to be queued up

Second Tasks:

Modifying ingest of csv's and existing track.json files to automatically populate attributes.  There is some code already in the desktop version to guess the type of the attribute when loading.
Exporting of all detections should include a metadata.json file which includes customTypeStyles as well as the attributes for import on desktop version.
Ensure that desktop version can import metadata.json and that it won't conflict with calculated attributes.
Enable ingestion of track.json along with metadata.json for importing the attributes and styling from the desktop or another project folder.

Final Tasks:

Sharing dialog to be able to import another public dataset's metadata for styling/attributes with the option to import both or a single one.
This will do a merge with the current metadata without overwriting any conflicting items.
Sharing of metadata for attributes and styling between projects.  Where a user can navigate to another public dataset and use it as a basis for their own project's attributes and types styling.

At the point now of doing Exporting and Importing of metadata.json files.  We have attribute calculations on both desktop and web.

Details on Metadata.json Exporting and importing.

Need to Specifically exporting on a full export the attributes and the custom styling.  This will allow for transferring between desktop and web those settings.
Desktop looks for metadata.json which will contain a structure for the attributes and styling data.
On Web it will need to be part of the import process.  Meaning that we will support multiple .json files and that validation will have to occur on each json to see if it is properly formatted
I may also want to provide a way to export and import only the metadata.json.  That way there is functionality to transfer data between systems/people.  This will flow eventually into sharing metadata.json files for custom types and attributes.

@BryonLewis is there something actionable here or can this be closed?

I think this can be closed, you basically implemented it.


cxx/AssistantCns.hxx has unfinished formulas (to produce+process databases of resources.) Do this as cxx/ClassCns.hxx or (new) cxx/ClassDatabases.hxx

Lots of this has general purpose uses, such as to do https://github.com/SwuduSusuwu/SubStack/blob/trunk/posts/CnsCompress.md
cxx/AssistantCns.hxx#L22
static std::vector<FilePath> assistantDefaultHosts = {
/* Universal Resources Locators of hosts which `questionsResponsesFromHosts()` uses
 * Wikipedia is a special case; has compressed downloads of databases ( https://wikipedia.org/wiki/Wikipedia:Database_download )
 * Github is a special case; has compressed downloads of repositories ( https://docs.github.com/en/get-started/start-your-journey/downloading-files-from-github )
 */
	"https://stackoverflow.com",
	"https://superuser.com",
	"https://quora.com"
};

/* @throw std::bad_alloc
 * @post If no question, `0 == questionsOrNull.bytecodes[x].size()` (new message synthesis).
 * If no responses, `0 == responsesOrNull.bytecodes[x].size()` (ignore).
 * `questionsOrNull.signatures[x] = Universal Resource Locator`
 * @code Sha2(ResultList.bytecodes[x]) == ResultList.hashes[x] @endcode */
void questionsResponsesFromHosts(ResultList &questionsOrNull, ResultList &responsesOrNull, const std::vector<FilePath> &hosts = assistantDefaultHosts);
void questionsResponsesFromXhtml(ResultList &questionsOrNull, ResultList &responsesOrNull, const FilePath &filepath = "index.xhtml");
const std::vector<FilePath> assistantParseUrls(const FilePath &filepath = "index.xhtml"); /* TODO: for XML/XHTML could just use [ https://www.boost.io/libraries/regex/ https://github.com/boostorg/regex ] or [ https://www.boost.org/doc/libs/1_85_0/doc/html/property_tree/parsers.html#property_tree.parsers.xml_parser https://github.com/boostorg/property_tree/blob/develop/doc/xml_parser.qbk ] */
const FileBytecode assistantParseQuestion(const FilePath &filepath = "index.xhtml"); /* TODO: regex or XML parser */
const std::vector<FileBytecode> assistantParseResponses(const FilePath &filepath = "index.xhtml"); /* TODO: regex or XML parser */

?questionsResponsesFromXhtml() "shadows outer" fix@6596f60 is an insignificant update, but has to do with this


fix ec2 tag resource type name

var ResourceType = "ec2Resource" was not a valid resource type to tag

/lgtm


Enhancement: get the export / import Timewarp feature of the old pipeline

Is there an existing issue for this?

[X] I have searched the existing issues.

Please describe the feature you have in mind and explain what the current shortcomings are?
We want to get the export / import Timewarp feature of the old pipeline :
From memory here is the main steps
-Export from flame : generate xml (check to see if dependant of shotgun)
-A script will convert them to json and publish the result to shotgun
-Import / Apply to camera / anim curve in Maya(dependent of shotgun data)
-Import / Apply to read node in Nuke(dependent of shotgun data)
How would you imagine the implementation of the feature?
First to get back the same fonctionnality with the same software.
But to be discuss with Vincent Goy after where it's better to export timewarp (Hiero or Flame)
Are there any labels you wish to add?

[X] I have added the relevant labels to the enhancement request.

Describe alternatives you've considered:
No response
Additional context:
No response

Here is the tool interface in maya :

accessible in the ShotGrid Menu : Launch Timewarp (wip)


500 Internal Server Error recurrent issue

Describe the bug
When running yarn install on a project that has momentjs as a dependency, I often get this error:
******** react-ui ...
yarn install v1.3.2
[1/4] Resolving packages...
[2/4] Fetching packages...
[91mwarning There appears to be trouble with our server. Retrying...
[0m[91mwarning There appears to be trouble with our server. Retrying...
[0m[91mwarning There appears to be trouble with our server. Retrying...
[0m[91mwarning There appears to be trouble with our server. Retrying...
[0m[91mwarning There appears to be trouble with our server. Retrying...
[0m[91mwarning There appears to be trouble with our server. Retrying...
[0m[91mwarning There appears to be trouble with our server. Retrying...
[0m[91mwarning There appears to be trouble with our server. Retrying...
[0m[91merror An unexpected error occurred: "http://verdaccio.internal/moment/-/moment-2.22.1.tgz: Request failed \"500 Internal Server Error\"".
[0minfo If you think this is a bug, please open a bug report with the information provided in "/opt/source/react-ui/yarn-error.log".
info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.

To Reproduce
Steps to reproduce the behavior:

Add moment js as a dependency
Run yarn install
See error

Expected behavior
Install to go through without error.

Thank you for the response, these are not momentjs related logs, but it seems to happen with other packages as well
http <-- 200, user: null(<IP_ADDRESS>), req: 'GET /rc-notification/-/rc-notification-3.1.1.tgz', bytes: 0/89832
http <-- 200, user: null(<IP_ADDRESS>), req: 'GET /rc-input-number/-/rc-input-number-4.0.7.tgz', bytes: 0/89730
http <-- 200, user: null(<IP_ADDRESS>), req: 'GET /rc-input-number/-/rc-input-number-4.0.7.tgz', bytes: 0/89730
error--- unexpected error: ESOCKETTIMEDOUT
Error: ESOCKETTIMEDOUT
    at ClientRequest.<anonymous> (/usr/local/app/node_modules/request/request.js:812:19)
    at Object.onceWrapper (events.js:273:13)
    at ClientRequest.emit (events.js:182:13)
    at TLSSocket.emitRequestTimeout (_http_client.js:652:40)
    at Object.onceWrapper (events.js:273:13)
    at TLSSocket.emit (events.js:182:13)
    at TLSSocket.Socket._onTimeout (net.js:448:8)
    at ontimeout (timers.js:427:11)
    at tryOnTimeout (timers.js:289:5)
    at listOnTimeout (timers.js:252:5)
http <-- 500, user: null(<IP_ADDRESS>), req: 'GET /rc-notification/-/rc-notification-3.1.1.tgz', error: internal server error
warn --> host registry.npmjs.org is now offline
error--- unexpected error: ESOCKETTIMEDOUT
Error: ESOCKETTIMEDOUT
    at ClientRequest.<anonymous> (/usr/local/app/node_modules/request/request.js:812:19)
    at Object.onceWrapper (events.js:273:13)
    at ClientRequest.emit (events.js:182:13)
    at TLSSocket.emitRequestTimeout (_http_client.js:652:40)
    at Object.onceWrapper (events.js:273:13)
    at TLSSocket.emit (events.js:182:13)
    at TLSSocket.Socket._onTimeout (net.js:448:8)
    at ontimeout (timers.js:427:11)
    at tryOnTimeout (timers.js:289:5)
    at listOnTimeout (timers.js:252:5)
http <-- 500, user: null(<IP_ADDRESS>), req: 'GET /rc-collapse/-/rc-collapse-1.9.2.tgz', error: internal server error
error--- unexpected error: ESOCKETTIMEDOUT
Error: ESOCKETTIMEDOUT
    at ClientRequest.<anonymous> (/usr/local/app/node_modules/request/request.js:812:19)
    at Object.onceWrapper (events.js:273:13)
    at ClientRequest.emit (events.js:182:13)
    at TLSSocket.emitRequestTimeout (_http_client.js:652:40)
    at Object.onceWrapper (events.js:273:13)
    at TLSSocket.emit (events.js:182:13)
    at TLSSocket.Socket._onTimeout (net.js:448:8)
    at ontimeout (timers.js:427:11)
    at tryOnTimeout (timers.js:289:5)
    at listOnTimeout (timers.js:252:5)
http <-- 500, user: null(<IP_ADDRESS>), req: 'GET /rc-editor-mention/-/rc-editor-mention-1.1.7.tgz', error: internal server error
error--- unexpected error: ESOCKETTIMEDOUT
Error: ESOCKETTIMEDOUT
    at ClientRequest.<anonymous> (/usr/local/app/node_modules/request/request.js:812:19)
    at Object.onceWrapper (events.js:273:13)
    at ClientRequest.emit (events.js:182:13)
    at TLSSocket.emitRequestTimeout (_http_client.js:652:40)
    at Object.onceWrapper (events.js:273:13)
    at TLSSocket.emit (events.js:182:13)
    at TLSSocket.Socket._onTimeout (net.js:448:8)
    at ontimeout (timers.js:427:11)
    at tryOnTimeout (timers.js:289:5)
    at listOnTimeout (timers.js:252:5)
http <-- 500, user: null(<IP_ADDRESS>), req: 'GET /rc-input-number/-/rc-input-number-4.0.7.tgz', error: internal server error
http <-- 500, user: null(<IP_ADDRESS>), req: 'GET /linebreak/-/linebreak-0.3.0.tgz', error: uplink is offline

It seems related to https://github.com/verdaccio/verdaccio/issues/877
Perhaps https://github.com/verdaccio/verdaccio/issues/720

I think it could be the same as #877 indeed, restarting the docker usually helps.

I think I have the same issue.. Although I don't get an error my virtual machine just hangs (at everything, I can't even SSH into it or login through the virtual console).
My CPU usage spikes at 100% with 1 core, at 50% with 2 cores and 25% with 4 cores.
It's a freshly installed Debian 9. I installed screen, curl, sudo and node10. Small packages with yarn add uuid and sometimes even yarn add @angular/cli work. But generating an angular library with ng generate library my-lib always crashes the server hosting verdaccio.
The machine has 2gb memory, 20GB disk space SSD and quad core 2.4Ghz.
I also tried verdaccio 4 alpha 0

My solution was to use debian 8 for the verdaccio VM. Been using it in production since the end of October

I'll close this one in favor of #877


Document patch

Python API call by Lazar at https://gist.github.com/lbodor/5b83c19ec4dd8d44415f1d23396f2fa3


[BUG] Default Web Browser not working.

Describe the bug
When clicking on a link from the Messages App for example, it takes me to Safari and not Chrome
To Reproduce
Steps to reproduce the behavior:

Go to 'Messages'
Click on a link someone has sent you
See that it opens to Safari

Expected behavior
I expect the link to open in Google Chrome, not Safari
Extra Info
Please include

iPhone 7
iOS 13.6
0.2.4

Ignored: com.apple.mobilesafari
<FBSOpenApplicationOptions: 0x283770ef0> {
    PayloadOptions = {
        UIApplicationLaunchOptionsSourceApplicationKey = com.apple.MobileSMS;
    }
    PayloadURL = https://uk.ign.com/wikis/the-legend-of-zelda-ocarina-of-time-3d/Biggoron%27s_Sword;
}

Closing. RIP Lorenzo.


kafka: Close response connection on any response read error

Signed-off-by: Romain Lenglet<EMAIL_ADDRESS>

test-me-please

test-me-please


Severity parameter is case-sensitive.

When running the command:
    invoke-scriptanalyzer .\a.ps1 -severity error

invoke-scriptanalyzer will throw an ArgumentException.

Hi @imfrancisd, thanks for reporting this! We will fix this issue as soon as possible!

Hi @imfrancisd , the bug is fixed and submitted into BugFixes. #148


[VT-98]  key not working while editing command

David Patrick on 2014-06-27T15:30:31Z says:

Migrated metadata:
Created: 2014-06-27T15:30:31Z
Modified: 2015-03-02T06:22:59Z

Scott Kostyshak on 2015-03-01T00:38:57Z says:
I can reproduce. Indeed, it doesn't work for any of VIT's prompts.

Scott Kostyshak on 2015-03-02T06:22:59Z says:
Fixed at 3c30a83e.


[CI] Use sdist on Linux and build without cython

Closes #11
Closes #12: use sdist on Linux
Rename README.md -> README.rst
Remove pip caching on Travis

Looks OK, tests are passing...


Removing working groups + leadership

Can re-introduce leadership pending an updated list of personnel

EDIT: we should preserve the KCIS page for now; the other pages may be removed / redirected to the home page.

Re-introduced Working Groups to show KCIS (which is now the only Working Group). Can handle this differently if need be.

This is a tricky situation–one of the WG leads has been continuing to promote the KCIS working group as the source of activities for the project despite the consortium transition away from working groups. There seems to be some miscommunication going on.
I will be discussing with the other VICC directors about how we want to handle this collectively. In the meantime, we should still hide the working group navbar even if we allow direct-link landing on the KCIS page.

Commented out for now. Still looking into autobuilding PRs for Jekyll.

Great. Let's open that feature as a separate issue.


[Improvement] Change namespaces

namespace SandboxPlus;

Yes, this might also happen to allow us to be compatible with addons built against/referencing Facepunch/Sandbox (though there might also be plenty of other issues with that preventing that from ever really being a possibility)


git-pull.js: Breaking game changes with folder paths

The game used to require leading slashes before a folder path is given, but now it has changed to not even allow them optionally. For those that choose to use git-pull.js to pull scripts into their own folder, things are no longer working.

#300 Was affected by this, but was not the correct solution


Practice 6 - simple_local_planner - simplifying code when adding objects and intersection problem

In the following part:
https://github.com/AnishShr/autoware_mini_practice/blob/d925e1e517a6656f8355264a1323ff155cd0e166/practice_6/nodes/planning/local/simple_local_planner.py#L144-L170

You can get rid of min_dist and you don't need to assign flaot(inf) to d if you just calculate all the distances from points to d (list) and later use min(d) where necessary.
This does not seem relevant: if d <= self.local_path_length: we already checked that they intersect with local_path_buffer so the min(d) must be within local_path
The original idea was to check if intersects then do intersection with convex hull (not just points like you are doing) and use the resulting polygon points from intersection. Currenlty the problem can be if object is big and original convex hull points are outside the local_path_buffer - then there will be no intersection! But there would be intersection between 2 polygons.

Removed additional variables like min_dist and assigning floats
Modified the code to ge tthe min distance from closest obstacle exterior coordinates
https://github.com/AnishShr/autoware_mini_practice/blob/84b0545f202deb38fdabf1e09150737125080dc0/practice_6/nodes/planning/local/simple_local_planner.py#L158-L170

Git commit: https://github.com/AnishShr/autoware_mini_practice/commit/84b0545f202deb38fdabf1e09150737125080dc0

ok


Create reference template(s) to demonstrate solution management through dev/test/prod

Demonstrate scenarios for both ISVs as well as customers/SIs building xRM apps.  Demonstrate when to use unmanaged/managed/etc.

Demonstrate how to apply patching and cloning across dev/test/prod:
https://technet.microsoft.com/en-us/library/mt628808.aspx

After further review, this really isn't necessary.  Trying to do this in a reference template would be hard to maintain.  The goal of this project is to "kick start" ones understanding and empower them to solve their own problems, not try to solve every possible scenario.  We can accomplish dev/test/prod explanation with documentation.  The "MSCRM Pack Solution" VSTS task enables you to set your package type to both managed and unmanaged.  You can then use VSTS Environments to deploy the unmanaged solution to one or many dev orgs and the managed to test/staging/prod.


⚠ Bofrost.de has degraded performance

In d9b40c5, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 977 ms

Resolved: Bofrost.de performance has improved in 371676e after 6 minutes.


Endless Warning and exception for no active Mixed Reality Toolkit in your scene

Overview
When playing a scene without configuring Mixed Reality Toolkit you will get a warning.
If you click Later the message will immediately pop up again. This will happen endlessly or until you click Run.
If you click on 'Run' it will throw exceptions.
Expected Behavior
No warning, no error. I may not want Mixed Reality Toolkit in my scene. Please do not pop a warning message each time I run a scene that does not have a toolkit configured just because I have a toolkit in the project.
Actual Behavior
This message pops up endlessly:

This error occurs if you hit run:

Steps to reproduce
Create new scene in a project hat has MixedRealityToolkit in the assets.  Click run without configuring.
Unity Editor Version
2018.3.6f1
Mixed Reality Toolkit Release Version
Latest from github on release MRTK branch

Related, and better stated than #3526 .
Thanks!

I've gone through replicating this and it seems a little more serious that this concludes.
Once ANY scene has instantiated any MRTK component that replies on the MRTK Manager, it would seem that something isn't destryed correctly as the MRTK persists in the editor.
Restarting the editor clears the issue.
Replicating

Create a clean project
Import MRTK - Ignore or later the VR check
Start a blank scene and save
Observe no warnings
Create a new scene and use the MRTK -> Configure and run scene
Observe no warnings
Create another new blank scene and run
Observe the MRTK warnings appear as if MRTK is being called
Load the original Blank scene
Observe the MRTK warnings appear as if MRTK is being called
Restart Unity
Load the original Blank scene
Observe no warnings

So MRTK is not being cleared down somewhere once called.
IF there is ANY MRTK component in the scene, then the behaviour is correct, as the MRTK Manager will create itself if it doesn't exist (removing the MRTK Manager from a scene is not enough, as we add a Gaze manager to the camera as well)

Somewhat necro-ing an older bug, but this is come up from numerous people - there are cases where people want to have the MRTK around in assets but not necessarily have things in scene (i.e. I can have things in source control and want to do non-MRTK things in an experimentation scene) but this popup keeps coming up.
Scheduling this to fix in GA

Alrighty, got a chance to take a look at this one today. This got removed as part of https://github.com/microsoft/MixedRealityToolkit-Unity/pull/4305. The warnings now are actually in the MRTK inspectors themselves (for example, if you attach an Interactable to a GameObject but you don't have an MRTK object in the scene, you'll see a warning in the Interactable's inspector, not a intrusive popup.


Remove jobs by wf name for force-complete and aborted - v1.0.8

1.0.8 version of #6028
Tested and good to be pushed to the production agents.

Beginning pylint and unit tests.
See https://cmssdt.cern.ch/jenkins/job/DMWM-PR-test/89/ for details


Some questions to be answered by the Windows App SDK team

Describe the bug
Some questions to be answered by the Windows App SDK team

Why don't you provide Microsoft. Windows. Management Deployment namespace provides a full explanation of the API documentation? It has been six months since the API collection was released.
Are there future opportunities to provide the Windows App SDK with classes and methods that UWP has limited access to in desktop applications? ShutdownManager, TimeZoneSettings, DateTimeSettings.
Why not provide improved FilePicker, FolderPicker, etc., in the Windows App SDK? These selectors are commonly used by developers, and the classes and methods provided by WinForm are currently inelegant when called directly through the COM interface or using WPF.
Why not provide the user's common SystemTray function in the Windows App SDK?

需要 Windows App SDK 团队回答的一些问题

为什么不提供 Microsoft.Windows.Management.Deployment 命名空间提供 API 文档的详细解释？截止到这个 API 集合的发布已经过去半年时间了。
未来有机会为 Windows App SDK 提供 UWP 在桌面应用中受限制访问的那些类和方法吗？比如 ShutdownManager，TimeZoneSettings，DateTimeSettings。
3.为什么不在 Windows App SDK 提供改进的 FilePicker，FolderPicker 等等？这些选取器是开发者经常使用的，目前直接通过 COM 接口调用或使用 WPF，WinForm 提供的类和方法很不优雅。
4.为什么不在 Windows App SDK 中提供用户常用的 SystemTray 功能？

Integration of WinUI issues in this place
WinUI 问题的整合在这个地方
https://github.com/microsoft/microsoft-ui-xaml/issues/9964
https://github.com/microsoft/microsoft-ui-xaml/discussions/9946
Steps to reproduce the bug
None
Expected behavior
None
Screenshots
None
NuGet package version
Windows App SDK 1.6.0: 1.6.240829007
Packaging type
Packaged (MSIX)
Windows version
Windows 11 version 22H2 (22621, 2022 Update)
IDE
Visual Studio 2022
Additional context
None

For 3, I will ask a counter question. While there has been some posts about fixing the UWP pickers, I haven't seen much of a fuss about implementing a new set of pickers in the Windows App SDK.
The last time this happened, I even mentioned that to really help, someone should make a feature request for pickers in the Windows App SDK and then get as many people as possible to react to it as possible to show there is a lot of support for this. But that feature request never happened.
So he counter question is, what demand is there for these pickers? If the pickers are on Microsoft's roadmap, what reason is there to make them high priority?
It is currently very hard to tell the difference between a few scattered people who just want to grumble and the demand being there but nobody is willing to do the work to make this known. Well, that is enough devil's advocate for one day, but remember that customer demand is everything.

For 1, yes WASDK docs team is responsible.
For 2, file a WASDK API request for the same.
For 3, upvote #8.
For 4, file a WASDK API request for the same.

For 1, yes the WASDK docs team is responsible. For 2, file a WASDK API request for the same. For 3, upvote #8. For 4, file a WASDK API request for the same.

2 link:https://github.com/microsoft/WindowsAppSDK/issues/4253
4.link:https://github.com/microsoft/WindowsAppSDK/issues/4063

For 1 is addressed; docs are available
P.S. 1 is redundant with Hope for Microsoft.Windows.Management.Deployment under the namespace all API to add the API documentation
#4253 which noted docs are available
For 2 and 4 please create a feature request issue (as mominshaikhdevs noted
For 3 see Improved StorageFile APIs
#8 and add your input and vote. Or if there's a different aspect to your request pleas create a new feature request issue with your details.


Add widget News

👷 Deploy Preview for cding processing.

Name
Link

🔨 Latest commit
82fc1fc6b81bf95f40e1cd08a09a367a42d280d1

🔍 Latest deploy log
https://app.netlify.com/sites/cding/deploys/62ef74854a77f7000966a616


Custom memory allocators

This PR adds the possibility to pass custom memory management functions. It can be done by specifying an additional header file where the custom functions are defined (or included) as -DOSQSP_CUSTOM_MEMORY_H=myfile.h
The file should have the structure
# include <mycustomem.h>
# define c_malloc  mymalloc
# define c_calloc mycalloc
# define c_realloc as myrealloc
# define c_free myfree

@nikalra does it solve your issue? #136. Feedbacks are welcome.

This would work great, thank you!

Added an example implementing malloc/free custom functions that count the net number of allocations.   These must be implemented in a separate file or library and linked to along with the osqp static library by user applications.
Changed the cmake syntax for the custom memory header to -DOSQSP_CUSTOM_MEMORY=myfile.h, since the previous way was likely to clash with include guards implemented around a custom header.


Visual studio build sets versions on .dll files but MSbuild does not

Visual studio build sets versions on .dll files but MSbuild leaves .dll versions as <IP_ADDRESS> on .Net Framework solution.
Any ideas?
Thank you in advance

I figured out what was going wrong, i had installed the nuget package and also did CLI installation. It worked in visual studio which made me think i had installed it correctly, but after days of trying in MSBuild i decided to start fresh. Upon following only the nuget installation it worked.


Port Ionide's release process over

@forki We need to change this line too.
I think we should write "publisher": "Ionide",

Other than that seems good for me.

This PR broke the plugin, I have the fix in my local version will publish it shortly

already fixed on master

https://twitter.com/MangelMaxime/status/973550199133130752
<3


pijul: 1.0.0-beta.2 -> 1.0.0-beta.4

Description of changes
Update pijul to 1.0.0-beta.4

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

There is beta 4 now: https://crates.io/crates/pijul/1.0.0-beta.4

@dywedir could you backport this to 22.11? Would be nice to have all the latest features. :)


Twitter fix

twitter meta information are only inserted in the head, if twitter.username is set. see https://github.com/mmistakes/minimal-mistakes/blob/master/_includes/seo.html
closes #189
@ClaireWyatt could you please test with https://346-267395254-gh.circle-artifacts.com/0/SORSE/index.html and the validator tool whether that works?

I still see the grey image using the validator tool and the link you provided

me too. I stil see the grey box and paper scroll.

does twitter maybe have a problem with the banner being .svg?

because the meta information are there in https://346-267395254-gh.circle-artifacts.com/0/SORSE/index.html
  <meta name="twitter:site" content="@Int_SORSE">
  <meta name="twitter:title" content="SORSE">
  <meta name="twitter:description" content="International Series of Online Research Software Events">
  <meta name="twitter:url" content="https://346-267395254-gh.circle-artifacts.com/0/SORSE/">

    <meta name="twitter:card" content="summary_large_image">
    <meta name="twitter:image" content="https://346-267395254-gh.circle-artifacts.com/0/SORSE/assets/images/sorse-banner.svg">

from https://developer.twitter.com/en/docs/tweets/optimize-with-cards/overview/summary-card-with-large-image

twitter:image
A URL to a unique image representing the content of the page. You should not use a generic image such as your website logo, author photo, or other image that spans multiple pages. Images for this Card support an aspect ratio of 2:1 with minimum dimensions of 300x157 or maximum of 4096x4096 pixels. Images must be less than 5MB in size. JPG, PNG, WEBP and GIF formats are supported. Only the first frame of an animated GIF will be used. SVG is not supported.

@ClaireWyatt or @TeriForey: Could you please try again with https://349-267395254-gh.circle-artifacts.com/0/SORSE/index.html?

It looks good to me!

great, thanks @TeriForey! can you also check another page, https://350-267395254-gh.circle-artifacts.com/0/SORSE/programme/call-for-contributions/index.html for instance? I don't have a twitter account, sorry

There's more text so it's only showing the centre of the image - I still think this looks okay though.

this is set by the og_image in _config.yml. shall we use the logo here?
https://github.com/SORSE/sorse.github.io/blob/7775f44a461cd1a0d148e610d20087a4918f351b/_config.yml#L18
shall we use the logo here?

Try it and I'll take a look at how it looks

alright: here is the new version: https://351-267395254-gh.circle-artifacts.com/0/SORSE/programme/call-for-contributions/index.html

Same problem, in fact now it's worse because it's a lower res!

hmm, I see. It needs to be quadratic...

this should work now
https://352-267395254-gh.circle-artifacts.com/0/SORSE/programme/call-for-contributions/index.html
sorry, I am in a workshop, so everything takes a while :wink:

Looks good!

alright. Feel free to approve and merge this PR if you like


Goody (DoorDash) API Integration - Gift Cards 🎁

Description
We typically reward members who are in the Top 75 of the leaderboard of a Fam Friday session with an AlgoExpert license as well as a DoorDash gift card. To administer the DoorDash gift cards, we use a service called Goody.
We want to build an integration with the Goody API so that we can make sending these gifts as easy as a few clicks in our Admin Dashboard. Eventually, we can build a deeper integration with the Airmeet API and automatically grab the top 75 in the leaderboard, so sending gift cards would be a 1-click task! The latter part we'll save for the next iteration.
For now, we'll build a simple integration with the Goody Automation API, which allows us to send gifts to multiple people.
Acceptance Criteria
This issue should do the following:

Implement the ability to send Goody gifts, using their API (ie: POST /v1/order_batches).

You can assume that the product_id is read from an environment variable (ie: GOODY_DOORDASH_PRODUCT_ID).

In the Admin Dashboard -> Students page, add an option when clicking on the row's option to "Send Goody Gift". This should open a modal which contains a form asking for the "Gift", which for now should have "DoorDash Gift Card" as the only option.
Upon submitting the form above, this should send the gift to the member. This should also send an internal Slack notification.

Don't worry about testing this - just implement 90% and I'll jump in to help testing with a real Goody API key.

@sajjad-source

ty! would love to work on this

Hey @sajjad-source! Are you still working on this issue? No worries if not, just let me know so we can open it up for someone else!

Going to open up this issue!

Hey @tomas-salgado. Really sorry I just saw your messages and had completely forgot about this issue 😅. Have had a lot on my plate since school ended, but I hope to be able to contribute again when space opens up. Thanks for opening up the issue again

If this issue is back up for grabs I'm down to work on it!

@nicholasg2001 You've got it!

@ramiAbdou Still learning the codebase so I just have a clarifying question. For the goody API code, should I be creating a folder goody inside of packages/core/src/modules?

@nicholasg2001 just realized we missed this question, apologies. For now it should be fine to just have the folder goody in that location.
Let me know if you still plan on working on this issue!

I'd be down to take it @nicholasg2001 if you're no longer working on it. LMK

You can take it!

@tomas-salgado Im working on it! assign me

Got a little bit of help from @RiosNicholas can we add him as an assignee as well? (bro barely did anything ngl but lets show him some love)

LMAO, nah I cooked with the frontend @omar-elbaz

Got a little bit of help from @RiosNicholas can we add him as an assignee as well? (bro barely did anything ngl but lets show him some love)

chill, I cooked with the frontend @omar-elbaz

I think you assigned the wrong Nicholas, lol @tomas-salgado

Ahh mb 😆


add kuberay on kind jupyter notebook

Why are these changes needed?
Add a jupyter notebook about how to setup a kuberay cluster on local kind environment
Related issue number
N/A
Checks

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

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

I have one more comment on the tutorial. I think user may be interested in the way to submit workloads to the cluster.  10001 port, can we add some guidance on where to retrieve the endpoint and how to verify connectivity is ok like below?
ray.util.connect('address')

I have one more comment on the tutorial. I think user may be interested in the way to submit workloads to the cluster. 10001 port, can we add some guidance on where to retrieve the endpoint and how to verify connectivity is ok like below?
ray.util.connect('address')

I made some changes to the notebook.

@wilsonwang371 Since controller already creates the service, is there a way we can avoid duplicate service creation?  I think extra port mapping should be enough for user to connect from localhost?
https://kind.sigs.k8s.io/docs/user/configuration/#extra-port-mappings

@wilsonwang371 Since controller already creates the service, is there a way we can avoid duplicate service creation? I think extra port mapping should be enough for user to connect from localhost?
https://kind.sigs.k8s.io/docs/user/configuration/#extra-port-mappings

We discussed this in #157. This service is needed for redirecting traffic from kind nodeport to kuberay.


java.lang.VerifyError: com/vincentbrison/openlibraries/android/dualcache/lib/DualCache

Why do I initialize DualCache complains
This is my initialization code
public void onCreate() {
super.onCreate();
DualCacheLogUtils.enableLog();
DualCacheContextUtils.setContext(getApplicationContext());
DualCache mCache = new DualCacheBuilder(mCacheId, 2, String.class)
.useDefaultSerializerInRam(mRamCacheSize)
.useDefaultSerializerInDisk(mDiskCacheSize, true);
}
This is the error information
java.lang.VerifyError: com/vincentbrison/openlibraries/android/dualcache/lib/DualCache
at com.vincentbrison.openlibraries.android.dualcache.lib.DualCacheBuilder.(DualCacheBuilder.java:18)
at com.jbp.minegold.MinegoldApplication.onCreate(MinegoldApplication.java:31)
at android.app.Instrumentation.callApplicationOnCreate(Instrumentation.java:1000)
at android.app.ActivityThread.handleBindApplication(ActivityThread.java:4502)
at android.app.ActivityThread.access$1300(ActivityThread.java:150)
at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1336)
at android.os.Handler.dispatchMessage(Handler.java:99)
at android.os.Looper.loop(Looper.java:213)
at android.app.ActivityThread.main(ActivityThread.java:5153)
at java.lang.reflect.Method.invokeNative(Native Method)
at java.lang.reflect.Method.invoke(Method.java:511)
at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:797)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:564)
at dalvik.system.NativeStart.main(Native Method)

Try to clear your project and rebuild it.

Did the issue solved doing what @auchri said ?

I clean up after the project is still the same，Maybe it's my build.gradle configuration is wrong? But I was written in document steps .

From your stacktrace I understand you are initing the cache during a test.
Have you the same issue when initing the cache from your regular
application ?
Le 6 oct. 2015 9:17 AM, "luckyliuqi"<EMAIL_ADDRESS>a écrit :

I clean up after the project is still the same，Maybe it's my build.gradle
configuration is wrong? But I was written in document steps .
—
Reply to this email directly or view it on GitHub
https://github.com/vincentbrison/android-easy-cache/issues/13#issuecomment-145765636
.

No enought information.

@luckyliuqi did u fix this problem. I am getting this problem when I initialize it on my activity

@luckyliuqi did u fix this problem. I am getting this problem when I initialize it on my activity

@yon86
Hi,
To have more context and be able to solve this, could you post your stacktrace, your initialization calls and the way you add the library to your gradle ?
BR

here is my intialization code
cache =  new DualCacheBuilder<ItemResponse>("mainCahche", 1, ItemResponse.class)
                        .useDefaultSerializerInRam(1000)
                        .useDefaultSerializerInDisk(1000, true);

in my app gradle file I just added the library path my build tool version is  buildToolsVersion "23.0.1"
 compile 'com.vincentbrison.openlibraries.android:dualcache:2.2.2@jar'

here is the stack trace
  java.lang.VerifyError: com/vincentbrison/openlibraries/android/dualcache/lib/DualCache
                                                                          at com.vincentbrison.openlibraries.android.dualcache.lib.DualCacheBuilder.<init>(DualCacheBuilder.java:18)
                                                                          at com.addiscan.analytics.Activity.SaleActivity.onCreate(SaleActivity.java:124)
                                                                          at android.app.Activity.performCreate(Activity.java:5426)
                                                                          at android.app.Instrumentation.callActivityOnCreate(Instrumentation.java:1105)
                                                                          at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2269)
                                                                          at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:2363)
                                                                          at android.app.ActivityThread.access$900(ActivityThread.java:161)
                                                                          at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1265)
                                                                          at android.os.Handler.dispatchMessage(Handler.java:102)
                                                                          at android.os.Looper.loop(Looper.java:157)
                                                                          at android.app.ActivityThread.main(ActivityThread.java:5356)
                                                                          at java.lang.reflect.Method.invokeNative(Native Method)
                                                                          at java.lang.reflect.Method.invoke(Method.java:515)
                                                                          at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:1265)
                                                                          at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1081)
                                                                          at dalvik.system.NativeStart.main(Native Method)

Thanks for the info, I will look into this ASAP.
BTW :

Do you use Progard ?
Do you use Java 6 or 7 in your project ?

I am not using proguard yet and use java 7 thanks

Hi,
I am able to recreate, I will submit a fix ASAP.
BR.

The issue is when importing library this way
compile 'com.vincentbrison.openlibraries.android:dualcache:2.2.2@jar' the dependencies are not resolved. Thus some class can not be found and you get this java.lang.VerifyErrorerror. To resolve this, just import the library without@jar:compile 'com.vincentbrison.openlibraries.android:dualcache:2.2.2'

thank you this fixed my issue


Change the folder patch operation type into a discriminated union

Motivation
When I want to modify asset folders, I have to specify patch operations. The SDK provides the AssetFolderModels.IModifyAssetFoldersData type for the operations. The problem is that this type doesn't guide me as to what properties are needed for different patch operations. I always have to check with the docs for what kind of properties I have to provide. It is unnecessarily difficult and opens up opportunities for mistakes. For example, when removing a folder I only have to specify the reference property, but when renaming, I also have to specify the value property.
Properties also can have different types based on the operation type. For example, when renaming a folder, the value property has to be of type string (the new name), but when adding a folder, the same property value has to be an object specifying the folder object.
Example:
// ...
.withData([
  {
    op: "addInto",
    value: "new folder name", // there is no error here, even though it is obviously wrong
  },
]);

Proposed solution
Change the AssetFolderModels.IModifyAssetFoldersData to be a discriminated union of available operations. Based on the op property, TypeScript would be able to identify what operation am I writing and would properly type-check them and offer me relevant IntelliSense. That would resolve both of the problems.
The type would look something like this:
export type ModifyAssetFoldersData =
  AddIntoOperation
  | RemoveOperation
  | RenameOperation;

type RemoveOperation = {
  op: "remove";
  // ...
};
// define the rest of the operations

There even is a bug, because the the of the value property is defined as IAddOrModifyAssetFolderData, which is an object with folder properties like name or folders. However, when I want to define the rename patch operation, I need to assign a string (the new name) into the value property (as per the documentation), but the type doesn't allow me to do so.


Can you prioritize it  ?

OKay. I will learn postgres USER-DEFINED feature first. Any document do you know about that?

http://www.sqlines.com/postgresql/how-to/create_user_defined_type