Cannot successfully run graphql codegen, as prisma-binding throws an error

Reproduction of the bug:
https://github.com/minheq/prisma-binding-bug
Bug
Cannot successfully run graphql codegen, as prisma-binding throws an error
Reproducing the bug
Clone repo and run following commands
yarn
yarn codegen

Expected result
It should generate schema.ts file with typescript types/interface for schema.graphql, based on .graphqlconfig.yml
Actual Result
Throws an error:
$ graphql codegen
warning command prepare both exists in plugin /Users/minheq/Documents/Repositories/prisma-binding-bug/node_modules/graphql-cli-prepare and is shipped with the graphql-cli.
The plugin is being ignored.
✖ Generating bindings for project app...
TypeError: Cannot read property 'type' of undefined
    at getWhere (/Users/minheq/Documents/Repositories/prisma-binding-bug/node_modules/prisma-binding/dist/utils.js:44:9)
    at /Users/minheq/Documents/Repositories/prisma-binding-bug/node_modules/prisma-binding/src/utils.ts:41:12
    at Array.map (<anonymous>)
    at getTypesAndWhere (/Users/minheq/Documents/Repositories/prisma-binding-bug/node_modules/prisma-binding/src/utils.ts:38:21)
    at Object.getExistsTypes (/Users/minheq/Documents/Repositories/prisma-binding-bug/node_modules/prisma-binding/src/utils.ts:9:17)
    at PrismaTypescriptGenerator.renderExists (/Users/minheq/Documents/Repositories/prisma-binding-bug/node_modules/prisma-binding/src/PrismaTypescriptGenerator.ts:71:20)
    at PrismaTypescriptGenerator.render (/Users/minheq/Documents/Repositories/prisma-binding-bug/node_modules/prisma-binding/src/PrismaTypescriptGenerator.ts:19:32)
    at /Users/minheq/Documents/Repositories/prisma-binding-bug/node_modules/prisma-binding/src/bin.ts:69:34
    at step (/Users/minheq/Documents/Repositories/prisma-binding-bug/node_modules/prisma-binding/dist/bin.js:33:23)
    at Object.next (/Users/minheq/Documents/Repositories/prisma-binding-bug/node_modules/prisma-binding/dist/bin.js:14:53)

I can reproduce.
@minheq --
If you modify your buggyTypes query to include some arguments, it will resolve your issue. For example:
type Query {
  buggyTypes(where: ID!): [BuggyType!]!
}

I will work on a PR for this.

@medelman17 Great! thanks for the tip :)

Hey @minheq @medelman17 👋,
Can you still reproduce this issue with the latest versions of the stack?

@maticzav I just got this issue by having a type with no properties other than id: ID! @unique defined in my Prisma schema. The OP's reproduction repo has the same setup, so I think that might be the source of the issue here.

Hey @minheq 👋,
I believe this is a design decision since prisma-binding assumes you are using it with Prisma itself where each type also features where input argument. I was able to fix the bug by adding where in the arguments list.
schema {
  query: Query
}

type Query {
  buggyTypes(where: Input!): [BuggyType!]!
}

input Input {
  foo: String
}

type BuggyType {
  id: ID!
}

We will discuss the next steps in the upcoming week and change the code accordingly. 🙂

i see :)
sounds good to me!

Hey @minheq 👋,
As promised, we discussed your case; prisma-binding is designed especially for Prisma. As a consequence, it doesn't work with other non-prisma schemas which you've already experienced.
I suggest you try using graphql-binding instead of prisma-binding to solve your problem.

Thank you for your response! @maticzav

Thank you for your response! @maticzav


Add @staticmethod decorators to avoid confusing the linter

Codecov Report

Merging #8 (0284966) into main (ad687e9) will decrease coverage by 2.59%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main       #8      +/-   ##
==========================================
- Coverage   91.00%   88.41%   -2.59%
==========================================
  Files           6        6
  Lines         678      682       +4
==========================================
- Hits          617      603      -14
- Misses         61       79      +18

Impacted Files
Coverage Δ

pulsectl_asyncio/pulsectl_async.py
85.56% <100.00%> (-4.28%)
:arrow_down:

examples/subscribe_peak_example.py
95.65% <0.00%> (-4.35%)
:arrow_down:

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

Hm, this does not work, because these are not actually used as static methods, but more like decorator functions that return (non-static) methods. So actually, they should be defined before defining the class, not as static members of the class.


Register Page design.

Design Register Page as Per Login Page theme.

Hey @harshalkaigaonkar can you please elaborate? I just took a look at the register page, and it looks similar to the login page.
What exactly should be done?

I want to refactor the design of the navbar of the auth page, such that it disappears for Auth pages and can only be shown on other than that pages.


Constraint to only exit fzf if processes base on key press are done?

Checklist

[X] I have read through the manual page (man fzf)
[X] I have searched through the existing issues
[X] For bug reports, I have checked if the bug is reproducible in the latest version of fzf

Output of fzf --version
0.53
OS

[ ] Linux
[X] macOS
[ ] Windows
[ ] Etc.

Shell

[X] bash
[X] zsh
[ ] fish

Problem / Steps to reproduce
So I found out why fzf causes prompt flick issue, especially with fzf --expect 'ctrl-h,ctrl-m',
this dues to users will run other processes base on the key press later after fzf already exit:
test=$(echo "1\n2\n3\n" | fzf --expect 'ctrl-h,ctrl-l')

key=$(head -1 $test)

if [[ $key == 'ctrl-h' ]]; then
  sleep 2 # this represents multiple processes which cause the slowness
  # this 100% cause prompt flick in which it will disappear and re-appear again when the processes are done.
fi

To fix this, fzf need methods to monitor processes after keys pressed with --expect, and make sure only exit fzf if they're all done.
I think this is better than requires all shells to give some kind of constraint for a special process (fzf) to aware of others before exit/return.

I simply can't put sleep 2 out of if [[ $key == 'ctrl-h' ]] dues to I'm processing result base on the key pressed.
If it's ctrl-h then do something with result but else do other things.

I'm not suggesting that you add "sleep" to your script, it doesn't make any sense. I was trying to demonstrate the effect of --no-clear.

If sleep 2 does not make sense then check this:
test=$(echo "1\n2\n3\n" | fzf --expect 'ctrl-h,ctrl-l')

key=$(head -1 <<< $test)

if [[ $key == 'ctrl-h' ]]; then
  cat ./test.txt # this file is 10MB text file
fi

--no-clear will cause prompt swallow which fzf takes whole prompt and after fzf exit, prompt still black until user manually redraw screen, is there a way to use it without swallow prompt?

For those who are banging your head against wall about prompt flickering issue TUI program like fzf and shells:
Stay away from bash/zsh/sh

Start to use elvish shell

That's the solution.
Also note, you can't completely remove flick because whatever text display on a screen always need a redraw, obviously under terminal the redraw can cause a lookalike flick but zsh/bash/sh with its way to spawn new program each time you need to do a small things would give you a big flick that drag you into hell. Good luck.

For those who are banging your head against wall about prompt flickering issue with TUI program like fzf and shells:
Stay away from bash/zsh/sh

Start to use elvish shell

That's the solution.
Also note, you can't completely remove flick because whatever text display on a screen always need a redraw, obviously under terminal the redraw can cause a lookalike flick but zsh/bash/sh with its way to spawn new program each time you need to do a small things would give you a big flick that drag you into hell. Good luck.

A bit misinfo, you can still use zsh/bash/sh but be aware of using external tools.


Remove explicit path to which

It is possible that on *nix systems, which program isn't located in /usr/bin/which. Thus, just use which and assume that it's in PATH. At least on NixOS, which is located in a complex path in nix store, thus the path to it cannot be hardcoded.
Is this ok? Are there cases when the path to which should be hardcoded?

Absolutely! Thanks :-)


fix: type import broken, fixes #133

I don’t know why the type import seems to work in the demo and in scalar/scalar, but the path is just wrong.
I just tried to find a way to catch this in CI, which seemed straight-forward, but it’s all just passing.
Anyway, here is the quick fix. I’ll find a solution to prevent this in the future.

There are some weird issues with the dts plugin it seems. Some files land in random places in the dist folder.


Finalise Overall Structure

This will be closed when the steering group is happy with the proposed overall structure.

The overall structures looks good to me


[Feature request] Support global color overrides

Hey there, thanks for writing owo-colors!
I'd love to use it except I'm running into one roadblock -- I wish owo-colors supported a global override for when e.g. a --color=always option is passed in to override any environment variables.
Even better would be to support thread-local and task-local overrides, which would make it easy to use owo-colors in libraries too.

@sunshowers alright so I started working on implementing this, but I think I should note: by default owo-colors does not do any checking if colors should be enabled/disabled, and that is only present if you use the if_supports_color API.
So I guess a few questions to help understand working towards a solution for you:

Is if_supports_color sufficient for your usecase if such an override API was provided? (Most owo-colors users end up abstracting this away in some form)
If not, I imagine you'd want it to function more like colored where it automatically checks this on every display operation, what would be your expectation as to the behavior? Part of why I didn't go this route originally was because assuming stdout vs stderr vs other wasn't something I wanted to do, but I'd be willing to put it behind a feature if you had a design idea

(Additional context: one usecase of owo-colors is doing things like outputting colors over debug for an embedded system, but even outside of that I find it hard to assume stdout (more common) or stderr (often colored because errors/warnings))
I'm definitely willing to support this feature (as well as whatever your usecase is), I just want to make sure I'm not assuming that if_supports_color is a good fit or if I should start by adding a feature for having something akin to if_supports_color be applied globally (and as I've said, if so, trying to understand what behavior users of such a feature would expect from it).

Thanks! How would a user of if_supports_color abstract over it? I don't fully understand the API

So if_supports_color is used something like this:
println!(
    "{}: this was incorrectly configured because reasons"
    "Error".if_supports_color(Stdout, |text| text.red())
);

but typically for applications you can make assumptions about things like where you use colors and what output stream they're going to.
So you might do something like:
fn print_error(err: &impl fmt::Display) {
    println!(
        "{}: {}",
        "Error".if_supports_color(Stderr, |x| x.red()),
        err
    );
}

Which is less convenient that having to pepper around "Error".if_supports_color(...) every error location. My point just being that usually by the time there's enough invocations of if_supports_error that it's annoying, your application is big enough that you'll end up abstracting your printing enough that all your coloring happens in 1-5 call sites anyways. (My example is somewhat bad because I imagine the Actual way error printing gets abstracted out is that all printable errors get bubbled up to a single match but hopefully you get the idea)

Ahh I see, that makes sense. What about using this for libraries?

Well for libraries I suppose it varies a lot. If your library immediately pipes your colored output to println/eprintln then your library can make similar assumptions, if not I would imagine you'd probably want this behavior in the first place, otherwise you'd have to decide between your users seeing overcoloring (if you assumed Stdout but they displayed to Stderr) or undercoloring (if you only colored if both Stdout and Stderr were found to support colors).
And to be clear--in a library which is providing configurable fancy/colored output I'd personally say to make a builder pattern for the printing, which would let you configure Stdout vs Stderr, something like ariadne's builder, which features print and eprint so it knows both where to output and would thus allow it to also figure out whether it that output stream supports color
Which, again, I could totally be wrong about what people's usecases are, and I'm definitely open to considering options to better support those whose usage doesn't fit these ideas

@sunshowers let me know your thoughts on #34. I'm going to spin off thread-local and task-local overrides into their own issues. Thread-local should be easy, but I can't said I've ever used task-locals before so I'll need to look into that.
Definitely let me know if either of those are something you need and if so I'll bump them up on my todo list.

Alright from my testing and from CI things look good. If this works well for you @sunshowers I'll publish these at 3.1.0, feel free to make another issue if what I have so far isn't enough, feel free to the PR if it has any issues for you, and if you have any thoughts on #35 or #36 please let me know! (Especially #36 as I have no idea what people would want/expect from task-locals, as I probably write 10x more sync code than async, but #35 is also open for comment on whether global or thread-local overrides should take priority)

Just to let you know @sunshowers this has been pushed in owo-colors 3.1.0

Thank you so much

After a bit of experimentation, I started finding the supports colors stuff somewhat unwieldy to use unfortunately. Part of the problem is that my use case doesn't really fall under the print_error model: my use case has some pretty ad-hoc messages, such as:
error!("unrecognized registry URL {} found for {}",
    registry_url.if_supports_color(Stream::Stderr, |s| s.bold())),
    package.name().if_supports_color(Stream::Stderr, |s| s.bold())),
);

and so on. It seems like there's two approaches I can use in this case:

Just give in and accept some boilerplate: define a method to abstract over, though auto-deref appears to make it so that I can't really do that.
Use the stylesheet approach: define a struct of styles somewhere and initialize it to certain colors. (That's basically what I've done for libraries anyway.)

I think correct usage on Unix terminals in this sort of ad-hoc situation necessarily has to imply not directly using the really simple coloring methods: .blue() etc.
(BTW I agree that the colored approach is incorrect because it assumes stdout -- but it's really clean in comparison since you don't need to use something like if_supports_colors).

(Note that the above only applies to these sorts of ad-hoc coloring schemes -- for more uniform ones if_supports_color works great!)


Is there any way to disable "swipe to open" but not swipe to close?

The thing is, I am using jquery slick carousel plugin which allows to swipe left/right to move carousel slides. But slideout's swipe from left event overrides it. So I want to disable swipe from left to open the menu. But would like to leave the swipe to right enabled, to close the menu. Is it possible?

I am afraid that is not possible atm, still, you can use the touch option to disable touch events.
touch: false

OK, thanks for the quick reply. Any chance to get that feature in future?

Hi @aslamdoctor!
You can do that using the current API. You should use the open and close events.
// Create a new instance with touch disabled
var slideout = new Slideout({
  'panel': document.getElementById('main'),
  'menu': document.getElementById('menu'),
  'touch': false
});

// Then, use the slideout `open` and `close` events
slideout.once('open', slideout._initTouchEvents);
slideout.on('open', slideout.enableTouch);
slideout.on('close', slideout.disableTouch);

Demo: http://codepen.io/anon/pen/WvavzZ


[WFLY-19504] Remove MyFaces-related test from JSFDeploymentProcessorTestCase

https://issues.redhat.com/browse/WFLY-19504
Remove obsolete tests
Clean up remaining code

@jasondlee Do we have the necessary coverage in the myfaces FP code?

@jasondlee Do we have the necessary coverage in the myfaces FP code?

@bstansberry There is some testing: https://github.com/wildfly-extras/wildfly-myfaces-feature-pack/blob/master/testsuite/subsystem/src/test/java/org/wildfly/extras/jsf/myfaces/test/subsystem/SubsystemSanityTestCase.java
That seems a bit light, but I'm not sure off hand how much more we need. I imagine running many of the same JSF tests from the WF testsuites could be useful there, but I have not given that much thought of late, tbh. I've filed an issue on that component to track the question: https://github.com/wildfly-extras/wildfly-myfaces-feature-pack/issues/11

Thanks @jasondlee


Update Electron to 9.4.3 to pickup bug fixes

Update Electron to 9.4.3 to pickup important bug fixes from upstream.

Ported to release/1.26


Doesn't work on 1.11.2

Hi! I updated my server to 1.11.2 and mcMMO Extras just stopped working. It worked perfectly fine on 1.10.2

Any errors?

Nope, there are none :/

Do you use any boss bar plugins? It works for me

I do, but on 1.10 the two plugins worked perfectly together. Now mcMMO Extras doesn't work, but the other boss bar plugin does work (it sends boss bar messages every minute or so that last around 5 seconds).

but the other boss bar plugin does work (it sends boss bar messages every minute or so that last around
5 seconds)

I think that plugin overrides mcMMOExtras. What's the name of it?

It doesn't, I removed it and it still didn't work. Regardless, its called BossMessenger, its Premium, and it worked together with mcMMO Extras on 1.10.2.
Does mcMMO extras rely on a certain config option being true in mcMMO though?

Does mcMMO extras rely on a certain config option being true in mcMMO though?
No it doesn't.

Could you try it on a local test server with just mcMMO and mcMMOExtras? Sadly there is no easy way to get the currently associated bossbars of a player.

Okay, that worked... Now I need to find the incompatibility whatever it is.
Is there a reason where there are no error messages? I know its not Boss
Messenger since they worked together flawlessly, and when removing it it
didn't start working.
The other somewhat annoying thing is that those are the two only plugins
that should be using the Boss Bar.
On Fri, Dec 23, 2016 at 1:24 PM, games647<EMAIL_ADDRESS>wrote:

Does mcMMO extras rely on a certain config option being true in mcMMO
though?
No it doesn't.
Could you try it on a local test server with just mcMMO and mcMMOExtras?
Sadly there is no easy way to get the currently associated bossbars of a
player.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/games647/McMMOExtras/issues/9#issuecomment-269047140,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AL6K-ZPl9Ebq9s8OnqAvSwrUf2dj9iszks5rLDvygaJpZM4LUjPo
.

 :)

What was the cause? I'm very interested

I had the mcmmoextras.expbar permission set to false. The weird thing is
even though its false, it said "expbar has been disabled/enabled", and not
"you do not have permission for this command", when executing /expbar.
On Mon, Dec 26, 2016 at 1:50 PM, games647<EMAIL_ADDRESS>wrote:

What was the cause? I'm very interested
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/games647/McMMOExtras/issues/9#issuecomment-269244038,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AL6K-VfIHpvr7BZ6Wfhp5d_jwMA2Ib9Rks5rMDaJgaJpZM4LUjPo
.

Thanks for the info


GEODE-8114: Refactor set

I tried to make all the "layers" of the set operation implement the RedisSetCommands interface.
Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[ ] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[ ] Is your initial contribution a single, squashed commit?

[ ] Does gradlew build run cleanly?

[ ] Have you written or updated unit tests to verify your changes?

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, check Concourse for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

you should update the description of this PR. You got rid of the statics on RedisSet and RedisHash. You also introduced a singleton EMPTY for both of them.


contracts: Add FeeCurrencyWhitelist

Adds the FeeCurrencyWhitelist contract, part of #57.
@pahor167 Please have a look at this. The changes I did:

Add the license modifier
Update solidity pragma
Fix imports
Removed the public visibility modifier from the constructor, because solc complained.

Mhg, I'm not sure why this complains about the ABIs. I'm suing the abigen version that is required:
% abigen --version
abigen version 1.10.25-stable

Mhg, I'm not sure why this complains about the ABIs. I'm using the abigen version that is required:
% abigen --version
abigen version 1.10.25-stable

Did you run make in op-bindings folder after all changes to smart contract were done ? I took the liberty to fetch branch, rebuild bindings and it seems that op-bindings-build is passing.
abigen --version abigen version 1.10.25-stable
My abigen version is same as yours.

This is included in #70


Fix apps broken by chain name change

Many apps broke because we now are returning the chainName directly from web, and so apps don't understand Ethereum as Mainnet or xDai as GnosisChain
We should try to fix this meanwhile in the SDK until we can get all apps affected updated to a recent version of the SDK

Fix will be created on safe-react side
https://github.com/gnosis/safe-react/issues/3339


Scale.get

Extended 'Scale.get' function to get notes outside of octave range. Added tests.

Very cool! I'll just have to check the tests when I get home to my computer.
Thank you for you contribution jaylupus (:

I'm not sure how the negative numbers should work. As of now I get a RangeError infinite-loop when trying to get scale.get(0), scale.get(-7), scale.get(-14), etc
Also it seems a bit counter-intuitive that scale.get(-1) isn't just one note down from unison? But I guess we have to keep that way

So I've added the functionality you've implemented. And thanks for that - it was an obvious addition to make to that method :)
Please let me now if it works out for you the way you expect. (and sorry for the late answer)

Thank you, I'll do so.
On Sun, Jul 2, 2017 at 8:05 AM, Jakob Miland<EMAIL_ADDRESS>wrote:

So I've added the functionality you've implemented. And thanks for that -
it was an obvious addition to make to that method :)
Please let me now if it works out for you the way you expect. (and sorry
for the late answer)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/saebekassebil/teoria/pull/113#issuecomment-312487918,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AK1KwRKd3kHbs0ua5LsdOSMoVR78vJJ-ks5sJ4eGgaJpZM4LDbR8
.


Mitosis => Angular:  Duplicated props

The variable name is declared once as an Input and then again as a class variable.
Add any other context about the problem here.](https://mitosis.builder.io/?outputTab=angular&code=import+{+useState%2C+onMount+}+from+'%40builder.io%2Fmitosis'%3B

export+default+function+ThrowAwayButton(props)+{
++const+state+%3D+useState({+name%3A+""+})%3B

++onMount(()+%3D>+{
++++state.name+%3D+props.name+%3F+props.name+%3A+'Default'%3B
++})%3B

++return+(
++++<div>
++++++Hello+from
++++++<span>+{name}<%2Fspan>
++++++!
++++<%2Fdiv>
++)%3B
}
)

Isn't that because you have both a prop and a state with the same name? To illustrate, here's the output if you name them differently: fiddle
Also note that you have a bug in your JSX: you reference name without prefixing it with state.  nor props.

You're right, this should be handled by the linter, closing this.


Add Insights Coverage Report doc

Description
Added documentation for the newly release isights => coverage feature.
Motivation and Context

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation fix (typos, incorrect content, missing content, etc.)

Checklist

[X] I have read the contributing document.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Hey @paweldudzinski -- hoping to get a review of this for early next week, if you are available. Let me know if you would like a screenshot of my local build to see how it looks rendered. Thanks!


Following storybook guide, vue example layout broken

When I follow the setup steps on the intro to storibook
on yarn serve I got this weird layout:

Windows 10 EDGE (chromium version)/chrome

@BuenGenio that is happening because the css  required to the tutorial is being added into the App.vue component. I left it as a trade-off as people in the past forgot to add it and asked why the styling was not being applied.


Mouseover tooltips

Suggested by /u/StrategicMagic on Reddit.
All kinds of information could be in this - pokedex-style mon descriptions (or even a short one-liner - "OU, Ground/Flying" for a mon, "100BP Physical Ground, hits adjacent" for a move, "Boosts Spe by 1.5x, locks user into one move" for an item, etc.)

This would require either:

Query some kind of external database (like Showdown's), or
Expand our pokedex

I'm inclined toward option 2 but it'd mean more work when updating the dex.

you could occasionally parse Showdown's files


App asks for file system access permission on Win10 2004 and doesn't show local drives.

Describe the bug
When Files app starts, it asks for granting file system access permission. After pressing 'Grant Permission' button, in settings page there's no Files app listed. However I'm able to access any folder or file on any of of my local drives by entering folder path manually. Also drives do not show on the side panel.
To Reproduce
Steps to reproduce the behavior:

Open Files UWP app.
Click on 'Grant Permission' button.
When file system settings page is shown, there's no Files app shown on the list of apps that can access file system.
Returning back to Files app, there's no drives at the side panel.

Expected behavior
The app must automatically access the file system and show all local drives in the side panel since I'm using Win10 version 2004 and Files UWP is a full-trust app.
Screenshots

Desktop (please complete the following information):

OS Version: Windows 10 2004 19041.450
App version: v<IP_ADDRESS>

Additional context
Complete re-installation didn't solve the problem.
Log file
debug.txt
debug_fulltrust.txt

@yarikv8601 could you check again with version 0.13.6? It may still crash but it will log more useful debug info

@gave92 logs from version 0.13.6:
debug.txt
2020-08-31 22:44:21.8290|INFO|Files.App|App launched| 2020-08-31 22:44:22.0290|INFO|Files.Filesystem.DrivesManager|Drive added: C:\, Fixed| 2020-08-31 22:44:22.0610|INFO|Files.Filesystem.DrivesManager|Drive added: D:\, Fixed| 2020-08-31 22:44:22.0940|INFO|Files.Filesystem.DrivesManager|Drive added: E:\, Fixed| 2020-08-31 22:44:22.1330|INFO|Files.Filesystem.DrivesManager|Drive added: F:\, CDRom| 2020-08-31 22:44:22.1590|WARN|Files.Filesystem.DrivesManager|One or more errors occurred. (Отказано в доступе. (Exception from HRESULT: 0x80070005))|System.UnauthorizedAccessException: Отказано в доступе. (Exception from HRESULT: 0x80070005)    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() + 0x21    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task) + 0x70    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task) + 0x38    at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd(Task) + 0x17    at Files.Filesystem.DrivesManager.<>c__DisplayClass20_0.<<GetDrives>b__1>d.MoveNext() + 0xb7
debug_fulltrust.txt
2020-08-31 22:44:23.0766|INFO|FilesFullTrust.QuickLook|QuickLook detected: False| 2020-08-31 22:44:43.0872|INFO|FilesFullTrust.Program|Argument: Terminate|

I'm getting the same Grant Permission request, and the same issue where the list of apps granted file system access is blank - on both my HP desktop and Lenovo laptop, using v14 of Files UWP. On my desktop, Files continues to work anyway, and I'm seeing my external drives. But on my laptop, it crashes immediately, and there's a toast message saying "something has gone wrong". LMK if you want log files from either one.
On my desktop, Files is running much faster now - the refresh of a new director level is far quicker, probably 2-3x faster than before (though I'd guess about half as fast as regular File Explorer)

@Talisman39ar yes, the log files for both would be useful. Thanks!

@Talisman39ar thank you. Looking at the log files, the issue you see on the HP is the same @yarikv8601 reported. On the lenovo is a different one, could you open a new issue describing the problem you experience on the lenovo?

@gave92 I think I should mention that before it all started I manually installed KB4568831 update and activated updated Start Menu using registry. Reverting to old Start menu didn't fixed the issue with Files app though.

@gave92 Having the same issue on the HP now too - Files starts (though window doesn't populate with files list), toast message about something went wrong appears, and Files disappears. Here's the log files from the HP - I'll do the Lenovo later and file a separate ticket for that, don't have it with me right now
debug.txt
debug_fulltrust.txt

@Talisman39ar could you check if disabling the "Continue where you left off" option in the app's settings "solves" the issue ?


Not able to use booleans or primitives when extending css properties in panda.config.

Description
I like to create a Grid component and use like this:
 <Grid mb="15vh" alignC justifyC>
  //
</Grid>

Where alignC and justifyC are shorthands for alignContent and justifyContent. In stitches, I would be able to map true to "center" or "100%", but it doesn't seem I can do that in panda config. In the config, the closet I could do was a mapped value:
  utilities: {
    extend: {
      alignContent: {
        shorthand: "alignC",
        property: "alignContent",
        values: {
          true: "center",
        },
        transform(value) {
          return { alignContent: value === true ? "center" : value };
        },
      },
  // ...
}

However, true is mapped to a string here and I have use this prop like alignC="true" instead of alignC.
Link to Reproduction
na
Steps to reproduce

Go to '...'
Click on '...'
Scroll down to '...'
See error

JS Framework
React (TS)
Panda CSS Version
^0.5.0
Browser
No response
Operating System

[X] macOS
[ ] Windows
[ ] Linux

Additional Information
No response

const config = {
  utilities: {
    extend: {
      alignContent: {
        shorthand: "alignC",
        property: "alignContent",
        values: { type: "boolean" },
        transform(value) {
          if (!value) return {};
          return { alignContent: "center" };
        },
      },
    },
  },
};


Basic Thread View with Quick Add and Time Map

Basic Thread view with:

Quick Add
Time Map

Need to add basic menu for links to e.g. timemap vizualization.


reference/test/solver/bicgstab_kernels.cpp tests fail when converted to float (i.e., single precision)

Having converted reference/test/solver/bicgstab_kernels.cpp to use single precision by copying it to reference/test/solver/bicgstab_kernels_float.cpp and making changes as per diff below:
51c51
<     using Mtx = gko::matrix::Dense<>;
---
>     using Mtx = gko::matrix::Dense<float>;
57c57
<               gko::solver::BicgstabFactory<>::create(exec, 8, 1e-15))
---
>               gko::solver::BicgstabFactory<float>::create(exec, 8, 1e-7))
62c62
<     std::unique_ptr<gko::solver::BicgstabFactory<>> bicgstab_factory;
---
>     std::unique_ptr<gko::solver::BicgstabFactory<float>> bicgstab_factory;
74c74
<     ASSERT_MTX_NEAR(x, l({-4.0, -1.0, 4.0}), 1e-8);
---
>     ASSERT_MTX_NEAR(x, l({-4.0, -1.0, 4.0}), 1e-4);
87c87
<     ASSERT_MTX_NEAR(x, l({{-4.0, 1.0}, {-1.0, 2.0}, {4.0, -1.0}}), 1e-8);
---
>     ASSERT_MTX_NEAR(x, l({{-4.0, 1.0}, {-1.0, 2.0}, {4.0, -1.0}}), 1e-4);
102c102
<     ASSERT_MTX_NEAR(x, l({-8.5, -3.0, 6.0}), 1e-8);
---
>     ASSERT_MTX_NEAR(x, l({-8.5, -3.0, 6.0}), 1e-4);
118c118
<     ASSERT_MTX_NEAR(x, l({{-8.5, 1.0}, {-3.0, 2.0}, {6.0, -5.0}}), 1e-8);
---
>     ASSERT_MTX_NEAR(x, l({{-8.5, 1.0}, {-3.0, 2.0}, {6.0, -5.0}}), 1e-4);

the following tests fail by not attaining the required accuracy:
[ RUN      ] Bicgstab.SolvesMultipleDenseSystems
.../ginkgo/reference/test/solver/bicgstab_kernels_float.cpp:87: Failure
Relative error between x and {{-4.0, 1.0}, {-1.0, 2.0}, {4.0, -1.0}} is 0.11072188922929171
	which is larger than 1e-4 (which is 0.0001)
x is:
	-4.0000228881835938	0.54083889722824097
	-1.0000052452087402	1.858891487121582
	4.000030517578125	-0.50263333320617676
{{-4.0, 1.0}, {-1.0, 2.0}, {4.0, -1.0}} is:
	-4	1
	-1	2
	4	-1
component-wise relative error is:
	5.7220131568155855e-06	0.45916110277175903
	5.2451812281639528e-06	0.070554256439208984
	7.6293363240331724e-06	0.49736666679382324

[  FAILED  ] Bicgstab.SolvesMultipleDenseSystems (1 ms)
[ RUN      ] Bicgstab.SolvesDenseSystemUsingAdvancedApply
.../ginkgo/reference/test/solver/bicgstab_kernels_float.cpp:102: Failure
Relative error between x and {-8.5, -3.0, 6.0} is 0.0014036462767795599
	which is larger than 1e-4 (which is 0.0001)
x is:
	-8.4900836944580078
	-2.9963669776916504
	5.9890694618225098
{-8.5, -3.0, 6.0} is:
	-8.5
	-3
	6
component-wise relative error is:
	0.0011666241814108455
	0.0012110074361165364
	0.0018217563629150391

[  FAILED  ] Bicgstab.SolvesDenseSystemUsingAdvancedApply (0 ms)
[ RUN      ] Bicgstab.SolvesMultipleDenseSystemsUsingAdvancedApply
.../ginkgo/reference/test/solver/bicgstab_kernels_float.cpp:118: Failure
Relative error between x and {{-8.5, 1.0}, {-3.0, 2.0}, {6.0, -5.0}} is 0.0015665962953109148
	which is larger than 1e-4 (which is 0.0001)
x is:
	-8.4900836944580078	0.99250507354736328
	-2.9963669776916504	1.9976270198822021
	5.9890694618225098	-4.9917192459106445
{{-8.5, 1.0}, {-3.0, 2.0}, {6.0, -5.0}} is:
	-8.5	1
	-3	2
	6	-5
component-wise relative error is:
	0.0011666241814108455	0.0074949264526367188
	0.0012110074361165364	0.0011864900588989258
	0.0018217563629150391	0.0016561508178710937

[  FAILED  ] Bicgstab.SolvesMultipleDenseSystemsUsingAdvancedApply (1 ms)

The problem here could be one of the following:

There's something wrong with the implementation of BiCGStab.
There's a problem with how we test. The number 1e-15 means "reduce the original residual by 15 orders of magnitude", not "obtain accuracy of 1e-15". Thus, if the original residual was to large (which we don't test anywhere), then the single precision version could might as well reduce the original residual by 7 orders of magnitude, and still get a solution which only has 1-3 correct digits. I think it would be more useful if we use a hard limit on residual norm for tests, but we have to implement different stopping criteria to do that.

Btw, there's an issue on the private repo about implementing tese stopping criteria already: https://gitlab.com/ginkgo-project/ginkgo/issues/60 Maybe we should consider moving it into the public repo.

Sorry if I misunderstood the parameters.
Please also note that the test with one RHS (SolvesDenseSystem) succeeds in float, if that may indicate you something.

Sorry if I misunderstood the parameters.

Even if you did, it's not an issue here. There's still something wrong with the tests.

Please also note that the test with one RHS (SolvesDenseSystem) succeeds in float, if that may indicate you something.

I noticed, and the explanation is probably this: https://gitlab.com/ginkgo-project/ginkgo/issues/55
TL; DR The convergence test is not exactly the same for multiple RHS as is for one RHS. Another thing we have to fix.

Yes, actually I remember now that we saw a similar problem for double when I integrated that code, and we had to settle with the current parameters for it to work. But we need to tackle the related issue in the private repo to make this work properly in the general case.

@tcojean, @hartwiganzt now that we have stopping criteria - maybe we should take another look into this?

I just checked this again. It looks like we have now fixed this problem in the meantime.
Result:
$ ./reference/test/solver/bicgstab_kernels
Running main() from /home/tcojean/ginkgo-github/build/third_party/gtest/src/googletest/src/gtest_main.cc
[==========] Running 7 tests from 1 test case.
[----------] Global test environment set-up.
[----------] 7 tests from Bicgstab
[ RUN      ] Bicgstab.SolvesDenseSystem
[       OK ] Bicgstab.SolvesDenseSystem (0 ms)
[ RUN      ] Bicgstab.SolvesMultipleDenseSystems
[       OK ] Bicgstab.SolvesMultipleDenseSystems (0 ms)
[ RUN      ] Bicgstab.SolvesDenseSystemUsingAdvancedApply
[       OK ] Bicgstab.SolvesDenseSystemUsingAdvancedApply (1 ms)
[ RUN      ] Bicgstab.SolvesMultipleDenseSystemsUsingAdvancedApply
[       OK ] Bicgstab.SolvesMultipleDenseSystemsUsingAdvancedApply (0 ms)
[ RUN      ] Bicgstab.SolvesBigDenseSystemForDivergenceCheck1
[       OK ] Bicgstab.SolvesBigDenseSystemForDivergenceCheck1 (0 ms)
[ RUN      ] Bicgstab.SolvesBigDenseSystemForDivergenceCheck2
[       OK ] Bicgstab.SolvesBigDenseSystemForDivergenceCheck2 (1 ms)
[ RUN      ] Bicgstab.SolvesMultipleDenseSystemsDivergenceCheck
[       OK ] Bicgstab.SolvesMultipleDenseSystemsDivergenceCheck (0 ms)
[----------] 7 tests from Bicgstab (2 ms total)

[----------] Global test environment tear-down
[==========] 7 tests from 1 test case ran. (3 ms total)
[  PASSED  ] 7 tests.

Patch I used:
diff --git a/reference/test/solver/bicgstab_kernels.cpp b/reference/test/solver/bicgstab_kernels.cpp
index bb89d71f..bf37f9b0 100644
--- a/reference/test/solver/bicgstab_kernels.cpp
+++ b/reference/test/solver/bicgstab_kernels.cpp
@@ -52,8 +52,8 @@ namespace {

 class Bicgstab : public ::testing::Test {
 protected:
-    using Mtx = gko::matrix::Dense<>;
-    using Solver = gko::solver::Bicgstab<>;
+    using Mtx = gko::matrix::Dense<float>;
+    using Solver = gko::solver::Bicgstab<float>;

     Bicgstab()
         : exec(gko::ReferenceExecutor::create()),
@@ -66,28 +66,28 @@ protected:
                       gko::stop::Time::build()
                           .with_time_limit(std::chrono::seconds(6))
                           .on(exec),
-                      gko::stop::ResidualNormReduction<>::build()
-                          .with_reduction_factor(1e-15)
+                      gko::stop::ResidualNormReduction<float>::build()
+                          .with_reduction_factor(1e-7)
                           .on(exec))
                   .on(exec)),
           bicgstab_factory_precision(
-              gko::solver::Bicgstab<>::build()
+              gko::solver::Bicgstab<float>::build()
                   .with_criteria(
                       gko::stop::Iteration::build().with_max_iters(50u).on(
                           exec),
                       gko::stop::Time::build()
                           .with_time_limit(std::chrono::seconds(6))
                           .on(exec),
-                      gko::stop::ResidualNormReduction<>::build()
-                          .with_reduction_factor(1e-15)
+                      gko::stop::ResidualNormReduction<float>::build()
+                          .with_reduction_factor(1e-7)
                           .on(exec))
                   .on(exec))
     {}

     std::shared_ptr<const gko::Executor> exec;
     std::shared_ptr<Mtx> mtx;
-    std::unique_ptr<gko::solver::Bicgstab<>::Factory> bicgstab_factory;
-    std::unique_ptr<gko::solver::Bicgstab<>::Factory>
+    std::unique_ptr<gko::solver::Bicgstab<float>::Factory> bicgstab_factory;
+    std::unique_ptr<gko::solver::Bicgstab<float>::Factory>
         bicgstab_factory_precision;
 };

@@ -100,7 +100,7 @@ TEST_F(Bicgstab, SolvesDenseSystem)

     solver->apply(b.get(), x.get());

-    ASSERT_MTX_NEAR(x, l({-4.0, -1.0, 4.0}), 1e-8);
+    ASSERT_MTX_NEAR(x, l({-4.0, -1.0, 4.0}), 1e-4);
 }

@@ -113,7 +113,7 @@ TEST_F(Bicgstab, SolvesMultipleDenseSystems)

     solver->apply(b.get(), x.get());

-    ASSERT_MTX_NEAR(x, l({{-4.0, 1.0}, {-1.0, 2.0}, {4.0, -1.0}}), 1e-8);
+    ASSERT_MTX_NEAR(x, l({{-4.0, 1.0}, {-1.0, 2.0}, {4.0, -1.0}}), 1e-4);
 }

@@ -128,7 +128,7 @@ TEST_F(Bicgstab, SolvesDenseSystemUsingAdvancedApply)
     solver->apply(alpha.get(), b.get(), beta.get(), x.get());

-    ASSERT_MTX_NEAR(x, l({-8.5, -3.0, 6.0}), 1e-8);
+    ASSERT_MTX_NEAR(x, l({-8.5, -3.0, 6.0}), 1e-4);
 }

@@ -144,7 +144,7 @@ TEST_F(Bicgstab, SolvesMultipleDenseSystemsUsingAdvancedApply)
     solver->apply(alpha.get(), b.get(), beta.get(), x.get());

-    ASSERT_MTX_NEAR(x, l({{-8.5, 1.0}, {-3.0, 2.0}, {6.0, -5.0}}), 1e-8);
+    ASSERT_MTX_NEAR(x, l({{-8.5, 1.0}, {-3.0, 2.0}, {6.0, -5.0}}), 1e-4);
 }

@@ -169,7 +169,7 @@ TEST_F(Bicgstab, SolvesBigDenseSystemForDivergenceCheck1)
         x,
         l({0.13853406350816114, -0.08147485210505287, -0.0450299311807042,
            -0.0051264177562865719, 0.11609654300797841, 0.1018688746740561}),
-        1e-9);
+        1e-4);
 }

@@ -193,16 +193,17 @@ TEST_F(Bicgstab, SolvesBigDenseSystemForDivergenceCheck2)
         x,
         l({0.13517641417299162, 0.75117689075221139, 0.47572853185155239,
            -0.50927993095367852, 0.13463333820848167, 0.23126768306576015}),
-        1e-9);
+        1e-4);
 }

-double infNorm(gko::matrix::Dense<> *mat, size_t col = 0)
+template <typename T>
+T infNorm(gko::matrix::Dense<T> *mat, size_t col = 0)
 {
     using std::abs;
-    double norm = 0.0;
+    T norm = 0.0;
     for (size_t i = 0; i < mat->get_size()[0]; ++i) {
-        double absEntry = abs(mat->at(i, col));
+        T absEntry = abs(mat->at(i, col));
         if (norm < absEntry) norm = absEntry;
     }
     return norm;
@@ -273,7 +274,7 @@ TEST_F(Bicgstab, SolvesMultipleDenseSystemsDivergenceCheck)

     // Not sure if this is necessary, the assertions above should cover what is
     // needed.
-    ASSERT_MTX_NEAR(xc, testMtx, 1e-14);
+    ASSERT_MTX_NEAR(xc, testMtx, 1e-4);
 }


Confusing interaction of top-level effects and lax mode

From @s-zanella  on slack:
module N
open FStar.ST
open FStar.HyperHeap
val rs: rs:(rid * rid){fst rs <> snd rs}
let rs =
  let r = new_region root in
  let s = new_region root in
  r, s

module P
open N
val bug: unit -> Lemma False
let bug _ = ()

What’s surprising from a user perspective is the difference between
Verified module: P (740 milliseconds)
All verification conditions discharged successfully

and
$ bin/fstar.exe --include ulib/hyperstack/ N.fst P.fst
./N.fst(7,0-10,6): (Warning) Top-level let-bindings must be total; this term may have effects
./P.fst(6,0-6,14): (Error) assertion failed(Also see: ./bin/../ulib/prims.fst(175,22-175,33))
Verified module: N (811 milliseconds)
Verified module: P (2490 milliseconds)
1 error was reported (see above)

that is weird behavior
seems like only lax-checking N causes P to succeed
this is quite bad actually, since we rely heavily on separately verifying individual modules while lax-checking their dependences.
will investigate and hopefully fix quickly


indexable graphic attributes

implements #77, for now just colors
kinda mesy right now, will refactor

color indexing is really nice :D
https://user-images.githubusercontent.com/9403332/208393879-0d373a23-9596-46f6-b07b-3a9facd6e95d.mp4

this insanity is possible now: https://www.youtube.com/watch?v=j_gwi-Wf1Ao
also this which is more useful :smile:

For scatter we can make size an indexable feature too.

@EricThomson, @clewis7 , @ArjunPutcha
guys this is pretty epic, changing a GraphicFeature can trigger events :smile:
Caitlin, pick_info formatting is the same as pygfx events to make life easy :D
https://user-images.githubusercontent.com/9403332/209122940-961f3157-5e81-4912-9953-e4c95221d2b8.mp4

Nice -- this will make certain other things so much easier.


feat(snap): Add core-common-config-bootstrapper

Signed-off-by: Mengyi Wang<EMAIL_ADDRESS>

If your build fails  due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/edgex-go/blob/main/.github/Contributing.md
PR Checklist
Please check if your PR fulfills the following requirements:

[x] I am not introducing a breaking change (if you are, flag in conventional commit message with BREAKING CHANGE: describing the break)
[ ] I am not introducing a new dependency (add notes below if you are)
[ ] I have added unit tests for the new feature or bug fix (if not, why?)
[ ] I have fully tested (add details below) this the new feature or bug fix (if not, why?)
[ ] I have opened a PR for the related docs change (if not, why?)

Testing Instructions

New Dependency Instructions (If applicable)

recheck

@lenny-intel Thank you. I still need to address some open topics for example:
level=ERROR ts=2023-02-14T17:06:50.072503544Z app=core-data source=bootstrap.go:48 msg="did not get boolean from config provider for IsCommonConfigReady: strconv.ParseBool: parsing \"\": invalid syntax"

I will let you know as soon as everything has been resolved.

@lenny-intel Thank you. I still need to address some open topics for example:
level=ERROR ts=2023-02-14T17:06:50.072503544Z app=core-data source=bootstrap.go:48 msg="did not get boolean from config provider for IsCommonConfigReady: strconv.ParseBool: parsing \"\": invalid syntax"

I will let you know as soon as everything has been resolved.

That will happen in the retry loops waiting for Common Config to be ready. Is it causing the service to exit?

Yes, the services exit immediately after that error and give up after several quick restarts. See the logs here, as an artifact: https://github.com/edgexfoundry/edgex-go/actions/runs/4175670981

@farshidtz , @MonicaisHer , the core/support service are failing because core-common-config-bootstrapper never ran. It should be in the list, but isn't:
Feb 14 17:35:49 fv-az358-706 edgexfoundry.configure[3098]: Processing autostart for: [              consul redis core-data core-metadata core-command support-notifications support-scheduler postgres kong-daemon vault security-secretstore-setup security-consul-bootstrapper security-proxy-setup security-bootstrapper-redis]
Feb 14 17:35:49 fv-az358-706 edgexfoundry.configure[3098]: consul will start and enable.
Feb 14 17:35:49 fv-az358-706 edgexfoundry.configure[3098]: redis will start and enable.
Feb 14 17:35:49 fv-az358-706 edgexfoundry.configure[3098]: core-data will start and enable.
Feb 14 17:35:49 fv-az358-706 edgexfoundry.configure[3098]: core-metadata will start and enable.
Feb 14 17:35:49 fv-az358-706 edgexfoundry.configure[3098]: core-command will start and enable.
Feb 14 17:35:49 fv-az358-706 edgexfoundry.configure[3098]: support-notifications will start and enable.
Feb 14 17:35:49 fv-az358-706 edgexfoundry.configure[3098]: support-scheduler will start and enable.
Feb 14 17:35:49 fv-az358-706 edgexfoundry.configure[3098]: postgres will start and enable.
Feb 14 17:35:49 fv-az358-706 edgexfoundry.configure[3098]: kong-daemon will start and enable.
Feb 14 17:35:49 fv-az358-706 edgexfoundry.configure[3098]: vault will start and enable.
Feb 14 17:35:49 fv-az358-706 edgexfoundry.configure[3098]: security-secretstore-setup will start and enable.
Feb 14 17:35:49 fv-az358-706 edgexfoundry.configure[3098]: security-consul-bootstrapper will start and enable.
Feb 14 17:35:49 fv-az358-706 edgexfoundry.configure[3098]: security-proxy-setup will start and enable.
Feb 14 17:35:49 fv-az358-706 edgexfoundry.configure[3098]: security-bootstrapper-redis will start and enable.


Ionic start issue

Hello guys, i am using proxy for my npm, because default settings don't work.
proxy=http://<IP_ADDRESS>:80
https-proxy=http://<IP_ADDRESS>:80
strict-ssl=false
I attempted to start my project by comand "ionic start app" and got this error:
http://i.stack.imgur.com/PHPHq.png
When i turned on my Apache server, i got this error.
http://i.stack.imgur.com/W50ua.png
So, solutions like "set proxy" don't work for me. Btw i wrote proxy in npm config but ionic still uses <IP_ADDRESS>:80.
I tried to turn off my antivirus and so on. 0 results.
How can i start my ionic project?

This issue was moved to driftyco/ionic-cli#1230

Hello, since this was an issue related to the ionic cli and not the framework i have moved this issue to that repo. Thanks for using Ionic!

This issue was moved to driftyco/ionic-site#738

This issue was moved to driftyco/ionic-cli#1231


Implement obsolete Marshal.Read APIs

If there's a bad obsolete API we added to .NET Core, chances are WinForms uses it.

Have you looked into what it would take to fix uses of these APIs in WinForms?

WinForms relies on object marshalling (AsAny) in public APIs. I looked at whether we can make WinForms and System.Drawing.Common not to be a stress test for marshalling APIs but I believe it's easier to implement the APIs and marshallers than to fix WinForm not to be unnecessarily confusing and inefficient.

I am not able to place where WinForms calls this API. Where is it?
I do see the AsAny marshaling.

Not at my PC anymore, but I think this was System.Drawing.Common - it was around where they use LOGFONT. It's how WinForms and Drawing  talk to each other.

It's this genially designed API: https://docs.microsoft.com/en-us/dotnet/api/system.drawing.font.fromlogfont?view=netframework-4.8


Distribute the macOS client as a standalone app in addition to app store

Investigate what changes would be required to move the Firezone macOS app to a system extension-based app, distributed as a standalone client. See https://developer.apple.com/documentation/technotes/tn3134-network-extension-provider-deployment
Why?

Ability to function as a multi-user system service
Not have to deal with the app review process
Ability to install older versions

- [ ] Package network extension as system extension
- [ ] Test release build with Test Flight using existing distribution process
- [ ] Update CI, Makefile to build and version a distributable `dmg` artifact to attach to releases
- [ ] Test drafted release artifact
- [ ] Update docs and test downgrades

Ability to function as a multi-user system service

The sandboxing to the logged-in user is also somewhat of a security feature, right?

Ability to function as a multi-user system service

The sandboxing to the logged-in user is also somewhat of a security feature, right?

So I think if we ever do a "multi-user service" it would require a service account token anyhow. I don't think we'd want it to be possible for a headless client to run with a regular user token for this reason.
The good news here is that this in effect still "runs" only when logged in as the user who installed the app. We would need to do more work (launchctl config at the least) to get this running for all users.

Just had this thought: I think we should not publish to the app store anymore to allow users to "downgrade" from the next release (system extension) to the current one (network extension).
That will make the next release a lot less risky.

I think we should not publish to the app store anymore to allow users to "downgrade" from the next release (system extension) to the current one (network extension).

The problem is I don't know how many admins are relying on app store as a distribution channel. Also anyone who wishes to upgrade won't receive a notification now their client is out of date upon launch, so we'd need at least one more app extension release to point them in the right direction.
That would be a lot of work now, though. I actually don't think the system extension is all that problematic given I understand a bit more of its inner workings now. There may be one or two more kinks to iron out but now that we can ship a standalone app we can ship updates more quickly.
I think it would be nice to have an in-place upgrade dialog available for the standalone app, as is common with many macOS apps. A pop up with a "Download & update" button.

I actually don't think the system extension is all that problematic given I understand a bit more of its inner workings now.

I was more thinking in regards to the connlib-side of things. Any bugs there might necessitate a downgrade.


SeisNoiseAWS module

@ttsukianto
I have been working to get this ready for you. In the dev branch you'll find an example in the example/test_SeisNoiseAWS.jl folder. This is a replica of your AWS folder structure and I am using as a test case for the module functions.
I have been struggling to find time to work on this, but I have it in a place where you can take over I think. We can chat about this in person some day soon.

I would like to get the log file working. e.g. in test_SeisNoiseAWS.jl you'll see the following lines.

using Logging io = open("xcor_log.txt", "w") logger = SimpleLogger(io)
This is not writing to the file yet, and I didn't spend much time digging into it, but I think it is important that we can track what happens during AWS processing with a log file like this. Can you please figure this one out?

All functions have been moved into SeisNoiseAWS.jl, a module that is then imported into test_SeisNoiseAWS.jl. I have everything working except the xcor() function. It has to do with the @everywhere declaration. I also haven't had time to dig into this, but I don't think I should declare @everywhere in the module like I do now. This is giving an error. Can you figure out what we should be doing so that we can use the xcor() function (written in the module) as an @everywhere function with pmap in test_SeisNoiseAWS.jl? I am sure this is close, I just can't find time to fix this myself.

@ttsukianto I left this issue earlier as a reminder where we were. Let me know if you have questions about where things ended. I can dig back into the code myself on Wednesday afternoon.

@ttsukianto I left this issue earlier as a reminder where we were. Let me know if you have questions about where things ended. I can dig back into the code myself on Wednesday afternoon.


add a code example to how-to-create-and-use-shared-ptr-instances.md

Fixes #4461

@TylerMSFT
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


[SwitchBase] Add a transition between checked and unchecked icons

Uses the Fade component in SwitchBase to transition between checked and unchecked icons. This animation is disabled by default for Switch, as it really only applies to Radio and Checkbox.

Resolves #12639.

From looking at the spec it looks like

the checkmark in Checkbox is animated from left to right
the circle in Radio grows outwards
the handle in Switch translates horizontal (which it already does)

I would prefer the right (or at least close to) transition instead of just any.

@calebissharp Thanks for working on this, it was a great first pass, but as it's been a while without progress, lets close for now. You're more than welcome to open a new PR an time!


[RUMBLE] Playlist downloads fail on /c/ but not on /user/ channels

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm reporting that yt-dlp is broken on a supported site
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
Norway
Provide a description that is worded well enough to be understood
This issue may be related to https://github.com/yt-dlp/yt-dlp/issues/8782
The issue replicates on two /c/ channels whereas the script behaves normally on a /user/ channel (yt-dlp -S res:480 --dateafter 20221001 --datebefore 20221031 https://rumble.com/user/JeffreyPrather) which plods through all videos and downloads the relevant ones.
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
[debug] Command-line config: ['-vU', '-S', 'res:720', '--dateafter', '20221001', '--datebefore', '20221010', 'https://rumble.com/c/RTNews']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp [24f827875] (pip)
[debug] Python 3.10.12 (CPython x86_64 64bit) - Linux-5.15.0-91-generic-x86_64-with-glibc2.35 (OpenSSL 3.0.2 15 Mar 2022, glibc 2.35)
[debug] exe versions: ffmpeg 4.4.2 (setts), ffprobe 4.4.2
[debug] Optional libraries: Cryptodome-3.15.0, brotli-1.0.9, certifi-2020.06.20, mutagen-1.45.1, requests-2.31.0, secretstorage-3.3.1, sqlite3-3.37.2, urllib3-2.1.0, websockets-10.3
[debug] Proxy map: {}
[debug] Request Handlers: urllib, requests
[debug] Loaded 1901 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version<EMAIL_ADDRESS>from yt-dlp/yt-dlp
yt-dlp is up to date<EMAIL_ADDRESS>from yt-dlp/yt-dlp)
[RumbleChannel] Extracting URL: https://rumble.com/c/RTNews
[download] Downloading playlist: RTNews
[RumbleChannel] RTNews: Downloading page 1
[RumbleChannel] RTNews: Downloading page 2
[RumbleChannel] RTNews: Downloading page 3
[RumbleChannel] RTNews: Downloading page 4
[RumbleChannel] RTNews: Downloading page 5
[RumbleChannel] RTNews: Downloading page 6
[RumbleChannel] RTNews: Downloading page 7
[RumbleChannel] RTNews: Downloading page 8
[RumbleChannel] RTNews: Downloading page 9
[RumbleChannel] RTNews: Downloading page 10
[RumbleChannel] RTNews: Downloading page 11
[RumbleChannel] RTNews: Downloading page 12
[RumbleChannel] RTNews: Downloading page 13
[RumbleChannel] RTNews: Downloading page 14
[RumbleChannel] RTNews: Downloading page 15
[RumbleChannel] RTNews: Downloading page 16
[RumbleChannel] RTNews: Downloading page 17
[RumbleChannel] RTNews: Downloading page 18
[RumbleChannel] RTNews: Downloading page 19
[RumbleChannel] RTNews: Downloading page 20
-
-
-
(lines deleted)
.
.
.
[RumbleChannel] RTNews: Downloading page 630
[RumbleChannel] RTNews: Downloading page 631
[RumbleChannel] RTNews: Downloading page 632
[RumbleChannel] RTNews: Downloading page 633
[RumbleChannel] RTNews: Downloading page 634
[RumbleChannel] RTNews: Downloading page 635
[RumbleChannel] RTNews: Downloading page 636
[RumbleChannel] RTNews: Downloading page 637
[RumbleChannel] RTNews: Downloading page 638
[RumbleChannel] RTNews: Downloading page 639
[RumbleChannel] RTNews: Downloading page 640
[RumbleChannel] RTNews: Downloading page 641
[RumbleChannel] RTNews: Downloading page 642
[RumbleChannel] Playlist RTNews: Downloading 0 items
[download] Finished downloading playlist: RTNews

Duplicate of #8782

@bashonly Why are you closing this issue? It's a major broken site issue!

because it's a duplicate!


Is valid_set needed？

Hi, I saw there is the train_set and dev_set in the "train.sh", dev_set is the test_set. So is valid_set necessary?

Yes, but you can remove it in train.py if you don't need.

Thanks.
For using valide_set, I think some lines like below can be added in the function of train_reinfoselect( ) of train.py, right?
best_mes = 0.0  with torch.no_grad():      valid_mes = dev(args, model, metric, valid_loader, device)     if valid_mes[args.metric] > best_mes:        dev_mes = dev(args, model, metric, dev_loader, device)        best_mes = dev_mes[args.metric]

Yes, for reinfoselect training, the valid_set is used as target dataset, the gains of the valid_mes is the reward for selection policy optimizing.

Thank you!

dev is valid_set, test_set is used in inference :)

Yep！Thank you for pointing out and correcting me!


Clipping the bar outside of the grid

Version
4.6.0
Reproduction link
https://gallery.echartsjs.com/editor.html?c=xs9BT4SnHR&v=1
Steps to reproduce
use this conf:
option = {
    legend: {},
    tooltip: {},
    dataset: {
        source: [
            ['val', 'id'],
            [0, 1234],
            [4, 12345]
        ]
    },
    xAxis: {type: 'value'},
    yAxis: {},
    series: [
        {type: 'bar'}
    ]
};

What is expected?
the graph is expected to be consistent
What is actually happening?
the layout is broken.
the columns come out of the graph and their size is not good

You can specify the width of bar

thanks for your feedback but even by reducing the width of the bars when zooming the layout is broken
use this cofig: option = {
dataZoom: [{ type: 'inside' }],
grid: { bottom: 40, left: 70, right: 50, top: 30 },
series: [
{
barWidth: '10%',
data: [
['val', 'id'],
[0, 377146],
[1, 620432],
],
name: 'id',
type: 'bar',
},
],
toolbox: {
feature: {
dataZoom: { title: { back: 'Zoom Reset', zoom: 'Zoom' } },
},
magicType: {
show: false,
title: {
bar: 'Switch to Bar Chart',
line: 'Switch to Line Chart',
stack: 'Stack',
tiled: 'Tile',
},
type: [],
},
saveAsImage: { title: 'Save image' },

show: false,

},
tooltip: { axisPointer: { type: 'shadow' }, trigger: 'axis' },
xAxis: {
axisLabel: {
rotate: 0,
},
axisLine: {
lineStyle: {
color: '#656565',
},
},
boundaryGap: ['20%', '20%'],
max: 2,
min: -1,
scale: true,

splitLine: {
  lineStyle: {
    color: '#DDD',
    type: 'dashed',
  },
},

type: 'value',

},
yAxis: {
axisLine: {
onZero: false,
},
scale: false,
splitLine: {
  lineStyle: { color: '#DDD', type: 'dashed' },
},

},
};
here is a link where you can test:
https://gallery.echartsjs.com/editor.html?c=xs9BT4SnHR&v=3
place your mouse to the right of the first bar and zoom step by step and you will see the outdated bar of the y-axis
here is a link where you can see the result:
https://zupimages.net/viewer.php?id=20/40/vs3s.png

Hi, this is the expected behavior for now. Bars will be only clipped (display nothing) when all of the bar are outside of the grid area. There's no way to clip the part outside of the grid for now but it may be supported afterwards.

Hi, thank you for your reply

hello I would like to know if you have been able to make any progress on this issue.


feat!: Add more locking possibilities

based on #163
see https://github.com/rustic-rs/rustic/discussions/1050
BREAKING CHANGES:

DeleteOptions in SnapshotFile now has more enum options.
Repository::save_snapshots now returns the list of Ids of the saved snapshots.
RepoId now must also impl Debug

Missing points:

[ ] Handle duplicate snapshots correctly - especially create at max. 1 new snapshot per "original" snapshot during lock
[ ] Handle locks in forget command (retention policy: only count one/the original snapshot for retention)
[ ] Handle locks correctly in the tag command (don't even try to remove locked snapshots)

Blocked on #163: PR requires some changes and maybe a discussion about Lock design as a type state for a backend? Not entirely sure, if that would be a good approach as the expectation on a backend with a lock is not completely clear to me.


Package.json main prop is incorrect

I think it should be: "main": "dist/umd/single-spa-layout.min.cjs"
Noticed the issue using parcel version 2 because it tries to use the main as an entry point to the module.

Oops, yes this is a bug that snuck into the 1.0.0 release. Mostly I've been relying on the exports field in the package.json instead of the main field, which is why this went undetected. I have put together a fix at #81 and will publish a new version with it shortly.

This is fixed in #81 and released in https://github.com/single-spa/single-spa-layout/releases/tag/v1.1.0


Updating Readme.md

I have added my name in the contributor's list. Also a part of Group 1 for reading papers on LiDAR + Camera.

Hey @MilindKharwade7, I could not see the paper or your name in Group 1.
As you added int he description, kindly check that.


Detail Callout Clipped

Sometimes it works sometimes it doesn't.  I'm not really sure why/when/how.
Here's my callout code
// annotation.detailCalloutView = object_context.detailCallout(annotation)
  detailCallout (rowData) {
    return (
      <TouchableOpacity style={{flex: 1}}>
        <Text>{`Case Number: ${rowData.caseNumber}`}</Text>
        <Text>{`Subject Name: ${rowData.subjectName}`}</Text>
        <Text>{`Date: ${rowData.dateTime}`}</Text>
        <Text>{`Address: ${rowData.address}`}</Text>
      </TouchableOpacity>
    )
  }

Here's my result in 0.18 - Sometimes this works, and sometimes it doesn't.  You can never tell which pin will work, which time.

Here's a gif of the issue being random.

For those having this issue/watching this ticket, I just verified it still exists in 0.19

+1

I had this problem too.
I ended up using react-native-maps which doesn't have this problem:
https://github.com/lelandrichardson/react-native-maps
Maybe you would like to see what they are doing there OR (even better) work together and merge it into react-native?

@af7 -  It seems react-native-maps works with Android in ways MapView can't!  @nicklockwood and @lelandrichardson - thoughts on possible merge conversation?

@GantMan I originally created that library in an effort to get it merged into RN. I would certainly love to see that happen, and we have talked about it. There are a couple of things to consider though:

react-native-maps requires google play services and an API key. This can make it extra difficult to package it with react native core, where we don't want to add that extra bulk to every RN app.
The react-native-maps library has significant API Surface Area. If it is merged into RN core, this can end up creating even more Issues/PRs for RN as a whole, where we are already struggling to keep up.
FB doesn't use react-native-maps internally

One alternative path is to remove the MapView component that's currently in core into a 3rd-party repo. This would then signal to everyone that a 3rd party solution is needed (just as it is for many things). We could add links to the main maps libraries in the docs if needed.
I'd be interested to see some more discussion around either path.

Hi there! This issue is being closed because it has been inactive for a while.
But don't worry, it will live on with ProductPains! Check out its new home: https://productpains.com/post/react-native/detail-callout-clipped
ProductPains helps the community prioritize the most important issues thanks to its voting feature.
It is easy to use - just login with GitHub. GitHub issues have voting too, nevertheless
Product Pains has been very useful in highlighting the top bugs and feature requests:
https://productpains.com/product/react-native?tab=top
Also, if this issue is a bug, please consider sending a pull request with a fix.
We're a small team and rely on the community for bug fixes of issues that don't affect fb apps.


Feature/way we do nuget

:white_check_mark: Build Microsoft.Owin.Security.ApiKey 5-sldynlks completed (commit https://github.com/jamesharling/Microsoft.Owin.Security.ApiKey/commit/a45aa26e11 by @hybridtechie)

:white_check_mark: Build Microsoft.Owin.Security.ApiKey 5-sldynlks completed (commit https://github.com/jamesharling/Microsoft.Owin.Security.ApiKey/commit/a45aa26e11 by @hybridtechie)


tweet the quote automagically

How to update it on every quote generation?

Put in some div, rebuild twitter;s iframe on quote generation ?


enables docsearch in development env

enables docsearch in development environment

previously a search hard redirect to https://getbootstrap.com/…

@m5o: can you keep both replace calls? We can remove the old one later if it's not needed.

@m5o: can you keep both replace calls? We can remove the old one later if it's not needed.

Sorry, I don't get it

I mean the replacement of https://v4-alpha.getbootstrap.com.

@maxiloc: looks good to you? Any suggestions, maybe?

👋 I am working on DocSearch, I will take over from @maxiloc .
From what I get you want to change the URL from the hit we have scrapped within you DocSearch index (bootstrap-v4).
Example forquery=changes:
Current result from your DocSearch index:
hit.url=https://getbootstrap.com/docs/4.0/migration/#stable-changes
From what I get:
data-baseurl={{ site.baseurl }}/docs/{{ site.docs_version }}="https://getbootstrap.com/docs/4.0/"

What you are doing:
hit.url.replace('https://v4-alpha.getbootstrap.com', '/docs/4.0')
``` This replacement is useless since 'https://v4-alpha.getbootstrap.com' doesn't match the URL `hit.url`.

Furthermore, you want to replace the pattern:
`'https://getbootstrap.com' + baseurl=https://getbootstrap.comhttps://getbootstrap.com/docs/4.0/`
This pattern will not be matched either.

After all, the url will be the same.

Let me know if I missed something.

@s-pace: first of all, thanks for the reply!
Now, what we want to do is replace the baseurl for local development where {{ site.baseurl }} is basically empty.
For production it's set to https://getbootstrap.com. The v4-alpha.getbootstrap.com part must be from the alpha time, I'm not sure we need it either, so we can probably indeed remove it.

Also, note that if you try this PR locally, it indeed works fine. I just feel something is not right in the implementation.

@XhmikosR
You are welcome.
This should work if the URL & the anchor (so the layout) are the same between your dev and production environment.

For production it's set to https://getbootstrap.com.

🙅♂ Not true, you can validate by compare: – Or in't this valid proof? https://github.com/twbs/bootstrap/blob/d01b4eb02560ba67b61b1b2c0496e9f8281c154d/index.html#L9
with the generated markup from http://getbootstrap.com/

History wise these changes are important https://github.com/twbs/bootstrap/pull/22731 and https://github.com/algolia/docsearch-configs/commit/ce46787bb133ed6f0833a6c601683ce386f2afed .
v4-alpha.getbootstrap.com is definitely a leftover and can be removed. I've tried it.
The main question is, maybe @s-pace can help. The returned hit.url is an absolute url, can this a relative url instead? A relative url would work in development and production environment.
+https://getbootstrap.com/docs/4.0/migration/#stable-changes
-/docs/4.0/migration/#stable-changes

algolia/docsearch-configs/configs/bootstrap-v4.json

True, baseurl is always empty. site.url changes which is why I mixed those two.

So, basically, all this PR does is this:

passes a data attribute with the baseurl which is translated to /docs/4.0 for both dev and production
It then replaces this from the search results: https://getbootstrap.com/docs/4.0 with /docs/4.0

So, it seems that we can simplify this and not rely to the empty baseurl.

Please check out the last patch I pushed. Basically I rely on site.url which is different for production and development. When in development, I just remove the url from the search results, making them relative.
Example searching for tooltip:
baseurl: http://<IP_ADDRESS>:9001
hit.url: /docs/4.0/components/tooltips/#overview

👍 thank you


BBE link fix

BBE link is broken, this pull fixes that

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

Codecov Report

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

@@           Coverage Diff           @@
##           master   #23855   +/-   ##
=======================================
  Coverage   14.59%   14.59%
=======================================
  Files          51       51
  Lines        1411     1411
  Branches      219      219
=======================================
  Hits          206      206
  Misses       1189     1189
  Partials       16       16

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 90d9f09...60fef00. Read the comment docs.


Add hexlet.io to the users list

Coverage remained the same at 69.679% when pulling 57f4ae53201da675af1bed986c86c45afcf99170 on nechitai:add-user into 618768c7ed727136db17d78ed07ce4a84fbbe247 on sourcelair:master.


Get rid of Tuple

This PR utilizes recently introduced EntityRecord.entity_id field for symmetry of conversion between an Entity and EntityRecord.
As the result we no longer need the Tuple class that hold an ID together with an EntityRecord.
This PR is based on #372.

@armiol, PTAL.


Strange Error in Cactbot ngld

Hey gus sicne the new change to nglds Cactbot Version i get a strange error in my overlay Log.
 Since it wil change tooo ngld soon i have no choice.
plz can tell me some1 for dummys what the problem is ?
i added a screenshot
https://www.file-upload.net/download-13772722/error.JPG.html
thx in regards

Don't put Cactbot in the addons folder. Put it pretty much anywhere else and add it as an ACT plugin. It should load correctly as long as it's not in the ACT folder and not in the addons folder.

yeah just did that  :D and it works perfectly now :D

but u said the same so it had to be that ;D thx anyways :)

man im so glad about this support here  great job ... keep it on

ngld support is amazing <3


Wrong file extension for router.esm and types.esm

Describe the bug
Module not found: Can't resolve './router.esm.mjs'
Module not found: Can't resolve './types.esm.mjs'

Package<EMAIL_ADDRESS>includes router.esm.js and types.esm.js files instead of router.esm.mjs and types.esm.mjs
https://www.npmjs.com/package/@ts-rest/serverless?activeTab=code
How to reproduce

Use next@15-rc
Install @ts-rest/serverless/next
Create a route /app/api/bff/[...ts-rest]
Use import { createNextHandler } from "@ts-rest/serverless/next";
Fetch /api/bff/something
An error occurs

Expected behavior
No response
Code reproduction
import { bffContract } from "@/contracts/bff-contract";
import { createNextHandler } from "@ts-rest/serverless/next";

const handler = createNextHandler(
  bffContract,
  {
    orders: async ({ params, query }) => {
      console.log("running orders");
      return {
        status: 200,
        body: {
          ...params,
          ...query,
          deleteMe: "response validation will delete me :(",
        },
      };
    },
  },
  {
    basePath: "/api/bff",
    jsonQuery: true,
    responseValidation: true,
    handlerType: "app-router",
  },
);

export { handler as GET, handler as POST, handler as PUT, handler as PATCH, handler as DELETE };

ts-rest version
"@ts-rest/core": "^3.49.1",
"@ts-rest/react-query": "^3.49.1",
"@ts-rest/serverless": "^3.49.1",

Oh wow! I was just about to commit a refactor for our build process, but without releasing a change. I had no idea @ts-rest/serverless was broken, but luckily the refactor fixes this problem.
Thanks for reporting!


add more free font sources

When google does not have font we can turn to more sources?
https://free-for.dev/#/?id=font

Well the project is tagged Google Fonts.
People maybe looking for free fonts and it's ok to add more sources as a fallback. It's just I need to think about the UX.


Add error message when shader file not found.

Currently we get a rather confusing error later on in the shader linking stage as we donet have an upfront error when the passed shader file cant be found.
So I added one.

Indeed, although the reasoning behind this is that in a typical pipeline, not all shader stages are required (for instance a graphics kernel that has only VS and no PS, or a mesh kernel that has no AS, etc.).
Wouldn’t this change now spam a lot of (invalid) errors when these shader files aren’t found?

Ah yes your right, it apeear I jumped the gun there. Ill close this but we should probably have some error message when no shaders are found somewhere as the current error is rather non-obvious


[FEATURE]Product listing- add button '+ Colors' on mobile

Describe how this feature will help us improve
Easy and quick way of choosing color of product
Describe the solution you'd considered
Add button '+ Colors' on mobile on product listing as below:

After pressing the '+ Colors' button a list of available colors of the product appears as below:

After pressing color, the choice is remembered. Then if user adds the product to the cart, the product is added with chosen color.
Acceptance criteria

[ ]   the '+ Colors' button is added to the product listing on mobile
[ ]  after pressing the '+ Color' a list of available colors appears
[ ] chosen color is remembered as long as the user is on that product listing
[ ] pressing any spot other than color square closes the list of available colors
[ ] after choosing the color, when user adds the product to the cart, it is added with tat chosen color

related link https://github.com/DivanteLtd/shopware-pwa/issues/406

related: #881


Doctor: fixFrontmatter command adds empty line under frontmatter block

Please select the area your bug applies to. (Multiple selections are Possible. You can leave blank if you're not sure.)

[ ] Workspace.  VSCode workspace, vaults, Intellisense/autocomplete, Dendron settings
[ ] Lookup.  Dendron's Lookup Command
[ ] Views.  Dendron Preview, Tree View, Side Panels in the UI
[ ] Schema.  Dendron Schemas
[ ] Pod.  Data import from / export to Dendron
[ ] Publish.  External Site Publish
[x] Markdown.  Markdown features, such as syntax support and features

Describe the bug
Running Doctor: fixFrontmatter command adds an extra empty line under the frontmatter block.

I use Prettier to format my notes and by default Prettier collapses multiple blank lines into a single blank line, and as a result, calling Doctor generates whitespace changes in all my notes when I just wanted to fix frontmatter in a single note.
To Reproduce
Steps to reproduce the behavior:

Make a note with a single empty line between the frontmatter block and the next non-empty line
Run Doctor: fixFrontmatter
Observe that another empty line was added after the frontmatter block

Expected behavior
Doctor: fixFrontmatter should not add empty lines (or reformat) notes themselves, or, at least, it should not do it in the notes with valid frontmatter.
Desktop (please complete the following information)

Version: 0.73

Thanks for the issue. We'll get back to you when we fix this.

Actually, just realized that doctor not only adds a single line at the top of your file so that you have two, it does it every time you call "fixFrontmatter"--if you use this command regularly, content of all your notes is being pushed down constantly :/.

This issue was selected by the community in the latest round of CROP.

Found a workaround--running npx prettier --write "**/*.md" removes empty lines added by the Doctor and I don't have 1k+ files to commit every time I'm trying to fix frontmatter on new notes. 🎉


Make values consistent in Introduction part 2 tutorial

Addresses #516.  Makes values of guess and measurement consistent throughout the tutorial.

Cancelling tests since this only changes a tutorial.
MayI merge?

Yep, go ahead


Ch#6. Arcade machine event never set True

In Chapter#6 there is no code to set Event flag (set() method).
THere is only create processor_free with default flag False:
https://github.com/luminousmen/grokking_concurrency/blob/667d37e16e3ab2b4bada0f75af38b26df607290f/Chapter 6/arcade_machine.py#L11
Coeerct me if I'm wrong, but at 12th line we have to call processor_free.set() and immediatelly after self.func() set Event once again.

Hey Evgeniy, I think you're right! Thanks for pointing that out - I will update the code


Not all topics synched

Hello,
I am currently working on a Kubernetes cluster using ROS melodic (for specific project reasons melodic needs to be used). As the ros master is a single point of failure, I want to make it redundant and came across your software. Thanks for sharing it!
In the cluster every node can directly address every node via its IP.
Each ros master is running in a Kubernetes pod, this means each container inside the pod shares storage and network. In total there are three containers for each pod running the following commands:

roscore: . /opt/ros/melodic/setup.bash; export ROS_IP="$(hostname -i)"; roscore
master discovery: . /opt/ros/melodic/setup.bash; export ROS_IP="$(hostname -i)"; rosrun master_discovery_fkie master_discovery
master sync: . /opt/ros/melodic/setup.bash; export ROS_IP="$(hostname -i)"; rosrun master_sync_fkie master_sync

The container image is created as following:
FROM ros:melodic-ros-base
RUN apt-get update && apt-get install -y ros-melodic-multimaster-fkie

Note: Broadcast and multicast get enabled before start.
The master discovery seems to work correctly:
Pod 1  (master discovery output):
[INFO] [1580121414.848866]: Check the ROS Master[Hz]: 1
[INFO] [1580121414.849375]: Heart beat [Hz]: 0.02
[INFO] [1580121414.849813]: Active request after [sec]: 60
[INFO] [1580121414.850235]: Remove after [sec]: 300
[INFO] [1580121414.850663]: Robot hosts: []
[INFO] [1580121414.851079]: Approx. mininum avg. network load: 1.36 bytes/s
[INFO] [1580121414.855287]: Start RPC-XML Server at ('<IP_ADDRESS>', 11611)
[INFO] [1580121414.855983]: Subscribe to parameter `/roslaunch/uris`
[INFO] [1580121414.858357]: + Bind to specified unicast socket @(<IP_ADDRESS>:11511)
[INFO] [1580121414.858861]: Create multicast socket at ('<IP_ADDRESS>', 11511)
[INFO] [1580121415.027750]: Detected master discovery: http://localhost:11611
[INFO] [1580121415.131294]: Added master with ROS_MASTER_URI=http://<IP_ADDRESS>:11311/
[INFO] [1580121415.443574]: Detected master discovery: http://<IP_ADDRESS>:11611
[INFO] [1580121415.547302]: Added master with ROS_MASTER_URI=http://<IP_ADDRESS>:11311/

Pod 2  (master discovery output):
[INFO] [1580121414.370170]: ROS Master URI: http://localhost:11311
[INFO] [1580121414.375819]: Check the ROS Master[Hz]: 1
[INFO] [1580121414.376521]: Heart beat [Hz]: 0.02
[INFO] [1580121414.377118]: Active request after [sec]: 60
[INFO] [1580121414.377603]: Remove after [sec]: 300
[INFO] [1580121414.378058]: Robot hosts: []
[INFO] [1580121414.378539]: Approx. mininum avg. network load: 1.36 bytes/s
[INFO] [1580121414.382736]: Start RPC-XML Server at ('<IP_ADDRESS>', 11611)
[INFO] [1580121414.383488]: Subscribe to parameter `/roslaunch/uris`
[INFO] [1580121414.386952]: + Bind to specified unicast socket @(<IP_ADDRESS>:11511)
[INFO] [1580121414.387578]: Create multicast socket at ('<IP_ADDRESS>', 11511)
[INFO] [1580121414.560276]: Detected master discovery: http://localhost:11611
[INFO] [1580121414.662968]: Added master with ROS_MASTER_URI=http://<IP_ADDRESS>:11311/
[INFO] [1580121415.025819]: Detected master discovery: http://<IP_ADDRESS>:11611
[INFO] [1580121415.128332]: Added master with ROS_MASTER_URI=http://<IP_ADDRESS>:11311/

Also I do not see any errors in the sync output:
Pod 1 (master sync output):
...
[INFO] [1580121446.300150]: ROS masters obtained from '/master_discovery/list_masters': ['<IP_ADDRESS>', '<IP_ADDRESS>']
[INFO] [1580121470.236891]: SyncThread[<IP_ADDRESS>] Requesting remote state from 'http://<IP_ADDRESS>:11611'
[INFO] [1580121470.240626]: SyncThread[<IP_ADDRESS>] Applying remote state...
[INFO] [1580121470.241990]: SyncThread[<IP_ADDRESS>] remote state applied.
[INFO] [1580121472.766345]: SyncThread[<IP_ADDRESS>] Requesting remote state from 'http://<IP_ADDRESS>:11611'
[INFO] [1580121472.770274]: SyncThread[<IP_ADDRESS>] Applying remote state...
[INFO] [1580121472.773116]: SyncThread[<IP_ADDRESS>] remote state applied.
[INFO] [1580121476.307809]: ROS masters obtained from '/master_discovery/list_masters': ['<IP_ADDRESS>', '<IP_ADDRESS>']
[INFO] [1580121506.315516]: ROS masters obtained from '/master_discovery/list_masters': ['<IP_ADDRESS>', '<IP_ADDRESS>']
[INFO] [1580121536.322906]: ROS masters obtained from '/master_discovery/list_masters': ['<IP_ADDRESS>', '<IP_ADDRESS>']
[INFO] [1580121566.329678]: ROS masters obtained from '/master_discovery/list_masters': ['<IP_ADDRESS>', '<IP_ADDRESS>']
...

Pod 2 (master sync output):
...
[INFO] [1580123187.126108]: ROS masters obtained from '/master_discovery/list_masters': ['<IP_ADDRESS>', '<IP_ADDRESS>']
[INFO] [1580123208.181042]: SyncThread[<IP_ADDRESS>] Requesting remote state from 'http://<IP_ADDRESS>:11611'
[INFO] [1580123208.185128]: SyncThread[<IP_ADDRESS>] Applying remote state...
[INFO] [1580123208.187537]: SyncThread[<IP_ADDRESS>] remote state applied.
[INFO] [1580123210.238873]: SyncThread[<IP_ADDRESS>] Requesting remote state from 'http://<IP_ADDRESS>:11611'
[INFO] [1580123210.242788]: SyncThread[<IP_ADDRESS>] Applying remote state...
[INFO] [1580123210.245158]: SyncThread[<IP_ADDRESS>] remote state applied.
[INFO] [1580123217.137040]: ROS masters obtained from '/master_discovery/list_masters': ['<IP_ADDRESS>', '<IP_ADDRESS>']
[INFO] [1580123217.641776]: SyncThread[<IP_ADDRESS>] Requesting remote state from 'http://<IP_ADDRESS>:11611'
[INFO] [1580123217.645935]: SyncThread[<IP_ADDRESS>] Applying remote state...
[INFO] [1580123217.648215]: SyncThread[<IP_ADDRESS>] remote state applied.
[INFO] [1580123247.147505]: ROS masters obtained from '/master_discovery/list_masters': ['<IP_ADDRESS>', '<IP_ADDRESS>']
[INFO] [1580123277.158128]: ROS masters obtained from '/master_discovery/list_masters': ['<IP_ADDRESS>', '<IP_ADDRESS>']
...

For test purpose there is realsense (camera) node which is publishing data from the camera. This will create a certain amount of topics and the realsense randomly registers add one of the masters.
At master one, I see the following topics:
/camera/accel/imu_info
/camera/accel/sample
/camera/aligned_depth_to_color/camera_info
/camera/aligned_depth_to_color/image_raw
/camera/aligned_depth_to_infra1/camera_info
/camera/aligned_depth_to_infra1/image_raw
/camera/color/camera_info
/camera/color/image_raw
/camera/color/image_rect_color
/camera/color_rectify_color/parameter_descriptions
/camera/color_rectify_color/parameter_updates
/camera/depth/camera_info
/camera/depth/image_rect_raw
/camera/depth_registered/points
/camera/extrinsics/depth_to_color
/camera/extrinsics/depth_to_infra1
/camera/extrinsics/depth_to_infra2
/camera/gyro/imu_info
/camera/gyro/sample
/camera/infra1/camera_info
/camera/infra1/image_rect_raw
/camera/infra2/camera_info
/camera/infra2/image_rect_raw
/camera/motion_module/parameter_descriptions
/camera/motion_module/parameter_updates
/camera/realsense2_camera_manager/bond
/camera/rgb_camera/auto_exposure_roi/parameter_descriptions
/camera/rgb_camera/auto_exposure_roi/parameter_updates
/camera/rgb_camera/parameter_descriptions
/camera/rgb_camera/parameter_updates
/camera/stereo_module/auto_exposure_roi/parameter_descriptions
/camera/stereo_module/auto_exposure_roi/parameter_updates
/camera/stereo_module/parameter_descriptions
/camera/stereo_module/parameter_updates
/diagnostics
/master_discovery/changes
/master_discovery/linkstats
/rosout
/rosout_agg
/tf
/tf_static

At master 2 there is only a small subset of the topics available:
/camera/color/image_rect_color
/camera/color_rectify_color/parameter_descriptions
/camera/color_rectify_color/parameter_updates
/camera/realsense2_camera_manager/bond
/master_discovery/changes
/master_discovery/linkstats
/rosout
/rosout_agg

It looks like there seems to be some synchronization but it seems to not work completely.
Do you see any problems in the set up? Is there some output/log files I should consult?
Thank you in advance!
Nils

Hey thanks for providing the update.
Unfortunately, I still do not see any change.
I added the relevant output lines below.
As you mentioned before there should be some more output if a topic gets registered, e.g.: https://github.com/fkie/multimaster_fkie/blob/fix_sync_ros_ip/fkie_master_sync/src/fkie_master_sync/sync_thread.py#L324
I will try to add some more debugging output to understand better why it is not working. Maybe some check is failing due to a specific setup I have.
Thanks a lot for your support!
[INFO] [1580744045.079913]: ROS masters obtained from '/master_discovery/list_masters': ['<IP_ADDRESS>', '<IP_ADDRESS>']
[DEBUG] [1580744068.563584]: SyncThread[<IP_ADDRESS>]: update notify new timestamp(1580744068.482115984), old(1580744024.720185041)
[INFO] [1580744069.369647]: SyncThread[<IP_ADDRESS>] Requesting remote state from 'http://<IP_ADDRESS>:11611'
[INFO] [1580744069.379375]: SyncThread[<IP_ADDRESS>] Applying remote state...
[DEBUG] [1580744069.382525]: SyncThread[<IP_ADDRESS>]: remote state: ['1580744068.482115984', '1580744068.482115984', 'http://<IP_ADDRESS>:11311/', '<IP_ADDRESS>', [['/test', ['/rostopic_256_1580744068021']]], [], [], [['/test', 'std_msgs/String']], [['/rostopic_256_1580744068021', 'http://basic-pipeline-nils-ros-vnc:33963/', 'http://<IP_ADDRESS>:11311/', None, 'remote']], []]
[DEBUG] [1580744069.387647]: SyncThread[<IP_ADDRESS>]: current timestamp<PHONE_NUMBER>.482115984, local<PHONE_NUMBER>.482115984
[INFO] [1580744069.390684]: SyncThread[<IP_ADDRESS>] Requesting remote md5sums 'http://<IP_ADDRESS>:11611'
[INFO] [1580744069.396152]: SyncThread[<IP_ADDRESS>] remote state applied.

[INFO] [1580744069.055136]: SyncThread[<IP_ADDRESS>] Requesting remote state from 'http://<IP_ADDRESS>:11611'
[INFO] [1580744069.064085]: SyncThread[<IP_ADDRESS>] Applying remote state...
[DEBUG] [1580744069.067311]: SyncThread[<IP_ADDRESS>]: remote state: ['1580744015.036187887', '1580744015.036187887', 'http://<IP_ADDRESS>:11311/', '<IP_ADDRESS>', [], [], [], [], [], []]
[DEBUG] [1580744069.072681]: SyncThread[<IP_ADDRESS>]: current timestamp<PHONE_NUMBER>.036187887, local<PHONE_NUMBER>.036187887
[INFO] [1580744069.075639]: SyncThread[<IP_ADDRESS>] Requesting remote md5sums 'http://<IP_ADDRESS>:11611'
[INFO] [1580744069.081018]: SyncThread[<IP_ADDRESS>] remote state applied.
[DEBUG] [1580744084.199874]: connecting to <IP_ADDRESS> 43161

thank you for testing!
The problem is in this entry:
['/rostopic_256_1580744068021', 'http://basic-pipeline-nils-ros-vnc:33963/', 'http://<IP_ADDRESS>:11311/', None, 'remote']
This is created by master_discovery. My fix [master_discovery/master_monitor.py#L817] (https://github.com/fkie/multimaster_fkie/blob/fix_sync_ros_ip/fkie_master_discovery/src/fkie_master_discovery/master_monitor.py#L817) and master_discovery/common.py#L85 should replace basic-pipeline-nils-ros-vnc by <IP_ADDRESS> if it is the same host! Remote nodes are not synchronized by master_sync. It could lead to problems...
If it is not the same host and you want synchronize remote nodes anyway, you have to set sync_remote_nodes of  master_sync to True.

I think the remote nodes is the issue in my case. Because both roscores are running on different pods (containers) with a separate network interface.
Unfortunately rosrun fkie_master_sync master_sync _log_level:=DEBUG sync_remote_nodes:=True  or rosrun fkie_master_sync master_sync _log_level:=DEBUG sync_remote_nodes:=true  does not work. I added a debug output but the expression self._filter.sync_remote_nodes() always evaluates to false.
Unfortunately, I am far from being a ROS expert, I could not find the reason why the command option is not taken into account.
I disabled the check manually and now the topics get synched. The only thing I get is a warning in the discovery output "[WARN] [1580825992.122942]: 'NoneType' object has no attribute '_md5sum'".
You mentioned that there might be problems if remote hosts are synched:
What could be the problems if remote synch is used (what is unfortunately what I am looking for)?
Thanks a lot for your support!

sync_remote_nodes is local variable of master_sync, so you have prepend _ and launch master_sync with:
rosrun fkie_master_sync master_sync _log_level:=DEBUG _sync_remote_nodes:=True
But you have changed the code and remote nodes are synchronized anyway ;-)

The warning is caused while master_sync checks the md5sum of remote topic with local one. I guess master_discovery can't resolve the topic class, since the node is not installed and not running in the same host/container as master_dsicovery.

(2.) is one of the problems with synchronization of remote nodes. Depending on ros-networks (multiple master_sync nodes) and timings the remote nodes could be stopped by roscore (multiple registration of nodes with same name.) It could also lead to visualization issues in node_manager do to dleays while tests for running state of the node.

You should start roscore, master_discovery and master_sync on each pod/container. This is the scenario what the multimaster was designed for.
I did not understand that the ROS nodes were started in another container than master_dsicovery. So the fix I did is for this problem not relevant.
Is there a reasen why you start the ROS-Nodes in different container as roscore and master_discovery?

Thanks. With setting it as local variable it works!
After all the setup is the following:
Inside the K8S cluster there is one pod for each ros program (node). Each of them is connecting to the ROS master(s). Because the ROS master is per default not backed up nor redundant, I was looking for running a redundant ROS master. Each instance of the ros master will run in their own pod. In that pod now the roscore, ros_master_sync and ros_master_discovery are running for each ros master.
But all of the other nodes never know to which master they connect, as they are connecting via a service which randomly resolves to one of the ros masters. If old of them are in synch and one crashes for what reason so ever, it should be transparent to the nodes, as they will just register or request a topic from one of the other masters, which should by in synch.
It looks that it is running now, but apparently not 100% reliable. A colleague told me that he encountered some problems, apparently some topics have not been available. I am right now trying to reproduce the issue.

is the issue still present?

I will close it.
There seem to be some reliability issues with the synch (sometimes topics don't show up or take a while to show up).
But overall it is working!


Truncate ECChain capacity when taking a prefix

We don't need to be thread-safe, but the chain.Prefix(n).Extend(tip) would previously have overwritten chain. I don't think this is an issue (we only use Extend in tests?) but we might as well fix it.

Hm. Actually, this is insufficient. multiple calls to Extend on the same chain can also cause issues.


Ability to register a error handlers for services

Description:
I propose there has to be a way to register (global and/or per service) error-handlers for ballerina 'service' construct.
Reason: I need to be able to control the behavior (and content) of how errors are transmitted back to client. For example: if I'm running my ballerina service in a local/dev environment I should be able send a descriptive message about the error that occurred while if I'm running in production I probably don't want to reveal too much information to the client.
Now I can do the same thing by checking for each error condition with-in the 'resource'. But reality of that is, amount of 'boiler-plate' code written becomes higher than code written for 'business-logic'.
As per my understanding, programmer can't customize the default behavior.
Affected Versions:
v 1.0.0
OS, DB, other environment details and versions:
Mac, Jdk 1.8
@sameerajayasoma

@ramith I created the following issue a few days ago. It is related to your requirement
https://github.com/ballerina-platform/ballerina-spec/issues/337

Ideally this would be made available to dependencies so services like sentry.io can hook and capture per/request and/or global errors.

We can achieve the requirement now using the on-fail clause support in Ballerina. Hence, closing the issue. Please reopen if you need any further clarification on this.

We can achieve the requirement now using the on-fail clause support in Ballerina. Hence, closing the issue. Please reopen if you need any further clarification on this.